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