Tuesday, 2020-03-31

*** jdandrea has quit IRC00:40
*** k_mouza has joined #openstack-ironic00:47
*** k_mouza has quit IRC00:52
*** TxGirlGeek has quit IRC01:04
*** ddurst has quit IRC01:18
*** ddurst has joined #openstack-ironic01:22
*** Goneri has quit IRC01:26
*** bnemec has quit IRC02:19
*** bnemec has joined #openstack-ironic02:28
*** zhenguo_ has joined #openstack-ironic02:33
*** k_mouza has joined #openstack-ironic02:51
*** k_mouza has quit IRC02:56
*** mkrai has joined #openstack-ironic03:10
openstackgerritMerged openstack/ironic master: Extend install_bootloader command timeout  https://review.opendev.org/71602103:12
*** threestrands has joined #openstack-ironic03:39
*** ociuhandu has joined #openstack-ironic03:43
*** ociuhandu has quit IRC03:48
*** zhenguo_ has quit IRC04:42
*** bdodd has quit IRC04:54
*** k_mouza has joined #openstack-ironic04:56
*** k_mouza has quit IRC05:01
*** tzumainn has quit IRC05:06
*** mkrai has quit IRC05:48
*** mkrai has joined #openstack-ironic05:48
openstackgerritAnkit Kumar proposed openstack/ironic master: Generalize ISO building for virtual media driver  https://review.opendev.org/67799606:34
*** etingof has joined #openstack-ironic06:45
uzumakimorning ironic! o/06:45
*** hjensas has quit IRC06:54
arne_wiebalckGood morning, ironic!06:56
*** k_mouza has joined #openstack-ironic06:57
uzumakihow you doing arne_wiebalck ?06:57
arne_wiebalckthanks, all good over here ... and you?06:58
uzumakii'm well thanks!06:59
*** k_mouza has quit IRC07:02
*** mkrai has quit IRC07:13
*** mkrai has joined #openstack-ironic07:14
*** rpittau|afk is now known as rpittau07:17
rpittaugood morning ironic! o/07:17
uzumakimorning rpittau o/07:18
rpittauhey uzumaki, how's it going ?07:19
uzumakigoing well! how about you?07:19
rpittaunot bad :)07:20
*** ociuhandu has joined #openstack-ironic07:24
uzumakiwell, I tested my code yesterday, and everything went really well, kinda glad for it!07:26
rpittauthat's great!07:28
*** iurygregory has quit IRC07:28
uzumakiyeah, PXE booting has been giving me nightmares, but as it turns out, server was booting from the wrong PXE port07:29
*** ociuhandu has quit IRC07:29
uzumakiso, finally, I get it to work, and i'm glad it did, I can wrap it up now07:29
*** iurygregory has joined #openstack-ironic07:29
iurygregorygood morning Ironic07:30
uzumakio/ iurygregory07:30
iurygregoryuzumaki, o/07:34
uzumakihow you doing?07:35
iurygregorydoing good, and you?07:35
uzumakii'm well! thanks07:35
uzumakimade any more adventurous trips lately?07:35
iurygregorynope07:36
uzumakicool!07:37
iurygregoryyeah XD07:37
*** hjensas has joined #openstack-ironic07:43
*** ociuhandu has joined #openstack-ironic07:55
openstackgerritRiccardo Pittau proposed openstack/python-ironic-inspector-client master: Bump hacking to 3.0.0  https://review.opendev.org/71619608:03
gudrutis2hi ironic o/08:07
iurygregoryhey gudrutis208:08
uzumakihey gudrutis2 o/08:12
*** lucasagomes has joined #openstack-ironic08:13
*** yaawang has quit IRC08:23
*** yaawang has joined #openstack-ironic08:25
*** threestrands has quit IRC08:26
iurygregoryrpittau, hey you around? =)08:35
rpittauiurygregory: yep08:36
iurygregoryrpittau, 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
patchbotpatch 714453 - ironic - Wholedisk+Uefi job - 8 patch sets08:37
iurygregoryI'm checking the logs from the nodes but haven't found anything suspicious08:38
*** derekh has joined #openstack-ironic08:42
rpittauiurygregory: 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.txt08:44
iurygregorydamm it08:44
iurygregoryD:08:44
iurygregoryso it's trying network boot08:46
*** ociuhandu has quit IRC08:49
*** ociuhandu has joined #openstack-ironic08:50
rpittauiurygregory: I would check the ironic.conf file to verify the boot interfaces08:50
iurygregoryenabled_boot_interfaces = fake,ipxe08:50
iurygregory[deploy]default_boot_option = netboot08:50
iurygregoryif I do remember wholedisk should be using local08:52
iurygregoryor maybe I need more coffee08:52
*** whizzz has quit IRC08:53
*** rcernin has quit IRC08:54
*** priteau has joined #openstack-ironic08:59
*** dtantsur|afk is now known as dtantsur09:00
dtantsurmorning ironic09:00
iurygregorymorning dtantsur09:01
dtantsurnice weather they have in some parts of Germany https://www.thelocal.de/userdata/images/1585579452_131358010.jpg09:02
*** alexmcleod has joined #openstack-ironic09:03
rpittaudtantsur: we're expecting snow too, if the temperatures stay this low09:03
dtantsurit's nice and sunny here for now, hopefully will stay this way09:03
openstackgerritDmitry Tantsur proposed openstack/ironic stable/train: Extend install_bootloader command timeout  https://review.opendev.org/71621409:05
iurygregoryno snow in Brno, but max of 6°C min of -3 °C09:07
iurygregorySpring? where are you? XD09:08
*** khansa has joined #openstack-ironic09:13
uzumakidtantsur, morning! o/09:18
uzumakiiurygregory, Brno? what's that? (excuse my ignorance on geography)09:19
iurygregoryuzumaki, a city in Czech Republic =)09:19
uzumakiIt's raining here, nice and slow, with occasional slight gusts of wind.. quite good weather indeed!09:20
uzumakiiurygregory, ah, I see!09:20
iurygregory=)09:21
uzumakiSo, if I happen to be in Brno, I can expect to see you there? iurygregory09:22
rpittauiurygregory: I can't not notice that uefi uses ovmf to boot, and I have nightmares, although seems working fine in other uefi jobs09:22
iurygregoryrpittau, yep =(09:23
iurygregoryI double check configs and seems correct09:23
iurygregoryit should use netboot and so on, maybe I forgot some ipxe param...09:23
iurygregoryuzumaki, yup09:23
uzumakithat would be great! always nice to have friends all over the world xD09:24
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Add an ability to run in-band deploy steps  https://review.opendev.org/69877009:24
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: WIP: add a deploy steps for writing an image  https://review.opendev.org/71621609:24
iurygregoryuzumaki, but I'm not Czech =)09:24
uzumakiwell, all the same, if I can find you there, it's the same for me, no?09:24
uzumakialthough, would be cooler to have a native09:24
iurygregoryuzumaki, yeah,09:25
uzumakiwhere are you from originally? iurygregory09:25
iurygregoryuzumaki, Brazil09:26
uzumakiOh! Moving all the way to CZ?09:26
iurygregoryuzumaki, yup =)09:26
uzumakiinteresting!09:27
iurygregoryyou probably heard about the actual president of Brazil (now people call him Captain Corona09:28
uzumakiI 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
iurygregory2018 the elections, yup it's him XD09:29
iurygregoryit's good that I moved before the elections hehe09:29
uzumakiyeah, i've heard of him, not the appellation of Captain Corona though09:29
uzumakiiurygregory, oh, so you've been in CZ >2 years?09:30
iurygregoryuzumaki, 1y 6 months09:31
uzumakiahan, that's cool. any luck with the local language?09:31
iurygregoryuzumaki, nope >.<09:31
uzumakiiurygregory, argh! I'm sure you've given it enough thought though09:32
iurygregoryI only know how to order a Beer09:32
uzumakiiurygregory, lol09:32
*** k_mouza has joined #openstack-ironic09:32
iurygregoryuzumaki, hehehe09:32
uzumakiwhen 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
iurygregoryuzumaki, only edit the reno file =)09:40
uzumakiand when I host the files locally, I browse to release notes and I should see my note there?09:40
iurygregoryyeah09:41
*** priteau has quit IRC09:41
iurygregoryreno new <name>09:42
iurygregoryCreated new notes file in releasenotes/notes/.......yaml09:42
iurygregoryyou will see this =)09:42
uzumakiyeah, 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
iurygregoryoh09:44
iurygregorylet me check here09:44
*** diga has joined #openstack-ironic09:45
digaHi Ironic09:46
uzumakihey diga o/09:46
digaAre we also having online Ironic PTG ?09:46
*** ociuhandu has quit IRC09:46
digaOr still planning for it09:46
digauzumaki: Hey :)09:48
dtantsurdiga: it's a bit early to say for sure, but I think it will happen in some form09:48
dtantsurthe first virtual PTG planning activities will happen this week09:49
dtantsurwe'll have more information as we go09:49
digadtantsur: okay09:49
dtantsurbut I'm sure *something* will happen09:49
digadtantsur: :) Good09:49
uzumakiyeah, we have meetings planned on april 2 7 and 10th I believe, on separate time zones09:49
digauzumaki:  Thanks :)09:49
dtantsuryep, I'll be able to attend, I think, two of them09:49
uzumakidiga, no problem09:50
uzumakidtantsur, I'll try the same09:50
*** ociuhandu has joined #openstack-ironic09:58
*** rcernin has joined #openstack-ironic09:59
uzumakiiurygregory, any luck with the reno?10:01
iurygregoryuzumaki, have you checked the unreleased.html file?10:03
uzumakilet me try10:03
uzumakilooking at it.. can't find my note there either10:05
uzumakihow does the doc build know where to put my release note though? I"m wondering..10:06
iurygregoryuzumaki, there is a logic in reno for this10:06
iurygregorybut yeah locally doesn't show .-.10:06
iurygregoryif the build goes ok it should be fine to update your patch10:07
uzumakiwell, yeah, the build succeeds successfully, then I believe I can jump onto writing the documentation update10:07
*** khansa has quit IRC10:07
iurygregoryyeah10:08
*** yaawang has quit IRC10:08
*** ociuhandu has quit IRC10:09
*** yaawang has joined #openstack-ironic10:09
*** ociuhandu has joined #openstack-ironic10:10
uzumakiiurygregory, oh, apparently, I have to cd into relesenotes/source when I run tox for it to recognize my new release note10:13
uzumakiit appears under "working-copy" as the release number10:13
*** rcernin has quit IRC10:14
*** ociuhandu has quit IRC10:14
*** ociuhandu has joined #openstack-ironic10:14
iurygregoryuzumaki, cool10:15
uzumakibuilding the docs now, to check..10:15
dtantsuruzumaki: reno takes release notes from git, so if you don't commit a note, it won't be recognized10:17
uzumakidtantsur, I see. I wanted to build the note locally to see if I had anything amiss there. Thanks for telling me!10:17
dtantsuryou can create a commit, build the notes, then fix if needed10:17
dtantsurmaybe `git add` is enough, I don't remember10:18
uzumakiAnother 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
iurygregoryyeah git add will do the job10:24
iurygregorydtantsur, ++10:24
*** jrist has quit IRC10:24
uzumakioh, then maybe it was the git add! yeah, it's working now10:25
dtantsuruzumaki: sorry, I don't remember how BIOS settings work by heart10:25
uzumakiit'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
dtantsurI guess you can always cache them again (explicitly) after the step is over10:26
openstackgerritMerged openstack/sushy-tools master: Bump hacking to 3.0.0  https://review.opendev.org/71589110:26
openstackgerritRiccardo Pittau proposed openstack/ironic master: Bump hacking to 3.0.0  https://review.opendev.org/71560210:26
openstackgerritRiccardo Pittau proposed openstack/ironic master: [WIP] Reduce complexity of some functions  https://review.opendev.org/71623010:26
uzumakidtantsur, that's what I thought to be the easiest fix10:27
dtantsurrpittau: the last patch conflicts with https://review.opendev.org/#/c/715907/ a bit10:27
patchbotpatch 715907 - ironic - Fix the remaining hacking issues - 2 patch sets10:27
rpittauoh snap, didn't see that one10:28
* uzumaki thinks it funny that 'snap' is a package manager in Ubuntu10:29
dtantsurheh10:29
rpittauoh dnf! (for par condicio)10:29
dtantsur++10:29
dtantsuroh yum did sound... yummier?10:30
iurygregoryyummy10:30
uzumakiikr? 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 al10:31
rpittaulooking at that patch, mine is probably not needed :/10:31
*** amoralej|off has quit IRC10:36
*** rnoriega_ has quit IRC10:36
rpittauyay ModuleNotFoundError: No module named 'pymemcache'10:37
uzumaki\o/ rpittau10:37
rpittauthis is tooz failing apparently10:38
rpittauit requires pymemcache BUT it doesn't install it10:38
*** zzzeek has quit IRC10:44
*** zzzeek has joined #openstack-ironic10:45
*** rnoriega_ has joined #openstack-ironic10:51
iurygregoryrpittau, damm =(10:53
openstackgerritNoor Muhammad Malik proposed openstack/ironic master: WIP: [RFE] Implement the bios-interface for idrac-wsman driver  https://review.opendev.org/71345010:55
uzumakigosh, finally, my patch makes it to Gerrit, took ages to send this one up. No idea why Gerrit was so slow..10:55
openstackgerritNoor Muhammad Malik proposed openstack/ironic master: WIP: [RFE] Implement the bios-interface for idrac-wsman driver  https://review.opendev.org/71345010:59
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: [DNM] test ci  https://review.opendev.org/71623811:02
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: [DNM] test ci  https://review.opendev.org/71623811:05
*** rpittau is now known as rpittau|bbl11:05
*** whizzz has joined #openstack-ironic11:21
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Add an ability to run in-band deploy steps  https://review.opendev.org/69877011:32
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: WIP: add a deploy steps for writing an image  https://review.opendev.org/71621611:32
dtantsuruzumaki: gerrit is usually not very fast11:32
dtantsurrpittau|bbl: we probably need to install it in our devstack plugin..11:33
uzumakiI know, it was slower than usual today which got me kinda worried/agitated. But everything's fine now..11:33
dtantsurhappens11:33
rpittau|bbldtantsur: I'm just very surprised it's failing now11:34
dtantsurrpittau|bbl: it's because devstack stopped installing test-requirements11:34
* uzumaki finds it cool that rpittau|bbl is not really 'back' yet and talking..11:34
dtantsuruzumaki: telepathy11:34
uzumakidtantsur, indeed! rpittau|bbl the magician, the sorcerer..11:35
dtantsur:)11:35
rpittau|bbldtantsur: ok, that was my guess too11:35
rpittau|bblThe open-sourcerer ?11:35
dtantsurlol11:35
uzumakirpittau|bbl, the sorcerer doesn't guess, it knows!11:36
uzumakirpittau|bbl, lol!11:36
dtantsur... and this is not even Friday11:37
uzumakiopen-sourcerers are not your everyday average joe sorcerer's now, are they? dtantsur11:38
dtantsur:D11:38
* uzumaki thinks rpittau|bbl should change his nick to rpittau|opensourcerer11:40
iurygregorydtantsur, 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
patchbotpatch 714453 - ironic - Wholedisk+Uefi job - 8 patch sets11:50
*** rnoriega_ has quit IRC11:50
openstackgerritAnkit Kumar proposed openstack/ironic master: Generalize ISO building for virtual media driver  https://review.opendev.org/67799611:56
*** hjensas has quit IRC12:01
*** dougsz has joined #openstack-ironic12:02
*** rh-jelabarre has joined #openstack-ironic12:03
*** ociuhandu has quit IRC12:04
*** ociuhandu has joined #openstack-ironic12:04
*** zzzeek has quit IRC12:05
*** zzzeek has joined #openstack-ironic12:06
openstackgerritMerged openstack/ironic-python-agent master: Rescan after restarting the md device  https://review.opendev.org/70663212:08
*** rnoriega_ has joined #openstack-ironic12:08
*** ociuhandu has quit IRC12:10
*** rpittau|bbl is now known as rpittau12:15
uzumaki\o/ he's finally back12:16
rpittaulol12:16
rpittautuesday is the new friday ?12:17
uzumakimore like, open-sourcerer is the stronger sorcerer12:17
iurygregoryrpittau, it can be since is end of month12:18
*** mkrai has quit IRC12:18
* uzumaki thinks iurygregory has a very good hypothesis! EOM special abilities12:20
iurygregoryyup12:20
rpittaudtantsur: I'm wondering if we should move pymemcache to requirements since it's needed for inspector-api12:23
iurygregoryI like the idea ^12:24
uzumakihas anybody else seen zoom misbehave? it was quite funny yesterday, in our meeting, it kept throwing people out, definitely overloaded12:24
* iurygregory never used zoom12:25
rpittauuzumaki: I'm too old for that zoom s...tuff12:25
uzumakirpittau, haha! well, we use it with our meetings with Dell, and the way it kept throwing people out yesterday was annoying.. but also kinda hilarious12:26
rpittauI feel like nowadays there are way too many tools for remote communication, and not a single one WAI12:28
rpittauwell maybe except pidgeons12:28
uzumakipideons? Oh I've heard of pidgeons, perhaps in context of IRC client12:28
uzumakirpittau, I agree, zoom is quite popular, and probably the reason why it gets very overloaded sometimes, same is the case with skype for business12:29
*** rh-jelabarre has quit IRC12:31
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Move pymemcache to requirements  https://review.opendev.org/71627612:31
*** rh-jelabarre has joined #openstack-ironic12:31
rpittauI got impatient :P12:32
uzumakiimpatience is the best kind of patience sometimes, rpittau12:33
openstackgerritVerification of a change to openstack/ironic-inspector failed: Added scope to introspection rules.  https://review.opendev.org/70221512:34
openstackgerritMerged openstack/ironic master: Default IRONIC_RAMDISK_TYPE to dib  https://review.opendev.org/71027812:34
iurygregory\o/12:34
rpittauoh wow12:34
rpittaunow I can start breaking stuff properly12:35
iurygregorywut12:35
iurygregorywtf12:35
iurygregoryzuul show -212:35
iurygregoryis showing12:35
uzumakiwell, apprently, zuul also got impatient..12:35
iurygregoryWTF O.o12:35
rpittauiurygregory: I don't see that12:35
iurygregoryLOL12:35
iurygregoryfor me it shows -212:35
iurygregoryI will clean the cache...12:36
openstackgerritMerged openstack/ironic-python-agent master: Bump hacking to 3.0.0  https://review.opendev.org/71591312:36
openstackgerritMerged openstack/python-ironicclient master: Bump hacking to 3.0.0  https://review.opendev.org/71588612:36
rpittauiurygregory: maybe it's the other patch ?12:37
iurygregoryrpittau, yup12:37
rpittauhttps://review.opendev.org/70221512:37
patchbotpatch 702215 - ironic-inspector - Added scope to introspection rules. - 17 patch sets12:37
rpittauok12:37
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: [DNM] test ci  https://review.opendev.org/71623812:39
*** hjensas has joined #openstack-ironic12:41
*** hjensas is now known as hjensas|afk12:41
*** rh-jelabarre has quit IRC12:44
dtantsurrpittau: we need to do it in devstack, it's not a requirement in a strict sense12:44
*** zzzeek has quit IRC12:44
dtantsur(analogous to mysql)12:44
rpittaudtantsur: ok, I'll change the patch12:44
*** ociuhandu has joined #openstack-ironic12:45
*** zzzeek has joined #openstack-ironic12:45
dtantsuriurygregory: re wholedisk+uefi, I'm not sure. maybe our cirros images are not capable of booting like that?12:47
iurygregorygotcha =(12:48
dtantsuriurygregory: although.. I don't see ironic calling into install_bootloader, that may be the problem12:48
dtantsuroh, we're trying to netboot, right12:48
iurygregoryyeah12:48
rpittauit could definitely be a problem with that specific interface not/wrongly defined in the cirros ipxe version12:52
dtantsurreading https://forum.ipxe.org/showthread.php?tid=10327 sanboot probably shouldn't work12:52
*** rloo has joined #openstack-ironic12:52
dtantsuri.e. netboot + UEFI + whole disk images is probably broken12:52
iurygregoryohh12:52
dtantsuriurygregory: what if you try explicitly using boot_option:local12:52
iurygregorydtantsur, will do12:52
rpittausame for the specific code https://ipxe.org/err/2c222012:52
dtantsuryeah12:53
openstackgerritMerged openstack/ironic stable/train: Extend install_bootloader command timeout  https://review.opendev.org/71621412:54
*** rh-jelabarre has joined #openstack-ironic12:54
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Install pymemcache explicitly  https://review.opendev.org/71627612:55
*** ociuhandu has quit IRC12:58
*** ociuhandu has joined #openstack-ironic12:58
iurygregoryfingers crossed13:00
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Wholedisk+Uefi job  https://review.opendev.org/71445313:00
dtantsuriurygregory: you seem to have an unrelated change at https://review.opendev.org/#/c/714453/9/zuul.d/project.yaml@1213:01
patchbotpatch 714453 - ironic - Wholedisk+Uefi job - 9 patch sets13:01
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Install pymemcache explicitly  https://review.opendev.org/71627613:01
iurygregorywut I was pretty sure I did the rebase O.o13:02
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Install pymemcache explicitly  https://review.opendev.org/71627613:02
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Wholedisk+Uefi job  https://review.opendev.org/71445313:02
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: [DNM] test ci  https://review.opendev.org/71623813:02
*** ociuhandu has quit IRC13:04
*** ociuhandu has joined #openstack-ironic13:04
*** mbeierl has joined #openstack-ironic13:09
*** ociuhandu has quit IRC13:15
*** ociuhandu has joined #openstack-ironic13:16
*** ociuhandu has quit IRC13:21
dtantsurmgoddard: hey, when you have a minute: https://review.opendev.org/#/c/698770/ seems more or less ready13:21
patchbotpatch 698770 - ironic-python-agent - Add an ability to run in-band deploy steps - 7 patch sets13:21
mgoddardhi dtantsur. It will probably thursday at this point as I'm working for a customer today and have holiday tomorrow13:22
dtantsurokay, no worries13:22
*** rnoriega_ has quit IRC13:22
openstackgerritRiccardo Pittau proposed openstack/ironic master: [WIP] Use openstacksdk to interact with swift  https://review.opendev.org/71407413:23
*** Goneri has joined #openstack-ironic13:26
TheJuliagood morning13:30
*** rnoriega_ has joined #openstack-ironic13:32
iurygregorygood morning TheJulia13:32
*** ddurst has quit IRC13:32
*** ociuhandu has joined #openstack-ironic13:32
*** ddurst has joined #openstack-ironic13:34
*** zzzeek has quit IRC13:38
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: Decompose core deploy step  https://review.opendev.org/69830913:38
*** zzzeek has joined #openstack-ironic13:39
openstackgerritDmitry Tantsur proposed openstack/ironic master: Fix the remaining hacking issues  https://review.opendev.org/71590713:40
dtantsurmorning TheJulia13:40
dtantsurTheJulia: could you also check https://review.opendev.org/#/c/715602/ please while you're on it?13:40
patchbotpatch 715602 - ironic - Bump hacking to 3.0.0 - 8 patch sets13:40
rpittaugood morning TheJulia :)13:42
openstackgerritJulia Kreger proposed openstack/ironic master: Change [deploy]/default_boot_option to local  https://review.opendev.org/49211413:44
openstackgerritJulia Kreger proposed openstack/ironic master: Add warning for deprecated get_default_boot_option  https://review.opendev.org/71603013:45
* rpittau stares into Python 3.8 deprecation abyss and it stares back13:46
*** uzumaki has quit IRC13:46
iurygregoryrpittau, do we have a lot of things that will need to change? O.o13:47
rpittauus? no... others, yes....13:48
dtantsuryeah, we have quite a bit of dependencies that do13:48
iurygregoryoh ok =)13:48
dtantsurENOTMYPROBLEM :D13:48
rpittaudtantsur: I *really* hope it won't be :)13:49
dtantsurit probably will, heh13:49
TheJuliarpittau: the question is does the abyss blink?13:50
iurygregory^ best question13:51
*** jdandrea has joined #openstack-ironic13:52
*** tzumainn has joined #openstack-ironic13:54
TheJuliatzumainn: https://review.opendev.org/#/c/708678/ <-- latest after yesterday's discussion13:57
patchbotpatch 708678 - ironic - Use auth values from neutron conf when managing Ne... - 3 patch sets13:57
tzumainnah yes! did you want me to change something?14:00
*** bnemec has quit IRC14:00
*** jtwill98 has joined #openstack-ironic14:01
*** mbeierl has quit IRC14:07
TheJuliatzumainn: just confirming14:10
TheJuliahopefully so I remember after my current meeting14:10
tzumainnTheJulia, thanks!14:11
*** mkrai has joined #openstack-ironic14:13
*** bnemec has joined #openstack-ironic14:16
*** bnemec has quit IRC14:22
*** ociuhandu has quit IRC14:24
*** ociuhandu has joined #openstack-ironic14:26
*** zzzeek has quit IRC14:28
*** ociuhandu has quit IRC14:31
rpittauTheJulia: it does not :/14:32
*** zzzeek has joined #openstack-ironic14:32
*** ociuhandu has joined #openstack-ironic14:35
TheJuliahttps://storyboard.openstack.org/#!/story/200747014:38
TheJuliadtantsur: testing jitter code fix now14:41
TheJuliaso patch should be up in ~10 minuts14:41
TheJuliaminutes14:41
TheJuliawell, acutally longer, I didn't tune values down14:42
*** tkajinam has quit IRC14:45
*** hjensas|afk is now known as hjensas14:59
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Add jitter to inspection command reporting  https://review.opendev.org/71500515:13
*** tsturm2 has joined #openstack-ironic15:13
dtantsurgood news, TripleO jobs started passing again15:25
TheJulia\o/15:25
TheJuliajust in time for us to merge more ?breaking? changes15:25
dtantsur\o/15:25
iurygregory\o/15:25
dtantsur2 weeks before the feature freeze, bring everything you have!!15:25
TheJuliamuahahahahaha15:25
TheJuliaIf I wasn't stressing and worried about my depression, I would have kexec working :(15:26
iurygregoryTheJulia, you mean https://review.opendev.org/688581 ?15:26
patchbotpatch 688581 - ironic-python-agent - kexec standby extension - 7 patch sets15:26
TheJuliaiurygregory: oh, well that is only part of the puzzle15:27
dtantsurwe're all not in the best shape, don't worry15:27
TheJuliathat is actually fairly good I think15:27
TheJuliait is all the conductor side logic that requires lots of work15:27
dtantsurI have a feeling I should have finished this deploy steps business long ago15:27
TheJuliabecause there are so many different possible cases15:27
dtantsuryou'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
TheJuliaIn my patches, I've been trying to focus on fairly narrow paths and building upon next logical steps, but less rebases15:28
TheJuliadtantsur: I don't hate you!15:28
dtantsurgood, either of us shouldn't :sadsmile:15:28
TheJuliadtantsur: It might be good to do so, it may simplify *some* conductor logic, the agent side stuff should basically stay the same though15:28
TheJuliaat least, that is my current perception, I don't see kexec beginning to merge until after we cut ironic 14.0.0 for this cycle15:29
TheJuliaI just want to get it up and working in my lab15:29
dtantsurfair enough15:29
dtantsurremind me, are we creating a new deploy interface for that?15:30
TheJuliaand 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 hours15:30
TheJuliadtantsur: 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 goal15:31
dtantsurThat'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
TheJuliaYeah, but that was also an interface explicitly the scientific community's usage15:32
dtantsurI borderline think a deploy interface should not have been a part of a driver15:32
dtantsurTime for the driver composition reform 2.0? *runs away screaming*15:32
TheJuliadtantsur: I'm kind of with you on that15:32
TheJuliagah!!!!15:32
TheJuliathe breaking concept is the ansible deploy interface which I guess proves the point, that there can be variations15:32
dtantsuryep15:33
TheJuliaand it does solve a nice use case of rapid prototyping of deployments15:33
dtantsurcan kexec be a deploy step that a deploy template enables instead of the standard reboot step?15:33
TheJuliaexample: OVH first doing their raid in it and then in the actual agent ocde15:33
dtantsuryep15:33
JayFThis is screaming for an ML2-style setup, a plugin for plugins :P15:34
TheJuliadtantsur: 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
TheJuliaand then call the appropriate methods15:34
dtantsurJayF: \o/15:34
dtantsurTheJulia: "can we kexec" is probably an operator's decision, no?15:34
JayFI mean, it's a combination of "decision" and "image that supports it"15:35
TheJuliadtantsur: mostly yes, although there need to be guards like "well, is the agent running" and "can we talk to the agent"15:35
JayFit seems only like it being kexec-able has to do with the image, right?15:35
dtantsurTheJulia: this applies to all deploy steps though15:35
JayFor is the belief you can kexec the installer kernel into any image/userspace ... including Windows? BSD? anything?15:35
dtantsurJayF: the other way around, I suspect there may be hardware that just doesn't like kexec15:35
JayFI think it goes both ways15:35
TheJuliaJayF: 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 understand15:36
dtantsurkexec into windows... sounds awesome, I can haz it?15:36
JayFNo, you can't, which was my point :P15:36
TheJuliaActually15:36
JayFthere's some intersection of supporting hardware and supporting image15:36
TheJuliaIt has been done... just nowhere widely15:36
dtantsurBlack magic is not for everyone indeed15:36
TheJulia++15:36
TheJuliaits the universal something something something magic thing15:37
JayFUnless 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 argument15:37
dtantsurSo, 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 approach15:37
TheJuliaI can't think of the words, but there is actually a bootloader bytecode standard for it15:37
TheJuliadtantsur: 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 something15:38
dtantsurmmm, fair. what's the best course of actions then: fail or succeed but slowly?15:39
TheJuliadtantsur: likely as an operator option15:39
TheJuliatrandles would be awesome to talk to about this15:40
dtantsurIt 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
JayFit is a really nice hammer15:40
TheJuliadtantsur: I mostly agree, but finish that hammer15:40
dtantsurhere comes the hardest part :)15:40
dtantsurI 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 accelerator15:41
dtantsurI have a strong feeling that I'll break out-of-tree deploy interfaces, at least ones based on agent15:41
dtantsur++15:41
TheJuliadtantsur: hmmmm15:41
* TheJulia ponders15:41
dtantsurI mean, I don't intend to, but given how invasive the patches are...15:42
dtantsurI wonder if we should just declare it a breaking change and not even try15:42
dtantsurto 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
TheJuliaYeah, I think your just going to need to delcare a breaking change15:45
TheJuliabut I'd be interested to look at the code as to how15:46
dtantsurthe 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 semantics15:47
JayFWhy do they need to be called the same?15:47
dtantsure.g. when it comes to handling states (DEPLOY WAIT or DEPLOYING) or polling for result15:47
JayFOh, in the conductor, not in the agent15:47
dtantsuryep15:47
TheJuliadtantsur: I think I just need to see it in code15:47
dtantsurgetting there15:47
JayFI need to stop trying to "help" with limited knowledge and go back to the actual things I'm supposed to do :D15:47
TheJuliaI know :)15:47
JayFyou all keep making awesome hammers o/15:48
dtantsurany input is useful at this point15:48
dtantsurespecially from you :)15:48
dtantsurspeaking of deploy steps, https://review.opendev.org/#/c/698770/ is reviewable, I think15:48
patchbotpatch 698770 - ironic-python-agent - Add an ability to run in-band deploy steps - 7 patch sets15:48
*** bdodd has joined #openstack-ironic15:51
*** etingof has quit IRC15:53
*** dougsz has quit IRC15:56
TheJuliaPerhaps we should make a reference to hammers in our next release notes15:57
TheJuliaprelude15:57
dtantsurprobably :)15:58
dtantsurdamn, the fact that IPA can only run one command at a time really breaks everything15:59
TheJuliawhy?15:59
dtantsurwe end up with some mix of get_deploy_steps and execute_deploy_step command16:00
dtantsur* commands16:00
*** dougsz has joined #openstack-ironic16:00
dtantsurmmm, I think I see how to handle it16:04
*** lucasagomes has quit IRC16:04
rpittaugood night! o/16:07
openstackgerritRiccardo Pittau proposed openstack/ironic master: [WIP] Use openstacksdk to interact with swift  https://review.opendev.org/71407416:07
*** rpittau is now known as rpittau|afk16:07
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: In-band deploy steps  https://review.opendev.org/69876816:08
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: Decompose core deploy step  https://review.opendev.org/69830916:08
*** dtantsur is now known as dtantsur|afk16:08
dtantsur|afkg'night16:08
*** bnemec has joined #openstack-ironic16:11
openstackgerritMerged openstack/ironic master: Bump hacking to 3.0.0  https://review.opendev.org/71560216:25
*** mkrai has quit IRC16:26
*** dustinc has joined #openstack-ironic17:00
*** k_mouza has quit IRC17:01
*** derekh has quit IRC17:02
*** k_mouza has joined #openstack-ironic17:03
arne_wiebalckbye everyone o/17:05
TheJuliao/17:06
*** dougsz has quit IRC17:06
*** alexmcleod has quit IRC17:17
*** k_mouza has quit IRC17:27
*** ociuhandu has quit IRC17:33
openstackgerritJulia Kreger proposed openstack/ironic master: Change [deploy]/default_boot_option to local  https://review.opendev.org/49211417:34
*** ociuhandu has joined #openstack-ironic17:34
*** ociuhandu has quit IRC17:39
openstackgerritMerged openstack/ironic master: Fix the remaining hacking issues  https://review.opendev.org/71590719:01
*** dougsz has joined #openstack-ironic19:04
*** tsturm2 has quit IRC19:06
TheJuliarpioso: 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
patchbotpatch 714491 - ironic - Fix issue where server fails to reboot - 2 patch sets19:11
*** dustinc has quit IRC19:23
*** iurygregory has quit IRC19:27
rpiosoTheJulia: 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
rpiosoTheJulia: Thank you so much for the review!19:27
TheJuliaYeah, My concern regarding having unit testing is really ensuring we track/maintain method behavior contracts or explicitly know when we make changes19:32
TheJuliaI'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 anyhow19:33
* TheJulia shrugs19:33
rpiosoTheJulia: Absolutely! :-)19:36
* rpioso doesn't shrug ;-)19:36
*** Lucas_Gray has joined #openstack-ironic19:41
*** Lucas_Gray has quit IRC19:48
openstackgerritAnkit Kumar proposed openstack/ironic master: Generalize ISO building for virtual media driver  https://review.opendev.org/67799619:49
*** Lucas_Gray has joined #openstack-ironic19:51
*** jtwill98 has quit IRC20:01
*** ociuhandu has joined #openstack-ironic20:12
*** ociuhandu has quit IRC20:18
*** Lucas_Gray has quit IRC20:31
*** dougsz has quit IRC20:34
*** Lucas_Gray has joined #openstack-ironic20:59
*** rcernin has joined #openstack-ironic21:02
*** Lucas_Gray has quit IRC21:03
*** Lucas_Gray has joined #openstack-ironic21:04
*** diga has quit IRC21:04
*** iurygregory has joined #openstack-ironic21:21
*** Wryhder has joined #openstack-ironic21:55
*** Lucas_Gray has quit IRC21:55
*** Wryhder is now known as Lucas_Gray21:56
*** rloo has quit IRC22:14
*** jdandrea has quit IRC22:25
*** Lucas_Gray has quit IRC22:45
*** tkajinam has joined #openstack-ironic22:49
*** ianychoi has quit IRC23:15
*** ianychoi has joined #openstack-ironic23:16
*** rh-jelabarre has quit IRC23:43
*** Goneri has quit IRC23:57

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!