Skip to content

added inventory check in live destroy/status - #4685

Open
OisinJohnston2005 wants to merge 2 commits into
kptdev:mainfrom
Nordix:inv-check-in-destroy-status
Open

added inventory check in live destroy/status#4685
OisinJohnston2005 wants to merge 2 commits into
kptdev:mainfrom
Nordix:inv-check-in-destroy-status

Conversation

@OisinJohnston2005

Copy link
Copy Markdown
Contributor

Fixes: #2190

Added a check for inventory mismatches in kpt live destroy and kpt live status

Signed-off-by: Oisin Johnston <oisin.johnston@ericsson.com>
@OisinJohnston2005
OisinJohnston2005 requested review from a team August 6, 2026 15:27
@dosubot dosubot Bot added size:M This PR changes 30-99 lines, ignoring generated files. area/live labels Aug 6, 2026
@netlify

netlify Bot commented Aug 6, 2026

Copy link
Copy Markdown

Deploy Preview for kptdocs ready!

Name Link
🔨 Latest commit 79c5a2f
🔍 Latest deploy log https://app.netlify.com/projects/kptdocs/deploys/6a8eb1e065dd2b00084e54d6
😎 Deploy Preview https://deploy-preview-4685--kptdocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@sonarqubecloud

sonarqubecloud Bot commented Aug 6, 2026

Copy link
Copy Markdown

Comment thread commands/live/destroy/cmddestroy.go Outdated
Comment thread commands/live/status/cmdstatus.go Outdated
Co-authored-by: Aravindhan Ayyanathan <aravindhan.a@est.tech>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/live size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

kpt live doesn't check the inventory id consistently across apply/destroy/status

2 participants