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

Not able to override Image Name in Simple Dockerfile Mode with jkube.generator.name #624

Closed
rohanKanojia opened this issue Mar 25, 2021 · 2 comments · Fixed by #629
Closed
Assignees
Labels
bug Something isn't working
Projects
Milestone

Comments

@rohanKanojia
Copy link
Member

rohanKanojia commented Mar 25, 2021

When using simple Dockerfile mode, we're not able to override the default image name using jkube.generator.name property. Seems like we always assume image name to be group:artifact:latest in zero dockerfile mode and don't consider whether jkube.generator.name property is set or not, we do pass a defaultImageName but somehow it seems to be passed as null:

https://github.com/eclipse/jkube/blob/fd9003cf04fd2bcd7407e87ffc30dbf41d8a24b7/jkube-kit/build/api/src/main/java/org/eclipse/jkube/kit/build/api/helper/DockerFileUtil.java#L249-L252

In order to reproduce this, you can go to quickstarts/maven/docker-file-simple and try overriding image name by adding some property. I noticed this when testing @ghassen1khalil sample: https://github.com/ghassen1khalil/jkube-error-reproducer

@rohanKanojia rohanKanojia added the bug Something isn't working label Mar 25, 2021
@manusa manusa added this to the 1.2.0 milestone Mar 25, 2021
@manusa manusa added this to Planned in Sprint #199 Mar 25, 2021
rohanKanojia added a commit to rohanKanojia/jkube that referenced this issue Mar 26, 2021
…file Mode with `jkube.generator.name`

Modify Mojos logic to consider image name configured from properties
rohanKanojia added a commit to rohanKanojia/jkube that referenced this issue Mar 26, 2021
…file Mode with `jkube.generator.name`

Modify Mojos logic to consider image name configured from properties
@manusa manusa moved this from Planned to Review in Sprint #199 Mar 26, 2021
@manusa manusa moved this from Review to In progress in Sprint #199 Mar 26, 2021
@manusa manusa moved this from In progress to Review in Sprint #199 Mar 26, 2021
rohanKanojia added a commit to rohanKanojia/jkube that referenced this issue Mar 29, 2021
…file Mode with `jkube.generator.name`

Modify Mojos logic to consider image name configured from properties
rohanKanojia added a commit to rohanKanojia/jkube that referenced this issue Mar 29, 2021
…file Mode with `jkube.generator.name`

Modify Mojos logic to consider image name configured from properties
rohanKanojia added a commit to rohanKanojia/jkube that referenced this issue Mar 31, 2021
…file Mode with `jkube.generator.name`

Modify Mojos logic to consider image name configured from properties
rohanKanojia added a commit to rohanKanojia/jkube that referenced this issue Mar 31, 2021
…file Mode with `jkube.generator.name`

Modify Mojos logic to consider image name configured from properties
Sprint #199 automation moved this from Review to Done Mar 31, 2021
manusa pushed a commit that referenced this issue Mar 31, 2021
…h `jkube.generator.name`

Modify Mojos logic to consider image name configured from properties
@rohanKanojia
Copy link
Member Author

@ghassen1khalil : We just released v1.2.0 which should fix your issue. Could you please try it out and provide feedback?

@ghassen1khalil
Copy link

@rohanKanojia I confirm that the release 1.2.0 fixed my bug 👍🏼
Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
No open projects
Sprint #199
  
Done
3 participants