-
Notifications
You must be signed in to change notification settings - Fork 23
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
735 feature request implement sonobuoy golang framework for kaas conformance #761
735 feature request implement sonobuoy golang framework for kaas conformance #761
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.
Looking good to me. I was able to run the example tests against the kind cluster using make commands. Just a few things that I would add/change.
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 went through the guide and could not find the ./build.sh
script in this repository. If it is not included, I would suggest adjusting the 'How to Use This Plugin' section accordingly.
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.
This command was intended to refer to a folder in which locally built Golang bineries should be stored.
But I have currently removed this, as executing the binaries outside a Kubernetes pod does not work without additional effort.
I will remove this command from this PR.
@@ -61,26 +60,30 @@ For test development with Sonobuoy, [KinD](https://kind.sigs.k8s.io/) is used as | |||
sonobuoy status |
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 don't think we need to use the sonobuoy status
command, as we are now using the --wait
flag in the make dev-run
command. I would say sonobuoy status
and the related text above apply when the user/developer is using the make dev-run-background
command now.
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.
That's right. I removed the sonobuoy status
commands.
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.
This command is also mentioned in the README of this directory. Will the README be updated in this PR, or will it be part of SovereignCloudStack/docs#221?
637963b
to
e965022
Compare
|
Which part are you referring to? |
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.
current state worked for me
Derived from the PoC, this commit presents the setup for the provision of SCS-kaas tests using sonoboy and golang. Signed-off-by: Toni Finger <[email protected]>
Rework Makefile Apply kind configuration Signed-off-by: Toni Finger <[email protected]>
Signed-off-by: Toni Finger <[email protected]>
Co-authored-by: Michal Gubricky <[email protected]> Signed-off-by: tonifinger <[email protected]>
This commit introduces a development mode that allows the test suite itself to create a Kubernetes cluster for testing Signed-off-by: Toni Finger <[email protected]>
Signed-off-by: Toni Finger <[email protected]>
Signed-off-by: Toni Finger <[email protected]>
bffe9d8
to
6561a31
Compare
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.
works for me
Restructure the current sonobuoy golang framework to provide a productive setup.