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
update tests for windows #181
Conversation
/hold (using prow to test the PR as having trouble with dev machine). Will remove hold and assign when ready for review |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aramase 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 |
@@ -5,7 +5,8 @@ metadata: | |||
spec: | |||
podInfoOnMount: true | |||
attachRequired: false | |||
{{- if and ( ge (.Capabilities.KubeVersion.Major | int) 1 ) ( ge (.Capabilities.KubeVersion.Minor | int) 16 ) }} | |||
volumeLifecycleModes: # Added in Kubernetes 1.16 with default mode of Persistent. Secrets store csi driver needs Ephermeral to be set. | |||
{{- if semverCompare ">=1.16-0" .Capabilities.KubeVersion.GitVersion }} |
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.
Using int compare does not work for pre-release versions (alpha/beta). So switched this to use semverCompare
instead.
/test pull-secrets-store-csi-driver-e2e-windows |
/test pull-secrets-store-csi-driver-e2e-windows |
/test pull-secrets-store-csi-driver-e2e-windows |
charts/secrets-store-csi-driver/templates/secrets-store-csi-driver.yaml
Outdated
Show resolved
Hide resolved
.PHONY: install-driver | ||
install-driver: | ||
ifdef TEST_WINDOWS | ||
helm install csi-secrets-store charts/secrets-store-csi-driver --namespace default --wait --timeout=15m -v=5 --debug \ |
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.
maybe use the same helm install cmd but set diff variables in the if/else?
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.
The variables for linux and windows are different which is why the helm install cmd is different in if/else
/test pull-secrets-store-csi-driver-e2e-windows |
@ritazh PTAL when you get a chance! Updated the charts to support hybrid scenarios and also added liveness probe sidecar for windows. |
/lgtm |
/hold cancel |
fixes #179