Skip to content
This repository has been archived by the owner on Jul 25, 2024. It is now read-only.

fix: complete terraform experiment cutover #481

Merged
merged 3 commits into from
Jun 4, 2022
Merged

Conversation

grayside
Copy link
Collaborator

@grayside grayside commented Jun 4, 2022

This PR completes the work in #476 by taking the following steps:

  • Delete the deprecated terraform configuration
  • Reduce the drift between original setup.sh and experimental-setup.sh
  • Replace setup.sh with experimental-setup.sh
  • Add error trap to setup.sh
  • Add more skip variables to setup.sh
  • Output section headers to more easily follow what setup.sh is doing
  • Command formatting polish (e.g., one flag per line)

I've made my way through a complete installation of setup.sh using the new skip variables & manually ran the seeding scripts. I found the system is enabled, logging is was fine. The Content API is throwing 500 errors but it's unclear whether that's related to these changes.

@grayside grayside self-assigned this Jun 4, 2022
@grayside grayside requested a review from a team as a code owner June 4, 2022 05:03
@github-actions github-actions bot added component: delivery Related to automation, testing, deployment of the application. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. labels Jun 4, 2022
@grayside grayside requested a review from ace-n June 4, 2022 05:04
@grayside grayside merged commit 2d5d470 into main Jun 4, 2022
@grayside grayside deleted the terraform-cutover branch June 4, 2022 22:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
component: delivery Related to automation, testing, deployment of the application. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants