-
Notifications
You must be signed in to change notification settings - Fork 352
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
Integrations fail with Jib publish strategy #5007
Comments
@gansheer can you please have a look? |
Hi @hernanDatgDev, I can't reproduce your issue, it works well for me. |
Install command camel-k v2.1.0: Integration Platform: Test integration (timer-log.groovy):
Operator logs:
|
Tried with camel-k v2.2.0 snapshot and received more information. Seems like a problem accessing the docker registry. The same registry secret works with Spectrum as the default publish strategy. I'm only seeing this when I use Jib. Also for more context I'm using docker-desktop as my test cluster.
|
@hernanDatgDev it is probably an issue with how the registry secret is managed by the Jib publish strategy. |
@gansheer Unfortunately I'm still facing this unauthorized 401 registry issue with the latest release 2.2.0 These are the latest steps I have for v 2.2.0:
Log snippet from builder pod init-container: jib
Full log: 📓 Note: I've tried both with and without build pod enabled. I thought builder pod was default for jib however I have not seen this behavior. By default my jib build occurs within operator. |
@hernanDatgDev Thanks for giving the detail of your installation. I identified what you report while working on this issue #5017. There are definitely some multiple custom behavior on what registry URL is used when dealing with docker hub registry via I will try to give you some workaround when I have more informations. |
@hernanDatgDev To make it work with Jib you have two possibilities:
|
Hey @gansheer, unfortunately neither of these options worked for me and I tried other possibilities as well, but no luck. After all attempts I tried with the latest 2.3.0-SNAPSHOT and I got the expected behavior. For now this will work since I can proceed with my task. I'm still not sure what the issue is however I suspect there is some default docker values that are overriding whatever I input as you mentioned in #5017. I'm guessing this is the case because of the "/v2" I saw appended to my registry parameters in the logs despite explicitly defining only "v1" |
You are right, it does not work in 2.2.0 release but works in 2.3.0-SNAPSHOT. The fix is not in the 2.2.0 release but will be part of the next one. |
@gansheer What would it look like to take the 2.2.0 fix RE this issue and patch it for v2.1.x? Could it really be as simple as taking the authentication refactoring you did and applying it to 2.1? My only concern is that the fix you made applies to errors seen w/ 2.2.0 and not with 2.1.0. |
@gansheer I tried applying your fix to 2.1.0 and the Jib strategy is working as expected. I believe that the logging in 2.1.0 w/ Jib simply was not detailed enough so the errors I was getting were hiding the fact that there was an issue with the container registry. I'm willing to make a PR for patching 2.1.0 but I must admit I'm not sure what the patching process looks like. |
We can backport to 2.1.x and if it's an issue that is blocking any production behavior we can make a patch release accordingly. |
@hernanDatgDev To make a backport you just need to do a PR on the release-2.1.x branch with the backported code. |
The jib strategy are tested in e2e test - the builder workflow. The e2e tests are thoses executed with a kubernetes environment. Also the default strategy is used in lots of the other e2e tests that build any integration kit to run an integration. |
What happened?
Integrations do not run with publish strategy Jib.
While using v2.1.0 the build fails before an image can be created with minimal logging detail
While using v2.0.0 the build succeeds but the container is not able to be created/deployed
Steps to reproduce
--build-publish-strategy Jib
Relevant log output
Camel K version
2.1.0 & 2.0.0
The text was updated successfully, but these errors were encountered: