Pass rendered resource sets to kubectl individually #57

tazjin · 2017-06-11T19:44:41Z

Please see the individual commits for more information. This resolves #51 and #56, but does not yet implement the missing resource separator warning inside of a single resource set.

tazjin · 2017-06-11T19:46:06Z

A side effect of this is that output now looks a bit nicer, for example:

Loading resources for cluster-config
Loading resources for kube-system/letsencrypt-controller
Loading resources for kube-system/kontemplate-runner
Passing file cluster-config/storage-classes.yaml to kubectl
storageclass "fast" configured (dry run)
storageclass "slow" configured (dry run)
Passing file kube-system/letsencrypt-controller/controller.yaml to kubectl
deployment "letsencrypt-controller" configured (dry run)
Passing file kube-system/kontemplate-runner/runner.yaml to kubectl
configmap "kontemplate-runner-config" configured (dry run)
deployment "kontemplate-runner" configured (dry run)

This makes it a lot easier to see which files potentially contain syntax errors or cause kubectl to fail in some way.

As a first step in resolving #51 this refactors the `templater` package to return rendered resource sets as a distinct type. This also fixes #56

Instead of passing the rendered output of all resource sets to kubectl simultaneously, build upon the previous commit and pass resource sets individually to new instances of kubectl. This resolves #51

tazjin added 3 commits June 11, 2017 21:46

refactor templater: Add intermediate type to represent rendered RSes

912f873

As a first step in resolving #51 this refactors the `templater` package to return rendered resource sets as a distinct type. This also fixes #56

refactor main: Call kubectl individually per resource set

0ace0cb

Instead of passing the rendered output of all resource sets to kubectl simultaneously, build upon the previous commit and pass resource sets individually to new instances of kubectl. This resolves #51

fix main: Fix 'kontemplate template' output

0e10b47

tazjin force-pushed the refactor/split-resource-set-applications branch from 9a7eb4c to 0e10b47 Compare June 11, 2017 19:46

tazjin changed the title ~~Pass rendered resource sets to kontemplate individually~~ Pass rendered resource sets to kubectl individually Jun 11, 2017

tazjin merged commit 3cba344 into master Jun 11, 2017

tazjin deleted the refactor/split-resource-set-applications branch June 11, 2017 20:09

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Pass rendered resource sets to kubectl individually #57

Pass rendered resource sets to kubectl individually #57

tazjin commented Jun 11, 2017

tazjin commented Jun 11, 2017

Pass rendered resource sets to kubectl individually #57

Pass rendered resource sets to kubectl individually #57

Conversation

tazjin commented Jun 11, 2017

tazjin commented Jun 11, 2017