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

A more thorough E2E test. #2

Merged
merged 3 commits into from
Jul 23, 2017
Merged

A more thorough E2E test. #2

merged 3 commits into from
Jul 23, 2017

Conversation

jlewi
Copy link
Contributor

@jlewi jlewi commented Jul 18, 2017

  • Rewrite the helm test as a go program (as opposed to Python).
  • Bake the go program into the Docker image for the operator; as opposed to using a config map.
  • The go program allows us to write more detailed tests such as verifying that the resources spawned by the TfJob are created and deleted.
  • The test is still pretty minimal.

Jeremy Lewi added 3 commits July 18, 2017 07:22
* Rewrite the helm test as a go program (as opposed to Python).
* Bake the go program into the Docker image for the operator; as opposed to using a config map.
* The go program allows us to write more detailed tests such as verifying that the resources spawned by the TfJob are created and deleted.
* The test is still pretty minimal.
* Rewrite the helm test as a go program (as opposed to Python).
* Bake the go program into the Docker image for the operator; as opposed to using a config map.
* The go program allows us to write more detailed tests such as verifying that the resources spawned by the TfJob are created and deleted.
* The test is still pretty minimal.
* Support TAP output.
@jlewi jlewi merged commit 052e178 into master Jul 23, 2017
@jlewi jlewi deleted the testing branch July 23, 2017 23:47
sutaakar pushed a commit to sutaakar/training-operator that referenced this pull request Apr 8, 2024
KPostOffice pushed a commit to KPostOffice/training-operator that referenced this pull request May 23, 2024
ChenYi015 pushed a commit to ChenYi015/training-operator that referenced this pull request Sep 20, 2024
bugfix: TF_CONFIG error when enable evaluator
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant