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.

Sign up for GitHub

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

Jump to bottom

Port #2525 and #2531 to master #2548

Merged
merged 5 commits into from
Aug 28, 2020
Merged

Port #2525 and #2531 to master #2548

merged 5 commits into from
Aug 28, 2020

Commits on Aug 28, 2020

  1. ChainSyncClient: gracefully terminate if the chain is uninteresting 

    Fixes #2499 for the ChainSyncClient.
    @mrBliss
    mrBliss committed Aug 28, 2020
    Configuration menu
    Copy the full SHA
    05837c5 View commit details
    Browse the repository at this point in the history

  2. test-infra: simplify mini-protocol restart logic 

    Now a directed edge in the topology (ie a bundle of mini-protocol instances)
restarts for two reasons.

1) The ChainSync client and/or server terminated *without* an exception. It
   restarts at the next slot onset.

2) The node was scheduled to restart at this slot onset. It restarts
   immediately.
    @nfrisby @mrBliss
    nfrisby authored and mrBliss committed Aug 28, 2020
    Configuration menu
    Copy the full SHA
    336cf1e View commit details
    Browse the repository at this point in the history

  3. Introduce the App type synonym for Apps 

    This allows us to later easily add arguments to an `App`, e.g.,
`NodeToClientVersion` or `ControlMessageSTM m`.
    @mrBliss
    mrBliss committed Aug 28, 2020
    Configuration menu
    Copy the full SHA
    5452492 View commit details
    Browse the repository at this point in the history

  4. Pass ControlMessageSTM to the NodeToNode apps 

    The control message isn't used yet.
    @mrBliss
    mrBliss committed Aug 28, 2020
    Configuration menu
    Copy the full SHA
    29896b1 View commit details
    Browse the repository at this point in the history

  5. ChainSyncClient: respond to ControlMessageSTM

    @mrBliss
    mrBliss committed Aug 28, 2020
    Configuration menu
    Copy the full SHA
    8be435e View commit details
    Browse the repository at this point in the history