Skip to content
This repository has been archived by the owner on Jul 30, 2021. It is now read-only.

render: add --cloud-provider #105

Merged
merged 1 commit into from
Nov 18, 2016
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
3 changes: 3 additions & 0 deletions cmd/bootkube/render.go
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@ var (
altNames string
selfHostKubelet bool
storageBackend string
cloudProvider string
}
)

Expand All @@ -48,6 +49,7 @@ func init() {
cmdRender.Flags().StringVar(&renderOpts.apiServers, "api-servers", "https://127.0.0.1:443", "List of API server URLs including host:port, commma seprated")
cmdRender.Flags().StringVar(&renderOpts.altNames, "api-server-alt-names", "", "List of SANs to use in api-server certificate. Example: 'IP=127.0.0.1,IP=127.0.0.2,DNS=localhost'. If empty, SANs will be extracted from the --api-servers flag.")
cmdRender.Flags().BoolVar(&renderOpts.selfHostKubelet, "self-host-kubelet", false, "Create a self-hosted kubelet daemonset.")
cmdRender.Flags().StringVar(&renderOpts.cloudProvider, "cloud-provider", "", "The provider for cloud services. Empty string for no provider")
}

func runCmdRender(cmd *cobra.Command, args []string) error {
Expand Down Expand Up @@ -117,6 +119,7 @@ func flagsToAssetConfig() (c *asset.Config, err error) {
AltNames: altNames,
SelfHostKubelet: renderOpts.selfHostKubelet,
StorageBackend: renderOpts.storageBackend,
CloudProvider: renderOpts.cloudProvider,
}, nil
}

Expand Down
1 change: 1 addition & 0 deletions pkg/asset/asset.go
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,7 @@ type Config struct {
AltNames *tlsutil.AltNames
SelfHostKubelet bool
StorageBackend string
CloudProvider string
}

// NewDefaultAssets returns a list of default assets, optionally
Expand Down
4 changes: 4 additions & 0 deletions pkg/asset/internal/templates.go
Original file line number Diff line number Diff line change
Expand Up @@ -142,6 +142,7 @@ spec:
- --tls-private-key-file=/etc/kubernetes/secrets/apiserver.key
- --service-account-key-file=/etc/kubernetes/secrets/service-account.pub
- --client-ca-file=/etc/kubernetes/secrets/ca.crt
- --cloud-provider={{ .CloudProvider }}
env:
- name: MY_POD_IP
valueFrom:
Expand Down Expand Up @@ -214,6 +215,8 @@ spec:
- --root-ca-file=/etc/kubernetes/secrets/ca.crt
- --service-account-private-key-file=/etc/kubernetes/secrets/service-account.key
- --leader-elect=true
- --cloud-provider={{ .CloudProvider }}
- --configure-cloud-routes=false
volumeMounts:
- name: secrets
mountPath: /etc/kubernetes/secrets
Expand All @@ -228,6 +231,7 @@ spec:
- name: ssl-host
hostPath:
path: /usr/share/ca-certificates
dnsPolicy: Default # Don't use cluster DNS.
`)
SchedulerTemplate = []byte(`apiVersion: extensions/v1beta1
kind: Deployment
Expand Down
2 changes: 1 addition & 1 deletion pkg/asset/k8s.go
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ const (
func newStaticAssets(selfHostKubelet bool) Assets {
var noData interface{}
assets := Assets{
mustCreateAssetFromTemplate(AssetPathControllerManager, internal.ControllerManagerTemplate, noData),
mustCreateAssetFromTemplate(AssetPathScheduler, internal.SchedulerTemplate, noData),
mustCreateAssetFromTemplate(AssetPathProxy, internal.ProxyTemplate, noData),
mustCreateAssetFromTemplate(AssetPathKubeDNSDeployment, internal.DNSDeploymentTemplate, noData),
Expand All @@ -38,6 +37,7 @@ func newStaticAssets(selfHostKubelet bool) Assets {

func newDynamicAssets(conf Config) Assets {
return Assets{
mustCreateAssetFromTemplate(AssetPathControllerManager, internal.ControllerManagerTemplate, conf),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The controller-manager entry should also be removed from newStaticAssets

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oops, must have bee a merge mistake at some point.

mustCreateAssetFromTemplate(AssetPathAPIServer, internal.APIServerTemplate, conf),
}
}
Expand Down