| clarkb | 2021-02-02 00:04:09,065 INFO cherrypy.access.139663382723600: ::ffff:172.17.0.1 - - [02/Feb/2021:00:04:09] "GET /v2/upstream/image/blobs/sha256:358792b44b713f905789d54d9733e75c702792522c07f812bf873b5ffc5efe77?ns=docker.io HTTP/1.1" 404 735 "" "containerd/1.4.0+unknown" from https://zuul.opendev.org/t/zuul/build/02a0af1e9c23470ab93eac0160354449/log/builder/docker/buildset_registry.txt#183 | 00:26 |
|---|---|---|
| clarkb | I'll get a change up shortly to handle the ns parameter there | 00:26 |
| openstackgerrit | Merged zuul/zuul master: gitlab: handle protected branches https://review.opendev.org/c/zuul/zuul/+/756900 | 00:30 |
| *** rlandy has quit IRC | 00:30 | |
| openstackgerrit | Clark Boylan proposed zuul/zuul-registry master: Handle ns parameter in get_blob https://review.opendev.org/c/zuul/zuul-registry/+/773549 | 00:30 |
| clarkb | corvus: ianw tristanC ^ fyi | 00:30 |
| ianw | clarkb: are we sure it's the buildx client and not some other part of the job using podman somehow? but otherwise lgtm | 00:31 |
| clarkb | ianw: yes it fails in the multiarch image build when buildx pulls the upstream image to build on top of | 00:32 |
| clarkb | I don't think that uses podman | 00:32 |
| ianw | i can't find the reference in the buildx code, but that doesn't mean much | 00:33 |
| clarkb | I bet its calling out to containerd | 00:35 |
| clarkb | I think all that container code gets passed around a lot. Like docker's failure to deal with ipv6 literals is everywhere too including podman | 00:35 |
| clarkb | and that is because they all source the same file in some docker repo | 00:35 |
| ianw | yeah, there is also a vendor/ directory with ... a lot of stuff | 00:39 |
| ianw | here's where it possibly came into buildx via vendored code updates https://github.com/docker/buildx/commit/2d720a1e0b99988f510b54f0192e3a612e564d17 | 00:47 |
| *** tjgresha has joined #zuul | 00:48 | |
| corvus | mhu: this look right for tagging? commit 7bab2f829165602af49ce42dfb05fd2a391f65ca (HEAD -> master, tag: 0.0.4, origin/master, origin/HEAD) | 00:52 |
| -openstackstatus- NOTICE: The Gerrit service on review.opendev.org is being quickly restarted to apply a new security patch | 00:58 | |
| *** tjgresha has quit IRC | 01:30 | |
| openstackgerrit | Merged zuul/nodepool master: Connect root logger to null handler by default https://review.opendev.org/c/zuul/nodepool/+/773248 | 02:07 |
| *** tjgresha has joined #zuul | 02:13 | |
| *** tjgresha has quit IRC | 02:30 | |
| *** hamalq has quit IRC | 02:36 | |
| *** irclogbot_0 has quit IRC | 03:27 | |
| *** ykarel has joined #zuul | 04:11 | |
| *** ykarel_ has joined #zuul | 04:15 | |
| *** ykarel has quit IRC | 04:17 | |
| *** irclogbot_3 has joined #zuul | 04:43 | |
| *** irclogbot_3 has quit IRC | 04:54 | |
| *** irclogbot_0 has joined #zuul | 04:59 | |
| *** vishalmanchanda has joined #zuul | 05:07 | |
| *** ykarel_ is now known as ykarel | 05:16 | |
| *** evrardjp has quit IRC | 05:33 | |
| *** evrardjp has joined #zuul | 05:33 | |
| *** jfoufas1 has joined #zuul | 05:49 | |
| *** ykarel_ has joined #zuul | 05:51 | |
| *** ykarel has quit IRC | 05:53 | |
| *** ykarel_ is now known as ykarel | 06:22 | |
| *** jcapitao has joined #zuul | 07:00 | |
| *** hashar has joined #zuul | 08:13 | |
| *** rpittau|afk is now known as rpittau | 08:25 | |
| *** jfoufas1 has quit IRC | 08:40 | |
| *** nils has joined #zuul | 08:53 | |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: gitlab: Add access token name, Update docs, Fix webhook https://review.opendev.org/c/zuul/zuul/+/771184 | 08:53 |
| *** jpena|off is now known as jpena | 08:56 | |
| *** tosky has joined #zuul | 09:02 | |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Update tests/base.py to use proper git data https://review.opendev.org/c/zuul/zuul/+/742746 | 09:09 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Fix gerrit merge commit change with zuul configuration https://review.opendev.org/c/zuul/zuul/+/762886 | 09:09 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Fix zuul-client enqueue-ref when oldrev/newrev aren't provided https://review.opendev.org/c/zuul/zuul/+/765767 | 09:09 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Improve merger getFilesChanges, Fix edge cases https://review.opendev.org/c/zuul/zuul/+/762887 | 09:09 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Add tests https://review.opendev.org/c/zuul/zuul/+/742747 | 09:09 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Reorg scheduler event process loop https://review.opendev.org/c/zuul/zuul/+/742748 | 09:09 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Scheduler ref-updated create/delete https://review.opendev.org/c/zuul/zuul/+/739198 | 09:10 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Allow ref-updated newrev+oldrev reconfiguration https://review.opendev.org/c/zuul/zuul/+/742749 | 09:10 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Get ref-updated modified files https://review.opendev.org/c/zuul/zuul/+/739078 | 09:10 |
| mhu | corvus: yes I think so | 09:13 |
| openstackgerrit | Clément Mondion proposed zuul/zuul master: [UI][Nodes] use patternfly 4 https://review.opendev.org/c/zuul/zuul/+/768192 | 09:54 |
| *** jfoufas1 has joined #zuul | 10:18 | |
| *** tosky has quit IRC | 10:33 | |
| *** tosky has joined #zuul | 10:34 | |
| openstackgerrit | Matthieu Huin proposed zuul/nodepool master: Move zuul integration testing to nodepool project https://review.opendev.org/c/zuul/nodepool/+/760564 | 10:37 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Move the testing framework under zuul https://review.opendev.org/c/zuul/zuul/+/760361 | 10:41 |
| *** zbr1 has joined #zuul | 11:14 | |
| *** zbr has quit IRC | 11:16 | |
| *** zbr1 is now known as zbr | 11:16 | |
| *** sshnaidm|ruck is now known as sshnaidm|afk | 11:38 | |
| *** jcapitao is now known as jcapitao_lunch | 12:08 | |
| *** piotrowskim has joined #zuul | 12:12 | |
| *** rlandy has joined #zuul | 12:26 | |
| *** hashar is now known as hasharLunch | 12:29 | |
| *** jpena is now known as jpena|lunch | 12:36 | |
| *** sshnaidm|afk is now known as sshnaidm|ruck | 12:36 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Remove obsolete testenv "zuul_client" https://review.opendev.org/c/zuul/zuul/+/773681 | 13:08 |
| *** jcapitao_lunch is now known as jcapitao | 13:14 | |
| *** hasharLunch is now known as hashar | 13:18 | |
| *** jpena|lunch is now known as jpena | 13:25 | |
| openstackgerrit | Clément Mondion proposed zuul/zuul master: [UI][Nodes] use patternfly 4 https://review.opendev.org/c/zuul/zuul/+/768192 | 13:51 |
| openstackgerrit | Clément Mondion proposed zuul/zuul master: [UI][Nodes] use patternfly 4 https://review.opendev.org/c/zuul/zuul/+/768192 | 14:47 |
| openstackgerrit | Slawek Kaplonski proposed zuul/zuul-jobs master: [multi-node-bridge] Add script to configure connectivity https://review.opendev.org/c/zuul/zuul-jobs/+/762650 | 14:50 |
| openstackgerrit | Clément Mondion proposed zuul/zuul master: [UI][Nodes] use patternfly 4 https://review.opendev.org/c/zuul/zuul/+/768192 | 14:50 |
| openstackgerrit | Clément Mondion proposed zuul/zuul master: [UI][Nodes] use patternfly 4 https://review.opendev.org/c/zuul/zuul/+/768192 | 14:51 |
| tobiash | mhu: I've commented on 734082 | 14:54 |
| openstackgerrit | Clément Mondion proposed zuul/zuul master: [web][Labels] update table to new lib https://review.opendev.org/c/zuul/zuul/+/773703 | 15:07 |
| mhu | tobiash, thanks, I'll have a look | 15:13 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Update tests/base.py to use proper git data https://review.opendev.org/c/zuul/zuul/+/742746 | 15:13 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Fix gerrit merge commit change with zuul configuration https://review.opendev.org/c/zuul/zuul/+/762886 | 15:13 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Fix zuul-client enqueue-ref when oldrev/newrev aren't provided https://review.opendev.org/c/zuul/zuul/+/765767 | 15:13 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Improve merger getFilesChanges, Fix edge cases https://review.opendev.org/c/zuul/zuul/+/762887 | 15:13 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Add tests https://review.opendev.org/c/zuul/zuul/+/742747 | 15:13 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Reorg scheduler event process loop https://review.opendev.org/c/zuul/zuul/+/742748 | 15:13 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Scheduler ref-updated create/delete https://review.opendev.org/c/zuul/zuul/+/739198 | 15:13 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Allow ref-updated newrev+oldrev reconfiguration https://review.opendev.org/c/zuul/zuul/+/742749 | 15:13 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Get ref-updated modified files https://review.opendev.org/c/zuul/zuul/+/739078 | 15:13 |
| mhu | zuul-maint, I've revisited moving zuul's test framework into the zuul package so it can be imported in functional tests for other projects: https://review.opendev.org/q/topic:%22zuul.tests%22+(status:open%20OR%20status:merged) | 15:15 |
| mhu | it works well for zuul-client but I have trouble doing it with the zuul-nodepool tests; does it actually need to be done for nodepool too? | 15:15 |
| zbr | mhu: good idea | 15:18 |
| openstackgerrit | Clément Mondion proposed zuul/zuul master: [web][Labels] update table to new lib https://review.opendev.org/c/zuul/zuul/+/773703 | 15:18 |
| *** hashar is now known as hasharAway | 15:27 | |
| fungi | mhu: one problem i expect that to bring is that testing something like zuul-client will result in you having to install all of zuul's dependencies just to use the test framework from it... though maybe extras is a way to work around that | 15:36 |
| tristanC | fungi: mhu: and the other way around, the test requirements will be zuul requirements too | 15:46 |
| tristanC | if the goal is to re-use the tests function, could they be published as a separate deliverable? | 15:47 |
| fungi | i think we can still artificially split those test requirements, it's how we've done unit tests with the tests inside the package | 15:48 |
| fungi | in other projects i mean | 15:49 |
| fungi | e.g., nodepool is already that way | 15:49 |
| fungi | the tests are accessible as nodepool.tests.something, but the requirements for running those tests are tracked in test-requirements.txt instead of requirements.txt so they don't pollute the setup-requires in the package | 15:50 |
| *** jamesmcarthur has joined #zuul | 16:02 | |
| *** jamesmcarthur has quit IRC | 16:03 | |
| *** jamesmcarthur has joined #zuul | 16:03 | |
| *** hasharAway is now known as hashar | 16:04 | |
| corvus | mhu: we explicityl decided not to do that | 16:20 |
| *** ykarel has quit IRC | 16:20 | |
| corvus | mhu: i understand the issue with zuul-client, but i don't think we should pursue that right now. the zuul tests are huge, and they're not actually something we consider to be a deliverable of zuul. | 16:21 |
| corvus | mhu: it might be better to try to brainstorm a different way of testing zuul-client with zuul. | 16:22 |
| corvus | mhu: maybe you could use fakes in z-c for unit tests, then set up a real zuul (in containers, like quickstart) for integration tests. | 16:30 |
| corvus | mhu: or maybe the status quo is okay? i know it's not ideal, but zuul-client shouldn't be changing too much; it may not be worth a huge investment to improve its tests. | 16:31 |
| tristanC | corvus: adding the tests to zuul deliverable would not be too heavy, compressed the tests directory adds 1.8MB to the 10.9MB zuul-3.19.1.tar.gz, and it would be barely noticable for the container image | 16:36 |
| tristanC | corvus: though i also think it's better to keep them separated | 16:37 |
| mhu | corvus, I can work with the status quo, provided we're okay with functional testing of new features in follow-up patches. Like https://review.opendev.org/c/zuul/zuul-client/+/752909 then https://review.opendev.org/c/zuul/zuul/+/758985 | 16:38 |
| avass | tristanC: I think software factory is being sad: https://review.opendev.org/c/zuul/zuul-jobs/+/771585 | 16:38 |
| avass | not sure if that's because of my change or not but it looks irrelevant | 16:38 |
| corvus | mhu: yep, i'm okay with the followup testing approach; i think that's a good balance of effort/reward for now. :) | 16:39 |
| tristanC | avass: oops, it's the reno test that doesn't work on python2, let me disable it quickly | 16:39 |
| clarkb | tristanC: corvus if you get a chance https://review.opendev.org/c/zuul/zuul-registry/+/773549 should fix the next thing with the buildx multiarch jobs | 16:52 |
| *** jcapitao has quit IRC | 16:52 | |
| clarkb | I'll continue to shepherd that along if the registry update lands | 16:52 |
| corvus | clarkb: lgtm thx | 16:52 |
| *** jamesmcarthur has quit IRC | 16:52 | |
| *** jamesmcarthur has joined #zuul | 16:54 | |
| *** jamesmcarthur has quit IRC | 16:57 | |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Update tests/base.py to use proper git data https://review.opendev.org/c/zuul/zuul/+/742746 | 16:59 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Fix gerrit merge commit change with zuul configuration https://review.opendev.org/c/zuul/zuul/+/762886 | 16:59 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Fix zuul-client enqueue-ref when oldrev/newrev aren't provided https://review.opendev.org/c/zuul/zuul/+/765767 | 16:59 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Improve merger getFilesChanges, Fix edge cases https://review.opendev.org/c/zuul/zuul/+/762887 | 16:59 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Add tests https://review.opendev.org/c/zuul/zuul/+/742747 | 16:59 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Reorg scheduler event process loop https://review.opendev.org/c/zuul/zuul/+/742748 | 16:59 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Scheduler ref-updated create/delete https://review.opendev.org/c/zuul/zuul/+/739198 | 16:59 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Allow ref-updated newrev+oldrev reconfiguration https://review.opendev.org/c/zuul/zuul/+/742749 | 16:59 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Get ref-updated modified files https://review.opendev.org/c/zuul/zuul/+/739078 | 17:00 |
| *** jamesmcarthur has joined #zuul | 17:00 | |
| openstackgerrit | Merged zuul/zuul-registry master: Handle ns parameter in get_blob https://review.opendev.org/c/zuul/zuul-registry/+/773549 | 17:06 |
| *** zbr1 has joined #zuul | 17:16 | |
| *** zbr has quit IRC | 17:18 | |
| *** zbr1 is now known as zbr | 17:18 | |
| openstackgerrit | Hemant Jain proposed zuul/nodepool master: Changed ExecStart address location in etc/nodepool-launcher.service.d/centos.conf as original configuration was giving an error. Explained in story https://storyboard.openstack.org/#!/story/2008569 https://review.opendev.org/c/zuul/nodepool/+/773667 | 17:20 |
| *** rlandy is now known as rlandy|biab | 17:40 | |
| clarkb | zbr: corvus ianw fungi that latest patch to the zuul-registry seems to have fixed the multiarch test jobs | 17:41 |
| corvus | clarkb: thank you for fiking that! | 17:41 |
| fungi | thanks clarkb! | 17:41 |
| corvus | fixing even | 17:41 |
| corvus | it was already fiked, i'd say | 17:42 |
| clarkb | my do not merge change is now only failing on the k8s crio job | 17:42 |
| corvus | (dvorak typos are the best) | 17:42 |
| clarkb | looks like a post run playbook does a health check and etcd returns a failure (not sure if that is a consistent or one off failure though) | 17:42 |
| clarkb | https://zuul.opendev.org/t/zuul/build/a6fbef4038824d8fbdc87fba870ceb73/log/job-output.txt#623-659 is the crio error | 17:43 |
| zbr | clarkb: a recheck to see if is random? | 17:44 |
| *** jamesmcarthur has quit IRC | 17:45 | |
| clarkb | zbr: https://zuul.opendev.org/t/zuul/builds?job_name=zuul-jobs-test-ensure-kubernetes-crio implies it isn't a consistent error | 17:46 |
| clarkb | unless it is a very new issue | 17:46 |
| *** jpena is now known as jpena|off | 17:55 | |
| *** jamesmcarthur has joined #zuul | 17:59 | |
| *** nils has quit IRC | 18:11 | |
| *** jfoufas1 has quit IRC | 18:14 | |
| *** fbo|off is now known as fbo | 18:22 | |
| *** jamesmcarthur has quit IRC | 18:24 | |
| *** sshnaidm|ruck is now known as sshnaidm|afk | 18:34 | |
| *** rpittau is now known as rpittau|afk | 18:39 | |
| avass | tristanC: thanks! | 18:39 |
| *** hashar is now known as hasharDinner | 18:41 | |
| *** jamesmcarthur has joined #zuul | 18:52 | |
| *** jamesmcarthur has quit IRC | 19:03 | |
| *** jamesmcarthur has joined #zuul | 19:15 | |
| *** rlandy|biab is now known as rlandy | 19:35 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/nodepool master: quota: ignore node without a type attribute https://review.opendev.org/c/zuul/nodepool/+/773788 | 19:37 |
| *** hasharDinner is now known as hashar | 19:39 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/nodepool master: quota: ignore node without a type attribute https://review.opendev.org/c/zuul/nodepool/+/773788 | 19:43 |
| *** evrardjp has quit IRC | 19:51 | |
| *** jamesmcarthur has quit IRC | 20:05 | |
| *** jamesmcarthur has joined #zuul | 20:06 | |
| *** jamesmcarthur has quit IRC | 20:07 | |
| *** jamesmcarthur has joined #zuul | 20:12 | |
| *** evrardjp has joined #zuul | 20:18 | |
| *** evrardjp has quit IRC | 20:26 | |
| *** evrardjp has joined #zuul | 20:27 | |
| *** jamesmcarthur has quit IRC | 20:40 | |
| *** vishalmanchanda has quit IRC | 21:15 | |
| masterpe | I'm sorry if I'm ask stupid questions but I'm trying to convert my static nodepool instance to a dynamic one with using https://zuul-ci.org/docs/nodepool/openstack.html | 21:24 |
| masterpe | but when I try to start nodepool-launcher I get a error: https://gist.github.com/mpiscaer/ef07d155f533ce089fda77a9f7bcc281 | 21:26 |
| corvus | masterpe: no stupid questions around here, only bad documentation (i'm allowed to say that since i wrote a lot of it) :) | 21:28 |
| ianw | diskimage = config.diskimages[i.name] | 21:28 |
| ianw | KeyError: 'ubuntu-bionic' | 21:28 |
| corvus | masterpe: the nodepool config is probably missing a standalone diskimages section | 21:28 |
| ianw | masterpe: ^ that's telling you it can't find the 'ubuntu-bionic' diskimage ... maybe it's not defined? | 21:28 |
| masterpe | Yes, in the /etc/nodepool/nodepool.yaml I have change that. | 21:29 |
| corvus | masterpe: however, if you aren't building images, then you probably don't need that. if that's the case, then you probably want to switch from using diskimages to 'cloud-images' in the launcher config | 21:29 |
| corvus | masterpe: so first question is: are you trying to build your own images, or do you just want to use an existing image in the cloud? | 21:29 |
| masterpe | ah | 21:29 |
| masterpe | No I want to use a existing image. | 21:30 |
| corvus | masterpe: i'm literally doing the exact same thing right now, let me paste you the config i just wrote :) | 21:30 |
| corvus | masterpe: http://paste.openstack.org/show/802261/ | 21:32 |
| corvus | masterpe: you may or may not need the "networks" section | 21:32 |
| corvus | masterpe: but that's a minimal launcher config file for an openstack cloud using an existing image | 21:33 |
| corvus | masterpe: the all caps stuff (like IMAGE_NAME_HERE) are things you'll need to fill in based on the values in the cloud provider. | 21:35 |
| *** hashar has quit IRC | 21:40 | |
| *** jamesmcarthur has joined #zuul | 21:41 | |
| masterpe | I also had to solve other issues, in my config. corvus Thanks for helping out. Is the openstack driver generating dynamic ssh-keys or what is the best solution to that? | 21:53 |
| clarkb | masterpe: no, you are expected to pregenerate a key and then bake that into the imge or supply it via nova's ssh key management | 21:55 |
| clarkb | masterpe: then on the zuul side it bootstraps the node and switches the keys over to a dynamic key | 21:55 |
| clarkb | the nodepool side is relatively simple in comparison | 21:55 |
| masterpe | so the private key needs to be in the home directory of the nodepool user? | 21:56 |
| clarkb | I want to say no, only zuul needs the private key side to bootstrap the dynamic keys | 21:57 |
| clarkb | but I could be wrong about that | 21:57 |
| corvus | zuul only, not nodepool | 22:04 |
| corvus | masterpe: in the config i pasted, it references an openstack 'keypair'. zuul needs the public key for that. | 22:04 |
| clarkb | corvus: unrelated, but do we need to consider a zuul-regsitry release? | 22:05 |
| corvus | clarkb: couldn't hurt, though i don't think it's urgent. | 22:10 |
| clarkb | ok | 22:10 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Get ref-updated modified files https://review.opendev.org/c/zuul/zuul/+/739078 | 22:10 |
| clarkb | it seems that the intermediate registry for opendev has updated in any case | 22:11 |
| corvus | mhu: i've pushed the 0.0.4 tag; can you please check pypi and dockerhub to make sure both updated (after the jobs finish) and let me know if it's okay or there are any anomalies | 22:15 |
| *** jamesmcarthur has quit IRC | 22:49 | |
| *** asettle has joined #zuul | 22:56 | |
| *** tjgresha has joined #zuul | 23:13 | |
| corvus | Open10K8S: fyi 772364 needs an update (see the log from the test failure) | 23:21 |
| tjgresha | my builds are passing, but on the buildset tab on the web ui, all are are result of error - any way to see what that error is? | 23:28 |
| corvus | tjgresha: a zuul admin (dunno if that's you or not :) will need to look at the zuul executor logs to find out | 23:30 |
| tjgresha | yeah that is me.. | 23:30 |
| corvus | tjgresha: either the executor logs or possible the scheduler. relevant log lines should have the build uuid in them, so you can grep for that. | 23:31 |
| corvus | tjgresha: but a result of error does mean an error related to zuul (as opposed to some kind of failure of the job) | 23:33 |
| tjgresha | i been called worse | 23:33 |
| *** tjgresha has quit IRC | 23:34 | |
| *** tjgresha has joined #zuul | 23:35 | |
| corvus | tjgresha: oh, also the error cause may be available in the web ui under the build page | 23:35 |
| tjgresha | ok that is good to know | 23:36 |
| corvus | i'm still looking into that | 23:36 |
| tjgresha | appreciate it.. will dig some more.. | 23:36 |
| corvus | huh; i think we may expose that in the api but not on the web page (doh) | 23:37 |
| corvus | yep that's the case | 23:37 |
| corvus | tjgresha: okay, so, in addition to checking the log files, you may have success with this: | 23:38 |
| corvus | here's an example error build: https://zuul.opendev.org/t/openstack/build/10a13bb2a40f490b9d6c3437bc3b41e6 | 23:38 |
| corvus | here's the api query for the same build: https://zuul.opendev.org/api/tenant/openstack/build/10a13bb2a40f490b9d6c3437bc3b41e6 | 23:38 |
| corvus | note the "error_detail" field there | 23:38 |
| corvus | we need to plumb that through to the web page, obviously | 23:39 |
| corvus | but in the interim, that might be an easy thing for you to check | 23:39 |
| tjgresha | good stuff - i will give it go.. | 23:42 |
| corvus | i'm working on a patch for the web ui | 23:46 |
| *** rlandy has quit IRC | 23:55 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!