13:33:36 <chandankumar> #startmeeting RDO Office Hour - 2017-09-26
13:33:36 <openstack> Meeting started Tue Sep 26 13:33:36 2017 UTC and is due to finish in 60 minutes.  The chair is chandankumar. Information about MeetBot at http://wiki.debian.org/MeetBot.
13:33:37 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
13:33:39 <openstack> The meeting name has been set to 'rdo_office_hour___2017_09_26'
13:33:51 <chandankumar> Welcome to another editions of RDO office hour.
13:33:59 <chandankumar> Here is the etherpad link:
13:34:13 <chandankumar> #link https://review.rdoproject.org/etherpad/p/rdo-office-hour-easyfixes
13:34:20 <chandankumar> #topic Roll Call
13:35:23 <chandankumar> jpena: amoralej|lunch jruzicka jschlueter aditya_r number80 office hour time
13:35:35 <number80> o/
13:35:39 <chandankumar> #chair number80
13:35:40 <openstack> Current chairs: chandankumar number80
13:35:49 <aditya_r> o/
13:36:01 <chandankumar> #chair aditya_r
13:36:02 <openstack> Current chairs: aditya_r chandankumar number80
13:36:04 <number80> I'm half here, half not-here since I have a cold :)
13:36:25 <Duck> mrunge: https://bugzilla.redhat.com/show_bug.cgi?id=1491017
13:36:25 <openstack> bugzilla.redhat.com bug 1491017 in Package Review "Review Request: needrestart -- checks which daemons need to be restarted after library upgrades" [Unspecified,New] - Assigned to nobody
13:36:45 <mrunge> Duck: thanks. Will have a look
13:36:57 <chandankumar> number80: same here, from morning, i also got cold, my nose is now red, due to liquid flow
13:37:33 <chandankumar> Here is the agenda for today's RDO office Hour.
13:37:46 <chandankumar> * https://review.rdoproject.org/r/#/q/status:open+branch:rpm-master+easyfix Reviews cleanup
13:38:01 <chandankumar> * Package Review Queue: https://bugzilla.redhat.com/buglist.cgi?component=Package%20Review&product=RDO&list_id=7897762
13:38:12 <aditya_r> chandankumar: please suggest me on https://review.rdoproject.org/r/#/c/9543/
13:38:16 <chandankumar> * Cleanup python3 packages: https://trunk-primary.rdoproject.org/f25/report.html
13:38:44 <chandankumar> So starting with Package review
13:39:36 <chandankumar> #info If a package in RDO still in review and Feel free to add Depends flag with RDO Queens Tracker https://bugzilla.redhat.com/show_bug.cgi?id=1486366
13:39:37 <openstack> bugzilla.redhat.com bug 1486366 in distribution "Tracker: Blockers and Review requests for new RDO Queens packages" [Unspecified,New] - Assigned to apevec
13:40:11 <rdogerrit> Merged openstack/novajoin-tempest-plugin-distgit rpm-master: Initial import of spec file  https://review.rdoproject.org/r/9310
13:40:25 <chandankumar> If a package is already reviewed and inital spec file is imported, feel to close the bug
13:41:06 <chandankumar> alee: python-novajoin-tempest-tests is now available
13:41:12 <chandankumar> aditya_r: checking
13:41:56 <aditya_r> chandankumar: I already done that, send me if any.
13:41:56 <chandankumar> jpena: regarding rpm-macros reviews, is rpm-macros issue is shorted out for fedora?
13:42:36 <jpena> chandankumar: not yet. We need https://softwarefactory-project.io/r/9699 to be merged, then we can recheck and merge https://review.rdoproject.org/r/9716
13:42:41 <jpena> and then it will be fixe
13:42:42 <jpena> d
13:43:18 <chandankumar> #info rpm-macros reviews depends on https://softwarefactory-project.io/r/9699 and https://review.rdoproject.org/r/#/c/9716/
13:43:46 <EmilienM> jaosorior: ok thx
13:48:53 <chandankumar> ykarel: if you are around https://review.rdoproject.org/r/#/q/status:open+branch:rpm-master+easyfix please go through this review.
13:49:00 <ykarel> chandankumar, ack
13:49:04 <chandankumar> #chair ykarel jpena
13:49:05 <openstack> Current chairs: aditya_r chandankumar jpena number80 ykarel
13:49:47 <jatanmalde> o/
13:50:08 <chandankumar> #chair jatanmalde
13:50:08 <openstack> Current chairs: aditya_r chandankumar jatanmalde jpena number80 ykarel
13:52:07 <chandankumar> aditya_r: what about doing this only http://paste.openstack.org/show/621939/ and add the rest of the description just below that
13:53:15 <chandankumar> aditya_r: http://paste.openstack.org/show/621941/
13:53:46 <chandankumar> aditya_r: you can ask about what other packages provides, like -tests contains what and improve the wording.
13:54:09 <Duck> mrunge: :-)
13:55:24 <ykarel> chandankumar, why this review is stuck: https://review.rdoproject.org/r/#/c/9377/?
13:55:56 <chandankumar> ykarel: no idea, done a recheck
13:56:02 <ykarel> Ok
13:58:43 <aditya_r> chandankumar: ok, will do that.
14:03:56 <rdogerrit> User vakwetu proposed rdoinfo master: Add queens tag to novajoin-tempest-plugin  https://review.rdoproject.org/r/9389
14:07:19 <alee> chandankumar, thanks!  I rebased https://review.rdoproject.org/r/#/c/9389/
14:08:31 <rdogerrit> Aditya Ramteke proposed openstack/sahara-distgit rpm-master: Moved package description to global variable and replace with it.  https://review.rdoproject.org/r/9543
14:10:25 <rdogerrit> Merged openstack/zaqar-distgit rpm-master: Replaced rm -rf requirements.txt step with proper rpm macro.  https://review.rdoproject.org/r/9566
14:13:24 <rdobot> [sensu] NEW: master.monitoring.rdoproject.org - check-delorean-master-current @ http://tinyurl.com/y8hxewm2 |#| Build failure on centos7-master/current: nova: http://trunk.rdoproject.org/centos7-master/report.html
14:13:40 <chandankumar> shreshtha: if you are around https://review.rdoproject.org/r/#/q/status:open+branch:rpm-master+easyfix feel free to review some
14:14:15 <shreshtha> ack chandankumar
14:14:27 <chandankumar> #chair shreshtha
14:14:27 <openstack> Current chairs: aditya_r chandankumar jatanmalde jpena number80 shreshtha ykarel
14:15:14 <chandankumar> amoralej: jpena needs one more +2 and workflow to get it merged https://review.rdoproject.org/r/#/q/topic:easyfix/23+status:open
14:16:48 <panda> jpena: hey, is there any test endpoint for dlrn api ? a instance to launch api commands, without really making any modifications ?
14:17:10 <chandankumar> alee: we need to make some comsmetic changes in the novajoin spec file: https://github.com/rdo-packages/novajoin-tempest-plugin-distgit/blob/rpm-master/python-novajoin-tests-tempest.spec
14:17:22 <chandankumar> like %global service novajoin
14:17:24 <chandankumar> %global plugin novajoin-tempest-pluginY
14:18:29 <chandankumar> sorry lost somewhere else.
14:18:58 <alee> chandankumar, whats the change to make? service -> novajoin-tests-tempest ?
14:19:06 <chandankumar> alee: yes
14:19:37 <alee> chandankumar, ok , what change to %global plugin ?
14:19:53 <chandankumar> alee: and URL: https://github.com/openstack/%{plugin}/ with git.openstack.org
14:20:29 <jpena> panda: no, I have not created any test endpoint. I think I'll fire up a test VM for that
14:20:38 <chandankumar> alee: plugin is ok, rest is good
14:21:02 <alee> chandankumar, ok - I'll add a new review with those changes
14:21:09 <chandankumar> alee: thanks :-)
14:21:42 <alee> chandankumar, whats the rpmlint error?
14:21:49 <rdogerrit> Merged openstack/heat-distgit rpm-master: Moved package description to global variable and replace with it.  https://review.rdoproject.org/r/9556
14:21:52 <rdogerrit> Merged openstack/neutron-vpnaas-distgit rpm-master: Replaced rm -rf requirements.txt step with proper rpm macro.  https://review.rdoproject.org/r/9626
14:21:55 <rdogerrit> Merged openstack/osops-tools-monitoring-oschecks-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro.  https://review.rdoproject.org/r/9603
14:21:57 <rdogerrit> Merged openstack/novajoin-distgit rpm-master: Replaced rm -rf requirements.txt step with proper rpm macro.  https://review.rdoproject.org/r/9562
14:22:13 <rdogerrit> Merged openstack/glance-distgit rpm-master: convert individual requires to macro  https://review.rdoproject.org/r/9518
14:22:29 <rdogerrit> Merged openstack/trove-distgit rpm-master: convert individual requires to macro  https://review.rdoproject.org/r/9506
14:22:35 <rdogerrit> Merged openstack/murano-agent-distgit rpm-master: convert individual requires to macro  https://review.rdoproject.org/r/9504
14:22:56 <chandankumar> alee: rpmlint error was false positive we can ignore that
14:23:06 <alee> chandankumar, ok cool
14:23:07 <chandankumar> alee: https://review.rdoproject.org/r/#/c/9389/ please remove depends on
14:23:15 <chandankumar> alee: as the patch is already merged
14:23:26 <alee> chandankumar, ok
14:24:19 <panda> jpena: ping me when you do, thanks
14:24:23 <rdogerrit> Aditya Ramteke proposed openstack/shaker-distgit rpm-master: Enable documentation build.  https://review.rdoproject.org/r/9752
14:25:06 <aditya_r> ykarel: done https://review.rdoproject.org/r/#/c/9752/
14:25:51 <ykarel> aditya_r, ack
14:28:26 <rdogerrit> User vakwetu proposed rdoinfo master: Add queens tag to novajoin-tempest-plugin  https://review.rdoproject.org/r/9389
14:29:09 <rdogerrit> Merged openstack/keystone-distgit rpm-master: convert individual requires to macro  https://review.rdoproject.org/r/9533
14:29:57 <alee> chandankumar, ^^ commit message updated
14:34:27 <rdogerrit> Aditya Ramteke proposed openstack/oslo-db-distgit rpm-master: Added with_doc conditional over -doc subpackage  https://review.rdoproject.org/r/9399
14:34:40 <jschlueter> number80: hmm how do we get this out of a stuck state? https://review.rdoproject.org/r/#/q/status:open+project:openstack/neutronclient-distgit+branch:rpm-master
14:35:16 * number80 looks
14:44:39 <rdogerrit> Aditya Ramteke proposed openstack/tripleo-common-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro.  https://review.rdoproject.org/r/9591
14:51:16 <rdogerrit> Aditya Ramteke proposed openstack/taskflow-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro.  https://review.rdoproject.org/r/9588
14:53:45 <rdogerrit> Merged openstack/ironic-distgit rpm-master: convert individual requires to macro.  https://review.rdoproject.org/r/9748
14:53:50 <rdogerrit> Merged openstack/swift-distgit rpm-master: convert individual requires to macro  https://review.rdoproject.org/r/9529
14:53:58 <rdogerrit> Merged openstack/cisco-ironic-contrib-distgit rpm-master: convert individual requires to macro  https://review.rdoproject.org/r/9512
14:54:48 <sfbender> Merged rdopkg master: Add topy to tox as a linting check  https://softwarefactory-project.io/r/9751
14:57:25 <rdogerrit> Alfredo Moralejo proposed openstack/zaqar-distgit rpm-master: convert individual requires to macro  https://review.rdoproject.org/r/9511
14:57:56 <rdogerrit> Aditya Ramteke proposed openstack/heat-distgit rpm-master: Replaced rm -rf requirements.txt step with proper rpm macro.  https://review.rdoproject.org/r/9557
15:00:45 <rdogerrit> Merged rdoinfo master: Add queens tag to novajoin-tempest-plugin  https://review.rdoproject.org/r/9389
15:02:46 <rdogerrit> Merged openstack/networking-cisco-distgit rpm-master: convert individual requires to macro  https://review.rdoproject.org/r/9532
15:02:55 <rdogerrit> Merged openstack/watcher-distgit rpm-master: convert individual requires to macro  https://review.rdoproject.org/r/9510
15:02:57 <rdogerrit> Merged openstack/kuryr-kubernetes-distgit rpm-master: convert individual requires to macro  https://review.rdoproject.org/r/9534
15:03:07 <rdogerrit> Merged openstack/ironic-inspector-distgit rpm-master: convert individual requires to macro  https://review.rdoproject.org/r/9527
15:03:26 <rdogerrit> Merged openstack/designate-distgit rpm-master: convert individual requires to macro  https://review.rdoproject.org/r/9530
15:03:28 <rdogerrit> Merged openstack/vitrage-distgit rpm-master: convert individual requires to macro  https://review.rdoproject.org/r/9537
15:03:30 <rdogerrit> Merged openstack/tacker-distgit rpm-master: convert individual requires to macro  https://review.rdoproject.org/r/9531
15:09:00 <rdogerrit> Merged openstack/aodh-distgit rpm-master: convert individual requires to macro  https://review.rdoproject.org/r/9515
15:09:26 <alee> chandankumar, whats the url supposed to be exactly? https://git.openstack.org/%{plugin}/  ?
15:09:36 <rdogerrit> Merged openstack/barbican-distgit rpm-master: convert individual requires to macro  https://review.rdoproject.org/r/9517
15:10:03 <alee> chandankumar, or http://git.openstack.org/cgit/openstack/{plugin} ?
15:10:48 <chandankumar> alee: https://git.openstack.org/openstack/novajoin-tempest-plugin
15:11:11 <alee> chandankumar, cool that was my third choice :)
15:11:17 <chandankumar> hehe
15:16:14 <number80> ok, my spare laptop is now dying
15:16:47 <number80> keyboard is inserting spurious characters randomly, I can't even unlock the HD
15:20:10 <openstackgerrit> Javier Peña proposed openstack/packstack master: Disable Keystone v2 tests for Tempest  https://review.openstack.org/507565
15:22:14 <rdogerrit> Aditya Ramteke proposed openstack/sahara-distgit rpm-master: Moved package description to global variable and replace with it.  https://review.rdoproject.org/r/9543
15:22:34 <rdogerrit> User vakwetu created openstack/novajoin-tempest-plugin-distgit rpm-master: Cosmetic changes to spec file  https://review.rdoproject.org/r/9757
15:22:45 <alee> chandankumar, ^^
15:24:56 <chandankumar> jschlueter: https://review.rdoproject.org/r/#/c/9757/ sorry i missed your comments in last patch set
15:25:08 <chandankumar> jschlueter: please have a look
15:25:18 <rdogerrit> Merged openstack/ironic-python-agent-distgit rpm-master: convert individual requires to macro  https://review.rdoproject.org/r/9520
15:25:38 <jschlueter> chandankumar: np looking now
15:25:44 <number80> alee: first change post-review is to enable build in trunk (this one doesn't need to keep the add-xxx topic)
15:26:24 <jschlueter> chandankumar: does it make sense to update the template?
15:26:26 <rdogerrit> Merged openstack/cloudkitty-dashboard-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro  https://review.rdoproject.org/r/9628
15:26:32 <chandankumar> jschlueter: yup
15:27:26 <rdogerrit> chkumar246 proposed openstack/congress-distgit rpm-master: convert individual requires to macro  https://review.rdoproject.org/r/9535
15:27:47 <jschlueter> chandankumar: one thing I noticed is python-six vs python2-six we have mixed usage between specs ... and RHEL python-six package is still python-six with no Provides ... :-(
15:28:29 <chandankumar> amoralej: jpena number80 https://review.rdoproject.org/r/#/q/topic:easyfix/23+status:open only 3 renaming
15:28:32 <jschlueter> python-six-1.9.0-2.el7                    rhel-7.2
15:28:55 <amoralej> chandankumar, good
15:29:28 <number80> qck
15:29:34 <jschlueter> so it's 1 minor build behind ...
15:29:57 <chandankumar> #info we have merged almost all reviews related related https://github.com/redhat-openstack/easyfix/issues/23
15:30:15 <chandankumar> #link https://review.rdoproject.org/r/#/q/topic:easyfix/23
15:30:59 <chandankumar> jpena: all patches merged related to easyfix23 systemd one \o/
15:31:13 <jschlueter> number80: did you figure out what was up with novaclient distgit reviews and why they are stuck currently?
15:31:46 <chandankumar> jschlueter: do we need to stick to python-six then?
15:31:59 <jschlueter> chandankumar: /me not sure
15:32:07 <number80> jschlueter: my laptop keyboard just died on me
15:32:46 <chandankumar> times up!
15:32:50 <jschlueter> chandankumar: for OSP we handle it just fine but anyone else attempting the build or trying to do RDO on RHEL will either pull python-six from RDO or fail with missing python2-six
15:33:01 <chandankumar> #endmeeting