Skip to content

Feat: support run kubectl on AKS by runcommand#286

Merged
Fei-Guo merged 4 commits intomainfrom
guwe/run-cmd-header
Jan 23, 2026
Merged

Feat: support run kubectl on AKS by runcommand#286
Fei-Guo merged 4 commits intomainfrom
guwe/run-cmd-header

Conversation

@gossion
Copy link
Member

@gossion gossion commented Jan 21, 2026

No description provided.

@gossion gossion force-pushed the guwe/run-cmd-header branch 2 times, most recently from f78dfa2 to cfb9af1 Compare January 21, 2026 07:37
"Comma-separated list of allowed Kubernetes namespaces (empty means all namespaces)")

// Multi-cluster configuration
flag.BoolVar(&cfg.EnableMultiCluster, "enable-multi-cluster", false,
Copy link
Member Author

Choose a reason for hiding this comment

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

is this flag good, or do we have better flag?

Copy link
Member Author

Choose a reason for hiding this comment

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

@Fei-Guo got a good idea to call it '--token-auth-only', thanks.

@gossion gossion marked this pull request as ready for review January 22, 2026 05:50
@gossion
Copy link
Member Author

gossion commented Jan 23, 2026

don't merge before HolmesGPT/holmesgpt#1397 is merged. maybe there is some change needed.

@gossion gossion force-pushed the guwe/run-cmd-header branch from d12188e to 73ff6c5 Compare January 23, 2026 06:46
@Fei-Guo Fei-Guo added this pull request to the merge queue Jan 23, 2026
Merged via the queue into main with commit 3ccd14f Jan 23, 2026
9 checks passed
@Fei-Guo Fei-Guo deleted the guwe/run-cmd-header branch January 23, 2026 21:56
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