-
Notifications
You must be signed in to change notification settings - Fork 33
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
Rultor Build Fails on Recent Versions of yegor256/rultor #82
Comments
@original-brownbear can't we just modify the code of this project so that it starts to work with the newest Docker image from Rultor? |
@yegor256 yes you can, but adding the custom image really is just:
... while fixing it in the code is adding a line to load some Maven To me it looks like the effort of both approaches is pretty much the same, but fixing the code instead of creating the image still keeps the risk of this happening again around. |
@original-brownbear I think, the best approach is to start tagging Docker images in Rultor project. now there is just one tag: |
@yegor256 jup as I said above that would be the easiest :) |
@original-brownbear let's fix that in Rultor first? to make that tags available. also, let's make it possible to provide a tag in |
@yegor256 ticket created: yegor256/rultor#1033, this is a job for you I'm afraid. |
@original-brownbear I'll take a look |
@yegor256 now that the Rultor ticket is dealt with you should be able to simply do this (tried it with my fork): git push origin :refs/tags/1.61.3
git push origin --tags That should trigger the build of
in the |
@yegor256 it errored :/, can you try again ? Maybe just some apt-get timeout. |
@yegor256 damn it failed again but the reason is clear at least:
This is a thing of 10 minutes and fixes the situation for good. All other projects working fine currently can simply lock in their version of the image, but with this one we need to intervene manually. |
@original-brownbear maybe it's better to create a new tag 1.61.3.1, which will be cloned from 1.61.3 and will have no rultor build inside? |
@yegor256 honestly I only see downsides to this approach. It's the same amount of work and creates a much harder to maintain situation for this project. If you want to upgrade dependencies here, you'll have to either count on there being a working tag of Rultor for the upgraded dependencies or add a Dockerfile here afterall. |
@original-brownbear yes, I agree. let's go for it. |
@original-brownbear what's the status here? :) |
@amihaiemil I'm not sure, as far as I understand the above conversation this ticket is a valid bug and needs to be handled. |
@yegor256 see above 2 comments, you don't need anything from me here right? |
@original-brownbear no, we're good, it's a bug |
@original-brownbear thanks a lot for reporting, 15 mins added to your acc, pmt ID |
@amihaiemil it's yours, please go ahead |
@original-brownbear I'm new to docker (trying to get a decent grip of it now, since I have this task) I got the big picture of what needs to be done from your comments, but I have a few questions, if you're willing to answer: First, what I know is that docker acts like a virtual machine and using the Dockerfile you simply setup the build environment and then specify some commands (in our case What is a unclear to me is:
|
@amihaiemil let me try :) answers can be found here:
|
@amihaiemil I'm the issue author, you need to ask me to close it ;) |
@original-brownbear sure, thanks. I asked the PM out of habbit. |
@amihaiemil Thanks for your contribution, 30 mins was added to your account, payment ID is |
@original-brownbear the code made here contains a puzzle |
@original-brownbear the last puzzle |
@original-brownbear I think something like this: |
This is the explaination for #81 failing.
You can find more information on this here: yegor256/rultor#1018
In essence this is what happened here specific to this project:
Currently the project sees this error in the build when merging for example here:
This is caused by:
yegor256/rultor@61ce6e1#diff-137272550fb0d38c0e229b21d901e20eR6
... forcing the use of http://repo.maven.apache.org/maven2 on Rultor.
Rultor needed this to build on Maven 3.3.9, but here it evidently is causing trouble.
As explained in the above Rultor ticket, this needs to be fixed in projects breaking supplying their own Docker image, not in Rultor. Rultor's Dockerfile/Image is there to build Rultor itself and maintaining the build across multiple users of the same base Docker image is simply an impossible task.
That being said, looking at the last PR here, that the merge worked fine with: The below Dockerfile results in a working image. Building it and setting the image property in the .rultor.yml to wherever you push the build result, will resolve the issues you're experiencing with merging #81.
From https://hub.docker.com/r/yegor256/rultor/builds/bh4tyahxbh2bfcngvfxnyt4/:
82-ba434d45
/Aether.java:97-100: Clean unnecessary docker setup (src/docler/Dockerfile),... #84 (by Mihai Emil Andronache)The text was updated successfully, but these errors were encountered: