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

Parameterise ES cluster yaml in tests #179

Merged
merged 2 commits into from
Jan 8, 2018

Conversation

wallrj
Copy link
Member

@wallrj wallrj commented Dec 15, 2017

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:

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
@wallrj
Copy link
Member Author

wallrj commented Dec 15, 2017

/test e2e

@wallrj
Copy link
Member Author

wallrj commented Dec 19, 2017

/test e2e

@wallrj
Copy link
Member Author

wallrj commented Dec 19, 2017

/test e2e

@munnerz
Copy link
Contributor

munnerz commented Jan 8, 2018

/lgtm
/approve

@jetstack-ci-bot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: munnerz

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

@jetstack-ci-bot
Copy link
Contributor

/test all [submit-queue is verifying that this PR is safe to merge]

@munnerz
Copy link
Contributor

munnerz commented Jan 8, 2018

/retest

@jetstack-ci-bot
Copy link
Contributor

/test all [submit-queue is verifying that this PR is safe to merge]

@jetstack-ci-bot
Copy link
Contributor

Automatic merge from submit-queue.

@jetstack-ci-bot jetstack-ci-bot merged commit 9a78702 into jetstack:master Jan 8, 2018
@wallrj wallrj deleted the parameterise-es-yaml 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