| *** jamesmcarthur has joined #zuul | 00:20 | |
| *** rlandy has quit IRC | 00:46 | |
| *** jamesmcarthur has quit IRC | 01:11 | |
| daniel2 | did nodepool 0.5.0 have support for the OpenStack v3 api? | 01:23 |
|---|---|---|
| clarkb | daniel2: keystone v3? I think so via the shade library | 01:50 |
| clarkb | you may have to explicitly enable it by setting the domain and project names | 01:50 |
| daniel2 | clarkb: in cloud.yaml or nodepool.yaml? | 01:52 |
| clarkb | cloud.yaml | 01:52 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: scheduler: add job's tags to the rpc job_list method https://review.openstack.org/633653 | 01:57 |
| daniel2 | clarkb: Well the project name is already specified. I have no idea where to specify the domain information as the docs don't show an option for it. | 02:01 |
| clarkb | daniel2: https://docs.openstack.org/openstacksdk/latest/user/config/configuration.html#auth-settings arethe docs and ya pretty sparse | 02:06 |
| clarkb | mordred: ^ fyi you may want to add moreinfo to that | 02:06 |
| clarkb | daniel2: https://git.openstack.org/cgit/openstack-infra/system-config/tree/playbooks/templates/clouds/nodepool_clouds.yaml.j2#n58 the vexxhost cloud there is an example you mightbe able to use | 02:08 |
| daniel2 | oh cool thanks! | 02:08 |
| daniel2 | clarkb: that worked, thanks again | 02:11 |
| *** jamesmcarthur has joined #zuul | 02:20 | |
| *** jamesmcarthur has quit IRC | 02:25 | |
| *** jamesmcarthur has joined #zuul | 02:38 | |
| *** jamesmcarthur has quit IRC | 02:43 | |
| *** jamesmcarthur has joined #zuul | 03:14 | |
| *** jamesmcarthur has quit IRC | 03:31 | |
| *** jamesmcarthur has joined #zuul | 03:31 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: scheduler: add job's tags to the rpc job_list method https://review.openstack.org/633653 | 03:39 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add tags to jobs list https://review.openstack.org/633654 | 03:39 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add job tag filter in job list https://review.openstack.org/644473 | 03:39 |
| SpamapS | I keep forgetting... my zuul components running in Kubernetes seem to double-log everything on stdout... | 03:53 |
| SpamapS | http://paste.openstack.org/show/747991/ is my logging.conf, but I also have an identical-ish gearman-logging.conf ... is Zuul maybe picking up like, all of the logging.conf's and registering two handlers? | 03:54 |
| SpamapS | (I have gearman-logging.conf referenced as the log_config in the [gearman_server] portion of zuul.conf) | 03:55 |
| *** raukadah is now known as chandankumar | 04:00 | |
| *** bhavikdbavishi has joined #zuul | 04:03 | |
| SpamapS | Looks like nodepool is doing it too | 04:14 |
| tobiash | corvus: it does install it into the tox dir. Did you delete the tox env? | 04:33 |
| tobiash | corvus: or do you mean that it's slow because every tox run updates it? | 04:35 |
| tobiash | corvus: hrm, but it should take less than a second if it's already there | 04:37 |
| *** jamesmcarthur has quit IRC | 04:40 | |
| *** bjackman has joined #zuul | 05:00 | |
| tobiash | corvus: I think I've understood your issue now. Currently the zuul-manage-ansible is run as a pre_command in tox and installs it into .tox/py36/lib/zuul. I guess you're creating the tox environment like 'tox -e py36 --notest' which skips the pre_command and thus doesn't install it and ttrun doesn't run the pre_command. | 05:15 |
| tobiash | corvus: maybe we can change that to run zuul-manage-ansible inside the install_command which would also handle your case | 05:16 |
| *** daniel2 has quit IRC | 05:21 | |
| *** saneax has joined #zuul | 05:46 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: trigger: add job filter event https://review.openstack.org/639905 | 06:00 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: webtrigger: add initial driver and event https://review.openstack.org/555153 | 06:00 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: webtrigger: add web route and rpclistener https://review.openstack.org/554839 | 06:00 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add build button to trigger job https://review.openstack.org/635716 | 06:00 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: webtrigger: add support for parameterized trigger https://review.openstack.org/644484 | 06:01 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add build modal with a parameter form https://review.openstack.org/644485 | 06:01 |
| *** snapiri has joined #zuul | 06:15 | |
| *** daniel2 has joined #zuul | 06:23 | |
| *** saneax has quit IRC | 07:06 | |
| *** saneax has joined #zuul | 07:07 | |
| *** pcaruana has joined #zuul | 07:14 | |
| *** hashar has joined #zuul | 07:31 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Move ansible installation from pre_command to install phase in tox https://review.openstack.org/644497 | 07:31 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Add py37 environment to tox https://review.openstack.org/644498 | 07:31 |
| tobiash | corvus: this should solve your issue ^ | 07:31 |
| *** pcaruana has quit IRC | 07:33 | |
| *** pcaruana has joined #zuul | 07:34 | |
| *** jpena|off is now known as jpena | 08:57 | |
| *** panda|pto is now known as panda | 09:04 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Move ansible installation from pre_command to install phase in tox https://review.openstack.org/644497 | 09:09 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Add py37 environment to tox https://review.openstack.org/644498 | 09:09 |
| *** sshnaidm|afk is now known as sshnaidm|rover | 09:43 | |
| *** swest has quit IRC | 09:43 | |
| *** swest has joined #zuul | 09:43 | |
| *** electrofelix has joined #zuul | 09:47 | |
| tobiash | mordred, tristanC: I think I remember you had thoughts to report with links to the build page instead of the log url directly. Are these just thoughts right now or are there already plans/patches in that direction? | 10:04 |
| tristanC | tobiash: iirc we need to make sql connection mandatory first | 10:08 |
| tobiash | tristanC: ok, so there are just thoughts floating around yet? | 10:08 |
| tristanC | tobiash: i'm not sure what's the plan to reports the build page, iiuc we would need to somehow update the upload-log role | 10:09 |
| *** markwork has joined #zuul | 10:23 | |
| markwork | Did something in the container builds change? I got an error spinning up zuul/zuul trying to call `ansible-playbook' | 10:33 |
| tristanC | markwork: ansible-playbook is now installed in /usr/local/lib/zuul/ansible/2.5/bin/ansible-playbook | 10:39 |
| *** bhavikdbavishi has quit IRC | 10:46 | |
| *** bhavikdbavishi has joined #zuul | 11:26 | |
| *** hashar is now known as hasharLunch | 11:46 | |
| *** klindgren_ has quit IRC | 11:51 | |
| *** klindgren has joined #zuul | 12:03 | |
| *** panda is now known as panda|lunch | 12:07 | |
| *** hasharLunch has quit IRC | 12:18 | |
| *** jpena is now known as jpena|lunch | 12:36 | |
| *** rfolco has joined #zuul | 12:37 | |
| *** rlandy has joined #zuul | 12:38 | |
| openstackgerrit | Markus Hosch proposed openstack-infra/zuul master: Evaluate CODEOWNERS setings during canMerge check https://review.openstack.org/644557 | 12:38 |
| openstackgerrit | Markus Hosch proposed openstack-infra/zuul master: Evaluate CODEOWNERS settings during canMerge check https://review.openstack.org/644557 | 12:41 |
| openstackgerrit | Markus Hosch proposed openstack-infra/zuul master: Evaluate CODEOWNERS settings during canMerge check https://review.openstack.org/644557 | 12:42 |
| *** hasharLunch has joined #zuul | 12:50 | |
| pabelanger | Was going to reply to the email that jimmy@openstack.org sent to ML about BoF/WG space, last I checked the zuul community isn't planning anything official at PTG correct? | 12:52 |
| *** hasharLunch is now known as hashar | 13:01 | |
| *** panda|lunch is now known as panda | 13:01 | |
| *** dmsimard has quit IRC | 13:07 | |
| *** dmsimard has joined #zuul | 13:07 | |
| *** bjackman has quit IRC | 13:12 | |
| *** pwhalen has quit IRC | 13:16 | |
| mordred | tristanC, markwork: we should maybe set PATH in the container images to include one of the ansible install locations - like maybe the 2.7 install | 13:24 |
| mordred | tobiash: yes - what tristanC said - we need to make sql mandatory - but I think we're all pretty eager to get build page to be the thing reported | 13:25 |
| *** pwhalen has joined #zuul | 13:27 | |
| *** jpena|lunch is now known as jpena | 13:33 | |
| openstackgerrit | Simon Westphahl proposed openstack-infra/zuul master: Ensure valid Ansible variable names in config https://review.openstack.org/644574 | 13:35 |
| corvus | pabelanger: correct, though to be clear, BoF/WG are during the summit, not PTG | 13:43 |
| corvus | tobiash: ah thanks! notests is exactly what i did :) | 13:44 |
| markwork | I've now a test setup for zuul running, from the quickstart. The `zuul-config' worked, but as soon as I do the `Test Zuul pipelines' part, zuul executes the job, but does not push this information to gerrit. What could be the reason for that? | 13:57 |
| *** bhavikdbavishi has quit IRC | 13:59 | |
| markwork | No actually I get `testjob: queued' but a confusing `Suceeding' icon. Hmmm. | 14:00 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Add ansible 2.7 path to zuul-executor image https://review.openstack.org/644581 | 14:06 |
| mordred | markwork: also - ansible installs are only in the zuul-executor image, not the zuul/zuul image | 14:07 |
| *** needscoffee is now known as kmalloc | 14:09 | |
| openstackgerrit | David Moreau Simard proposed openstack-infra/zuul-jobs master: Do not merge: test zuul-jobs-upload-git-mirror job https://review.openstack.org/643437 | 14:12 |
| tobiash | mordred: commented on | 14:17 |
| tobiash | 644581 | 14:17 |
| *** daniel3 has joined #zuul | 14:17 | |
| mordred | tobiash: yes. I agree. I'll WIP that patch and come back with one that makes some container images | 14:17 |
| tobiash | ++ | 14:17 |
| corvus | markwork: if the test job is queued, then there may be a problem with nodepool supplying the test node for the job. you might want to check the logs for the zuul executor or the nodepool launcher containers | 14:19 |
| *** saneax has quit IRC | 14:19 | |
| *** saneax has joined #zuul | 14:19 | |
| *** daniel2 has quit IRC | 14:21 | |
| markwork | corvus: thanks, I'll have a look | 14:21 |
| *** daniel3 is now known as daniel2 | 14:21 | |
| *** saneax has quit IRC | 14:24 | |
| *** bhavikdbavishi has joined #zuul | 14:25 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Fix metadata not passed in provides/requires https://review.openstack.org/644445 | 14:27 |
| *** hashar is now known as hasharAway | 14:28 | |
| tobiash | corvus: responded on 644497 | 14:34 |
| corvus | tobiash: ooooh. tricky. thanks. | 14:37 |
| *** jamesmcarthur has joined #zuul | 14:39 | |
| tobiash | corvus: yes, mordred invented this trick for npm and yarn :) | 14:39 |
| tobiash | I just reuse that trick :) | 14:39 |
| openstackgerrit | Markus Hosch proposed openstack-infra/zuul master: Evaluate CODEOWNERS settings during canMerge check https://review.openstack.org/644557 | 14:43 |
| *** bhavikdbavishi has quit IRC | 14:46 | |
| *** daniel2 has quit IRC | 14:48 | |
| *** saneax has joined #zuul | 15:03 | |
| *** hasharAway has quit IRC | 15:33 | |
| *** daniel2 has joined #zuul | 15:47 | |
| *** bhavikdbavishi has joined #zuul | 16:01 | |
| *** jamesmcarthur has quit IRC | 16:04 | |
| *** jamesmcarthur has joined #zuul | 16:05 | |
| *** hasharAway has joined #zuul | 16:12 | |
| pabelanger | corvus: thanks, replied to email on ML | 16:13 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/zuul-preview master: Finish refactor https://review.openstack.org/644609 | 16:16 |
| corvus | pabelanger: i'm confused by your reply | 16:19 |
| pabelanger | corvus: okay, possible I misunderstood your reply to my original quesiton | 16:20 |
| pabelanger | the zuul community is not going to be holding BoF/ WG in Devner? | 16:21 |
| corvus | pabelanger: you asked if we were planning anything at the PTG. we are not. but jimmymcarthur was asking about the *summit*. | 16:21 |
| pabelanger | Ah, my mistake. I thought summit / PTG was grouped as single event | 16:22 |
| corvus | pabelanger: it's confusing. :) | 16:22 |
| pabelanger | does that mean, there will be BoF / WG sessions for the summit? | 16:22 |
| corvus | pabelanger: if we want them :) | 16:22 |
| pabelanger | ack, understand now. So guess we should discuss that. | 16:24 |
| pabelanger | For some reason, I assumed there was going to be no zuul related topics (BoF / WG) outside of the accepted talks. With more of an unoffical hallway track | 16:25 |
| corvus | pabelanger: i'd never heard of BoF/WG sessions before now | 16:28 |
| pabelanger | okay, let me update the email I sent saying we still need to discuss | 16:29 |
| corvus | pabelanger: i replied | 16:29 |
| corvus | also, we can discuss on the list :) | 16:29 |
| pabelanger | ack, see thta now | 16:30 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/zuul-preview master: Partial code refactor for unit testing https://review.openstack.org/643666 | 16:30 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/zuul-preview master: Add unit testing framework and sample test. https://review.openstack.org/644247 | 16:30 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/zuul-preview master: Finish refactor https://review.openstack.org/644609 | 16:30 |
| openstackgerrit | David Moreau Simard proposed openstack-infra/zuul-jobs master: Add a role to mirror a git repository to a remote git server https://review.openstack.org/643394 | 16:41 |
| openstackgerrit | David Moreau Simard proposed openstack-infra/zuul-jobs master: Do not merge: test zuul-jobs-upload-git-mirror job https://review.openstack.org/643437 | 16:41 |
| *** bhavikdbavishi has quit IRC | 16:58 | |
| fungi | corvus: for historical context, "wg" sessions were originally used by osf board working groups, while "bof" sessions were for topics not directly aligned with any particular group (like efforts in openstack which are now referred to as sigs) | 17:27 |
| fungi | there were usually a scant handful on the schedule and so were easy to overlook | 17:28 |
| fungi | but i guess over time they've expanded the use cases for those and are applying the terms more broadly | 17:28 |
| *** hasharAway is now known as hashar | 17:34 | |
| *** jamesmcarthur has quit IRC | 17:39 | |
| tobiash | zuul-maint: I'd love some reviews on a tiny change that improves error reporting if zuul cannot access a repo in github: https://review.openstack.org/583841 | 17:51 |
| tobiash | I co-authored it so I didn't vote myself | 17:51 |
| tobiash | tristanC, jhesketh: is the zuul-runner stack rebase onto multi-ansible ready for review? | 17:52 |
| *** jamesmcarthur has joined #zuul | 17:54 | |
| *** saneax has quit IRC | 17:54 | |
| corvus | tobiash: https://review.openstack.org/644445 fixes a bug we noticed in production yesterday | 17:56 |
| tobiash | corvus: yeah, just reviewed it one minute ago :) | 17:56 |
| corvus | nice :) | 17:57 |
| *** chandankumar is now known as raukadah | 18:05 | |
| corvus | tobiash: that change also affects your change 642857; i left comments describing that | 18:10 |
| tobiash | corvus: thanks for review , I'll look into that :) | 18:11 |
| *** hashar is now known as hasharDinner | 18:12 | |
| *** jamesmcarthur has quit IRC | 18:12 | |
| *** jamesmcarthur has joined #zuul | 18:14 | |
| *** jamesmcarthur has joined #zuul | 18:14 | |
| *** jpena is now known as jpena|off | 18:19 | |
| openstackgerrit | Merged openstack-infra/zuul master: Move ansible installation from pre_command to install phase in tox https://review.openstack.org/644497 | 18:32 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Extract an abstract base Parser class https://review.openstack.org/545610 | 18:36 |
| *** hasharDinner is now known as hashar | 18:39 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Forward artifacts to child jobs within buildset https://review.openstack.org/642857 | 18:40 |
| *** markwork has quit IRC | 18:47 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Forward artifacts to child jobs within buildset https://review.openstack.org/642857 | 19:09 |
| *** electrofelix has quit IRC | 19:27 | |
| *** gouthamr_ has joined #zuul | 19:34 | |
| *** gouthamr_ has quit IRC | 19:38 | |
| *** jamesmcarthur has quit IRC | 19:41 | |
| *** jamesmcarthur has joined #zuul | 19:42 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Fix deprecation warning around yaml.load https://review.openstack.org/644651 | 19:43 |
| *** jamesmcarthur has quit IRC | 19:45 | |
| *** jamesmcarthur_ has joined #zuul | 19:45 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Skip ansible validation for non-ansible tests https://review.openstack.org/644655 | 19:48 |
| tobiash | corvus: this improves test performance quite a bit ^ | 19:48 |
| tobiash | from 420s -> 320s for a full py37 run on my local machine | 19:49 |
| tobiash | I think that's important to stabilize the gate | 19:58 |
| *** jamesmcarthur_ has quit IRC | 20:02 | |
| *** jamesmcarthur has joined #zuul | 20:03 | |
| *** jamesmcarthur has quit IRC | 20:08 | |
| *** jamesmcarthur has joined #zuul | 20:08 | |
| corvus | tobiash: nice! i have a suggestion for an alternative impl | 20:18 |
| openstackgerrit | Merged openstack-infra/zuul master: Fix metadata not passed in provides/requires https://review.openstack.org/644445 | 20:19 |
| tobiash | corvus: ah yes, that makes sense | 20:23 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Skip ansible validation for non-ansible tests https://review.openstack.org/644655 | 20:42 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Skip ansible validation for non-ansible tests https://review.openstack.org/644655 | 20:44 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Skip ansible validation for non-ansible tests https://review.openstack.org/644655 | 20:45 |
| tobiash | corvus: that switches to your suggestion but keeps one conditional. Otherwise we would sacrifice the module override copy skip which is 50-100ms per test so I think it's worth to keep that conditional | 20:46 |
| corvus | tobiash: yeah, and at least it's in the tests where it's more clear | 20:46 |
| corvus | the version where there was a conditional in the server __init__ was unclear -- it looked like something that could be optimized out :) | 20:46 |
| tobiash | Yeah, that really felt like an ugly hack, thanks for that trick :) | 20:49 |
| *** hashar has quit IRC | 20:50 | |
| *** hashar has joined #zuul | 20:51 | |
| *** hashar has quit IRC | 20:54 | |
| *** hashar has joined #zuul | 20:55 | |
| *** hashar has quit IRC | 20:56 | |
| *** hashar has joined #zuul | 20:56 | |
| *** hashar has quit IRC | 20:57 | |
| *** hashar has joined #zuul | 20:57 | |
| openstackgerrit | Merged openstack-infra/zuul master: github: log the project name when getBranches fails https://review.openstack.org/583841 | 20:59 |
| *** hashar has quit IRC | 21:01 | |
| mnaser | tristanC, mordred: given your frontend work, this may be of interest https://usercontent.irccloud-cdn.com/file/OjopcRSJ/image.png | 21:18 |
| mnaser | aah | 21:19 |
| mnaser | i think i find it | 21:19 |
| mnaser | https://www.irccloud.com/pastebin/2vw44rpr/ | 21:19 |
| mnaser | im assuming that should be response.responseURL or something | 21:20 |
| mnaser | "`error.request` is an instance of XMLHttpRequest in the browser" | 21:23 |
| mnaser | so its possible that error.request is not set *if* setting up the request failed | 21:24 |
| mnaser | i dont have time to dig more into this.. but yeah, web/src/actions/status.js assumes all requests always suceed | 21:27 |
| *** hashar has joined #zuul | 21:35 | |
| openstackgerrit | Merged openstack-infra/zuul master: Add py37 environment to tox https://review.openstack.org/644498 | 21:57 |
| *** pcaruana has quit IRC | 22:08 | |
| *** jamesmcarthur has quit IRC | 22:08 | |
| *** hashar has quit IRC | 22:16 | |
| *** jamesmcarthur has joined #zuul | 22:30 | |
| *** jamesmcarthur has quit IRC | 22:45 | |
| *** jamesmcarthur has joined #zuul | 22:46 | |
| *** jamesmcarthur has quit IRC | 22:52 | |
| *** rlandy has quit IRC | 23:13 | |
| *** jamesmcarthur has joined #zuul | 23:14 | |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul-jobs master: Move chdir under args for shell task https://review.openstack.org/644696 | 23:17 |
| *** jamesmcarthur has quit IRC | 23:30 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/zuul-jobs master: Add vars to write-inventory https://review.openstack.org/644701 | 23:42 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Move chdir under args for shell task https://review.openstack.org/644696 | 23:45 |
| *** jamesmcarthur has joined #zuul | 23:51 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
