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.

Sign up for GitHub

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

Jump to bottom

Add Docker Configuration for Executor #189

Merged

mergify merged 1 commit into develop from pate/docker-container

Nov 13, 2022

Collaborator

RPate97 commented Nov 13, 2022 •

edited

Loading

Purpose

Adds Docker container configuration for the executor.

You'll need to install Docker Desktop to test this feature locally.

Testing

In the root directory run yarn build.
Navigate to the plugins directory and run a local node npx hardhat node --local false.
Navigate to the executor directory, make a copy of .env.example, rename it to .env, and fill in the values (the Amplitude API key is optional).
Build the container yarn build-container
Run the container yarn run-container
You should see the container start up and output some logs. After ~20 seconds, you'll see logs indicating that the test contract from the plugins package was executed.

You can also try out deploying other projects with the executor using npx hardhat chugsplash-deploy --local false.

Todo

Test against Optimism Goerli

github-actions bot added the executor label

RPate97 force-pushed the pate/docker-container branch 2 times, most recently from 63d8851 to 423326e Compare

November 13, 2022 01:16

RPate97 requested a review from sam-goldman

November 13, 2022 01:16

sam-goldman reviewed

View reviewed changes

packages/executor/src/index.ts Outdated Show resolved Hide resolved

sam-goldman reviewed

View reviewed changes

packages/executor/tsconfig.json Show resolved Hide resolved

sam-goldman reviewed

View reviewed changes

packages/executor/src/index.ts Outdated Show resolved Hide resolved

RPate97 force-pushed the pate/docker-container branch from 5c74010 to 17f262c Compare

November 13, 2022 22:08

sam-goldman reviewed

View reviewed changes

packages/executor/src/index.ts Outdated Show resolved Hide resolved

sam-goldman reviewed

View reviewed changes

packages/executor/src/index.ts Outdated Show resolved Hide resolved

RPate97 force-pushed the pate/docker-container branch from 17f262c to 2e70cde Compare

November 13, 2022 22:32

RPate97 requested a review from sam-goldman

November 13, 2022 22:35


          Added Docker configuration for executor

RPate97 force-pushed the pate/docker-container branch from 2e70cde to 6669060 Compare

November 13, 2022 23:40

sam-goldman approved these changes

View reviewed changes

Contributor

mergify bot commented Nov 13, 2022

This PR has been added to the merge queue, and will be merged soon.

mergify bot added the on-merge-train label

mergify bot merged commit a43e0e3 into develop

mergify bot deleted the pate/docker-container branch

November 13, 2022 23:42

Contributor

mergify bot commented Nov 13, 2022

This PR is next in line to be merged, and will be merged as soon as checks pass.

mergify bot removed the on-merge-train label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels