| mnasiadka | Hello - any reason https://review.opendev.org/c/openstack/kolla/+/886380 is not merging? gate pipeline succeeded... | 08:27 |
|---|---|---|
| frickler | 2023-11-25 07:16:18,235 DEBUG zuul.Pipeline.openstack.gate: [e: e6d6c565a03f4421b8b9e8903fcfc13e] <QueueItem c7130523284747ff9835bbb946b2f351 live for <Change 0x7fb2a0558790 openstack/kolla 886380,2> in gate> is a failing item because ['it did not merge'] | 11:21 |
| frickler | very helpful ;-D | 11:21 |
| frickler | infra-root: https://paste.opendev.org/show/b4a9K5UJBa51rTLFoDjn/ has a bit more context, I'll leave it in that state for now in case someone wants to take a deeper look | 11:22 |
| frickler | mnasiadka: tkajinam: usually the answer would be to rebase and try again, but leave wait with that for a bit | 11:22 |
| frickler | s/leave/please/ | 11:23 |
| frickler | hmm, this looks like an error in gerrit for sure. maybe a bug or some inconsistency in our db after the upgrade? java.lang.NullPointerException: Cannot invoke "com.google.gerrit.entities.Account$Id.get()" because "this.account" is null | 11:29 |
| frickler | https://paste.opendev.org/show/b615KTW52ITobC7n4zsT/ | 11:30 |
| fungi | frickler: mnasiadka: it looks like maybe the committer address on patchset #1 is no longer associated with any account in gerrit (the change owner removed that address from their gerrit account). i wonder if that could be what's triggering the bug | 13:12 |
| fungi | though maybe not. the "author" still reflects the nonexistent address but the "owner" is associated with their current address, so the account is still the same | 13:15 |
| frickler | a similar patch was merged 5 days ago, so after the gerrit upgrade https://review.opendev.org/c/openstack/nova/+/889787 | 17:24 |
| Clark[m] | It looks like an issue with gerrit metrics (which we don't use) and checking the change for specific user attributes. I'm not sure the author is the problem | 17:31 |
| fungi | okay, so unlikely to be the problem | 17:31 |
| fungi | yeah | 17:31 |
| fungi | i missed where the metrics plugin was mentioned in the backtrace | 17:32 |
| Clark[m] | Line 10 and a few above it | 17:33 |
| Clark[m] | mnasiadka rebaser the change and it is failing in the code that counts rebaser approvals. I think maybe drop mnasiadka's votes and try to merge it again. If that works we file a bug describing the situation and workaround | 17:36 |
| opendevreview | Harry Kominos proposed openstack/diskimage-builder master: feat: Add new fail2ban elemenent https://review.opendev.org/c/openstack/diskimage-builder/+/892541 | 20:23 |
| *** dhill is now known as Guest8141 | 21:59 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!