-
Notifications
You must be signed in to change notification settings - Fork 92
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Tentacle kickoff! #2315
base: main
Are you sure you want to change the base?
Tentacle kickoff! #2315
Conversation
See ceph/ceph-build#2315 Signed-off-by: Patrick Donnelly <[email protected]>
ceph/ceph-build#2315 Signed-off-by: Patrick Donnelly <[email protected]>
See ceph/ceph-build#2315 Signed-off-by: Patrick Donnelly <[email protected]>
See ceph/ceph-build#2315 Signed-off-by: Patrick Donnelly <[email protected]>
Not a review, just a note that chacra will have to be updated before CI builds will work |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FWIW LGTM.
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. |
See ceph/ceph-build#2315 Signed-off-by: Patrick Donnelly <[email protected]>
See ceph/ceph-build#2315 Signed-off-by: Patrick Donnelly <[email protected]>
Signed-off-by: Patrick Donnelly <[email protected]>
Signed-off-by: Patrick Donnelly <[email protected]>
3a593e3
to
af8ebc2
Compare
@dmick do we need to delay merging this until ceph/ceph#61321 is merged? |
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. |
See: ceph/ceph#61321