Thursday, 2023-09-28

-@gerrit:opendev.org- Daniel Blixt proposed: [zuul/zuul-jobs] 896640: Make prepare-workspace-git work with unique scheme https://review.opendev.org/c/zuul/zuul-jobs/+/89664006:31
-@gerrit:opendev.org- Daniel Blixt proposed: [zuul/zuul-jobs] 896640: Make prepare-workspace-git work with unique scheme https://review.opendev.org/c/zuul/zuul-jobs/+/89664006:34
-@gerrit:opendev.org- daniel.pawlik https://matrix.to/#/@dpawlik:matrix.org proposed: [zuul/zuul] 889314: Change start_time and end_time field type to integer; add new fields https://review.opendev.org/c/zuul/zuul/+/88931406:38
-@gerrit:opendev.org- Daniel Blixt proposed: [zuul/zuul-jobs] 896640: Make prepare-workspace-git work with unique scheme https://review.opendev.org/c/zuul/zuul-jobs/+/89664006:39
-@gerrit:opendev.org- daniel.pawlik https://matrix.to/#/@dpawlik:matrix.org proposed: [zuul/zuul-jobs] 896646: Add workaround to accept traffic in/out the bridge; add dns test https://review.opendev.org/c/zuul/zuul-jobs/+/89664606:44
-@gerrit:opendev.org- Daniel Blixt proposed: [zuul/zuul-jobs] 896640: Make prepare-workspace-git work with unique scheme https://review.opendev.org/c/zuul/zuul-jobs/+/89664006:45
-@gerrit:opendev.org- Daniel Blixt proposed: [zuul/zuul-jobs] 896640: Make prepare-workspace-git work with unique scheme https://review.opendev.org/c/zuul/zuul-jobs/+/89664006:48
-@gerrit:opendev.org- Daniel Blixt proposed: [zuul/zuul-jobs] 896640: Make prepare-workspace-git work with unique scheme https://review.opendev.org/c/zuul/zuul-jobs/+/89664007:34
-@gerrit:opendev.org- Daniel Blixt proposed: [zuul/zuul-jobs] 896640: Make prepare-workspace-git work with unique scheme https://review.opendev.org/c/zuul/zuul-jobs/+/89664007:39
-@gerrit:opendev.org- Daniel Blixt proposed: [zuul/zuul-jobs] 896640: Make prepare-workspace-git work with unique scheme https://review.opendev.org/c/zuul/zuul-jobs/+/89664007:39
-@gerrit:opendev.org- Daniel Blixt proposed: [zuul/zuul-jobs] 896640: Make prepare-workspace-git work with unique scheme https://review.opendev.org/c/zuul/zuul-jobs/+/89664007:55
-@gerrit:opendev.org- Daniel Blixt proposed: [zuul/zuul-jobs] 896640: Make prepare-workspace-git work with unique scheme https://review.opendev.org/c/zuul/zuul-jobs/+/89664008:07
-@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 896501: Store frozen jobs using UUID instead of name https://review.opendev.org/c/zuul/zuul/+/89650108:09
-@gerrit:opendev.org- daniel.pawlik https://matrix.to/#/@dpawlik:matrix.org proposed: [zuul/zuul-jobs] 896646: WIP Add workaround to accept traffic in/out the bridge; add dns test https://review.opendev.org/c/zuul/zuul-jobs/+/89664608:27
-@gerrit:opendev.org- daniel.pawlik https://matrix.to/#/@dpawlik:matrix.org proposed: [zuul/zuul-jobs] 896646: WIP Add workaround to accept traffic in/out the bridge; add dns test https://review.opendev.org/c/zuul/zuul-jobs/+/89664608:59
-@gerrit:opendev.org- Daniel Blixt proposed wip: [zuul/zuul-jobs] 896640: Make prepare-workspace-git work with unique scheme https://review.opendev.org/c/zuul/zuul-jobs/+/89664009:01
-@gerrit:opendev.org- daniel.pawlik https://matrix.to/#/@dpawlik:matrix.org proposed: [zuul/zuul-jobs] 896646: WIP Add workaround to accept traffic in/out the bridge; add dns test https://review.opendev.org/c/zuul/zuul-jobs/+/89664609:10
-@gerrit:opendev.org- Daniel Blixt proposed wip: [zuul/zuul-jobs] 896640: Make prepare-workspace-git work with unique scheme https://review.opendev.org/c/zuul/zuul-jobs/+/89664009:10
-@gerrit:opendev.org- Daniel Blixt proposed wip: [zuul/zuul-jobs] 896640: Make prepare-workspace-git work with unique scheme https://review.opendev.org/c/zuul/zuul-jobs/+/89664009:28
-@gerrit:opendev.org- Daniel Blixt proposed wip: [zuul/zuul-jobs] 896640: Make prepare-workspace-git work with unique scheme https://review.opendev.org/c/zuul/zuul-jobs/+/89664009:38
-@gerrit:opendev.org- Daniel Blixt marked as active: [zuul/zuul-jobs] 896640: Make prepare-workspace-git work with unique scheme https://review.opendev.org/c/zuul/zuul-jobs/+/89664009:39
@phoenikzz:matrix.orgRegarding unique workspace scheme:09:49
I noticed that for debian-buster, ubuntu-bionic and ubuntu-focal, the default shell executable uses a PATH search that interprets % as a flag starter (see Path search at e.g. https://www.unix.com/man-page/minix/1/ash/). In the use case of unique workspace scheme and a project with a / in it, and if that project's zuul.project(s{}).src_dir is added to path, those shells will not be able to find commands from that path. This could for example be an activated virtual environment, which was what happend in the fetch-subunit-output test.
The work-around is to use e.g. /bin/bash as the shell executable, but this could be difficult to know?
-@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 896501: Store frozen jobs using UUID instead of name https://review.opendev.org/c/zuul/zuul/+/89650109:58
-@gerrit:opendev.org- daniel.pawlik https://matrix.to/#/@dpawlik:matrix.org proposed: [zuul/zuul-jobs] 896646: WIP Add workaround to accept traffic in/out the bridge; add dns test https://review.opendev.org/c/zuul/zuul-jobs/+/89664610:22
-@gerrit:opendev.org- Daniel Blixt proposed: [zuul/zuul-jobs] 764152: Allow mirror push to delete current branch https://review.opendev.org/c/zuul/zuul-jobs/+/76415211:39
@clarkb:matrix.org> <@phoenikzz:matrix.org> Regarding unique workspace scheme:17:41
> I noticed that for debian-buster, ubuntu-bionic and ubuntu-focal, the default shell executable uses a PATH search that interprets % as a flag starter (see Path search at e.g. https://www.unix.com/man-page/minix/1/ash/). In the use case of unique workspace scheme and a project with a / in it, and if that project's zuul.project(s{}).src_dir is added to path, those shells will not be able to find commands from that path. This could for example be an activated virtual environment, which was what happend in the fetch-subunit-output test.
>
> The work-around is to use e.g. /bin/bash as the shell executable, but this could be difficult to know?
I don't know that I have a good solution for that off the top of my head. At the very least we could document this somewhere? Maybe iwth a pointer to the ansible shell module docs executable override so that bash can be explicitly chosen?

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!