search

projectcalico / canal

Policy based networking for cloud native applications
717 stars 100 forks source link

Adding hostport config to all canal manifests #103

Closed heschlie closed 7 years ago

heschlie commented 7 years ago

Description

Fixes #102

Updating Canal manifests to support hostport via portmap chaining.

I also noticed we're not on the latest Calico, I'd be happy to update and test that as well where it applies. We also were only changing the conf name in the canal_etcd_tls.yaml to 10-canal, should we make them all the same? I feel like 10-canal.conflist would be a good name to standardize on for Canal.

Todos

caseydavenport commented 7 years ago

@heschlie I think it's fine to keep as 10-calico.conflist given Calico is the CNI plugin in use.

caseydavenport commented 7 years ago

@heschlie have you tried these out on v1.6, v1.7, etc clusters?

heschlie commented 7 years ago

I did not test it on a 1.8 cluster, I can test that before I merge.

caseydavenport commented 7 years ago

Feel free to merge once you've tested on v1.6, v1.7, and v1.8.