Skip to content

qa/cephfs: add ceph-osd-{classic,crimson} to exclude list of LTS releases#66475

Merged
vshankar merged 1 commit intoceph:mainfrom
vshankar:wip-74026
Feb 23, 2026
Merged

qa/cephfs: add ceph-osd-{classic,crimson} to exclude list of LTS releases#66475
vshankar merged 1 commit intoceph:mainfrom
vshankar:wip-74026

Conversation

@vshankar
Copy link
Contributor

@vshankar vshankar commented Dec 2, 2025

Fixes: http://tracker.ceph.com/issues/74026

Contribution Guidelines

  • To sign and title your commits, please refer to Submitting Patches to Ceph.

  • If you are submitting a fix for a stable branch (e.g. "quincy"), please refer to Submitting Patches to Ceph - Backports for the proper workflow.

  • When filling out the below checklist, you may click boxes directly in the GitHub web UI. When entering or editing the entire PR message in the GitHub web UI editor, you may also select a checklist item by adding an x between the brackets: [x]. Spaces and capitalization matter when checking off items this way.

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

You must only issue one Jenkins command per-comment. Jenkins does not understand
comments with more than one command.

@vshankar vshankar requested a review from a team December 2, 2025 04:39
@vshankar vshankar added the cephfs Ceph File System label Dec 2, 2025
@github-actions github-actions bot added the tests label Dec 2, 2025
vshankar added a commit to vshankar/ceph that referenced this pull request Dec 2, 2025
@dparmar18
Copy link
Contributor

@vshankar is this something temporary or will we always need to exclude these packages?

@vshankar
Copy link
Contributor Author

vshankar commented Dec 2, 2025

@vshankar is this something temporary or will we always need to exclude these packages?

Always I presume. The changes were in main branch, but that effects tests which use LTS releases.

@dparmar18
Copy link
Contributor

@vshankar is this something temporary or will we always need to exclude these packages?

Always I presume. The changes were in main branch, but that effects tests which use LTS releases.

oh okay how about patching teuthology code to exclude these packages here https://github.com/ceph/teuthology/blob/258eb6279f4d7fcd4b45c82e521f2a2e799d7f33/teuthology/task/install/__init__.py#L165-L166

@vshankar
Copy link
Contributor Author

vshankar commented Dec 2, 2025

@vshankar is this something temporary or will we always need to exclude these packages?

Always I presume. The changes were in main branch, but that effects tests which use LTS releases.

oh okay how about patching teuthology code to exclude these packages here https://github.com/ceph/teuthology/blob/258eb6279f4d7fcd4b45c82e521f2a2e799d7f33/teuthology/task/install/__init__.py#L165-L166

Why does that change needs to go in teuthology? The yaml can list the packages to exclude installation. Moreover, this is release specific.

@dparmar18
Copy link
Contributor

@vshankar is this something temporary or will we always need to exclude these packages?

Always I presume. The changes were in main branch, but that effects tests which use LTS releases.

oh okay how about patching teuthology code to exclude these packages here https://github.com/ceph/teuthology/blob/258eb6279f4d7fcd4b45c82e521f2a2e799d7f33/teuthology/task/install/__init__.py#L165-L166

Why does that change needs to go in teuthology? The yaml can list the packages to exclude installation. Moreover, this is release specific.

okay i thought this might be needed universally.

@vshankar
Copy link
Contributor Author

vshankar commented Dec 3, 2025

This change need a bit more changes to get upgrade tests and other LTS tests working.

@vshankar
Copy link
Contributor Author

vshankar commented Dec 3, 2025

So, this could be the problem. During upgrade

2025-12-02T05:37:59.267 INFO:teuthology.orchestra.run.smithi001.stdout:  Upgrading        : ceph-osd-2:20.3.0-4253.g9422a98d.el9.x86_64          19/82
2025-12-02T05:38:00.770 INFO:teuthology.orchestra.run.smithi001.stdout:  Running scriptlet: ceph-osd-2:20.3.0-4253.g9422a98d.el9.x86_64          19/82
2025-12-02T05:38:00.987 INFO:teuthology.orchestra.run.smithi001.stdout:  Installing       : ceph-osd-classic-2:20.3.0-4253.g9422a98d.el9.x86_6   20/82
2025-12-02T05:38:01.010 INFO:teuthology.orchestra.run.smithi001.stdout:  Running scriptlet: ceph-osd-classic-2:20.3.0-4253.g9422a98d.el9.x86_6   20/82
2025-12-02T05:38:01.010 INFO:teuthology.orchestra.run.smithi001.stdout:failed to link /usr/bin/ceph-osd -> /etc/alternatives/ceph-osd: /usr/bin/ceph-osd exists and it is not a symlink

/usr/bin/ceph-osd is from the squid installation.

Copy link
Contributor

@Matan-B Matan-B left a comment

Choose a reason for hiding this comment

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

Thanks for fixing!
Note: Similarly to #66372 and #66268.

@vshankar
Copy link
Contributor Author

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

@vshankar
Copy link
Contributor Author

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

@vshankar
Copy link
Contributor Author

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

@vshankar
Copy link
Contributor Author

@vshankar vshankar merged commit e5e841c into ceph:main Feb 23, 2026
13 of 14 checks passed
vshankar added a commit to vshankar/ceph that referenced this pull request Feb 25, 2026
* refs/pull/66475/head:

Reviewed-by: Matan Breizman <mbreizma@redhat.com>
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