great job Simon David and Will. Given that we dont have github labels, this is very useful for the RM to pickup the prs to merge and for others to review pending.
~Rajani On Tue, May 3, 2016 at 9:47 AM, Simon Weller <swel...@ena.com> wrote: > All, > > Will mentioned a little project we've been working on to try and help get > a grasp on the open PR list. > You can access it here: http://acs:a...@acspr.ena.net > > Due to the lack of github labels currently, there's quite a bit of > regex/pattern matching involved in determining data sets, so please consume > the data with that in mind. The data gets refreshed every 20 mins in order > to avoid hitting the github api limit. > > The colour codes mean the following: > > Yellow - Merge Conflict > Gray - 2 or more LGTMs and needs CI run > Dark Green - 2 or more LGTMS and has completed a CI run, but hasn't been > reviewed for merge > Light Green - has been marked as ready to merge > > > RTM is determined by any of the following strings: "This is Ready to > Merge","This can be merged.","tag:mergeready" > Note that if someone places an "LGTM" in a comment or issue note, it will > be picked up. > CI Status is determined by any of the following strings: "CI RESULTS", > "ACS CI BVT" > > I hope you find this useful and please feel free to send me feedback and > bug reports. > > - Si > ________________________________________ > From: Will Stevens <williamstev...@gmail.com> > Sent: Monday, May 2, 2016 5:33 PM > To: dev@cloudstack.apache.org > Subject: ACS PRs Status - 2016/05/02 > > ACS PRs > > - 1433 - master (ready) > - 1230 - master (ready) > - 1350 - master (ready) > - 1463 - master (ready) > - 1502 - master (ready, pending comments) > - 1054 - master (ready, do grep) > - 1515 (1375) - 4.8 (ready, pending new 4.7 PR) > - 1477 - 4.7 (ready, pending verification) > - 1473 - 4.7 (ready) > - 1321 - 4.7 (ready) > - 1490 - 4.7 (ready) > - 1365 - 4.7 (ready) > - 1493 - master (ready, pending Jenkins) > - 1385 - master (ready, pending Jenkins) > - 956 - master (ready, pending Jenkins) > - 1516 - master (ready, pending Travis) > - 1455 - master (ready, pending LGTM) > - 1520 - master (ready, pending LGTM) > - 1376 - master (ready, pending 1515 conflict) > - 1374 - master (ready, pending 1515 + 1376 + rebase) > - 1489 - master (ready, pending LGTM, test upgrade path(s)) > - 1444 - master (ready, pending LGTM) > - 1280 - master (ready, pending LGTM) > - 1520 - master (pending LGTM) > - 1464 - master (pending LGTM) > - 1403 - master (pending LGTM) > - 1124 - master (today - pending CI) > - 1482 - 4.7 (pending LGTM) > - 1417 - master (marvin fails to import) > - 1466 - master (CI on cs2) > - 1436 - master (CI posted, needs review) > - 1304 - master (needs work) > - 1408 - master (pending verification) > - 1423 - master + svm (*pending CI) > - 1496 - master (pending CI) > - 1428 - master (pending ALL) > - 1450 - 4.7 (pending ALL) > - 1453 - master (pending ALL) > - 1331 - 4.7 (pending ALL) > - 1475 - 4.7 (pending ALL) > - 1297 - master (pending CI) > - 1410 - 4.7 (pending ALL) > - 1483 - 4.7 (pending ALL) > - 1470 - 4.7 (pending ALL) > - 1471 - 4.7 (pending ALL) > - 1472 - 4.7 (pending ALL) > - 1488 - master (pending CI) > - 1474 - 4.7 (pending ALL) > - 1486 - 4.7 (pending ALL) > - 1483 - 4.7 (pending ALL) > - 872 - master + svm (pending CI) > - 1456 - 4.7 (pending ALL) > - 1412 - 4.6 (pending ALL) > - 1406 - 4.6 (pending LGTM) > - 1378 - 4.6 (pending LGTM) > - 1491 - 4.7 (pending ALL) > - 1360 - master (pending LGTM) > - 1499 - master (pending ALL) > - 1371 - master + svm (pending ALL) > - 1497 - master (pending ALL) > - 883 - master (pending ALL) > - 956 - master (needs work) > - 880 - master (pending ALL) > - 866 - master (pending ALL) > - 846 - master (pending CI) > - 1244 - master (needs work) > - 1514 - 4.7 (pending ALL) > - 1511 - master (pending ALL) > - 1508 - master, 4.8? (pending ALL) > - 1351 - 4.7 (pending tests) > - 1523 - master (pending ALL) > - 1519 - 4.7 (pending ALL) > - 1440 - master (pending ALL) > - 1437 - master (pending HyperV CI) > - 985 - master (pending CI) > - 1269 - master (pending LGTM) > - 816 - master (pending ALL) > - 669 - master (pending ALL) > > NOTE: Merged into 4.7 and 4.8, but not yet fwd-merged to master > > ​Sorry for the delay on this status. I have a bunch of stuff pending that > will be getting merged, but I have a bit of a hiccup which I am working > through. > > I have merged some 4.7 PRs and have successfully forward merged them into > 4.8, but there are merge conflicts when forward merging into master. I > want to resolve the forward merge conflicts from 4.8 -> master before I get > all the ready PRs merged into master to try to keep things as simple as I > can. > > We continue to make good progress. I expect to see quite a bit of action > on the CI front this week, so hopefully we can continue to power through > these PRs. > > Thanks again to everyone who has been helping keep things moving. I know > it is busy times for everyone so all the support is very appreciated. > > I know the status of some of these items is wrong at this point, but I have > not had a chance to update everything. > > I wanted to give a big shout out to Simon Weller and David Mabry who have > been putting in a lot of work behind the scenes to help out with CI and > helping with some automation to help discover the status of the different > PRs. > > @Simon, I will let you elaborate if you would like. :) > > I may move this status update to Tuesdays as Mondays are pretty much a > write-off due to the influx of activity and it basically takes me all day > to work through the hundreds of emails. :) >
