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

log: fix "unsupported value" error in a log line #309

Open
huntergregory opened this issue Apr 24, 2024 · 2 comments
Open

log: fix "unsupported value" error in a log line #309

huntergregory opened this issue Apr 24, 2024 · 2 comments
Labels
area/plugins good first issue Good for newcomers help wanted Extra attention is needed lang/go The Go Programming Language priority/2 P2 type/bug Something isn't working

Comments

@huntergregory
Copy link
Contributor

huntergregory commented Apr 24, 2024

zap.Any() does not support the field passed at packetparser/packetparser_linux.go:217

We should find another way to log the eth0Link:

level=info caller=packetparser/packetparser_linux.go:217 msg="Attaching Packetparser to eth0" goversion=go1.21.9 os=linux arch=amd64 numcores=4 hostname=aks-nodepool1-17703939-vmss000000 podname=retina-agent-cqnxv version=b8538dc apiserver=https://vakr-04-16-vakr-04-16-9b8218-ezit2obj.hcp.westus2.azmk8s.io:443 plugins=linuxutil,packetforward,packetparser,dns,dropreason eth0Link= eth0LinkError="unsupported value type"
@huntergregory huntergregory added the good first issue Good for newcomers label Apr 24, 2024
@rbtr rbtr added type/bug Something isn't working help wanted Extra attention is needed lang/go The Go Programming Language area/plugins priority/2 P2 labels Apr 25, 2024
@roopeshsn
Copy link

zap.Any() does not support the field passed at packetparser/packetparser_linux.go:217

We should find another way to log the eth0Link:

level=info caller=packetparser/packetparser_linux.go:217 msg="Attaching Packetparser to eth0" goversion=go1.21.9 os=linux arch=amd64 numcores=4 hostname=aks-nodepool1-17703939-vmss000000 podname=retina-agent-cqnxv version=b8538dc apiserver=https://vakr-04-16-vakr-04-16-9b8218-ezit2obj.hcp.westus2.azmk8s.io:443 plugins=linuxutil,packetforward,packetparser,dns,dropreason eth0Link= eth0LinkError="unsupported value type"

It's a netlink.Link interface right?

@timraymond
Copy link
Member

@huntergregory is this still an issue? It seems like the offending log line has been changed to remove zap.Any entirely...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/plugins good first issue Good for newcomers help wanted Extra attention is needed lang/go The Go Programming Language priority/2 P2 type/bug Something isn't working
Projects
Status: Accepted
Development

No branches or pull requests

4 participants