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

Add Annotations to created ServiceAccount #480

Merged

Conversation

endzyme
Copy link
Contributor

@endzyme endzyme commented Oct 5, 2022

In order to support AWS IAM Role assumption via the pod-identity-webhook, there need to be annotations added to the ServiceAccount.

This change enables users to add annotations to the ServiceAccount created for the SolrCloud instance.

Copy link
Contributor

@HoustonPutman HoustonPutman left a comment

Choose a reason for hiding this comment

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

This looks good to me.

Could you add a changelog entry in the Chart.yaml, and add it to the list of available options in the chart's README?

This is also something that we could add to the solr-operator chart.

In order to support AWS IAM Role assumption via the
[pod-identity-webhook](https://github.com/aws/amazon-eks-pod-identity-webhook),
there need to be annotations added to the ServiceAccount.
@endzyme endzyme force-pushed the add-annotations-to-solr-service-account branch from 024a991 to f152054 Compare November 3, 2022 18:12
@endzyme
Copy link
Contributor Author

endzyme commented Nov 3, 2022

@HoustonPutman I've updated the changelog and added the same feature to the solr-operator chart.

@endzyme endzyme requested a review from HoustonPutman November 3, 2022 18:14
@endzyme endzyme mentioned this pull request Nov 7, 2022
@HoustonPutman
Copy link
Contributor

Made a fix for the serviceAccount when annotations are not provided, but otherwise should be good to go! This will make it into v0.7.0

@HoustonPutman HoustonPutman added this to the main (v0.7.0) milestone Nov 7, 2022
@HoustonPutman HoustonPutman added custom kube options Adding options related to customizing parts of the default Kubernetes resources. helm labels Nov 7, 2022
@HoustonPutman HoustonPutman merged commit 4052556 into apache:main Nov 7, 2022
@endzyme endzyme deleted the add-annotations-to-solr-service-account branch November 10, 2022 03:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
custom kube options Adding options related to customizing parts of the default Kubernetes resources. helm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants