Skip to content

fix: add timeout for live patching service#7956

Merged
YaoC merged 1 commit intomainfrom
chyao/snapshot-timeout
Feb 27, 2026
Merged

fix: add timeout for live patching service#7956
YaoC merged 1 commit intomainfrom
chyao/snapshot-timeout

Conversation

@YaoC
Copy link
Contributor

@YaoC YaoC commented Feb 25, 2026

What this PR does / why we need it:

add timeout for live patching service

Which issue(s) this PR fixes:

Fixes #

Copy link
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 adds timeout configuration for live patching services on both Ubuntu and Mariner distributions to prevent indefinite service hangs.

Changes:

  • Added 60-minute timeout for snapshot-update.service on Ubuntu
  • Added 60-minute timeout for package-update.service on Mariner
  • Updated all test data files with new gzipped binary content reflecting these service configuration changes

Reviewed changes

Copilot reviewed 61 out of 61 changed files in this pull request and generated no comments.

File Description
parts/linux/cloud-init/artifacts/ubuntu/snapshot-update.service Added timeout and kill signal configuration for Ubuntu snapshot update service
parts/linux/cloud-init/artifacts/mariner/package-update.service Added timeout and kill signal configuration for Mariner package update service
pkg/agent/testdata/*/CustomData Updated test data files with new gzipped binary content reflecting the service configuration changes

Copilot AI review requested due to automatic review settings February 27, 2026 17:57
@YaoC YaoC force-pushed the chyao/snapshot-timeout branch from cdac4fc to 290dfcb Compare February 27, 2026 17:57
Copy link
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 62 out of 62 changed files in this pull request and generated no new comments.

Copy link
Contributor

@yagmurbaydogan yagmurbaydogan left a comment

Choose a reason for hiding this comment

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

lgtm!

@YaoC YaoC enabled auto-merge (squash) February 27, 2026 21:54
Copy link
Member

@yewmsft yewmsft left a comment

Choose a reason for hiding this comment

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

LGTM

@YaoC YaoC merged commit c94651a into main Feb 27, 2026
31 of 33 checks passed
@YaoC YaoC deleted the chyao/snapshot-timeout branch February 27, 2026 23:22
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.

4 participants