| opendevreview | cid proposed openstack/ironic master: Migrates Inspector rules over to Ironic https://review.opendev.org/c/openstack/ironic/+/918303 | 00:28 |
|---|---|---|
| opendevreview | cid proposed openstack/ironic master: Apply Rules: Migrate Inspection Rules https://review.opendev.org/c/openstack/ironic/+/939183 | 00:28 |
| opendevreview | cid proposed openstack/ironic master: Testing: inspection rules migration https://review.opendev.org/c/openstack/ironic/+/939189 | 00:28 |
| opendevreview | Steve Baker proposed openstack/ironic-specs master: Graphical Console Support https://review.opendev.org/c/openstack/ironic-specs/+/938526 | 04:44 |
| opendevreview | Steve Baker proposed openstack/ironic master: WIP Add novnc-proxy service https://review.opendev.org/c/openstack/ironic/+/939191 | 04:48 |
| opendevreview | Steve Baker proposed openstack/ironic master: WIP add VNC auth type to novnc-proxy https://review.opendev.org/c/openstack/ironic/+/939192 | 04:48 |
| opendevreview | Steve Baker proposed openstack/ironic master: WIP create idrac graphical console driver https://review.opendev.org/c/openstack/ironic/+/939193 | 04:48 |
| rpittau | good morning ironic! o/ | 07:41 |
| opendevreview | cid proposed openstack/ironic master: Resolve portgroup deletion failure https://review.opendev.org/c/openstack/ironic/+/939085 | 08:25 |
| opendevreview | Verification of a change to openstack/ironic unmaintained/yoga failed: [Stable Only] pin virtualbmc/sushy-tools/ironic-tempest-plugin to last released tag https://review.opendev.org/c/openstack/ironic/+/939099 | 09:18 |
| kubajj | TheJulia: is the ironic-networking meeting tomorrow still over video or is the plan to switch it already to irc? | 10:03 |
| opendevreview | Andrew Bonney proposed openstack/networking-generic-switch master: Add support for Neutron routed segments https://review.opendev.org/c/openstack/networking-generic-switch/+/939211 | 10:17 |
| janders | TheJulia I have a node servicing related question: what's your thinking about dealing with cases of servicing failures and in particular what is the recommended way of backing out of such cases to get the node back to normal operations if servicing can complete successfully for whatever reason? | 10:25 |
| janders | I was thinking ABORT but that will get us / keep us in error state while I'd want to be able to get back to ACTIVE | 10:26 |
| janders | (at least that's my understanding of what ABORT will do) | 10:26 |
| janders | dtantsur suggsted perhaps passing empty service steps could be a tool used in whatever we may need/want to do | 10:29 |
| janders | but before we get too deep into this we figured we'll check in with you | 10:29 |
| opendevreview | cid proposed openstack/ironic master: API: Inspection rules migration https://review.opendev.org/c/openstack/ironic/+/939217 | 10:31 |
| janders | looking at state transitions in https://docs.openstack.org/ironic/latest/_images/states.svg doesn't make me feel great, IIUC we'd need to go through RESCUE and UNRESCUE to go back to ACTIVE which feels quite complicated (but there may be a reason) | 10:33 |
| opendevreview | cid proposed openstack/ironic master: Apply Rules: inspection rules migration https://review.opendev.org/c/openstack/ironic/+/939218 | 10:35 |
| opendevreview | cid proposed openstack/ironic master: Testing: migration inspection rules https://review.opendev.org/c/openstack/ironic/+/939219 | 10:36 |
| opendevreview | Merged openstack/ironic unmaintained/yoga: [Stable Only] pin virtualbmc/sushy-tools/ironic-tempest-plugin to last released tag https://review.opendev.org/c/openstack/ironic/+/939099 | 11:59 |
| opendevreview | Merged openstack/ironic stable/2023.2: Calculate missing checksum for file:// based images https://review.opendev.org/c/openstack/ironic/+/938619 | 11:59 |
| JayF | janders: The thing to think about is if a service fails, ironic has to be work to make sure the node is able to be given back to the tenant... Think about things like network switching | 13:07 |
| JayF | So even if we implemented the transition you speak of, there would be plenty of situations where the node would still be stuck | 13:07 |
| dtantsur | JayF: yeah, this is what made me think about using empty service steps: this would involve the whole machinery | 13:38 |
| dtantsur | problem: it can fail again if we cannot boot IPA | 13:38 |
| dtantsur | I wonder if the initial plan was to have something similar to unrescue | 13:38 |
| TheJulia | janders: I was always thinking empty service steps would bethe way | 14:20 |
| TheJulia | we could add unservice to explicitly do the breakout actions to bring the node back to active, for some reason the empty steps list resonates with me | 14:22 |
| JayF | Unservice just doesn't make sense verbally. Service is not a stable state that we should be in that we should need to go away from. | 14:28 |
| JayF | If anything, hook up a service failed to active transition with a verb cancel or something | 14:28 |
| TheJulia | that could also work. | 14:28 |
| TheJulia | i just explicitly remember thinking about the no step case a long time ago | 14:29 |
| * TheJulia grumbles about a need to reboot because work laptop is registering random mouse clicks and won't scroll this morning | 14:29 | |
| * TheJulia suspects a "CAT" has something to do with it | 14:29 | |
| opendevreview | Riccardo Pittau proposed openstack/metalsmith master: CI: Remove metalsmith legacy jobs https://review.opendev.org/c/openstack/metalsmith/+/933154 | 14:31 |
| opendevreview | Elod Illes proposed openstack/ironic-python-agent unmaintained/yoga: [CI] Remove centos-8 based jobs https://review.opendev.org/c/openstack/ironic-python-agent/+/938306 | 15:34 |
| opendevreview | Verification of a change to openstack/ironic master failed: trivial: remove xclarity remenent https://review.opendev.org/c/openstack/ironic/+/935973 | 15:35 |
| TheJulia | JayF: have we disabled CI jobs on ironic-lib yet? | 15:57 |
| JayF | it doesn't make sense to edit it at all until the fixes land | 15:57 |
| JayF | in case we have cve | 15:57 |
| JayF | er, deprecations land | 15:58 |
| TheJulia | oreI'm suspecting once we land the ironic change, those jobs won't provide valid resultsanym | 15:58 |
| TheJulia | err | 15:58 |
| TheJulia | stupid mouse | 15:58 |
| TheJulia | I'm worried once merged, the jobs won't provide anyvalid results | 15:58 |
| JayF | they won't, and I'll disable and deprecate the repo then | 15:59 |
| dtantsur | yeah, realistically we should close the master branch at this point | 15:59 |
| JayF | I've already looked into the work needed for that | 15:59 |
| dtantsur | and start any backports from the last supported stable branch | 15:59 |
| TheJulia | dtantsur: ++ | 15:59 |
| dtantsur | (what tripleo did back then) | 15:59 |
| JayF | I'm happy to take these actions once the ironic_lib removals merge | 16:01 |
| JayF | and I've laid out time for it | 16:01 |
| TheJulia | dtantsur: regarding mdns and likely the old metrics driver, and other similar "seemingly never used" features, it might make sense to make a poll and solicit operator feedback to make a data driven decision. At best, we find users, at worst we have data to justify removing. Plus side: the poll is free feature advertising and also might help break impression molds of "that hasn't changed since 2015" | 16:03 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP OCI container adjacent artifact support https://review.opendev.org/c/openstack/ironic/+/937896 | 16:04 |
| * TheJulia goes to reboot before first meeting of the day. | 16:05 | |
| opendevreview | cid proposed openstack/ironic master: API: Inspection rules migration https://review.opendev.org/c/openstack/ironic/+/939217 | 16:05 |
| opendevreview | cid proposed openstack/ironic master: Apply Rules: inspection rules migration https://review.opendev.org/c/openstack/ironic/+/939218 | 16:05 |
| opendevreview | cid proposed openstack/ironic master: Testing: migration inspection rules https://review.opendev.org/c/openstack/ironic/+/939219 | 16:05 |
| rpittau | we should probably also announce deprecation/removal of ironic-lib to the mailing list, if it was not already done | 16:05 |
| JayF | rpittau: Unless you wanna own that, I'll take care of al lthat | 16:11 |
| JayF | basically trying to one-man the ironic-lib deprecation at this point | 16:12 |
| rpittau | :D | 16:14 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/937948 can be approved, I'll get the IPA patch revised for review comments once my morning meetings are over | 16:14 |
| JayF | maybe by EOW we can have ironic-lib buried | 16:14 |
| rpittau | just approved that | 16:15 |
| JayF | good stuff, and when I move hints stuff in IPA to a new module as suggested, I can follow up on the ironic change to make it be in a similarly shaped module for easy syncing | 16:16 |
| * TheJulia glares at work laptop and wonders if it is close enough to a refresh to get a new one | 16:30 | |
| opendevreview | Merged openstack/ironic unmaintained/2023.1: Calculate missing checksum for file:// based images https://review.opendev.org/c/openstack/ironic/+/938621 | 16:52 |
| opendevreview | Merged openstack/ironic master: Migrate ironic_lib to ironic https://review.opendev.org/c/openstack/ironic/+/937948 | 17:34 |
| rpittau | good night! o/ | 17:37 |
| JayF | woohoo thanks for the approval and have a good night o/ | 17:52 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP OCI container adjacent artifact support https://review.opendev.org/c/openstack/ironic/+/937896 | 18:27 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Automatic zstd detection and decompression... https://review.opendev.org/c/openstack/ironic/+/938904 | 18:29 |
| TheJulia | you know your getting into a pretty good place with a patch when you make some major changes and it works first try. | 19:18 |
| JayF | that usually sends me into a spiral of making CERTAIN my changes made it into the test environment lol | 19:19 |
| TheJulia | I already verified that ;) | 19:20 |
| TheJulia | i have a driver_internal_info['image_source'] value now which was in my last revision ;) | 19:20 |
| TheJulia | *should* just be able to trigger rebuild and a full re-eval should occur... once my tinycore ipa vm is done downloading | 19:21 |
| TheJulia | .... surprisingly slow | 19:21 |
| TheJulia | then again, not optimized for speed... it is optimized for size | 19:21 |
| JayF | I wouldn't mind someone pairing with me on the "make a small not-tinyipa image" stuff | 19:23 |
| JayF | because it seems borderline impossible to get something that small which isn't similarly "build it yourself" | 19:23 |
| JayF | I'm mainly curious what a more reasonable target is because "as small as tinyipa" is not reasonable | 19:23 |
| TheJulia | tinyipa only that that size because it cheats by doing evil things like... removing the source and only running compiled bytecode | 19:25 |
| TheJulia | and carrying no firmware | 19:25 |
| TheJulia | "firmware" | 19:25 |
| TheJulia | Also, I'm fighting nested virt ultimately which is always slow | 19:25 |
| TheJulia | just... painfully slow to move the 10gb of this image across | 19:26 |
| JayF | I can do no firmware trivially | 19:26 |
| JayF | the "remove everything but bytecode" is probably a bridge too far for most normal cases, if we can get away with it | 19:26 |
| TheJulia | yeah | 19:28 |
| JayF | I think really answering the question of "what's the actual size it needs to dip below" would probably help in the endeavor | 19:28 |
| TheJulia | You got things to slightly smaller than coreos right? | 19:28 |
| JayF | I got it smaller than the centos one | 19:28 |
| JayF | but not even in the ballpark of tinyipa | 19:29 |
| JayF | which is partially why I punted until I got better requirements | 19:29 |
| JayF | (and also because it's nice to do something more doable after spinning wheels a while) | 19:29 |
| TheJulia | basically, if we can get a gentoo based vm running in under 1g, which means your maximum ramdisk size is somewhere around 300 megabytes compressed, it might work | 19:30 |
| JayF | I might be there, or close | 19:30 |
| JayF | I don't have a test image still available at hand | 19:32 |
| JayF | but I think I was in the ballpark of high 400s / low 500s MB | 19:32 |
| opendevreview | Merged openstack/ironic unmaintained/zed: Calculate missing checksum for file:// based images https://review.opendev.org/c/openstack/ironic/+/938622 | 19:32 |
| JayF | I'm going to update the ironic-lib patch now but then I'll get back to a "it's working but not quite small enough" spot | 19:32 |
| JayF | but it sounds like the litmus test is "can you deploy cirros with image streaming enabled on a 1GB VM" | 19:33 |
| TheJulia | reminds me, we're also going to need to update ipa-b in general | 19:33 |
| JayF | for what? | 19:34 |
| TheJulia | regarding ironic-lib | 19:34 |
| JayF | oh, yeah | 19:34 |
| JayF | good point | 19:34 |
| * JayF adds it to his list | 19:34 | |
| JayF | I'm going to swap away from IRC so I can update that IPA patch, and write the IPA-B, ironic-lib and governance patches :) | 19:34 |
| TheJulia | cool cool | 19:35 |
| TheJulia | I'm doing lunch | 19:35 |
| TheJulia | and depending on the wind will govern if I get into authentication after my post-lunch meeting | 19:35 |
| TheJulia | or just say "no way I'm going to make progress with this wind and go do a grocery run | 19:35 |
| * TheJulia files a bug against IPA | 19:43 | |
| JayF | cardoe: are you going to finish the set of "run lint via pre-commit" patches? I just noticed that IPA isn't migrated | 19:49 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: Remove dependency on ironic-lib https://review.opendev.org/c/openstack/ironic-python-agent/+/937743 | 19:59 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Migrate documentation from ironic-lib https://review.opendev.org/c/openstack/ironic/+/939275 | 21:00 |
| opendevreview | Jay Faulkner proposed openstack/ironic-lib master: Deprecate ironic-lib master branch https://review.opendev.org/c/openstack/ironic-lib/+/939277 | 21:11 |
| JayF | PRs going up to various governance/release/requirements repos for ironic-lib retirement as well | 21:17 |
| opendevreview | Jay Faulkner proposed openstack/ironic-lib master: Deprecate ironic-lib master branch https://review.opendev.org/c/openstack/ironic-lib/+/939277 | 21:20 |
| opendevreview | Jay Faulkner proposed openstack/ironic-lib master: Deprecate ironic-lib master branch https://review.opendev.org/c/openstack/ironic-lib/+/939277 | 21:25 |
| janders | TheJulia JayF thank you for your responses. In this case, what would be the "correct" way of passing empty service steps? I tried a couple of variations yesterday: https://paste.opendev.org/show/bWJTGWVkjY0b4EItfWez/ | 21:45 |
| janders | only the last one gets accepted (others fail validation) but then fails on attempt to run servicing (trying to reproduce to share the exact message) | 21:46 |
| JayF | janders: That matches what I would expect, I wasn't sure what syntax Julia was thinking :) | 21:46 |
| JayF | Maybe a bug we need to fix up to allow that | 21:46 |
| JayF | (& document it) | 21:46 |
| JayF | rpittau: I suspect TC will want your +1 here https://review.opendev.org/c/openstack/governance/+/939278 | 21:50 |
| janders | here is the error I get in the last service step call: https://paste.opendev.org/show/bpxKCR7kEQP758B5UGTA/ | 21:58 |
| janders | let's see what Julia thinks and we'll come up with some plan | 21:59 |
| janders | gotta take the little one for a med checkup now, back in the arvo | 21:59 |
| JayF | https://review.opendev.org/q/topic:%22ironic-lib-deprecation%22 | 22:21 |
| JayF | oh yeah, I need an IPA-B patch, huh | 22:21 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent-builder master: Deprecate ironic-lib https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/939288 | 22:26 |
| TheJulia | janders: do you have hold/unhold ? ;) | 22:29 |
| TheJulia | in term of a serivce step | 22:29 |
| JayF | OH | 22:32 |
| JayF | lol | 22:32 |
| JayF | I forget we have noop steps | 22:32 |
| JayF | step: wait 1 second | 22:32 |
| JayF | servicing done | 22:32 |
| TheJulia | yeah, wait is a good one too | 22:33 |
| cardoe | JayF: yes. I must have missed that whoops. | 23:20 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
