ceph_test_rados_api_{watch_notify,misc}: tolerate some timeouts #34011
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
liewegas
force-pushed
the
valgrind-tests
branch
from
cfc0920
to
7955dd2
Compare
When running under valgrind (and thrashing) things can be slow. Tell tests in case they need to tolerate timeouts. Signed-off-by: Sage Weil <sage@redhat.com>
See https://tracker.ceph.com/issues/44582 Signed-off-by: Sage Weil <sage@redhat.com>
liewegas
force-pushed
the
valgrind-tests
branch
from
d0abfef
to
99114ee
Compare
See https://tracker.ceph.com/issues/44062 Signed-off-by: Sage Weil <sage@redhat.com>
This reverts commit 65e81e6. This slows things down too much with valgrind. Signed-off-by: Sage Weil <sage@redhat.com>
This reverts commit 4f742f2. This was in the wrong file.. see valgrind.yaml Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
liewegas
force-pushed
the
valgrind-tests
branch
from
99114ee
to
7c19c15
Compare
badone
reviewed
Mar 18, 2020
|
the only failures are the kernel install thing (unrelated) and https://tracker.ceph.com/issues/43861 (which is still a mystery) |
neha-ojha
approved these changes
Mar 18, 2020
liewegas
added a commit
that referenced
this pull request
Mar 19, 2020
* refs/pull/34060/head: Merge PR #34027 into octopus Merge PR #34045 into octopus Merge pull request #34035 from dillaman/wip-rbd-permissions mgr/progress: fix duration strings Merge PR #34014 into octopus Merge PR #34001 into octopus Merge PR #34011 into octopus qa/workunits/rbd: use context managers to control Rados lifespan Merge pull request #34032 from dillaman/wip-rbd-octopus-docs doc/releases/octopus: add additional RBD improvements qa/workunits/cephadm/test_cephadm: mark services unmanaged for test mgr/cephadm: do not reconfig unmanaged services Merge PR #33981 into octopus Merge pull request #34018 from ajarr/octopus-subvolume-clone-cancel qa/workunits/cephadm/test_cephadm: output file for pub key Merge PR #33866 into octopus Merge PR #34005 into octopus Merge PR #34013 into octopus mgr/cephadm: pytest: Enable SpecStore mgr/orchestrator: add test for default implementation for apply() python-common: validate ServiceSpec.service_type fixup mgr/cephadm: Fix ceph orch apply -i mgr/dashbaord: orchestrator service: Revert wait_api_result to a single completion mgr/orchestrator: `orch daemon add` accepts a yaml mgr/cephadm: apply_drivegroups() returns a single Completion mgr/cephadm: remove `trivial_result()` mgr/cephadm: Fix `ceph orch apply -i` Merge pull request #33994 from dillaman/wip-librbd-poll-event-race doc: document `clone cancel` command test: add `clone cancel` tests mgr/volumes: introduce "clone cancel" volume command mgr/volumes: allow canceling a single asynchronous job for a volume mgr/volumes: helper for looking up a clone entry index mgr/volumes: periodically check if clone operations should be canceled mgr/volumes: periodically check if copy operations should be canceled mgr/volumes: introduce 'canceled' state in clone op state machine qa/suites/rados/verify/validater/valgrind: tolerate SLOW_OPS qa/suites/rados/verify/validater/valgrind: less bluestore logging qa/suites/rados/verify/validater: increase heartbeat grace Revert "qa/suites/rados/verify: debug_ms = 1, osd_heartbeat_grace = 60" Revert "qa/suites/rados/verify/validator/valgrind: debug refs = 5" ceph_test_watch_notify: try notify 10x if ALLOW_TIMEOUTS is set ceph_test_rados_api_misc: ShutdownRace timeout if ALLOW_TIMEOUTS is set qa/suites/rados/verify: set ALLOW_TIMEOUTS for workunits doc/install: edits doc/cephadm: more edits doc/cephadm/install: edits doc/cephadm/adoption: improvements doc/cephadm/install: a few edits doc/cephadm/install: do not install ceph-common on host (by default) doc/cephadm: drop os recs link doc/cephadm/upgrade: improvements doc/cephadm/upgrade: document upgrade doc/cephadm/install: revamp install docs doc: reorganize cephadm docs doc/cephadm/administration: update docs on customizing SSH config doc/cephadm/administration: add a note about the 'removed' dir mgr/balancer: tolerate pgs outside of target weight map qa/workunits/cephadm/test_cephadm: --skip-monitoring-stack Merge PR #33974 into octopus Merge PR #33442 into octopus Merge PR #33997 into octopus Merge PR #34000 into octopus use quay octopus tip until 15.2 tag is available python-common: reduce output of ServiceSpec.to_json() python-common,mgr/cephadm: move assert_valid_host to service_spec mgr/cephadm: add HostAssignment.validate() mgr/dashboard: adapt create_osds interface change mon/MgrMonitor: make 'mgr fail' work with no arguments cephadm: add allow_ptrace option to enable SYS_PTRACE update default container images mgr/cephadm: limit number of times check host is performed in the serve loop Merge PR #33961 into octopus Merge PR #33952 into octopus Merge PR #33990 into octopus Merge PR #33955 into octopus Merge PR #33936 into octopus mgr/orch: add --all-available-devices to 'orch apply osd' qa/workunits/cephadm: --skip-mon-network when using 127.0.0.1 cephadm: add tests qa/tasks/cephadm: pass -v to bootstrap mgr/cephadm: only try to place mons on hosts matching public_network mgr/cephadm: keep track of host networks, ips cephadm: automatically infer mon public_network, if we can cephadm: add list-networks command cephadm: bootstrap: deploy monitoring stack by default librbd: defer event socket completion until after callback issued cephadm: add-repo: add --version mgr/cephadm: respect 'unmanaged' flag in spec mgr/orch: orch ls: show <no spec> or <unmanaged> as appropriate mgr/orch: orch ls: rename SPEC -> PLACEMENT mgr/orch: add 'unmanaged' property to ServiceSpec cephadm: rename distro args in repo methods mgr/orch: combine 'orch daemon add <type> ...' into one command mgr/orch: combine 'orch apply <type> [<placement>]' into one command Reviewed-by: Laura Paduano <lpaduano@suse.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
have the valgrind teuthology test set ALLOW_TIMEOUTS=1, and adjust
some tests to tolerate timeouts when that is set.