New "Weather Item" process with full preparation on topic and verification instead of breaking the world
(RELENG-101)
|
|
| Status: | Resolved |
| Project: | releng |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Sub-task | Priority: | Medium |
| Reporter: | Michael Vorburger | Assignee: | Luis Gomez |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||
| Description |
|
Before we have RELENG-102, we want to just use the existing integration-multipatch-test-fluorine for what we are after in the parent JIRA: https://jenkins.opendaylight.org/releng/job/integration-multipatch-test-fluorine/ and http://docs.opendaylight.org/projects/releng-builder/en/latest/jenkins.html#jenkins-job-templates. What we are missing is be able to run builds on that job without -Pq so that we can catch not only compile time impacts, but also broken tests and other failures (such as the FindBugs failure which Luis volunteered to do something about this in today's integration call. Either just drop the -Pq or introduce a new keyword. This is better than a new parameter, because not veeryone has access to https://jenkins.opendaylight.org/releng/job/integration-multipatch-test-fluorine/build?delay=0sec. |
| Comments |
| Comment by Luis Gomez [ 27/Apr/18 ] |
|
This should work: https://git.opendaylight.org/gerrit/#/c/71492/ |
| Comment by Michael Vorburger [ 30/Apr/18 ] |
|
Thanks! I'll test this ASAP, for |
| Comment by Luis Gomez [ 30/Apr/18 ] |
|
Michael, fyi this is the list of patches I tried (based on the topic binding-tlc-rpc): mdsal=55/69355/33:62/69362/30,controller=05/71205/6,aaa=19/71219/1,netconf=22/71222/2,bgpcep=27/71227/4 Next step could be either:
List of projects we need a patch (in dependency order): |
| Comment by Michael Vorburger [ 30/Apr/18 ] |
|
ecelgp how about we track what you posted above in But did that REALLY run an autorelease with FindBugs?? If so, then this means we can just merge the controller and mdsal changes, as the job should prove if any impact or not. I'm just sceptical if there really is no impact, because I don't entirely trust our new job from From what (little) I understand here, my integration-multipatch-test-fluorine/18/ JUST built mdsal and controller - NOT all of autorelease WITH those two patches - is that right? How can I get it to do what we want? |
| Comment by Luis Gomez [ 30/Apr/18 ] |
|
That is correct multi-patch job only builds whatever you tell in the patch list + distribution at the end. Building all projects is more like a new AR job we require. |
| Comment by Michael Vorburger [ 30/Apr/18 ] |
|
ecelgp I'm trying "mdsal:61/66361/6,controller:60/66360/5,genius,netvirt" on (currently queued) integration-multipatch-test-fluorine/19/, hoping that means "please try mvn install with those 2 patches (making the breaking change), and then genius / netvirt master - to see if those still pass or need patches" - will that syntax work? Or if does not, would it be possible to make something like that work? |
| Comment by Luis Gomez [ 30/Apr/18 ] |
|
I can start looking at that job by the end of this week but maybe for the next immediate breakage we can just use the multi-patch job. |
| Comment by Luis Gomez [ 30/Apr/18 ] |
|
I agree we can close this ticket, I already moved performed test to |
| Comment by Michael Vorburger [ 30/Apr/18 ] |
|
Hang on, but then this approach is not going work for the short term, like we had hoped, and we would have to have a new job (RELENG-102) rather sooner than later, right? What I really want is a job to SEE which projects FAIL and are affected. I don't want to have to "trust" projects to raise patches IFF they are affected. For a better Weather Item process (parent issue), we need to validate IFF projects are affected, and test their proposed patches. |
| Comment by Luis Gomez [ 30/Apr/18 ] |
|
multi-patch job just verifies individual patches entered manually, it does not detect if a downstream project breaks because an upstream patch or similar. That will require a new job in AR. |
| Comment by Luis Gomez [ 30/Apr/18 ] |
|
OK, it is been noted (I have never tested this way before) that if a project is mentioned in the multi-patch build without any patch, e.g. "mdsal:61/66361/6,controller:60/66360/5,genius,netvirt", then the job will use latest code in branch. So a new approach here is to enter the list of projects in AR and see which one breaks, i.e. needs a patch. |
| Comment by Michael Vorburger [ 02/May/18 ] |
|
> approach here is to enter the list of projects in AR and see which one breaks, i.e. needs a patch That's what The Bot now does. Let us close this sub task, as the goal originally set up for it here has been achieved. More about specific builds in the respective TSC ticket, and particular tooling issues in new sub-tasks, such as |