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

Failed to watch *v1alpha1.CNINode log messages #2574

Closed
tulanian opened this issue Sep 18, 2023 · 4 comments
Closed

Failed to watch *v1alpha1.CNINode log messages #2574

tulanian opened this issue Sep 18, 2023 · 4 comments
Labels

Comments

@tulanian
Copy link

What happened:
Upgraded to v1.15.0-eksbuild.2 using Addon Update. Pods are up but now seeing these logs:

023-09-18T10:08:00.724060030Z E0918 10:08:00.723932      10 reflector.go:148] pkg/mod/k8s.io/client-go@v0.27.3/tools/cache/reflector.go:231: Failed to watch *v1alpha1.CNINode: unknown (get cninodes.vpcresources.k8s.aws)
2023-09-18T10:08:33.576510894Z E0918 10:08:33.576407      10 reflector.go:148] pkg/mod/k8s.io/client-go@v0.27.3/tools/cache/reflector.go:231: Failed to watch *v1alpha1.CNINode: unknown (get cninodes.vpcresources.k8s.aws)

Attach logs

What you expected to happen:
Not to see these log messages.

How to reproduce it (as minimally and precisely as possible):
Upgrade to v1.15.0-eksbuild.2 via the Addon Update mechanism.

Anything else we need to know?:

Environment:

  • Kubernetes version (use kubectl version): 1.26
  • CNI Version: v1.15.0-eksbuild.2
  • OS (e.g: cat /etc/os-release): Amazon Linux 2
  • Kernel (e.g. uname -a): 5.10.186-179.751.amzn2.x86_64
@tulanian tulanian added the bug label Sep 18, 2023
@tulanian
Copy link
Author

@tulanian tulanian changed the title Failed to watch *v1alpha1.CNINode Failed to watch *v1alpha1.CNINode log messages Sep 18, 2023
@jdn5126
Copy link
Contributor

jdn5126 commented Sep 18, 2023

@tulanian this log message is harmless and was first reported via #2567. The TL;DR is that the Kubernetes client wants to cache CNINode objects, but we do not need or want them to be cached.

The long-term fix, which will ship in the next v1.15.x release, is #2570. If you want to stop the log, you can add watch permissions as #2567 did, but note that this will slightly increase the amount of memory consumed by the aws-node pod.

@jdn5126
Copy link
Contributor

jdn5126 commented Sep 22, 2023

Closing as this merged in #2567 and will ship in v1.15.1

@jdn5126 jdn5126 closed this as completed Sep 22, 2023
@github-actions
Copy link

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants