13:00:47 <dirk> #startmeeting rpm_packaging
13:00:48 <openstack> Meeting started Thu Jun 23 13:00:47 2016 UTC and is due to finish in 60 minutes.  The chair is dirk. Information about MeetBot at http://wiki.debian.org/MeetBot.
13:00:49 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
13:00:50 <dirk> hey everyone
13:00:51 <openstack> The meeting name has been set to 'rpm_packaging'
13:01:00 <dirk> please add your topics to
13:01:03 <dirk> #link https://etherpad.openstack.org/p/openstack-rpm-packaging
13:02:47 <number80> well, does not seem we have much people this week either
13:03:15 <dirk> IgorYozhikov seemed to be active in reviews today :)
13:03:32 <dirk> yeah, there is an openSUSE conference today so don't expect attendance from SUSE side
13:03:36 <number80> ack
13:03:54 <dirk> and you're there :)
13:04:04 <IgorYozhikov> o/
13:04:10 <number80> \o/
13:04:19 <number80> yeah, I'll be more active from now on
13:04:51 <IgorYozhikov> our CI folks made some upgrades in infrastructure
13:05:07 <IgorYozhikov> I'm working on stabilization of it
13:05:09 <number80> The Error 404?
13:05:28 <IgorYozhikov> yes
13:05:44 <number80> ack
13:05:45 <IgorYozhikov> issues with mirrrors sync :(
13:06:10 <dirk> hey Igor!
13:06:11 <number80> well, it was training to read fuel CI logs :)
13:06:17 <IgorYozhikov> dirk, :)
13:06:20 <dirk> yeah, its challenging to find the real error there
13:07:44 <IgorYozhikov> there are artifacts in jobs with detailed logs
13:07:48 <dirk> okay, we're out of time for collecting agenda items :)
13:07:58 <dirk> #topic reviews
13:08:27 <dirk> so easy ones for Igor are:
13:08:39 <dirk> https://review.openstack.org/#/c/331655/
13:08:51 <dirk> https://review.openstack.org/#/c/331658/
13:09:47 <dirk> https://review.openstack.org/#/c/331539/
13:10:01 <IgorYozhikov> https://review.openstack.org/#/c/332758/
13:13:03 <number80> dirk: minor nitpick on https://review.openstack.org/#/c/331539/3
13:13:28 <number80> except that, it'll get +2
13:13:30 <dirk> number80: hmm, so you don't like armv7hl ? :)
13:14:13 <number80> we can add it, I just don't remember the macros :)
13:14:46 <dirk> for us its %arm
13:15:09 <number80> yep generic rpm macros
13:15:14 <dirk> IgorYozhikov: yeah, thats a good one. in general I like the review, I haven't digested it in detail. todo
13:15:18 <number80> then add it, and it works for me
13:16:07 <number80> Yeah, this other review needs thinking
13:16:07 <IgorYozhikov> number80, looks fine but you set WIP in topic :)
13:16:23 <number80> IgorYozhikov: I wanted to get your feedback
13:16:32 <number80> but let's change topic then
13:16:43 <IgorYozhikov> I'm fine
13:16:54 <IgorYozhikov> further usage will show everything
13:17:24 <number80> I also templatized openstack-macros => https://review.openstack.org/#/c/331539/3
13:18:08 <dirk> yep, looks good to me. there is a series of reviews.. can you post them as individual ones?
13:18:21 <number80> It's work to prepare for our future gate
13:18:24 <number80> sure
13:18:49 <number80> my colleague jpena is fixing our tooling to build continuously upstream packaging
13:19:53 <dirk> nice
13:20:31 <dirk> I need to reach out to pbellanger for the build-publish step
13:20:46 <dirk> one particular review I want to point out is this one:
13:20:48 <dirk> https://review.openstack.org/#/c/311285/
13:20:58 <dirk> its becoming increasingly difficult to rebase that one
13:21:16 <number80> dirk: just to be sure, you want me to squash all reviews for openstack-macros, right?
13:21:23 <dirk> number80: no
13:21:30 <number80> ack,
13:21:51 <dirk> number80: currently priv.sep is in the same "branch" than the openstack-macro change
13:21:59 <number80> ok
13:22:05 <dirk> so the jenkins jobs for openstack-macro are also including the priv.sep change, which is why they fail
13:22:26 <dirk> it would be better to post privsep under a separate topic (git branch) and remove it from the branch of the openstack-macros one
13:23:15 <dirk> the other two are related so they can stay in one branch
13:23:22 <dirk> I hope I was able to explain this better :)
13:23:32 <number80> done
13:23:49 <dirk> thank you!
13:23:58 <number80> I likely played too much w/ branches, I didn't notice
13:24:32 <dirk> so coming back to the spec-cleaner review, aka https://review.openstack.org/#/c/311285/
13:24:54 <dirk> other than the jenkins failures, what are the opinions about this? I can do a final rebase to get the jenkins to pass, but I'd like to get it merged then
13:25:05 <dirk> it will have my +2, any opinion from number80  / Igor ?
13:25:18 <IgorYozhikov> Ш ыуте куигшде
13:25:19 <number80> It'll get my +2
13:25:52 <dirk> IgorYozhikov: sorry? :)
13:25:54 <IgorYozhikov> I sent rebuild - even if our CI will fail I will merge it
13:26:09 <dirk> ok..
13:26:10 <IgorYozhikov> we have some post upgrade issues
13:26:22 <IgorYozhikov> with mirrors sync as I mnetioned before
13:26:26 <dirk> well, the jenkins job is still failing. I'll go fixing that then and ping you both to review/approve
13:26:32 <dirk> thanks!
13:26:36 <dirk> any other topic?
13:26:46 <IgorYozhikov> pymo2pkg CR
13:26:50 <number80> yes
13:27:13 <number80> I've updated mapping for RDO and added a new RegexRule
13:27:30 <IgorYozhikov> just saw your comments dirk
13:27:41 <IgorYozhikov> will address them https://review.openstack.org/#/c/330527/
13:28:27 <number80> oh yeah, two pymod2pkg topics :)
13:28:37 <IgorYozhikov> dirk, should be using the global requirements, aka remove the versioning override
13:29:09 <IgorYozhikov> May be I didn't understand your previous comments for versions
13:30:19 <IgorYozhikov> https://review.openstack.org/#/c/330527/2..3/openstack/pymod2pkg/pymod2pkg.spec.j2
13:30:47 <dirk> number80: oh, sorry, that ended up in a different folder (the pymod2pkg reviews). will go over them
13:30:58 <number80> np
13:31:49 <dirk> IgorYozhikov: replace the line {{py2pkg(foo, >=, 1.2.3)}} with {{py2pkg(foo) }}
13:32:17 <IgorYozhikov> that is how it was earlier
13:34:22 <IgorYozhikov> ok? I'll address your comment today
13:34:58 <dirk> IgorYozhikov: hmm, maybe I'm confused then. let me check
13:35:43 <dirk> IgorYozhikov: ah. what I meant is that the %doc package should have a "Requires: %name = %version"
13:35:55 <dirk> e.g. the -doc package requires the main package in the same version
13:36:03 <IgorYozhikov> O I C
13:36:13 <IgorYozhikov> will fix that
13:36:18 <dirk> thanks!
13:39:09 <dirk> any last topic?
13:39:45 <number80> for now, I'm good, we have enough work to keep us busy
13:41:41 <dirk> number80: if you have time, please also untangle https://review.openstack.org/#/c/332651 and friends into separate reviews
13:41:58 <dirk> ok, then lets end it for today.. thanks for attending and see you next week same time same place!
13:42:03 <dirk> #endmeeting