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.

Sign up for GitHub

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

Jump to bottom

v1.11 backports 2023-04-26 #25139

Merged
merged 2 commits into from Apr 28, 2023
Merged

v1.11 backports 2023-04-26 #25139

merged 2 commits into from Apr 28, 2023

Conversation

sayboras
Copy link
Member

@sayboras sayboras commented Apr 26, 2023 •
edited by harsimran-pabla

Once this PR is merged, you can update the PR labels via:

$ for pr in 25043 25046; do contrib/backporting/set-labels.py $pr done 1.11; done

@harsimran-pabla @sayboras
bgp: do not advertise ipv6 prefixes via metallb
d03bd01 
[ upstream commit 922aa1b ]

When using metallb as BGP speaker, if IPv6 advertisement is made - metallb will return error as unsupported.
This error is logged and error is returned to control loop, which continues retrying causing log flooding and high CPU.
This change filters out IPv6 prefixes before sending them to metallb library and logs one time error message.

Signed-off-by: harsimran pabla <hpabla@isovalent.com>
Signed-off-by: Tam Mach <tam.mach@cilium.io>
@sayboras sayboras requested a review from a team as a code owner April 26, 2023 11:50
@sayboras sayboras added kind/backports This PR provides functionality previously merged into master. backport/1.11 This PR represents a backport for Cilium 1.11.x of a PR that was merged to main. labels Apr 26, 2023
@nbusseneau @sayboras
ci: remove STATUS commands from upstream tests' Jenkinsfile
a51d91a 
[ upstream commit 46de5bc ]

These are remnants of a past before GHPRB. At best they create
uncessary noise in the logs, at worst they can interfere with the
default behaviour, so let's just remove them.

Signed-off-by: Nicolas Busseneau <nicolas@isovalent.com>
Signed-off-by: Tam Mach <tam.mach@cilium.io>
@sayboras
Copy link
Member Author

sayboras commented Apr 26, 2023 •
edited by maintainer-s-little-helper bot

/test-backport-1.11

Job 'Cilium-PR-Runtime-net-next' failed:

Click to show.

Test Name

RuntimePrivilegedUnitTests Run Tests

Failure Output

FAIL: Failed to run privileged unit tests

Jenkins URL: https://jenkins.cilium.io/job/Cilium-PR-Runtime-net-next/6311/

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

Then please upload the Jenkins artifacts to that issue.

Job 'Cilium-PR-K8s-1.19-kernel-4.9' hit: #22555 (92.92% similarity)

Job 'Cilium-PR-K8s-1.16-net-next' failed:

Click to show.

Test Name

K8sDatapathConfig Host firewall With VXLAN

Failure Output

FAIL: Failed to reach 192.168.56.11:80 from testclient-9mkh2

Jenkins URL: https://jenkins.cilium.io/job/Cilium-PR-K8s-1.16-net-next/2662/

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

Then please upload the Jenkins artifacts to that issue.

Job 'Cilium-PR-K8s-1.19-kernel-4.9' failed:

Click to show.

Test Name

K8sKafkaPolicyTest Kafka Policy Tests KafkaPolicies

Failure Output

FAIL: Found 2 k8s-app=cilium logs matching list of errors that must be investigated:

Jenkins URL: https://jenkins.cilium.io/job/Cilium-PR-K8s-1.19-kernel-4.9/2835/

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

Then please upload the Jenkins artifacts to that issue.

Job 'Cilium-PR-K8s-1.16-net-next' failed:

Click to show.

Test Name

K8sPolicyTest Clusterwide policies Test clusterwide connectivity with policies

Failure Output

FAIL: Test suite timed out after 2h50m0s

Jenkins URL: https://jenkins.cilium.io/job/Cilium-PR-K8s-1.16-net-next/2663/

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

Then please upload the Jenkins artifacts to that issue.

@sayboras
Copy link
Member Author

/test-1.16-netnext

@sayboras
Copy link
Member Author

/test-1.20-4.9

@sayboras
Copy link
Member Author

/test-1.21-5.4

@sayboras
Copy link
Member Author

/test-1.23-4.9

@sayboras
Copy link
Member Author

sayboras commented Apr 26, 2023 •
edited

Above re-trigger due to recent github issue, which fails checkout steps in Jenkins.

@maintainer-s-little-helper maintainer-s-little-helper bot mentioned this pull request Apr 26, 2023
Closed
@sayboras
Copy link
Member Author

/test-runtime

@sayboras
Copy link
Member Author

/test-1.19-4.9

@sayboras
Copy link
Member Author

/test-1.16-netnext

@sayboras
Copy link
Member Author

/test-1.17-4.9

@sayboras
Copy link
Member Author

Test Summary

Reviews are in, and all other required CI jobs are green. Marking this ready to merge.

@sayboras sayboras added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Apr 27, 2023
@pchaigno pchaigno merged commit e026bf6 into cilium:v1.11 Apr 28, 2023
49 of 52 checks passed
@sayboras sayboras deleted the pr/v1.11-backport-2023-04-26 branch April 29, 2023 00:50
This was referenced May 3, 2023
Closed
Merged
This was referenced May 12, 2023
Closed
Closed
Closed
@jrajahalme jrajahalme mentioned this pull request May 15, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nbusseneau nbusseneau nbusseneau approved these changes

@michi-covalent michi-covalent michi-covalent approved these changes

@harsimran-pabla harsimran-pabla harsimran-pabla approved these changes

Assignees
No one assigned
Labels
backport/1.11 This PR represents a backport for Cilium 1.11.x of a PR that was merged to main. kind/backports This PR provides functionality previously merged into master. ready-to-merge This PR has passed all tests and received consensus from code owners to merge.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@sayboras @nbusseneau @michi-covalent @harsimran-pabla @pchaigno