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

[Bytesafe] Add support for serviceAccountName #44

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

conclusionlogic
Copy link

This PR introduces a few updates to the statefulSet.yaml template:

  • Added conditions to check the creation of a ServiceAccount and set automountServiceAccountToken to true as well as referenced the correct ServiceAccountName value.
  • The serviceName attribute under spec has been explicitly set to use the bytesafe.fullname template, ensuring clarity in service references.
  • Indentation has been corrected across various sections to ensure consistency and improve the visual hierarchy of the YAML structure.
  • Conditional blocks for statefulSet.labels, statefulSet.annotations, pod.labels, and pod.annotations have been adjusted to include proper indentation and spacing, making it easier to distinguish between nested properties.

This commit introduces several changes to the `statefulSet.yaml` template to enhance readability and maintainability:
- Indentation has been corrected across various sections to ensure consistency and improve the visual hierarchy of the YAML structure.
- Conditional blocks for `statefulSet.labels`, `statefulSet.annotations`, `pod.labels`, and `pod.annotations` have been adjusted to include proper indentation and spacing, making it easier to distinguish between nested properties.
- The `serviceName` attribute under `spec` has been explicitly set to use the `bytesafe.fullname` template, ensuring clarity in service references.
- Added conditions to check the creation of a ServiceAccount and set `automountServiceAccountToken` to true as well as referenced the correct ServiceAccount name.
@conclusionlogic conclusionlogic changed the title Add support for serviceAccountName [Bytesafe] Add support for serviceAccountName Jul 21, 2024
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.

1 participant