Skip to content

mon/MDSMonitor: set birth time on FSMap during encode#53503

Merged
batrick merged 6 commits intoceph:mainfrom
batrick:i62849
Jun 25, 2024
Merged

mon/MDSMonitor: set birth time on FSMap during encode#53503
batrick merged 6 commits intoceph:mainfrom
batrick:i62849

Conversation

@batrick
Copy link
Member

@batrick batrick commented Sep 18, 2023

See also: rook/rook#12789

So we can begin to answer questions like: when did we last see an MDS?

Fixes: https://tracker.ceph.com/issues/62849

TODO:

  • docs
  • tests
  • release note
  • add command "last-seen "

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 batrick added the cephfs Ceph File System label Sep 18, 2023
@batrick batrick force-pushed the i62849 branch 2 times, most recently from 304c3a0 to 6606a3b Compare September 19, 2023 01:32
@batrick batrick force-pushed the i62849 branch 2 times, most recently from cd9bcf5 to 57ce744 Compare September 19, 2023 19:22
@batrick
Copy link
Member Author

batrick commented Sep 19, 2023

So I think this works now. Just need to add testing.

@github-actions github-actions bot removed the stale label Jan 17, 2024
@ceph ceph deleted a comment from github-actions bot Mar 13, 2024
@ceph ceph deleted a comment from github-actions bot Mar 13, 2024
@batrick batrick force-pushed the i62849 branch 4 times, most recently from 96c1338 to dbfca43 Compare June 18, 2024 15:57
@batrick
Copy link
Member Author

batrick commented Jun 19, 2024

jenkins test make check

@batrick
Copy link
Member Author

batrick commented Jun 19, 2024

jenkins test make check arm64

@batrick
Copy link
Member Author

batrick commented Jun 19, 2024

jenkins test dashboard

@batrick
Copy link
Member Author

batrick commented Jun 19, 2024

jenkins test dashboard cephadm

batrick added 6 commits June 20, 2024 21:32
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
So we can begin to answer questions like: when did we last see an MDS?

Fixes: https://tracker.ceph.com/issues/62849
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
For use by high-level storage operators like Rook.

Fixes: https://tracker.ceph.com/issues/62849
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
@batrick
Copy link
Member Author

batrick commented Jun 21, 2024

jenkins test make check

@batrick
Copy link
Member Author

batrick commented Jun 21, 2024

jenkins test make check arm64

@batrick
Copy link
Member Author

batrick commented Jun 21, 2024

jenkins test dashboard cephadm

@batrick
Copy link
Member Author

batrick commented Jun 21, 2024

jenkins test make check arm64

@batrick
Copy link
Member Author

batrick commented Jun 22, 2024

jenkins test make check

@batrick
Copy link
Member Author

batrick commented Jun 22, 2024

This PR is under test in https://tracker.ceph.com/issues/66609.

@batrick
Copy link
Member Author

batrick commented Jun 23, 2024

Ready for merge. @ceph/cephfs @vshankar requires approval please.

https://tracker.ceph.com/issues/66609#note-1

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.

Nice work!

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

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants