*** stendulker has joined #openstack-ironic | 01:18 | |
*** stendulker has quit IRC | 01:41 | |
*** jelabarre-rh has joined #openstack-ironic | 02:35 | |
*** jelabarre-rh has quit IRC | 03:47 | |
*** stendulker has joined #openstack-ironic | 04:26 | |
*** ricolin_ has joined #openstack-ironic | 04:32 | |
*** ricolin has quit IRC | 04:36 | |
*** ricolin_ is now known as ricolin | 04:36 | |
*** stendulker has quit IRC | 04:57 | |
*** stendulker has joined #openstack-ironic | 05:00 | |
arne_wiebalck | Good morning janders and ironic! | 05:43 |
---|---|---|
janders | hey arne_wiebalck o/ | 05:43 |
cenne | good morning ironic ! | 05:44 |
cenne | hey janders , arne_wielbalck | 05:44 |
arne_wiebalck | hey cenne o/ | 05:45 |
janders | hey cenne o/ | 06:04 |
arne_wiebalck | dtantsur: https://storyboard.openstack.org/#!/story/2008937 | 06:33 |
iurygregory | morning janders arne_wiebalck cenne and Ironic o/ | 07:11 |
cenne | morning iurygregory o/ | 07:12 |
arne_wiebalck | hi iurygregory o/ | 07:12 |
*** rpittau|afk is now known as rpittau | 07:44 | |
rpittau | good morning ironic! o/ | 07:44 |
iurygregory | morning rpittau o/ | 07:45 |
rpittau | hey iurygregory :) | 07:45 |
*** tosky has joined #openstack-ironic | 07:48 | |
*** tosky has quit IRC | 07:48 | |
janders | hey iurygregory rpittau o/ | 07:51 |
rpittau | hey janders :) | 07:51 |
iurygregory | janders, o/ | 07:53 |
*** lourot has quit IRC | 07:54 | |
*** lourot has joined #openstack-ironic | 07:55 | |
*** lucasagomes has joined #openstack-ironic | 08:07 | |
*** tosky has joined #openstack-ironic | 08:08 | |
*** derekh has joined #openstack-ironic | 08:14 | |
opendevreview | liuzhuangzhuang proposed openstack/ironic master: API: Node Tags API https://review.opendev.org/c/openstack/ironic/+/793573 | 08:49 |
opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Follow-up to Include bios registry fields in bios API https://review.opendev.org/c/openstack/ironic/+/793907 | 09:22 |
iurygregory | bfournie, rpittau dtantsur ^ | 09:22 |
*** sshnaidm|off is now known as sshnaidm | 09:57 | |
*** sshnaidm has joined #openstack-ironic | 09:58 | |
dtantsur | morning ironic | 10:03 |
rpittau | good morning dtantsur :) | 10:04 |
iurygregory | morning dtantsur | 10:04 |
dtantsur | thank you arne_wiebalck! | 10:05 |
opendevreview | Merged openstack/sushy-tools master: Add support for returning message and attribute registries https://review.opendev.org/c/openstack/sushy-tools/+/790361 | 10:05 |
*** dtantsur_ has joined #openstack-ironic | 10:13 | |
*** dtantsur has quit IRC | 10:17 | |
*** dtantsur_ is now known as dtantsur | 10:28 | |
janders | hey dtantsur o/ | 10:28 |
*** dtantsur_ has joined #openstack-ironic | 10:32 | |
*** dtantsur has quit IRC | 10:36 | |
*** dtantsur_ is now known as dtantsur | 10:36 | |
*** Luzi has joined #openstack-ironic | 10:57 | |
*** hjensas is now known as hjensas|afk | 10:59 | |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: Move provision states documentation to the user guide https://review.opendev.org/c/openstack/ironic/+/793946 | 11:00 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: Trivial: fix an outdated link https://review.opendev.org/c/openstack/ironic/+/793947 | 11:05 |
dtantsur | arne_wiebalck: do you know if `smartctl -H` returns a failure exit code? | 11:07 |
dtantsur | seems like yes | 11:18 |
* dtantsur has now read man more thoroughly | 11:18 | |
*** ftarasenko has joined #openstack-ironic | 11:19 | |
dtantsur | actually, it seems like we can run with -a and still get an exit code | 11:20 |
dtantsur | (with -a we'll get more information in the ramdisk logs) | 11:20 |
* dtantsur is reading https://alexander.kirk.at/2013/02/07/munin-smart-plugin-ignore-error-in-the-past/ now | 11:24 | |
*** stendulker has quit IRC | 11:47 | |
opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: [WIP] Check the target root device with smartctl https://review.opendev.org/c/openstack/ironic-python-agent/+/793953 | 11:50 |
dtantsur | arne_wiebalck: tl;dr something like ^^^ | 11:50 |
*** sshnaidm has quit IRC | 11:51 | |
arne_wiebalck | dtantsur: nice, was not aware the return code is a bit mask :) | 11:52 |
dtantsur | arne_wiebalck: yeah, check the manpage, it's quite interesting | 11:52 |
* arne_wiebalck is reading ... | 11:52 | |
arne_wiebalck | "prefail attributes <= threshold" confuses me | 11:56 |
arne_wiebalck | it seems to mean we have crossed the threshold to the failing side of things | 11:57 |
dtantsur | yep. I don't fully understand it either | 11:57 |
arne_wiebalck | or does it mean: there are prefail values, but not yet above the threshold where we should worry | 11:57 |
dtantsur | in theory, if it's non-zero, it's dangerous | 11:58 |
dtantsur | do you have anyone to ask? | 11:58 |
arne_wiebalck | sure | 11:58 |
*** sshnaidm has joined #openstack-ironic | 11:59 | |
arne_wiebalck | it seems to indicate an upcoming failure, general state "impending failure" | 12:02 |
*** jelabarre-rh has joined #openstack-ironic | 12:03 | |
dtantsur | I'm not capturing the difference | 12:03 |
arne_wiebalck | the device has not failed yet, but is expected to fail soon | 12:04 |
arne_wiebalck | I agree, there is danger. The question is: is it dangerous enough? | 12:04 |
arne_wiebalck | For instance, in our deployment we would not replace such a disk. | 12:04 |
arne_wiebalck | So, for us, failing deployment would be a false positive. | 12:05 |
dtantsur | okay, so maybe make it tunable for cleaning and only check bit 3 for deployment? | 12:05 |
arne_wiebalck | sounds good to me | 12:05 |
dtantsur | okay, I'll update it. I probably won't have cycles to work on the cleaning one. | 12:06 |
arne_wiebalck | the health check is maybe sth which should also run as part of the burn-in ... but could also be a second step (which is maybe more flexible) | 12:06 |
arne_wiebalck | I can see if I can add a cleaning step, inspired by the deploy code. | 12:07 |
opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: Refactor: use convert_image from ironic_lib https://review.opendev.org/c/openstack/ironic-python-agent/+/793955 | 12:07 |
dtantsur | arne_wiebalck: do you know if we can actually run smartctl --xall and then only fail if bit 3 is set? | 12:09 |
arne_wiebalck | dtantsur: not sure I get the question ... | 12:10 |
dtantsur | arne_wiebalck: --health has a very short output, pretty much only pass for fail | 12:11 |
dtantsur | this is why I'm calling --xall in case of a failure to leave the full output in the logs | 12:11 |
dtantsur | but I wonder if I can just call it with --xall right away and check the exit code for bit 3? | 12:11 |
arne_wiebalck | ah ... erm ... good question | 12:12 |
arne_wiebalck | any issue with calling --xall all the time, then calling --health | 12:12 |
arne_wiebalck | or do you want logs only when a failure is detected? | 12:13 |
*** tzumainn has joined #openstack-ironic | 12:15 | |
arne_wiebalck | on every deploy is maybe too much | 12:16 |
rpittau | --xall does look like a bit overkill | 12:17 |
arne_wiebalck | yeah | 12:17 |
dtantsur | maybe just --all? | 12:17 |
dtantsur | I want to leave something useful in logs | 12:17 |
arne_wiebalck | I am not sure smartctl computes the return value or if that is sth that xall reports as well somewhere ... | 12:17 |
rpittau | we're probably ok with just --all, let's give that a try ? | 12:21 |
arne_wiebalck | --all and --xall include -H, so the return code should be the same in both cases | 12:24 |
rpittau | yeah | 12:25 |
arne_wiebalck | both meaning: we can run --(x)all and rely on the bit map, no need to call --health | 12:25 |
arne_wiebalck | I think this is my answer to your question dtantsur ;-) | 12:26 |
dtantsur | okie, thanks! | 12:28 |
opendevreview | Merged openstack/bifrost master: Add a benchmark job for ironic https://review.opendev.org/c/openstack/bifrost/+/792705 | 12:53 |
rpittau | see ya later! o/ | 12:57 |
*** rpittau is now known as rpittau|afk | 12:57 | |
*** lmcgann has joined #openstack-ironic | 12:58 | |
*** sdanni has joined #openstack-ironic | 13:00 | |
*** zaneb has joined #openstack-ironic | 13:10 | |
TheJulia | Good morning Ironic | 13:12 |
iurygregory | good morning TheJulia =) | 13:14 |
*** rloo has joined #openstack-ironic | 13:14 | |
*** hjensas|afk has quit IRC | 13:17 | |
*** hjensas has joined #openstack-ironic | 13:17 | |
*** Luzi has quit IRC | 13:21 | |
opendevreview | Merged openstack/ironic master: Follow-up to Include bios registry fields in bios API https://review.opendev.org/c/openstack/ironic/+/793907 | 13:23 |
TheJulia | it breaks my brain a little that it is tuesday | 13:23 |
TheJulia | Also that my first meeting is in a half hour :( | 13:23 |
dtantsur | morning TheJulia | 13:39 |
iurygregory | better Tuesday than Monday I would say =) | 13:43 |
arne_wiebalck | Good morning TheJulia o/ | 13:46 |
opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: Check the target root device with smartctl https://review.opendev.org/c/openstack/ironic-python-agent/+/793953 | 13:56 |
dtantsur | okay, this should be it now ^^ would be good to fit into the release | 13:57 |
*** ftarasenko has quit IRC | 14:11 | |
*** paras333 has joined #openstack-ironic | 14:14 | |
*** kkillsfirst has joined #openstack-ironic | 14:32 | |
*** opendevreview has quit IRC | 14:44 | |
TheJulia | Any chance I can get eyes on https://review.opendev.org/c/openstack/ironic-python-agent/+/793453 ? | 14:46 |
TheJulia | it is of course different and needs to be backported | 14:46 |
iurygregory | TheJulia, looking now | 14:46 |
TheJulia | but the commit message should explain it | 14:46 |
iurygregory | the note does help to explain why we have the change in write-image.sh :D | 14:47 |
iurygregory | +2 | 14:48 |
TheJulia | okay, I've got a downstream blocker that I need to get that merged for and backported :( | 14:51 |
arne_wiebalck | On a tyoical prod node in our deployment, this will reduce the arenas from 32 to 3 ... seems like a drastic reduction ... any known side-effects? | 14:52 |
arne_wiebalck | s/tyoical/typical/ | 14:53 |
TheJulia | 32 for how many cores | 14:54 |
TheJulia | by default, it is 8 * CPU core count * memory footprint | 14:54 |
arne_wiebalck | 4 cores | 14:54 |
TheJulia | no, basically if it is out of arenas, the process should just hold on the lock being released instead of trying to malloc an entirely new arena | 14:54 |
TheJulia | the second arena only gets created when one thread tries to access memory locked space of the first thread | 14:55 |
arne_wiebalck | so, the parallelism will go down? | 14:56 |
TheJulia | on physical cores in a server, that shouldn't really be an issue since you don't have the variability of one core getting other work scheduld on it and a thread getting delayed out of view from the OS | 14:56 |
arne_wiebalck | the controller is a VM | 14:56 |
TheJulia | no, it won't. If your creating additional arenas, then your machine is already on a path to running out of memory or the process running out of memory. | 14:57 |
TheJulia | arne_wiebalck: iscsi deploy? | 14:57 |
arne_wiebalck | iscsi as well as direct | 14:57 |
arne_wiebalck | image conversion so far was an issue when instantiation 10s or 100s of nodes, so I am wondering if this will reduce the number of instances I can create in parallel | 14:58 |
TheJulia | Okay, so we've already got backports ?merged? for ironic-lib w/r/t iscsi deploy because the issue we've been able to identify is should this cross-thread locking start occuring, then the process runs out of memory rapidly | 14:58 |
arne_wiebalck | ok | 14:58 |
TheJulia | and it is really just malloc'ing to get a new arena to abandon the old one | 14:58 |
TheJulia | because the lock was hit | 14:59 |
TheJulia | so it is a memory leak in essence | 14:59 |
arne_wiebalck | ok, I see | 14:59 |
TheJulia | So, it might actually help some of the parallel conversions because you could possibly hit similar cross-thread issues on controllers | 14:59 |
TheJulia | this should keep it under the 1-2gb memory limit for the process which is still needed because qemu-img is also handling an untrusted byte stream and the input data *can* depending on what is present cause it to try and consume as much memory as possible for block reassembly. | 15:00 |
TheJulia | but humans have to be *super* evil to do stuff like that, it is not normal in any regards | 15:00 |
arne_wiebalck | ok, thanks for the explanation! | 15:01 |
*** opendevreview has joined #openstack-ironic | 15:02 | |
opendevreview | Bob Fournier proposed openstack/ironic master: Add a section to redfish doc for BIOS registry fields https://review.opendev.org/c/openstack/ironic/+/793982 | 15:02 |
* arne_wiebalck will start to sprinkle new nodes across conductor groups anyway soon so make use of all controllers in parallel, rather than moving new nodes all into the same group | 15:02 | |
* TheJulia had a database platform that would use cross-cpu shared memory.... and would actually crash on virtual machines because of it. | 15:02 | |
*** zbitter has joined #openstack-ironic | 15:03 | |
dtantsur | TheJulia: FYI when a patch is stuck on a merged dependency, use an additional W+1, recheck won't help | 15:05 |
TheJulia | dtantsur: not all the time, but just kind of hoping it would unwedge it without having to go all the way back through review | 15:06 |
dtantsur | TheJulia: you're a core, you can just add one more W+1 | 15:06 |
dtantsur | it's trickier for non-core people | 15:06 |
* TheJulia is conflicted about just re +2+A it since changing it would also drop all reviews | 15:06 | |
opendevreview | Julia Kreger proposed openstack/ironic master: Add bifrost-benchmark-ironic job https://review.opendev.org/c/openstack/ironic/+/792706 | 15:07 |
TheJulia | done | 15:08 |
dtantsur | TheJulia: you didn't have to reupload the patch.. | 15:08 |
TheJulia | to change the topic to remove the dependency tag which it didn't gracefully, I do | 15:08 |
dtantsur | yeah, but you don't have to remove the dependency (and changing topic doesn't require re-uploading) | 15:09 |
*** opendevreview has quit IRC | 15:09 | |
dtantsur | this is what I'm trying to hint at: adding another W+1 triggers re-verification of the dependency | 15:09 |
*** zaneb has quit IRC | 15:09 | |
dtantsur | even if there is already a W+1 present from another core | 15:09 |
TheJulia | well, at least it is better than github | 15:10 |
dtantsur | no objections :) | 15:10 |
*** opendevreview has joined #openstack-ironic | 15:14 | |
opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: Check the target root device with smartctl https://review.opendev.org/c/openstack/ironic-python-agent/+/793953 | 15:14 |
NobodyCam | Good Morning Ironic folks! | 15:17 |
arne_wiebalck | hey NobodyCam o/ | 15:18 |
NobodyCam | hey hey Good Morning arne_wiebalck o/ | 15:19 |
iurygregory | morning NobodyCam o/ | 15:19 |
NobodyCam | I posted a repo last week with several older hardware drivers for cent8 may be of interest for you | 15:19 |
NobodyCam | morning iurygregory o/ | 15:20 |
arne_wiebalck | NobodyCam: Thanks! It is not so much I cannot get hold of the drivers in general, it is more that they may not exist for newer versions of kernels/OS I use for the IPA image ... which means I would need to pin a successful (older) image to the node ... which will only get me so far as the IPA in that image ages as well. | 15:25 |
NobodyCam | ++ | 15:27 |
arne_wiebalck | While I did have nodes which ended in dracut emergency shells, I did not have nodes yet which I could not clean due to the IPA image not being able to see the disks ... but I guess this may happen at some point. | 15:29 |
TheJulia | eyes on https://review.opendev.org/c/openstack/ironic/+/790142 would be nice :) | 15:34 |
*** stendulker has joined #openstack-ironic | 15:36 | |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: Improve agent_client logging https://review.opendev.org/c/openstack/ironic/+/793995 | 15:46 |
*** lucasagomes has quit IRC | 16:00 | |
opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: Stop accepting duplicated configdrive https://review.opendev.org/c/openstack/ironic-python-agent/+/790471 | 16:07 |
*** zbitter has quit IRC | 16:10 | |
*** zaneb has joined #openstack-ironic | 16:22 | |
*** stendulker has quit IRC | 16:24 | |
iurygregory | TheJulia, it's on my list for today =) I will look after dinner | 16:26 |
TheJulia | Thanks | 16:26 |
dtantsur | g'night folks! | 16:28 |
opendevreview | Merged openstack/ironic-python-agent stable/wallaby: Limit qemu-img execution arenas https://review.opendev.org/c/openstack/ironic-python-agent/+/793453 | 16:30 |
iurygregory | TheJulia, re ^ you need in other stable branches right? if you push the backports I can approve without problems | 16:31 |
arne_wiebalck | iurygregory TheJulia while you're updating stable branches: https://review.opendev.org/c/openstack/ironic-python-agent/+/790672 and https://review.opendev.org/c/openstack/ironic-python-agent/+/790671 ;) | 16:43 |
arne_wiebalck | bye everyone o/ | 16:43 |
*** zaneb has quit IRC | 16:50 | |
*** zaneb has joined #openstack-ironic | 16:51 | |
TheJulia | Do we need to review jam today? | 16:58 |
*** derekh has quit IRC | 17:00 | |
JayF | I won't be at the jam, sorry | 17:01 |
TheJulia | No worries, I'm on endless hold with a government agency at the moment | 17:01 |
*** pas-ha is now known as Guest486 | 17:38 | |
*** jlvillal has quit IRC | 17:56 | |
*** jlvillal has joined #openstack-ironic | 17:57 | |
*** jlvillal has quit IRC | 18:01 | |
*** jlvillal has joined #openstack-ironic | 18:02 | |
*** Guest486 has quit IRC | 18:10 | |
*** pas-ha[m] has joined #openstack-ironic | 18:15 | |
*** rloo has quit IRC | 18:18 | |
*** rloo has joined #openstack-ironic | 18:18 | |
opendevreview | Bob Fournier proposed openstack/ironic master: Trivial: Fix version number in comment https://review.opendev.org/c/openstack/ironic/+/794025 | 18:49 |
*** Gue______ has quit IRC | 19:02 | |
*** Gue______ has joined #openstack-ironic | 19:11 | |
opendevreview | Merged openstack/ironic-python-agent stable/wallaby: Software RAID: RAID the ESPs https://review.opendev.org/c/openstack/ironic-python-agent/+/790671 | 19:11 |
opendevreview | Merged openstack/ironic-python-agent stable/victoria: Software RAID: RAID the ESPs https://review.opendev.org/c/openstack/ironic-python-agent/+/790672 | 19:11 |
opendevreview | Merged openstack/ironic master: Secure RBAC - Efficent node santiziation https://review.opendev.org/c/openstack/ironic/+/790142 | 19:12 |
opendevreview | Merged openstack/ironic master: Add bifrost-benchmark-ironic job https://review.opendev.org/c/openstack/ironic/+/792706 | 19:13 |
opendevreview | Julia Kreger proposed openstack/ironic stable/wallaby: Secure RBAC - Efficent node santiziation https://review.opendev.org/c/openstack/ironic/+/793929 | 19:23 |
opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/victoria: Limit qemu-img execution arenas https://review.opendev.org/c/openstack/ironic-python-agent/+/794030 | 19:26 |
*** rpioso has joined #openstack-ironic | 19:27 | |
iurygregory | MERGE ALL THE THINGS :D | 19:29 |
opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/ussuri: Increase the memory limit for qemu-img https://review.opendev.org/c/openstack/ironic-python-agent/+/793930 | 19:30 |
opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/train: Increase the memory limit for qemu-img https://review.opendev.org/c/openstack/ironic-python-agent/+/793931 | 19:31 |
rpioso | Good morning, ironicers :-) | 19:31 |
opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/ussuri: Limit qemu-img execution arenas https://review.opendev.org/c/openstack/ironic-python-agent/+/794033 | 19:34 |
iurygregory | morning rpioso | 19:34 |
opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/train: Limit qemu-img execution arenas https://review.opendev.org/c/openstack/ironic-python-agent/+/794034 | 19:35 |
iurygregory | chain of patches :D | 19:36 |
rpioso | iurygregory: o/ | 19:36 |
TheJulia | I proposed the memory size backports along with the memory arenas change for ipa down to train | 19:37 |
iurygregory | yeah I noticed | 19:39 |
*** rpioso is now known as rpioso|afk | 19:40 | |
TheJulia | iurygregory: if you want to +2A the memory ulimit changes, I think that will be fine. | 19:40 |
TheJulia | since my backporting them gets considered as a +2 | 19:41 |
iurygregory | TheJulia, yeah doing now | 19:41 |
iurygregory | CI will probably report in 2hrs and I will be around to +W (time to watch netflix :D) | 19:44 |
NobodyCam | anyone happen to know off the tops of there head, how / where we handle dns for a remote conductor group? | 19:48 |
NobodyCam | s/dns/dhcp/ ;p | 19:50 |
*** rpioso|afk is now known as rpioso | 19:52 | |
TheJulia | NobodyCam: neutron connected? or not neutron connected? | 20:00 |
NobodyCam | connected | 20:00 |
NobodyCam | going to need a neutron node in the remote location too :' | 20:01 |
NobodyCam | ( | 20:01 |
*** rpioso is now known as rpioso|afk | 20:07 | |
TheJulia | ml2 integrated... or not? | 20:12 |
NobodyCam | looks like yes | 20:14 |
NobodyCam | do we have any configuration guides for remote conductors? | 20:22 |
NobodyCam | I'm trying to avoid setting up a whole region for the remote location | 20:26 |
TheJulia | not exactly, do you have a dedicated cleaning network for that remote location? | 20:32 |
TheJulia | not exactly being the guide difference | 20:34 |
TheJulia | and is the neutron present for that remote location already aware of its physical networks | 20:34 |
NobodyCam | Neutron is aware of the network, as of me typing this there is no neutron in the remote location | 20:37 |
NobodyCam | was hoping it could be something like install neutron-dhcp-agent on the remote conductor and call it a day | 20:41 |
*** rpioso|afk is now known as rpioso | 20:41 | |
*** dking has joined #openstack-ironic | 20:56 | |
*** dking is now known as Guest504 | 20:56 | |
*** paras333 has quit IRC | 20:59 | |
*** paras333 has joined #openstack-ironic | 20:59 | |
*** jroll has joined #openstack-ironic | 21:00 | |
*** rpioso is now known as rpioso|afk | 21:08 | |
trandles | Question about IPA (I think?): I'm doing a tripleo (RHOSP) install, which means RHEL8. I have a server with the root disk hanging off a megaraid_sas device, which means I have to add back the megaraid_sas driver. I did that in the IPA ramdisk, it sees the drives and installs the overcloud_full image to the correct drive. I'd modified the overcloud_full image with the | 21:18 |
trandles | megaraid_sas device and the full modules.alias file. I've also modified the overcloud_full initrd with the megaraid_sas driver. After deploy the machine reboots, grub is fine, it starts booting but dracut in the initrd can't find the drive. It might not be an IPA issue, but I can't figure out why the initrd installed by IPA doesn't seem to have the driver set up correctly. | 21:18 |
TheJulia | NobodyCam: so technically you'd need a neutron agent that was aware of the physnet and setup on the remote, and as long as the conducotr config points to the proper network and the remote is also aware of it, then it should just work... | 21:29 |
TheJulia | NobodyCam: and by just work I mean your networking stars align | 21:29 |
TheJulia | trandles: did you rebuild the ramdisk image for the partition image? | 21:30 |
NobodyCam | :) woot | 21:30 |
trandles | TheJulia: rebuild the /boot/initramfs-blah-blah in the overcloud_full image? | 21:31 |
TheJulia | NobodyCam: and if your doing switch port toggling, your neutron endpoint would need to be aware of things | 21:31 |
TheJulia | trandles: *and* extract that initramfs from with-in the image | 21:32 |
TheJulia | because your writing a partition image out by default unless your building a whole disk image to deploy | 21:32 |
trandles | TheJulia: I'm using the overcloud_full image that RHAT distributes via RPM | 21:32 |
TheJulia | at which point, the ramdisk needs to be aware of the driver... and I guess dracut might need to know more :\ | 21:32 |
NobodyCam | Thank you TheJulia as always your wisdom is the guiding light I strive to follow | 21:33 |
* TheJulia gives NobodyCam a look | 21:33 | |
NobodyCam | :) | 21:33 |
trandles | I modified that image and also the initramfs that comes with it, but I didn't _replace_ the /boot/initramfs in the image itself. | 21:33 |
trandles | I didn't know if IPA ran dracut to write a new initramfs on the partition as part of its install work. | 21:34 |
trandles | Sounds like no? | 21:34 |
TheJulia | trandles: no, ipa does not run dracut so yes, update the one in the image | 21:34 |
TheJulia | trandles: sounds like you might be running a whole disk iamge? | 21:35 |
TheJulia | We're trying to push osp to using a whole disk image by default | 21:35 |
trandles | TheJulia: ok, thanks for the pointer. And if you can get someone to put those drivers back in RHEL8 you could probably make a reasonable demand for free lifetime of beer from a lot of people. ;) | 21:35 |
kkillsfirst | Hello, I am about to start working with the Ironic repo. Is there a preferred operating system that I should use? Most of experience is in Ubuntu but I am versatile. | 21:42 |
TheJulia | kkillsfirst: ubuntu is good | 21:42 |
TheJulia | kkillsfirst: fwiw, the bulk of upstream CI uses ubuntu | 21:42 |
NobodyCam | I like ubuntu, mainly because Devstack likes Ubuntu | 21:42 |
kkillsfirst | Thank you for the info. :) | 21:44 |
*** rpioso|afk is now known as rpioso | 21:46 | |
*** rpioso is now known as erbarr|afk | 21:47 | |
*** erbarr|afk is now known as rpioso | 21:48 | |
opendevreview | Julia Kreger proposed openstack/ironic master: Only return the requested fields https://review.opendev.org/c/openstack/ironic/+/792274 | 22:05 |
opendevreview | Julia Kreger proposed openstack/ironic master: Set stage for objects to handle selected field lists. https://review.opendev.org/c/openstack/ironic/+/792275 | 22:07 |
*** krkillsfirst has joined #openstack-ironic | 22:20 | |
*** kkillsfirst has quit IRC | 22:20 | |
opendevreview | Merged openstack/ironic master: Trivial: Fix version number in comment https://review.opendev.org/c/openstack/ironic/+/794025 | 22:31 |
opendevreview | Steve Baker proposed openstack/ironic-python-agent stable/train: Add fstab pointer to EFI partition https://review.opendev.org/c/openstack/ironic-python-agent/+/782886 | 22:42 |
TheJulia | stevebaker: so https://review.opendev.org/c/openstack/ironic-python-agent/+/782885 first ? | 22:48 |
stevebaker | TheJulia: yes first | 22:52 |
opendevreview | Julia Kreger proposed openstack/ironic master: WIP: API to pass fields to node object list https://review.opendev.org/c/openstack/ironic/+/792296 | 22:53 |
TheJulia | heh, second should just arrive just after it | 22:55 |
opendevreview | Verification of a change to openstack/ironic-python-agent failed: Increase the memory limit for qemu-img https://review.opendev.org/c/openstack/ironic-python-agent/+/793931 | 22:58 |
opendevreview | Verification of a change to openstack/ironic-python-agent failed: Limit qemu-img execution arenas https://review.opendev.org/c/openstack/ironic-python-agent/+/794034 | 22:58 |
opendevreview | Steve Baker proposed openstack/ironic-python-agent stable/train: Add fstab pointer to EFI partition https://review.opendev.org/c/openstack/ironic-python-agent/+/782886 | 23:00 |
TheJulia | blarg, ci transient connectivity failure | 23:03 |
*** krkillsfirst has quit IRC | 23:05 | |
*** tosky has quit IRC | 23:11 | |
*** sdanni_ has joined #openstack-ironic | 23:46 | |
*** sdanni has quit IRC | 23:51 | |
opendevreview | Verification of a change to openstack/ironic-python-agent failed: Increase the memory limit for qemu-img https://review.opendev.org/c/openstack/ironic-python-agent/+/793930 | 23:59 |
opendevreview | Verification of a change to openstack/ironic-python-agent failed: Limit qemu-img execution arenas https://review.opendev.org/c/openstack/ironic-python-agent/+/794033 | 23:59 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!