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

[Pyrra v0.7.0] Missing changes in CRD? #925

Closed
hsolberg opened this issue Sep 27, 2023 · 3 comments
Closed

[Pyrra v0.7.0] Missing changes in CRD? #925

hsolberg opened this issue Sep 27, 2023 · 3 comments

Comments

@hsolberg
Copy link
Contributor

hsolberg commented Sep 27, 2023

Hi!

We noticed some strange things while testing the new version in our dev cluster. When applying the new CRD from the v0.7.0 tag (or master, both seems to be identical) this change seems to be missing:

"absent": {
"default": true,
"type": "boolean"
},
"burnrates": {
"default": true,
"type": "boolean"
},

absent:
default: true
type: boolean
burnrates:
default: true
type: boolean

Here's the CRD from main/master-branch:
https://github.com/pyrra-dev/pyrra/blob/main/examples/kubernetes/manifests/setup/pyrra-slo-CustomResourceDefinition.yaml#L50-L57

And the one tagged with v0.7.0:
https://github.com/pyrra-dev/pyrra/blob/v0.7.0/examples/kubernetes/manifests/setup/pyrra-slo-CustomResourceDefinition.yaml#L50-L57

Both seem to be missing the new option for disabling absent alerts and burnrates.

Current workaround is to use this CRD https://github.com/pyrra-dev/pyrra/blob/0dce67252ae42e59cde4500603a65fa49dfc5bc4/config/crd/bases/pyrra.dev_servicelevelobjectives.yaml

@metalmatze
Copy link
Member

Oops. Thank you for reporting this.
Sorry for replying only now, due to organising PromCon the last days.

@hsolberg
Copy link
Contributor Author

hsolberg commented Oct 3, 2023

No worries, we used the CRD here as a workaround.
Seems the CRD is still missing the new options though, but the schema is updated.

@metalmatze
Copy link
Member

The generated examples have been updated and are available on the release-0.7. As the examples/ is mostly documentation purposes I'll wait for another bit before releasing v0.7.2.

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

No branches or pull requests

2 participants