Wednesday, 2021-09-15

kata-irc-bot<simon.kaegi> hmm... let me try some variations but I believe this was for the case where `optional: true` was specified but the configmap/secret was not present03:50
kata-irc-bot<simon.kaegi> oh interesting... this fails also if the configmap or secret is present "but" it's empty04:14
kata-irc-bot<eric.ernst> Interesting. Can you reproduce on 2.1 Kata?05:00
kata-irc-bot<eric.ernst> I’m wondering if this may be dealing w watchable mount handling that was introduced in the 2.2 release timeframe. 05:01
kata-irc-bot<archana.m.shinde> +1 for going with useradd rather than updating files manually05:55
kata-irc-bot<christophe> Interested in getting feedback on the "pod API" https://github.com/kata-containers/kata-containers/issues/263706:13
kata-irc-bot<simon.kaegi> yep seems to be a 2.2 problem - 2.1 worked.15:19
kata-irc-bot<fidencio> @eric.ernst, do you take it over from here? :slightly_smiling_face:15:29
kata-irc-bot<eric.ernst> ugh.17:02
kata-irc-bot<eric.ernst> yeah, let me TAL right away.17:02
kata-irc-bot<eric.ernst> Looks like the path must not be created until the first copy is carried out.17:22
kata-irc-bot<feng.wang> @eric.ernst @fidencio can you two please approve the PR if there is no more feedback?18:16
kata-irc-bot<eric.ernst> @fidencio you had some outstanding comments? https://github.com/kata-containers/kata-containers/pull/2432/files#r70570604818:23
kata-irc-bot<feng.wang> I have added a defer block in line 309. Let me if that addressed the concern. @fidencio @eric.ernst18:24
kata-irc-bot<eric.ernst> Left 2 more nits for questions that I think were unresolved.18:26
kata-irc-bot<eric.ernst> The defer on that line looks good — just want to give fabiano a chance to resolve the query18:27
kata-irc-bot<feng.wang> Thanks. I’ll look into them.18:28
kata-irc-bot<fidencio> If CI is green, I'm fine merging it. :slightly_smiling_face:19:26

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!