Skip to content

ci: Add check for cilium error logs to cilium nightly pipeline #3592

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

jpayne3506
Copy link
Contributor

Reason for Change:

Adds an additional level of alerting for breaking changes occurring in upstream cilium.

Issue Fixed:

Requirements:

Notes:

@jpayne3506 jpayne3506 added the ci Infra or tooling. label Apr 14, 2025
@jpayne3506 jpayne3506 self-assigned this Apr 14, 2025
@Copilot Copilot AI review requested due to automatic review settings April 14, 2025 17:43
@jpayne3506 jpayne3506 requested a review from a team as a code owner April 14, 2025 17:43
@jpayne3506 jpayne3506 requested a review from nddq April 14, 2025 17:43
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

@jpayne3506 jpayne3506 force-pushed the jpayne3506/nightly-error branch from 0069d78 to 411ce18 Compare April 14, 2025 17:48
@vipul-21
Copy link
Contributor

If we want to test the cilium logs for error, we can use the connectivity test for that. They have check log error and specify the log-check-levels as error so that it does not fail on warning.

@jpayne3506 jpayne3506 force-pushed the jpayne3506/nightly-error branch from 411ce18 to 0ca8d8a Compare April 14, 2025 21:25
@jpayne3506
Copy link
Contributor Author

https://github.com/cilium/cilium/pull/36231/files , got it. I think this only checks agent logs, so for now we can still leverage this for operator.

@jpayne3506 jpayne3506 force-pushed the jpayne3506/nightly-error branch from 3cac522 to 1e977db Compare April 15, 2025 18:22
@jpayne3506 jpayne3506 force-pushed the jpayne3506/nightly-error branch from 1e977db to 595267e Compare April 17, 2025 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Infra or tooling.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants