OOM Meeting Notes - 2021-03-24
new rules for merging:
if component part of core (AAI,DMAAP,SDC,SDNC,SO,VID), we mandate 100% healtcheck and e2e tests
if not, 1 HC and 1 e2e test is allowed to fail
we prefer to do commit msg update / rebase instead of "oom_redeploy" in order to keep all results.
RC0 Honolu: @Sylvain Desbureaux @Krzysztof Opasiak
still some patches that are candidate to merge: https://gerrit.onap.org/r/q/hashtag:%22honoluluCandidate%22+(status:open%20OR%20status:merged)
18 to go for now
some "outstanding" review that are still on the queue:
SO → known issue to be solved https://gerrit.onap.org/r/c/oom/+/119309
SDNC → waiting for 100% healtcheck + e2e https://gerrit.onap.org/r/c/oom/+/118284
VFC → a pod is crashing https://gerrit.onap.org/r/c/oom/+/119012
UUI → -2 from @Krzysztof Opasiak https://gerrit.onap.org/r/c/oom/+/119124
HOLMES → AAF and nodeport issue https://gerrit.onap.org/r/c/oom/+/117395
Policy (?) → HC issue https://gerrit.onap.org/r/c/oom/+/119102
DMAAP → if not, certificate issue on Message Router... https://gerrit.onap.org/r/c/oom/+/118602
MR Guilin: @Sylvain Desbureaux @Fiachra Corcoran
3 patches that are left (certs only):
CDS: https://gerrit.onap.org/r/c/oom/+/118917 → waiting for good gate
VID: https://gerrit.onap.org/r/c/oom/+/119128 → waiting for good gate
DMAAP MR: https://gerrit.onap.org/r/c/oom/+/118913 → issue with dmaap chart
Prometheus: @Marat Salakhutdinov @Lukasz Grech
Proposal from Lukasz in order to have a single Service Monitor for spring boot actuator based components (see last week) with the right labels
merge request from Marat: https://gerrit.onap.org/r/c/oom/+/119648
these two proposals seems to be compatible and complentary
DCAE: @Ajay Deep Singh
DCAE is moving to helm based components
basic ones are already in the pipe or merged (the static ones)
now the works on more dynamic components
and some needs specific topics created by dmaap bc
ask for help in order to create a template doing that (DMAAP-1561: [DMAAP][OOM] Move bc provisioning job to common chart/templateClosed)
Ingress part enhancement: Patrik Romsak
https://github.com/onap/oom/blob/master/kubernetes/common/common/templates/_ingress.tpl
it's nginx only on SSL part for now
proposal:
Add a global value for ingress type (defaulted to nginx)
if value is empty, default also to nginx
look at https://github.com/onap/oom/blob/master/kubernetes/common/common/templates/_storage.tpl#L35 for engine selection overriding