generated from kubernetes/kubernetes-template-project
-
Notifications
You must be signed in to change notification settings - Fork 105
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
This change updates the config and Makefile to produce multiple variants of the HNC manifests. The initial list is "default" (same as today), "default-cm" (same, but configured for cert-manager instead of the internal cert-controller), and ha (replicated webhooks, using the internal cert controller). Also made some updates to the Prow tests to allow for variant testing, as well as upgrading the ability to test the Prow tests locally. Tested: verified that default.yaml is the same as the prior hnc-manager.yaml, apart from some reordered command line args. Verified that smoke tests work on ha.yaml. Did not do _any_ testing on the cert-manager variants. Locally ran the prow e2e tests on both the HA and default configs. Also did an experimental release from my personal repo and bumped up the timeout since we've added more build targets.
- Loading branch information
1 parent
55c0c2a
commit a2962af
Showing
18 changed files
with
344 additions
and
215 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,4 @@ | ||
This directory is the default configuration of HNC, including internal cert | ||
management ("cc" = "Cert Controller"). It's published as "default_cc.yaml" when | ||
building the manifests ("made manifests" in the root directory) and aliased as | ||
"default.yaml." In versions of HNC prior to v1.0, this was hnc_manager.yaml. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,30 @@ | ||
# Adds namespace to all resources. | ||
namespace: hnc-system | ||
|
||
# Value of this field is prepended to the | ||
# names of all resources, e.g. a deployment named | ||
# "wordpress" becomes "alices-wordpress". | ||
# Note that it should also match with the prefix (text before '-') of the namespace | ||
# field above. | ||
namePrefix: hnc- | ||
|
||
bases: | ||
- ../../crd | ||
- ../../internalcert | ||
- ../../manager | ||
- ../../rbac | ||
- ../../webhook | ||
|
||
patches: | ||
- patch: |- | ||
- op: add | ||
path: /spec/template/spec/containers/0/args/- | ||
value: --enable-internal-cert-management | ||
- op: add | ||
path: /spec/template/spec/containers/0/args/- | ||
value: --cert-restart-on-secret-refresh | ||
target: | ||
group: apps | ||
version: v1 | ||
kind: Deployment | ||
name: controller-manager |
Oops, something went wrong.