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 peers to use store methods #2893

Merged
merged 132 commits into from
Jan 20, 2025
Merged

[management] Refactor peers to use store methods #2893

merged 132 commits into from
Jan 20, 2025

Conversation

bcmmbaga
Copy link
Contributor

@bcmmbaga bcmmbaga commented Nov 14, 2024 •
edited
Loading

Describe your changes

Remove calls to get and save whole account in peers 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 added 30 commits November 8, 2024 00:31
@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 and others added 26 commits January 14, 2025 22:13
@bcmmbaga
Refactor peer scheduler to retry every 3 seconds on errors
84aea32 
Signed-off-by: bcmmbaga <[email protected]>
@pascal-fischer
update expectations
29ea44b
@pascal-fischer
fix validator
61b38e5
@pascal-fischer
fix validator
a5731fe
@pascal-fischer
fix validator
44f69c7
@pascal-fischer
update timeouts
b1e8ed8
@bcmmbaga
Refactor ToGroupsInfo to process slices of groups
b15ee5c 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
Merge remote-tracking branch 'origin/peers-get-account-refactoring' i…
7baef8c 
…nto peers-get-account-refactoring
@pascal-fischer
update expectations
30b023d
@pascal-fischer
Merge remote-tracking branch 'origin/peers-get-account-refactoring' i…
7241a16 
…nto peers-get-account-refactoring
@pascal-fischer
update expectations
167c80d
@pascal-fischer
update expectations
e27db94
@bcmmbaga
Bump integrations version
6a81ca2 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
Refactor GetValidatedPeers
2fb399a 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
Fix tests
22798ad 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
go mod tidy
6367fe4 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
Use peers and groups map for peers validation
4afaabb 
Signed-off-by: bcmmbaga <[email protected]>
@pascal-fischer
remove mysql from api benchmark tests
aeb6e53
@bcmmbaga
Skip user check for system initiated peer deletion
30ae3aa 
Signed-off-by: bcmmbaga <[email protected]>
@bcmmbaga
Remove context in db calls
d962f7f 
Signed-off-by: bcmmbaga <[email protected]>
@pascal-fischer
[management] Improve group peer/resource counting (#3192)
97ff9a3
@pascal-fischer
add metrics
93c8f94
@pascal-fischer
revert: add some logs
232c978
@pascal-fischer
Merge branch 'main' into peers-get-account-refactoring
5acfe00 
# Conflicts:
#	management/server/peer.go
@pascal-fischer
fix import
f1ee844
@pascal-fischer
[management] extract getSettings calls from transactions (#3206)
9af9573
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
4 Accepted issues

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

See analysis details on SonarQube Cloud

@pascal-fischer pascal-fischer merged commit 1ad2cb5 into main Jan 20, 2025
41 checks passed
@pascal-fischer pascal-fischer deleted the peers-get-account-refactoring branch January 20, 2025 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pascal-fischer pascal-fischer pascal-fischer approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bcmmbaga @pascal-fischer