| clarkb | ianw: I'm seeing https://review.opendev.org/c/openstack/diskimage-builder/+/747220 is getting some traction again. Do you recall what opendev did to ensure git was installed on its images anyway. Thinking about your concerns and wondering if whatever we did is a viable generic workaround | 15:22 |
|---|---|---|
| ianw | clarkb: i'd say that https://opendev.org/openstack/project-config/src/branch/master/nodepool/elements/infra-package-needs/package-installs.yaml#L17 keeps git installed in opendev images | 20:56 |
| clarkb | ianw: but isn't my change making it so that explicitly listing the package doesn't lead to it being uninstalled automatically? | 20:57 |
| clarkb | I think that is why I'm confused. Maybe this issue doesn't exist anymore since it has been so long? | 20:57 |
| ianw | i also forget, i mean your change looks ok. "if a build-dep but also asked to be installed, then keep it" | 20:59 |
| ianw | and for opendev the "also ask to be installed" would be the above | 21:00 |
| clarkb | right but isn't that behavior new in my chagne which isn't merged. | 21:00 |
| clarkb | I just wanted to call it out since this has been resurrected and maybe we need to take a closer look at it in 2023 | 21:01 |
| clarkb | but I'm very distracted today unfortunately | 21:01 |
| ianw | yeah, i guess opendev doesn't use any elements that build-dep on git, that ended up removing it? i can't remember what the driver element of that was | 21:09 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!