openstackgerrit | Monty Taylor proposed openstack-infra/zuul feature/zuulv3: Gate on zuul-stream-functional https://review.openstack.org/502864 | 00:01 |
---|---|---|
dmsimard | pabelanger: http://ara.readthedocs.io/en/latest/configuration.html#ara-ignore-parameters | 00:08 |
dmsimard | pabelanger: export ARA_IGNORE_PARAMETERS= | 00:08 |
dmsimard | or [ara] ignore_parameters= | 00:08 |
openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Rename success ansible variable to zuul_success https://review.openstack.org/502863 | 00:09 |
mordred | remote: https://review.openstack.org/502867 Revert "Add debug logging for cachecontrol library" | 00:09 |
openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Shift zuul_stream socket reading to use a for loop https://review.openstack.org/502492 | 00:12 |
*** jkilpatr has quit IRC | 00:18 | |
openstackgerrit | Jesse Keating proposed openstack-infra/zuul feature/zuulv3: Document the existence of zuul_success job var https://review.openstack.org/502869 | 00:18 |
jlk | So ^^ documents zuul_success, but we also need to document zuul_execution_phase and zuul_execution_trusted | 00:19 |
jlk | and zuul_execution_canonical_name_and_path and zuul_execution_branch | 00:19 |
*** yolanda has quit IRC | 00:30 | |
*** ianw_ has joined #zuul | 02:09 | |
*** ianw has quit IRC | 02:09 | |
*** ianw_ is now known as ianw | 02:10 | |
*** yolanda has joined #zuul | 03:48 | |
dmsimard | wow what did I miss, one of my reviews in zuul-jobs failed on a shade syntax error !? | 04:22 |
dmsimard | that's kind of an achievement mordred | 04:23 |
*** bhavik1 has joined #zuul | 04:41 | |
clarkb | dmsimard: has to do with zuul mergers will be fixed in the morning | 04:58 |
*** bhavik1 has quit IRC | 05:21 | |
*** hashar has joined #zuul | 05:54 | |
*** toabctl has joined #zuul | 06:36 | |
haypo | clarkb, ianw : "in summary, seems related to an invalid gi_frame in a generator object ... if that rings any bells" sorry, it doesn't ring any bell :-p | 07:02 |
haypo | clarkb, ianw : the best would be to test python 3.6, or at least python 3.5.4, to check if it's a known python bug (if the bug goes away with a more recent python version) or not | 07:02 |
haypo | "refcnt of 2 but blank object type?" https://storyboard.openstack.org/#!/story/2001186 this is an obvious bug. running your code in a python compiled in debug mode should help to detect such inconsistency, thanks to many assertions (removed in debug mode) | 07:13 |
ianw | haypo: yeah ... jeblair or pabelanger might be a be able to run under python3.5-dbg, however we have also merged https://review.openstack.org/#/c/502492/5 which avoids using a generator in this context | 07:31 |
haypo | ianw: "using a generator in this context" well, that's another option to workaround the issue ;) | 07:33 |
ianw | haypo: since this is happening in a multiprocessed forked() child, and then it's doing imports with imp.load_module() stuff ... there's so much going on it's mind bending :) | 07:35 |
haypo | ianw: since python3, multiprocessing gives you the choice how to spawn child processes: https://docs.python.org/dev/library/multiprocessing.html#contexts-and-start-methods spawn, fork or forkserver | 07:38 |
*** HTTP_____GK1wmSU has joined #zuul | 07:43 | |
*** HTTP_____GK1wmSU has left #zuul | 07:44 | |
haypo | ianw: but i agree that the traceback is scary :-) SSH, import, ansible, multiprocessing, many things at once :) | 07:47 |
ianw | haypo: yeah, thanks for looking in. i think at this point we're just going to leave it as a wait-and-see, because i don't think it can hold up the much larger rollout if we have the workaround. but we can all keep an eye out | 07:56 |
ianw | i will try to distil it to something for bugs.python.org too; at least it may come up in searches | 07:56 |
haypo | ianw: ok, no problem. at least, i'm happy so see python3 used ;) | 07:56 |
ianw | https://bugs.python.org/issue31426 | 08:37 |
*** hashar has quit IRC | 09:36 | |
haypo | ianw: i replied ;-) | 09:38 |
*** hashar has joined #zuul | 10:16 | |
*** jkilpatr has joined #zuul | 11:15 | |
electrofelix | SpamapS: it was a problem with running the job in Jenkins rather than Ansible, but it was traced to missing a 'tty' but there could be other culprits (jvm), which I guess could also occur with Ansible if you disabled the 'pty' allocation | 12:11 |
*** olaph has joined #zuul | 12:28 | |
*** olaph1 has quit IRC | 12:29 | |
*** dkranz has joined #zuul | 12:35 | |
*** olaph1 has joined #zuul | 12:51 | |
*** jkilpatr has quit IRC | 12:51 | |
*** olaph has quit IRC | 12:53 | |
jeblair | remote: https://review.openstack.org/502985 Run logstash submit jobs on localhost | 13:07 |
jeblair | mordred, clarkb: ^ doh | 13:07 |
*** jkilpatr has joined #zuul | 13:11 | |
openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Abort reconfiguration when cat jobs fail https://review.openstack.org/502989 | 13:16 |
tobiash | jeblair: have a comment ^^ (not sure if it justifies the -1 but we should discuss this) | 13:25 |
jeblair | tobiash: that may be a problem, but in general we expect the network connections between all the zuul components to be fairly reliable. if any component disconnects from gearman, lots of things get aborted without retrying, including merges if a merger is disconnected today. the only exception to that is that jobs should be retried if their executor disconnects. i think it might be a nice enhancement to add retries to all merge operations, ... | 13:31 |
jeblair | ... but i don't think we need to do it now since it doesn't change the status quo. | 13:31 |
mordred | jeblair: nice on the logstach on localhost :) | 13:40 |
mordred | jeblair: tox does not like you | 13:43 |
*** jkilpatr has quit IRC | 13:52 | |
SpamapS | so, removing the generator worked around the segfault? | 13:54 |
Shrews | mordred: is the 'project' inside a skip-if supposed to match project template names as well? | 13:54 |
Shrews | or even jeblair? ^^^ | 13:55 |
SpamapS | tristanC: dmsimard is there a software-factory channel? I don't want our back and forth about it to interfere with the zuul roll out chatter. | 13:58 |
SpamapS | and at the moment.. I'm stuck :-P | 14:00 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add tox bashate job https://review.openstack.org/503005 | 14:02 |
mordred | Shrews: no - just project name | 14:09 |
dmsimard | SpamapS: #softwarefactory | 14:10 |
mordred | Shrews: I have one tiny patch to migration script I'm going to push up - I do not think it should cause merge problems for you | 14:19 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul feature/zuulv3: Always strip {name} from name https://review.openstack.org/503013 | 14:19 |
mordred | Shrews: also, https://review.openstack.org/503014 and https://review.openstack.org/502329 in project-config make some of the renaming much happier | 14:20 |
Shrews | mordred: finding myself passing project name around to many places now just so mapNewJob() can use it. perhaps we can look at the code when you get here and find a better way | 14:21 |
mordred | Shrews: cool | 14:22 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul feature/zuulv3: Always strip {name} from name https://review.openstack.org/503013 | 14:35 |
openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Abort reconfiguration when cat jobs fail https://review.openstack.org/502989 | 14:39 |
clarkb | jeblair: do you want ^ to be in before we restart services to pick up mordreds zuul stream enerator removal? | 14:55 |
clarkb | jeblair: comment on ^ as well | 14:57 |
clarkb | skimming the python 3.5.4 commit log there is a non zero number of segfault fixes that I don't think are what we have already patched in | 15:10 |
clarkb | however none of them seem related to generators | 15:10 |
*** yolanda has quit IRC | 15:13 | |
*** hashar is now known as hasharAway | 15:15 | |
SpamapS | clarkb: did we ever find a reasonably concise reproducer? | 15:15 |
*** yolanda has joined #zuul | 15:17 | |
openstackgerrit | Jesse Keating proposed openstack-infra/zuul feature/zuulv3: Document the existence of zuul_success job var https://review.openstack.org/502869 | 15:18 |
openstackgerrit | David Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Matchers: voting, branch https://review.openstack.org/503034 | 15:19 |
*** yolanda has quit IRC | 15:29 | |
*** haypo has left #zuul | 15:36 | |
openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Abort reconfiguration when cat jobs fail https://review.openstack.org/502989 | 15:40 |
jlk | mordred: pabelanger: Do we already have something in v3 project-config, et al that take care of prepping upper constraints file? | 15:42 |
jlk | (or will that be part of the migration?) | 15:44 |
*** yolanda has joined #zuul | 15:46 | |
openstackgerrit | David Shrewsbury proposed openstack-infra/zuul feature/zuulv3: DNM Matchers: voting, branch https://review.openstack.org/503034 | 15:49 |
openstackgerrit | Tobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Don't fail on quota exceeded https://review.openstack.org/503051 | 15:51 |
jeblair | when the next change lands, i'm going to do a full zuul restart to get that and mordred's change in | 15:57 |
jeblair | well, i'll skip the mergers; they shouldn't be affected. but i will restart executors and scheduler | 15:57 |
tobiash | whoops, had a segfault in nodepool launcher... | 16:04 |
tobiash | my trust in python is disappearing slowly :( | 16:05 |
openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Abort reconfiguration when cat jobs fail https://review.openstack.org/502989 | 16:06 |
pabelanger | jlk: yes, for tox jobs we do. you can see shade as an example of how that works | 16:11 |
jeblair | i'm clearing out /var/lib/zuul/builds on the executors | 16:11 |
jlk | ah, okay. Hopefully that's a re-usable bit of stuff, since other projects will need it | 16:11 |
pabelanger | ya | 16:11 |
jlk | it's not in the shade repo is it? | 16:12 |
jeblair | zuul is restarted | 16:16 |
openstackgerrit | David Shrewsbury proposed openstack-infra/zuul feature/zuulv3: DNM Matchers: voting, branch https://review.openstack.org/503034 | 16:19 |
SpamapS | tobiash: time to rewrite python in Rust! :) | 16:19 |
SpamapS | and no, I'm not 100% snark on that. | 16:20 |
tobiash | SpamapS: at least I noticed that my nodepool was still running under py27 now I upgraded to py36... | 16:21 |
jeblair | SpamapS: indeed, though i'm afraid if we did that, we'd "just" end up with an unrecoverable runtime error instead of a segfault for this kind of bug. :| | 16:21 |
openstackgerrit | David Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Matchers: voting, branch https://review.openstack.org/503034 | 16:22 |
tobiash | we should return to assembler to get any buggy runtime out of the way ;) | 16:22 |
tobiash | but honestly, would python still be the choice if we would start from scratch with zuul and nodepool? | 16:23 |
jeblair | tobiash: i would seriously consider c++ or rust. | 16:24 |
jeblair | tobiash: zuulv5? :) | 16:24 |
SpamapS | jeblair: That's entirely possible, however, one thing that happens when writing rust is the compiler catches your mistakes quite often. | 16:25 |
SpamapS | But I imagine when writing a python interpreter one might have to choose the more dynamic memory models that it supports more often than not. | 16:25 |
SpamapS | and that does still mean panics ;) | 16:26 |
tobiash | uh, zuul -> python -> rust | 16:26 |
tobiash | (not) any problem can be solved with an additional layer :) | 16:27 |
SpamapS | no I think we'd replace the python with the rust :) | 16:27 |
jeblair | mordred, Shrews: i'm starting on change queue combining | 16:38 |
tristanC | jeblair: mordred: a preview of the zuul-web dashboard route is running here: https://38.145.32.112/zuul3/web/local/status.html | 16:47 |
clarkb | SpamapS: they already rewrote python in (r)python | 16:54 |
clarkb | and java and c# | 16:55 |
*** yolanda has quit IRC | 16:58 | |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add tox bashate job https://review.openstack.org/503005 | 17:06 |
SpamapS | clarkb: did not know about the C# one | 17:10 |
clarkb | fwiw pypy looks really solid but python3 support is still not quite there aiui | 17:11 |
SpamapS | pypy is done in C and some other language yeah? | 17:18 |
clarkb | rpython | 17:19 |
clarkb | my understanding is it is mostly (or maybe even all) rpython | 17:19 |
*** yolanda has joined #zuul | 17:20 | |
SpamapS | so in all seriousness, anything I can do to help? | 17:25 |
pabelanger | I'm still working on get ssh keys on to zuulv3 mergers, then was just going to see what new jobs need to be written | 17:26 |
clarkb | SpamapS: right now we are trying to reproduce the segfault with monty's don't use generator chang in place | 17:26 |
clarkb | SpamapS: so far I don't see any? but I may not be the best to look | 17:26 |
clarkb | SpamapS: I think if possible having a python 3.5.4 that we can install and test too would be useful | 17:26 |
*** yolanda has quit IRC | 17:27 | |
clarkb | SpamapS: apparently 3.5.4 has a security fix of some sort as well (so generally just a good idea) | 17:27 |
pabelanger | clarkb: SpamapS: so, do we need a new SRU? | 17:27 |
pabelanger | I haven't been following | 17:27 |
clarkb | 3.6 is also possibility? problem with that is newer ubuntu 3.6 doesn't necessarily have all the patches that hav emade it into current 3.6 and 3.5 | 17:27 |
clarkb | jeblair: ^ thoughts? | 17:27 |
*** yolanda has joined #zuul | 17:36 | |
SpamapS | clarkb: I'd guess that fix will be backported for Ubuntu. Links? | 17:39 |
clarkb | SpamapS: haypo didn't specify what the security bug was, let me look at change logs | 17:41 |
SpamapS | artful is python 3.6.2 btw | 17:41 |
SpamapS | so we can 'backport-package python3.6 xenial' | 17:42 |
clarkb | ok 3.6.2 is latest | 17:43 |
clarkb | so maybe we just check that | 17:43 |
SpamapS | Pretty easy to upload that to a PPA | 17:43 |
SpamapS | but would be good if we had an actual reproducer. ;) | 17:43 |
* SpamapS is sorry for saying that over and over | 17:43 | |
clarkb | SpamapS: currently busy in session but I think rough reproducer is run lots of data through a generator | 17:44 |
clarkb | in our case its megabytes 4096 bytes at a time | 17:44 |
clarkb | I can try to condense that down into a script that just does that from dev/urandom or something | 17:44 |
SpamapS | ahhh | 17:45 |
SpamapS | should work w/ /dev/zero in that case :) | 17:45 |
SpamapS | or even just copying big strings | 17:46 |
SpamapS | if it's really that | 17:46 |
clarkb | ya not sure if the actually content matters or not yet. but yes | 17:46 |
*** yolanda has quit IRC | 17:47 | |
mordred | jeblair: the merge conflict error was a "cannot read from remote repository" error | 17:49 |
mordred | git.exc.GitCommandError: Cmd('git') failed due to: exit code(128) | 17:49 |
mordred | cmdline: git fetch --tags -v origin | 17:49 |
mordred | stderr: 'fatal: Could not read from remote repository. | 17:49 |
mordred | jeblair: and was on zm07 | 17:51 |
mordred | pabelanger: ^^ | 17:51 |
mordred | pabelanger: we still have bogus keys or something | 17:54 |
mordred | on the new mergers | 17:54 |
pabelanger | mordred: fixed, just waiting for puppet wheel. We had incorrect hostkeys in ~/.ssh/known_hosts | 17:58 |
pabelanger | mordred: jeblair: clarkb: okay, zm08.o.o is back online for zuulv3 merger | 18:12 |
pabelanger | I'll hold off on other mergers for the moment | 18:12 |
pabelanger | jeblair: I did notice http://paste.openstack.org/show/620957/ in merger debug logs, going to see if I can fix | 18:14 |
openstackgerrit | Paul Belanger proposed openstack-infra/zuul feature/zuulv3: Fix typo with merger.getRepoState https://review.openstack.org/503092 | 18:24 |
pabelanger | I _think_ that is right, but mostly copied how it works in executor/server.py | 18:25 |
clarkb | my little reproducer is running on ze03 and unfortunately not reproducing | 18:31 |
clarkb | I'm using the job log output from a successful run with the idea that maybe something spceific in the output is the problem? | 18:32 |
clarkb | I am also using a file and not a socket I guess it is time to use netcat | 18:32 |
SpamapS | clarkb: make it use multiprocessing | 18:36 |
SpamapS | and import after fork | 18:36 |
clarkb | switching back to socket and recv isn't enough | 18:39 |
mordred | jeblair: http://paste.openstack.org/show/620958/ | 18:40 |
clarkb | SpamapS: http://paste.openstack.org/show/620959/ with nc serving http://logs.openstack.org/65/500365/22/check/shade-ansible-functional-devstack/045d1e1/job-output.txt.gz | 18:40 |
clarkb | SpamapS: also I just noticed that the old generator never really ended iteration. The outer control loop did | 18:45 |
clarkb | perhaps leainv a generator hanging around is a way to trip the gc up? | 18:45 |
Shrews | mordred: if you have a sec... | 18:47 |
openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Fix typo with merger.getRepoState https://review.openstack.org/503092 | 18:48 |
*** hasharAway is now known as hashar | 18:49 | |
*** yolanda has joined #zuul | 18:50 | |
*** electrofelix has quit IRC | 18:51 | |
SpamapS | clarkb: that seems as likely as anything else. | 18:52 |
openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Add max-job-timeout tenant setting https://review.openstack.org/502332 | 18:53 |
pabelanger | jeblair: mordred: clarkb: I've restarte zm08.o.o to pickup 503092^ | 18:56 |
*** yolanda has quit IRC | 18:56 | |
mordred | pabelanger: cool | 18:58 |
clarkb | SpamapS: http://paste.openstack.org/show/620960/ that just ran for a few minutes with no problems and is a more aggressive attempt at making the generator that isn't closed possibly more problematic | 19:02 |
mordred | jlk: in case it's useful to you : https://review.openstack.org/#/c/500627/ | 19:03 |
clarkb | I'm about ready to say meh on reproducing. If we want to go back to a generator we test it with proper iteration ending and/or use newer python3 | 19:03 |
clarkb | mordred: jeblair ^ | 19:03 |
* clarkb is going to make the input data much bigger just to see if that "helps" | 19:03 | |
SpamapS | clarkb: I'm going to try it with multiprocessing forking and doing the imports after that | 19:10 |
mordred | clarkb: http://logs.openstack.org/65/500365/22/check/shade-functional-devstack-legacy/539ceb5/job-output.txt.gz | 19:10 |
mordred | clarkb: http://logs.openstack.org/65/500365/22/check/shade-functional-devstack-legacy/539ceb5/job-output.txt.gz#_2017-09-12_17_51_33_951573 | 19:10 |
clarkb | SpamapS: ok, also keep in mind that end line check we look for isn't actually in the test input | 19:10 |
clarkb | SpamapS: so I just added it as the last line | 19:10 |
clarkb | SpamapS: otherwise the break condition to kill the generator early isn't tripped | 19:11 |
clarkb | mordred: http://logs.openstack.org/65/500365/22/check/shade-functional-devstack/7ef2644/job-output.txt.gz#_2017-09-12_18_03_03_693677 | 19:11 |
openstackgerrit | David Shrewsbury proposed openstack-infra/zuul feature/zuulv3: DNM Expand templates for project-specific matchers https://review.openstack.org/503143 | 19:12 |
mordred | https://review.openstack.org/#/c/500365/23/playbooks/devstack/run.yaml | 19:14 |
SpamapS | clarkb: what input do I put into nc? | 19:16 |
SpamapS | clarkb: I have dumped a few text files in and it exits early on empty lines | 19:16 |
SpamapS | (ah n/m that was EOF) | 19:17 |
clarkb | SpamapS: I used http://logs.openstack.org/65/500365/22/check/shade-ansible-functional-devstack/045d1e1/job-output.txt.gz with "[Zuul] Task exit code" appended to it to trip the stop condition before end of iteration | 19:18 |
SpamapS | ahh | 19:18 |
SpamapS | kk, with a small file python3.6 on OS X works ;) | 19:19 |
clarkb | since tat is roughly what it was doing in production | 19:19 |
clarkb | ya I can't get that to crash on 3.5 on ze03 | 19:19 |
SpamapS | what python are you thinking SHOULD die? | 19:19 |
SpamapS | stock xenial? | 19:19 |
SpamapS | or xenial-proposed python3.5? | 19:19 |
clarkb | I am testing on the proposed one with the dict patches | 19:22 |
clarkb | since that is what segfaulted in production | 19:22 |
*** yolanda has joined #zuul | 19:23 | |
openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Decode stdout from ansible-playbook before logging https://review.openstack.org/502362 | 19:25 |
openstackgerrit | Tobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Add connection_type to build and upload information https://review.openstack.org/503148 | 19:33 |
openstackgerrit | Tobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Rename ssh_port to connection_port https://review.openstack.org/500800 | 19:35 |
openstackgerrit | Tobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Add username to build and upload information https://review.openstack.org/453968 | 19:35 |
openstackgerrit | Tobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Support username also for unmanaged cloud images https://review.openstack.org/500808 | 19:35 |
openstackgerrit | Tobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Add connection_type to build and upload information https://review.openstack.org/503148 | 19:35 |
mordred | OSError: Multiple exceptions: [Errno 111] Connect call failed ('2001:4800:7819:104:be76:4eff:fe05:302', 79), [Errno 111] Connect call failed ('104.239.137.167', 79) | 19:40 |
mordred | clarkb, pabelanger: ^^ | 19:40 |
*** yolanda has quit IRC | 19:45 | |
SpamapS | clarkb: does not die on xenial python3.5 | 19:46 |
clarkb | SpamapS: no it isn't dying for me | 19:47 |
clarkb | (so it is a bad reproducer) | 19:47 |
SpamapS | agreed :( | 19:49 |
SpamapS | trying with proposed too | 19:49 |
SpamapS | just.. to try :-P | 19:49 |
SpamapS | and I'm importing time and socket in the multiprocessed fork too btw | 19:49 |
SpamapS | oh, n/m, there's no python3.5 in proposed | 19:50 |
pabelanger | mordred: where did you see the OSerror? | 19:53 |
*** jkilpatr has joined #zuul | 19:54 | |
SpamapS | clarkb: http://paste.openstack.org/show/620964/ <-- with multiprocessing in the mix | 19:55 |
SpamapS | but yah, nothing seems to reproduce this | 19:56 |
openstackgerrit | Monty Taylor proposed openstack-infra/zuul feature/zuulv3: Bind to :: instead of 0.0.0.0 in log streamer https://review.openstack.org/503158 | 19:59 |
jeblair | INFO:zuul.Migrate: <__main__.ChangeQueue object at 0x7fc695abd4a8> containing openstack-dev/devstack, openstack-dev/grenade, openstack-infra/devstack-gate, openstack-infra/pypi-mirror, openstack/cinder, openstack/keystone, openstack/keystone-tempest-plugin, openstack/neutron, openstack/nova, openstack/python-swiftclient, openstack/requirements, openstack/swift, openstack/tempest | 20:01 |
jeblair | mordred, Shrews: ^ progress | 20:01 |
SpamapS | oooooo | 20:02 |
jlk | https://review.openstack.org/502208 got harder than expected. Had to transpose some puppet into ansible to prep things for Zanata client. Ready for review. | 20:03 |
openstackgerrit | Tobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Don't gather host keys for non ssh connections https://review.openstack.org/503166 | 20:06 |
mordred | jeblair: \o/ | 20:06 |
*** yolanda has joined #zuul | 20:07 | |
mordred | clarkb, pabelanger: https://review.openstack.org/#/c/491805/ | 20:09 |
openstackgerrit | David Shrewsbury proposed openstack-infra/zuul feature/zuulv3: DNM Expand templates for project-specific matchers https://review.openstack.org/503143 | 20:09 |
pabelanger | +2 | 20:15 |
jeblair | http://paste.openstack.org/show/620970/ | 20:18 |
jeblair | those are our 15 shared queues | 20:18 |
openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Migration: handle shared queues https://review.openstack.org/503170 | 20:19 |
clarkb | I have approved the migration tool | 20:24 |
openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Bind to :: instead of 0.0.0.0 in log streamer https://review.openstack.org/503158 | 20:25 |
SpamapS | I'm Clark Boylan and I Approve This Migration | 20:25 |
*** yolanda has quit IRC | 20:27 | |
mordred | remote: https://review.openstack.org/503173 Don't map gate-cross jobs automatically to tox jobs | 20:27 |
openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Document the existence of zuul_success job var https://review.openstack.org/502869 | 20:30 |
*** jkilpatr has quit IRC | 20:35 | |
openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Add migration tool for v2 to v3 conversion https://review.openstack.org/491805 | 20:35 |
*** jkilpatr has joined #zuul | 20:40 | |
pabelanger | clarkb: jeblair mordred: ze01 / ze02 restarted | 21:00 |
jlk | jeblair: only tangentially related to your script, I'm curious about how 'vinz-webclient' wound up being a specific shared queue, and in exploring that curiosity, I realized that the vinz project(s) are vapor ware. There's been no commits to them other than site wide type commits. | 21:17 |
openstackgerrit | Jamie Lennox proposed openstack-infra/zuul-jobs master: A nodejs install and npm upload role https://review.openstack.org/503201 | 21:20 |
*** yolanda has joined #zuul | 21:22 | |
openstackgerrit | Jamie Lennox proposed openstack-infra/zuul-jobs master: A nodejs install and npm upload role https://review.openstack.org/503201 | 21:26 |
clarkb | jlk: yes no one ever made commits to it | 21:26 |
clarkb | jlk: I think we haven't retired it formally in hopes it might get pciekd up again but with new gerrit ui things upstream maybe its not worth worrying about | 21:27 |
jlk | I think that was a project idea from Philip Schwartz, but he never did anything. | 21:27 |
clarkb | ya at least upstream code never showed up | 21:27 |
clarkb | but we have talked about resurrecting it in some form or another over time | 21:28 |
clarkb | (due to gertty success) | 21:28 |
clarkb | but not gerrit is doing its own UI rewrite using proper js instead of gwt | 21:28 |
clarkb | s/not/now/ | 21:28 |
openstackgerrit | Jamie Lennox proposed openstack-infra/zuul-jobs master: A nodejs install and npm upload role https://review.openstack.org/503201 | 21:33 |
pabelanger | jlk: jamielennox: keep in mind, we'd run upload-npm from executor, so npm_target would likely be {{ zuul.executor.work_root }}/artifacts/foobar.npm (right extension) | 21:37 |
jamielennox | pabelanger: bah, right, that would make sense - fyi got that from: https://github.com/openstack-infra/zuul-jobs/blob/master/roles/upload-pypi/defaults/main.yaml#L2 | 21:40 |
*** hashar has quit IRC | 21:43 | |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: web: add /{tenant}/builds route https://review.openstack.org/466561 | 21:47 |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: web: add /{source}/{project}.pem route https://review.openstack.org/502530 | 21:47 |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: web: add /tenants route https://review.openstack.org/503268 | 21:47 |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: web: add /{tenant}/status route https://review.openstack.org/503269 | 21:47 |
openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: web: add /{tenant}/jobs route https://review.openstack.org/503270 | 21:47 |
openstackgerrit | David Shrewsbury proposed openstack-infra/zuul feature/zuulv3: DNM Expand templates for project-specific matchers https://review.openstack.org/503143 | 22:05 |
Shrews | jeblair: mordred: ^^^ may or may not actually work. left the DNM part in the commit message so we can review the output | 22:06 |
openstackgerrit | Jesse Keating proposed openstack-infra/zuul feature/zuulv3: Drop use_app boolean from github client https://review.openstack.org/503403 | 22:15 |
openstackgerrit | Tobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Fix typo in variable name https://review.openstack.org/503422 | 22:18 |
openstackgerrit | Jesse Keating proposed openstack-infra/zuul-jobs master: Add content to support translation jobs https://review.openstack.org/502207 | 22:19 |
Shrews | definitely NOT correct. looks like EVERYTHING expanded | 22:23 |
openstackgerrit | David Shrewsbury proposed openstack-infra/zuul feature/zuulv3: DNM Expand templates for project-specific matchers https://review.openstack.org/503143 | 22:36 |
Shrews | that should be at least a little more correct | 22:38 |
Shrews | ugh, forget to actually set irrelevant-files | 22:38 |
openstackgerrit | Jesse Keating proposed openstack-infra/zuul-jobs master: Improve the check for sudo not working https://review.openstack.org/503448 | 22:41 |
openstackgerrit | Jesse Keating proposed openstack-infra/zuul-jobs master: Improve the check for sudo not working https://review.openstack.org/503448 | 22:53 |
openstackgerrit | Tobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Don't fail on quota exceeded https://review.openstack.org/503051 | 23:07 |
openstackgerrit | Tobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Support several paused request handlers https://review.openstack.org/503456 | 23:07 |
mordred | Shrews: this time it seems to have exapnded nothing | 23:12 |
*** jkilpatr has quit IRC | 23:17 | |
openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Drop use_app boolean from github client https://review.openstack.org/503403 | 23:42 |
openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Always strip {name} from name https://review.openstack.org/503013 | 23:45 |
*** pabelanger_ has joined #zuul | 23:46 | |
*** jesusaurum has joined #zuul | 23:48 | |
*** jianghuaw_ has joined #zuul | 23:48 | |
*** pabelanger has quit IRC | 23:48 | |
*** jesusaur has quit IRC | 23:48 | |
*** jianghuaw has quit IRC | 23:48 | |
openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Matchers: voting, branch https://review.openstack.org/503034 | 23:50 |
*** jianghuaw_ has quit IRC | 23:52 | |
*** jianghuaw has joined #zuul | 23:54 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!