Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ioc: add stage without IOC building. #20

Merged
merged 1 commit into from
Sep 13, 2023
Merged

ioc: add stage without IOC building. #20

merged 1 commit into from
Sep 13, 2023

Conversation

henriquesimoes
Copy link
Collaborator

This allows us to run an IOC which was built along with its support module. This pattern is used for some community modules, including motor's and areaDetector's ones.

A new (empty) stage has been created only to specify the base image in the COPY command, since --from argument does not support variable expansion.

build-stage has been moved below no-build stage, so bad multi-stage resolvers won't fail for the new stage.

This allows us to run an IOC which was built along with its support
module. This pattern is used for some community modules, including
motor's and areaDetector's ones.

A new (empty) stage has been created only to specify the base image in
the COPY command, since `--from` argument does not support variable
expansion [1].

`build-stage` has been moved below `no-build` stage, so bad multi-stage
resolvers won't fail for the new stage.

[1]: https://forums.docker.com/t/how-can-i-expand-a-variable-within-a-copy-command-in-the-dockerfile/113538/2
Copy link
Member

@ericonr ericonr left a comment

Choose a reason for hiding this comment

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

Assuming the build has been tested with podman/non buildkit docker already, approved.

@henriquesimoes henriquesimoes merged commit 30a9e5f into main Sep 13, 2023
1 check passed
@henriquesimoes henriquesimoes deleted the no-build branch September 13, 2023 18:11
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