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

feat : Add support for Pull Image options - createImageOptions #1524

Merged
merged 2 commits into from
Feb 6, 2022

Conversation

rohanKanojia
Copy link
Member

Fix #1516

  • Add support for Pull Image options - createImageOptions (Port of feat: enable Create Image (pull) HTTP API options eclipse-jkube/jkube#1241 like we have buildOptions for providing build API query parameters via plugin, add support for providing API query parameters for pulling images via providing them as map in <image> > <build> > <createImageOptions>
  • Add a check in DockerfileUtil to ignore token starting with -- so that it doesn't fail the
    build. For specifying pull platform it would need to be set in image
    configuration build createImageOptions

@rohanKanojia rohanKanojia force-pushed the pr/pull-image-options branch 2 times, most recently from bfc43e1 to b7272ff Compare February 6, 2022 08:37
Port of eclipse-jkube/jkube#1241

Just like we have buildOptions for providing build API query parameters
via plugin, add support for providing API query parameters for pulling
images via providing them as map in `<image>` > `<build>` >
`<createImageOptions>`

Signed-off-by: Rohan Kumar <rohaan@redhat.com>
)

Right now we need to pull base images (ones specified in `FROM`
statements) before doing the actual build. DockerfileUtil doesn't seem
to be handling the case when an option is specified in `FROM` statement.
It just assumes second token to be image name.

Add check to ignore token starting with `--` so that it doesn't fail the
build. For specifiying pull platform it would need to be set in image
configuration build createImageOptions

Signed-off-by: Rohan Kumar <rohaan@redhat.com>
@rohanKanojia rohanKanojia marked this pull request as ready for review February 6, 2022 09:25
@rohanKanojia rohanKanojia merged commit dd086ee into fabric8io:master Feb 6, 2022
@rohanKanojia rohanKanojia deleted the pr/pull-image-options branch February 6, 2022 13:06
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.

FROM --platform in Dockerfile causes error
1 participant