Skip to content

feat(mounts): Implement mounts option for Recipe and Module#716

Open
Aex12 wants to merge 9 commits intoblue-build:mainfrom
Aex12:feat/recipe-mounts
Open

feat(mounts): Implement mounts option for Recipe and Module#716
Aex12 wants to merge 9 commits intoblue-build:mainfrom
Aex12:feat/recipe-mounts

Conversation

@Aex12
Copy link

@Aex12 Aex12 commented Feb 27, 2026

This PR implements the mounts option as proposed in blue-build/schema#16

@Aex12 Aex12 requested a review from gmpinder as a code owner February 27, 2026 02:27
@gmpinder gmpinder changed the title Implement mounts option in recipe feat(mounts): Implement mounts option in recipe Feb 27, 2026
@Aex12 Aex12 changed the title feat(mounts): Implement mounts option in recipe feat(mounts): Implement mounts option in recipe and module Feb 27, 2026
@Aex12 Aex12 requested a review from gmpinder February 27, 2026 05:39
@Aex12 Aex12 changed the title feat(mounts): Implement mounts option in recipe and module feat(mounts): Implement mounts option for Recipe and Module Feb 27, 2026
@Aex12 Aex12 force-pushed the feat/recipe-mounts branch from b43e9a5 to ff840e2 Compare February 27, 2026 22:29
@Aex12 Aex12 force-pushed the feat/recipe-mounts branch from ff840e2 to f84e1e9 Compare February 27, 2026 22:47
Copy link
Member

@gmpinder gmpinder left a comment

Choose a reason for hiding this comment

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

Looking good so far can you add some of this new functionality to the recipe in ./integration-test/test-repo/? That one is one that gets built in the integration tests. The one you were updating, which was needed, is for testing the schema.

@gmpinder
Copy link
Member

I also left a comment in discord about updating the constant for the schema base URL. Doing that will pull the new schema changes before they get fully merged in.

@Aex12
Copy link
Author

Aex12 commented Feb 28, 2026

Hi @gmpinder,

I just updated the integration tests and the schema base url constant

@Aex12
Copy link
Author

Aex12 commented Mar 1, 2026

Hi @gmpinder I just updated all the other recipes in the integration tests to include the cache mount option so that the test defined in common.yml wont fail for those tests. This should fix the integration tests not passing.

@gmpinder
Copy link
Member

gmpinder commented Mar 1, 2026

I'm seeing the error message unrecognized value "private" for field 'sharing'

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.

2 participants