Commit Graph

10 Commits

Author SHA1 Message Date
jetstack-ci-bot
97ce5ca2b3
Merge pull request #329 from munnerz/default-cluster-namespace
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Set default cluster resource namespace to current pod namespace

**What this PR does / why we need it**:

Changes the default cluster resource namespace from kube-system to the current namespace of the cert-manager deployment.

**Which issue this PR fixes**: fixes #103 

**Release note**:
```release-note
Supporting resources for ClusterIssuer's (e.g. signing CA certificates, or ACME account private keys) will now be stored in the same namespace as cert-manager, instead of kube-system in previous versions. Action required: you will need to ensure to properly manually migrate these referenced resources across into the deployment namespace of cert-manager, else cert-manager may not be able to find account private keys or signing CA certificates.
```

/cc @mikebryant
2018-02-23 19:56:21 +00:00
James Munnelly
105c6c149e Create a Namespace resource as part of the static manifest bundle 2018-02-23 09:13:06 +00:00
James Munnelly
4afc72d166 Update default deployment namespace to be 'cert-manager' 2018-02-22 07:35:54 +00:00
James Munnelly
ce73a22f6f Set default cluster resource namespace to current pod namespace 2018-02-22 07:24:29 +00:00
jetstack-ci-bot
362735f8f1
Merge pull request #312 from Mikulas/pr/cert-crd-alias
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Add default shortNames to certificates CRD

Defaults to `[cert, certs]` and is configurable with `certificateCRDShortNames` parameter.

**What this PR does / why we need it**:

Simplifies manual certificate management with kubectl.

Fixes #311

<div name="review-notes" />

**Special notes for your reviewer**:

Instead of a boolean switch do/dont include the shortNames, the value defines the aliases. This may be handy if anybody prefers `[crt, crts]` instead.

I'm not too keen on the `certificateCRDShortNames` variable name. It might be better to use `Resource` instead of `CRD` to be consistent with the `createCustomResource` var.

Other CRDs are probably ok without an alias, but other people workflows may differ. Should these also be configurable? In that case, the variables could be `shortNames: {certificates: [], …}`.

**Release note**:

```release-note
Add Certificate CRD shortnames `cert` and `certs`. This is configurable in the Helm Chart with `certificateResourceShortNames`.
```
2018-02-21 20:48:17 +00:00
Mikuláš Dítě
d884404159 Add default shortNames to certificates CRD
Defaults to [cert, certs] and is configurable with
`certificateCRDShortNames` parameter.

[Closes #311]
2018-02-21 20:37:25 +01:00
Mike Bryant
0274964100 fix: Use ConfigMaps for leaderelection
Fixes #293
2018-02-21 17:48:13 +00:00
James Munnelly
5aa3c091e9 Run hack/update-deploy-gen.sh 2018-02-05 19:16:47 +00:00
James Munnelly
c9f1c0e07d Run hack/update-deploy-gen.sh 2018-01-25 16:28:15 +00:00
James Munnelly
0d3790567a Add deployment manifest generation scripts 2018-01-25 16:27:29 +00:00