Monday, 2022-08-22

-@gerrit:opendev.org- Zuul merged on behalf of Ian Wienand: [zuul/nodepool] 853887: Update DIB to 3.23.1 https://review.opendev.org/c/zuul/nodepool/+/85388700:56
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 853909: Fedora : update to Fedora 36 release nodes https://review.opendev.org/c/zuul/zuul-jobs/+/85390904:38
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 853913: zuul-jobs-test-ensure-python-pyenv: update matchers https://review.opendev.org/c/zuul/zuul-jobs/+/85391305:16
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/nodepool] 853914: [wip] update openshift job nodes https://review.opendev.org/c/zuul/nodepool/+/85391405:22
-@gerrit:opendev.org- Ian Wienand proposed:05:27
- [zuul/zuul-jobs] 853913: zuul-jobs-test-ensure-python-pyenv: update matchers https://review.opendev.org/c/zuul/zuul-jobs/+/853913
- [zuul/zuul-jobs] 853909: Fedora : update to Fedora 36 release nodes https://review.opendev.org/c/zuul/zuul-jobs/+/853909
- [zuul/zuul-jobs] 853916: [wip] zuul-jobs-test-registry-buildset-registry-openshift-docker: use 9-stream https://review.opendev.org/c/zuul/zuul-jobs/+/853916
-@gerrit:opendev.org- Ian Wienand proposed:05:45
- [zuul/zuul-jobs] 853916: [wip] zuul-jobs-test-registry-buildset-registry-openshift-docker: use 9-stream https://review.opendev.org/c/zuul/zuul-jobs/+/853916
- [zuul/zuul-jobs] 853918: zuul-jobs-test-registry-buildset-registry : update matcher https://review.opendev.org/c/zuul/zuul-jobs/+/853918
@iwienand:matrix.orghrm, updating openshift is going to require much more context switching in than i'd hoped06:31
@avass:vassast.org> <@tobias.henkel:matrix.org> all gdb back traces I got so far from hanging tasks look the same:07:53
> ```
> Thread 1 (Thread 0x7f9ba968a740 (LWP 1758660)):
> #0 __lll_lock_wait () at ../sysdeps/unix/sysv/linux/x86_64/lowlevellock.S:103
> #1 0x00007f9ba99e67d1 in __GI___pthread_mutex_lock (mutex=0x7f9ba9dfc990 <_rtld_global+2352>) at ../nptl/pthread_mutex_lock.c:115
> #2 0x00007f9ba9ddf1ce in _dl_add_to_namespace_list (new=0x55b53d175e10, nsid=0) at dl-object.c:33
> #3 0x00007f9ba9dda792 in _dl_map_object_from_fd (name=name@entry=0x7f9ba58b54b0 "/usr/local/lib/python3.8/lib-dynload/_crypt.cpython-38-x86_64-linux-gnu.so", origname=origname@entry=0x0, fd=-1, fbp=fbp@entry=0x7ffc97c7ca90, realname=<optimized out>, loader=loader@entry=0x0, l_type=<optimized out>, mode=<optimized out>, stack_endp=<optimized out>, nsid=<optimized out>) at dl-load.c:1382
> #4 0x00007f9ba9ddca8d in _dl_map_object (loader=0x0, loader@entry=0x7f9ba9dce000, name=name@entry=0x7f9ba58b54b0 "/usr/local/lib/python3.8/lib-dynload/_crypt.cpython-38-x86_64-linux-gnu.so", type=type@entry=2, trace_mode=trace_mode@entry=0, mode=mode@entry=-1879048190, nsid=<optimized out>) at dl-load.c:2466
> #5 0x00007f9ba9de6feb in dl_open_worker (a=a@entry=0x7ffc97c7cfe0) at dl-open.c:228
> #6 0x00007f9ba97c157f in __GI__dl_catch_exception (exception=exception@entry=0x7ffc97c7cfc0, operate=operate@entry=0x7f9ba9de6f60 <dl_open_worker>, args=args@entry=0x7ffc97c7cfe0) at dl-error-skeleton.c:196
> #7 0x00007f9ba9de6bba in _dl_open (file=0x7f9ba58b54b0 "/usr/local/lib/python3.8/lib-dynload/_crypt.cpython-38-x86_64-linux-gnu.so", mode=-2147483646, caller_dlopen=0x7f9ba9c675d1 <_PyImport_FindSharedFuncptr+113>, nsid=<optimized out>, argc=17, argv=0x7ffc97c82d58, env=0x7ffc97c82de8) at dl-open.c:599
> ```
We ran into that as well, and upgrading to using a newer ansible version will fix it: https://github.com/ansible/ansible/pull/72412, https://github.com/ansible/ansible/issues/59642
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 853372: Convert GCE to state machine driver and remove simple https://review.opendev.org/c/zuul/nodepool/+/85337214:03
@jpew:matrix.orgI'm having trouble running zuul-web in a sub-URL (foo.bar.com/zuul); it seems to want to fetch some Javascript without the correct prefix, e.g. `GET /static/js/main.e157fcda.chunk.js` should be `GET /zuul/static/js/main.e157fcda.chunk.js`. I'm already setting `web.root` to `https://foo.bar.com/zuul` in the zuul config, am I missing something else?14:31
-@gerrit:opendev.org- Marvin Becker proposed wip: [zuul/nodepool] 853993: [WIP] Add disable and enable node commands https://review.opendev.org/c/zuul/nodepool/+/85399314:32
@tristanc_:matrix.orgjpew: have you checked https://zuul-ci.org/docs/zuul/latest/installation.html#sub-directory-serving ?14:35
@jpew:matrix.orgtristanC: RTFM I guess :)14:35
@jpew:matrix.orgAh, I have to rebuild the website. That's unfortunate14:37
-@gerrit:opendev.org- Marvin Becker proposed wip: [zuul/nodepool] 853993: [WIP] Add disable and enable node commands https://review.opendev.org/c/zuul/nodepool/+/85399314:38
@tristanc_:matrix.orgjpew: i meant, perhaps the doc was incomplete, but it should contains the instructions for your setup.14:38
-@gerrit:opendev.org- Marvin Becker proposed wip: [zuul/nodepool] 853993: [WIP] Add disable and enable node commands https://review.opendev.org/c/zuul/nodepool/+/85399314:39
@clarkb:matrix.orgI think an OpenDev user has discovered a bug with Zuul's github driver: https://paste.opendev.org/show/bQelUAO1ppQHhjqkoxTF/15:15
@clarkb:matrix.orgIt appears we are making a request of graphql that we are not authorized to make? Is this a corner case where we need an app installation just to fulfill depends-on?15:16
@clarkb:matrix.orghttps://opendev.org/zuul/zuul/src/branch/master/zuul/driver/github/graphql/canmerge.graphql this is the query we appear to be running. I'm not familiar enough with github's graphql api and permissions model to know if anything in there might be sensitive and require extra privs15:29
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed:15:51
- [zuul/zuul] 853548: Deprecate Ansible 2, make Ansible 5 default https://review.opendev.org/c/zuul/zuul/+/853548
- [zuul/zuul] 854009: Fix Ansible version testing https://review.opendev.org/c/zuul/zuul/+/854009
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 854009: Fix Ansible version testing https://review.opendev.org/c/zuul/zuul/+/85400915:52
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 853548: Deprecate Ansible 2, make Ansible 5 default https://review.opendev.org/c/zuul/zuul/+/85354815:52
-@gerrit:opendev.org- Zuul merged on behalf of Tristan Cacqueray: [zuul/zuul-operator] 845851: Update CRD apiVersion to v1 (from v1beta) https://review.opendev.org/c/zuul/zuul-operator/+/84585115:55
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 853552: Add Ansible 6 https://review.opendev.org/c/zuul/zuul/+/85355215:57
@jim:acmegating.comfungi: can you take a look at https://review.opendev.org/853579 ?16:07
@fungicide:matrix.orgthanks for the heads up, i've commented on it17:15
@clarkb:matrix.org> <@clarkb:matrix.org> https://opendev.org/zuul/zuul/src/branch/master/zuul/driver/github/graphql/canmerge.graphql this is the query we appear to be running. I'm not familiar enough with github's graphql api and permissions model to know if anything in there might be sensitive and require extra privs17:29
Looking into this further, I am beginning to suspect that we may actually need the app installed on the repo to process these graphql queries successfully. In particular it seems that we get auth here: https://opendev.org/zuul/zuul/src/branch/master/zuul/driver/github/githubconnection.py#L1208-L1255 and that assumes if zuul has an app install then we use it or fallback to anonymous. Since the repo doesn't have the app installed we must be falling back to anonymous requests which fail because checks results require auth
@jim:acmegating.comi think it would be worthwhile to support depends-on without the app installed, so i think it's worth someone looking into how to accomplish that.17:31
-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/nodepool] 851941: AWS: Use snapshot instead of image import https://review.opendev.org/c/zuul/nodepool/+/85194117:32
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed:17:43
- [zuul/zuul] 854009: Fix Ansible version testing https://review.opendev.org/c/zuul/zuul/+/854009
- [zuul/zuul] 853548: Deprecate Ansible 2, make Ansible 5 default https://review.opendev.org/c/zuul/zuul/+/853548
- [zuul/zuul] 853552: Add Ansible 6 https://review.opendev.org/c/zuul/zuul/+/853552
-@gerrit:opendev.org- Zuul merged on behalf of Stephen Finucane: [zuul/zuul] 853420: setup: Replace dash-separated config https://review.opendev.org/c/zuul/zuul/+/85342018:13
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 853552: Add Ansible 6 https://review.opendev.org/c/zuul/zuul/+/85355220:19
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 854044: Add config-error reporter and report config errors to DB https://review.opendev.org/c/zuul/zuul/+/85404422:12
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 853552: Add Ansible 6 https://review.opendev.org/c/zuul/zuul/+/85355223:08
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 854046: Remove merge-failure pipeline configuration https://review.opendev.org/c/zuul/zuul/+/85404623:21
@iwienand:matrix.orgit looks like getting the openshift test bits off centos-7 involves a re-write to use openshift-installer.  unfortunately you have to pull and build from source to use libvirt/kvm backend.  i have some concern it may be unhappy with a nested virt situation but maybe not.23:29
@iwienand:matrix.orgi'm debating how much effort this is worth ...23:29
@clarkb:matrix.orgianw: yes, every time I look at it the impression I get is that it is a significant amount of work and that the openshift project doesn't seem ease of deployment or testability to be important functionality23:32
@clarkb:matrix.orgIt is unfortunate because all they do is run a lot of containers. Nothing about it should be this complicated 23:33
@iwienand:matrix.orgyeah it's a bit unfortunate the two install options for the binary i downloaded was aws and vsphere23:34
@iwienand:matrix.orgi spun up a local 2gb vm to try and build it, but it seems the go processes are all getting killed.  so i assume that's not enough of a machine to build the installer ...23:36
@iwienand:matrix.orghttps://github.com/code-ready/crc is maybe more what we want for testing23:46
@clarkb:matrix.orgOh interesting 23:46
@iwienand:matrix.orgthis seems to require a login and a "pull secret" 23:47
@iwienand:matrix.orgfrom https://console.redhat.com/openshift/create/local23:47
@clarkb:matrix.orgI think there is some public pull secret thing that the open source installer documents23:48
@iwienand:matrix.orgthere is something about the crc installer for the okd distribution, that points to a binary23:54
@iwienand:matrix.orghttps://dl.fedoraproject.org/pub/alt/okd-crc/release/linux-amd64/23:54
@iwienand:matrix.orgthat says "download and run this"23:54
@iwienand:matrix.orgthis 4gb blob is also coming up on being a year old, which isn't inspiring for it's general maintenance status23:55

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