Skip to content
This repository has been archived by the owner on Sep 19, 2022. It is now read-only.

Updated the usrer steps necessary to run the out of the box #244

Closed
wants to merge 3 commits into from
Closed

Updated the usrer steps necessary to run the out of the box #244

wants to merge 3 commits into from

Conversation

MATRIX4284
Copy link
Contributor

image name gcr.io/<your_project>/pytorch_dist_mnist:latest wil cause the image pull to fail in case of non gcp users.
So included the yaml specs with the local image i.e. kubeflow/pytorch-dist-mnist-test:1.0.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
image name gcr.io/<your_project>/pytorch_dist_mnist:latest wil cause the image pull to fail in case of non gcp users.
So included the yaml specs with the local image i.e. kubeflow/pytorch-dist-mnist-test:1.0.
@k8s-ci-robot
Copy link

Hi @MATRIX4284. Thanks for your PR.

I'm waiting for a kubeflow member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@k8s-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign johnugeorge
You can assign the PR to them by writing /assign @johnugeorge 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 files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@coveralls
Copy link

coveralls commented Jan 1, 2020

Coverage Status

Coverage remained the same at 22.97% when pulling 0a7e04f on MATRIX4284:master into f117363 on kubeflow:master.

@gaocegege
Copy link
Member

/ok-to-test

@k8s-ci-robot k8s-ci-robot added size/L and removed size/M labels Jan 2, 2020
@k8s-ci-robot
Copy link

@MATRIX4284: The following test failed, say /retest to rerun them all:

Test name Commit Details Rerun command
kubeflow-pytorch-operator-presubmit 0a7e04f link /test kubeflow-pytorch-operator-presubmit

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

libcanberra-gtk-module \
&& sudo rm -rf /var/lib/apt/lists/*
RUN conda install -y -c menpo opencv3=3.1.0 \
&& conda clean -ya
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do we need such a Dockerfile

@MATRIX4284
Copy link
Contributor Author

Added the required Dockerfile For mitigating the issue #245

@MATRIX4284 MATRIX4284 closed this Jan 2, 2020
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.

None yet

4 participants