prometheus-operator icon indicating copy to clipboard operation
prometheus-operator copied to clipboard

ScrapeConfig CRD: Proposal For Extending The CRD Surface

Open mviswanathsai opened this issue 1 year ago • 5 comments

Description

Proposal for extending the ScrapeConfig API surface to pave the way for graduation to v1beta1.

Type of change

What type of changes does your code introduce to the Prometheus operator? Put an x in the box that apply.

  • [ ] CHANGE (fix or feature that would cause existing functionality to not work as expected)
  • [ ] FEATURE (non-breaking change which adds functionality)
  • [ ] BUGFIX (non-breaking change which fixes an issue)
  • [ ] ENHANCEMENT (non-breaking change which improves existing functionality)
  • [ x] NONE (if none of the other choices apply. Example, tooling, build system, CI, docs, etc.)

Verification

Please check the Prometheus-Operator testing guidelines for recommendations about automated tests.

Changelog entry

Please put a one-line changelog entry below. This will be copied to the changelog file during the release process.


mviswanathsai avatar May 30 '24 08:05 mviswanathsai

Hey! any further comments?

mviswanathsai avatar Jul 31 '24 09:07 mviswanathsai

This is still valid, can we mention which 3 SD's you meant here?

I don't think this is valid anymore, I have changed it to reflect the current scenario.

mviswanathsai avatar Aug 27 '24 06:08 mviswanathsai

Above comment still valid

Not sure I understand what is implied here :)

mviswanathsai avatar Aug 27 '24 06:08 mviswanathsai

This is still valid, can we mention which 3 SD's you meant here?

I don't think this is valid anymore, I have changed it to reflect the current scenario.

Hmmm not sure while checking file changed it was still showing me same. But if you already updated it's ok.

slashpai avatar Aug 27 '24 07:08 slashpai

Ah! my bad, I failed to push it.

mviswanathsai avatar Aug 27 '24 07:08 mviswanathsai

PTAL @simonpasquier @slashpai

mviswanathsai avatar Oct 07 '24 09:10 mviswanathsai

@mviswanathsai my changes don't pass the doc linter. Would you mind fixing the format and I'll merge :)

simonpasquier avatar Oct 10 '24 15:10 simonpasquier

:partying_face:

simonpasquier avatar Oct 11 '24 15:10 simonpasquier

Woah, finally!

mviswanathsai avatar Oct 13 '24 10:10 mviswanathsai