-
Notifications
You must be signed in to change notification settings - Fork 83
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
refactor: update test projects and build references #1200
refactor: update test projects and build references #1200
Conversation
b92a9a9
to
e041ab3
Compare
|
Seems up again. But I don't find a way to restart failing builds on https://ci.eclipse.org/repairnator/job/repairnator-ci/. @javierron any idea? |
@monperrus I've restarted the failing builds on all pull requests |
Hey @andre15silva, I want to rebase these changes into my PR #1215. What's left to do here? |
I'm waiting for the projects mentioned in #1195 to be moved. When that is done, it's a matter of changing the references in the tests (maybe some builds need to be triggered again too). @javierron |
@andre15silva do we need to move all projects or can we proceed as is, with the ones which have already been moved? if we can proceed, we would unwip and merge this one, and @javierron would rebase #1195 afterwards. WDYT? |
Yes, we can merge as is and leave the rest of the changes pending. I'll rebase this PR and mark it ready. |
e041ab3
to
15f2957
Compare
perfect, and @javierron will merge it afterwards |
15f2957
to
ae68530
Compare
ae68530
to
c8219a7
Compare
Signed-off-by: André Silva <andre15andre@hotmail.com>
c8219a7
to
4669de4
Compare
@javierron do you know what might be causing the build failure in The first build failed with test cases returning 404 errors, and the two others timed-out. |
I've fixed those 404s in my PR, the timeouts might be related. I will cherry-pick my commit to this branch to see if we can get the tests passing. |
Signed-off-by: Javier Ron <javierron90@gmail.com>
@andre15silva I fixed the It seems like travis-ci.org is removing old builds that our tests depend on (example). I think the best solution is to create our own builds (maybe in repairnator/failingProject ?) that fulfill the tests' requirements. |
I think so too. I'm doing that change and adding it to this PR. Thanks! |
Signed-off-by: André Silva <andre15andre@hotmail.com>
Signed-off-by: André Silva <andre15andre@hotmail.com>
Signed-off-by: André Silva <andre15andre@hotmail.com>
Signed-off-by: André Silva <andre15andre@hotmail.com>
@javierron There are two issues right now:
WDYT we should do with these? The latter issue doesn't affect (currently) the tests, as the builds are still available. The first one needs a change tho. |
@andre15silva I believe the plugins in the test project should be outside the |
Signed-off-by: André Silva <andre15andre@hotmail.com>
It was that indeed. Fixed! Thanks!
I don't have the permission to add you, needs to be @monperrus |
Thanks @andre15silva! We're finally green again. 🚀 Since this is blocking our other PRs, let's merge these changes and take care of the 2 remaining builds in a separate PR. |
Thanks a lot! |
Waiting for project moves from #1195