-
Notifications
You must be signed in to change notification settings - Fork 56
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
filter tests to save time on terraform-only PRs #715
Conversation
hack/setup-kind.sh
Outdated
@@ -174,8 +174,8 @@ kubeadmConfigPatches: | |||
"service-account-signing-key-file": "/etc/kubernetes/pki/sa.key" | |||
"service-account-api-audiences": "api,spire-server" | |||
"service-account-jwks-uri": "https://kubernetes.default.svc/openid/v1/jwks" | |||
networking: | |||
dnsDomain: "${CLUSTER_SUFFIX}" | |||
# networking: |
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.
Why are we commenting this now ?
.github/workflows/test-release.yaml
Outdated
@@ -52,9 +55,10 @@ jobs: | |||
|
|||
- name: Setup Cluster | |||
# TODO: update after next release. | |||
uses: chainguard-dev/actions/setup-kind@52eca3baf7c09ec1dbc4195449779fa59cc618e6 | |||
uses: bobcallaway/cg-actions/setup-kind@f807492463bc85bc56ddbac9ddc265e3caa4c8ce |
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.
why do we need to use this ?
@bobcallaway I am facing the same issue, happy to help fixing it! I was looking at #704 or #705 after talking with @wlynch. |
Since this is fixing the bug of not running tests unnecessarlly, I'm wondering if we should consider:
|
Signed-off-by: Bob Callaway <bcallaway@google.com>
sigstore/scaffolding#715 changed the test behavior of `scaffolding` to optionally only run certain tests on terraform vs non-terraform PRs Signed-off-by: Bob Callaway <bobcallaway@users.noreply.github.com>
sigstore/scaffolding#715 changed the test behavior of `scaffolding` to optionally only run certain tests on terraform vs non-terraform PRs Signed-off-by: Bob Callaway <bobcallaway@users.noreply.github.com>
sigstore/scaffolding#715 changed the test behavior of `scaffolding` to optionally only run certain tests on terraform vs non-terraform PRs Signed-off-by: Bob Callaway <bobcallaway@users.noreply.github.com>
Signed-off-by: Bob Callaway <bcallaway@google.com>
this change adds
paths-ignore
filters to make terraform-only changes run fasterFixes: #699