Skip to content

Comments

feat: add dockerfile and entrypoint script to deploy hyperlane#51

Merged
damiannolan merged 7 commits intomainfrom
damian/dockerize-deploying-the-hype
May 20, 2025
Merged

feat: add dockerfile and entrypoint script to deploy hyperlane#51
damiannolan merged 7 commits intomainfrom
damian/dockerize-deploying-the-hype

Conversation

@damiannolan
Copy link
Collaborator

@damiannolan damiannolan commented May 19, 2025

Overview

  • Adds a dockerfile to automate the deployment of hyperlane components on EVM and cosmosnative (celestia).
  • Adjusts the Makefile targets: start, stop to use the new docker-compose services.

closes: #1

@damiannolan damiannolan force-pushed the damian/dockerize-deploying-the-hype branch from 64da300 to 2c22921 Compare May 20, 2025 07:55
Copy link
Collaborator Author

@damiannolan damiannolan left a comment

Choose a reason for hiding this comment

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

Ideally would like to keep PRs well scoped so I will publish this for review and create more follow up PRs.

Next I would like to:

  • Clean out unused files from the ibc demo which are bloating this repo #47
  • Address the comment I left about extending the init container for hyperlane deployment to complete the last step. #53
  • Add the hyperlane relayer to docker compose #54

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This should be extended to install foundry tools in order to complete the last step: enroll-remote-router on both celestia-app and evm.

I think this can be done in a follow up PR.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

#53

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

simple build and push workflow, if we need to adjust this to use the one provided by celestia ops team then we can do that later, afaik the only difference is certain arch support for running k8s which we don't need here

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This should just work when merged to main. Can take a look when this PR lands

Copy link
Collaborator

Choose a reason for hiding this comment

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

hahaha I think the same thing every time I touch Docker CI workflows but I would not be surprised if a permission issue prevented this from working out of the box.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

lol

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Copy link
Collaborator

@rootulp rootulp left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

Choose a reason for hiding this comment

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

hahaha I think the same thing every time I touch Docker CI workflows but I would not be surprised if a permission issue prevented this from working out of the box.

@damiannolan damiannolan merged commit 07d5557 into main May 20, 2025
@rootulp rootulp deleted the damian/dockerize-deploying-the-hype branch May 20, 2025 14:50
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.

Modify make setup to setup Hyperlane contracts, ISMs, warp routes

2 participants