Skip to content

Inform developer how to restore OLM CSV after cleanup with local webhook#1919

Open
ciecierski wants to merge 1 commit into
openstack-k8s-operators:mainfrom
ciecierski:cleanup-message
Open

Inform developer how to restore OLM CSV after cleanup with local webhook#1919
ciecierski wants to merge 1 commit into
openstack-k8s-operators:mainfrom
ciecierski:cleanup-message

Conversation

@ciecierski
Copy link
Copy Markdown
Contributor

Print a message in the cleanup handler with the exact oc patch command needed to restore the operator's original CSV after the local webhook run exits.

@openshift-ci openshift-ci Bot requested review from fultonj and rebtoor May 14, 2026 11:40
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 14, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: ciecierski
Once this PR has been reviewed and has the lgtm label, please assign rebtoor for approval. For more information see the Code Review Process.

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

Details Needs approval from an approver in each of these files:

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

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 14, 2026

OpenStackControlPlane CRD Size Report

Metric Value
CRD JSON size 322464 bytes (315KB)
Base branch size 322464 bytes
Change +0.00%
Status yellow — growing
Threshold reference
Color Range Meaning
🟢 green < 300KB Comfortable
🟡 yellow 300–400KB Growing
🟠 orange 400–750KB Concerning
🔴 red > 750KB Approaching 1.5MB etcd limit (cut in half to allow space for update)

@ciecierski ciecierski requested a review from abays May 14, 2026 11:42
@ciecierski ciecierski changed the title Inform user how to restore OLM CSV after cleanup with local web hook Inform developer how to restore OLM CSV after cleanup with local webhook May 14, 2026
Comment thread hack/run_with_local_webhook.sh Outdated
Print a message in the cleanup handler with the exact `oc patch`
command needed to restore the operator's original CSV after the
local webhook run exits.
@ciecierski ciecierski requested a review from dprince May 14, 2026 19:30
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