| opendevreview | Steve Baker proposed openstack/networking-generic-switch master: Create service plugin to configure security groups https://review.opendev.org/c/openstack/networking-generic-switch/+/949834 | 03:41 |
|---|---|---|
| opendevreview | Steve Baker proposed openstack/networking-generic-switch master: Support security groups in netmiko base class https://review.opendev.org/c/openstack/networking-generic-switch/+/949835 | 03:41 |
| opendevreview | Steve Baker proposed openstack/networking-generic-switch master: Add security group support to netmiko_cisco_nxos https://review.opendev.org/c/openstack/networking-generic-switch/+/952314 | 03:41 |
| opendevreview | Steve Baker proposed openstack/networking-generic-switch master: Add security group support to netmiko_sonic https://review.opendev.org/c/openstack/networking-generic-switch/+/955252 | 03:41 |
| opendevreview | Steve Baker proposed openstack/networking-generic-switch master: WIP Add security group support to ovs https://review.opendev.org/c/openstack/networking-generic-switch/+/956519 | 03:41 |
| opendevreview | Steve Baker proposed openstack/networking-generic-switch master: Enhance doc generation https://review.opendev.org/c/openstack/networking-generic-switch/+/957300 | 03:41 |
| rpittau | good morning ironic! o/ | 05:25 |
| queensly[m] | Good morning o/ | 06:02 |
| darkhackernc | morning \o | 06:03 |
| opendevreview | Verification of a change to openstack/ironic master failed: api: Add schema for allocations API (requests) https://review.opendev.org/c/openstack/ironic/+/945218 | 08:44 |
| opendevreview | Elod Illes proposed openstack/ironic-prometheus-exporter master: DNM: gate health test https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/957361 | 09:41 |
| opendevreview | Elod Illes proposed openstack/ironic-tempest-plugin master: DNM: gate health test https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/957362 | 09:42 |
| opendevreview | Elod Illes proposed openstack/metalsmith master: DNM: gate health test https://review.opendev.org/c/openstack/metalsmith/+/957366 | 09:43 |
| opendevreview | Elod Illes proposed openstack/python-ironic-inspector-client master: DNM: gate health test https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/957392 | 09:49 |
| opendevreview | Elod Illes proposed openstack/python-ironicclient master: DNM: gate health test https://review.opendev.org/c/openstack/python-ironicclient/+/957393 | 09:49 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Fix servicing abort to respect abortable flag https://review.opendev.org/c/openstack/ironic/+/957189 | 12:29 |
| janders | TheJulia when you're online I have a couple questions about https://review.opendev.org/c/openstack/ironic/+/956972/3 | 12:33 |
| janders | It works for me overall. Other than consensus of the verb (I think we settled on unservice) is there anything else we need? | 12:33 |
| janders | WDYT about https://review.opendev.org/c/openstack/ironic/+/957189 ? - I pushed this to see if it delays aborting servicing till after the step completes - seems to do the job (and blocks the ability to reboot when node is in "service wait" while the BIOS is actually updating). | 12:33 |
| janders | thanks in advance! | 12:34 |
| dtantsur | Folks, I'll be out until next Wednesday, see you then o/ | 12:49 |
| TheJulia | janders: time/capacity to cycle back to it is all I need at this point | 13:07 |
| TheJulia | janders: does the logic around https://review.opendev.org/c/openstack/ironic/+/957189/2/ironic/conductor/manager.py#1533 mirror other abort logic paths? | 13:11 |
| TheJulia | Yeah, I guess that works. | 13:20 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Optional indirection API use https://review.opendev.org/c/openstack/ironic/+/956504 | 13:45 |
| TheJulia | rpittau, cid: a review pass on eventlet stuff would be greatly appreciated while dmitry is out. I'd really like to get some of this stuff rolling towards merging | 14:14 |
| rpittau | I'm actually also be out until Monday :D | 14:16 |
| rpittau | I'm going offline now, I will check it on Monday | 14:16 |
| TheJulia | Okay, have a great weekend | 14:16 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Revert "ci: temporary metal3 integration job disable" https://review.opendev.org/c/openstack/ironic/+/956953 | 14:18 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Clean-up misc eventlet references https://review.opendev.org/c/openstack/ironic/+/955632 | 14:18 |
| cid | TheJulia. I will make sure to review that today. tks. | 14:26 |
| TheJulia | cid: thanks | 14:26 |
| TheJulia | Looks like CI is super slammed today. | 15:05 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic-prometheus-exporter master: Move bifrost cs9 job to non-voting https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/957454 | 17:15 |
| opendevreview | Merged openstack/ironic master: api: Add schema for allocations API (requests) https://review.opendev.org/c/openstack/ironic/+/945218 | 17:49 |
| opendevreview | Merged openstack/networking-generic-switch master: Enhance doc generation https://review.opendev.org/c/openstack/networking-generic-switch/+/957300 | 18:13 |
| opendevreview | Stephen Finucane proposed openstack/ironic master: api: Add schema for allocations API (responses) https://review.opendev.org/c/openstack/ironic/+/928921 | 18:27 |
| TheJulia | cardoe: if you have some spoons, eventlet stuff could use some eyes since it is also going to (pick 2) [epic, awesome, scary] | 18:31 |
| opendevreview | cid proposed openstack/ironic master: Add periodic cleanup of stale conductors https://review.opendev.org/c/openstack/ironic/+/956500 | 20:53 |
| opendevreview | cid proposed openstack/ironic master: Add periodic cleanup of stale conductors https://review.opendev.org/c/openstack/ironic/+/956500 | 21:01 |
| janders | TheJulia noted, big thanks for looking into this | 22:22 |
| TheJulia | I've started working on an unrescue patch, fwiw | 22:23 |
| TheJulia | but obviously, that is a heavier lift | 22:23 |
| TheJulia | I should be able to post that tomorrow | 22:23 |
| janders | w/r/t https://review.opendev.org/c/openstack/ironic/+/957189/2/ironic/conductor/manager.py#1533 yes, it does mirror cleaning with one exception being it goes servicing > service failed > active (first fail, then abort). Aborting clean doesn't go through that intermediate failed state. | 22:24 |
| cardoe | TheJulia: absolutely tonight. I’m my wife’s assistant and bag boy for her work so on PTO but I’ll make time. | 22:27 |
| TheJulia | cardoe: no worries, thanks! | 22:27 |
| TheJulia | go back to that gear bag, that stuff is heavy! | 22:27 |
| TheJulia | :) | 22:27 |
| JayF | looks like we need to pay some love to IPE CI https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/957361 before release | 22:33 |
| JayF | https://review.opendev.org/c/openstack/metalsmith/+/957366 metalsmith, too | 22:34 |
| JayF | iury already looked at ipe: https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/957454 | 22:37 |
| TheJulia | we need to stop trying to push metalsmith along | 22:45 |
| JayF | > Please note the Metalsmith project is in maintenance mode and it's functionality is slowly being introduced into Ironic's API and usual clients. No additional features are expected to be added to metalsmith directly. | 22:48 |
| JayF | is in the readme | 22:48 |
| JayF | not sure if that counts as deprecation-enough for us to drop the deliverable this cycle | 22:48 |
| JayF | but I don't use it or really care about it at all | 22:48 |
| TheJulia | Yeah, I only care about wallaby and if upstream is closed then its fine | 22:51 |
| JayF | I'll hit the list | 22:52 |
| opendevreview | Steve Baker proposed openstack/ironic master: Always rebuild sphinx env on doc build https://review.opendev.org/c/openstack/ironic/+/957471 | 22:56 |
| TheJulia | Thanks | 23:35 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!