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

[stable/yoga] Adding fixing broken configuration test for mysql-router #1218

Merged
merged 1 commit into from
Jun 17, 2024

Conversation

xtrusia
Copy link
Contributor

@xtrusia xtrusia commented Jun 12, 2024

(cherry picked from commit 0a88897)

@ajkavanagh
Copy link
Collaborator

Note: stable/caracal, stable/bobcat, stable/antelope and stable/zed all require this cherry-pick for consistency. Otherwise, odd things may happen if testing with say stable/antelope.

@xtrusia
Copy link
Contributor Author

xtrusia commented Jun 17, 2024

Note: stable/caracal, stable/bobcat, stable/antelope and stable/zed all require this cherry-pick for consistency. Otherwise, odd things may happen if testing with say stable/antelope.

Thanks a lot. I created PR

#1219 #1220 #1221 #1222

@ajkavanagh ajkavanagh changed the title Adding fixing broken configuration test for mysql-router [stable/yoga] Adding fixing broken configuration test for mysql-router Jun 17, 2024
@ajkavanagh
Copy link
Collaborator

Note that: https://review.opendev.org/c/openstack/charm-mysql-router/+/921716 is using this PR; don't delete the branch until that review has merged.

@ajkavanagh ajkavanagh merged commit fd4674d into openstack-charmers:stable/yoga Jun 17, 2024
2 checks passed
openstack-mirroring pushed a commit to openstack/charm-mysql-router that referenced this pull request Jun 17, 2024
A customer faced an issue when they face full disk. Mysql router configuration file was broken. As this charm doesn't handle whole configuration file, charm wrote just part of them based on template. so re-bootstrapping is needed when this symptom happens.

To do that, I put code to check configuration file size, and if it is under 1000bytes, run bootstrap.

func-test-pr: openstack-charmers/zaza-openstack-tests#1218
Closes-Bug: #2004088
Change-Id: Ic28f0406eb8d5dbd9131ae83090498297ac48c31
(cherry picked from commit d334665)
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.

2 participants