Skip to content
This repository has been archived by the owner on Oct 31, 2024. It is now read-only.

fix: wip update local docker setup #351

Closed
wants to merge 2 commits into from
Closed

fix: wip update local docker setup #351

wants to merge 2 commits into from

Conversation

gruberb
Copy link
Contributor

@gruberb gruberb commented Oct 26, 2023

Description

Fixing the local docker setup. Peers cannot connect to the bootnode since they cannot reach the address in the genesis.json, which doesn't point to a valid node at the moment. This PR aims to fix that.

Fixes TP-769

PR Checklist:

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added or updated tests that comprehensively prove my change is effective or that my feature works


export TOPOS_HOME=$TOPOS_HOME
export TCE_LOCAL_KS=$HOSTNAME
export TCE_EXT_HOST
export TCE_LOCAL_KS="b2d9e427c3be"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Just a quick fix for now. We might want to also store this in a file.

@@ -1,6 +1,5 @@
{
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I didn't implement a better solution for the keys yet. We might not want to delete them from the file in the future.

@@ -43,6 +43,6 @@
"burnContractDestinationAddress": "0x0000000000000000000000000000000000000000"
},
"bootnodes": [
"/ip4/10.101.232.59/tcp/10001/p2p/12D3KooWKfbuXufub9aRs3HQJ2T8geHDM8mkfA6DaZ8KTHzQzzPU"
"/dns4/b2d9e427c3be/tcp/9090/p2p/12D3KooWKfbuXufub9aRs3HQJ2T8geHDM8mkfA6DaZ8KTHzQzzPU"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's the "old address".

@gruberb gruberb closed this Oct 27, 2023
@gruberb
Copy link
Contributor Author

gruberb commented Oct 27, 2023

In favour of the extended version #349

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant