Skip to content

[tenant-namespace-operator] Upgraded to 1.32.0 - #70

Merged
plnordquist-pnnl merged 16 commits into
pnnl-miscscripts:masterfrom
plnordquist-pnnl:tno-latest
Nov 6, 2023
Merged

[tenant-namespace-operator] Upgraded to 1.32.0#70
plnordquist-pnnl merged 16 commits into
pnnl-miscscripts:masterfrom
plnordquist-pnnl:tno-latest

Conversation

@plnordquist-pnnl

Copy link
Copy Markdown
  • Upgraded the operator to the latest operator-sdk
  • Addressed ansible-lint issues
  • Removed obsolete ingress upgrade code
  • Added leader election role for coordination.k8s.io
  • Added default restricted securityContext settings
  • Added node affinity based on image arch/os
  • Switched to using check/diff mode in Ansible
  • Used helm task diff output as our diff output

I spent time testing each incremental version through the commits. I think I need one last final test from the 1.4.0 operator to this version again with k8s 1.23 and it should be tested well enough.

@plnordquist-pnnl

Copy link
Copy Markdown
Author

Testing with 1.23.x exposed an issue with the newer ingress chart where Kubernetes 1.24 granted leases.coordination.k8s.io to namespace administrators by default. The operator is unable to deploy the tenant-namespace chart in 1.23 due to the missing RBAC rules. I've added these rules now and cleaned up some duplicates I found.

@kfox1111kfox1111 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM. Thanks! :)

@plnordquist-pnnl
plnordquist-pnnl merged commit 4b98561 into pnnl-miscscripts:masterNov 6, 2023
@plnordquist-pnnl
plnordquist-pnnl deleted the tno-latest branch April 2, 2024 18:00
Sign up for freeto 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

@plnordquist-pnnl@kfox1111