Skip to content
This repository has been archived by the owner on Apr 4, 2023. It is now read-only.

Allow selection of the test to run #177

Closed
wants to merge 1 commit into from

Conversation

wallrj
Copy link
Member

@wallrj wallrj commented Dec 14, 2017

Allows you to limit the E2E tests that are run by setting a TEST_PREFIX environment variable. E.g.
TEST_PREFIX=test_cassandra ./hack/e2e.sh

To bypass the Elasticsearch tests when I'm running E2E tests manually on my GKE cluster.

Part of #23

Release note:

NONE

Allows you to limit the E2E tests that are run by setting a `TEST_PREFIX` environment variable. E.g.
TEST_PREFIX=test_cassandra ./hack/e2e.sh

To bypass the Elasticsearch tests when I'm running E2E tests manually on my GKE cluster.

Part of jetstack#23
@jetstack-ci-bot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
We suggest the following additional approver: munnerz

Assign the PR to them by writing /assign @munnerz in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@wallrj
Copy link
Member Author

wallrj commented Dec 14, 2017

/test e2e

@munnerz
Copy link
Contributor

munnerz commented Dec 14, 2017

@wallrj failing due to unbound var:

W1214 13:19:27.152] hack/e2e.sh: line 135: TEST_PREFIX: unbound variable

@wallrj
Copy link
Member Author

wallrj commented Dec 14, 2017

/test e2e

@munnerz
Copy link
Contributor

munnerz commented Jan 8, 2018

Not really sure whether this or #179 is meant to be merged, but given #179 includes this and some, I'm going to close this PR.

/close

jetstack-ci-bot added a commit that referenced this pull request Jan 8, 2018
Automatic merge from submit-queue.

Parameterise ES cluster yaml in tests

Allows me to override the ES pilot image parameters when I'm running tests manually on GKE.

Using `envsubst` as a much lighter weight alternative to using helm chart. Discovered via: kubernetes/kubernetes#23896 (comment)


Stacked on #177 

**Release note**:
```release-note
NONE
```
@wallrj wallrj deleted the 23-choose-a-test branch January 9, 2018 09:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants