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

Bump operator-sdk to 1.17 #692

Merged
merged 12 commits into from
Feb 8, 2022
Merged

Conversation

pavolloffay
Copy link
Member

@pavolloffay pavolloffay commented Feb 7, 2022

This PR bumps operator-skd from 1.13 to 1.17. It follows the SDK upgrade docs and aligns project with those changes.

Notable changes:

  • tools are installed to ./bin
  • unit tests that use envtest require additional settings based on the k8s version used in the makefile - added contributing docs
  • Removed support from admissionregistration.k8s.io/v1beta1, now only v1 is supported (as part of operator-sdk 1.14 upgrade). The v1beta1 was removed in 1.22 and deprecated in 1.16 (added v1). OpenShift 4.6 LTS is based on k8s 1.19.

1.17 update

1.14 update

Signed-off-by: Pavol Loffay p.loffay@gmail.com

* Remove admissionregistration.k8s.io/v1beta1 removed from 1.22, v1 was added in 1.16
* Remove crd:trivialVersions=true,preserveUnknownFields=false" that produce CRDs compatible with 1.11

Signed-off-by: Pavol Loffay <p.loffay@gmail.com>
@pavolloffay pavolloffay changed the title Bump operator-sdk to 1.14 Bump operator-sdk to 1.17 Feb 7, 2022
Signed-off-by: Pavol Loffay <p.loffay@gmail.com>
Signed-off-by: Pavol Loffay <p.loffay@gmail.com>
Signed-off-by: Pavol Loffay <p.loffay@gmail.com>
@@ -15,11 +15,18 @@ spec:
- "--secure-listen-address=0.0.0.0:8443"
- "--upstream=http://127.0.0.1:8080/"
- "--logtostderr=true"
- "--v=10"
- "--v=0"
Copy link
Member Author

Choose a reason for hiding this comment

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

@@ -2,7 +2,7 @@ apiVersion: apiextensions.k8s.io/v1
kind: CustomResourceDefinition
metadata:
annotations:
controller-gen.kubebuilder.io/version: v0.6.0-beta.0
controller-gen.kubebuilder.io/version: v0.8.0
Copy link
Member Author

Choose a reason for hiding this comment

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

Signed-off-by: Pavol Loffay <p.loffay@gmail.com>
Signed-off-by: Pavol Loffay <p.loffay@gmail.com>
Signed-off-by: Pavol Loffay <p.loffay@gmail.com>
@pavolloffay pavolloffay marked this pull request as ready for review February 8, 2022 11:35
@pavolloffay pavolloffay requested review from a team and tigrannajaryan February 8, 2022 11:35
Signed-off-by: Pavol Loffay <p.loffay@gmail.com>
Signed-off-by: Pavol Loffay <p.loffay@gmail.com>
@pavolloffay
Copy link
Member Author

@VineethReddy02 @jpkrohling this is ready for review.

Signed-off-by: Pavol Loffay <p.loffay@gmail.com>
@jpkrohling
Copy link
Member

Can you ping me again once the CI passes? I'll then review and merge this PR.

Signed-off-by: Pavol Loffay <p.loffay@gmail.com>
Signed-off-by: Pavol Loffay <p.loffay@gmail.com>
@pavolloffay
Copy link
Member Author

@jpkrohling all green now!

@jpkrohling jpkrohling merged commit 22c3ccf into open-telemetry:main Feb 8, 2022
ItielOlenick pushed a commit to ItielOlenick/opentelemetry-operator that referenced this pull request May 1, 2024
* Bump operator-sdk to 1.14

* Remove admissionregistration.k8s.io/v1beta1 removed from 1.22, v1 was added in 1.16
* Remove crd:trivialVersions=true,preserveUnknownFields=false" that produce CRDs compatible with 1.11

Signed-off-by: Pavol Loffay <p.loffay@gmail.com>

* Bump operator-sdk to 1.17.0

Signed-off-by: Pavol Loffay <p.loffay@gmail.com>

* Fix

Signed-off-by: Pavol Loffay <p.loffay@gmail.com>

* remove wait

Signed-off-by: Pavol Loffay <p.loffay@gmail.com>

* fix lint

Signed-off-by: Pavol Loffay <p.loffay@gmail.com>

* Add phony

Signed-off-by: Pavol Loffay <p.loffay@gmail.com>

* install tools only to ./bin

Signed-off-by: Pavol Loffay <p.loffay@gmail.com>

* use go install

Signed-off-by: Pavol Loffay <p.loffay@gmail.com>

* guide

Signed-off-by: Pavol Loffay <p.loffay@gmail.com>

* remove ech

Signed-off-by: Pavol Loffay <p.loffay@gmail.com>

* remvoe deprecation

Signed-off-by: Pavol Loffay <p.loffay@gmail.com>

* Remove comment

Signed-off-by: Pavol Loffay <p.loffay@gmail.com>
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