Skip to content

Use the release/test python scripts pulled from the repo. #237

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 7 commits into from
Dec 22, 2017
Merged

Use the release/test python scripts pulled from the repo. #237

merged 7 commits into from
Dec 22, 2017

Conversation

@jlewi
Copy link
Contributor

@jlewi jlewi commented Dec 21, 2017 •
edited
Loading

  • The steps in our E2E tests should use the version of the test scripts
    at the revision that is being tested as opposed to the version baked into
    the container.

This change is Reviewable

@jlewi
Use the release/test python scripts pulled from the repo.
262a634 
* The steps in our E2E tests should use the version of the test scripts
  at the revision that is being tested as opposed to the version baked into
  the container.
@jlewi jlewi requested a review from wbuchwalter December 21, 2017 06:21
@jlewi jlewi mentioned this pull request Dec 21, 2017
Merged
4 tasks
@coveralls
Copy link

coveralls commented Dec 21, 2017 •
edited
Loading

Coverage Status

Coverage decreased (-0.1%) to 37.782% when pulling 262a634 on jlewi:python_path into cb1e053 on tensorflow:master.

@coveralls
Copy link

coveralls commented Dec 21, 2017 •
edited
Loading

Coverage Status

Coverage decreased (-0.1%) to 37.782% when pulling ab407e2 on jlewi:python_path into cb1e053 on tensorflow:master.

@coveralls
Copy link

coveralls commented Dec 21, 2017 •
edited
Loading

Coverage Status

Coverage decreased (-0.1%) to 37.782% when pulling 09c4868 on jlewi:python_path into cb1e053 on tensorflow:master.

@coveralls
Copy link

coveralls commented Dec 21, 2017 •
edited
Loading

Coverage Status

Coverage decreased (-0.1%) to 37.782% when pulling 0ec90b9 on jlewi:python_path into cb1e053 on tensorflow:master.

@coveralls
Copy link

coveralls commented Dec 21, 2017 •
edited
Loading

Coverage Status

Coverage decreased (-0.1%) to 37.782% when pulling 37ffae6 on jlewi:python_path into cb1e053 on tensorflow:master.

@coveralls
Copy link

coveralls commented Dec 21, 2017 •
edited
Loading

Coverage Status

Coverage decreased (-0.1%) to 37.782% when pulling 1c359fd on jlewi:python_path into cb1e053 on tensorflow:master.

@jlewi jlewi merged commit e108d55 into kubeflow:master Dec 22, 2017
sutaakar pushed a commit to sutaakar/training-operator that referenced this pull request Mar 24, 2025
@konflux-internal-p02
RPM updates (kubeflow#237)
f8bd610 
Signed-off-by: konflux-internal-p02 <170854209+konflux-internal-p02[bot]@users.noreply.github.com>
Co-authored-by: konflux-internal-p02[bot] <170854209+konflux-internal-p02[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@wbuchwalter wbuchwalter wbuchwalter approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jlewi @coveralls @wbuchwalter