Skip to content

cover external bootc status changes - #264

Merged
alicefr merged 2 commits into
bootc-dev:mainfrom
HarshwardhanPatil07:test/e2e-external-bootc-status-change
Oct 30, 2026
Merged

alicefr merged 2 commits into
bootc-dev:mainfrom
HarshwardhanPatil07:test/e2e-external-bootc-status-change

Conversation

@HarshwardhanPatil07

Copy link
Copy Markdown
Member

TestExternalBootcStatusChange verifies that an external host operation is reflected in BootcNode status through fsnotify, without a spec change or daemon restart triggering reconciliation.

@HarshwardhanPatil07
HarshwardhanPatil07 requested a balanced review from Copilot October 27, 2026 06:02

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

@HarshwardhanPatil07
HarshwardhanPatil07 force-pushed the test/e2e-external-bootc-status-change branch from fc97c79 to 850ae45 Compare October 27, 2026 06:06
Comment thread test/e2e/watcher_test.go Resolved
@HarshwardhanPatil07
HarshwardhanPatil07 force-pushed the test/e2e-external-bootc-status-change branch 3 times, most recently from 7e55a6a to 9aa8974 Compare October 29, 2026 14:11
Comment thread test/e2e/watcher_test.go Resolved
alicefr
alicefr previously approved these changes Oct 30, 2026
Verify that external host staging reaches BootcNode status without a spec change or daemon restart. Share the daemon polling default with the test so its timing bound excludes fallback polling.

Generated-by: AI
Signed-off-by: HarshwardhanPatil07 <harshpat@redhat.com>
Mark the Milestone 4d validation as implemented and document the timing and identity checks that distinguish watcher refresh from polling or restart.

Generated-by: AI
Signed-off-by: HarshwardhanPatil07 <harshpat@redhat.com>
@alicefr
alicefr merged commit 996d2e1 into bootc-dev:main Oct 30, 2026
12 of 17 checks 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