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

Setup multiple connections on same home #127

Open
orkunkl opened this issue Mar 17, 2021 · 4 comments
Open

Setup multiple connections on same home #127

orkunkl opened this issue Mar 17, 2021 · 4 comments
Projects

Comments

@orkunkl
Copy link
Contributor

orkunkl commented Mar 17, 2021

Currently you can only setup one connection from registry and this information is saved to app.yaml
For setting up new relayers on the same machine, the registry.yaml must be copied to a new relayer home and set up must be done again for the connection.
app.yaml:

src: hackatom-ru-ics20
dest: swap-testnet-2001
mnemonic: ***
srcConnection: connection-0
destConnection: connection-1

In this case what ibc-relayer start --src --dst flags are for?

Please close the issue if you are aware of this issue.

@ethanfrey ethanfrey added this to To do in Relayer Jun 21, 2021
@ethanfrey
Copy link
Member

This is a good question.
I think we wanted to remove these unused (and unusable flags).

I need to check the current state of the CLI

@ethanfrey
Copy link
Member

Ah, this exists as we wanted to make ibc-relayer work with 12 factor style.

You can use an app.yaml file which is created via the various ibc-setup commands. This is great for local development and can be used in production.

You can also run it without any app.yaml, specifying everything via cli flags (--src --src-connection) or env variables (RELAYER_SRC, RELAYER_SRC_CONNECTION).

If this is all to confusing, I could remove the cli flags here and make it only app.yaml or env variables.

Or error if multiple elements are both set and are not identical.

What do you think?

@orkunkl
Copy link
Contributor Author

orkunkl commented Jun 25, 2021

Considering 12 factor style, flags makes sense.

Or error if multiple elements are both set and are not identical.

This could work. Also specifying which app.yaml to run as flag could make sense.

@ethanfrey
Copy link
Member

Also specifying which app.yaml to run as flag could make sense.

That is what the --home flag is for. We read the app.yaml and registry.yaml from that dir, nothing else.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Relayer
To do
Development

No branches or pull requests

2 participants