Skip to content

Render CSR roles when NonClusterHost feature is enabled#3834

Merged
rene-dekker merged 1 commit intotigera:masterfrom
hjiawei:noncluster-host-csr-enable
Mar 25, 2025
Merged

Render CSR roles when NonClusterHost feature is enabled#3834
rene-dekker merged 1 commit intotigera:masterfrom
hjiawei:noncluster-host-csr-enable

Conversation

@hjiawei
Copy link
Contributor

@hjiawei hjiawei commented Mar 25, 2025

Description

This change adds the NonClusterHost resource check to render CSR roles for the CSR controller. It is required to validate and sign CSRs generated from non-cluster hosts.

Should be part of #3817.

For PR author

  • Tests for change.
  • If changing pkg/apis/, run make gen-files
  • If changing versions, run make gen-versions

For PR reviewers

A note for code reviewers - all pull requests must have the following:

  • Milestone set according to targeted release.
  • Appropriate labels:
    • kind/bug if this is a bugfix.
    • kind/enhancement if this is a a new feature.
    • enterprise if this PR applies to Calico Enterprise only.

This change adds the NonClusterHost resource check to render
CSR roles for the CSR controller. It is required to validate and
sign CSRs generated from non-cluster hosts.
@rene-dekker rene-dekker merged commit 2db4bfd into tigera:master Mar 25, 2025
5 checks passed
@hjiawei hjiawei deleted the noncluster-host-csr-enable branch March 25, 2025 21:08
hjiawei added a commit to hjiawei/operator that referenced this pull request Mar 25, 2025
This change adds the NonClusterHost resource check to render
CSR roles for the CSR controller. It is required to validate and
sign CSRs generated from non-cluster hosts.
rene-dekker pushed a commit that referenced this pull request Mar 26, 2025
* Support non-cluster host scaling with Typha (#3817)

* Support non-cluster host scaling with Typha

* Create Kubernetes Clientset from config only once

* Allow certificate signing request from non-cluster hosts

* Add and fix unit tests

* Create a separate key pair for non-cluster host Typha

* Start the non-cluster host Typha autoscaler for enterprise only

* Perform SubjectAccessReview for non-cluster host CSRs

* Start Typha auto-scaler when NonClusterHost resource exists

* Add a 10 second timeout when performing access reviews

* Rerun make generate

* Render CSR roles when NonClusterHost feature is enabled (#3834)

This change adds the NonClusterHost resource check to render
CSR roles for the CSR controller. It is required to validate and
sign CSRs generated from non-cluster hosts.

* FV test with Calico OSS release-v3.30
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.

3 participants