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