Skip to content

Conversation

@manusa
Copy link
Member

@manusa manusa commented Nov 28, 2025

Follows up on #502 to better organize the code and improve its maintenance and sustainability.

Changes:

@manusa manusa added this to the 0.1.0 milestone Nov 28, 2025
@manusa
Copy link
Member Author

manusa commented Nov 28, 2025

For f54d30e

IMO we should actually move the SIGHUP logic to the pkg/kubernetes-mcp-server/cmd package where it makes more sense.

@nader-ziada
Copy link
Collaborator

i like the moving of watcher logic in a separate package, much cleaner. Also the move of open shift check. 👍

/lgtm

@nader-ziada
Copy link
Collaborator

thanks for the refactor @manusa, i will rebase the drop-in config and make the suggested changes

/lgtm
/approve

@nader-ziada nader-ziada merged commit 69ed78e into containers:main Nov 28, 2025
6 checks passed
@manusa manusa deleted the kubernetes/watch branch November 28, 2025 14:51
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.

2 participants