| *** gyee has quit IRC | 02:56 | |
| *** evrardjp has quit IRC | 04:33 | |
| *** evrardjp has joined #openstack-rpm-packaging | 04:33 | |
| *** ykarel has joined #openstack-rpm-packaging | 04:44 | |
| *** ykarel has quit IRC | 05:59 | |
| *** ykarel has joined #openstack-rpm-packaging | 06:00 | |
| *** apevec has joined #openstack-rpm-packaging | 06:23 | |
| *** amoralej|off is now known as amoralej | 06:34 | |
| *** sboyron has joined #openstack-rpm-packaging | 07:11 | |
| openstackgerrit | Boyron Sébastien proposed openstack/rpm-packaging master: Add validations-libs https://review.opendev.org/756305 | 07:19 |
|---|---|---|
| openstackgerrit | Boyron Sébastien proposed openstack/rpm-packaging master: Add validations-libs https://review.opendev.org/756305 | 07:21 |
| *** jpena|off is now known as jpena | 07:25 | |
| sboyron | Hi jpena: damani: Do you know if the rpmbuild regenration is performed as root in the CI ? For review 756305, there is a lack in setup.py of ansible roles. When I try to install them manually I get this error locally: install: cannot create directory '/builddir/build/BUILDROOT/python-validations-libs-1.0.4-0.20201006165224.20b0b65.el8.x86_64/usr/share/ansible': Permission denied | 07:27 |
| sboyron | These ansible roles will be installed directly using setup.py in the next releases (already merged but not yet released) | 07:28 |
| jpena | sboyron: in the RDO CI, it's done as the zuul user | 07:29 |
| sboyron | jpena, yes that's what I see in the latest build. So do we have a way to install them ? | 07:30 |
| jpena | sboyron: I think we need to remove the -p parameter from "install" | 07:35 |
| jpena | but not sure | 07:35 |
| *** rpittau|afk is now known as rpittau | 07:41 | |
| *** ykarel_ has joined #openstack-rpm-packaging | 07:43 | |
| *** ykarel has quit IRC | 07:44 | |
| *** ykarel__ has joined #openstack-rpm-packaging | 07:44 | |
| sboyron | jpena: it's not the -p; it comes from the 644 on the folder ;p | 07:46 |
| jpena | ohhh, of course | 07:46 |
| sboyron | zuul cannot copy files into it after | 07:46 |
| * jpena facepalms | 07:46 | |
| sboyron | yep, the same | 07:46 |
| sboyron | should the roles files itself should be in 644 ? | 07:47 |
| sboyron | folder will be in 755 | 07:47 |
| *** ykarel_ has quit IRC | 07:47 | |
| sboyron | by the way jpena; I've got a working local RDO CI now, which is reflecting well the official one ;) | 07:50 |
| jpena | nice! | 07:50 |
| *** ykarel__ is now known as ykarel | 07:52 | |
| sboyron | jpena: latest question (I hope) on this subject: regarding the dependencies vs the submodules. Do we need to add some dependancies on docs and ansible on the "master" rpm ? or is it already done by subrpm mechanism ? | 07:55 |
| jpena | sboyron: it's not done. Usually, the main subpackage does not add dependencies on the -doc subpackage (docs are usually optional). We could add a dep on the ansible subpackage, since I think they're meant to be installed together | 07:56 |
| sboyron | ok got it | 07:58 |
| openstackgerrit | Boyron Sébastien proposed openstack/rpm-packaging master: Add validations-libs https://review.opendev.org/756305 | 08:02 |
| sboyron | jpena: I think we're good now. | 08:02 |
| *** jpich has joined #openstack-rpm-packaging | 08:06 | |
| openstackgerrit | Boyron Sébastien proposed openstack/rpm-packaging master: Add validations-libs https://review.opendev.org/756305 | 08:23 |
| openstackgerrit | Daniel Bengtsson proposed openstack/rpm-packaging stable/train: Update oslo.privsep to 1.33.4. https://review.opendev.org/756011 | 08:57 |
| *** ykarel_ has joined #openstack-rpm-packaging | 09:03 | |
| *** ykarel has quit IRC | 09:03 | |
| openstackgerrit | Riccardo Pittau proposed openstack/rpm-packaging master: Update ironic requirements to latest master https://review.opendev.org/756471 | 09:18 |
| openstackgerrit | Riccardo Pittau proposed openstack/rpm-packaging master: Remove ironicclient dependency https://review.opendev.org/756472 | 09:23 |
| *** ykarel_ is now known as ykarel | 09:25 | |
| openstackgerrit | Riccardo Pittau proposed openstack/rpm-packaging master: Update ironic-python-agent requirements https://review.opendev.org/756473 | 09:28 |
| *** jpich has quit IRC | 09:33 | |
| *** jpich has joined #openstack-rpm-packaging | 09:33 | |
| rpittau | good morning everyone, I'm checking the linters errors in https://review.opendev.org/756471 but I can't reproduce the failure locally, linters test goes fine, any hint ? | 09:43 |
| toabctl | rpittau, I guess you have a different speccleaner version locally | 09:45 |
| rpittau | toabctl: I'm using the one in the rpm-packaging repo, just running linters test in tox | 09:46 |
| toabctl | rpittau, maybe recreate your tox env | 09:47 |
| toabctl | or just fix the linter errors from the logs and continue :) | 09:47 |
| rpittau | to be honest, I don't see what went wrong, that's why I ran it locally | 09:48 |
| rpittau | it just says command failed | 09:48 |
| toabctl | there is the diff output. basically the ordering of your Requires is wrong | 09:48 |
| rpittau | ah ok | 09:48 |
| toabctl | python3-neutronclient needs to be after python3-keystoneclient | 09:48 |
| toabctl | and so on. remove the ones with "-" and add the ones with "+" | 09:49 |
| rpittau | toabctl: oh, I see, I assumed the order was the same expanded, my bad, thanks! | 09:50 |
| openstackgerrit | Riccardo Pittau proposed openstack/rpm-packaging master: Update ironic requirements to latest master https://review.opendev.org/756471 | 09:55 |
| openstackgerrit | Boyron Sébastien proposed openstack/rpm-packaging master: Add validations-libs https://review.opendev.org/756305 | 09:59 |
| openstackgerrit | Boyron Sébastien proposed openstack/rpm-packaging master: Add validations-libs https://review.opendev.org/756305 | 10:15 |
| openstackgerrit | Merged openstack/rpm-packaging master: Remove ironicclient dependency https://review.opendev.org/756472 | 10:15 |
| openstackgerrit | Riccardo Pittau proposed openstack/rpm-packaging master: Update ironic-python-agent requirements https://review.opendev.org/756473 | 10:29 |
| openstackgerrit | Boyron Sébastien proposed openstack/rpm-packaging master: Add validations-libs https://review.opendev.org/756305 | 10:46 |
| *** ykarel has quit IRC | 10:50 | |
| *** ykarel has joined #openstack-rpm-packaging | 10:50 | |
| sboyron | jpena, toabctl, dirk; I finally had several questions on https://review.opendev.org/756305 so I listed it in the review. | 11:23 |
| *** jpena is now known as jpena|lunch | 11:28 | |
| *** amoralej is now known as amoralej|lunch | 11:41 | |
| *** jpena|lunch is now known as jpena | 12:28 | |
| *** amoralej|lunch is now known as amoralej | 12:41 | |
| *** sboyron_ has joined #openstack-rpm-packaging | 13:03 | |
| *** jpich has quit IRC | 13:03 | |
| *** sboyron has quit IRC | 13:03 | |
| *** evrardjp has quit IRC | 13:04 | |
| *** evrardjp has joined #openstack-rpm-packaging | 13:04 | |
| *** jpich has joined #openstack-rpm-packaging | 13:08 | |
| sboyron_ | jpena: I'm fighting with 756305; what should be the name of the final package : python-validations-libs or python3-validations-libs ? | 13:34 |
| sboyron_ | shoudl I use {{ py2name() }} or {{ py2name(py_versions='py3') }} for the packages names ? | 13:34 |
| jpena | sboyron_: the built package will be python3-validations-libs (which is what we get from py2name(py_versions='py3')) | 13:34 |
| jpena | the Name tag (in line 6) should be python-validations-libs (which we get from py2name()) | 13:35 |
| sboyron_ | ok and all sub package ? | 13:36 |
| sboyron_ | jpena: should they use python or python3 ? | 13:36 |
| jpena | for the -doc subpackage, it should be python-validations-libs-doc | 13:36 |
| sboyron_ | python-validations-libs-doc and ansible-role-python-validations-libs | 13:37 |
| jpena | right | 13:37 |
| jpena | the ansible role could be "ansible-role-validations-libs" | 13:37 |
| sboyron_ | yes | 13:37 |
| sboyron_ | and so should I une the pypi name var for that | 13:37 |
| sboyron_ | or directly fixed ? | 13:38 |
| jpena | for tne ansible role, we should set it directly | 13:38 |
| sboyron_ | jpena: thanks a lot, it's less "mystical" now | 13:42 |
| openstackgerrit | Boyron Sébastien proposed openstack/rpm-packaging master: Add validations-libs https://review.opendev.org/756305 | 14:22 |
| openstackgerrit | Boyron Sébastien proposed openstack/rpm-packaging master: Update heat-agents to 2.1.1 https://review.opendev.org/756524 | 14:33 |
| openstackgerrit | Boyron Sébastien proposed openstack/rpm-packaging master: Add validations-libs https://review.opendev.org/756305 | 14:41 |
| *** ykarel is now known as ykarel|away | 15:00 | |
| *** ykarel|away has quit IRC | 15:32 | |
| *** amoralej is now known as amoralej|off | 15:51 | |
| *** rpittau is now known as rpittau|afk | 16:06 | |
| *** jpich has quit IRC | 16:22 | |
| *** ykarel has joined #openstack-rpm-packaging | 17:10 | |
| *** ykarel has quit IRC | 17:18 | |
| *** jpena is now known as jpena|off | 17:46 | |
| openstackgerrit | Merged openstack/rpm-packaging stable/train: Update oslo.privsep to 1.33.4. https://review.opendev.org/756011 | 18:13 |
| openstackgerrit | Merged openstack/rpm-packaging master: update telemetry-tempest-plugin to 1.1.0 https://review.opendev.org/756215 | 18:13 |
| openstackgerrit | Merged openstack/rpm-packaging master: update octavia-tempest-plugin to 1.5.0 https://review.opendev.org/756208 | 18:13 |
| openstackgerrit | Merged openstack/rpm-packaging master: Update ironic requirements to latest master https://review.opendev.org/756471 | 18:21 |
| openstackgerrit | Merged openstack/rpm-packaging stable/ussuri: stackviz: update buildrequires https://review.opendev.org/755923 | 18:21 |
| *** gyee has joined #openstack-rpm-packaging | 19:12 | |
| openstackgerrit | Merged openstack/rpm-packaging master: Update heat-agents to 2.1.1 https://review.opendev.org/756524 | 19:26 |
| *** openstackgerrit has quit IRC | 22:07 | |
| *** sboyron_ has quit IRC | 22:16 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!