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

[CSA] No longer read AE.isCombinableWithRemedialAction attribute #1130

Merged
merged 12 commits into from
Oct 24, 2024

Conversation

bqth29
Copy link
Collaborator

@bqth29 bqth29 commented Sep 11, 2024

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

Does this PR already have an issue describing the problem?
The field isCombinableWithRemedialAction was making the CRAC heavier than needed by adding OnConstraint usage rules even though the behavior is already covered by other usage rules.

What kind of change does this PR introduce?
The attribute is no longer read which avoid creating several useless or undesired OnConstraint usage rules for remedial actions.

Signed-off-by: Thomas Bouquet <[email protected]>
@bqth29 bqth29 added cleaning This issue or pull request only concerns improving the overall state of the code swe-csa This issue or pull request addresses a feature or a bug relative to the SWE CSA process labels Sep 11, 2024
@bqth29 bqth29 marked this pull request as ready for review October 16, 2024 09:19
Signed-off-by: Thomas Bouquet <[email protected]>
@bqth29 bqth29 added the PR: waiting-for-review This PR is waiting to be reviewed label Oct 16, 2024
Signed-off-by: Thomas Bouquet <[email protected]>
@bqth29 bqth29 requested a review from jipea October 17, 2024 08:46
jipea
jipea previously approved these changes Oct 17, 2024
@bqth29 bqth29 merged commit 4e82922 into main Oct 24, 2024
10 checks passed
bqth29 added a commit that referenced this pull request Nov 26, 2024
* No longer read attribute

Signed-off-by: Thomas Bouquet <[email protected]>

* Add geo filter

Signed-off-by: Thomas Bouquet <[email protected]>

* remove geographical filter

Signed-off-by: Thomas Bouquet <[email protected]>

* remove getBranchLocation

Signed-off-by: Thomas Bouquet <[email protected]>

* sonar

Signed-off-by: Thomas Bouquet <[email protected]>

---------

Signed-off-by: Thomas Bouquet <[email protected]>
Co-authored-by: jipea <[email protected]>
Signed-off-by: Thomas Bouquet <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleaning This issue or pull request only concerns improving the overall state of the code PR: waiting-for-review This PR is waiting to be reviewed swe-csa This issue or pull request addresses a feature or a bug relative to the SWE CSA process
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants