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

ci: cilium-nightly skip test check-log-errors #2763

Merged
merged 1 commit into from
Jun 5, 2024
Merged

Conversation

camrynl
Copy link
Contributor

@camrynl camrynl commented Jun 4, 2024

Reason for Change:
New test case in latest cilium cil checks for log errors and is failing nightly pipeline. Found that the error reported in the cilium-agent logs recovers on its own, but still causes a failure in this test case. Skipping this check to unblock the pipeline, and will follow up with cilium-cli on this.

time=2024-05-30T22:07:56Z level=info msg="Devices changed" module=agent.datapath.devices-controller devices=[]
time="2024-05-30T22:07:57Z" level=info msg="Node addresses updated" device=eth0 node-addresses= subsys=node-address
**time="2024-05-30T22:07:57Z" level=error msg="Unable to write node config header" error="failed to write node configuration file at /var/run/cilium/state/globals/node_config.h: IPv4 direct routing device not found" subsys=datapath-loader**
time="2024-05-30T22:07:57Z" level=warning msg="Failed to reload datapath" error="unable to recompile base programs from devices changed: failed to write node configuration file at /var/run/cilium/state/globals/node_config.h: IPv4 direct routing device not found" subsys=daemon
**time=2024-05-30T22:07:57Z level=info msg="Devices changed" module=agent.datapath.devices-controller devices=[eth0]
time="2024-05-30T22:07:57Z" level=info msg="Node addresses updated" device=eth0 node-addresses="10.10.0.5 (primary, nodeport)" subsys=node-address**
time="2024-05-30T22:07:58Z" level=info msg="Updated link /sys/fs/bpf/cilium/socketlb/links/cgroup/cil_sock6_post_bind for program cil_sock6_post_bind" subsys=socketlb
time="2024-05-30T22:07:58Z" level=info msg="Updated link /sys/fs/bpf/cilium/socketlb/links/cgroup/cil_sock6_connect for program cil_sock6_connect" subsys=socketlb
time="2024-05-30T22:07:58Z" level=info msg="Updated link /sys/fs/bpf/cilium/socketlb/links/cgroup/cil_sock6_sendmsg for program cil_sock6_sendmsg" subsys=socketlb

Issue Fixed:

Requirements:

Notes:

@camrynl camrynl requested a review from a team as a code owner June 4, 2024 00:00
@camrynl camrynl requested a review from peterfelts June 4, 2024 00:00
@camrynl camrynl enabled auto-merge June 4, 2024 00:01
Copy link
Contributor

@jpayne3506 jpayne3506 left a comment

Choose a reason for hiding this comment

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

Approved, pending a 🟢 on nightly pipeline

@camrynl
Copy link
Contributor Author

camrynl commented Jun 5, 2024

/azp run Azure Container Networking PR

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@camrynl camrynl added this pull request to the merge queue Jun 5, 2024
Merged via the queue into master with commit 3e10379 Jun 5, 2024
15 of 18 checks passed
@camrynl camrynl deleted the camrynl/skiplogcheck branch June 5, 2024 21:43
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