| *** adam_g has quit IRC | 00:14 | |
| *** adam_g has joined #zuul | 00:14 | |
| *** adam_g has quit IRC | 00:23 | |
| *** wuchunyang has joined #zuul | 00:31 | |
| *** sgw1 has quit IRC | 00:33 | |
| *** wuchunyang has quit IRC | 00:35 | |
| *** Goneri has quit IRC | 00:41 | |
| *** pots has quit IRC | 02:03 | |
| *** pots has joined #zuul | 02:03 | |
| *** mmedvede has quit IRC | 02:04 | |
| *** cloudnull8 has joined #zuul | 02:04 | |
| *** cloudnull has quit IRC | 02:04 | |
| *** cloudnull8 is now known as cloudnull | 02:04 | |
| *** mgoddard has quit IRC | 02:05 | |
| *** weshay|ruck has quit IRC | 02:05 | |
| *** flaper87 has quit IRC | 02:05 | |
| *** weshay has joined #zuul | 02:06 | |
| *** kgz has quit IRC | 02:06 | |
| *** mmedvede has joined #zuul | 02:12 | |
| *** mgoddard has joined #zuul | 02:13 | |
| *** flaper87 has joined #zuul | 02:19 | |
| *** kgz has joined #zuul | 02:23 | |
| *** rlandy|bbl is now known as rlandy | 02:38 | |
| *** rlandy has quit IRC | 02:38 | |
| *** rfolco has quit IRC | 02:54 | |
| *** vishalmanchanda has joined #zuul | 03:30 | |
| *** bhavikdbavishi has joined #zuul | 03:56 | |
| ianw | ERROR: Package 'openstacksdk' requires a different Python: 3.5.2 not in '>=3.6' ... did openstacksdk drop python3.5 support? | 03:57 |
|---|---|---|
| clarkb | pip should handle that with the index requires python metadata | 03:59 |
| clarkb | but we've been seeing some unhappy pypi cdn nodes and when that happens it seems we dont always get that metadata? | 03:59 |
| ianw | that must be it, this is a zuul py35 tox job | 03:59 |
| ianw | https://zuul.opendev.org/t/zuul/build/156f85c8a62a48a69a0ed4d7926c4912 | 03:59 |
| ianw | nothing has changed afaics | 03:59 |
| clarkb | we can use env markers to specify it in zuul too if we want | 04:00 |
| clarkb | that will be more reliable | 04:00 |
| *** bhavikdbavishi has quit IRC | 04:01 | |
| ianw | i think it's probably transitive via nodepool | 04:01 |
| *** bhavikdbavishi has joined #zuul | 04:18 | |
| *** saneax has joined #zuul | 04:18 | |
| *** raukadah is now known as chkumar|rover | 04:25 | |
| *** evrardjp has quit IRC | 04:33 | |
| *** evrardjp has joined #zuul | 04:33 | |
| *** yolanda has quit IRC | 04:41 | |
| *** mach1na has joined #zuul | 05:13 | |
| *** cloudnull is now known as kecarter | 05:19 | |
| *** kecarter is now known as cloudnull | 05:19 | |
| *** mach1na has quit IRC | 05:23 | |
| *** mach1na has joined #zuul | 05:23 | |
| *** ianychoi_ has joined #zuul | 05:25 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [wip] json edit https://review.opendev.org/746834 | 05:27 |
| *** mach1na has quit IRC | 05:28 | |
| *** ianychoi has quit IRC | 05:28 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [wip] json edit https://review.opendev.org/746834 | 05:30 |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [wip] json edit https://review.opendev.org/746834 | 05:40 |
| *** mach1na has joined #zuul | 05:55 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [wip] json edit https://review.opendev.org/746834 | 05:56 |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [wip] json edit https://review.opendev.org/746834 | 06:05 |
| *** bhavikdbavishi has quit IRC | 06:09 | |
| *** dmellado has quit IRC | 06:11 | |
| *** mach1na has quit IRC | 06:28 | |
| *** mach1na has joined #zuul | 06:28 | |
| *** mach1na has quit IRC | 06:39 | |
| *** dmellado has joined #zuul | 06:41 | |
| *** yolanda has joined #zuul | 06:50 | |
| *** mach1na has joined #zuul | 07:05 | |
| *** dmellado has quit IRC | 07:07 | |
| *** bhavikdbavishi has joined #zuul | 07:15 | |
| *** piotrowskim has joined #zuul | 07:26 | |
| *** mach1na has quit IRC | 07:51 | |
| *** mach1na has joined #zuul | 07:52 | |
| *** dmellado has joined #zuul | 07:53 | |
| *** mach1na has quit IRC | 07:54 | |
| *** mach1na has joined #zuul | 07:54 | |
| *** jpena|off is now known as jpena | 07:58 | |
| openstackgerrit | Merged zuul/zuul-jobs master: terraform: Add parameter for plan file https://review.opendev.org/746657 | 08:21 |
| *** tosky has joined #zuul | 08:28 | |
| *** nils has joined #zuul | 08:43 | |
| openstackgerrit | Benoît Bayszczak proposed zuul/nodepool master: [provider][aws] now performs a multi-describe at once https://review.opendev.org/740328 | 08:59 |
| *** bhavikdbavishi has quit IRC | 09:27 | |
| *** bhavikdbavishi has joined #zuul | 09:29 | |
| *** bhavikdbavishi has quit IRC | 09:52 | |
| openstackgerrit | Benoît Bayszczak proposed zuul/nodepool master: [provider][aws] now performs a multi-describe at once https://review.opendev.org/740328 | 09:52 |
| *** korkakak has joined #zuul | 09:52 | |
| *** bhavikdbavishi has joined #zuul | 09:54 | |
| korkakak | Hi! I was trying to execute an ansible playbook that used the lookup's pipe. Apparently this is forbidden in Zuul for the last 3 years according to https://opendev.org/zuul/zuul/commit/d08b4ce374150c1b25017e11ab91256c4ec62e80 | 09:54 |
| korkakak | my question is, how is pipe more dangerous compared to using the `shell` or the `console` ? | 09:55 |
| korkakak | by `pipe` in the previous msg I denote the ansible's lookup pipe plugin | 09:56 |
| tobiash | korkakak: local shell tasks are forbidden as well | 10:15 |
| korkakak | @tobiash, how about the console which also has similar properties? | 10:47 |
| tobiash | korkakak: do you mean command or really console? | 10:48 |
| korkakak | you are right, I meant command | 10:48 |
| tobiash | command is forbidden as well locally ;) | 10:50 |
| korkakak | hmmm, I can see that I am allowed to execute command(s) if there are in a role | 10:52 |
| korkakak | and the job is not trusted | 10:53 |
| avass | korkakak: any arbitrary code execution is forbidden on the executor unless it's done in a trusted context ) | 11:01 |
| avass | korkakak: and so are most lookups, but I believe some of them are whitelisted | 11:01 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: REST API: improve tenant scoping of autohold, authorizations https://review.opendev.org/728118 | 11:03 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: CLI: add autohold-info, autohold-delete via REST https://review.opendev.org/728410 | 11:03 |
| *** mach1na has quit IRC | 11:13 | |
| *** mach1na has joined #zuul | 11:14 | |
| *** mach1na has joined #zuul | 11:14 | |
| *** mach1na has quit IRC | 11:26 | |
| *** mach1na has joined #zuul | 11:26 | |
| *** mach1na has quit IRC | 11:31 | |
| korkakak | avass was kind enough to educate me about ansible, how this is deployed and executed by Zuul, thanks @avass :) | 11:31 |
| *** jpena is now known as jpena|lunch | 11:31 | |
| *** _erlon_ has joined #zuul | 11:44 | |
| *** hashar has joined #zuul | 12:01 | |
| *** rlandy has joined #zuul | 12:02 | |
| *** mach1na has joined #zuul | 12:02 | |
| *** rfolco has joined #zuul | 12:03 | |
| *** bhavikdbavishi has quit IRC | 12:06 | |
| *** sshnaidm is now known as sshnaidm|afk | 12:26 | |
| avass | korkakak: :) | 12:27 |
| *** jpena|lunch is now known as jpena | 12:37 | |
| *** weshay is now known as weshay|interview | 12:47 | |
| *** Goneri has joined #zuul | 12:47 | |
| *** sgw1 has joined #zuul | 12:50 | |
| *** weshay|interview is now known as weshay | 13:10 | |
| openstackgerrit | Benoît Bayszczak proposed zuul/nodepool master: [provider][aws] now performs a multi-describe at once https://review.opendev.org/740328 | 13:12 |
| openstackgerrit | Benoît Bayszczak proposed zuul/nodepool master: [provider][aws] now performs a multi-describe at once https://review.opendev.org/740328 | 13:18 |
| *** weshay is now known as weshay|ruck | 13:21 | |
| openstackgerrit | Benoît Bayszczak proposed zuul/nodepool master: [provider][aws] use one API call to create tags https://review.opendev.org/746921 | 13:22 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: REST API: improve tenant scoping of autohold, authorizations https://review.opendev.org/728118 | 13:40 |
| *** hashar has quit IRC | 14:06 | |
| openstackgerrit | Benoît Bayszczak proposed zuul/nodepool master: [provider][aws] fix exception when instance not found during cleanup https://review.opendev.org/746932 | 14:07 |
| *** korkakak has quit IRC | 14:13 | |
| *** saneax is now known as saneax_ | 14:26 | |
| *** hashar has joined #zuul | 14:33 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: CLI: add autohold-info, autohold-delete via REST https://review.opendev.org/728410 | 14:35 |
| *** sshnaidm|afk is now known as sshnaidm | 14:42 | |
| openstackgerrit | Benoît Bayszczak proposed zuul/nodepool master: [provider][aws] now performs a multi-describe at once https://review.opendev.org/740328 | 15:04 |
| openstackgerrit | Benoît Bayszczak proposed zuul/nodepool master: [provider][aws] use one API call to create tags https://review.opendev.org/746921 | 15:04 |
| *** chkumar|rover is now known as raukadah | 15:14 | |
| fungi | clarkb: ianw: catching up, are you sure it's not that we're using a too-old pip that doesn't know to query that field from the pypi metadata when selecting package versions? | 15:20 |
| clarkb | fungi: the ensure-pip job is supposed to address that and tosky is reporting similar for unrelated jobs in #opendev | 15:21 |
| fungi | ensure-pip installs from distro packages by default though, right? so if this is on ubuntu-xenial (to get a default python3.5) then it may also use a fairly old pip? | 15:21 |
| clarkb | fungi: on xenial it installs a backported bionic pip 9 from our ppa | 15:23 |
| clarkb | primarily to address the pip 8 doesn't fallback if an index doesn't have a package behavior, but that should also ensure we ave a new enough pip to handle python requires | 15:23 |
| tosky | fungi: the jobs where I've seen pypi failures are not consistently failing; a recheck usually pass | 15:25 |
| clarkb | tosky: and at least the one example of yours I checked ran on bionic | 15:31 |
| clarkb | which I think rules out any xenial related problems | 15:31 |
| clarkb | ah though tosky's example is the package missing from the index entirely | 15:32 |
| clarkb | not python3.5 or python2.7 installing package for newer python | 15:32 |
| clarkb | but I think both are related to CDN is unhappy | 15:32 |
| tosky | no, I think it's just rax's mirror missing something | 15:33 |
| clarkb | well its a proxy, the problem is in pypi's cdn | 15:33 |
| tosky | oh | 15:34 |
| tosky | so I guess there is nothing to do apart from maybe letting them know they have a problem | 15:35 |
| fungi | clarkb: tosky: thanks, so it's random-ish then, got it | 15:36 |
| *** mach1na has quit IRC | 15:37 | |
| *** jpena is now known as jpena|off | 16:07 | |
| *** nils has quit IRC | 16:10 | |
| *** hashar has quit IRC | 16:18 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: REST API: improve tenant scoping of autohold, authorizations https://review.opendev.org/728118 | 16:25 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: CLI: add autohold-info, autohold-delete via REST https://review.opendev.org/728410 | 16:27 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: REST API: add promote endpoint https://review.opendev.org/728489 | 16:32 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: REST API: add promote endpoint https://review.opendev.org/728489 | 16:36 |
| openstackgerrit | Clark Boylan proposed zuul/zuul-jobs master: Generate build ssh keys in PEM format https://review.opendev.org/746994 | 16:38 |
| *** hashar has joined #zuul | 16:50 | |
| clarkb | hrm it doens't look like we do much testing of ^ problably because it is privileged and in base? | 17:07 |
| clarkb | do ya'll think I should make a copy of that role, add it to opendev's base-test and test it that way? | 17:07 |
| corvus | clarkb: i thought there was a test job for that | 17:08 |
| corvus | oh, it's the "add-sshkey" role i was thinking of; that's different | 17:09 |
| clarkb | older changes run the same set of jobs too https://review.opendev.org/#/c/740350/ | 17:10 |
| tobiash | mhu: sorry, it looks like I've overlooked your promote endpoint change. In the meantime https://review.opendev.org/742848 has merged | 17:13 |
| corvus | clarkb: we might be able to construct a test job for that role; it'll take a little bit of work, but i bet we could start an ssh-agent on a remote node, add a key to it like zuul does, then run that role on it. | 17:15 |
| tobiash | clarkb: it would be great to get a second review on https://review.opendev.org/744679 in order to not block zuul-web when enqueuing/dequeuing changes during event backlog | 17:23 |
| clarkb | tobiash: noted. I'm about to get a bike ride in but will put that at the top of the list when I get back | 17:25 |
| tobiash | clarkb: of course a bike ride is more important, enjoy :) | 17:26 |
| mhu | tobiash, ah, that makes more sense! I was scratching my head on why the rebase was failing | 17:35 |
| mhu | tobiash, I think my change is still valid, it adds some test coverage at least | 17:36 |
| tobiash | mhu: yeah, test_options makes sense, the test_promote is duplicated then I think | 17:39 |
| mhu | right - I was interrupted during the rebase, I'm just seeing that | 17:39 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: REST API: improve promote endpoint testing https://review.opendev.org/728489 | 17:48 |
| mhu | I'm in favor of keeping the testing of the web API in addition to the CLI test, because we might split the CLI between admin stuff and user stuff | 17:49 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Builds: add "held" attribute https://review.opendev.org/728994 | 17:50 |
| *** bhavikdbavishi has joined #zuul | 17:51 | |
| mhu | felixedel, do you have an ongoing change on the TableFilters component related to the PF4 migration? I'm asking before I rebase this one: https://review.opendev.org/#/c/729265/ | 17:52 |
| *** hashar is now known as hasharWineMusic | 17:55 | |
| *** vishalmanchanda has quit IRC | 18:49 | |
| AJaeger | tristanC: clarkb suggested you review https://review.opendev.org/#/c/746423/ , could you do so, please? | 19:20 |
| *** bhavikdbavishi has quit IRC | 19:28 | |
| *** saneax_ has quit IRC | 19:32 | |
| *** y2kenny has joined #zuul | 20:01 | |
| y2kenny | Are "Zookeeper suspended. Waiting" message from Nodepool normal? | 20:03 |
| y2kenny | Nodepool's log | 20:03 |
| clarkb | y2kenny: I don't think so? is zookeeper running | 20:03 |
| y2kenny | clarkb: it is... and another nodepool is working... ok I think I know where to dig, thanks. | 20:05 |
| mhu | AJaeger, I think tristanC is on vacation at the moment | 20:16 |
| y2kenny | clarkb: does nodepool retry on lost zk connection? I am wondering if I broke the retry mechanism because my driver didn't keep a reference to the zk_conn | 20:18 |
| y2kenny | clarkb: I noticed the addition of "self._zk = zk_conn" while reviewing your other patch | 20:20 |
| clarkb | y2kenny: yes it should retry | 20:22 |
| corvus | clarkb: good comment from zbr on 746994. if you haven't started on anything similar, i'll take a quick stab at making a test job for that | 20:47 |
| corvus | (ssh pem) | 20:47 |
| clarkb | corvus: I havne't. I'm just about ot dig into tobiash's change now | 20:48 |
| clarkb | corvus: that would be appreciated (the test job) | 20:48 |
| corvus | cool, i'll do that | 20:48 |
| *** dustinc has quit IRC | 20:49 | |
| *** zenkuro has joined #zuul | 20:51 | |
| corvus | clarkb: :( i think there are some critical "delegate-to: localhost" shell tasks there so we won't be able to run it speculatively, sorry | 20:51 |
| corvus | clarkb: i think the base-test dance is the way to go on that one | 20:51 |
| corvus | (later, if we lift the localhost shell restriction, we may be able to test it) | 20:52 |
| clarkb | ok, I'll work on an update that puts it in a test role thta we can merge, as well as a base-test update | 20:53 |
| clarkb | corvus: tobiash for https://review.opendev.org/#/c/744679/3 we aren't worried about races between the workers right? basically all the pipeline write functions are in slow. And all the hold functions are in not slow so we shouldn't have any interacting writes in badways between threads? | 20:55 |
| corvus | clarkb: right -- i think if the fast ones are read-only we should be okay | 20:55 |
| zenkuro | hi, is priblem related to ara-report: error 30 read-only file system known? | 20:58 |
| corvus | zenkuro: is this a problem with zuul.opendev.org? | 20:58 |
| clarkb | corvus: the autoholds are also in the fast list, but those shouldn't interact with pipelines right? | 20:59 |
| zenkuro | corvus: do not think so. more like s local deployment issue that happens after 2-3 weeks of work | 20:59 |
| corvus | zenkuro: ok. that doesn't sound familiar. https://review.opendev.org/742971 is the most recent significant change to ara-report | 21:01 |
| openstackgerrit | Clark Boylan proposed zuul/zuul-jobs master: Generate build ssh keys in PEM format https://review.opendev.org/746994 | 21:02 |
| openstackgerrit | Clark Boylan proposed zuul/zuul-jobs master: Add test-add-build-sshkey role https://review.opendev.org/747035 | 21:02 |
| corvus | zenkuro: if you previously had a value for ara_report_path set, that change may have broken your setup and could result in an error like that | 21:02 |
| clarkb | corvus: remote: https://review.opendev.org/747036 Use test-add-build-sshkey in base-test | 21:03 |
| *** rfolco has quit IRC | 21:05 | |
| corvus | zenkuro: here's a little more info about that: http://lists.zuul-ci.org/pipermail/zuul-announce/2020-July/000080.html | 21:05 |
| *** smyers has quit IRC | 21:12 | |
| *** smyers_ has joined #zuul | 21:12 | |
| *** smyers_ is now known as smyers | 21:12 | |
| *** dmsimard7 has joined #zuul | 21:14 | |
| *** dmsimard has quit IRC | 21:15 | |
| *** dmsimard7 is now known as dmsimard | 21:15 | |
| clarkb | ok think I convinced myself the split rpc worker setup should be fine (no interaction between writers in different threads) | 21:16 |
| clarkb | I've approved the change | 21:16 |
| *** zenkuro has quit IRC | 21:27 | |
| *** zenkuro has joined #zuul | 21:37 | |
| zenkuro | corvus: thanks! looks like this is my case. I am using software factory and disabling upstream playbooks helped | 21:37 |
| corvus | zenkuro: do you think you can fix it by adjusting the value of that variable? | 21:38 |
| corvus | mhu: ^ fyi | 21:39 |
| *** tosky has quit IRC | 21:47 | |
| zenkuro | corvus: hm, may be, I will think about it with a fresh head | 21:50 |
| *** hasharWineMusic is now known as hahsar | 21:59 | |
| *** hahsar is now known as hashar | 21:59 | |
| *** zenkuro has quit IRC | 22:07 | |
| clarkb | corvus: I +2'd https://review.opendev.org/#/c/744801/ but left a couple of thoughts. Didn't approve and will let you decide if those are worth addressing before approving | 22:15 |
| *** rlandy is now known as rlandy|bbl | 22:28 | |
| corvus | clarkb: let me see if i can remember the answers to those q's :) | 22:29 |
| *** hashar has quit IRC | 22:34 | |
| corvus | clarkb: replied | 22:37 |
| clarkb | corvus: sound sgood to me if you want to approve it. Or should I? | 22:39 |
| corvus | done | 22:40 |
| clarkb | corvus: https://review.opendev.org/#/c/740328 that is one you may be interested in. I think that may be better served by the task manager rate limiting | 22:56 |
| corvus | clarkb: does it seem like that driver could use the simple driver interface, or at least the taskmanager? | 23:14 |
| clarkb | I think at least the taskmanager | 23:14 |
| corvus | i sort of designed the simple driver for both aws and gce | 23:15 |
| clarkb | not sure if it can be simplified to the simple driver | 23:15 |
| openstackgerrit | Ian Wienand proposed zuul/zuul master: Doc updates for Github interaction https://review.opendev.org/746815 | 23:15 |
| clarkb | (but maybe simple driver too) | 23:15 |
| openstackgerrit | Ian Wienand proposed zuul/zuul master: Doc updates for Github interaction https://review.opendev.org/746815 | 23:15 |
| corvus | clarkb: oh, heh, i just saw your cover comment (i missed it earlier and only saw the inline comments before) | 23:17 |
| *** Goneri has quit IRC | 23:27 | |
| openstackgerrit | Merged zuul/nodepool master: Finish adding GCE quota support https://review.opendev.org/744801 | 23:29 |
| corvus | tobiash: can you see comment on https://review.opendev.org/741157 ? | 23:37 |
| *** piotrowskim has quit IRC | 23:39 | |
| openstackgerrit | Merged zuul/zuul master: github connection: remove unused depends_on_re https://review.opendev.org/746437 | 23:45 |
| openstackgerrit | Merged zuul/zuul master: Fix Multi Node Builds with k8s Build Nodes https://review.opendev.org/746515 | 23:52 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!