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

swift-proxy: wait for model to settle after running action #1139

Merged
merged 2 commits into from
Sep 28, 2023

Conversation

freyes
Copy link
Member

@freyes freyes commented Sep 27, 2023

Let the units run all the hooks before attempting to check if the ring is synced, because the action set-weight triggers the hook swift-storage-relation-changed in the swift-storage units which it's the one in charge of getting the new ring data.

Let the units run all the hooks before attempting to check if the ring
is synced, because the action set-weight triggers the hook
swift-storage-relation-changed in the swift-storage units which it's the
one in charge of getting the new ring data.
@freyes
Copy link
Member Author

freyes commented Sep 27, 2023

this PR is being exercised by https://review.opendev.org/c/openstack/charm-swift-proxy/+/896723

@freyes freyes requested a review from ajkavanagh September 28, 2023 11:52
@ajkavanagh ajkavanagh merged commit f143d7c into openstack-charmers:master Sep 28, 2023
3 checks passed
openstack-mirroring pushed a commit to openstack/charm-swift-proxy that referenced this pull request Sep 28, 2023
Func-Test-Pr: openstack-charmers/zaza-openstack-tests#1139
Change-Id: Id06e0e004133a157c447af31f3cb308eefa722d1
openstack-mirroring pushed a commit to openstack/openstack that referenced this pull request Sep 28, 2023
* Update charm-swift-proxy from branch 'master'
  to cf1ffe68666a019c51ba3550dad2e3b02ce4718f
  - Charm-helpers sync
    
    Func-Test-Pr: openstack-charmers/zaza-openstack-tests#1139
    Change-Id: Id06e0e004133a157c447af31f3cb308eefa722d1
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