Skip to content
This repository was archived by the owner on Feb 22, 2022. It is now read-only.

Deprecate gcloud-sqlproxy chart - #9219

Merged
k8s-ci-robot merged 1 commit into
helm:masterfrom
rimusz:deprecate-gloud-sqlproxy
Nov 22, 2018
Merged

Deprecate gcloud-sqlproxy chart#9219
k8s-ci-robot merged 1 commit into
helm:masterfrom
rimusz:deprecate-gloud-sqlproxy

Conversation

@rimusz

@rimusz rimusz commented Nov 13, 2018

Copy link
Copy Markdown
Contributor

Signed-off-by: rimas rmocius@gmail.com

Deprecating gcloud-sqlproxy chart in preparation for the new Helm Distributed repo (hub).
gcloud-sqlproxy chart is already hosted at https://github.com/rimusz/charts/tree/master/stable/gcloud-sqlproxy

Signed-off-by: rimas <rmocius@gmail.com>
@helm-bot helm-bot added Contribution Allowed If the contributor has signed the DCO or the CNCF CLA (prior to the move to a DCO). size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Nov 13, 2018
@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 13, 2018
@rimusz

rimusz commented Nov 13, 2018

Copy link
Copy Markdown
Contributor Author

/assign @unguiculus

@rimusz

rimusz commented Nov 13, 2018

Copy link
Copy Markdown
Contributor Author

/assign @prydonius

@rimusz

rimusz commented Nov 16, 2018

Copy link
Copy Markdown
Contributor Author

can we get this one merged, thanks

@unguiculus

Copy link
Copy Markdown
Member

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Nov 22, 2018
@k8s-ci-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: rimusz, unguiculus

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot
k8s-ci-robot merged commit 06f8db9 into helm:master Nov 22, 2018
@monotek

monotek commented Nov 23, 2018

Copy link
Copy Markdown
Collaborator

@unguiculus
@rimusz

Hearing about "the new Helm Distributed repo (hub)" the first time.

Found this pr when looking for it: https://github.com/helm/community/pull/24/files

So Bitnami owned site https://hub.kubeapps.com will be the new place where we have to look for charts?

Is this decission already finished or is there an ongoing discussion about it anywhere where one can atend?

@monotek

monotek commented Nov 23, 2018

Copy link
Copy Markdown
Collaborator

Thanks for the heads-up :-)

@rimusz

rimusz commented Nov 23, 2018

Copy link
Copy Markdown
Contributor Author

@rimusz
rimusz deleted the deprecate-gloud-sqlproxy branch November 23, 2018 16:05
davidkarlsen pushed a commit to davidkarlsen/charts that referenced this pull request Dec 2, 2018
Signed-off-by: rimas <rmocius@gmail.com>
yuchaoran2011 pushed a commit to yuchaoran2011/charts that referenced this pull request Dec 11, 2018
Signed-off-by: rimas <rmocius@gmail.com>
Signed-off-by: Chaoran Yu <yuchaoran2011@gmail.com>
wgiddens pushed a commit to wgiddens/charts that referenced this pull request Jan 18, 2019
Signed-off-by: rimas <rmocius@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. Contribution Allowed If the contributor has signed the DCO or the CNCF CLA (prior to the move to a DCO). lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants