13:33:17 #startmeeting RDO Office Hour - 2017-11-14 13:33:17 Meeting started Tue Nov 14 13:33:17 2017 UTC. The chair is chandankumar. Information about MeetBot at http://wiki.debian.org/MeetBot. 13:33:17 Useful Commands: #action #agreed #halp #info #idea #link #topic. 13:33:17 The meeting name has been set to 'rdo_office_hour_-_2017-11-14' 13:33:17 Meeting started Tue Nov 14 13:33:16 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:18 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 13:33:20 The meeting name has been set to 'rdo_office_hour___2017_11_14' 13:33:44 #topic Roll Call 13:33:49 \o/ 13:34:38 jpena|lunch: amoralej|lunch apevec office hour timing 13:35:17 Agenda for today's RDO office hour. 13:35:45 #info Easyfix 4 review cleanup https://review.rdoproject.org/r/#/q/topic:easyfix/5+status:open 13:36:05 #undo 13:36:05 Removing item from minutes: INFO by chandankumar at 13:35:45 : Easyfix 4 review cleanup https://review.rdoproject.org/r/#/q/topic:easyfix/5+status:open 13:36:06 Removing item from minutes: #info Easyfix 4 review cleanup https://review.rdoproject.org/r/#/q/topic:easyfix/5+status:open 13:36:12 #info Easyfix 4 review cleanup https://review.rdoproject.org/r/#/q/topic:easyfix/4+status:open 13:36:22 #info Easyfix 5 review cleanup https://review.rdoproject.org/r/#/q/topic:easyfix/5+status:open 13:37:07 Next week i will work on adding more easyfixes for the project. 13:37:37 aditya_ra: \o/ 13:38:02 aditya_ra: thanks for updating the merge conflicts, few more are left feel free to do that 13:41:00 o/ 13:42:04 #chair aditya_r 13:42:04 Current chairs: aditya_r chandankumar 13:42:05 Current chairs: aditya_r chandankumar 13:44:09 chandankumar: on it. I have just come from the gym :) 13:45:11 Merged openstack/neutron-lbaas-dashboard-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro https://review.rdoproject.org/r/9604 13:46:00 Merged openstack/troveclient-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro https://review.rdoproject.org/r/9640 13:46:00 Merged openstack/muranoclient-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro https://review.rdoproject.org/r/9644 13:46:01 Merged openstack/designateclient-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro https://review.rdoproject.org/r/9646 13:46:01 Merged openstack/oslo-utils-distgit rpm-master: Replaced rm -rf requirements.txt step with proper rpm macro. https://review.rdoproject.org/r/9577 13:46:31 [sensu] NEW: master.monitoring.rdoproject.org - check-delorean-master-current @ http://tinyurl.com/y8hxewm2 |#| Build failure on centos7-master/current: horizon: http://trunk.rdoproject.org/centos7-master/report.html 13:46:53 Merged openstack/kuryr-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro https://review.rdoproject.org/r/9625 13:47:13 Merged openstack/keystonemiddleware-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro https://review.rdoproject.org/r/9639 13:48:17 Aditya Ramteke proposed openstack/monascaclient-distgit rpm-master: Replaced rm -rf requirements.txt step with proper rpm macro. https://review.rdoproject.org/r/9655 13:48:35 amoralej|lunch: jpena|lunch feel free to go through this list https://review.rdoproject.org/r/#/q/topic:easyfix/4+status:open when you are free. 13:50:32 Aditya Ramteke proposed openstack/swiftclient-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro. https://review.rdoproject.org/r/9660 13:52:33 Aditya Ramteke proposed openstack/magnumclient-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro. https://review.rdoproject.org/r/9658 13:54:18 Merged openstack/django_openstack_auth-distgit rpm-master: Replaced rm -rf requirements.txt step with proper rpm macro. https://review.rdoproject.org/r/9547 13:54:29 Aditya Ramteke proposed openstack/heatclient-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro https://review.rdoproject.org/r/9647 13:56:36 Aditya Ramteke proposed openstack/neutron-lib-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro https://review.rdoproject.org/r/9641 13:56:46 Merged openstack/oslo-db-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro https://review.rdoproject.org/r/9596 13:56:47 Merged openstack/mistral-lib-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro https://review.rdoproject.org/r/9642 13:57:19 Merged openstack/os-xenapi-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro https://review.rdoproject.org/r/9634 13:57:19 Merged openstack/oslo-i18n-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro. https://review.rdoproject.org/r/9669 13:57:25 apevec, amoralej|lunch, weshay, adarazs|ruck: definitely a good trend since the registry was upgraded to 3.7 13:57:26 https://review.rdoproject.org/jenkins/job/periodic-tripleo-centos-7-pike-containers-build/buildTimeTrend 13:57:33 https://review.rdoproject.org/jenkins/job/periodic-tripleo-centos-7-master-containers-build/buildTimeTrend 13:58:02 looks like build times are fairly consistent and reliable, also 13:58:17 Merged openstack/oslo-context-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro https://review.rdoproject.org/r/9586 13:58:28 Aditya Ramteke proposed openstack/gnocchiclient-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro https://review.rdoproject.org/r/9638 14:00:02 Merged openstack/swiftclient-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro. https://review.rdoproject.org/r/9660 14:01:02 Aditya Ramteke proposed openstack/aodhclient-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro. https://review.rdoproject.org/r/9656 14:01:54 dmsimard|off, nice 14:01:56 thanks 14:02:41 panda: did you figure out why containers aren't tagged ? 14:02:47 panda: it's a pretty important issue... 14:03:15 Aditya Ramteke proposed openstack/osprofiler-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro https://review.rdoproject.org/r/9657 14:03:23 dmsimard|off, there was a patch replacing docker role with direct CLI call in promotion... lemme find it 14:03:49 apevec: yeah tagging 125 containers with origin client takes around one minute (no joke) 14:03:52 damn, sf token expiring... 14:04:19 patch was: https://review.rdoproject.org/r/#/c/10453/1/ci-scripts/container-push/container-push.yml 14:04:36 it was merged, but I guess we didn't have promotion since then 14:04:58 should use oc tag.. 14:05:00 it is still not using oc tag 14:05:15 like here https://github.com/rdo-infra/ansible-role-rdo-kolla-build/blob/master/tag-promotion.yml 14:05:28 panda, ^ 14:05:28 docker push has i/o lookups on registry 14:05:52 but promotion also pushes to dockerhub? 14:05:53 Aditya Ramteke proposed openstack/hacking-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro. https://review.rdoproject.org/r/9668 14:06:08 apevec: that specific task mentions "rdo registry" 14:07:36 Aditya Ramteke proposed openstack/barbicanclient-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro https://review.rdoproject.org/r/9661 14:07:39 TIL skopeo has a "copy" command to copy images from a registry to the other 14:07:55 https://github.com/projectatomic/skopeo#copying-images 14:08:13 I'll create a bz for doing registry backups. 14:08:19 ah that could be used for pushing to dockerhub? 14:08:30 panda, adarazs|ruck ^ 14:08:49 * adarazs|ruck back from a meeting. 14:08:55 reads back 14:08:58 adarazs|ruck, who is your rover? 14:09:16 rlandy|rover :) 14:09:20 apevec: ^ 14:09:36 and amoralej|lunch is my |lunch ;) 14:09:41 ah, should be prefixed vs suffix :) 14:09:51 Aditya Ramteke proposed openstack/cinderclient-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro. https://review.rdoproject.org/r/9662 14:09:54 apevec: that would screw with people's autocomplete muscle memory. 14:10:08 at least when they try to talk to me. 14:12:29 sshnaidm: was that what you had implemented a while back ? https://docs.docker.com/registry/recipes/mirror/ 14:13:04 * adarazs|ruck got overwhelmed with meetings, can't check stuff for now. 14:13:10 dmsimard, yep 14:13:28 sshnaidm: is that still used ? 14:13:37 dmsimard, no, I don't think so 14:13:43 sshnaidm: why not ? 14:13:47 dmsimard, is it needed? 14:14:01 dmsimard, I thought we are gonna use rdoproject registry 14:14:03 sshnaidm: I think it'd go along the same lines as requiring a mirror in RDO cloud 14:14:18 sshnaidm: there's still some stuff pulled from dockerhub, like centos base image, ceph.. 14:14:33 sshnaidm: it's okay, though, I'll put it on our todo list 14:14:40 Aditya Ramteke proposed openstack/oslo-versionedobjects-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro. https://review.rdoproject.org/r/9664 14:14:47 dmsimard, well, I can revive it I think.. 14:14:58 sshnaidm: no rush 14:15:46 dmsimard, needs support for this config changes too 14:15:53 yeah 14:17:56 Aditya Ramteke proposed openstack/stevedore-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro. https://review.rdoproject.org/r/9670 14:20:35 Aditya Ramteke proposed openstack/oslotest-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro. https://review.rdoproject.org/r/9665 14:22:00 apevec: what is rover here? 14:23:08 Aditya Ramteke proposed openstack/oslo-sphinx-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro. https://review.rdoproject.org/r/9666 14:25:31 Aditya Ramteke proposed openstack/os-client-config-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro. https://review.rdoproject.org/r/9659 14:27:50 #info aditya_r is working on updating all the merge conflicts reviews related to easyfix 4 and 5 14:27:51 Aditya Ramteke proposed openstack/ironicclient-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro https://review.rdoproject.org/r/9599 14:28:12 #info reviews related to easyfix 4 and 5 got cleaned up 14:28:38 Merged openstack/ceilometerclient-distgit rpm-master: Moved package description to global variable. https://review.rdoproject.org/r/9954 14:28:42 Merged openstack/cliff-distgit rpm-master: Moved package description to global variable. https://review.rdoproject.org/r/9952 14:29:55 dmsimard, could that good trend be because of registry is almost empty?, or do you think it will maintain over time? 14:31:04 Aditya Ramteke proposed openstack/oslo-messaging-distgit rpm-master: Replace rm -rf requirements.txt step with proper rpm macro. https://review.rdoproject.org/r/9590 14:31:43 dmsimard, if you are back from pto, could you take a look into https://review.rdoproject.org/r/#/c/10466/ whe you have a chance? 14:31:49 time to end office hour 14:31:52 #endmeeting