Monday, 2019-02-18

*** hwoarang has joined #openstack-infra00:01
*** weshay|ruck is now known as weshay_PTO00:28
*** markvoelker has joined #openstack-infra00:33
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: connections: always register the scheduler  https://review.openstack.org/63744400:37
*** rkukura has quit IRC00:39
*** markvoelker has quit IRC01:07
*** wolverineav has joined #openstack-infra01:15
*** ruffian_sheep has joined #openstack-infra01:17
*** wolverineav has quit IRC01:19
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: connections: always register the scheduler  https://review.openstack.org/63744401:20
*** dave-mccowan has joined #openstack-infra01:21
*** wolverineav has joined #openstack-infra01:35
*** wolverineav has quit IRC01:38
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: connections: remove unused getSchema procedure  https://review.openstack.org/63744501:46
*** sdake has joined #openstack-infra01:59
*** yikun has joined #openstack-infra01:59
*** markvoelker has joined #openstack-infra02:04
*** whoami-rajat has joined #openstack-infra02:13
*** sdake has quit IRC02:18
*** sdake has joined #openstack-infra02:22
*** mgoddard has quit IRC02:32
*** mgoddard has joined #openstack-infra02:34
*** markvoelker has quit IRC02:38
*** sdake has quit IRC02:44
*** hwoarang has quit IRC02:49
*** hwoarang has joined #openstack-infra02:51
*** psachin has joined #openstack-infra02:51
*** hwoarang has quit IRC02:56
*** hwoarang has joined #openstack-infra03:00
*** wolverineav has joined #openstack-infra03:05
*** hwoarang has quit IRC03:05
*** hwoarang has joined #openstack-infra03:06
*** wolverineav has quit IRC03:09
*** hongbin has joined #openstack-infra03:10
*** ykarel|away has joined #openstack-infra03:10
*** janki has joined #openstack-infra03:21
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: fixtures: use default _cleanUp procedure  https://review.openstack.org/63744903:23
*** markvoelker has joined #openstack-infra03:34
*** ricolin has joined #openstack-infra03:38
*** udesale has joined #openstack-infra03:44
*** ykarel|away is now known as ykarel04:00
*** markvoelker has quit IRC04:06
*** ramishra has joined #openstack-infra04:07
*** dave-mccowan has quit IRC04:07
*** dave-mccowan has joined #openstack-infra04:09
*** verdurin has quit IRC04:14
*** hwoarang has quit IRC04:15
*** agopi has quit IRC04:15
*** hwoarang has joined #openstack-infra04:17
*** verdurin has joined #openstack-infra04:26
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: config: add playbooks to job.toDict()  https://review.openstack.org/62134304:37
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: Add API endpoint to get frozen jobs  https://review.openstack.org/60707704:37
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: Get executor job params  https://review.openstack.org/60707804:37
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: Separate out executor server from runner  https://review.openstack.org/60707904:37
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: zuul-runner: implement prep-workspace  https://review.openstack.org/60708204:37
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: web: add /connections route  https://review.openstack.org/63170304:37
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: zuul-runner: use connections endpoint for prepare-workspace  https://review.openstack.org/63170404:37
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: zuul-runner: add execute sub-command  https://review.openstack.org/63094404:37
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: zuul-runner: add support for depends-on  https://review.openstack.org/63206404:37
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: zuul-runner: add quick-start integration test  https://review.openstack.org/63570104:37
*** lpetrut has joined #openstack-infra04:43
*** hwoarang has quit IRC04:46
*** hwoarang has joined #openstack-infra04:47
*** ykarel has quit IRC04:58
*** markvoelker has joined #openstack-infra05:03
*** lpetrut has quit IRC05:10
openstackgerritIan Wienand proposed openstack-infra/system-config master: [dnm] letsencrypt rax dns prototype implementation  https://review.openstack.org/63745605:14
*** auristor has quit IRC05:17
*** ykarel has joined #openstack-infra05:19
*** auristor has joined #openstack-infra05:22
*** markvoelker has quit IRC05:37
*** hongbin has quit IRC05:42
*** tkajinam_ has joined #openstack-infra05:54
*** tkajinam has quit IRC05:57
*** jbadiapa has joined #openstack-infra06:02
*** chandankumar is now known as chkumar|ruck06:17
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: amqp: add basic trigger  https://review.openstack.org/63745806:20
*** hwoarang has quit IRC06:23
*** hwoarang has joined #openstack-infra06:29
*** markvoelker has joined #openstack-infra06:34
*** sdake has joined #openstack-infra06:38
*** e0ne has joined #openstack-infra06:41
*** kjackal has quit IRC06:41
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add /connections route  https://review.openstack.org/63170306:46
*** e0ne has quit IRC06:47
*** jtomasek has joined #openstack-infra06:50
*** xek has joined #openstack-infra06:52
*** AJaeger has quit IRC06:55
*** AJaeger has joined #openstack-infra07:01
*** yboaron_ has quit IRC07:02
openstackgerritFelix Schmidt proposed openstack-infra/zuul master: Add new merger job to get role definitions from a repository  https://review.openstack.org/63718107:04
*** markvoelker has quit IRC07:06
*** tkajinam_ has quit IRC07:10
*** slaweq has joined #openstack-infra07:10
*** tkajinam_ has joined #openstack-infra07:12
*** hwoarang has quit IRC07:12
*** tkajinam__ has joined #openstack-infra07:14
*** hwoarang has joined #openstack-infra07:16
openstackgerritIan Wienand proposed openstack-infra/infra-specs master: letsencrypt spec  https://review.openstack.org/58728307:17
*** tkajinam_ has quit IRC07:17
*** aojea has joined #openstack-infra07:19
*** apetrich has joined #openstack-infra07:19
*** kjackal has joined #openstack-infra07:22
*** quiquell|off is now known as quiquell|rover07:23
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: amqp: add basic trigger  https://review.openstack.org/63745807:24
*** jtomasek has quit IRC07:27
*** dpawlik has joined #openstack-infra07:27
*** jtomasek has joined #openstack-infra07:27
*** sdake has quit IRC07:30
*** xek has quit IRC07:33
*** e0ne has joined #openstack-infra07:35
*** quiquell|rover is now known as quiquell|rover|b07:36
*** e0ne has quit IRC07:38
*** pgaxatte has joined #openstack-infra07:40
*** dtantsur|afk is now known as dtantsur07:42
*** pcaruana has joined #openstack-infra07:43
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: amqp: add basic trigger  https://review.openstack.org/63745807:48
*** sshnaidm|off is now known as sshnaidm07:56
*** ykarel is now known as ykarel|lunch07:58
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: amqp: add basic trigger  https://review.openstack.org/63745808:02
*** yboaron_ has joined #openstack-infra08:02
*** markvoelker has joined #openstack-infra08:03
*** kopecmartin|off is now known as kopecmartin08:04
*** wolverineav has joined #openstack-infra08:05
*** quiquell|rover|b is now known as quiquell|rovee08:05
*** quiquell|rovee is now known as quiquell|rover08:05
*** yboaron_ has quit IRC08:06
*** yboaron_ has joined #openstack-infra08:07
*** panda|off is now known as panda08:09
*** ginopc has joined #openstack-infra08:09
*** rascasoft has joined #openstack-infra08:09
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: amqp: add basic trigger  https://review.openstack.org/63745808:12
*** egonzalez has quit IRC08:15
*** rpittau has joined #openstack-infra08:17
*** tkajinam__ has quit IRC08:28
*** gfidente has joined #openstack-infra08:29
*** tosky has joined #openstack-infra08:33
*** gfidente has quit IRC08:36
*** egonzalez has joined #openstack-infra08:36
*** markvoelker has quit IRC08:37
*** xek has joined #openstack-infra08:37
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: amqp: add basic trigger  https://review.openstack.org/63745808:43
*** rossella_s has quit IRC08:46
*** jpena|off is now known as jpena08:52
*** ruffian_sheep has quit IRC08:52
*** jpich has joined #openstack-infra08:57
*** aojea has quit IRC08:57
*** ykarel|lunch is now known as ykarel08:59
*** rossella_s has joined #openstack-infra09:00
*** pcaruana|afk| has joined #openstack-infra09:01
*** pcaruana has quit IRC09:02
*** ramishra has quit IRC09:05
*** e0ne has joined #openstack-infra09:08
*** ramishra has joined #openstack-infra09:09
*** roman_g has joined #openstack-infra09:11
*** ccamacho has joined #openstack-infra09:13
*** sdake has joined #openstack-infra09:15
*** kjackal has quit IRC09:17
*** kjackal has joined #openstack-infra09:18
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: amqp: add basic trigger  https://review.openstack.org/63745809:20
*** gfidente has joined #openstack-infra09:21
*** ramishra_ has joined #openstack-infra09:21
*** ramishra has quit IRC09:23
*** markvoelker has joined #openstack-infra09:34
*** derekh has joined #openstack-infra09:37
*** trident has joined #openstack-infra09:38
openstackgerritSimon Westphahl proposed openstack-infra/zuul master: wip: Jaeger tracing PoC  https://review.openstack.org/63750509:42
openstackgerritSimon Westphahl proposed openstack-infra/nodepool master: wip: Jaeger tracing PoC  https://review.openstack.org/63750609:42
*** electrofelix has joined #openstack-infra09:42
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: amqp: add basic trigger  https://review.openstack.org/63745809:47
openstackgerritTobias Henkel proposed openstack-infra/nodepool master: WIP: Second try to fix single cloud config reload  https://review.openstack.org/60869509:50
*** pcaruana|afk| has quit IRC10:01
*** ramishra_ has quit IRC10:06
*** gfidente has quit IRC10:06
*** pcaruana has joined #openstack-infra10:07
*** markvoelker has quit IRC10:07
*** ramishra has joined #openstack-infra10:09
*** xek has quit IRC10:09
*** ociuhandu has joined #openstack-infra10:14
*** gfidente has joined #openstack-infra10:14
*** xek has joined #openstack-infra10:18
*** ociuhandu has quit IRC10:20
*** xek has quit IRC10:24
*** xek has joined #openstack-infra10:27
*** ociuhandu has joined #openstack-infra10:30
openstackgerritJakub Bielecki proposed openstack-infra/nodepool master: doc bugfix for static provider  https://review.openstack.org/63751810:32
*** xek has quit IRC10:34
*** tobias-urdin has joined #openstack-infra10:40
openstackgerritladjadj proposed openstack-infra/grafyaml master: Add capability to use grafana folderid parameter  https://review.openstack.org/63752010:40
*** cgoncalves has quit IRC10:52
*** cgoncalves has joined #openstack-infra10:56
openstackgerritladjadj proposed openstack-infra/grafyaml master: Add capability to use grafana folderid parameter  https://review.openstack.org/63752010:56
*** wolverineav has quit IRC11:03
*** markvoelker has joined #openstack-infra11:04
openstackgerritFabien Boucher proposed openstack-infra/zuul master: URLTrigger driver time based - artifact change jobs triggering driver  https://review.openstack.org/63556711:05
openstackgerritFabien Boucher proposed openstack-infra/zuul master: URLTrigger driver time based - artifact change jobs triggering driver  https://review.openstack.org/63556711:06
*** luizbag has joined #openstack-infra11:08
openstackgerritFabien Boucher proposed openstack-infra/zuul master: URLTrigger driver time based - artifact change jobs triggering driver  https://review.openstack.org/63556711:13
*** sdake has quit IRC11:16
*** sdake has joined #openstack-infra11:20
*** udesale has quit IRC11:33
*** udesale has joined #openstack-infra11:34
*** markvoelker has quit IRC11:37
*** jpena is now known as jpena|brb11:42
*** janki has quit IRC11:49
*** janki has joined #openstack-infra11:54
*** ginopc has quit IRC12:10
*** apetrich has quit IRC12:11
*** sdake has quit IRC12:12
*** sdake has joined #openstack-infra12:16
*** ginopc has joined #openstack-infra12:20
*** kaiokmo has joined #openstack-infra12:25
*** apetrich has joined #openstack-infra12:25
*** ginopc has quit IRC12:28
*** ginopc has joined #openstack-infra12:29
*** apetrich has quit IRC12:30
*** xek has joined #openstack-infra12:40
*** xek_ has joined #openstack-infra12:47
*** xek has quit IRC12:47
*** apetrich has joined #openstack-infra12:48
*** jcoufal has joined #openstack-infra12:49
*** rlandy has joined #openstack-infra12:54
openstackgerritladjadj proposed openstack-infra/grafyaml master: Add capability to use grafana folderid parameter  https://review.openstack.org/63752012:56
*** xek__ has joined #openstack-infra12:59
*** xek_ has quit IRC13:01
*** rh-jelabarre has joined #openstack-infra13:01
*** smarcet has joined #openstack-infra13:04
*** xek__ has quit IRC13:06
*** xek__ has joined #openstack-infra13:06
*** trown|outtypewww is now known as trown13:06
*** janki has quit IRC13:06
*** eharney has joined #openstack-infra13:20
*** gfidente has quit IRC13:22
openstackgerritladjadj proposed openstack-infra/grafyaml master: Add capability to use grafana folderid parameter  https://review.openstack.org/63752013:22
*** gfidente has joined #openstack-infra13:23
*** priteau has joined #openstack-infra13:26
*** jpena|brb is now known as jpena13:27
*** sdake has quit IRC13:28
*** quiquell|rover is now known as quique|rover|eat13:28
openstackgerritsebastian marcet proposed openstack-infra/system-config master: Updated OpenStackId Production Node  https://review.openstack.org/63754113:31
*** rh-jelabarre has quit IRC13:37
*** rh-jelabarre has joined #openstack-infra13:37
*** kaisers has quit IRC13:42
*** priteau has quit IRC13:45
*** kaisers has joined #openstack-infra13:49
*** fried_rice is now known as efried13:50
*** mriedem has joined #openstack-infra13:50
*** sdake has joined #openstack-infra13:56
openstackgerritladjadj proposed openstack-infra/grafyaml master: Add capability to use grafana folderid parameter  https://review.openstack.org/63752013:58
*** agopi has joined #openstack-infra13:59
*** priteau has joined #openstack-infra14:02
*** yboaron_ has quit IRC14:04
*** yboaron_ has joined #openstack-infra14:04
*** quique|rover|eat is now known as quiquell|rover14:09
*** jroll has quit IRC14:13
*** ykarel is now known as ykarel|away14:14
*** jroll has joined #openstack-infra14:14
*** ykarel|away is now known as ykarel14:15
*** gfidente has quit IRC14:22
*** rkukura has joined #openstack-infra14:23
*** eharney has quit IRC14:23
*** psachin has quit IRC14:28
*** chkumar|ruck is now known as chandankumar14:28
*** gfidente has joined #openstack-infra14:32
*** bnemec-pto is now known as bnemec14:37
*** ricolin_ has joined #openstack-infra14:40
*** ricolin has quit IRC14:42
*** ykarel is now known as ykarel|away14:42
*** rh-jelabarre has quit IRC14:43
*** priteau has quit IRC14:47
*** rh-jelabarre has joined #openstack-infra14:48
*** ykarel|away has quit IRC14:52
*** aojea has joined #openstack-infra14:53
*** kgiusti has joined #openstack-infra14:55
*** aojea has quit IRC14:57
*** dave-mccowan has joined #openstack-infra14:58
*** iurygregory has joined #openstack-infra14:58
*** sdake has quit IRC14:58
*** priteau has joined #openstack-infra14:59
*** janki has joined #openstack-infra15:00
iurygregoryhave anyone saw an error from LIBS_FROM_GIT: http://logs.openstack.org/69/636169/11/check/ironic-tempest-dsvm-ironic-inspector-pike/be70a42/job-output.txt.gz#_2019-02-18_13_27_13_465763 ?15:00
*** rh-jelabarre has quit IRC15:01
*** dave-mccowan has quit IRC15:03
fungiiurygregory: i haven't seen that before, but you might have more luck inquiring in #openstack-qa if you haven't already15:03
iurygregoryfungi, ty =D15:04
fungilooks like according to `pip list` these packages weren't installed from git checkouts: ironic-tempest-plugin ironic-python-agent requirements tempest placement cinder devstack15:05
*** aojea has joined #openstack-infra15:06
iurygregorybut before it shows that they were found strange15:07
iurygregoryLOOP [use-cached-repos : Find locally cached git repos]15:08
fungithat lib_installed_from_git function looks fragile since it's parsing `pip list` stdout15:08
fungiand looking for lines mentioning those packages to contain a comma and a space and then more characters15:08
fungii have a feeling a future pip minor release could break it completely15:09
*** quiquell|rover is now known as quiquell|off15:11
iurygregoryD:15:12
iurygregorythe function is very different in master15:12
*** munimeha1 has joined #openstack-infra15:13
*** ricolin_ has quit IRC15:16
*** ricolin_ has joined #openstack-infra15:17
*** priteau has quit IRC15:18
*** rh-jelabarre has joined #openstack-infra15:19
openstackgerritMerged openstack-infra/zuul master: Make UnsafeTag self registering  https://review.openstack.org/63702315:20
*** yboaron_ has quit IRC15:26
*** rh-jelabarre has quit IRC15:28
*** rh-jelabarre has joined #openstack-infra15:31
*** smarcet has quit IRC15:34
fungiinfra-root: ovh has sent us a notification of a ceph outage which occurred in gra1 on january 27. dmesg does show some i/o errors on /dev/sda from that timeframe but it doesn't look like the rootfs mounted from a partition on it got marked read-only. i'm tempted not to bother rebooting that instance unless we see evidence of a problem, but can certainly be convinced otherwise15:35
*** rh-jelabarre has quit IRC15:36
*** rh-jelabarre has joined #openstack-infra15:36
smcginnisfungi: Maybe worth rebooting now to make sure there's not a problem? Versus a reboot happening somewhere down the line, something going wrong, and not having the context to tie it to this event?15:39
fungimaybe, however it's been over 3 weeks already and this is the first we've even found out it happened15:41
*** udesale has quit IRC15:41
*** e0ne has quit IRC15:42
fungi#status log according to ovh, there was a ceph outage which affected the rootfs for our gra1 mirror there on 2018-01-27 between 11:20 and 16:20 utc15:42
openstackstatusfungi: finished logging15:42
fungithat was a sunday, which is probably why we didn't notice15:43
fungijob volume is usually almost nonexistent for us at that time of week15:43
*** rh-jelabarre has quit IRC15:44
*** rh-jelabarre has joined #openstack-infra15:45
*** sdake has joined #openstack-infra15:45
*** e0ne has joined #openstack-infra15:48
*** ekultails has joined #openstack-infra15:49
*** ykarel|away has joined #openstack-infra15:50
*** rh-jelabarre has quit IRC15:50
*** ykarel|away is now known as ykarel15:51
*** rh-jelabarre has joined #openstack-infra15:56
*** jamesmcarthur has joined #openstack-infra16:01
*** smarcet has joined #openstack-infra16:04
*** janki has quit IRC16:08
*** pcaruana has quit IRC16:10
*** ramishra has quit IRC16:13
*** rh-jelabarre has quit IRC16:13
*** ykarel is now known as ykarel|away16:13
fungiprobably also what caused us to not end up with a broken rootfs after the outage concluded16:16
*** ramishra has joined #openstack-infra16:17
*** kjackal has quit IRC16:18
*** kjackal_v2 has joined #openstack-infra16:18
*** rh-jelabarre has joined #openstack-infra16:18
*** kjackal_v2 has quit IRC16:24
*** kjackal has joined #openstack-infra16:25
openstackgerritJan Kundrát proposed openstack-infra/git-review master: Support usernames that contain '@' and ssh Git URLs  https://review.openstack.org/42870016:26
*** smarcet has quit IRC16:30
*** rh-jelabarre has quit IRC16:30
openstackgerritClark Boylan proposed openstack-infra/system-config master: PBX is running on xenial now, not trusty  https://review.openstack.org/63758016:30
clarkber I should update the inventory file too16:31
* clarkb makes new patchset16:31
openstackgerritClark Boylan proposed openstack-infra/system-config master: PBX is running on xenial now, not trusty  https://review.openstack.org/63758016:33
*** rh-jelabarre has joined #openstack-infra16:33
*** e0ne has quit IRC16:34
clarkb#status log Deleted Trusty pbx.openstack.org (038e80f5-15aa-4f69-8c6c-0f43b3587778) as new Xenial pbx01.opendev.org is up and running16:37
openstackstatusclarkb: finished logging16:37
clarkbinfra-root https://review.openstack.org/637580 is additional cleanup for pbx.o.o removal. DNS is already pointed over to the new stuff so I think we are done once that gets in16:38
*** eharney has joined #openstack-infra16:40
*** ramishra has quit IRC16:42
clarkbinfra-root (and mordred in particular) are we ready to delete the old puppetmaster?16:46
clarkbI think we've kept it around out of caution, but its been a while and I have not logged in or needed the old server in months16:46
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Deploy gitea with docker-compose  https://review.openstack.org/63733016:46
corvusclarkb: 'yes' from me.16:46
corvusclarkb, mordred: when we're all around, let's talk about gitea/k8s/docker.16:47
clarkb++16:48
*** emccormick has quit IRC16:50
*** kjackal has quit IRC16:52
*** pgaxatte has quit IRC16:57
*** panda is now known as panda|off16:58
*** rh-jelabarre has quit IRC16:59
roman_gHello team. Do you know if there was any discussing I could have a look at regarding readthdocs job failures? https://zuul.openstack.org/builds?job_name=trigger-readthedocs-webhook17:00
roman_g*discussion17:00
*** ricolin_ has quit IRC17:00
*** sdake has quit IRC17:00
clarkbroman_g: yes it is a known issue with the rtd api17:00
clarkbianw filed a bug with them. Let me see if I can find it17:01
*** rh-jelabarre has joined #openstack-infra17:02
fungiinfra-root: related, cleanup from the storyboard xenial replacement work is 637388 and the next step (production replacement) for openstackid xenial upgrade is 63754117:02
fungiclarkb: and i'm +2 on old puppetmaster deletion as well17:03
*** aojea has quit IRC17:04
roman_gclarkb: https://github.com/rtfd/readthedocs.org/issues/4986 - this one?17:04
*** sdake has joined #openstack-infra17:05
corvusfungi, clarkb: could you please review https://review.openstack.org/637386 and https://review.openstack.org/637387 -- next steps in docker registry work.17:05
clarkbroman_g: yes that is it17:05
fungicorvus: thanks! on it17:05
roman_gclarkb: Thank you. I've left a comment there, asking if there is anything I can help with.17:06
roman_gianw: Thanks for your work on RTD issue ^^^ .17:06
* clarkb opens all the changes17:10
*** rh-jelabarre has quit IRC17:11
*** rh-jelabarre has joined #openstack-infra17:12
*** kjackal has joined #openstack-infra17:12
clarkbcorvus: one question on https://review.openstack.org/#/c/637387/117:14
clarkbfungi: oh cacti thats a piece I need to update in my pbx cleanup change too17:14
clarkbfungi: reviewing your change is pointing out all the problems with mine :)17:14
*** smarcet has joined #openstack-infra17:15
corvusclarkb: answered17:15
fungiclarkb: yeah, i still need to set up backups too17:15
*** ricolin has joined #openstack-infra17:16
*** rpittau has quit IRC17:16
*** rh-jelabarre has quit IRC17:16
*** ginopc has quit IRC17:19
*** rpittau has joined #openstack-infra17:19
*** rh-jelabarre has joined #openstack-infra17:19
openstackgerritClark Boylan proposed openstack-infra/system-config master: pbx.openstack.org cleanup  https://review.openstack.org/63758017:20
clarkbcorvus: fungi ^ is hopefully a bit more complete now17:20
*** rh-jelabarre has quit IRC17:24
*** rh-jelabarre has joined #openstack-infra17:25
*** rpittau has quit IRC17:26
*** rh-jelabarre has quit IRC17:29
*** rh-jelabarre has joined #openstack-infra17:30
*** smarcet has quit IRC17:32
*** rh-jelabarre has quit IRC17:34
*** rh-jelabarre has joined #openstack-infra17:35
clarkbfungi: re cleaning up the regexes we should probably do a pass through those in a month or two and bulk update things that are missed17:39
*** kjackal has quit IRC17:39
*** smarcet has joined #openstack-infra17:40
openstackgerritMerged openstack-infra/system-config master: Fix htpasswd creation for registry  https://review.openstack.org/63738617:42
fungiyeah, feel free to self-approve that one17:43
fungijust trying to fix them up as i go, if i remember to17:43
*** hwoarang has quit IRC17:47
*** hwoarang has joined #openstack-infra17:47
*** kjackal has joined #openstack-infra17:50
*** jpich has quit IRC17:50
openstackgerritMerged openstack-infra/zuul-jobs master: Update docker image roles  https://review.openstack.org/63738717:51
*** ricolin has quit IRC17:52
openstackgerritMerged openstack-infra/system-config master: Updated OpenStackId Production Node  https://review.openstack.org/63754117:53
*** priteau has joined #openstack-infra17:53
openstackgerritMerged openstack-infra/zuul master: Add spec for multi ansible version support  https://review.openstack.org/62392717:56
*** kopecmartin is now known as kopecmartin|off17:56
*** gtmanfred has quit IRC17:59
*** gtmanfred has joined #openstack-infra18:00
*** trown is now known as trown|lunch18:01
*** derekh has quit IRC18:02
*** Vadmacs has joined #openstack-infra18:06
*** jpena is now known as jpena|off18:09
*** rh-jelabarre has quit IRC18:15
*** rh-jelabarre has joined #openstack-infra18:15
*** ykarel|away has quit IRC18:16
*** ykarel|away has joined #openstack-infra18:17
*** xek__ has quit IRC18:18
openstackgerritMerged openstack-infra/zuul master: web: prevent status update loop in background  https://review.openstack.org/63634318:18
*** xek__ has joined #openstack-infra18:19
*** xek__ has quit IRC18:19
*** xek__ has joined #openstack-infra18:20
*** rh-jelabarre has quit IRC18:20
*** gfidente is now known as gfidente|afk18:20
*** rh-jelabarre has joined #openstack-infra18:20
*** ykarel|away has quit IRC18:22
*** xek__ has quit IRC18:24
*** xek__ has joined #openstack-infra18:24
*** e0ne has joined #openstack-infra18:24
*** ociuhandu_ has joined #openstack-infra18:24
*** ociuhandu has quit IRC18:28
*** ociuhandu_ has quit IRC18:29
*** sdake has quit IRC18:29
*** xek__ has quit IRC18:30
*** sdake_ has joined #openstack-infra18:30
openstackgerritMerged openstack-infra/system-config master: pbx.openstack.org cleanup  https://review.openstack.org/63758018:32
*** smarcet has quit IRC18:36
openstackgerritJan Kundrát proposed openstack-infra/git-review master: Support usernames that contain '@' and ssh Git URLs  https://review.openstack.org/42870018:37
clarkbcorvus: I've got gitea on the infra agenda for tomorrow if we don't get to it before then we can talk then and if we do talk before then we can recap18:38
corvuskk18:38
*** rh-jelabarre has quit IRC18:39
*** rh-jelabarre has joined #openstack-infra18:39
*** ociuhandu has joined #openstack-infra18:40
*** ociuhandu has quit IRC18:45
fungigotta go grab grub18:46
*** dtantsur is now known as dtantsur|afk18:47
openstackgerritJames E. Blair proposed openstack-infra/zuul-jobs master: Load buildset_registry in intermediate pull  https://review.openstack.org/63760318:48
corvusfungi, clarkb: ^ fix for the next registry error18:48
*** rh-jelabarre has quit IRC18:49
*** rh-jelabarre has joined #openstack-infra18:50
openstackgerritMerged openstack-infra/zuul master: connections: always register the scheduler  https://review.openstack.org/63744418:54
*** rh-jelabarre has quit IRC18:54
clarkbcorvus: is there already an artifact at that point?18:55
clarkbcorvus: we run buildset registry in job A then job B runs and starts with pull, I guess the way we tell B about A's registry if via zuul return so this should work18:55
*** rh-jelabarre has joined #openstack-infra18:56
openstackgerritMerged openstack-infra/zuul master: connections: remove unused getSchema procedure  https://review.openstack.org/63744518:58
openstackgerritClark Boylan proposed openstack-infra/zuul master: Rename project to project_name in getPullBySha  https://review.openstack.org/63721819:01
openstackgerritClark Boylan proposed openstack-infra/zuul master: Test GithubShaCache  https://review.openstack.org/63722819:01
*** rh-jelabarre has quit IRC19:05
*** rh-jelabarre has joined #openstack-infra19:06
*** trown|lunch is now known as trown19:06
*** electrofelix has quit IRC19:15
openstackgerritAdam Coldrick proposed openstack-infra/storyboard-webclient master: Remember scroll offset of lanes when refreshing board view  https://review.openstack.org/63760719:15
*** rh-jelabarre has quit IRC19:16
*** rh-jelabarre has joined #openstack-infra19:17
*** jamesmcarthur has quit IRC19:19
openstackgerritDoug Wiegley proposed openstack/gertty master: Add ctrl-v/meta-v bindings for page-up/page-down  https://review.openstack.org/63761019:24
*** Vadmacs has quit IRC19:25
*** priteau has quit IRC19:25
*** rh-jelabarre has quit IRC19:26
*** rh-jelabarre has joined #openstack-infra19:27
*** e0ne has quit IRC19:30
*** smarcet has joined #openstack-infra19:32
*** rh-jelabarre has quit IRC19:32
TheJuliaHas there been any discussion, anywhere... if possibly enabling gophercloud to trigger a job defined in ironic's zuul config?19:32
*** rh-jelabarre has joined #openstack-infra19:32
*** jamesmcarthur has joined #openstack-infra19:33
*** stbenjam has joined #openstack-infra19:33
mordredTheJulia: you mean running an ironic job when someone pushes up a gophercloud change? or just using gophercloud in ironic jobs?19:35
*** hongbin has joined #openstack-infra19:36
mordredTheJulia: openlab are running gophercloud changes, so it should be possible for them to pull in job config from the ironic repo and re-use it - although IIRC in practice there are still a few roadblocks for that to 100% work (due to dependency chains)19:36
*** e0ne has joined #openstack-infra19:37
openstackgerritsebastian marcet proposed openstack-infra/openstackid-resources master: Fixed TimeZoneEntity  https://review.openstack.org/63761219:38
openstackgerritMerged openstack-infra/openstackid-resources master: Fixed TimeZoneEntity  https://review.openstack.org/63761219:39
openstackgerritGorka Eguileor proposed openstack-infra/project-config master: Add cinderlib project  https://review.openstack.org/63761319:40
openstackgerritClark Boylan proposed openstack-infra/zuul master: Switch to LRU + TTL based sha to PR cache  https://review.openstack.org/63761519:40
TheJuliamordred: kind of thinking both. Mainly report back to gophercloud, but I see value in ironic for us to be able to test the support being written in gophercloud for ironic19:40
clarkbfwiw I don't think we should add any github based projects to zuul until the situation in the change I just pushed is addressed, but otherwise we should be able to do both? I think that falls under our third party testing rule and if you have that integration that opens the door to depends on and all that19:42
openstackgerritMerged openstack-infra/zuul-jobs master: Load buildset_registry in intermediate pull  https://review.openstack.org/63760319:43
*** hongbin has quit IRC19:43
TheJuliaclarkb: ack, thinking more long term anyway :)19:43
openstackgerritGorka Eguileor proposed openstack-infra/project-config master: Add cinderlib project  https://review.openstack.org/63761319:49
*** zaneb has quit IRC19:50
*** sdake_ has quit IRC19:54
*** wolverineav has joined #openstack-infra19:58
openstackgerritClark Boylan proposed openstack-infra/zuul master: Switch to LRU based sha to PR cache  https://review.openstack.org/63761519:59
fungiokay, returned from grub-getting so catching back up on registry things20:02
*** mriedem has quit IRC20:03
*** mriedem has joined #openstack-infra20:04
fungiacting as a third party to test gophercloud pull requests don't break ironic sounds awesome20:05
fungibut will check out clarkb'20:05
fungis lru implementation first20:05
*** wolverineav has quit IRC20:06
openstackgerritsebastian marcet proposed openstack-infra/openstackid-resources master: Speakers/Moderators Refactoring  https://review.openstack.org/63762020:09
openstackgerritMerged openstack-infra/openstackid-resources master: Speakers/Moderators Refactoring  https://review.openstack.org/63762020:10
fungicaught up and getting to work launching the new openstackid01.openstack.org node20:12
fungilooks like i need some regex/glob updates first20:12
*** smarcet has quit IRC20:15
fungioh, actually i guess i don't20:15
fungipast me was smarter than present me20:15
fungithis does not bode well for my future20:15
*** wolverineav has joined #openstack-infra20:16
*** luizbag has quit IRC20:19
*** rh-jelabarre has quit IRC20:21
*** rh-jelabarre has joined #openstack-infra20:22
*** whoami-rajat has quit IRC20:23
*** rh-jelabarre has quit IRC20:26
*** rh-jelabarre has joined #openstack-infra20:26
*** eharney has quit IRC20:29
mnaseropenstackid01.openstack.org20:30
mnaseroops misclick20:30
*** rh-jelabarre has quit IRC20:31
*** rh-jelabarre has joined #openstack-infra20:32
fungiit will exist soon (hopefully in minutes) serving the openstackid.org site20:34
*** rh-jelabarre has quit IRC20:37
*** rh-jelabarre has joined #openstack-infra20:37
*** agopi has quit IRC20:37
*** eernst has joined #openstack-infra20:39
ianwroman_g: yes, it's kind of frustrating how long it's gone on :/  however zuul recently merged features to pass secrets "up" I think, which would make it significantly easier to use the private token approach20:41
ianwi should probably escalate that on my todo list ...20:41
openstackgerritIan Wienand proposed openstack-infra/system-config master: [dnm] letsencrypt rax dns prototype implementation  https://review.openstack.org/63745620:55
*** smarcet has joined #openstack-infra20:59
*** rh-jelabarre has quit IRC21:08
*** rh-jelabarre has joined #openstack-infra21:08
*** eernst has quit IRC21:11
*** zaneb has joined #openstack-infra21:12
corvusclarkb: mordred is mostly afk today -- so let's regroup tomorrow morning on gitea21:21
*** rh-jelabarre has quit IRC21:22
*** rh-jelabarre has joined #openstack-infra21:23
openstackgerritJeremy Stanley proposed openstack-infra/system-config master: Add new openstackid01 host to inventory and cacti  https://review.openstack.org/63763321:24
fungismarcet: ^ is the replacement production openstackid server21:24
smarcetfungi: thx u21:24
fungiit'll get more thoroughly puppeted once that change to add it to our inventory merges21:24
*** ijw has joined #openstack-infra21:27
*** rh-jelabarre has quit IRC21:27
*** rh-jelabarre has joined #openstack-infra21:28
*** wolverineav has quit IRC21:29
openstackgerritIan Wienand proposed openstack-infra/infra-specs master: letsencrypt spec  https://review.openstack.org/58728321:30
*** ijw has quit IRC21:32
*** ijw has joined #openstack-infra21:32
*** rh-jelabarre has quit IRC21:32
*** ijw has quit IRC21:32
*** rh-jelabarre has joined #openstack-infra21:33
*** ijw has joined #openstack-infra21:34
*** rh-jelabarre has quit IRC21:34
*** zaneb has quit IRC21:37
*** rcernin has joined #openstack-infra21:40
*** jtomasek has quit IRC21:40
*** eernst has joined #openstack-infra21:43
openstackgerritIan Wienand proposed openstack-infra/system-config master: [dnm] letsencrypt rax dns prototype implementation  https://review.openstack.org/63745621:49
clarkbcorvus: ok21:49
*** eernst has quit IRC21:50
*** smarcet has quit IRC21:52
ianwclarkb / corvus / fungi / anyone : letsencrypt spec is ready for review https://review.openstack.org/587283 (could keep fiddling wording all day).  i'm happy enough with the prototypes that, as written, it's actually implementable21:53
clarkbcool, I got it on the meeting agenda with updated text noting it was updated. I do need to reread it though21:53
clarkbmaybe I should just go ahead and do that21:54
corvusianw: i just finished up reading and posting some comments21:54
*** smarcet has joined #openstack-infra21:56
ianwthanks21:57
*** smarcet has quit IRC21:57
clarkbdoes apache config language have conditional section imports based on files existing or not (trying to understand how setting that up conditionally would look)22:01
corvusclarkb: if not, self-signed would probably be fine?22:01
clarkbhttps://httpd.apache.org/docs/2.4/mod/core.html#iffile TIL22:02
corvusneat :)22:02
*** trown is now known as trown|outtypewww22:02
fungiapache can do *anything*22:02
corvusfungi: well, i read clarkb's question as ending in an implicit "without using mod_perl" :)22:02
corvuswhich is the best way to read most apache questions22:03
* fungi tries to remember to ask apache for stock tips22:03
*** e0ne has quit IRC22:03
*** e0ne has joined #openstack-infra22:04
*** kjackal has quit IRC22:05
fungicorvus: worth mentioning, unless we expect to send packets for an already established tcp socket to a different server, it's okay of each server in the load-balanced pool has a different cert for the same hostname(s)22:05
clarkbfungi: does validating a new cert in letsencrypt update a revocation list for the old certs?22:05
corvusfungi: that is a good point -- i feel certain that's something we can live with for the current git LB setup.  i *suspect* that's also okay for the gitea/k8s setup, but i'm not 100% sure about that, having discovered surprising new behavior there on Friday.  :)22:07
fungiclarkb: i can't say for certain, but i hope not as that would invalidate a whole set of assumptions on how people use x.509 certificates22:07
corvusclarkb: i'm *pretty sure* i remember reading something about how they concluded revocation lists are terrible and don't want to use them, and that's why lifetimes are short.22:08
fungiyeah, that's also a big part of it for letsencrypt in particular22:08
corvusright, they==letsencrypt22:08
fungithough their reasoning was mostly around client implementations generally not implementing standardized (ocsp or similar) crl retrieval mechanisms22:09
*** kgiusti has quit IRC22:09
clarkbya clients don't actually check/update the revocation lists22:09
funginot that the idea of a crl is necessarily bad, just that most client-side implementations have little incentive to do it because it's additional network connections and user-facing delays for some silly security which is almost never necessary22:10
fungi(except when it is, of course)22:10
clarkbre DNS I do like that we can bootstrap things on the cert side without needing a fully deployed instance (or even half deployed in the case of iffile )22:11
fungithere's also the somewhat legitimate argument that a trivial mitm can block crl checks and allow clients they're compromising to continue using revoked certs22:11
clarkbso I boot a new storyboard and before I update dns to point at that server I can have a fully configured host22:11
clarkbthen do the dns switchover for the service itself22:11
clarkb(whereas dns has to point at the new server before the switch for letsencrypt to validate aiui22:12
fungiclarkb: the same could be achieved by starting with a snakeoil cert and configuring certbot to overwrite that22:12
fungi(using http-based validation)22:12
clarkbfungi: no this is a different concern22:12
fungioh, got it, before we have a record for the server in dns at all22:13
clarkbthe way letsencrypt knows you own the domain is by HTTP GETting a known file22:13
clarkbyes22:13
clarkbso in your case of replacing storyboard servers we'd have to bootstrap the cert or take a longer outage22:13
clarkbstill doable, just less nice imo22:13
clarkbwith the DNS validation we can fully bootstrap the server before updating DNS to point at the server22:13
fungithis is a good point. if we want to avoid copying a cert to a replacement server which is not yet reachable at that name, dns-based validation does have an up-side22:14
corvusagreed22:14
corvusi hope someone is writing these rebuttals to my points in review comments on the spec, i don't want to have to argue with myself :)22:16
clarkbI haven't yet but will go do so now22:16
corvusthat leaves me with the question as to whether having this hybrid centralized/distributed approach is worth the extra complexity.  basically, that gives us the ability to use dns validation on ephemeral certs only held on the hosts, but it's fairly complicated.  if we had precious certs, the system could be simpler (as described in the openstack.org section).22:18
fungii can rebut, just wanted to think through a rebuttal out loud first22:20
corvusyes, it's been good.  i just noticed nothing posted yet and wanted to make sure it didn't end here :)22:21
*** sdake has joined #openstack-infra22:26
*** sdake has quit IRC22:27
clarkbfungi: let me know if you want me to add the notes about being able to validate multiple certs for the same domain(s)22:30
fungii just did22:30
ianwcorvus: even with the precious certs held on bridge.o.o, if we stick with dns based validation we still have the swizzle of bridge.o.o having to update the dns entries on the dns server.22:31
ianwso that's boiling down to "does the token come from the remote host, or bridge.o.o"22:31
corvusianw: yeah, i guess i was thinking that having 2 hosts involved (bridge/dns) instead of 3 (bridge/host/dns) would be simpler, but maybe it's not that big of a deal because ansible makes it easy enough?22:33
*** sdake has joined #openstack-infra22:33
ianwcorvus: yeah, I think it's just a "with_inventory_hostnames" -> https://review.openstack.org/#/c/636759/23/playbooks/roles/letsencrypt-install-txt-record/tasks/main.yaml22:33
ianweach host can just set the token in a known var in it's host variables, like -> https://review.openstack.org/#/c/636759/23/playbooks/roles/letsencrypt-request-certs/tasks/main.yaml22:34
corvusianw: i haven't internalized what 636759 is doing yet -- but will it be possible to run a smaller playbook?22:38
corvusie, right now, we're doing everything from base.yaml, but at some point, we need to stop doing that and just have, for example a playbook which just manages the elasticsearch cluster.22:38
corvusbut are those letsencrypt roles something that has to run for every host?  and if so, are they still compatible with a playbook that just manages one or a small set of hosts?22:39
ianwcorvus: so the basic pattern in 636759 is 1) host starts ACME and generates TXT record, puts it in host var 2) zone.db is updated on adns1 3) host now finishes auth and writes out keys22:41
corvusianw: so those same 3 steps could be applied just to a single host22:41
ianwi think it is compatible, but you *might* end up being a bit racy22:41
corvusianw: maybe we put some lock files around letsencrytp-install-txt-record ?22:41
corvusso that only one dns update happens at once?22:42
ianwif, two hosts decided to renew at exactly the same time.  but a bit of flock could sort that out i think22:42
ianwright ... it's much simpler to just overwrite the zone.db file.  i guess it would involve one more step of removing the lock, after the key is issued22:44
*** eharney has joined #openstack-infra22:45
clarkbwe can add a fair bit of randomness to renewal cycle if we renew early enough. We should still have a lock but chances of contention would be low22:45
openstackgerritJames E. Blair proposed openstack-infra/system-config master: DNM: sample letsencrypt playbook for small service  https://review.openstack.org/63764522:46
ianwright, we're talking every few months22:46
corvusianw: ^ that's an example of what i'm thinking about22:46
corvusbasically, can we do that -- i think the thing that worries me is that i've seen roles reference the 'letsencrypt' group22:46
*** sdake has quit IRC22:46
corvusbut i think our "add servers to groups and run everything from base.yaml" is getting out of hand -- i'd like to be able to just add a letsecrypt role to a server22:48
corvus(or 4 letsencrypt roles, that's fine :)22:48
ianwcorvus: well the prototype as written gets the key configuration from the group variable -> https://review.openstack.org/#/c/636759/23/playbooks/group_vars/letsencrypt.yaml22:48
ianwi figured on one host, you'd potentially want many different certificates22:49
*** sdake has joined #openstack-infra22:49
clarkbwe should be able to define those on host vars too right? then resolution of vars will sort it out22:49
corvusianw: maybe we could make that just a "letencrypt_certs" hostvar?  then we don't need to have a central list, we could add that variable to individual hostvars files...22:49
corvus(same way we have what iptables ports need to be opened on a host in its hostvars file, we'd have what certs need to be requested)22:50
ianwi don't see a problem switching to that.  my first thought was it was nice to have a big list of certs right in front of you, but all that matters is the host knows what certs to request22:51
corvusoh, back to the central vs distributed question -- how do we handle testing if certs are not precious?  for example, we have self-signed certs as hostvars for the registry: http://git.openstack.org/cgit/openstack-infra/system-config/tree/playbooks/zuul/templates/group_vars/registry.yaml.j222:54
corvusthat means we can deploy the registry in a ci job complete with cert -- but how do we do that if we expect the registry to get it cert on startup?22:55
clarkbcorvus: wrap the include role for letsencrypt in a when registry_tls_cert not defined?22:55
clarkbcorvus: then we can separately test that role against their test server. Or configure it to talk to the test server in the test suite instead of my when condition?22:56
ianwthere could be an alternative role used for testing that instead of creating a real key creates a self-signed certificate in the same location as letsencrypt would.  anything connecting to it for testing would also then have to be in testing mode accepting self-signed certs22:58
corvusif we wrap it in a conditional, then we won't test the interaction of the letsencrypt role with the rest of the system (eg, is the registry configured to read the certs from the correct location?) until we run it in prod, so there's a hole in our testing...22:58
*** eharney has quit IRC22:58
corvusthe alternative testing role that makes the cert (or having the real role switch on testing) sounds better as it's a bit less of a hole in test coverage22:59
ianwor, instead of running the actual renew in -> https://review.openstack.org/#/c/636759/23/playbooks/roles/letsencrypt-create-certs/tasks/main.yaml22:59
corvusif we can use letsencrypt's test server, maybe that's the best?  does that do any validation?23:00
ianwit could generate self-signed certs.  just flip the behaviour based on a var23:00
ianwcorvus: the staging servers still validate, you just don't get a real cert23:00
ianwand you don't hit failure rate limits, etc23:00
*** tkajinam has joined #openstack-infra23:00
corvusokay, validation is probably too hard since it's dns based and i don't think we'll be delegating a zone to ephemeral ci nodes :)23:01
corvusso self-signed triggered by var may be the best23:01
*** mriedem has quit IRC23:03
*** dklyle has quit IRC23:03
ianwwe *could* ... it would certainly fall out easier with designate or some DNS-as-an-api type thing.  then you wouldn't worry about races, serial numbers etc23:03
corvusi don't think we could do that in a check job23:03
ianwbut for immediate purposes, i think we could do self-signed23:04
*** rascasoft has quit IRC23:06
ianwoh reading it you've still got limits on le staging; 60 failed validations an hour.  i could DOS that just trying to figure things out :)23:08
openstackgerritGorka Eguileor proposed openstack-infra/project-config master: Add cinderlib project  https://review.openstack.org/63761323:09
ianw... it's a standalone docker image to run your own version of Let's Encrypt easily ...23:10
corvusianw: that's... that's a really cool idea :)23:11
*** munimeha1 has quit IRC23:12
*** e0ne has quit IRC23:12
clarkbzbr: for https://review.openstack.org/#/c/634438/1 we normally just add people via chanserv directly23:20
*** e0ne has joined #openstack-infra23:23
*** sdake has quit IRC23:24
*** sdake has joined #openstack-infra23:26
clarkbfungi can we approve  https://review.openstack.org/#/c/636574/1/gerrit/projects.yaml or does that need a coordinated cut over?23:27
*** e0ne has quit IRC23:30
*** ociuhandu has joined #openstack-infra23:30
clarkbfungi: also any idea what the expected relationship between SIG repos and sigs is/will be? https://review.openstack.org/#/c/637125/423:31
*** sdake has quit IRC23:32
openstackgerritJames E. Blair proposed openstack-infra/zuul-jobs master: Fix build-docker-image when using buildset_registry  https://review.openstack.org/63765023:33
corvusfungi, clarkb: ^ sorry, there was an error in an earlier change23:33
*** jcoufal has quit IRC23:34
clarkbah right. If that was shell it would be ok (maybe I'd have to double check wht that quoting type does)23:34
corvusit folds onto one line23:34
corvusso yeah, the other fix would be to change it to "shell: |"23:35
*** ociuhandu has quit IRC23:35
corvusinstead of "command: >-"23:35
clarkbre earlier ironic question, https://review.openstack.org/#/c/636093/1 is similar. I think in both cases we would need to confirm with the upstream they want our third party input (and we should fix the github perforamnce issure and get that deployed first). Also be extra clear we aren't gating or main CI for that project (as noted in that email thread I wrote a little while back) and instead are a23:36
clarkbthird party input23:36
openstackgerritMerged openstack-infra/zuul master: Re-use the github PR object when fetching reviews  https://review.openstack.org/63670523:36
openstackgerritMerged openstack-infra/zuul master: Add comment about extra issues request  https://review.openstack.org/63670623:36
*** gfidente|afk has quit IRC23:37
corvusclarkb: https://governance.openstack.org/tc/resolutions/20180215-third-party-check.html is a handy link if you need to share it23:37
openstackgerritClark Boylan proposed openstack-infra/zuul master: Don't request PR issue data  https://review.openstack.org/63672823:40
openstackgerritClark Boylan proposed openstack-infra/zuul master: Don't request PR issue data  https://review.openstack.org/63672823:41
clarkbcorvus: thanks23:41
corvusclarkb: also.... technically.... 636093 won't increase our volume -- adding the github app will, and that's independent of 636093.  it's unclear if the goal is to have gnocchi trigger zuul (eg, third-party-check), or just have telementry be able to use required-projects and depends-on with gnocchi.  if it's the latter, then they don't have to add the app.  but if you still want to hold that change23:41
corvusuntil lru is in out of an abundance of caution, that wfm.23:41
clarkbya I think we are really close to fixing the sha to pr mapping so may as well try and get that in first. Also good point re depends on or adding the app23:42
openstackgerritMerged openstack-infra/project-config master: Remove rtd_web_hook from networking-ansible  https://review.openstack.org/63665123:43
*** rascasoft has joined #openstack-infra23:44
*** agopi has joined #openstack-infra23:47
clarkbsomehow X has decided to consume an entire cpu core. /me performs a long overdue reboot23:48
*** rascasoft has quit IRC23:50
*** smarcet has joined #openstack-infra23:50
*** smarcet has quit IRC23:56
openstackgerritIan Wienand proposed openstack-infra/system-config master: [dnm] letsencrypt prototype implementation  https://review.openstack.org/63675923:58
*** spsurya has quit IRC23:58

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!