Skip to content

Update composite disk mtime when disk spec matches existing - #3142

Open
Elie-Kh wants to merge 1 commit into
google:mainfrom
Elie-Kh:update_composite_mtime
Open

Update composite disk mtime when disk spec matches existing#3142
Elie-Kh wants to merge 1 commit into
google:mainfrom
Elie-Kh:update_composite_mtime

Conversation

@Elie-Kh

@Elie-Kh Elie-Kh commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

When CreateOrUpdateCompositeDisk determines that an existing composite disk already matches the target partition layout, it skips regenerating the disk file. However, if a component partition (such as persistent_vbmeta.img) was regenerated or touched during instance setup, the composite disk's modification time would remain older than that component. This causes subsequent assemble_cvd checks to falsely detect that a component was updated and fail with an mtime mismatch error.

Update the modification time of output_composite_path using utimensat when the existing disk spec is retained, ensuring the composite disk mtime remains up to date.

Bug: 554545294

When CreateOrUpdateCompositeDisk determines that an existing composite
disk already matches the target partition layout, it skips regenerating
the disk file. However, if a component partition (such as
persistent_vbmeta.img) was regenerated or touched during instance setup,
the composite disk's modification time would remain older than that
component. This causes subsequent assemble_cvd checks to falsely detect
that a component was updated and fail with an mtime mismatch error.

Update the modification time of output_composite_path using utimensat
when the existing disk spec is retained, ensuring the composite disk
mtime remains up to date.

Bug: 554545294
@Elie-Kh
Elie-Kh force-pushed the update_composite_mtime branch from da7c225 to e1f39e6 Compare September 4, 2026 16:21
@Databean Databean added the kokoro:run Run e2e tests. label Sep 5, 2026
@GoogleCuttlefishTesterBot GoogleCuttlefishTesterBot removed the kokoro:run Run e2e tests. label Sep 5, 2026
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