Skip to content

Cutover review-apps to new cluster#10791

Merged
stephencshelton merged 5 commits intomainfrom
sshelton/cutover-reviewapp
Jun 14, 2024
Merged

Cutover review-apps to new cluster#10791
stephencshelton merged 5 commits intomainfrom
sshelton/cutover-reviewapp

Conversation

@stephencshelton
Copy link
Copy Markdown
Contributor

@stephencshelton stephencshelton commented Jun 11, 2024

🛠 Summary of changes

Switches the kubectl context to instead use the new reviewapp cluster, and modifies the dns names to use the new zone for that cluster.

📜 Testing Plan

Provide a checklist of steps to confirm the changes.

  • All reviewapp dns names are up, resolvable, and functioning

Copy link
Copy Markdown
Contributor

@dskgsa dskgsa left a comment

Choose a reason for hiding this comment

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

LGTM

.gitlab-ci.yml Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

don't forget to switch this back

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

yep yep will do that once I get the autodiscovery stuff in place

@stephencshelton stephencshelton force-pushed the sshelton/cutover-reviewapp branch from 47dc55e to 172b254 Compare June 14, 2024 17:47
@stephencshelton stephencshelton force-pushed the sshelton/cutover-reviewapp branch from 172b254 to e458704 Compare June 14, 2024 18:59
@stephencshelton stephencshelton merged commit f84e557 into main Jun 14, 2024
@stephencshelton stephencshelton deleted the sshelton/cutover-reviewapp branch June 14, 2024 20:07
brandemix pushed a commit to brandemix/18F-identity-idp that referenced this pull request Jun 17, 2024
* changelog: CI, Internal, Cutover review-apps to new cluster
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.

4 participants