Skip to content

Conversation

@fintelia
Copy link
Contributor

This PR adds new fields to COSI so that we can avoid losing information when converting VHD -> COSI. Specifically, it adds information from the disk partition table that may be important, like what the part UUIDs were and what order the partitions appeared on disk.

The plan is for trident stream-image to honor these additional fields while trident install would continue its current approach of generating new random UUIDs and create partitions in the order specified in the Host Configuration.

@fintelia fintelia requested a review from a team as a code owner December 11, 2025 00:05
Copilot AI review requested due to automatic review settings December 11, 2025 00:05
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 introduces COSI specification version 1.2 to support baremetal image workflows by preserving partition table metadata during VHD to COSI conversion. The changes add four new required fields to the ImageFile object that capture original partition characteristics.

Key changes:

  • Add COSI version 1.2 to the revision summary with spec date 2025-12-10
  • Extend ImageFile object with partition metadata fields: originalSize, partUuid, partLabel, and partNumber
  • Update documentation to clarify that ImageFile objects represent disk partitions with both content and partition table information

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings December 11, 2025 18:59
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 1 out of 1 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings December 11, 2025 19:04

This comment was marked as outdated.

Copilot AI review requested due to automatic review settings December 11, 2025 22:20

This comment was marked as outdated.

frhuelsz
frhuelsz previously approved these changes Dec 17, 2025
Copy link
Contributor Author

@fintelia fintelia left a comment

Choose a reason for hiding this comment

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

Add discussion from API design meeting

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