-
Notifications
You must be signed in to change notification settings - Fork 73
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Documenting acceptance test #175
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good start, LGTM. Will add further ideas myself.
docs/design/acceptance_test.md
Outdated
# Acceptance test for kube-arangodb operator on specific Kubernetes platform | ||
|
||
This acceptance test plan describes all test scenario's that must be executed | ||
succesfully in order to consider the kube-arangodb operator production ready |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
successfully
|
||
```bash | ||
kubectl apply -f examples/arango-local-storage.yaml | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We probably want to create an arango-local-storage on all platforms, just to see if it works, and because performance will be best with local disks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. See 3 little comments.
executed on the following platforms: | ||
|
||
- Google GKE, with Kubernetes version 1.10 | ||
- Amazon EKS, with Kubernetes version 1.10 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would like to add "Amazon/kops, with Kubernetes version 1.10" to this mix.
- [ ] The deployment must yield a `Service` named `<deployment-name>-sync` | ||
- [ ] The `Service` named `<deployment-name>-ea` must be accessible from outside (LoadBalancer or NodePort) and show WebUI | ||
|
||
### Test 2a: Scale an active failover deployment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed, but note that we "officially" do not support this yet.
docs/design/acceptance_test.md
Outdated
- [ ] The deployment must yield a `Service` named `<deployment-name>-ea` | ||
- [ ] The `Service` named `<deployment-name>-ea` must be accessible from outside (LoadBalancer or NodePort) and show WebUI | ||
|
||
### Test 4b: Create cluster deployment with a platform provides `StorageClass` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"provides" -> "provided"
No description provided.