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

feat: add Dockerfile #8315

Merged
merged 1 commit into from Jul 25, 2022
Merged

feat: add Dockerfile #8315

merged 1 commit into from Jul 25, 2022

Conversation

OrKoN
Copy link
Collaborator

@OrKoN OrKoN commented May 4, 2022

This PR adds an official Dockerfile for Puppeteer. The content of the Dockerfile is practically the same as documented in troubleshooting.md:

  1. It installs chrome-stable and dependencies via apt-get.
  2. it installs a local Puppeteer build into the docker user's home folder.
  3. configures required permissions for the user.
  4. outputs licenses into the THIRD_PARTY_NOTICES file.

The local Puppeteer build is created by docker/pack.sh which is meant to be used in CI. This PR also includes a GitHub action that would build a docker image and run a smote test inside of it.

The next step would be actually publishing the docker image from GitHub Actions to GitHub Registry.

Copy link

@Ruslan34395 Ruslan34395 left a comment

Choose a reason for hiding this comment

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

[removed reference to issue 8356]

@OrKoN OrKoN force-pushed the docker branch 2 times, most recently from fff3de2 to df4e77b Compare June 23, 2022 14:50
@OrKoN OrKoN force-pushed the docker branch 5 times, most recently from 2f8b7a6 to 2d50b60 Compare July 25, 2022 08:18
@OrKoN OrKoN requested a review from mathiasbynens July 25, 2022 08:18
This PR adds an official Dockerfile for Puppeteer. The content of the Dockerfile is practically the same as documented in troubleshooting.md:

1) It installs chrome-stable and dependencies via apt-get.
2) it installs a local Puppeteer build into the docker user's home folder.
3) configures required permissions for the user.
4) outputs licenses into the THIRD_PARTY_NOTICES file.

The local Puppeteer build is created by `docker/pack.sh` which is meant to be used in CI. This PR also includes a GitHub action that would build a docker image and run a smote test inside of it.

The next step would be actually publishing the docker image from GitHub Actions to GitHub Registry.
@OrKoN OrKoN enabled auto-merge (squash) July 25, 2022 09:21
@OrKoN OrKoN disabled auto-merge July 25, 2022 09:21
@OrKoN OrKoN enabled auto-merge (squash) July 25, 2022 09:22
@OrKoN OrKoN changed the title chore: add Dockerfile feat: add Dockerfile Jul 25, 2022
@OrKoN OrKoN merged commit b72d949 into main Jul 25, 2022
@OrKoN OrKoN deleted the docker branch July 25, 2022 12:34
jrandolf pushed a commit that referenced this pull request Aug 2, 2022
This PR adds an official Dockerfile for Puppeteer. The content of the Dockerfile is practically the same as documented in troubleshooting.md:

1) It installs chrome-stable and dependencies via apt-get.
2) it installs a local Puppeteer build into the docker user's home folder.
3) configures required permissions for the user.
4) outputs licenses into the THIRD_PARTY_NOTICES file.

The local Puppeteer build is created by `docker/pack.sh` which is meant to be used in CI. This PR also includes a GitHub action that would build a docker image and run a smote test inside of it.

The next step would be actually publishing the docker image from GitHub Actions to GitHub Registry.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants