| *** wolverineav has joined #openstack-infra | 00:04 | |
| *** sflanigan has joined #openstack-infra | 00:04 | |
| *** sflanigan has joined #openstack-infra | 00:04 | |
| *** sree_ has joined #openstack-infra | 00:06 | |
| *** sree_ is now known as Guest89443 | 00:06 | |
| *** SumitNaiksatam has quit IRC | 00:08 | |
| *** gmann has quit IRC | 00:10 | |
| *** gmann has joined #openstack-infra | 00:10 | |
| *** slaweq has joined #openstack-infra | 00:14 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Rename zuul-revoke-sudo to revoke-sudo https://review.openstack.org/481331 | 00:15 |
|---|---|---|
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Copy in run-docs role as zuul-run-docs https://review.openstack.org/481332 | 00:15 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Rename zuul-run-docs to run-docs https://review.openstack.org/481333 | 00:15 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-roles master: Remove unneeded roles https://review.openstack.org/481334 | 00:15 |
| openstackgerrit | Clark Boylan proposed openstack-infra/git-review master: Handle http queries below / https://review.openstack.org/478325 | 00:15 |
| openstackgerrit | Clark Boylan proposed openstack-infra/git-review master: Fix mock assertions for python3 https://review.openstack.org/481335 | 00:15 |
| clarkb | fungi: ^ I'm hopeful that actuall passes testing now | 00:15 |
| mordred | ianw: yes! :) | 00:16 |
| *** slaweq has quit IRC | 00:18 | |
| *** Sukhdev has quit IRC | 00:21 | |
| *** thorst has joined #openstack-infra | 00:23 | |
| *** thorst has quit IRC | 00:26 | |
| *** armax has joined #openstack-infra | 00:26 | |
| *** felipemonteiro has quit IRC | 00:28 | |
| clarkb | huh apparnetly something in my bug fix patch causes that bytestring stuff to be a problem? | 00:30 |
| clarkb | I'm going to squash for now to get reviews going | 00:30 |
| clarkb | so confused | 00:30 |
| *** Guest89443 has quit IRC | 00:30 | |
| *** wolverineav has quit IRC | 00:31 | |
| openstackgerrit | Clark Boylan proposed openstack-infra/git-review master: Handle http queries below / https://review.openstack.org/478325 | 00:31 |
| *** LindaWang has joined #openstack-infra | 00:42 | |
| *** claudiub has quit IRC | 00:42 | |
| *** zhurong has joined #openstack-infra | 00:44 | |
| *** gildub has joined #openstack-infra | 00:44 | |
| *** mat128 has joined #openstack-infra | 00:44 | |
| *** vhosakot has quit IRC | 00:46 | |
| *** ijw has joined #openstack-infra | 00:46 | |
| *** gildub has quit IRC | 00:52 | |
| *** ijw has quit IRC | 00:52 | |
| *** gildub has joined #openstack-infra | 00:54 | |
| *** dangers_away has quit IRC | 00:54 | |
| *** mat128 has quit IRC | 00:54 | |
| *** dangers_away has joined #openstack-infra | 00:54 | |
| *** lbragstad has quit IRC | 00:56 | |
| *** cuongnv has joined #openstack-infra | 00:56 | |
| openstackgerrit | Zhenguo Niu proposed openstack-infra/devstack-gate master: DNM: test mogan gate https://review.openstack.org/481341 | 00:57 |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Rename git_host to server in github driver https://review.openstack.org/477594 | 00:59 |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Clarify canonical_hostname documentation https://review.openstack.org/479020 | 00:59 |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Add docs on allow-secrets https://review.openstack.org/480726 | 00:59 |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Fix indentation error in docs https://review.openstack.org/480740 | 00:59 |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Move status_url to webapp config section https://review.openstack.org/480759 | 01:00 |
| *** ihrachys_ has joined #openstack-infra | 01:01 | |
| *** ykarel has joined #openstack-infra | 01:02 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Pass result data back from ansible https://review.openstack.org/479442 | 01:05 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: WIP: remove success/failure url options https://review.openstack.org/481342 | 01:05 |
| *** sree_ has joined #openstack-infra | 01:05 | |
| *** sree_ is now known as Guest12148 | 01:06 | |
| openstackgerrit | Jamie Lennox proposed openstack-infra/zuul feature/zuulv3: Cleanup on ssh-agent failure https://review.openstack.org/481344 | 01:06 |
| *** thorst has joined #openstack-infra | 01:06 | |
| *** thorst has quit IRC | 01:07 | |
| *** jkilpatr has quit IRC | 01:08 | |
| *** ijw has joined #openstack-infra | 01:08 | |
| *** lbragstad has joined #openstack-infra | 01:10 | |
| *** ijw has quit IRC | 01:13 | |
| *** slaweq has joined #openstack-infra | 01:15 | |
| *** yamahata has quit IRC | 01:19 | |
| *** slaweq has quit IRC | 01:20 | |
| *** ym200 has quit IRC | 01:20 | |
| *** iyamahat has quit IRC | 01:20 | |
| openstackgerrit | Merged openstack-infra/jenkins-job-builder master: Update logstash to make use of convert_mapping_to_xml() https://review.openstack.org/479903 | 01:22 |
| *** ym200 has joined #openstack-infra | 01:22 | |
| *** Apoorva has quit IRC | 01:30 | |
| *** s-shiono has joined #openstack-infra | 01:34 | |
| *** liujiong has joined #openstack-infra | 01:34 | |
| *** ijw has joined #openstack-infra | 01:36 | |
| fungi | jeblair: #thanks support is not yet fully landed... entrypoint change for system-config is 473536 and still needs reviews | 01:41 |
| *** Julien-zte has joined #openstack-infra | 01:41 | |
| clarkb | oh I misunderstood and thought it had landed for some reason. I've made a note to review in the morning | 01:42 |
| fungi | it's been on my starred list for a while but lately reviewing teh flurry of docs refactor and eol-related stuff has sucked up my review time | 01:42 |
| openstackgerrit | David Moreau Simard proposed openstack-infra/project-config master: Update branches for gerritbot notifications for Packstack https://review.openstack.org/481354 | 01:46 |
| *** ijw has quit IRC | 01:47 | |
| *** makowals has quit IRC | 01:50 | |
| *** hongbin has joined #openstack-infra | 01:55 | |
| *** makowals has joined #openstack-infra | 01:58 | |
| *** ramishra has joined #openstack-infra | 02:07 | |
| *** dhinesh has quit IRC | 02:14 | |
| *** mat128 has joined #openstack-infra | 02:17 | |
| *** Mutter has joined #openstack-infra | 02:20 | |
| *** Julien-zte has quit IRC | 02:20 | |
| *** cody-somerville has joined #openstack-infra | 02:22 | |
| *** gyee has quit IRC | 02:23 | |
| *** Julien-zte has joined #openstack-infra | 02:23 | |
| dmsimard | AJaeger: so how does one actually retire a branch ? :/ | 02:27 |
| *** csomerville has joined #openstack-infra | 02:27 | |
| dmsimard | I'm part of the release ACL for packstack fwiw, just never had to do something like this before | 02:27 |
| *** cody-somerville has quit IRC | 02:30 | |
| AJaeger | dmsimard: send a followup to http://lists.openstack.org/pipermail/openstack-infra/2017-July/005480.html, please | 02:30 |
| AJaeger | you missed the last round... | 02:30 |
| dmsimard | AJaeger: we have as far back as kilo :( | 02:31 |
| dmsimard | AJaeger: I'll ask the person who was supposed to be taking care of that tomorrow | 02:31 |
| *** wolverineav has joined #openstack-infra | 02:31 | |
| *** Mutter has quit IRC | 02:35 | |
| *** wolverineav has quit IRC | 02:38 | |
| *** ykarel has quit IRC | 02:42 | |
| openstackgerrit | Merged openstack-infra/project-config master: tripleo: change featureset number for COE job https://review.openstack.org/480590 | 02:50 |
| *** Guest12148 has quit IRC | 02:51 | |
| *** hieulq_ has joined #openstack-infra | 02:52 | |
| *** wolverineav has joined #openstack-infra | 02:53 | |
| openstackgerrit | Amrith Kumar proposed openstack/diskimage-builder master: fix readme.rst to reflect correct environment variable https://review.openstack.org/481385 | 02:54 |
| *** hieulq_ has quit IRC | 02:58 | |
| *** hieulq_ has joined #openstack-infra | 02:58 | |
| *** hieulq_ has quit IRC | 02:59 | |
| *** gagehugo has quit IRC | 03:00 | |
| *** Julien-zte has quit IRC | 03:00 | |
| *** wolverineav has quit IRC | 03:02 | |
| *** gagehugo has joined #openstack-infra | 03:03 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/tripleo-ci master: Do not duplicate the postci.txt file https://review.openstack.org/481386 | 03:04 |
| *** ym200 has quit IRC | 03:05 | |
| *** ym200 has joined #openstack-infra | 03:06 | |
| *** hieulq_ has joined #openstack-infra | 03:07 | |
| *** wolverineav has joined #openstack-infra | 03:07 | |
| *** ykarel has joined #openstack-infra | 03:07 | |
| *** thorst has joined #openstack-infra | 03:07 | |
| *** chlong_ has quit IRC | 03:08 | |
| *** chlong_ has joined #openstack-infra | 03:09 | |
| *** jamesmcarthur has joined #openstack-infra | 03:10 | |
| *** thorst has quit IRC | 03:13 | |
| *** jamesmcarthur has quit IRC | 03:15 | |
| *** chlong_ has quit IRC | 03:17 | |
| *** chlong_ has joined #openstack-infra | 03:18 | |
| *** makowals has quit IRC | 03:19 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Remove obsolete python34-jobs https://review.openstack.org/481389 | 03:20 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: WIP: Create python-jobs-trusty https://review.openstack.org/481390 | 03:20 |
| *** felipemonteiro has joined #openstack-infra | 03:23 | |
| *** gagehugo has quit IRC | 03:25 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Remove obsolete python-jobs-linters template https://review.openstack.org/481392 | 03:25 |
| *** psachin has joined #openstack-infra | 03:26 | |
| *** gagehugo has joined #openstack-infra | 03:27 | |
| *** hieulq_ has quit IRC | 03:32 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: mitaka-eol: Handle astara https://review.openstack.org/481397 | 03:34 |
| *** vhosakot has joined #openstack-infra | 03:35 | |
| *** wolverineav has quit IRC | 03:37 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: WIP: Create python-jobs-trusty https://review.openstack.org/481390 | 03:39 |
| *** makowals has joined #openstack-infra | 03:39 | |
| *** mat128 has quit IRC | 03:39 | |
| AJaeger | ianw, could you put https://review.openstack.org/#/q/status:open+project:openstack-infra/project-config+branch:master+topic:mitaka-eol on your review list, please? For further cleanup for mitaka, I need a clean slate ;) thanks | 03:40 |
| *** felipemonteiro has quit IRC | 03:43 | |
| AJaeger | thanks, ianw ! | 03:48 |
| *** iyamahat has joined #openstack-infra | 03:48 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: mitaka-eol: Handle networking-odl https://review.openstack.org/481399 | 03:48 |
| *** iyamahat has quit IRC | 03:50 | |
| *** iyamahat has joined #openstack-infra | 03:51 | |
| openstackgerrit | Merged openstack-infra/project-config master: mitaka-eol: Remove obsolete translation setup https://review.openstack.org/481180 | 03:51 |
| *** iyamahat has quit IRC | 03:54 | |
| *** iyamahat has joined #openstack-infra | 03:54 | |
| openstackgerrit | Merged openstack-infra/project-config master: mitaka-eol: Remove trusty from grafana https://review.openstack.org/481142 | 03:54 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Fix tripleo skip conditions https://review.openstack.org/481403 | 03:54 |
| *** wolverineav has joined #openstack-infra | 03:55 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: WIP: Create python-jobs-trusty https://review.openstack.org/481390 | 03:55 |
| *** jascott1 has joined #openstack-infra | 03:55 | |
| *** yamamoto has joined #openstack-infra | 03:56 | |
| *** rlandy has quit IRC | 03:59 | |
| *** iyamahat_ has joined #openstack-infra | 04:00 | |
| *** iyamahat has quit IRC | 04:00 | |
| *** wolverineav has quit IRC | 04:02 | |
| *** gagehugo has quit IRC | 04:02 | |
| *** wolverineav has joined #openstack-infra | 04:02 | |
| *** gagehugo has joined #openstack-infra | 04:03 | |
| *** iyamahat_ has quit IRC | 04:04 | |
| openstackgerrit | Andreas Florath proposed openstack/diskimage-builder master: Introduce manual setting of DIB_INIT_SYSTEM https://review.openstack.org/414179 | 04:05 |
| *** thorst has joined #openstack-infra | 04:09 | |
| openstackgerrit | Merged openstack/diskimage-builder master: fix readme.rst to reflect correct environment variable https://review.openstack.org/481385 | 04:10 |
| *** wolverineav has quit IRC | 04:11 | |
| *** thorst has quit IRC | 04:14 | |
| *** lbragstad has quit IRC | 04:14 | |
| *** wolverineav has joined #openstack-infra | 04:15 | |
| *** yamahata has joined #openstack-infra | 04:17 | |
| *** slaweq has joined #openstack-infra | 04:19 | |
| *** SumitNaiksatam has joined #openstack-infra | 04:20 | |
| *** wolverineav has quit IRC | 04:20 | |
| *** wolverineav has joined #openstack-infra | 04:21 | |
| openstackgerrit | Andreas Florath proposed openstack/diskimage-builder master: Docker matrix build https://review.openstack.org/414347 | 04:22 |
| *** slaweq has quit IRC | 04:24 | |
| *** wolverineav has quit IRC | 04:25 | |
| *** links has joined #openstack-infra | 04:29 | |
| *** winggundamth has joined #openstack-infra | 04:34 | |
| *** claudiub has joined #openstack-infra | 04:35 | |
| *** ihrachys_ has quit IRC | 04:35 | |
| *** wolverineav has joined #openstack-infra | 04:39 | |
| *** adisky__ has joined #openstack-infra | 04:40 | |
| *** faizy has joined #openstack-infra | 04:42 | |
| *** wolverineav has quit IRC | 04:44 | |
| *** armax has quit IRC | 04:45 | |
| *** armax has joined #openstack-infra | 04:46 | |
| *** makowals has quit IRC | 04:46 | |
| *** liujiong is now known as liujiong|away | 04:52 | |
| *** SumitNaiksatam has quit IRC | 04:53 | |
| *** wolverineav has joined #openstack-infra | 04:54 | |
| *** hongbin has quit IRC | 04:55 | |
| *** zzzeek has joined #openstack-infra | 04:55 | |
| *** shu-mutou-AWAY is now known as shu-mutou | 04:56 | |
| *** zzzeek has quit IRC | 04:56 | |
| openstackgerrit | Andreas Florath proposed openstack/diskimage-builder master: zypper: Repect proxy settings https://review.openstack.org/481419 | 04:58 |
| *** makowals has joined #openstack-infra | 05:00 | |
| *** SumitNaiksatam has joined #openstack-infra | 05:00 | |
| *** zzzeek has joined #openstack-infra | 05:02 | |
| *** zzzeek has quit IRC | 05:02 | |
| *** zzzeek has joined #openstack-infra | 05:04 | |
| *** zzzeek has quit IRC | 05:04 | |
| *** zzzeek has joined #openstack-infra | 05:08 | |
| *** wolverineav has quit IRC | 05:09 | |
| *** zzzeek has quit IRC | 05:09 | |
| *** zzzeek has joined #openstack-infra | 05:10 | |
| *** pratik_ has joined #openstack-infra | 05:11 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Create python-trusty-jobs https://review.openstack.org/481390 | 05:12 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Create python-jobs-trusty zuul template https://review.openstack.org/481425 | 05:12 |
| *** wolverineav has joined #openstack-infra | 05:14 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Create python-jobs-trusty job group https://review.openstack.org/481390 | 05:15 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Create python-jobs-trusty zuul template https://review.openstack.org/481425 | 05:15 |
| *** zzzeek has quit IRC | 05:16 | |
| openstackgerrit | Andreas Florath proposed openstack/diskimage-builder master: Docker matrix build https://review.openstack.org/414347 | 05:18 |
| *** slaweq has joined #openstack-infra | 05:19 | |
| *** thorst has joined #openstack-infra | 05:21 | |
| *** zzzeek has joined #openstack-infra | 05:22 | |
| *** zzzeek has quit IRC | 05:23 | |
| *** slaweq has quit IRC | 05:25 | |
| *** wolverineav has quit IRC | 05:25 | |
| *** prateek has joined #openstack-infra | 05:25 | |
| *** thorst has quit IRC | 05:25 | |
| *** wolverineav has joined #openstack-infra | 05:28 | |
| *** jascott1 has quit IRC | 05:30 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Move all trusty jobs to python-jobs-trusty job group https://review.openstack.org/481430 | 05:31 |
| *** jascott1 has joined #openstack-infra | 05:31 | |
| *** yamahata has quit IRC | 05:32 | |
| *** wolverineav has quit IRC | 05:34 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul feature/zuulv3: Optionally bind /lib64 https://review.openstack.org/481432 | 05:35 |
| *** SumitNaiksatam has quit IRC | 05:37 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: mitaka-eol: remove trusty-job from fuel-ccp-component https://review.openstack.org/481434 | 05:38 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: mitaka-eol: Remove trusty linters job https://review.openstack.org/481435 | 05:38 |
| *** zzzeek has joined #openstack-infra | 05:40 | |
| *** zzzeek has quit IRC | 05:40 | |
| openstackgerrit | Andreas Florath proposed openstack/diskimage-builder master: Docker matrix build https://review.openstack.org/414347 | 05:41 |
| *** hrubi has quit IRC | 05:44 | |
| *** bhavik1 has joined #openstack-infra | 05:44 | |
| *** wolverineav has joined #openstack-infra | 05:45 | |
| *** rcernin has joined #openstack-infra | 05:46 | |
| *** hrubi has joined #openstack-infra | 05:46 | |
| *** florianf has joined #openstack-infra | 05:48 | |
| *** wolverineav has quit IRC | 05:49 | |
| *** pgadiya has joined #openstack-infra | 05:50 | |
| *** markvoelker has quit IRC | 05:50 | |
| *** markvoelker has joined #openstack-infra | 05:51 | |
| *** wolverineav has joined #openstack-infra | 05:53 | |
| *** ianw is now known as ianw_pto | 05:55 | |
| *** markvoelker has quit IRC | 05:55 | |
| *** vhosakot has quit IRC | 05:55 | |
| *** wolverineav has quit IRC | 05:57 | |
| *** junbo has quit IRC | 06:01 | |
| *** zzzeek has joined #openstack-infra | 06:02 | |
| *** dhajare has joined #openstack-infra | 06:03 | |
| *** andreas_s has joined #openstack-infra | 06:04 | |
| *** armax has quit IRC | 06:07 | |
| *** slaweq has joined #openstack-infra | 06:07 | |
| pratik_ | AJaeger, clarkb : can you please review https://review.openstack.org/#/c/475300/? | 06:07 |
| *** armax has joined #openstack-infra | 06:07 | |
| *** armax has quit IRC | 06:08 | |
| *** armax has joined #openstack-infra | 06:08 | |
| *** armax has quit IRC | 06:08 | |
| *** armax has joined #openstack-infra | 06:09 | |
| *** armax has quit IRC | 06:09 | |
| *** armax has joined #openstack-infra | 06:10 | |
| *** armax has quit IRC | 06:10 | |
| *** armax has joined #openstack-infra | 06:10 | |
| *** armax has quit IRC | 06:11 | |
| *** armax has joined #openstack-infra | 06:11 | |
| *** armax has quit IRC | 06:12 | |
| *** e0ne has joined #openstack-infra | 06:13 | |
| *** wolverineav has joined #openstack-infra | 06:13 | |
| *** zzzeek has quit IRC | 06:17 | |
| openstackgerrit | Andreas Florath proposed openstack/diskimage-builder master: zypper: Respect proxy settings https://review.openstack.org/481419 | 06:22 |
| *** wolverineav has quit IRC | 06:23 | |
| *** markus_z has joined #openstack-infra | 06:24 | |
| *** ajafo has quit IRC | 06:26 | |
| *** ajafo has joined #openstack-infra | 06:26 | |
| *** wolverineav has joined #openstack-infra | 06:26 | |
| *** liujiong|away is now known as liujiong | 06:26 | |
| *** wolverineav has quit IRC | 06:31 | |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack-infra/project-config master: jenkins: jobs: ansible-role-jobs: Always enable the EPEL repo https://review.openstack.org/478095 | 06:33 |
| *** mrodden has quit IRC | 06:37 | |
| *** timburke has quit IRC | 06:37 | |
| *** pabelanger has quit IRC | 06:37 | |
| *** beagles has quit IRC | 06:37 | |
| *** hrubi has quit IRC | 06:37 | |
| *** yolanda_ has quit IRC | 06:38 | |
| *** charz has quit IRC | 06:38 | |
| *** cinerama has quit IRC | 06:38 | |
| *** notmyname has quit IRC | 06:38 | |
| *** hrubi has joined #openstack-infra | 06:38 | |
| *** lastmikoi has quit IRC | 06:38 | |
| *** mhayden has quit IRC | 06:38 | |
| *** notmyname has joined #openstack-infra | 06:38 | |
| *** timburke has joined #openstack-infra | 06:38 | |
| *** kzaitsev1pi has quit IRC | 06:39 | |
| *** fabo has quit IRC | 06:39 | |
| *** mrodden has joined #openstack-infra | 06:39 | |
| *** charz has joined #openstack-infra | 06:39 | |
| *** wolverineav has joined #openstack-infra | 06:39 | |
| *** fabo has joined #openstack-infra | 06:39 | |
| *** fabo has quit IRC | 06:39 | |
| *** fabo has joined #openstack-infra | 06:39 | |
| *** kzaitsev_pi has joined #openstack-infra | 06:39 | |
| *** pabelanger has joined #openstack-infra | 06:40 | |
| *** mhayden has joined #openstack-infra | 06:40 | |
| *** cinerama has joined #openstack-infra | 06:40 | |
| *** lastmikoi has joined #openstack-infra | 06:43 | |
| *** b3nt_pin has joined #openstack-infra | 06:43 | |
| *** b3nt_pin is now known as Guest9640 | 06:43 | |
| *** jbadiapa has quit IRC | 06:43 | |
| *** yolanda_ has joined #openstack-infra | 06:44 | |
| *** gouthamr has quit IRC | 06:47 | |
| *** ijw has joined #openstack-infra | 06:48 | |
| *** wolverineav has quit IRC | 06:51 | |
| *** ijw has quit IRC | 06:52 | |
| *** e0ne has quit IRC | 06:53 | |
| openstackgerrit | Dirk Mueller proposed openstack/diskimage-builder master: opensuse-minimal: install glibc-locale https://review.openstack.org/473959 | 06:57 |
| *** jbadiapa has joined #openstack-infra | 06:58 | |
| *** ramishra has quit IRC | 06:58 | |
| *** wolverineav has joined #openstack-infra | 07:00 | |
| *** ramishra has joined #openstack-infra | 07:00 | |
| *** gildub has quit IRC | 07:01 | |
| *** liujiong has quit IRC | 07:01 | |
| *** jtomasek has joined #openstack-infra | 07:04 | |
| *** slaweq has quit IRC | 07:05 | |
| *** Julien-zte has joined #openstack-infra | 07:05 | |
| *** martinkopec has joined #openstack-infra | 07:05 | |
| *** shardy has joined #openstack-infra | 07:06 | |
| *** wolverineav has quit IRC | 07:06 | |
| *** sshnaidm is now known as sshnaidm|off | 07:07 | |
| *** mriedem has quit IRC | 07:10 | |
| *** jamesmcarthur has joined #openstack-infra | 07:11 | |
| *** martinkopec has quit IRC | 07:15 | |
| *** martinkopec has joined #openstack-infra | 07:15 | |
| *** jamesmcarthur has quit IRC | 07:15 | |
| *** alexchadin has joined #openstack-infra | 07:16 | |
| *** tesseract has joined #openstack-infra | 07:17 | |
| *** jtomasek has quit IRC | 07:18 | |
| *** jpena|off is now known as jpena | 07:18 | |
| *** thorst has joined #openstack-infra | 07:21 | |
| *** thorst has quit IRC | 07:27 | |
| *** wolverineav has joined #openstack-infra | 07:27 | |
| *** wolverineav has quit IRC | 07:31 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul feature/zuulv3: Add status_pattern to zuul-web config https://review.openstack.org/481474 | 07:33 |
| *** iyamahat has joined #openstack-infra | 07:34 | |
| openstackgerrit | Alexander Chadin proposed openstack-infra/project-config master: Add functional job for watcherclient https://review.openstack.org/480495 | 07:38 |
| *** iyamahat has quit IRC | 07:40 | |
| *** jpich has joined #openstack-infra | 07:41 | |
| *** ccamacho has joined #openstack-infra | 07:42 | |
| *** ralonsoh has joined #openstack-infra | 07:48 | |
| *** markvoelker has joined #openstack-infra | 07:51 | |
| *** pratik_ has quit IRC | 07:55 | |
| *** jtomasek has joined #openstack-infra | 07:55 | |
| *** efoley has joined #openstack-infra | 07:59 | |
| *** zzzeek_ has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-infra | 08:00 | |
| *** bhavik1 has quit IRC | 08:02 | |
| *** Julien-zte has quit IRC | 08:05 | |
| *** Julien-zte has joined #openstack-infra | 08:05 | |
| *** jascott1 has quit IRC | 08:06 | |
| *** jascott1 has joined #openstack-infra | 08:08 | |
| *** sflanigan has quit IRC | 08:10 | |
| *** funzo_ has joined #openstack-infra | 08:11 | |
| *** funzo has quit IRC | 08:13 | |
| *** e0ne has joined #openstack-infra | 08:14 | |
| AJaeger | odyssey4me: could you review https://review.openstack.org/#/c/478095 , please? | 08:18 |
| *** david-lyle has quit IRC | 08:19 | |
| *** david-lyle has joined #openstack-infra | 08:19 | |
| *** jtomasek has quit IRC | 08:25 | |
| *** markvoelker has quit IRC | 08:26 | |
| openstackgerrit | Carlos Camacho proposed openstack-infra/tripleo-ci master: Enable the HA deployment by default https://review.openstack.org/415179 | 08:28 |
| *** jascott1 has quit IRC | 08:31 | |
| *** jascott1 has joined #openstack-infra | 08:32 | |
| *** lucas-afk is now known as lucasagomes | 08:33 | |
| *** rossella_s has quit IRC | 08:34 | |
| *** s-shiono has quit IRC | 08:35 | |
| *** jascott1 has quit IRC | 08:36 | |
| *** rossella_s has joined #openstack-infra | 08:37 | |
| *** tsuzuki has quit IRC | 08:42 | |
| odyssey4me | AJaeger done | 08:50 |
| *** priteau has joined #openstack-infra | 08:51 | |
| odyssey4me | I don't think it does any harm, but we'll need pabelanger's final say on that one. | 08:51 |
| AJaeger | odyssey4me: ok | 08:52 |
| *** mandre_away is now known as mandre | 08:56 | |
| *** electrofelix has joined #openstack-infra | 08:57 | |
| *** hashar has joined #openstack-infra | 08:59 | |
| AJaeger | yolanda_: could you put https://review.openstack.org/#/q/status:open+project:openstack-infra/project-config+branch:master+topic:mitaka-eol on your review queue for *later*, please? | 08:59 |
| yolanda_ | hi AJaeger , i'll review | 09:01 |
| AJaeger | thanks, yolanda_ ! | 09:03 |
| *** faizy has quit IRC | 09:04 | |
| *** psachin_ has joined #openstack-infra | 09:04 | |
| *** psachin_ has quit IRC | 09:05 | |
| *** faizy has joined #openstack-infra | 09:05 | |
| openstackgerrit | Merged openstack-infra/project-config master: puppet: remove workaround for mitaka/hammer branches https://review.openstack.org/481149 | 09:09 |
| *** tosky has joined #openstack-infra | 09:12 | |
| openstackgerrit | Merged openstack-infra/project-config master: Enable OSC plugin gates for python-karborclient https://review.openstack.org/480881 | 09:15 |
| openstackgerrit | Merged openstack-infra/project-config master: mitaka-eol: Handle astara https://review.openstack.org/481397 | 09:16 |
| openstackgerrit | Merged openstack-infra/project-config master: Remove obsolete python34-jobs https://review.openstack.org/481389 | 09:16 |
| openstackgerrit | Merged openstack-infra/project-config master: Skip integration tests/non-doc jobs for certain patrole changes https://review.openstack.org/478373 | 09:16 |
| openstackgerrit | Merged openstack-infra/project-config master: Add fuel-plugin-emailnotifyd https://review.openstack.org/479830 | 09:16 |
| openstackgerrit | Merged openstack-infra/project-config master: Add fuel-plugin-sensu https://review.openstack.org/480238 | 09:16 |
| openstackgerrit | Merged openstack-infra/project-config master: Add fuel-plugin-ceph-multibackend https://review.openstack.org/477869 | 09:17 |
| openstackgerrit | Merged openstack-infra/project-config master: Add fuel-plugin-watcher https://review.openstack.org/479834 | 09:18 |
| openstackgerrit | Merged openstack-infra/project-config master: Add fuel-plugin-dns-updater https://review.openstack.org/479838 | 09:18 |
| openstackgerrit | Merged openstack-infra/project-config master: Add fuel-plugin-tempest https://review.openstack.org/479835 | 09:19 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Fix tripleo skip conditions https://review.openstack.org/481403 | 09:20 |
| openstackgerrit | Dirk Mueller proposed openstack/diskimage-builder master: opensuse-minimal: install glibc-locale https://review.openstack.org/473959 | 09:20 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Create python-jobs-trusty job group https://review.openstack.org/481390 | 09:21 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Create python-jobs-trusty zuul template https://review.openstack.org/481425 | 09:21 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Move all trusty jobs to python-jobs-trusty job group https://review.openstack.org/481430 | 09:21 |
| *** neiljerram has joined #openstack-infra | 09:22 | |
| openstackgerrit | Merged openstack-infra/project-config master: Add fuel-plugin-detach-horizon https://review.openstack.org/477878 | 09:22 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack-infra/project-config master: Normalize projects.yaml https://review.openstack.org/481540 | 09:22 |
| *** markvoelker has joined #openstack-infra | 09:23 | |
| *** e0ne has quit IRC | 09:23 | |
| *** thorst has joined #openstack-infra | 09:23 | |
| openstackgerrit | Merged openstack-infra/project-config master: Add os-service-types project https://review.openstack.org/475080 | 09:24 |
| openstackgerrit | Merged openstack-infra/project-config master: Use xenial for check-osc-plugin https://review.openstack.org/481171 | 09:25 |
| *** thorst has quit IRC | 09:28 | |
| *** e0ne has joined #openstack-infra | 09:29 | |
| openstackgerrit | Alfredo Moralejo proposed openstack-infra/tripleo-ci master: DNM - test https://review.openstack.org/#/c/481535/ with consistent https://review.openstack.org/481546 | 09:29 |
| *** rossella_s has quit IRC | 09:29 | |
| openstackgerrit | Merged openstack-infra/project-config master: Remove tempest-dsvm-ironic-pxe_ssh-{node} job https://review.openstack.org/480619 | 09:30 |
| openstackgerrit | Merged openstack-infra/project-config master: Cleanup ironic jobs https://review.openstack.org/480620 | 09:31 |
| openstackgerrit | Merged openstack-infra/project-config master: Remove *_ssh drivers from all ironic jobs https://review.openstack.org/480621 | 09:31 |
| openstackgerrit | Merged openstack-infra/project-config master: mitaka-eol: Remove unused charm trusty jobs https://review.openstack.org/481166 | 09:31 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Fix tripleo skip conditions https://review.openstack.org/481403 | 09:32 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Create python-jobs-trusty job group https://review.openstack.org/481390 | 09:32 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Create python-jobs-trusty zuul template https://review.openstack.org/481425 | 09:33 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Move all trusty jobs to python-jobs-trusty job group https://review.openstack.org/481430 | 09:33 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: mitaka-eol: remove trusty-job from fuel-ccp-component https://review.openstack.org/481434 | 09:33 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: mitaka-eol: Remove trusty linters job https://review.openstack.org/481435 | 09:33 |
| *** MatthewLi has quit IRC | 09:37 | |
| *** gildub has joined #openstack-infra | 09:41 | |
| *** yamahata has joined #openstack-infra | 09:41 | |
| *** alexchadin has quit IRC | 09:42 | |
| *** rossella_s has joined #openstack-infra | 09:43 | |
| openstackgerrit | Merged openstack-infra/project-config master: Normalize projects.yaml https://review.openstack.org/481540 | 09:43 |
| *** arif-ali has quit IRC | 09:44 | |
| *** sulo has quit IRC | 09:47 | |
| *** sulo has joined #openstack-infra | 09:47 | |
| *** openstackgerrit has quit IRC | 09:48 | |
| *** e0ne has quit IRC | 09:48 | |
| *** ZZelle has joined #openstack-infra | 09:51 | |
| *** sulo has quit IRC | 09:52 | |
| *** slaweq has joined #openstack-infra | 09:53 | |
| *** sulo has joined #openstack-infra | 09:55 | |
| *** markvoelker has quit IRC | 09:56 | |
| *** cuongnv has quit IRC | 09:59 | |
| *** sambetts|afk is now known as sambetts | 10:06 | |
| *** jamesmcarthur has joined #openstack-infra | 10:10 | |
| *** openstackgerrit has joined #openstack-infra | 10:13 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: mitaka-eol: remove trusty-job from fuel-ccp-component https://review.openstack.org/481434 | 10:13 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: mitaka-eol: Remove trusty linters job https://review.openstack.org/481435 | 10:13 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Move all trusty jobs to python-jobs-trusty job group https://review.openstack.org/481430 | 10:13 |
| *** jamesmcarthur has quit IRC | 10:15 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Add functional job for watcherclient https://review.openstack.org/480495 | 10:16 |
| *** stakeda has quit IRC | 10:16 | |
| *** slaweq has quit IRC | 10:20 | |
| *** slaweq has joined #openstack-infra | 10:21 | |
| *** MatthewLi has joined #openstack-infra | 10:24 | |
| *** slaweq has quit IRC | 10:26 | |
| *** yamahata has quit IRC | 10:26 | |
| *** MatthewLi has quit IRC | 10:29 | |
| *** LindaWang has quit IRC | 10:32 | |
| *** rossella_ has joined #openstack-infra | 10:32 | |
| *** rossella_s has quit IRC | 10:35 | |
| *** alexchadin has joined #openstack-infra | 10:35 | |
| *** hashar is now known as hasharLunch | 10:37 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: mitaka-eol: remove trusty-job from fuel-ccp-component https://review.openstack.org/481434 | 10:38 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: mitaka-eol: Remove trusty linters job https://review.openstack.org/481435 | 10:38 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Create python-jobs-trusty zuul template https://review.openstack.org/481425 | 10:38 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Move all trusty jobs to python-jobs-trusty job group https://review.openstack.org/481430 | 10:38 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: mitaka-eol: Simplify zuul branch conditions https://review.openstack.org/481567 | 10:38 |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack-infra/project-config master: Do some openstack-ansible job housekeeping https://review.openstack.org/481568 | 10:39 |
| *** alexchadin has quit IRC | 10:42 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack-infra/project-config master: Do some openstack-ansible job housekeeping https://review.openstack.org/481568 | 10:45 |
| *** katkapilatova has joined #openstack-infra | 10:45 | |
| *** yamamoto has quit IRC | 10:46 | |
| *** zhurong has quit IRC | 10:46 | |
| *** slaweq has joined #openstack-infra | 10:49 | |
| *** slaweq has quit IRC | 10:50 | |
| *** slaweq has joined #openstack-infra | 10:50 | |
| AJaeger | Yeah, my mitaka-eol stack passes. And it moves us down from 17170 jobs defined to 14239. Which also gives smaller log files (18 MB -> 15 MB)! | 10:53 |
| *** markvoelker has joined #openstack-infra | 10:53 | |
| AJaeger | project-config cores, please review https://review.openstack.org/#/q/status:open+project:openstack-infra/project-config+branch:master+topic:mitaka-eol | 10:53 |
| AJaeger | odyssey4me: regarding https://review.openstack.org/#/c/481435/4/jenkins/jobs/projects.yaml - the change is needed, otherwise the tests do not pass | 10:56 |
| odyssey4me | AJaeger odd, that must mean that repo has not had the eol tag applied | 10:57 |
| AJaeger | odyssey4me: let me give a reference in review.o. | 10:57 |
| odyssey4me | hmm, odd - it has no mitaka branch | 10:57 |
| AJaeger | odyssey4me: then let's merge the change as is - and I'll add one more change on the stack to remove the trusty jobs. | 10:58 |
| odyssey4me | AJaeger sure, no worries | 10:59 |
| AJaeger | odyssey4me: or wait: Don't you run trusty jobs on *newton* as well - and that repo has stable/newton | 10:59 |
| odyssey4me | yes, we absolutely do - but not the linters jobs | 10:59 |
| odyssey4me | only our functional tests use trusty for newton | 10:59 |
| AJaeger | Ah! Then I'll do the followup -and you can review both and +1 both, ok? | 10:59 |
| odyssey4me | sure, no worries | 10:59 |
| *** thorst has joined #openstack-infra | 11:00 | |
| AJaeger | odyssey4me: so, we can remove gate-{name}-linters-ubuntu-trusty and gate-{name}-docs-ubuntu-trusty from ansible-role-jobs completely, can't we? | 11:00 |
| odyssey4me | yes, unless pabelanger needs them for whatever reason | 11:01 |
| AJaeger | windmill has no branches, so he won't need them. Cool, I simplify some more | 11:02 |
| odyssey4me | great :) | 11:02 |
| *** rossella_ has quit IRC | 11:03 | |
| *** aviau has quit IRC | 11:04 | |
| *** aviau has joined #openstack-infra | 11:04 | |
| *** thorst has quit IRC | 11:05 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: mitaka-eol: Remove trusty docs, linters for ansible https://review.openstack.org/481580 | 11:08 |
| AJaeger | odyssey4me: here it is ^ | 11:08 |
| AJaeger | please review both changes | 11:08 |
| odyssey4me | awesome, done - thanks AJaeger | 11:11 |
| *** ym200 has quit IRC | 11:14 | |
| *** ym200 has joined #openstack-infra | 11:15 | |
| *** rossella_ has joined #openstack-infra | 11:19 | |
| *** ym200 has quit IRC | 11:19 | |
| *** ym200 has joined #openstack-infra | 11:20 | |
| *** katkapilatova has left #openstack-infra | 11:21 | |
| *** ym200 has quit IRC | 11:24 | |
| *** markvoelker has quit IRC | 11:26 | |
| *** wolverineav has joined #openstack-infra | 11:26 | |
| *** lucasagomes is now known as lucas-hungry | 11:27 | |
| *** wfoster has quit IRC | 11:28 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Make openstack-docs-build job https://review.openstack.org/481586 | 11:28 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Remove the old tox-linter job https://review.openstack.org/481322 | 11:29 |
| openstackgerrit | Monty Taylor proposed openstack-infra/openstack-zuul-jobs master: Make openstack-docs-build job https://review.openstack.org/481586 | 11:30 |
| *** yamamoto has joined #openstack-infra | 11:30 | |
| *** wfoster has joined #openstack-infra | 11:31 | |
| *** ccamacho is now known as ccamacho|lunch | 11:34 | |
| AJaeger | thanks, odyssey4me for the review. Glad to be able to get this in ;) | 11:35 |
| AJaeger | Now only 14 MB of log file! | 11:35 |
| *** psachin has quit IRC | 11:36 | |
| AJaeger | and 14090 jobs defined | 11:36 |
| *** sdague has joined #openstack-infra | 11:36 | |
| AJaeger | mordred: if you have some time for mitaka-eol reviews today, please look at https://review.openstack.org/#/q/status:open+project:openstack-infra/project-config+branch:master+topic:mitaka-eol | 11:37 |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Remove old tox jobs https://review.openstack.org/481325 | 11:38 |
| *** winggundamth has quit IRC | 11:39 | |
| openstackgerrit | Honza Pokorny proposed openstack-infra/project-config master: Create tempest-tripleo project https://review.openstack.org/481158 | 11:41 |
| *** ldnunes has joined #openstack-infra | 11:43 | |
| *** markvoelker has joined #openstack-infra | 11:44 | |
| *** slaweq has quit IRC | 11:48 | |
| *** pcaruana has joined #openstack-infra | 11:49 | |
| *** Julien-zte has quit IRC | 11:50 | |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Improve debugging at start of job https://review.openstack.org/480761 | 11:51 |
| *** LindaWang has joined #openstack-infra | 11:52 | |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: github: gracefully handle unknown event https://review.openstack.org/473249 | 11:53 |
| *** rhallisey has joined #openstack-infra | 11:53 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul feature/zuulv3: Add html based websocket client for console stream https://review.openstack.org/481599 | 11:54 |
| *** jsavak has joined #openstack-infra | 11:55 | |
| *** thorst has joined #openstack-infra | 11:56 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Remove unused centos7 chef multinode job https://review.openstack.org/481602 | 12:00 |
| *** jpena is now known as jpena|lunch | 12:01 | |
| *** jsavak has quit IRC | 12:03 | |
| *** jsavak has joined #openstack-infra | 12:03 | |
| *** rwsu has quit IRC | 12:05 | |
| *** pgadiya has quit IRC | 12:08 | |
| *** mat128 has joined #openstack-infra | 12:09 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Remove unused python-jobs https://review.openstack.org/481607 | 12:11 |
| *** rwsu has joined #openstack-infra | 12:12 | |
| openstackgerrit | Merged openstack-infra/project-config master: mitaka-eol: Handle networking-odl https://review.openstack.org/481399 | 12:13 |
| *** ccamacho|lunch is now known as ccamacho | 12:14 | |
| *** jamesdenton has joined #openstack-infra | 12:14 | |
| *** shardy has quit IRC | 12:18 | |
| *** arif-ali has joined #openstack-infra | 12:20 | |
| *** gildub has quit IRC | 12:22 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: Remove unused python-jobs https://review.openstack.org/481607 | 12:24 |
| *** lucas-hungry is now known as lucasagomes | 12:25 | |
| *** faizy has quit IRC | 12:28 | |
| *** rlandy has joined #openstack-infra | 12:30 | |
| *** e0ne has joined #openstack-infra | 12:35 | |
| *** ijw has joined #openstack-infra | 12:36 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul feature/zuulv3: Add some indexes to allow efficient dashboard sql https://review.openstack.org/481614 | 12:37 |
| *** trown|outtypewww is now known as trown | 12:38 | |
| *** gordc has joined #openstack-infra | 12:39 | |
| *** hasharLunch is now known as hashar | 12:39 | |
| *** ijw has quit IRC | 12:41 | |
| *** dprince has joined #openstack-infra | 12:42 | |
| *** chlong_ has quit IRC | 12:43 | |
| *** kgiusti has joined #openstack-infra | 12:44 | |
| openstackgerrit | Carlos Camacho proposed openstack-infra/tripleo-ci master: Enable the HA deployment by default https://review.openstack.org/415179 | 12:44 |
| stephenfin | mordred, dhellmann: You guys around? | 12:47 |
| * dhellmann checks | 12:47 | |
| dhellmann | yes | 12:47 |
| stephenfin | :) I've been working on https://bugs.launchpad.net/pbr/+bug/1702872 | 12:47 |
| openstack | Launchpad bug 1702872 in PBR "pbr + Sphinx 1.6+ ignores the '[sphinx_build] builders' setuptools option" [Undecided,New] | 12:47 |
| dhellmann | oh, boy | 12:48 |
| stephenfin | It seems we made a mistake in commit c024066 | 12:48 |
| *** slaweq has joined #openstack-infra | 12:48 | |
| stephenfin | We provided an overriden version of an internal function here https://github.com/openstack-dev/pbr/blob/master/pbr/builddoc.py#L123 | 12:49 |
| dhellmann | was that renamed? | 12:49 |
| stephenfin | However, with that commit, we no longer call that function https://github.com/openstack-dev/pbr/blob/master/pbr/builddoc.py#L195 vs https://github.com/openstack-dev/pbr/blob/master/pbr/builddoc.py#L200 | 12:49 |
| dhellmann | oh | 12:50 |
| stephenfin | So I see two paths forward | 12:50 |
| *** jaypipes is now known as leakypipes | 12:50 | |
| dhellmann | I see some logic in there related to the newer sphinx | 12:50 |
| dhellmann | is that what's triggering this? | 12:50 |
| dhellmann | because line 197 on https://review.openstack.org/#/c/465489/2/pbr/builddoc.py does show a call to _sphinx_run() | 12:51 |
| stephenfin | Yeah, but we early return on line 192 | 12:51 |
| dhellmann | if the OP thought the underlying class dealt with multiple builders, did the option name change? | 12:51 |
| stephenfin | so for Sphinx 1.6+, we'll never call that | 12:52 |
| dhellmann | right, so the fact that we're now requiring sphinx 1.6.2 for our projects triggers the failure | 12:52 |
| stephenfin | exactly | 12:52 |
| dhellmann | so, why did jd_ think the base class handled multiple builders? does it do that some other way (different option name, specifying them on the command line, ?) | 12:53 |
| stephenfin | Sphinx does handle multiple builders now | 12:53 |
| stephenfin | I added that feature | 12:53 |
| dhellmann | oh, it wasn't jd, it was you :-) | 12:53 |
| stephenfin | Yup :) https://github.com/sphinx-doc/sphinx/commit/2afa0b6627f7e5afb188d5a60c8c4767f6250774 | 12:53 |
| stephenfin | Basically we don't need 'builders' anymore | 12:54 |
| *** slaweq has quit IRC | 12:54 | |
| dhellmann | ok | 12:55 |
| dhellmann | how do we specify that we want both html and man pages? | 12:55 |
| stephenfin | builder | 12:55 |
| stephenfin | (vs. builders) | 12:55 |
| dhellmann | ok, the option name needs to be changed in the setup.cfg files? | 12:55 |
| stephenfin | Yup, but that's not all | 12:55 |
| stephenfin | I've fixed that issue with this commit, which should be backportable | 12:55 |
| stephenfin | Wait - I didn't push it yet :) | 12:56 |
| openstackgerrit | Stephen Finucane proposed openstack-dev/pbr master: builddoc: Use '[sphinx_build] builders' with Sphinx < 1.6 https://review.openstack.org/481620 | 12:56 |
| openstackgerrit | Stephen Finucane proposed openstack-dev/pbr master: Deprecate support for Sphinx < 1.6 https://review.openstack.org/481621 | 12:56 |
| stephenfin | Those changes ^ | 12:56 |
| stephenfin | dhellmann: However, that's not the whole issue | 12:57 |
| dhellmann | ok | 12:57 |
| stephenfin | Because we also need to provide our own confoverrides | 12:57 |
| stephenfin | https://github.com/openstack-dev/pbr/blob/master/pbr/builddoc.py#L138-L142 | 12:57 |
| dhellmann | I don't know what that is | 12:57 |
| dhellmann | can suppress_warnings be set in setup.cfg, too? | 12:58 |
| dhellmann | oh, we pass all sorts of things, don't we | 12:58 |
| dhellmann | joy | 12:58 |
| stephenfin | Yup | 12:58 |
| dhellmann | I wonder if we can do some of this in the openstackdocstheme | 12:58 |
| stephenfin | I can't find the commit right now, but if you don't provide those confoverrides then the build fails | 12:59 |
| dhellmann | or just deprecate running sphinx this way and invoke it from the command line directly | 12:59 |
| dhellmann | ah, I bet that's what is causing the issue with projects using sphinxcontrib.pecanwsme | 13:00 |
| stephenfin | Yeah, I'd like to deprecate that custom builddoc implementation altogether | 13:00 |
| dhellmann | mistral ran into that | 13:00 |
| stephenfin | That sounds about right, yeah | 13:00 |
| stephenfin | However, I'm not aware of a good migration strategy | 13:00 |
| dhellmann | why don't you start a ML thread about deprecating the sphinx integration | 13:00 |
| dhellmann | we would need to change the doc build job to run sphinx directly, or to use "tox -e docs" | 13:01 |
| stephenfin | I will, but I was hoping to have some solutions before going that way | 13:01 |
| dhellmann | we've said recently that would be ok | 13:01 |
| stephenfin | Nah, there's no issue with that aspect of it | 13:01 |
| dhellmann | well, we can use the thread to figure out what we need to do | 13:01 |
| dhellmann | will sphinx read setup.cfg for settings on its own? or do we need to move all of those settings into the tox cli? | 13:02 |
| dhellmann | or do we need a different wrapper for sphinx to do that? | 13:02 |
| stephenfin | What that custom implementation does is provide some helpful defaults (setting the version using distribute, for example) and, in the past, some things that Sphinx didn't support natively (multiple builders) | 13:02 |
| dhellmann | we might be able to have the theme set the version string | 13:02 |
| stephenfin | Some of those are less helpful now (multiple builders, warnerrors), while others could be handled by themes and the likes | 13:03 |
| stephenfin | Yeah, exactly | 13:03 |
| stephenfin | But at a minimum, everyone needs to s/builders/builder/ | 13:03 |
| AJaeger | dhellmann: what about non-official repos that don't use the openstackdocstheme? Do we have to do something for them as well? | 13:03 |
| dhellmann | that sounds like a mechanical change | 13:03 |
| *** gcb has quit IRC | 13:03 | |
| stephenfin | AJaeger: I'd recommend using a Sphinx plugin for that | 13:03 |
| dhellmann | maybe we write a new sphinx plugin that sets the version info? | 13:03 |
| dhellmann | maybe one already exists? | 13:04 |
| dhellmann | https://pypi.python.org/pypi/sphinxcontrib-versioning | 13:04 |
| stephenfin | I'm pretty sure one does. I'm looking now | 13:04 |
| *** xarses_ has quit IRC | 13:05 | |
| stephenfin | I think that plugin allows you to switch between different releases using a tooltip, like on readthedocs | 13:05 |
| dhellmann | ah, yeah | 13:06 |
| *** links has quit IRC | 13:06 | |
| dhellmann | ok, well, a separate tool that does this one thing seems like the right way to go | 13:06 |
| *** slaweq has joined #openstack-infra | 13:07 | |
| dhellmann | mordred : when you're around you'll want to read the scrollback ^^ | 13:07 |
| stephenfin | OK, I'll try to find one if I can. If not, we'll do one ourselves | 13:08 |
| stephenfin | Will send a mail to the list too. openstack-dev? | 13:08 |
| mordred | dhellmann: will I realy? | 13:08 |
| *** jpena|lunch is now known as jpena | 13:08 | |
| dhellmann | stephenfin : yeah, please do start a ML thread | 13:08 |
| dhellmann | mordred : the last ~20 minutes or so | 13:09 |
| mordred | ok - so - what's the right way to "run sphinx directly" ? | 13:10 |
| dhellmann | mordred : sphinx-build ... | 13:10 |
| dhellmann | or "tox -e venv -- sphinx-build ..." | 13:10 |
| dhellmann | or even just "tox -e docs" | 13:10 |
| stephenfin | 'python setup.py build_sphinx' is also fine | 13:11 |
| mordred | k. so - yeah - I think the _easiest_ step forward would be to change the job to run tox -e venv -- sphinx-build ... | 13:11 |
| dhellmann | only after we actually remove the pbr overrides | 13:11 |
| stephenfin | exactly | 13:11 |
| *** slaweq has quit IRC | 13:11 | |
| stephenfin | *true | 13:11 |
| mordred | because our projects are not guaranteed to _have_ a docs env | 13:11 |
| mordred | since that's not part of the PTI | 13:11 |
| dhellmann | mordred : right, but before we do that we need to deal with the overrides in some other ways | 13:11 |
| mordred | gotcha | 13:11 |
| dhellmann | sure. we discussed changing that when we talked about how golang projects would build docs | 13:11 |
| dhellmann | so step 1 is to come up with whatever extensions replace the pbr magic | 13:12 |
| stephenfin | Switching to 'sphinx-build' won't resolve all the issues though | 13:12 |
| stephenfin | because we'll lose all the overrides anyway | 13:12 |
| dhellmann | step 2 is to decide on "tox -e docs" or "tox -e venv -- ..." and roll that change out | 13:12 |
| *** dhajare has quit IRC | 13:12 | |
| stephenfin | so maybe hold off til we get feedback on the ML, because we might be able to dump that version and bump pbr pretty quickly | 13:13 |
| dhellmann | step 3 is to drop the integration in pbr | 13:13 |
| mordred | I agree with these steps | 13:13 |
| dhellmann | stephenfin : I'm not sure what you mean by "dump that version" | 13:14 |
| stephenfin | Sorry - I meant dump that customized version of the 'build_sphinx' command | 13:14 |
| mordred | stephenfin: we have to figure out the overrides to do that - we have information pbr injects that is not in conf.py that I _believe_ sphinx will be unhappy without | 13:15 |
| dhellmann | yeah, if we work out how to do those things in sphinx plugins instead of through pbr we should be able to avoid writing a new wrapper for build_sphinx | 13:15 |
| mordred | ++ | 13:15 |
| *** bobh has joined #openstack-infra | 13:15 | |
| stephenfin | If we did, it would just be a series of s/builders/builder/ changes to projects, and some work in openstackdocthemes to set project name and version | 13:16 |
| *** slaweq has joined #openstack-infra | 13:16 | |
| stephenfin | mordred: There's three things I see it doing | 13:16 |
| mordred | dhellmann: while we're considering this problem broadly - I'd like to suggest we consider introducing a new file as well - docs-requirements.txt - so that we could also drop a docs-requirements.txt into go projects and it would be posible to know, regardless of use of tox or not, what needs to be installed to successfully run sphinx for a given project | 13:16 |
| dhellmann | stephenfin : does sphinx read setup.cfg if you run it as sphinx-build? | 13:17 |
| mordred | which would also allow us to drop sphinx from our normal test-requirements across our python projects - which means not installing doc utils for unittests :) | 13:17 |
| AJaeger | dhellmann: you wonder about warning-is-error on where to set it? | 13:17 |
| dhellmann | mordred : I like the idea. I wonder if we want to use extras in setup.cfg instead of a separate file? | 13:17 |
| dhellmann | "pip install .[docs]" | 13:17 |
| stephenfin | (a) setting version and name from pkg_resources, (b) automatically running 'autodoc' and a custom (?) script that does something similar, and (c) allowing multiple builders for Sphinx < 1.6 | 13:18 |
| mordred | dhellmann: well - go projects don't have setup.cfg files | 13:18 |
| dhellmann | boo | 13:18 |
| dhellmann | ok | 13:18 |
| *** markmcd has quit IRC | 13:18 | |
| mordred | dhellmann: BUT - we could make pbr grok the file and add a docs extras | 13:18 |
| mordred | for python projects | 13:18 |
| dhellmann | we could also just list the requirements in tox.ini | 13:18 |
| mordred | so that pip install .[docs] _does_ work | 13:18 |
| stephenfin | We can set version with openstackdocstheme, simply add 'sphinx.ext.autodoc' to extensions for projects that use it, and forget about Sphinx < 1.6 | 13:18 |
| dhellmann | although I guess having a separate file helps with tooling | 13:18 |
| *** markmcd has joined #openstack-infra | 13:19 | |
| stephenfin | dhellmann: No, it doesn't. Only 'build_docs' reads setup.cfg | 13:19 |
| mordred | dhellmann: yah - we can sync it with g-r better in a separate file - and it allows us to write tooling to detect the file and install in a docs job even if a project doesn't use tox | 13:19 |
| mordred | dhellmann: I'll write up a proposal for that | 13:19 |
| stephenfin | You need to pass those options as flags if you want it | 13:19 |
| dhellmann | mordred : we're going to stop syncing requirements soon | 13:19 |
| openstackgerrit | zhurong proposed openstack-infra/project-config master: Add python-pankoclient to $PROJECTS variable in dsvm-integration-panko https://review.openstack.org/481628 | 13:19 |
| mordred | dhellmann: nod. I still think it'll give us the best ability to detect in automation | 13:19 |
| dhellmann | stephenfin : that's what I thought. | 13:19 |
| dhellmann | mordred : ok, I can go along with that | 13:20 |
| mordred | dhellmann: (I'm working on zuulv3 versions of hte docs build jobs right now - so this is very present in my brain :) ) | 13:20 |
| openstackgerrit | Carlos Camacho proposed openstack-infra/tripleo-ci master: Enable the HA deployment by default https://review.openstack.org/415179 | 13:20 |
| *** rossella_ has quit IRC | 13:20 | |
| stephenfin | mordred: So what's the tl;dr: version of what you're going to do? | 13:20 |
| mordred | dhellmann: for v3 - we're trying to have a set of jobs that extra generic that can be used by non-openstack folks too - do a "build-sphinx" job that can figure out how to do sphinx with or without tox would be a nice job to have | 13:20 |
| stephenfin | and do I still need to send a question to the mailing list about dropping this feature from pbr? | 13:21 |
| mordred | stephenfin: oh - right now just going to write up the suggestion about docs-requirements.txt - it's orthogonal | 13:21 |
| stephenfin | ah, gotcha | 13:21 |
| mordred | stephenfin: no - please continue with your drop-from-pbr work - I think that's essential | 13:21 |
| stephenfin | (y) | 13:21 |
| dhellmann | stephenfin : yes, it would be good to have all of these things written down in one place and to get feedback on the plan forward | 13:21 |
| mordred | just - if we're going to wind up updating the PTI and doing a big patch bomb - we might as well make it count | 13:21 |
| dhellmann | yeah, I'd be happy to have the doc-requirements.txt thing rolled into this | 13:22 |
| mordred | oh - so ... here's an idea ... | 13:22 |
| dhellmann | that broadens the scope, so we'll need some of the requirements team to get involved in that | 13:22 |
| mordred | nope. nevermind. ignore me | 13:22 |
| AJaeger | so, cross-openstack spec? Infra-spec? Or just a plain doc? | 13:22 |
| mordred | oh! | 13:23 |
| mordred | stephenfin: WHAT IF ... | 13:23 |
| * AJaeger ignores mordred ;) | 13:23 | |
| mordred | stephenfin: the pbr build_sphinx command just subprocess invoked sphinx-build | 13:23 |
| dhellmann | AJaeger : good question | 13:23 |
| mordred | stephenfin: as a bridge to us invoking sphinx-build | 13:23 |
| *** zhurong has joined #openstack-infra | 13:23 | |
| mordred | stephenfin: can we pass the override things we need on the command line? | 13:23 |
| dhellmann | can we pass values like version through the command line? | 13:23 |
| stephenfin | mordred: But that's exactly what it does now? | 13:24 |
| dhellmann | jinx | 13:24 |
| dhellmann | it uses subprocess? | 13:24 |
| dhellmann | I thought it imported it | 13:24 |
| stephenfin | Oh, no, not subprocess | 13:24 |
| mordred | I thought it subclassed the sphinx setuptools integration | 13:24 |
| stephenfin | Missed that distinction | 13:24 |
| mordred | yah- if we can pass our overrides as command line arguments | 13:24 |
| stephenfin | OK, but we can't call it 'build_sphinx' | 13:24 |
| mordred | then it'll give us a way forward in short-order from pbr - and then we can construct a plan to get pbr out of the game | 13:24 |
| *** alexchadin has joined #openstack-infra | 13:24 | |
| stephenfin | Or can we... | 13:25 |
| dhellmann | I guess a transition like that would let us fix this bug in the short term, and do the doc build changes over a longer period. | 13:25 |
| mordred | stephenfin: sure - let's make a new command that does the rightthing ... release pbr - all of openstack will have it - we can update the job definition easily to call the new command | 13:25 |
| dhellmann | stephenfin : if we rewrite the implementation within pbr, we can use the same command | 13:25 |
| dhellmann | wait, why do we need a new command? | 13:25 |
| mordred | not sure - but I think we can do a new command if we need to | 13:26 |
| mordred | withouth too much pain | 13:26 |
| dhellmann | ok | 13:26 |
| dhellmann | I guess that does save us having to ensure all projects have a docs tox setup | 13:26 |
| *** eharney has joined #openstack-infra | 13:26 | |
| mordred | yah | 13:26 |
| stephenfin | I think we need to take a step back because I'm a little confused | 13:27 |
| stephenfin | We want to remove pbr's version of 'build_sphinx' | 13:27 |
| mordred | kk. yes. let's start from the beginning | 13:27 |
| mordred | yes | 13:27 |
| stephenfin | ...because it's painful to maintain, and hacks internal functions of the Sphinx version that change somewhat regularly | 13:27 |
| mordred | yes | 13:27 |
| stephenfin | But to do that, we probably need to provide a migration path for users that rely on some of the value add that pbr's variant provides | 13:28 |
| mordred | yes | 13:28 |
| *** esberglu has joined #openstack-infra | 13:28 | |
| mordred | most likely through one or more sphinx extension modules | 13:28 |
| stephenfin | namely, auto-naming/versioning, automatic running of autodoc, and the other thing | 13:28 |
| mordred | yes | 13:28 |
| mordred | what's the other thing? | 13:28 |
| *** bobh has quit IRC | 13:28 | |
| stephenfin | I'm trying to remember - sec :) | 13:28 |
| dhellmann | we'll want to list exactly how we're going to replace each of the existing features of the pbr integration so we make sure we cover them all | 13:28 |
| mordred | ++ | 13:29 |
| stephenfin | Oh, multiple builders for Sphinx < 1.6. I think we can ignore that one | 13:29 |
| dhellmann | yeah | 13:29 |
| mordred | the replacement for that is "if a project wants multiple builders, list it in sphinx's config" | 13:29 |
| stephenfin | So my question is what's the migration strategy. I see two options | 13:29 |
| mordred | so - let's list it as a thing - and it has an easy solution | 13:29 |
| stephenfin | (a) hard break. "You need to add these things to 'doc/source/conf.py' to retain your existing functionality" | 13:30 |
| *** LindaWang has quit IRC | 13:31 | |
| stephenfin | (b) soft break. Merge in changes from upstream Sphinx's '_sphinx_run' function with pbr's confoverrides and calls to 'autodoc', and stop calling 'setup_command.BuildDoc.run(self)' | 13:31 |
| stephenfin | but add warnings that this feature is going away, and you need to do steps in (a) asap | 13:32 |
| *** Dinesh_Bhor has quit IRC | 13:32 | |
| *** slaweq_ has joined #openstack-infra | 13:32 | |
| *** apetrich_ has joined #openstack-infra | 13:32 | |
| mordred | well - before we go down those roads ... | 13:32 |
| mordred | I think we need to answer the quesiton "can we implement confoverrides and apidoc-run in a sphinx extension | 13:33 |
| mordred | or in two sphinx extensions | 13:33 |
| *** rossella_ has joined #openstack-infra | 13:33 | |
| dhellmann | I am sure we can do the apidoc run one | 13:33 |
| *** apetrich_ has quit IRC | 13:33 | |
| dhellmann | using the builder-inited event | 13:33 |
| stephenfin | afaik, we can | 13:33 |
| dhellmann | I'm less sure of confoverrides | 13:33 |
| dhellmann | we probably can, but I'm not sure exactly how it would work | 13:34 |
| mordred | if we CAN - then we've got a third option - which is "add the sphinx extensions, have pbr detect their presence and not run its stuff if they are there" | 13:34 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul feature/zuulv3: Optionally bind /lib64 https://review.openstack.org/481432 | 13:34 |
| dhellmann | good point | 13:34 |
| stephenfin | Wait - what's apidoc-run? | 13:34 |
| stephenfin | do you mean autodoc? | 13:34 |
| mordred | stephenfin: running apidoc - which is the pbr added autodoc thing | 13:35 |
| stephenfin | mordred: Are you sure? | 13:35 |
| stephenfin | ahhh | 13:35 |
| mordred | stephenfin: yah autodoc_tree_index_modules | 13:35 |
| mordred | _sphinx_tree in pbr/builddoc.py | 13:36 |
| dhellmann | there seemed to be 2 implementations of that. I'm not clear how they are different. Doesn't sphinx have something native like that, though? | 13:36 |
| mordred | it does - let's see if anyone is using the old one | 13:36 |
| mordred | http://codesearch.openstack.org/?q=autodoc_tree_index_modules&i=nope&files=&repos= | 13:36 |
| mordred | yes | 13:36 |
| *** slaweq_ has quit IRC | 13:37 | |
| mordred | it's not many though - so we could go through and change them to use the sphinx provided thing | 13:37 |
| stephenfin | mordred: Which some folks are already doing, from the looks of things http://codesearch.openstack.org/?q=sphinx.ext.autodoc&i=nope&files=&repos= | 13:37 |
| dhellmann | mordred : I think I've been updating projects to not use tree_index but to use the other thing? I'd have to check. | 13:38 |
| dhellmann | but yeah, either way | 13:38 |
| stephenfin | OK, that sounds like a good plan | 13:38 |
| stephenfin | I'll try figure out what the two things do differently, and if/how native Sphinx can replace one or both | 13:39 |
| *** sdague has quit IRC | 13:39 | |
| mordred | yah - keystoneauth has both sphinx.ext.autodoc and autodoc_tree_index | 13:39 |
| stephenfin | then I'll fire off a mail to the ML with the general idea to remove that plugin in favour of Sphinx features/discrete extensions | 13:39 |
| mordred | yes | 13:39 |
| mordred | I'm looking at sphinx now to see if I can figure out how to do our confoverrides thing in an extension | 13:40 |
| mordred | maybe worst case we convince the sphinx folks at let us add an extension point to sphinx | 13:40 |
| stephenfin | what overrides are you looking for? | 13:41 |
| stephenfin | mordred: version and project name can (and probably should) be set in 'conf.py'. I know a couple of folks do this manually using pkg_resources | 13:42 |
| stephenfin | (probably because they didn't know pbr would do it for them) | 13:42 |
| fungi | if we hit on a magic combination, please update the templates in the cookiecutter repos as some of the first | 13:43 |
| openstackgerrit | wes hayutin proposed openstack-infra/tripleo-ci master: record the size of the tripleo logs directory https://review.openstack.org/481306 | 13:43 |
| stephenfin | As for 'suppress_warnings', only one of those is necessary - 'image.nonlocal_uri' - which we could ask folks to set manually or set with openstackdocstheme | 13:43 |
| fungi | i always get annoyed when i discover we've gone through some major overhaul of a basic design pattern throughout the community but neglected to update the cookiecutters and so people continue creating new projects using the old stuff | 13:44 |
| stephenfin | fungi: Will do. Think I've updated that in the past for just that reason | 13:44 |
| mordred | dhellmann: the setup(app) function for a plugin gets a config object on app.config ... I wonder if we can set the values there | 13:44 |
| mordred | stephenfin: right- but we explicitly don't want version and project_name ot be set in conf.py | 13:44 |
| dhellmann | mordred : yeah, we'll have to experiment | 13:44 |
| dhellmann | fungi : ++ | 13:45 |
| stephenfin | mordred: Even if they're pulled from pkg_resources? | 13:45 |
| mordred | stephenfin: because that requires someone putting a version in a file in a git repo that is versioned by git tags | 13:45 |
| mordred | stephenfin: the repo may not be installed yet | 13:45 |
| mordred | so the values in pkg_resources may not be valid | 13:45 |
| stephenfin | Ah, gotcha | 13:45 |
| mordred | I mean - I imagine the sphinx extension to do this would be one provided by pbr itself | 13:46 |
| fungi | also having your build modify in-place files which are tracked in your vcs is aggravating | 13:46 |
| mordred | fungi: ++ | 13:46 |
| stephenfin | So we'll need to use pbr's machinery for that | 13:46 |
| *** mat128 has quit IRC | 13:46 | |
| stephenfin | fungi: Hmm, what files would be modified? | 13:46 |
| mordred | hrm. there is an annoying chicken and egg here- if we do a pbr extension, we'd still need everyone to put the pbr extension in their extensions list | 13:46 |
| dhellmann | mordred : if there is not already a sphinxcontrib-version-from-git we should write that as a separate thing | 13:47 |
| *** xarses has joined #openstack-infra | 13:47 | |
| dhellmann | and yes, adding it explicitly would be part of the rollout | 13:47 |
| *** xarses has quit IRC | 13:47 | |
| *** jsavak has quit IRC | 13:47 | |
| stephenfin | mordred: Yup, that's why I suggested the soft break option (b) above | 13:47 |
| *** dizquierdo has joined #openstack-infra | 13:48 | |
| *** slaweq_ has joined #openstack-infra | 13:48 | |
| *** xarses has joined #openstack-infra | 13:48 | |
| mordred | stephenfin, dhellmann: yah - we could also just do this: https://github.com/openstack/keystoneauth/blob/master/doc/source/conf.py#L65 | 13:48 |
| mordred | and add a hacking check to verify that projects have that cantrip in their conf.py | 13:48 |
| stephenfin | Folks are going to have to do something, but it's a question of how far down the road we kick that can | 13:48 |
| dhellmann | mordred : yeah | 13:48 |
| stephenfin | mordred: yup | 13:48 |
| *** mat128 has joined #openstack-infra | 13:49 | |
| openstackgerrit | Gabriele Cerami proposed openstack-infra/tripleo-ci master: WIP: containers periodic test https://review.openstack.org/475747 | 13:49 |
| stephenfin | or, like dhellmann said, a Sphinx extension that could do that for us | 13:49 |
| mordred | we could even expand it to be a single line: "project, copyright, version, release = pbr.get_sphinx_settings()" | 13:49 |
| dhellmann | less magic here is better, so maybe just saying "set the version; here's a way to do it using pbr" is best | 13:49 |
| mordred | ++ | 13:49 |
| dhellmann | I like the 1 liner, too | 13:49 |
| stephenfin | +1 | 13:49 |
| * mordred writes the one-liner real quick | 13:49 | |
| dhellmann | stephenfin : does any of this change how we do the reno integration you've been working on? | 13:50 |
| dhellmann | that's an sdist hook, right? so maybe not. | 13:50 |
| stephenfin | dhellmann: Not really. I'd structured that as a setuptools plugin so I don't think so | 13:51 |
| dhellmann | ok | 13:51 |
| dhellmann | oh, I meant the pbr side of it. | 13:51 |
| dhellmann | but I think you're still right | 13:51 |
| fungi | stephenfin: you were asking if values could be automatically added to setup.cfg, which would dirty your vcs tracked state (unless operating on a copy) | 13:51 |
| fungi | er, conf.py i mean | 13:52 |
| stephenfin | dhellmann: Nah, I don't think we're going to touch any of the setuptools side of pbr - only the Sphinx bit | 13:52 |
| * dhellmann nods | 13:52 | |
| stephenfin | fungi: Oh, no. conf.py is python so we'd be adding a function call to...something in there | 13:52 |
| stephenfin | We wouldn't hard code it or anything | 13:52 |
| stephenfin | Like this https://github.com/openstack/keystoneauth/blob/master/doc/source/conf.py#L65-L72 | 13:53 |
| fungi | yeah, i had originally read it as asking why we couldn't set name and version in setup.cfg, sorry | 13:53 |
| stephenfin | Ah, no worries :) | 13:53 |
| dhellmann | stephenfin : you haven't come across a sphinx extension that build sitemap files, have you? | 13:53 |
| stephenfin | No, but I can't say I've been looking for one tbh | 13:54 |
| *** jsavak has joined #openstack-infra | 13:54 | |
| fungi | i am not opposed at all to _determining_ those values through some function calls in the sphinx setup.py if it can be done safely | 13:54 |
| *** baoli has joined #openstack-infra | 13:54 | |
| *** baoli has quit IRC | 13:54 | |
| zxiiro | larainema: FYI we have a #openstack-jjb channel now. | 13:54 |
| stephenfin | dhellmann: Should be easy enough to put one together though and include it with openstackdoctheme, though I guess you want to avoid reinventing the wheel | 13:54 |
| *** baoli has joined #openstack-infra | 13:55 | |
| stephenfin | I didn't think sitemap.xml files were still a thing, given how advanced search engines are now | 13:55 |
| *** dansmith is now known as superdan | 13:55 | |
| fungi | er, sphinx conf.py. i should just stop typing and wait for my coffee to kick in ;) | 13:55 |
| *** LindaWang has joined #openstack-infra | 13:56 | |
| dhellmann | stephenfin : apparently you can still influence relative priority on a given site using them | 13:57 |
| dhellmann | the format isn't hard, so we'd probably just bake a generator into the theme | 13:57 |
| *** slaweq_ has quit IRC | 13:58 | |
| *** mriedem has joined #openstack-infra | 13:58 | |
| fungi | apparently all the seo-savvy peeps are into sitemaps | 14:00 |
| *** Goneri has joined #openstack-infra | 14:00 | |
| fungi | and at least one popular search engine will provide "intelligent" results with grouped deep links arranged based on your sitemap metadata | 14:00 |
| *** cshen has quit IRC | 14:01 | |
| dhellmann | I have not-completely-formed plans to replace our current one-big-sitemap with a bunch of little ones generated each time we build docs, and then linked together with a sitemap index file managed in the docs repo using a template. | 14:01 |
| dhellmann | I need to write that up, though, and it's not yet high on the priority list | 14:02 |
| *** vhosakot has joined #openstack-infra | 14:03 | |
| *** alexchadin has quit IRC | 14:03 | |
| *** bobh has joined #openstack-infra | 14:04 | |
| *** camunoz has quit IRC | 14:06 | |
| *** jcoufal has joined #openstack-infra | 14:07 | |
| *** adisky__ has quit IRC | 14:08 | |
| *** hongbin has joined #openstack-infra | 14:08 | |
| *** chlong_ has joined #openstack-infra | 14:09 | |
| *** amotoki is now known as amotoki_away | 14:09 | |
| *** slaweq has quit IRC | 14:09 | |
| *** jamesmcarthur has joined #openstack-infra | 14:09 | |
| *** slaweq has joined #openstack-infra | 14:10 | |
| AJaeger | clarkb, fungi, mordred, could you put my mitaka-eol cleanup on your review list, please? https://review.openstack.org/#/q/status:open+project:openstack-infra/project-config+branch:master+topic:mitaka-eol This reduces also the size of defined jobs and thus the log file from 18 MB to 14 MB. | 14:10 |
| *** jsavak has quit IRC | 14:11 | |
| *** slaweq has quit IRC | 14:14 | |
| *** wolverineav has quit IRC | 14:14 | |
| *** prateek has quit IRC | 14:14 | |
| *** slaweq has joined #openstack-infra | 14:14 | |
| *** wolverineav has joined #openstack-infra | 14:14 | |
| AJaeger | I tried to make smaller changes this time so that it is not so repititve and hurtful ;) | 14:17 |
| *** camunoz has joined #openstack-infra | 14:18 | |
| fungi | AJaeger: the earlier changes were fine too; smaller changes to make it easier to review part of the work and then pick up the rest later though | 14:18 |
| fungi | s/to make/do make/ | 14:18 |
| *** wolverineav has quit IRC | 14:19 | |
| *** rbrndt has joined #openstack-infra | 14:19 | |
| *** rbrndt has quit IRC | 14:19 | |
| AJaeger | yep, I tried to split also in either JJB or zuul changes - easier to verify that I did the right things | 14:19 |
| *** gouthamr has joined #openstack-infra | 14:19 | |
| fungi | agreed, that helps | 14:19 |
| *** slaweq has quit IRC | 14:19 | |
| *** rbrndt has joined #openstack-infra | 14:19 | |
| *** felipemonteiro_ has joined #openstack-infra | 14:19 | |
| *** askb has quit IRC | 14:21 | |
| *** felipemonteiro__ has joined #openstack-infra | 14:21 | |
| *** ldnunes has quit IRC | 14:21 | |
| *** ldnunes has joined #openstack-infra | 14:22 | |
| *** pradk has joined #openstack-infra | 14:22 | |
| fungi | AJaeger: also we should probably bring up the driverfixes/mitaka testing with smcginnis. i believe the plan there was that we should not block trusty removal on those, and either try to run the jobs on xenial or replace them with custom no-op jobs if that doesn't work | 14:23 |
| fungi | but we can figure that out after the bulk of the cleanup is worked through | 14:23 |
| *** felipemonteiro_ has quit IRC | 14:24 | |
| openstackgerrit | Merged openstack-infra/project-config master: Fix tripleo skip conditions https://review.openstack.org/481403 | 14:24 |
| openstackgerrit | Carlos Camacho proposed openstack-infra/tripleo-ci master: Enable the HA deployment by default https://review.openstack.org/415179 | 14:24 |
| fungi | essentially the only reason they're not using the general no-op job is to avoid bridging those into the integrated queue | 14:24 |
| fungi | so the pep8 job was seen as a viable substitute for basically no testing at all | 14:25 |
| EmilienM | AJaeger: good afternoon - https://review.openstack.org/#/c/481177 broke Puppet OpenStack CI - I'm trying to fix it now, I'll probably send a patch over the next minutes | 14:25 |
| *** ykarel has quit IRC | 14:25 | |
| fungi | EmilienM: how did it break? | 14:25 |
| fungi | curious what we missed/misunderstood | 14:26 |
| AJaeger | EmilienM: sorry to hear - interested in your change... | 14:26 |
| *** zhurong has quit IRC | 14:26 | |
| *** slaweq has joined #openstack-infra | 14:26 | |
| *** csomerville has quit IRC | 14:26 | |
| EmilienM | fungi, AJaeger : see https://review.openstack.org/#/c/481003/ - we are not supposed to run gate-puppet-openstack-integration-3-* | 14:27 |
| *** csomerville has joined #openstack-infra | 14:27 | |
| openstackgerrit | Merged openstack-infra/project-config master: Create python-jobs-trusty job group https://review.openstack.org/481390 | 14:27 |
| *** armax has joined #openstack-infra | 14:27 | |
| fungi | EmilienM: oh! yep, i see the error in the change now | 14:28 |
| fungi | we dropped the conditional which said to only run that on old branches rather than dropping the job itself | 14:28 |
| AJaeger | Argh ;( | 14:28 |
| EmilienM | no problem :) | 14:29 |
| * AJaeger cannot check right now | 14:29 | |
| EmilienM | AJaeger: no worries, I'll send a patch | 14:29 |
| fungi | name: ^gate-puppet-openstack-integration-3-.*$ | 14:29 |
| fungi | branch: ^stable/(liberty|mitaka|hammer).*$ | 14:29 |
| fungi | AJaeger: ^ we removed that | 14:29 |
| AJaeger | Ah, seems we thought it was negated ;( | 14:30 |
| AJaeger | EmilienM: will you remove the job definitions as well in that case, please? | 14:30 |
| fungi | right, that's how i likely read it as well when reviewing | 14:30 |
| dhellmann | mordred, fungi : while you're thinking about setuptools things, this pbr review could use some attention: https://review.openstack.org/#/c/475034/2 | 14:30 |
| *** slaweq has quit IRC | 14:31 | |
| fungi | EmilienM: sorry about that! something about an omelet and broken eggs | 14:31 |
| EmilienM | fungi: again, no worries at all | 14:31 |
| EmilienM | you're doing a complex work in EOLing things, of course mistakes happen :) | 14:31 |
| EmilienM | I'll propose a patch | 14:31 |
| fungi | thanks for understanding! | 14:32 |
| AJaeger | I would have hoped none happens - but it's indeed some work. Thanks for your understanding and help, EmilienM ! | 14:32 |
| EmilienM | cool, I have a quick easy fix | 14:33 |
| fungi | mordred: on a related note to the build_reno extension, do you happen to know if pbr ever grew a mechanism for generating manpages for inclusion during sdist builds? | 14:34 |
| *** bobh has quit IRC | 14:35 | |
| fungi | i'm wondering how separating out the special sphinx bits might impact any possibility for that | 14:35 |
| openstackgerrit | Emilien Macchi proposed openstack-infra/project-config master: Remove puppet-openstack-integration-3-.* jobs https://review.openstack.org/481647 | 14:36 |
| fungi | worst case, i guess manpage generation as a separate step can be punted to the distro package maintainers instead | 14:36 |
| fungi | i just always thought including them in the source tarballs would be useful for things like command-line tools | 14:37 |
| openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck master: Remove query for fixed bug 1654393 https://review.openstack.org/481648 | 14:38 |
| openstack | bug 1654393 in OpenStack Compute (nova) ""keystoneauth1.exceptions.catalog.EmptyCatalog: The service catalog is empty." in test_cinderclient_unsupported_v1" [High,Fix released] https://launchpad.net/bugs/1654393 - Assigned to Matt Riedemann (mriedem) | 14:38 |
| EmilienM | AJaeger, fungi : thx for the quick ninja review | 14:39 |
| fungi | EmilienM: thx for the quick ninja fix! | 14:39 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack-infra/project-config master: Add networking-ovn job to ovsdbapp gate https://review.openstack.org/481650 | 14:41 |
| AJaeger | EmilienM: we just trust youit will work | 14:41 |
| AJaeger | ;) | 14:41 |
| *** andreas_s has quit IRC | 14:41 | |
| fungi | it looked fine to me, and if it doesn't pass ci it won't merge, so no harm in approving | 14:42 |
| EmilienM | clarkb: good morning - quick update fyi on re: logs collection - here's the main efforts to improve the situation: https://review.openstack.org/#/q/topic:bug/1698172 - our team is working on it | 14:42 |
| EmilienM | clarkb: some patches are failing because we had a CI downtime last night, due do an upgrade of docker version in centos. | 14:43 |
| fungi | mordred: 475034 still depends-on the corresponding entrypoint in reno which seems to still be incomplete | 14:43 |
| AJaeger | fungi: agreed | 14:43 |
| *** sdague has joined #openstack-infra | 14:44 | |
| *** bobh has joined #openstack-infra | 14:44 | |
| *** xarses_ has joined #openstack-infra | 14:45 | |
| *** lbragstad has joined #openstack-infra | 14:45 | |
| openstackgerrit | Merged openstack-infra/elastic-recheck master: Remove query for fixed bug 1654393 https://review.openstack.org/481648 | 14:46 |
| openstack | bug 1654393 in OpenStack Compute (nova) ""keystoneauth1.exceptions.catalog.EmptyCatalog: The service catalog is empty." in test_cinderclient_unsupported_v1" [High,Fix released] https://launchpad.net/bugs/1654393 - Assigned to Matt Riedemann (mriedem) | 14:46 |
| *** dougs2 has joined #openstack-infra | 14:46 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack-infra/project-config master: Add networking-ovn job to ovsdbapp gate https://review.openstack.org/481650 | 14:47 |
| *** andreww has joined #openstack-infra | 14:47 | |
| dougs2 | Please could I request that the branch stable/mitaka is deleted from the bareon-ironic repo? We don't plan to push any changes for this. Thanks | 14:47 |
| AJaeger | dougs2: best answer on my email on openstack-infra - and I'll collect and hand it over to jhesketh | 14:48 |
| AJaeger | dougs2: http://lists.openstack.org/pipermail/openstack-infra/2017-July/005480.html - or my last on openstack-dev | 14:49 |
| *** xarses_ has quit IRC | 14:49 | |
| dougs2 | ok, thanks | 14:50 |
| openstackgerrit | Merged openstack-infra/project-config master: Remove puppet-openstack-integration-3-.* jobs https://review.openstack.org/481647 | 14:51 |
| fungi | dougs2: thanks. we're trying to batch these up since there are apparently several projects who missed requesting an addition the original batch (and at least one it looks like we missed from the original batch as well) | 14:51 |
| *** bobh has quit IRC | 14:53 | |
| *** bobh has joined #openstack-infra | 14:55 | |
| stephenfin | dhellmann: What tags would I use for that "deprecating pbr's build_sphinx" mail? | 14:56 |
| stephenfin | if any | 14:56 |
| dhellmann | hmm | 14:56 |
| dhellmann | docs, pbr, ptl | 14:56 |
| dhellmann | maybe throw in oslo, since technically that team owns the project repo | 14:57 |
| stephenfin | (y) | 14:57 |
| stephenfin | done and dusted | 14:58 |
| *** LindaWang has quit IRC | 14:59 | |
| *** LindaWang has joined #openstack-infra | 14:59 | |
| openstackgerrit | Merged openstack-infra/project-config master: Create python-jobs-trusty zuul template https://review.openstack.org/481425 | 14:59 |
| *** camunoz has quit IRC | 15:01 | |
| *** bobh has quit IRC | 15:02 | |
| *** rcernin has quit IRC | 15:06 | |
| dmsimard | mtreinish: can I bother you for a min ? Need a refresher on how the data ends up being collected for openstack-health. Is it a post job that triggers a pull of the stackviz log file for a particular job or something like that ? | 15:06 |
| *** bobh has joined #openstack-infra | 15:08 | |
| dmsimard | mtreinish: or something that listens on firehose maybe ? | 15:08 |
| fungi | dmsimard: it's handled with a persistent daemon similar to our logstash indexer | 15:08 |
| *** ociuhandu has joined #openstack-infra | 15:09 | |
| fungi | dmsimard: it could be reworked to rely on firehose events once we get zuul/nodepool reporting into it, but for now it watches zuul's gearman queue i believe | 15:09 |
| * fungi digs up the docs | 15:09 | |
| *** rossella_ has quit IRC | 15:09 | |
| *** markus_z has quit IRC | 15:11 | |
| openstackgerrit | Darragh Bailey proposed openstack-infra/irc-meetings master: Add OpenStack Jenkins Job Builder team meeting https://review.openstack.org/481664 | 15:13 |
| *** jamesmcarthur has quit IRC | 15:14 | |
| *** camunoz has joined #openstack-infra | 15:14 | |
| *** jamesmcarthur has joined #openstack-infra | 15:14 | |
| dmsimard | fungi: thanks -- I was trying to dig in https://github.com/openstack/openstack-health but that seems to be mostly the web application itself | 15:15 |
| mtreinish | dmsimard, fungi: yeah it leverages zmq and gearman right now | 15:16 |
| clarkb | fungi dmsimard it uses the same mechansim as the logstash workers. zuul emits events over 0mq pub sub when jobs complete. We generate gearman jobs to handle log artifacts from that | 15:16 |
| mtreinish | it's the same mechanism as logstash, we just have a different gearman queue for subunit | 15:16 |
| *** zzzeek_ has joined #openstack-infra | 15:17 | |
| mtreinish | dmsimard: yeah openstack-health is just a web app, but the data store it uses is a subunit2sql db | 15:17 |
| mtreinish | (and logstash for elastic-recheck data) | 15:17 |
| fungi | dmsimard: https://docs.openstack.org/infra/system-config/logstash.html#subunit-gearman-worker | 15:17 |
| *** zzzeek_ has quit IRC | 15:17 | |
| fungi | right, for some reason i forgot that zuul grew a zeromq publisher at one point | 15:17 |
| mordred | dhellmann: we also need to add a sphinx extension that can run the changelog and authors generation - since that's a thing that happens a doc build time currently too | 15:17 |
| dmsimard | fungi, clarkb, mtreinish: awesome, thanks for the info | 15:17 |
| mordred | dhellmann: I think that one can safely be a sphinx extension provided by pbr | 15:18 |
| fungi | mordred: that seems like a good pattern for the manpage generation i'm thinking of as well in that case | 15:18 |
| fungi | the extension already exists, but we would potentially call it during sdist | 15:18 |
| dhellmann | mordred : don't we get those files when tox does the equivalent of "pip install ." | 15:18 |
| mordred | dhellmann: no - they're currently only generated on sdist and on build_sphinx | 15:19 |
| *** bobh has quit IRC | 15:19 | |
| mordred | dhellmann: so if you just pip install . you skip the sdist step and then no changelog/authors | 15:19 |
| mordred | which makes some folks' docs sad | 15:20 |
| dhellmann | ah | 15:20 |
| dhellmann | stephenfin : ^^ | 15:20 |
| mordred | dhellmann: but I think just a tiny extension that does the builder-inited event and calls the 2 pbr functions is likely fine | 15:21 |
| *** bobh has joined #openstack-infra | 15:21 | |
| dhellmann | mordred : maybe you can follow up to http://lists.openstack.org/pipermail/openstack-dev/2017-July/119396.html | 15:21 |
| dhellmann | yeah, although it means adding that explicitly to projects, but that's ok, too | 15:21 |
| mordred | fungi, dhellmann: so - for building manpages before sdist, that means we'd need to tell pbr to run whatever our sphinx command winds up being | 15:22 |
| mordred | fungi, dhellmann: OR - we could just make "run docs, then run python setup.py sdist" part of the PTI for making a tarball | 15:22 |
| dhellmann | I think I would rather do that | 15:22 |
| mordred | it would be easy to duplicate for other languages that make tarballs | 15:22 |
| *** burgerk has joined #openstack-infra | 15:22 | |
| dhellmann | then we can ship all of the built docs in whatever format | 15:22 |
| dhellmann | that's also a good point | 15:22 |
| mordred | basically - the CI system would just do "run the PTI entry for docs generation, then the PTI entry for tarball generation" | 15:23 |
| dhellmann | right | 15:23 |
| fungi | if it's more than one command, we likely need a wrapper (which probably means a new tox env) | 15:24 |
| *** rossella_ has joined #openstack-infra | 15:25 | |
| fungi | and if memory serves, the main argument against relying on `tox -e docs` was that the majority of the python ecosystem expects to be able to call `python setup.py build_sphinx` | 15:27 |
| dhellmann | fungi : I'm not sure that's all that standard. | 15:27 |
| fungi | so i would thing the same goes for `tox -e sdist` vs `python setup.py sdist` | 15:27 |
| dhellmann | it might be, I don't know | 15:27 |
| fungi | s/thing/think/ | 15:27 |
| dhellmann | the sdist command is definitely standard | 15:28 |
| dhellmann | I don't know how most people actually run sphinx | 15:28 |
| *** trown is now known as trown|brb | 15:28 | |
| dhellmann | it still generates Makefiles, though, so it's possible "make" is as much a standard as anything | 15:28 |
| *** ihrachys_ has joined #openstack-infra | 15:28 | |
| fungi | i would buy that | 15:28 |
| clarkb | I think it was more you can use standard sphinx tool whether the setup.py extension or the equivalent cli command or make to build the docs without special extras | 15:29 |
| clarkb | since make build_sphinc and buildsphinx or whatever it is called should all be roughly equivalent | 15:29 |
| dhellmann | but you don't get the same results if you run sphinx through setuptools and directly, because when it runs directly it doesn't read settings from setup.cfg | 15:30 |
| clarkb | ya I think in practice sphinx has produced disparate tooli g that doesnt do ehat you expect | 15:30 |
| clarkb | byt that was the intention | 15:30 |
| fungi | dhellmann: doesn't it though, by way of pbr having a setuptools entrypoint? | 15:31 |
| dhellmann | fungi : right, "python setup.py build_sphinx" gets data from setup.cfg but "sphinx-build ..." does not | 15:31 |
| clarkb | read the docs uses the cli command iirc | 15:31 |
| fungi | dhellmann: oh, got it. i missed the "and" in your original statement | 15:32 |
| dhellmann | yeah, that's how I always used sphinx before I started using pbr | 15:32 |
| *** ykarel has joined #openstack-infra | 15:32 | |
| fungi | so, yes i think in general the choice to get the extra magic somewhere that the sphinx conf.py can call into it regardless of how it gets invoked is in the spirit of the pti | 15:33 |
| fungi | more than the current implementation with pbr and its setuptools entrypoint | 15:33 |
| *** jascott1 has joined #openstack-infra | 15:33 | |
| dhellmann | fungi : yes, I think that's it | 15:34 |
| fungi | whereas having special extra pre-building commands before sdist generation is probably going in the other direction | 15:34 |
| dhellmann | maybe :-( | 15:34 |
| dhellmann | maybe what we want is a pbr option for "tell me how to build docs before I build an sdist" | 15:34 |
| dhellmann | instead of having logic hard-coded into pbr | 15:34 |
| fungi | so any plan that involves extra steps before invoking setup.py sdist or else you don't get the same tarball contents we release is probably anti-pti | 15:35 |
| *** jamesmcarthur has quit IRC | 15:35 | |
| *** g^k_1^w^m_s^u has joined #openstack-infra | 15:36 | |
| *** g^k_1^w^m_s^u has left #openstack-infra | 15:36 | |
| dhellmann | that makes sense | 15:36 |
| fungi | not sure who else follows the distutils-sig ml, but the plan around pep 517 backends is probably relevant here too | 15:36 |
| *** jascott1 has quit IRC | 15:38 | |
| fungi | tl;dr is how to make it possible to replace setuptools for sdist and bdist(wheel) generation | 15:38 |
| openstackgerrit | Monty Taylor proposed openstack-dev/pbr master: Add two helper functions for setting sphinx config https://review.openstack.org/481674 | 15:38 |
| openstackgerrit | Monty Taylor proposed openstack-dev/pbr master: Add suppress warnings to set_sphinx_config https://review.openstack.org/481675 | 15:38 |
| openstackgerrit | Monty Taylor proposed openstack-dev/pbr master: Grab package name from setup.cfg https://review.openstack.org/481676 | 15:38 |
| fungi | and the hook mechanisms for different stages of the build process are still being hashed out a bit on the ml | 15:38 |
| openstackgerrit | Monty Taylor proposed openstack-dev/pbr master: Add openstackdocstheme support and infer names https://review.openstack.org/481677 | 15:38 |
| openstackgerrit | Monty Taylor proposed openstack-dev/pbr master: Add htmlhelp and latex options https://review.openstack.org/481678 | 15:38 |
| *** trown|brb is now known as trown | 15:38 | |
| *** jamesmcarthur has joined #openstack-infra | 15:38 | |
| *** amotoki_away is now known as amotoki | 15:39 | |
| mordred | dhellmann: there is a stack with an increasing set of options | 15:39 |
| fungi | but basically there would be a stage where we could specify the hook that sets our package metadata based on queries to git or whatever before the tree is potentially copied/filtered at the start of the sdist generation stage | 15:39 |
| mordred | dhellmann: ++ to pbr option for "tell me how to build docs" - although I think it should have a default value potentially? | 15:40 |
| mordred | that way we can have python setup.py sdist DTRT for folks in most cases, but give an escape hatch if there is a different invocation people are expecting to be able to use | 15:41 |
| *** g^k_1^w^m_s^u has joined #openstack-infra | 15:41 | |
| hongbin | hi, does anyone know who is maintaining the project navigator https://www.openstack.org/software/project-navigator/ ? I have a question for it | 15:41 |
| fungi | hongbin: the openstack foundation web dev team | 15:41 |
| *** bobh has quit IRC | 15:41 | |
| clarkb | AJaeger: small item on https://review.openstack.org/#/c/481567/1 | 15:42 |
| mordred | yah. although it does get data from a git repo in openstack | 15:42 |
| *** bobh has joined #openstack-infra | 15:43 | |
| hongbin | fungi: i observed that zun doesn't have the logo (or it doesn't show properly), do you know who is the person i should contact? | 15:43 |
| *** jamesmcarthur has quit IRC | 15:43 | |
| fungi | hongbin: i think anne@openstack.org has taken over the logo/mascot work now that heidi joy has moved on to another position | 15:43 |
| hongbin | fungi: ack, thx | 15:44 |
| *** jamesmcarthur has joined #openstack-infra | 15:44 | |
| *** hashar is now known as hasharAway | 15:45 | |
| *** e0ne has quit IRC | 15:47 | |
| cmurphy | if any project-config cores would like to +w https://review.openstack.org/#/c/477239/ for me that will help me get some more momentum on puppet 4 stuff (thanks AJaeger for fixing my derp) | 15:48 |
| *** amotoki is now known as amotoki_away | 15:49 | |
| openstackgerrit | Jesse Keating proposed openstack-infra/zuul feature/zuulv3: Ensure github change body is not a NoneType https://review.openstack.org/481681 | 15:50 |
| clarkb | cmurphy: I'll add it to the list | 15:53 |
| cmurphy | clarkb: tyty | 15:53 |
| openstackgerrit | Merged openstack-infra/project-config master: Remove unused centos7 chef multinode job https://review.openstack.org/481602 | 15:54 |
| clarkb | AJaeger: out of curiousity re https://review.openstack.org/#/c/481430/5/jenkins/jobs/projects.yaml those projects are keeping old branches around or they run trusty against newer branches? | 15:55 |
| openstackgerrit | Merged openstack-infra/project-config master: Remove unused python-jobs https://review.openstack.org/481607 | 15:55 |
| *** Apoorva has joined #openstack-infra | 15:55 | |
| openstackgerrit | Merged openstack-infra/project-config master: Remove obsolete python-jobs-linters template https://review.openstack.org/481392 | 15:56 |
| *** lucasagomes is now known as lucas-afk | 15:58 | |
| *** gyee has joined #openstack-infra | 15:58 | |
| *** martinkopec has quit IRC | 15:58 | |
| *** waynr has quit IRC | 16:01 | |
| openstackgerrit | Merged openstack-infra/project-config master: Move all trusty jobs to python-jobs-trusty job group https://review.openstack.org/481430 | 16:03 |
| *** LindaWang has quit IRC | 16:05 | |
| EmilienM | fungi: did you already hear the feedback that RAX cloud is slower than other clouds? We have been experiencing job timeouts when running in RAX - I've started to debug it and I found out it's not only network but also VM performances I think | 16:07 |
| *** waynr has joined #openstack-infra | 16:07 | |
| *** yamahata has joined #openstack-infra | 16:07 | |
| openstackgerrit | Merged openstack-infra/project-config master: Co-gate spec helper gem with puppet-openstackci https://review.openstack.org/477239 | 16:07 |
| openstackgerrit | Melvin Hillsman (mrhillsman) proposed openstack-infra/project-config master: Add UC IRC channel to accessbot Add governance-uc watching to UC IRC channel https://review.openstack.org/479461 | 16:07 |
| dougs2 | I'm having an issue with Jenkins not respecting the openstack requirements upper-constraints file. On stable newton, oslo.db==4.24.0 is getting installed, whereas it should be pinned to oslo.db===4.13.6. I figure this is something I need to fix in openstack-infra. Any pointers? | 16:08 |
| dougs2 | here is a log, showing the wrong package being installed: http://logs.openstack.org/53/481653/1/check/gate-bareon-ironic-python27-ubuntu-xenial/e4d5cdd/console.html | 16:08 |
| openstackgerrit | Hironori Shiina proposed openstack-infra/project-config master: Add an experimental job to test ironic boot from volume https://review.openstack.org/480332 | 16:09 |
| clarkb | dougs2: https://git.openstack.org/cgit/openstack/bareon/tree/tox.ini#n8 needs to be updated to install with constraints applied | 16:10 |
| dougs2 | ah, as simple as that | 16:11 |
| clarkb | dougs2: there should be pletny of examples in other tox.ini files | 16:11 |
| fungi | EmilienM: news to me but i'll keep an ear to the ground | 16:12 |
| *** jascott1 has joined #openstack-infra | 16:12 | |
| dougs2 | clarkb, got it thanks | 16:12 |
| EmilienM | fungi: i'm preparing a bug report | 16:13 |
| *** thorst has quit IRC | 16:15 | |
| *** jamesmcarthur has quit IRC | 16:16 | |
| clarkb | fungi: there are a few more changes from AJaeger that I think are ready to merge under that mitaka-eol topic if you want to give it another pass | 16:17 |
| *** jamesmcarthur has joined #openstack-infra | 16:17 | |
| EmilienM | fungi: https://bugs.launchpad.net/tripleo/+bug/1702955 | 16:18 |
| openstack | Launchpad bug 1702955 in tripleo "tripleo upgrade jobs timeout when running in RAX cloud" [Critical,Triaged] | 16:18 |
| *** ramishra has quit IRC | 16:19 | |
| clarkb | worth noting that osic is very much the best case scenario as far as performance goes. That hardware is shiny | 16:20 |
| *** _1_w_m__s_u__g_k has joined #openstack-infra | 16:20 | |
| *** ramishra has joined #openstack-infra | 16:20 | |
| *** _1_w_m__s_u__g_k has quit IRC | 16:20 | |
| EmilienM | fungi: if you know someone who we should inform about this problem | 16:21 |
| EmilienM | clarkb: I guess it would be unfair to schedule tripleo upgrade jobs on osic only. | 16:21 |
| EmilienM | I would rather try to understand why rax is slower and how we can improve that | 16:21 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Fix AttributeError in executor client https://review.openstack.org/481691 | 16:26 |
| clarkb | EmilienM: ya understanding why it is slower is good. Just pointing out everything is slower than osic | 16:26 |
| EmilienM | clarkb: rax is really slower I think | 16:27 |
| *** rossella_ has quit IRC | 16:27 | |
| fungi | i don't see any issues highlighted on https://rackspace.service-now.com/system_status/ | 16:33 |
| fungi | we can file a trouble ticket, but i'm not even sure what i'd begin to say in it | 16:34 |
| *** ociuhandu has quit IRC | 16:36 | |
| *** ramishra has quit IRC | 16:36 | |
| openstackgerrit | Emilien Macchi proposed openstack-infra/project-config master: puppet: remove unit 3.3, 3.4, 3.7 and 4.0 https://review.openstack.org/481692 | 16:36 |
| EmilienM | fungi: right, I don't have _details_ on what's wrong | 16:36 |
| EmilienM | mwhahaha, fungi, AJaeger: another one ^ (let's wait on CI first, this one was a bit trickier) | 16:36 |
| *** jpich has quit IRC | 16:37 | |
| *** ykarel has quit IRC | 16:38 | |
| *** jpena is now known as jpena|off | 16:38 | |
| EmilienM | clarkb: could you please tell us which jobs were collecting too much logs? | 16:39 |
| EmilienM | clarkb: we measured one, here: http://logs.openstack.org/06/481306/2/check/gate-tripleo-ci-centos-7-containers-multinode/bb6281b/logs/log-size.txt.gz | 16:39 |
| EmilienM | and it takes 49M | 16:39 |
| *** amotoki_away is now known as amotoki | 16:40 | |
| clarkb | EmilienM: http://logs.openstack.org/15/481115/1/check/gate-tripleo-ci-centos-7-undercloud-containers/2889c15/ was my example yesterday | 16:41 |
| EmilienM | wendar: I don't see the log size file in http://logs.openstack.org/06/481306/2/check/gate-tripleo-ci-centos-7-undercloud-containers/c3c7608/logs/ | 16:41 |
| fungi | EmilienM: tab complete fail? | 16:42 |
| EmilienM | weshay: ^ | 16:42 |
| EmilienM | wendar: apologize | 16:42 |
| EmilienM | weshay: oh it's because it's not oooq job - can you maybe update the patch to also incluse legacy jobs? | 16:43 |
| weshay | EmilienM, we'd have to merge that review | 16:43 |
| EmilienM | weshay: I guess we didn't optimize log collection for legacy jobs (non oooq) | 16:43 |
| weshay | https://review.openstack.org/#/c/481306/ | 16:43 |
| fungi | EmilienM: i've also proposed https://review.openstack.org/481179 to allow us to get that for any job's logs in the future | 16:43 |
| EmilienM | weshay: +2 on https://review.openstack.org/#/c/481306/ | 16:43 |
| weshay | I'll grab trown | 16:43 |
| EmilienM | grab him :D | 16:44 |
| fungi | but turn-around on getting that into our running zuul-launchers will likely take a while | 16:44 |
| trown | im here | 16:44 |
| EmilienM | weshay: but again, the issue is on legacy jobs | 16:44 |
| trown | but about to go to lunsh | 16:44 |
| trown | lunch even | 16:44 |
| *** jascott1 has quit IRC | 16:44 | |
| EmilienM | fungi: indeed, very helpful. Thanks | 16:44 |
| *** Douhet has quit IRC | 16:45 | |
| *** Douhet has joined #openstack-infra | 16:46 | |
| *** iyamahat has joined #openstack-infra | 16:46 | |
| *** dhinesh has joined #openstack-infra | 16:46 | |
| *** psachin has joined #openstack-infra | 16:47 | |
| clarkb | hrm my browser just started rendering the wrong fonts and glyphs | 16:47 |
| clarkb | worried I may have bad memory | 16:48 |
| *** iyamahat has quit IRC | 16:48 | |
| clarkb | (so if I disappear I may be memchecking and following up with a fsck | 16:48 |
| *** iyamahat has joined #openstack-infra | 16:48 | |
| fungi | eep! | 16:49 |
| fungi | g'luck | 16:49 |
| *** amotoki is now known as amotoki_away | 16:49 | |
| *** trown is now known as trown|lunch | 16:50 | |
| *** iyamahat has quit IRC | 16:51 | |
| openstackgerrit | Emilien Macchi proposed openstack-infra/project-config master: puppet: remove unit 3.3, 3.4, 3.7 and 4.0 https://review.openstack.org/481692 | 16:55 |
| *** psachin has quit IRC | 16:57 | |
| *** efoley has quit IRC | 16:58 | |
| openstackgerrit | Darragh Bailey proposed openstack-infra/jenkins-job-builder master: Fix workspace cleanup code https://review.openstack.org/481701 | 16:58 |
| *** ralonsoh has quit IRC | 17:03 | |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Fix AttributeError in executor client https://review.openstack.org/481691 | 17:03 |
| electrofelix | Anyone for reviewing a few git-review changes? https://review.openstack.org/389701 - fixes listing reviews on python3, https://review.openstack.org/244083 - adds topic field to list output, https://review.openstack.org/398353 - refactor displaying reviews code, https://review.openstack.org/312056 - support limit listing reviews by branch | 17:03 |
| fungi | i think clarkb was looking at them recently. it's on my to do list to get to them as well and at least get through any fixing bugs (i doubt i have bandwidth to go through new feature proposals and explain why more features is not always more better) | 17:06 |
| EmilienM | AJaeger, fungi : when you have a min, please look https://review.openstack.org/#/c/481692/ (I think that's all for us today) | 17:06 |
| *** iyamahat has joined #openstack-infra | 17:06 | |
| *** dhinesh has quit IRC | 17:06 | |
| *** sdague has quit IRC | 17:09 | |
| *** ccamacho has quit IRC | 17:11 | |
| *** ccamacho has joined #openstack-infra | 17:11 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Add flag to include all images in image list https://review.openstack.org/481702 | 17:11 |
| *** tosky has quit IRC | 17:14 | |
| *** dizquierdo has quit IRC | 17:15 | |
| *** Swami has joined #openstack-infra | 17:15 | |
| openstackgerrit | Monty Taylor proposed openstack-dev/pbr master: Add two helper functions for setting sphinx config https://review.openstack.org/481674 | 17:22 |
| openstackgerrit | Monty Taylor proposed openstack-dev/pbr master: Add openstackdocstheme support and infer names https://review.openstack.org/481677 | 17:22 |
| openstackgerrit | Monty Taylor proposed openstack-dev/pbr master: Grab package name from setup.cfg https://review.openstack.org/481676 | 17:22 |
| openstackgerrit | Monty Taylor proposed openstack-dev/pbr master: Add htmlhelp and latex options https://review.openstack.org/481678 | 17:22 |
| *** dave-mccowan has joined #openstack-infra | 17:24 | |
| *** jamesmcarthur has quit IRC | 17:24 | |
| openstackgerrit | Merged openstack-infra/project-config master: mitaka-eol: remove trusty-job from fuel-ccp-component https://review.openstack.org/481434 | 17:25 |
| *** rbrndt has quit IRC | 17:25 | |
| *** links has joined #openstack-infra | 17:25 | |
| *** jascott1 has joined #openstack-infra | 17:25 | |
| *** iyamahat_ has joined #openstack-infra | 17:26 | |
| *** baoli has quit IRC | 17:27 | |
| *** baoli has joined #openstack-infra | 17:28 | |
| *** iyamahat_ has quit IRC | 17:28 | |
| *** iyamahat_ has joined #openstack-infra | 17:28 | |
| *** iyamahat_ has quit IRC | 17:29 | |
| *** iyamahat_ has joined #openstack-infra | 17:29 | |
| *** links has quit IRC | 17:29 | |
| *** iyamahat has quit IRC | 17:30 | |
| *** links has joined #openstack-infra | 17:30 | |
| openstackgerrit | Merged openstack-infra/project-config master: mitaka-eol: Remove trusty linters job https://review.openstack.org/481435 | 17:31 |
| *** tosky has joined #openstack-infra | 17:32 | |
| *** sambetts is now known as sambetts|afk | 17:34 | |
| *** ociuhandu has joined #openstack-infra | 17:36 | |
| *** links has quit IRC | 17:36 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Read config from .zuul.yaml or zuul.yaml https://review.openstack.org/481706 | 17:36 |
| *** iyamahat_ has quit IRC | 17:37 | |
| *** iyamahat_ has joined #openstack-infra | 17:37 | |
| *** slaweq has joined #openstack-infra | 17:37 | |
| *** jamesmcarthur has joined #openstack-infra | 17:38 | |
| *** iyamahat_ has quit IRC | 17:38 | |
| *** iyamahat_ has joined #openstack-infra | 17:38 | |
| fungi | jeblair: quick question... i'm preparing to create the new freenode account for openstackptg and, looking at our account list, it seems we've used separate mail aliases in the past. is that a pattern we should try to stick with? | 17:39 |
| *** Douhet has quit IRC | 17:39 | |
| openstackgerrit | James E. Blair proposed openstack-infra/puppet-zuul master: Update zuulv3.conf https://review.openstack.org/481707 | 17:40 |
| openstackgerrit | Darragh Bailey proposed openstack-infra/project-config master: Add dedicated channel for jenkins-job-builder https://review.openstack.org/481708 | 17:40 |
| fungi | i'm going to try to write up a brief section for https://docs.openstack.org/infra/system-config/irc.html with steps for doing this when adding new bots in the future | 17:40 |
| *** electrofelix has quit IRC | 17:40 | |
| *** Douhet has joined #openstack-infra | 17:40 | |
| *** baoli has quit IRC | 17:40 | |
| fungi | since i guess things like some of the nickserv options we want to set would be nice to standardize on | 17:40 |
| *** amotoki_away is now known as amotoki | 17:40 | |
| jeblair | fungi: i believe they are all email aliases. i'm assuming that separate nickserv accounts should have different emails, but i'm not certain as to the requirements. | 17:41 |
| jeblair | fungi: i very likely just started with that assumption (since it seemed safe) and proceeded from there without verifying | 17:41 |
| fungi | and we no longer have anyone on the infra team who has the ability to add more aliases as far as you're aware, right? | 17:41 |
| *** baoli has joined #openstack-infra | 17:41 | |
| fungi | thinking i may need to arrange it through jbryce if we want to stick with that pattern going forward | 17:42 |
| *** jamesmcarthur has quit IRC | 17:42 | |
| jbryce | fungi: I can give you access to create aliases if that's what you're needing | 17:43 |
| *** jascott1 has quit IRC | 17:44 | |
| fungi | jbryce: that might help. i think jeblair was doing them for us way back when but we haven't needed any new ones for a while | 17:44 |
| *** bnemec is now known as beekneemech | 17:45 | |
| *** jascott1 has joined #openstack-infra | 17:45 | |
| fungi | jbryce: we don't need to add new accounts, just another alias for the infra-root@openstack.org account | 17:45 |
| fungi | i'm checking now to see whether it has the ability to do that itself | 17:46 |
| fungi | for consistency we'll make ptgbot@openstack.org an alias for infra-root@openstack.org | 17:47 |
| fungi | though it's taking me a while to log into the infra-root webmail interface. i should probably clean up some of the filtered inboxes there | 17:48 |
| *** slaweq has quit IRC | 17:49 | |
| *** yamahata has quit IRC | 17:49 | |
| *** thorst has joined #openstack-infra | 17:49 | |
| *** slaweq has joined #openstack-infra | 17:50 | |
| *** amotoki is now known as amotoki_away | 17:50 | |
| *** iyamahat_ has quit IRC | 17:50 | |
| *** thorst has quit IRC | 17:52 | |
| *** thorst has joined #openstack-infra | 17:52 | |
| *** dhinesh has joined #openstack-infra | 17:53 | |
| dhellmann | fungi : I added you as a reviewer to this docs patch to create a script to move the old docs.o.o/developer content into the new structure: https://review.openstack.org/#/c/481710/ | 17:53 |
| dhellmann | no rush on looking at it | 17:53 |
| fungi | dhellmann: thanks! | 17:54 |
| *** bit_lySLH2uSZHed has joined #openstack-infra | 17:55 | |
| dhellmann | fungi : I want to get AJaeger and asettle to look that over before we run it, to make sure I haven't missed any directories | 17:55 |
| openstackgerrit | Ben Nemec proposed openstack-infra/project-config master: Make tripleo undercloud-upgrades job voting https://review.openstack.org/481711 | 17:55 |
| openstackgerrit | Rafael Folco proposed openstack/diskimage-builder master: Enable console during kernel boot on Power https://review.openstack.org/481712 | 17:55 |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Read config from .zuul.yaml or zuul.yaml https://review.openstack.org/481706 | 17:56 |
| *** bit_lySLH2uSZHed has left #openstack-infra | 17:56 | |
| fungi | clarkb: you worked out our initial account setup with ovh, right? trying to figure out why the infra-root mailbox is getting spammed half a dozen times each day with payment reminders, maybe you remember whether this is a known thing? | 18:03 |
| clarkb | fungi: it was jeblair iirc. I did osic and citycloud and vexxhost | 18:03 |
| fungi | ahh | 18:03 |
| fungi | at least i _think_ they're payment reminders, but my french is really rusty | 18:03 |
| jeblair | i am bad at paying bills | 18:03 |
| clarkb | I want to say that we have to reup that account periodically like once a year and ran into this before? | 18:03 |
| fungi | Subject: Société Ovh : obligation de paiement ... | 18:04 |
| openstackgerrit | Merged openstack-infra/puppet-zuul master: Update zuulv3.conf https://review.openstack.org/481707 | 18:04 |
| fungi | hpe likes to spam jenkins@o.o about their webinars | 18:06 |
| mordred | fungi: I don't thik I did ovh - I remember doing internap (their billing human was confused by me) | 18:06 |
| openstackgerrit | Merged openstack-infra/shade master: Add support for list available QoS rule types https://review.openstack.org/481282 | 18:08 |
| openstackgerrit | Merged openstack-infra/shade master: Add validation of required QoS extensions in Neutron https://review.openstack.org/481169 | 18:08 |
| *** dtantsur|afk has quit IRC | 18:10 | |
| *** dtantsur has joined #openstack-infra | 18:13 | |
| fungi | looks like the server-side filters for this mailbox aren't being applied or they're configured incorrectly | 18:13 |
| *** e0ne has joined #openstack-infra | 18:14 | |
| *** tesseract has quit IRC | 18:21 | |
| *** rbrndt has joined #openstack-infra | 18:24 | |
| *** trown|lunch is now known as trown | 18:24 | |
| *** yamahata has joined #openstack-infra | 18:24 | |
| fungi | huh, apparently there's some administrator contact feature in zanata where users can report problems? the result is their message goes to infra-root@o.o, and never noticed :( | 18:28 |
| fungi | i've only found one in the past month and it was more of a help request that would have been better suited to the i18n ml | 18:29 |
| *** bit_lySLH2uSZHed has joined #openstack-infra | 18:30 | |
| *** bit_lySLH2uSZHed has quit IRC | 18:31 | |
| *** bit_lySLH2uSZHed has joined #openstack-infra | 18:34 | |
| *** wolverineav has joined #openstack-infra | 18:35 | |
| *** bit_lySLH2uSZHed has left #openstack-infra | 18:35 | |
| *** wolverineav has quit IRC | 18:38 | |
| openstackgerrit | Merged openstack-infra/tripleo-ci master: record the size of the tripleo logs directory https://review.openstack.org/481306 | 18:38 |
| *** wolverineav has joined #openstack-infra | 18:38 | |
| *** chlong_ has quit IRC | 18:40 | |
| openstackgerrit | wes hayutin proposed openstack-infra/tripleo-ci master: calculate and record the size of the log files for tripleo-ci https://review.openstack.org/481727 | 18:40 |
| *** amotoki_away is now known as amotoki | 18:41 | |
| openstackgerrit | Merged openstack-infra/shade master: De-client-ify Domain List https://review.openstack.org/478322 | 18:45 |
| openstackgerrit | Merged openstack-infra/shade master: De-client-ify Domain Get https://review.openstack.org/478323 | 18:45 |
| openstackgerrit | Merged openstack-infra/shade master: De-client-ify Domain Search https://review.openstack.org/478324 | 18:45 |
| *** bit_lySLH2uSZHed has joined #openstack-infra | 18:46 | |
| *** bit_lySLH2uSZHed has quit IRC | 18:47 | |
| EmilienM | clarkb: if you have some time before the weekend, https://review.openstack.org/#/c/481692/ would unblock puppet openstack gate, thanks | 18:47 |
| *** slaweq has quit IRC | 18:49 | |
| openstackgerrit | Merged openstack-infra/shade master: Use the right variable name in userdata encoding https://review.openstack.org/480592 | 18:49 |
| *** slaweq has joined #openstack-infra | 18:49 | |
| *** amotoki is now known as amotoki_away | 18:51 | |
| openstackgerrit | wes hayutin proposed openstack-infra/tripleo-ci master: reduce the log size for undercloud-containers https://review.openstack.org/481731 | 18:52 |
| *** slaweq has quit IRC | 18:54 | |
| clarkb | EmilienM: in https://review.openstack.org/#/c/481692/2/zuul/layout.yaml puppet-midenet removes puppet 3.6 but 3.6 is kept elsewhere is that intentional? | 18:54 |
| *** Sukhdev has joined #openstack-infra | 18:58 | |
| * AJaeger is back and reading backscroll now... | 18:59 | |
| AJaeger | clarkb: regarding https://review.openstack.org/#/c/481430/5/jenkins/jobs/projects.yaml - I kept the branches for projects that did not retire mitaka. jhesketh linked to a github gist with a list and that's the one I used. | 19:00 |
| EmilienM | clarkb: I think it wasn't, let me fix that | 19:00 |
| *** yland has joined #openstack-infra | 19:01 | |
| *** bit_lySLH2uSZHed has joined #openstack-infra | 19:02 | |
| *** bit_lySLH2uSZHed has left #openstack-infra | 19:03 | |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: mitaka-eol: Simplify zuul branch conditions https://review.openstack.org/481567 | 19:05 |
| AJaeger | clarkb: addressed your comment - thanks for spotting that ^ | 19:06 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: mitaka-eol: Remove trusty docs, linters for ansible https://review.openstack.org/481580 | 19:06 |
| *** jamesden_ has joined #openstack-infra | 19:06 | |
| *** jamesdenton has quit IRC | 19:07 | |
| openstackgerrit | Emilien Macchi proposed openstack-infra/project-config master: puppet: remove unit 3.3, 3.4, 3.7 and 4.0 https://review.openstack.org/481692 | 19:07 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Remove base tox jobs and rename tox-docs https://review.openstack.org/481327 | 19:08 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Rename .zuul.yaml to zuul.yaml https://review.openstack.org/481328 | 19:08 |
| openstackgerrit | Dirk Mueller proposed openstack/diskimage-builder master: opensuse-minimal: install glibc-locale https://review.openstack.org/473959 | 19:09 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: mitaka-eol: Remove trusty docs, linters for ansible https://review.openstack.org/481580 | 19:09 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Duplicate the zuul-tox jobs to just be named tox https://review.openstack.org/481323 | 19:09 |
| AJaeger | clarkb: using ansible-role now as suggested ^ | 19:09 |
| AJaeger | argh, missed one place | 19:09 |
| EmilienM | clarkb: done, let's see how it works | 19:09 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Switch from zuul-tox to tox https://review.openstack.org/481329 | 19:09 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/project-config master: mitaka-eol: Remove trusty docs, linters for ansible https://review.openstack.org/481580 | 19:10 |
| AJaeger | clarkb, fungi, so 481580 and 481567 are updated, thanks for review. | 19:13 |
| *** camunoz has quit IRC | 19:14 | |
| *** e0ne has quit IRC | 19:14 | |
| AJaeger | I think we only need one final go over zuul/layout - but if any of you wants to do it, I would be happy. | 19:15 |
| AJaeger | EmilienM: zuul is happy with your change | 19:16 |
| * AJaeger calls it a weekend and waves good night | 19:16 | |
| *** bit_lySLH2uSZHed has joined #openstack-infra | 19:22 | |
| *** bit_lySLH2uSZHed has quit IRC | 19:22 | |
| *** camunoz has joined #openstack-infra | 19:22 | |
| *** florianf has quit IRC | 19:32 | |
| *** Apoorva has quit IRC | 19:34 | |
| *** yamamoto has quit IRC | 19:34 | |
| openstackgerrit | Merged openstack-infra/shade master: De-client-ify User Create https://review.openstack.org/474397 | 19:35 |
| openstackgerrit | wes hayutin proposed openstack-infra/tripleo-ci master: calculate and record the size of the log files for tripleo-ci https://review.openstack.org/481727 | 19:40 |
| *** amotoki_away is now known as amotoki | 19:42 | |
| *** dtantsur has quit IRC | 19:45 | |
| *** sdague has joined #openstack-infra | 19:46 | |
| *** bobh has quit IRC | 19:47 | |
| *** ihrachys_ has quit IRC | 19:47 | |
| fungi | it seems the infra-root mailbox has swallowed my afternoon. i've given up trying to sort through it all with the webui and am manually classifying and filtering things with mutt to get caught up on the ~8k message backlog there | 19:49 |
| *** bobh has joined #openstack-infra | 19:49 | |
| *** slaweq has joined #openstack-infra | 19:49 | |
| *** slaweq_ has joined #openstack-infra | 19:50 | |
| *** jamesden_ has quit IRC | 19:50 | |
| *** dtantsur has joined #openstack-infra | 19:50 | |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Consume tox instead of zuul-tox https://review.openstack.org/481326 | 19:50 |
| fungi | some 75% were review spam from gerrit | 19:50 |
| fungi | a bunch were release job failure notifications which i tracked back to jenkins@o.o being subscribed to that ml so it could post there, but didn't need to actually be set to receive messages from the list | 19:52 |
| *** amotoki is now known as amotoki_away | 19:52 | |
| fungi | and the bulk of the remainder were ovh payment reminders | 19:52 |
| *** slaweq has quit IRC | 19:53 | |
| *** esberglu has quit IRC | 19:53 | |
| *** esberglu has joined #openstack-infra | 19:54 | |
| openstackgerrit | Dirk Mueller proposed openstack/diskimage-builder master: opensuse-minimal: install glibc-locale https://review.openstack.org/473959 | 19:54 |
| *** dtantsur has quit IRC | 19:57 | |
| *** tjones has joined #openstack-infra | 19:57 | |
| *** tjones has quit IRC | 19:58 | |
| *** tjones has joined #openstack-infra | 19:58 | |
| *** esberglu has quit IRC | 19:58 | |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Handle GitHub comment reviews carefully https://review.openstack.org/481314 | 19:58 |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Remove unnecessary loop in github test https://review.openstack.org/481315 | 19:59 |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Ensure github change body is not a NoneType https://review.openstack.org/481681 | 20:00 |
| EmilienM | AJaeger: thx for the review | 20:01 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Pass result data back from ansible https://review.openstack.org/479442 | 20:02 |
| *** dtantsur has joined #openstack-infra | 20:02 | |
| openstackgerrit | wes hayutin proposed openstack-infra/tripleo-ci master: WIP: add log instructions as a footer in tripleo-ci https://review.openstack.org/481746 | 20:02 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Pass result data back from ansible https://review.openstack.org/479442 | 20:04 |
| EmilienM | weshay, clarkb: undercloud-container takes 86M: http://logs.openstack.org/31/481731/1/check/gate-tripleo-ci-centos-7-undercloud-containers/5f14b20/logs/log-size.txt.gz | 20:04 |
| *** zara_the1lemur__ is now known as zara_the_lemur__ | 20:04 | |
| *** jamesdenton has joined #openstack-infra | 20:05 | |
| fungi | which looks like is before some things get compressed/filtered since that one's reported at 78mib on the filesystem for logs.o.o | 20:06 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Pass result data back from ansible https://review.openstack.org/479442 | 20:07 |
| weshay | EmilienM, fungi what size are we shooting for? | 20:11 |
| *** jamesdenton has quit IRC | 20:11 | |
| weshay | how big are devstack logs for example? | 20:11 |
| fungi | i'll nab a sample | 20:12 |
| *** jamesdenton has joined #openstack-infra | 20:12 | |
| fungi | weshay: http://logs.openstack.org/82/474282/2/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/8af5c32/ just ran a couple hours ago and takes up 32mib on disk | 20:15 |
| fungi | so that's a little less than half what that gate-tripleo-ci-centos-7-undercloud-containers run saved | 20:15 |
| fungi | http://logs.openstack.org/82/474282/2/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv/0c1c1a1/ takes up slightly more at 35mib on disk | 20:17 |
| weshay | fungi, ok.. that is where are non-containerized jobs are at as well http://logs.openstack.org/06/481306/2/gate/gate-tripleo-ci-centos-7-nonha-multinode-oooq/b74ffa1/logs/log-size.txt.gz | 20:18 |
| *** esberglu has joined #openstack-infra | 20:18 | |
| weshay | fungi, I'm working to fix that one job now | 20:18 |
| fungi | awesome--thanks! | 20:19 |
| fungi | hah, going through the remaining unclassified messages in the infra-root inbox, i found a message from me inviting the release bot to join us in barcelona | 20:21 |
| *** wolverineav has quit IRC | 20:22 | |
| *** burgerk_ has joined #openstack-infra | 20:22 | |
| *** wolverineav has joined #openstack-infra | 20:22 | |
| *** kgiusti has left #openstack-infra | 20:24 | |
| *** burgerk has quit IRC | 20:25 | |
| jeblair | fungi: pretty sure i saw you in a robot costume | 20:25 |
| fungi | i swear it wasn't me | 20:26 |
| EmilienM | fungi: if you have time before the weekend, https://review.openstack.org/#/c/481692/ thanks a lot | 20:27 |
| openstackgerrit | Dirk Mueller proposed openstack-infra/project-config master: Build openSUSE 42.3 nodepool images https://review.openstack.org/481752 | 20:27 |
| *** markvoelker has quit IRC | 20:27 | |
| *** markvoelker has joined #openstack-infra | 20:27 | |
| *** wolverineav has quit IRC | 20:27 | |
| *** burgerk_ has quit IRC | 20:28 | |
| *** bobh has quit IRC | 20:29 | |
| fungi | yay, inbox zero reached on infra-root mailbox. now i'm going to see if i can purge the >30k messages from gerrit in the review mailbox there since the webui was falling over itself failing to do so earlier | 20:29 |
| jeblair | a store 1:* +FLAGS \Deleted | 20:31 |
| jeblair | b expunge | 20:31 |
| jeblair | fungi: ^ | 20:32 |
| openstackgerrit | Dirk Mueller proposed openstack-infra/system-config master: Prepare nodepool-builder logging config for openSUSE 42.3 https://review.openstack.org/468117 | 20:32 |
| *** bobh has joined #openstack-infra | 20:32 | |
| fungi | jeblair: luckily it didn't come to that | 20:33 |
| fungi | apparently the webui _did_ purge the contents there, it was simply failing to update its status afterward to indicate that | 20:33 |
| jeblair | aww, sorry you missed the joy of chatting with an imap server | 20:34 |
| fungi | i do actually miss the days of being a human imap client | 20:34 |
| jeblair | they're almost as friendly as smtp servers (which are sticklers for courtesy, you should always say helo to them) | 20:34 |
| fungi | and bye! | 20:34 |
| *** Apoorva has joined #openstack-infra | 20:35 | |
| *** yamamoto has joined #openstack-infra | 20:35 | |
| openstackgerrit | Dirk Mueller proposed openstack-infra/system-config master: Bump puppet-ntp to 4.2.0 https://review.openstack.org/447058 | 20:36 |
| *** bobh has quit IRC | 20:37 | |
| fungi | no, wait... i misremembered. sendmail used to _respond_ bye when you gave the quit command. it's the ftp protocol which defines bye as a synonym for its quit command | 20:38 |
| *** ijw has joined #openstack-infra | 20:38 | |
| *** yamamoto has quit IRC | 20:39 | |
| openstackgerrit | Merged openstack-infra/project-config master: puppet: remove unit 3.3, 3.4, 3.7 and 4.0 https://review.openstack.org/481692 | 20:39 |
| jeblair | i started saying 'quit' when leaving parties and just never got invited back | 20:39 |
| *** Apoorva has quit IRC | 20:39 | |
| fungi | still, better than ^D i guess | 20:40 |
| *** mriedem1 has joined #openstack-infra | 20:40 | |
| EmilienM | fungi: thx a lot | 20:41 |
| EmilienM | mwhahaha: Puppet OpenStack CI job should be like before now, let me know if you still see some weird things | 20:41 |
| mwhahaha | yea i'm checking it out now thanks | 20:41 |
| fungi | EmilienM: no problem. trying to get caught up on the remaining open eol changes | 20:41 |
| EmilienM | fungi: good luck | 20:42 |
| fungi | mwhahaha: give it 15-30 minutes before rechecking | 20:42 |
| *** sambetts|afk has quit IRC | 20:42 | |
| mwhahaha | k | 20:42 |
| EmilienM | mwhahaha: maybe we could send DNM patches in all branches and see how it goes (in 15-30m) | 20:42 |
| *** priteau has quit IRC | 20:42 | |
| fungi | our puppet automation still needs to hand that configuration update off to zuul | 20:42 |
| *** mriedem has quit IRC | 20:42 | |
| *** amotoki_away is now known as amotoki | 20:43 | |
| *** sambetts has joined #openstack-infra | 20:43 | |
| *** bobh has joined #openstack-infra | 20:44 | |
| *** Apoorva has joined #openstack-infra | 20:45 | |
| *** ijw has quit IRC | 20:45 | |
| *** ijw has joined #openstack-infra | 20:46 | |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Remove the transitionary zuul-tox jobs https://review.openstack.org/481324 | 20:47 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Copy in run-docs role as zuul-run-docs https://review.openstack.org/481332 | 20:47 |
| openstackgerrit | wes hayutin proposed openstack-infra/tripleo-ci master: WIP: add log instructions as a footer in tripleo-ci https://review.openstack.org/481746 | 20:48 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-roles master: Remove unneeded roles https://review.openstack.org/481334 | 20:49 |
| openstackgerrit | Merged openstack-infra/openstack-zuul-jobs master: Rename zuul-run-docs to run-docs https://review.openstack.org/481333 | 20:49 |
| *** yland has quit IRC | 20:49 | |
| *** dprince has quit IRC | 20:49 | |
| *** yland has joined #openstack-infra | 20:50 | |
| *** yland is now known as Guest24751 | 20:50 | |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Rename zuul-revoke-sudo to revoke-sudo https://review.openstack.org/481331 | 20:50 |
| mwhahaha | EmilienM: we have a bunch of open stuff we can recheck | 20:51 |
| *** Guest24751 has quit IRC | 20:52 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Pass result data back from ansible https://review.openstack.org/479442 | 20:52 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Support relative urls in success-url https://review.openstack.org/481342 | 20:52 |
| *** amotoki is now known as amotoki_away | 20:52 | |
| *** Guest24751 has joined #openstack-infra | 20:53 | |
| *** mriedem1 is now known as mriedem | 20:55 | |
| EmilienM | mwhahaha: good | 20:57 |
| *** ijw has quit IRC | 20:57 | |
| *** rbrndt has quit IRC | 20:59 | |
| *** ijw has joined #openstack-infra | 20:59 | |
| *** trown is now known as trown|outtypewww | 21:00 | |
| openstackgerrit | Merged openstack-infra/project-config master: mitaka-eol: Simplify zuul branch conditions https://review.openstack.org/481567 | 21:04 |
| *** ijw has quit IRC | 21:04 | |
| *** camunoz has quit IRC | 21:07 | |
| *** mat128 has quit IRC | 21:08 | |
| *** yamamoto has joined #openstack-infra | 21:08 | |
| mrhillsman | if anyone has time before eod/weekend, couple things, uc irc channel - https://review.openstack.org/#/c/479460/ and https://review.openstack.org/#/c/479461/ and SIG ML https://review.openstack.org/#/c/481013/ - apologies if this is not proper etiquette; be gentle | 21:09 |
| openstackgerrit | Merged openstack-infra/project-config master: mitaka-eol: Remove trusty docs, linters for ansible https://review.openstack.org/481580 | 21:10 |
| fungi | mrhillsman: ooh, yes this is probably a good and nondisruptive time to approve the meetbot addition for channel logging | 21:15 |
| fungi | thanks for the reminder | 21:15 |
| *** thorst has quit IRC | 21:16 | |
| * mrhillsman not sure if fungi is being sarcastic :) | 21:16 | |
| mrhillsman | if not, you're welcome :) | 21:17 |
| mrhillsman | if so, gentle man, dang | 21:17 |
| *** yamamoto has quit IRC | 21:17 | |
| fungi | mrhillsman: not sarcastic | 21:20 |
| mrhillsman | whew :sweat_smile: | 21:20 |
| fungi | since channel logging is implemented by the meetbot, and the meetbot needs a restart to reread its configuration, and it doesn't persist state over restarts, merging channel logging additions/changes when meetings are going on causes meeting logs to get truncated | 21:20 |
| fungi | but late on fridays or over the weekend is fine since nobody wants to hold meetings now | 21:21 |
| mrhillsman | ah, and no meetings right now :) | 21:21 |
| fungi | mrhillsman: they look good to me. infra-root: project-config-core: any takers to approve 479461(project-config), 479460(system-config) and 481013(system-config) to set up irc bots for teh user committee and a sigs ml for the uc/tc? | 21:28 |
| mrhillsman | woohoo :) | 21:28 |
| *** jesusaur has quit IRC | 21:28 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add work_dir option to tox role https://review.openstack.org/481762 | 21:29 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add a job that runs zuul's py35 unit tests https://review.openstack.org/481763 | 21:29 |
| *** gordc has quit IRC | 21:29 | |
| mordred | fungi: on it | 21:30 |
| *** jesusaur has joined #openstack-infra | 21:31 | |
| mordred | fungi: also - if you want to see a crazy new thing - https://review.openstack.org/481763 is a fun patch :) | 21:32 |
| mordred | I mean - it failed - but meh | 21:32 |
| fungi | failure is inevitable | 21:32 |
| mordred | aha! I see the error ... tricky tricky | 21:33 |
| fungi | that's a nifty hack | 21:35 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add a job that runs zuul's py35 unit tests https://review.openstack.org/481763 | 21:35 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add work_dir option to tox and bindep roles https://review.openstack.org/481762 | 21:35 |
| fungi | (which i expect to stop thinking of as a hack by this time next year) | 21:35 |
| *** thorst has joined #openstack-infra | 21:38 | |
| *** thorst has quit IRC | 21:38 | |
| openstackgerrit | Merged openstack-infra/project-config master: Add UC IRC channel to accessbot Add governance-uc watching to UC IRC channel https://review.openstack.org/479461 | 21:39 |
| openstackgerrit | Merged openstack-infra/system-config master: Add openstack-uc IRC channel https://review.openstack.org/479460 | 21:40 |
| *** dave-mccowan has quit IRC | 21:41 | |
| openstackgerrit | Merged openstack-infra/system-config master: Add mailing list for OpenStack SIGs https://review.openstack.org/481013 | 21:41 |
| jeblair | mordred, fungi: i get pretty excited when i think about what that job looks like in v2. :) | 21:42 |
| *** amotoki_away is now known as amotoki | 21:43 | |
| *** ldnunes has quit IRC | 21:44 | |
| *** yamamoto has joined #openstack-infra | 21:45 | |
| openstackgerrit | Gabriele Cerami proposed openstack-infra/tripleo-ci master: Add settings for ovb in rdocloud https://review.openstack.org/480950 | 21:46 |
| *** Guest24751 has quit IRC | 21:46 | |
| *** Guest24751 has joined #openstack-infra | 21:46 | |
| *** openstackgerrit has quit IRC | 21:47 | |
| *** thorst has joined #openstack-infra | 21:50 | |
| *** yamamoto has quit IRC | 21:51 | |
| *** ijw has joined #openstack-infra | 21:51 | |
| *** priteau has joined #openstack-infra | 21:52 | |
| *** esberglu has quit IRC | 21:52 | |
| *** rbrndt has joined #openstack-infra | 21:52 | |
| *** esberglu has joined #openstack-infra | 21:52 | |
| *** amotoki is now known as amotoki_away | 21:53 | |
| *** priteau has quit IRC | 21:56 | |
| *** openstackstatus has quit IRC | 21:56 | |
| *** openstack has joined #openstack-infra | 21:58 | |
| *** openstackstatus has joined #openstack-infra | 21:58 | |
| *** ChanServ sets mode: +v openstackstatus | 21:58 | |
| mordred | fungi: I've only been doing v3 jobs for a couple of weeks now - but i'm happy to report that I'm already thinking in v3 terms as I work | 21:58 |
| *** yland has joined #openstack-infra | 21:58 | |
| openstackgerrit | Gabriele Cerami proposed openstack-infra/tripleo-ci master: WIP: containers periodic test https://review.openstack.org/475747 | 21:59 |
| *** yland is now known as Guest13991 | 21:59 | |
| *** hasharAway has quit IRC | 22:00 | |
| * clarkb got got totally nerd sniped today. But memory seems to check out so firefox bug? | 22:00 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add a job that runs zuul's py35 unit tests https://review.openstack.org/481763 | 22:02 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add zuul_work_dir default to one-project roles https://review.openstack.org/481762 | 22:02 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Permit config shadowing https://review.openstack.org/479084 | 22:03 |
| *** apetrich has quit IRC | 22:05 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add a job that runs zuul's py35 unit tests https://review.openstack.org/481763 | 22:05 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add zuul_work_dir default to one-project roles https://review.openstack.org/481762 | 22:05 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Delete run-bindep role https://review.openstack.org/481770 | 22:05 |
| *** apetrich has joined #openstack-infra | 22:05 | |
| *** felipemonteiro__ has quit IRC | 22:07 | |
| *** claudiub has quit IRC | 22:08 | |
| *** zzzeek_ has joined #openstack-infra | 22:08 | |
| *** zzzeek_ has quit IRC | 22:09 | |
| *** rhallisey has quit IRC | 22:12 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Delete run-bindep role https://review.openstack.org/481770 | 22:13 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add a job that runs zuul's py35 unit tests https://review.openstack.org/481763 | 22:13 |
| *** zzzeek_ has joined #openstack-infra | 22:14 | |
| *** zzzeek_ has quit IRC | 22:14 | |
| *** zzzeek_ has joined #openstack-infra | 22:14 | |
| *** zzzeek_ has quit IRC | 22:15 | |
| mordred | clarkb: you may also find https://review.openstack.org/481763 fun - it's using almost all of the new v3 features | 22:16 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul-jobs master: DNM: test per-build ssh key https://review.openstack.org/481772 | 22:17 |
| clarkb | is required projects going to clone that repo to the root of the job? | 22:17 |
| clarkb | iirc those zuul-jobs still run old style with expectation that tox can run out of pwd? | 22:18 |
| clarkb | s/root of the job/root of the wherever the next things expects to be/ | 22:18 |
| mordred | all repos are cloned into src/{{ canonical_project }} | 22:18 |
| mordred | but these have already been updated for all of that | 22:18 |
| clarkb | oh thats the zuul_work_dir? | 22:18 |
| mordred | yah - in the previous patch I added that as a default value on the tox roles | 22:18 |
| clarkb | ya thats the piece I thought was missing neat | 22:19 |
| mordred | so it defaults to "src/{{ zuul.project.canonical_name }}" | 22:19 |
| *** zzzeek_ has joined #openstack-infra | 22:19 | |
| mordred | we're getting somewhere :) | 22:19 |
| openstackgerrit | James E. Blair proposed openstack-infra/project-config master: Add per-build ssh key to base job https://review.openstack.org/481773 | 22:19 |
| *** askb has joined #openstack-infra | 22:21 | |
| clarkb | if I can get reviews on https://review.openstack.org/#/c/453323/ and https://review.openstack.org/#/c/478216/ those are necessary to turn puppet back on on review-dev and start testing gerrit 2.13 a bit more. I wouldn't approve right now though since its end of week. I can approve when votes accumulate (hopefully monday) | 22:22 |
| jeblair | mordred, clarkb: ^ that's ready to land. | 22:22 |
| clarkb | jeblair: curious why you need to remove the key at the end? | 22:23 |
| *** zzzeek_ has quit IRC | 22:24 | |
| *** jamesdenton has quit IRC | 22:24 | |
| mordred | clarkb: on them | 22:24 |
| jeblair | clarkb: it's not necessary for nodepool nodes, but it will clean things up if we run it on a hypothetical static node. (of course, we might want to avoid doing it at all) | 22:24 |
| jeblair | clarkb, mordred: that does make me think of something though -- if that fails, our upload logs playbook will fail, right? do we need some more ansible there to make sure that doesn't happen? | 22:25 |
| *** jamesdenton has joined #openstack-infra | 22:25 | |
| clarkb | jeblair: but if removed on a static node you'd have no way of getting in at that point? | 22:25 |
| clarkb | jeblair: yes I think that is correct you can wrap it in a block that says meh if it fails | 22:25 |
| jeblair | clarkb: the key used to log into the server is still in the server's authorized keys | 22:25 |
| *** bobh has quit IRC | 22:26 | |
| clarkb | jeblair: oh! got it | 22:26 |
| clarkb | its just that the job only has access to the new key | 22:26 |
| jeblair | clarkb: the only thing we do related to that is have the local ssh agent forget that key once the new one is installed | 22:26 |
| mordred | jeblair: which ... oh - if the remove-key role fails | 22:26 |
| jeblair | mordred: right | 22:26 |
| mordred | jeblair: yah - we should do something about that | 22:26 |
| clarkb | ignore_errors: yes | 22:26 |
| clarkb | does that work on roles? | 22:26 |
| mordred | no - but works on tasks - so we could set that on the task in the remove_ssh_key role | 22:28 |
| *** zzzeek_ has joined #openstack-infra | 22:29 | |
| *** jamesdenton has quit IRC | 22:29 | |
| mordred | also - woo https://review.openstack.org/#/c/481763/ actually worked! :) | 22:30 |
| *** zzzeek_ has quit IRC | 22:30 | |
| clarkb | also hunting down that role is :/ my first impression of git repoing all the things is not so great | 22:30 |
| clarkb | I kinda like how dib ends up carrying a lot of that stuff internally because it makes grep super easy :) | 22:30 |
| *** zzzeek_ has joined #openstack-infra | 22:31 | |
| *** jamesdenton has joined #openstack-infra | 22:31 | |
| mordred | clarkb: there's a few cases for us there stuff is spanning repos - but for the most part the roles should wind up in the same place as the jobs | 22:31 |
| jeblair | clarkb: yeah, both the base job and that role are going to end up in zuul-roles | 22:31 |
| jeblair | we just can't land that quite yet | 22:31 |
| clarkb | mordred: well in this case its in openstack roles but it seems like this would be useful in the base zuul roles? | 22:31 |
| clarkb | jeblair: gotcha | 22:31 |
| jeblair | that's waiting on https://review.openstack.org/479084 | 22:31 |
| mordred | clarkb: yah - there's a little extra awkward with that - I just finished collapsing a bunch of stuff into zuul-jobs for a similar reason and it's MUCH more pleasant now | 22:32 |
| jeblair | which lets us put 'base' in zuul-jobs, but gives us the escape valve of being able to override base in project-config should we need. (and we probably will.) | 22:32 |
| clarkb | as for thinking about when the role would fail. If the ansible_ssh_user doesn't have permissions to the file (which could happen) or if the key is already removed (which seems like would cause a failure earlier in the job? | 22:32 |
| clarkb | oh except ssh will fail if the ssh user doesn't own the file either | 22:32 |
| jeblair | clarkb: yeah, i'm thinking like if the node went belly up right at the end of the job, or even if the job hosed the node, or the authorized_keys file. | 22:33 |
| clarkb | jeblair: in which case we should have proper errors before we ignore this one | 22:33 |
| clarkb | so should be safe to ignore errors there on the off chance there is a less catastrophic failure | 22:33 |
| jeblair | clarkb: yep. and we just need to ignore this to make sure we get logs uploaded. | 22:33 |
| *** zzzeek- has joined #openstack-infra | 22:33 | |
| mordred | jeblair: it kinda seems to me like this should really be two playbooks so that the zuul "run all the post playbooks" takes care of it - but that we get to see that something failed | 22:34 |
| jeblair | mordred: that works; i momentarily forgot we can have more than one in the same job. :) | 22:35 |
| jeblair | i'll rework the patch to do that instead of ignore_error on the role | 22:35 |
| *** zzzeek_ has quit IRC | 22:35 | |
| mordred | tbh- I did too - I was thinking I was suggesting implementing a new feature :) | 22:35 |
| mordred | I guess we maybe thought about this already | 22:35 |
| jeblair | someone did | 22:35 |
| mordred | thank god | 22:36 |
| openstackgerrit | Merged openstack-infra/puppet-gerrit master: Support setting accountPatchReviewDb.url in config https://review.openstack.org/478216 | 22:36 |
| openstackgerrit | Morgan Fainberg proposed openstack-infra/shade master: Make sure we pass propert dicts to validate https://review.openstack.org/478582 | 22:36 |
| openstackgerrit | James E. Blair proposed openstack-infra/project-config master: Add per-build ssh key to base job https://review.openstack.org/481773 | 22:37 |
| openstackgerrit | Morgan Fainberg proposed openstack-infra/shade master: Make sure we don't fail open on bad input to validate https://review.openstack.org/478589 | 22:37 |
| jeblair | mordred, clarkb: 481773 does that | 22:37 |
| mordred | jeblair: woot! | 22:40 |
| *** zzzeek- has quit IRC | 22:41 | |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Add zuul_work_dir default to one-project roles https://review.openstack.org/481762 | 22:41 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Delete run-bindep role https://review.openstack.org/481770 | 22:41 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Add a job that runs zuul's py35 unit tests https://review.openstack.org/481763 | 22:41 |
| *** leakypipes has quit IRC | 22:43 | |
| *** amotoki_away is now known as amotoki | 22:44 | |
| *** dhinesh has quit IRC | 22:44 | |
| *** sdague has quit IRC | 22:47 | |
| *** slaweq_ has quit IRC | 22:47 | |
| *** slaweq has joined #openstack-infra | 22:48 | |
| *** askb has quit IRC | 22:49 | |
| *** rbrndt has quit IRC | 22:49 | |
| *** rbrndt has joined #openstack-infra | 22:49 | |
| *** rbrndt has quit IRC | 22:49 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Permit config shadowing https://review.openstack.org/479084 | 22:52 |
| *** slaweq has quit IRC | 22:53 | |
| *** tosky has quit IRC | 22:53 | |
| *** amotoki is now known as amotoki_away | 22:54 | |
| openstackgerrit | wes hayutin proposed openstack-infra/tripleo-ci master: reduce the log size for undercloud-containers https://review.openstack.org/481731 | 22:55 |
| *** tjones has left #openstack-infra | 22:56 | |
| *** jcoufal has quit IRC | 22:57 | |
| *** andreww has quit IRC | 23:05 | |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Fix test_bubblewrap_leak https://review.openstack.org/481123 | 23:12 |
| *** rlandy has quit IRC | 23:15 | |
| *** Guest13991 has quit IRC | 23:15 | |
| *** Guest13991 has joined #openstack-infra | 23:16 | |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Optionally bind /lib64 https://review.openstack.org/481432 | 23:16 |
| *** zzzeek_ has joined #openstack-infra | 23:17 | |
| *** Guest13991 has quit IRC | 23:18 | |
| *** Guest13991 has joined #openstack-infra | 23:19 | |
| openstackgerrit | Yolande Amate proposed openstack-infra/jenkins-job-builder master: Update trigger_project to use convert_mapping_to_xml() https://review.openstack.org/481776 | 23:19 |
| *** zzzeek_ has quit IRC | 23:21 | |
| *** zzzeek_ has joined #openstack-infra | 23:21 | |
| *** Apoorva has quit IRC | 23:26 | |
| openstackgerrit | Merged openstack-infra/jenkins-job-builder master: Fix workspace cleanup code https://review.openstack.org/481701 | 23:29 |
| *** amotoki_away is now known as amotoki | 23:45 | |
| *** Apoorva has joined #openstack-infra | 23:47 | |
| *** ijw has quit IRC | 23:52 | |
| *** ijw has joined #openstack-infra | 23:52 | |
| *** amotoki is now known as amotoki_away | 23:55 | |
| *** ijw has quit IRC | 23:57 | |
| *** thorst has joined #openstack-infra | 23:57 | |
| *** ijw has joined #openstack-infra | 23:58 | |
| *** Apoorva has quit IRC | 23:58 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!