| rpittau | good morning ironic! o/ | 08:10 |
|---|---|---|
| rpittau | adam-metal3: I went through the discussion here and from my experience I can unfortunately confirm the general slowness with iDRAC job queue and more weirdness, although that usually happens when the BMC firmware version has something "wrong" | 08:21 |
| rpittau | being a 16G I suggest to verify that you have iDRAC9 v7.00.60.00 first to exclude any known issue | 08:21 |
| adam-metal3 | rpittau, thanks for the inofo ! | 08:47 |
| SvenKieske | hi o/ can we get this still in for the release? it has one +2 https://review.opendev.org/c/openstack/networking-baremetal/+/903995 I didn't see any networking-baremetal release just yet, did I miss that? | 10:02 |
| opendevreview | Kamil Gustab proposed openstack/sushy master: Adds Ethernet Interfaces for manager. https://review.opendev.org/c/openstack/sushy/+/909451 | 11:41 |
| iurygregory | good morning Ironic | 11:46 |
| dtantsur | SvenKieske: it's probably too early for networking-* releases | 12:02 |
| SvenKieske | dtantsur: I was referring to the fact that a release was demanded "ASAP" from the release team, see: https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/EGD6IP3W2KFT33HJDLJNK2KM5ZXFFWM3/#IIFQQPNMEAOUUXF7XVDQJK7E5FBKKMOM | 12:13 |
| SvenKieske | so I just wanted to make sure that patch makes the release, seems there are 7 days to go | 12:14 |
| dtantsur | Hmm, maybe my internal clock is off, and you're right. JayF and rpittau are the people to talk to. | 12:19 |
| opendevreview | cid proposed openstack/ironic master: Fix data length exceeding limit error https://review.opendev.org/c/openstack/ironic/+/910973 | 12:22 |
| opendevreview | cid proposed openstack/ironic master: Fix data length exceeding limit error https://review.opendev.org/c/openstack/ironic/+/910973 | 12:25 |
| rpittau | SvenKieske: I think we're still in time, I've approved the patch | 13:11 |
| rpittau | once that merges we'll add the release | 13:14 |
| SvenKieske | rpittau: thank you :) | 13:14 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [PoC] Verify agent TLS certificate and provide diagnostics https://review.opendev.org/c/openstack/ironic/+/911116 | 13:31 |
| TheJulia | Good morning | 14:16 |
| rpittau | hey TheJulia :) | 14:16 |
| TheJulia | Quick and easy backport: https://review.opendev.org/q/I33e76929ca9bf7869b3b4ef4d6501e692cf0a922 | 14:21 |
| TheJulia | And a tempest fix: https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/909939 :) | 14:21 |
| jrosser | maybe this is old news, but it looks interesting https://github.com/IBM/DPFS | 14:31 |
| TheJulia | neat | 14:32 |
| dtantsur | wow | 14:32 |
| TheJulia | Not exactly what nvidia would prefer you do, *and* on an end of life card, but cool | 14:32 |
| jrosser | i stumbled across that becasue i have a use case for ironic + large shared dataset but we have multi-tenancy | 14:34 |
| jrosser | so it's all a bit /o\ to implement nicely | 14:34 |
| rpittau | interesting | 14:34 |
| jrosser | btw if i'm missing an obvious answer for multitenancy vs. ironic vs. shared filesystems thats self-service for users then do let me know :) | 14:39 |
| TheJulia | for filesystems, that is a bit of a niche thing. :) | 14:40 |
| TheJulia | For networking, I'd <3 us to be able to program DPUs | 14:40 |
| TheJulia | to specific networks, but to do it securely, we would need to have OOB programming access | 14:41 |
| TheJulia | and... well... the likely only way that will happen is if someone just submits patches or existing contributors suddenly have some cards | 14:41 |
| TheJulia | or a company or two partners on the topic | 14:41 |
| opendevreview | cid proposed openstack/ironic master: Fix data length exceeding limit error https://review.opendev.org/c/openstack/ironic/+/910973 | 14:44 |
| jrosser | TheJulia: do you think most of that programming is already acheived here? https://docs.openstack.org/neutron/latest/admin/ovn/smartnic_dpu.html | 14:47 |
| TheJulia | nope | 14:47 |
| jrosser | ah interesting | 14:48 |
| TheJulia | wlel | 14:48 |
| TheJulia | well, actually, that looks like it has been updated | 14:48 |
| TheJulia | so that *might* do the needful | 14:50 |
| TheJulia | just nobody has likely pieced it together in a non-virtualized context to see if the physical function is presented to the host sufficently for use | 14:50 |
| opendevreview | Merged openstack/networking-baremetal master: don't force amqp_auto_delete for quorum queues https://review.opendev.org/c/openstack/networking-baremetal/+/903995 | 15:06 |
| opendevreview | Sven Kieske proposed openstack/networking-baremetal stable/2023.2: don't force amqp_auto_delete for quorum queues https://review.opendev.org/c/openstack/networking-baremetal/+/911611 | 15:08 |
| opendevreview | Sven Kieske proposed openstack/networking-baremetal stable/2023.1: don't force amqp_auto_delete for quorum queues https://review.opendev.org/c/openstack/networking-baremetal/+/911612 | 15:08 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-lib stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/ironic-lib/+/911870 | 15:35 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-lib stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/ironic-lib/+/911871 | 15:35 |
| opendevreview | OpenStack Release Bot proposed openstack/metalsmith stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/metalsmith/+/911872 | 15:35 |
| opendevreview | OpenStack Release Bot proposed openstack/metalsmith stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/metalsmith/+/911873 | 15:35 |
| opendevreview | OpenStack Release Bot proposed openstack/metalsmith master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/metalsmith/+/911874 | 15:35 |
| opendevreview | OpenStack Release Bot proposed openstack/python-ironic-inspector-client stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/911875 | 15:35 |
| opendevreview | OpenStack Release Bot proposed openstack/python-ironic-inspector-client stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/911876 | 15:35 |
| opendevreview | OpenStack Release Bot proposed openstack/python-ironic-inspector-client master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/911877 | 15:35 |
| opendevreview | OpenStack Release Bot proposed openstack/python-ironicclient stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/python-ironicclient/+/911882 | 15:35 |
| opendevreview | OpenStack Release Bot proposed openstack/python-ironicclient stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/python-ironicclient/+/911884 | 15:35 |
| opendevreview | OpenStack Release Bot proposed openstack/python-ironicclient master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/python-ironicclient/+/911886 | 15:35 |
| opendevreview | OpenStack Release Bot proposed openstack/sushy stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/sushy/+/911887 | 15:36 |
| opendevreview | OpenStack Release Bot proposed openstack/sushy stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/sushy/+/911888 | 15:36 |
| opendevreview | OpenStack Release Bot proposed openstack/sushy master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/sushy/+/911889 | 15:36 |
| rpittau | going to aske release of networking-baremetal once https://review.opendev.org/c/openstack/networking-baremetal/+/907552 merges | 15:54 |
| opendevreview | Merged openstack/ironic-lib stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/ironic-lib/+/911870 | 16:07 |
| opendevreview | Verification of a change to openstack/ironic-lib stable/2024.1 failed: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/ironic-lib/+/911871 | 16:07 |
| opendevreview | Merged openstack/sushy stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/sushy/+/911887 | 16:10 |
| opendevreview | Merged openstack/sushy master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/sushy/+/911889 | 16:13 |
| opendevreview | Merged openstack/python-ironic-inspector-client stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/911875 | 16:15 |
| opendevreview | Verification of a change to openstack/python-ironic-inspector-client stable/2024.1 failed: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/911876 | 16:15 |
| opendevreview | Merged openstack/ironic master: fix errors messaging around network mappings https://review.opendev.org/c/openstack/ironic/+/909938 | 16:17 |
| opendevreview | Merged openstack/sushy stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/sushy/+/911888 | 16:19 |
| opendevreview | Merged openstack/python-ironicclient stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/python-ironicclient/+/911882 | 16:20 |
| opendevreview | Verification of a change to openstack/python-ironicclient stable/2024.1 failed: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/python-ironicclient/+/911884 | 16:20 |
| opendevreview | Merged openstack/ironic stable/2023.2: neutron: do not error if no cleaning/provisioning on launch https://review.opendev.org/c/openstack/ironic/+/910310 | 16:21 |
| opendevreview | Verification of a change to openstack/ironic stable/2023.1 failed: neutron: do not error if no cleaning/provisioning on launch https://review.opendev.org/c/openstack/ironic/+/910311 | 16:21 |
| opendevreview | Verification of a change to openstack/ironic-tempest-plugin master failed: Invoke tests with fake interfaces https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/909939 | 16:27 |
| JayF | I think our gate on master is busted in a new wya | 16:34 |
| JayF | *way | 16:34 |
| TheJulia | Please submit liquor. | 16:35 |
| JayF | > E: Unable to locate package ksmtuned | 16:35 |
| TheJulia | Please submit liquor. | 16:35 |
| JayF | https://zuul.opendev.org/t/openstack/build/9a66172336da4a7880310599d2501297 | 16:35 |
| JayF | ironic-tempest-functional-python3 | 16:35 |
| TheJulia | Please submit liquor. | 16:35 |
| * TheJulia has decided, next career will be that of a bending robot | 16:35 | |
| JayF | TheJulia: https://usercontent.irccloud-cdn.com/file/w6GFb07u/image.png | 16:36 |
| clarkb | JayF: the actual error is ab it higher than that | 16:36 |
| clarkb | Could not resolve 'mirror.dfw.rackspace.opendev.org' | 16:36 |
| JayF | oh phew | 16:37 |
| JayF | I was already digging myself down a hole | 16:37 |
| JayF | thanks for the rope | 16:37 |
| clarkb | which can happen for one of a million reasons. The simplest of which is dns resolution locally got broken | 16:37 |
| JayF | wait, I thought clouds had 100% reliable networks | 16:37 |
| clarkb | oh I know what the problem is | 16:38 |
| JayF | and that no problems in computer science were dns | 16:38 |
| JayF | oooh, a not-random-failure is more interesting for sure :D | 16:38 |
| clarkb | arg this was an unexpected side effect of us testing new rax credentials to deal with their new MFA mandate that will break normal API access | 16:38 |
| clarkb | we have a second set of credentials with a different cloud name and that changes everything down the line apparently. It should be mirror.dfw.rax.opendev.org | 16:39 |
| TheJulia | JayF: did you ever watch futurama? | 16:39 |
| JayF | TheJulia: I have no strong feelings one way or the other. | 16:39 |
| clarkb | I'm going to revert the change but jobs will need to be rechecked. Give me a few | 16:39 |
| JayF | TheJulia: (that's a futurama reference, btw) | 16:40 |
| TheJulia | JayF: :) | 16:40 |
| TheJulia | JayF: I was thinking about the time Fry was admitted to an robot insane asylumn | 16:41 |
| TheJulia | "cloud networking is 100% reliable" | 16:41 |
| JayF | TheJulia: why do you think I took April 1-5 off? Gotta practice my stabbin' | 16:41 |
| JayF | TheJulia: that is like, top 5 futurama episodes for me. I also love the part (I think it's same episode?) where bender is like "I had a nightmare! There were all these 1s and 0s .... but I think I saw a 2!" "No Bender, there's no such thing as 2!" | 16:42 |
| clarkb | JayF: ok I manually reverted the change and jobs starting on new test nodes should be good now | 16:44 |
| opendevreview | Alex Welsh proposed openstack/bifrost master: Improve downloaded deployment image support https://review.opendev.org/c/openstack/bifrost/+/884888 | 16:46 |
| *** jamesdenton_ is now known as jamesdenton | 16:48 | |
| jamesdenton | Good morning, Ironic. Question about scheduling -- if a user wanted to target a specific baremetal node for a deployment, is the best/only way to do that via a unique trait and flavor? Or is there another way? | 16:49 |
| JayF | So, lets contextualize | 16:49 |
| JayF | you can do that *via Ironic APIs directly* | 16:49 |
| JayF | but I assume you mean scheduling thru nova? | 16:50 |
| jamesdenton | nova, yes | 16:50 |
| JayF | Then, yes. I've worked places that carried a patch to do it, but it's really not part of the OpenStack model to do it that way | 16:50 |
| jamesdenton | agreed | 16:50 |
| JayF | Using Ironic standalone, you can target whatever node you want (and we don't really have a concept of scheduling) | 16:50 |
| jamesdenton | and i don't think we'd want to carry a patch, either | 16:50 |
| JayF | Like, I suspect direct Ironic API access might be your answer | 16:50 |
| JayF | but the complexity of your environment can change that up some | 16:51 |
| JayF | we support "BMaaS" without having Nova involved, node.owner sets project that owns the node, you can even use node.lessee to temporarily "lease" it out to another project | 16:51 |
| jamesdenton | Direct Ironic is likely the long term approach, but for now we're using Nova. Although, it might simplify things to not use Nova, i just haven't had time | 16:51 |
| JayF | it just gets hairy if you are making all the coordinating API calls yourself to do the needful in neutron and all that | 16:51 |
| JayF | well, I'm trying to say you don't have to choose | 16:51 |
| JayF | take nodes for nova, mark them as "owned" by a project, ensure nova user for ironic is scoped to that project | 16:52 |
| jamesdenton | ok | 16:52 |
| JayF | nodes for direct-deploy, set owner to another project (preferably end user project), then use those separately | 16:52 |
| JayF | This is not only "it works" but we explicitly support this mixed mode of operation | 16:52 |
| jamesdenton | oh i gotcha | 16:52 |
| jamesdenton | in the direct-deploy model, is it still possible leverage neutron and ml2 driver? | 16:53 |
| JayF | yes; I think you might have to make some calls to neutron that nova would make yourself | 16:53 |
| dtantsur | JayF: nit: we do have a concept of scheduling, it's called allocation API :) | 16:54 |
| JayF | jamesdenton: dtantsur and Julia do know more about this stuff than I do, I just think it's cool ;) | 16:54 |
| jamesdenton | it IS cool | 16:54 |
| -opendevstatus- NOTICE: Jobs that fail due to being unable to resolve mirror.dfw.rackspace.opendev.org can be rechecked. This error was an unexpected side effect of some nodepool configuration changes which have been reverted. | 16:54 | |
| JayF | jamesdenton: basically end result is you could have "N" separate Novas, just segregated by project, none of which can see nodes from the other | 16:55 |
| JayF | e.g. openstack baremetal node list while scoped to project "X" only sees nodes with node.owner/node.lessee=X | 16:56 |
| JayF | IDK if anyone's deployed it this way, but you shuold be able to go full baremetal multitenancy with Ironic only | 16:56 |
| JayF | (meaning, no Nova, not, no other openstack-y bits) | 16:56 |
| jamesdenton | ok, i understand that model. I think the difficulty here for the teams involved is the mental shift of "this server is device 123456 and it has IP address x.x.x.x" and that lives in CORE tooling. There's no separation of the logical device number and physical device. In this new model, there would be. We don't have a way to reflect that relationship *in our existing tooling* even though the Ironic API can provide that info | 16:58 |
| jamesdenton | that will change (and is changing) but a slow change, indeed | 16:59 |
| JayF | jamesdenton: you wanna have a sync chat about this? You know I spent almost a decade at Rackspace so I have some insights at least to how things might fit | 17:02 |
| JayF | jamesdenton: getting Rackspace back using and contributing to Ironic is well worth a half hour of my time :D | 17:03 |
| jamesdenton | we definitely can | 17:03 |
| jamesdenton | i will email you an invite and bring Kevin along | 17:03 |
| JayF | fun | 17:03 |
| JayF | if you do something especially cool, amybe I'll mail you this classic Rackspace OnMetal bear flag LOL | 17:04 |
| jamesdenton | OnMetal is still rocking along | 17:05 |
| jamesdenton | :D | 17:05 |
| jamesdenton | just like you left it | 17:05 |
| JayF | do I wanna know ...yeah | 17:05 |
| JayF | that's mainly what I'm afraid of lol | 17:05 |
| JayF | I remember when I worked on a team building custom linux LBs/firewalls. I left that team to do other things at the company. Three years later I see someone working on them, ask them to run `uname -a` and see nobody has built a patched kernel for them in that period of time. | 17:06 |
| JayF | I'd rather something I built get torn down than bitrot :( | 17:06 |
| rpittau | good night! o/ | 17:06 |
| JayF | o/ | 17:06 |
| jamesdenton | <insert side-eyed monkey meme here> | 17:06 |
| JayF | lol exactly | 17:06 |
| dtantsur | JayF: hey, not sure if you saw my ping the other day, this is the TLS skew detection PoC we talked about: https://review.opendev.org/c/openstack/ironic/+/911116 | 17:10 |
| dtantsur | I mean, one of the approaches. I don't know how to extract information from an SSLError | 17:10 |
| JayF | I saw the ping, but was getting massively squirreled yesterday and it fell outta my brain | 17:10 |
| * dtantsur is fascinated by "squirreled" | 17:11 | |
| JayF | like a golden retreiever, when they see a squirrel, their attention just goes BOOM over to that | 17:11 |
| JayF | it's a good analogy for how some people are easily distracted. and yesterday was a day of shiny objects distracting me at every turn (like, actual things I needed to do, just little that I *planned* to do) | 17:12 |
| * dtantsur has just learned he is a golden retriever | 17:12 | |
| JayF | we all are, we all are | 17:12 |
| TheJulia | wait, what do I know more about? what should I hide from?!?!?!?! | 17:13 |
| TheJulia | dtantsur: wait, what?! | 17:13 |
| dtantsur | squirrels? golden retrievers? | 17:14 |
| * dtantsur got distracted and does not know any more | 17:14 | |
| TheJulia | squirrel?!? where!?! | 17:14 |
| TheJulia | jamesdenton: did nova kill the undocumented "place on compute node" hint via the command line? some folks at one point in time would just put the ComputeNode hostname which matched the ironic node uuid, but this long in the past. | 17:16 |
| JayF | honestly TheJulia's talk at last summit about Ironic RBAC | 17:22 |
| JayF | is basically required watching for this topic | 17:22 |
| TheJulia | who? what? huh? why? | 17:22 |
| TheJulia | Please hold, I need to go place a thumper and ride a sand worm away from my computer. | 17:23 |
| jamesdenton | TheJulia I do think that still exists as --availability-zone nova:<compute> .... might be worth trying that | 17:31 |
| TheJulia | so, I'm still kind of worried about the RBAC topic | 17:32 |
| JayF | in what way? | 17:32 |
| TheJulia | well, the mention of rbac has me wanting to go join the fremen | 17:33 |
| * TheJulia might have been flying an ornothopter the last few nights and working on landing | 17:34 | |
| * TheJulia needs to map the wind break to a break button | 17:34 | |
| TheJulia | s/wind/wing/ | 17:34 |
| JayF | I recommend you, also, take a week off soon LOL | 17:36 |
| TheJulia | I've been thinking about doing so | 17:36 |
| opendevreview | Verification of a change to openstack/networking-baremetal master failed: Bump hacking to 6.1.0 https://review.opendev.org/c/openstack/networking-baremetal/+/907552 | 17:43 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Fix error handling in the virtual media attach API https://review.opendev.org/c/openstack/ironic/+/911952 | 17:49 |
| dtantsur | a small patch for a nasty defect if anyone has time ^^ | 17:52 |
| TheJulia | ohhh ahhh | 17:52 |
| TheJulia | so whatever is calling do_virtual_media_attach was not catching any exceptions/errors? | 17:53 |
| dtantsur | TheJulia: it's called inside spawn_worker | 17:54 |
| TheJulia | oh | 17:54 |
| dtantsur | there is basically nothing above it | 17:54 |
| TheJulia | sigh | 17:54 |
| JayF | yeah | 17:54 |
| TheJulia | yeah, that would do it | 17:54 |
| JayF | confirmed that myself just now, too | 17:54 |
| dtantsur | I got really bamboozled while testing my gophercloud patch for this feature | 17:54 |
| dtantsur | Everything is successful and then... silence? :) | 17:55 |
| JayF | > exc_info=not isinstance(exc, exception.IronicException) | 17:55 |
| JayF | is the only thing weirding me out about that change | 17:55 |
| TheJulia | that is about as bad as a customer who I have a call with tomorrow who the media attaches, but the hardware boots via pxe | 17:55 |
| JayF | why don't we want exc_info if it's an IronicException? | 17:55 |
| dtantsur | JayF: We do it quite often. What it says is "log traceback if unexpected (= not ironic-produced) exception" | 17:55 |
| JayF | ack | 17:55 |
| JayF | that is what I assumed re: desire but wanted to validate | 17:55 |
| dtantsur | think about stuff like KeyError. Without a traceback it's impossible to debug | 17:56 |
| dtantsur | (it's error message is the missing key) | 17:56 |
| JayF | my question would've been pointed more towards "why not always true" | 17:56 |
| JayF | not the other way around :D | 17:56 |
| dtantsur | Ideally, our exceptions should be self-explanatory without a traceback :) otherwise, putting them in last_error is no so helpful | 17:56 |
| dtantsur | Python's tracebacks are helpful but also very verbose, especially now that they also print traceback for the whole exception chain | 17:57 |
| TheJulia | https://bugs.launchpad.net/ironic/+bug/2056248 <-- I wonder if such an old version of eventlet would cause an issue like that | 18:26 |
| JayF | I'm more thinking, bugs in graceful shutdown, perhaps | 18:27 |
| TheJulia | oh, conductor-0 was working, lost connectivity to the db | 18:28 |
| JayF | yeah | 18:28 |
| JayF | the errors from greenlet are timeouts on the socket | 18:28 |
| JayF | this just looks like networking done wrong? | 18:28 |
| TheJulia | yeah | 18:28 |
| JayF | Now, I'd buy we might have some bugs where | 18:28 |
| JayF | if you spin up conductor, no sql server | 18:28 |
| JayF | sql server comes online/available some time later | 18:28 |
| JayF | that's a weird case we don't test and I wouldn't be stunned to find dragons around those cases; but this ain't it | 18:28 |
| TheJulia | well, it won't start without it | 18:32 |
| TheJulia | but if you bounce the server, depending on log level/settings, it might never log anything depending on the sql command line | 18:32 |
| TheJulia | also interestingly enough, they have an error regarding Portgroup model being unknown. | 18:32 |
| TheJulia | all very weird. | 18:32 |
| Sandzwerg[m] | <TheJulia> "jamesdenton: did nova kill the..." <- Still works for us but you need to know the availability zone of the node and the node UUID | 20:39 |
| TheJulia | Sandzwerg[m]: awesome, thanks | 20:40 |
| Sandzwerg[m] | We are still on xena, but from my understand stand shouldn' t change. As for OpenStack a single node is a hypervisor and, at least with enough rights, you can force a instance on a specific hypervisor | 20:41 |
| Sandzwerg[m] | *ironic node | 20:42 |
| TheJulia | Yeah, now that you mention it, I think you have to be an admin to use that option | 21:12 |
| jamesdenton_ | well, i'm not so sure -- a nova availability-zone-list only shows the conductor nodes themselves as eligible hosts, not the individual baremetal nodes. Although, those show up in 'openstack hypervisor list' | 21:17 |
| jamesdenton_ | throwback: https://docs.openstack.org/project-deploy-guide/tripleo-docs/zed/provisioning/node_placement.html | 21:42 |
| JayF | cid has a couple of his first (okay, okay, it's #2 and #3) Ironic changes up for review, https://review.opendev.org/c/openstack/ironic/+/909851 and https://review.opendev.org/c/openstack/ironic/+/910973 -- both are pretty simple reviews if you have a moment :) | 23:23 |
| JayF | that node_history.user change would be good to get in C, TBH | 23:23 |
| iurygregory | I probably missed since they weren't with the hashtag, will add to my list | 23:29 |
| JayF | I added ht to at least one of em | 23:29 |
| JayF | but they weren't V+1 until this morning | 23:29 |
| iurygregory | ack | 23:29 |
| JayF | also sorta | 23:29 |
| JayF | if there's something at that level we need fixed | 23:29 |
| iurygregory | I'm checking 851 | 23:29 |
| JayF | let me/cid know and we can line it up | 23:29 |
| iurygregory | ack | 23:29 |
| JayF | he's basically blown through my planned MLH projects :D | 23:29 |
| JayF | TheJulia: https://review.opendev.org/c/openstack/ironic/+/901090 needs a rebase now; but I'm unsure if the changes here have already been wrapped in elsewhere; lmk if they are good to land and I can rebase if I'm still workin | 23:31 |
| opendevreview | Jay Faulkner proposed openstack/python-ironicclient master: Fix functional test run https://review.opendev.org/c/openstack/python-ironicclient/+/907094 | 23:33 |
| TheJulia | yeah, a rebase will be needed | 23:33 |
| JayF | I think IPA unit tests might be in pain too | 23:34 |
| JayF | going to make a note to look in the morning | 23:34 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/900846 needs an added release note, I might make that change today if I have time as well | 23:35 |
| JayF | trying to get the stragglers in since we gotta cut a release soonish | 23:36 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Add redfish https boot CI job https://review.opendev.org/c/openstack/ironic/+/901090 | 23:38 |
| opendevreview | Verification of a change to openstack/ironic-python-agent master failed: Force constraints when installing a package during tox test https://review.opendev.org/c/openstack/ironic-python-agent/+/908787 | 23:41 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Fix Reopen Web Console Duplicate Sol Session https://review.opendev.org/c/openstack/ironic/+/900846 | 23:41 |
| JayF | I still chose to +2 900846 even though I changed it, it was only a release note. JFYI | 23:42 |
| JayF | 901090 I will likely +2 as well, but wanna see CI results (I rebased that one) | 23:42 |
| opendevreview | Verification of a change to openstack/ironic stable/2023.1 failed: neutron: do not error if no cleaning/provisioning on launch https://review.opendev.org/c/openstack/ironic/+/910311 | 23:46 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
