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

[management] Refactor policy to use store methods #2878

Open
wants to merge 41 commits into
base: main
Choose a base branch
from
Open

[management] Refactor policy to use store methods #2878

wants to merge 41 commits into from

Conversation

bcmmbaga
Copy link
Contributor

Describe your changes

Remove calls to get and save whole account in policy operations.

Issue ticket number and link

Checklist

  • Is it a bug fix
  • Is a typo/documentation fix
  • Is a feature enhancement
  • It is a refactor
  • Created tests that fail without the change (if possible)
  • Extended the README / documentation, if necessary

@bcmmbaga
Refactor setup key handling to use store methods
389c961 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
add lock to get account groups
78044c2 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
add check for regular user
1a5f3c6 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
get only required groups for auto-group validation
931521d 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
add account lock and return auto groups map on validation
f8b5eed 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
refactor account peers update
106fc75 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
Refactor groups to use store methods
0a70e4c 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
refactor GetGroupByID and add NewGroupNotFoundError
8126d95 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
fix tests
ac05f69 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
Merge branch 'feature/get-account-refactoring' into setupkey-get-acco…
40af1a5 
…unt-refactoring

# Conflicts:
#	management/server/sql_store.go
@bcmmbaga
Merge branch 'setupkey-get-account-refactoring' into groups-get-accou…
d58cf50 
…nt-refactoring

# Conflicts:
#	management/server/sql_store.go
@bcmmbaga
Add AddPeer and RemovePeer methods to Group struct
7100be8 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
Preserve store engine in SqlStore transactions
6dc185e 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
Run groups ops in transaction
bdeb95c 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
fix missing group removed from setup key activity
3ed8b9c 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
Merge branch 'setupkey-get-account-refactoring' into groups-get-accou…
cc04aef 
…nt-refactoring
@bcmmbaga
fix merge
871500c 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
Refactor posture checks to remove get and save account
174e07f 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
fix refactor
d54b696 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
fix tests
601d429 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
Merge branch 'main' into groups-get-account-refactoring
010a8bf 
# Conflicts:
#	management/server/group.go
#	management/server/group/group.go
#	management/server/setupkey.go
#	management/server/sql_store.go
#	management/server/status/error.go
#	management/server/store.go
@bcmmbaga
fix merge
664d138 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
fix sonar
ab00c41 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
Change setup key log level to debug for missing group
113c21b 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
Retrieve modified peers once for group events
d23b5c8 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
Merge branch 'groups-get-account-refactoring' into policy-get-account…
ffce48c 
…-refactoring
@bcmmbaga
Refactor policy get and save account to use store methods
0c0fd38 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
Fix tests
2d7f08c 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
Add tests
2806d73 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
Merge branch 'groups-get-account-refactoring' into posturechecks-get-…
00023bf 
…account-refactoring
@bcmmbaga
Add tests
a3abc21 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
Merge branch 'main' into groups-get-account-refactoring
ed259a6 
# Conflicts:
#	management/server/account.go
#	management/server/status/error.go
@bcmmbaga
Merge branch 'groups-get-account-refactoring' into policy-get-account…
147971f 
…-refactoring
@bcmmbaga
Merge branch 'groups-get-account-refactoring' into posturechecks-get-…
446de5e 
…account-refactoring
@bcmmbaga
Merge branch 'posturechecks-get-account-refactoring' into policy-get-…
2a59f04 
…account-refactoring
@bcmmbaga
Retrieve policy groups and posture checks once for validation
32d1b2d 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
Fix typo
bbaee18 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
Merge branch 'posturechecks-get-account-refactoring' into policy-get-…
50e6389 
…account-refactoring
@bcmmbaga
Add policy tests
3a915de 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga bcmmbaga force-pushed the policy-get-account-refactoring branch from 03bb8b0 to 3a915de Compare November 12, 2024 17:16
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Nov 12, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.2% Duplication on New Code

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@bcmmbaga