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

Merges gitrepo and gitops controllers #2537

Merged
merged 3 commits into from
Jun 25, 2024

Conversation

0xavi0
Copy link
Contributor

@0xavi0 0xavi0 commented Jun 19, 2024

Moves all the functionality implemented in the gitrepo controller to the gitopts controller.

Refers to: #2513

@0xavi0 0xavi0 self-assigned this Jun 19, 2024
@0xavi0 0xavi0 force-pushed the 2513-merge-gitrepo-gitopts branch 2 times, most recently from 5061a33 to a9447f0 Compare June 19, 2024 08:54
@0xavi0 0xavi0 force-pushed the 2513-merge-gitrepo-gitopts branch 6 times, most recently from 04f8436 to 08ff212 Compare June 20, 2024 10:06
@0xavi0 0xavi0 marked this pull request as ready for review June 20, 2024 12:07
@0xavi0 0xavi0 requested a review from a team as a code owner June 20, 2024 12:07
@manno manno changed the title Merges gitrepo and gitopts controllers Merges gitrepo and gitops controllers Jun 21, 2024
@0xavi0 0xavi0 force-pushed the 2513-merge-gitrepo-gitopts branch from ec35369 to 3a1a210 Compare June 25, 2024 07:26
@0xavi0 0xavi0 requested a review from manno June 25, 2024 07:51
Copy link
Contributor

@weyfonk weyfonk left a comment

Choose a reason for hiding this comment

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

Nice one! 😎

e2e/single-cluster/status_test.go Show resolved Hide resolved
e2e/single-cluster/status_test.go Show resolved Hide resolved
internal/cmd/controller/grutil/status.go Outdated Show resolved Hide resolved
internal/cmd/controller/grutil/status.go Show resolved Hide resolved
0xavi0 added 2 commits June 25, 2024 13:02
Moves all the functionality implemented in the `gitrepo` controller to
the `gitopts` controller.

Signed-off-by: Xavi Garcia <[email protected]>
It also renames the `finalizerutils` to just `finalizer` and documents
its functions

Signed-off-by: Xavi Garcia <[email protected]>
@0xavi0 0xavi0 force-pushed the 2513-merge-gitrepo-gitopts branch from 3a1a210 to dc4522c Compare June 25, 2024 11:02
Changes documentatin as suggested in code review.

Signed-off-by: Xavi Garcia <[email protected]>
@0xavi0 0xavi0 force-pushed the 2513-merge-gitrepo-gitopts branch from dc4522c to a85ccff Compare June 25, 2024 12:05
@0xavi0 0xavi0 requested a review from weyfonk June 25, 2024 12:26
Copy link
Contributor

@weyfonk weyfonk left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@0xavi0 0xavi0 merged commit f9450f6 into rancher:main Jun 25, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants