Skip to content

fix(sdm-relay): switch kubectl images (again)#95

Open
jaredallard-laurel wants to merge 2 commits into
strongdm:mainfrom
jaredallard-laurel:patch-2
Open

fix(sdm-relay): switch kubectl images (again)#95
jaredallard-laurel wants to merge 2 commits into
strongdm:mainfrom
jaredallard-laurel:patch-2

Conversation

@jaredallard-laurel
Copy link
Copy Markdown
Contributor

Description of changes

Unfortunately, the way we deployed helm charts made it seem like the other image was working, but it was not. I didn't notice the bash call, and that Kubernetes image is kubectl only, no shell.

There isn't a great replacement with a shell, aside from this opensuse image, so I opted for that, but totally open to suggestions (e.g., rancher/kubectl).

I triple checked this one works this time :)

Validation steps

  • installed the pre-commit hooks with pre-commit install
  • (optionally) deployed this change to k8s cluster.

Unfortunately, the way we deployed helm charts made it seem
like the other image was working, but it was not. I didn't notice
the bash call, and that Kubernetes image is kubectl only, no shell.

There isn't a great replacement with a shell, aside from this opensuse
image, so I opted for that, but totally open to suggestions (e.g., rancher/kubectl).
@jaredallard-laurel jaredallard-laurel requested a review from a team as a code owner March 13, 2026 23:09
@jaredallard-laurel
Copy link
Copy Markdown
Contributor Author

Ping on this @kellenanker7 (sorry!)

@kellenanker7
Copy link
Copy Markdown
Contributor

interestingly, it seems like some k8s versions don't have associated images. for example:

~: docker pull registry.suse.com/suse/kubectl:1.35
1.35: Pulling from suse/kubectl
Digest: sha256:164d73450e981f6996c196472dd1191367b12c3eb0f45780a5cc444652af9130
Status: Downloaded newer image for registry.suse.com/suse/kubectl:1.35
registry.suse.com/suse/kubectl:1.35

~: docker pull registry.suse.com/suse/kubectl:1.33
1.33: Pulling from suse/kubectl
7a6c20d59360: Pull complete
Digest: sha256:3262d05e247c1df9ffb663b17608167a445cbfc20117b67abe8c152d013611d2
Status: Downloaded newer image for registry.suse.com/suse/kubectl:1.33
registry.suse.com/suse/kubectl:1.33

and yet:

~: docker pull registry.suse.com/suse/kubectl:1.34
Error response from daemon: failed to resolve reference "registry.suse.com/suse/kubectl:1.34": registry.suse.com/suse/kubectl:1.34: not found

not entirely sure what's wrong with 1.34, but we likely can't move forward with this new image if an entire minor version is not supported.

any other images you may have in mind? a public ECR one, for example?

@kellenanker7
Copy link
Copy Markdown
Contributor

@jaredallard-laurel apologies for the late replies!

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.

2 participants