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

bgpv2/ci: added watch reactor for bgp cluster config #31381

Merged

Conversation

harsimran-pabla
Copy link
Contributor

Wait for BGPClusterConfig and CiliumNodes watcher to initialize before starting the tests.

Fixes: #31054, #31363

Signed-off-by: harsimran pabla <hpabla@isovalent.com>
@harsimran-pabla harsimran-pabla added the release-note/ci This PR makes changes to the CI. label Mar 13, 2024
@harsimran-pabla harsimran-pabla requested a review from a team as a code owner March 13, 2024 15:59
@harsimran-pabla
Copy link
Contributor Author

/test

Copy link
Contributor

@rastislavs rastislavs left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@harsimran-pabla
Copy link
Contributor Author

AWS-CNI test failed to create cluster.

Error: failed to create cluster "cilium-cilium-8268424990-1"
2024-03-13 17:07:47 [!]  1 error(s) occurred and cluster hasn't been created properly, you may wish to check CloudFormation console
2024-03-13 17:07:47 [ℹ]  to cleanup resources, run 'eksctl delete cluster --region=us-east-2 --name=cilium-cilium-8268424990-1'
2024-03-13 17:07:47 [✖]  error creating Clientset: getting list of API resources for raw REST client: the server has asked for the client to provide credentials

@harsimran-pabla
Copy link
Contributor Author

ci-ginkgo failure : #31147

@harsimran-pabla
Copy link
Contributor Author

IPsec tests failing because of quay timeouts

Error: cannot pull image quay.io/lvh-images/kind:5.10-20240305.092417: Error response from daemon: Get "https://quay.io/v2/": net/http: request canceled (Client.Timeout exceeded while awaiting headers)

@harsimran-pabla
Copy link
Contributor Author

Another IPSec test failure (client-egress-l7) :
#27762

[.] Action [client-egress-l7/pod-to-pod/curl-ipv4-2: cilium-test/client2-ccd7b8bdf-dzqvc (10.244.1.200) -> cilium-test/echo-other-node-5d67f9786b-lfjb6 (10.244.2.123:8080)]
  ❌ command "curl -w %{local_ip}:%{local_port} -> %{remote_ip}:%{remote_port} = %{response_code} --silent --fail --show-error --output /dev/null --connect-timeout 2 --max-time 10 http://10.244.2.123:8080/" failed: error with exec request (pod=cilium-test/client2-ccd7b8bdf-dzqvc, container=client2): command terminated with exit code 28

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Mar 14, 2024
@aditighag
Copy link
Member

Merging the fix based on @harsimran-pabla's confirmation that it's ready.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Operator bgpv2 Test_NodeLabels is flaky
4 participants