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

JIB Image Build Regession in 1.4.0 #895

Closed
rohanmars opened this issue Sep 8, 2021 · 2 comments · Fixed by #933
Closed

JIB Image Build Regession in 1.4.0 #895

rohanmars opened this issue Sep 8, 2021 · 2 comments · Fixed by #933
Assignees
Labels
bug Something isn't working
Milestone

Comments

@rohanmars
Copy link
Contributor

Seems like there has been a bug/side effect introduced in 1.4.0 with build jib images.

The same image build that worked in versions before now gets an error when I run mvn install:

[ERROR] Failed to execute goal org.eclipse.jkube:kubernetes-maven-plugin:1.4.0:build (default) on project jkube-java-builder: Failed to execute the build: Error when building JIB image: Failed to create directory: /home/user/s2i-test/target/docker/test/1.0.0-SNAPSHOT/build/ -> [Help 1]

What is strange, is after that build, the folder has been created and if you run mvn install again it works.

The config is:

test:1.0.0-SNAPSHOT
@manusa
Copy link
Member

manusa commented Sep 9, 2021

Hi @rohanmars,
Thanks for your report. Could you please provide some way to reproduce this issue?

@manusa manusa added the bug Something isn't working label Sep 20, 2021
@manusa
Copy link
Member

manusa commented Sep 20, 2021

I managed to reproduce the issue (https://github.com/marcnuri-demo/202109-quarkus-cafe/runs/3650824725?check_suite_focus=true)

Problem in:
https://github.com/eclipse/jkube/blob/93a66d007ad1c6169ba804959c81c7d3e279b6f0/jkube-kit/common/src/main/java/org/eclipse/jkube/kit/common/util/FileUtil.java#L251

Directory is created but returns false (I think last canonical / messes the method).

Could be replaced by FileUtils#forceMkdir

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
None yet
Development

Successfully merging a pull request may close this issue.

2 participants