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

Add support for AWS Secrets Manager #6245

Merged
merged 3 commits into from
Jun 26, 2018
Merged

Add support for AWS Secrets Manager #6245

merged 3 commits into from
Jun 26, 2018

Conversation

kurtmc
Copy link
Contributor

@kurtmc kurtmc commented Jun 21, 2018

What this PR does / why we need it:
Adds support for AWS Secrets Manager: https://concourse-ci.org/creds.html#asm

@k8s-ci-robot k8s-ci-robot added needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Jun 21, 2018
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jun 21, 2018
@kurtmc
Copy link
Contributor Author

kurtmc commented Jun 22, 2018

/assign @william-tran

@william-tran
Copy link
Collaborator

@kurtmc can we call it credentialManager.awsSecretsManager? I should have asked for the same with SSM :S

@kurtmc
Copy link
Contributor Author

kurtmc commented Jun 25, 2018

@william-tran Done. Is it ok for me to rebase this pull request so that there is no conflict?

@william-tran
Copy link
Collaborator

YEs, please rebase, also bump the minor version of the chart instead as this is a new feature

@kurtmc
Copy link
Contributor Author

kurtmc commented Jun 26, 2018

@william-tran Done.

@william-tran
Copy link
Collaborator

/ok-to-test

@k8s-ci-robot k8s-ci-robot removed the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label Jun 26, 2018
@william-tran
Copy link
Collaborator

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jun 26, 2018
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: kurtmc, william-tran

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

The pull request process is described here

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 added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 26, 2018
@william-tran
Copy link
Collaborator

thanks @kurtmc!

@k8s-ci-robot k8s-ci-robot merged commit de0a9b1 into helm:master Jun 26, 2018
jainishshah17 added a commit to jainishshah17/charts that referenced this pull request Jun 26, 2018
* artifactory-rbac-support:
  updated readme
  bumped Artifactory version
  RBAC support
  upgrade alertmanager, prometheus, configmap-reload and node-exporter (helm#6276)
  Add support for AWS Secrets Manager (helm#6245)
  Dokuwiki - Several improvements (helm#6326)
  Mediawiki fix probes & improve notes (helm#6320)
  Odoo - Several improvements (helm#6316)
  Joomla! - Several improvements (helm#6322)
  update signalfx-agent helm chart to v0.1.2 to deploy agent v3.2.2 (helm#6202)
  cert-manager: fast-forward to upstream f804cb56 (helm#6308)
  [stable/concourse] correct the parameter name  (helm#6283)

# Conflicts:
#	stable/artifactory/Chart.yaml
#	stable/artifactory/README.md
#	stable/artifactory/values.yaml
jainishshah17 added a commit to jainishshah17/charts that referenced this pull request Jun 26, 2018
…configmap

* artifactory-ha-rbac-support: (60 commits)
  updated readme
  RBAC support
  upgrade alertmanager, prometheus, configmap-reload and node-exporter (helm#6276)
  Add support for AWS Secrets Manager (helm#6245)
  Dokuwiki - Several improvements (helm#6326)
  Mediawiki fix probes & improve notes (helm#6320)
  Odoo - Several improvements (helm#6316)
  Joomla! - Several improvements (helm#6322)
  update signalfx-agent helm chart to v0.1.2 to deploy agent v3.2.2 (helm#6202)
  cert-manager: fast-forward to upstream f804cb56 (helm#6308)
  [stable/concourse] correct the parameter name  (helm#6283)
  [stable/envoy] chart version 1.0.0 (helm#5955)
  nginx-ingress/values.yaml: Updated tag to 0.15.0. (helm#5947)
  [stable/keycloak] Documentation fixes (helm#6289)
  [sumologic-fluentd] Update to latest image and expose new configuration to control stat watcher on fluentD in_tail plugins. (helm#6304)
  Adds tolerations labels for Jaeger pods (helm#6255)
  Upgrade image version for kube-slack (helm#6305)
  Add katafygio chart (helm#5543)
  [stable/k8s-spot-rescheduler] Add support for node selector and tolerations (helm#5789)
  schema-registry - Templating for liveness/readiness probes (helm#6301)
  ...

# Conflicts:
#	stable/artifactory-ha/Chart.yaml
#	stable/artifactory-ha/README.md
#	stable/artifactory-ha/values.yaml
or1can pushed a commit to or1can/charts that referenced this pull request Jul 10, 2018
* Add support for AWS Secrets Manager

* Rename credentialManager.secretsmanager to credentialManager.awsSecretsManager

* Bump minor version for new feature
voron pushed a commit to dysnix/helm-charts that referenced this pull request Sep 5, 2018
* Add support for AWS Secrets Manager

* Rename credentialManager.secretsmanager to credentialManager.awsSecretsManager

* Bump minor version for new feature

Signed-off-by: voron <av@arilot.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. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants