Skip to content

fix: improve recipe parsing in build workflows#1654

Merged
RoyalOughtness merged 2 commits intosecureblue:livefrom
HastD:provenance-parsing
Dec 2, 2025
Merged

fix: improve recipe parsing in build workflows#1654
RoyalOughtness merged 2 commits intosecureblue:livefrom
HastD:provenance-parsing

Conversation

@HastD
Copy link
Copy Markdown
Collaborator

@HastD HastD commented Dec 2, 2025

Use yq instead of sed to extract the image name and base image data from recipe files. This ensures correct parsing of the YAML so the parsed values match what BlueBuild reads from the recipe file, even in the presence of trailing whitespace, quotation marks around the string, or other similar syntax elements allowed by YAML but not handled by the previous parsing method.

Also cleaned up trailing whitespace in recipe files.

@HastD HastD requested a review from RoyalOughtness as a code owner December 2, 2025 01:36
Use `yq` instead of `sed` to extract the image name and base image data
from recipe files. This ensures correct parsing of the YAML so the
parsed values match what BlueBuild reads from the recipe file, even in
the presence of trailing whitespace, quotation marks around the string,
or other similar syntax elements allowed by YAML but not handled by the
previous parsing method.

Also cleaned up trailing whitespace in recipe files.
@HastD HastD force-pushed the provenance-parsing branch from 283f665 to 0b819aa Compare December 2, 2025 02:02
@HastD HastD requested a review from RoyalOughtness December 2, 2025 02:03
@RoyalOughtness RoyalOughtness merged commit 3b3713f into secureblue:live Dec 2, 2025
19 of 22 checks passed
@HastD HastD deleted the provenance-parsing branch December 2, 2025 19:35
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