-
Notifications
You must be signed in to change notification settings - Fork 150
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
Update Operator SDK dependency #203
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not an expert of this area, but matches the changes I made before my attempt to lock in version 0.8.2 (see #199). hence LGTM.
os.Exit(1) | ||
for _, f := range []func(*apiruntime.Scheme) error{ | ||
apis.AddToScheme, | ||
cdiv1alpha1.AddToScheme, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's weird that we aren't adding kubevirt to the Scheme.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's because kubevirt adds itself when you import the pkg. In other words, there's an init
somewhere in kubevirt that does it.
It seems Gopkg.toml and Gopkg.lock gone offsync. Gopkg.lock already updated in kubevirt#203 with the right ssp-operator version. Signed-off-by: Francesco Romani <fromani@redhat.com>
* Gopkg.toml: dep: bump the ssp-operator dependency to v1.0.8 It seems Gopkg.toml and Gopkg.lock gone offsync. Gopkg.lock already updated in #203 with the right ssp-operator version. Signed-off-by: Francesco Romani <fromani@redhat.com> * ssp: deploy MetricsAggregation CR Starting version 1.0.6, the SSP operator gained the ability to deploy the prometheus aggregation rules that kubevirt on openshift wants to expose (e.g. for telemetry). In this patch, we bump the vendored version to 1.0.7 (last released at this point in time), and we make sure to create the new CR. This patch only wants to add code, not to change any of the existing behaviour. Signed-off-by: Francesco Romani <fromani@redhat.com>
No description provided.