Commit Graph

82 Commits

Author SHA1 Message Date
James Munnelly
1b8a286206 Rename Setup->Init and use Start to start informers
Signed-off-by: James Munnelly <james@munnelly.eu>
2019-08-22 13:11:33 +01:00
jetstack-bot
dcba8ebd06
Merge pull request #2002 from munnerz/internal-apiversion
Add internal API version, enable defaulter & conversion-gen and implement mutating webhook
2019-08-21 21:53:36 +01:00
James Munnelly
6052e0558e Refactor base controller and allow Running additional informers
Signed-off-by: James Munnelly <james@munnelly.eu>
2019-08-21 13:22:57 +01:00
James Munnelly
73d66c44ab Move API validations package
Signed-off-by: James Munnelly <james@munnelly.eu>
2019-08-21 13:11:01 +01:00
JoshVanL
08c245ad7e Set venafi RetrieveError as failed
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-19 16:04:12 +01:00
JoshVanL
a2f9d9935c Updates venafi CR tests to use full sync from base controller
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-19 16:04:12 +01:00
JoshVanL
efe1d4e670 Use DefaultCertDuration in ./pkg/api/util
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-19 16:04:12 +01:00
JoshVanL
646f87424d Update venafi CR controller to use new reporter functionality
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-19 16:04:12 +01:00
JoshVanL
ab346453e3 Moves DefaultCertDruation to ./api/util/duration.go
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-19 16:04:12 +01:00
JoshVanL
5432354679 Adds venafi CR controller tests
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-19 16:04:12 +01:00
JoshVanL
d96c7d2e1c Adds venafi internal package with venafi CR controller
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-19 16:04:12 +01:00
JoshVanL
eb6e38a7d2 Changes WithResource -> WithRelatedResource
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-16 16:11:43 +01:00
JoshVanL
5ef9700f35 Updates acme CR controller tests to include controller sync
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-15 18:24:32 +01:00
JoshVanL
7869f99edf Fix test package name and acme tests
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-15 17:18:11 +01:00
JoshVanL
10402613c2 Update the ACME CR controller to use the new reporter signature
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-15 17:18:11 +01:00
JoshVanL
64b2dde1f1 Improve event wording and some code nesting
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-15 17:18:11 +01:00
JoshVanL
0f08f40316 Move fake order lister to ./test/unit/listers
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-15 17:18:11 +01:00
JoshVanL
b9188871ad Add extra informers to cr controller struct on New
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-15 17:18:11 +01:00
JoshVanL
eabdf0444a Updates acme to use new reporter
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-15 17:18:11 +01:00
JoshVanL
e2b9b21766 Adds acme CertificateRequest controller with unit tests
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-15 17:18:11 +01:00
JoshVanL
92a6e32a73 Remove the possibility to set the generic CR controller to set issuance
pending

Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-15 13:20:57 +01:00
JoshVanL
ec204def7e Updates vault CR controller tests to include Sync
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-14 23:40:37 +01:00
JoshVanL
1814a09580 Update CA CR controller tests with now correct conditions
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-14 22:54:47 +01:00
JoshVanL
ebf38dbfbb Refactor and cleans up SelfSigned CR controller tests
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-14 22:24:14 +01:00
JoshVanL
2948280f01 Fixes CR condition reporting and cleans up main sync tests
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-14 21:17:18 +01:00
JoshVanL
6e983fd7e6 Refactors CA cr controller to use sync
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-14 18:17:26 +01:00
JoshVanL
b11d69afa6 Update reporter in vault CR controller to use new signature
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-13 23:40:59 +01:00
JoshVanL
d588d5feba Improve code flow and remove unused issuer helper in vault cr controller
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-13 23:37:35 +01:00
JoshVanL
5f70998acb Cleans up some code and clearer naming
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-13 23:37:35 +01:00
JoshVanL
25f374da19 Fix pkg/internal/vault/fake
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-13 23:37:35 +01:00
JoshVanL
f6075035ee Move pkg/internal/internal.go into ./pkg/internal/vault
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-13 23:37:35 +01:00
JoshVanL
ea84dfdb2b Code organisation and cleaning up code
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-13 23:37:35 +01:00
JoshVanL
1bbdd03bfe Move DefaultCertDuration to pkg/util/api
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-13 23:37:35 +01:00
JoshVanL
cefd45b7de Rebase vault CR controller to use new reporter
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-13 23:37:35 +01:00
JoshVanL
179fc0349b Adds issuer to CR sign function
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-13 23:37:06 +01:00
JoshVanL
635aae6ffc Fix vault duration panic + util.CertDuration to default
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-13 23:37:06 +01:00
JoshVanL
779c07baad Fix circular dependency
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-13 23:37:06 +01:00
JoshVanL
a8e7584b36 Adds unit tests to vault CR controller
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-13 23:37:06 +01:00
JoshVanL
bf133665a3 Both CR controller and legacy issuer use new vault client for signing
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-13 23:37:06 +01:00
JoshVanL
5923c0bdfe Adds new internal vault client pkg
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-13 23:37:06 +01:00
JoshVanL
ad5e109983 Adds initial cr vault unit tests
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-13 23:37:06 +01:00
JoshVanL
312a050560 Fixes CR sync api reporting
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-13 23:37:06 +01:00
JoshVanL
356953c476 Adds status reports to vault sign method
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-13 23:36:43 +01:00
JoshVanL
6b1341505e Update CA CR controller to use a shared reporter util
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-13 23:34:18 +01:00
JoshVanL
60b349e162 Adds initial vault CR controller
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-13 23:34:17 +01:00
JoshVanL
7e56ce1c01 Adds CR reporter tests and cleans up code
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-13 17:24:38 +01:00
JoshVanL
eed1e4ba21 Move certificates cr controller to use a switch on condition status
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-13 12:10:13 +01:00
JoshVanL
0eb4ef385b Change CR reporter to be a long lived struct
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-13 11:36:53 +01:00
JoshVanL
0361a83c20 Fix reporter not setting correct conditions
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-13 10:02:53 +01:00
JoshVanL
f8c617a582 Adds failure time to CertificateRequest Status
Signed-off-by: JoshVanL <vleeuwenjoshua@gmail.com>
2019-08-12 12:41:54 +01:00