Mike Zappa comments

Results 103 comments of


                                            Mike Zappa

Set stderr to empty string when using terminal

@mikebrow when you are free :-)

[release/1.7] Handle teardown failure to avoid blocking cleanup

> @mikebrow @dmcgowan please hold on this until we can investigate a bit more, we have an internal issue that is causing CNI plugin to leak IPs and this may...

Still seeing the issue for endpoints staying out of sync

/assign @tnqn Are you able to triage this since you are already looking at this?

new tag for merged PRs ?

Second this as we want to have the latest fixes. @mattfarina

new tag for merged PRs ?

@mattfarina hopefully this can get done?

Pass down resources to CRI

@marquiz I was talking with @mikebrow @kad and Krisztian Litkey about including labels/annotations with this KEP. Are you able to discuss this?

CNIConfigWriter writes .temp file that throws off containerd

This is because of the file watcher, any changes to the specified directory would cause this. We probably should downgrade the error to warn or filter on specific file extensions...

CNIConfigWriter writes .temp file that throws off containerd

@dims it probably makes sense to update the documentation in containerd to reflect that we only would watch these specific file types

CNIConfigWriter writes .temp file that throws off containerd

I just need to test this PR (https://github.com/containerd/containerd/pull/10497)

CNIConfigWriter writes .temp file that throws off containerd

> My read of the go-cni code is we filter for the aforementioned extensions when loading all config from a directory... > > However there is no filtering by extension...