Add CNI integration about cri-o HOT 12 CLOSED

@mrunalp Do you agree this is fixed with commit a4f5c3a ?

from cri-o.

why not having people checkout CNI repo since they basically have the same examples?

I was assuming "people" meant me packaging ocid. I'll add an example tomorrow (/me -> bed). 😉

from cri-o.

@sameo Can we please include the "sample CNI configuration" in the repo so that distributions can package it, and that the README is just "here's a sample file".

from cri-o.

@sameo Can we please include the "sample CNI configuration" in the repo so that distributions can package it, and that the README is just "here's a sample file".

@cyphar mmm, why not having people checkout CNI repo since they basically have the same examples? it won't change, but if the config changes we'll need to track CNI from this repo

from cri-o.

(no strong opinion on this though, feel free to include a config sample if you feel strongly about it)

from cri-o.

Including the CNI source code in an RPM just to get an example configuration seems to me to be a little overkill. ;)

from cri-o.

Including the CNI source code in an RPM just to get an example configuration seems to me to be a little overkill. ;)

nah, I didn't mean that (if I got correctly what you said)

from cri-o.

@cyphar the misunderstanding lies in me saying checkout CNI repo - by checkout I meant have a look on the github repo on the web

from cri-o.

Ah, right. "go check out" not git checkout. 😸

from cri-o.

Ah, right. "go check out" not git checkout.

I should take another english class soon...myb

from cri-o.

@sameo Yeah, I think we are good. I am okay either ways on adding the config files. Will wait for @cyphar

from cri-o.

This is in #295.

from cri-o.