Skip to content

Conversation

rodrigovalin
Copy link
Contributor

All Submissions:

A service will be updated on each reconciliation. Needed if Prometheus is added to a MongoDB resource with no Prometheus beforehand.

  • Have you opened an Issue before filing this PR?
  • Have you signed our CLA?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if such).

@@ -5,12 +5,6 @@ kind: CustomResourceDefinition
metadata:
annotations:
controller-gen.kubebuilder.io/version: v0.4.1
service.binding/type: 'mongodb'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I will remove this deletions.

Copy link
Member

@mircea-cosbuc mircea-cosbuc left a comment

Choose a reason for hiding this comment

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

LGTM

@rodrigovalin rodrigovalin force-pushed the fix-prometheus-service-update branch from 36617d0 to 50d10ea Compare March 8, 2022 12:44
@rodrigovalin rodrigovalin merged commit 205b286 into master Mar 8, 2022
@rodrigovalin rodrigovalin deleted the fix-prometheus-service-update branch March 8, 2022 13:20
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.

2 participants