| ihalomi | hey, im not sure message before was sent...I was working on mounting /run but I couldnt find solution that would work way it is supposed. We want to mount /run because it is tmpfs storage and if we would create these directories it would go against that idea. Do we really need to be these mounts shared amount containers? Isnt it there just to communicate with host machine? | 08:07 |
|---|---|---|
| *** mmalchuk_ is now known as mmalchuk | 08:58 | |
| mmalchuk | mgoddard mnasiadka please review https://review.opendev.org/c/openstack/kayobe/+/881679 | 09:21 |
| opendevreview | Maksim Malchuk proposed openstack/kolla stable/yoga: https: remove dhparams.pem before running gencerts https://review.opendev.org/c/openstack/kolla/+/881467 | 09:23 |
| SvenKieske | mmalchuk: I believe mnasiadka is on vacation this week | 09:39 |
| SvenKieske | ihalomi: I don't understand? "shared among containers" and "communicate with host machine" are the same thing in this case, no? /run needs to be shared between containers in order to be able to communicate with the host? if you want non shared /run between containers - which is afaik the default in podman - you also do not share /run with the host, no? | 09:42 |
| opendevreview | Maksim Malchuk proposed openstack/kolla-ansible master: Fix passwords.yml permissions https://review.opendev.org/c/openstack/kolla-ansible/+/882100 | 10:02 |
| opendevreview | Maksim Malchuk proposed openstack/kolla stable/yoga: Bump docker library to use 5.x version https://review.opendev.org/c/openstack/kolla/+/881729 | 12:23 |
| SvenKieske | mmalchuk: does 881729 interact with this change as well? https://review.opendev.org/c/openstack/kolla/+/825786 might be good to coordinate these two, no? | 12:48 |
| mmalchuk | no, its WIP | 12:48 |
| mmalchuk | and only Yoga | 12:48 |
| mmalchuk | we have dependency problem there | 12:49 |
| mmalchuk | looks like we either should fix tests or fix docker library itself | 12:50 |
| mmalchuk | pinning version works but need changes in the global-requirements | 12:50 |
| mmalchuk | which is an emergency action, and may be not merged | 12:51 |
| mmalchuk | https://review.opendev.org/c/openstack/requirements/+/881814 | 12:51 |
| mmalchuk | the root cause: https://github.com/docker/docker-py/blob/6.0.0/setup.py#L15 | 12:52 |
| mmalchuk | 5.0.3 doesn't pin urlib3: https://github.com/docker/docker-py/blob/5.0.3/setup.py#L12-L15 | 12:53 |
| mmalchuk | and urlib3 released 2.x with breaking changes: https://urllib3.readthedocs.io/en/latest/v2-migration-guide.html | 12:53 |
| SvenKieske | there's already an open issue for fixing this upstream: https://github.com/docker/docker-py/issues/3113 | 12:58 |
| SvenKieske | the urllib author themselves even commented there and suggested a fix: https://github.com/docker/docker-py/issues/3113#issuecomment-1531570788 | 13:01 |
| ihalomi | is weekly canceled this week? | 13:13 |
| mmalchuk | I know, and there is support for new version. But on the other hand we can have the same issue with any other library in the feature, so we should fix an issue ourselves | 13:13 |
| mmalchuk | ihalomi yep | 13:13 |
| opendevreview | Maksim Malchuk proposed openstack/kolla stable/yoga: Upgrade pip/setuptools/wheel for tests https://review.opendev.org/c/openstack/kolla/+/881729 | 13:15 |
| opendevreview | Maksim Malchuk proposed openstack/kolla stable/yoga: https: remove dhparams.pem before running gencerts https://review.opendev.org/c/openstack/kolla/+/881467 | 13:27 |
| opendevreview | Maksim Malchuk proposed openstack/kolla-ansible master: Abort on bad passwords.yml permissions https://review.opendev.org/c/openstack/kolla-ansible/+/882177 | 20:18 |
| opendevreview | Maksim Malchuk proposed openstack/kolla-ansible master: Abort on bad passwords.yml permissions https://review.opendev.org/c/openstack/kolla-ansible/+/882177 | 20:48 |
| opendevreview | Maksim Malchuk proposed openstack/kolla-ansible master: Abort on bad passwords.yml permissions https://review.opendev.org/c/openstack/kolla-ansible/+/882177 | 20:52 |
| opendevreview | Maksim Malchuk proposed openstack/kolla-ansible master: Abort on bad passwords.yml permissions https://review.opendev.org/c/openstack/kolla-ansible/+/882177 | 22:16 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!