Skip to content

feat: Implement Volumes specification subset#73

Merged
upils merged 5 commits intomainfrom
work/CRAFT-3961-Prototype-partitions-definition-and-usage-in-Imagecraft
Jan 29, 2025
Merged

feat: Implement Volumes specification subset#73
upils merged 5 commits intomainfrom
work/CRAFT-3961-Prototype-partitions-definition-and-usage-in-Imagecraft

Conversation

@upils
Copy link
Copy Markdown
Contributor

@upils upils commented Jan 20, 2025

Implement the subset of the Volumes specification defined in https://docs.google.com/document/d/12LxcUIUEoXvzK619CvK9eHU_8n_rEKnLPEC4clsriXA/edit?pli=1&tab=t.0

CRAFT-3972

@upils upils self-assigned this Jan 20, 2025
@upils upils changed the title feat: Implement partitions support feat: Implement Volumes specification subset Jan 21, 2025
@upils upils force-pushed the work/CRAFT-3961-Prototype-partitions-definition-and-usage-in-Imagecraft branch 2 times, most recently from b883f82 to f7c5b5f Compare January 21, 2025 10:10
Comment thread imagecraft/models/volume.py
Comment thread imagecraft/models/volume.py Outdated
Comment thread imagecraft/models/project.py Outdated
@upils upils force-pushed the work/CRAFT-3961-Prototype-partitions-definition-and-usage-in-Imagecraft branch 2 times, most recently from dba6801 to 0301b5d Compare January 21, 2025 11:18
@upils upils marked this pull request as ready for review January 21, 2025 12:33
@upils upils requested review from cmatsuoka and tigarmo January 21, 2025 13:49
Comment thread imagecraft/models/project.py
Comment thread imagecraft/models/project.py Outdated
Comment thread imagecraft/models/volume.py Outdated
@upils upils requested a review from tigarmo January 23, 2025 08:23
Copy link
Copy Markdown
Contributor

@tigarmo tigarmo left a comment

Choose a reason for hiding this comment

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

thanks!

@upils upils force-pushed the work/CRAFT-3961-Prototype-partitions-definition-and-usage-in-Imagecraft branch 2 times, most recently from 6da1003 to 81f210b Compare January 24, 2025 07:24
Comment thread imagecraft/models/volume.py
Comment thread imagecraft/models/volume.py Outdated
@upils upils requested a review from cmatsuoka January 27, 2025 08:58
@upils upils force-pushed the work/CRAFT-3961-Prototype-partitions-definition-and-usage-in-Imagecraft branch 2 times, most recently from 5f7f50d to 7df13a5 Compare January 28, 2025 07:34
Copy link
Copy Markdown

@cmatsuoka cmatsuoka left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

upils added 5 commits January 29, 2025 13:48
Signed-off-by: Paul Mars <paul.mars@canonical.com>
Signed-off-by: Paul Mars <paul.mars@canonical.com>
Signed-off-by: Paul Mars <paul.mars@canonical.com>
Instead of copying it, use VALID_PARTITION_REGEX and make it more strict (and explain why).

Signed-off-by: Paul Mars <paul.mars@canonical.com>
Signed-off-by: Paul Mars <paul.mars@canonical.com>
@upils upils force-pushed the work/CRAFT-3961-Prototype-partitions-definition-and-usage-in-Imagecraft branch from eecb48e to 224cb91 Compare January 29, 2025 12:48
@upils upils merged commit 95256c7 into main Jan 29, 2025
@upils upils deleted the work/CRAFT-3961-Prototype-partitions-definition-and-usage-in-Imagecraft branch January 29, 2025 13:00
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