Skip to content

fix: openstack command in hyperconverged-lab.sh need -c id instead … #892

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Mar 11, 2025
Merged

fix: openstack command in hyperconverged-lab.sh need -c id instead … #892

merged 1 commit into from
Mar 11, 2025

Conversation

@awfabian-rs
Copy link
Contributor

@awfabian-rs awfabian-rs commented Mar 11, 2025

…of -C ID

@awfabian-rs
Copy link
Contributor Author

awfabian-rs commented Mar 11, 2025 •
edited
Loading

I generally saw unknown field ID with -c ID from the OpenStack command and it seemed fine when I replaced it with -c id

@awfabian-rs awfabian-rs requested a review from cloudnull March 11, 2025 22:49
@awfabian-rs awfabian-rs force-pushed the hyperconverged-id-case branch from f1819ba to 893eb2c Compare March 11, 2025 22:50
@awfabian-rs awfabian-rs marked this pull request as draft March 11, 2025 22:52
@awfabian-rs awfabian-rs force-pushed the hyperconverged-id-case branch from 893eb2c to ff09146 Compare March 11, 2025 22:52
@awfabian-rs awfabian-rs marked this pull request as ready for review March 11, 2025 22:53
@cloudnull
Copy link
Contributor

cloudnull commented Mar 11, 2025

image
both "id" and "ID" work for me, I wonder if this is related to the client version? That said, this does work, so +1 from me.

@cloudnull cloudnull merged commit 504e4cc into rackerlabs:main Mar 11, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cloudnull cloudnull cloudnull approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@awfabian-rs @cloudnull