| *** tosky has quit IRC | 00:18 | |
| *** cgoncalves has quit IRC | 00:28 | |
| *** cgoncalves has joined #openstack-requirements | 01:06 | |
| *** ianw is now known as ianw_pto | 05:24 | |
| *** evrardjp has quit IRC | 05:33 | |
| *** evrardjp has joined #openstack-requirements | 05:33 | |
| *** hberaud has quit IRC | 05:51 | |
| *** hberaud has joined #openstack-requirements | 05:53 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: Updated from generate-constraints https://review.opendev.org/c/openstack/requirements/+/765244 | 06:30 |
|---|---|---|
| *** sboyron has joined #openstack-requirements | 07:20 | |
| *** e0ne has joined #openstack-requirements | 07:22 | |
| *** rpittau|afk is now known as rpittau | 07:27 | |
| *** e0ne has quit IRC | 07:32 | |
| *** e0ne has joined #openstack-requirements | 08:06 | |
| *** e0ne has quit IRC | 08:08 | |
| *** e0ne has joined #openstack-requirements | 08:13 | |
| *** tosky has joined #openstack-requirements | 08:34 | |
| *** redrobot has quit IRC | 08:41 | |
| *** vishalmanchanda has joined #openstack-requirements | 08:47 | |
| openstackgerrit | Merged openstack/requirements master: update constraint for os-win to new release 5.3.0 https://review.opendev.org/c/openstack/requirements/+/765189 | 09:24 |
| openstackgerrit | Merged openstack/requirements master: update constraint for openstackdocstheme to new release 2.2.7 https://review.opendev.org/c/openstack/requirements/+/765207 | 09:42 |
| openstackgerrit | Merged openstack/requirements master: update constraint for python-masakariclient to new release 6.2.0 https://review.opendev.org/c/openstack/requirements/+/765102 | 10:11 |
| *** dtantsur|afk is now known as dtantsur | 10:26 | |
| openstackgerrit | Merged openstack/requirements master: update constraint for python-vitrageclient to new release 4.2.0 https://review.opendev.org/c/openstack/requirements/+/765106 | 12:21 |
| openstackgerrit | Merged openstack/requirements master: update constraint for mistral-lib to new release 2.4.0 https://review.opendev.org/c/openstack/requirements/+/765104 | 12:27 |
| *** e0ne has quit IRC | 12:55 | |
| *** e0ne has joined #openstack-requirements | 13:09 | |
| *** e0ne has quit IRC | 13:25 | |
| *** mugsie has quit IRC | 13:32 | |
| *** e0ne has joined #openstack-requirements | 14:00 | |
| *** felipe_rodrigues has joined #openstack-requirements | 15:05 | |
| *** redrobot has joined #openstack-requirements | 15:35 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for castellan to new release 3.7.0 https://review.opendev.org/c/openstack/requirements/+/765340 | 15:35 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for python-manilaclient to new release 2.4.0 https://review.opendev.org/c/openstack/requirements/+/765346 | 16:03 |
| *** e0ne has quit IRC | 16:44 | |
| *** rpittau is now known as rpittau|afk | 16:49 | |
| prometheanfire | I'm kinda at a loss for https://review.opendev.org/763931 dhellmann it's your package (sphinxcontrib.datatemplates using runcmd), not sure why it can't find runcmd | 17:10 |
| *** tosky has quit IRC | 17:29 | |
| *** dtantsur is now known as dtantsur|afk | 17:39 | |
| dhellmann | it's not something dumb like the fact that the requirements list of sphinxcontrib.datatemplates says "sphinxcontrib.runcmd" but the actual package is "sphinxcontrib-runcmd" is it? | 17:39 |
| prometheanfire | it probably is, but it was autogenerated, not manually added | 17:42 |
| dhellmann | looking at https://zuul.opendev.org/t/openstack/build/72ad573b150744c7af2ab98ac8d5c209 I see in the output that sphinxcontrib-runcmd==0.2.0 is installed so the failure isn't in installing it, it's in the code later that tries to verify that everything requested was installed | 17:42 |
| prometheanfire | https://github.com/sphinx-contrib/datatemplates/blob/master/requirements.txt#L8 shows the .runcmd version | 17:43 |
| dhellmann | so I think it's a pkg_resources bug | 17:43 |
| prometheanfire | ya, likely | 17:43 |
| prometheanfire | I'm tyring to debug pgk_resources now | 17:43 |
| dhellmann | if you run pkg_resources.require("sphinxcontrib-runcmd") in an environment where the package is installed, does it work? | 17:44 |
| dhellmann | yes, it does | 17:44 |
| dhellmann | https://paste.centos.org/view/ec00081b | 17:45 |
| prometheanfire | ok, guess I report the upstream bug, thanks for the help | 17:45 |
| dhellmann | I'd also take a patch in sphinxcontrib.datatemplates to use the "right" name for that dependency | 17:46 |
| dhellmann | I honestly don't know where the bug is. Either pip shouldn't have installed the thing with the wrong name or pkg_resources should recognize the name with the dot | 17:47 |
| prometheanfire | dhellmann: I think that the requirement on sphinxcontrib.runcmd in the datatemplates project is incorrect given pypi lists the project as https://pypi.org/project/sphinxcontrib-runcmd/ | 18:58 |
| dhellmann | yeah. although pip seems happy enough. I don't understand the name translation rules, but if you want to update that requirements list I can push out a new release. | 18:59 |
| prometheanfire | dhellmann: if you don't mind, want a PR? | 18:59 |
| dhellmann | that would be good. I'll try to do the release dance later today | 18:59 |
| prometheanfire | kk | 18:59 |
| prometheanfire | dhellmann: https://github.com/sphinx-contrib/datatemplates/pull/74 pypi has a redirect for the .runcmd (dot) version | 19:05 |
| dhellmann | ah, fun | 19:05 |
| dhellmann | so maybe we should have a test on the requirements/constraints lists to ensure they use the "right" name | 19:06 |
| prometheanfire | heh, that'd be fun, check for a 301? :P | 19:08 |
| prometheanfire | I generally don't like tests that reach out to the network as a general policy, not a big deal in this case as it'd be explicitly needed and the existing error is correct, if obtuse | 19:09 |
| smcginnis | I think I had that in a script. (the name check) | 19:17 |
| smcginnis | It pointed out a couple mismatches we had. | 19:17 |
| *** vishalmanchanda has quit IRC | 19:22 | |
| *** samueldmq has quit IRC | 19:27 | |
| *** tosky has joined #openstack-requirements | 20:55 | |
| *** openstackgerrit has quit IRC | 21:08 | |
| dhellmann | it wouldn't have to go to the network. it could run after everything is installed and look for this specific class of naming mismatch. | 21:12 |
| dhellmann | the error being triggered here is an example, it's just that the error message isn't clear, right? | 21:12 |
| dhellmann | it might also be simpler to just change that check job to try both forms for names | 21:52 |
| dhellmann | sphinxcontrib-datatemplates 0.7.2 includes the fix | 21:57 |
| prometheanfire | dhellmann: thanks | 22:21 |
| *** felipe_rodrigues has quit IRC | 22:44 | |
| *** openstackgerrit has joined #openstack-requirements | 22:54 | |
| openstackgerrit | Matthew Thode proposed openstack/requirements master: Updated from generate-constraints https://review.opendev.org/c/openstack/requirements/+/763931 | 22:54 |
| *** bnemec has quit IRC | 23:22 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!