-@gerrit:opendev.org- lotorev vitaly marked as active: [zuul/nodepool] 833515: Log hostname on socket.getaddrinfo() exception https://review.opendev.org/c/zuul/nodepool/+/833515 | 05:58 | |
-@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 833536: Fix change list corruption on local layout update https://review.opendev.org/c/zuul/zuul/+/833536 | 08:21 | |
-@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 833536: Fix change list corruption on local layout update https://review.opendev.org/c/zuul/zuul/+/833536 | 08:24 | |
-@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 833538: Test SQL connection before checkout from pool https://review.opendev.org/c/zuul/zuul/+/833538 | 09:03 | |
-@gerrit:opendev.org- Zuul merged on behalf of Tobias Henkel: [zuul/nodepool] 771968: Handle ports exceeded error as quota error https://review.opendev.org/c/zuul/nodepool/+/771968 | 09:49 | |
-@gerrit:opendev.org- Zuul merged on behalf of Andy Ladjadj: [zuul/zuul] 800664: Improve performance of _saveRepoState https://review.opendev.org/c/zuul/zuul/+/800664 | 11:21 | |
-@gerrit:opendev.org- Zuul merged on behalf of Tobias Henkel: [zuul/zuul] 830902: Simplify _saveRepoState https://review.opendev.org/c/zuul/zuul/+/830902 | 11:45 | |
-@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 832517: Include original path of renamed file for a PR https://review.opendev.org/c/zuul/zuul/+/832517 | 11:54 | |
@clarkb:matrix.org | corvus: I have approved https://review.opendev.org/c/zuul/zuul/+/830707 but gerrit says its child needs a rebase. I figure once the parent lands it will be easier to do that? | 16:20 |
---|---|---|
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 16:21 | |
- [zuul/zuul] 830707: Use a transaction for BuildCompletedEvent https://review.opendev.org/c/zuul/zuul/+/830707 | ||
- [zuul/zuul] 830896: Use kazoo.retry in zkobject https://review.opendev.org/c/zuul/zuul/+/830896 | ||
@jim:acmegating.com | Clark: doh! i was rebasing the pair of them while you wrote that, sorry! | 16:21 |
@clarkb:matrix.org | oh heh looks like both just got rebased | 16:21 |
@clarkb:matrix.org | its ok let me take a look and maybe we can reenqueue them | 16:21 |
@clarkb:matrix.org | looks like the conflict was around the accounting tracking? Seems pretty clean as far as rebases go | 16:25 |
@clarkb:matrix.org | I went ahead and approved both | 16:26 |
@jim:acmegating.com | Clark: yeah, that's the only issue that i saw | 16:26 |
@jim:acmegating.com | thanks :) | 16:26 |
-@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 833536: Fix change list corruption on local layout update https://review.opendev.org/c/zuul/zuul/+/833536 | 17:40 | |
-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 831316: Check if job request is still queued before yielding https://review.opendev.org/c/zuul/zuul/+/831316 | 17:40 | |
-@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 831661: Ignore errors in merge request cleanup https://review.opendev.org/c/zuul/zuul/+/831661 | 18:09 | |
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 833676: Clear branch cache on full reconfiguration https://review.opendev.org/c/zuul/zuul/+/833676 | 18:35 | |
-@gerrit:opendev.org- Pedro Moritz de Carvalho Neto proposed: [zuul/nodepool] 833683: Add override-container-command and override-container-args config https://review.opendev.org/c/zuul/nodepool/+/833683 | 19:44 | |
-@gerrit:opendev.org- Pedro Moritz de Carvalho Neto proposed: [zuul/nodepool] 833683: Add override-container-command and override-container-args config https://review.opendev.org/c/zuul/nodepool/+/833683 | 19:47 | |
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 833676: Clear branch cache on full reconfiguration https://review.opendev.org/c/zuul/zuul/+/833676 | 20:07 | |
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 833676: Clear branch cache on full reconfiguration https://review.opendev.org/c/zuul/zuul/+/833676 | 20:09 | |
-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/nodepool] 830524: Add provider image config to statemachine image upload https://review.opendev.org/c/zuul/nodepool/+/830524 | 20:40 | |
-@gerrit:opendev.org- Pedro Moritz de Carvalho Neto proposed: [zuul/nodepool] 833683: Add override-container-command and override-container-args config https://review.opendev.org/c/zuul/nodepool/+/833683 | 21:26 | |
@clarkb:matrix.org | 833676 hit the javascript heap out of memory error running npx browserlist@latest command | 22:09 |
@clarkb:matrix.org | https://github.com/browserslist/browserslist#browsers-data-updating seems to be what triggers it | 22:13 |
@clarkb:matrix.org | Is our browser list config in web/package.json too complicated for node to handle in memory? | 22:14 |
@clarkb:matrix.org | Looks like react-scripts build invokes that npx command so I'm not sure how to pass the flag to node saying please use a more memory | 22:18 |
@clarkb:matrix.org | To be fair it appears to already have almost 2GB of memory | 22:18 |
@clarkb:matrix.org | I'm not sure how much more it needs to update browser configs, but maybe the search space for that is quite large | 22:19 |
@clarkb:matrix.org | I'm not sure what we should be doing here. | 22:23 |
@clarkb:matrix.org | hrm maybe update our nodejs version? | 22:23 |
-@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul] 833691: Update nodejs runtime to version 16 https://review.opendev.org/c/zuul/zuul/+/833691 | 22:30 | |
@clarkb:matrix.org | I think ^ is a good next step either way as upstreams are unlikely to want to debug old nodejs for us? | 22:30 |
-@gerrit:opendev.org- Pedro Moritz de Carvalho Neto proposed: [zuul/nodepool] 833683: Add override-container-command and override-container-args config https://review.opendev.org/c/zuul/nodepool/+/833683 | 22:36 | |
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 821711: Add IBM Cloud VPC driver https://review.opendev.org/c/zuul/nodepool/+/821711 | 22:37 | |
@clarkb:matrix.org | corvus: left a thought on https://review.opendev.org/c/zuul/zuul/+/833676 if you want to look at it quickly before approving | 22:50 |
@jim:acmegating.com | Clark: thanks; replied and approved | 22:59 |
@jim:acmegating.com | Clark: and yeah, i definitely spent some time in the repl checking my assumptions on that one :) | 23:00 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!