Skip to content

Add deployment test#8

Open
lebauce wants to merge 1 commit into
skydive-project:masterfrom
lebauce:add-tests
Open

Add deployment test#8
lebauce wants to merge 1 commit into
skydive-project:masterfrom
lebauce:add-tests

Conversation

@lebauce

@lebauce lebauce commented Jun 9, 2020

Copy link
Copy Markdown
Member

No description provided.

@lebauce

lebauce commented Jun 9, 2020

Copy link
Copy Markdown
Member Author

run operator-tests

@lebauce

lebauce commented Jun 9, 2020

Copy link
Copy Markdown
Member Author

ping @sa-cloud

Comment thread tests/run-tests.sh
# Create a general Skydive resource
kubectl create -f deploy/crds/charts.helm.k8s.io_v1alpha1_skydive_cr.yaml
# Create Skydive as netflow collector resource
kubectl create -f deploy/crds/charts.helm.k8s.io_v1alpha1_netflowcollector_cr.yaml

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Only one of the _cr.yaml should be installed at a time. Each of them deploys a Skydive, and only one Skydive instance can run at a cluster (unless this was fixed)

Comment thread tests/run-tests.sh
# Create Skydive as netflow collector resource
kubectl create -f deploy/crds/charts.helm.k8s.io_v1alpha1_netflowcollector_cr.yaml

# Do some tests...

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

  1. For kubectl create -f deploy/crds/charts.helm.k8s.io_v1alpha1_netflowcollector_cr.yaml
    kubectl get pods
    We expect the same as with helm deployment - skydive pods are up and running
    NAME READY STATUS RESTARTS AGE
    netflow-collector-skydive-agent-f8zgq 1/1 Running 0 3m8s
    netflow-collector-skydive-agent-klbmb 1/1 Running 0 3m8s
    netflow-collector-skydive-agent-zmb5m 1/1 Running 0 3m8s
    netflow-collector-skydive-analyzer-866d9778c4-bw4pk 3/3 Running 0 3m8s
    skydive-operator-67c958f454-fkg9j 1/1 Running 0 3m28s

  2. The same as above for kubectl create -f deploy/crds/charts.helm.k8s.io_v1alpha1_netflowcollector_cr.yaml (after removing the above by running kubectl delete -f deploy/crds/charts.helm.k8s.io_v1alpha1_skydive_cr.yaml)

  3. When deploying the charts.helm.k8s.io_v1alpha1_netflowcollector_cr.yaml we can optionally go deeper:
    kubectl describe pods netflow-collector-skydive-analyzer-866
    and see it has 3 containers: skydive-analyzer, skydive-exporter and skydive-minio, and all are running

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants