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

fix: add handling of dns multiaddrs + bootstrapping + CLI / Conn changes #547

Merged
merged 6 commits into from
Feb 5, 2024

Conversation

zeeshanlakhani
Copy link
Contributor

@zeeshanlakhani zeeshanlakhani commented Feb 2, 2024

fix: add handling of dns multiaddrs and added kad bootstrapping

Closes #545.
Closes #492.

refactor: transport handling for ws & quic as fallback

Includes:

@zeeshanlakhani zeeshanlakhani force-pushed the zl/dns+star branch 2 times, most recently from 04962ea to 02f07f4 Compare February 2, 2024 19:13
Copy link

codecov bot commented Feb 2, 2024

Codecov Report

Attention: 80 lines in your changes are missing coverage. Please review.

Comparison is base (d420416) 74.91% compared to head (2104385) 77.07%.

❗ Current head 2104385 differs from pull request most recent head 43640c9. Consider uploading reports for the commit 43640c9 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #547      +/-   ##
==========================================
+ Coverage   74.91%   77.07%   +2.15%     
==========================================
  Files          83       83              
  Lines        9537     9778     +241     
==========================================
+ Hits         7145     7536     +391     
+ Misses       2392     2242     -150     
Files Coverage Δ
homestar-runtime/src/cli/show.rs 61.11% <100.00%> (-26.89%) ⬇️
homestar-runtime/src/event_handler.rs 100.00% <100.00%> (ø)
homestar-runtime/src/network/webserver/rpc.rs 71.13% <ø> (ø)
homestar-runtime/src/settings/libp2p_config.rs 100.00% <100.00%> (ø)
homestar-runtime/src/cli.rs 63.76% <88.88%> (+3.76%) ⬆️
homestar-runtime/src/event_handler/event.rs 63.72% <86.66%> (+22.13%) ⬆️
homestar-runtime/src/network/rpc.rs 83.73% <73.33%> (-1.45%) ⬇️
homestar-runtime/src/runner.rs 87.58% <81.81%> (+0.26%) ⬆️
homestar-runtime/src/runner/nodeinfo.rs 45.83% <9.09%> (-18.46%) ⬇️
homestar-runtime/src/runner/response.rs 81.20% <81.03%> (-0.62%) ⬇️
... and 3 more

... and 6 files with indirect coverage changes

@zeeshanlakhani zeeshanlakhani marked this pull request as ready for review February 5, 2024 13:43
@zeeshanlakhani zeeshanlakhani requested a review from a team as a code owner February 5, 2024 13:43
@zeeshanlakhani zeeshanlakhani changed the title fix: add handling of dns multiaddrs fix: add handling of dns multiaddrs + CLI / Conn changes Feb 5, 2024
refactor: transport handling for ws & quic as fallback

Includes:

- dns fallback update
- CLI: removal of -w for running workflows; it's now a positional argument (the first one on run)
  * related to #489
- CLI: node command which showcases nodeinfo, e.g. peer_id and listener addrs/connections
  * related to #489
- Closes #492 with local node bootstrapping
- Closes #297
- Closes #497
Co-authored-by: Brian Ginsburg <[email protected]>
Signed-off-by: Zeeshan Lakhani <[email protected]>
Co-authored-by: Brian Ginsburg <[email protected]>
Signed-off-by: Zeeshan Lakhani <[email protected]>
Zeeshan Lakhani and others added 2 commits February 5, 2024 14:13
Co-authored-by: Brian Ginsburg <[email protected]>
Signed-off-by: Zeeshan Lakhani <[email protected]>
Co-authored-by: Brian Ginsburg <[email protected]>
Signed-off-by: Zeeshan Lakhani <[email protected]>
@zeeshanlakhani zeeshanlakhani changed the title fix: add handling of dns multiaddrs + CLI / Conn changes fix: add handling of dns multiaddrs + bootstrapping + CLI / Conn changes Feb 5, 2024
Copy link
Contributor

@bgins bgins left a comment

Choose a reason for hiding this comment

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

Looks good! Lots of nice changes in here. 💯

@zeeshanlakhani zeeshanlakhani merged commit e0bb469 into main Feb 5, 2024
31 checks passed
@zeeshanlakhani zeeshanlakhani deleted the zl/dns+star branch February 5, 2024 23:04
zeeshanlakhani pushed a commit that referenced this pull request Feb 21, 2024
## 🤖 New release
* `homestar-runtime`: 0.1.1 -> 0.2.0 (⚠️ API breaking changes)
* `homestar-invocation`: 0.1.1 -> 0.2.0 (✓ API compatible changes)
* `homestar-wasm`: 0.1.1 -> 0.2.0 (✓ API compatible changes)
* `homestar-workflow`: 0.1.1 -> 0.2.0 (✓ API compatible changes)

### ⚠️ `homestar-runtime` breaking changes

```
--- failure enum_variant_added: enum variant added on exhaustive enum ---

Description:
A publicly-visible enum without #[non_exhaustive] has a new variant.
        ref: https://doc.rust-lang.org/cargo/reference/semver.html#enum-variant-new
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.28.0/src/lints/enum_variant_added.ron

Failed in:
  variant Command:Node in /tmp/.tmp7PLiiL/homestar/homestar-runtime/src/cli.rs:149
  variant Command:Info in /tmp/.tmp7PLiiL/homestar/homestar-runtime/src/cli.rs:155
```

<details><summary><i><b>Changelog</b></i></summary><p>

## `homestar-runtime`
<blockquote>

##
[0.2.0](homestar-runtime-v0.1.1...homestar-runtime-v0.2.0)
- 2024-02-20

### Added
- Add OpenRPC API docs and associated JSON Schemas
([#534](#534))
- redial `node_addresses` at an interval on connection close
([#529](#529))

### Fixed
- add handling of dns multiaddrs + bootstrapping + CLI / Conn changes
([#547](#547))

### Other
- deps + flake cleanup
([#581](#581))
- Allow dead code default timeout
([#577](#577))
- Update homestar-functions to use cargo component
([#576](#576))
- fix transport order for wss possibility
([#563](#563))
- small comment, sorry
([#561](#561))
- move away from deadlines dealing w/ the runner and wasi-preview 2
wasmtime ([#560](#560))
- docker updates with info command and rpc host update
([#558](#558))
- just test conn ([#544](#544))
- handle this evil workflow_info test
([#543](#543))
- remove unnecessary deps and add tooling for those checks
([#541](#541))
- [chore(cargo)](deps): bump puffin from 0.18.1 to 0.19.0
([#537](#537))
- updates/flaky kills on ci
([#540](#540))
- release docs and cp readmes
([#530](#530))
- port selection and test config generation macro
([#528](#528))
- [chore(cargo)](deps): bump serde_with from 3.4.0 to 3.5.0
([#524](#524))
- [chore(cargo)](deps): bump moka from 0.12.3 to 0.12.4
([#525](#525))
</blockquote>

## `homestar-invocation`
<blockquote>

##
[0.2.0](homestar-invocation-v0.1.1...homestar-invocation-v0.2.0)
- 2024-02-20

### Added
- Add OpenRPC API docs and associated JSON Schemas
([#534](#534))

### Other
- deps + flake cleanup
([#581](#581))
- Update homestar-functions to use cargo component
([#576](#576))
- move away from deadlines dealing w/ the runner and wasi-preview 2
wasmtime ([#560](#560))
- remove unnecessary deps and add tooling for those checks
([#541](#541))
- release docs and cp readmes
([#530](#530))
</blockquote>

## `homestar-wasm`
<blockquote>

##
[0.2.0](homestar-wasm-v0.1.1...homestar-wasm-v0.2.0)
- 2024-02-20

### Other
- deps + flake cleanup
([#581](#581))
- Update homestar-functions to use cargo component
([#576](#576))
- move away from deadlines dealing w/ the runner and wasi-preview 2
wasmtime ([#560](#560))
- remove unnecessary deps and add tooling for those checks
([#541](#541))
- release docs and cp readmes
([#530](#530))
</blockquote>

## `homestar-workflow`
<blockquote>

##
[0.2.0](homestar-workflow-v0.1.1...homestar-workflow-v0.2.0)
- 2024-02-20

### Added
- Add OpenRPC API docs and associated JSON Schemas
([#534](#534))

### Other
- deps + flake cleanup
([#581](#581))
- remove unnecessary deps and add tooling for those checks
([#541](#541))
- release docs and cp readmes
([#530](#530))
</blockquote>


</p></details>

---
This PR was generated with
[release-plz](https://github.com/MarcoIeni/release-plz/).

Signed-off-by: release-plz-ipvm-wg[bot] <144082651+release-plz-ipvm-wg[bot]@users.noreply.github.com>
Co-authored-by: release-plz-ipvm-wg[bot] <144082651+release-plz-ipvm-wg[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants