Skip to content
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

Conversation

tonifinger
Copy link
Contributor

Restructure the current sonobuoy golang framework to provide a productive setup.

Copy link
Contributor

@michal-gubricky michal-gubricky left a 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.

Tests/kaas/kaas-sonobuoy-tests/Makefile Outdated Show resolved Hide resolved
Tests/kaas/kaas-sonobuoy-tests/README.md Outdated Show resolved Hide resolved
Copy link
Contributor

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.

Copy link
Contributor Author

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
Copy link
Contributor

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.

Copy link
Contributor Author

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.

Copy link
Contributor

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?

@tonifinger tonifinger marked this pull request as ready for review October 1, 2024 07:21
@tonifinger tonifinger force-pushed the 735-feature-request-implement-sonobuoy-golang-framework-for-kaas-conformance branch from 637963b to e965022 Compare October 1, 2024 08:04
@fraugabel
Copy link
Contributor

yq is not in the requirements.. should be updated

@tonifinger
Copy link
Contributor Author

yq is not in the requirements.. should be updated

Which part are you referring to? yq is part of the make dev-prerequests
See: https://github.com/SovereignCloudStack/standards/pull/761/files#diff-2f673fe7de6ad88820c96b50f8862b96e1b6dbe54b8f7ba8c11907dde677a57cR53

Copy link
Contributor

@fraugabel fraugabel left a 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

tonifinger and others added 7 commits October 1, 2024 11:57
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]>
@tonifinger tonifinger force-pushed the 735-feature-request-implement-sonobuoy-golang-framework-for-kaas-conformance branch from bffe9d8 to 6561a31 Compare October 1, 2024 09:58
Copy link
Contributor

@fraugabel fraugabel left a comment

Choose a reason for hiding this comment

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

works for me

@tonifinger tonifinger merged commit c16cfac into main Oct 2, 2024
7 checks passed
@tonifinger tonifinger deleted the 735-feature-request-implement-sonobuoy-golang-framework-for-kaas-conformance branch October 2, 2024 08:36
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.

Make sonobuoy golang framework useable for production
3 participants