Skip to content

fix: missing image for monitoring#1242

Open
bk201 wants to merge 1 commit intoharvester:masterfrom
bk201:fix-monitor-images
Open

fix: missing image for monitoring#1242
bk201 wants to merge 1 commit intoharvester:masterfrom
bk201:fix-monitor-images

Conversation

@bk201
Copy link
Contributor

@bk201 bk201 commented Mar 13, 2026

Problem:

A new cluster can't be provisioned in an air-gapped environment because the docker.io/rancher/kuberlr-kubectl:v6.0.0 image is missing. Monitoring CRDs is a prerequisite for many charts. The monitoring addon also need this image.

Screenshot 2026-03-13 at 8 54 36 AM Screenshot 2026-03-13 at 9 16 13 AM

Solution:

Add the image back.

Related Issue(s):

Test plan:

  • Ensure a new cluster can run in an air-gapped and non-air-gapped environment.
  • Ensure monitoring add on can be enabled and functional.

Additional documentation or context

@w13915984028 I'd keep the v7.0.1 image because it's supposed to be used by Rancher v2.14.0-alpha9.

The image is required for several one-shot pods.

Signed-off-by: Kiefer Chang <kiefer.chang@suse.com>
@bk201 bk201 marked this pull request as ready for review March 13, 2026 01:38
@bk201 bk201 requested review from Copilot and w13915984028 March 13, 2026 01:38
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a missing Rancher utility image to the repository’s Rancher image allowlist so air-gapped cluster provisioning (and enabling Monitoring/CRDs) doesn’t fail due to an unavailable image.

Changes:

  • Add docker.io/rancher/kuberlr-kubectl:v6.0.0 back to the Rancher image list used for image mirroring/air-gap preparation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@Vicente-Cheng Vicente-Cheng left a comment

Choose a reason for hiding this comment

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

thanks for the quick fix!

Copy link
Member

@w13915984028 w13915984028 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 for the fix.

BTW:
On commit 483d831, when bumping rancher-monitoring, this image was set to docker.io/rancher/kuberlr-kubectl:v6.0.0

and due to the image was shared by rancher-monitoring and rancher

it was replaced on later commit b0992d0

Not like the keeping bumping rancher, the rancher-monitoring normally keeps a version for some time.

We need to keep the docker.io/rancher/kuberlr-kubectl:v6.0.0 for rancher-monitoring on v180/master-head, and also keep another version (v7.0.1 for now) which matches embedded Rancher's (v2.14.0-alpha9) usage.

The txt based file seems not easy to comment on some images saying they are locked by some charts :)

I will try to set a used image list on rancher-monitoring addon side, and use a script to check if the expected images are on the list, to avoid such mistake.

harvester/harvester#10210

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.

4 participants