| *** wolverineav has joined #openstack-infra | 00:02 | |
| *** eernst has quit IRC | 00:03 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul-jobs master: Add prepare-workspace-openshift role https://review.openstack.org/631402 | 00:16 |
|---|---|---|
| *** sdake has quit IRC | 00:23 | |
| *** sdake has joined #openstack-infra | 00:24 | |
| openstackgerrit | Clark Boylan proposed openstack-infra/puppet-zuul master: Allow for both global zuul web and whitelabeled tenant https://review.openstack.org/633586 | 00:25 |
| *** hongbin has joined #openstack-infra | 00:27 | |
| pabelanger | clarkb: ianw_pto: just confirmed fedora-29, non network manager still work. But haven't looked more into current failures | 00:28 |
| *** kei-ichi has joined #openstack-infra | 00:29 | |
| clarkb | cool I'm still trying to sort out the last issues on that puppet change for running an opendev zuul web frontend | 00:33 |
| clarkb | I'm oh so very close but now running into the specific behaviors of puppet hashes in erb templates when accessed via ruby | 00:33 |
| *** wolverineav has quit IRC | 00:34 | |
| *** wolverineav has joined #openstack-infra | 00:35 | |
| *** jcoufal has quit IRC | 00:36 | |
| *** edmondsw has quit IRC | 00:38 | |
| *** trown is now known as trown|outtypewww | 00:39 | |
| *** wolverineav has quit IRC | 00:39 | |
| pabelanger | okay, fedora-29 running on limestone current | 00:40 |
| pabelanger | I haven't seen one of ovh today | 00:40 |
| pabelanger | Hmm, still having issues accessing github.com from limestone | 00:45 |
| *** gyee has quit IRC | 00:51 | |
| *** wolverineav has joined #openstack-infra | 00:52 | |
| openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: Fall back to legacy networking for fedora-29 https://review.openstack.org/633631 | 00:52 |
| pabelanger | clarkb: ^should allow fedora-29 on all providers, until ianw_pto is back to look more into network manager. We in ansible-network are still using legacy network and images still boot properly on vexxhost / limestone | 00:54 |
| *** smarcet has quit IRC | 00:54 | |
| *** smarcet has joined #openstack-infra | 00:55 | |
| *** wolverineav has quit IRC | 00:56 | |
| *** armax has quit IRC | 00:58 | |
| *** jamesmcarthur has joined #openstack-infra | 01:04 | |
| *** jamesmcarthur has quit IRC | 01:09 | |
| *** yamamoto has joined #openstack-infra | 01:09 | |
| *** markvoelker has joined #openstack-infra | 01:10 | |
| *** rlandy is now known as rlandy|bbl | 01:15 | |
| *** markvoelker has quit IRC | 01:15 | |
| *** hongbin has quit IRC | 01:18 | |
| *** hongbin has joined #openstack-infra | 01:20 | |
| *** rh-jelabarre has quit IRC | 01:24 | |
| *** sdake has quit IRC | 01:28 | |
| *** dulek has quit IRC | 01:28 | |
| *** bhavikdbavishi has joined #openstack-infra | 01:33 | |
| *** ruffian_sheep has joined #openstack-infra | 01:33 | |
| *** tosky has quit IRC | 01:34 | |
| *** armax has joined #openstack-infra | 01:36 | |
| *** wolverineav has joined #openstack-infra | 01:37 | |
| *** bhavikdbavishi has quit IRC | 01:51 | |
| *** bhavikdbavishi has joined #openstack-infra | 01:51 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: model: remove unused addImpliedBranchMatcher procedure https://review.openstack.org/633643 | 01:53 |
| *** bobh has quit IRC | 01:53 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: model: remove unused job's BranchMatcher procedures https://review.openstack.org/633643 | 01:56 |
| *** ddurst has joined #openstack-infra | 01:58 | |
| *** bobh has joined #openstack-infra | 02:00 | |
| *** yamamoto has quit IRC | 02:03 | |
| *** sdake has joined #openstack-infra | 02:31 | |
| *** sdake has quit IRC | 02:37 | |
| *** bhavikdbavishi has quit IRC | 02:37 | |
| *** whoami-rajat has joined #openstack-infra | 02:37 | |
| *** sdake has joined #openstack-infra | 02:37 | |
| *** yamamoto has joined #openstack-infra | 02:41 | |
| *** yamamoto has quit IRC | 02:46 | |
| *** psachin has joined #openstack-infra | 02:52 | |
| *** ykarel|away has joined #openstack-infra | 02:55 | |
| *** armstrong has quit IRC | 02:57 | |
| *** sdake has quit IRC | 02:58 | |
| *** sdake has joined #openstack-infra | 02:59 | |
| *** bgmccollum has quit IRC | 03:00 | |
| *** bgmccollum has joined #openstack-infra | 03:02 | |
| *** rlandy|bbl is now known as rlandy | 03:04 | |
| *** rlandy has quit IRC | 03:04 | |
| *** apetrich has quit IRC | 03:15 | |
| *** jamesmcarthur has joined #openstack-infra | 03:23 | |
| *** jamesmcarthur has quit IRC | 03:24 | |
| *** jamesmcarthur has joined #openstack-infra | 03:24 | |
| *** yamamoto has joined #openstack-infra | 03:25 | |
| *** sdake has quit IRC | 03:26 | |
| *** sdake has joined #openstack-infra | 03:29 | |
| *** yamamoto has quit IRC | 03:34 | |
| *** sdake has quit IRC | 03:42 | |
| *** ykarel|away is now known as ykarel | 03:42 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: config: add playbooks to job.toDict() https://review.openstack.org/621343 | 03:48 |
| *** anteaya has quit IRC | 03:49 | |
| *** ykarel is now known as ykarel|afk | 03:50 | |
| *** bhavikdbavishi has joined #openstack-infra | 03:52 | |
| *** udesale has joined #openstack-infra | 04:02 | |
| *** yamamoto has joined #openstack-infra | 04:06 | |
| *** ykarel|afk is now known as ykarel | 04:06 | |
| *** bobh has quit IRC | 04:11 | |
| *** diablo_rojo has quit IRC | 04:12 | |
| *** jamesmcarthur has quit IRC | 04:15 | |
| *** jamesmcarthur has joined #openstack-infra | 04:16 | |
| *** bobh has joined #openstack-infra | 04:18 | |
| *** jamesmcarthur has quit IRC | 04:21 | |
| *** chkumar|out is now known as chandankumar | 04:22 | |
| *** bobh has quit IRC | 04:27 | |
| *** eernst has joined #openstack-infra | 04:32 | |
| *** bhavikdbavishi has quit IRC | 04:38 | |
| *** bhavikdbavishi has joined #openstack-infra | 04:39 | |
| *** sdake has joined #openstack-infra | 04:49 | |
| *** spsurya has joined #openstack-infra | 05:05 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: scheduler: add job's variants to the rpc job_list method https://review.openstack.org/573473 | 05:16 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: switch jobs list to a tree view https://review.openstack.org/633437 | 05:16 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add jobs list filter https://review.openstack.org/633652 | 05:16 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: scheduler: add job's tags to the rpc job_list method https://review.openstack.org/633653 | 05:16 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add tags to jobs list https://review.openstack.org/633654 | 05:16 |
| *** auristor has quit IRC | 05:20 | |
| *** auristor has joined #openstack-infra | 05:20 | |
| *** ruffian_sheep has quit IRC | 05:21 | |
| *** bhavikdbavishi has quit IRC | 05:27 | |
| *** udesale has quit IRC | 05:29 | |
| *** jtomasek has joined #openstack-infra | 05:36 | |
| *** eernst has quit IRC | 05:38 | |
| *** sdake has quit IRC | 05:41 | |
| *** sdake has joined #openstack-infra | 05:49 | |
| *** udesale has joined #openstack-infra | 05:51 | |
| *** hwoarang has quit IRC | 05:52 | |
| *** bhavikdbavishi has joined #openstack-infra | 05:53 | |
| *** hwoarang has joined #openstack-infra | 05:55 | |
| *** udesale has quit IRC | 05:59 | |
| *** udesale has joined #openstack-infra | 06:00 | |
| *** jtomasek has quit IRC | 06:02 | |
| *** ykarel has quit IRC | 06:09 | |
| *** edmondsw has joined #openstack-infra | 06:19 | |
| *** markvoelker has joined #openstack-infra | 06:20 | |
| *** hongbin has quit IRC | 06:21 | |
| *** ykarel has joined #openstack-infra | 06:21 | |
| *** jamesmcarthur has joined #openstack-infra | 06:24 | |
| *** markvoelker has quit IRC | 06:24 | |
| *** noama has joined #openstack-infra | 06:24 | |
| *** mandre is now known as mandre|off | 06:25 | |
| *** jamesmcarthur has quit IRC | 06:28 | |
| tobiash | infra-root: are there dns problems in some providers? I think I remember several job failures due to 'Temporary failure in name resolution' during the last days. Example log is here: http://logs.openstack.org/43/633643/2/check/tox-py36/e3d3969/job-output.txt.gz#_2019-01-29_02_04_11_799858 | 06:29 |
| tobiash | this one happened in rax-dfw | 06:33 |
| *** ramishra has joined #openstack-infra | 06:33 | |
| *** strigazi has quit IRC | 06:42 | |
| *** strigazi has joined #openstack-infra | 06:43 | |
| *** ccamacho has quit IRC | 06:44 | |
| *** kjackal has joined #openstack-infra | 06:50 | |
| AJaeger | corvus: https://review.openstack.org/#/c/632785 is not handled by zuul - I think we merged 633591 too soon, should have waited for 785 to merge first - shouldn't it? Or how do you want to force-merge? | 07:01 |
| *** quiquell|off is now known as quiquell | 07:01 | |
| *** wolverineav has quit IRC | 07:03 | |
| *** wolverineav has joined #openstack-infra | 07:04 | |
| *** wolverineav has quit IRC | 07:09 | |
| *** udesale has quit IRC | 07:10 | |
| *** udesale has joined #openstack-infra | 07:18 | |
| *** ccamacho has joined #openstack-infra | 07:20 | |
| *** udesale has quit IRC | 07:22 | |
| *** udesale has joined #openstack-infra | 07:26 | |
| *** yamamoto has quit IRC | 07:28 | |
| *** yamamoto has joined #openstack-infra | 07:32 | |
| *** pgaxatte has joined #openstack-infra | 07:33 | |
| *** kjackal has quit IRC | 07:34 | |
| *** wolverineav has joined #openstack-infra | 07:34 | |
| *** kjackal has joined #openstack-infra | 07:37 | |
| *** wolverineav has quit IRC | 07:38 | |
| *** slaweq has joined #openstack-infra | 07:43 | |
| *** aojea has joined #openstack-infra | 07:46 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: config: add tenant.toDict() method and REST endpoint https://review.openstack.org/621344 | 07:48 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add config page https://review.openstack.org/633667 | 07:48 |
| *** e0ne has joined #openstack-infra | 07:59 | |
| *** shardy has joined #openstack-infra | 08:02 | |
| *** rpittau has joined #openstack-infra | 08:04 | |
| *** apetrich has joined #openstack-infra | 08:09 | |
| *** sdake has quit IRC | 08:11 | |
| *** ginopc has joined #openstack-infra | 08:11 | |
| *** shardy has quit IRC | 08:11 | |
| *** shardy has joined #openstack-infra | 08:12 | |
| *** dulek has joined #openstack-infra | 08:15 | |
| *** wolverineav has joined #openstack-infra | 08:15 | |
| *** kjackal has quit IRC | 08:16 | |
| *** kjackal_v2 has joined #openstack-infra | 08:16 | |
| *** kopecmartin|off is now known as kopecmartin | 08:19 | |
| *** wolverineav has quit IRC | 08:19 | |
| *** markvoelker has joined #openstack-infra | 08:20 | |
| *** gfidente has joined #openstack-infra | 08:21 | |
| *** gfidente has quit IRC | 08:25 | |
| *** ginopc has quit IRC | 08:26 | |
| *** bnemec has joined #openstack-infra | 08:29 | |
| *** jpena|off is now known as jpena | 08:33 | |
| *** ramishra_ has joined #openstack-infra | 08:34 | |
| *** ramishra has quit IRC | 08:35 | |
| *** electrofelix has joined #openstack-infra | 08:35 | |
| *** yamamoto has quit IRC | 08:35 | |
| *** yamamoto has joined #openstack-infra | 08:36 | |
| *** peschk_l has quit IRC | 08:36 | |
| *** janki has joined #openstack-infra | 08:36 | |
| chandankumar | Hello | 08:39 |
| chandankumar | can we hold the node openstack-ansible-functional-centos-7 node next time we retrigger it we need to debug a tempest failure on centos7 for os_tempest side/ | 08:40 |
| *** tosky has joined #openstack-infra | 08:42 | |
| frickler | chandankumar: which project/patch do you need this for? | 08:45 |
| chandankumar | frickler: https://review.openstack.org/#/c/633208 | 08:46 |
| *** tkajinam has quit IRC | 08:48 | |
| *** pcaruana has joined #openstack-infra | 08:51 | |
| *** ykarel is now known as ykarel|lunch | 08:52 | |
| elod | hi, infra team! I've found a small issue last week around "editable git installed packages" and created a minor fix for it: https://review.openstack.org/#/c/633049/ | 08:53 |
| elod | could someone review it? :) | 08:53 |
| *** priteau has joined #openstack-infra | 08:54 | |
| *** markvoelker has quit IRC | 08:54 | |
| tobias-urdin | infra-root: ping on https://review.openstack.org/#/c/632699/ | 08:54 |
| elod | without it there are some (stable) job failures (and some cases it doesn't cause real problem however does not install deps for editable packages) | 08:54 |
| frickler | chandankumar: ok, hold set up, you'll get a node once that job fails. which ssh key shall we use to give you access? | 08:56 |
| AJaeger | tobias-urdin: could you ask pabelanger to review 632699, please? | 08:58 |
| frickler | elod: that fix looks to me like it will need some extended testing, would it be possible for you to pin pip<19 as a workaround? | 09:00 |
| *** jpich has joined #openstack-infra | 09:01 | |
| tobias-urdin | AJaeger: ack | 09:02 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: config: add playbooks to job.toDict() https://review.openstack.org/621343 | 09:04 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: config: add tenant.toDict() method and REST endpoint https://review.openstack.org/621344 | 09:04 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add config page https://review.openstack.org/633667 | 09:04 |
| *** jtomasek has joined #openstack-infra | 09:06 | |
| *** gfidente has joined #openstack-infra | 09:06 | |
| chandankumar | frickler: sending in pm | 09:09 |
| elod | frickler AJaeger : i've tested only the function itself, but i'll do a depends on test now, thanks for the review! | 09:13 |
| elod | frickler: i can create pip<19 pin workaround if you think that is needed | 09:14 |
| *** soumplis has quit IRC | 09:19 | |
| *** ykarel|lunch is now known as ykarel | 09:20 | |
| *** panda is now known as panda|numb | 09:28 | |
| *** dtantsur|afk is now known as dtantsur | 09:35 | |
| *** derekh has joined #openstack-infra | 09:36 | |
| *** luizbag has joined #openstack-infra | 09:40 | |
| *** ginopc has joined #openstack-infra | 09:44 | |
| *** markvoelker has joined #openstack-infra | 09:51 | |
| *** kashyap has joined #openstack-infra | 09:53 | |
| kashyap | Hi folks, this patch _keeps_ getting some kind of time out for the last 10 days: https://review.openstack.org/#/c/630980/ (libvirt: A few miscellaneous items related to "native TLS") | 09:53 |
| kashyap | And the failures are not related to my patch at all. | 09:53 |
| kashyap | I was pointed out this: http://logs.openstack.org/80/630980/7/gate/tempest-full/5b6ebba/ara-report/ | 09:54 |
| kashyap | ... where the 'run-devstack' took 30 minutes. | 09:54 |
| kashyap | Any hints on how to put that patch out of its misery? | 09:54 |
| *** baysao has joined #openstack-infra | 10:01 | |
| *** zaneb has quit IRC | 10:16 | |
| openstackgerrit | Merged openstack-infra/irc-meetings master: Change of agenda url for weekly meetings https://review.openstack.org/633466 | 10:17 |
| *** yboaron has joined #openstack-infra | 10:20 | |
| ttx | AJaeger: any chance you could review https://review.openstack.org/#/c/631032/ ? I need that repo up to propose changes to it | 10:21 |
| ttx | just needs the second +2a | 10:21 |
| *** ramishra_ has quit IRC | 10:23 | |
| ttx | or frickler ^ | 10:23 |
| *** markvoelker has quit IRC | 10:24 | |
| *** adriancz has joined #openstack-infra | 10:24 | |
| *** rf0lc0 has joined #openstack-infra | 10:27 | |
| *** rfolco has quit IRC | 10:27 | |
| frickler | ttx: hmm, since this is the first repo in a new namespace, I'd like to wait for corvus' vote on that one | 10:28 |
| *** ramishra has joined #openstack-infra | 10:30 | |
| ttx | frickler: sure. He did comment no a previous patchset but did not vote on the final one. | 10:30 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack-infra/project-config master: Stop project gating before retirement https://review.openstack.org/631218 | 10:30 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack-infra/project-config master: Stop project gating before retirement https://review.openstack.org/631218 | 10:31 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add roles usage information to the build page https://review.openstack.org/633697 | 10:43 |
| *** bhavikdbavishi has quit IRC | 10:45 | |
| *** baysao has quit IRC | 10:49 | |
| frickler | chandankumar: slaweq: root@158.69.65.144 is your held node. please ping infra-root once you are finished with debugging | 10:49 |
| *** rf0lc0 has quit IRC | 10:53 | |
| chandankumar | frickler: sure | 10:53 |
| slaweq | frickler: thx | 10:54 |
| frickler | ttx: clarkb: corvus: added some comment regarding uniqueness of repo names vs. gerrit group names to https://review.openstack.org/#/c/631032/ | 11:00 |
| *** ssbarnea|bkp2 has quit IRC | 11:01 | |
| *** ssbarnea|rover has joined #openstack-infra | 11:02 | |
| *** janki has quit IRC | 11:02 | |
| *** rfolco has joined #openstack-infra | 11:03 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add roles usage information to the build page https://review.openstack.org/633697 | 11:04 |
| *** udesale has quit IRC | 11:09 | |
| *** yamamoto has quit IRC | 11:14 | |
| chandankumar | jrosser: slaweq is already there in the box | 11:20 |
| chandankumar | slaweq: jrosser can help on osa | 11:20 |
| *** markvoelker has joined #openstack-infra | 11:21 | |
| *** bhavikdbavishi has joined #openstack-infra | 11:39 | |
| chandankumar | frickler: we are done with debugging on the nodes | 11:51 |
| chandankumar | frickler: you can remove the hold | 11:51 |
| chandankumar | frickler: thanks for the help :-) | 11:51 |
| *** markvoelker has quit IRC | 11:53 | |
| *** yamamoto has joined #openstack-infra | 11:54 | |
| *** yamamoto has quit IRC | 12:04 | |
| *** kukacz has quit IRC | 12:04 | |
| *** kukacz has joined #openstack-infra | 12:04 | |
| *** lucasagomes is now known as lucas-hungry | 12:07 | |
| *** fdegir is now known as fdegir_ | 12:10 | |
| *** agopi|off has joined #openstack-infra | 12:12 | |
| *** jpena is now known as jpena|lunch | 12:15 | |
| *** yamamoto has joined #openstack-infra | 12:23 | |
| *** jamesmcarthur has joined #openstack-infra | 12:24 | |
| *** jamesmcarthur has quit IRC | 12:29 | |
| *** fdegir has joined #openstack-infra | 12:31 | |
| *** fdegir_ has left #openstack-infra | 12:32 | |
| elod | AJaeger frickler : the depends-on test finished and I've commented the log details | 12:39 |
| *** pcaruana has quit IRC | 12:40 | |
| elod | I've tested on vmware-nsx stable/ocata, is it enough or should I do some other test on other project and branch? | 12:40 |
| *** yboaron has quit IRC | 12:42 | |
| *** markmcd has quit IRC | 12:43 | |
| *** ginopc has quit IRC | 12:44 | |
| *** quiquell is now known as quiquell|brb | 12:48 | |
| *** markmcd has joined #openstack-infra | 12:48 | |
| *** pcaruana has joined #openstack-infra | 12:50 | |
| *** kaiokmo has joined #openstack-infra | 12:50 | |
| *** markvoelker has joined #openstack-infra | 12:50 | |
| *** lucas-hungry is now known as lucasagomes | 12:51 | |
| *** apetrich has quit IRC | 12:52 | |
| *** apetrich has joined #openstack-infra | 12:55 | |
| *** agopi|off has quit IRC | 12:55 | |
| *** priteau has quit IRC | 13:04 | |
| *** priteau has joined #openstack-infra | 13:04 | |
| *** sean-k-mooney has joined #openstack-infra | 13:04 | |
| sean-k-mooney | im not sure if this is the right place to ask but im wondering if we encountered this error http://paste.openstack.org/show/744167/ upstream and how it was corrected in the devstack post playbook | 13:09 |
| sean-k-mooney | specifcally with a ubuntu 18.04 guest https://github.com/openstack-dev/devstack/blob/94ca9f6756e7b677b1ee3fd2e32b555447e950dd/roles/fetch-devstack-log-dir/tasks/main.yaml is failing to sync the logs | 13:09 |
| *** xek has joined #openstack-infra | 13:11 | |
| sean-k-mooney | i had considerd submitting a patch to devstack to disable preserving permissions/ownership but i was wonderign if infra had another solution | 13:11 |
| *** priteau has quit IRC | 13:12 | |
| sean-k-mooney | this is all in relation to seting up a third party ci. at this point my devstack job is reusign the upstream devstack job and stacking but this final role in the post playbook is failing an causeing the job to fail as a result | 13:13 |
| *** trown|outtypewww is now known as trown | 13:13 | |
| *** e0ne has quit IRC | 13:13 | |
| *** markvoelker has quit IRC | 13:20 | |
| *** sean-k-mooney has quit IRC | 13:20 | |
| *** sean-k-mooney has joined #openstack-infra | 13:21 | |
| *** quiquell|brb is now known as quiquell | 13:26 | |
| *** agopi|off has joined #openstack-infra | 13:26 | |
| *** agopi|off is now known as agopi | 13:26 | |
| *** kgiusti has joined #openstack-infra | 13:27 | |
| *** mriedem has joined #openstack-infra | 13:28 | |
| *** e0ne has joined #openstack-infra | 13:29 | |
| *** rlandy has joined #openstack-infra | 13:30 | |
| *** jpena|lunch is now known as jpena | 13:36 | |
| openstackgerrit | Matt Riedemann proposed openstack-infra/project-config master: Fix the ceph grafana dashboard https://review.openstack.org/633735 | 13:44 |
| *** jcoufal has joined #openstack-infra | 13:44 | |
| *** rh-jelabarre has joined #openstack-infra | 13:47 | |
| *** bhavikdbavishi has quit IRC | 13:48 | |
| *** sthussey has joined #openstack-infra | 13:54 | |
| *** panda|numb is now known as panda | 13:56 | |
| *** psachin has quit IRC | 13:56 | |
| mriedem | i noticed that https://bugs.launchpad.net/cinder/+bug/1810526 is predominantly on ovh nodes, | 13:58 |
| openstack | Launchpad bug 1810526 in Cinder "volume delete fails due to ToozConnectionError in the gate" [Undecided,Confirmed] | 13:58 |
| mriedem | is there some trend we've seen about ovh nodes that might indicate why that could be failing so much, i.e. i/o issues that might cause etcd delays? | 13:58 |
| openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck master: Add query for cinder tooz connection error bug 1810526 https://review.openstack.org/633738 | 13:59 |
| *** yamamoto has quit IRC | 14:00 | |
| *** ykarel is now known as ykarel|away | 14:01 | |
| *** jcoufal has quit IRC | 14:07 | |
| openstackgerrit | Merged openstack-infra/elastic-recheck master: fix tox python3 overrides https://review.openstack.org/605618 | 14:09 |
| *** yboaron has joined #openstack-infra | 14:14 | |
| *** efried has quit IRC | 14:15 | |
| *** efried has joined #openstack-infra | 14:15 | |
| *** jcoufal has joined #openstack-infra | 14:16 | |
| *** efried has quit IRC | 14:19 | |
| *** efried has joined #openstack-infra | 14:19 | |
| *** roman_g has joined #openstack-infra | 14:20 | |
| frickler | mriedem: seems https://review.openstack.org/626885 tackles the same thing. maybe we should consider switching this on by default instead of only for kuryr | 14:22 |
| *** yamamoto has joined #openstack-infra | 14:23 | |
| *** bobh has joined #openstack-infra | 14:23 | |
| *** udesale has joined #openstack-infra | 14:24 | |
| *** ykarel|away has quit IRC | 14:25 | |
| mriedem | hmm yeah | 14:26 |
| mriedem | the other thing i was thinking about was just not using etcd as the cinder DLM backend on single host jobs since it's kind of pointless | 14:26 |
| *** dkehn has joined #openstack-infra | 14:27 | |
| fungi | infra-root: i'm not really here, but i see rackspace fanatical support opened a ticket to let us know the host for afs01.ord.openstack.org has suffered some sort of critical failure so likely want to keep an eye on our afs infrastructure and check out that fileserver when it comes back online | 14:31 |
| openstackgerrit | Merged openstack-infra/elastic-recheck master: Add query for cinder tooz connection error bug 1810526 https://review.openstack.org/633738 | 14:34 |
| openstack | bug 1810526 in Cinder "volume delete fails due to ToozConnectionError in the gate" [Undecided,Confirmed] https://launchpad.net/bugs/1810526 | 14:34 |
| *** smarcet has quit IRC | 14:34 | |
| *** smarcet has joined #openstack-infra | 14:36 | |
| *** dave-mccowan has joined #openstack-infra | 14:39 | |
| *** weshay has quit IRC | 14:43 | |
| *** dave-mccowan has quit IRC | 14:45 | |
| *** pcaruana has quit IRC | 14:45 | |
| *** eharney has joined #openstack-infra | 14:49 | |
| *** sdake has joined #openstack-infra | 14:49 | |
| *** bhavikdbavishi has joined #openstack-infra | 14:50 | |
| *** sdake has quit IRC | 14:51 | |
| *** roman_g has quit IRC | 14:51 | |
| *** e0ne has quit IRC | 14:51 | |
| *** pcaruana has joined #openstack-infra | 14:53 | |
| *** sdake has joined #openstack-infra | 14:54 | |
| *** e0ne has joined #openstack-infra | 14:54 | |
| *** roman_g has joined #openstack-infra | 14:58 | |
| smcginnis | mriedem: We could switch most jobs to use local filesystem locks. That's actually the default when you deploy system packages, so devstack kind of sets things up in a non-standardish way. | 15:01 |
| mriedem | smcginnis: yeah i just haven't dug into that yet, i need to re-learn how to determine from devstack if it's going to be a multi-node job | 15:02 |
| mriedem | i don't want to have to set a flag in all zuul jobs (either single or multinode) | 15:03 |
| *** sshnaidm is now known as sshnaidm|mtg | 15:04 | |
| *** smarcet has quit IRC | 15:05 | |
| *** smarcet has joined #openstack-infra | 15:08 | |
| openstackgerrit | Merged openstack-infra/project-config master: Add ansible roles for managing HSM software https://review.openstack.org/631326 | 15:09 |
| mriedem | smcginnis: having said that, i'm not actively working on that one right now if you're bored | 15:12 |
| mriedem | http://status.openstack.org/elastic-recheck/#1813198 is the current killer | 15:12 |
| *** yamamoto has quit IRC | 15:15 | |
| *** yamamoto has joined #openstack-infra | 15:16 | |
| *** yamamoto has quit IRC | 15:16 | |
| *** yamamoto has joined #openstack-infra | 15:17 | |
| mnaser | infra-root: i see a change that merged but showed this - infra-prod-zuul_reconfigure finger://ze02.openstack.org/b0db56e18f104bfa8f08350254d8405a : RETRY_LIMIT in 23s | 15:18 |
| *** udesale has quit IRC | 15:18 | |
| pabelanger | mnaser: expected, I blieve the job is missing known_hosts task | 15:19 |
| clarkb | mnaser: that is noise. Should probably remove the job if I cant get votes on the fix. | 15:19 |
| pabelanger | I believe* | 15:19 |
| mnaser | pabelanger, clarkb: ok cool, thanks, just wanted to make sure zuul didn't fail reconfig or something | 15:20 |
| clarkb | pabelanger: the issue is we cant ssh as root. So I have changes up to ssh as a zuul user | 15:20 |
| pabelanger | looking | 15:20 |
| AJaeger | pabelanger could you review 632699, please? | 15:20 |
| *** wolverineav has joined #openstack-infra | 15:21 | |
| *** eharney has quit IRC | 15:21 | |
| pabelanger | clarkb: left comment on 604925 | 15:23 |
| *** bobh has quit IRC | 15:23 | |
| *** yamamoto has quit IRC | 15:23 | |
| pabelanger | AJaeger: Hmm, I for puppet modules we installed dependencies via bindep and build target | 15:23 |
| pabelanger | AJaeger: will lookup and reply | 15:24 |
| *** eharney has joined #openstack-infra | 15:24 | |
| clarkb | pabelanger: that is possible | 15:25 |
| *** wolverineav has quit IRC | 15:25 | |
| pabelanger | clarkb: I also proposed https://review.openstack.org/633631/ to roll back nm on fedora-29 nodes, since online limestone is booting them currently | 15:25 |
| pabelanger | clarkb: yah, https://github.com/ansible-network/windmill-config/blob/master/tests/playbooks/bastion.yaml is how I CD from zuul today | 15:26 |
| clarkb | pabelanger: I thought fedora 29 required nm? or is it rhel8 that makes the switch required? | 15:27 |
| pabelanger | clarkb: not required, I'm still using legacy script for ansible-network, you just need to install them manually. As I say that, let me check if our elements will do that | 15:28 |
| *** smarcet has quit IRC | 15:29 | |
| openstackgerrit | Paul Belanger proposed openstack-infra/project-config master: Fall back to legacy networking for fedora-29 https://review.openstack.org/633631 | 15:32 |
| *** sdake has quit IRC | 15:32 | |
| *** kjackal_v2 has quit IRC | 15:32 | |
| pabelanger | clarkb: that's the setup we use in ansible-network^, but can understand if don't want to do it. | 15:32 |
| *** kjackal has joined #openstack-infra | 15:32 | |
| pabelanger | I know ianw_pto is on PTO, so unsure when we're get around to looking at the actualy issue | 15:33 |
| clarkb | It doesnt fix the broken udev and dbus which may still cause jobs to fail | 15:33 |
| *** sshnaidm|mtg is now known as sshnaidm | 15:33 | |
| pabelanger | yah, this is mostly just to get fedora-29 working on all providers again. Right now, jobs depending on it are having high failure rate. Only running in limestone, and there seems to be dns issue going on there too | 15:35 |
| *** sdake has joined #openstack-infra | 15:35 | |
| clarkb | dns problems can be related to dbus if systemd's resolveris in the mix | 15:36 |
| pabelanger | http://logs.openstack.org/44/633544/12/check/ansible-role-borgbackup-src-fedora-latest/f77d065/ara-report/result/321da77d-b4cd-4e4d-83c5-1cd1f45e28ad/ | 15:36 |
| pabelanger | yah | 15:36 |
| clarkb | my point is fedora 29 is broken. Our results reflect that. The fix is to fix dbus and nm | 15:36 |
| pabelanger | I won't have much time to do that this week sadly | 15:37 |
| *** ykarel|away has joined #openstack-infra | 15:38 | |
| *** ykarel|away is now known as ykarel | 15:39 | |
| *** bhavikdbavishi has quit IRC | 15:43 | |
| *** ccamacho has quit IRC | 15:45 | |
| *** ccamacho has joined #openstack-infra | 15:46 | |
| openstackgerrit | Matt Riedemann proposed openstack-infra/elastic-recheck master: Add query for evacuate vif plugging timeout bug 1813789 https://review.openstack.org/633766 | 15:49 |
| corvus | AJaeger: yes, let's force-merge 785; i think that is the safest thing to do (so we are extra careful not to mess up the openstack tenant) | 15:49 |
| openstack | bug 1813789 in OpenStack Compute (nova) "Evacuate test intermittently fails with network-vif-plugged timeout exception" [Medium,Confirmed] https://launchpad.net/bugs/1813789 | 15:49 |
| *** weshay has joined #openstack-infra | 15:49 | |
| *** openstackgerrit has quit IRC | 15:51 | |
| clarkb | pabelanger: thinking about it more limestone likely works asit is an ipv6 cloud and the RAs configure the interface automatically | 15:51 |
| AJaeger | corvus: works for me - please go ahead with force merging... | 15:51 |
| corvus | frickler made a good point on 631032, or really, 2 good points: should we relax the uniqueness restriction on repo names (eg, is it okay now to have kata/foo and openstack/foo). and how should we name gerrit groups? | 15:54 |
| corvus | we have some jobs in openstack that assume unique repo names. i think as long as we maintain repo name uniqueness in the openstack zuul tenant, that should be okay. | 15:55 |
| pabelanger | clarkb: yah, believe so. I've send an email to ML asking for help from the community. For now, I'll have to flip jobs non-voting and hope I can get some spare cycles later this week to look more | 15:55 |
| *** bhavikdbavishi has joined #openstack-infra | 15:55 | |
| corvus | (eg, doc, tarball, all work on repo names). | 15:55 |
| *** eernst has joined #openstack-infra | 15:56 | |
| corvus | we should avoid setting up a pypi publishing job in another tenant with the same credential. | 15:56 |
| corvus | otherwise starlingx/nova could overwrite openstack/nova | 15:56 |
| corvus | clarkb: ^ | 15:57 |
| clarkb | corvus credential uniqueness probably desireable for other reasons too | 15:57 |
| clarkb | If we make the rule we must be unique within a tenant/namespace that shoyld be relatively easy to check and we can prefix group names if we want to avoid collisions | 15:58 |
| *** kjackal has quit IRC | 16:03 | |
| *** kjackal has joined #openstack-infra | 16:03 | |
| ttx | from a practical perspective, allowing for duplicate names would prevent intentional/unintentional landgrab | 16:09 |
| ttx | We have lots of things like openstack/governance | 16:09 |
| *** dkehn has quit IRC | 16:10 | |
| ttx | Maybe uniqueness within a Zuul tenant is the right way | 16:10 |
| ttx | at least for now | 16:11 |
| ttx | corvus: should I just rename my repo to kata-containers/kata-zuul-config and table that question (I kinda need that repo up asap)? Would that work for you? | 16:13 |
| dmsimard | Is Zuul not picking up new jobs right now ? I have two patches up that haven't triggered anything | 16:13 |
| dmsimard | Hmm, there are new jobs starting.. I'll try a recheck | 16:14 |
| corvus | ttx: i'm happy with /zuul-config, sounds like clarkb is too. i'm pretty sure we want to be as hands-off as possible with opendev, so i'd leave it. | 16:14 |
| ttx | corvus: alright! Just +2a at your convenience then | 16:15 |
| dmsimard | Oh, I see this message in logs: 2019-01-29 16:12:16,842 DEBUG zuul.Pipeline.openstack.check: <QueueItem 0x7fd9c69599b0 for <Change 0x7fd9beeafa90 openstack/ara-web 633764,1> in check> is a failing item because ['is a non-live item with no items behind'] | 16:15 |
| *** quiquell is now known as quiquell|off | 16:16 | |
| *** jamesmcarthur has joined #openstack-infra | 16:16 | |
| corvus | dmsimard: that's likely a red herring | 16:17 |
| dmsimard | okay, false alarm -- file matcher was wrong | 16:17 |
| dmsimard | sorry about the noise | 16:17 |
| *** jamesmcarthur has quit IRC | 16:21 | |
| *** wolverineav has joined #openstack-infra | 16:21 | |
| redrobot | Hello infra friends! | 16:22 |
| redrobot | I have a couple of new repos that were created earlier today. Wondering how I can get myself added to the XXX-core groups on gerrit for them? | 16:23 |
| *** jamesmcarthur has joined #openstack-infra | 16:23 | |
| redrobot | https://review.openstack.org/#/admin/groups/2005,members | 16:23 |
| redrobot | https://review.openstack.org/#/admin/groups/2006,members | 16:23 |
| *** fdegir has quit IRC | 16:26 | |
| clarkb | redrobot: I'll take a look in a few minutes | 16:27 |
| redrobot | Awesome, thanks clarkb | 16:27 |
| *** bobh has joined #openstack-infra | 16:29 | |
| *** wolverineav has quit IRC | 16:30 | |
| clarkb | redrobot: because they were added as official openstack-ansible deliverables I've added mnaser as the initial user. Mnaser can add additional group members (and new group members can manage from there too) | 16:31 |
| *** e0ne has quit IRC | 16:31 | |
| redrobot | clarkb, awesome, appreciate it. | 16:31 |
| *** openstackgerrit has joined #openstack-infra | 16:32 | |
| openstackgerrit | Merged openstack-infra/project-config master: Add Kata tenant Zuul configuration repository https://review.openstack.org/631032 | 16:32 |
| *** chandankumar is now known as chkumar|out | 16:35 | |
| clarkb | infra-root I got https://review.openstack.org/#/c/633586/ passing tests yesterday. This change adds support for running multiple vhosts in puppet-zuul so that we can run zuul.opendev.org on zuul.openstack.org | 16:38 |
| clarkb | the changes there are backward compatilbe and if we like them we'll need a follow on to systemconfig to configure the new vhost stuff in puppet-zuul | 16:38 |
| *** agopi is now known as agopi|lunch | 16:41 | |
| corvus | clarkb: lgtm | 16:45 |
| clarkb | I guess I should recheck to get that last non voting job to go green. I believe it failed due to rubygem download failures | 16:45 |
| *** sdake has quit IRC | 16:46 | |
| openstackgerrit | Merged openstack-infra/elastic-recheck master: Add query for evacuate vif plugging timeout bug 1813789 https://review.openstack.org/633766 | 16:47 |
| openstack | bug 1813789 in OpenStack Compute (nova) "Evacuate test intermittently fails with network-vif-plugged timeout exception" [Medium,Confirmed] https://launchpad.net/bugs/1813789 | 16:47 |
| clarkb | mriedem: according to the kuryr emails they found that fsync calls on ovh gra1 take >1s ~1.5% of the time which is far greater than the other clouds. They were working with ovh on the issue though | 16:50 |
| *** hamzy has quit IRC | 16:56 | |
| corvus | clarkb: any other zuul multi-tenant stuff i should be doing/reviewing? | 16:57 |
| clarkb | corvus: I think that is the current set. I guess I should go ahead and draft the change that depends on ^ | 16:57 |
| clarkb | I don't have ssl data yet but we can plug that in fairly easily once ready | 16:58 |
| *** bhavikdbavishi has quit IRC | 16:58 | |
| corvus | clarkb: have you started the ssl process? i think last one i saw used dns, so it might be a bit slower, so good to get that going in the background | 16:58 |
| *** noama has quit IRC | 16:59 | |
| clarkb | corvus: I have not | 16:59 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/zuul-jobs master: install-nodejs: add support for RPM-based OSes https://review.openstack.org/631049 | 16:59 |
| clarkb | I wasn't sure if we'd have a working lets encrypt setup though it is looking like no | 16:59 |
| corvus | that seems to be the case | 17:00 |
| *** hamzy has joined #openstack-infra | 17:00 | |
| *** pcaruana has quit IRC | 17:01 | |
| *** pgaxatte has quit IRC | 17:03 | |
| *** gyee has joined #openstack-infra | 17:03 | |
| openstackgerrit | Clark Boylan proposed openstack-infra/system-config master: Add zuul.opendev.org webserver https://review.openstack.org/633783 | 17:08 |
| clarkb | corvus: ^ that is the consumption of the puppet-zuul side of things | 17:08 |
| clarkb | I can spin up a new cert after the infra meeting today | 17:10 |
| clarkb | its "only" $8 so may as well keep pushing on that | 17:11 |
| *** hamzy has quit IRC | 17:11 | |
| clarkb | but then I think its just a matter of merging stuff and checking it works after | 17:11 |
| clarkb | I've been asked to start doing similar for ehterpad, but with an early am flight tomorrow I think I'm going to have to constrain what I start digging into. On the etherpad side of things I think we can simplify a bit and have etherpad.openstack.org redirect to etherpad.opendev.org since their behavior will be the same otherwise (unlike zuul with the multiple tenants) | 17:14 |
| *** bhavikdbavishi has joined #openstack-infra | 17:16 | |
| *** sdake has joined #openstack-infra | 17:17 | |
| clarkb | hrm though we'll still need https certs managed for all of the redirects os maybe not as simple as I first though | 17:17 |
| corvus | would really be great to have le in place :) | 17:18 |
| *** sdake has quit IRC | 17:18 | |
| clarkb | ya finding the balance between moving opendev wforward not blocking on things like that and actually paying that tech debt down currently difficult ;) | 17:18 |
| *** ccamacho has quit IRC | 17:18 | |
| clarkb | I do think the zuul side is worth pushing in particular though | 17:18 |
| *** yamamoto has joined #openstack-infra | 17:21 | |
| *** rpittau has quit IRC | 17:21 | |
| *** rpittau has joined #openstack-infra | 17:21 | |
| *** anteaya has joined #openstack-infra | 17:22 | |
| *** yboaron has quit IRC | 17:24 | |
| *** yboaron has joined #openstack-infra | 17:24 | |
| *** kopecmartin is now known as kopecmartin|off | 17:24 | |
| *** ramishra has quit IRC | 17:25 | |
| *** yboaron has quit IRC | 17:26 | |
| *** trown is now known as trown|lunch | 17:26 | |
| *** yamamoto has quit IRC | 17:27 | |
| *** yboaron has joined #openstack-infra | 17:27 | |
| *** bnemec has quit IRC | 17:27 | |
| *** anteaya has quit IRC | 17:28 | |
| openstackgerrit | Sorin Sbarnea proposed openstack-infra/git-review master: Allow user to control Java version used by tox https://review.openstack.org/630661 | 17:30 |
| *** yboaron has quit IRC | 17:31 | |
| *** yboaron has joined #openstack-infra | 17:31 | |
| *** jpich has quit IRC | 17:32 | |
| *** yboaron has quit IRC | 17:35 | |
| *** rpittau has quit IRC | 17:35 | |
| openstackgerrit | Sorin Sbarnea proposed openstack-infra/git-review master: Allow user to control Java version used by tox https://review.openstack.org/630661 | 17:36 |
| *** ykarel is now known as ykarel|away | 17:41 | |
| *** sdake has joined #openstack-infra | 17:43 | |
| openstackgerrit | Clark Boylan proposed openstack-infra/system-config master: Add zuul.opendev.org webserver https://review.openstack.org/633783 | 17:45 |
| *** agopi|lunch is now known as agopi | 17:46 | |
| clarkb | is there a new release of pep8? seems like a lot of failures happening in openstack integrated gate due to pep8 failures | 17:46 |
| *** e0ne has joined #openstack-infra | 17:46 | |
| clarkb | yup pycodestyle 2.5.0 just released and that is what the jobs that fail are using | 17:47 |
| clarkb | prometheanfire: tonyb ^ fyi that may be something requirements team needs to look at | 17:47 |
| openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient master: Add a resource subcontroller for attachments https://review.openstack.org/633611 | 17:49 |
| openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient master: Display download links for files attached to stories https://review.openstack.org/633612 | 17:49 |
| *** anteaya has joined #openstack-infra | 17:53 | |
| *** aojea has quit IRC | 17:58 | |
| *** derekh has quit IRC | 18:01 | |
| *** diablo_rojo has joined #openstack-infra | 18:01 | |
| openstackgerrit | David Shrewsbury proposed openstack-infra/nodepool master: WIP Revert "Revert "Add a timeout for the image build"" https://review.openstack.org/633792 | 18:02 |
| prometheanfire | clarkb: https://review.openstack.org/633783 ? | 18:02 |
| clarkb | prometheanfire: no pycodestyle 2.5.0 released a few hours ago and has broken pep8 jobs for at least neutron and keystone | 18:02 |
| prometheanfire | k | 18:02 |
| clarkb | prometheanfire: I've pinged them about possibly fixing their errors, but wasn't sure if requirements team had thoughts about maybe pinning? | 18:03 |
| prometheanfire | clarkb: that's controlled per project | 18:03 |
| clarkb | roger | 18:03 |
| prometheanfire | https://github.com/openstack/requirements/blob/master/blacklist.txt#L14 | 18:03 |
| clarkb | ah ok | 18:03 |
| prometheanfire | since everyone likes to use a diferent versions for linting | 18:03 |
| clarkb | so then neutron and keystone should fix locally | 18:03 |
| prometheanfire | they can just pin/whatever it | 18:04 |
| prometheanfire | ya | 18:04 |
| *** jamesmcarthur has quit IRC | 18:06 | |
| *** jamesmcarthur has joined #openstack-infra | 18:08 | |
| *** jtomasek has quit IRC | 18:10 | |
| *** sdake has quit IRC | 18:10 | |
| ssbarnea|rover | clarkb: prometheanfire as you mentioned the new linter breaking things kind of issue. that is one of the reasons why I love running linters via pre-commit as it always locks them, you never get surprised by a new linter and updating all linter is as simple as running "pre-commit autoupdate". | 18:15 |
| clarkb | ssbarnea|rover: the problem in this case is pycodestyle broke on the existing code | 18:15 |
| *** jpena is now known as jpena|off | 18:15 | |
| diablo_rojo | frickler, around? | 18:16 |
| ssbarnea|rover | clarkb: yeah, but it broke because the project was not pinning pycodestyle to specific version, right? | 18:16 |
| clarkb | ssbarnea|rover: ya I think they must do <3.0 | 18:16 |
| clarkb | so its bound but not specific | 18:16 |
| prometheanfire | likely | 18:16 |
| ssbarnea|rover | yep, that's the easy/fast fix | 18:17 |
| prometheanfire | iirc upper-constraints can only be called once, so combining it with another local linters-requirements list would be needed (or two pip install calls) | 18:17 |
| *** kjackal has quit IRC | 18:17 | |
| corvus | wasn't hacking supposed to pin pep8? forget i asked. | 18:18 |
| openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient master: Make tags section more consistent https://review.openstack.org/633613 | 18:18 |
| *** bhavikdbavishi has quit IRC | 18:20 | |
| clarkb | http://git.openstack.org/cgit/openstack-dev/hacking/tree/requirements.txt apparently not | 18:20 |
| *** trown|lunch is now known as trown | 18:21 | |
| prometheanfire | I'd hope not | 18:21 |
| clarkb | prometheanfire: it did in the past | 18:21 |
| prometheanfire | ya, it was annoying (as a packager) | 18:22 |
| ssbarnea|rover | corvus: clarkb: unrelated to current issue with pycodestyle, i am curious if you would be pro/con starting to run linters via pre-commit in infra related projects. | 18:22 |
| clarkb | ssbarnea|rover: I don't think it would help us, as we'd still ahve to enforce it on the server side | 18:23 |
| clarkb | you can already set up pre commit hooks locally if you want to (and maybe we should document that), but we can't enforce it remotely | 18:24 |
| ssbarnea|rover | clarkb: to be clear: when I say pre-commit i mean the project to the "git-pre-commit hooks". the tool does not depend on being run as a hook, this being fully optional to each user. on CI we use it inside tox -e linters | 18:24 |
| clarkb | ssbarnea|rover: git pre commit hooks run when you git commit, How does that fit into tox? | 18:25 |
| clarkb | I guess I don't quite understand | 18:26 |
| ssbarnea|rover | clarkb: sure, everyone that I told about this got confused because we all know how git commit hooks work and when we hear "pre-commit" we assume is a git hook. | 18:29 |
| *** smarcet has joined #openstack-infra | 18:30 | |
| clarkb | well there is a pre-commit git hook so ya | 18:30 |
| ssbarnea|rover | check https://github.com/openstack/tripleo-quickstart/blob/master/tox.ini#L49 which is how you call it as a module (pre-commit cli also works). | 18:30 |
| clarkb | "The pre-commit hook is run first, before you even type in a commit message." | 18:30 |
| clarkb | ssbarnea|rover: how is using a different tool better than just running the linters? | 18:30 |
| clarkb | ssbarnea|rover: does that buy us something we don't already have? | 18:30 |
| ssbarnea|rover | its config looks like https://github.com/openstack/tripleo-quickstart/blob/master/.pre-commit-config.yaml -- as you can see there "bashare" ansible-lint and other linters defined, All of them pinned to specific revisions. | 18:31 |
| ssbarnea|rover | clarkb: you know hacking, right? pre-commit is like hacking-next-gen. | 18:31 |
| ssbarnea|rover | it does pinning, easy updating versions, optimize installation (reuse linters across projects when version matches). | 18:32 |
| clarkb | ok so pre-commit is a generic tool to run linters and similar tools | 18:32 |
| ssbarnea|rover | clarkb: exactly! | 18:32 |
| clarkb | and it manages versions very specifically | 18:32 |
| clarkb | we do already have the ability to pin these tools to specific versions, so we probably need to decide if that is desireable and if it is why we haven't done it with the existing tools | 18:33 |
| *** electrofelix has quit IRC | 18:34 | |
| *** diablo_rojo has quit IRC | 18:35 | |
| clarkb | ok I'm going to step out for a bit prior to the meeting. See you all in ~25 minutes | 18:35 |
| *** sdake has joined #openstack-infra | 18:37 | |
| openstackgerrit | Sorin Sbarnea proposed openstack-infra/git-review master: Switch from testrepository to stestr https://review.openstack.org/628655 | 18:43 |
| *** hamzy has joined #openstack-infra | 18:52 | |
| *** e0ne has quit IRC | 18:53 | |
| *** yamamoto has joined #openstack-infra | 19:04 | |
| *** ykarel|away has quit IRC | 19:06 | |
| *** whoami-rajat has quit IRC | 19:07 | |
| *** e0ne has joined #openstack-infra | 19:07 | |
| *** dtantsur is now known as dtantsur|afk | 19:17 | |
| openstackgerrit | James E. Blair proposed openstack-infra/system-config master: WIP: run a gerrit container https://review.openstack.org/630406 | 19:20 |
| rm_work | anyone aware of an issue with the publish jobs due to Twine being broken? | 19:39 |
| rm_work | like: http://logs.openstack.org/94/624294/9/gate/test-release-openstack/d804b41/job-output.txt.gz#_2019-01-29_18_38_41_409378 | 19:39 |
| *** kjackal has joined #openstack-infra | 19:40 | |
| *** harlowja has joined #openstack-infra | 19:44 | |
| rm_work | http://logs.openstack.org/94/624294/9/check/test-release-openstack/891bbd4/ara-report/result/8513865e-c4a4-48ff-ae90-2d47c88d9a3e/ | 19:52 |
| *** raissa has joined #openstack-infra | 19:53 | |
| *** bobh has quit IRC | 19:54 | |
| *** luizbag has quit IRC | 19:59 | |
| clarkb | rm_work: looks like ubuntu xenial python3.5 doesn't have an OpenSSL module | 20:01 |
| clarkb | rm_work: I just checked locally and get a similar traceback from import OpenSSL | 20:01 |
| *** eernst has quit IRC | 20:01 | |
| clarkb | rm_work: my hunch is that the fix is to add whatever package adds that module to the bindep.txt for your project | 20:02 |
| cmurphy | clarkb: am i right in remembering that the puppet master no longer has puppet installed on it? should get rid of these silly mgmt_puppet_version things http://git.openstack.org/cgit/openstack-infra/ansible-role-puppet/tree/tasks/main.yaml#n3 | 20:02 |
| cmurphy | though i'm not sure how that doesn't error out if puppet isn't installed | 20:03 |
| clarkb | rm_work: python3-openssl ? | 20:03 |
| *** raissa has quit IRC | 20:03 | |
| clarkb | cmurphy: we must set mgmt_hieradata ? | 20:03 |
| *** yamamoto has quit IRC | 20:05 | |
| *** raissa has joined #openstack-infra | 20:05 | |
| *** raissa has quit IRC | 20:05 | |
| rm_work | clarkb: hmm but this worked until today | 20:05 |
| clarkb | cmurphy: http://git.openstack.org/cgit/openstack-infra/system-config/tree/playbooks/group_vars/puppet.yaml | 20:05 |
| rm_work | so i assume twine broke something (though i need to check their releases) | 20:05 |
| rm_work | but i guess we can add the bindep... | 20:05 |
| clarkb | rm_work: its possible twine was relying on cryptography before? | 20:05 |
| rm_work | i would just assume that this broke much more than us | 20:06 |
| rm_work | but possibly other projects just happened to have that installed | 20:06 |
| rm_work | clarkb: do i need some magic to only install python3-openssl on py3 tests? | 20:07 |
| *** raissa has joined #openstack-infra | 20:07 | |
| rm_work | or is it safe to just do that generically | 20:07 |
| *** raissa has quit IRC | 20:07 | |
| *** eernst has joined #openstack-infra | 20:08 | |
| clarkb | rm_work: it should be safe to do it generically, it will just add a small amount of time to the job. | 20:09 |
| *** agopi is now known as agopi|brb | 20:10 | |
| *** agopi|brb has quit IRC | 20:10 | |
| *** eernst has quit IRC | 20:12 | |
| *** fdegir_ has joined #openstack-infra | 20:14 | |
| *** eernst has joined #openstack-infra | 20:14 | |
| openstackgerrit | Clark Boylan proposed openstack-infra/zone-opendev.org master: Verify zuul.opendev.org cert https://review.openstack.org/633817 | 20:18 |
| openstackgerrit | Clark Boylan proposed openstack-infra/zone-opendev.org master: Cleanup cert verification record https://review.openstack.org/633818 | 20:18 |
| clarkb | infra-root ^ I think that is the last major piece of that puzzle | 20:18 |
| clarkb | now to review the dns management docs change | 20:19 |
| *** eernst has quit IRC | 20:19 | |
| *** eernst has joined #openstack-infra | 20:21 | |
| corvus | +3 | 20:22 |
| openstackgerrit | Merged openstack-infra/zone-opendev.org master: Verify zuul.opendev.org cert https://review.openstack.org/633817 | 20:22 |
| *** eernst has quit IRC | 20:25 | |
| *** eernst has joined #openstack-infra | 20:27 | |
| *** eernst has quit IRC | 20:31 | |
| openstackgerrit | Brian Rosmaita proposed openstack-infra/irc-meetings master: Update glance meeting info https://review.openstack.org/633824 | 20:31 |
| *** jamesmcarthur has quit IRC | 20:32 | |
| *** eernst has joined #openstack-infra | 20:33 | |
| *** jamesmcarthur has joined #openstack-infra | 20:33 | |
| *** zaneb has joined #openstack-infra | 20:36 | |
| *** eernst has quit IRC | 20:37 | |
| *** eernst has joined #openstack-infra | 20:39 | |
| *** jtomasek has joined #openstack-infra | 20:41 | |
| tobias-urdin | pabelanger: could you check out https://review.openstack.org/#/c/632699/ thanks! :) | 20:41 |
| *** e0ne has quit IRC | 20:42 | |
| openstackgerrit | Paul Vinciguerra proposed openstack-infra/zuul master: configloader.py: Not all jobs have an .updated attribute. https://review.openstack.org/633259 | 20:43 |
| *** eernst has quit IRC | 20:43 | |
| *** sdake has quit IRC | 20:45 | |
| *** eernst has joined #openstack-infra | 20:45 | |
| *** eernst has quit IRC | 20:49 | |
| *** eernst has joined #openstack-infra | 20:51 | |
| *** agopi|brb has joined #openstack-infra | 20:54 | |
| *** agopi|brb is now known as agopi | 20:54 | |
| *** eernst has quit IRC | 20:54 | |
| *** xek has quit IRC | 20:54 | |
| *** eernst has joined #openstack-infra | 20:55 | |
| *** fdegir_ is now known as fdegir | 20:55 | |
| openstackgerrit | Tobias Henkel proposed openstack/gertty master: Add inline comments to change overview https://review.openstack.org/633825 | 21:02 |
| * corvus is running with that change now ^ :) | 21:03 | |
| tobiash | corvus: this is an attempt to add the inline comments on the change screen in gertty so one can follow discussions on changes without having to switch to the diff screen ^ | 21:03 |
| mriedem | do we have a bug reported for that pep8 error? | 21:03 |
| mriedem | prometheanfire: ^? | 21:03 |
| clarkb | mriedem: I didn't file a bug but notified the keystone and neutron teams (they seemed to be affected) | 21:04 |
| mriedem | tempest is also http://logs.openstack.org/66/633566/1/check/openstack-tox-pep8/b1da858/job-output.txt.gz#_2019-01-29_17_13_12_367701 | 21:04 |
| mriedem | which is blocking getting the skip in for the other test that is killing the gate | 21:04 |
| mriedem | the udhcp one | 21:04 |
| corvus | tobiash: i think i might suggest a different color, possibly dark cyan, as light cyan gets a little confused with the header for my own messages. | 21:04 |
| tobiash | yeah, noticed that too now | 21:05 |
| tobiash | corvus: also I have no idea how the inline messages could be indented after the line number | 21:06 |
| corvus | tobiash: yeah, i was thinking the same thing... possibly something in https://docs.python.org/3/library/textwrap.html might help | 21:07 |
| prometheanfire | mriedem: pep8 is in blacklist.txt, it's controlled via devstack initilization iirc | 21:07 |
| mriedem | this wouldn't be devstack jobs though | 21:07 |
| mriedem | and it's pycodestyle right? | 21:07 |
| prometheanfire | also in blacklist | 21:08 |
| prometheanfire | we don't track those | 21:08 |
| mriedem | ok so i just need to fix tempest | 21:08 |
| prometheanfire | ya | 21:08 |
| mriedem | ok | 21:08 |
| corvus | tobiash: though right now, urwid is handling the line wrapping, so that could get weird if the terminal got too small. so textwrap is probably not a good idea. | 21:08 |
| prometheanfire | constrain it there, if reqs gate rejects the change we are doing something wrong | 21:08 |
| prometheanfire | https://github.com/openstack/requirements/blob/master/blacklist.txt | 21:09 |
| corvus | tobiash: we'd probably need a new urwid widget, which is a fun weekend activity. :) | 21:09 |
| tobiash | urwid is totally new to me... ;) | 21:09 |
| corvus | tobiash: like maybe adding it as a two-column box, with the line number in the first column, and the message in the second. | 21:09 |
| prometheanfire | it's neat, I've not needed it yet though | 21:10 |
| corvus | tobiash: it's powerful and fun, except that if you get *anything* wrong, it tends to tell you by raising an exception. :) | 21:10 |
| corvus | tobiash: i think the color change is the only thing we should do before landing that. we can improve the formatting later. | 21:10 |
| openstackgerrit | Tobias Henkel proposed openstack/gertty master: Add inline comments to change overview https://review.openstack.org/633825 | 21:12 |
| corvus | tobiash: oh i think i just saw a bug but was too dense to realize it and lost the traceback. | 21:13 |
| corvus | got it | 21:13 |
| corvus | tobiash: http://paste.openstack.org/show/744200/ | 21:14 |
| tobiash | that's triggered by a debugging leftover | 21:15 |
| tobiash | just a sec | 21:15 |
| openstackgerrit | Tobias Henkel proposed openstack/gertty master: Add inline comments to change overview https://review.openstack.org/633825 | 21:15 |
| openstackgerrit | Merged openstack/gertty master: Do not decode failed POST response https://review.openstack.org/626076 | 21:16 |
| corvus | tobiash: same thing when i just review without leaving a comment: http://paste.openstack.org/show/744201/ | 21:17 |
| corvus | and that's with the latest ps | 21:17 |
| openstackgerrit | Merged openstack/gertty master: Fix crash on python3 https://review.openstack.org/619938 | 21:17 |
| corvus | oh hrm, it couldn't be with the latest, sorry | 21:17 |
| openstackgerrit | Sean McGinnis proposed openstack-infra/zuul-jobs master: Make sure urllib3[secure] is installed for Twine use https://review.openstack.org/633829 | 21:19 |
| corvus | tobiash: ok, confirmed that fixes it | 21:19 |
| tobiash | :) | 21:19 |
| corvus | tobiash: i added some edge cases there too. a file comment shows up as "None:" which is not wrong, but maybe could be better. :) | 21:20 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/nodepool master: WIP Revert "Revert "Add a timeout for the image build"" https://review.openstack.org/633792 | 21:20 |
| corvus | at least it doesn't crash :) | 21:21 |
| clarkb | corvus: https://review.openstack.org/633818 can be merged at this point (I am adding cert details to hiera now) | 21:22 |
| corvus | clarkb: done | 21:22 |
| openstackgerrit | Tobias Henkel proposed openstack/gertty master: Add inline comments to change overview https://review.openstack.org/633825 | 21:22 |
| openstackgerrit | Merged openstack-infra/zone-opendev.org master: Cleanup cert verification record https://review.openstack.org/633818 | 21:23 |
| tobiash | that makes the file comment nicer ^ | 21:23 |
| corvus | ++ | 21:23 |
| tonyb | prometheanfire: Nope you can have multiple constraint files but if a library is in both it'll cause pip to barf | 21:24 |
| prometheanfire | ah | 21:25 |
| prometheanfire | maybe it was requirements that had that behavior | 21:25 |
| clarkb | hiera/ansible vars all configured https://review.openstack.org/#/c/633783/2 and its depends on should be ready when we are | 21:27 |
| corvus | clarkb: how long until you drop? | 21:27 |
| clarkb | corvus: its getting close :) though I can probably keep an eyeball on things if you want to hit go buttons | 21:28 |
| clarkb | I was packing while waiting on namecheap to verify :) | 21:28 |
| *** kgiusti has left #openstack-infra | 21:28 | |
| clarkb | can pack while waiting for puppet to run | 21:28 |
| corvus | buttons have been hit | 21:29 |
| clarkb | exciting | 21:29 |
| *** diablo_rojo has joined #openstack-infra | 21:31 | |
| *** gfidente has quit IRC | 21:32 | |
| *** jcoufal has quit IRC | 21:39 | |
| *** hamzy has quit IRC | 21:42 | |
| smcginnis | clarkb, others: We appear to have release failures due to twine that this is (hopefully) fixing: https://review.openstack.org/#/c/633829/ | 21:48 |
| smcginnis | Any eyes there would be appreciated. | 21:48 |
| *** eharney has quit IRC | 21:48 | |
| corvus | smcginnis: have a link to the failure, and do you know what changed? | 21:48 |
| clarkb | smcginnis: I think that will require libssl-dev (or whatever it is called on $distro) for the linkaging to happen | 21:49 |
| smcginnis | corvus: This is an example - http://logs.openstack.org/76/767f3ea3efcedb4c8df1dae1a6b40d0b0a230d3d/release/release-openstack-python/9701f8c/ara-report/result/03681c3b-d551-4581-9f7b-96fb4f5df174/ | 21:49 |
| smcginnis | clarkb: I believe that's in our default set, IIRC. | 21:49 |
| smcginnis | corvus: Not sure what changed, but twine requires requests which requires urllib3 which only installs all necessary packages with urllib3[security]. | 21:50 |
| corvus | this is running on the executor | 21:51 |
| corvus | libssl-dev is installed there | 21:51 |
| smcginnis | Would I need to update a bindep somewhere to make sure that has it? | 21:51 |
| smcginnis | OK, good. | 21:51 |
| johnsom | There were both changes in project-config and urllib3 about the same time, both could be contributing | 21:51 |
| clarkb | if I had to guess urllib3 stopped pulling it by default | 21:52 |
| clarkb | probably because it requires the header packages to build most places | 21:52 |
| smcginnis | That area of their setup.cfg changed in Nov I think. | 21:52 |
| johnsom | They moved how they were handling the "extras_requires" | 21:52 |
| clarkb | though it like twine hasn't changed super recently | 21:52 |
| smcginnis | Couple layers of indirection with this one I think. | 21:53 |
| johnsom | https://github.com/urllib3/urllib3/commit/f80ff34bffb2018528ccd2ccf82e533d321f2af8 | 21:53 |
| corvus | johnsom: know the project-config changes that might be relevant? | 21:53 |
| johnsom | corvus This: https://review.openstack.org/#/c/616673 and https://github.com/openstack-infra/project-config/commit/4f3b36607738be78985982c3dc935836e9a1814d | 21:54 |
| johnsom | Switched this over to python3 | 21:54 |
| johnsom | About the same time window with the the urllib3 change. | 21:55 |
| corvus | that merged a long time ago, right? but it looks like release-openstack-python succeded at 15:00 today | 21:55 |
| clarkb | is it possible that our newer executors don't have python3-openssl installed but the older onces do (or vice versa?) | 21:55 |
| clarkb | maybe it is executor specific | 21:56 |
| smcginnis | Or it could depend if the project in question has requirements that pull it in. | 21:56 |
| corvus | ran on ze07 | 21:56 |
| clarkb | smcginnis: oh ya or that | 21:56 |
| corvus | smcginnis: i don't believe the upload job is affected by the project in any way | 21:56 |
| smcginnis | Hmm | 21:56 |
| corvus | libssl-dev is installed on ze07 | 21:57 |
| johnsom | It wasn't getting as far as libssl-dev though, it is missing the python module | 21:58 |
| clarkb | ya it never tries to install the pyopenssl or cryptography stuff. It just assumes it is there | 21:59 |
| clarkb | but that change would've been back in november? maybe some other dep was pulling it in and they fixed that? | 22:00 |
| corvus | https://pypi.org/project/requests-toolbelt/#history | 22:00 |
| *** jtomasek has quit IRC | 22:01 | |
| *** trown is now known as trown|outtypewww | 22:01 | |
| corvus | and here's a diff between the last success and first failure: http://paste.openstack.org/show/744210/ | 22:02 |
| corvus | the surprising thing is -- shouldn't we see pyopenssl or crytography or something on the old line? | 22:03 |
| smcginnis | I was wondering the same thing. | 22:04 |
| clarkb | corvus: ya if it was transitive I think you would | 22:04 |
| smcginnis | Not seeing anyting yet in https://github.com/requests/toolbelt/compare/0.8.0...0.9.0 | 22:05 |
| smcginnis | But there's the typical pull request commit message quality. | 22:05 |
| johnsom | It's this patch: https://github.com/requests/toolbelt/commit/675fab790b0e9fefd6beb0f38d9a5da4b056c59c | 22:05 |
| *** eernst has quit IRC | 22:05 | |
| johnsom | They added the dependency in tox.ini and dev-requirements.txt but not setup.py | 22:05 |
| pabelanger | I was just about to link that.. | 22:05 |
| smcginnis | Good eye(s) | 22:06 |
| pabelanger | yah, I think pyopenssl should be in requirements / setup.py | 22:06 |
| johnsom | https://github.com/requests/toolbelt/issues/241 | 22:06 |
| pabelanger | https://github.com/requests/toolbelt/pull/242 | 22:07 |
| corvus | so would urllib3[secure] get what we need, or do we nee to add pyopenssl? | 22:08 |
| johnsom | urllib3[secure] includes pyopenssl and the other dependencies. Question is, do we want to add this or somehow exclude that version of requests_toolbelt | 22:09 |
| clarkb | corvus: I think urllib3[secure] pulls in pyopenssl (and others). Probably thye most correct thing is to install only pyopenssl | 22:09 |
| clarkb | johnsom: oh ya if we just exclude 0.9.0 of requests-toolbelt then theoretically their next release will have that fix pabelangerlinked | 22:09 |
| smcginnis | It shouldn't hurt anything, but we could also block 0.9.0. | 22:09 |
| *** roman_g has quit IRC | 22:09 | |
| *** ekcs has joined #openstack-infra | 22:10 | |
| *** eernst has joined #openstack-infra | 22:10 | |
| *** eernst has quit IRC | 22:11 | |
| corvus | maybe that's what we should do? | 22:11 |
| clarkb | ya I think that is the best option, block 0.9.0 | 22:11 |
| smcginnis | Problem with that is we don't install requests-toolbelt, twine does. | 22:11 |
| clarkb | smcginnis: top level overrides should win | 22:12 |
| clarkb | so if we do pip install requests-toolbelt!=0.9.0 twine then the version we specified in that isntall will win | 22:12 |
| clarkb | (this is not true if you install them in two different pip runs) | 22:12 |
| smcginnis | I can update my patch to do that. | 22:12 |
| corvus | smcginnis: if you respin, can you throw in a link to https://github.com/requests/toolbelt/issues/241 in the commit message? | 22:13 |
| openstackgerrit | Sean McGinnis proposed openstack-infra/zuul-jobs master: Block installation of requests-toolbelt 0.9.0 https://review.openstack.org/633829 | 22:15 |
| corvus | normally we'd do a lot of testing for a change like that, but i'm pretty sure everyone using that is broken so we can just merge it and test in prod. | 22:17 |
| smcginnis | The best way to test. :) | 22:17 |
| *** sdake has joined #openstack-infra | 22:17 | |
| johnsom | We can re-check https://review.openstack.org/#/c/624294/, which is a quick job to test and revert if it doesn't solve it. | 22:18 |
| corvus | we have a test-release-openstack job? that's awesome :) | 22:18 |
| *** rh-jelabarre has quit IRC | 22:18 | |
| pabelanger | yah, I think it still uploads to test-pypi | 22:19 |
| corvus | i don't see any secrets for it | 22:20 |
| corvus | if we moved that job to openstack-zuul-jobs, we could use it to live-test changes like this | 22:20 |
| corvus | i assume it's in project-config to share the playbook | 22:20 |
| corvus | i bet we could rework this to do what we want though. | 22:21 |
| openstackgerrit | Merged openstack-infra/puppet-zuul master: Allow for both global zuul web and whitelabeled tenant https://review.openstack.org/633586 | 22:21 |
| openstackgerrit | Adam Coldrick proposed openstack-infra/storyboard-webclient master: WIP: Add a button for uploading attachments https://review.openstack.org/633840 | 22:21 |
| openstackgerrit | Merged openstack-infra/system-config master: Add zuul.opendev.org webserver https://review.openstack.org/633783 | 22:21 |
| corvus | pre and run in untrusted, then post in trusted | 22:23 |
| clarkb | here we go. The last step if ^ looks good is to set up dns I think | 22:23 |
| corvus | clarkb: let's go ahead and do dns | 22:23 |
| clarkb | ok I'll get a change up for that | 22:24 |
| corvus | there is no existing zuul.opendev.org, so if it fails, it's no big deal | 22:24 |
| openstackgerrit | Clark Boylan proposed openstack-infra/zone-opendev.org master: Add zuul.opendev.org record https://review.openstack.org/633841 | 22:26 |
| clarkb | that should do it | 22:26 |
| openstackgerrit | Merged openstack-infra/zone-opendev.org master: Add zuul.opendev.org record https://review.openstack.org/633841 | 22:27 |
| clarkb | looking at puppet logs on zuul01 I expect that 23:06UTC ish is when the new stuff will apply | 22:32 |
| *** gyee has quit IRC | 22:35 | |
| *** slaweq has quit IRC | 22:36 | |
| ekcs | not sure if this is the right place, but anyone got pointers on how to add a new member to project core team in Gerrit? | 22:45 |
| ekcs | The ptl guide just says: "Update the core team in Gerrit." And I have scoured codesearch.o.o but so far failed to find what to do. | 22:45 |
| clarkb | ekcs: in gerrit itself if you are logged in you can browse the groups at https://review.openstack.org/#/admin/groups/ use the little search bar there to find teh group you want, click on it, then add members on the members pane | 22:46 |
| corvus | ekcs: this may help: https://review.openstack.org/Documentation/intro-project-owner.html#groups | 22:46 |
| ekcs | ahhh of course. had to be the most obvious place =p thanks guys! | 22:47 |
| clarkb | that link above is navigable from gerrit via the people tab -> list groups | 22:47 |
| ekcs | got it. thanks a lot! | 22:55 |
| *** tkajinam has joined #openstack-infra | 22:55 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul-jobs master: add-build-sshkey: remove previously authorized build-sshkey https://review.openstack.org/632620 | 23:06 |
| clarkb | well that seems to have made apache have a sad :/ | 23:07 |
| clarkb | I'm looking into why now | 23:07 |
| *** diablo_rojo has quit IRC | 23:07 | |
| smcginnis | Not sure if that ^ impacted anything, but I am getting a Not Found (404) error on my zuul.openstack.org page I have up. | 23:07 |
| *** jamesmcarthur has quit IRC | 23:08 | |
| corvus | aw man, we're 0 for 2. | 23:08 |
| johnsom | Me too | 23:08 |
| clarkb | https works | 23:08 |
| clarkb | but http does not | 23:08 |
| smcginnis | Full refresh, Not Found error page. | 23:08 |
| johnsom | +1 https | 23:08 |
| corvus | we should just redirect http to https (which, i expect, many of us thought was happening already) | 23:08 |
| clarkb | smcginnis: try https | 23:08 |
| smcginnis | Confirmed switching to https works. | 23:08 |
| clarkb | corvus: ya I'm confused as to how http worked in the past | 23:09 |
| smcginnis | corvus: I thought it used to work that way. | 23:09 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Block installation of requests-toolbelt 0.9.0 https://review.openstack.org/633829 | 23:09 |
| clarkb | the good new is its only half broken :) | 23:09 |
| corvus | yeah. i think we should roll forward | 23:09 |
| clarkb | zuul.opendev.org https vhost also working for me. GIven that ya I think roll forward is good if we can figure out why http stopped working | 23:11 |
| clarkb | how about I send #status notice http://zuul.openstack.org is not working. https://zuul.openstack.org does work. Please use that while we investigate. | 23:11 |
| corvus | ++ | 23:11 |
| clarkb | #status notice http://zuul.openstack.org is not working. https://zuul.openstack.org does work. Please use that while we investigate. | 23:12 |
| openstackstatus | clarkb: sending notice | 23:12 |
| -openstackstatus- NOTICE: http://zuul.openstack.org is not working. https://zuul.openstack.org does work. Please use that while we investigate. | 23:13 | |
| corvus | clarkb: we used to always create an http vhost, and optionally create an ssl vhost | 23:13 |
| clarkb | ah | 23:14 |
| clarkb | corvus: what if we add a port 80 to the vhost template that unconditionally redirects to $port | 23:14 |
| clarkb | (and start assuming https?) | 23:14 |
| corvus | clarkb: that wfm, but may not be backwards compatible for other users of the module? | 23:15 |
| clarkb | ya it is less backward compatible whereas I tried to be backward compat with the older changes | 23:15 |
| openstackstatus | clarkb: finished sending notice | 23:15 |
| corvus | (mind you, i'm trying to get out of the business of maintaining modules for other users, but i think we decided to do that by switching to ansible) | 23:15 |
| clarkb | what we can do is add back in port 80 vhosts | 23:15 |
| clarkb | just list them without the ssl | 23:15 |
| corvus | clarkb: basically duplicate the data structure that's there to also have a non-ssl version? | 23:16 |
| clarkb | but I think what we really want is to redirect | 23:16 |
| clarkb | corvus: ya I think so | 23:16 |
| corvus | clarkb: i think we should add the extra vhost so that we don't have to deal with puppet-zuul more than necessary, then redirect when we switch to ansible. | 23:16 |
| clarkb | wfm | 23:16 |
| corvus | clarkb: you want to write that or do you want me to? | 23:17 |
| clarkb | I'll do it really quick | 23:17 |
| johnsom | Yep, that requests_toolbelt fix worked | 23:18 |
| corvus | johnsom: \o/ need me to re-enqueue anything? | 23:18 |
| johnsom | No, I am good. Thanks | 23:18 |
| *** jamesmcarthur has joined #openstack-infra | 23:19 | |
| openstackgerrit | Clark Boylan proposed openstack-infra/system-config master: Add http vhosts for zuul.o.o https://review.openstack.org/633845 | 23:20 |
| clarkb | corvus: ^ I think that should work? | 23:20 |
| corvus | +3 | 23:21 |
| *** jamesmcarthur has quit IRC | 23:23 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add /{tenant}/buildsets route https://review.openstack.org/630035 | 23:28 |
| clarkb | ok I'm going to pop out now for early dinner/late lunch. I'll be back to check on the http situation after | 23:29 |
| *** sdake has quit IRC | 23:35 | |
| *** sdake has joined #openstack-infra | 23:37 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: WIP add provides/requires support https://review.openstack.org/633605 | 23:39 |
| smcginnis | Would someone be able to reenqueu a failed job from earlier? | 23:44 |
| smcginnis | http://logs.openstack.org/76/767f3ea3efcedb4c8df1dae1a6b40d0b0a230d3d/release/release-openstack-python/9701f8c/ | 23:44 |
| smcginnis | This was due to the twine>requests-toolbelt>urllib3 openssl issue. | 23:44 |
| smcginnis | I've verified the config change fixed this with a release-test. | 23:44 |
| *** rcernin has quit IRC | 23:53 | |
| *** sdake has quit IRC | 23:55 | |
| *** sdake has joined #openstack-infra | 23:55 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!