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

Fixes kind create cluster script that contained a hardcoded IP #383

Merged

Conversation

jordigilh
Copy link
Contributor

Removes a hardcoded IP that defined the directory inside /etc/containerd for the registry configuration. When the IPs mismatched it caused a failure to pull or push images from the insecure registry via kaniko builder with Kind. The correct approach is to use the IP from the docker network configuration for the registry container for both the directory name and the configuration contained within.

@wmedvede @ricardozanini PTAL.

Copy link
Member

@ricardozanini ricardozanini left a comment

Choose a reason for hiding this comment

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

Thank you!

@ricardozanini
Copy link
Member

Can you please rebase your fork? 🙏
This is the reason why e2e failed.

…e registry configuration in containerd. Instead it now uses the associated IP of the registry container running in Docker

Signed-off-by: Jordi Gil <[email protected]>
@jordigilh jordigilh force-pushed the fix/kind_hardcoded_ip_in_config branch from ace8a88 to e6f214e Compare February 4, 2024 10:11
@ricardozanini ricardozanini merged commit ed4770a into apache:main Feb 5, 2024
4 checks passed
rgdoliveira pushed a commit to rgdoliveira/kogito-serverless-operator that referenced this pull request Mar 11, 2024
…e registry configuration in containerd. Instead it now uses the associated IP of the registry container running in Docker (apache#383)

Signed-off-by: Jordi Gil <[email protected]>
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.

3 participants