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.

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

feature: enhance K8s cluster operational and observable capabilities #7

Merged
merged 30 commits into from
Oct 28, 2024

Conversation

bearslyricattack
Copy link

  • I have registered the PR changes.

Ⅰ. Describe what this PR did

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

@ptyin ptyin changed the title New feature: Enhance the operational and observable capabilities of the tool. feat: enhance K8s cluster operational and observable capabilities Oct 11, 2024
@ptyin ptyin changed the title feat: enhance K8s cluster operational and observable capabilities feature: enhance K8s cluster operational and observable capabilities Oct 11, 2024
@ptyin ptyin self-assigned this Oct 21, 2024
action/common/parser.go Outdated Show resolved Hide resolved
go 2.sum Outdated Show resolved Hide resolved
seata.json Outdated Show resolved Hide resolved
qodana.yaml Outdated Show resolved Hide resolved
cmd/root.go Outdated Show resolved Hide resolved
for key, value := range doc {
fmt.Printf(" %s: %v\n", key, value)
}
fmt.Println("----------------------------")
Copy link
Member

Choose a reason for hiding this comment

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

Logging should be more formal.

action/log/log.go Outdated Show resolved Hide resolved
action/log/log.go Outdated Show resolved Hide resolved
action/k8s/status.go Show resolved Hide resolved
action/k8s/install.go Outdated Show resolved Hide resolved
@ptyin
Copy link
Member

ptyin commented Oct 23, 2024

@bearslyricattack Please supplement usage in README.md

@ptyin
Copy link
Member

ptyin commented Oct 23, 2024

@bearslyricattack CI failed, plz check

@ptyin ptyin changed the base branch from main to summer-code_ctl October 28, 2024 12:55
Copy link

sonarcloud bot commented Oct 28, 2024

@ptyin ptyin merged commit ca2aad4 into apache:summer-code_ctl Oct 28, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants