Skip to content

verify_os_update - mark node dirty after OS update#4509

Merged
LiliDeng merged 3 commits into
mainfrom
smyakam/verify_os_update/2026_05_31
Jun 2, 2026
Merged

verify_os_update - mark node dirty after OS update#4509
LiliDeng merged 3 commits into
mainfrom
smyakam/verify_os_update/2026_05_31

Conversation

@SRIKKANTH
Copy link
Copy Markdown
Collaborator

@SRIKKANTH SRIKKANTH commented Jun 1, 2026

After OS update sometimes kernel version may change. When LISA is used to validate a VM image with specific kernel version, changing the kernel will result in running tests with undesired kernel which should be avoided. Marking the node dirty after OS update to avoid such issues.

Description

Related Issue

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Refactoring
  • Documentation update

Checklist

  • Description is filled in above
  • No credentials, secrets, or internal details are included
  • Peer review requested (if not, add required peer reviewers after raising PR)
  • Tests executed and results posted below

Test Validation

Key Test Cases:

verify_os_update
Impacted LISA Features:

Tested Azure Marketplace Images:

  • Canonical ubuntu-26_04-lts server latest

Test Results

Image VM Size Result
Canonical ubuntu-26_04-lts server latest Standard_DC2as_v6 PASSED

After OS update sometimes kernel version may change.
When LISA is used to validate a VM image with specific kernel version,
changing the kernel will result in running tests with undesired kernel which should be avoided.
Marking the node dirty after OS update to avoid such issues.
Copilot AI review requested due to automatic review settings June 1, 2026 15:30
@SRIKKANTH SRIKKANTH requested a review from umfranci June 1, 2026 15:30
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the verify_os_update Azure image validation test to ensure that if an OS package update changes the kernel version, the node is marked dirty so it won’t be reused by subsequent test cases that may require the original kernel/version state.

Changes:

  • Mark the node dirty after update_packages() to prevent reusing a potentially kernel-changed VM for later validations.
  • Move the reboot call into the POSIX branch so it only happens when the update actually runs.

Comment thread lisa/microsoft/testsuites/core/azure_image_standard.py Outdated
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 1, 2026

✅ AI Test Selection — PASSED

1 test case(s) selected (view run)

Marketplace image: canonical 0001-com-ubuntu-server-jammy 22_04-lts-gen2 latest

Count
✅ Passed 1
❌ Failed 0
⏭️ Skipped 0
Total 1
Test case details
Test Case Status Time (s) Message
verify_os_update (lisa_0_0) ✅ PASSED 83.010

Comment thread lisa/microsoft/testsuites/core/azure_image_standard.py Outdated
Copilot AI review requested due to automatic review settings June 2, 2026 06:36
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 2, 2026

✅ AI Test Selection — PASSED

1 test case(s) selected (view run)

Marketplace image: canonical 0001-com-ubuntu-server-jammy 22_04-lts-gen2 latest

Count
✅ Passed 1
❌ Failed 0
⏭️ Skipped 0
Total 1
Test case details
Test Case Status Time (s) Message
verify_os_update (lisa_0_0) ✅ PASSED 86.050

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread lisa/microsoft/testsuites/core/azure_image_standard.py
@LiliDeng LiliDeng merged commit a18cef0 into main Jun 2, 2026
62 checks passed
@LiliDeng LiliDeng deleted the smyakam/verify_os_update/2026_05_31 branch June 2, 2026 06:43
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 2, 2026

✅ AI Test Selection — PASSED

1 test case(s) selected (view run)

Marketplace image: canonical 0001-com-ubuntu-server-jammy 22_04-lts-gen2 latest

Count
✅ Passed 1
❌ Failed 0
⏭️ Skipped 0
Total 1
Test case details
Test Case Status Time (s) Message
verify_os_update (lisa_0_0) ✅ PASSED 83.360

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