Skip to content

rgw/logging: fix canonical names#63909

Merged
yuvalif merged 4 commits intoceph:mainfrom
yuvalif:wip-yuval-71638
Jul 7, 2025
Merged

rgw/logging: fix canonical names#63909
yuvalif merged 4 commits intoceph:mainfrom
yuvalif:wip-yuval-71638

Conversation

@yuvalif
Copy link
Contributor

@yuvalif yuvalif commented Jun 12, 2025

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

split to 3 commits so it is easier to review. can squash into one before merge

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

@yuvalif yuvalif marked this pull request as ready for review June 12, 2025 12:33
@yuvalif yuvalif requested a review from a team as a code owner June 12, 2025 12:33
@github-actions github-actions bot added the rgw label Jun 12, 2025
@adamemerson
Copy link
Contributor

jenkins test make check arm64

@yuvalif
Copy link
Contributor Author

yuvalif commented Jun 24, 2025

yuvalif added 4 commits July 2, 2025 13:25
Fixes: https://tracker.ceph.com/issues/71638

Signed-off-by: Yuval Lifshitz <ylifshit@ibm.com>
function is moved up in the inheritance hierarchy
when possible

Signed-off-by: Yuval Lifshitz <ylifshit@ibm.com>
Signed-off-by: Yuval Lifshitz <ylifshit@ibm.com>
@yuvalif yuvalif force-pushed the wip-yuval-71638 branch from 3576872 to 56c7537 Compare July 3, 2025 09:30
@adamemerson
Copy link
Contributor

@anrao19 Is this done with QA? (If not that's fine! I just want to make sure things are moving forward.)

@anrao19
Copy link
Contributor

anrao19 commented Jul 4, 2025

@anrao19 Is this done with QA? (If not that's fine! I just want to make sure things are moving forward.)

Hi @adamemerson , pr test https://tracker.ceph.com/issues/71817 is in "QA Needs Rerun/Rebuilt"
will trigger rerun , then will update the result for the same

@yuvalif
Copy link
Contributor Author

yuvalif commented Jul 7, 2025

jenkins test make check arm64

@yuvalif yuvalif merged commit ae67c20 into ceph:main Jul 7, 2025
13 checks passed
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.

3 participants