-
Notifications
You must be signed in to change notification settings - Fork 759
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
Create Dockerfile #542
base: uinverse
Are you sure you want to change the base?
Create Dockerfile #542
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
FROM java:8 | ||
|
||
COPY . /var/www/java | ||
|
||
WORKDIR ./src | ||
|
||
ADD https://images.pexels.com/photos/4587959/pexels-photo-4587959.jpeg | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Please replace the hyperlink with a locally stored Here is the cat image, enhanced and upscaled using Machine Learning (i.e., future-proofed for devices using 16k resolution): You should add this image to the base/root of the project and then change line 7 of the Dockerfile as follows:
Note that
In summary, it is my expert opinion that the proposed Dockerfile in full should appear like so: FROM java:8
COPY . /var/www/java
WORKDIR ./src
COPY cat.jpeg .
RUN ./main/java/com/seriouscompany/business/java/fizzbuzz/packagenamingpackage/impl/Main.java There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thank you very much for your work so we doing huge steps forward. Another big problem here is the copyright of the cat image. We dont have a written contract to use this image in our software and we should avoid getting sued for 120m damages. So i propose to replace the cat image with an ai generated cat image that has no copyright. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I agree that a copyright free image would be the best option here. However, we should be cognizant to the developing field of AI-rights wherein the ownership of AI-generated works of art may belong to the AI, or perhaps the creator of said AI, and not necessarily the public domain. We should acquire an AI-generated cat image and revisit this issue of legality in 20 years from today's date. Put this on your calendars. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can you prove the ownership and sign a contract that is added to the docker image? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm also happy to fly out to our Enterprise Headquarters in the corporate private jet to provide additional proof. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I guess that is enough. We should use a picture of Kimchi. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Now we just need @Tylersuard to update his Dockerfile and resubmit his PR. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I believe relying on pexels is not sustainable; I'd commit the image into directly the repository instead. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
We did the research a year ago. This project needs to cut some costs so we cant do it again. One image of kimchi would not add much bloat to the current state of our goal. |
||
|
||
RUN ./main/java/com/seriouscompany/business/java/fizzbuzz/packagenamingpackage/impl/Main.java | ||
|
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.
Please remove this line.
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.
I can't. If I remove the cat picture everything stops working.
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.
How are we gonna resolve this issue?
Company management wants us to scale into cloud but the cat image is blocking it.
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.
@PurHur please see my comment below regarding a possible resolution for this issue. bd44545#r901147477