13:32:39 #startmeeting RDO Office Hour - 2017-10-03 13:32:39 Meeting started Tue Oct 3 13:32:39 2017 UTC. The chair is chandankumar. Information about MeetBot at http://wiki.debian.org/MeetBot. 13:32:39 Useful Commands: #action #agreed #halp #info #idea #link #topic. 13:32:39 The meeting name has been set to 'rdo_office_hour_-_2017-10-03' 13:32:40 Meeting started Tue Oct 3 13:32:39 2017 UTC and is due to finish in 60 minutes. The chair is chandankumar. Information about MeetBot at http://wiki.debian.org/MeetBot. 13:32:42 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 13:32:44 The meeting name has been set to 'rdo_office_hour___2017_10_03' 13:33:20 Here is the agenda link https://review.rdoproject.org/etherpad/p/rdo-office-hour-easyfixes 13:33:21 apevec: ack 13:33:27 #topic Roll Call 13:34:12 o/ 13:34:16 o/ 13:34:23 #chair rbowen jpena 13:34:23 Current chairs: chandankumar jpena rbowen 13:34:24 Current chairs: chandankumar jpena rbowen 13:34:41 o/ 13:35:08 #chair amoralej 13:35:08 Current chairs: amoralej chandankumar jpena rbowen 13:35:08 Current chairs: amoralej chandankumar jpena rbowen 13:36:10 Here is the today's agenda: 13:36:31 * Easyfix#4 Reviews: https://review.rdoproject.org/r/#/q/topic:easyfix/4+status:open 13:36:48 * Easyfix#5 Reviews: https://review.rdoproject.org/r/#/q/status:open+branch:rpm-master+topic:easyfix/5 13:38:17 that's it 13:38:25 we have a long review queue 13:39:06 Let's start it cleaning up 13:39:18 #link Easyfix#4 Reviews: https://review.rdoproject.org/r/#/q/topic:easyfix/4+status:open 13:39:29 #link Easyfix#5 Reviews: https://review.rdoproject.org/r/#/q/status:open+branch:rpm-master+topic:easyfix/5 13:40:06 Haïkel Guémar proposed graffiti master: Use rdoinfo database to detect CBS tags https://softwarefactory-project.io/r/9805 13:40:07 o/ 13:41:13 #chair number80 13:41:13 Current chairs: amoralej chandankumar jpena number80 rbowen 13:41:14 Current chairs: amoralej chandankumar jpena number80 rbowen 13:41:32 o/ 13:42:25 #chair jatanmalde 13:42:25 Current chairs: amoralej chandankumar jatanmalde jpena number80 rbowen 13:42:26 Current chairs: amoralej chandankumar jatanmalde jpena number80 rbowen 13:44:07 o/ 13:45:50 #chair aditya_r 13:45:50 Current chairs: aditya_r amoralej chandankumar jatanmalde jpena number80 rbowen 13:45:51 Current chairs: aditya_r amoralej chandankumar jatanmalde jpena number80 rbowen 13:46:35 amoralej: number80: jpena I am looking for new chair to rotate RDO office hour each weekly or can we make RDO office hour bi-weekly? 13:49:06 Haïkel Guémar proposed graffiti master: Use rdoinfo database to detect CBS tags https://softwarefactory-project.io/r/9805 13:49:11 o/ 13:49:59 chandankumar: there are only three reviews left for https://review.rdoproject.org/r/#/q/topic:easyfix/11 13:50:12 once these are done, easyfix/11 can be closed :) 13:50:24 chandankumar: I can take it next week 13:51:05 snecklifter: https://review.rdoproject.org/r/#/c/9866/1/python-oslo-db.spec Is there any need to add Common Description? 13:52:04 Merged openstack/barbicanclient-distgit rpm-master: Moved package description to global variable. https://review.rdoproject.org/r/9886 13:52:53 snecklifter: please send one more patch to cleanup changelog from tackerclient 13:52:58 aditya_r: just my preference but yes, because this is generally how we are handling other commits 13:53:40 snecklifter: for python-os-vif hacking is also not need, need one more patch 13:53:53 Merged openstack/kolla-distgit rpm-master: Moved the description to global variable https://review.rdoproject.org/r/9226 13:54:16 chandankumar: ack, perhaps another easyfix to check for hacking or have we done that already? 13:54:26 snecklifter: i think it was there 13:54:52 snecklifter: hacking should be also removed python-k8sclient 13:55:56 snecklifter: done 13:56:10 snecklifter: feel free to close it 13:56:26 Merged openstack/tackerclient-distgit rpm-master: Removed BuildRequires/Requires: python-coverage from .spec https://review.rdoproject.org/r/9785 13:57:08 chandankumar: done, good work! 13:58:39 Merged openstack/os-vif-distgit rpm-master: Removed BuildRequires/Requires: python-coverage from .spec https://review.rdoproject.org/r/9783 13:59:25 #info https://github.com/redhat-openstack/easyfix/issues/11 -> Closed 14:00:41 Christopher Brown created openstack/tackerclient-distgit rpm-master: clean up changelog https://review.rdoproject.org/r/9911 14:01:06 Merged openstack/k8sclient-distgit rpm-master: Removed BuildRequires/Requires: python-coverage from .spec https://review.rdoproject.org/r/9782 14:03:33 Aditya Ramteke proposed openstack/oslo-db-distgit rpm-master: Moved package description to global variable. https://review.rdoproject.org/r/9866 14:03:37 Merged openstack/shaker-distgit rpm-master: Moved package description to global variable and replace with it. https://review.rdoproject.org/r/9333 14:03:54 Christopher Brown created openstack/os-vif-distgit rpm-master: remove hacking BR https://review.rdoproject.org/r/9912 14:06:20 Merged openstack/magnumclient-distgit rpm-master: Moved package description to global variable. https://review.rdoproject.org/r/9890 14:06:26 Aditya Ramteke proposed openstack/designateclient-distgit rpm-master: Moved package description to global variable. https://review.rdoproject.org/r/9897 14:07:44 Christopher Brown created openstack/k8sclient-distgit rpm-master: remove hacking BR https://review.rdoproject.org/r/9913 14:08:01 snecklifter: https://review.rdoproject.org/r/#/c/9870/ please ack 14:09:02 Merged openstack/kuryr-kubernetes-distgit rpm-master: Moved package description to global variable and replace with it. https://review.rdoproject.org/r/9550 14:09:59 aditya_r: ? 14:10:25 Merged openstack/oslo-context-distgit rpm-master: Moved package description to global variable and replace with it. https://review.rdoproject.org/r/9587 14:10:50 snecklifter: The global hasn't been used anywhere in this commit. ?? 14:11:33 aditya_r: what do you mean please ack? 14:12:20 aditya_r: you have defined the global but not used it anywhere 14:13:22 Merged openstack/cinderclient-distgit rpm-master: Moved package description to global variable. https://review.rdoproject.org/r/9885 14:14:30 number80: Hi, thanks for dependencies reviews in tripleo-ui! This is the last dependencies patch which needs your eyes currently. https://review.openstack.org/#/c/505609/ (It also moves React from BSD+Patents to MIT license) 14:14:44 Merged openstack/muranoclient-distgit rpm-master: Moved package description to global variable. https://review.rdoproject.org/r/9898 14:14:51 ack 14:15:13 snecklifter: Ok, I got it now. Sorry for that. 14:15:32 aditya_r: np 14:19:45 aditya_r, https://review.rdoproject.org/r/#/c/9395/ 14:19:58 check this aditya_r if you can rebase it 14:21:04 whats with the "no restorecon" errors? 14:22:27 Merged openstack/oslo-serialization-distgit rpm-master: Moved the description to global variables. https://review.rdoproject.org/r/9381 14:23:06 Merged openstack/neutron-lib-distgit rpm-master: Moved package description to global variable. https://review.rdoproject.org/r/9901 14:23:15 Aditya Ramteke proposed openstack/networking-bigswitch-distgit rpm-master: Moved package description to global variable. https://review.rdoproject.org/r/9871 14:24:30 jatanmalde: Sure, give a minute. 14:27:17 Merged openstack/monascaclient-distgit rpm-master: Moved package description to global variable. https://review.rdoproject.org/r/9893 14:27:27 David Moreau Simard proposed rdo-infra/rdo-infra-playbooks master: Add monitoring for http and https for RDO domains https://review.rdoproject.org/r/9910 14:27:56 Merged openstack/openstacksdk-distgit rpm-master: Moved the description to global variables. https://review.rdoproject.org/r/9374 14:28:57 Jatan Malde proposed openstack/oslo-serialization-distgit rpm-master: Added the doc_guard https://review.rdoproject.org/r/9446 14:30:35 hguemar created gating_scripts master: Prepare transition to newer graffiti https://review.rdoproject.org/r/9914 14:30:36 hguemar created gating_scripts master: WIP: Follow-up cleanup on graffiti jobs config file https://review.rdoproject.org/r/9915 14:31:57 Merged openstack/mistral-lib-distgit rpm-master: Moved package description to global variable. https://review.rdoproject.org/r/9900 14:32:30 Merged openstack/senlinclient-distgit rpm-master: Add python3-os-testr as a build requirement https://review.rdoproject.org/r/9896 14:32:49 panda, dmsimard - reading back, why would we not tag the DLRN hash? It's nicely self-documenting what's the source 14:33:18 jpena: if it doesn't take too much time, can you update the test endpoint with the latest hashes ? older hashes have no associated containers anymore. 14:33:34 apevec: the DLRN hash is tagged by default IIUC 14:34:05 panda: ok, let me import the latest db contents once I finish the current meeting 14:34:12 for pike, they are up to date 14:34:14 jpena: thanks. 14:34:18 panda, yes, and I'd like we keep that 14:34:40 I understood from backchat that you're considering removing it? 14:35:02 apevec: we will, not touching that part, just didn't remember if we did it and what was the format 14:35:19 ok 14:35:27 Alfredo Moralejo created rdoinfo master: Pin puppet-firewall https://review.rdoproject.org/r/9916 14:36:54 ^ second reviewer needed for pin 14:43:48 Merged openstack/tackerclient-distgit rpm-master: clean up changelog https://review.rdoproject.org/r/9911 14:48:18 Merged openstack/oslo-sphinx-distgit rpm-master: Moved package description to global variable. https://review.rdoproject.org/r/9881 14:48:45 Merged openstack/osprofiler-distgit rpm-master: Moved package description to global variable. https://review.rdoproject.org/r/9891 14:49:25 Merged openstack/ironicclient-distgit rpm-master: Moved package description to global variable. https://review.rdoproject.org/r/9805 14:49:26 Merged openstack/hacking-distgit rpm-master: Moved package description to global variable. https://review.rdoproject.org/r/9880 14:50:05 Merged rdoinfo master: Pin puppet-firewall https://review.rdoproject.org/r/9916 14:50:49 hi, can anyone check those reviews for *sahara* stuff? 14:50:50 https://review.rdoproject.org/r/#/c/9717/ 14:50:55 https://review.rdoproject.org/r/#/c/9543/ 14:50:59 https://review.rdoproject.org/r/#/c/9544/ 14:51:11 ignore the last one, it should be rebased 14:51:45 and https://review.rdoproject.org/r/#/c/9450/ 14:52:27 Merged openstack/os-testr-distgit rpm-master: Moved package description to global variable. https://review.rdoproject.org/r/9869 14:53:12 Alfredo Moralejo created puppet/puppet-firewall-distgit rpm-master: Dummy commit to force rebuild https://review.rdoproject.org/r/9917 14:53:19 Merged openstack/oslo-vmware-distgit rpm-master: Moved the description to global variable https://review.rdoproject.org/r/9231 14:55:31 Merged openstack/ironic-ui-distgit rpm-master: Moved package description to global variable. https://review.rdoproject.org/r/9872 14:56:28 panda: updated with fresh DB dump 14:57:31 Merged openstack/neutronclient-distgit rpm-master: Moved package description to global variable. https://review.rdoproject.org/r/9868 14:58:01 Merged openstack/keystonemiddleware-distgit rpm-master: Moved package description to global variable. https://review.rdoproject.org/r/9874 14:58:06 jpena: thanks! 15:04:36 Javier Peña proposed openstack/sahara-distgit rpm-master: Replaced rm -rf requirements.txt step with proper rpm macro. https://review.rdoproject.org/r/9544 15:10:59 Javier Peña created openstack/oslo-rootwrap-distgit rpm-master: Add iproutes as build requirement, re-enable unit tests https://review.rdoproject.org/r/9918 15:25:32 hi guys, we have a ton of these cloudsig promote jobs on CentOS CI: https://ci.centos.org/view/rdo/view/cbs-promote-pipelines/ -- what are we using these for, can somebody tell me? 15:25:51 Jakub Ružička proposed rdopkg master: normalize commit messages https://softwarefactory-project.io/r/9781 15:30:44 Times up :-) 15:30:49 #endmeeting