opendevreview | Ian Wienand proposed zuul/zuul-jobs master: Add introduction to the container jobs documentation https://review.opendev.org/c/zuul/zuul-jobs/+/798969 | 01:59 |
---|---|---|
opendevreview | Simon Westphahl proposed zuul/zuul master: Make reporting asynchronous https://review.opendev.org/c/zuul/zuul/+/691253 | 07:09 |
opendevreview | Simon Westphahl proposed zuul/zuul master: Make reporting asynchronous https://review.opendev.org/c/zuul/zuul/+/691253 | 07:13 |
avass[m] | boha: either on the job level or where the role is included | 07:22 |
*** bhagyashris_ is now known as bhagyashris|ruck | 07:40 | |
*** rpittau|afk is now known as rpittau | 07:45 | |
*** sshnaidm|afk is now known as sshnaidm | 08:35 | |
*** jpena|off is now known as jpena | 08:35 | |
*** dmellado_ is now known as dmellado | 08:43 | |
opendevreview | Simon Westphahl proposed zuul/zuul master: Refactor pipeline processing in run handler https://review.opendev.org/c/zuul/zuul/+/795985 | 08:45 |
*** iurygregory is now known as iury|holiday | 09:18 | |
swest | corvus: there seems to be a race condition with the executor API: https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_a7c/795985/9/check/zuul-tox-py36/a7c1eb9/testr_results.html | 10:06 |
swest | corvus: might be something we are just hitting in the test | 10:07 |
*** bhagyashris_ is now known as bhagyashris|ruck | 10:08 | |
swest | corvus: I think this is similar to the race condition we observed with the tree cache. the watched node is deleted while a slow "changed" event tries to retrieve the data of the now gone znode. | 10:12 |
swest | maybe something we just have to account for in the _watchBuildState() callback | 10:13 |
*** jpena is now known as jpena|lunch | 11:33 | |
opendevreview | Simon Westphahl proposed zuul/zuul master: Don't process build state watches w/o data https://review.opendev.org/c/zuul/zuul/+/799460 | 11:49 |
swest | corvus: ^ not sure if we should stop the data watch in this case | 11:49 |
opendevreview | Simon Westphahl proposed zuul/zuul master: Move tenant validation to separate method https://review.opendev.org/c/zuul/zuul/+/799463 | 12:07 |
opendevreview | Simon Westphahl proposed zuul/zuul master: Combine full and smart reconfiguration events https://review.opendev.org/c/zuul/zuul/+/799464 | 12:07 |
opendevreview | Felix Edel proposed zuul/zuul master: Move parent provider determination to pipeline manager https://review.opendev.org/c/zuul/zuul/+/797631 | 12:28 |
*** jpena|lunch is now known as jpena | 12:28 | |
*** jpena is now known as jpena|away | 12:36 | |
opendevreview | Benjamin Schanzel proposed zuul/zuul master: Add tenant name on NodeRequests for Nodepool https://review.opendev.org/c/zuul/zuul/+/788680 | 12:45 |
opendevreview | Benjamin Schanzel proposed zuul/zuul master: Add tenant name on NodeRequests for Nodepool https://review.opendev.org/c/zuul/zuul/+/788680 | 12:50 |
opendevreview | Felix Edel proposed zuul/zuul master: Move parent provider determination to pipeline manager https://review.opendev.org/c/zuul/zuul/+/797631 | 13:20 |
*** jpena|away is now known as jpena | 14:29 | |
*** marios is now known as marios|out | 15:56 | |
*** rpittau is now known as rpittau|afk | 16:20 | |
*** jpena is now known as jpena|off | 16:33 | |
boha[m] | <avass[m] "boha: either on the job level or"> Thanks :). That got me past the add-build-sshkey. Is the prepare_workspace-role supported on windows? I get an unexpected failure when it tries to start zuul_console daemon. Any chance of finding a complete base setup for a windows node publicly? | 19:18 |
avass[m] | boha: I think there's a change for one somewhere, let me checl | 19:19 |
avass[m] | boha: this should do it: https://review.opendev.org/c/zuul/zuul-jobs/+/740005 | 19:21 |
boha[m] | nice, so it could replace the prepare_workspace in the future? the zuul_console-part in prepare_workspace - is that for live streaming to the webui or something else? | 19:28 |
tobiash[m] | boha: zuul-console daemon is for live log streaming and not supported on windows | 19:30 |
tobiash[m] | you can just skip that for windows | 19:30 |
boha[m] | ok, thanks 🙂 | 19:31 |
avass[m] | not supported on windows yet, maybe someone will make that happen some day :) | 20:13 |
boha[m] | how is the status on windows and zuul in general? is it used in production? | 20:15 |
boha[m] | <avass[m] "boha: this should do it: https:/"> I got the winrm-part working with you synchronize_repos 🙂. SSH did not find git, but that should be my fault ;-) | 20:17 |
avass[m] | Yeah we use windows extensively at Volvo | 20:17 |
avass[m] | boha: you may need to reboot or reload the environment somehow if git is in path and then restart sshd | 20:19 |
boha[m] | Cool 🙂, but the zuul-jobs-repo are mainly concerned with linux for now? | 20:19 |
avass[m] | Yeah, opendev doesn't really have any windows machines to run tests on either | 20:20 |
corvus | we accept windows-related roles in zuul/zuul-jobs | 20:21 |
corvus | we would also accept a test platform with windows machines to run tests on them | 20:22 |
corvus | but historically there's a lot of non-windows contributions, so that's what the bulk of the roles are aiming for. of course, linux subsystem on windows levels the playing field a lot. | 20:24 |
boha[m] | <avass[m] "boha: you may need to reboot or "> it always helps with a restart 😀. Now it runs using ssh also. Last step failing now is the remove-build-sshkey. | 20:32 |
boha[m] | looks like it cannot find the key... | 20:33 |
avass[m] | boha: i think we may have a patched version of remove-build-sshkey, i can check tomorrow | 20:34 |
boha[m] | sounds good, much appreciated :). Thanks for all the help. | 20:36 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!