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

qa/suites: add test exercising workunits/mon/auth_caps.sh #15754

Merged
merged 3 commits into from Jul 5, 2017

Conversation

Projects
None yet
4 participants
@tchaikov
Contributor

tchaikov commented Jun 19, 2017

No description provided.

qa/workunits/mon/auth_caps.sh: Revert "qa/workunits: mon: auth_caps: …
…account for mon blank caps"

This reverts commit f0653c0.

--force is not implemented by AuthMonitor. so revert this change to test
it.

Signed-off-by: Kefu Chai <kchai@redhat.com>

@tchaikov tchaikov added the tests label Jun 19, 2017

@tchaikov tchaikov requested review from jecluis and badone Jun 19, 2017

qa/workunits/mon/auth_caps.sh: remove tests for blank cap
* remove tests for blank cap: this feature is not supported/implemented by AuthMonitor.
* remove cap for client.baz after done with it. so we don't have error
  like: "entity client.baz exists but caps do not match" when trying to
  re-set the cap of it.

Signed-off-by: Kefu Chai <kchai@redhat.com>
@badone

badone approved these changes Jun 19, 2017

LGTM

@tchaikov

This comment has been minimized.

Contributor

tchaikov commented Jun 19, 2017

per discussion with @jecluis, will make this a part of "make check". as it's just a smoke test.

@liewegas liewegas added the mon label Jun 20, 2017

@tchaikov

This comment has been minimized.

Contributor

tchaikov commented Jun 26, 2017

@jecluis ping?

@jecluis

lgtm

@jecluis

This comment has been minimized.

Member

jecluis commented Jun 27, 2017

jenkins retest this please

qa/suites: add test exercising workunits/mon/auth_caps.sh
Signed-off-by: Kefu Chai <kchai@redhat.com>

@tchaikov tchaikov merged commit 04e0ef5 into ceph:master Jul 5, 2017

4 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
default Build finished.
Details
make check make check succeeded
Details

@tchaikov tchaikov deleted the tchaikov:wip-test-auth-caps branch Jul 5, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment