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

Adding fixing broken configuration test for mysql-router #1196

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

xtrusia
Copy link
Contributor

@xtrusia xtrusia commented Apr 9, 2024

try to fix broken conf automatically if size is under 1000bytes

openstack-mirroring pushed a commit to openstack/charm-mysql-router that referenced this pull request May 22, 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#1196
Closes-Bug: #2004088
Change-Id: Ic28f0406eb8d5dbd9131ae83090498297ac48c31
openstack-mirroring pushed a commit to openstack/openstack that referenced this pull request May 22, 2024
* Update charm-mysql-router from branch 'master'
  to 2cc90ec1f1a2f449dd785bed57a51c5632d29b0a
  - Merge "Fixing broken mysql-router configuration"
  - Fixing broken mysql-router configuration
    
    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#1196
    Closes-Bug: #2004088
    Change-Id: Ic28f0406eb8d5dbd9131ae83090498297ac48c31
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.

None yet

1 participant