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

Update the embedded config data in mysqlctl #10094

Closed
wants to merge 6 commits into from

Conversation

mattlord
Copy link
Contributor

@mattlord mattlord commented Apr 14, 2022

Description

I will need to rebase this on main after #10080 is resolved and merged as that also makes some changes to the config files that get embedded. But the bulk of the work in this PR is about getting everything to work once super_read_only is actually applied.

Related Issue(s)

Checklist

  • "Backport me!" label has been added if this change should be backported
  • Tests were added or are not required
  • Documentation was added or is not required

Deployment Notes

@mattlord mattlord added Component: General Changes throughout the code base Type: Bug Type: Enhancement Logical improvement (somewhere between a bug and feature) release notes labels Apr 14, 2022
@mattlord mattlord force-pushed the update_mysqlctl_ricebox branch from c842011 to 9395e13 Compare April 14, 2022 05:04
@mattlord mattlord force-pushed the update_mysqlctl_ricebox branch 21 times, most recently from 9823bcd to dd0ecf9 Compare May 21, 2022 23:33
@mattlord mattlord force-pushed the update_mysqlctl_ricebox branch from dd0ecf9 to 5a893b9 Compare May 22, 2022 04:13
@mattlord mattlord force-pushed the update_mysqlctl_ricebox branch from 5a893b9 to a03de1f Compare May 22, 2022 04:19
@mattlord mattlord force-pushed the update_mysqlctl_ricebox branch 3 times, most recently from 0c132a2 to 39db12d Compare May 22, 2022 15:22
Restore test files

Signed-off-by: Matt Lord <[email protected]>
@mattlord mattlord force-pushed the update_mysqlctl_ricebox branch from 39db12d to ec030ae Compare May 22, 2022 15:23
mattlord added a commit to planetscale/vitess that referenced this pull request May 24, 2022
This ended up causing various failures when that change was applied
in mysqlctl by updating its rice box (make embed_config). You can see
this in: vitessio#10094

We're instead going to schedule work to add full super_read_only
support to Vitess in: vitessio#10363

Signed-off-by: Matt Lord <[email protected]>
@mattlord
Copy link
Contributor Author

We're going to revert the super_read_only config file changes from #9312 in #10366 and we created a new ticket for adding full super_read_only support to Vitess in 15+: #10363.

@mattlord mattlord closed this May 24, 2022
@mattlord mattlord deleted the update_mysqlctl_ricebox branch May 24, 2022 16:00
mattlord added a commit that referenced this pull request May 25, 2022
* Revert super_read_only config file changes

This ended up causing various failures when that change was applied
in mysqlctl by updating its rice box (make embed_config). You can see
this in: #10094

We're instead going to schedule work to add full super_read_only
support to Vitess in: #10363

* Remove MySQL 5.6 config

This is a follow-up to #10295
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: General Changes throughout the code base Type: Bug Type: Enhancement Logical improvement (somewhere between a bug and feature)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant