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

install control-plane on GCP outputs warning to stdout #5554

Closed
lahabana opened this issue Dec 23, 2022 · 1 comment · Fixed by #5590
Closed

install control-plane on GCP outputs warning to stdout #5554

lahabana opened this issue Dec 23, 2022 · 1 comment · Fixed by #5590
Assignees
Labels
area/kumactl kind/feature New feature triage/accepted The issue was reviewed and is complete enough to start working on it

Comments

@lahabana
Copy link
Contributor

Description

With GCP logging warning when doing kumactl install control-plane to stdout, you can therefore not pipe to kubectl.

@lahabana lahabana added area/kumactl triage/accepted The issue was reviewed and is complete enough to start working on it kind/feature New feature labels Dec 23, 2022
@lahabana
Copy link
Contributor Author

lahabana commented Jan 6, 2023

Looked at this and it seems that the log message comes from K8s's klog. Trying to find ways to ensure klog goes to stderr

lahabana added a commit to lahabana/kuma that referenced this issue Jan 6, 2023
Logs from klog would go to stdout which was ruining the output
of commands like install-control-plane

Fix kumahq#5554

Signed-off-by: Charly Molter <[email protected]>
@lahabana lahabana self-assigned this Jan 6, 2023
lahabana added a commit that referenced this issue Jan 6, 2023
Logs from klog would go to stdout which was ruining the output
of commands like install-control-plane

Fix #5554

Signed-off-by: Charly Molter <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/kumactl kind/feature New feature triage/accepted The issue was reviewed and is complete enough to start working on it
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant