Thanks Michael and Jun for the quick turnaround. -Gary From: Michael O'Brien [mailto:[email protected]] Sent: Wednesday, August 08, 2018 1:49 PM To: [email protected]; Michael O'Brien <[email protected]>; Gary Wu <[email protected]> Cc: 'HU, JUN NICOLAS' <[email protected]> Subject: RE: [onap-discuss] [oom] Please merge these OOM changes
All 3 in - thanks Gary for the rebase on the merge conflict that merging #2 forced on #1 /michael From: [email protected]<mailto:[email protected]> <[email protected]<mailto:[email protected]>> On Behalf Of Michael O'Brien Sent: Wednesday, August 8, 2018 4:38 PM To: [email protected]<mailto:[email protected]>; [email protected]<mailto:[email protected]> Cc: 'HU, JUN NICOLAS' <[email protected]<mailto:[email protected]>> Subject: Re: [onap-discuss] [oom] Please merge these OOM changes Gary, Update: #2 and #3 were just merged by Jun. Hi, we raised this issue in the oom meet today - sorry for the delay - essentially all the amdocs committers are out on vac=Mike, vac=Mandeep, business-trip=Borislav (which are in some cases all the reviewers in a couple of your jiras) - I am a contributor not a committer in OOM - but I can assist with reviews. https://wiki.onap.org/display/DW/OOM+Meeting+Notes+-+2018-08-08 I would like to say a special THANK YOU to Jun for all the assistance getting patches in today. To get reviews in (3 in addition to yours) - Jun (Nicolas) Hu is merging everything reviewed now. I checked your reviews - it would help a bit more if in general test results were posted and (even though contributors cannot do a +1 verified) - add a comment that the patch is verified - then Jun can just hit +2 and merge the patch The first patch has no +1 or test results - add a reviewer that does a +1 and we can +2 /merge it The second and third patches have +1 and can be merged (#2 was just merged) Jun has +2'd the 3rd patch just now If Jun is unable to merge all three - we can see if Jerome or Alexis can get them merged. Committer list = https://git.onap.org/oom/tree/INFO.yaml Jun has just merged 58109 - patch 2 Thank you /michael From: [email protected]<mailto:[email protected]> <[email protected]<mailto:[email protected]>> On Behalf Of Gary Wu Sent: Wednesday, August 8, 2018 2:59 PM To: [email protected]<mailto:[email protected]> Subject: [onap-discuss] [oom] Please merge these OOM changes Hi OOM team, Can we please merge the following? They've been pending for more than a week with no updates. https://gerrit.onap.org/r/#/c/58053/ https://gerrit.onap.org/r/#/c/58109/ https://gerrit.onap.org/r/#/c/59279/ Until these are merged we'll be getting a ton of failures in our daily OOM staging deployment tests. Thanks, Gary This message and the information contained herein is proprietary and confidential and subject to the Amdocs policy statement, you may review at https://www.amdocs.com/about/email-disclaimer This message and the information contained herein is proprietary and confidential and subject to the Amdocs policy statement, you may review at https://www.amdocs.com/about/email-disclaimer -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#11770): https://lists.onap.org/g/onap-discuss/message/11770 Mute This Topic: https://lists.onap.org/mt/24232740/21656 Group Owner: [email protected] Unsubscribe: https://lists.onap.org/g/onap-discuss/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
