Skip to content

Conversation

@mrunalpagnis
Copy link
Member

@mrunalpagnis mrunalpagnis commented Apr 28, 2020

DEMO -

The e2e tests have been run on k8s cluster with version 1.14 and 1.15.

k8s 1.15 non-oke cluster nodes - 

mpagnis@mpagnis-mac  ~  k get nodes
NAME                                                     STATUS   ROLES    AGE   VERSION
k8s-master-ad1-0.k8smasterad1.kubernetes.oraclevcn.com   Ready    master   27d   v1.15.11
k8s-master-ad2-0.k8smasterad2.kubernetes.oraclevcn.com   Ready    master   27d   v1.15.11
k8s-master-ad3-0.k8smasterad3.kubernetes.oraclevcn.com   Ready    master   27d   v1.15.11
k8s-worker-ad1-0.k8sworkerad1.kubernetes.oraclevcn.com   Ready    node     27d   v1.15.11
k8s-worker-ad2-0.k8sworkerad2.kubernetes.oraclevcn.com   Ready    node     27d   v1.15.11
k8s-worker-ad3-0.k8sworkerad3.kubernetes.oraclevcn.com   Ready    node     27d   v1.15.11

k8s 1.14 non-oke cluster nodes - 
mpagnis@mpagnis-mac  ~  k get nodes
NAME                                                     STATUS   ROLES    AGE     VERSION
k8s-master-ad1-0.k8smasterad1.kubernetes.oraclevcn.com   Ready    master   9m50s   v1.14.10
k8s-master-ad2-0.k8smasterad2.kubernetes.oraclevcn.com   Ready    master   9m21s   v1.14.10
k8s-master-ad3-0.k8smasterad3.kubernetes.oraclevcn.com   Ready    master   9m34s   v1.14.10
k8s-worker-ad1-0.k8sworkerad1.kubernetes.oraclevcn.com   Ready    node     8m34s   v1.14.10
k8s-worker-ad2-0.k8sworkerad2.kubernetes.oraclevcn.com   Ready    node     8m22s   v1.14.10
k8s-worker-ad3-0.k8sworkerad3.kubernetes.oraclevcn.com   Ready    node     8m15s   v1.14.10
Ran 13 of 13 Specs in 1632.420 seconds
SUCCESS! -- 13 Passed | 0 Failed | 0 Pending | 0 Skipped
PASS

Ginkgo ran 1 suite in 27m20.864192183s
Test Suite Passed

The logs of above results have been attached.
k8s115_ccm_installed_080_rbac.txt
k8s114_ccm_installed_080_rbac.txt

Flexvolume driver master and worker -

 mpagnis@mpagnis-mac  ~  ks get ds | grep flexvolume
oci-flexvolume-driver-master   3         3         3       3            3           node-role.kubernetes.io/master=   26m
oci-flexvolume-driver-worker   3         3         3       3            3           <none>                            26m

Volume provisioner -

 mpagnis@mpagnis-mac  ~  ks get deploy | grep oci
oci-block-volume-provisioner         1/1     1            1           21m
oci-file-system-volume-provisioner   1/1     1            1           21m

Volume provisioner demo -

 mpagnis@mpagnis-mac  ~/oke5754  kubectl create -f claim_flex.yaml
persistentvolumeclaim/oci-flex-claim created
 mpagnis@mpagnis-mac  ~/oke5754  kubectl create -f pod_flex.yaml
pod/app created
 mpagnis@mpagnis-mac  ~/oke5754  k get pvc
NAME             STATUS   VOLUME                                                                              CAPACITY   ACCESS MODES   STORAGECLASS   AGE
oci-flex-claim   Bound    ocid1.volume.oc1.phx.abyhqljsno7k2a7hu4dsm2ekyfdmtsudm5dxag7ajiag2sc5aygiujm6stua   50Gi       RWO            oci            2m21s
 mpagnis@mpagnis-mac  ~/oke5754  k get po
NAME   READY   STATUS    RESTARTS   AGE
app    1/1     Running   0          112s

All oke pods running in kube-system namespaces -

 mpagnis@mpagnis-mac  ~  ks get po | grep oci
oci-block-volume-provisioner-5746c94584-ndj26                                    1/1     Running   0          22m
oci-cloud-controller-manager-2k5kd                                               1/1     Running   0          31h
oci-cloud-controller-manager-6t5xl                                               1/1     Running   0          31h
oci-cloud-controller-manager-npf9f                                               1/1     Running   0          31h
oci-file-system-volume-provisioner-65d44bd4bd-27jll                              1/1     Running   0          21m
oci-flexvolume-driver-master-2wjgg                                               1/1     Running   0          28m
oci-flexvolume-driver-master-hp9zg                                               1/1     Running   0          28m
oci-flexvolume-driver-master-q9qlb                                               1/1     Running   0          28m
oci-flexvolume-driver-worker-4dxb4                                               1/1     Running   0          28m
oci-flexvolume-driver-worker-c7lbq                                               1/1     Running   0          28m
oci-flexvolume-driver-worker-p8fsr                                               1/1     Running   0          28m

