-
Notifications
You must be signed in to change notification settings - Fork 20
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] scd scenarios should confirm that entity deletion with a wrong OVN/version is impossible #759
Labels
Comments
5 tasks
Shastick
added a commit
to Orbitalize/monitoring
that referenced
this issue
Aug 28, 2024
…the correct OVN First step towards fixing interuss#759
Shastick
added a commit
to Orbitalize/monitoring
that referenced
this issue
Aug 28, 2024
…the correct OVN First step towards fixing interuss#759
Shastick
added a commit
to Orbitalize/monitoring
that referenced
this issue
Aug 28, 2024
…the correct OVN First step towards fixing interuss#759
Shastick
added a commit
to Orbitalize/monitoring
that referenced
this issue
Aug 29, 2024
…the correct OVN First step towards fixing interuss#759
Shastick
added a commit
to Orbitalize/monitoring
that referenced
this issue
Aug 29, 2024
…the correct OVN First step towards fixing interuss#759
Shastick
added a commit
to Orbitalize/monitoring
that referenced
this issue
Sep 5, 2024
…the correct OVN First step towards fixing interuss#759
Shastick
added a commit
to Orbitalize/monitoring
that referenced
this issue
Sep 5, 2024
…the correct OVN First step towards fixing interuss#759
Shastick
added a commit
to Orbitalize/monitoring
that referenced
this issue
Sep 5, 2024
…the correct OVN First step towards fixing interuss#759
Shastick
added a commit
to Orbitalize/monitoring
that referenced
this issue
Sep 6, 2024
…the correct OVN First step towards fixing interuss#759
Shastick
added a commit
to Orbitalize/monitoring
that referenced
this issue
Sep 10, 2024
…the correct OVN First step towards fixing interuss#759
Shastick
added a commit
to Orbitalize/monitoring
that referenced
this issue
Sep 10, 2024
…the correct OVN First step towards fixing interuss#759
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
The DSS has a missing OVN check in its OIR deletion endpoint, which the uss_qualifier currently does not detect. The same problem affects the CR deletion endpoint.
Request
This a kind of bug that we would want the qualifier to catch.
We should either create a new scenario, or extend an existing one, with a step that explicitly tries to delete an OIR with the wrong OVN.
Notes
We could easily add some steps to cover the same feature for Subscriptions and Constraint References, if their deletion with the proper version/OVN is not already covered
We should also verify that entity mutation with a wrong OVN is properly by scenarios.
Plan
OIRSimple
equivalent to theSubscriptionSimple
scenario. We should probably create one.CRSimple
equivalent to theSubscriptionSimple
scenario. We should probably create one.Each of the scenarios above can also be enriched with an explicit test that will attempt mutating an entity. This is covered by #760.
Progress:
The text was updated successfully, but these errors were encountered: