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 <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' <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> 
<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]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to