| arozman | Hi Ironic! | 07:47 |
|---|---|---|
| rpittau | good morning ironic! o/ | 08:50 |
| opendevreview | kamlesh chauvhan proposed openstack/sushy master: Retry on iDRAC SYS518 errors for all requests https://review.opendev.org/c/openstack/sushy/+/864911 | 11:02 |
| iurygregory | good morning Ironic | 11:11 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-inspector bugfix/10.9: Fix upper-constraints in unit tests https://review.opendev.org/c/openstack/ironic-inspector/+/866337 | 14:19 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-inspector bugfix/10.9: CI: Various fixes https://review.opendev.org/c/openstack/ironic-inspector/+/860170 | 14:20 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-inspector bugfix/10.9: CI: Various fixes https://review.opendev.org/c/openstack/ironic-inspector/+/860170 | 14:23 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-inspector bugfix/10.9: CI: Various fixes https://review.opendev.org/c/openstack/ironic-inspector/+/860170 | 14:24 |
| JayF | rpittau: o/ | 14:54 |
| JayF | rpittau: mind if I point you at another that's been perplexing me? | 14:54 |
| rpittau | JayF: hey! sure! :) | 14:54 |
| JayF | lemme get in | 14:54 |
| rpittau | sorry I hijacked that change :P | 14:54 |
| JayF | https://review.opendev.org/c/openstack/ironic-inspector/+/860166 | 14:55 |
| JayF | never apologize for unbreaking my broken stuff lol | 14:55 |
| JayF | I pushed like forty million patches to fix the zuul queue issue; I've been curating them through as I have time | 14:55 |
| JayF | most of the ones left are hard cases; or at least stuff IDK how to fix :) | 14:55 |
| opendevreview | Merged openstack/ironic stable/yoga: Add support auth protocols for iRMC https://review.opendev.org/c/openstack/ironic/+/864467 | 15:05 |
| opendevreview | Merged openstack/ironic stable/xena: Add support auth protocols for iRMC https://review.opendev.org/c/openstack/ironic/+/865947 | 15:17 |
| opendevreview | Merged openstack/ironic bugfix/19.0: Add support auth protocols for iRMC https://review.opendev.org/c/openstack/ironic/+/865946 | 15:17 |
| rpittau | JayF: wow that's on train, I've rechecked it, will have a look later | 15:51 |
| JayF | yep; I think it's the only one outstanding for train, too | 15:52 |
| hjensas | no json_rpc for inspector? I need rabbits? | 15:52 |
| hjensas | dtantsur: ^ ? | 15:52 |
| rpittau | ah fricking requirements | 15:59 |
| JayF | yuuup | 16:00 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-inspector bugfix/10.9: CI: Various fixes https://review.opendev.org/c/openstack/ironic-inspector/+/860170 | 16:02 |
| opendevreview | Merged openstack/ironic-inspector bugfix/10.9: CI: Various fixes https://review.opendev.org/c/openstack/ironic-inspector/+/860170 | 16:45 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-inspector stable/train: CI: Zuul no longer respects queue https://review.opendev.org/c/openstack/ironic-inspector/+/860166 | 17:16 |
| rpittau | JayF: this ^ should work | 17:16 |
| rpittau | good night! o/ | 17:16 |
| JayF | thanks! | 17:16 |
| opendevreview | Merged openstack/ironic-inspector stable/train: CI: Zuul no longer respects queue https://review.opendev.org/c/openstack/ironic-inspector/+/860166 | 19:34 |
| JayF | rpittau: FWIW; remaining changes that need help for getting queue param stuff fixed thruout https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/860184 https://review.opendev.org/c/openstack/ironic-inspector/+/860171 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/860181 https://review.opendev.org/c/openstack/ironic-lib/+/860176 | 19:50 |
| JayF | https://review.opendev.org/c/openstack/ironic-python-agent/+/860190 | 19:50 |
| JayF | That looks like a lot, but believe me, 5 is super awesome considering how many have been outstanding :D | 19:51 |
| * JayF going to recheck all those to see if we get lucky or, more likely, get fresh logs | 19:51 | |
| JayF | I added an item to Monday's agenda; we'll need to get ACLs updated to be able to delete branches and tag EOL for old bugfix branches; and we gotta talk about how we're going to do that (I'm going to suggest a second, much smaller core group since the actions are extra-destructive) | 19:59 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: API support for CRUD node.shard https://review.opendev.org/c/openstack/ironic/+/866235 | 22:40 |
| JayF | I'm going live w/my office hours in ~6 minutes (2300 UTC). youtube.com/jayofdoom -- bring any patches you need reviewed or questions, otherwise I'll be working live on the next patch in ^^ that series | 22:55 |
| TheJulia | JayF: that will need a policy check and a new policy added to ironic/common/policy.py to enable operators to lock out all ability to change the field. I would be happy to help with that on Monday | 22:58 |
| JayF | I'm trying to figure it out on office hours | 23:12 |
| JayF | wanna get the node side done/done and if that's required, it's required | 23:12 |
| JayF | TheJulia: about to push a change with policy implemented (I think?). It'll need to be manually tested with the other stuff at the end | 23:20 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: API support for CRUD node.shard https://review.opendev.org/c/openstack/ironic/+/866235 | 23:22 |
| TheJulia | So turns out there is a habit testing framework for rbac already ;) | 23:23 |
| TheJulia | Test_ACL and the yanks | 23:24 |
| JayF | is it magic? | 23:24 |
| JayF | I looked in there and saw magic that implied it's either tested automatically or I couldn't dervive how it was setup lol | 23:25 |
| TheJulia | Eh, ddt testing | 23:26 |
| TheJulia | So semi magical until you wrap your head around it | 23:26 |
| JayF | the only acronym I know of all the ones you used | 23:26 |
| JayF | is ACL | 23:26 |
| JayF | lol | 23:26 |
| TheJulia | DDT is data driven test | 23:31 |
| JayF | ack; I'll dig into it | 23:32 |
| JayF | I may take you up on the Monday offer; but to help with /v1/shards | 23:32 |
| JayF | just to get me started | 23:32 |
| JayF | weird to have an endpoint that doesn't return an object-proper | 23:32 |
| JayF | I don't think we have many examples of that | 23:32 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!