| opendevreview | Verification of a change to openstack/ironic failed: Scoped RBAC Devstack Plugin support https://review.opendev.org/c/openstack/ironic/+/778957 | 02:38 |
|---|---|---|
| *** pmannidi is now known as pmannidi|brb | 04:28 | |
| *** pmannidi|brb is now known as pmannidi | 05:34 | |
| iurygregory | hello Ironicers o/ | 06:09 |
| iurygregory | Service Unavailable | 06:10 |
| iurygregory | The server is temporarily unable to service your request due to maintenance downtime or capacity problems. Please try again later. | 06:10 |
| iurygregory | Gerrit is down =( | 06:10 |
| iurygregory | yay gerrit is back | 06:16 |
| iurygregory | if any core has some time https://review.opendev.org/c/openstack/ironic/+/801064 | 06:16 |
| -opendevstatus- NOTICE: Due to a configuration error unfortunately the Zuul queue was lost. Please recheck any in-flight changes | 06:56 | |
| opendevreview | Jonas Schäfer proposed openstack/ironic-python-agent master: Expose BMC MAC address in inventory data https://review.opendev.org/c/openstack/ironic-python-agent/+/801260 | 07:06 |
| jssfr | TheJulia (I think it was you), thanks for your feedback! I addressed the nits. I am not sure what to do about the ipmitool hangs. Are there any specific circumstances we could avoid? Did you mean "it hangs when asking for a non-existent LAN channel"? | 07:07 |
| *** rpittau|afk is now known as rpittau | 07:27 | |
| rpittau | good morning ironic! o/ | 07:27 |
| iurygregory | morning rpittau o/ | 07:29 |
| rpittau | hey iurygregory :) | 07:29 |
| iurygregory | finally got zuul +1 \o/ | 07:29 |
| rpittau | oh great! | 07:30 |
| rpittau | I'l have a look soon | 07:30 |
| iurygregory | tks! | 07:30 |
| iurygregory | stendulker, morning o/ if you have some time can you check https://review.opendev.org/c/openstack/ironic/+/801064 ? I saw some failures from HP CI but nto sure if they are related | 07:48 |
| rpittau | if anyone has a minute please check these 3 quick changes in bifrost https://review.opendev.org/c/openstack/bifrost/+/800461 https://review.opendev.org/c/openstack/bifrost/+/799965 https://review.opendev.org/c/openstack/bifrost/+/799975 | 07:50 |
| iurygregory | looking now | 07:51 |
| iurygregory | I was sure I've reviewed the upgrade one hehe | 07:51 |
| dtantsur | good morning ironic | 07:55 |
| iurygregory | hey dtantsur | 07:56 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: [DNM] Test latest ansible https://review.opendev.org/c/openstack/bifrost/+/800440 | 08:00 |
| opendevreview | Verification of a change to openstack/ironic-inspector failed: Do not reset FSM when ironic ports cant be queried https://review.opendev.org/c/openstack/ironic-inspector/+/791251 | 08:01 |
| dtantsur | folks, could we maybe merge https://review.opendev.org/c/openstack/bifrost/+/801457 https://review.opendev.org/c/openstack/bifrost/+/801465 and https://review.opendev.org/c/openstack/bifrost/+/801298 before we cut bifrost? | 08:02 |
| dtantsur | just to avoid backporting them right away | 08:02 |
| iurygregory | dtantsur, I will look at the other two in a few minutes | 08:03 |
| dtantsur | thx! | 08:03 |
| iurygregory | the one for vendors has my +2 already | 08:03 |
| dtantsur | we also need to release 18.0.1 from bugfix/18.0 because of the BIOSVersion fix | 08:04 |
| dtantsur | this requires manual intervention by the release team, could someone talk to them or should I? | 08:05 |
| iurygregory | I'm trying to understand why requires manual intervention... | 08:11 |
| iurygregory | we need to ask them for 18.0.1 to point to bugfix/18.0.0 that would be the reason? | 08:15 |
| rpittau | iurygregory: I'm reading the patch and I don't see anything wrong, just the name of 2 functions are really similar and I would change them to avoid confusion, I left a comment | 08:16 |
| rpittau | iurygregory: re manual intervention: since we need to add a version to a bugfix and there's no automation for that, yes | 08:16 |
| iurygregory | rpittau, tks should be fine to rename the functions (get_all_subscriptions and maybe get_subscription instead of get_single_subscription?) | 08:18 |
| iurygregory | rpittau, I see | 08:18 |
| rpittau | iurygregory: re rename functions: yep, that works | 08:18 |
| iurygregory | ack updating now | 08:18 |
| dtantsur | iurygregory: I mean, the releases repository machinery is (or at least was the last time I checked) incapable of doing bugfix releases | 08:23 |
| dtantsur | * from bugfix branches | 08:23 |
| dtantsur | oh, I see rpittau already answered that | 08:23 |
| rpittau | dtantsur: I can take care of that in a couple of minutes | 08:23 |
| dtantsur | many thanks! I suspect we may need to wait until our afternoon when folks from the USA wake up | 08:24 |
| opendevreview | Verification of a change to openstack/ironic failed: Scoped RBAC Devstack Plugin support https://review.opendev.org/c/openstack/ironic/+/778957 | 08:24 |
| rpittau | mmm right, I'll leave a message in the channel and patiently wait :) | 08:24 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Add vendor_passthru method for subscriptions https://review.opendev.org/c/openstack/ironic/+/801064 | 08:27 |
| iurygregory | rpittau, done ^ =) | 08:27 |
| * rpittau please insert more coffee to continue.... | 08:36 | |
| iurygregory | I'm already on my second mug of coffee | 08:38 |
| iurygregory | yesterday I had 5 (my record was 4..) | 08:39 |
| rpittau | I usually get to 5 when I'm in Sardegna, because people making coffee at any time of day/night | 08:47 |
| dtantsur | :) | 08:48 |
| * dtantsur is back to coffee with caffeine, at least for now | 08:48 | |
| iurygregory | woot?! dtantsur is back to coffee? <O> | 08:50 |
| dtantsur | yeah, want to see how it goes now | 08:51 |
| rpittau | the problem with coffee in Sardegna is that after a certain hour it always get followed but some form of very strong alcoholic beverage, and then more coffee, and then more alcoholic drink, in a very dangerous loop | 08:51 |
| iurygregory | good luck, today I'm just trying to keep my sanity and do some work | 08:51 |
| rpittau | s/but/by/ | 08:52 |
| iurygregory | rpittau, now I want to go to Sardegna =) | 08:52 |
| rpittau | :D | 08:52 |
| * dtantsur hasn't stopped wanting to go to Sardegna | 08:52 | |
| iurygregory | yeah | 08:52 |
| rpittau | heh hope to meet you all there at some point :) | 08:53 |
| opendevreview | Merged openstack/bifrost master: Perform upgrade by default when installing ansible https://review.opendev.org/c/openstack/bifrost/+/800461 | 08:54 |
| opendevreview | Merged openstack/bifrost master: Force selinux presence when testing https://review.opendev.org/c/openstack/bifrost/+/799965 | 08:54 |
| opendevreview | Merged openstack/bifrost master: Collect explicit list of installed packages https://review.opendev.org/c/openstack/bifrost/+/799975 | 08:54 |
| rpittau | mmm may be worth merging this also before the cut https://review.opendev.org/c/openstack/bifrost/+/798825 | 08:56 |
| dtantsur | yep, approved | 08:56 |
| opendevreview | Merged openstack/bifrost master: Install sushy-oem-idrac for idrac-redfish-virtual-media https://review.opendev.org/c/openstack/bifrost/+/801457 | 09:02 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost bugfix/11.0: Install sushy-oem-idrac for idrac-redfish-virtual-media https://review.opendev.org/c/openstack/bifrost/+/801580 | 09:04 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost stable/wallaby: Install sushy-oem-idrac for idrac-redfish-virtual-media https://review.opendev.org/c/openstack/bifrost/+/801606 | 09:05 |
| opendevreview | Merged openstack/bifrost master: Support ilo5 hardware type https://review.opendev.org/c/openstack/bifrost/+/801465 | 09:14 |
| iurygregory | merge all the things :D | 09:15 |
| dtantsur | \o/ | 09:15 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Show debug messages during upgrade ci job https://review.opendev.org/c/openstack/bifrost/+/801608 | 09:16 |
| opendevreview | Merged openstack/bifrost master: bifrost-cli: fix relative paths in --extra-vars https://review.opendev.org/c/openstack/bifrost/+/801298 | 09:18 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost bugfix/11.0: bifrost-cli: fix relative paths in --extra-vars https://review.opendev.org/c/openstack/bifrost/+/801581 | 09:19 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost stable/wallaby: bifrost-cli: fix relative paths in --extra-vars https://review.opendev.org/c/openstack/bifrost/+/801582 | 09:19 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Remove ansible-base when upgrading ansible https://review.opendev.org/c/openstack/bifrost/+/801609 | 09:19 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: [DNM] Test latest ansible https://review.opendev.org/c/openstack/bifrost/+/800440 | 09:20 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Add lower-constraints job to current development branch https://review.opendev.org/c/openstack/ironic/+/801350 | 09:25 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Add lower-constraints job to current development branch https://review.opendev.org/c/openstack/ironic/+/801350 | 09:26 |
| iurygregory | ironic-tempest-ipa-partition-pxe_ipmitool - failure -.-' really?! | 09:47 |
| iurygregory | recheck ftw | 09:47 |
| iurygregory | ofc it's because of cleanning | 09:48 |
| opendevreview | Merged openstack/bifrost master: Allow HTTP headers to be customised for IPA image download https://review.opendev.org/c/openstack/bifrost/+/798825 | 09:54 |
| iurygregory | rpittau, tks for the +2 | 09:59 |
| iurygregory | weird zuul didn't trigger another run after my recheck .-. | 10:11 |
| iurygregory | lol I had to wait for the run to finish | 10:32 |
| opendevreview | Merged openstack/bifrost stable/wallaby: Install sushy-oem-idrac for idrac-redfish-virtual-media https://review.opendev.org/c/openstack/bifrost/+/801606 | 11:11 |
| opendevreview | Merged openstack/bifrost stable/wallaby: bifrost-cli: fix relative paths in --extra-vars https://review.opendev.org/c/openstack/bifrost/+/801582 | 11:11 |
| opendevreview | Merged openstack/ironic master: Scoped RBAC Devstack Plugin support https://review.opendev.org/c/openstack/ironic/+/778957 | 11:27 |
| iurygregory | ironic-tempest-ipa-partition-pxe_ipmitool failed again -.-' | 11:46 |
| stendulker | iurygregory: The failures on HPE CI are not relaetd to your patch https://review.opendev.org/c/openstack/ironic/+/801064 | 12:02 |
| iurygregory | stendulker, tks! | 12:02 |
| iurygregory | Some nodes did not finish cleaning =( oh god why this job doesn't help | 12:03 |
| rpittau | if anyone has a minute this https://review.opendev.org/c/openstack/bifrost/+/801609 finally fixes ansible upgrades | 12:19 |
| iurygregory | rpittau, done | 12:19 |
| rpittau | thanks :) | 12:19 |
| iurygregory | I wish the ipmitooljob stopped causing failures -.-' | 12:20 |
| TheJulia | hmm | 13:03 |
| TheJulia | sounds like we're just hitting sporatic failures, because one yesterday was on networking for post deployment | 13:04 |
| TheJulia | Also, good morning everyone | 13:06 |
| TheJulia | jssfr: we don't have enough info, to be honest, so I think proceed as your doing to is fine for now. If we get enoguh information then we can just do the needful then | 13:09 |
| jssfr | thanks! | 13:09 |
| jssfr | I'll "recheck" then, as Zuul complained again... | 13:09 |
| * TheJulia needs coffee | 13:10 | |
| jssfr | ☕ | 13:11 |
| TheJulia | ++ | 13:11 |
| cenne | Good morning TheJulia | 13:21 |
| iurygregory | morning TheJulia and cenne | 13:32 |
| cenne | o/ | 13:32 |
| opendevreview | Merged openstack/ironic-inspector stable/ussuri: Remove grenade jobs from old stable branches https://review.opendev.org/c/openstack/ironic-inspector/+/800466 | 13:34 |
| * iurygregory gives cookies to ironic-tempest-ipa-partition-pxe_ipmitool job | 13:34 | |
| iurygregory | if any core has some time to look at https://review.opendev.org/c/openstack/ironic/+/801064 I would appreciate =) | 13:50 |
| iurygregory | the rechecks are because so me nodes did not finish cleaning in the ironic-tempest-ipa-partition-pxe_ipmitool (random failures) | 13:51 |
| rpittau | we should really try to get that in by today | 13:57 |
| iurygregory | yeah | 14:02 |
| iurygregory | seems like this time the ipmitool job didn't fail \o/ | 14:02 |
| iurygregory | finally +1 from zuul \o/ | 14:20 |
| iurygregory | I really don't want to update the commit message =X ( I just noticed it says "adds two new vendor_passthru methods" /facepalm | 14:21 |
| TheJulia | iurygregory: so does this obsolete the spec at this point, or are we just holding on the spec or.... ? | 14:22 |
| iurygregory | TheJulia, it does =) | 14:22 |
| iurygregory | spec == obsolete =) | 14:22 |
| TheJulia | I think we should put it into the ideas folder and merge it, fwiw | 14:23 |
| TheJulia | or backlog It think what it is, and maybe note the vendor passthroughs for now | 14:23 |
| TheJulia | Just to wrap it up nicely | 14:23 |
| iurygregory | I can totally do that =) | 14:23 |
| iurygregory | TheJulia, I don't see ideas folder I will move to backlog folder and update a few things | 14:27 |
| TheJulia | iurygregory: yeah, blacklog folder is the right place | 14:31 |
| iurygregory | TheJulia, should I keep the spec the way it's and only add that we have vendor_passthru support as an Alternative? | 14:35 |
| TheJulia | iurygregory: specs are about standardization and cross-vendor. I don't think listing it as an alternative across the board is very viable :) | 14:44 |
| iurygregory | well the vendor_passthru should work in Dell/HP when using redfish :D | 14:46 |
| iurygregory | and the API would only work on HW that has Redfish support also... | 14:46 |
| iurygregory | I will keep the spec the way it's and only move to the backlog =) | 14:47 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic-specs master: Event Subscription Idea https://review.opendev.org/c/openstack/ironic-specs/+/785742 | 14:50 |
| iurygregory | done =) | 14:51 |
| TheJulia | \o/ | 15:04 |
| *** tcr1 is now known as trandles | 16:15 | |
| dtantsur | TheJulia: I've started prototyped something, curious to hear your opinion https://hackmd.io/q9aHu0WjQT2yQfE9U3LkTg | 16:32 |
| TheJulia | Left one comment | 16:35 |
| TheJulia | I think it is reasonable and I've kind of been hoping someone woudl do this | 16:36 |
| TheJulia | a couple minor things to keep in mind, technically it breaks the traits schema | 16:36 |
| TheJulia | But we *should* | 16:36 |
| TheJulia | There is a limit of 50 traits, and we've done the needful to keep from auto-detonating the database interactions | 16:37 |
| dtantsur | yep. maybe we should keep the number of user traits to 50, while also being modest with the number of auto-traits | 16:38 |
| TheJulia | the db join fix *really* doesn't prohibit us from growing that, fwiw | 16:38 |
| TheJulia | I don't remember why it was 50 though | 16:39 |
| TheJulia | do you? | 16:39 |
| dtantsur | for sanity reasons I guess | 16:40 |
| TheJulia | What is this sanity word you use? | 16:40 |
| iurygregory | placement limit | 16:40 |
| iurygregory | according to the spec | 16:40 |
| TheJulia | Is that were I get the special warm self hugging jacket? | 16:40 |
| TheJulia | oh, heh | 16:40 |
| iurygregory | https://review.opendev.org/c/openstack/ironic-specs/+/504531/14/specs/approved/node-traits.rst | 16:40 |
| iurygregory | L180 | 16:40 |
| TheJulia | thanks iurygregory | 16:40 |
| iurygregory | TheJulia, yw | 16:41 |
| dtantsur | aha, I see | 16:48 |
| dtantsur | hmm, we should make sure we don't report the auto-traits to placement. | 16:49 |
| dtantsur | or... do we? | 16:49 |
| TheJulia | *we* do not | 16:50 |
| TheJulia | nova-compute nova.virt.ironic *does* | 16:50 |
| TheJulia | I want to flip that | 16:50 |
| dtantsur | yep, that's the question: should we exclude auto-traits? | 16:50 |
| dtantsur | I can see arguments for and against it | 16:50 |
| TheJulia | but spoons/time and hoping nobodycam beats me to it | 16:50 |
| TheJulia | Yeah, I dunno. I'm not 1000% sure it would accept a non-compliant label right now | 16:50 |
| TheJulia | but if it doesn't, we should make it handle it | 16:51 |
| TheJulia | so I think prototype it, shove it in to gerrit and lets see if it blows up our nova integration! | 16:51 |
| TheJulia | so who were my volunteers for an ironic session again? | 16:53 |
| dtantsur | I was IIRC | 16:53 |
| dtantsur | and iurygregory and arne_wiebalck | 16:54 |
| iurygregory | correct | 16:54 |
| dtantsur | re nova: I'm planning to hide the auto-traits behind a microversion, so that nova doesn't see them immediately | 16:54 |
| TheJulia | arne_wiebalck is out, so I'll share my prototype deck and we can kick some ideas around | 16:54 |
| TheJulia | Yeah, it has a most recent and last known good setting if memory serves | 16:54 |
| TheJulia | err, most recent known | 16:55 |
| iurygregory | "if you can't see, you can't complain" :D | 16:55 |
| iurygregory | hide behind a config option :D | 16:55 |
| TheJulia | -1 to configuration options | 16:56 |
| dtantsur | oh no | 16:56 |
| dtantsur | microversions are an instrument precisely for this | 16:56 |
| dtantsur | (breaking changes is the only thing they're good at) | 16:56 |
| TheJulia | what dtantsur said | 16:56 |
| TheJulia | we do have the effective max setting to allow rolling upgrades | 16:57 |
| TheJulia | The start of a deck shared | 16:57 |
| dtantsur | okay, thanks! when is the thing? | 16:58 |
| iurygregory | I don't think we have a date yet, do we? | 16:58 |
| iurygregory | I remember TheJulia said there is a place where we can choose the slot | 16:59 |
| dtantsur | unless it's tomorrow, I'll take a look tomorrow - that's what I'm trying to say :) | 16:59 |
| iurygregory | ++ | 16:59 |
| TheJulia | So we're going to try for the 12th of august | 16:59 |
| TheJulia | so tomorrow++ | 16:59 |
| TheJulia | or even friday++ | 17:00 |
| TheJulia | or Monday++ as long as we start hammering ideas out | 17:00 |
| dtantsur | Friday sounds even better given that it's a no-meeting-day for me | 17:00 |
| dtantsur | (SPUC doesn't count) | 17:00 |
| TheJulia | :) | 17:00 |
| TheJulia | I'm going to go think of something for lunch | 17:00 |
| dtantsur | enjoy! | 17:00 |
| iurygregory | enjoy your lunch =) | 17:01 |
| iurygregory | wow it's 7pm already (this would explain why I'm hungry :D) | 17:01 |
| dtantsur | indeed | 17:01 |
| iurygregory | see you tomorrow Ironic o/ | 17:01 |
| * dtantsur loves that hackmd has a vi mode | 17:03 | |
| rpittau | dtantsur: if you have a moment can you please check this https://review.opendev.org/c/openstack/bifrost/+/801609 | 17:05 |
| dtantsur | rpittau: +A | 17:05 |
| rpittau | thanks! | 17:05 |
| dtantsur | I wonder if we need to force-update the openstack collection | 17:05 |
| dtantsur | wdyt? | 17:05 |
| rpittau | in the upgrade job? that's a good idea | 17:08 |
| TheJulia | snacking complete, lunch needs to be in a couple hours for reasons | 17:09 |
| opendevreview | cenne proposed openstack/ironic master: [WIP] Add api endpoint for setting boot_mode on managed node https://review.opendev.org/c/openstack/ironic/+/800084 | 17:09 |
| iurygregory | https://review.opendev.org/c/openstack/ironic/+/801469 lgtm I'm just wondering if we really need the major bump since we already have 18.0 with the iscsi removal... | 17:10 |
| rpittau | dtantsur, TheJulia, if you're not too busy do you mind having a look at https://review.opendev.org/c/openstack/ironic/+/801469 ? I'd like to ask for the bugfix cut as soon as possible | 17:11 |
| dtantsur | I'll leave the englishing to TheJulia :) | 17:12 |
| dtantsur | rpittau: but I'll ask my favourite question: why a major bump? :) | 17:13 |
| TheJulia | iscsi interface removal? | 17:13 |
| iurygregory | it was 18.0 | 17:13 |
| dtantsur | was in 18.0 | 17:13 |
| dtantsur | time flies, yeah? | 17:13 |
| TheJulia | oh, right | 17:13 |
| iurygregory | =) | 17:13 |
| TheJulia | yeah | 17:13 |
| rpittau | I know, I know, I saw we had quite some changes and I thought a major was maybe due | 17:14 |
| * TheJulia looks | 17:14 | |
| dtantsur | rpittau: it's less about how many changes we have, more about how invasive (or outright breaking) they are | 17:14 |
| dtantsur | nothing stands out as such in the release notes to me | 17:14 |
| rpittau | alright, I'll change the patch | 17:16 |
| dtantsur | cool, thanks! sorry for being a pain :) | 17:16 |
| rpittau | btw I didn't get any answer form the release team | 17:16 |
| TheJulia | the oslo.db minimum is *really* the only thing that could justify it | 17:16 |
| dtantsur | I don't think we tend to raise the major version for a requirement bump, unless particularly problematic | 17:17 |
| rpittau | dtantsur: no worries! | 17:17 |
| TheJulia | dtantsur: that is my recollection as well | 17:17 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Prepare 19.0 and clean up release notes https://review.opendev.org/c/openstack/ironic/+/801469 | 17:19 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Prepare 18.1 and clean up release notes https://review.opendev.org/c/openstack/ironic/+/801469 | 17:19 |
| TheJulia | heh | 17:20 |
| iurygregory | hehehe | 17:20 |
| rpittau | one more time ^ | 17:20 |
| * rpittau the typo master hits again | 17:20 | |
| dtantsur | +2 and I'm ready to leave for a day | 17:21 |
| dtantsur | see you! | 17:21 |
| TheJulia | goodnight dtantsur | 17:21 |
| rpittau | I'm going too, good night! o/ | 17:21 |
| *** rpittau is now known as rpittau|afk | 17:21 | |
| TheJulia | goodnight rpittau|afk | 17:22 |
| opendevreview | Merged openstack/bifrost master: Remove ansible-base when upgrading ansible https://review.opendev.org/c/openstack/bifrost/+/801609 | 18:11 |
| opendevreview | Julia Kreger proposed openstack/ironic-specs master: Clean up released features/items https://review.opendev.org/c/openstack/ironic-specs/+/801650 | 18:54 |
| TheJulia | o/ kkillsfirst How goes it? | 19:04 |
| kkillsfirst | o/ TheJulia I am to demo kexec on my testbed today. Do you have any methods for measuring if it is functioning? | 19:08 |
| TheJulia | typically we attempt to ssh into the instance | 19:09 |
| TheJulia | good luck! | 19:09 |
| kkillsfirst | I am able to ssh into the node after deployment. I have a direct and ramdisk node to test. | 19:11 |
| TheJulia | cool! | 19:11 |
| stevebaker | good morning | 20:18 |
| TheJulia | Good morn ing | 20:44 |
| opendevreview | Merged openstack/ironic master: Prepare 18.1 and clean up release notes https://review.opendev.org/c/openstack/ironic/+/801469 | 20:56 |
| timeu | Hi all, I have one question regarding the virtual media boot support in OSP16.1 (train). As far as I understand ironic will automatically create an iso from the kernel and ramdisk and then mount in a virtual media slot via http(s). It seems to use the swift endpoint to do so. Unfortunately in our case the BMC can only reach the provisioning network, but not the swift | 21:03 |
| timeu | endpoint. is there any way to tell ironic to serve the image via the provisioning network similar to how it is done for pxe/tftp/http ? | 21:03 |
| TheJulia | timeu: uhhh | 21:06 |
| TheJulia | hmmmm | 21:06 |
| TheJulia | hmmmm | 21:06 |
| timeu | I checked the ironic config and image_download_source=http is set | 21:06 |
| TheJulia | I *feel* like this came up | 21:07 |
| TheJulia | and I think it was fixed as a bug | 21:07 |
| timeu | and it seems to work for the regular non virtual media deploy | 21:07 |
| NobodyCam | Good Afternoon Ironic folks! | 21:07 |
| NobodyCam | happy hump day | 21:07 |
| timeu | but in case of virtual media I see following url: 'http://172.16.53.5:8080/v1/AUTH_d8cf4d60de994cbcaa91dc06ef642dbb/ironic_redfish_container/boot-01c9e448-b833-4a11-a561-3282b27e1205 | 21:07 |
| timeu | 172.16 is the storage network where the swift admin/internal endpoint is listening | 21:08 |
| TheJulia | timeu: pulling up the code, this is sounding like deja vu | 21:08 |
| timeu | TheJulia: thanks a lot. I was thinking it could be a bug but wanted to make sure | 21:09 |
| timeu | I had to install latest sushy library into the ironic-conductor container because our Lenovo servers don't support insert_media action but only PATCH. | 21:09 |
| TheJulia | timeu: hmm | 21:10 |
| timeu | but I don't think that this is the issue. Funnily I see that swift temporary url for deploy iso's are supported in xena: https://review.opendev.org/c/openstack/ironic/+/801469/3/releasenotes/notes/deploy-iso-swift-355ad6eba6c511b4.yaml | 21:11 |
| timeu | but don't worry. I can dig around in the code and see if I can find the issue. I just wanted to make sure that http source is also supported for virtual media ;-) | 21:13 |
| TheJulia | Looks like dtantsur made some changes there really recently | 21:13 |
| timeu | the stackframe where the conductor throws an error is around: task.driver.boot.prepare_ramdisk(task, ramdisk_opts) | 21:14 |
| timeu | I can do a git blame and check what changed there | 21:15 |
| TheJulia | timeu: looks like you may need/want https://review.opendev.org/q/I6ecbe8ae4c775f5a39b68ff698bde655795e5db7 | 21:16 |
| TheJulia | hmm | 21:18 |
| TheJulia | maybe not | 21:18 |
| timeu | hmm yeah as far as I understand the patch, it's a new flag to disable caching it on the conductor | 21:19 |
| TheJulia | yeah | 21:21 |
| TheJulia | a lot of chagnes went in between vitoria and wallaby | 21:22 |
| timeu | yeah I will dig a bit more in the code and check some of the PRs | 21:23 |
| TheJulia | hmm | 21:23 |
| TheJulia | okay | 21:23 |
| timeu | on that note: do you by any chance have some insights whether OSP16.2 (train) and OSP17.0 (wallaby) will be released together ? | 21:25 |
| TheJulia | [redfish]use_swift | 21:25 |
| TheJulia | That is what you need | 21:25 |
| TheJulia | https://github.com/openstack/ironic/blob/stable/train/ironic/drivers/modules/redfish/boot.py#L252-L2653 | 21:25 |
| TheJulia | err https://github.com/openstack/ironic/blob/stable/train/ironic/drivers/modules/redfish/boot.py#L252-L265 | 21:25 |
| timeu | TheJulia: ah super thanks ! | 21:26 |
| timeu | that looks like it | 21:26 |
| TheJulia | enjoy! | 21:26 |
| TheJulia | also, let us know how it goes, specifically because your the first person I've heard trying on lenovo gear | 21:27 |
| timeu | yeah wlll do ! would be super great if we could get virtual media working. I opened a storyboard issue with some information recently: https://storyboard.openstack.org/#!/story/2008852 | 21:29 |
| timeu | will update it as soon as I have something working | 21:29 |
| TheJulia | oh joy | 21:33 |
| TheJulia | Thanks! | 21:33 |
| timeu | ok instant partial success. after adding the config flag, it tries to download the iso but fails with: PermissionError: [Errno 13] Permission denied: '/var/lib/ironic/httpboot/redfish/boot-01c9e448-b833-4a11-a561-3282b27e1205' | 21:34 |
| TheJulia | Oo | 21:35 |
| timeu | will check dig into it myself | 21:35 |
| TheJulia | ok | 21:35 |
| timeu | maybe I messed to much with the container ;-) | 21:35 |
| TheJulia | likely good to check | 21:35 |
| TheJulia | or maybe something is wrong in the permissions it ships with and nobody has reported it in that build or soemthing | 21:35 |
| TheJulia | janders_: FYI w/r/t lenovo hardware above | 21:36 |
| timeu | seems like that the created redfish directory has some weird permissions: dr-x---r-t. 2 ironic ironic 6 Jul 21 21:33 redfish | 21:41 |
| rpioso | TheJulia, timeu: That rings a bell. I believe that bug was fixed. Perhaps it wasn't backported to stable/train and/or OSP 16.1? | 21:45 |
| timeu | ok after fixing the directory permission, the cleaning starts, however the console shows now the GRUB menu | 21:45 |
| timeu | rpioso: do you mean the directory permission ? good to know. I fixed it for now manually. It starts the cleaning but something doesn't work with the iso. Will look into it tomorrow tough | 21:47 |
| rpioso | timeu: Yep, the directory permission. Note the permission is different on master: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/image_utils.py#L199-L200 | 21:52 |
| timeu | ah good to know thanks for the link | 21:52 |
| rpioso | timeu: Some refactoring was later done. | 21:53 |
| rpioso | timeu: You're most welcome. | 21:54 |
| kkillsfirst | Hello TheJulia, Do you know how to add the kexec package to my images using disk-image-create? | 21:55 |
| TheJulia | kkillsfirst: ad-hoc you can add -p and a list of packages | 21:56 |
| TheJulia | kkillsfirst: for like stock images, I would modify ironic-python-agent-builder | 21:57 |
| TheJulia | Each element has a package list and is pulled in by diskimage-builder element package lists | 21:58 |
| rpioso | TheJulia, timeu: Here's the change I was thinking of: https://review.opendev.org/c/openstack/ironic/+/732413 | 22:01 |
| TheJulia | rpioso: your memory is much better than mine today ... So that is own of mine | 22:02 |
| rpioso | TheJulia: Probs not. I recently caffeinated :-) | 22:03 |
| TheJulia | rpioso: ahh, yes. Caffeine! | 22:04 |
| * TheJulia resumes exercising... | 22:04 | |
| opendevreview | Steve Baker proposed openstack/ironic stable/train: Provide a path to set explicit ipxe bootloaders https://review.opendev.org/c/openstack/ironic/+/801669 | 22:50 |
| TheJulia | timeu:I've got a note on my calendar to try and backport it down to train tomorrow morning | 23:14 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!