Skip to content
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

OCM-5131 | fix: don't show additional compute SGs describing cluster #584

Merged
merged 1 commit into from
Dec 11, 2023
Merged

OCM-5131 | fix: don't show additional compute SGs describing cluster #584

merged 1 commit into from
Dec 11, 2023

Conversation

gdbranco
Copy link
Collaborator

@gdbranco gdbranco commented Dec 8, 2023 •
edited
Loading

Related issue: OCM-5131

@renan-campos
Copy link
Collaborator

Could you provide some example output?

@renan-campos
Copy link
Collaborator

Got it:

[rcampos@rcampos-thinkpadt14sgen2i ocm-cli]$ ./ocm describe cluster 280ijfruedtlbufo0u84vjdamdnif311

ID:			<REDACTED>
External ID:		
Name:			rc-test
Display Name:		rc-test
State:			error 
Details:		<REDACTED>
API URL:		
API Listening:		external
Console URL:		
Control Plane:
	Replicas: 3
Infra:
	Replicas: 2
Compute:
	Replicas: 4
...

@renan-campos
Copy link
Collaborator

/lgtm

vkareh
vkareh approved these changes Dec 11, 2023
View reviewed changes
Copy link
Member

@vkareh vkareh left a comment

Choose a reason for hiding this comment

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

/lgtm

@gdbranco gdbranco merged commit 9d144fd into openshift-online:main Dec 11, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vkareh vkareh vkareh 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.
3 participants
@gdbranco @renan-campos @vkareh