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

chore: add keycloak to dev bundle and rename #262

Merged
merged 15 commits into from
Mar 19, 2024
Merged

Conversation

zachariahmiller
Copy link
Contributor

@zachariahmiller zachariahmiller commented Mar 14, 2024

Description

This change replaces the core-istio bundle with a dev-slim bundle that includes keycloak in addition to istio and pepr.

Related Issue

#251

@zachariahmiller zachariahmiller marked this pull request as draft March 14, 2024 23:00
@zachariahmiller zachariahmiller linked an issue Mar 14, 2024 that may be closed by this pull request
@zachariahmiller zachariahmiller changed the title wip: Optional keycloak in dev fix: optional keycloak in dev bundle Mar 15, 2024
@zachariahmiller
Copy link
Contributor Author

Will update to fix linting errors and add a couple lines to to docs in a bit, but if there is other feedback to address lmk

MxNxPx
MxNxPx previously approved these changes Mar 15, 2024
README.md Outdated Show resolved Hide resolved
Copy link
Contributor

@mjnagel mjnagel left a comment

Choose a reason for hiding this comment

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

We'll also need to update release-please-config.json to have the new filepaths (instead of packages/istio/zarf.yaml and bundles/k3d-istio/uds-bundle.yaml).

Otherwise this LGTM - validated the task changes for create/publish and all seems in line.

README.md Outdated Show resolved Hide resolved
tasks/deploy.yaml Outdated Show resolved Hide resolved
@zachariahmiller
Copy link
Contributor Author

@mjnagel Updated with your suggested changes and re-requested review. Assuming CI passes, which it should, I will go ahead and update the pr description and title after this next meeting.

@zachariahmiller zachariahmiller changed the title fix: optional keycloak in dev bundle fix: add keylaock to dev bundle and rename Mar 19, 2024
@zachariahmiller zachariahmiller changed the title fix: add keylaock to dev bundle and rename fix: add keycloak to dev bundle and rename Mar 19, 2024
@zachariahmiller
Copy link
Contributor Author

@mjnagel i resolved your comments and updated the description and pr title.

Copy link
Contributor

@mjnagel mjnagel left a comment

Choose a reason for hiding this comment

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

(last one I promise) this readme should be updated to the new name/description.

@mjnagel mjnagel enabled auto-merge (squash) March 19, 2024 22:36
@mjnagel mjnagel changed the title fix: add keycloak to dev bundle and rename chore: add keycloak to dev bundle and rename Mar 19, 2024
@mjnagel mjnagel merged commit f9b905c into main Mar 19, 2024
8 checks passed
@mjnagel mjnagel deleted the optiona_keycloak_in_dev branch March 19, 2024 23:17
jeff-mccoy pushed a commit that referenced this pull request Mar 22, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.17.0](v0.16.1...v0.17.0)
(2024-03-22)


### Features

* introduce sso secret templating
([#276](#276))
([e0832ec](e0832ec))


### Bug Fixes

* add keycloak to dev bundle and rename
([#262](#262))
([f9b905c](f9b905c))
* registration robot check form id
([#269](#269))
([c6419b9](c6419b9))
* sticky sessions for keycloak in ha
([#281](#281))
([5ccd557](5ccd557))


### Miscellaneous

* align mutation annotations
([#268](#268))
([f18ad4d](f18ad4d))
* **deps:** update loki
([#209](#209))
([03ca499](03ca499))
* **deps:** update pepr to v0.28.6
([#254](#254))
([54ef7de](54ef7de))
* **deps:** update zarf to v0.32.5
([#243](#243))
([ee93612](ee93612))
* typo fix in README.md
([#280](#280))
([f9727e0](f9727e0))

---
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.17.0](defenseunicorns/uds-core@v0.16.1...v0.17.0)
(2024-03-22)


### Features

* introduce sso secret templating
([#276](defenseunicorns/uds-core#276))
([e0832ec](defenseunicorns/uds-core@e0832ec))


### Bug Fixes

* add keycloak to dev bundle and rename
([#262](defenseunicorns/uds-core#262))
([f9b905c](defenseunicorns/uds-core@f9b905c))
* registration robot check form id
([#269](defenseunicorns/uds-core#269))
([c6419b9](defenseunicorns/uds-core@c6419b9))
* sticky sessions for keycloak in ha
([#281](defenseunicorns/uds-core#281))
([5ccd557](defenseunicorns/uds-core@5ccd557))


### Miscellaneous

* align mutation annotations
([#268](defenseunicorns/uds-core#268))
([f18ad4d](defenseunicorns/uds-core@f18ad4d))
* **deps:** update loki
([#209](defenseunicorns/uds-core#209))
([03ca499](defenseunicorns/uds-core@03ca499))
* **deps:** update pepr to v0.28.6
([#254](defenseunicorns/uds-core#254))
([54ef7de](defenseunicorns/uds-core@54ef7de))
* **deps:** update zarf to v0.32.5
([#243](defenseunicorns/uds-core#243))
([ee93612](defenseunicorns/uds-core@ee93612))
* typo fix in README.md
([#280](defenseunicorns/uds-core#280))
([f9727e0](defenseunicorns/uds-core@f9727e0))

---
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>
rjferguson21 pushed a commit that referenced this pull request Jul 11, 2024
## Description
This change replaces the core-istio bundle with a dev-slim bundle that
includes keycloak in addition to istio and pepr.

## Related Issue
#251

---------

Co-authored-by: Micah Nagel <micah.nagel@defenseunicorns.com>
rjferguson21 pushed a commit that referenced this pull request Jul 11, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.17.0](v0.16.1...v0.17.0)
(2024-03-22)


### Features

* introduce sso secret templating
([#276](#276))
([e0832ec](e0832ec))


### Bug Fixes

* add keycloak to dev bundle and rename
([#262](#262))
([f9b905c](f9b905c))
* registration robot check form id
([#269](#269))
([c6419b9](c6419b9))
* sticky sessions for keycloak in ha
([#281](#281))
([5ccd557](5ccd557))


### Miscellaneous

* align mutation annotations
([#268](#268))
([f18ad4d](f18ad4d))
* **deps:** update loki
([#209](#209))
([03ca499](03ca499))
* **deps:** update pepr to v0.28.6
([#254](#254))
([54ef7de](54ef7de))
* **deps:** update zarf to v0.32.5
([#243](#243))
([ee93612](ee93612))
* typo fix in README.md
([#280](#280))
([f9727e0](f9727e0))

---
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.

Update core-istio-dev to optionally deploy keycloak
3 participants