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

Fix #1384: Building spring-boot-with-jib fails with NoSuchMethodEror #1386

Merged
merged 2 commits into from
Oct 10, 2020

Conversation

rohanKanojia
Copy link
Member

Fix #1384

  • Updating jib-core to 0.15.0 broke our current JIB support due to
    incompatibility with httpclient. Updated apache httpclient to 4.5.10
  • Somehow behavior of Jib setting ImageFormat has been changed(maybe it was
    not working earlier. Introduced a new parameter in mojo docker.build.jib.imageFormat
    to allow users specify image format(OCI or Docker) with Docker as
    default ImageFormat.
  • Minor Cleanup in Integration tests; Added an integration test for JIB mode
    which builds image tarball and verifies it by loading into docker daemon

This should also fix #1385

@rohanKanojia rohanKanojia force-pushed the pr/issue1384 branch 4 times, most recently from c160cca to b6349d1 Compare October 2, 2020 07:36
…thodEror

+ Updating jib-core to 0.15.0 broke our current JIB support due to
  incompatibility with httpclient. Updated apache httpclient to 4.5.10
+ Somehow behavior of Jib setting ImageFormat has been changed(maybe it was
  not working earlier. Introduced a new parameter in mojo `docker.build.jib.imageFormat`
  to allow users specify image format(OCI or Docker) with Docker as
  default ImageFormat.
+ Minor Cleanup in Integration tests; Added an integration test for JIB mode
  which builds image tarball and verifies it by loading into docker daemon

This should also fix fabric8io#1385
Copy link
Collaborator

@rhuss rhuss left a comment

Choose a reason for hiding this comment

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

Thanks ! I don't have much time to make a rigorous review, but from a first look it totally looks reasonable.

Let's get it merged now

@rhuss rhuss merged commit 7f184fd into fabric8io:master Oct 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants