Skip to content

Conversation

sd109
Copy link
Member

@sd109 sd109 commented Dec 8, 2022

No description provided.

@sd109 sd109 requested a review from markgoddard December 8, 2022 17:10
@sd109 sd109 requested a review from a team as a code owner December 8, 2022 17:10
@mnasiadka
Copy link
Member

Why before? cephadm shell won't work before deploy...

@markgoddard
Copy link
Contributor

Why before? cephadm shell won't work before deploy...

Yeah, should go just after cephadm-deploy.yml

@sd109
Copy link
Member Author

sd109 commented Dec 9, 2022

Well spotted, thanks guys. My fault for trying to cut corners...

@markgoddard markgoddard merged commit 065a773 into stackhpc/xena Dec 12, 2022
@markgoddard markgoddard deleted the extra-cephadm-commands branch December 12, 2022 10:36
jovial added a commit that referenced this pull request Jan 9, 2023
Support for this feature was added in #290. This adds a variable
and some documentation to raise awareness of its existence.
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.

3 participants