| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [FOR DISCUSSION] Revert "Ensure wheel exists for build-release-python" https://review.opendev.org/736402 | 00:03 |
|---|---|---|
| *** rfolco|rover has joined #zuul | 00:05 | |
| *** jamesmcarthur has joined #zuul | 00:10 | |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "Use zuul jobs" https://review.opendev.org/732068 | 00:12 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "gate pipeline" https://review.opendev.org/732069 | 00:12 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "job secrets" https://review.opendev.org/732070 | 00:12 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "job dependencies" https://review.opendev.org/732071 | 00:12 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Fix zuul_run_command ret when command is not found https://review.opendev.org/735475 | 00:12 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: zuul_stream: close streamer when task is skipped https://review.opendev.org/735476 | 00:12 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: DNM: execute zuul tutorial jobs with pending zuul-jobs changes https://review.opendev.org/735477 | 00:12 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: POC: test job multinode (zuul_stream+callback test) https://review.opendev.org/735483 | 00:12 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: WIP: Add 'uuid' to 'src_dir' in order to allow parallel jobs for a static node https://review.opendev.org/735981 | 00:12 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: POC: test 'uuid' in 'src_dir', 1 static/4 parallel jobs https://review.opendev.org/735576 | 00:12 |
| *** Goneri has quit IRC | 00:22 | |
| *** jamesmcarthur has quit IRC | 00:24 | |
| mordred | ianw: the original fix for wheel used become - and broken ansible's zuul | 00:27 |
| ianw | mordred: https://review.opendev.org/#/c/736001/5/roles/build-python-release/tasks/main.yaml isn't in the pre-run phase where it already uses "ensure-pip" though, right? | 00:29 |
| mordred | ianw: there was no ensure-pip in a pre at that point - so mayb ethat would be the fix | 00:32 |
| mordred | honestly - I'm super confused by what's supposed to work and what's not at this point | 00:33 |
| mordred | so if there is a better way - cool | 00:34 |
| ianw | name: build-python-release | 00:34 |
| ianw | pre-run: playbooks/python/pre.yaml | 00:34 |
| ianw | which runs ensure-pip | 00:34 |
| ianw | it looks like the "wheel" binary is in python-wheel-common, which is separate to python3-wheel ... sigh | 00:35 |
| mordred | well - we don't need the wheel binary | 00:36 |
| mordred | we need bdist_wheel | 00:36 |
| mordred | so that python setup.py bdist wheel works :) | 00:36 |
| mordred | so maybe that makes it easier | 00:37 |
| * mordred has to EOD ... | 00:37 | |
| ianw | mordred: yeah i think it does, and if for some reason we *need* the wheel binary, it would be logical to add it along there if someone just wants to unravel what package goes where | 00:37 |
| ianw | mordred: ttyl :) | 00:38 |
| *** rfolco|rover has quit IRC | 00:47 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [FOR DISCUSSION] Revert "Ensure wheel exists for build-release-python" https://review.opendev.org/736402 | 00:55 |
| *** jamesmcarthur has joined #zuul | 00:58 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [FOR DISCUSSION] Revert "Ensure wheel exists for build-release-python" https://review.opendev.org/736402 | 00:58 |
| *** jamesmcarthur has quit IRC | 01:00 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [FOR DISCUSSION] Revert "Ensure wheel exists for build-release-python" https://review.opendev.org/736402 | 01:01 |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [FOR DISCUSSION] Revert "Ensure wheel exists for build-release-python" https://review.opendev.org/736402 | 01:11 |
| *** jamesmcarthur has joined #zuul | 01:14 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [FOR DISCUSSION] Revert "Ensure wheel exists for build-release-python" https://review.opendev.org/736402 | 01:19 |
| *** rfolco|rover has joined #zuul | 01:23 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [FOR DISCUSSION] Revert "Ensure wheel exists for build-release-python" https://review.opendev.org/736402 | 01:26 |
| ianw | writing it is easy ... testing it properly is ... not :) | 01:26 |
| ianw | why are we testing stretch and not buster? | 01:30 |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [FOR DISCUSSION] Revert "Ensure wheel exists for build-release-python" https://review.opendev.org/736402 | 01:41 |
| *** jamesmcarthur has quit IRC | 01:43 | |
| *** swest has quit IRC | 01:50 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [FOR DISCUSSION] Revert "Ensure wheel exists for build-release-python" https://review.opendev.org/736402 | 01:51 |
| *** jamesmcarthur has joined #zuul | 02:00 | |
| *** swest has joined #zuul | 02:04 | |
| *** bhavikdbavishi has joined #zuul | 02:49 | |
| *** bhavikdbavishi1 has joined #zuul | 02:52 | |
| *** bhavikdbavishi has quit IRC | 02:53 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 02:53 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [FOR DISCUSSION] Revert "Ensure wheel exists for build-release-python" https://review.opendev.org/736402 | 03:21 |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [FOR DISCUSSION] Revert "Ensure wheel exists for build-release-python" https://review.opendev.org/736402 | 03:31 |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [WIP] Revert "Ensure wheel exists for build-release-python"; move to ensure-pip https://review.opendev.org/736402 | 03:57 |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [WIP] Revert "Ensure wheel exists for build-release-python"; move to ensure-pip https://review.opendev.org/736402 | 04:04 |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: Partial revert "Ensure wheel exists for build-release-python"; move to ensure-pip https://review.opendev.org/736402 | 04:19 |
| *** evrardjp has quit IRC | 04:33 | |
| *** evrardjp has joined #zuul | 04:33 | |
| *** jamesmcarthur has quit IRC | 04:52 | |
| *** jamesmcarthur has joined #zuul | 04:52 | |
| *** ysandeep|away is now known as ysandeep | 04:58 | |
| *** jamesmcarthur has quit IRC | 04:58 | |
| *** sgw1 has quit IRC | 05:03 | |
| *** rlandy|bbl has quit IRC | 05:08 | |
| *** felixedel has joined #zuul | 05:13 | |
| *** jamesmcarthur has joined #zuul | 05:26 | |
| *** bhavikdbavishi has quit IRC | 05:26 | |
| *** jamesmcarthur has quit IRC | 05:38 | |
| *** bhavikdbavishi has joined #zuul | 05:42 | |
| *** paulalbertella has quit IRC | 05:42 | |
| *** reiterative has joined #zuul | 05:43 | |
| *** sanjayu_ has joined #zuul | 05:51 | |
| AJaeger | ianw: could you check https://review.opendev.org/#/c/736379/1/test-playbooks/ensure-pip.yaml , please? | 05:53 |
| AJaeger | mordred, ianw, corvus, clarkb, publish-openstack-python-branch-tarball still fails, see https://zuul.opendev.org/t/openstack/build/61fdd71509e6456397eeb6ca05d186b1 | 05:58 |
| ianw | AJaeger: hrm, that says it's missing setuptools | 06:00 |
| AJaeger | and test-release-openstack is failing as well, https://zuul.opendev.org/t/openstack/build/4f3e5bcc6e77409c90eb3dc08b5fe472 - invalid command 'bdist_wheel' | 06:00 |
| ianw | if it hasn't run ensure-pip, nothing may have brought in setuptools | 06:00 |
| ianw | https://review.opendev.org/736402 would fix that; it brings in setuptools along with pip, as pip: module needs setuptools | 06:01 |
| AJaeger | ianw: great! I didn't have time to go through 402 yet | 06:01 |
| AJaeger | ianw: can we do a depends-on testing test-release-openstack with 402? | 06:02 |
| ianw | ummm, maybe? is there an example from when the ensure-wheel role went in? | 06:03 |
| AJaeger | what about taking one of the fails? | 06:06 |
| ianw | but does that work in the check queue? | 06:06 |
| AJaeger | or check https://zuul.opendev.org/t/openstack/builds?job_name=test-release-openstack | 06:06 |
| AJaeger | yes | 06:06 |
| AJaeger | and test-release-openstack tests parts of the publish part, so if that works, we're nearly there ;) | 06:07 |
| openstackgerrit | Merged zuul/zuul-jobs master: Fix ansible-lint rules tests https://review.opendev.org/736387 | 06:11 |
| ianw | AJaeger: i put in https://review.opendev.org/736443 to test | 06:12 |
| AJaeger | ianw: thanks. I added a lengthy comment to your review to point out what I know so that we can evaluate those together. | 06:15 |
| AJaeger | ianw: found a nit in case you want to respin or do a followup. | 06:15 |
| ianw | AJaeger: thanks ... i mean the original pip-and-virtualenv must have been bringing in wheel somewhat accidentally. afair it was certainly never explicitly installed as part of that, and now it's gone missing | 06:16 |
| AJaeger | ianw: oh fun :( | 06:18 |
| *** yolanda has quit IRC | 06:31 | |
| *** yolanda has joined #zuul | 06:36 | |
| *** noonedeadpunk has joined #zuul | 06:41 | |
| *** rpittau|afk is now known as rpittau | 06:43 | |
| *** hashar has joined #zuul | 06:54 | |
| *** bhavikdbavishi has quit IRC | 07:05 | |
| *** jcapitao has joined #zuul | 07:16 | |
| *** tosky has joined #zuul | 07:25 | |
| felixedel | ianw: What's the state about https://review.opendev.org/#/c/733564/ Did the runs of the test-job went well so far? :) | 07:28 |
| *** harrymichal has joined #zuul | 07:31 | |
| AJaeger | requirements-check is failing sometimes, see https://zuul.opendev.org/t/openstack/builds?job_name=requirements-check with pip3 not found - "pip3 install --user src/opendev.org/openstack/requirements " | 07:34 |
| *** bhavikdbavishi has joined #zuul | 07:36 | |
| * AJaeger is on it | 07:36 | |
| *** bhavikdbavishi1 has joined #zuul | 07:39 | |
| *** bhavikdbavishi has quit IRC | 07:41 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 07:41 | |
| *** harrymichal has quit IRC | 07:43 | |
| *** jpena|off is now known as jpena | 07:57 | |
| *** nils has joined #zuul | 08:08 | |
| *** ysandeep is now known as ysandeep|lunch | 08:30 | |
| ianw | felixedel: i'm not sure? was a test job proposed to test it? | 08:37 |
| ianw | felixedel: the testing procedure is described at https://opendev.org/opendev/base-jobs/src/branch/master/zuul.d/jobs.yaml#L10 | 08:39 |
| jkt | guilhermesp: a gentle ping about zuul.vexxhost.dev | 08:45 |
| felixedel | ianw: Sry, than maybe I understood that wrong. I thought that there is already a testjob in place and you just wanted to execute it :D | 08:58 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Add linting rule to enforce no-same-owner policy https://review.opendev.org/727642 | 09:06 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Add linting rule to enforce no-same-owner policy https://review.opendev.org/727642 | 09:08 |
| *** felixedel has quit IRC | 09:10 | |
| *** ysandeep|lunch is now known as ysandeep | 09:12 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: REST API: improve tenant scoping of autohold, authorizations https://review.opendev.org/728118 | 09:21 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: CLI: add autohold-info, autohold-delete via REST https://review.opendev.org/728410 | 09:21 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: user login with OpenID Connect https://review.opendev.org/734082 | 09:21 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: allow a privileged user to dequeue a change https://review.opendev.org/734850 | 09:22 |
| avass | zuul-jobs-maint: synchronize/unarchive linting rule is ready: https://review.opendev.org/#/c/727642/ | 09:22 |
| *** ysandeep is now known as ysandeep|afk | 09:35 | |
| *** jamesmcarthur has joined #zuul | 09:35 | |
| *** jamesmcarthur has quit IRC | 09:49 | |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Add ensure-pip to zuul-stream playbook https://review.opendev.org/736520 | 09:57 |
| *** rpittau is now known as rpittau|bbl | 10:06 | |
| *** dennis_effa has joined #zuul | 10:13 | |
| *** ysandeep|afk is now known as ysandeep | 10:14 | |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Drop support for ansible 2.6 https://review.opendev.org/727158 | 10:15 |
| avass | it's been 4 weeks since we dropped support for ansible 2.6 in zuul so should we go ahead with that ^ now? | 10:17 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "gate pipeline" https://review.opendev.org/732069 | 10:18 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "job secrets" https://review.opendev.org/732070 | 10:18 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "job dependencies" https://review.opendev.org/732071 | 10:18 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Fix zuul_run_command ret when command is not found https://review.opendev.org/735475 | 10:18 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Drop support for ansible 2.7 https://review.opendev.org/727410 | 10:23 |
| *** jcapitao is now known as jcapitao_lunch | 10:37 | |
| mordred | ianw: that update looks great to me! | 11:23 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: nit: Fix missing word in roles/ensure-pip/README.rst https://review.opendev.org/736617 | 11:25 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: zuul_stream: close streamer when task is skipped https://review.opendev.org/735476 | 11:25 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: DNM: execute zuul tutorial jobs with pending zuul-jobs changes https://review.opendev.org/735477 | 11:25 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: POC: test job multinode (zuul_stream+callback test) https://review.opendev.org/735483 | 11:25 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: WIP: Add 'uuid' to 'src_dir' in order to allow parallel jobs for a static node https://review.opendev.org/735981 | 11:25 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: POC: test 'uuid' in 'src_dir', 1 static/4 parallel jobs https://review.opendev.org/735576 | 11:25 |
| *** jcapitao_lunch is now known as jcapitao | 11:27 | |
| mordred | avass: after all of yesterday's work around getting wheel installed, ianw made a patch that's a bit different but I think is better | 11:33 |
| mordred | avass: https://review.opendev.org/#/c/736402/ | 11:36 |
| mordred | avass: tested by https://review.opendev.org/#/c/736443/ | 11:36 |
| *** jpena is now known as jpena|lunch | 11:38 | |
| *** jcapitao has quit IRC | 11:46 | |
| *** jcapitao has joined #zuul | 11:49 | |
| *** rlandy has joined #zuul | 11:50 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-jobs master: phoronix-test-suite: extract ensure- role from run playbook https://review.opendev.org/736368 | 11:54 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-jobs master: Add ensure pre-run policy to ansible-lint https://review.opendev.org/736367 | 11:54 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-jobs master: Add ensure pre-run policy to ansible-lint https://review.opendev.org/736367 | 11:55 |
| *** wuchunyang has joined #zuul | 12:03 | |
| *** bhavikdbavishi has quit IRC | 12:04 | |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: POC: test 'uuid' in 'src_dir', 1 static/4 parallel jobs https://review.opendev.org/735576 | 12:12 |
| *** sgw1 has joined #zuul | 12:14 | |
| *** rpittau|bbl is now known as rpittau | 12:20 | |
| *** wuchunyang has quit IRC | 12:24 | |
| *** dennis_effa has quit IRC | 12:26 | |
| *** jpena|lunch is now known as jpena | 12:38 | |
| *** bhavikdbavishi has joined #zuul | 12:43 | |
| *** ysandeep is now known as ysandeep|brb | 12:47 | |
| avass | mordred: should Suse install python3-wheel for python2? | 12:50 |
| avass | AJaeger: ^? | 12:51 |
| mordred | avass: almost certainly not :) | 12:51 |
| mordred | avass: lemme update that real quick | 12:51 |
| openstackgerrit | Monty Taylor proposed zuul/zuul-jobs master: Partial revert "Ensure wheel exists for build-release-python"; move to ensure-pip https://review.opendev.org/736402 | 12:52 |
| mordred | avass: published edit | 12:52 |
| mordred | great catch! | 12:52 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: WIP: Add 'uuid' to 'src_dir' in order to allow parallel jobs for a static node https://review.opendev.org/735981 | 12:52 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: POC: test 'uuid' in 'src_dir', 1 static/4 parallel jobs https://review.opendev.org/735576 | 12:52 |
| avass | mordred: also not sure why we're checking ansible_python.version.major 2 for Redhat but nowhere else | 12:53 |
| mordred | avass: yeah - I'm not sure why we care about ansible-python at all tbh | 12:55 |
| mordred | avass: but the ansible_python.version.major == 3 check was already in there | 12:55 |
| mordred | so - shrug? | 12:55 |
| avass | I guess so huh | 12:56 |
| *** noonedeadpunk is now known as noonedeadpunk_ | 12:57 | |
| AJaeger | avass, mordred, good catch, thanks! | 12:58 |
| avass | mordred: otherwise it looks good, I'll get back to preparing for midsummer tomorrow :) | 12:58 |
| avass | but I'd rather someone add a comment to why Redhat checks ansible_python version | 12:59 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Drop support for ansible 2.7 https://review.opendev.org/727410 | 13:11 |
| *** sshnaidm is now known as sshnaidm|mtg | 13:23 | |
| *** ysandeep|brb is now known as ysandeep | 13:29 | |
| openstackgerrit | Jens Harbott (frickler) proposed zuul/zuul-jobs master: Fix with_* loop for ansible 2.9 https://review.opendev.org/736714 | 13:33 |
| zbr | another zuul UX issue: page lacking essential information: project, patch and job-name: https://review.rdoproject.org/zuul/build/97eac8964a5646f98a541b6d13fa898b/console | 13:38 |
| zbr | unless I chick on summary tab, it will be impossible to figure it out where this run from. | 13:39 |
| zbr | not only that this should be included in the page, but it should also be sticky/floating, like the top menubar. | 13:39 |
| mhu | I get you, but the summary tab is just one click away ... | 13:40 |
| zbr | yes, but you need to browse back if you go there, changes context twice | 13:40 |
| zbr | also to reach summary page you need to scroll up from where you are | 13:40 |
| zbr | context can easily be mentioned in a single line, it does not need to take more then one line, but it needs to stay on screen all the time. | 13:41 |
| zbr | neither the build URL helps because either it does not include the build name or is so long that is no longer visible in the address bar. | 13:42 |
| fungi | though also relying on the url to be displayed at all is probably a poor choice, regardless of whether breadcrubms about where you are have been embedded there or not | 13:44 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: allow a privileged user to dequeue a change https://review.opendev.org/734850 | 13:45 |
| * fungi often runs browsers in chromeless mode without buttons, tabs, titles or location bar showing, to save screen real estate | 13:45 | |
| *** jamesmcarthur has joined #zuul | 13:46 | |
| *** Goneri has joined #zuul | 13:47 | |
| *** jamesmcarthur has quit IRC | 13:51 | |
| *** Goneri has quit IRC | 13:55 | |
| *** bhavikdbavishi has quit IRC | 13:56 | |
| zbr | fungi: what I wanted to check is if others would support a change that aims to make these "basic" information visibale on the webpage at all time. | 13:57 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: nit: Fix missing word in roles/ensure-pip/README.rst https://review.opendev.org/736617 | 13:57 |
| AJaeger | fungi: that's the followup change to address one of your comments ^ | 13:57 |
| zbr | as i do not want to spend weeks working on that feature and discover that some don't like it, of don't find it useful. | 13:57 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: phoronix-test-suite: extract ensure- role from run playbook https://review.opendev.org/736368 | 13:58 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: Add ensure pre-run policy to ansible-lint https://review.opendev.org/736367 | 13:58 |
| zbr | i personally find the ensure-wheel role result of lack of vision. Are we now looking to create another one ensure-pep517.build one? or another one for twine,.... | 13:59 |
| zbr | is not that we do not need a role to install it, but we need a reusable role to ensure a pypi requirement in general, one that is extensible. | 14:00 |
| *** jamesmcarthur has joined #zuul | 14:00 | |
| zbr | in fact, maybe we can even use a ensure-pip to install extra tools, and have a variable that defines them. | 14:01 |
| *** rfolco|rover is now known as rfolco | 14:01 | |
| fungi | zbr: the outcome there was that pip already needs wheel anyway, and the chances you're going to use wheel without having pip are low, so installing wheel with the ensure-pip role makes sense | 14:01 |
| *** dennis_effa has joined #zuul | 14:01 | |
| fungi | (pip doesn't install sdists directly, it creates a wheel from them and then installs that) | 14:02 |
| mordred | also - it turns out the correct solution from ianw is not to install wheel from pypi at all - but, like pip, to instal it from distro packages | 14:07 |
| fungi | that also gets us a wheel version contemporary with the pip version | 14:08 |
| mordred | yeah. | 14:08 |
| *** Goneri has joined #zuul | 14:08 | |
| mordred | the reason to have some ensure- roles (I agree, it's not necessary to have one for each thing you can install from pypi) - is to nagivate figuring out if someone has pre-installed a given common tool, which can happen in multiple ways, and if not to intsall it- again usually supporting multiple different site-specific deployment choices - so that a given job can express "I need fundamental tool x" - and we can | 14:10 |
| mordred | provide that without needing to have stuff pre-baked into image but also not having job authors having to understand everything about structural choices deployers have made | 14:10 |
| mordred | it's always going to be a judgement call as to whether something needs an ensure- role | 14:10 |
| mordred | I agree - if all the role does is "pip install foo" - that would be silly | 14:11 |
| mordred | most of our ensure- roles are a bit more complex than that | 14:11 |
| openstackgerrit | Merged zuul/zuul-jobs master: Partial revert "Ensure wheel exists for build-release-python"; move to ensure-pip https://review.opendev.org/736402 | 14:12 |
| mnaser | mordred: i think ensure-foo being simple does have another benefit of running inside pre so if it depends on any network resource that fails, it gives you that 3 retry thing | 14:16 |
| mnaser | so even if its really simple, it does add the benefit of making the job a bit more reliable, but we can also totally replace that with retries inside ansible (but that wouldn't protect us from landing on a bad node and failing a job) | 14:16 |
| openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul-jobs master: tox: allow tox to be upgraded https://review.opendev.org/690057 | 14:18 |
| openstackgerrit | Merged zuul/zuul-jobs master: nit: Fix missing word in roles/ensure-pip/README.rst https://review.opendev.org/736617 | 14:19 |
| zbr | i think that ianw is not aware about the fact that newer wheel is required for building projects using pyproject.toml | 14:19 |
| zbr | i am almost sure that centos-7 will fail to build with some distro wheel (i have one in particular in mind) | 14:20 |
| mordred | zbr: then I think in that case people wanting to run build-python-release should not use a centos-7 node | 14:21 |
| mordred | or - if that's important for an administrator of a zuul system providing centos-7 nodes, pre-installing a newer wheel in their images should avoid the use of that role | 14:21 |
| *** hashar has quit IRC | 14:21 | |
| zbr | mordred: i am not using it, but the same issue applies to other distros. they may be working today, but sooner or later the wheel package may lack behind long enough to break some builds. | 14:22 |
| mordred | the ensure-pip system as it is written installs pip from packages, because people feel that using the version of the tools that comes with the distro under test is important - especially when people are doing integration testing of production deployments | 14:22 |
| zbr | i think that all agreed that pre-hacking system images was a bad idea | 14:22 |
| mordred | while I *personally* would *never* install pip from packages | 14:22 |
| mordred | this isn't about my personal preferences :) | 14:23 |
| zbr | mordred: we share that personal preference, when I see pip 9.0 ..., i step back and scream. | 14:23 |
| mordred | yup. but overall the design was agreed on that distro packages would be used for these things since they are easy for job content to unintsall and replace with something else of their choosing as needed | 14:26 |
| mordred | and - in general, for jobs like "build-python-release" - as long as that is run on a recent base image things should be fine | 14:26 |
| *** noonedeadpunk_ is now known as noonedeadpunk | 14:29 | |
| openstackgerrit | Merged zuul/zuul master: Shift docker and pip setup to pre-playbook for quick-start https://review.opendev.org/735910 | 14:29 |
| corvus | mordred: i think i'm caught up on the latest wheel stuff in 736402; makes sense. anything else need attention? | 14:33 |
| *** Goneri has quit IRC | 14:35 | |
| fungi | i'm working with the openstack release team to test the updated state of that stuff by reenqueuing a recent failure in opendev | 14:36 |
| mordred | corvus: I don't think so - to my knowledge I think it's solid now | 14:40 |
| avass | mnaser: well, you don't really need a role for running something like "pip install this specific package" you can just run that in the playbook directly :) | 14:43 |
| avass | corvus: should we announce that we're dropping ansible 2.6 support for zuul-jobs or just drop it since zuul already dropped it? https://review.opendev.org/#/c/727158/ | 14:45 |
| avass | mordred, mnaser: that's my thoughts on https://review.opendev.org/#/c/730360/9 as well. the ensure-ansible role doesn't really do a lot (and I'd like others opinion on that) | 14:48 |
| mnaser | avass: maybe ensure-pip-package should be a thing, similar to ensure-package-repositories | 14:49 |
| avass | mnaser: yeah I had that idea as well | 14:49 |
| mnaser | and then we can iron out all of the little kinks of getting a pip package installed, which can run in pre to save failures | 14:49 |
| fungi | avass: mnaser: we may still want to announce that, as folks could be directly using zuul-jobs but not upgrading zuul continuously | 14:50 |
| mnaser | and have improvements to that role fix anything else that needs to pull things from pip | 14:50 |
| openstackgerrit | Merged zuul/zuul-jobs master: phoronix-test-suite: extract ensure- role from run playbook https://review.opendev.org/736368 | 14:50 |
| *** Goneri has joined #zuul | 14:51 | |
| corvus | fungi, avass: ++ to announcing; we don't actually have an 'ansible version' policy in zuul-jobs, but i think trailing zuul itself by a little bit seems reasonable. | 14:56 |
| fungi | just got confirmation from openstack release jobs the latest updates to ensure-pip et cetera have worked | 15:04 |
| fungi | they're able to process new releases again | 15:04 |
| *** jamesmcarthur has quit IRC | 15:04 | |
| *** harrymichal has joined #zuul | 15:06 | |
| AJaeger | great news! | 15:07 |
| avass | corvus, fungi: alright I'll send an email announcing that zuul-jobs is dropping 2.6 support later. I also remembered that I forgot to announce dropping tox_envlist: venv default so I'll do that too :) | 15:07 |
| avass | should we wait another week with dropping 2.6 support then? | 15:10 |
| avass | corvus, fungi: ^? | 15:11 |
| *** harrymichal has quit IRC | 15:11 | |
| corvus | avass: yes, let's wait 2 weeks to give people time to pin zuul-jobs if they need to | 15:11 |
| avass | alright | 15:12 |
| *** jcapitao has quit IRC | 15:17 | |
| avass | corvus: send! | 15:24 |
| avass | sent* | 15:25 |
| zbr | technically speaking zuul is in breach of EU cookie low due to the timezone feature..... :p | 15:26 |
| zbr | i wonder if I will sleep well knowing this... | 15:27 |
| mordred | zbr: we do not belive that to be the case | 15:27 |
| zbr | no? i think that the law required prompt regardless if they are tracked or not. | 15:27 |
| mordred | no - there's specific language that allows thigns like what we're doing | 15:27 |
| zbr | anyway, i think we are likely better using localStorage instead and avoid sending data to the server. is there a reason for not doing it? | 15:28 |
| mordred | what data on the server? | 15:28 |
| mhu | probably so that it's transient? | 15:28 |
| mordred | cookies are stored in browser | 15:28 |
| mhu | and you don't have to re-set your TZ every time you open a tab | 15:29 |
| zbr | cookies are sent always, there is no such thing as local cookie, cookie are sent with each request. | 15:29 |
| mhu | zbr: cookies are stored locally AND sent as headers - but the content is stored in the browser, it's the whole point | 15:29 |
| mordred | yeah. I suppose if we used local storage we could avoid the cookie in the header | 15:30 |
| mordred | but - also I think browser support for local storage is much newer and requires a user to opt-in no? | 15:30 |
| zbr | i will investigate use of localStorage, as if we add more settings, this may prove more practical. | 15:30 |
| zbr | no opt-in afaik, https://www.w3schools.com/html/html5_webstorage.asp | 15:31 |
| mhu | don't take my word for it, but I'm not certain localStorage is shared among tabs | 15:31 |
| zbr | in fact is quite old | 15:31 |
| avass | mordred: ready https://www.privacypolicies.com/blog/eu-cookie-law/ I'd say the timezone cookie would fall under 'customized experience' | 15:31 |
| * mordred has no issues with local storage - likes it for things it's good for | 15:31 | |
| zbr | there are two things localStorage and localSession, we want the first. | 15:31 |
| avass | but we've had this discussion before and I'm no lawyer so I have no idea :) | 15:32 |
| mordred | avass: I would not interpret customeized user experience in the way they use it there to be what we're talking about here | 15:32 |
| zbr | avass: yep, customized experience for sure. | 15:32 |
| mordred | I believe they're talking about things that change the behavior - like, on espn.com I dont' want to see boxing news,so I set a prefernece saying that | 15:33 |
| mordred | that's a customized user experience - it changes the UI of the site in a way that is personal to me | 15:33 |
| fungi | the rules also have a carve-out for using cookies in ways which are necessary for functionality of the site/application | 15:36 |
| fungi | all the language in gdpr is around cookies used for tracking behavior and other personal information for marketing purposes and sharing with third parties | 15:37 |
| mordred | yeah | 15:37 |
| fungi | but people who are scared by the legalese tend to just decide that to be safe they have to mention any cookie being used, regardless of what it's for or how it's used | 15:37 |
| mordred | actually, specifically this: "as strictly necessary in order for the provider of an information society service explicitly requested by the subscriber or user to provide the service" - the fact that the cookie in question is in service of providing a service that is explicitly requested by the user | 15:38 |
| mordred | the only thing the cookie does is allow us to provide a timezone-modified date time that the user asks us to show them - and it is necessary to do that | 15:38 |
| avass | looking at: https://wikis.ec.europa.eu/display/WEBGUIDE/04.+Cookies most of the exemptions are 'for the duration of a session' | 15:39 |
| avass | mordred: wouldn't it be a 'user input cookie'? | 15:39 |
| mordred | yeah - but since we don't have authentication - a session is of indeterminate duration - I do not believe anyone could define when a zuul dashboard session ends | 15:40 |
| mhu | mordred: we don't have authentication YET! | 15:40 |
| avass | well, it ends when you close the tab imo :) | 15:40 |
| mordred | avass: I think we use the dashboard differently :) | 15:41 |
| mordred | or tabs | 15:41 |
| clarkb | close the tab? | 15:41 |
| fungi | wait... you *close* tabs? | 15:41 |
| *** sshnaidm|mtg is now known as sshnaidm|ruck | 15:41 | |
| avass | yeah, after usually when I've opened >100 to solve a problem | 15:41 |
| fungi | i just throw my computer away and replace it when browser tabs build up to the point where it's no longer usable | 15:41 |
| mhu | you guys have tabs? | 15:41 |
| mordred | but anyway - I argue that we're covered by strictly necessary to provide requested service | 15:41 |
| zbr | back to practical stuff: i will look into use of localstorage | 15:42 |
| *** hashar has joined #zuul | 15:42 | |
| openstackgerrit | Merged zuul/zuul master: Restore release jobs https://review.opendev.org/736327 | 15:42 |
| zbr | reason for that is that i may need it to implement the requested feature to change the wrapping behavior ... :( | 15:42 |
| avass | mordred: and I'd argue that it would fall under being a customized experience and/or user input | 15:42 |
| mordred | avass: I know - I'd argue differently | 15:42 |
| mordred | which means that it's literally impossible to know unless we get sued | 15:43 |
| avass | but I think we need a lawyer for this :) | 15:43 |
| clarkb | we could use the timezone reported by the browser | 15:43 |
| zbr | you know a good lawyer is a ... one. | 15:43 |
| mordred | a lawyer can't tell us the answer - a lawyer can give advice as to what our relative risk profiles might be and what they believe the answer might be | 15:43 |
| *** ysandeep is now known as ysandeep|away | 15:43 | |
| tristanC | zbr: where do you see the cookie being sent to the server? | 15:43 |
| clarkb | but then that has the problem of showing peopel different timestamps | 15:44 |
| clarkb | we could also just force utc (I'd be fine with that) | 15:44 |
| zbr | i did not check, but i know how cookies work. any cookie set is sent to the server with each request. | 15:44 |
| *** rlandy is now known as rlandy|ruck2 | 15:44 | |
| avass | mordred: they can give us advice based on experience :) | 15:44 |
| zbr | is how cookies was designed to work. | 15:44 |
| zbr | if the server does somethign with them or not, is less important. | 15:44 |
| mordred | avass: they can indeed - but they're also typically overly risk averse because they don't want to be wrong in the wrong direction | 15:44 |
| tristanC | zbr: i thought that only happens for http cookie, like the one used for session, not for any cookie, like the one used for site preference | 15:44 |
| mordred | avass: so they tend to give the _Absolute_ safest advice even. like, the laywer would say "just show a consent, you know you won't be in violation then" | 15:45 |
| tristanC | zbr: unless i'm mistaken, i do not see any cookie being exchanged between the client and the zuul server | 15:45 |
| AJaeger | publish-openstack-python-branch-tarball is still failing - I'm confused | 15:45 |
| AJaeger | https://zuul.opendev.org/t/openstack/builds?job_name=publish-openstack-python-branch-tarball | 15:45 |
| zbr | probably because none of us did customize the timezone preference | 15:45 |
| avass | mordred: probably | 15:47 |
| zbr | anyone reason to write ensure-setuptools? ;) -- i am out now. | 15:47 |
| * mordred throws a bucket of pies at zbr | 15:47 | |
| avass | not cookies? | 15:48 |
| avass | mordred: I can't say I want zuul to have a cookie banner though :) | 15:49 |
| *** bhagyashris is now known as bhagyashris|away | 15:49 | |
| clarkb | avass: I think that is my biggest concern. Those banners are so completely useless | 15:49 |
| clarkb | they've just trained everyone to accept all the cookies we didn't want them to have in the first place | 15:49 |
| clarkb | which is worse because now we've consented to the problem | 15:49 |
| clarkb | rather than eliminating the underlying issue /grump | 15:49 |
| avass | I usually go 'ugh I'll read it somewhere else' | 15:50 |
| fungi | heh, yeah, i mostly just try not to click any consent banners, and if stuff doesn't work without consenting, i leave the page | 15:50 |
| clarkb | fungi: the new thing is if you don't explicitly consent but start scrolling they have verbage to say that is implicit consent | 15:51 |
| avass | yeah | 15:51 |
| clarkb | whether that would hold up in court i have no idea | 15:51 |
| clarkb | but the problem remains we're just training people to give up their info without a second thought | 15:51 |
| clarkb | its completely backfired the intent/goal of the law imo | 15:51 |
| tristanC | oops, i was wrong, the timezone is being sent to the server when it is set | 15:52 |
| fungi | and anyway, i would leave it up to the deployers/sysadmins to decide if they're going to overlay something like that. there's plenty of other open source software we run in opendev which uses cookies for a variety of similar purposes and the upstream authors of those services haven't felt they need to add in "gdpr support" or anything like that | 15:52 |
| fungi | (though i know there's an open feature request against jitsi-meet for it, that's a little different since it directly handles video and audio recordings of users) | 15:53 |
| mhu | tristanC, maybe sending the tz to the server is unneeded actually - where is the UI code executed? | 15:53 |
| *** harrymichal has joined #zuul | 15:53 | |
| clarkb | mhu: ya it isnt that is why zbr is looking at local storage aiui | 15:55 |
| mhu | oooh ok according to Stack Overflow localStorage can be shared among tabs, but it needs to be set explicitly https://stackoverflow.com/questions/20325763/browser-sessionstorage-share-between-tabs | 15:57 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Deprecate default tox_envlist: venv https://review.opendev.org/726830 | 16:00 |
| *** bhavikdbavishi has joined #zuul | 16:01 | |
| *** harrymichal has quit IRC | 16:03 | |
| *** harrymichal has joined #zuul | 16:03 | |
| avass | corvus: I sent the tox_envlist announcement as well | 16:03 |
| *** bhavikdbavishi1 has joined #zuul | 16:04 | |
| *** bhavikdbavishi has quit IRC | 16:05 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 16:05 | |
| openstackgerrit | Albin Vass proposed zuul/zuul master: Add commit id and owner to Change for mqtt reporter https://review.opendev.org/722478 | 16:06 |
| *** dennis_effa has quit IRC | 16:11 | |
| *** jamesmcarthur has joined #zuul | 16:19 | |
| openstackgerrit | Jeremy Stanley proposed zuul/zuul-jobs master: Simplify twine invocation for PyPI uploads https://review.opendev.org/735932 | 16:40 |
| *** jamesmcarthur has quit IRC | 16:43 | |
| *** rlandy|ruck2 is now known as rlandy|ruck2|brb | 16:45 | |
| *** jamesmcarthur has joined #zuul | 16:51 | |
| *** sshnaidm|ruck is now known as sshnaidm|off | 16:52 | |
| openstackgerrit | Merged zuul/zuul-jobs master: Fix with_* loop for ansible 2.9 https://review.opendev.org/736714 | 16:54 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: allow a privileged user to re-enqueue a change https://review.opendev.org/736772 | 16:55 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: allow a privileged user to re-enqueue a change https://review.opendev.org/736772 | 16:57 |
| *** olaph has joined #zuul | 17:00 | |
| *** rpittau is now known as rpittau|afk | 17:00 | |
| *** rlandy|ruck2|brb is now known as rlandy|ruck2 | 17:04 | |
| *** bhavikdbavishi has quit IRC | 17:04 | |
| *** bhavikdbavishi has joined #zuul | 17:05 | |
| *** hashar has quit IRC | 17:05 | |
| *** jamesmcarthur has quit IRC | 17:08 | |
| *** jamesmcarthur has joined #zuul | 17:08 | |
| openstackgerrit | Jeremy Stanley proposed zuul/zuul-jobs master: Simplify twine invocation for PyPI uploads https://review.opendev.org/735932 | 17:09 |
| *** nils has quit IRC | 17:14 | |
| openstackgerrit | Jeremy Stanley proposed zuul/zuul-jobs master: Simplify twine invocation for PyPI uploads https://review.opendev.org/735932 | 17:15 |
| *** jpena is now known as jpena|off | 17:21 | |
| fungi | webknjaz: ^ thanks for all your help on that one! | 17:22 |
| webknjaz | 👍 | 17:22 |
| fungi | i'm still struggling to come up to speed with ansible | 17:23 |
| avass | fungi: should that fail if it detects unsafe files? | 17:25 |
| fungi | avass: er, yes, do i need to add a failed_when on the twnie check task? | 17:26 |
| avass | fungi: you can use 'fail' instead of 'debug' on the 'Report files unsafe for upload' task | 17:27 |
| avass | unless the check already sets an exit code, then you might as well remove the debug task because it will never report any unsafe files :) | 17:27 |
| fungi | oh, cool, still learning ;) | 17:27 |
| clarkb | it will actually fail on line 12 | 17:27 |
| clarkb | if the output from twine is verbsoe enough you probably don't need the extra debug/fail task | 17:28 |
| clarkb | but if not then you want a failed_when: false on the line 12 task and then switch debug to fail | 17:28 |
| fungi | basically the twine check command will have a nonzero exit code if it detects a problem sich as a file not fit for uploading | 17:28 |
| avass | clarkb, fungi: alright, then I guess you can remove the register and just run 'twine check' and if that fails nothing else will run | 17:28 |
| fungi | and it does provide verbose output | 17:28 |
| avass | (in that playbook) | 17:28 |
| fungi | i didn't know if it was important for the cleanup task to still run | 17:28 |
| clarkb | fungi: followup tasks like that are usually for when the previous failure is going to be obtuse to the end user | 17:29 |
| avass | if it is you can use a block and use the 'always' keyword | 17:29 |
| clarkb | or if you're relying on somethign other than exit code to determine failure | 17:29 |
| avass | fungi: like https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/tox/tasks/main.yaml#L37 | 17:30 |
| fungi | does the failing task need to be inside the block, i guess? | 17:31 |
| avass | that will run tox and return comments even if tox fails, sort of like a try/finally | 17:32 |
| avass | yeah | 17:32 |
| *** michael-beaver has joined #zuul | 17:32 | |
| openstackgerrit | Jeremy Stanley proposed zuul/zuul-jobs master: Simplify twine invocation for PyPI uploads https://review.opendev.org/735932 | 17:33 |
| fungi | avass: so like that? ^ | 17:33 |
| avass | fungi: lgtm :) | 17:33 |
| fungi | thanks! | 17:34 |
| avass | oh, you might actually need to indent the block another level | 17:35 |
| avass | not sure about that though | 17:35 |
| avass | let me check | 17:35 |
| fungi | oh, i see, looking at the example, the list entry markers are two spaces deeper than the always | 17:36 |
| avass | yeah | 17:36 |
| fungi | the list entry markers earlier in the block i mean | 17:36 |
| fungi | i guess the top level data structure is a list, and then one of the list items is a dict with keys "block" and "always" and then each has list items within it | 17:37 |
| avass | both seem to work but you might get a linter yelling at you | 17:37 |
| openstackgerrit | Jeremy Stanley proposed zuul/zuul-jobs master: Simplify twine invocation for PyPI uploads https://review.opendev.org/735932 | 17:38 |
| fungi | one of those nuances of yaml which is easy to not spot | 17:38 |
| fungi | it's technically correct because the list markers count as indentation for the yaml parser, basically | 17:39 |
| fungi | but it can make things hard to follow, and a number of linters will yell at you about it, yes | 17:39 |
| mordred | fungi, avass: both are correct - we've chosen in zuul to adopt the extra-2-spaces indentation as a yaml style | 17:45 |
| fungi | yeah, i'm down with that | 17:47 |
| avass | changed my vote to a +1 since the ansible looks correct, but I've never used twine :) | 17:48 |
| mordred | fungi: so - I have a suggestion ... | 17:48 |
| fungi | avass: it's okay, i'm pretty confident in the command line bits, i appreciate the deeper ansible review | 17:49 |
| mordred | fungi: (ansible looks good) | 17:49 |
| mordred | fungi: {{ pypi_path }}/* will work - but will upload things in shell-expansion order according to your commit message | 17:49 |
| mordred | this could result in uploading the tarball before the wheel - but it's important to upload the wheel first so that people installing don't get caught in the moment between tarball and wheel only having the tarball available and thus having to build the wheel | 17:50 |
| avass | oh, actually, we have tests for those | 17:50 |
| fungi | mordred: yeah, that's a great point | 17:50 |
| mordred | fungi: so maybe {{ pypi_path }}/*whl {{ pypi_path }}/*tar.gz instead of just the bare * ? | 17:50 |
| fungi | i think if we have a project only uploading a tarball, that will cause an error | 17:51 |
| mordred | we also apparently were excluding excludes: "*-linux_x86_64.whl" | 17:51 |
| mordred | I'm not sure why were we doing that | 17:51 |
| fungi | oh, because pypi won't allow platform-specific wheels for linux abi | 17:51 |
| mordred | ah | 17:52 |
| fungi | in theory twine check will also complain there, but either way it's a bit late at that point | 17:52 |
| mordred | so maybe the two finds are worth keeping - but then combine on the single twine command line | 17:52 |
| mordred | or something | 17:52 |
| avass | yeah | 17:53 |
| fungi | i was struggling with how to do that in ansible, as you'll see in earlier patchsets | 17:53 |
| fungi | find is returning a list of dicts | 17:53 |
| fungi | so turning dict keys from each element into a concatenated list of strings is... well if it was just python i could use a list comprehension but... | 17:54 |
| fungi | rather, turning the values associated with specific dict keys in each of the list elements into a concatenated string | 17:55 |
| avass | fungi: you can use map with join for that I believe: https://docs.ansible.com/ansible/latest/user_guide/playbooks_filters.html#id8 | 17:57 |
| avass | fungi: something like {{ files | map(attribute='path') | join(' ') }} | 17:58 |
| fungi | yeah, i see the map join example in there | 17:59 |
| fungi | i can give that a shot | 17:59 |
| fungi | and i guess if files is an empty list then that will result in an empty string, so i don't have to test for whether they're empty? | 18:00 |
| avass | fungi: here's a better link: https://jinja.palletsprojects.com/en/2.11.x/templates/#map | 18:00 |
| avass | I would think so | 18:00 |
| fungi | i'll give that a shot after my next meeting | 18:01 |
| *** hashar has joined #zuul | 18:09 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Add authentication-realm attribute to tenants https://review.opendev.org/735586 | 18:39 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: allow a privileged user to re-enqueue a change https://review.opendev.org/736772 | 18:39 |
| openstackgerrit | Jeremy Stanley proposed zuul/zuul-jobs master: Simplify twine invocation for PyPI uploads https://review.opendev.org/735932 | 18:45 |
| openstackgerrit | Ahmad Mahmoudi proposed zuul/zuul-jobs master: (fix) - Added tox location to PATH https://review.opendev.org/736801 | 18:47 |
| *** bhavikdbavishi has quit IRC | 19:13 | |
| *** gmann is now known as gmann_afk | 19:22 | |
| corvus | avass: note i pinged you in http://lists.zuul-ci.org/pipermail/zuul-discuss/2020-June/001262.html | 19:31 |
| avass | corvus: yep I saw, and no there's no rush | 19:36 |
| avass | corvus: ah wait, that reply didn't do what I wanted it to right? | 19:37 |
| *** gmann_afk is now known as gmann | 19:37 | |
| corvus | avass: your second one is right :) | 19:44 |
| corvus | avass: and sounds great. you want to send an updated -announce post, or want me to? | 19:45 |
| avass | you can do it :) | 19:49 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Run upload-docker-image on release https://review.opendev.org/736333 | 19:50 |
| corvus | mordred, clarkb: can you re-review https://review.opendev.org/736333 ^ pls? need one more fix for nodepool's build-python-release | 19:50 |
| mordred | corvus: lgtm | 20:00 |
| *** rlandy|ruck2 is now known as rlandy|ruck2|mtg | 20:00 | |
| *** gmann is now known as gmann_afk | 20:10 | |
| clarkb | looking | 20:21 |
| clarkb | corvus: I +2'd but didn't approve as I'm not sure how much coordination that needs | 20:22 |
| corvus | none, i +wd thx | 20:22 |
| *** rlandy|ruck2|mtg is now known as rlandy|ruck2 | 20:31 | |
| *** hashar has quit IRC | 20:47 | |
| *** rfolco has quit IRC | 20:52 | |
| *** harrymichal has quit IRC | 21:33 | |
| *** harrymichal has joined #zuul | 21:35 | |
| openstackgerrit | Merged zuul/nodepool master: Run upload-docker-image on release https://review.opendev.org/736333 | 21:37 |
| openstackgerrit | Merged zuul/zuul-jobs master: Add linting rule to enforce no-same-owner policy https://review.opendev.org/727642 | 21:37 |
| ianw | zbr: just back to your previous note about people wanting later wheel for toml or wanting later pip for dependencies when that's around or whatever -- no problem with that -- and you can tell ensure-pip to install from source. | 21:38 |
| ianw | however, i think that is something you should opt-in to ... if people think they're testing on bionic and then we've installed yesterdays pip etc., then they build something relying on that and someone tries it on bionic outside the gate it all blows up | 21:39 |
| mordred | ianw: ++ | 21:45 |
| mordred | ianw: also - they can just a focal nodeset or something | 21:45 |
| mordred | many options | 21:45 |
| avass | speaking of focal, any idea what's going on here https://review.opendev.org/#/c/731995/ ? | 21:46 |
| *** gmann_afk is now known as gmann | 22:04 | |
| fungi | anybody have a spare moment to take a look at a couple of new user case study summaries for the website? https://review.opendev.org/736053 (and thanks corvus for reviewing already!) | 22:09 |
| clarkb | fungi: I can get to that in a bit probably | 22:10 |
| fungi | it's not super urgent, just figure it's also an easy one to get out of the way | 22:11 |
| mnaser | fungi: it would be nice if we landed the gatsby comment :( | 22:22 |
| mnaser | s/comment/change/ | 22:22 |
| clarkb | mnaser: did you see the feedback on it? | 22:23 |
| clarkb | mnaser: there were a few things that corvus asked be fixed frist | 22:23 |
| mordred | mnaser, clarkb: yeah - I keep meaning to circle back and fix those | 22:23 |
| mnaser | ah yes, i see those now, my bad. | 22:24 |
| mnaser | mordred: think you can get to those? i dont know if i have the bandwidth now to get them done right away, if not, i can try to get soemone here to work on it but that might take a little bit | 22:25 |
| *** tosky has quit IRC | 22:25 | |
| clarkb | fungi: one small thing on https://review.opendev.org/#/c/736053/ but I +2'd in case my english is just bad or you want to fix in a followup | 22:26 |
| clarkb | fungi: feel free to approve if either of those is the case | 22:26 |
| mordred | mnaser: yeah - I'l give them a look tomorrow - and if I can't figure them out I might ask some specific questions | 22:37 |
| mnaser | mordred: ok i can route those questions to the right people here | 22:37 |
| fungi | clarkb: ahh, yeah, i can fix that real quick. i copied and pasted aprice's prose verbatim | 22:37 |
| mordred | mnaser: the menu thing is one that I looked at for a second and then just kind of cried and moved on to something else | 22:37 |
| mnaser | i think the menu thing could be trivial because we might steal it from the component we use/built for vexxhost.com | 22:37 |
| mnaser | i can talk to the guy who owns that site, he's cool with us using it, i think | 22:37 |
| mnaser | :P | 22:37 |
| mordred | mnaser: cool. because it's like - nontrivial but seems like it should be trivial | 22:38 |
| openstackgerrit | Jeremy Stanley proposed zuul/zuul-website master: Add user story summaries for Wazo and T-Systems https://review.opendev.org/736053 | 22:40 |
| fungi | clarkb: ^ better? | 22:40 |
| clarkb | yup +2 thanks | 22:41 |
| corvus | i approved that; mordred, mnaser: that means the gatsby stuff will need a rebase to pick up the new content, but that should be easy | 22:43 |
| mordred | corvus: yah. the adds there are not hard adds | 22:43 |
| fungi | also those user case study blocks are repetitive | 22:48 |
| fungi | format wise | 22:48 |
| openstackgerrit | Merged zuul/zuul-website master: Add user story summaries for Wazo and T-Systems https://review.opendev.org/736053 | 22:48 |
| mordred | fungi: yah - once we have the base thing done, we can replace them with a directory of markdown files | 22:53 |
| fungi | that'll be slick | 22:54 |
| mordred | yeah | 23:14 |
| mordred | I very much enjoy my gatsby site | 23:14 |
| *** jamesmcarthur has quit IRC | 23:43 | |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Rework quick-start and prepare for other tutorials https://review.opendev.org/732066 | 23:57 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "gate your first patch" https://review.opendev.org/732067 | 23:57 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "Use zuul jobs" https://review.opendev.org/732068 | 23:57 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "gate pipeline" https://review.opendev.org/732069 | 23:57 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "job secrets" https://review.opendev.org/732070 | 23:57 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "job dependencies" https://review.opendev.org/732071 | 23:57 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Fix zuul_run_command ret when command is not found https://review.opendev.org/735475 | 23:57 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: zuul_stream: close streamer when task is skipped https://review.opendev.org/735476 | 23:57 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: [DNM] TEST run zuul tutorials with pending zuul-jobs changes https://review.opendev.org/735477 | 23:57 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: [DNM] TEST zuul_stream+callback test using multinode jobs https://review.opendev.org/735483 | 23:57 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Add 'uuid' to 'src_dir' in order to allow parallel jobs for a static node https://review.opendev.org/735981 | 23:57 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: [DNM] TEST 'uuid' in 'src_dir': 1 node, 4 parallel jobs https://review.opendev.org/735576 | 23:57 |
| *** rfolco has joined #zuul | 23:58 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!