Skip to content

Add RN for the etcd / turtles bug#1060

Merged
mkrutov-suse merged 4 commits intosuse-edge:mainfrom
mkrutov-suse:mkr_rn_turtlesbug
Feb 18, 2026
Merged

Add RN for the etcd / turtles bug#1060
mkrutov-suse merged 4 commits intosuse-edge:mainfrom
mkrutov-suse:mkr_rn_turtlesbug

Conversation

@mkrutov-suse
Copy link
Contributor

No description provided.

@hardys hardys requested a review from ranjinimn February 17, 2026 09:22
@hardys
Copy link
Contributor

hardys commented Feb 17, 2026

Please rebase now that #1059 merged, this should solve the check job failure

Copy link
Collaborator

@ranjinimn ranjinimn left a comment

Choose a reason for hiding this comment

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

@mkrutov-suse I had a few suggestions. Please let me know if you have any questions. Thanks.

Co-authored-by: ranjinimn <ranjini.n@suse.com>
----
kubectl patch crd crd-item-name \
--type='json' \
-p='[{"op": "remove", "path": "/metadata/annotations/cert-manager.io~1inject-ca-from"}]'
Copy link
Contributor

Choose a reason for hiding this comment

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

To clarify for other reviewers, the ~1 here is from https://jsonpatch.com/#json-pointer as we need to escape the / in the key name

Copy link
Contributor

@hardys hardys 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!

@mkrutov-suse mkrutov-suse merged commit 2ede082 into suse-edge:main Feb 18, 2026
1 check passed
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