| JayF | Hey all, simple change to enable basic testing of nova sharding, I'd love to get it in this cycle so I can get it running in CI before we brnach for stable/2023.2 in Ironic :) https://review.opendev.org/c/openstack/devstack/+/894459 | 17:34 |
|---|---|---|
| gmann | +2. lgtm | 17:42 |
| JayF | gmann: is Needed-By a project convention or something gerrit understands? I linked that change in a comment but your request is only the second time I've seen that (I noticed it on a change the other day; the election stack for ATC->AC iirc) | 17:43 |
| dansmith | it's for reviewers | 17:43 |
| JayF | it's a pretty nice convention | 17:43 |
| gmann | yeah, for reviewer to know | 17:43 |
| opendevreview | Jay Faulkner proposed openstack/devstack master: [nova][ironic] Support configuring 1 shard on n-cpu https://review.opendev.org/c/openstack/devstack/+/894459 | 17:44 |
| JayF | whelp I didn't know it was landing, I fixed the commit message and chucked it outta the gate /o\ | 17:44 |
| dansmith | I got it | 17:44 |
| gmann | JayF: yeah, it was not need to update but no issue | 17:44 |
| clarkb | note gerrit doesn't even understand depends-on. Thats all happenign in zuul | 18:21 |
| JayF | ack; that's what I effectively meant but you are correct I should use better terminology :D | 19:24 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!