| *** rlandy has quit IRC | 00:16 | |
| *** xinliang has quit IRC | 00:22 | |
| *** odyssey4me has quit IRC | 00:29 | |
| *** odyssey4me has joined #zuul | 00:29 | |
| *** xinliang has joined #zuul | 00:34 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add /{tenant}/jobs/{job_name} route https://review.openstack.org/550978 | 01:39 |
|---|---|---|
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add /{tenant}/projects routes https://review.openstack.org/550979 | 01:39 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add /{tenant}/pipelines route https://review.openstack.org/541521 | 01:39 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: Rename javascript package to zuul-dashboard https://review.openstack.org/551999 | 01:40 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: dashboard: add /{tenant}/job.html page to display job details https://review.openstack.org/535545 | 01:40 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: dashboard: add /{tenant}/projects.html web page https://review.openstack.org/537870 | 01:40 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: Fix indentation and renable the eslint rule https://review.openstack.org/545671 | 01:41 |
| *** Wei_Liu has joined #zuul | 01:42 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: Shift html templates into components https://review.openstack.org/551327 | 01:43 |
| *** harlowja has quit IRC | 01:59 | |
| dmsimard | Does nodepool have a mechanism where if you have two providers for a label, to prefer a provider over the other ? | 02:11 |
| dmsimard | As in, only start using the second provider once the first one is maxed out | 02:11 |
| clarkb | no | 02:11 |
| clarkb | the currwnt behavior aiui is each provider polls zk looking for labels it can satisfy and when it finds them immediately services the reques | 02:12 |
| clarkb | so its a rough fifo based on what providers that arent at quota can provide | 02:12 |
| dmsimard | Ah, it would be a great feature for cost optimization | 02:16 |
| dmsimard | For example if I'm using clouds with different billing structures, I could "burst" into the second cloud after a certain point | 02:17 |
| dmsimard | or if the first cloud is out of quota | 02:17 |
| dmsimard | A better example would be an on-premise cloud with limited resources and then I only want to burst into RAX/INAP/etc if that on-prem cloud is full | 02:18 |
| dmsimard | tobiash came up with great ideas around quota management, maybe he has something in mind for ^ :p | 02:18 |
| clarkb | for this case providers would need to coordinate | 02:19 |
| clarkb | maybe by setting a flag in zk indicating they are at quota unlocking providers that were waiting | 02:19 |
| clarkb | it is doable just need to add coordination of some form between provider handlers | 02:25 |
| dmsimard | yeah I'm not familiar with the internals of nodepool, I guess I could see a "priority" sort of field in the label provider config ? Something like https://gist.github.com/dmsimard/e884b749f07c64a177078fadb8b4f8b6 | 02:25 |
| dmsimard | I guess the implication here is that "onprem cloud" (as per my example) would need to tag the request somehow and tell zk it's out of quota | 02:26 |
| dmsimard | so that the next provider knows it's "his" turn | 02:27 |
| * dmsimard hand waving | 02:27 | |
| dmsimard | Request for a label has "priority" 1 when it comes in. Provider with priority 1 for that sees the request but it's out of quota so priority++. Provider with priority 2 sees it, has room so it picks up the request and does whatever it is. | 02:31 |
| * dmsimard more hand waving | 02:32 | |
| dmsimard | So it'd be fifo but for the same level of priority | 02:49 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add jobs graph rendering https://review.openstack.org/537869 | 03:18 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add project pipeline rendering https://review.openstack.org/537871 | 03:18 |
| *** harlowja has joined #zuul | 04:11 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add project pipeline rendering https://review.openstack.org/537871 | 04:21 |
| tristanC | is it normal the DiskAccountant is running on executor merge jobs? | 04:39 |
| *** harlowja has quit IRC | 04:50 | |
| tobiash | It should only run during job execution | 04:56 |
| tobiash | dmsimard: that won't work as there can be several providers of prio 1 | 04:58 |
| tobiash | But the use case is interesting and might also something we want some time in the future | 04:59 |
| tristanC | it seems like secret needs to be added with a name now, it doesn't seems to use the name set in the secret anymore | 05:05 |
| tristanC | or something else is going on, post jobs are now failing with "'my_site' is undefined" | 05:18 |
| *** swest has joined #zuul | 05:35 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: model: freeze parent playbooks https://review.openstack.org/553144 | 05:37 |
| tristanC | it seems like we need this ^ | 05:37 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: Revert "Don't store references to secret objects from jobs" https://review.openstack.org/553147 | 05:45 |
| *** sjmurdoch795 has joined #zuul | 05:51 | |
| sjmurdoch795 | THIS IS A FREENODE BREAKING NEWS ALERT!! Hitechcg AND opal ARE GOING AT IT RIGHT NOW WITH A LOT OF FIGHTING AND ARGUING WOW YOU DON'T WANT TO MISS THIS!! TYPE /JOIN ## TO SEE THE ACTION...AGAIN TYPE /JOIN ## TO SEE THE ACTION!! | 05:51 |
| sjmurdoch795 | THIS IS A FREENODE BREAKING NEWS ALERT!! Hitechcg AND opal ARE GOING AT IT RIGHT NOW WITH A LOT OF FIGHTING AND ARGUING WOW YOU DON'T WANT TO MISS THIS!! TYPE /JOIN ## TO SEE THE ACTION...AGAIN TYPE /JOIN ## TO SEE THE ACTION!! | 05:51 |
| sjmurdoch795 | swest Wei_Liu xinliang odyssey4me ssbarnea sshnaidm pwhalen JasonCL dmellado yolanda_ fbo _ari_ openstackgerrit dtruong myoung|afk dmsimard robled AJaeger fungi snapiri toabctl patriciadomin pbrobinson sdoran patrickeast gregdek zhuli adam_g mhu Diabelko weshay pabelanger openstack openstackstatus abelur jimi|ansible qwc haint_ smyers zigo jhesketh rcarrillocruz tristanC TheJulia kklimonda robcresswell jtanner zaro_ mrhillsman mnaser f | 05:51 |
| *** sjmurdoch795 has quit IRC | 05:51 | |
| *** AJaeger has quit IRC | 07:56 | |
| *** hashar has joined #zuul | 07:56 | |
| *** AJaeger has joined #zuul | 08:20 | |
| *** bhavik1 has joined #zuul | 08:26 | |
| *** bhavik1 has quit IRC | 08:29 | |
| *** jpena|off is now known as jpena | 08:43 | |
| openstackgerrit | Matthieu Huin proposed openstack-infra/zuul master: zuul autohold: allow operator to specify nodes TTL https://review.openstack.org/543403 | 09:11 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/nodepool master: Add separate modules for management commands https://review.openstack.org/536303 | 09:54 |
| mhu | corvus, jhesketh tobiash I rebased https://review.openstack.org/#/c/543403/, it should pass the gate now | 09:56 |
| *** electrofelix has joined #zuul | 10:05 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Generate symlinks during tests https://review.openstack.org/553316 | 10:21 |
| tobiash | pabelanger: this should fix your symlink issue ^ | 10:21 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/nodepool master: webapp: add optional admin endpoint https://review.openstack.org/536319 | 10:35 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Generate symlinks during tests https://review.openstack.org/553316 | 10:55 |
| openstackgerrit | Matthieu Huin proposed openstack-infra/zuul master: zuul web: add admin endpoint, enqueue & autohold commands https://review.openstack.org/539004 | 10:55 |
| *** JasonCL has quit IRC | 10:59 | |
| *** dtruong_ has joined #zuul | 11:18 | |
| *** snapiri- has joined #zuul | 11:18 | |
| *** maxamillion_ has joined #zuul | 11:20 | |
| *** zaro__ has joined #zuul | 11:20 | |
| *** mnaser_ has joined #zuul | 11:20 | |
| openstackgerrit | Matthieu Huin proposed openstack-infra/nodepool master: webapp: add optional admin endpoint https://review.openstack.org/536319 | 11:21 |
| *** gundalow1 has joined #zuul | 11:24 | |
| *** colettecello has joined #zuul | 11:25 | |
| *** timrchavez_ has joined #zuul | 11:25 | |
| *** pbrobinson_ has joined #zuul | 11:27 | |
| *** dtruong has quit IRC | 11:27 | |
| *** snapiri has quit IRC | 11:27 | |
| *** pbrobinson has quit IRC | 11:27 | |
| *** zaro_ has quit IRC | 11:27 | |
| *** mnaser has quit IRC | 11:27 | |
| *** maxamillion has quit IRC | 11:27 | |
| *** timrc has quit IRC | 11:27 | |
| openstackgerrit | Matthieu Huin proposed openstack-infra/zuul master: zuul web: add admin endpoint, enqueue & autohold commands https://review.openstack.org/539004 | 11:27 |
| *** mnaser_ is now known as mnaser | 11:27 | |
| *** maxamillion_ is now known as maxamillion | 11:27 | |
| *** pbrobinson_ is now known as pbrobinson | 11:27 | |
| *** sshnaidm_ has joined #zuul | 11:27 | |
| *** jlk` has joined #zuul | 11:27 | |
| *** fbo_ has joined #zuul | 11:28 | |
| *** _ari_ has quit IRC | 11:28 | |
| *** fbo has quit IRC | 11:28 | |
| *** gundalow has quit IRC | 11:28 | |
| *** sshnaidm has quit IRC | 11:28 | |
| *** gothicmindfood has quit IRC | 11:28 | |
| *** jlk has quit IRC | 11:28 | |
| *** _ari_ has joined #zuul | 11:28 | |
| *** maxamillion has quit IRC | 11:28 | |
| *** maxamillion has joined #zuul | 11:28 | |
| *** mnaser has quit IRC | 11:28 | |
| *** mnaser has joined #zuul | 11:28 | |
| *** zaro__ has quit IRC | 11:28 | |
| *** zaro__ has joined #zuul | 11:28 | |
| *** fbo_ has quit IRC | 11:28 | |
| *** fbo_ has joined #zuul | 11:28 | |
| *** gundalow1 is now known as gundalow | 11:35 | |
| *** gundalow has quit IRC | 11:36 | |
| *** gundalow has joined #zuul | 11:36 | |
| *** dmellado has quit IRC | 11:50 | |
| *** colettecello has quit IRC | 11:53 | |
| *** gothicmindfood has joined #zuul | 11:53 | |
| *** dmellado has joined #zuul | 11:58 | |
| *** dmellado has quit IRC | 12:05 | |
| *** odyssey4me has quit IRC | 12:05 | |
| *** odyssey4me has joined #zuul | 12:05 | |
| *** jpena is now known as jpena|lunch | 12:21 | |
| *** dmellado has joined #zuul | 12:23 | |
| *** snapiri- is now known as snapiri | 12:27 | |
| *** snapiri has quit IRC | 12:27 | |
| *** snapiri has joined #zuul | 12:28 | |
| *** dmellado has quit IRC | 12:30 | |
| *** rlandy has joined #zuul | 12:35 | |
| *** sshnaidm_ is now known as sshnaidm | 12:35 | |
| *** snapiri has quit IRC | 12:41 | |
| *** snapiri has joined #zuul | 12:41 | |
| *** JasonCL has joined #zuul | 13:01 | |
| *** JasonCL has joined #zuul | 13:02 | |
| *** JasonCL has quit IRC | 13:07 | |
| *** myoung|afk is now known as myoung|rover | 13:09 | |
| *** JasonCL has joined #zuul | 13:16 | |
| *** dmellado has joined #zuul | 13:19 | |
| *** dmellado has quit IRC | 13:20 | |
| *** hashar has quit IRC | 13:24 | |
| *** jpena|lunch is now known as jpena | 13:29 | |
| *** snapiri has quit IRC | 13:29 | |
| *** snapiri has joined #zuul | 13:32 | |
| *** JasonCL has quit IRC | 13:32 | |
| *** JasonCL has joined #zuul | 13:37 | |
| *** myoung|rover is now known as myoung|logdiver | 13:39 | |
| *** myoung|logdiver is now known as myoung|rover | 13:39 | |
| *** hashar has joined #zuul | 13:41 | |
| *** JasonCL has quit IRC | 13:42 | |
| *** JasonCL has joined #zuul | 13:43 | |
| *** JasonCL has quit IRC | 13:44 | |
| *** JasonCL has joined #zuul | 13:44 | |
| *** JasonCL has quit IRC | 13:49 | |
| Shrews | corvus: tobiash: so my current theory on how to fix the no_log breakage is we either: a) whitelist certain modules (like the 'file' module in the apache-logs-conf role failure), or b) only replace the item with a dict if it is a string or dict (so based on type) | 13:54 |
| *** hashar has quit IRC | 13:54 | |
| Shrews | i have to step away to get ready for a chiropractor appointment | 13:54 |
| *** JasonCL has joined #zuul | 13:55 | |
| *** JasonCL has quit IRC | 13:59 | |
| *** JasonCL has joined #zuul | 13:59 | |
| -openstackstatus- NOTICE: POST_FAILURE results on Tempest-based jobs since the most recent Zuul security fixes are being investigated; rechecking those won't help for now but we'll keep you posted once a solution is identified | 14:12 | |
| *** ChanServ changes topic to "POST_FAILURE results on Tempest-based jobs since the most recent Zuul security fixes are being investigated; rechecking those won't help for now but we'll keep you posted once a solution is identified" | 14:12 | |
| *** JasonCL has quit IRC | 14:23 | |
| *** JasonCL has joined #zuul | 14:24 | |
| *** JasonCL has quit IRC | 14:25 | |
| *** JasonCL has joined #zuul | 14:26 | |
| corvus | Shrews, tobiash, dmsimard: is the no_log problem perhaps that we only make a shallow copy of the dict, so it's safe to modify the first level, but when we modify anything deeper than that, we're modifying the actual data ansible uses? | 14:32 |
| Shrews | corvus: about to leave, but i think the problem is that we blindly always replace the item data with a dict | 14:33 |
| corvus | hrm... strip_internal_keys should end up doing a deep copy... | 14:33 |
| corvus | Shrews: oh, i see what you mean | 14:33 |
| Shrews | which, in the case of the 'file' module, is a file object, thus the 'when' breaks | 14:33 |
| tobiash | Maybe, but I'm currently afk | 14:33 |
| Shrews | and that results in: "The conditional check 'item.stat.isreg or item.stat.islnk' failed. The error was: error while evaluating conditional (item.stat.isreg or item.stat.islnk): 'dict object' has no attribute 'stat'" | 14:34 |
| Shrews | b/c we've made the item a dict | 14:34 |
| corvus | Shrews: so our traverse of that structure just needs to be more sophisticated | 14:34 |
| Shrews | corvus: right | 14:34 |
| corvus | we should be able to make a test case for this. i'll start on that | 14:35 |
| Shrews | so either whitelist by module, or replace based on type (which i'm not sure if that is good enough) | 14:35 |
| Shrews | i have to run now... bbs | 14:35 |
| tobiash | Ah, so we're replacing some munch like structure wuth a dict? | 14:36 |
| *** JasonCL has quit IRC | 14:38 | |
| *** JasonCL has joined #zuul | 14:39 | |
| *** JasonCL has quit IRC | 14:41 | |
| *** JasonCL has joined #zuul | 14:42 | |
| *** JasonCL has quit IRC | 14:44 | |
| *** dmellado_ has joined #zuul | 14:46 | |
| *** maxamillion has quit IRC | 14:49 | |
| *** maxamillion has joined #zuul | 14:49 | |
| *** JasonCL has joined #zuul | 14:49 | |
| *** JasonCL has joined #zuul | 14:49 | |
| *** dkranz has joined #zuul | 14:51 | |
| *** dmellado_ is now known as dmellado | 14:54 | |
| *** JasonCL has quit IRC | 14:55 | |
| *** myoung|rover is now known as myoung|rover|brb | 14:56 | |
| *** JasonCL has joined #zuul | 15:01 | |
| tristanC | corvus: can you have a look at https://review.openstack.org/#/c/553147/ please, it seems like master is unusable without it or https://review.openstack.org/#/c/553144/ | 15:06 |
| *** JasonCL has quit IRC | 15:07 | |
| corvus | tristanC: i don't immediately understand that. openstack-infra has been running with that change for a few days | 15:11 |
| corvus | tristanC: can you precisely describe a failure? | 15:12 |
| tristanC | corvus: without the revert, base post-run fail with "'secret' is undefined" | 15:13 |
| tristanC | has the scheduler been restarted with it? | 15:14 |
| corvus | tristanC: oh, you're right, we haven't restarted the scheduler :) | 15:14 |
| corvus | we've only been restarting the executors | 15:14 |
| corvus | tristanC: so the test case we need to make is: a base job with a secret in a post playbook, and have that inherited by some other job (which doesn't need to use secrets) ? | 15:15 |
| tristanC | corvus: yes, like a base job using upload-logs | 15:17 |
| corvus | tristanC: does your example rename the secret when the job uses it, or not? | 15:17 |
| tristanC | corvus: i tried both, but by default we don't rename it | 15:17 |
| corvus | tristanC: ok, thanks. i've approved the revert | 15:18 |
| tristanC | you're welcome :) | 15:19 |
| clarkb | please let me know if I can help review or debug the apache no_log thing. I'm booting up now. I have tea :) | 15:22 |
| pabelanger | tobiash: let me test 553316 in a few moments | 15:37 |
| openstackgerrit | Merged openstack-infra/zuul master: Revert "Don't store references to secret objects from jobs" https://review.openstack.org/553147 | 15:37 |
| *** JasonCL has joined #zuul | 15:37 | |
| *** myoung|rover|brb is now known as myoung|rover | 15:38 | |
| *** JasonCL has quit IRC | 15:40 | |
| *** JasonCL has joined #zuul | 15:43 | |
| *** JasonCL has quit IRC | 15:48 | |
| *** JasonCL has joined #zuul | 15:48 | |
| Shrews | corvus: ok, i'm back. anything i can help with on the no_log thing? | 15:51 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: WIP: improve no_log test https://review.openstack.org/553450 | 15:53 |
| corvus | Shrews, clarkb: ^ i'm working on a test, but that isn't failing | 15:53 |
| *** kmalloc has joined #zuul | 15:53 | |
| Shrews | corvus: maybe a 'when' clause is processed later? | 15:54 |
| Shrews | try adding: | 15:54 |
| Shrews | when: | 15:55 |
| Shrews | - item.stat.isreg | 15:55 |
| Shrews | after the with_items | 15:55 |
| corvus | Shrews: "msg": "The conditional check 'item.stat.isreg' failed. The error was: error while evaluating conditional (item.stat.isreg): 'dict object' has no attribute 'stat' | 15:56 |
| tobiash | Trying to use the registered var probably breaks it | 15:56 |
| corvus | Shrews: w00t, that looks like that's it, yeah? | 15:56 |
| Shrews | Yep | 15:56 |
| corvus | oh, er... that still happens without no_log | 15:56 |
| Shrews | oh? | 15:57 |
| *** dtruong_ has quit IRC | 15:58 | |
| corvus | oh, i need a third thing that operates on the stat | 15:58 |
| *** dtruong has joined #zuul | 15:58 | |
| corvus | something to replace "create hard links" in the example | 15:58 |
| corvus | based on http://git.openstack.org/cgit/openstack-dev/devstack/tree/roles/apache-logs-conf/tasks/main.yaml#n27 i thought it was "Dereference files" that was failing, but it's "Create hard links" | 15:59 |
| corvus | i'll see if a can just do a debug in place of 'create hard links', but use a 'when' | 15:59 |
| Shrews | i'm not following the need for the 3rd thing | 16:01 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: WIP: improve no_log test https://review.openstack.org/553450 | 16:01 |
| corvus | Shrews: the second thing gets us the 'item.stat' structure, so the third thing is the one that tries to access that structure | 16:02 |
| corvus | Shrews: ^ updated -- that seems to show the error with no_log (and works without no_log) | 16:02 |
| Shrews | corvus: ah, i see now | 16:02 |
| Shrews | the previous successful failure was actually a false failure O.o :) | 16:03 |
| Shrews | that may be my most favorite new sentence | 16:03 |
| *** harlowja has joined #zuul | 16:03 | |
| *** electrofelix has quit IRC | 16:11 | |
| *** myoung|rover is now known as myoung|lunch | 16:15 | |
| clarkb | corvus: Shrews maybe you know with_items: "{{ src_paths.results }}" is also what the apache thing in tempest does (specifically with items being foo.results | 16:16 |
| clarkb | but reading the stat module's docs there is no results structure returned | 16:16 |
| clarkb | is that put in lpace by the register? | 16:16 |
| pabelanger | tobiash: +2 on 553316, fixes synchronize from zuul-executor in untrusted jobs | 16:17 |
| pabelanger | thanks | 16:17 |
| Shrews | clarkb: are you asking about the test in 553450? or what's happening in the tempest job itself. sorry i'm not groking | 16:27 |
| clarkb | Shrews: both, they both register a stat result then access it via registered_var.results.stat.entry rather than registered_var.stat.entry which is what the docs seem to say you should do instead | 16:28 |
| corvus | maybe it comes from with_items ? | 16:29 |
| corvus | like, if you use stat in a list, you get 'results' | 16:29 |
| corvus | oh this may be docs: http://docs.ansible.com/ansible/latest/playbooks_loops.html#using-register-with-a-loop | 16:30 |
| corvus | so i think i'm narrowing down on the issue -- strip_internal_keys does not deep-copy lists: https://github.com/ansible/ansible/blob/stable-2.3/lib/ansible/vars/__init__.py#L82 | 16:32 |
| clarkb | yup that seems to explain it. Thanks | 16:32 |
| corvus | it deep copies *dicts*, but not lists | 16:32 |
| corvus | and since results is a list, when we modify our "copy" for the json log, we are also modifying the original | 16:33 |
| clarkb | oh hey my question was relevant to the behavior too :) | 16:33 |
| corvus | :) | 16:33 |
| Shrews | oh, i actually did not realize that we had a copy | 16:34 |
| corvus | probably the quickest fix would just be to do our own *actual* deep copy before calling strip_internal_keys. we'll end up copying a bit too much (because strip_internal_keys will copy the dicts again). but it's easy... | 16:34 |
| corvus | Shrews: well, we only have one by virtue of the fact that strip_internal_keys makes a "copy" | 16:34 |
| corvus | just... not very well. | 16:34 |
| clarkb | corvus: extra copies is probably not the worst thing if it results in consistent behavior | 16:35 |
| Shrews | corvus: hrm, i believe that import at the top for strip_internal_keys is documented incorrectly. versions should be swapped there | 16:37 |
| Shrews | (just as an annoying aside) | 16:37 |
| corvus | Shrews: ok, i can go ahead and swap them in my patch | 16:38 |
| *** myoung|lunch is now known as myoung|rover | 16:45 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Fix no_log bug with result lists https://review.openstack.org/553450 | 16:46 |
| corvus | Shrews, clarkb, tobiash: ^ | 16:46 |
| Shrews | i wonder if we should submit a PR to ansible for them to use deepcopy() instead of copy()? i'd be afraid that would break the world somehow though | 16:48 |
| clarkb | corvus: lgtm | 16:49 |
| Shrews | gundalow: jimi|ansible: ^^^ | 16:49 |
| clarkb | Shrews: won't break this change at least. Will just do more copying than necessary | 16:49 |
| Shrews | corvus: lgtm too. didn't +A in case others want to look | 16:51 |
| tobiash | corvus: thanks for fixing that :) | 16:51 |
| tobiash | I guess we should start adding more positive tests to tox-remote | 16:52 |
| tobiash | Our break openstack ratio when fixing security stuff is not that good atm :( | 16:53 |
| clarkb | tobiash: ya a few representative jobs are probably a good idea. I don't expect we'll ever catch everything like pabelanger's copy zuul thing | 16:54 |
| rcarrillocruz | Folks, I have seen shrews change about adding nodepool provider 'supported label' in zookeeper. Is there going to be something like 'allowed-labels' on Zuul executors to constrain them to run only on $labels? | 16:56 |
| corvus | the remote tests are brand new. i'm just thrilled we can now add tests like this so we don't break things in the future. :) | 16:57 |
| clarkb | rcarrillocruz: yes, the storyboard bug for that should capture all the details (corvus was quite complete in capturing that) | 16:57 |
| Shrews | corvus: you mean, "so we don't break the SAME things in the future" :) | 16:57 |
| corvus | Shrews: yep -- and with the callback plugins we *have* been breaking the same things over and over :) | 16:58 |
| corvus | never again :) | 16:58 |
| tobiash | Or don't break things on 2/3 of the sec fixes | 16:58 |
| rcarrillocruz | clarkb: cool, will search for SB story later (typing from phone) | 16:58 |
| clarkb | rcarrillocruz: it should be linked to from the ptg etherpad | 16:58 |
| rcarrillocruz | ++ thx | 16:59 |
| *** harlowja has quit IRC | 17:09 | |
| openstackgerrit | Merged openstack-infra/zuul master: Fix no_log bug with result lists https://review.openstack.org/553450 | 17:15 |
| *** jlk` is now known as jlk | 17:31 | |
| *** JasonCL has quit IRC | 17:50 | |
| *** JasonCL has joined #zuul | 18:07 | |
| *** JasonCL has quit IRC | 18:08 | |
| *** JasonCL has joined #zuul | 18:14 | |
| *** jpena is now known as jpena|off | 18:14 | |
| *** JasonCL has quit IRC | 18:18 | |
| *** JasonCL has joined #zuul | 18:18 | |
| *** JasonCL has quit IRC | 18:20 | |
| *** harlowja has joined #zuul | 18:20 | |
| *** JasonCL has joined #zuul | 18:25 | |
| *** JasonCL has quit IRC | 18:26 | |
| *** JasonCL has joined #zuul | 18:30 | |
| *** JasonCL has quit IRC | 18:36 | |
| *** JasonCL has joined #zuul | 18:36 | |
| *** JasonCL has quit IRC | 18:38 | |
| *** JasonCL has joined #zuul | 18:38 | |
| *** JasonCL has quit IRC | 18:39 | |
| *** JasonCL has joined #zuul | 18:42 | |
| pabelanger | clarkb: did you see the fix from tobiash for symlinks in zuul source: https://review.openstack.org/553316/ | 18:49 |
| clarkb | pabelanger: I hadn't | 18:49 |
| pabelanger | that fixed the issue from yesterday | 18:49 |
| *** JasonCL has quit IRC | 18:50 | |
| *** JasonCL has joined #zuul | 18:51 | |
| *** sshnaidm is now known as sshnaidm|afk | 18:58 | |
| *** JasonCL has quit IRC | 19:03 | |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: Make Zuul able to start with a broken config https://review.openstack.org/535511 | 19:13 |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: Make Zuul able to start with a broken config https://review.openstack.org/535511 | 19:13 |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: Make Zuul able to start with a broken config https://review.openstack.org/535511 | 19:17 |
| clarkb | gerrit doesn't seem to have diffed that symlink change very well. Thankfully local diff works great | 19:24 |
| dmsimard | Is the nodepool storyboard an adequate place to create feature requests ? I just haven't seen it used much relative to zuul's storyboard | 19:28 |
| *** ChanServ changes topic to "Discussion of the project gating system Zuul | Docs: http://docs.openstack.org/infra/zuul/ | Source: https://git.openstack.org/cgit/openstack-infra/zuul/ | Roadmap: https://storyboard.openstack.org/#!/board/53 | Channel logs: http://eavesdrop.openstack.org/irclogs/%23zuul/" | 19:28 | |
| -openstackstatus- NOTICE: The regression stemming from one of yesterday's Zuul security fixes has been rectified, and Devstack/Tempest jobs which encountered POST_FAILURE results over the past 24 hours can safely be rechecked now | 19:28 | |
| corvus | dmsimard: yep | 19:28 |
| dmsimard | ok, thanks | 19:29 |
| dmsimard | I'll formalize what I asked about yesterday :D | 19:29 |
| *** JasonCL has joined #zuul | 19:41 | |
| *** JasonCL has quit IRC | 19:42 | |
| *** JasonCL has joined #zuul | 19:44 | |
| *** JasonCL has quit IRC | 19:45 | |
| *** JasonCL has joined #zuul | 19:47 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Expand secret inheritance test https://review.openstack.org/553595 | 20:06 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Don't store references to secret objects from jobs https://review.openstack.org/553596 | 20:06 |
| corvus | tristanC: i can't create a failing test based on the description you gave. can you take a look at that and let me know what you think? | 20:06 |
| *** rlandy is now known as rlandy|afk | 20:09 | |
| corvus | oh, i forgot some git adds | 20:10 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Expand secret inheritance test https://review.openstack.org/553595 | 20:11 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Don't store references to secret objects from jobs https://review.openstack.org/553596 | 20:11 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Perform late validation of secrets https://review.openstack.org/553041 | 20:17 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Perform late validation of nodesets https://review.openstack.org/553088 | 20:17 |
| *** LinuxJedi has joined #zuul | 20:38 | |
| LinuxJedi | SpamapS: I hear you had a call with Max (MariaDB) today :) | 20:39 |
| *** JasonCL has quit IRC | 20:41 | |
| *** JasonCL has joined #zuul | 20:41 | |
| *** JasonCL has quit IRC | 20:46 | |
| *** hashar has joined #zuul | 20:54 | |
| corvus | LinuxJedi: hello there! sounds cool :) | 20:55 |
| LinuxJedi | corvus: certainly is. I'm definitely going to need some help from you all in the coming weeks if that is OK. I've been asked to lead the charge on this at our end :) | 20:57 |
| corvus | LinuxJedi: i'm happy to help! | 20:58 |
| SpamapS | LinuxJedi: I did. It went well! | 20:59 |
| SpamapS | been so long since I played with buildbot. | 20:59 |
| LinuxJedi | I've never played with Buildbot, I don't get much time do deal with CI personally (until I'm suddenly put in charge of educating about it) | 21:00 |
| LinuxJedi | the ironic thing is my particular team has the worst CI story in the company | 21:01 |
| LinuxJedi | and I finally have an opportunity to fix things | 21:01 |
| LinuxJedi | if any of you happen to be near Helsinki 4-6 April and want to talk about Zuul we will be having a meeting there | 21:03 |
| *** JasonCL has joined #zuul | 21:06 | |
| *** dkranz has quit IRC | 21:06 | |
| *** JasonCL has quit IRC | 21:07 | |
| * SpamapS so wants to make an excuse to be in Helsinki.. well.. ever. | 21:07 | |
| LinuxJedi | lol! I've drunk so much black vodka in Helsinki in the last 6 months (I'm there almost every month lately) | 21:08 |
| *** myoung|rover is now known as myoung|afk | 21:12 | |
| LinuxJedi | also, if any of you are at Cloud Expo Europe next week I'm at the Big Data conference running in parallel to it in the same building | 21:20 |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: Make Zuul able to start with a broken config https://review.openstack.org/535511 | 21:32 |
| *** JasonCL has joined #zuul | 21:35 | |
| *** hashar has quit IRC | 21:39 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: WIP: late bind pipelines https://review.openstack.org/553618 | 21:41 |
| *** eandersson has joined #zuul | 21:43 | |
| jimi|ansible | Shrews: just saw your ping here. I'm generally ok with deepcopy, as long as it doesn't impact performance (which it does when, for instance, doing copies of heavily nested task objects) | 21:43 |
| eandersson | AJaeger, I assume it's supposed to look something like this? https://review.openstack.org/#/c/553619/ | 21:45 |
| *** JasonCL has quit IRC | 21:50 | |
| *** JasonCL has joined #zuul | 21:50 | |
| *** JasonCL has quit IRC | 21:51 | |
| *** JasonCL has joined #zuul | 21:58 | |
| openstackgerrit | David Moreau Simard proposed openstack-infra/zuul-jobs master: Refactor "emit-ara-html" into a new role: "ara-report" https://review.openstack.org/513874 | 21:59 |
| *** JasonCL has quit IRC | 21:59 | |
| *** JasonCL has joined #zuul | 22:00 | |
| *** rlandy|afk is now known as rlandy | 22:02 | |
| *** JasonCL has quit IRC | 22:04 | |
| *** JasonCL has joined #zuul | 22:13 | |
| *** JasonCL has quit IRC | 22:18 | |
| *** JasonCL has joined #zuul | 22:30 | |
| *** JasonCL has quit IRC | 22:39 | |
| *** JasonCL has joined #zuul | 23:05 | |
| *** JasonCL has quit IRC | 23:11 | |
| clarkb | LinuxJedi: I learned a long time ago that you have to be careful drinking that stuff | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!