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.

Sign up for GitHub

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

Jump to bottom

rgw: replace period update --commit with function #8911

Merged
merged 2 commits into from Oct 12, 2021
Merged

rgw: replace period update --commit with function #8911

merged 2 commits into from Oct 12, 2021

Commits on Oct 11, 2021

  1. rgw: replace period update --commit with function 

    Replace calls to 'radosgw-admin period update --commit' with an
idempotent function.

Resolves rook#8879

Signed-off-by: Blaine Gardner <blaine.gardner@redhat.com>
    @BlaineEXE
    BlaineEXE committed Oct 11, 2021
    Copy the full SHA
    eadcd75 View commit details
    Browse the repository at this point in the history

  2. rgw: add integration test for committing period 

    Add to the RGW multisite integration test a verification that the RGW
period is committed on the first reconcile and not committed on the
second reconcile.

Do this in the multisite test so that we verify that this works for
both the primary and secondary multi-site cluster.

To add this test, the github-action-helper.sh script had to be modified
to
1. actually deploy the version of Rook under test
2. adjust how functions are called to not lose the `-e` in a subshell
3. fix wait_for_prepare_pod helper that had a failure in the middle
   of its operation that didn't cause failures in the past

Signed-off-by: Blaine Gardner <blaine.gardner@redhat.com>
    @BlaineEXE
    BlaineEXE committed Oct 11, 2021
    Copy the full SHA
    9564308 View commit details
    Browse the repository at this point in the history