*** jdandrea has quit IRC | 00:40 | |
*** k_mouza has joined #openstack-ironic | 00:47 | |
*** k_mouza has quit IRC | 00:52 | |
*** TxGirlGeek has quit IRC | 01:04 | |
*** ddurst has quit IRC | 01:18 | |
*** ddurst has joined #openstack-ironic | 01:22 | |
*** Goneri has quit IRC | 01:26 | |
*** bnemec has quit IRC | 02:19 | |
*** bnemec has joined #openstack-ironic | 02:28 | |
*** zhenguo_ has joined #openstack-ironic | 02:33 | |
*** k_mouza has joined #openstack-ironic | 02:51 | |
*** k_mouza has quit IRC | 02:56 | |
*** mkrai has joined #openstack-ironic | 03:10 | |
openstackgerrit | Merged openstack/ironic master: Extend install_bootloader command timeout https://review.opendev.org/716021 | 03:12 |
---|---|---|
*** threestrands has joined #openstack-ironic | 03:39 | |
*** ociuhandu has joined #openstack-ironic | 03:43 | |
*** ociuhandu has quit IRC | 03:48 | |
*** zhenguo_ has quit IRC | 04:42 | |
*** bdodd has quit IRC | 04:54 | |
*** k_mouza has joined #openstack-ironic | 04:56 | |
*** k_mouza has quit IRC | 05:01 | |
*** tzumainn has quit IRC | 05:06 | |
*** mkrai has quit IRC | 05:48 | |
*** mkrai has joined #openstack-ironic | 05:48 | |
openstackgerrit | Ankit Kumar proposed openstack/ironic master: Generalize ISO building for virtual media driver https://review.opendev.org/677996 | 06:34 |
*** etingof has joined #openstack-ironic | 06:45 | |
uzumaki | morning ironic! o/ | 06:45 |
*** hjensas has quit IRC | 06:54 | |
arne_wiebalck | Good morning, ironic! | 06:56 |
*** k_mouza has joined #openstack-ironic | 06:57 | |
uzumaki | how you doing arne_wiebalck ? | 06:57 |
arne_wiebalck | thanks, all good over here ... and you? | 06:58 |
uzumaki | i'm well thanks! | 06:59 |
*** k_mouza has quit IRC | 07:02 | |
*** mkrai has quit IRC | 07:13 | |
*** mkrai has joined #openstack-ironic | 07:14 | |
*** rpittau|afk is now known as rpittau | 07:17 | |
rpittau | good morning ironic! o/ | 07:17 |
uzumaki | morning rpittau o/ | 07:18 |
rpittau | hey uzumaki, how's it going ? | 07:19 |
uzumaki | going well! how about you? | 07:19 |
rpittau | not bad :) | 07:20 |
*** ociuhandu has joined #openstack-ironic | 07:24 | |
uzumaki | well, I tested my code yesterday, and everything went really well, kinda glad for it! | 07:26 |
rpittau | that's great! | 07:28 |
*** iurygregory has quit IRC | 07:28 | |
uzumaki | yeah, PXE booting has been giving me nightmares, but as it turns out, server was booting from the wrong PXE port | 07:29 |
*** ociuhandu has quit IRC | 07:29 | |
uzumaki | so, finally, I get it to work, and i'm glad it did, I can wrap it up now | 07:29 |
*** iurygregory has joined #openstack-ironic | 07:29 | |
iurygregory | good morning Ironic | 07:30 |
uzumaki | o/ iurygregory | 07:30 |
iurygregory | uzumaki, o/ | 07:34 |
uzumaki | how you doing? | 07:35 |
iurygregory | doing good, and you? | 07:35 |
uzumaki | i'm well! thanks | 07:35 |
uzumaki | made any more adventurous trips lately? | 07:35 |
iurygregory | nope | 07:36 |
uzumaki | cool! | 07:37 |
iurygregory | yeah XD | 07:37 |
*** hjensas has joined #openstack-ironic | 07:43 | |
*** ociuhandu has joined #openstack-ironic | 07:55 | |
openstackgerrit | Riccardo Pittau proposed openstack/python-ironic-inspector-client master: Bump hacking to 3.0.0 https://review.opendev.org/716196 | 08:03 |
gudrutis2 | hi ironic o/ | 08:07 |
iurygregory | hey gudrutis2 | 08:08 |
uzumaki | hey gudrutis2 o/ | 08:12 |
*** lucasagomes has joined #openstack-ironic | 08:13 | |
*** yaawang has quit IRC | 08:23 | |
*** yaawang has joined #openstack-ironic | 08:25 | |
*** threestrands has quit IRC | 08:26 | |
iurygregory | rpittau, hey you around? =) | 08:35 |
rpittau | iurygregory: yep | 08:36 |
iurygregory | rpittau, can you check the config for the wholedisk + uefi https://review.opendev.org/#/c/714453/ I have the feeling I'm doing something wrong since it's failling =( | 08:37 |
patchbot | patch 714453 - ironic - Wholedisk+Uefi job - 8 patch sets | 08:37 |
iurygregory | I'm checking the logs from the nodes but haven't found anything suspicious | 08:38 |
*** derekh has joined #openstack-ironic | 08:42 | |
rpittau | iurygregory: have you seen this? https://d42863ba1342ece836c2-0c3df1d15a10c3537a68ce17608ebd20.ssl.cf5.rackcdn.com/714453/8/check/ironic-tempest-ipa-wholedisk-uefi-ipmi-direct-dib/3a8fd10/controller/logs/ironic-bm-logs/node-0_no_ansi_2020-03-30-18%3A49%3A26_log.txt | 08:44 |
iurygregory | damm it | 08:44 |
iurygregory | D: | 08:44 |
iurygregory | so it's trying network boot | 08:46 |
*** ociuhandu has quit IRC | 08:49 | |
*** ociuhandu has joined #openstack-ironic | 08:50 | |
rpittau | iurygregory: I would check the ironic.conf file to verify the boot interfaces | 08:50 |
iurygregory | enabled_boot_interfaces = fake,ipxe | 08:50 |
iurygregory | [deploy]default_boot_option = netboot | 08:50 |
iurygregory | if I do remember wholedisk should be using local | 08:52 |
iurygregory | or maybe I need more coffee | 08:52 |
*** whizzz has quit IRC | 08:53 | |
*** rcernin has quit IRC | 08:54 | |
*** priteau has joined #openstack-ironic | 08:59 | |
*** dtantsur|afk is now known as dtantsur | 09:00 | |
dtantsur | morning ironic | 09:00 |
iurygregory | morning dtantsur | 09:01 |
dtantsur | nice weather they have in some parts of Germany https://www.thelocal.de/userdata/images/1585579452_131358010.jpg | 09:02 |
*** alexmcleod has joined #openstack-ironic | 09:03 | |
rpittau | dtantsur: we're expecting snow too, if the temperatures stay this low | 09:03 |
dtantsur | it's nice and sunny here for now, hopefully will stay this way | 09:03 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/train: Extend install_bootloader command timeout https://review.opendev.org/716214 | 09:05 |
iurygregory | no snow in Brno, but max of 6°C min of -3 °C | 09:07 |
iurygregory | Spring? where are you? XD | 09:08 |
*** khansa has joined #openstack-ironic | 09:13 | |
uzumaki | dtantsur, morning! o/ | 09:18 |
uzumaki | iurygregory, Brno? what's that? (excuse my ignorance on geography) | 09:19 |
iurygregory | uzumaki, a city in Czech Republic =) | 09:19 |
uzumaki | It's raining here, nice and slow, with occasional slight gusts of wind.. quite good weather indeed! | 09:20 |
uzumaki | iurygregory, ah, I see! | 09:20 |
iurygregory | =) | 09:21 |
uzumaki | So, if I happen to be in Brno, I can expect to see you there? iurygregory | 09:22 |
rpittau | iurygregory: I can't not notice that uefi uses ovmf to boot, and I have nightmares, although seems working fine in other uefi jobs | 09:22 |
iurygregory | rpittau, yep =( | 09:23 |
iurygregory | I double check configs and seems correct | 09:23 |
iurygregory | it should use netboot and so on, maybe I forgot some ipxe param... | 09:23 |
iurygregory | uzumaki, yup | 09:23 |
uzumaki | that would be great! always nice to have friends all over the world xD | 09:24 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Add an ability to run in-band deploy steps https://review.opendev.org/698770 | 09:24 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: WIP: add a deploy steps for writing an image https://review.opendev.org/716216 | 09:24 |
iurygregory | uzumaki, but I'm not Czech =) | 09:24 |
uzumaki | well, all the same, if I can find you there, it's the same for me, no? | 09:24 |
uzumaki | although, would be cooler to have a native | 09:24 |
iurygregory | uzumaki, yeah, | 09:25 |
uzumaki | where are you from originally? iurygregory | 09:25 |
iurygregory | uzumaki, Brazil | 09:26 |
uzumaki | Oh! Moving all the way to CZ? | 09:26 |
iurygregory | uzumaki, yup =) | 09:26 |
uzumaki | interesting! | 09:27 |
iurygregory | you probably heard about the actual president of Brazil (now people call him Captain Corona | 09:28 |
uzumaki | I do remember there were elections, last year? bolsonaro, or something his name was, if i'm not mistaken, you're talking about him? | 09:28 |
iurygregory | 2018 the elections, yup it's him XD | 09:29 |
iurygregory | it's good that I moved before the elections hehe | 09:29 |
uzumaki | yeah, i've heard of him, not the appellation of Captain Corona though | 09:29 |
uzumaki | iurygregory, oh, so you've been in CZ >2 years? | 09:30 |
iurygregory | uzumaki, 1y 6 months | 09:31 |
uzumaki | ahan, that's cool. any luck with the local language? | 09:31 |
iurygregory | uzumaki, nope >.< | 09:31 |
uzumaki | iurygregory, argh! I'm sure you've given it enough thought though | 09:32 |
iurygregory | I only know how to order a Beer | 09:32 |
uzumaki | iurygregory, lol | 09:32 |
*** k_mouza has joined #openstack-ironic | 09:32 | |
iurygregory | uzumaki, hehehe | 09:32 |
uzumaki | when i write a release note, do I have to 'index' it somewhere as well? Or I just edit that reno file and I'm done? | 09:40 |
iurygregory | uzumaki, only edit the reno file =) | 09:40 |
uzumaki | and when I host the files locally, I browse to release notes and I should see my note there? | 09:40 |
iurygregory | yeah | 09:41 |
*** priteau has quit IRC | 09:41 | |
iurygregory | reno new <name> | 09:42 |
iurygregory | Created new notes file in releasenotes/notes/.......yaml | 09:42 |
iurygregory | you will see this =) | 09:42 |
uzumaki | yeah, I have the file, I edited it, built the with tox -ereleasenotes and now hosting in the /releasenotes/build/html but can't see my note :'( | 09:44 |
iurygregory | oh | 09:44 |
iurygregory | let me check here | 09:44 |
*** diga has joined #openstack-ironic | 09:45 | |
diga | Hi Ironic | 09:46 |
uzumaki | hey diga o/ | 09:46 |
diga | Are we also having online Ironic PTG ? | 09:46 |
*** ociuhandu has quit IRC | 09:46 | |
diga | Or still planning for it | 09:46 |
diga | uzumaki: Hey :) | 09:48 |
dtantsur | diga: it's a bit early to say for sure, but I think it will happen in some form | 09:48 |
dtantsur | the first virtual PTG planning activities will happen this week | 09:49 |
dtantsur | we'll have more information as we go | 09:49 |
diga | dtantsur: okay | 09:49 |
dtantsur | but I'm sure *something* will happen | 09:49 |
diga | dtantsur: :) Good | 09:49 |
uzumaki | yeah, we have meetings planned on april 2 7 and 10th I believe, on separate time zones | 09:49 |
diga | uzumaki: Thanks :) | 09:49 |
dtantsur | yep, I'll be able to attend, I think, two of them | 09:49 |
uzumaki | diga, no problem | 09:50 |
uzumaki | dtantsur, I'll try the same | 09:50 |
*** ociuhandu has joined #openstack-ironic | 09:58 | |
*** rcernin has joined #openstack-ironic | 09:59 | |
uzumaki | iurygregory, any luck with the reno? | 10:01 |
iurygregory | uzumaki, have you checked the unreleased.html file? | 10:03 |
uzumaki | let me try | 10:03 |
uzumaki | looking at it.. can't find my note there either | 10:05 |
uzumaki | how does the doc build know where to put my release note though? I"m wondering.. | 10:06 |
iurygregory | uzumaki, there is a logic in reno for this | 10:06 |
iurygregory | but yeah locally doesn't show .-. | 10:06 |
iurygregory | if the build goes ok it should be fine to update your patch | 10:07 |
uzumaki | well, yeah, the build succeeds successfully, then I believe I can jump onto writing the documentation update | 10:07 |
*** khansa has quit IRC | 10:07 | |
iurygregory | yeah | 10:08 |
*** yaawang has quit IRC | 10:08 | |
*** ociuhandu has quit IRC | 10:09 | |
*** yaawang has joined #openstack-ironic | 10:09 | |
*** ociuhandu has joined #openstack-ironic | 10:10 | |
uzumaki | iurygregory, oh, apparently, I have to cd into relesenotes/source when I run tox for it to recognize my new release note | 10:13 |
uzumaki | it appears under "working-copy" as the release number | 10:13 |
*** rcernin has quit IRC | 10:14 | |
*** ociuhandu has quit IRC | 10:14 | |
*** ociuhandu has joined #openstack-ironic | 10:14 | |
iurygregory | uzumaki, cool | 10:15 |
uzumaki | building the docs now, to check.. | 10:15 |
dtantsur | uzumaki: reno takes release notes from git, so if you don't commit a note, it won't be recognized | 10:17 |
uzumaki | dtantsur, I see. I wanted to build the note locally to see if I had anything amiss there. Thanks for telling me! | 10:17 |
dtantsur | you can create a commit, build the notes, then fix if needed | 10:17 |
dtantsur | maybe `git add` is enough, I don't remember | 10:18 |
uzumaki | Another thing, dtantsur, the cache_bios_settings wrapper is supposed to cache the settings _after_ a clean step of BIOS config is run, yes? But if the clean step is async, the cache will happen _before_ the step, not afterwards. | 10:19 |
iurygregory | yeah git add will do the job | 10:24 |
iurygregory | dtantsur, ++ | 10:24 |
*** jrist has quit IRC | 10:24 | |
uzumaki | oh, then maybe it was the git add! yeah, it's working now | 10:25 |
dtantsur | uzumaki: sorry, I don't remember how BIOS settings work by heart | 10:25 |
uzumaki | it's okay, it was just something interesting I came upon when testing yesterday. Will work with rpioso to determine the best course of action. That seems to be the only outstanding thing in my patch, everything else is in shape. | 10:26 |
dtantsur | I guess you can always cache them again (explicitly) after the step is over | 10:26 |
openstackgerrit | Merged openstack/sushy-tools master: Bump hacking to 3.0.0 https://review.opendev.org/715891 | 10:26 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Bump hacking to 3.0.0 https://review.opendev.org/715602 | 10:26 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [WIP] Reduce complexity of some functions https://review.opendev.org/716230 | 10:26 |
uzumaki | dtantsur, that's what I thought to be the easiest fix | 10:27 |
dtantsur | rpittau: the last patch conflicts with https://review.opendev.org/#/c/715907/ a bit | 10:27 |
patchbot | patch 715907 - ironic - Fix the remaining hacking issues - 2 patch sets | 10:27 |
rpittau | oh snap, didn't see that one | 10:28 |
* uzumaki thinks it funny that 'snap' is a package manager in Ubuntu | 10:29 | |
dtantsur | heh | 10:29 |
rpittau | oh dnf! (for par condicio) | 10:29 |
dtantsur | ++ | 10:29 |
dtantsur | oh yum did sound... yummier? | 10:30 |
iurygregory | yummy | 10:30 |
uzumaki | ikr? and it's weird they call it a "Yellowdog Update Manager" or something. It's just an excuse. Yum is a cool name anyway, they didn't want to admit it, is al | 10:31 |
rpittau | looking at that patch, mine is probably not needed :/ | 10:31 |
*** amoralej|off has quit IRC | 10:36 | |
*** rnoriega_ has quit IRC | 10:36 | |
rpittau | yay ModuleNotFoundError: No module named 'pymemcache' | 10:37 |
uzumaki | \o/ rpittau | 10:37 |
rpittau | this is tooz failing apparently | 10:38 |
rpittau | it requires pymemcache BUT it doesn't install it | 10:38 |
*** zzzeek has quit IRC | 10:44 | |
*** zzzeek has joined #openstack-ironic | 10:45 | |
*** rnoriega_ has joined #openstack-ironic | 10:51 | |
iurygregory | rpittau, damm =( | 10:53 |
openstackgerrit | Noor Muhammad Malik proposed openstack/ironic master: WIP: [RFE] Implement the bios-interface for idrac-wsman driver https://review.opendev.org/713450 | 10:55 |
uzumaki | gosh, finally, my patch makes it to Gerrit, took ages to send this one up. No idea why Gerrit was so slow.. | 10:55 |
openstackgerrit | Noor Muhammad Malik proposed openstack/ironic master: WIP: [RFE] Implement the bios-interface for idrac-wsman driver https://review.opendev.org/713450 | 10:59 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: [DNM] test ci https://review.opendev.org/716238 | 11:02 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: [DNM] test ci https://review.opendev.org/716238 | 11:05 |
*** rpittau is now known as rpittau|bbl | 11:05 | |
*** whizzz has joined #openstack-ironic | 11:21 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Add an ability to run in-band deploy steps https://review.opendev.org/698770 | 11:32 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: WIP: add a deploy steps for writing an image https://review.opendev.org/716216 | 11:32 |
dtantsur | uzumaki: gerrit is usually not very fast | 11:32 |
dtantsur | rpittau|bbl: we probably need to install it in our devstack plugin.. | 11:33 |
uzumaki | I know, it was slower than usual today which got me kinda worried/agitated. But everything's fine now.. | 11:33 |
dtantsur | happens | 11:33 |
rpittau|bbl | dtantsur: I'm just very surprised it's failing now | 11:34 |
dtantsur | rpittau|bbl: it's because devstack stopped installing test-requirements | 11:34 |
* uzumaki finds it cool that rpittau|bbl is not really 'back' yet and talking.. | 11:34 | |
dtantsur | uzumaki: telepathy | 11:34 |
uzumaki | dtantsur, indeed! rpittau|bbl the magician, the sorcerer.. | 11:35 |
dtantsur | :) | 11:35 |
rpittau|bbl | dtantsur: ok, that was my guess too | 11:35 |
rpittau|bbl | The open-sourcerer ? | 11:35 |
dtantsur | lol | 11:35 |
uzumaki | rpittau|bbl, the sorcerer doesn't guess, it knows! | 11:36 |
uzumaki | rpittau|bbl, lol! | 11:36 |
dtantsur | ... and this is not even Friday | 11:37 |
uzumaki | open-sourcerers are not your everyday average joe sorcerer's now, are they? dtantsur | 11:38 |
dtantsur | :D | 11:38 |
* uzumaki thinks rpittau|bbl should change his nick to rpittau|opensourcerer | 11:40 | |
iurygregory | dtantsur, do you have any ideas if im missing a configuration for wholedisk+uefi https://review.opendev.org/#/c/714453/ seems like the node coldn't netboot =( | 11:50 |
patchbot | patch 714453 - ironic - Wholedisk+Uefi job - 8 patch sets | 11:50 |
*** rnoriega_ has quit IRC | 11:50 | |
openstackgerrit | Ankit Kumar proposed openstack/ironic master: Generalize ISO building for virtual media driver https://review.opendev.org/677996 | 11:56 |
*** hjensas has quit IRC | 12:01 | |
*** dougsz has joined #openstack-ironic | 12:02 | |
*** rh-jelabarre has joined #openstack-ironic | 12:03 | |
*** ociuhandu has quit IRC | 12:04 | |
*** ociuhandu has joined #openstack-ironic | 12:04 | |
*** zzzeek has quit IRC | 12:05 | |
*** zzzeek has joined #openstack-ironic | 12:06 | |
openstackgerrit | Merged openstack/ironic-python-agent master: Rescan after restarting the md device https://review.opendev.org/706632 | 12:08 |
*** rnoriega_ has joined #openstack-ironic | 12:08 | |
*** ociuhandu has quit IRC | 12:10 | |
*** rpittau|bbl is now known as rpittau | 12:15 | |
uzumaki | \o/ he's finally back | 12:16 |
rpittau | lol | 12:16 |
rpittau | tuesday is the new friday ? | 12:17 |
uzumaki | more like, open-sourcerer is the stronger sorcerer | 12:17 |
iurygregory | rpittau, it can be since is end of month | 12:18 |
*** mkrai has quit IRC | 12:18 | |
* uzumaki thinks iurygregory has a very good hypothesis! EOM special abilities | 12:20 | |
iurygregory | yup | 12:20 |
rpittau | dtantsur: I'm wondering if we should move pymemcache to requirements since it's needed for inspector-api | 12:23 |
iurygregory | I like the idea ^ | 12:24 |
uzumaki | has anybody else seen zoom misbehave? it was quite funny yesterday, in our meeting, it kept throwing people out, definitely overloaded | 12:24 |
* iurygregory never used zoom | 12:25 | |
rpittau | uzumaki: I'm too old for that zoom s...tuff | 12:25 |
uzumaki | rpittau, haha! well, we use it with our meetings with Dell, and the way it kept throwing people out yesterday was annoying.. but also kinda hilarious | 12:26 |
rpittau | I feel like nowadays there are way too many tools for remote communication, and not a single one WAI | 12:28 |
rpittau | well maybe except pidgeons | 12:28 |
uzumaki | pideons? Oh I've heard of pidgeons, perhaps in context of IRC client | 12:28 |
uzumaki | rpittau, I agree, zoom is quite popular, and probably the reason why it gets very overloaded sometimes, same is the case with skype for business | 12:29 |
*** rh-jelabarre has quit IRC | 12:31 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: Move pymemcache to requirements https://review.opendev.org/716276 | 12:31 |
*** rh-jelabarre has joined #openstack-ironic | 12:31 | |
rpittau | I got impatient :P | 12:32 |
uzumaki | impatience is the best kind of patience sometimes, rpittau | 12:33 |
openstackgerrit | Verification of a change to openstack/ironic-inspector failed: Added scope to introspection rules. https://review.opendev.org/702215 | 12:34 |
openstackgerrit | Merged openstack/ironic master: Default IRONIC_RAMDISK_TYPE to dib https://review.opendev.org/710278 | 12:34 |
iurygregory | \o/ | 12:34 |
rpittau | oh wow | 12:34 |
rpittau | now I can start breaking stuff properly | 12:35 |
iurygregory | wut | 12:35 |
iurygregory | wtf | 12:35 |
iurygregory | zuul show -2 | 12:35 |
iurygregory | is showing | 12:35 |
uzumaki | well, apprently, zuul also got impatient.. | 12:35 |
iurygregory | WTF O.o | 12:35 |
rpittau | iurygregory: I don't see that | 12:35 |
iurygregory | LOL | 12:35 |
iurygregory | for me it shows -2 | 12:35 |
iurygregory | I will clean the cache... | 12:36 |
openstackgerrit | Merged openstack/ironic-python-agent master: Bump hacking to 3.0.0 https://review.opendev.org/715913 | 12:36 |
openstackgerrit | Merged openstack/python-ironicclient master: Bump hacking to 3.0.0 https://review.opendev.org/715886 | 12:36 |
rpittau | iurygregory: maybe it's the other patch ? | 12:37 |
iurygregory | rpittau, yup | 12:37 |
rpittau | https://review.opendev.org/702215 | 12:37 |
patchbot | patch 702215 - ironic-inspector - Added scope to introspection rules. - 17 patch sets | 12:37 |
rpittau | ok | 12:37 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: [DNM] test ci https://review.opendev.org/716238 | 12:39 |
*** hjensas has joined #openstack-ironic | 12:41 | |
*** hjensas is now known as hjensas|afk | 12:41 | |
*** rh-jelabarre has quit IRC | 12:44 | |
dtantsur | rpittau: we need to do it in devstack, it's not a requirement in a strict sense | 12:44 |
*** zzzeek has quit IRC | 12:44 | |
dtantsur | (analogous to mysql) | 12:44 |
rpittau | dtantsur: ok, I'll change the patch | 12:44 |
*** ociuhandu has joined #openstack-ironic | 12:45 | |
*** zzzeek has joined #openstack-ironic | 12:45 | |
dtantsur | iurygregory: re wholedisk+uefi, I'm not sure. maybe our cirros images are not capable of booting like that? | 12:47 |
iurygregory | gotcha =( | 12:48 |
dtantsur | iurygregory: although.. I don't see ironic calling into install_bootloader, that may be the problem | 12:48 |
dtantsur | oh, we're trying to netboot, right | 12:48 |
iurygregory | yeah | 12:48 |
rpittau | it could definitely be a problem with that specific interface not/wrongly defined in the cirros ipxe version | 12:52 |
dtantsur | reading https://forum.ipxe.org/showthread.php?tid=10327 sanboot probably shouldn't work | 12:52 |
*** rloo has joined #openstack-ironic | 12:52 | |
dtantsur | i.e. netboot + UEFI + whole disk images is probably broken | 12:52 |
iurygregory | ohh | 12:52 |
dtantsur | iurygregory: what if you try explicitly using boot_option:local | 12:52 |
iurygregory | dtantsur, will do | 12:52 |
rpittau | same for the specific code https://ipxe.org/err/2c2220 | 12:52 |
dtantsur | yeah | 12:53 |
openstackgerrit | Merged openstack/ironic stable/train: Extend install_bootloader command timeout https://review.opendev.org/716214 | 12:54 |
*** rh-jelabarre has joined #openstack-ironic | 12:54 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: Install pymemcache explicitly https://review.opendev.org/716276 | 12:55 |
*** ociuhandu has quit IRC | 12:58 | |
*** ociuhandu has joined #openstack-ironic | 12:58 | |
iurygregory | fingers crossed | 13:00 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Wholedisk+Uefi job https://review.opendev.org/714453 | 13:00 |
dtantsur | iurygregory: you seem to have an unrelated change at https://review.opendev.org/#/c/714453/9/zuul.d/project.yaml@12 | 13:01 |
patchbot | patch 714453 - ironic - Wholedisk+Uefi job - 9 patch sets | 13:01 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: Install pymemcache explicitly https://review.opendev.org/716276 | 13:01 |
iurygregory | wut I was pretty sure I did the rebase O.o | 13:02 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: Install pymemcache explicitly https://review.opendev.org/716276 | 13:02 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Wholedisk+Uefi job https://review.opendev.org/714453 | 13:02 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-inspector master: [DNM] test ci https://review.opendev.org/716238 | 13:02 |
*** ociuhandu has quit IRC | 13:04 | |
*** ociuhandu has joined #openstack-ironic | 13:04 | |
*** mbeierl has joined #openstack-ironic | 13:09 | |
*** ociuhandu has quit IRC | 13:15 | |
*** ociuhandu has joined #openstack-ironic | 13:16 | |
*** ociuhandu has quit IRC | 13:21 | |
dtantsur | mgoddard: hey, when you have a minute: https://review.opendev.org/#/c/698770/ seems more or less ready | 13:21 |
patchbot | patch 698770 - ironic-python-agent - Add an ability to run in-band deploy steps - 7 patch sets | 13:21 |
mgoddard | hi dtantsur. It will probably thursday at this point as I'm working for a customer today and have holiday tomorrow | 13:22 |
dtantsur | okay, no worries | 13:22 |
*** rnoriega_ has quit IRC | 13:22 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [WIP] Use openstacksdk to interact with swift https://review.opendev.org/714074 | 13:23 |
*** Goneri has joined #openstack-ironic | 13:26 | |
TheJulia | good morning | 13:30 |
*** rnoriega_ has joined #openstack-ironic | 13:32 | |
iurygregory | good morning TheJulia | 13:32 |
*** ddurst has quit IRC | 13:32 | |
*** ociuhandu has joined #openstack-ironic | 13:32 | |
*** ddurst has joined #openstack-ironic | 13:34 | |
*** zzzeek has quit IRC | 13:38 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: WIP: Decompose core deploy step https://review.opendev.org/698309 | 13:38 |
*** zzzeek has joined #openstack-ironic | 13:39 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Fix the remaining hacking issues https://review.opendev.org/715907 | 13:40 |
dtantsur | morning TheJulia | 13:40 |
dtantsur | TheJulia: could you also check https://review.opendev.org/#/c/715602/ please while you're on it? | 13:40 |
patchbot | patch 715602 - ironic - Bump hacking to 3.0.0 - 8 patch sets | 13:40 |
rpittau | good morning TheJulia :) | 13:42 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Change [deploy]/default_boot_option to local https://review.opendev.org/492114 | 13:44 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Add warning for deprecated get_default_boot_option https://review.opendev.org/716030 | 13:45 |
* rpittau stares into Python 3.8 deprecation abyss and it stares back | 13:46 | |
*** uzumaki has quit IRC | 13:46 | |
iurygregory | rpittau, do we have a lot of things that will need to change? O.o | 13:47 |
rpittau | us? no... others, yes.... | 13:48 |
dtantsur | yeah, we have quite a bit of dependencies that do | 13:48 |
iurygregory | oh ok =) | 13:48 |
dtantsur | ENOTMYPROBLEM :D | 13:48 |
rpittau | dtantsur: I *really* hope it won't be :) | 13:49 |
dtantsur | it probably will, heh | 13:49 |
TheJulia | rpittau: the question is does the abyss blink? | 13:50 |
iurygregory | ^ best question | 13:51 |
*** jdandrea has joined #openstack-ironic | 13:52 | |
*** tzumainn has joined #openstack-ironic | 13:54 | |
TheJulia | tzumainn: https://review.opendev.org/#/c/708678/ <-- latest after yesterday's discussion | 13:57 |
patchbot | patch 708678 - ironic - Use auth values from neutron conf when managing Ne... - 3 patch sets | 13:57 |
tzumainn | ah yes! did you want me to change something? | 14:00 |
*** bnemec has quit IRC | 14:00 | |
*** jtwill98 has joined #openstack-ironic | 14:01 | |
*** mbeierl has quit IRC | 14:07 | |
TheJulia | tzumainn: just confirming | 14:10 |
TheJulia | hopefully so I remember after my current meeting | 14:10 |
tzumainn | TheJulia, thanks! | 14:11 |
*** mkrai has joined #openstack-ironic | 14:13 | |
*** bnemec has joined #openstack-ironic | 14:16 | |
*** bnemec has quit IRC | 14:22 | |
*** ociuhandu has quit IRC | 14:24 | |
*** ociuhandu has joined #openstack-ironic | 14:26 | |
*** zzzeek has quit IRC | 14:28 | |
*** ociuhandu has quit IRC | 14:31 | |
rpittau | TheJulia: it does not :/ | 14:32 |
*** zzzeek has joined #openstack-ironic | 14:32 | |
*** ociuhandu has joined #openstack-ironic | 14:35 | |
TheJulia | https://storyboard.openstack.org/#!/story/2007470 | 14:38 |
TheJulia | dtantsur: testing jitter code fix now | 14:41 |
TheJulia | so patch should be up in ~10 minuts | 14:41 |
TheJulia | minutes | 14:41 |
TheJulia | well, acutally longer, I didn't tune values down | 14:42 |
*** tkajinam has quit IRC | 14:45 | |
*** hjensas|afk is now known as hjensas | 14:59 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Add jitter to inspection command reporting https://review.opendev.org/715005 | 15:13 |
*** tsturm2 has joined #openstack-ironic | 15:13 | |
dtantsur | good news, TripleO jobs started passing again | 15:25 |
TheJulia | \o/ | 15:25 |
TheJulia | just in time for us to merge more ?breaking? changes | 15:25 |
dtantsur | \o/ | 15:25 |
iurygregory | \o/ | 15:25 |
dtantsur | 2 weeks before the feature freeze, bring everything you have!! | 15:25 |
TheJulia | muahahahahaha | 15:25 |
TheJulia | If I wasn't stressing and worried about my depression, I would have kexec working :( | 15:26 |
iurygregory | TheJulia, you mean https://review.opendev.org/688581 ? | 15:26 |
patchbot | patch 688581 - ironic-python-agent - kexec standby extension - 7 patch sets | 15:26 |
TheJulia | iurygregory: oh, well that is only part of the puzzle | 15:27 |
dtantsur | we're all not in the best shape, don't worry | 15:27 |
TheJulia | that is actually fairly good I think | 15:27 |
TheJulia | it is all the conductor side logic that requires lots of work | 15:27 |
dtantsur | I have a feeling I should have finished this deploy steps business long ago | 15:27 |
TheJulia | because there are so many different possible cases | 15:27 |
dtantsur | you're going to hate me probably, but should we delay it till we have deploy steps? otherwise we may need to rework it again? | 15:27 |
dtantsur | (or maybe not) | 15:28 |
TheJulia | In my patches, I've been trying to focus on fairly narrow paths and building upon next logical steps, but less rebases | 15:28 |
TheJulia | dtantsur: I don't hate you! | 15:28 |
dtantsur | good, either of us shouldn't :sadsmile: | 15:28 |
TheJulia | dtantsur: It might be good to do so, it may simplify *some* conductor logic, the agent side stuff should basically stay the same though | 15:28 |
TheJulia | at least, that is my current perception, I don't see kexec beginning to merge until after we cut ironic 14.0.0 for this cycle | 15:29 |
TheJulia | I just want to get it up and working in my lab | 15:29 |
dtantsur | fair enough | 15:29 |
dtantsur | remind me, are we creating a new deploy interface for that? | 15:30 |
TheJulia | and the code is intertwined in palces that we're trying to merge other changes/fixes, so it basically stays out of merge conflict for only about 12 hours | 15:30 |
TheJulia | dtantsur: no, the idea is optional invocation using existing interfaces if the case is right, and eventually be able to kexec to a ramdisk from an active instance if so enabled. | 15:30 |
TheJulia | ^^^ that is like my goal | 15:31 |
dtantsur | That's probably correct. It's also inconsistent (IMO) with what we did for the ramdisk deploy interface (as JayF mentioned the other day) | 15:31 |
TheJulia | Yeah, but that was also an interface explicitly the scientific community's usage | 15:32 |
dtantsur | I borderline think a deploy interface should not have been a part of a driver | 15:32 |
dtantsur | Time for the driver composition reform 2.0? *runs away screaming* | 15:32 |
TheJulia | dtantsur: I'm kind of with you on that | 15:32 |
TheJulia | gah!!!! | 15:32 |
TheJulia | the breaking concept is the ansible deploy interface which I guess proves the point, that there can be variations | 15:32 |
dtantsur | yep | 15:33 |
TheJulia | and it does solve a nice use case of rapid prototyping of deployments | 15:33 |
dtantsur | can kexec be a deploy step that a deploy template enables instead of the standard reboot step? | 15:33 |
TheJulia | example: OVH first doing their raid in it and then in the actual agent ocde | 15:33 |
dtantsur | yep | 15:33 |
JayF | This is screaming for an ML2-style setup, a plugin for plugins :P | 15:34 |
TheJulia | dtantsur: possibly, there may need to be a meta step that could be invoked instead that just contains the "Can we kexec over or do we need to do a reboot" | 15:34 |
TheJulia | and then call the appropriate methods | 15:34 |
dtantsur | JayF: \o/ | 15:34 |
dtantsur | TheJulia: "can we kexec" is probably an operator's decision, no? | 15:34 |
JayF | I mean, it's a combination of "decision" and "image that supports it" | 15:35 |
TheJulia | dtantsur: mostly yes, although there need to be guards like "well, is the agent running" and "can we talk to the agent" | 15:35 |
JayF | it seems only like it being kexec-able has to do with the image, right? | 15:35 |
dtantsur | TheJulia: this applies to all deploy steps though | 15:35 |
JayF | or is the belief you can kexec the installer kernel into any image/userspace ... including Windows? BSD? anything? | 15:35 |
dtantsur | JayF: the other way around, I suspect there may be hardware that just doesn't like kexec | 15:35 |
JayF | I think it goes both ways | 15:35 |
TheJulia | JayF: this is also true, if we get just a wholedisk image, we can't kexec to it realistically... yet. Someone has code to re-trigger bootloaders but it is like early kexec from what I understand | 15:36 |
dtantsur | kexec into windows... sounds awesome, I can haz it? | 15:36 |
JayF | No, you can't, which was my point :P | 15:36 |
TheJulia | Actually | 15:36 |
JayF | there's some intersection of supporting hardware and supporting image | 15:36 |
TheJulia | It has been done... just nowhere widely | 15:36 |
dtantsur | Black magic is not for everyone indeed | 15:36 |
TheJulia | ++ | 15:36 |
TheJulia | its the universal something something something magic thing | 15:37 |
JayF | Unless you're willing to assert that any image can be kexec'd into (like some weird chainloader from a booted kernel stuff) it doesn't change the argument | 15:37 |
dtantsur | So, to my point: if the operators will have to explicitly enable/request kexec, and potentially do it per node, it may fit into the deploy template approach | 15:37 |
TheJulia | I can't think of the words, but there is actually a bootloader bytecode standard for it | 15:37 |
TheJulia | dtantsur: indeed, even in that, the logic likely needs to have "If this then that" sort of logic in case the machine is just in the non-kexec-able state OR the kexec operation fails to load if it is out of ram or something | 15:38 |
dtantsur | mmm, fair. what's the best course of actions then: fail or succeed but slowly? | 15:39 |
TheJulia | dtantsur: likely as an operator option | 15:39 |
TheJulia | trandles would be awesome to talk to about this | 15:40 |
dtantsur | It does feel suitable for a deploy step/template. Maybe I'm just seeing nails everywhere after spending a week or two working on a hammer.. | 15:40 |
JayF | it is a really nice hammer | 15:40 |
TheJulia | dtantsur: I mostly agree, but finish that hammer | 15:40 |
dtantsur | here comes the hardest part :) | 15:40 |
dtantsur | I think I'm close to getting it working (famous last words), but the question of backward compatibility... | 15:41 |
* TheJulia notes we'll then take the hammer, work on a plasma pulse generator mixed with a linear accelerator | 15:41 | |
dtantsur | I have a strong feeling that I'll break out-of-tree deploy interfaces, at least ones based on agent | 15:41 |
dtantsur | ++ | 15:41 |
TheJulia | dtantsur: hmmmm | 15:41 |
* TheJulia ponders | 15:41 | |
dtantsur | I mean, I don't intend to, but given how invasive the patches are... | 15:42 |
dtantsur | I wonder if we should just declare it a breaking change and not even try | 15:42 |
dtantsur | to be clear: I'm talking about the split of the core deploy step. Introducing the in-band deploy steps mechanism is probably fine. | 15:43 |
TheJulia | Yeah, I think your just going to need to delcare a breaking change | 15:45 |
TheJulia | but I'd be interested to look at the code as to how | 15:46 |
dtantsur | the problem is: we'll have agent deploy steps that are similar to existing functions and are sometimes even called the same, but have subtly different semantics | 15:47 |
JayF | Why do they need to be called the same? | 15:47 |
dtantsur | e.g. when it comes to handling states (DEPLOY WAIT or DEPLOYING) or polling for result | 15:47 |
JayF | Oh, in the conductor, not in the agent | 15:47 |
dtantsur | yep | 15:47 |
TheJulia | dtantsur: I think I just need to see it in code | 15:47 |
dtantsur | getting there | 15:47 |
JayF | I need to stop trying to "help" with limited knowledge and go back to the actual things I'm supposed to do :D | 15:47 |
TheJulia | I know :) | 15:47 |
JayF | you all keep making awesome hammers o/ | 15:48 |
dtantsur | any input is useful at this point | 15:48 |
dtantsur | especially from you :) | 15:48 |
dtantsur | speaking of deploy steps, https://review.opendev.org/#/c/698770/ is reviewable, I think | 15:48 |
patchbot | patch 698770 - ironic-python-agent - Add an ability to run in-band deploy steps - 7 patch sets | 15:48 |
*** bdodd has joined #openstack-ironic | 15:51 | |
*** etingof has quit IRC | 15:53 | |
*** dougsz has quit IRC | 15:56 | |
TheJulia | Perhaps we should make a reference to hammers in our next release notes | 15:57 |
TheJulia | prelude | 15:57 |
dtantsur | probably :) | 15:58 |
dtantsur | damn, the fact that IPA can only run one command at a time really breaks everything | 15:59 |
TheJulia | why? | 15:59 |
dtantsur | we end up with some mix of get_deploy_steps and execute_deploy_step command | 16:00 |
dtantsur | * commands | 16:00 |
*** dougsz has joined #openstack-ironic | 16:00 | |
dtantsur | mmm, I think I see how to handle it | 16:04 |
*** lucasagomes has quit IRC | 16:04 | |
rpittau | good night! o/ | 16:07 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [WIP] Use openstacksdk to interact with swift https://review.opendev.org/714074 | 16:07 |
*** rpittau is now known as rpittau|afk | 16:07 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: WIP: In-band deploy steps https://review.opendev.org/698768 | 16:08 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: WIP: Decompose core deploy step https://review.opendev.org/698309 | 16:08 |
*** dtantsur is now known as dtantsur|afk | 16:08 | |
dtantsur|afk | g'night | 16:08 |
*** bnemec has joined #openstack-ironic | 16:11 | |
openstackgerrit | Merged openstack/ironic master: Bump hacking to 3.0.0 https://review.opendev.org/715602 | 16:25 |
*** mkrai has quit IRC | 16:26 | |
*** dustinc has joined #openstack-ironic | 17:00 | |
*** k_mouza has quit IRC | 17:01 | |
*** derekh has quit IRC | 17:02 | |
*** k_mouza has joined #openstack-ironic | 17:03 | |
arne_wiebalck | bye everyone o/ | 17:05 |
TheJulia | o/ | 17:06 |
*** dougsz has quit IRC | 17:06 | |
*** alexmcleod has quit IRC | 17:17 | |
*** k_mouza has quit IRC | 17:27 | |
*** ociuhandu has quit IRC | 17:33 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Change [deploy]/default_boot_option to local https://review.opendev.org/492114 | 17:34 |
*** ociuhandu has joined #openstack-ironic | 17:34 | |
*** ociuhandu has quit IRC | 17:39 | |
openstackgerrit | Merged openstack/ironic master: Fix the remaining hacking issues https://review.opendev.org/715907 | 19:01 |
*** dougsz has joined #openstack-ironic | 19:04 | |
*** tsturm2 has quit IRC | 19:06 | |
TheJulia | rpioso: If chris is around, please let him know I have a comment on https://review.opendev.org/#/c/714491. That being said, if there is already testing to that effect, I'm good with +2'ing it. | 19:11 |
patchbot | patch 714491 - ironic - Fix issue where server fails to reboot - 2 patch sets | 19:11 |
*** dustinc has quit IRC | 19:23 | |
*** iurygregory has quit IRC | 19:27 | |
rpioso | TheJulia: I'll pass your message along to cdearborn. I agree with your comment. We're integration testing against baremetal, but, as you well know, that's different from unit testing. | 19:27 |
rpioso | TheJulia: Thank you so much for the review! | 19:27 |
TheJulia | Yeah, My concern regarding having unit testing is really ensuring we track/maintain method behavior contracts or explicitly know when we make changes | 19:32 |
TheJulia | I'm kind of surprised moving the code block didn't raise any issue, but maybe the unit testing is still good enough as is, but the helper method seems like it should at least have something specific now anyhow | 19:33 |
* TheJulia shrugs | 19:33 | |
rpioso | TheJulia: Absolutely! :-) | 19:36 |
* rpioso doesn't shrug ;-) | 19:36 | |
*** Lucas_Gray has joined #openstack-ironic | 19:41 | |
*** Lucas_Gray has quit IRC | 19:48 | |
openstackgerrit | Ankit Kumar proposed openstack/ironic master: Generalize ISO building for virtual media driver https://review.opendev.org/677996 | 19:49 |
*** Lucas_Gray has joined #openstack-ironic | 19:51 | |
*** jtwill98 has quit IRC | 20:01 | |
*** ociuhandu has joined #openstack-ironic | 20:12 | |
*** ociuhandu has quit IRC | 20:18 | |
*** Lucas_Gray has quit IRC | 20:31 | |
*** dougsz has quit IRC | 20:34 | |
*** Lucas_Gray has joined #openstack-ironic | 20:59 | |
*** rcernin has joined #openstack-ironic | 21:02 | |
*** Lucas_Gray has quit IRC | 21:03 | |
*** Lucas_Gray has joined #openstack-ironic | 21:04 | |
*** diga has quit IRC | 21:04 | |
*** iurygregory has joined #openstack-ironic | 21:21 | |
*** Wryhder has joined #openstack-ironic | 21:55 | |
*** Lucas_Gray has quit IRC | 21:55 | |
*** Wryhder is now known as Lucas_Gray | 21:56 | |
*** rloo has quit IRC | 22:14 | |
*** jdandrea has quit IRC | 22:25 | |
*** Lucas_Gray has quit IRC | 22:45 | |
*** tkajinam has joined #openstack-ironic | 22:49 | |
*** ianychoi has quit IRC | 23:15 | |
*** ianychoi has joined #openstack-ironic | 23:16 | |
*** rh-jelabarre has quit IRC | 23:43 | |
*** Goneri has quit IRC | 23:57 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!