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

EL-1238 Part B: Update hostnames and values #1064

Merged
merged 1 commit into from
Nov 6, 2023
Merged

Conversation

willc-work
Copy link
Contributor

@willc-work willc-work commented Nov 3, 2023

Jira ticket

What changed and why

Swapped the hostname for main from the existing NS to the new NS
Update values files to correspond e.g oauth
Add a wait in circleci to prevent ingress conflicts

Guidance to review

This PR is the 2nd for this ticket. It must not be merged until the redis and postgres DBs have been successfully migrated to the new NS. This merge will take the domain name offline briefly (a few minutes, likely less than 5), so even the maintenance page would not be visible to someone trying to access the site.

Part C: #1065

Checklist

Before you ask people to review this PR:

  • Tests and rubocop should be passing
  • Branch is generally up to date with main Github - definitely no conflicts
  • No unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
  • PR description says what changed and why, with a link to the JIRA story.
  • Diff has been checked for unexpected changes being included.
  • Commit messages say why the change was made.

@willc-work willc-work marked this pull request as ready for review November 3, 2023 15:24
@willc-work willc-work requested a review from a team as a code owner November 3, 2023 15:24
Base automatically changed from el-1238-switch-uat to main November 3, 2023 15:36
@patrick-laa patrick-laa merged commit b7d2cca into main Nov 6, 2023
12 checks passed
@patrick-laa patrick-laa deleted the el-1238-switch-uat2 branch November 6, 2023 10:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants