Agenda


Reviews must have healthcheck and a get pods output to get reviews - ideally with testing results on the review or Jira

# deploy robot pod in addition to your pod tree
ubuntu@a-ons0-master:~/oom/kubernetes/robot$ sudo ./ete-k8s.sh onap health
...your pods only need to pass
Basic DMAAP Message Router Health Check                               | PASS |

ubuntu@a-ons0-master:~$ kubectl get pods --all-namespaces
NAMESPACE     NAME                                                    READY     STATUS             RESTARTS   AGE
kube-system   heapster-7b48b696fc-cs22c                               1/1       Running            0          1d
kube-system   kube-dns-6655f78c68-dfjsf                               3/3       Running            0          1d
kube-system   kubernetes-dashboard-6f54f7c4b-2rzlx                    1/1       Running            0          1d
kube-system   monitoring-grafana-7877679464-cxbns                     1/1       Running            0          1d
kube-system   monitoring-influxdb-64664c6cf5-trggz                    1/1       Running            0          1d
kube-system   tiller-deploy-78db58d887-v8nbd                          1/1       Running            0          1d
# your components pods - and any dependencies - like dmaap,aaf - we will account for known issues unrelated to your merge
onap          onap-aai-aai-babel-86bd78c6b4-dgdrn                     2/2       Running            0          21h
onap          onap-aai-aai-bf7cdb594-6vcjq                            0/1       Init:0/1           96         17h
onap          onap-aai-aai-cassandra-0                                1/1       Running            2          21h


Reviews should verify nodeport collision - either deploy all of onap - or verify OOM NodePort List

Reviews need to have all committers to get reviewed faster (Alexis de Talhouët , Borislav Glozman , James MacNider, Mike Elliott ) and contributors like myself (Michael O'Brien), Prudence Au and integration team members affected Gary Wu Brian Freeman Yang Xu


  • No labels

2 Comments

  1. Q) 4th review - https://gerrit.onap.org/r/#/c/79652/ looks good - just need a 2nd review - Sylvain Desbureaux will also check an issue on the CD side


    for https://gerrit.onap.org/r/#/c/81992/ for Dominic Lunanuova - just rebase the merge conflict - after this we will merge it - thanks


    q) yang: docker manifest with oom 

    see  OOM Meeting Notes - 2019-03-06

    Sylvain Desbureaux looking into master and snapshot versions

    We will wait for Mike to return after the 25th

    Jiras for Orange and Integration team for now

    really required for the RC0-3 stage - ie: portals snapshot version needs to be looked at

    master in mixed snapshot/release state needs to be fixed first

    docker staging does not always have latest images

    Reduction effort between Casablanca and Dublin

    for example SO has a higher CMR than master (not cherry picked from 3.0.0/3.0.1) - see SO pods

    Decision until the 25th : use the current manifest (from integration repo) to be able to get the staging versions (see middle column in Sylvain Desbureaux's table - 


    Check 

    https://hub.docker.com/r/oomk8s/readiness-check/tags

    review image uploads Mandeep Khindawas doing it - we will need to take over uploads and retrofit of the latest tags 

    Also check Sylvain Desbureaux's page for "actual" vm size 6xxx not the 2xx posted on dockerhub - need a full check

    Michael O'Brien todo: get the DockerFile for each repo in dockerhub and post a page until we remove all these from oomk8s - onap

    Q)

    Dominic Lunanuova help with merge conflict - 

    DMAAP-866 - Getting issue details... STATUS

    DMAAP-904 - Getting issue details... STATUS

    https://gerrit.onap.org/r/#/c/81988/ was merged before https://gerrit.onap.org/r/#/c/81992/

    in the future you can put a -1 (don't merge) on all reviews and put them +1 in sequence to merge them 1 at a time to avoid a merge conflict - especially on renamed files - which this review had - thanks Dominic


    1. I messed up resolving the merge conflict so abandoned https://gerrit.onap.org/r/#/c/81992/  and have a new commit https://gerrit.onap.org/r/#/c/82150/

      I put the +1 as suggested.  Any additional help to get this merged would be appreciated.