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

Add operator cluster commands #460

Merged
merged 2 commits into from
Feb 15, 2024
Merged

Add operator cluster commands #460

merged 2 commits into from
Feb 15, 2024

Conversation

Quinn-With-Two-Ns
Copy link
Contributor

What was changed

Add some operator cluster commands

Why?

Parity with old CLI

  1. How was this tested:
    Integration tests added for new commands

Doesn't fully implement all operator cluster commands, since those are much more painful to test so I pulled out the simple ones first.

@Quinn-With-Two-Ns
Add operator cluster commands
290b732 
Add operator cluster describe, health, and system.
cretz
cretz reviewed Feb 15, 2024
View reviewed changes
temporalcli/commands.operator.go Outdated Show resolved Hide resolved
temporalcli/commands.operator.go Outdated Show resolved Hide resolved
temporalcli/commands.operator.go Outdated Show resolved Hide resolved
@Quinn-With-Two-Ns Quinn-With-Two-Ns merged commit 8d8ac3b into cli-rewrite Feb 15, 2024
5 checks passed
@Quinn-With-Two-Ns Quinn-With-Two-Ns deleted the cli-rewrite-operator-cluster-init branch February 15, 2024 20:40
@josh-berry josh-berry mentioned this pull request Apr 29, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cretz cretz cretz 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
@Quinn-With-Two-Ns @cretz