*** coolsvap has joined #openstack-rpm-packaging | 04:34 | |
*** pcaruana has joined #openstack-rpm-packaging | 06:04 | |
*** pcaruana is now known as pcaruana|afk| | 06:13 | |
*** pcaruana|afk| is now known as pcaruana | 06:40 | |
*** jpena|off is now known as jpena | 07:24 | |
*** amoralej|off is now known as amoralej | 07:34 | |
*** eggmaster has quit IRC | 08:03 | |
*** eggmaster has joined #openstack-rpm-packaging | 08:03 | |
openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Add initial spec for taskflow https://review.openstack.org/339458 | 08:23 |
---|---|---|
openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Add initial spec for taskflow https://review.openstack.org/339458 | 08:23 |
openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Add initial spec for taskflow https://review.openstack.org/339458 | 08:25 |
*** apevec has joined #openstack-rpm-packaging | 08:53 | |
*** apevec has joined #openstack-rpm-packaging | 08:53 | |
*** itxaka has joined #openstack-rpm-packaging | 09:17 | |
openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Add initial spec for taskflow https://review.openstack.org/339458 | 09:48 |
number80 | astsmtl: ping #rdo (do you have a email on our gerrit?) | 09:49 |
openstackgerrit | Haïkel Guémar proposed openstack/rpm-packaging: Add initial spec for python-zaqarclient https://review.openstack.org/340476 | 10:09 |
IgorYozhikov | folks - new word in commit messaging - https://review.openstack.org/#/c/370636/1 :D | 10:22 |
*** jpena is now known as jpena|lunch | 12:02 | |
*** amoralej is now known as amoralej|lunch | 12:07 | |
number80 | lol | 12:12 |
*** pradk has joined #openstack-rpm-packaging | 12:13 | |
number80 | dirk: will you be around for today's meeting? | 12:13 |
openstackgerrit | Alexander Tsamutali proposed openstack/rpm-packaging: Add spec template for django_openstack_auth https://review.openstack.org/370425 | 12:50 |
openstackgerrit | Alexander Tsamutali proposed openstack/rpm-packaging: Add spec template for django_openstack_auth https://review.openstack.org/370425 | 12:53 |
number80 | meeting time! | 12:59 |
number80 | ping dirk toabctl IgorYozhikov number80 jruzicka | 12:59 |
openstackgerrit | Alexander Tsamutali proposed openstack/rpm-packaging: Add spec template for django_openstack_auth https://review.openstack.org/370425 | 12:59 |
astsmtl | number80, pong. Don't think I have an account on RDO gerrit. | 13:01 |
*** jpena|lunch is now known as jpena | 13:02 | |
number80 | astsmtl: can you log in (using github authentication), so I can give perms on murano packages? | 13:03 |
number80 | IgorYozhikov added you as a comaintainer | 13:03 |
astsmtl | number80, okay. | 13:04 |
astsmtl | number80, logged in as astsmtl, astsmtl@yandex.ru. | 13:07 |
number80 | ack | 13:07 |
*** amoralej|lunch is now known as amoralej | 13:15 | |
jpena | number80: about the RDO gate, once the current Newton rush has passed we should sit down for a moment and remove the last few hurdles to get it done (it's mostly a couple dependencies that we don't/can't have in our current openstack-newton repos) | 14:01 |
dirk | jpena: if you have sufficient reasons some of the existing deps are not needed then feel free to write a patch that removes those | 14:03 |
IgorYozhikov | dirk, I set wf +1 before I realized that there is no vote from SUSE CI - https://review.openstack.org/#/c/370425/ :( | 14:04 |
jpena | dirk: in most cases, the issue is that some requirements.txt reqs include versions we don't ship for some reasons (e.g. we have coverage 3.6b3 which should be API-equivalent to 3.6, but python setup.py doesn't care) | 14:05 |
dirk | IgorYozhikov: I'll check it | 14:05 |
IgorYozhikov | sorry | 14:05 |
number80 | jpena: I created a card, please add deps you know that are missing => https://trello.com/c/YDC0t3OB/367-rpm-packaging-gate | 14:05 |
openstackgerrit | Merged openstack/rpm-packaging: Add spec template for django_openstack_auth https://review.openstack.org/370425 | 14:05 |
jpena | in our specs, we always do "rm -f *requirements.txt", so we handle the requirements on the packaging side | 14:06 |
jpena | number80: ack | 14:06 |
number80 | jpena: some of them should be explicitly overridden => http://docs.openstack.org/developer/renderspec/usage.html#handling-requirements | 14:20 |
jpena | number80: I didn't know that was possible, very interesting | 14:20 |
* jpena reads | 14:20 | |
number80 | yes, pytz and inotify are unlikely to be updated as they're in base RHEL | 14:21 |
number80 | still checking for coverage | 14:21 |
jpena | so we can just have a rdo-requirements.txt file in DLRN and use it for the overrides | 14:22 |
number80 | yes, if you take a look at MOS CI, that's what they've been doing | 14:22 |
jpena | number80: actually, this is only fixing versions in the spec itself, right? We have the issue during build/check, because they're checking for the python package version (for example http://209.132.178.209/repos/6e/9f/6e9f7226c86f110e3b537eeeaf5f61e9edc355f2_16c7e131/rpmbuild.log) | 14:24 |
number80 | jpena: then, we need to fix that in the template generation | 14:27 |
number80 | we need a block for post setup cleanups here : https://github.com/openstack/renderspec/blob/master/renderspec/dist-templates/fedora.spec.j2 | 14:28 |
jpena | I'll have a look at it | 14:29 |
number80 | jruzicka may help you in getting this right | 14:30 |
jruzicka | oh yeah, finally some use case for blocks :) | 14:34 |
jruzicka | note there blocks are unique, you can't have two in .spec with different name | 14:34 |
jruzicka | .spec.j2 that is | 14:35 |
jpena | jruzicka: aha, so let's say I want to add an "rm -rf *requirements.txt" at the end of the %prep section | 14:35 |
jruzicka | only for RDO, right? | 14:36 |
jruzicka | jpena, | 14:36 |
jpena | yes | 14:36 |
jruzicka | mhm, so just add that to dist-templates/fedora.spec.j2 as suggested by number80 above | 14:37 |
jpena | jruzicka, something like https://paste.fedoraproject.org/428440/14739502/ ? | 14:37 |
jruzicka | and then {% block reqs_cleanup %}{% enblock %} in .spec | 14:37 |
jruzicka | jpena, yeah | 14:38 |
jruzicka | s/reqs_cleanup/prep/ | 14:38 |
number80 | that was why I was keen on having such features, it gives us a lot of flexibility | 14:38 |
number80 | -s | 14:38 |
jruzicka | yes, yes | 14:38 |
number80 | jruzicka++ for implementing that :) | 14:38 |
number80 | I need to bring you a crown | 14:39 |
jruzicka | \o/ | 14:39 |
number80 | tooling kingpin | 14:39 |
jruzicka | you bet ;) | 14:39 |
jpena | jruzicka, I should then add something to the spec.j2 template to get it rendered, right? | 14:39 |
jruzicka | someone packages 7 billion packages a day, someone just tinkers with tools :) | 14:39 |
jruzicka | together we are stronger ;) | 14:40 |
jruzicka | jpena, yes. If you use it with renderspec without the dist-template, it's just be empty | 14:40 |
jruzicka | jpena, {% block prep %}{% enblock %} | 14:41 |
jruzicka | I mean that it won't break with older renderspec without the new blocks defined | 14:41 |
jpena | ohhh this is über-cool | 14:42 |
jruzicka | HA! | 14:42 |
*** pradk has quit IRC | 14:42 | |
jpena | ok, I'll propose the update to renderspec, then as soon as we have a release I'll prepare a mass-update patch :) | 14:42 |
jruzicka | yes, only disadvantage is you can't have two blocks with same name | 14:43 |
jpena | well, req-cleanup or something similar should be unique enouch | 14:43 |
jpena | s/enouch/enough/ | 14:43 |
jruzicka | yeah but for things shared between subpackages it can't be used :-/ | 14:43 |
jpena | ah, I see | 14:43 |
jruzicka | let me know if you hit that, I might be able to hack around that with some dark magic | 14:44 |
*** pradk has joined #openstack-rpm-packaging | 14:45 | |
jpena | now my only concern is if this will be ok for MOS, I understand they use --spec-style=fedora | 14:46 |
jruzicka | hmm, in that case we might wanna introduce new spec-style | 14:46 |
jruzicka | I was also thinking about using RDO instead of fedora... | 14:47 |
jruzicka | naming | 14:47 |
jpena | let's see during review, but it might make sense | 14:47 |
jruzicka | it's really easy to do if needed | 14:47 |
openstackgerrit | Javier Peña proposed openstack/renderspec: Add req_cleanup block for Fedora templates https://review.openstack.org/370906 | 15:05 |
*** pcaruana has quit IRC | 15:19 | |
*** itxaka has quit IRC | 15:21 | |
*** jpena is now known as jpena|off | 17:10 | |
*** amoralej is now known as amoralej|off | 17:43 | |
*** pcaruana has joined #openstack-rpm-packaging | 18:22 | |
*** pcaruana has quit IRC | 19:32 | |
*** tbarron has quit IRC | 21:47 | |
*** pradk has quit IRC | 21:48 | |
*** tbarron has joined #openstack-rpm-packaging | 21:52 | |
*** apevec has left #openstack-rpm-packaging | 21:57 | |
*** number80 has quit IRC | 22:50 | |
*** number80 has joined #openstack-rpm-packaging | 23:12 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!