Skip to content

fix: message relayer docker build#1964

Merged
tynes merged 2 commits intodevelopfrom
fix/message-relayer-docker-build
Dec 23, 2021
Merged

fix: message relayer docker build#1964
tynes merged 2 commits intodevelopfrom
fix/message-relayer-docker-build

Conversation

@tynes
Copy link
Copy Markdown
Contributor

@tynes tynes commented Dec 22, 2021

Description
After 3a67332
was merged, it broke the docker build of the message-relayer. See:
https://github.com/ethereum-optimism/optimism/runs/4600810001?check_suite_focus=true

#25 [stage-1 18/20] COPY --from=builder /optimism/packages/message-relayer/exec ./exec
#25 ERROR: "/optimism/packages/message-relayer/exec": not found
------
 > [stage-1 18/20] COPY --from=builder /optimism/packages/message-relayer/exec ./exec:
------
Dockerfile.message-relayer:31
--------------------
  29 |     COPY --from=builder /optimism/packages/message-relayer/dist ./dist
  30 |     COPY --from=builder /optimism/packages/message-relayer/package.json .
  31 | >>> COPY --from=builder /optimism/packages/message-relayer/exec ./exec
  32 |     COPY --from=builder /optimism/packages/message-relayer/node_modules ./node_modules
  33 |     
--------------------

This PR removes the reference to exec which no longer exists

tynes added 2 commits December 21, 2021 16:28
@tynes
ops: fix message-relayer docker build
4d963ad 
The `exec` directory was removed, so no longer
copy it over during the docker build.
@tynes
chore: add changeset for message-relayer
e7c65ad
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Dec 22, 2021

🦋 Changeset detected

Latest commit: e7c65ad

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@eth-optimism/message-relayer Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@tynes tynes requested a review from smartcontracts December 22, 2021 00:40
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Dec 22, 2021 •
edited
Loading

Codecov Report

Merging #1964 (e7c65ad) into develop (dddc160) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #1964   +/-   ##
========================================
  Coverage    73.42%   73.42%           
========================================
  Files           80       80           
  Lines         2559     2559           
  Branches       399      399           
========================================
  Hits          1879     1879           
  Misses         680      680
Flag Coverage Δ
batch-submitter 62.50% <ø> (ø)
contracts 87.96% <ø> (ø)
core-utils 57.50% <ø> (ø)
data-transport-layer 38.64% <ø> (ø)
message-relayer 70.86% <ø> (ø)
sdk 87.06% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dddc160...e7c65ad. Read the comment docs.

@tynes tynes merged commit 0a20266 into develop Dec 23, 2021
@tynes tynes deleted the fix/message-relayer-docker-build branch December 23, 2021 18:04
theochap pushed a commit that referenced this pull request Dec 10, 2025
@refcell
chore(rpc): Disconnect Peer RPC Request (#1964)
37f53b8 
### Description

Implements the `opp2p_disconnectPeer` rpc request.

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

Reviewers

@smartcontracts smartcontracts Awaiting requested review from smartcontracts

Assignees

No one assigned

Labels

A-ops Area: ops

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tynes @codecov-commenter