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

Increase logging verbosity of Kubernetes API Server in kind #24384

Merged
merged 1 commit into from Mar 16, 2023

Conversation

marseel
Copy link
Contributor

@marseel marseel commented Mar 15, 2023

This will allow us to see what requests are made by Cilium components against k8s apiserver.

Example:

kubectl logs -n kube-system kube-apiserver-kind-control-plane

This will allow us to see what requests are made by Cilium components.

Signed-off-by: Marcel Zieba <marcel.zieba@isovalent.com>
@marseel marseel added the release-note/misc This PR makes changes that have no direct user impact. label Mar 15, 2023
@marseel marseel requested a review from a team as a code owner March 15, 2023 11:11
@marseel marseel requested a review from bimmlerd March 15, 2023 11:11
@marseel
Copy link
Contributor Author

marseel commented Mar 15, 2023

/test

Job 'Cilium-PR-K8s-1.25-kernel-4.19' failed:

Click to show.

Test Name

K8sAgentPolicyTest Multi-node policy test with L7 policy using connectivity-check to check datapath

Failure Output

FAIL: cannot install connectivity-check

If it is a flake and a GitHub issue doesn't already exist to track it, comment /mlh new-flake Cilium-PR-K8s-1.25-kernel-4.19 so I can create one.

Copy link
Member

@bimmlerd bimmlerd left a comment

Choose a reason for hiding this comment

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

nice

@marseel
Copy link
Contributor Author

marseel commented Mar 16, 2023

/test

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Mar 16, 2023
@aditighag aditighag merged commit 3eaa764 into cilium:master Mar 16, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants