Skip to content
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

ref(e2e): Refactor e2e test charts to complement the refactored test suite #218

Merged
merged 1 commit into from
Apr 12, 2016
Merged

ref(e2e): Refactor e2e test charts to complement the refactored test suite #218

merged 1 commit into from
Apr 12, 2016

Conversation

krancour
Copy link
Contributor

Supersedes #214

Relates to deis/workflow-e2e#157

cc @vdice @arschles

@krancour
Copy link
Contributor Author

Also, reviewers please note I moved these out from under deis-e2e and under workflow-dev-e2e. The "deis" moniker doesn't seem correct when the test suite is specifically for the Workflow product.

@jchauncey
Copy link
Member

The readme needs to be updated to reflect the change in the pod name

@jchauncey jchauncey removed the LGTM1 label Apr 11, 2016
@krancour
Copy link
Contributor Author

Nice catch.

@krancour
Copy link
Contributor Author

Commit has now been amended to remove the workflow-dev-e2e-serial chart. Those wishing to run tests in serial can adjust the GINKGO_NODES env var to equal "1".

@krancour
Copy link
Contributor Author

Thanks for the reviews, guys. Will hold off on merging this until deis/workflow-e2e#157 is ready to be merged.

@vdice
Copy link
Member

vdice commented Apr 12, 2016

Closes #215

vdice pushed a commit to vdice/chart-mate that referenced this pull request Apr 12, 2016
This represents the necessary update to go along with changes in
deis/workflow-e2e#157 and
deis/charts#218
vdice pushed a commit to vdice/chart-mate that referenced this pull request Apr 12, 2016
This represents the necessary update to go along with changes in
deis/workflow-e2e#157 and
deis/charts#218
vdice pushed a commit to vdice/chart-mate that referenced this pull request Apr 12, 2016
This represents the necessary update to go along with changes in
deis/workflow-e2e#157 and
deis/charts#218
vdice pushed a commit to vdice/chart-mate that referenced this pull request Apr 12, 2016
This represents the necessary update to go along with changes in
deis/workflow-e2e#157 and
deis/charts#218
@krancour krancour merged commit bfb5f09 into deis:master Apr 12, 2016
@krancour krancour deleted the support-refactored-tests branch April 12, 2016 22:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants