Skip to content
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

Open
wants to merge 1 commit into
base: uinverse
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 10 additions & 0 deletions dockerfile
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
Copy link

Choose a reason for hiding this comment

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

Please remove this line.

Copy link
Author

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.

Copy link

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.

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

Copy link

@knakamura13 knakamura13 Jun 19, 2022

Choose a reason for hiding this comment

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

Please replace the hyperlink with a locally stored jpeg file to appease corporate demand for cloud scalability while avoiding cross-hosting headaches as well as, God-forbid, 404 missing file issues if pexels.com goes offline.

Here is the cat image, enhanced and upscaled using Machine Learning (i.e., future-proofed for devices using 16k resolution):
https://we.tl/t-okV1rNmg3d

You should add this image to the base/root of the project and then change line 7 of the Dockerfile as follows:

- ADD https://images.pexels.com/photos/4587959/pexels-photo-4587959.jpeg
+ ADD cat.jpeg

Note that ADD is no longer necessary as it is the same as COPY when using locally-referenced files. Therefore, the same line can be rewritten as follows:

COPY cat.jpeg .

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

Copy link

@PurHur PurHur Jun 19, 2022

Choose a reason for hiding this comment

The 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.

Choose a reason for hiding this comment

The 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.

Copy link

@knakamura13 knakamura13 Jun 19, 2022

Choose a reason for hiding this comment

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

How about we use a cat image that I took? Copyright: me.

Artboard 2

Copy link

Choose a reason for hiding this comment

The 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?

Choose a reason for hiding this comment

The 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.

Copy link

Choose a reason for hiding this comment

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

I guess that is enough. We should use a picture of Kimchi.

Choose a reason for hiding this comment

The 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.

Copy link
Contributor

Choose a reason for hiding this comment

The 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.
But I do agree we should investigate why the Dockerfile is not working unless we add more cats to it.

Copy link

Choose a reason for hiding this comment

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

But I do agree we should investigate why the Dockerfile is not working unless we add more cats to it.

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