| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 821783: Fix erroneous semaphore leak cleanup https://review.opendev.org/c/zuul/zuul/+/821783 | 00:36 | |
| @jim:acmegating.com | Clark: ^ i think you're going to enjoy that :) | 00:37 |
|---|---|---|
| @jim:acmegating.com | oh forgot a git add | 00:38 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 821783: Fix erroneous semaphore leak cleanup https://review.opendev.org/c/zuul/zuul/+/821783 | 00:39 | |
| @clarkb:matrix.org | Cool I'll look at it first thing tomorrow | 01:04 |
| @srigowthami:matrix.org | Hi, | 08:54 |
| We are using Zuul, nodepool to test the commits on IBM Power hardware. | ||
| We are trying to re-deploy the nodepool-builder vm on focal vm as Xenial has reached EOM. | ||
| I have installed libjemalloc1 package but still could notice below error messages in build logs so many times and the builds aren't successful | ||
| '/usr/lib/x86_64-linux-gnu/libjemalloc.so.1' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored' | ||
| Could someone help me with these ? | ||
| @srigowthami:matrix.org | nodepool@nodepool-buildv3:/var/log/nodepool/builds$ nodepool --version | 09:07 |
| Nodepool version: 3.13.0 | ||
| @avass:vassast.org | Reading the docs for the openshift pods driver, is it not possible to use a serviceaccount configured for the pod instead of using a kubeconfig? | 09:15 |
| > Before using the driver, Nodepool services need a kube/config file manually installed. Make sure the context is present in oc config get-contexts command output. | ||
| -@gerrit:opendev.org- Riccardo Pittau proposed on behalf of Matthieu Huin https://matrix.to/#/@mhuin:matrix.org: [zuul/zuul] 796270: Tests: add non-voting unit testing for Python 3.10 https://review.opendev.org/c/zuul/zuul/+/796270 | 13:17 | |
| -@gerrit:opendev.org- Riccardo Pittau proposed on behalf of Matthieu Huin https://matrix.to/#/@mhuin:matrix.org: [zuul/zuul] 796270: Tests: add non-voting unit testing for Python 3.10 https://review.opendev.org/c/zuul/zuul/+/796270 | 13:18 | |
| -@gerrit:opendev.org- Riccardo Pittau proposed on behalf of Matthieu Huin https://matrix.to/#/@mhuin:matrix.org: [zuul/zuul] 796270: Tests: add non-voting unit testing for Python 3.10 https://review.opendev.org/c/zuul/zuul/+/796270 | 13:22 | |
| @maheshbarai1234:matrix.org | Albin Vass: : we are not using AWS. the windows node which we are trying to add is on-premises | 13:38 |
| @avass:vassast.org | Mahesh Barai: that option is not specific to aws and is available for all drivers: https://www.zuul-ci.org/docs/nodepool/static.html#attr-providers.[static].pools.nodes.shell-type | 13:41 |
| @avass:vassast.org | it's there to tell configure ansible for ssh on windows hosts. https://docs.ansible.com/ansible/latest/user_guide/windows_setup.html#configuring-ansible-for-ssh-on-windows | 13:42 |
| @avass:vassast.org | * it's there to configure ansible for ssh on windows hosts. https://docs.ansible.com/ansible/latest/user_guide/windows_setup.html#configuring-ansible-for-ssh-on-windows | 13:42 |
| @mhuin:matrix.org | Hello zuul-maint, all I want for christmas is +3s on https://review.opendev.org/q/topic:%22web_results_pagination%22+(status:open%20OR%20status:merged) 🙂 | 14:25 |
| @jim:acmegating.com | mhu: i haven't forgot... things came up. | 14:52 |
| @fungicide:matrix.org | srigowthami: We did this in our executor configs for a while in order to avoid preloading libjemalloc on our executors when upgrading them to focal: https://opendev.org/opendev/system-config/src/commit/39495ffdd5f2ed226e4d97ae2f709ee747d6d9dc/playbooks/roles/zuul-executor/templates/zuul-executor.defaults.j2#L2-L4 | 16:06 |
| @fungicide:matrix.org | mhu: the pipeline manager type icons in the dashboard are great! just noticed them showing up in opendev's deployment | 17:36 |
| @clarkb:matrix.org | The matrix eavesdrop bot has updated. | 18:07 |
| @clarkb:matrix.org | Now to see if I see my own messages | 18:07 |
| @clarkb:matrix.org | I do, I think all is well there | 18:08 |
| @jim:acmegating.com | tristanC: mhu tobiash i had to leave a -2 instead of a -1 on https://review.opendev.org/820194 because tristanC approved it while i was reviewing it. | 18:47 |
| @jim:acmegating.com | stack before that looks good though; i'm very much looking forward to having pagination :) | 18:51 |
| @tristanc_:matrix.org | corvus: oops, well I replied to your comment | 18:51 |
| @jim:acmegating.com | tristanC: thanks, that helps. i re-ran the tests without offset and got better results, so have re-approved it. | 19:04 |
| @clarkb:matrix.org | corvus: are you wanting the semaphore fix in 4.11.0? (I just had to recheck it) | 19:31 |
| @jim:acmegating.com | Yep, thx | 19:39 |
| @mhuin:matrix.org | > <@fungicide:matrix.org> mhu: the pipeline manager type icons in the dashboard are great! just noticed them showing up in opendev's deployment | 20:01 |
| Have you tried hovering the mouse on them? You get a description of the pipeline type this way | ||
| @fungicide:matrix.org | yep, checked them all, they're very helpful. the supercedent one is a little vague, but i can't think of a concise way to describe how it works anyway | 20:31 |
| @fungicide:matrix.org | something about collapsing multiple builds by skipping intermediate ones | 20:32 |
| @fungicide:matrix.org | * something about collapsing multiple buildsets by skipping intermediate queue items | 20:32 |
| @fungicide:matrix.org | if inspiration strikes with improved wording for that, i'll propose an update | 20:33 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 21:23 | |
| - [zuul/zuul] 820593: Add a version selector to docs https://review.opendev.org/c/zuul/zuul/+/820593 | ||
| - [zuul/zuul] 820915: Remove cross-project-gating howto https://review.opendev.org/c/zuul/zuul/+/820915 | ||
| - [zuul/zuul] 820916: Remove zuul-from-scratch howto https://review.opendev.org/c/zuul/zuul/+/820916 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 821881: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/nodepool/+/821881 | 21:31 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 21:32 | |
| - [zuul/zuul] 820585: Reorganize docs https://review.opendev.org/c/zuul/zuul/+/820585 | ||
| - [zuul/zuul] 820593: Add a version selector to docs https://review.opendev.org/c/zuul/zuul/+/820593 | ||
| - [zuul/zuul] 820915: Remove cross-project-gating howto https://review.opendev.org/c/zuul/zuul/+/820915 | ||
| - [zuul/zuul] 820916: Remove zuul-from-scratch howto https://review.opendev.org/c/zuul/zuul/+/820916 | ||
| @jim:acmegating.com | hrm, zuul said there was a merge conflict; i didn't see it but i rebased anyway | 21:32 |
| @clarkb:matrix.org | Diffing the patchsets shows only the commit msg changed. I wonder if it is one of those file level things that jgit can't handle | 21:33 |
| @jim:acmegating.com | well this was zuul not gerrit.. but :/ | 21:36 |
| @clarkb:matrix.org | Doesn't zuul try to emulate the jgit behavior? | 21:37 |
| @clarkb:matrix.org | it sets the merge type to one closer to what jgit does iirc | 21:37 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-client] 821883: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/zuul-client/+/821883 | 21:41 | |
| @fungicide:matrix.org | ahh, it looked to me like it had a minor conflict over the doc change from the prometheus liveliness probes commit | 21:41 |
| @fungicide:matrix.org | but maybe not if it was able to fuzzy apply in rebase | 21:41 |
| @clarkb:matrix.org | corvus: note on 821883 | 21:48 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-sphinx] 821888: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/zuul-sphinx/+/821888 | 21:50 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-client] 821883: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/zuul-client/+/821883 | 21:53 | |
| @jim:acmegating.com | Clark: thanks; easy to get wires crossed with changes like this :) | 21:53 |
| @jim:acmegating.com | every conf.py is different and getting better... i might do one more pass through to try to unify them | 21:56 |
| @clarkb:matrix.org | corvus: ya few more thoughts on the zuul-client one that may apply to the others as well | 21:59 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-operator] 821911: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/zuul-operator/+/821911 | 22:03 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-operator] 821911: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/zuul-operator/+/821911 | 22:04 | |
| @clarkb:matrix.org | and I left some notes on the zuul-sphinx and nodepool chagnes too. | 22:04 |
| @jim:acmegating.com | Clark: agree -- take a look at zuul-operator; if that looks good i'll update all the others with that | 22:04 |
| @jim:acmegating.com | (there should be a really small delta between them at that point) | 22:05 |
| @clarkb:matrix.org | Oh ya doing the min version as more of a constant looks like a good idea | 22:05 |
| @clarkb:matrix.org | And then you can use the same lookup for all of them just change the version value | 22:06 |
| @jim:acmegating.com | maybe we can even put that as a method in zuul-sphinx and call it | 22:06 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: | 22:19 | |
| - [zuul/zuul] 821783: Fix erroneous semaphore leak cleanup https://review.opendev.org/c/zuul/zuul/+/821783 | ||
| - [zuul/zuul] 820608: Quantize relative priority https://review.opendev.org/c/zuul/zuul/+/820608 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-sphinx] 821888: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/zuul-sphinx/+/821888 | 22:35 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-operator] 821911: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/zuul-operator/+/821911 | 22:38 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-sphinx] 821888: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/zuul-sphinx/+/821888 | 22:39 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-client] 821883: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/zuul-client/+/821883 | 22:43 | |
| @jim:acmegating.com | okay, i think operator, sphinx, and client are gtg with the simplied form; i'm going back for nodepool and zuul now | 22:43 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 821881: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/nodepool/+/821881 | 22:50 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-client] 821883: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/zuul-client/+/821883 | 22:51 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-sphinx] 821888: Switch docs theme to versioned RTD https://review.opendev.org/c/zuul/zuul-sphinx/+/821888 | 22:52 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 821917: Simplify doc conf.py https://review.opendev.org/c/zuul/zuul/+/821917 | 22:54 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 821918: Switch docs theme to RTD https://review.opendev.org/c/zuul/zuul-jobs/+/821918 | 22:59 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-base-jobs] 821919: Switch docs theme to RTD https://review.opendev.org/c/zuul/zuul-base-jobs/+/821919 | 23:00 | |
| @jim:acmegating.com | whew; okay let's see what those look like :) | 23:00 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 821918: Switch docs theme to RTD https://review.opendev.org/c/zuul/zuul-jobs/+/821918 | 23:08 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-base-jobs] 821919: Switch docs theme to RTD https://review.opendev.org/c/zuul/zuul-base-jobs/+/821919 | 23:08 | |
| @fungicide:matrix.org | corvus: another merge failure on 820585 when i rechecked it immediately following a test failure... i'm starting to think there's some deeper issue with the merge conflict checking | 23:13 |
| @jim:acmegating.com | fungi: yeah...that's looking sus... | 23:16 |
| @jim:acmegating.com | (i wonder if we could put event ids in merge failure messages to make it esaier to grep) | 23:17 |
| @jim:acmegating.com | ``` | 23:24 |
| Trying really trivial in-index merge... | ||
| Nope. | ||
| Trying simple merge. | ||
| Simple merge failed, trying Automatic merge. | ||
| Automatic merge failed; fix conflicts and then commit the result.' | ||
| stderr: 'error: Merge requires file-level merging | ||
| ERROR: doc/source/discussion/components.rst: Not handling case 975a32c6f6e345acab58fb4458b13df3344f258b -> b729538666e90131759688b36368da01219d4a4a -> | ||
| fatal: merge program failed' | ||
| ``` | ||
| @jim:acmegating.com | i don't think there's anything systemically wrong... | 23:25 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 23:25 | |
| - [zuul/zuul] 820585: Reorganize docs https://review.opendev.org/c/zuul/zuul/+/820585 | ||
| - [zuul/zuul] 820593: Add a version selector to docs https://review.opendev.org/c/zuul/zuul/+/820593 | ||
| - [zuul/zuul] 820915: Remove cross-project-gating howto https://review.opendev.org/c/zuul/zuul/+/820915 | ||
| - [zuul/zuul] 820916: Remove zuul-from-scratch howto https://review.opendev.org/c/zuul/zuul/+/820916 | ||
| - [zuul/zuul] 821917: Simplify doc conf.py https://review.opendev.org/c/zuul/zuul/+/821917 | ||
| @jim:acmegating.com | rebased again | 23:26 |
| @clarkb:matrix.org | The diff still shows only the commit message changing so I +2'd | 23:27 |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 804602: web: JobVariant : pull description into a card https://review.opendev.org/c/zuul/zuul/+/804602 | 23:38 | |
| @fungicide:matrix.org | yeah, this time it looks like it was probably fuzzed due to the priority batching paragraph added to the docs, i wonder if it's just such a large list of file changes that any fuzz on the files is too much for the simple merge strategy | 23:52 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 821918: Switch docs theme to RTD https://review.opendev.org/c/zuul/zuul-jobs/+/821918 | 23:59 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!