| kubajj | good morning ironic! | 06:49 |
|---|---|---|
| opendevreview | Aija Jauntēva proposed openstack/sushy master: Capture requests errors https://review.opendev.org/c/openstack/sushy/+/853209 | 07:04 |
| *** RamonaRautenberg[m] is now known as RamonaBeermann[m] | 07:55 | |
| dtantsur | happy monday folks | 09:14 |
| kubajj | hey dtantsur | 09:19 |
| ftarasenko | gm ironic! have question for you about mdraid that is created on server by client (not by IPA). When client decommission this server, it goes to cleaning, but IPA sees mdraid and tries to clean it instead of physical disks. Can delete_configuration for mdraid be explicitly enabled while automated cleaning? arne_wiebalck: do you clean md devices with automated clean or you have different workflow? | 09:29 |
| iurygregory | good morning Ironic | 10:43 |
| dtantsur | morning iurygregory | 10:58 |
| TheJulia | good morning | 13:10 |
| opendevreview | Julia Kreger proposed openstack/ironic master: CI: Changes to support Anaconda CI jobs https://review.opendev.org/c/openstack/ironic/+/849587 | 13:10 |
| iurygregory | good morning TheJulia | 13:14 |
| TheJulia | Did everyone have a good weekend? | 13:15 |
| TheJulia | ftarasenko: can it today... no. With the model of raid configuration, I'm not sure we can run explicit delete_configuration, but maybe it would be okay if the raidset is torn down if the agent knows nothing of it based upon the configuration from ironic?!? | 13:19 |
| iurygregory | I did | 13:27 |
| iurygregory | re - good weekend | 13:27 |
| arne_wiebalck | ftarasenko: we do it during auto-cleaning | 13:28 |
| arne_wiebalck | ftarasenko: we have our own h/w manager to do this | 13:28 |
| arne_wiebalck | ftarasenko: but I think you could crank up the prio of this step to be included as well | 13:28 |
| * arne_wiebalck *thought* you could do this, but has now read TheJulia's comment, so is maybe wrong | 13:30 | |
| TheJulia | arne_wiebalck: I didn't think about resetting/overriding the priorities | 13:30 |
| TheJulia | I mean... it is not something any of us have tested... | 13:30 |
| arne_wiebalck | TheJulia: I never tried either, but this was my understanding of what you could do with clean step prios. | 13:31 |
| opendevreview | Merged openstack/sushy master: Capture requests errors https://review.opendev.org/c/openstack/sushy/+/853209 | 13:32 |
| TheJulia | \o/ | 13:32 |
| arne_wiebalck | another topic: 'node history' ... this is unfinished work from xena ... I guess we could pick this up w/o interfering with anyone working on this, right? kaifeng was working on this at the time, but I don't think he got round to do much about it lately ... does anyone know what the status is? | 13:33 |
| TheJulia | I *think* it works | 13:34 |
| TheJulia | I've seen the tests pass | 13:35 |
| arne_wiebalck | oh, yeah? | 13:39 |
| arne_wiebalck | seems like xena added sth on the API yes | 13:41 |
| TheJulia | ugh | 13:42 |
| TheJulia | well, this is fun. Google now prefers mitaka and newton docs | 13:43 |
| TheJulia | https://docs.openstack.org/python-ironicclient/latest/cli/osc/v1/index.html#baremetal-node-history-list | 13:44 |
| TheJulia | the db side code journals, you cannot delete entries for obvious reasons... the conductor will purge them. I don't remember what the defaults were | 13:47 |
| TheJulia | speaking of kaifeng, has anyone heard from him in say the last year? | 13:57 |
| dtantsur | not me | 14:04 |
| dtantsur | good morning TheJulia | 14:04 |
| iurygregory | nope =( | 14:04 |
| iurygregory | I think he only joined one time after we moved to OFTC... | 14:04 |
| TheJulia | https://www.stackalytics.io/?release=all&metric=commits&user_id=kaifeng | 14:09 |
| TheJulia | I suspect they have moved on, they haven't responded to any emails for quite some time if I'm remembering correctly | 14:10 |
| arne_wiebalck | is the node history fully done then? | 14:13 |
| arne_wiebalck | or is there sth left to do? | 14:13 |
| iurygregory | TheJulia, yeah | 14:14 |
| TheJulia | arne_wiebalck: I think the work is done, although to be totally honest, I've not really "kicked the tires" | 14:19 |
| arne_wiebalck | TheJulia: ok, we will try to check ... | 14:20 |
| arne_wiebalck | TheJulia: thanks! | 14:21 |
| TheJulia | iurygregory: that is a great email, thanks! | 14:53 |
| iurygregory | TheJulia, dtantsur also helped a lot with the email | 14:54 |
| TheJulia | great job both of you! | 14:54 |
| JayF | :( I'm sorry to hear about etingof. Condolances to those of you who got to know them better than I did. | 14:56 |
| * TheJulia sighs | 14:57 | |
| iurygregory | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon Aug 22 15:00:16 2022 UTC and is due to finish in 60 minutes. The chair is iurygregory. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| opendevmeet | The meeting name has been set to 'ironic' | 15:00 |
| iurygregory | Hello everyone, welcome to our weekly meeting | 15:00 |
| JayF | o/ | 15:00 |
| erbarr | o/ | 15:00 |
| matfechner | o/ | 15:00 |
| kubajj | o/ | 15:00 |
| TheJulia | o/ | 15:00 |
| rloo | o/ | 15:00 |
| dtantsur | o/ | 15:01 |
| iurygregory | you can find the agenda for the meeting in the wiki | 15:01 |
| iurygregory | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:01 |
| ajya | o/ | 15:01 |
| iurygregory | #topic Announcements / Reminder | 15:01 |
| kamlesh6808c | o/ | 15:01 |
| rpioso | o/ | 15:02 |
| iurygregory | the first announcement is a sad one | 15:02 |
| iurygregory | #info Ilya Etingof Passed Away. Goodbye, etingof! | 15:03 |
| iurygregory | #link https://lists.openstack.org/pipermail/openstack-discuss/2022-August/030062.html | 15:03 |
| ajya | Sorry to hear that, my condolences. | 15:03 |
| iurygregory | I would like to share with our community, some of us knew him a lot | 15:04 |
| arne_wiebalck | o/ | 15:04 |
| rloo | OH no, I'm so sorry. | 15:05 |
| rpioso | I am very saddened to learn of etingof's passing. My condolences to the ironic community, Red Hat, and Ilya's family and friends. | 15:07 |
| iurygregory | #info This week we will release our non-client libraries | 15:07 |
| ajya | that's sushy also? | 15:07 |
| iurygregory | ajya, correct | 15:08 |
| dtantsur | sushy, ironic-lib, metalsmith | 15:08 |
| iurygregory | yeah | 15:08 |
| dtantsur | we need to check for outstanding patches (I have one, has some comments) | 15:08 |
| iurygregory | so we will focus on reviewing this 3 to make sure we have included what we want in Zed =) | 15:08 |
| iurygregory | dtantsur, yeah | 15:08 |
| ajya | can I get 2nd reviewer for this https://review.opendev.org/c/openstack/sushy/+/850899 and include that in release? | 15:09 |
| iurygregory | ajya, sure we will try to include the open patches =) | 15:09 |
| JayF | I'll look; I haven't traditionally worked much on sushy but should proabbly ramp it up. | 15:09 |
| ajya | thanks | 15:09 |
| iurygregory | #info Antelope PTG etherpad | 15:09 |
| iurygregory | #link https://etherpad.opendev.org/p/ironic-antelope-ptg | 15:10 |
| iurygregory | just a reminder that our etherpad for the PTG is this one =) | 15:10 |
| iurygregory | #info PTG registration | 15:11 |
| iurygregory | #link https://openinfra-ptg.eventbrite.com/ | 15:11 |
| iurygregory | don't forget to register for the PTG | 15:11 |
| TheJulia | Please register for the PTG so the foundation knows how many attendees plan to actively engage. This allows them to have information for future planning as well, so everyone attending registering would help them a lot. | 15:12 |
| iurygregory | #info ironic-ui is fixed =) | 15:13 |
| * TheJulia suspects we all need to dance now | 15:13 | |
| dtantsur | yay! | 15:14 |
| iurygregory | I don't know the irc handle of Vishal, tks for the help! | 15:14 |
| iurygregory | #link https://review.opendev.org/c/openstack/ironic-ui/+/852702 | 15:14 |
| iurygregory | no action items from previous meeting, skipping | 15:15 |
| iurygregory | #topic Review subteam status reports | 15:15 |
| iurygregory | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 15:15 |
| iurygregory | starting around L90 | 15:15 |
| JayF | Are there even meaningful updates there to review? | 15:18 |
| TheJulia | I guess the one w/r/t anaconda | 15:18 |
| iurygregory | Anaconda CI | 15:18 |
| iurygregory | yup =) | 15:18 |
| TheJulia | Uhh... so... tl;dr is I cannot use opendev's mirror system without hacking in another feature (maybe) into the interface to explicitly delineate package repositories versus all the other artifacts | 15:19 |
| iurygregory | =( | 15:19 |
| TheJulia | in essence, the mirror can't take on a more stuff without there being an increasingly negative impact, and the guidance is to just use public mirrors for folks doing Rocky linux | 15:20 |
| JayF | Have we looked at if we can make the install lighter/faster in any way to help get past timeouts? | 15:20 |
| TheJulia | so.. I *think* the net effect is I just need to get the timing right | 15:20 |
| JayF | I'm mainly curious if we can pass flags to anaconda, disable some of the setup steps to get a thinner test | 15:20 |
| TheJulia | it is fairly minimal, downloads can just take a ton of time | 15:20 |
| JayF | but I haven't looked at a breakdown of what it's spending most of the time in | 15:20 |
| JayF | except that one were we saw it was taking ~5 minutes for all the packages | 15:20 |
| TheJulia | possibly, although again, I'm thinking we're talking borderline featurey things | 15:21 |
| TheJulia | maybe those are okay since they would just be jinja2 parsing | 15:21 |
| JayF | but really ... 5 minutes is small in context of an hour+ job | 15:21 |
| TheJulia | yeah, latest run got to configuring the kernel post-install at 1hr. I think we're also having the lack of paravirt kill us too | 15:21 |
| TheJulia | since it is a lot of CPU overhead on uncompressing | 15:21 |
| TheJulia | I'm going to push forward, already working to make it it's own single job | 15:22 |
| TheJulia | just wanted folks to generally be aware | 15:22 |
| JayF | if you want to pair on this at any point, or just get a second set of eyes, feel free to ping me and/or we can even set aside some time | 15:22 |
| TheJulia | The "evil" option in my brain is just look to see if anaconda checked in with ironic, and then abort the deployu | 15:22 |
| TheJulia | which could be valid too... | 15:22 |
| TheJulia | Maybe we should discuss that instead | 15:23 |
| JayF | that's exactly the kind of short circuiting I'm looking for | 15:23 |
| TheJulia | Ultimately, it all comes down to the template and it's contents, it is hyper customizable | 15:23 |
| JayF | ironic does all the orchestration up front; do we want the CI to "catch" anaconda breakages, or just Ironic's ability to set the table for anaconda? | 15:23 |
| TheJulia | It wouldn't catch the close out of the deploy... but that should be fairly clear if it breaks based upon reports/issues | 15:24 |
| TheJulia | but starting/template processing/data handling, it would catch any issues there | 15:24 |
| JayF | we could even intentionally have a deploy break and return an error to ironic | 15:24 |
| JayF | testing the unhappy path is arguably more important than the happy one | 15:24 |
| TheJulia | depending on what, yeah | 15:24 |
| JayF | get anaconda going far enough for it to be able to do the err callback to ironic; then we know at least we setup anaconda as expected | 15:25 |
| TheJulia | Anyway, I could use opinions here, I'm a bit tired of fighting this :) | 15:25 |
| JayF | yeah, use me as the help for that, I'm somewhat guilty for helping upstream that sans-CI | 15:25 |
| TheJulia | That might be a really good separate test, fwiw | 15:25 |
| JayF | I am proposing it potentially as the only test, as we could probably skip all the package installs | 15:25 |
| TheJulia | do one that aborts, do one that errors, call it a day | 15:25 |
| JayF | then you get something working sooner, then worry about the happy/aborted path | 15:25 |
| TheJulia | oh, error would get called before package installs I believe | 15:25 |
| JayF | that's what I'm saying | 15:26 |
| TheJulia | I think you just need to feed it invalid stuffs and it calls %onerror | 15:26 |
| rloo | what exactly are we trying to test wrt anaconda interface? | 15:26 |
| rloo | minimum test I guess... | 15:26 |
| JayF | Heh. You could go an even step further. Write test code in anaconda template, run it, use the returned err as an indication that things were setup as expected (or not) | 15:26 |
| TheJulia | I'd like to know end to end it works | 15:26 |
| TheJulia | *but* there is a lot of overhead to make it happy in our CI | 15:27 |
| rloo | end-to-end == actually install an OS image? | 15:27 |
| TheJulia | well, in this case, install from a repository | 15:27 |
| JayF | rloo: yeah TheJulia has it working with an actual install, it just times out in the last few % | 15:27 |
| TheJulia | like last 10% it looks like | 15:27 |
| JayF | using the support added for repo-based deploys instead of liveimg based deploys | 15:27 |
| TheJulia | but it is quite variable based upon the mirrors | 15:27 |
| rloo | gad. hmm... and it needs everything from the mirrors? | 15:28 |
| rloo | and where does the time out come from? can we increase it? | 15:29 |
| TheJulia | not *everything* only like 320 rpms | 15:29 |
| TheJulia | infra cannot carry the stage2/install image | 15:29 |
| TheJulia | so... no local mirrors | 15:29 |
| iurygregory | only... | 15:29 |
| TheJulia | we should move on | 15:30 |
| TheJulia | and continue in open discussion | 15:30 |
| iurygregory | I was about to say this =) | 15:30 |
| iurygregory | #topic Deciding on priorities for the coming week | 15:30 |
| iurygregory | #link https://review.opendev.org/q/status:open+hashtag:ironic-week-prio | 15:30 |
| JayF | I'm working on trying to get several Nova-Ironic driver patches backported, probably wouldn't hurt to get more Ironic +1s on them if folks want to add them to their review list as well. I'm tracking it here: https://etherpad.opendev.org/p/NovaPatchesFromJay | 15:31 |
| JayF | (I don't think we can put weekly-prio tag on nova patches) | 15:31 |
| kamlesh6808c | Can you please help to add this patch to week priority list : https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/853621 | 15:32 |
| TheJulia | I think we can hashtag it... | 15:32 |
| iurygregory | TheJulia, normally only the owner of the patch can do that if I recall | 15:32 |
| TheJulia | ahh, yeah! | 15:32 |
| iurygregory | also depends on the config for the project | 15:32 |
| iurygregory | for example https://review.opendev.org/c/openstack/nova/+/813897 has the hashtag | 15:33 |
| iurygregory | so JayF you can probably try to add the hashtag (I think it should work...) | 15:33 |
| iurygregory | kamlesh6808c, added | 15:34 |
| kamlesh6808c | thanks ! | 15:34 |
| iurygregory | I'm adding dtantsur's patch https://review.opendev.org/c/openstack/sushy/+/851023 also | 15:34 |
| JayF | iurygregory, all: I updated the hashtag on those nova stable patches owned by me (many are owned by others and I'm just playing frontman to get them merged lol) | 15:35 |
| iurygregory | JayF, no worries! | 15:35 |
| iurygregory | tks! | 15:35 |
| iurygregory | not sure if Eric Lei is around to update https://review.opendev.org/c/openstack/ironic-lib/+/844666 | 15:36 |
| iurygregory | I'll push an edit later today so we can merge =) | 15:36 |
| iurygregory | metalsmith doesn't seem to have patches we would need to review | 15:38 |
| iurygregory | moving on o/ | 15:38 |
| iurygregory | #topic Baremetal SIG | 15:38 |
| TheJulia | given the time, I think one of us should jsut make the change | 15:38 |
| arne_wiebalck | NTR for the SIG | 15:39 |
| iurygregory | tks arne_wiebalck =) | 15:39 |
| iurygregory | #topic RFE review | 15:39 |
| iurygregory | I'm a bit puzzled if the topic from open discussion would be rfe review... =) | 15:40 |
| arne_wiebalck | yep, could be | 15:40 |
| iurygregory | #info Discussion of the software RAID story | 15:40 |
| iurygregory | #link https://storyboard.openstack.org/#!/story/2010233 | 15:40 |
| iurygregory | kubajj, o/ | 15:41 |
| arne_wiebalck | kubajj has been working on extending the disk protection to s/w RAID devices | 15:41 |
| arne_wiebalck | one question we ran into is what to do with create_configuration | 15:41 |
| arne_wiebalck | i.e. when the devices are re-created | 15:42 |
| JayF | Can I ask a question a step behind that? | 15:42 |
| arne_wiebalck | sure | 15:42 |
| kubajj | Sure | 15:42 |
| JayF | Why do we need the ability to explicitly skip disks that hold RAID partitions | 15:42 |
| JayF | if the operator already has the (thanks to kubajj) ability to skip disks based on device hints? | 15:42 |
| kubajj | Because RAIDs are skipped by default anyway. They are handled in a different function | 15:43 |
| JayF | Ah, and you just want to add software raids to those that are skipped. | 15:43 |
| JayF | We have to be careful how we implement this to prevent a malicious actor from putting something that looks like a raid superblock on a disk to prevent being cleaning | 15:43 |
| kubajj | Yeah, the goal is just to extend the functionality. | 15:43 |
| JayF | **cleaned | 15:43 |
| arne_wiebalck | JayF: everything but the partitions which form the RAID are cleaned | 15:44 |
| arne_wiebalck | well, almost everything :-D | 15:45 |
| JayF | Yeah, I'm not saying we shouldn't do it, I'm saying we should be careful and make sure there's an opt-out for anyone with a higher security bar | 15:45 |
| arne_wiebalck | JayF: sure, unless you explicit say on the node that you would like to skip sth, all will be cleaned | 15:46 |
| arne_wiebalck | like before | 15:46 |
| JayF | awesome | 15:46 |
| arne_wiebalck | this is about the special case where you have multiple s/w RAID devices | 15:46 |
| JayF | I apologize, some of this stuff, I don't know what happened when I wasn't looking so I appreciate you filling in the context | 15:46 |
| arne_wiebalck | and you would like to skip cleaning *some* | 15:46 |
| kubajj | exactly, the plan is to use the volume_name as mentioned in the story and include it in the skip_block_devices list in the properties section like with normal disks | 15:47 |
| arne_wiebalck | the volume name is the name you give the device itself (not the block device file), it is md device metadata | 15:48 |
| kubajj | https://review.opendev.org/c/openstack/ironic-python-agent/+/853182 enables actually creating logical disks with volume name enabled | 15:49 |
| kubajj | I have tested it out on our testing node and it works | 15:49 |
| arne_wiebalck | and the question was if there is an obvious problem with this ... I think dtantsur mentioned the inspector as one potential source of problems | 15:49 |
| arne_wiebalck | otherwise we go ahead and see where it gets us :) | 15:50 |
| iurygregory | I liked the idea, just trying to understand the inspector problem ... | 15:52 |
| arne_wiebalck | the inspector adds the root device to the inspector data (I think) | 15:52 |
| JayF | Are we not worried at all about the ability for whoever got that device provisioned to them being able to change that volume name? | 15:53 |
| JayF | Like, if that's not a case we're worried about; awesome... but it's trivial for that volume name to change | 15:53 |
| JayF | I was going to suggest PARTUUID but pretty much any unique identifier is changable from the system :( | 15:54 |
| * TheJulia wonders if we're scope creeping to cover all possibilities as opposed to trying to cover 90% | 15:54 | |
| arne_wiebalck | if you change the volume name, the next cleaning would erase your data | 15:54 |
| JayF | TheJulia: that's why I asked if we were worried about it :D | 15:54 |
| TheJulia | (not to say everything is good!, but obviously we need to start somewhere) | 15:54 |
| JayF | arne_wiebalck: of course, so it fails safe | 15:54 |
| JayF | aight, sounds like fun :) I look forward to reviewing it | 15:54 |
| arne_wiebalck | heh | 15:54 |
| arne_wiebalck | ok, we can check with dtantsur directly once more, seems he is not here atm | 15:55 |
| dtantsur | I'm kinda here, not following the discussion tho | 15:55 |
| JayF | I have a small item for open discussion, if we've talked this one through | 15:55 |
| iurygregory | JayF, go ahead | 15:56 |
| JayF | So, most of you know I've been working a new job, with 20% time dedicated generally to openstack and 80% to the other project I work on, Armada | 15:56 |
| dtantsur | iurygregory: the problem with inspector is that it does not have access to any symlinks on the node | 15:56 |
| JayF | over the next few weeks, those percentages will be swapping and you'll be seeing more of me around | 15:56 |
| JayF | I'm going to focus, until I get into a good cycle of my own work, on upstreaming some stuff from downstream here, stable maintenance, and reviews | 15:57 |
| JayF | but feel free to nerd snipe me in for bigger stuff as I will have the time to give | 15:57 |
| iurygregory | JayF, nice | 15:57 |
| arne_wiebalck | dtantsur: I fail to see how that is a problem | 15:57 |
| dtantsur | the discussion was around resolving /dev/md/<something> | 15:58 |
| JayF | I know we don't have a formal role for it anymore, but generally I am going to try to be the grand-poo-bah of stable branches, getting stuff backported (and starting with heralding our long-outstanding patches to our nova collegues) | 15:58 |
| dtantsur | I still haven't read the full scrollback | 15:58 |
| arne_wiebalck | dtantsur: we can do this tomorrow or so | 15:58 |
| dtantsur | JayF: I feel bad asking someone to look at our bug list.. but someone needs to look at our bug list | 15:58 |
| JayF | That /dev/md/<something> is going away in newer mdadm aiui | 15:58 |
| JayF | when you said volume name, I assumed that was shorthand for the partition label | 15:59 |
| TheJulia | cool! | 15:59 |
| JayF | dtantsur: when I get ramped up, i'll specifically cut aside some time to work on bugs | 15:59 |
| TheJulia | JayF: is it hiding in under the lvm interface? | 15:59 |
| arne_wiebalck | JayF: no, what we mean is the name of the md device (which is sth you can set on an md device) | 15:59 |
| JayF | arne_wiebalck: that is going away in newer mdadm/kernel combinations | 15:59 |
| JayF | arne_wiebalck: people were complaining about it being gone in gentoo channels the other day | 15:59 |
| dtantsur | arne_wiebalck: without /dev/md/<something>, volume name seems useless | 16:00 |
| TheJulia | oh noes | 16:00 |
| kubajj | JayF: you mean the --name option? | 16:00 |
| arne_wiebalck | dtantsur: why? | 16:00 |
| dtantsur | arne_wiebalck: how else do you access it? | 16:00 |
| JayF | kubajj: I *think* so? I'd have to look in depth to remember exactly | 16:00 |
| arne_wiebalck | mdadm --detail | 16:00 |
| dtantsur | arne_wiebalck: okay, here is the trick: how are you going to do it on the inspector side? :) | 16:00 |
| dtantsur | assuming you want it as part of the root device hints | 16:00 |
| arne_wiebalck | dtantsur: why do I need to do it on the inspector side? | 16:01 |
| dtantsur | arne_wiebalck: because we already do | 16:01 |
| iurygregory | we are at the top of the hour for the meeting , going to close and we can continue talking :D | 16:01 |
| dtantsur | we parse and process root device hints in inspector | 16:01 |
| iurygregory | #endmeeting | 16:01 |
| opendevmeet | Meeting ended Mon Aug 22 16:01:24 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:01 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-08-22-15.00.html | 16:01 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-08-22-15.00.txt | 16:01 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-08-22-15.00.log.html | 16:01 |
| dtantsur | now, you could argue we should just deprecate and throw away this from inspector, and I'll likely agree :) | 16:01 |
| arne_wiebalck | thanks iurygregory | 16:01 |
| dtantsur | but it's there, and we cannot just ignore it | 16:01 |
| arne_wiebalck | I most likely do not understand how that is used | 16:02 |
| arne_wiebalck | the inspector gets the hints, and populates the inspection data with root_device ... and then? | 16:02 |
| arne_wiebalck | what is that used for ? | 16:02 |
| TheJulia | target of where to write the image | 16:02 |
| arne_wiebalck | root_device in the inspection data I mean | 16:02 |
| dtantsur | arne_wiebalck: largely, local_gb detection | 16:02 |
| arne_wiebalck | this read from the node and not redone upon deployment? | 16:03 |
| opendevreview | Julia Kreger proposed openstack/ironic master: CI: Changes to support Anaconda CI jobs https://review.opendev.org/c/openstack/ironic/+/849587 | 16:04 |
| kubajj | dtantsur: but could we pass the volume names to inspector somehow? | 16:04 |
| kubajj | I have to admit that I am not really sure when the inspector gets called though | 16:05 |
| arne_wiebalck | kubajj: we could add them to the inspection data, but not if there is no reason to do it | 16:05 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: Initial tempest test idea anaconda deploy https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/854031 | 16:06 |
| * TheJulia wonders if we should instead just be working to peel back some of local_gb | 16:07 | |
| * TheJulia is just thinking outloud | 16:07 | |
| dtantsur | yeah, that's what I mean: this code can probably be killed. but with deprecation sooo... | 16:07 |
| TheJulia | yeah | 16:07 |
| arne_wiebalck | ok ... we will let code speak :) | 16:14 |
| TheJulia | JayF: I think if my latest changes don't result in a passing job, I'll just head on the "oh, we got to deploy wait, abort!" | 16:14 |
| TheJulia | But if it gains conciousness.... | 16:15 |
| arne_wiebalck | TheJulia: the code? hmm ... | 16:15 |
| * arne_wiebalck goes home now to think about that ... | 16:16 | |
| TheJulia | lol | 16:16 |
| arne_wiebalck | bye everyone o/ | 16:16 |
| TheJulia | goodnight | 16:20 |
| *** tkajinam is now known as tkajinam|off | 16:35 | |
| *** rcastillo|rover is now known as rcastillo | 18:22 | |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: Initial tempest test idea anaconda deploy https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/854031 | 18:38 |
| TheJulia | dtantsur: so I read in the mboot source after I remembered iso booting drops the rest of the iso contents from memory... and cried even more because they do things.... | 19:49 |
| * TheJulia largely eradicates the doc items he wrote up | 19:54 | |
| TheJulia | she | 19:54 |
| opendevreview | Merged openstack/sushy master: Add new Storage controllers https://review.opendev.org/c/openstack/sushy/+/850899 | 20:33 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add docs for VMware deployment https://review.opendev.org/c/openstack/ironic/+/853549 | 21:27 |
| TheJulia | JayF: well... https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/854031/ (i have no idea how it is now a new change set....) but it passed it seems | 21:31 |
| TheJulia | JayF: I don't feel too horrible about a selector in the config file... | 21:36 |
| JayF | aight, looking | 21:53 |
| opendevreview | Julia Kreger proposed openstack/ironic master: CI: Changes to support Anaconda CI jobs https://review.opendev.org/c/openstack/ironic/+/849587 | 23:21 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Docs: Add considerations to anaconda docs https://review.opendev.org/c/openstack/ironic/+/854045 | 23:21 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: Tempest test for anaconda deploy https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/854031 | 23:21 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!