Skip to content
This repository has been archived by the owner on Jan 11, 2023. It is now read-only.

network policy: add cilium #2429

Merged
merged 3 commits into from
Mar 13, 2018
Merged

network policy: add cilium #2429

merged 3 commits into from
Mar 13, 2018

Conversation

jessfraz
Copy link
Contributor

What this PR does / why we need it: Add cilium as a network policy provider.

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

Special notes for your reviewer:

Release note:

Add cilium as a network policy provider.

Signed-off-by: Jess Frazelle <acidburn@microsoft.com>
"orchestratorRelease": "1.9",
"kubernetesConfig": {
"networkPolicy": "cilium",
"etcdVersion": "3.1.10",
Copy link
Member

Choose a reason for hiding this comment

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

out of curiosity is there an etcd version dependency for cilium? current default in acs-engine is 3.2.16.

Signed-off-by: Jess Frazelle <acidburn@microsoft.com>
@ghost ghost assigned jackfrancis Mar 13, 2018
Copy link
Member

@jackfrancis jackfrancis left a comment

Choose a reason for hiding this comment

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

lgtm, thanks @jessfraz!

@jackfrancis jackfrancis merged commit df44a7c into Azure:master Mar 13, 2018
@ghost ghost removed the in progress label Mar 13, 2018
@jessfraz jessfraz deleted the add-cilium branch March 13, 2018 17:23
tesharp pushed a commit to tesharp/acs-engine that referenced this pull request Mar 16, 2018
* docs: add cilium to network policy

Signed-off-by: Jess Frazelle <acidburn@microsoft.com>

* network policy: add cilium

Signed-off-by: Jess Frazelle <acidburn@microsoft.com>

* remove cilium from legacy e2e
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants