| *** rlandy is now known as rlandy|holiday | 00:05 | |
| ianw | initial report on the login popup -> https://bugs.chromium.org/p/gerrit/issues/detail?id=16018 | 00:23 |
|---|---|---|
| ianw | i searched and couldn't find a similar bug | 00:23 |
| fungi | thanks! | 01:39 |
| ianw | https://gerrit-review.googlesource.com/c/gerrit/+/339494 is a potential fix that is running on the test server @ https://173.231.255.107/ | 01:47 |
| ianw | it seems to work, but i have no idea if it's acceptable as a review, i very much hacked this from vague references in the javascript traceback of the 403 call and bits of git history | 01:48 |
| ianw | i.e. standard javascript development :) | 01:48 |
| Clark[m] | Looks like it failed some linting CI upstream | 01:56 |
| ianw | looks to me like it's actually a job setup failure? https://gerrit-ci.gerritforge.com/job/Gerrit-codestyle/84841/console | 02:07 |
| ianw | Caused by: hudson.plugins.git.GitException: Command "/usr/bin/git fetch --tags --force --progress -- https://gerrit.googlesource.com/a/gerrit refs/changes/94/339494/1" returned status code 128: | 02:08 |
| Clark[m] | Maybe rerunning it is sufficient. I think there is a button you can click to do that | 02:24 |
| *** pojadhav is now known as pojadhav|ruck | 04:30 | |
| opendevreview | yatin proposed zuul/zuul-jobs master: [DNM] Test proposal playbooks https://review.opendev.org/c/zuul/zuul-jobs/+/846546 | 06:32 |
| opendevreview | yatin proposed zuul/zuul-jobs master: [DNM] Test proposal playbooks https://review.opendev.org/c/zuul/zuul-jobs/+/846546 | 06:40 |
| opendevreview | yatin proposed zuul/zuul-jobs master: [DNM] Test proposal playbooks https://review.opendev.org/c/zuul/zuul-jobs/+/846546 | 06:45 |
| *** pojadhav|ruck is now known as pojadhav|lunch | 07:02 | |
| opendevreview | yatin proposed zuul/zuul-jobs master: [DNM] Test proposal playbooks https://review.opendev.org/c/zuul/zuul-jobs/+/846546 | 07:13 |
| *** yoctozepto[pto] is now known as yoctozepto | 07:14 | |
| *** yoctozepto is now known as Guest2593 | 07:15 | |
| opendevreview | yatin proposed zuul/zuul-jobs master: [DNM] Test proposal playbooks https://review.opendev.org/c/zuul/zuul-jobs/+/846546 | 07:25 |
| *** jpena|off is now known as jpena | 07:36 | |
| *** yoctozepto_ is now known as yoctozepto | 07:50 | |
| *** pojadhav|lunch is now known as pojadhav|ruck | 07:52 | |
| opendevreview | yatin proposed zuul/zuul-jobs master: [DNM] Test proposal playbooks https://review.opendev.org/c/zuul/zuul-jobs/+/846546 | 07:58 |
| opendevreview | yatin proposed zuul/zuul-jobs master: [DNM] Test proposal playbooks https://review.opendev.org/c/zuul/zuul-jobs/+/846546 | 08:06 |
| *** rlandy_ is now known as rlandy|holiday | 09:54 | |
| *** pojadhav is now known as pojadhav|ruck | 09:55 | |
| *** dviroel_ is now known as dviroel | 12:05 | |
| frickler | while looking for log upload errors, I found these failures on ze01, maybe someone wants to have a closer look https://paste.opendev.org/show/bL8rneWkjwDTXKx5Kxu3/ https://paste.opendev.org/show/bB5t2gxwJXc2Naa7Dyxx/ | 12:19 |
| frickler | hmm, did we change the merge conflict check in gerrit? it now sticks on a change that is definitively rebased https://review.opendev.org/c/openstack/kolla/+/836664 | 12:24 |
| frickler | can't see it on the patch itself, but on the index https://review.opendev.org/q/project:openstack%252Fkolla | 12:25 |
| fungi | for that first paste, i can't tell if the missing inventory key in the traceback is the reason the build was cancelled, or something that's happened because the job was cancelled (perhaps cancelled too early to have the node in its inventory? though that seems unlikely as well) | 12:35 |
| *** ysandeep is now known as ysandeep|afk | 13:17 | |
| Clark[m] | frickler: re the merge conflict thing that may be a cache problem since the info doesn't align across two different views. | 13:21 |
| fungi | it looked like it was claiming the change was parented to an abandoned change | 13:53 |
| fungi | ahh, no i always read that table backwards. the abandon change has 836664 as its parent | 13:55 |
| Clark[m] | The mobile change listing doesn't show that info so not easy for me to confirm what frickler was seeing. But if I had to guess caches are involved | 13:56 |
| fungi | that seems likely | 13:56 |
| fungi | i wonder if something didn't get updated during the reindex | 13:57 |
| Clark[m] | I wonder if we need to reindex more indexes. It only did changes automatically | 13:59 |
| *** ysandeep|afk is now known as ysandeep | 14:00 | |
| frickler | this seems somehow be related to WIP-ness, if I mark the patch as active, both WIP+Merge Conflict disappear in the index. once I mark it as WIP again, both are back | 14:08 |
| frickler | and the issue persists even after pushing a new revision | 14:09 |
| Clark[m] | Interesting, that may be worth an upstream bug since it is tied to the WIP state | 14:10 |
| Clark[m] | I half wonder if all WIP changes are merge conflicted now for some reason | 14:11 |
| Clark[m] | Cannot merge because WIP ~= merge conflict maybe? | 14:11 |
| frickler | that would be an interesting logic. but indeed a random patch I mark WIP shows up with MC at once, too https://review.opendev.org/q/project:cirros%252Fcirros+status:open | 14:13 |
| ykarel_ | Clark[m], can you check https://review.opendev.org/c/openstack/project-config/+/846427 | 15:14 |
| ykarel_ | and see which route can be taken to clear it | 15:14 |
| ykarel_ | fungi, wrt https://review.opendev.org/c/zuul/zuul-jobs/+/844704, do you mean we need to handle setting that var on neutron side itself? | 15:18 |
| ykarel_ | and other projects on their side? | 15:18 |
| ykarel_ | something like https://review.opendev.org/c/openstack/neutron/+/846545/1/playbooks/enable-fips.yaml | 15:18 |
| ykarel_ | or may be you meant something else | 15:19 |
| Clark[m] | ykarel re fips, yes you need to set it in your jobs | 15:20 |
| Clark[m] | And ya there isn't a good default because networking varies so much | 15:20 |
| ykarel_ | Clark[m], okk i just see tempest unmerged patch also does the same https://review.opendev.org/c/openstack/tempest/+/831607/23/zuul.d/integrated-gate.yaml#339 | 15:22 |
| ykarel_ | ok will add on our side for neutron jobs | 15:22 |
| *** ysandeep is now known as ysandeep|out | 15:29 | |
| ykarel_ | Clark[m], can you also check other one too https://review.opendev.org/c/openstack/project-config/+/846427 | 15:35 |
| ykarel_ | just in case you missed | 15:35 |
| *** dviroel is now known as dviroel|lunch | 15:36 | |
| *** marios is now known as marios|out | 16:03 | |
| Adri2000 | hello, how can I further debug a "Permission denied (publickey)" when using git-review? it used to work and I don't see what could have changed on my side recently... | 16:17 |
| Adri2000 | trying to ssh manually to check if authentication work: ssh adri2000@review.opendev.org -p 29418 - I get the same error | 16:17 |
| Clark[m] | Adri2000: did you upgrade your operating system to include newer openssh? If so and you use rsa keys that is the likely issue | 16:18 |
| Clark[m] | Newer openssh dropped support for rsa + sha1 by default and will only do rsa + sha2 but gerrit's ssh server doesn't understand how to negotiate that in a way that makes openssh happy. This should be addressed when we upgrade to Gerrit 3.6 | 16:19 |
| Clark[m] | If you aren't using rsa keys then this may be a result of our recent Gerrit 3.5 upgrade, but this sounds like the rsa issue at first glance | 16:20 |
| Adri2000 | ah, I upgraded to Ubuntu 22.04 recently, it's possible that I haven't tried pushing anything since then. and I use a 4096 rsa key indeed. | 16:21 |
| Clark[m] | There are two workarounds. First is to use an ed25519 or ecdsa key. The other is to set an ssh config to allow rsa + sha1. We aren't recommending this as it circumvents default security choices for your platform | 16:21 |
| jrosser_ | could i get some help with this https://review.opendev.org/c/openstack/ansible-role-systemd_service/+/846646 | 16:27 |
| jrosser_ | it doesnt appear to have run any jobs and i'm not seeing why that would be | 16:27 |
| Adri2000 | Clark[m]: thank you, problem fixed using a new, ed25519, key :) | 16:34 |
| *** dviroel|lunch is now known as dviroel | 16:43 | |
| *** jpena is now known as jpena|off | 16:45 | |
| clarkb | jrosser_: typically the cause is file matches not matching. However, in this case it seems there is no stable/yoga branch defined for openstack/openstack-ansible and it appears to supply the job definitions for the templates on that project. I think that may be the reason | 16:45 |
| jrosser_ | aaah ok | 16:45 |
| jrosser_ | interesting - as we generally have to branch all these ansible roles first in order to then reference them when creating the yoga branch in openstack-ansible | 16:46 |
| jrosser_ | noonedeadpunk: ^ | 16:46 |
| jrosser_ | clarkb: thanks for the pointer on missing branches... recheck has kicked the jobs off, it was probably all very racey with creating the branches and then finding the jobs across so many repos | 17:21 |
| clarkb | you're welcome | 17:21 |
| opendevreview | yatin proposed openstack/project-config master: Only install python3.8-venv in translation update jobs https://review.opendev.org/c/openstack/project-config/+/846794 | 17:23 |
| ykarel_ | Thanks Clark[m] done ^ | 17:24 |
| ykarel_ | fungi, frickler you too please check when get chance | 17:26 |
| ykarel_ | thx in advance | 17:27 |
| * ykarel_ out | 17:27 | |
| *** dviroel is now known as dviroel|afk | 20:05 | |
| *** rlandy|holiday is now known as rlandy|out | 20:16 | |
| *** dviroel|afk is now known as dviroel|out | 21:05 | |
| clarkb | ianw: upstream gerrit gave you the votes you need to merge your login required fix. They let the change authors self submit once you have the necessary approvals. Its great to see them review that so quickly | 22:36 |
| clarkb | then we can trigger a rebuild of our images to pull in the fix and deploy it | 22:40 |
| clarkb | last call for meeting agenda items. I'm about to send it out | 22:46 |
| opendevreview | Merged openstack/project-config master: Only install python3.8-venv in translation update jobs https://review.opendev.org/c/openstack/project-config/+/846794 | 22:50 |
| clarkb | and sent | 22:58 |
| fungi | thanks! | 22:58 |
| ianw | clarkb: yeah, will do today, i guess needs to forward port too | 23:11 |
| clarkb | ianw: ya once you merge the change to 3.5 you will want to merge the resulting 3.5 into 3.6 and push that then merge 3.6 into master and push that. If you want to be lazy usptream does do those periodically to pick up changes that people didn't do them for. Half the time I try to be nice and push those forward ports they end up doing it themselves anyway (to pick up even more changes | 23:13 |
| clarkb | that have landed since I pushed my changes) | 23:13 |
| ianw | looks like it has a conflict on 3.6 -- back to the javascript :) | 23:19 |
| ianw | seems it has had a bit of a rewrite, so there's a new fn setReviewedStatus() that looks like it probes for logged in status. also, it doesn't do it on gerrit-review.googlesource.com (3.6.1) | 23:23 |
| ianw | we can double check it as we get 3.6 in our ci | 23:24 |
| fungi | so maybe this is something they (accidentally?) fixed in 3.6 | 23:24 |
| clarkb | ++ | 23:24 |
| ianw | fungi: yep, https://gerrit-review.googlesource.com/c/gerrit/+/322716 was part of other work, but i think did also fix this issue | 23:26 |
| clarkb | ianw: I closed your issue | 23:33 |
| clarkb | (at some point my gerrit issue tracker perms got bumped up) | 23:33 |
| ianw | probably when you started showing an aptitude for enterprise java development :) thanks | 23:35 |
| fungi | that sounds terrifying | 23:42 |
| opendevreview | Ian Wienand proposed opendev/system-config master: gerrit: trigger rebuild for anonymous browsing fix https://review.opendev.org/c/opendev/system-config/+/846809 | 23:44 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!