Thursday, 2016-09-15

*** coolsvap has joined #openstack-rpm-packaging04:34
*** pcaruana has joined #openstack-rpm-packaging06:04
*** pcaruana is now known as pcaruana|afk|06:13
*** pcaruana|afk| is now known as pcaruana06:40
*** jpena|off is now known as jpena07:24
*** amoralej|off is now known as amoralej07:34
*** eggmaster has quit IRC08:03
*** eggmaster has joined #openstack-rpm-packaging08:03
openstackgerritHaïkel Guémar proposed openstack/rpm-packaging: Add initial spec for taskflow  https://review.openstack.org/33945808:23
openstackgerritHaïkel Guémar proposed openstack/rpm-packaging: Add initial spec for taskflow  https://review.openstack.org/33945808:23
openstackgerritHaïkel Guémar proposed openstack/rpm-packaging: Add initial spec for taskflow  https://review.openstack.org/33945808:25
*** apevec has joined #openstack-rpm-packaging08:53
*** apevec has joined #openstack-rpm-packaging08:53
*** itxaka has joined #openstack-rpm-packaging09:17
openstackgerritHaïkel Guémar proposed openstack/rpm-packaging: Add initial spec for taskflow  https://review.openstack.org/33945809:48
number80astsmtl: ping #rdo (do you have a email on our gerrit?)09:49
openstackgerritHaïkel Guémar proposed openstack/rpm-packaging: Add initial spec for python-zaqarclient  https://review.openstack.org/34047610:09
IgorYozhikovfolks - new word in commit messaging - https://review.openstack.org/#/c/370636/1 :D10:22
*** jpena is now known as jpena|lunch12:02
*** amoralej is now known as amoralej|lunch12:07
number80lol12:12
*** pradk has joined #openstack-rpm-packaging12:13
number80dirk: will you be around for today's meeting?12:13
openstackgerritAlexander Tsamutali proposed openstack/rpm-packaging: Add spec template for django_openstack_auth  https://review.openstack.org/37042512:50
openstackgerritAlexander Tsamutali proposed openstack/rpm-packaging: Add spec template for django_openstack_auth  https://review.openstack.org/37042512:53
number80meeting time!12:59
number80ping dirk toabctl IgorYozhikov number80 jruzicka12:59
openstackgerritAlexander Tsamutali proposed openstack/rpm-packaging: Add spec template for django_openstack_auth  https://review.openstack.org/37042512:59
astsmtlnumber80, pong. Don't think I have an account on RDO gerrit.13:01
*** jpena|lunch is now known as jpena13:02
number80astsmtl: can you log in (using github authentication), so I can give perms on murano packages?13:03
number80IgorYozhikov added you as a comaintainer13:03
astsmtlnumber80, okay.13:04
astsmtlnumber80, logged in as astsmtl, astsmtl@yandex.ru.13:07
number80ack13:07
*** amoralej|lunch is now known as amoralej13:15
jpenanumber80: 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
dirkjpena: if you have sufficient reasons some of the existing deps are not needed then feel free to write a patch that removes those14:03
IgorYozhikovdirk, I set wf +1 before I realized that there is no vote from SUSE CI - https://review.openstack.org/#/c/370425/ :(14:04
jpenadirk: 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
dirkIgorYozhikov: I'll check it14:05
IgorYozhikovsorry14:05
number80jpena: I created a card, please add deps you know that are missing => https://trello.com/c/YDC0t3OB/367-rpm-packaging-gate14:05
openstackgerritMerged openstack/rpm-packaging: Add spec template for django_openstack_auth  https://review.openstack.org/37042514:05
jpenain our specs, we always do "rm -f *requirements.txt", so we handle the requirements on the packaging side14:06
jpenanumber80: ack14:06
number80jpena: some of them should be explicitly overridden => http://docs.openstack.org/developer/renderspec/usage.html#handling-requirements14:20
jpenanumber80: I didn't know that was possible, very interesting14:20
* jpena reads14:20
number80yes, pytz and inotify are unlikely to be updated as they're in base RHEL14:21
number80still checking for coverage14:21
jpenaso we can just have a rdo-requirements.txt file in DLRN and use it for the overrides14:22
number80yes, if you take a look at MOS CI, that's what they've been doing14:22
jpenanumber80: 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
number80jpena: then, we need to fix that in the template generation14:27
number80we need a block for post setup cleanups here : https://github.com/openstack/renderspec/blob/master/renderspec/dist-templates/fedora.spec.j214:28
jpenaI'll have a look at it14:29
number80jruzicka may help you in getting this right14:30
jruzickaoh yeah, finally some use case for blocks :)14:34
jruzickanote there blocks are unique, you can't have two in .spec with different name14:34
jruzicka.spec.j2 that is14:35
jpenajruzicka: aha, so let's say I want to add an "rm -rf *requirements.txt" at the end of the %prep section14:35
jruzickaonly for RDO, right?14:36
jruzickajpena,14:36
jpenayes14:36
jruzickamhm, so just add that to dist-templates/fedora.spec.j2 as suggested by number80 above14:37
jpenajruzicka, something like https://paste.fedoraproject.org/428440/14739502/ ?14:37
jruzickaand then {% block reqs_cleanup %}{% enblock %} in .spec14:37
jruzickajpena, yeah14:38
jruzickas/reqs_cleanup/prep/14:38
number80that was why I was keen on having such features, it gives us a lot of flexibility14:38
number80-s14:38
jruzickayes, yes14:38
number80jruzicka++ for implementing that :)14:38
number80I need to bring you a crown14:39
jruzicka\o/14:39
number80tooling kingpin14:39
jruzickayou bet ;)14:39
jpenajruzicka, I should then add something to the spec.j2 template to get it rendered, right?14:39
jruzickasomeone packages 7 billion packages a day, someone just tinkers with tools :)14:39
jruzickatogether we are stronger ;)14:40
jruzickajpena, yes. If you use it with renderspec without the dist-template, it's just be empty14:40
jruzickajpena, {% block prep %}{% enblock %}14:41
jruzickaI mean that it won't break with older renderspec without the new blocks defined14:41
jpenaohhh this is über-cool14:42
jruzickaHA!14:42
*** pradk has quit IRC14:42
jpenaok, I'll propose the update to renderspec, then as soon as we have a release I'll prepare a mass-update patch :)14:42
jruzickayes, only disadvantage is you can't have two blocks with same name14:43
jpenawell, req-cleanup or something similar should be unique enouch14:43
jpenas/enouch/enough/14:43
jruzickayeah but for things shared between subpackages it can't be used :-/14:43
jpenaah, I see14:43
jruzickalet me know if you hit that, I might be able to hack around that with some dark magic14:44
*** pradk has joined #openstack-rpm-packaging14:45
jpenanow my only concern is if this will be ok for MOS, I understand they use --spec-style=fedora14:46
jruzickahmm, in that case we might wanna introduce new spec-style14:46
jruzickaI was also thinking about using RDO instead of fedora...14:47
jruzickanaming14:47
jpenalet's see during review, but it might make sense14:47
jruzickait's really easy to do if needed14:47
openstackgerritJavier Peña proposed openstack/renderspec: Add req_cleanup block for Fedora templates  https://review.openstack.org/37090615:05
*** pcaruana has quit IRC15:19
*** itxaka has quit IRC15:21
*** jpena is now known as jpena|off17:10
*** amoralej is now known as amoralej|off17:43
*** pcaruana has joined #openstack-rpm-packaging18:22
*** pcaruana has quit IRC19:32
*** tbarron has quit IRC21:47
*** pradk has quit IRC21:48
*** tbarron has joined #openstack-rpm-packaging21:52
*** apevec has left #openstack-rpm-packaging21:57
*** number80 has quit IRC22:50
*** number80 has joined #openstack-rpm-packaging23:12

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!