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: setup 0.8.0 #991

Merged

Conversation

svghadi
Copy link
Collaborator

@svghadi svghadi commented Sep 11, 2023

What type of PR is this?
/kind chore

What does this PR do / why we need it:
cherry-pick #966 to refactoring branch to set it up for 0.8

Signed-off-by: Siddhesh Ghadi <[email protected]>
name: argocd-operator-controller-manager-metrics-service
spec:
ports:
- name: https
port: 8443
targetPort: https
selector:
control-plane: argocd-operator
control-plane: controller-manager
Copy link
Collaborator

@jaideepr97 jaideepr97 Sep 14, 2023

Choose a reason for hiding this comment

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

@svghadi is this an intentional change? for the 0.7 bundle?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I cherry-picked the changes from #966.

According to #966 (comment) looks like a intentional change.

@jaideepr97 jaideepr97 merged commit b529e86 into argoproj-labs:operator-redesign Sep 14, 2023
2 of 3 checks passed
@svghadi svghadi deleted the redesign-setup-0.8 branch October 5, 2023 10:56
Julik24 pushed a commit to Julik24/argocd-operator that referenced this pull request Apr 24, 2024
Signed-off-by: Siddhesh Ghadi <[email protected]>
Co-authored-by: Regina Scott <[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