Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show HashiCorp Vault Address when using 'kubectl get ta' or 'kubectl get cta' #1862

Merged
merged 3 commits into from
Jun 7, 2021

Conversation

Shubham82
Copy link
Contributor

@Shubham82 Shubham82 commented Jun 4, 2021

Signed-off-by: Shubham Kuchhal shubham.kuchhal@india.nec.com

Add VaultAdress field in TriggerAuthentication and ClusterTriggerAuthentication CRD so on using kubectl get TriggerAuthentication or kubectl get ClusterTriggerAuthentication it would show the Address of HashiCorp Vault.

Checklist

  • Commits are signed with Developer Certificate of Origin (DCO - learn more)
  • Changelog has been updated

Fixes #1826

… cta

Signed-off-by: Shubham Kuchhal <shubham.kuchhal@india.nec.com>
@Shubham82 Shubham82 changed the title Show HashiCorp Vault Address when using Kubectl get ta or kubectl get cta Show HashiCorp Vault Address when using Kubectl get ta or kubectl get cta Jun 4, 2021
@Shubham82 Shubham82 changed the title Show HashiCorp Vault Address when using Kubectl get ta or kubectl get cta Show HashiCorp Vault Address when using 'kubectl get ta' or 'kubectl get cta' Jun 4, 2021
@Shubham82
Copy link
Contributor Author

Hi @tomkerkhove @zroubalik
PTAL

Copy link
Member

@tomkerkhove tomkerkhove left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@zroubalik zroubalik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@zroubalik
Copy link
Member

And please update changelog as well :)

Signed-off-by: Shubham Kuchhal <shubham.kuchhal@india.nec.com>
@Shubham82
Copy link
Contributor Author

And please update changelog as well :)

Addressed the comment

@Shubham82
Copy link
Contributor Author

Looking good, could you please open another PR for charts, so the CRDs are updated there as well?

https://github.com/kedacore/charts/blob/master/keda/templates/05-crd-triggerauthentications.keda.sh.yaml

https://github.com/kedacore/charts/blob/master/keda/templates/02-crd-clustertriggerauthentications.yaml

Sure I will soon create a PR for it also.

Signed-off-by: Shubham Kuchhal <shubham.kuchhal@india.nec.com>
@Shubham82
Copy link
Contributor Author

Copy link
Member

@zroubalik zroubalik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @Shubham82 !

@zroubalik zroubalik merged commit d75dbee into kedacore:main Jun 7, 2021
@zroubalik zroubalik added this to the v2.4.0 milestone Jul 14, 2021
nilayasiktoprak pushed a commit to nilayasiktoprak/keda that referenced this pull request Oct 23, 2021
…get cta' (kedacore#1862)

Signed-off-by: Shubham Kuchhal <shubham.kuchhal@india.nec.com>
Signed-off-by: nilayasiktoprak <nilayasiktoprak@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Provide information concerning HashiCorp Vault in CRD
3 participants