Below changes have been picked up -

  • Change documentation as per new release. fix rbac manifest and readme compatibility matrix
  • add/delete/update backend servers instead of updating the entire backendsets
  • Fix the documentation around how to run e2e tests for CCM
  • Fix issues with ccm e2e tests.
  • CCM is not honouring oci-load-balancer-connection-idle-timeout annotation during update event
  • Change log level to debug for config file not found on worker node
  • Fixing CCM build target
  • Resolve dependency conflict for Sirupsen by bumping up the version of docker/docker
  • CCM should ignore nodes for backends that aren't setup yet.
  • move to go mod
  • Avoid CCM panic due to index out of range
  • update oci-go-sdk
  • correction to boilerplate header validation and remove e2e test automation
  • Enable provisioning BV from another BV
  • Deploy single container with CCM and FVP
  • Cert rotation should avoid listener recreation
  • Fix test to avoid appending secret name to listeners name

@mrunalpagnis mrunalpagnis force-pushed the sync_with_internal branch 2 times, most recently from 5e408a4 to a6f10a4 Compare April 28, 2020 11:54
@mrunalpagnis mrunalpagnis force-pushed the sync_with_internal branch 2 times, most recently from d46e73a to dd8d3a6 Compare May 4, 2020 05:33
Copy link
Contributor

@IamPrvn IamPrvn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@mrunalpagnis
Copy link
Member Author

Support for 1.16 tested -

 mpagnis@mpagnis-mac  ~  k get no
NAME        STATUS   ROLES    AGE    VERSION
k8smaster   Ready    master   7d3h   v1.16.8
k8sworker   Ready    <none>   7d3h   v1.16.8
mpagnis@mpagnis-mac  ~/src/github.com/oracle/oci-cloud-controller-manager   sync_with_internal  make run-ccm-e2e-tests-local > $HOME/releaseOSS/testresuls116.txt

mpagnis@mpagnis-mac  ~  tail -f releaseOSS/testresuls116.txt
    /Users/mpagnis/src/github.com/oracle/oci-cloud-controller-manager/test/e2e/cloud-provider-oci/load_balancer.go:476
------------------------------
Jul 10 11:45:37.658: INFO: Running AfterSuite actions on all node

Ran 17 of 17 Specs in 1220.318 seconds
SUCCESS! -- 17 Passed | 0 Failed | 0 Pending | 0 Skipped
PASS

Ginkgo ran 1 suite in 20m25.685467543s
Test Suite Passed

testresuls116.txt

@mrunalpagnis mrunalpagnis force-pushed the sync_with_internal branch 2 times, most recently from 641f4b8 to 794328a Compare July 10, 2020 10:38
@mrunalpagnis
Copy link
Member Author

Tests pass on 1.17 Kubernetes Cluster as well.

Jul 13 12:46:08.355: INFO: Running AfterSuite actions on all node

Ran 17 of 17 Specs in 1029.997 seconds
SUCCESS! -- 17 Passed | 0 Failed | 0 Pending | 0 Skipped
PASS

Ginkgo ran 1 suite in 17m15.774283155s
Test Suite Passed

testresults117.txt

@mrunalpagnis mrunalpagnis linked an issue Jul 29, 2020 that may be closed by this pull request
Enable provisioning BV from another BV

Fix review comments

Fix review comments

Add unit tests

Deploy single container with CCM and FVP

Fix polling error

fix review comments

Refactored to move command to app package, replace fatalf refrences with error, remove nodeName reference

Cert rotation should avoid listener recreation

fix review comments

match params with called func signature

Signed-off-by: Steve Brunton <steve.brunton@oracle.com>

Fix test to avoid appending secret name to listeners name

correction to boilerplate header validation and remove e2e test automation

OKE-8694 - CCM is not honouring oci-load-balancer-connection-idle-timeout annotation during update event

OKE-8991: add/delete/update backend servers instead of updating the entire backendsets

JIRA: OKE-9388 - [TKM] [TKW] Migrate oci-cloud-controller-manager to use Go SDK v17.0.0 or greater.

OKE-9846: 1. CCM is calling update listener in time of new events
2. Adding logs to show the difference between the spec and LB in time of UpdateListener and UpdatebackendSet 3. Add Backend name in Backend creation log

OKE-9860 - Remove CCM internal rate limiting by adding a backward compatible option

JIRA: OKE-10232 - Investigate and fix the storage tests in CCM repo

JIRA: OKE-10232 - Investigate and fix the storage tests in CCM repo

JIRA: OKE-9860 Remove CCM internal rate limiting by adding a backward compatible option. refactoring NewRateLimiter in cloud_provider_framework.go

correct the copyright

remove lint error

Add documentation for 1.16 support and missing FVD and FVP installation

1.17 supported change in readme

JIRA: OKE-8771 Change documentation as per new release. remove real ocids

make changes to push to oss repo
@mrunalpagnis mrunalpagnis merged commit c5b87ae into master Aug 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants