<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jan 5, 2021 at 9:44 AM Cédric Jeanneret <<a href="mailto:cjeanner@redhat.com">cjeanner@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hello there,<br>
<br>
Since we introduced Podman instead of Docker in tripleo, we are running<br>
after the right version.<br>
For now, the version we should use is provided by a specific<br>
container-tools module stream (container-tools:2.0). The default stream<br>
being container-tools:rhel8, we might end with a deployed undercloud<br>
running the wrong podman version, leading to some nice issues,<br>
especially in a SELinux-enabled environment.<br></blockquote><div><br></div><div>o/ Tengu thanks for socialising that issue, it's a fun one for sure...</div><div><br></div><div>first question is _where_ is that the default and it is mostly rhetorical as I can guess. In general though i think it's worth filing a LP bug with some of these details so we can also track whatever is decided as a fix here.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Currently, the main dependency tree is:<br>
python3-tripleoclient requires podman<br>
That's mostly all.<br>
<br>
While we can, of course, edit the python3-tripleoclient spec file in<br>
order to pin the precise podman version (or, at least, set upper and<br>
lower constraints) in order to target the version provided by the right<br>
stream, it's not ideal:<br>
- package install error will lead to more questions (since ppl might<br>
skip doc for #reason)<br>
- if we change the target version, we'll need to ensure it's present in<br>
a stream (and, well, update the doc if we need to switch the stream)<br>
- and probably some other reasons.<br>
<br>
Since we can NOT depend on a specific stream in rpm (for instance, we<br>
can't put "Requires @container-tools:2.0/podman" like we might in<br>
ansible), we're a bit stuck.<br>
<br>
An ugly hack is tested in the spec file[1], using a bash thing in order<br>
to check the activated stream, but this is not ideal either, especially<br>
since it makes the whole package building/checking fail during the<br>
"mock" stage. In order to make it pass, we'd need to modify the whole<br>
RDO in order to take into account stream switching. This isn't<br>
impossible, but it leads to other concerns, especially when it comes to<br>
"hey we need to switch the stream for <module>". And I'm pretty sure no<br>
one will want to dig into it, for good reasons ;).<br>
<br>
This leads to a last possibility:<br>
drop the "Requires podman" from tripleo dependency tree, and rely on<br></blockquote><div><br></div><div>so how does that work in practice though I mean what about python-tripleoclient. Quick grep just now tells me at least the heat-launcher is directly using podman. How do we drop that requirement.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
tripleo-ansible to actually enable the correct module, and install podman.<br>
This means podman will be installed during the undercloud deploy, for<br>
instance as a host_prep_tasks (step_0).<br>
<br></blockquote><div><br></div><div>I am missing how using tripleo-ansible helps us with the stream question. Do you mean add logic in tripleo-ansible that tests the stream and sets the correct version ?</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
While I'm not that happy with this idea, it's probably not as bad as the<br>
other hacks we've tested so far, and would, at last, prevent once for<br>
all the usual "it's not working" due to a wrong podman version (and,<br>
yes, we get plenty of them, especially in the OSP version).<br>
<br>
What do you think? Any other way of solving this issue? Remember, we're<br>
talking about "ensuring we get the right version, coming from a specific<br>
stream" and, especially, "ensure operator can't install the wrong one"<br>
(if they don't follow the doc, of if they are using automation that<br>
doesn't check actual requirements in the doc"...<br></blockquote><div><br></div><div>in general i lean more towards 'ansible tasks' rather than 'bash thing in the specfile' to solve that. Either way though this is us reacting (with some fix/hack/workaround) to a problem that isn't anything to do with TripleO or even OpenStack but rather an OS/packaging issue. So do we have any longer term plans on what you mentioned on IRC just now regarding  getting module support for RPM deps (do you mean something in the spec file here?) or is that a complete dead end?</div><div><br></div><div>thanks, marios</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Thank you for your thoughts!<br>
<br>
Cheers,<br>
<br>
C.<br>
<br>
<br>
[1] <a href="https://review.rdoproject.org/r/#/c/31411/" rel="noreferrer" target="_blank">https://review.rdoproject.org/r/#/c/31411/</a><br>
<br>
<br>
-- <br>
Cédric Jeanneret (He/Him/His)<br>
Sr. Software Engineer - OpenStack Platform<br>
Deployment Framework TC<br>
Red Hat EMEA<br>
<a href="https://www.redhat.com/" rel="noreferrer" target="_blank">https://www.redhat.com/</a><br>
<br>
</blockquote></div></div>