Skip to content

qa/workunits/cephtool/test.sh: a handful fixes#31689

Merged
tchaikov merged 9 commits intoceph:masterfrom
tchaikov:wip/workunits/cephtool/test.sh
Nov 18, 2019
Merged

qa/workunits/cephtool/test.sh: a handful fixes#31689
tchaikov merged 9 commits intoceph:masterfrom
tchaikov:wip/workunits/cephtool/test.sh

Conversation

@tchaikov
Copy link
Contributor

introduced by 9095f67

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

Checklist

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug

Show available Jenkins commands
  • jenkins retest this please
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test dashboard backend
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

@tchaikov
Copy link
Contributor Author

@tchaikov tchaikov force-pushed the wip/workunits/cephtool/test.sh branch from 4614fb3 to 00da5ca Compare November 18, 2019 01:40
@tchaikov
Copy link
Contributor Author

changelog

  • tests SI units related test with a tier pool. as target_max_objects only applies to tier pools.

@tchaikov
Copy link
Contributor Author

@tchaikov tchaikov force-pushed the wip/workunits/cephtool/test.sh branch from 00da5ca to e097959 Compare November 18, 2019 02:43
@tchaikov tchaikov changed the title qa/workunits/cephtool/test.sh: s/TEST_POOLGETSET/TEST_POOL_GETSET/ qa/workunits/cephtool/test.sh: fix iec/si tests Nov 18, 2019
@tchaikov tchaikov force-pushed the wip/workunits/cephtool/test.sh branch 3 times, most recently from b6ea126 to 2d13b53 Compare November 18, 2019 05:49
tests all IEC and SI units related test with a tier pool. as
`target_max_objects` and `target_size_bytes` only apply to tier
pools. so, for the sake of simplicity, tests all of them using
a tier pool.

introduced by 9095f67

Signed-off-by: Kefu Chai <kchai@redhat.com>
after the big asok/tell cleanup, "heap" is now an asok command, so
update the manpage accordingly

Signed-off-by: Kefu Chai <kchai@redhat.com>
"heap" cli commands are removed in the big asok/tell cleanup, they are now
asock commands.

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

ceph cli will be able to route the command to the target daemon when
handling "tell" commands.

Signed-off-by: Kefu Chai <kchai@redhat.com>
@tchaikov tchaikov force-pushed the wip/workunits/cephtool/test.sh branch from 2d13b53 to 6f78420 Compare November 18, 2019 05:52
"version" is not an asok command anymore in the sense that it's served
by registered asock hook. so in this change, we replace "version" with
"sessions", so we can verify that audit channel does not the dispatched
"sessions" command sent from ceph cli.

also, restructure the test as a loop for better readability

Signed-off-by: Kefu Chai <kchai@redhat.com>
@tchaikov tchaikov force-pushed the wip/workunits/cephtool/test.sh branch from fc71d41 to e96df8c Compare November 18, 2019 09:09
…ted_commands()

both "compact" and "scrub" are now cli command, so we cannot issue them
using "tell" anymore.

Signed-off-by: Kefu Chai <kchai@redhat.com>
otherwise wait_for_health() fails like:

wait_for_health:  ceph health detail
HEALTH_WARN 1 pool(s) have non-power-of-two pg_num
[WRN] POOL_PG_NUM_NOT_POWER_OF_TWO: 1 pool(s) have non-power-of-two pg_num
    pool 'rbd' pg_num 10 is not a power of two
../qa/workunits/cephtool/../../standalone/ceph-helpers.sh:1613: wait_for_health:  return 1

the failure was found when testing test_mon_pg().

this behavior was introduced by 6e46b1c

Signed-off-by: Kefu Chai <kchai@redhat.com>
otherwise ceph-mgr will refuse to perform reweight

Signed-off-by: Kefu Chai <kchai@redhat.com>
@tchaikov tchaikov force-pushed the wip/workunits/cephtool/test.sh branch from cb3771d to e097959 Compare November 18, 2019 13:18
in cephtool/test.sh, we

ceph fs set cephfs inline_data {1,0}

so the health check fails when the test ends, like

mon.a (mon.0) 3498 : cluster [WRN] Health check failed: 1 filesystem
with deprecated feature inline_data (FS_INLINE_DATA_DEPRECATED)" in
cluster log

so, before we remove the test, we need to whitelist this warning

Signed-off-by: Kefu Chai <kchai@redhat.com>
@tchaikov tchaikov force-pushed the wip/workunits/cephtool/test.sh branch from e097959 to 153311a Compare November 18, 2019 14:25
@tchaikov tchaikov changed the title qa/workunits/cephtool/test.sh: fix iec/si tests qa/workunits/cephtool/test.sh: a handful fixes Nov 18, 2019
@tchaikov
Copy link
Contributor Author

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants