-
-
Notifications
You must be signed in to change notification settings - Fork 77
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
Dockerfile for PipelineDP #246
base: main
Are you sure you want to change the base?
Conversation
This is the initial commit for Issue Support Dockerfile for Env Dependencies Issue #245. @chinmayshah99 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for late reply, the last 3 weeks with the situation in Ukraine were crazy.
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 <#246 (comment)>:
> +# 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 <#246 (comment)>:
> @@ -116,6 +116,45 @@ sample of it. Here's how to take a subset of the data in bash:
3. 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 <#246 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AARUYPPUYANOCXME4XB6EUTVG3PGNANCNFSM5OIAUVYA>.
You are receiving this because you authored the thread.
|
|
||
# download source | ||
RUN git clone https://github.com/OpenMined/PipelineDP.git | ||
RUN cd /PipelineDP && git checkout v0.1.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
need to change this version
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