*** mhen_ is now known as mhen | 01:12 | |
frickler | gmaan: you are right. python3-oslo.utils gets pulled in via the novnc pkg and then somehow overrides the pkg installed in the venv. if I purge the pkg, nova-manage works: https://paste.opendev.org/show/bEWr0MsGDtU4vM7VmprU/ | 07:54 |
---|---|---|
frickler | sean-k-mooney: ^^ that also explains why you could not reproduce it locally | 07:54 |
frickler | the issue only seems to happen with editable installs, so as a workaround we could maybe drop the "-e" in devstack. but the correct solution seems to be to switch all deliverables to add a pyproject.toml file | 08:01 |
frickler | let me know if you want access to the held node and look at it yourself | 08:01 |
frickler | (and yeah I know you were up late so take your time ;) | 08:02 |
*** elodille1 is now known as elodilles | 10:11 | |
opendevreview | Xiang Wang proposed openstack/oslo.messaging master: Implement ConsulOperator as a service broker to support HTTP Driver https://review.opendev.org/c/openstack/oslo.messaging/+/912499 | 10:14 |
opendevreview | Xiang Wang proposed openstack/oslo.messaging master: Add HTTP driver to oslo.messaging.drivers https://review.opendev.org/c/openstack/oslo.messaging/+/914840 | 10:14 |
opendevreview | Xiang Wang proposed openstack/oslo.messaging master: Implement HTTPListener and HTTPClient https://review.opendev.org/c/openstack/oslo.messaging/+/915056 | 10:14 |
opendevreview | Xiang Wang proposed openstack/oslo.messaging master: Add flask app for http-driver https://review.opendev.org/c/openstack/oslo.messaging/+/938283 | 10:14 |
opendevreview | Xiang Wang proposed openstack/oslo.messaging master: Add Broadcaster for RPC Fanout https://review.opendev.org/c/openstack/oslo.messaging/+/938536 | 10:14 |
elodilles | damani[m]: tkajinam: hi, sorry for pinging you, but could you take a look at the 2023.2 Bobcat EOL patch of oslo please: https://review.opendev.org/c/openstack/releases/+/948198 | 11:02 |
elodilles | it would be nice to have an ACK from the team o:) | 11:03 |
elodilles | i've checked and hashes are still all correct | 11:03 |
sean-k-mooney | frickler: droping -e in devstack is not an option | 11:29 |
sean-k-mooney | that a core part of its design. however we could have a flag for using it or not in ci | 11:30 |
sean-k-mooney | perhaps on a pre depenency basic. kind of like libs form git. | 11:31 |
sean-k-mooney | frickler: a better short term solution may be for neturon to just disable vnc in tehre job | 11:31 |
sean-k-mooney | if its coming form novnc they coudl just not install it | 11:31 |
sean-k-mooney | if pyproject.toml works then that would be good in general but i dont knwo if that will fix it in this case, reodering novnc install earlier in devstack coudl help as then the lib_form_git install of oslo will take precedence | 11:33 |
frickler | sean-k-mooney: the latter doesn't help since we do use --system-site-packages. the error persists if I uninstall oslo.utils from the venv and re-install it | 11:38 |
sean-k-mooney | hum thats unfortunet | 11:38 |
sean-k-mooney | i think site packages is there because of libivrt or libvirt python to be more precise | 11:39 |
frickler | yes | 11:41 |
sean-k-mooney | we have the option to pull novnc form source in devstack | 11:41 |
sean-k-mooney | im not sure if that would help | 11:41 |
sean-k-mooney | https://github.com/openstack/devstack/blob/3fe8873a15db27d2d8b7df4e708210ee3ca1465d/.zuul.yaml#L377 | 11:42 |
sean-k-mooney | we coudl set that to false perhaps | 11:43 |
sean-k-mooney | taht would avoid installing oslo in site packages | 11:43 |
sean-k-mooney | frickler: ill push a patch toi disabel that and put a dnm to oslo to see if that helps | 11:44 |
frickler | I'm running a test on the held node now | 11:44 |
sean-k-mooney | oh ok shall i wait then. | 11:45 |
sean-k-mooney | nova vnc consol proxy can be used without horizon but we also may not need to enabel that service in devstack minimal | 11:47 |
sean-k-mooney | oh actully its enabeld in devstack not devstack-minimal but even so | 11:48 |
sean-k-mooney | im not saying other project should not run the 1 or 2 vnc console tempest test but if your change to neutron broke nova vnc console that some impressive python you have there | 11:49 |
frickler | sean-k-mooney: ok, the test worked, I did have to clone the novnc repo manually, though. not sure if we have that in zuul so it could be added to projects list | 12:14 |
sean-k-mooney | am we shoudl its cofniguable in the jobs. i proably needs to be added as a required project to the base devstack job if we are going ot flip the default but it shoudl be in the zuul tenant | 12:16 |
sean-k-mooney | frickler: yep https://github.com/openstack/project-config/blob/master/zuul/main.yaml#L1405 | 12:16 |
sean-k-mooney | it might be better to contiue this converstaion in the qa channel but we have a few options in teh short term. we could update the neutron job to use it form souce and add that as a require proejct | 12:18 |
sean-k-mooney | or we coudl do that in the main devstack job for everyone | 12:18 |
sean-k-mooney | frickler: by the way im kind of wondering why does installing novnc form package install oslo | 12:19 |
sean-k-mooney | it does not actully depend on it | 12:19 |
sean-k-mooney | its almost entirly javascript so this sould like a ubunut? packaging bug | 12:19 |
sean-k-mooney | unless its a devstack bug? | 12:19 |
sean-k-mooney | frickler: hum ok that weird | 12:23 |
sean-k-mooney | https://ubuntu.pkgs.org/24.04/ubuntu-universe-amd64/python3-novnc_1.3.0-2_all.deb.html | 12:23 |
sean-k-mooney | they list oslo.config as a dep of the python3-novnc package | 12:23 |
sean-k-mooney | and that is a dep of novnc | 12:24 |
sean-k-mooney | frickler: ok so it is a debian packaging bug sort of | 12:28 |
sean-k-mooney | frickler: in the before times https://github.com/novnc/noVNC/pull/244/files novnc used to import not jsut oslo but nova code | 12:29 |
sean-k-mooney | that code was removed form the novnce packate in v0.6 | 12:31 |
sean-k-mooney | they are packaging 1.3.0 in noble | 12:32 |
sean-k-mooney | i think they just never drop the dep | 12:32 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!