Commit Graph

390 Commits

Author SHA1 Message Date
James Munnelly
0706fb8847 Use cert-manager installed via helm in e2e tests
Print pod logs on test failure

Remove old flags from e2e_test makefile target
2017-12-01 18:53:42 +00:00
jetstack-ci-bot
6ad35b3a18
Merge pull request #212 from munnerz/ingress-shim-fixes
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>.

Ingress shim fixes

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

Fixes ingress-shim image name in helm chart
Allow specifying extraArgs to ingress-shim in helm chart

**Release note**:
```release-note
NONE
```

/assign
2017-12-01 16:41:02 +00:00
James Munnelly
638733f617 Add ingressShim.extraArgs param to helm chart 2017-12-01 15:28:02 +00:00
James Munnelly
d9e44d2af9 Correctly set default image-shim image repo 2017-12-01 13:43:27 +00:00
jetstack-ci-bot
3f62be9421
Merge pull request #210 from munnerz/ingress-shim
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 ingress-shim controller to create Certificates based on annotations on ingress resources

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

Adds a new controller, ingress-shim, that will automatically create Certificate resources for ingress resources with annotations on them.

By setting defaults on the ingress-shim CLI, it's possible to re-implement the `kubernetes.io/tls-acme` behavior of kube-lego.

There also needs to be some documentation written about this, as well as a user guide. This can be our migration story for users coming from kube-lego as well. It'd be interesting to know how easily it'd be to support a seamless migration from [kube-cert-manager](https://github.com/PalmStoneGames/kube-cert-manager) through this tool too.

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #

Fixes #19 

**Special notes for your reviewer**:

This includes unit tests for sync.go in the new controller, as well as a basic e2e test that ensures an ACME certificate is obtained correctly when an annotation is set on an ingress.

**Release note**:
```release-note
Add ingress-shim controller to automatically create Certificate resources based on annotations on ingresses. This allows for easy creation of Certificate resources when using ingress.
```

Slightly related to #97
2017-12-01 12:46:55 +00:00
James Munnelly
5a18c1aff2 Add unit test cases for dns01 buildCertificates 2017-12-01 12:30:51 +00:00
James Munnelly
d91963db85 Raise boulder e2e test rate limits 2017-12-01 11:59:17 +00:00
James Munnelly
2068f7bb57 Add dummy ingress path 2017-12-01 11:22:40 +00:00
James Munnelly
b2f5698145 Fix flag redefined error 2017-12-01 11:05:46 +00:00
James Munnelly
45c3c8b283 Remove unused var 2017-12-01 10:27:47 +00:00
James Munnelly
9b8a55d55c Import ingress e2e test suite 2017-12-01 01:53:57 +00:00
James Munnelly
70e2a2ec60 Fix check for certificate in ingress-shim e2e 2017-12-01 01:46:27 +00:00
James Munnelly
3ccadb5f73 Fix image-shim image in e2e tests 2017-12-01 01:38:30 +00:00
James Munnelly
2958698686 Include ingress-shim in default helm chart deployment 2017-12-01 01:38:19 +00:00
James Munnelly
9d5620a84f Add ingress-shim e2e test 2017-12-01 01:20:20 +00:00
James Munnelly
1542db57d2 Add ingress-shim sync unit tests 2017-12-01 00:56:26 +00:00
James Munnelly
56d3bd5eca Add dockerfile for ingress-shim 2017-11-30 22:56:17 +00:00
James Munnelly
c3be0f204e Add ingress-shim controller to create Certificates based on annotations on ingress resources 2017-11-30 22:53:54 +00:00
jetstack-ci-bot
2accacb219
Merge pull request #200 from euank/go-1.9
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>.

Update to go 1.9

1.10 isn't too far off, at which point 1.8 will no longer receive security updates. It seems simple enough to bump sooner rather than wait until the last second.

**Release note**:
```release-note
NONE
```
2017-11-22 14:26:41 +00:00
Euan Kemp
82c4119539 Update to go 1.9 2017-11-21 18:00:08 -08:00
jetstack-ci-bot
e56d051b72
Merge pull request #202 from munnerz/refactor-makefile
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>.

Clean up Makefile and place Dockerfiles in subdirectory

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

Cleans up the Makefile to have more consistent targets and more readable/usable

**Release note**:
```release-note
NONE
```

/assign
2017-11-22 01:49:02 +00:00
James Munnelly
8c7eaa1db8 Clean up Makefile and place Dockerfiles in subdirectory 2017-11-22 01:19:46 +00:00
jetstack-bot
e6cdbce52a
Merge pull request #196 from euank/]]]
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>.

Fix doc link typo

```release-note
NONE
```
2017-11-13 16:24:06 +00:00
Euan Kemp
0dbee6df5b Fix doc link typo 2017-11-12 22:27:18 -08:00
jetstack-bot
ea3a6310f7
Merge pull request #193 from aaron-trout/no-drupal
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>.

s/Drupal/cert-manager/

Replace the word 'Drupal' with 'cert-manager'. Do I detect copypasta? :D
2017-11-07 17:41:51 +00:00
Aaron Trout
8c0919381c s/Drupal/cert-manager/ 2017-11-07 17:27:15 +00:00
James Munnelly
31cc4f8123 Update for 0.2.1 release 2017-11-04 13:50:05 +00:00
jetstack-bot
e32147b695
Merge pull request #183 from munnerz/fix-issue-loop
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>.

Improve validation of certificates. Fix bug in checking certificate validity

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

Improves the validation of dnsNames and commonNames on certificate resources.
Fixes a bug in checking certificate validity.

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #

Fixes #176, fixes #175

**Release note**:
```release-note
Fix a bug in checking certificate validity and improve validation of dnsNames and commonName
```
2017-11-04 01:00:30 +00:00
James Munnelly
ffbfe2da3d Use ordered RemoveDuplicates. Add additional test cases. 2017-11-04 00:44:08 +00:00
jetstack-bot
de32d7d8d7
Merge pull request #186 from munnerz/skip-generated-packages
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>.

Don't test pkg/client/ and third_party/ packages

Don't run tests for generated files

**Release note**:
```release-note
NONE
```
2017-11-04 00:34:57 +00:00
jetstack-bot
04a6705ce5
Merge pull request #184 from munnerz/acme-validation
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 basic validation to ACME issuer

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

Improves validation of existing certificate resources

**Release note**:
```release-note
Fix bugs when checking validity of certificate resources
```
2017-11-04 00:29:24 +00:00
James Munnelly
a5b954658c Fix panic in certificates controller 2017-11-04 00:27:22 +00:00
James Munnelly
4cf492e729 Don't test pkg/client/ and third_party/ packages 2017-11-04 00:21:50 +00:00
jetstack-bot
91c543c208
Merge pull request #185 from munnerz/dns-unit-tests
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 solverFor and wait.go unit tests for acme dns solver

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

Adds some unit tests for ACME dns solver

**Release note**:
```release-note
NONE
```
2017-11-04 00:14:20 +00:00
James Munnelly
7f656f9ed4 Update CA issuer 2017-11-04 00:12:33 +00:00
James Munnelly
74d711ceef Add unit tests for cert->dnsName and cert->CN functions 2017-11-04 00:10:42 +00:00
James Munnelly
f3db0df7b6 Add RemoveDuplicates unit test 2017-11-03 23:58:25 +00:00
James Munnelly
6ac437699d Improve validation of certificates. Fix bug in checking certificate validity 2017-11-03 23:48:18 +00:00
James Munnelly
422f8fc4be Add basic validation to ACME issuer 2017-11-03 23:35:58 +00:00
James Munnelly
01fa76e00a Add solverFor and wait.go unit tests for acme dns solver 2017-11-03 23:34:37 +00:00
James Munnelly
d6fcee3b8f Prepare v0.2.0 release 2017-11-03 18:23:46 +00:00
jetstack-bot
2f159a0f00
Merge pull request #145 from jetstack-experimental/move-org
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>.

Move to github.com/jetstack/cert-manager repo

**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #

Closes #134 

**Release note**:

```release-note
Move to 'jetstack' organisation.

Action required: this will require updating your existing deployments to point to the new image repository, as new tags will not be pushed to the old 'jetstackexperimental/cert-manager-controller` repository.
```

/area admin
/assign
/release-note-action-required
2017-11-03 17:03:43 +00:00
James Munnelly
fa7e052ac1 Move to github.com/jetstack/cert-manager repo 2017-11-03 16:41:39 +00:00
jetstack-bot
16e0770d06
Merge pull request #179 from munnerz/tidy-up-informers
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>.

Tidy up informers

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

Refactors the controllers and issuers to use the new vendored and filtered informers

**Special notes for your reviewer**:

Do not merge until #178 has merged

**Release note**:
```release-note
No longer support ClusterIssuer resources when cert-manager is running with `--namespace` flag set
```

/assign
2017-11-03 16:11:06 +00:00
James Munnelly
6bd606a7ce Fix ClusterIssuers. Update test name. Add comments. 2017-11-03 15:51:34 +00:00
James Munnelly
eb4be6859e Update controllers and issuers for new SharedInformerFactory 2017-11-03 15:26:19 +00:00
James Munnelly
c77ee5e5b2 Run update-client-gen.sh 2017-11-03 15:26:19 +00:00
James Munnelly
3552a21a71 Generate informers for Kubernetes types 2017-11-03 15:26:19 +00:00
James Munnelly
a4468c76c5 Use munnerz/code-generator 2017-11-03 15:26:19 +00:00
jetstack-bot
568079ee97
Merge pull request #178 from munnerz/codegen-script
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>.

Switch to using generate-group script in code-generator

**Release note**:
```release-note
NONE
```
2017-11-03 15:25:01 +00:00