Re: [onap-discuss] [oom] Please merge these OOM changes
Thanks Michael and Jun for the quick turnaround. -Gary From: Michael O'Brien [mailto:frank.obr...@amdocs.com] Sent: Wednesday, August 08, 2018 1:49 PM To: onap-discuss@lists.onap.org; Michael O'Brien ; Gary Wu Cc: 'HU, JUN NICOLAS' 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: onap-discuss@lists.onap.org<mailto:onap-discuss@lists.onap.org> mailto:onap-discuss@lists.onap.org>> On Behalf Of Michael O'Brien Sent: Wednesday, August 8, 2018 4:38 PM To: onap-discuss@lists.onap.org<mailto:onap-discuss@lists.onap.org>; gary.i...@huawei.com<mailto:gary.i...@huawei.com> Cc: 'HU, JUN NICOLAS' mailto:jh2...@att.com>> 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: onap-discuss@lists.onap.org<mailto:onap-discuss@lists.onap.org> mailto:onap-discuss@lists.onap.org>> On Behalf Of Gary Wu Sent: Wednesday, August 8, 2018 2:59 PM To: onap-discuss@lists.onap.org<mailto:onap-discuss@lists.onap.org> 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: onap-discuss+ow...@lists.onap.org Unsubscribe: https://lists.onap.org/g/onap-discuss/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
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: onap-discuss@lists.onap.org On Behalf Of Michael O'Brien Sent: Wednesday, August 8, 2018 4:38 PM To: onap-discuss@lists.onap.org; gary.i...@huawei.com Cc: 'HU, JUN NICOLAS' 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: onap-discuss@lists.onap.org<mailto:onap-discuss@lists.onap.org> mailto:onap-discuss@lists.onap.org>> On Behalf Of Gary Wu Sent: Wednesday, August 8, 2018 2:59 PM To: onap-discuss@lists.onap.org<mailto:onap-discuss@lists.onap.org> 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 <https://www.amdocs.com/about/email-disclaimer> -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#11769): https://lists.onap.org/g/onap-discuss/message/11769 Mute This Topic: https://lists.onap.org/mt/24232740/21656 Group Owner: onap-discuss+ow...@lists.onap.org Unsubscribe: https://lists.onap.org/g/onap-discuss/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
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: onap-discuss@lists.onap.org On Behalf Of Gary Wu Sent: Wednesday, August 8, 2018 2:59 PM To: onap-discuss@lists.onap.org 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 <https://www.amdocs.com/about/email-disclaimer> -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#11767): https://lists.onap.org/g/onap-discuss/message/11767 Mute This Topic: https://lists.onap.org/mt/24232740/21656 Group Owner: onap-discuss+ow...@lists.onap.org Unsubscribe: https://lists.onap.org/g/onap-discuss/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
[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 -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#11765): https://lists.onap.org/g/onap-discuss/message/11765 Mute This Topic: https://lists.onap.org/mt/24232740/21656 Group Owner: onap-discuss+ow...@lists.onap.org Unsubscribe: https://lists.onap.org/g/onap-discuss/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
