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

Replace identity_mappings.json with cli variables #35

Merged
merged 2 commits into from
Mar 8, 2022

Conversation

DieracDelta
Copy link
Contributor

@DieracDelta DieracDelta commented Mar 4, 2022

Fixes a subset of the items in this issue. Notably:

  • Generates peer ids dynamically. In order to do this, we add a new type of message and ClientRequest that the Conductor node repeatedly broadcasts its peers. The reason for this message is prevent Conductor nodes from being pruned.
  • Moves the bound port and ip address into a cli variable in the style $IP:$PORT
  • Add "expected" number of nodes to cli variable. This is needed for all nodes in order to perform pruning.
  • Specify node's role via cli variable (e.g. bootstrap, conductor, regular)
  • Provides example of sample usage.
  • Moves the static shell to a separate shell denoted staticShell, s.t. nix develop allows for compiling to work on macos and linux. To access the shell, run nix develop .#staticShell.

Note this runs on our aws cluster. The next step (in a separate PR) will be to include a dockerfile for usage on fargate.

nmccarty pushed a commit that referenced this pull request Mar 7, 2022
chore(deps): bump libp2p from 0.42.2 to 0.43.0
@DieracDelta DieracDelta force-pushed the jr/remove_json_config branch 2 times, most recently from edeaafb to 081c9f9 Compare March 7, 2022 17:50
libp2p-networking/Cargo.toml Outdated Show resolved Hide resolved
libp2p-networking/examples/common/mod.rs Outdated Show resolved Hide resolved
libp2p-networking/README Outdated Show resolved Hide resolved
libp2p-networking/examples/common/mod.rs Show resolved Hide resolved
libp2p-networking/examples/common/mod.rs Outdated Show resolved Hide resolved
libp2p-networking/examples/common/mod.rs Outdated Show resolved Hide resolved
@DieracDelta DieracDelta force-pushed the jr/remove_json_config branch from 081c9f9 to 1866107 Compare March 8, 2022 14:55
VictorKoenders
VictorKoenders previously approved these changes Mar 8, 2022
libp2p-networking/README Outdated Show resolved Hide resolved
@VictorKoenders
Copy link
Contributor

Oh look new CVE just dropped, right when your PR was ready to be merged https://rustsec.org/advisories/RUSTSEC-2022-0013

@DieracDelta DieracDelta mentioned this pull request Mar 8, 2022
@VictorKoenders VictorKoenders merged commit e902862 into main Mar 8, 2022
@VictorKoenders VictorKoenders deleted the jr/remove_json_config branch March 8, 2022 17:46
VictorKoenders added a commit that referenced this pull request Mar 8, 2022
Replace identity_mappings.json with cli variables
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.

2 participants