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

operator: Update golang to 1.20; Update dependencies to latest #8776

Merged
merged 15 commits into from Mar 13, 2023

Conversation

Red-GV
Copy link
Contributor

@Red-GV Red-GV commented Mar 10, 2023

What this PR does / why we need it:
This PR updates go to 1.20, Kubernetes dependencies to 1.26.2, OpenShift dependencies to release-4.13, and updates most of the bingo tools.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Checklist

  • Reviewed the CONTRIBUTING.md guide (required)
  • Documentation added
  • Tests updated
  • CHANGELOG.md updated
  • Changes that require user attention or interaction to upgrade are documented in docs/sources/upgrading/_index.md

@Red-GV Red-GV requested a review from a team as a code owner March 10, 2023 19:30
@Red-GV Red-GV force-pushed the operator-update-dependencies branch from 9feaafb to 1479585 Compare March 10, 2023 21:43
Copy link
Collaborator

@periklis periklis left a comment

Choose a reason for hiding this comment

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

Looks overall good to me. Just double checking: Aren't any operator-sdk migrations needed between 1.24 and 1.27?

@@ -5,5 +5,5 @@ apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
images:
- name: controller
newName: quay.io/openshift-logging/loki-operator
newTag: v0.1.0
Copy link
Collaborator

Choose a reason for hiding this comment

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

Needs to be reverted back

@periklis
Copy link
Collaborator

@Red-GV I suggest to consider adding the following TLSOpts in a follow-up PR (kubernetes-sigs/controller-runtime#2023) to support TLSProfile on the webhook. I recall we had a request about this.

operator/go.mod Outdated Show resolved Hide resolved
operator/go.mod Show resolved Hide resolved
Copy link
Collaborator

@periklis periklis left a comment

Choose a reason for hiding this comment

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

Testing this on upcoming releases of OpenShift 4.13 requires the following patch:

@@ -259,7 +259,7 @@ olm-deploy:  ## Deploy the operator bundle and the operator via OLM into an Kube
 	$(error Set variable REGISTRY_BASE to use a custom container registry org account for local development)
 else
 olm-deploy: olm-deploy-bundle olm-deploy-operator $(OPERATOR_SDK)
-	$(OPERATOR_SDK) run bundle -n $(LOKI_OPERATOR_NS) --install-mode AllNamespaces $(BUNDLE_IMG)
+	$(OPERATOR_SDK) run bundle -n $(LOKI_OPERATOR_NS) --install-mode AllNamespaces $(BUNDLE_IMG) --security-context-config restricted
 endif

@Red-GV Red-GV force-pushed the operator-update-dependencies branch from 7c9692a to 5049406 Compare March 13, 2023 13:21
@Red-GV Red-GV force-pushed the operator-update-dependencies branch from 5049406 to d50426e Compare March 13, 2023 13:21
@Red-GV
Copy link
Contributor Author

Red-GV commented Mar 13, 2023

@periklis No major operator-sdk migrations. The last one was for 1.24 -> 1.25, but we are updating to k8s 1.26 instead of 1.25, so I ignored that one.

@periklis periklis merged commit a926ae3 into grafana:main Mar 13, 2023
7 checks passed
@Red-GV Red-GV deleted the operator-update-dependencies branch March 21, 2023 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants