-
Notifications
You must be signed in to change notification settings - Fork 84
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
220 useproductized jolokia fuse 7.x.redhat #221
220 useproductized jolokia fuse 7.x.redhat #221
Conversation
Signed-off-by: Aurélien Pupier <[email protected]>
@apupier thanks for your interest and PR for this project. Full disclosure: I am maintaining the community version ( |
I was pointed to this repo when i asked what is used to create the Fuse images: https://access.redhat.com/containers/?tab=overview#/registry.access.redhat.com/fuse7/fuse-java-openshift to have the merge with jboss-container-images/openjdk might be a very good idea. It might explain why VS Code OpenShift connector is working with https://access.redhat.com/containers/?tab=overview#/registry.access.redhat.com/redhat-openjdk-18/openjdk18-openshift and not with https://access.redhat.com/containers/?tab=overview#/registry.access.redhat.com/fuse7/fuse-java-openshift . it is why I landed here.
thanks for the proposition. For now, I don't feel that I know enough about these images to be responsible to merge modifications with my only point of view. Maybe after I sort out which images relies on which one and why and in which way they diverged. |
OK. Hope you understand that I won't merge this PR. Perhaps other old committers on this project will. Up to you know if you want to keep this open or when you want to Abandon to Close it - or let me know if you are interested after all. |
@vorburger I see your point, but please note that we had from the very beginning a variant based on RHEL as a base image, which is used productized variations. So this repos supports both, plain community images but also RHEL based images. So I'm totally for merging this PR for RHEL images only. |
@rhuss I will not be able to help maintain the |
closign as either things has moved from another side or still not moved and so there will be low probability that it happens oen day. |
should we change also at these places?
s2i/java/images/rhel/Dockerfile
Line 15 in 41cf3ea
s2i/java/images/rhel/Dockerfile
Line 61 in 41cf3ea
Why is it retrieving with curl? should we do it also with curl but with the maven redhat repository and productized version?