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

code-refactoring: Add argocd server controller #1063

Merged

Conversation

svghadi
Copy link
Collaborator

@svghadi svghadi commented Nov 9, 2023

What type of PR is this?
/kind code-refactoring

What does this PR do / why we need it:
Adds server controller

How to test changes / Special notes to the reviewer:

@svghadi svghadi changed the title WIP: Server pkg code-refactoring: Add argocd server controller Nov 13, 2023
@svghadi svghadi marked this pull request as ready for review November 13, 2023 13:13
controllers/argocd/server/deployment.go Outdated Show resolved Hide resolved
controllers/argocd/server/route_test.go Outdated Show resolved Hide resolved
controllers/argocd/server/route_test.go Outdated Show resolved Hide resolved
controllers/argocd/sso/dex/dex.go Outdated Show resolved Hide resolved
controllers/argocd/sso/dex/constants.go Outdated Show resolved Hide resolved
controllers/argocd/server/constants.go Outdated Show resolved Hide resolved
controllers/argocd/server/constants.go Outdated Show resolved Hide resolved
common/keys.go Show resolved Hide resolved
common/values.go Outdated Show resolved Hide resolved
@svghadi svghadi force-pushed the server-pkg-redesign branch 6 times, most recently from cda0cd4 to 03d6e07 Compare January 10, 2024 13:51
@jaideepr97
Copy link
Collaborator

@svghadi added some generic comments that should be applied across all resources managed by the server
Please rebase after all these PRs are merged and I will do another round of reivew

#1161
#1167
#1168
#1170
#1171

Additionally, please make sure to move all the server related functions that you have already implemented from the old code into the correct TOBEREMOVED.go file so we can be sure that we didn't miss refactoring any existing logic

@svghadi svghadi force-pushed the server-pkg-redesign branch from 03d6e07 to 30a8fc3 Compare January 21, 2024 15:13
@svghadi svghadi force-pushed the server-pkg-redesign branch from 30a8fc3 to 2dec80f Compare January 29, 2024 06:10
Signed-off-by: Siddhesh Ghadi <[email protected]>
Signed-off-by: Siddhesh Ghadi <[email protected]>
Signed-off-by: Siddhesh Ghadi <[email protected]>
Signed-off-by: Siddhesh Ghadi <[email protected]>
@svghadi svghadi force-pushed the server-pkg-redesign branch from 2dec80f to e71feaa Compare February 28, 2024 14:47
Signed-off-by: Siddhesh Ghadi <[email protected]>
Copy link
Collaborator

@jaideepr97 jaideepr97 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks @svghadi !

@jaideepr97 jaideepr97 merged commit 22338f2 into argoproj-labs:operator-redesign Feb 28, 2024
1 of 3 checks passed
@svghadi svghadi deleted the server-pkg-redesign branch February 29, 2024 04:42
Julik24 pushed a commit to Julik24/argocd-operator that referenced this pull request Apr 24, 2024
* add server controller

---------

Signed-off-by: Siddhesh Ghadi <[email protected]>
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.

3 participants