Skip to content

Revert "Rename values in SiteConfiguration (2/3) attempt #2" - #23224

Merged
pwnage101 merged 1 commit into
masterfrom
revert-23214-pwnage101/rename_site_configuration_siteconfiguration_values_2.1
Feb 27, 2020
Merged

Revert "Rename values in SiteConfiguration (2/3) attempt #2"#23224
pwnage101 merged 1 commit into
masterfrom
revert-23214-pwnage101/rename_site_configuration_siteconfiguration_values_2.1

Conversation

@pwnage101

Copy link
Copy Markdown
Contributor

Reverts edx/edx-platform#23214

The same migration failed again, but this time a different reason. JSONDecodeError on the site_values column in the history table, which apparently did not inherit the same default value as the non-history table.

@pwnage101
pwnage101 requested review from a team February 27, 2020 18:26
@pwnage101
pwnage101 requested a review from a team as a code owner February 27, 2020 18:26

@doctoryes doctoryes left a comment

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.

😭

@edx-status-bot

Copy link
Copy Markdown

Your PR has finished running tests. There were no failures.

@pwnage101
pwnage101 merged commit ac4fd6f into master Feb 27, 2020
@pwnage101
pwnage101 deleted the revert-23214-pwnage101/rename_site_configuration_siteconfiguration_values_2.1 branch February 27, 2020 18:58
@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production.

@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR may have caused e2e tests to fail on Stage. If you're a member of the edX org, please visit #e2e-troubleshooting on Slack to help diagnose the cause of these failures. Otherwise, it is the reviewer's responsibility. E2E tests have failed. https://gocd.tools.edx.org/go/tab/pipeline/history/deploy_to_stage

@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the production environment.

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