Hi Xu Ran, On 29.03.2021 03:40, Xu Ran wrote: > Dear Opasiak and Catherine, > > I have noticed that the block thing of UUI’s merge into OOM is: > > https://gerrit.onap.org/r/c/oom/+/118930 > <https://protect2.fireeye.com/v1/url?k=1e966c5a-410d5543-1e97e715-000babff3793-f7ea0549a854ee69&q=1&e=26b28653-b4cf-4482-aed3-4b5732aa0935&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F118930> > https://gerrit.onap.org/r/c/oom/+/119124 > <https://protect2.fireeye.com/v1/url?k=43d07bfd-1c4b42e4-43d1f0b2-000babff3793-1e123c16f51c1874&q=1&e=26b28653-b4cf-4482-aed3-4b5732aa0935&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F119124> > > > > They’re all related to the new NLP server, however, since there’re some > several problems appearing in these two commits, we accept the result to > delay this new part to Istanbul release. We’ll abandon these two commits > and make further changes. And the H release plan of UUI will *NOT* > include this part.
No need to abandon. You can always squash the two and move ahead with a new patchset on one of the reviews;) > > As for the commit: > https://gerrit.onap.org/r/c/oom/+/119844 > <https://protect2.fireeye.com/v1/url?k=430831be-1c9308a7-4309baf1-000babff3793-15fb88f07d0c1278&q=1&e=26b28653-b4cf-4482-aed3-4b5732aa0935&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F119844> > > > > This commit only contains the changes of E2E slicing service which are > some minor ones. Yes + there is an implementation of NLP server introduced by commit 7643cc5b373b167000d676c48d741e830081f3ab but I expect that as NLP is delayed to Istanbul then this is just disabled by default? > We hope that OOM team can review this change and merge > it. If there’s any problem with the E2E part, we’ll be free to change. > And also, Doctor @Dong Wang is the sponsor of the new NLP part, he is > also available to response the problem of this part. > > We will attend today’s PTL meeting to make some more explanation, hope > we can reach an agreement. Gating seems to be fine on those patches. My only concern, that I expressed in the ticket is what is the rationale for not providing this container update before the deadline which is M3? Your REQ tickets say that you don't expect a need to update OOM configuration for this release... > > > B.R. > --------------------------------------------- > Xu Ran > China Mobile Research Institute > No.32 Xuanwumen west street,Xicheng District, Beijing 100053, China > > Mobile: +86 15010868144 > Email: [email protected] <mailto:[email protected]> > --------------------------------------------- > > On 03/27/2021 03:05，Krzysztof Opasiak via > lists.onap.org<[email protected]> > <mailto:[email protected]> wrote： > > Dear Catherine, > > On 26.03.2021 19:13, Lefevre, Catherine wrote: > > Dear OOM Team, > > David and I we are trying to understand what it is left from OOM > Backlog > that the project team(s) need to consider before the next PTL > call (3/29) > > We understand from the TSC call (3/25) - SO, SDNC and UUI had an > action > but it was not clear that there were other projects. > > Here is our understanding and the path to move forward. > > After the following items are solved then we need to stabilize the > release and no more accept any OOM code submission except > > for *versioning, documentation and show stoppers blocking E2E > Testing.* > > #1 UUI > > Remaining code submitted to change version is > "https://gerrit.onap.org/r/c/oom/+/119844 > > <https://protect2.fireeye.com/v1/url?k=46b413cc-192f2ad6-46b59883-000babff24ad-7502168be2f1c9a9&q=1&e=bb5ddae6-e181-4352-8e25-2bcfcc413086&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F119844>" > > I have reviewed it and did not notice anything wrong so I > believe it is > ready to merge > > > The UUI action was related to: > https://gerrit.onap.org/r/c/oom/+/118930 > > <https://protect2.fireeye.com/v1/url?k=96cdd1a2-c956e8bb-96cc5aed-000babff3793-7441e05435231c2a&q=1&e=26b28653-b4cf-4482-aed3-4b5732aa0935&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F118930> > https://gerrit.onap.org/r/c/oom/+/119124 > > <https://protect2.fireeye.com/v1/url?k=ae05fae4-f19ec3fd-ae0471ab-000babff3793-f6bee883b61eacbc&q=1&e=26b28653-b4cf-4482-aed3-4b5732aa0935&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F119124> > > Not the patch that you linked above. The one that you linked has been > created *yesterday* 40 mins before the TSC meeting which is 4 weeks > after the deadline for starting OOM review with new containers for > Honolulu release. > > In my personal opinion it's very unfair for other project teams to > accept such changes with a monthly delay. Think how much extra features > could CPS or DCAE Team implement if they had extra month for the > development... > > > The following items - we will follow-up if these are required > for RC1 > containers > > It does not look like new code but more modifications of config > to build > UUI Container. To be confirmed by *XU* > > https://gerrit.onap.org/r/c/oom/+/118930 > > <https://protect2.fireeye.com/v1/url?k=f0e13dae-af7a04b7-f0e0b6e1-000babff3793-5eaad4d8761cd6f9&q=1&e=26b28653-b4cf-4482-aed3-4b5732aa0935&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F118930> > > > <https://protect2.fireeye.com/v1/url?k=a47f0bfc-fbe432e6-a47e80b3-000babff24ad-e702636d9b0b5453&q=1&e=bb5ddae6-e181-4352-8e25-2bcfcc413086&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F118930> > > https://gerrit.onap.org/r/c/oom/+/119124 > > <https://protect2.fireeye.com/v1/url?k=fdd4954a-a24fac53-fdd51e05-000babff3793-55475913da8ddb84&q=1&e=26b28653-b4cf-4482-aed3-4b5732aa0935&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F119124> > > > <https://protect2.fireeye.com/v1/url?k=07a5321d-583e0b07-07a4b952-000babff24ad-6ac11a2aeca53cf1&q=1&e=bb5ddae6-e181-4352-8e25-2bcfcc413086&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F119124> > > > This is a *brand new* microservice that they are willing to add to ONAP > It's not a change in config, it's not any kind of bugfix, it's a brand > new Helm chart under UUI project. > > It was uploaded to gerrit a little bit after the deadline but it was > first sent to me via email so we decided to give it a chance. > Unfortunately when we start reviewing it more deeply multiple issues > has > been discovered but never fixed and I believe that we cannot compromise > the quality because of the deadline. > > > #2 SO > > We check the remaining open defects, > > SO-3584 - https://gerrit.onap.org/r/c/oom/+/119522/11 > > <https://protect2.fireeye.com/v1/url?k=ac9aa5c5-f3019cdc-ac9b2e8a-000babff3793-2c33c31e7d2a30ef&q=1&e=26b28653-b4cf-4482-aed3-4b5732aa0935&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F119522%2F11> > > > <https://protect2.fireeye.com/v1/url?k=37e382ff-6878bbe5-37e209b0-000babff24ad-4af727887ea4b80c&q=1&e=bb5ddae6-e181-4352-8e25-2bcfcc413086&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F119522%2F11> > > (can we merge the code)? > > > As stated in the review, the patch itself is fine (that's why it has +2 > from me and Sylvain) but we are waiting for a successful gating in > patch that you referred below as this is actually where SO will start > making use of that. If we merge it just now it would be just a blind > merge as without the patch above we don't know if it really works. > > > SO-3590 - https://gerrit.onap.org/r/c/oom/+/119309 > > <https://protect2.fireeye.com/v1/url?k=31cf5339-6e546a20-31ced876-000babff3793-a8eb7c361ddb5e27&q=1&e=26b28653-b4cf-4482-aed3-4b5732aa0935&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F119309> > > > <https://protect2.fireeye.com/v1/url?k=7c0eff7c-2395c666-7c0f7433-000babff24ad-9d6fd0d44e1a675a&q=1&e=bb5ddae6-e181-4352-8e25-2bcfcc413086&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F119309> > > (waiting for review) > > > Waiting for gating... > > > *SESHU, * > > SO-3473 > > <https://protect2.fireeye.com/v1/url?k=e1cd66c7-be565fdd-e1cced88-000babff24ad-9b6b971174a78770&q=1&e=bb5ddae6-e181-4352-8e25-2bcfcc413086&u=https%3A%2F%2Fjira.onap.org%2Fbrowse%2FSO-3473> > > - suggestion is to descope the remaining item of SO factoring to > Istanbul - too many code submitted and not yet reviewed. > > SO-3553 > > <https://protect2.fireeye.com/v1/url?k=d2d706b8-8d4c3fa2-d2d68df7-000babff24ad-710911c68ded527a&q=1&e=bb5ddae6-e181-4352-8e25-2bcfcc413086&u=https%3A%2F%2Fjira.onap.org%2Fbrowse%2FSO-3553> > > - https://gerrit.onap.org/r/c/oom/+/118331 > > <https://protect2.fireeye.com/v1/url?k=c5a23f69-9a390670-c5a3b426-000babff3793-88b5f0c7eadcec63&q=1&e=26b28653-b4cf-4482-aed3-4b5732aa0935&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F118331> > > > <https://protect2.fireeye.com/v1/url?k=b5a00398-ea3b3a82-b5a188d7-000babff24ad-2c0d2ce12129518d&q=1&e=bb5ddae6-e181-4352-8e25-2bcfcc413086&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F118331> > > - can we descope and shift it to Istanbul? > > CCSDK/SDNC > > https://gerrit.onap.org/r/c/oom/+/118284 > > <https://protect2.fireeye.com/v1/url?k=e0647eb3-bfff47aa-e065f5fc-000babff3793-aa115ba8a23e365d&q=1&e=26b28653-b4cf-4482-aed3-4b5732aa0935&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F118284> > > > <https://protect2.fireeye.com/v1/url?k=b0398acd-efa2b3d7-b0380182-000babff24ad-4cd8bfca6accb446&q=1&e=bb5ddae6-e181-4352-8e25-2bcfcc413086&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F118284> > > -- (version update + 1 big fix) Dan T to review the comments > from Morgan > > > I believe that Dan has already fixed the issue reported by Morgan. We > are waiting for gating to confirm this. > > > https://gerrit.onap.org/r/c/oom/+/117808 > > <https://protect2.fireeye.com/v1/url?k=688fde52-3714e74b-688e551d-000babff3793-0ab6d8c12a2d43b6&q=1&e=26b28653-b4cf-4482-aed3-4b5732aa0935&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F117808> > > > <https://protect2.fireeye.com/v1/url?k=c5400328-9adb3a32-c5418867-000babff24ad-e8cac77324f766a1&q=1&e=bb5ddae6-e181-4352-8e25-2bcfcc413086&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F117808> > > -- verified job ok, ready for review > > > I need to follow up with Alexander on this patch as I missed his > comment. Sorry for that. > > > CPS - OK For Honolulu > > https://gerrit.onap.org/r/c/oom/+/118995 > > <https://protect2.fireeye.com/v1/url?k=5bf5e645-046edf5c-5bf46d0a-000babff3793-127381b38cb02780&q=1&e=26b28653-b4cf-4482-aed3-4b5732aa0935&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F118995> > > > <https://protect2.fireeye.com/v1/url?k=1ce1defb-437ae7e1-1ce055b4-000babff24ad-f02eadf288773474&q=1&e=bb5ddae6-e181-4352-8e25-2bcfcc413086&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F118995> > > ) verified job ok, ready for review > > > It's marked as WIP as Bruno is working to address comments from > previous > revision. Nevertheless changes would not require releasing a new > container image. They just need to switch to use certInitializer for > their https termination on ingress. > > Considering the progress that they made in this release, their nice > cooperation with us almost from the beginning of the release and amount > of comments that they addressed I believe that it's fair to take this > patch even as a bugfix after RC0 as it would affect OOM only > > > OOF - OK For Honolulu > > https://gerrit.onap.org/r/c/oom/+/113414 > > <https://protect2.fireeye.com/v1/url?k=09e47a5f-567f4346-09e5f110-000babff3793-9a434a737a2ed37b&q=1&e=26b28653-b4cf-4482-aed3-4b5732aa0935&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F113414> > > > <https://protect2.fireeye.com/v1/url?k=26742d43-79ef1459-2675a60c-000babff24ad-c6374855ff7d5692&q=1&e=bb5ddae6-e181-4352-8e25-2bcfcc413086&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F113414> > > verified job ok, ready for review > > > It's not ready, it doesn't work. I got some hint from Krishna how to > fix > it so I'll give it a try this evening if it works we'll merge if not we > just postpone it. It's not a functional change it's just enabling > logging to STDOUT for OOF so I believe that it can be merge even > after RC0 > > > Holmes - What's level of confidence that it will work? Can we > move this > to Istanbul. > > https://gerrit.onap.org/r/c/oom/+/117395 > > <https://protect2.fireeye.com/v1/url?k=b514f4cc-ea8fcdd5-b5157f83-000babff3793-2d475082764a36ff&q=1&e=26b28653-b4cf-4482-aed3-4b5732aa0935&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F117395> > > > <https://protect2.fireeye.com/v1/url?k=9235dd10-cdaee40a-9234565f-000babff24ad-c6d9314de228eb06&q=1&e=bb5ddae6-e181-4352-8e25-2bcfcc413086&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F117395> > > > Guangrong Fu made recently some fixes but due to bad certs in DMAAP we > haven't had a chance to see it in the action. After we have gating > results we'll be able to say anything more. > > > AAI - OK For Honolulu (required for Certification) > > https://gerrit.onap.org/r/c/oom/+/118248 > > <https://protect2.fireeye.com/v1/url?k=e4592b0a-bbc21213-e458a045-000babff3793-aa7a3646d133fee6&q=1&e=26b28653-b4cf-4482-aed3-4b5732aa0935&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F118248> > > > <https://protect2.fireeye.com/v1/url?k=ef44ab91-b0df928b-ef4520de-000babff24ad-7bcfb89218b9c689&q=1&e=bb5ddae6-e181-4352-8e25-2bcfcc413086&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F118248> > > - verified job ok, ready for review > > > Waiting for gating results... > Code-wise looks fine. If it passes in the gating I'll merge it. > > > Multicloud - submitted before RC0 > > https://gerrit.onap.org/r/c/oom/+/119125 > > <https://protect2.fireeye.com/v1/url?k=e5ff3623-ba640f3a-e5febd6c-000babff3793-af0f070c60c1f0ea&q=1&e=26b28653-b4cf-4482-aed3-4b5732aa0935&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F119125> > > > <https://protect2.fireeye.com/v1/url?k=ee9c325a-b1070b40-ee9db915-000babff24ad-e9894bd205478539&q=1&e=bb5ddae6-e181-4352-8e25-2bcfcc413086&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F119125> > > - verified job ok, ready for review > > > Already reviewed;) > As I'm coauthor of this patch, Sylvain is the one to merge it but he is > waiting for jenkins job review before doing that. Code-wise looks fine > so as soon as we have results from jenkins I hope it can be merged. > > > *OOM Team*- do we miss anything else? THANK YOU > > > There is one more bugfix from the SDNC (CCSDK) team that has been > submitted yesterday: > https://gerrit.onap.org/r/c/oom/+/119847 > > <https://protect2.fireeye.com/v1/url?k=53d0af33-0c4b962a-53d1247c-000babff3793-ebcb86fc8c9b8cc1&q=1&e=26b28653-b4cf-4482-aed3-4b5732aa0935&u=https%3A%2F%2Fgerrit.onap.org%2Fr%2Fc%2Foom%2F%2B%2F119847> > > I'm working with them to find the best solution but I'm sure we'll sort > this out quickly especially that it's just a change in the helm charts > that is easy to gate. > > Best regards, > -- > Krzysztof Opasiak > Samsung R&D Institute Poland > Samsung Electronics > > > > > -- Krzysztof Opasiak Samsung R&D Institute Poland Samsung Electronics -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#23077): https://lists.onap.org/g/onap-discuss/message/23077 Mute This Topic: https://lists.onap.org/mt/81584261/21656 Group Owner: [email protected] Unsubscribe: https://lists.onap.org/g/onap-discuss/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
