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

Fix CI job KinD network error #167

Closed
wants to merge 1 commit into from

Conversation

yxun
Copy link

@yxun yxun commented Nov 14, 2024

This PR is following openshift-service-mesh/istio#135 and fixing the CI job error " failed to ensure docker network: command"

Signed-off-by: Yuanlin Xu <yuanlin.xu@redhat.com>
@openshift-ci openshift-ci bot added the size/XS label Nov 14, 2024
@FilipB
Copy link

FilipB commented Nov 15, 2024

/retest

Copy link

openshift-ci bot commented Nov 15, 2024

@yxun: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-ocp-4-17 ff5578f link false /test e2e-ocp-4-17
ci/prow/e2e-ocp-4-16-arm ff5578f link true /test e2e-ocp-4-16-arm

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

Comment on lines +168 to +173
# Create the docker network beforehand so "kind create cluster" doesn't create it with ipv6
echo "Deleting previous KinD docker network"
docker network rm kind || true
echo "Creating the docker network for kind"
docker network create -d=bridge -o com.docker.network.bridge.enable_ip_masquerade=true kind

Copy link

Choose a reason for hiding this comment

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

This change will be overridden when the update-common make target is run. Because we copy the entire file from the upstream repo: https://github.com/istio/common-files. So, make sense to push this change to the upstream repository and then update using the make target?

@@ -165,6 +165,12 @@ function setup_kind_cluster() {
echo "No existing kind cluster with name ${NAME}. Continue..."
fi

# Create the docker network beforehand so "kind create cluster" doesn't create it with ipv6

Choose a reason for hiding this comment

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

@yxun kind create cluster creates the kind cluster based on the IP_FAMILY.
Can you explain why this change is required?

@dgn
Copy link

dgn commented Jan 28, 2025

Let's make functional changes like this one in the community repo

@dgn dgn closed this Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants