Skip to content

Tentacle kickoff!#2315

Merged
rzarzynski merged 2 commits intoceph:mainfrom
batrick:tentacle-kickoff
Mar 31, 2025
Merged

Tentacle kickoff!#2315
rzarzynski merged 2 commits intoceph:mainfrom
batrick:tentacle-kickoff

Conversation

@batrick
Copy link
Member

@batrick batrick commented Jan 10, 2025

@batrick batrick mentioned this pull request Jan 10, 2025
14 tasks
batrick added a commit to batrick/ceph that referenced this pull request Jan 10, 2025
See ceph/ceph-build#2315

Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
@batrick batrick changed the title scripts: add tentacle Tentacle kickoff! Jan 10, 2025
@batrick batrick marked this pull request as draft January 10, 2025 16:42
batrick added a commit to batrick/ceph that referenced this pull request Jan 10, 2025
ceph/ceph-build#2315

Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
batrick added a commit to batrick/ceph that referenced this pull request Jan 10, 2025
See ceph/ceph-build#2315

Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
@batrick batrick marked this pull request as ready for review January 10, 2025 17:57
batrick added a commit to batrick/ceph that referenced this pull request Jan 10, 2025
See ceph/ceph-build#2315

Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
@dmick
Copy link
Member

dmick commented Jan 15, 2025

Not a review, just a note that chacra will have to be updated before CI builds will work

Copy link
Contributor

@rzarzynski rzarzynski left a comment

Choose a reason for hiding this comment

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

FWIW LGTM.

@dmick
Copy link
Member

dmick commented Feb 6, 2025

Did some investigation into chacra. We have some technical debt to resolve there to allow deploying on modern releases; it would probably be good to do that along with the update for tentacle at the same time. I don't know if we have any Python developers with any relevant expertise at this point; I can probably personally muddle along, but it's not scoped yet.

batrick added a commit to batrick/ceph that referenced this pull request Feb 6, 2025
See ceph/ceph-build#2315

Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
batrick added a commit to batrick/ceph that referenced this pull request Feb 6, 2025
See ceph/ceph-build#2315

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

batrick commented Feb 6, 2025

@dmick do we need to delay merging this until ceph/ceph#61321 is merged?

@batrick batrick closed this Feb 6, 2025
@batrick batrick reopened this Feb 6, 2025
@dmick
Copy link
Member

dmick commented Feb 7, 2025

not sure. I haven't tried to consider interdependencies and would have to spend some time on it. I hope you're hearing me about chacra tho.

@batrick
Copy link
Member Author

batrick commented Feb 7, 2025

not sure. I haven't tried to consider interdependencies and would have to spend some time on it. I hope you're hearing me about chacra tho.

I hear you but I'm not sure what needs done from your comment.

batrick added a commit to batrick/ceph that referenced this pull request Feb 17, 2025
See ceph/ceph-build#2315

Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
rzarzynski pushed a commit to rzarzynski/ceph that referenced this pull request Feb 21, 2025
See ceph/ceph-build#2315

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

batrick commented Mar 3, 2025

ceph/ceph#61321 is merged.

@rzarzynski
Copy link
Contributor

@dmick, @batrick: fine to to rebase and merge?

@batrick
Copy link
Member Author

batrick commented Mar 10, 2025

@dmick, @batrick: fine to to rebase and merge?

I'm waiting for @dmick to decide next steps before rebasing again.

@rzarzynski
Copy link
Contributor

@dmick: what's the step on this?

@dmick
Copy link
Member

dmick commented Mar 17, 2025

Not sure what Patrick is referring to, but one thing I know that needs to be done, still, is to update the chacra configuration to allow for tentacle as a release name

@batrick
Copy link
Member Author

batrick commented Mar 18, 2025

Not sure what Patrick is referring to, but one thing I know that needs to be done, still, is to update the chacra configuration to allow for tentacle as a release name

I was referring to: #2315 (comment)

Should I rebase and then we can merge?

@cbodley
Copy link
Contributor

cbodley commented Mar 24, 2025

just noting that windows builds on main are all failing with ERROR: Unknown release for version '20.0.0' until this merges

@rzarzynski
Copy link
Contributor

It's also reflected by https://tracker.ceph.com/issues/70215.
@batrick, @cbodley: are we ready to merge after the rebase?

batrick added 2 commits March 24, 2025 14:20
Signed-off-by: Patrick Donnelly <batrick@batbytes.com>
Signed-off-by: Patrick Donnelly <batrick@batbytes.com>
@batrick
Copy link
Member Author

batrick commented Mar 24, 2025

It's also reflected by https://tracker.ceph.com/issues/70215. @batrick, @cbodley: are we ready to merge after the rebase?

I guess? At least let's just merge and rip the band-aid off. Fix whatever gets broken.

@rzarzynski rzarzynski merged commit 7693d90 into ceph:main Mar 31, 2025
1 check failed
@dmick
Copy link
Member

dmick commented Mar 31, 2025

I performed the configuration steps at https://docs.ceph.com/en/latest/dev/release-process/#new-major-releases (built the ceph-release package for tentacle with the jenkins job, and updated chacra to include tentacle as a known release, albeit manually because I didn't want to get caught up in ansible/OS upgrade issues). I'll put the change back to the chacra ansible anyway and it's an ongoing task to update the chacra hosts.

I also believe that having the separate version of ceph-release automagically included in the package repo is the wrong way to solve this particular problem and will be pondering a better less-magical solution.

@batrick batrick deleted the tentacle-kickoff branch April 28, 2025 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants