Skip to content

Revert "osd/PeeringState: proc_lease_ack break once found from OSD"#49537

Merged
ljflores merged 1 commit intoceph:mainfrom
rzarzynski:wip-bug-57546
Jan 26, 2023
Merged

Revert "osd/PeeringState: proc_lease_ack break once found from OSD"#49537
ljflores merged 1 commit intoceph:mainfrom
rzarzynski:wip-bug-57546

Conversation

@rzarzynski
Copy link
Contributor

@rzarzynski rzarzynski commented Dec 21, 2022

This reverts commit 9337fa6.

It's a main's counterpart of quincy's PRs #48104 and #49304.

Fixes: https://tracker.ceph.com/issues/57546
Signed-off-by: Radoslaw Zarzynski rzarzyns@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

@rzarzynski
Revert "osd/PeeringState: proc_lease_ack break once found from OSD"
91a445c 
This reverts commit 9337fa6.

It's a `main`'s counterpart of quincy's PRs ceph#48104 and ceph#49304.

Fixes: https://tracker.ceph.com/issues/57546
Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
@rzarzynski rzarzynski requested a review from a team as a code owner December 21, 2022 22:15
@github-actions github-actions bot added the core label Dec 21, 2022
@ljflores
Copy link
Member

ljflores commented Jan 26, 2023

Rados suite review: https://pulpito.ceph.com/?branch=wip-yuri-testing-2023-01-23-0926

Failures, unrelated:
1. https://tracker.ceph.com/issues/58587 -- new tracker
2. https://tracker.ceph.com/issues/58585
3. https://tracker.ceph.com/issues/58098 -- fix merged to latest main
4. https://tracker.ceph.com/issues/58256 -- fix merged to latest main
5. https://tracker.ceph.com/issues/57900
6. https://tracker.ceph.com/issues/58475
7. https://tracker.ceph.com/issues/58560

Details:
1. test_dedup_tool.sh: test_dedup_object fails when pool 'dedup_chunk_pool' does not exist - Ceph - RADOS
2. rook: failed to pull kubelet image - Ceph - Orchestrator
3. qa/workunits/rados/test_crash.sh: crashes are never posted - Ceph - RADOS
4. ObjectStore/StoreTestSpecificAUSize.SpilloverTest/2: Expected: (logger->get(l_bluefs_slow_used_bytes)) >= (16 * 1024 * 1024), actual: 0 vs 16777216 - Ceph - RADOS
5. mon/crush_ops.sh: mons out of quorum - Ceph - RADOS
6. test_dashboard_e2e.sh: Conflicting peer dependency: postcss@8.4.21 - Ceph - Mgr - Dashboard
7. test_envlibrados_for_rocksdb.sh failed to subscrib repo - Ceph

@ljflores ljflores merged commit 03ec790 into ceph:main Jan 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@neha-ojha neha-ojha neha-ojha approved these changes

@ljflores ljflores Awaiting requested review from ljflores

Assignees

No one assigned

Labels

bug-fix core needs-qa wip-yuri-testing

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@rzarzynski @ljflores @neha-ojha