Skip to content

systest: fix race with pool delete after socket injection#46099

Merged
yuriw merged 1 commit intoceph:masterfrom
NitzanMordhai:wip-nitzan-rados-test-delete-pools-parallel
May 24, 2022
Merged

systest: fix race with pool delete after socket injection#46099
yuriw merged 1 commit intoceph:masterfrom
NitzanMordhai:wip-nitzan-rados-test-delete-pools-parallel

Conversation

@NitzanMordhai
Copy link
Contributor

@NitzanMordhai NitzanMordhai commented May 1, 2022 •
edited
Loading

Adding another semaphore for the second test to create pools only after the
first test delete complete.

Fixes: https://tracker.ceph.com/issues/43887
Signed-off-by: Nitzan Mordechai nmordec@redhat.com

Contribution Guidelines

Checklist

  • Tracker (select at least one)
    • References tracker ticket
    • Very recent bug; references commit where it was introduced
    • New feature (ticket optional)
    • Doc update (no ticket needed)
    • Code cleanup (no ticket needed)
  • Component impact
    • Affects Dashboard, opened tracker ticket
    • Affects Orchestrator, opened tracker ticket
    • No impact that needs to be tracked
  • Documentation (select at least one)
    • Updates relevant documentation
    • No doc update is appropriate
  • Tests (select at least one)
Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • 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 cephadm
  • jenkins test api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox
  • jenkins test windows

@NitzanMordhai
systest: fix race with pool delete after socket injection
4c00b44 
Adding another semaphore for the second test to create pools only after the
first test delete complete.

Fixes: https://tracker.ceph.com/issues/43887
Signed-off-by: Nitzan Mordechai <nmordec@redhat.com>
@github-actions github-actions bot added the tests label May 1, 2022
@NitzanMordhai NitzanMordhai requested a review from jdurgin May 1, 2022 05:38
@ljflores
Copy link
Member

ljflores commented May 23, 2022

http://pulpito.front.sepia.ceph.com/yuriw-2022-05-19_01:43:57-rados-wip-yuri7-testing-2022-05-18-1636-distro-default-smithi/

Failures, unrelated:
1. https://tracker.ceph.com/issues/52124
2. https://tracker.ceph.com/issues/55741
3. https://tracker.ceph.com/issues/51835
4. https://tracker.ceph.com/issues/52321

Details:
1. Invalid read of size 8 in handle_recovery_delete() - Ceph - RADOS
2. cephadm/test_dashboard_e2e.sh: Unable to find element cd-modal .custom-control-label when testing on orchestrator/01-hosts.e2e-spec.ts - Ceph - Mgr - Dashboard
3. mgr/DaemonServer.cc: FAILED ceph_assert(pending_service_map.epoch > service_map.epoch) - Ceph - RADOS
4. qa/tasks/rook times out: 'check osd count' reached maximum tries (90) after waiting for 900 seconds - Ceph - Orchestrator

@yuriw yuriw merged commit 123c574 into ceph:master May 24, 2022
@kamoltat
Copy link
Member

kamoltat commented Mar 2, 2023 •
edited
Loading

Encountered this error https://tracker.ceph.com/issues/43887, PR in the tracker points to this PR in: yuriw-2023-03-02_00:09:05-rados-wip-yuri11-testing-2023-03-01-1424-distro-default-smithi/7191503

PTAL @NitzanMordhai

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

Reviewers

@jdurgin jdurgin jdurgin approved these changes

Assignees

No one assigned

Labels

core needs-qa tests wip-yuri7-testing

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@NitzanMordhai @ljflores @kamoltat @jdurgin @yuriw