Skip to content

Conversation

@nammn
Copy link
Collaborator

@nammn nammn commented Dec 29, 2025

Summary

This pr fixes some more minikube static container setup steps which i encountered after merging the prior pr.

  • Fix registry container name conflict by adding --replace flag to podman run (in case it already exists)
  • Fix IPv6 connection refused by using 127.0.0.1 instead of localhost for local registry
  • Fix minikube status check to use sudo podman ps instead of minikube status (rootful vs rootless podman namespace mismatch)

Proof of Work

  • manual patch
    • running it multiple times still passes. Re-running causes the scheduler to schedule the task to the same static machine

Checklist

  • Have you linked a jira ticket and/or is the ticket in the title?
  • Have you checked whether your jira ticket required DOCSP changes?
  • Have you added changelog file?

@nammn nammn changed the title try more minikube fixes add more minikube fixes Dec 29, 2025
@nammn nammn changed the title add more minikube fixes add more static cluster minikube fixes Dec 29, 2025
@github-actions
Copy link

⚠️ (this preview might not be accurate if the PR is not rebased on current master branch)

MCK 1.6.2 Release Notes

Bug Fixes

  • Fixed an issue where monitoring agents would fail after disabling TLS on a MongoDB deployment.
  • Persistent Volume Claim resize fix: Fixed an issue where the Operator ignored namespaces when listing PVCs, causing conflicts with resizing PVCs of the same name. Now, PVCs are filtered by both name and namespace for accurate resizing.

echo "Removing ~/.minikube/machines/minikube directory..."
rm -rf ~/.minikube/machines/minikube
fi
sudo rm -rf ~/.minikube/machines/minikube 2>/dev/null || true
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

since we use sudo podman (rootful) we also need to ensure minikube gets cleaned up in root space

@nammn nammn added the skip-changelog Use this label in Pull Request to not require new changelog entry file label Dec 29, 2025
sudo podman rm -f registry 2>/dev/null || true

if ! sudo podman run -d -p 127.0.0.1:5000:5000 --name registry --restart=always docker.io/library/registry:2; then
if ! sudo podman run -d -p 127.0.0.1:5000:5000 --replace --name registry --restart=always docker.io/library/registry:2; then
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

sometimes cleanup didn't work, so we should just replace the existing/running one

sudo tee /root/.config/containers/registries.conf << 'EOF' >/dev/null
[[registry]]
location = "localhost:5000"
location = "127.0.0.1:5000"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

some loopback misses from prior pr

@nammn nammn changed the title add more static cluster minikube fixes ibm power fixes - add more static cluster minikube fixes Dec 29, 2025
@nammn nammn marked this pull request as ready for review December 29, 2025 13:44
@nammn nammn requested a review from a team as a code owner December 29, 2025 13:44
@nammn nammn requested review from a team and fealebenpae December 29, 2025 14:14
@nammn nammn merged commit 259b878 into master Dec 29, 2025
32 of 35 checks passed
@nammn nammn deleted the fix-ibm-docker-4 branch December 29, 2025 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Use this label in Pull Request to not require new changelog entry file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants