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

Add additional HA requirements from https://github.com/Azure/DASH/pull/56/ into high-availability-and-scale.md #57

Open
chrispsommers opened this issue Feb 16, 2022 · 4 comments
Assignees
Labels
documentation Improvements or additions to documentation engineering high availability

Comments

@chrispsommers
Copy link
Collaborator

#56 contains the following content in the proposal document. If this is indeed the case, these requirements should go into https://github.com/Azure/DASH/blob/main/documentation/high-avail/design/high-availability-and-scale.md

Microsoft has provided some additional requirements:

  1. HA Interoperability is required between vendors
    • Pairing cards from different vendors is not the typical deployment, but must work
  2. The HA packet format and protocol must be public
    • This allows sniffed/mirrored HA messages to be analyzed
    • No vendor-private protocol is allowed
  3. The HA protocol for syncing active flows could have a base mode and optional modes
    • Additional modes could be defined, for example to the reduce PPS/bps needed for the active sync messages
    • A vendor only needs to support the base mode
    • Any optional modes must also be public
  4. The HA protocol does not need to reliably sync 100% of the flows between cards
    • Ideally all flows are synced. But is ok if a small number of flows (hundreds out of 10s of millions) are missed.
@chrispsommers
Copy link
Collaborator Author

Hi @budgrise we triaged this and the consensus is: could you take the additional "MSFT requirements" captured above, which must have arisen from some conversation and do a PR to add it to the defining HA doc https://github.com/Azure/DASH/blob/main/documentation/high-avail/design/high-availability-and-scale.md? This way MSFT can review and hopefully just get it into the existing spec while we debate the new content you've proposed. Thanks.

@chrispsommers chrispsommers added documentation Improvements or additions to documentation engineering labels Mar 10, 2022
@mmiele
Copy link
Collaborator

mmiele commented Jun 13, 2022

Please, see also Xsight HA proposal.

@mgheorghe
Copy link
Collaborator

@r12f will you be able to review this to see if those are still the correct requirements so we can close this issue?

in https://github.com/sonic-net/DASH/blob/main/documentation/high-avail/high-availability-and-scale.md i see
anything pending for this item ?
image

@r12f r12f self-assigned this Nov 20, 2024
@r12f
Copy link
Collaborator

r12f commented Nov 20, 2024

thanks @mgheorghe for bringing it up! will check on this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation engineering high availability
Projects
None yet
Development

No branches or pull requests

4 participants