-
Notifications
You must be signed in to change notification settings - Fork 29
feat: add COSI test suite using Chainsaw #94
feat: add COSI test suite using Chainsaw #94
Conversation
Skipping CI for Draft Pull Request. |
60bfc05
to
b30d99a
Compare
b30d99a
to
84679f5
Compare
/cc @BlaineEXE |
84679f5
to
fd2c62f
Compare
In this other PR, I'm adding the 3 remaining e2e/lint targets that are suggested by Kubebuilder's template: https://github.com/kubernetes-sigs/container-object-storage-interface-api/pull/96/files I think it would be nice to see how this PR would integrate with make in the If you think it would be better to do the tooling in another PR, maybe a |
157fafe
to
4cf10fb
Compare
Signed-off-by: Mateusz Urbanek <[email protected]>
Signed-off-by: Mateusz Urbanek <[email protected]>
Signed-off-by: Mateusz Urbanek <[email protected]>
Signed-off-by: Mateusz Urbanek <[email protected]>
Signed-off-by: Mateusz Urbanek <[email protected]>
Signed-off-by: Mateusz Urbanek <[email protected]>
Signed-off-by: Mateusz Urbanek <[email protected]>
Signed-off-by: Mateusz Urbanek <[email protected]>
Signed-off-by: Mateusz Urbanek <[email protected]>
4cf10fb
to
77e688e
Compare
I fully tested that on Linode COSI Driver using the following values: driverName: objectstorage.cosi.linode.com
deletionPolicy: "Delete" # "Retain"
bucketClassParams:
cosi.linode.com/v1/region: us-east
cosi.linode.com/v1/acl: private
cosi.linode.com/v1/cors: disabled
bucketAccessClassParams:
cosi.linode.com/v1/permissions: read_write
authenticationType: "Key" # "IAM"
bucketClaimProtocols: ["S3"]
bucketAccessProtocol: "S3" Test suite run in ~22s, including full |
Signed-off-by: Mateusz Urbanek <[email protected]>
I included a simple readme, however, I am not adding any |
Looks like a great starting point to me. Thanks @shanduur ! |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: BlaineEXE, shanduur The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This pull request introduces a new COSI (Container Object Storage Interface) test suite using Chainsaw, along with several enhancements and preparations for dynamic test generation.
COSI Test Suite Implementation:
Test Generator:
KinD Cluster Configuration:
ctlptl
for setting up a KinD (Kubernetes in Docker) cluster to facilitate testing in a controlled environment.TODO: this needs future updates, e.g. Makefile targets, to ensure that tests can be run locally, including installation of sample driver, CRDs and controller.
The dynamic test generator is now in place, which will allow for easier updates and extensions to the test suite in future iterations. This PR is foundational for future enhancements and ensures that the COSI functionality is thoroughly tested across different authentication mechanisms.