Thursday, 2020-04-16

*** cdearborn has quit IRC00:01
TheJuliaI did! :)00:57
TheJuliadtantsur|afk: w/r/t your comment earlier about erase_devices. It is erase_devices_metadata that is running, not erase_devices from what I saw00:58
*** ricolin has joined #openstack-ironic01:32
openstackgerritMerged openstack/ironic-inspector stable/stein: [stable-only] Make ironicclient initialization future-proof  https://review.opendev.org/71872801:39
*** mkrai has joined #openstack-ironic01:59
*** Qianbiao has joined #openstack-ironic02:04
*** tzumainn has quit IRC02:07
*** yaawang has quit IRC02:10
*** yaawang has joined #openstack-ironic02:11
*** lifeless_ is now known as liffeless02:40
*** liffeless is now known as lifeless02:40
*** ociuhandu has joined #openstack-ironic03:14
*** ociuhandu has quit IRC03:19
Qianbiaohello ironic03:29
*** threestrands has joined #openstack-ironic03:29
Qianbiaogood morning.03:30
Qianbiaois there any official pycharm(or other IDE) code format style for openstack project?03:31
Qianbiaolike google style, export lints as a xml or something.03:31
*** gyee has quit IRC04:02
TheJuliaQianbiao: That is a really good quesstion...04:40
TheJuliaquestion04:40
TheJuliaQianbiao: All I have, off hand is, https://docs.openstack.org/hacking/latest/user/hacking.html04:41
QianbiaoTheJulia thanks, i have read the doc before. But still, it's hard for someone like me(not long time contributor) to remember all of them.04:53
QianbiaoWhen review code, there are those code style issues waste a lot of time for both contributor and reviewer.04:54
*** Qianbiao is now known as Qianbiao|afk05:30
*** jtomasek has joined #openstack-ironic05:45
*** Qianbiao|afk has quit IRC06:02
*** early has quit IRC06:04
*** early has joined #openstack-ironic06:09
*** Qianbiao|afk has joined #openstack-ironic06:10
*** ociuhandu has joined #openstack-ironic06:11
*** Qianbiao|afk is now known as Qianbiao06:17
*** ociuhandu has quit IRC06:38
*** ociuhandu has joined #openstack-ironic06:39
*** ociuhandu has quit IRC06:44
arne_wiebalckGood morning, ironic!06:50
Qianbiaohello, arne_wiebalck06:58
arne_wiebalckHey Qianbiao o/06:58
iurygregorygood morning arne_wiebalck Qianbiao and Ironic o/07:03
Qianbiao:) morning. where are u guys located. From some place?07:05
iurygregoryCzech Republic XD07:06
Qianbiaolol, about Czech, first thing occur in my mind is football.07:08
*** yolanda has quit IRC07:08
Qianbiaosecond is Czechoslovakia in Titanic.07:08
arne_wiebalckhey iurygregory, gm o/07:09
*** yolanda has joined #openstack-ironic07:09
iurygregoryI'm Czech so I have no idea XD07:09
arne_wiebalckQianbiao: France07:09
arne_wiebalckiurygregory: I thought you were Brazilian?07:10
iurygregoryarne_wiebalck, damm I forgot the not!07:10
iurygregoryI'm not Czech LOL07:10
arne_wiebalckiurygregory: LOL07:10
iurygregoryI'm sleeping07:11
* iurygregory needs more coffee07:11
Qianbiao._.07:11
iurygregoryD:07:11
arne_wiebalckQianbiao: where are you based?07:11
QianbiaoI am chinese.07:11
iurygregoryNice =)07:11
QianbiaoSafest place in the world for now..07:11
arne_wiebalckYeah, restrictions have just been prolonged by 4 weeks here.07:12
iurygregoryCzech Republic announced some measures so things can starting going back to normal07:12
iurygregorythe R0 here is in 0.807:13
QianbiaoShould do that earlier.07:13
arne_wiebalck"R0 here is in 0.8" ?07:14
QianbiaoR0 0.8 is pretty safe ?07:14
Qianbiaothings will get better.07:14
iurygregorywell it's better than 2.5 XD07:14
iurygregorythe lowest the better07:14
iurygregoryhttps://en.wikipedia.org/wiki/Basic_reproduction_number07:15
Qianbiao0.8 is pretty good. i know that.07:15
Qianbiaoif anyone need mask, goggles, i can help to buy some. just ping me.07:19
iurygregoryawesome =)07:19
iurygregoryarne_wiebalck, in France masks are mandatory?07:19
*** etingof has quit IRC07:26
*** ociuhandu has joined #openstack-ironic07:30
*** rpittau|afk is now known as rpittau07:34
rpittaugood morning ironic! o/07:34
rpittauiurygregory: no, not mandatory, for now07:34
rpittauQianbiao: we have a style guide for Openstack, usually it's reflected in the pep8 tests that we run on the projects07:39
rpittauQianbiao: I personally use pycharm and it helps a lot07:40
iurygregorymorning rpittau o/07:46
rpittauhey iurygregory :)07:47
arne_wiebalckiurygregory: sorry, had to join meetings07:51
iurygregoryarne_wiebalck, no worries =)07:52
arne_wiebalckiurygregory: no (I don't think so), recommended, though07:52
kaifengHi folks o/07:54
iurygregoryhey kaifeng o/07:54
kaifengiurygregory: wrt https://review.opendev.org/#/c/492114 I wonder what you mean by mentioning the iscsi deploy?07:55
patchbotpatch 492114 - ironic - Change [deploy]/default_boot_option to local - 34 patch sets07:55
Qianbiao<rpittau> i use pycharm too. what i expect is a customer pycharm code style xml.07:56
iurygregorykaifeng, 1min let me find the code07:56
* kaifeng doesn't see difference since it roots to the get_default_boot_option, am i missing something?07:56
Qianbiaoi can import it to pycharm, then it will auto follow the openstack lints and format code for me.07:56
arne_wiebalckhey kaifeng o/07:56
Qianbiaorpittau something like https://github.com/google/styleguide07:57
iurygregorykaifeng, I mean that we use the function there https://github.com/openstack/ironic/blob/8a2866f25f22a3a090e5ed7fc14fb8ab0a429312/ironic/drivers/modules/iscsi_deploy.py#L28607:57
kaifengQianbiao: basically, passing pep8 would be fine I guess, one style I remember clearly is avoid hard line break.07:57
rpittauQianbiao: I'm not aware of such thing for the Openstack style guide07:58
rpittauiurygregory: isn't that  already mentioned as deprecated ?07:59
Qianbiao<kaifeng> not only pep8, openstack has more special lint different from pep807:59
iurygregoryi thought since we are deprecating we would need to change there no?07:59
rpittauQianbiao: if you check the tox.ini file on each project you'll see that we use hacking that includes some specific tests for the Openstack style07:59
iurygregoryor we keep and only change after removal of the function?08:00
kaifengQianbiao: i didn't aware of a strong style, i guess it's just a feeling, and you will be used to it once you get familiar with the code08:00
kaifengiurygregory: so your suggesting we insert a TODO there?08:00
Qianbiaosure. kaifeng08:01
iurygregorykaifeng, it would be worth i assume =) so we don't miss since we will remove the function after Ussuri08:01
rpittauiurygregory: unittest and pep8 will faile :)08:02
iurygregoryrpittau, what? O.o because of a TODO?08:02
kaifengiurygregory: when we remove the method, tests will complain i would say (mimic the tone like iurygregory :)08:02
rpittauiurygregory: no, if you remove the function and run unittest or pep8, they will fail08:02
iurygregoryoh yeah ofc =)08:02
*** lucasagomes has joined #openstack-ironic08:05
QianbiaoOn page: https://docs.openstack.org/ironic/latest/admin/raid.html. what is "Backing physical disks" means?08:07
Qianbiaowhat is the different between example3 and example 408:07
arne_wiebalckI think one point here is to show how to create multiple RAID devices.08:20
arne_wiebalckE.g. you can have only  one where is_root_volume is true.08:21
*** ociuhandu has quit IRC08:25
Qianbiaoarne_wiebalck i see08:26
Qianbiaois "Backing physical disks" means how to identify physical disks(drive)08:27
Qianbiaoexample 3 and example 4 both show how to ""Backing physical disks"08:27
*** dtantsur|afk is now known as dtantsur08:28
*** Qianbiao is now known as Qianbiao|afk08:28
dtantsurmorning ironic08:29
iurygregorymorning dtantsur08:29
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Add variable to set discovery timeout in tempest tests  https://review.opendev.org/72039408:29
iurygregoryrpittau, dtantsur  I would like your thoughts in https://review.opendev.org/#/c/712041/08:31
patchbotpatch 712041 - ironic-tempest-plugin - Split Tempest tests between ipmi and redfish - 8 patch sets08:31
dtantsursure, just lemme wake up a bit (may take a week or two :)08:31
rpittauiurygregory: I was getting to that, like a sloth, but I was08:31
arne_wiebalckQianbiao|afk: yes08:32
iurygregorydtantsur, vodka would help you?08:32
iurygregoryrpittau, LOL08:32
dtantsuriurygregory: it usually has the opposite effect :)08:32
iurygregorydtantsur, you can get vodka after08:32
iurygregoryit's a prize08:32
dtantsurdo you suggest I may need to? :D08:32
iurygregorynow I know dtantsur has a stock...08:33
dtantsurI don't think I have a bottle stashed nowadays. I used to quite recently :)08:33
iurygregorynow I can only offer cute animals photos08:34
iurygregoryyou can choose the animal XD08:34
rpittausloth ?08:34
* rpittau can't wait to have centos8 official support for devstack08:35
iurygregoryrpittau, https://www.google.com/search?q=cute+sloth&source=lnms&tbm=isch&sa=X&ved=2ahUKEwj7hd2IxezoAhXvsaQKHYG5BO0Q_AUoAXoECBMQAw&biw=1600&bih=81308:35
iurygregorya whole list for you08:36
rpittauiurygregory: awesome08:36
*** ociuhandu has joined #openstack-ironic08:36
kaifengrpittau: do you know has centos8 supported packstack? last time i checked without luck08:36
rpittaukaifeng: the centos8 situation at the moment is a bit blurry, still some unfinished work08:37
kaifengrpittau: seems so, some packages are not available yet, even openvswitch is not there.08:38
rpittaukaifeng: I usually refer to this page for news https://hackmd.io/HrQd03c9SxOMtFPFrq50tg?view08:39
rpittauor the rdo channel08:39
*** alexmcleod has joined #openstack-ironic08:41
* kaifeng is typing..08:42
* kaifeng blames to he has two computers08:43
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Convert jobs to dib  https://review.opendev.org/71251608:44
*** threestrands has quit IRC08:45
*** tkajinam has quit IRC08:50
*** ociuhandu has quit IRC09:05
*** ociuhandu has joined #openstack-ironic09:08
*** mkrai has quit IRC09:09
openstackgerritDmitry Tantsur proposed openstack/networking-baremetal master: Remove the unused oslo.i18n bits  https://review.opendev.org/72040509:13
openstackgerritDmitry Tantsur proposed openstack/networking-generic-switch master: Remove the unused oslo.i18n bits  https://review.opendev.org/72040609:14
iurygregorydtantsur, is on fire =)09:17
dtantsur:D09:17
* iurygregory thinks dtantsur has a clone because he is pushing patches and sending emails...09:18
openstackgerritVerification of a change to openstack/ironic-inspector failed: Change default ram for tinyipa jobs  https://review.opendev.org/71861309:19
dtantsurI cannot confirm nor deny this statement09:20
iurygregoryhehehe09:20
*** mkrai has joined #openstack-ironic09:23
kaifengdtantsur gets a magic gerrit number ends with 404 :)09:27
iurygregoryyup and it's a serious patch09:30
*** k_mouza has joined #openstack-ironic09:36
Qianbiao|afkhello.09:49
*** Qianbiao|afk is now known as qianbiao09:49
qianbiaoI am confused on RAID settings.09:49
qianbiaoIt says RAID 1+0, RAID 5+0, RAID 6+0 is supported.09:50
iurygregorycorrect09:50
qianbiaobut if no span number is specified, how could we setup raid 10,50,6009:50
iurygregoryarne_wiebalck, do you know? ^09:51
iurygregoryI think you can only choose between the specific types we have support09:51
qianbiao> quote from document: "RAID level for the logical disk. Ironic supports the following RAID levels: 0, 1, 2, 5, 6, 1+0, 5+0, 6+0."09:52
iurygregory0, 1, 2, 5, 6, 1+0, 5+0, 6+009:52
iurygregoryyeah09:52
qianbiaobut AFAIK, span number is required to create volume for those +0 RAID level09:53
qianbiaoand span number does not appear in the document.09:53
iurygregorythis would be the volume_name ?09:53
qianbiaono. span number is used to indicate how many (0) volume should be created09:54
qianbiao+009:54
qianbiaoRAID 1+0 is RAID1 + RAID009:54
qianbiaoraid5+0 is raid5 + raid009:54
qianbiaothe span number is used for this09:54
qianbiaolike if we have 8 drives, there are several situation to create raid1+0.09:55
iurygregoryyeah I understand09:55
qianbiaolike 4-raid1+4-raid109:55
qianbiaoor 2-raid1+2raid1+2raid+2raid09:55
iurygregorynow I'm wondering how we set this XD09:56
qianbiaothis is span number for09:56
arne_wiebalckI don't know, this is for h/w RAID only09:56
qianbiaoI am reading idrac document. wonder how they calculate span numbers.09:59
*** ociuhandu has quit IRC10:02
*** ociuhandu has joined #openstack-ironic10:03
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-ui master: Imported Translations from Zanata  https://review.opendev.org/72042410:10
*** jiriprox has joined #openstack-ironic10:11
*** rpittau is now known as rpittau|bbl10:19
*** yolanda has quit IRC10:26
*** mkrai has quit IRC10:29
*** qianbiao is now known as Qianbiao|afk10:33
*** mkrai has joined #openstack-ironic10:36
*** yolanda has joined #openstack-ironic10:38
*** bjolo_ has quit IRC10:39
*** derekh has joined #openstack-ironic10:58
openstackgerritMerged openstack/ironic-python-agent master: Fix the token logic to be compatible with older ironic  https://review.opendev.org/72018511:11
*** jtomasek has quit IRC11:13
openstackgerritMerged openstack/ironic master: Fix missing print format in log messages  https://review.opendev.org/72011411:13
*** jtomasek has joined #openstack-ironic11:13
*** mkrai has quit IRC11:16
*** mkrai has joined #openstack-ironic11:16
*** Qianbiao|afk has quit IRC11:26
*** Qianbiao|afk has joined #openstack-ironic11:35
*** cdearborn has joined #openstack-ironic11:37
*** etingof has joined #openstack-ironic11:50
*** mkrai has quit IRC11:54
*** rh-jelabarre has joined #openstack-ironic11:56
*** Goneri has joined #openstack-ironic11:59
*** ociuhandu has quit IRC12:04
*** ociuhandu has joined #openstack-ironic12:05
*** Qianbiao|afk has quit IRC12:06
*** Qianbiao|afk has joined #openstack-ironic12:06
*** rpittau|bbl is now known as rpittau12:07
*** ociuhandu has quit IRC12:09
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Convert jobs to dib  https://review.opendev.org/71251612:10
*** ociuhandu has joined #openstack-ironic12:28
*** mkrai has joined #openstack-ironic12:33
*** ociuhandu has quit IRC12:46
*** ociuhandu has joined #openstack-ironic12:46
*** rloo has joined #openstack-ironic12:49
openstackgerritHarald JensÃ¥s proposed openstack/ironic-inspector master: Port physical network CIDR map hook  https://review.opendev.org/71859412:50
*** ociuhandu has quit IRC12:51
*** mkrai has quit IRC12:55
*** yolanda has quit IRC12:55
*** yolanda has joined #openstack-ironic12:55
*** mkrai has joined #openstack-ironic13:04
* TheJulia yawns13:05
openstackgerritRiccardo Pittau proposed openstack/python-ironic-inspector-client master: No need to import print function anymore  https://review.opendev.org/72046813:07
rpittaugood morning TheJulia :)13:07
*** tzumainn has joined #openstack-ironic13:11
*** dougsz has joined #openstack-ironic13:13
dougszarne_wiebalck I'm just looking at some of your good work on software RAID13:15
dougszAre there any gotchas I should be aware of for UEFI?13:15
*** ociuhandu has joined #openstack-ironic13:16
TheJuliadougsz: add them to the documentation if you find them! :)13:17
dougszWill do :) I've hit a failure installing the bootloader to the RAID1 partition, just trying to understand it a bit better. Will keep digging :)13:18
arne_wiebalckhey dougsz13:18
arne_wiebalckdougsz:  the main thing is to have a UEFI capable image I think13:18
arne_wiebalckdougsz: and have the root fs UUID in the glance metadata (true for BIOS as well, but not for stand-alone)13:19
arne_wiebalckdougsz: oh13:19
arne_wiebalckdougsz: there is another thing we are just struggling with13:20
arne_wiebalckdougsz: which is that grub and efi installed at the same time do not play well together (for us on Centos7 at least)13:20
arne_wiebalckdougsz: we are trying to have a one-size-fits-all image13:21
arne_wiebalckdougsz: but this is for BIOS (and s/w RAID) ... nevermind if you're on UEFI only13:21
openstackgerritMerged openstack/ironic-ui master: Imported Translations from Zanata  https://review.opendev.org/72042413:21
arne_wiebalckdougsz: if you have some logs to share I am happy to have a look to see if I recognize the issue13:22
arne_wiebalckdougsz: minfd you, I have only tried this on a backport to stein (not on master itself) with real hardware13:22
arne_wiebalckmind13:22
dougszhttps://pastebin.com/nN0danYd13:23
arne_wiebalckhm, no ... this is one of the bind mounts right before grub13:24
arne_wiebalcknot grub itself13:24
dougszYeah, just installing. Will get a break point in there are try and figure out what is going wrong.13:25
arne_wiebalckthis is master?13:25
dougszThanks for the hints though, I am tempted to switch to legacy BIOS mode :)13:25
dougszThis is train13:26
arne_wiebalckwith backported UEFI patches?13:26
arne_wiebalckUEFI support for s/w RAID only comes in Ussuri13:26
dougszAh :) That could well be it!13:27
arne_wiebalck:)13:27
TheJuliadtantsur: did also find an issue in the raid code path, I remember reviewing the change yesterday13:27
TheJuliaor was it agent token13:28
TheJuliaor was it both *brain explodes*13:28
arne_wiebalcklol13:28
dougszThanks TheJulia, I have found the RAID setup part works well for me.13:28
TheJuliaout of curiosity, who knows what the term "cult of personality" ?13:32
* TheJulia is answering questions for someone writing an article and they didn't get the reference so I'm wondering how widely the expression is used.13:32
arne_wiebalckthis is a common term in German, if that is of any help :)13:33
arne_wiebalck"Personenkult" (for dtantsur ;)13:34
dtantsur:D13:34
dtantsurTheJulia: I think there is a direct analogy in Russian. if I understand the English one correctly, that is :)13:36
dtantsuralso good morning13:36
dtantsuralso the patches yesterday were 1. for token, 2. for UEFI13:36
*** mkrai has quit IRC13:36
dougszIt was new to me, but not any more!13:37
TheJuliaarne_wiebalck: dtantsur: thanks13:43
TheJuliadougsz: hmm if you don't mind, where about in the world are you?13:44
* TheJulia is getting full corgi talk this morning for some... unknown reason13:44
* TheJulia needs to wake up more13:45
dougszUK - I'm sure i'd heard it, but couldn't really define what it meant until I looked it up, could be just me13:45
TheJuliahmm13:45
TheJuliaInteresting13:45
* TheJulia goes puts on video call presentable clothing.. i.e. not pajamas13:45
*** k_mouza has quit IRC13:47
dtantsurTheJulia: don't forget to take corgi!13:47
* dtantsur is sure corgis make any video call better13:47
iurygregorycorgis in the video call ++13:48
TheJuliathis morning we also have a cat that has attached herself to my chair, this is going to be an interesting day.13:50
TheJuliarandom question, do we need a SPUC next week?13:52
arne_wiebalckI'd be in!13:57
*** yaawang_ has joined #openstack-ironic13:59
iurygregorywe do13:59
iurygregoryTheJulia, I assume you are now using a random chair =)14:00
*** yaawang has quit IRC14:01
*** k_mouza has joined #openstack-ironic14:01
iurygregoryhttps://review.opendev.org/#/c/492114/ is in the gate \o/14:03
patchbotpatch 492114 - ironic - Change [deploy]/default_boot_option to local - 34 patch sets14:03
*** dougsz has quit IRC14:08
*** ociuhandu has quit IRC14:17
*** dougsz has joined #openstack-ironic14:24
dtantsur\o/14:28
TheJuliaiurygregory: no, I managed to claim my chair... *evil grin*14:29
iurygregoryTheJulia, \o/ awesome14:35
*** gyee has joined #openstack-ironic14:42
*** mkrai has joined #openstack-ironic14:43
*** akahat is now known as akahat|ruck14:50
openstackgerritDmitry Tantsur proposed openstack/ironic master: Improve the first boot check in agent's process_next_step  https://review.opendev.org/71835814:53
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: Support executing in-band deploy steps  https://review.opendev.org/69876814:53
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: Decompose core deploy step  https://review.opendev.org/69830914:53
etingofrpittau, o/ I accidentally encountered a centos8 ramdisk not mounting root (unknown device). Is is something already known?15:07
openstackgerritDmitry Tantsur proposed openstack/ironic master: Improve the command status checks in the agent's process_next_step  https://review.opendev.org/71835815:07
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: Support executing in-band deploy steps  https://review.opendev.org/69876815:07
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: Decompose core deploy step  https://review.opendev.org/69830915:07
rpittauetingof: mmmm I didn't see anything like that so far15:08
dtantsuretingof: where did you encounter that? The first thing to check is that you have enough RAM.15:08
etingofin devstack15:09
dtantsurCI, locally?15:09
etingoflocally15:09
dtantsurhow much RAM do your nodes have?15:09
etingof1310720KiB15:10
rpittauetingof: try with 2GB15:10
etingofokay15:10
dtantsur++15:10
rpittauetingof: the default for centos8 ramdisk images should be 2GB, if it's less in some jobs we need to change that15:12
etingofalright, I will give it a try15:13
openstackgerritMerged openstack/ironic-inspector master: Change default ram for tinyipa jobs  https://review.opendev.org/71861315:22
openstackgerritMerged openstack/ironic master: Change [deploy]/default_boot_option to local  https://review.opendev.org/49211415:22
dtantsurTheJulia: ^^^ \o/15:23
openstackgerritMerged openstack/python-ironic-inspector-client master: No need to import print function anymore  https://review.opendev.org/72046815:28
*** Qianbiao|afk has quit IRC15:33
TheJuliadtantsur: omg \o/15:45
* TheJulia dances15:45
*** ociuhandu has joined #openstack-ironic15:45
TheJuliajust... omg that makes me SOOO happy15:45
*** ociuhandu has quit IRC15:50
dtantsur++15:54
iurygregory\o/15:56
*** lucasagomes has quit IRC15:56
iurygregorybye everyone see you tomorrow o/15:58
*** ociuhandu has joined #openstack-ironic16:02
* etingof got a feeling that boot interface validation is never called on in-band cleaning16:12
etingofsupposedly, that should happen as part of agent/deploy interface validation16:13
dtantsurit should be, really16:14
etingofhmmm16:14
* etingof focuses both eyes at the single xterm16:15
rpittaubye, good night! o/16:18
*** rpittau is now known as rpittau|afk16:18
*** alexmcleod has quit IRC16:18
dtantsurTheJulia: it seems that we should start collecting timezone information for the virtual PTG16:20
arne_wiebalckbye everyone o/16:21
* etingof is getting convinced that boot interface validation is not happening on cleaning, however power and network validations are performed16:22
openstackgerritMerged openstack/ironic-python-agent master: A boot partition on a GPT disk should be considered an EFI partition  https://review.opendev.org/72018416:22
dtantsurTheJulia: I have a feeling that everyone will be competing for more or less the same slot (US-Europe)16:23
etingofthis is where it's tool late to validate boot interface -- https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/deploy_utils.py#L59816:24
etingofthis is where some other interfaces (but boot) are validated -- https://github.com/openstack/ironic/blob/master/ironic/conductor/cleaning.py#L6516:26
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent stable/train: A boot partition on a GPT disk should be considered an EFI partition  https://review.opendev.org/72054916:31
*** dsneddon has joined #openstack-ironic16:31
*** yaawang has joined #openstack-ironic16:32
*** yaawang_ has quit IRC16:33
dtantsurg'night folks16:36
*** dtantsur is now known as dtantsur|afk16:36
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Add variable to set discovery timeout in tempest tests  https://review.opendev.org/72039416:37
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Convert jobs to dib  https://review.opendev.org/71251616:37
openstackgerritAndreas Jaeger proposed openstack/ironic-inspector master: Cleanup py27 support  https://review.opendev.org/72055116:46
*** ricolin has quit IRC16:48
openstackgerritAndreas Jaeger proposed openstack/ironic-inspector master: Cleanup py27 support  https://review.opendev.org/72055116:49
dougszarne_wiebalck did you have to do anything special with your deploy image for the grub install?16:52
dougszNow I have a legacy BIOS deploy image I hit: grub2-install: error: disk `md0,1' not found.16:52
openstackgerritAndreas Jaeger proposed openstack/ironic-python-agent-builder master: Cleanup py27 support  https://review.opendev.org/72055416:52
dougszBut RAID config and rootfs deploy look good at least :)16:53
*** mkrai has quit IRC17:12
*** ociuhandu has quit IRC17:13
*** k_mouza has quit IRC17:31
*** rcernin has quit IRC17:37
*** k_mouza has joined #openstack-ironic17:39
*** k_mouza has quit IRC17:41
*** ociuhandu has joined #openstack-ironic18:09
*** yolanda has quit IRC18:14
*** k_mouza has joined #openstack-ironic18:25
*** k_mouza has quit IRC18:27
*** yolanda has joined #openstack-ironic18:28
*** ociuhandu has quit IRC18:31
*** Lucas_Gray has joined #openstack-ironic18:34
TheJuliadtantsur|afk: Yeah, I was kind of suspecting that and I was planning on it on Monday. I'm honestly a little confused by the format and actual requirement there, but I'm sure more details will be revealed18:53
TheJuliadougsz: interesting, partition or whole disk image?19:01
dougszwhole disk image, still digging into exactly what the problem is19:14
*** ociuhandu has joined #openstack-ironic19:16
*** ociuhandu has quit IRC19:22
dougsz"grub2-install: error: disk `md0,1' not found" is user error, no mdadm installed into deploy image.19:46
openstackgerritAndreas Jaeger proposed openstack/ironic-inspector master: Cleanup py27 support  https://review.opendev.org/72055120:06
*** derekh has quit IRC20:26
*** derekh has joined #openstack-ironic20:35
TheJuliaoh, interesting, that makes tons of sense20:36
TheJuliadougsz: feel free to post a doc update! :)20:37
dougszTheJulia: Will do, although to be fair the docs are very good and I missed that detail :) If I find anything else of value on my journey however..20:44
*** ociuhandu has joined #openstack-ironic20:45
*** ociuhandu has quit IRC20:49
*** dougsz has quit IRC21:23
*** ociuhandu has joined #openstack-ironic21:38
*** ociuhandu has quit IRC21:44
*** rloo has quit IRC22:14
*** Lucas_Gray has quit IRC22:21
*** k_mouza has joined #openstack-ironic22:27
*** k_mouza has quit IRC22:32
*** rcernin has joined #openstack-ironic22:39
*** tkajinam has joined #openstack-ironic22:39
*** bdodd has quit IRC22:46
*** bdodd has joined #openstack-ironic22:50
TheJuliaA relatively easy/quick review: https://review.opendev.org/#/c/718342/22:53
patchbotpatch 718342 - ironic - Release reservation when stoping the ironic-conduc... - 3 patch sets22:53
*** threestrands has joined #openstack-ironic23:14
*** threestrands has quit IRC23:15
*** threestrands has joined #openstack-ironic23:15
openstackgerritJulia Kreger proposed openstack/ironic master: Block port deletions where vif is present  https://review.opendev.org/66583523:19
TheJuliaironic-cores, I'm sure dell would appreciate eyes on https://review.opendev.org/#/c/713450/2323:25
patchbotpatch 713450 - ironic - Implement the bios-interface for idrac-wsman driver - 23 patch sets23:25
*** michchap has quit IRC23:29
*** jiriprox has quit IRC23:29

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