PipelineDP icon indicating copy to clipboard operation
PipelineDP copied to clipboard

Dockerfile for PipelineDP

Open wenhuizhang opened this issue 3 years ago • 2 comments

Description

Dockerfile for one of the test

Affected Dependencies

readme file Dockerfile folder

How has this been tested?

as stated in the readme file

Checklist

wenhuizhang avatar Feb 13 '22 02:02 wenhuizhang

This is the initial commit for Issue Support Dockerfile for Env Dependencies Issue #245. @chinmayshah99

wenhuizhang avatar Feb 13 '22 02:02 wenhuizhang

Nope, I will update the pull request then

On Apr 25, 2022, at 11:22 AM, Chinmay Shah @.***> wrote:

@chinmayshah99 requested changes on this pull request.

In Dockerfile/restaraunt_visits_without_frameworks/Dockerfile https://github.com/OpenMined/PipelineDP/pull/246#discussion_r805493657:

+# See the License for the specific language governing permissions and +# limitations under the License.

+# base image +FROM ubuntu:20.04 + +# maintainer +MAINTAINER wenhui @.> @.> @.***> + +# commands executed in terminal +RUN set -x + +# build deps +RUN apt-get update +RUN apt-get -y install git vim
Is vim required?

In README.md https://github.com/OpenMined/PipelineDP/pull/246#discussion_r857906105:

@@ -116,6 +116,45 @@ sample of it. Here's how to take a subset of the data in bash:

  1. Run python movie_view_ratings.py --input_file=<path to data.txt from 2> --output_file=<...>

+## Dockerfile to tryout +To build and test a Dockerfile, please: +``` +cd ./Dockerfile + +docker image build -t $test_case:$version this seems to be a typo. In the current version of the dockerfile, there is no support for different files

— Reply to this email directly, view it on GitHub https://github.com/OpenMined/PipelineDP/pull/246#pullrequestreview-881141423, or unsubscribe https://github.com/notifications/unsubscribe-auth/AARUYPPUYANOCXME4XB6EUTVG3PGNANCNFSM5OIAUVYA. You are receiving this because you authored the thread.

wenhuizhang avatar Apr 25 '22 18:04 wenhuizhang