Skip to content

mds: respawn standby-replay if read_pos is <= expire_pos#50615

Merged
vshankar merged 1 commit intoceph:mainfrom
batrick:i59119
Nov 3, 2023
Merged

mds: respawn standby-replay if read_pos is <= expire_pos#50615
vshankar merged 1 commit intoceph:mainfrom
batrick:i59119

Conversation

@batrick
Copy link
Member

@batrick batrick commented Mar 21, 2023

Debugging for https://tracker.ceph.com/issues/59119

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

batrick added a commit to batrick/ceph that referenced this pull request Mar 22, 2023
batrick added a commit to batrick/ceph that referenced this pull request Mar 24, 2023
batrick added a commit to batrick/ceph that referenced this pull request Mar 28, 2023
batrick added a commit to batrick/ceph that referenced this pull request Mar 28, 2023
batrick added a commit to batrick/ceph that referenced this pull request May 9, 2023
batrick added a commit to batrick/ceph that referenced this pull request May 18, 2023
batrick added a commit to batrick/ceph that referenced this pull request May 20, 2023
batrick added a commit to batrick/ceph that referenced this pull request Aug 30, 2023
batrick added a commit to batrick/ceph that referenced this pull request Sep 6, 2023
batrick added a commit to batrick/ceph that referenced this pull request Sep 12, 2023
batrick added a commit to batrick/ceph that referenced this pull request Oct 20, 2023
@batrick
Copy link
Member Author

batrick commented Oct 20, 2023

@vshankar I've had this PR in my test branches for a long time. I didn't run across the problem so wasn't able to debug with this PR. On its own I think it's useful so I'm going to set this as ready-for-review + merge, with your approval.

@batrick batrick marked this pull request as ready for review October 20, 2023 13:10
@batrick batrick requested review from a team and vshankar October 20, 2023 13:10
@batrick batrick added needs-review and removed DNM labels Oct 20, 2023
This adds debugging for https://tracker.ceph.com/issues/59119

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
@batrick
Copy link
Member Author

batrick commented Nov 2, 2023

@batrick
Copy link
Member Author

batrick commented Nov 2, 2023

ping @vshankar RFM w/ approval

Copy link
Contributor

@vshankar vshankar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vshankar vshankar merged commit 59f5087 into ceph:main Nov 3, 2023
@batrick batrick deleted the i59119 branch November 3, 2023 14:28
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