Skip to content
New issue

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

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

Already on GitHub? # to your account

Environment Variable Updates #994

Merged
merged 10 commits into from
Jun 15, 2024
Merged

Environment Variable Updates #994

merged 10 commits into from
Jun 15, 2024

Conversation

v0lkan
Copy link
Contributor

@v0lkan v0lkan commented Jun 15, 2024

Environment Variable Updates

Description

This update moves environment variables strings to symbolic constants to
make the code less error-prone, and easier to refactor.

Changes

  • Moved env vars to ./core/constants/constants.go.
  • Sorted the variables alphabetically.
  • Created constants for the default values of the env vars oo.
  • Updated relevant documentation.
  • Cross-checked that all env vars in the code also exist in the public documentation.
  • Created constants.GetEnv() that ensures that a constant is already defined
    for an env var to be used.
  • Created helpers like constants.True(s) and constants.Never(s).
  • Renamed certain environment variables to keep them together when sorted alphabetically.
  • updated helm values and regenerated manifests.
  • added retries to certain make targets (sometimes, some k8s hooks were not ready and it was resulting in ClusterSPIFFEIDs not reconciling properly; retrying the make command after a 5-10 second wait seemed to have fixed that)

Test Policy Compliance

  • I have added or updated unit tests for my changes.
  • I have included integration tests where applicable.
  • All new and existing tests pass successfully.

Code Quality

  • I have followed the coding standards for this project.
  • I have performed a self-review of my code.
  • My code is well-commented, particularly in areas that may be difficult
    to understand.

Documentation

  • I have made corresponding changes to the documentation (if applicable).
  • I have updated any relevant READMEs or wiki pages.

Checklist

Before you submit this PR, please make sure:

  • You have read the contributing guidelines and
    especially the test policy.
  • You have thoroughly tested your changes.
  • You have followed all the contributing guidelines for this project.
  • You understand and agree that your contributions will be publicly available
    under the project's license.

By submitting this pull request, you confirm that my contribution is made under
the terms of the project's license and that you have the authority to grant
these rights.


Thank you for your contribution to VMware Secrets Manager
🐢⚡️!

v0lkan added 10 commits June 14, 2024 00:08
Signed-off-by: Volkan Ozcelik <me@volkan.io>
Signed-off-by: Volkan Ozcelik <me@volkan.io>
Signed-off-by: Volkan Ozcelik <me@volkan.io>
Signed-off-by: Volkan Ozcelik <me@volkan.io>
Signed-off-by: Volkan Ozcelik <me@volkan.io>
Signed-off-by: Volkan Ozcelik <me@volkan.io>
Signed-off-by: Volkan Ozcelik <me@volkan.io>
Signed-off-by: Volkan Ozcelik <me@volkan.io>
Signed-of-by: Volkan Ozcelik <me@volkan.io>
Signed-off-by: Volkan Ozcelik <me@volkan.io>
@v0lkan v0lkan self-assigned this Jun 15, 2024
@v0lkan v0lkan requested a review from BulldromeQ as a code owner June 15, 2024 09:00
@vmwclabot
Copy link

@v0lkan, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding Signed-off-by: John Doe <john.doe@email.org> to the last line of each Git commit message. The e-mail address used to sign must match the e-mail address of the Git author. Click here to view the Developer Certificate of Origin agreement.

@v0lkan v0lkan merged commit 3125570 into main Jun 15, 2024
@v0lkan v0lkan deleted the ovolkan/env branch June 15, 2024 09:00
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants