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

feat: expose tls certs as UDS bundle variables #48

Merged
merged 2 commits into from
Nov 19, 2023

Conversation

jeff-mccoy
Copy link
Member

@jeff-mccoy jeff-mccoy commented Nov 19, 2023

Description

This PR leverages the new UDS Helm Variables mapping introduce in UDS CLI v0.3.0. While this is currently at the bundle-level, this should eventually make it down to the Zarf level. However, for UDS deployers, the change will be transparent.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Other (security config, docs update, etc)

Checklist before merging

@jeff-mccoy jeff-mccoy requested a review from a team as a code owner November 19, 2023 05:46
Copy link
Contributor

@zachariahmiller zachariahmiller left a comment

Choose a reason for hiding this comment

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

Change looks good. Not sure how I feel about the scalability of the bundle overrides syntax as more inputs are added, but it will take time to flesh that out.

@jeff-mccoy
Copy link
Member Author

Change looks good. Not sure how I feel about the scalability of the bundle overrides syntax as more inputs are added, but it will take time to flesh that out.

Yeah agreed, we're doing it at this level for now until we settle on the final style to push down to zarf.

@jeff-mccoy jeff-mccoy merged commit c1f8286 into main Nov 19, 2023
7 checks passed
@jeff-mccoy jeff-mccoy deleted the bundle-helm-variables branch November 19, 2023 22:41
jeff-mccoy pushed a commit that referenced this pull request Nov 19, 2023
🤖 I have created a release *beep* *boop*
---


##
[0.5.0](v0.4.1...v0.5.0)
(2023-11-19)


### Features

* expose tls certs as UDS bundle variables
([#48](#48))
([c1f8286](c1f8286))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
robmcelvenny pushed a commit to owen-grady/uds-core-slim-dev that referenced this pull request Jun 3, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.5.0](defenseunicorns/uds-core@v0.4.1...v0.5.0)
(2023-11-19)


### Features

* expose tls certs as UDS bundle variables
([#48](defenseunicorns/uds-core#48))
([c1f8286](defenseunicorns/uds-core@c1f8286))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.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.

2 participants