| opendevreview | Merged openstack/sushy stable/yoga: Fix volume deletion on newer iDRACs https://review.opendev.org/c/openstack/sushy/+/869312 | 00:29 |
|---|---|---|
| opendevreview | Merged openstack/sushy master: Remove setuptools workaround https://review.opendev.org/c/openstack/sushy/+/869971 | 00:40 |
| opendevreview | Merged openstack/sushy stable/zed: Retry on iDRAC SYS518 errors for all requests https://review.opendev.org/c/openstack/sushy/+/867639 | 00:40 |
| vanou | TheJulia JayF: Thanks for review on https://review.opendev.org/c/openstack/ironic/+/865074 and https://review.opendev.org/c/openstack/ironic/+/865075 . If it's ok, please add workflow +2. Thanks. | 00:45 |
| opendevreview | Merged openstack/ironic-inspector bugfix/10.12: Fix CI https://review.opendev.org/c/openstack/ironic-inspector/+/870581 | 03:33 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/zed: [iRMC] identify BMC firmware version https://review.opendev.org/c/openstack/ironic/+/870880 | 04:56 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/zed: [iRMC] Handle IPMI incompatibility in iRMC S6 2.x https://review.opendev.org/c/openstack/ironic/+/870881 | 04:56 |
| arne_wiebalck | Good morning, Ironic! | 07:28 |
| samuelkunkel[m] | Good morning | 07:34 |
| samuelkunkel[m] | Shall I file a bug before contributing a little bugfix into sushy? Or is it enough to explain the circumstances within the commit message? | 07:35 |
| rpittau | good morning ironic! o/ | 07:41 |
| arne_wiebalck | samuelkunkel[m]: if it is a little fix, the commit message is good enough | 08:01 |
| arne_wiebalck | samuelkunkel[m]: a bug is usually filed when there is a timely gap between the bug and the fix, or if it is important for others to find it | 08:07 |
| samuelkunkel[m] | arne_wiebalck: thanks for the explanation | 08:08 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent bugfix/8.6: Fix CI for bugfix/8.6 https://review.opendev.org/c/openstack/ironic-python-agent/+/868065 | 08:15 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent bugfix/8.6: Drop python2 from bindep.txt https://review.opendev.org/c/openstack/ironic-python-agent/+/867915 | 08:16 |
| rpittau | looking for another review for https://review.opendev.org/c/openstack/bifrost/+/868303 now that the fix on ironic has merged, thanks! | 08:32 |
| opendevreview | Samuel Kunkel proposed openstack/sushy master: bugfix: fix requests verify handling if env is set https://review.opendev.org/c/openstack/sushy/+/870888 | 08:59 |
| opendevreview | Oleksandr Kozachenko proposed openstack/networking-generic-switch stable/zed: Add ArubaOS-CX switch support https://review.opendev.org/c/openstack/networking-generic-switch/+/870896 | 09:28 |
| opendevreview | Merged openstack/bifrost master: Fix CI https://review.opendev.org/c/openstack/bifrost/+/868303 | 11:03 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent bugfix/8.6: Fix CI for bugfix/8.6 https://review.opendev.org/c/openstack/ironic-python-agent/+/868065 | 11:28 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent bugfix/8.6: Drop python2 from bindep.txt https://review.opendev.org/c/openstack/ironic-python-agent/+/867915 | 11:29 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Fix jinja ansible lint error https://review.opendev.org/c/openstack/bifrost/+/866137 | 11:30 |
| kubajj | Hello everyone! | 13:53 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [WIP] [PoC] A metal3 CI job https://review.opendev.org/c/openstack/ironic/+/863873 | 14:33 |
| opendevreview | Merged openstack/bifrost master: Use ansible 6.x https://review.opendev.org/c/openstack/bifrost/+/865969 | 15:06 |
| JayF | vanou: Julia and I must have crossed reviews, lol! I'll approve em | 15:08 |
| vanou | JayF: Thanks a lot :) And if you have time, please take a look at backport patches https://review.opendev.org/c/openstack/ironic/+/870880 and https://review.opendev.org/c/openstack/ironic/+/870881. There is no modification on these. | 15:22 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [WIP] [PoC] A metal3 CI job https://review.opendev.org/c/openstack/ironic/+/863873 | 15:44 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add `service` role RBAC policy support https://review.opendev.org/c/openstack/ironic/+/869614 | 16:00 |
| opendevreview | Merged openstack/bifrost master: Create /run/ironic runtime directory when service starts https://review.opendev.org/c/openstack/bifrost/+/868211 | 16:03 |
| kubajj | dtantsur: How would you go about testing the store_ and get_introspection_data (https://review.opendev.org/c/openstack/ironic/+/870799/4/ironic/drivers/modules/inspect_utils.py#61) I guess these need to be just functional (using the api)? What should be the next step? | 16:10 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Make metrics names a little more consistent https://review.opendev.org/c/openstack/ironic/+/869879 | 16:10 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/wallaby: Suppress Bandit B509 in snmp.py https://review.opendev.org/c/openstack/ironic/+/860847 | 16:16 |
| dtantsur | kubajj: sorry, need to run now, could you ask tomorrow? | 16:29 |
| kubajj | dtantsur: yeah, sure | 16:29 |
| opendevreview | Merged openstack/ironic master: [iRMC] identify BMC firmware version https://review.opendev.org/c/openstack/ironic/+/865074 | 16:32 |
| rpittau | good night! o/ | 16:41 |
| opendevreview | Merged openstack/ironic master: [iRMC] Handle IPMI incompatibility in iRMC S6 2.x https://review.opendev.org/c/openstack/ironic/+/865075 | 16:46 |
| opendevreview | Merged openstack/ironic-python-agent bugfix/8.6: Fix CI for bugfix/8.6 https://review.opendev.org/c/openstack/ironic-python-agent/+/868065 | 17:21 |
| opendevreview | Merged openstack/ironic-python-agent bugfix/8.6: Drop python2 from bindep.txt https://review.opendev.org/c/openstack/ironic-python-agent/+/867915 | 17:21 |
| *** gmann is now known as gmann_afk | 17:29 | |
| *** gmann_afk is now known as gmann | 17:41 | |
| * TheJulia raises an eyebrow on failing unit tests in ci | 17:53 | |
| *** gmann is now known as gmann_afk | 18:06 | |
| NobodyCam | Good morning OpenStack Folks | 18:15 |
| JayF | o/ mornin | 18:17 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Permit node level override of Grub ESP config path https://review.opendev.org/c/openstack/ironic/+/869230 | 18:33 |
| TheJulia | good morning NobodyCam | 18:42 |
| NobodyCam | Good morning TheJulia o/ | 18:43 |
| NobodyCam | Morning JayF | 18:43 |
| *** gmann_afk is now known as gmann | 18:56 | |
| opendevreview | Julia Kreger proposed openstack/ironic master: CI: Fix race prone unit test in networking https://review.opendev.org/c/openstack/ironic/+/870995 | 18:59 |
| TheJulia | so I think ^ will address some of the unit test failures we've had pop up in the last few patches | 18:59 |
| JayF | I assume that's a debugging PR? | 19:05 |
| JayF | or you didn't add the actual fix lol | 19:05 |
| TheJulia | rofl | 19:10 |
| TheJulia | oh wow! | 19:10 |
| TheJulia | I'm out of it | 19:10 |
| opendevreview | Julia Kreger proposed openstack/ironic master: CI: Fix race prone unit test in networking https://review.opendev.org/c/openstack/ironic/+/870995 | 19:11 |
| TheJulia | much better | 19:13 |
| JayF | TheJulia: ...service is not a scope? | 20:40 |
| JayF | are the only scopes project and admin, then? | 20:40 |
| * JayF feels like he took a crazy pill | 20:40 | |
| JayF | was service, at one point, going to be a scope or something? | 20:40 |
| TheJulia | scopes are system, domain, and project | 20:41 |
| JayF | domain ? | 20:41 |
| TheJulia | roles are in projects | 20:41 |
| TheJulia | don't worry, we all take the crazy pills some days :) | 20:41 |
| JayF | what is domain | 20:41 |
| TheJulia | it is a keystone only construct for managing projects | 20:41 |
| TheJulia | we don't need to know/think/support it afaik | 20:42 |
| JayF | so you're adding a service role | 20:42 |
| TheJulia | at least, that is the impression I've gained after talking with the keystone folks | 20:42 |
| JayF | to both project and system scopes ? | 20:42 |
| TheJulia | yes | 20:42 |
| TheJulia | in our model, a service account can have project scoped rights or not project scoped rights, so it only made sense to populate both | 20:43 |
| JayF | yeah, this makes sense | 20:43 |
| JayF | for some reason, I had it in my head that service was a scope | 20:43 |
| JayF | which never really made sense for obvious reasons | 20:43 |
| TheJulia | yeah.... | 20:43 |
| TheJulia | I suspect some old timer keystone folks would table flip at the idea, tbh | 20:44 |
| TheJulia | (of it not being a scope) | 20:44 |
| TheJulia | but yeah. :( | 20:44 |
| TheJulia | stevebaker[m]: When you have a few minutes, you have a question on https://review.opendev.org/c/openstack/bifrost/+/807220/ which you can kind of see highlighted with a quick glance at https://review.opendev.org/c/openstack/bifrost/+/807220/32..45 | 21:26 |
| stevebaker[m] | TheJulia: the comment about the disappearing release note? | 21:30 |
| TheJulia | yeah | 21:31 |
| stevebaker[m] | I found it | 21:33 |
| opendevreview | Steve Baker proposed openstack/bifrost master: Copy shim and grub into tftp and http directories https://review.opendev.org/c/openstack/bifrost/+/849247 | 21:33 |
| opendevreview | Steve Baker proposed openstack/bifrost master: Remove enable_uefi_ipxe https://review.opendev.org/c/openstack/bifrost/+/849248 | 21:33 |
| opendevreview | Steve Baker proposed openstack/bifrost master: Support PXE network boot with grub https://review.opendev.org/c/openstack/bifrost/+/807220 | 21:33 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: DB & Object layer for node.shard https://review.opendev.org/c/openstack/ironic/+/864236 | 21:42 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: API support for CRUD node.shard https://review.opendev.org/c/openstack/ironic/+/866235 | 21:42 |
| TheJulia | JayF: looks like the apiref is still unhappy on the shard api change | 22:03 |
| JayF | I'll look at the test when it goes kaput | 22:04 |
| JayF | trying to figure out /v1/ports?node_shard=lol,cats,bbq | 22:04 |
| TheJulia | wait, how did we go from lol, cats, and *then* to bbq | 22:05 |
| TheJulia | ?! | 22:05 |
| TheJulia | Inquiring minds want some tasty ribs | 22:05 |
| JayF | lolcats wtf bbqcakes; etc used to be like, textual memeing back in the day | 22:06 |
| TheJulia | roflcopter | 22:09 |
| JayF | exactly | 22:09 |
| *** dmellado_ is now known as dmellado | 22:31 | |
| opendevreview | Jay Faulkner proposed openstack/ironic master: API support for CRUD node.shard https://review.opendev.org/c/openstack/ironic/+/866235 | 23:48 |
| JayF | hopefully that fixes apiref; the tox target is busted for me locally it seems | 23:48 |
| JayF | aha, I think I fixed it; I was running tox itself from inside a venv | 23:52 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!