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

[uss_qualifier] check constraint reference synchronization across multiple DSS instances #687

Closed
wants to merge 2 commits into from

Conversation

Shastick
Copy link
Contributor

@Shastick Shastick commented May 8, 2024

Adds a scenario the checks that Constraint References are properly synchronized across a set of DSS instances.

This is a relatively big PR. The new validator and scenario are an almost 1-1 equivalent of what is done for the operational intents.

Part of the effort tracked by #434

@Shastick Shastick force-pushed the constraint-ref-sync branch from 3b38e4f to 2d9b7fc Compare May 28, 2024 15:14
@BenjaminPelletier
Copy link
Member

I haven't been able to find a block of time big enough to fully cache this entire PR -- I think we should split it into smaller pieces unless @mickmis or @barroco are able to review instead

@Shastick
Copy link
Contributor Author

I put that on my list of things to do for Friday. Splitting this according to each step of the scenario should not be problematic.

@Shastick
Copy link
Contributor Author

Shastick commented May 30, 2024

@BenjaminPelletier the first slice has been pushed to #704

edit: Note, a series of PRs is now ready on:

All except the first one are left as draft, but they are ready for review: if anyone would want to look at them, you may see the relevant changes by only showing the changes of the last commit.

@Shastick Shastick closed this Jun 10, 2024
@Shastick Shastick deleted the constraint-ref-sync branch June 11, 2024 09:05
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