Skip to content

add cpfs.imagePullPrefix, remove nss roles#2449

Merged
ibm-ci-bot merged 2 commits intoIBM:masterfrom
bluzarraga:migration-update2
Mar 20, 2025
Merged

add cpfs.imagePullPrefix, remove nss roles#2449
ibm-ci-bot merged 2 commits intoIBM:masterfrom
bluzarraga:migration-update2

Conversation

@bluzarraga
Copy link
Member

What this PR does / why we need it: Further testing revealed additional problems with migration chart.

Signed-off-by: Ben Luzarraga <luzarragaben@gmail.com>
Signed-off-by: Ben Luzarraga <luzarragaben@gmail.com>
@ibm-ci-bot ibm-ci-bot added size/S and removed size/XS labels Mar 20, 2025
@bluzarraga
Copy link
Member Author

Also add watch permission to rbac to avoid noisy error:

E0320 16:51:33.451004 105 reflector.go:150] k8s.io/client-go/tools/watch/informerwatcher.go:146: Failed to watch *unstructured.Unstructured: unknown

Copy link
Contributor

@Daniel-Fan Daniel-Fan left a comment

Choose a reason for hiding this comment

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

/lgtm

@ibm-ci-bot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bluzarraga, Daniel-Fan

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:
  • OWNERS [Daniel-Fan,bluzarraga]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ibm-ci-bot ibm-ci-bot merged commit 2b21d0e into IBM:master Mar 20, 2025
6 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants