Thursday, 2019-12-05

*** ociuhandu has joined #openstack-ironic00:39
*** ociuhandu has quit IRC00:48
*** goldyfruit_ has joined #openstack-ironic01:20
*** ociuhandu has joined #openstack-ironic01:39
*** ociuhandu has quit IRC01:43
*** chenhaw has joined #openstack-ironic01:51
openstackgerritMerged openstack/ironic-inspector stable/train: Get details when listing ports upon inspection of active nodes  https://review.opendev.org/69728602:09
*** ricolin has joined #openstack-ironic02:11
openstackgerritMerged openstack/ironic-inspector stable/train: Do not update/delete ports upon inspection of active nodes  https://review.opendev.org/69728702:18
openstackgerritMerged openstack/ironic-inspector master: Update docs on aarch64 PXE  https://review.opendev.org/69722102:18
openstackgerritKaifeng Wang proposed openstack/ironic-python-agent-builder master: Add words on building image for other architecture  https://review.opendev.org/69722202:25
*** ociuhandu has joined #openstack-ironic02:38
*** ociuhandu has quit IRC02:47
*** igordc has quit IRC02:54
*** gyee has quit IRC03:03
*** rloo has quit IRC03:10
*** michchap has quit IRC04:03
*** tzumainn has quit IRC04:10
*** rh-jelabarre has quit IRC04:18
cp-https://pastebin.com/mL14MUxa05:05
cp-Should I be cleaning something ?05:06
*** ociuhandu has joined #openstack-ironic05:30
*** ociuhandu has quit IRC05:35
*** pcaruana has joined #openstack-ironic05:38
*** pcaruana has quit IRC05:42
*** rcernin has quit IRC06:09
*** ociuhandu has joined #openstack-ironic06:16
*** jtomasek has joined #openstack-ironic06:18
*** ociuhandu has quit IRC06:36
*** ociuhandu has joined #openstack-ironic07:03
*** pcaruana has joined #openstack-ironic07:17
*** ociuhandu has quit IRC07:27
*** ociuhandu has joined #openstack-ironic07:27
*** rcernin has joined #openstack-ironic08:01
*** iurygregory has joined #openstack-ironic08:01
iurygregorygood morning Ironic08:04
arne_wiebalckGood morning, iurygregory and ironic!08:10
iurygregorymorning arne_wiebalck o/08:10
*** tesseract has joined #openstack-ironic08:21
*** rpittau|afk is now known as rpittau08:32
rpittaugood morning ironic! o/08:32
*** iurygregory has quit IRC08:45
*** iurygregory has joined #openstack-ironic08:47
iurygregorymorning rpittau o/08:47
rpittauhey iurygregory :)08:47
*** khansa has joined #openstack-ironic08:54
hjensasgood morning!08:58
rpittauhey hjensas :)08:58
hjensasrpittau: the removing six, this is breaking package build's in rdo. https://review.rdoproject.org/r/#/c/23945/08:58
hjensasrpittau: I'm not sure we should fix it, or just tell RDO they need py3 now ...08:59
rpittauhjensas: oh cool....08:59
hjensasrpittau: I guess similar errors are going to pop up in other packages as well.09:00
rpittauhjensas: the cut with py2 is kind of already happened, we blocked only sushy-tools at the moment09:01
rpittauwe can always revert the change, but I think similar errors will start happening more frequently as soon as people stop supporting python 209:01
hjensasrpittau: yes, let's wait for dtantsur|afk and get his opinion as well?09:02
rpittauhjensas: absolutely09:02
*** lucasagomes has joined #openstack-ironic09:05
rpittauhjensas: fun fact, the python interpreter in the job is version 3.6.8 :)09:05
hjensasrpittau: it's for sure includes other packages as well - https://review.rdoproject.org/r/23928 is for ironic-python-agent09:06
*** ociuhandu has quit IRC09:09
rpittauhjensas: we have 5 changes merged so far https://review.opendev.org/#/q/topic:goodbye-six+(status:open+OR+status:merged)09:13
iurygregorymorning hjensas09:19
hjensasrpittau: ok, and more are coming in. From Ironic's perspective the changes are correct, the project need to move on. Let's wait for more people and discuss.09:19
hjensasmorning iurygregory09:19
*** iurygregory is now known as iurygregory_cour09:20
*** iurygregory_cour is now known as iury_course09:20
*** derekh has joined #openstack-ironic09:43
*** khansa has quit IRC09:54
*** jtomasek has quit IRC09:54
*** ricolin has quit IRC09:54
*** goldyfruit_ has quit IRC09:54
*** hjensas has quit IRC09:54
*** hamzy has quit IRC09:54
*** dmellado has quit IRC09:54
*** mbeierl has quit IRC09:54
*** yaawang has quit IRC09:54
*** sziviani has quit IRC09:54
*** andrein has quit IRC09:54
*** donnyd has quit IRC09:54
*** bfournier has quit IRC09:54
*** ddurst has quit IRC09:54
*** TheJulia has quit IRC09:54
*** mnasiadka has quit IRC09:54
*** rpittau has quit IRC09:54
*** gregwork has quit IRC09:54
*** toanster has quit IRC09:54
*** seongsoocho has quit IRC09:54
*** iury_course has quit IRC09:54
*** bobmel has quit IRC09:54
*** trident has quit IRC09:54
*** rnoriega_ has quit IRC09:54
*** _gryf_ has quit IRC09:54
*** openstackgerrit has quit IRC09:54
*** antonym has quit IRC09:54
*** brtknr has quit IRC09:54
*** mgoddard has quit IRC09:54
*** naved001 has quit IRC09:54
*** logan- has quit IRC09:54
*** adrianc has quit IRC09:54
*** hwoarang has quit IRC09:54
*** dtantsur|afk has quit IRC09:54
*** mmethot has quit IRC09:54
*** samc-bbc has quit IRC09:54
*** mcarden has quit IRC09:54
*** kaifeng has quit IRC09:54
*** nicholas has quit IRC09:54
*** rpioso has quit IRC09:54
*** amotoki has quit IRC09:54
*** Anticimex has quit IRC09:54
*** KeenWarrior has quit IRC09:54
*** ildikov has quit IRC09:54
*** ajya[m] has quit IRC09:54
*** spotz has quit IRC09:54
*** JayF has quit IRC09:54
*** mkowalski has quit IRC09:54
*** lucasagomes has quit IRC09:54
*** tesseract has quit IRC09:54
*** pcaruana has quit IRC09:54
*** lennyb has quit IRC09:54
*** yolanda has quit IRC09:54
*** jlvillal has quit IRC09:54
*** hoonetorg has quit IRC09:55
*** baha has quit IRC09:55
*** irclogbot_2 has quit IRC09:55
*** bnemec has quit IRC09:55
*** benj_ has quit IRC09:55
*** cp- has quit IRC09:55
*** ianw has quit IRC09:55
*** mroh has quit IRC09:55
*** zer0c00l has quit IRC09:55
*** larsks has quit IRC09:55
*** mjturek has quit IRC09:55
*** jmccrory has quit IRC09:55
*** strigazi has quit IRC09:55
*** rcernin has quit IRC09:55
*** Goneri has quit IRC09:55
*** dhellmann has quit IRC09:55
*** persia has quit IRC09:55
*** zaneb has quit IRC09:55
*** gudrutiss has quit IRC09:55
*** jroll has quit IRC09:55
*** mrda has quit IRC09:55
*** sshnaidm has quit IRC09:55
*** johnhaan has quit IRC09:55
*** jrist has quit IRC09:55
*** fungi has quit IRC09:55
*** rajinir has quit IRC09:55
*** gmann has quit IRC09:55
*** dustinc has quit IRC09:55
*** derekh has quit IRC09:55
*** chenhaw has quit IRC09:55
*** yoctozepto has quit IRC09:55
*** jerrywang has quit IRC09:55
*** Blinkiz has quit IRC09:55
*** jistr has quit IRC09:55
*** gryf has quit IRC09:55
*** stevebaker has quit IRC09:55
*** xXraphXx has quit IRC09:55
*** patchbot has quit IRC09:55
*** tonyb has quit IRC09:55
*** chihhsin has quit IRC09:55
*** bdodd has quit IRC09:55
*** cxc99 has quit IRC09:55
*** dansmith has quit IRC09:55
*** mbuil has quit IRC09:55
*** jhesketh has quit IRC09:55
*** dtruong has quit IRC09:55
*** andreaf has quit IRC09:55
*** johnsom has quit IRC09:55
*** portdirect has quit IRC09:55
*** jungleboyj has quit IRC09:55
*** sw3 has quit IRC09:55
*** csmart has quit IRC09:55
*** andy_ has quit IRC09:55
*** vesper11 has quit IRC09:55
*** clarkb has quit IRC09:55
*** verdurin has quit IRC09:55
*** mgagne has quit IRC09:55
*** lifeless has quit IRC09:55
*** early has quit IRC09:55
*** zzzeek has quit IRC09:55
*** guilhermesp has quit IRC09:55
*** tonyb[m] has quit IRC09:55
*** ab-a has quit IRC09:55
*** arne_wiebalck has quit IRC09:55
*** vdrok has quit IRC09:55
*** mnaser has quit IRC09:55
*** NobodyCam has quit IRC09:55
*** cjloader has quit IRC09:55
*** melwitt has quit IRC09:55
*** alexmcleod has joined #openstack-ironic10:04
*** mbeierl has joined #openstack-ironic10:04
*** dmellado has joined #openstack-ironic10:04
*** hamzy has joined #openstack-ironic10:04
*** hjensas has joined #openstack-ironic10:04
*** goldyfruit_ has joined #openstack-ironic10:04
*** ricolin has joined #openstack-ironic10:04
*** jtomasek has joined #openstack-ironic10:04
*** seongsoocho has joined #openstack-ironic10:04
*** toanster has joined #openstack-ironic10:04
*** gregwork has joined #openstack-ironic10:04
*** rpittau has joined #openstack-ironic10:04
*** mnasiadka has joined #openstack-ironic10:04
*** TheJulia has joined #openstack-ironic10:04
*** ddurst has joined #openstack-ironic10:04
*** bfournier has joined #openstack-ironic10:04
*** donnyd has joined #openstack-ironic10:04
*** andrein has joined #openstack-ironic10:04
*** sziviani has joined #openstack-ironic10:04
*** yaawang has joined #openstack-ironic10:04
*** brtknr has joined #openstack-ironic10:04
*** antonym has joined #openstack-ironic10:04
*** openstackgerrit has joined #openstack-ironic10:04
*** _gryf_ has joined #openstack-ironic10:04
*** rnoriega_ has joined #openstack-ironic10:04
*** trident has joined #openstack-ironic10:04
*** bobmel has joined #openstack-ironic10:04
*** iury_course has joined #openstack-ironic10:04
*** derekh has joined #openstack-ironic10:04
*** lucasagomes has joined #openstack-ironic10:04
*** tesseract has joined #openstack-ironic10:04
*** rcernin has joined #openstack-ironic10:04
*** pcaruana has joined #openstack-ironic10:04
*** chenhaw has joined #openstack-ironic10:04
*** lennyb has joined #openstack-ironic10:04
*** Goneri has joined #openstack-ironic10:04
*** dhellmann has joined #openstack-ironic10:04
*** JayF has joined #openstack-ironic10:04
*** spotz has joined #openstack-ironic10:04
*** yolanda has joined #openstack-ironic10:04
*** yoctozepto has joined #openstack-ironic10:04
*** vesper11 has joined #openstack-ironic10:04
*** jhesketh has joined #openstack-ironic10:04
*** jlvillal has joined #openstack-ironic10:04
*** jerrywang has joined #openstack-ironic10:04
*** gudrutiss has joined #openstack-ironic10:04
*** Blinkiz has joined #openstack-ironic10:04
*** jistr has joined #openstack-ironic10:04
*** gryf has joined #openstack-ironic10:04
*** clarkb has joined #openstack-ironic10:04
*** persia has joined #openstack-ironic10:04
*** zaneb has joined #openstack-ironic10:04
*** jroll has joined #openstack-ironic10:04
*** mkowalski has joined #openstack-ironic10:04
*** stevebaker has joined #openstack-ironic10:04
*** hoonetorg has joined #openstack-ironic10:04
*** mrda has joined #openstack-ironic10:04
*** csmart has joined #openstack-ironic10:04
*** xXraphXx has joined #openstack-ironic10:04
*** verdurin has joined #openstack-ironic10:04
*** ab-a has joined #openstack-ironic10:04
*** patchbot has joined #openstack-ironic10:04
*** sshnaidm has joined #openstack-ironic10:04
*** tonyb has joined #openstack-ironic10:04
*** baha has joined #openstack-ironic10:04
*** irclogbot_2 has joined #openstack-ironic10:04
*** bnemec has joined #openstack-ironic10:04
*** benj_ has joined #openstack-ironic10:04
*** chihhsin has joined #openstack-ironic10:04
*** cp- has joined #openstack-ironic10:04
*** mgagne has joined #openstack-ironic10:04
*** ianw has joined #openstack-ironic10:04
*** bdodd has joined #openstack-ironic10:04
*** tonyb[m] has joined #openstack-ironic10:04
*** lifeless has joined #openstack-ironic10:04
*** early has joined #openstack-ironic10:04
*** zzzeek has joined #openstack-ironic10:04
*** guilhermesp has joined #openstack-ironic10:04
*** johnhaan has joined #openstack-ironic10:04
*** jungleboyj has joined #openstack-ironic10:04
*** cxc99 has joined #openstack-ironic10:04
*** jrist has joined #openstack-ironic10:04
*** mroh has joined #openstack-ironic10:04
*** fungi has joined #openstack-ironic10:04
*** rajinir has joined #openstack-ironic10:04
*** jmccrory has joined #openstack-ironic10:04
*** dtruong has joined #openstack-ironic10:04
*** zer0c00l has joined #openstack-ironic10:04
*** andreaf has joined #openstack-ironic10:04
*** dansmith has joined #openstack-ironic10:04
*** larsks has joined #openstack-ironic10:04
*** mjturek has joined #openstack-ironic10:04
*** strigazi has joined #openstack-ironic10:04
*** andy_ has joined #openstack-ironic10:04
*** mbuil has joined #openstack-ironic10:04
*** gmann has joined #openstack-ironic10:04
*** dustinc has joined #openstack-ironic10:04
*** johnsom has joined #openstack-ironic10:04
*** portdirect has joined #openstack-ironic10:04
*** sw3 has joined #openstack-ironic10:04
*** melwitt has joined #openstack-ironic10:04
*** mnaser has joined #openstack-ironic10:04
*** NobodyCam has joined #openstack-ironic10:04
*** vdrok has joined #openstack-ironic10:04
*** cjloader has joined #openstack-ironic10:04
*** arne_wiebalck has joined #openstack-ironic10:04
*** khansa has joined #openstack-ironic10:04
*** ajya[m] has joined #openstack-ironic10:04
*** mgoddard has joined #openstack-ironic10:04
*** naved001 has joined #openstack-ironic10:04
*** logan- has joined #openstack-ironic10:04
*** adrianc has joined #openstack-ironic10:04
*** hwoarang has joined #openstack-ironic10:04
*** dtantsur|afk has joined #openstack-ironic10:04
*** mmethot has joined #openstack-ironic10:04
*** samc-bbc has joined #openstack-ironic10:04
*** mcarden has joined #openstack-ironic10:04
*** kaifeng has joined #openstack-ironic10:04
*** nicholas has joined #openstack-ironic10:04
*** rpioso has joined #openstack-ironic10:04
*** KeenWarrior has joined #openstack-ironic10:04
*** amotoki has joined #openstack-ironic10:04
*** Anticimex has joined #openstack-ironic10:04
*** ildikov has joined #openstack-ironic10:04
*** logan- has quit IRC10:05
*** dtantsur|afk is now known as dtantsur10:06
dtantsurmorning ironic10:06
dtantsurhjensas, rpittau, tell RDO they need python 3..10:06
*** logan- has joined #openstack-ironic10:06
dtantsuror, well, they know it10:07
dtantsurit just takes time due to delays on centos side10:07
hjensasdtantsur: :) it's a pita that centos delay.10:08
dtantsurwell, yes10:08
rpittaudtantsur: lol10:22
iury_coursemorning dtantsur10:30
*** xXraphXx has quit IRC10:31
*** chenhaw has quit IRC10:33
*** ociuhandu has joined #openstack-ironic10:39
*** pcaruana has quit IRC10:42
* etingof is wondering if switching ironic-standalone job on vmedia boot would harden the CI...?10:48
dtantsurmaybe?11:14
dtantsurcurrently it uses ipmi11:14
*** ociuhandu has quit IRC11:31
*** derekh has quit IRC11:33
*** ociuhandu has joined #openstack-ironic11:44
*** khansa has quit IRC11:48
*** khansa has joined #openstack-ironic11:49
openstackgerritDmitry Tantsur proposed openstack/ironic master: Correct power state handling for managed in-band inspection  https://review.opendev.org/69745111:49
dtantsurTheJulia: I hope this ^^ addresses your concerns11:50
*** iury_course has quit IRC11:55
*** iurygregory has joined #openstack-ironic11:56
*** iurygregory is now known as iury_course11:56
*** Lucas_Gray has joined #openstack-ironic12:07
iury_courserpittau, you around?12:09
rpittauiury_course: I am12:09
*** Lucas_Gray has quit IRC12:20
*** Lucas_Gray has joined #openstack-ironic12:22
*** khansa has quit IRC12:23
*** bfournier has quit IRC12:28
*** khansa has joined #openstack-ironic12:41
*** rcernin has quit IRC12:52
*** pcaruana has joined #openstack-ironic12:54
*** ociuhandu has quit IRC13:06
*** rh-jelabarre has joined #openstack-ironic13:07
TheJuliagood morning everyone13:12
rpittauGood morning TheJulia :)13:13
TheJuliaiury_course: fyi, the file systems are case insensitive afaik. I noted that in my original huge long reply about finding the various bootloader options.13:14
iury_courseTheJulia, ack13:14
* TheJulia tries to wake up13:14
*** goldyfruit_ has quit IRC13:20
openstackgerritDmitry Tantsur proposed openstack/ironic-lib master: mdns: default to listing on both V4 and V6 interfaces  https://review.opendev.org/69746713:22
dtantsurone missing bit in the mdns-v6 story ^^^13:22
dtantsurmorning TheJulia13:22
*** rloo has joined #openstack-ironic13:23
khansagood morning TheJulia13:24
*** Goneri has quit IRC13:24
*** belmoreira has joined #openstack-ironic13:25
*** ociuhandu has joined #openstack-ironic13:27
TheJuliadtantsur: doh!13:31
openstackgerritDmitry Tantsur proposed openstack/ironic-lib master: mdns: correctly handle loopback addresses  https://review.opendev.org/69747013:36
dtantsurand a corner case ^^13:36
*** bfournie has joined #openstack-ironic13:37
TheJuliaetingof: dtantsur: switching that job does make a lot of sense since, although part of the issue is also the default netboot action and ports reattaching in time....  Remastering new iso for ramdisks could at least be interesting although maybe a little more disk heavy. Likely need to absolutely make sure full disk wipe cleaning is disabled13:38
openstackgerritHamdy Khader proposed openstack/ironic master: Keep port's binding profile when unbinding in case of SmartNIC  https://review.opendev.org/69747113:38
* TheJulia blinks13:38
etingofTheJulia, those boot ISOs are not large... much less than a VM drive I think13:39
etingofgood morning!13:40
TheJuliathe downside of a global change is we really just also need to get most of those jobs off of the network boot default13:40
TheJuliasince local updates of kernel/ramdisk can't update tftp/iso images managed by the conductor13:41
etingofis not conductor rebuilding isos out of the most current kernel/ramdisk?13:42
* etingof is confused13:42
TheJuliaetingof: I'm thinking long term13:43
TheJuliabecause bare metal instances may be out there for a long time in a more BMaaS fleet13:44
etingofso to update BM instance we need to reboot the BM13:45
etingofunless it's local boot, new boot will always be with the latest kernel/ramdisk13:45
TheJuliafar worse.13:45
TheJuliaWe need to recognize a change has occured with the remote files either in glance or otherwise, and then update the files managed by the conductor.13:46
TheJuliaIt is something we as a community have basically said "we hear you, but that is not very cloud like and then that may introduce additional breaking risk"13:46
etingofdo you mean booting from remote volume where kernel/ramdisk are embedded once and forever?13:47
*** Lucas_Gray has quit IRC13:47
TheJuliathat is basically the way netboot works today, and it is bad for long running instances.13:48
etingofthose remote files you mentioned - can you give me an example?13:49
TheJulia /httpboot/uuid/kernel13:49
TheJuliaplease don't focus on this, it is a path we shouldn't be focusing on13:50
TheJuliaideally everyone should local boot because that means the conductor can go away and the machines continues to run13:50
etingofbut updates? I thought network booting ensures updates which is a feature, no?13:51
*** bobmel has quit IRC13:55
* etingof still needs to give a personal cdrom drive to each manager at sushy-tools13:55
TheJuliaThat is not a feature and not a feature that brings value to BMaaS because we could end up desyncing from the operating system. The best case is for the OS to self manage. Ramdisk/network booting is useful for short lived instances though13:56
*** derekh has joined #openstack-ironic13:56
etingoffor sake of argument, I can think of use-cases where the entire OS (but not data) resides in the ramdisk... diskless belongs to that13:58
TheJuliaand for that reason we built the ramdisk case. The operator needs to update that through rebuild13:59
etingofon other note, perhaps we can introduce boot ISO caching to conductor to save some seeks14:00
TheJuliawould be better to just let operators figure that out since we're starting to see operators scale the number of nodes to conductors to huge numbers14:03
dtantsurI think we're only stuck with netboot because of cirros..14:04
*** khansa has quit IRC14:04
etingofthat might not always be easy for the operators to manage boot ISOs by hand for caching purposes...14:04
TheJuliayeah, the copy to emtpy disk from ramdisk stuff14:04
TheJuliaI started working to try and kill that somewhere14:04
*** khansa has joined #openstack-ironic14:05
TheJuliaetingof: Lets wait for a concrete case from operators. Some BMCs out there do auto-disconnect the media, and ramdisks are supposed to be mapped into ram14:05
* etingof hopes that virtual cdroms out there refuse to eject when the media is mounted14:07
dtantsuretingof: better leave that hope :(14:07
etingofthat worked wonderfully in my windows 9514:07
TheJuliathat was an os level lock14:08
dtantsurit depends on how it is implemented14:08
*** jtomasek has quit IRC14:08
dtantsurif it's through ACPI (?) to the OS - yes.14:08
dtantsurif it's just "oops, it's gone", then no14:08
TheJuliaBMC's just "make it go away"14:08
etingofI am not entirely sure that virtual cdrom as a device is present to the OS before the media is inserted14:10
dtantsurWe can safely assume this to be implementation-specific14:10
TheJuliain _many_ cases it is14:10
TheJulia++14:10
TheJuliavery safely14:10
etingofyeah, it is not impossible with BMCs that bringing device on-line and inserting media is merged into one operation14:10
*** jtomasek has joined #openstack-ironic14:11
dtantsurit makes sense with redfish-based implementations where a CD does not belong to a machine (doh!)14:11
etingoframdisk mmaping is at risk then14:12
etingofin that light, ramdisk should not mmap on anything14:13
dtantsurwell, it just means that the redfish implementation is insane :)14:13
dtantsurwe can only wait and see how it actually works on hardware14:13
iury_coursedtantsur, fyi https://review.opendev.org/697345 the function is not used anywhere yet, but I'll be using on the efi patch and will be used by software raid https://review.opendev.org/#/c/696156/2/ironic_python_agent/extensions/image.py , I pushed a separate patch so my patch wouldn't need to be using 696156 as base patch14:16
patchbotpatch 697345 - ironic-python-agent - Search for efi partition - 1 patch set14:16
*** mkrai has joined #openstack-ironic14:16
patchbotpatch 696156 - ironic-python-agent - Software RAID: make it bootable with uefi - 2 patch sets14:16
dtantsuriury_course: ehm. let's introduce the function where it's actually used?14:16
*** ociuhandu has quit IRC14:16
dtantsurI'm afraid of us changing the course and leaving the function orphaned14:16
iury_courseI'm doing a chain using the function patch to my efi patch (I'll be using) and https://review.opendev.org/#/c/696156/ will also use .-.14:17
patchbotpatch 696156 - ironic-python-agent - Software RAID: make it bootable with uefi - 2 patch sets14:17
iury_courseI wanted to avoid using a huge patch as base to create a chain14:18
dtantsurwhy not?14:18
iury_courseso it wouldn't block the work on software raid / efi14:18
*** Goneri has joined #openstack-ironic14:18
iury_courseTheJulia, thoughts on this? ^14:19
dtantsurhmm, okay, I guess the commit message could be a bit clearer on this14:19
TheJuliaon a conference call at the moment14:19
iury_coursedtantsur, I can update the commit message =)14:19
dtantsuriury_course: don't worry14:19
dtantsurjust keep in mind next time please14:19
iury_courseI promise you the function won't be orphan =) , sure I will keep in mind14:20
*** belmoreira has quit IRC14:21
*** belmoreira has joined #openstack-ironic14:24
dtantsuretingof: do we have a fake System driver in sushy-tools?14:27
dtantsurI'm thinking of checking how much a conductor can handle14:27
TheJulia++14:30
TheJuliaThat should  be something we think a lot about this cycle14:30
dtantsurright14:31
TheJuliaIf we can somehow end up with some jobs, it would be ++++++ good14:32
TheJuliaI'd love a performance based gating job14:32
TheJuliabut, noisy neighbors is always an issue14:32
TheJuliaalthough, maybe we need a scale CI14:33
*** pcaruana has quit IRC14:38
*** cdearborn has joined #openstack-ironic14:38
*** ociuhandu has joined #openstack-ironic14:40
*** goldyfruit_ has joined #openstack-ironic14:48
*** ociuhandu has quit IRC14:57
*** ociuhandu has joined #openstack-ironic14:57
etingofdtantsur, no, System is real14:57
*** goldyfruit___ has joined #openstack-ironic15:00
dtantsurokay, I may look into it at some point15:01
ricolindtantsur, can you re-review? https://review.opendev.org/#/q/project:openstack/heat+topic:story/2006403 :)15:02
*** goldyfruit_ has quit IRC15:02
dtantsursure15:02
ricolindtantsur, thanks!15:02
*** rpittau is now known as rpittau|afk15:03
dtantsurricolin: I cannot +1 anything that has max_ironic_api_microversion15:03
ricolindtantsur, that's fair15:03
etingofdtantsur, what kind of load test we could possibly do with a fake system i.e. not backed by anything alive?15:05
*** gudrutis has joined #openstack-ironic15:05
dtantsuretingof: periodic tasks of the conductor, largely15:06
openstackgerritMerged openstack/ironic-python-agent-builder master: Add words on building image for other architecture  https://review.opendev.org/69722215:06
ricolinbesides that part I need more comments on BaremetalDeployment and BaremetalAllocation resource type15:06
*** tzumainn has joined #openstack-ironic15:07
*** gudrutis has left #openstack-ironic15:07
etingofdtantsur, yeah, we'd need to copy&paste fake something into a fake system then15:07
*** gudrutis has joined #openstack-ironic15:07
dtantsurricolin: I'm looking at them now15:08
*** gudrutis has quit IRC15:08
etingofdtantsur, or we could probably just employ static simulator...15:09
*** gudrutis has joined #openstack-ironic15:09
ricolindtantsur, thanks I will keep your comments on the microversion handling thing in my mind and see if I can get some discussion for that in heat team15:09
dtantsuretingof: probably yes15:10
dtantsurspeaking of which, TheJulia, could you get back to https://storyboard.openstack.org/#!/story/2006910 when you have time again?15:11
dtantsurricolin: this ^^ is re deployment resources, we're shaping a similar API in ironic15:11
*** gudrutis has quit IRC15:17
TheJuliaokay off call15:18
*** ociuhandu has quit IRC15:21
*** ociuhandu has joined #openstack-ironic15:22
openstackgerritDmitry Tantsur proposed openstack/ironic master: docs: add a missing heading  https://review.opendev.org/69749915:23
cdearbornhey folks, we have had this as our vendor priority for some time, but haven't gotten any reviews in quite a while: https://review.opendev.org/#/c/671038/.  Hoping people can break out some time to review this so we can move it forward.  Tx in advance!15:23
patchbotpatch 671038 - ironic - DRAC: Drives conversion from raid to jbod - 21 patch sets15:23
dtantsurcdearborn: +215:26
TheJuliaHeh, dmitry +2, I -1'ed15:26
TheJuliawoot15:26
TheJuliacdearborn: I'm good with a quick follow-up, my issue is only with the release note15:26
TheJuliaas the agent can now do software raid, it would be preferable to indicate the "raid_interface" value required.15:27
dtantsurif it's a feature (i.e. won't be backported), we can probably follow-up15:27
TheJuliaetingof: re sushy-oem-dell, flicker apparently commented about the group name looking weird :(15:28
cdearborndtantsur, TheJulia, LOL! Tx for the quick reviews!  Will get the reno updated.15:29
TheJuliacdearborn: thanks15:30
TheJuliacdearborn: and if there is some desire on dell's part to backport, that change does look super feature-ish15:30
dtantsurdoes or does not?15:31
TheJuliait does15:31
TheJuliato me at least, at a glance15:31
TheJuliawith a reno indicating its a feature15:31
cdearbornTheJulia, dtantsur, it definitely is very boarder line.  I'm good with calling it a feature.15:32
dtantsurif the story had any *why* it would be easier to judge15:32
TheJuliaiury_course: smaller patches are always better, but yeah, just commit message clarity as dmitry suggested should help. Just start keeping a running list somewhere so we don't miss anything15:32
cdearborndtantsur, will update the story to clarify15:32
TheJuliawhy is always important. It makes sense to do to break a raid set, but yeah15:33
iury_courseTheJulia, sure15:33
dtantsurTheJulia: I mean, it can be called a bug fix if the node is not usable after delete_configuration without creating a new configuration15:33
TheJuliaundo the things you've done because a different major configuration could be requsted in the future15:33
TheJuliadtantsur: very very very true15:33
TheJuliacdearborn: ^^^15:33
* TheJulia feels like we're on the same page15:34
*** mkrai has quit IRC15:42
*** mkrai_ has joined #openstack-ironic15:42
etingofTheJulia,  re sushy-oem-dell - copy&paste programming always backfires15:45
TheJuliawoot15:46
dtantsuralways15:46
*** belmoreira has quit IRC15:47
*** belmoreira has joined #openstack-ironic15:51
cdearbornTheJulia, dtantsur: Just added detail to https://storyboard.openstack.org/#!/story/2006234 to further clarify what https://review.opendev.org/#/c/679093/ is intended to do if you're interested15:53
patchbotpatch 679093 - ironic - DRAC: Drives conversion from JBOD to RAID (MERGED) - 25 patch sets15:53
TheJuliathanks!15:53
cdearbornTheJulia: Sure thing!15:53
dtantsurcdearborn: and now it actually looks like a bug fix15:54
TheJuliaDefinitely15:55
TheJulialike...  of the we should backport kind15:55
*** iury_course has quit IRC15:55
cdearborndtantsur, TheJulia: yeah, it's one of those boarder line things.  The end user would definitely perceive it as a bug.15:56
dtantsurcdearborn: I'd recommend reframing as a bug fix and backporting, unless it requires a newer version of dracclient15:57
TheJuliaI concur, the user's perception is key in that15:58
*** khansa has quit IRC16:00
* TheJulia feels so much better that there is one less meeting today16:00
TheJuliabfournie: thanks!16:01
bfournie\o/16:01
dtantsur:)16:02
cdearborndtantsur, TheJulia: ok, sounds good.  It certainly would be nice to have it in prior releases.  Will take a look.  The reno in the JBOD to RAID patch, which already merged lists it as a feature: https://review.opendev.org/#/c/679093/.  This patch is only in master.  Do you think we should do a follow-on patch to that to change the reno to indicate bug?16:02
patchbotpatch 679093 - ironic - DRAC: Drives conversion from JBOD to RAID (MERGED) - 25 patch sets16:02
*** stevebaker has quit IRC16:02
dtantsurouch, so it requires more code?16:03
cdearbornwell, there are 2 parts: automatically convert drives from JBOD to RAID during RAID creation (this part merged already), and automatically convert drives from RAID to JBOD during RAID deletion16:04
dtantsurwhat happens if the first part is not present? RAID will fail?16:04
cdearbornit depends on what mode the disks are in that will be included in the virtual disk.  if they're all in RAID mode, then it will work.  if 1 or more are in JBOD mode, then it will fail16:05
*** mkrai_ has quit IRC16:05
dtantsurugh16:05
dtantsuryeah, it looks like the older patch was also a bug fix16:06
dtantsurcdearborn: I need to note that neither the commit message nor the story provided enough information for us to realize that :-/16:06
*** stevebaker has joined #openstack-ironic16:07
dtantsurgiven that it needs a new version of dracclient, I guess we cannot backport it16:07
dtantsuris it at least documented?16:08
cdearborndtantsur: y, I agree.  Will try to do a better job with that in the future16:08
cdearborndtantsur: I thought we had included that in the docs patch we did for Train, but it looks like we missed it16:13
dtantsurcdearborn: okay, let's at least make this requirement clear in the docs (can be train-only + further backports)16:14
cdearborndtantsur, sounds good.  will fix.  will also ask Rachit to provide follow on patch to reno for JBOD to RAID conversion patch making it a bug fix16:16
*** ociuhandu has quit IRC16:23
*** igordc has joined #openstack-ironic16:27
*** gudrutis has joined #openstack-ironic16:34
TheJuliathanks cdearborn16:34
*** tesseract has quit IRC16:39
*** gyee has joined #openstack-ironic16:41
*** openstackgerrit has quit IRC16:42
*** goldyfruit_ has joined #openstack-ironic16:43
*** goldyfruit___ has quit IRC16:46
*** ricolin has quit IRC16:54
*** rloo has quit IRC16:57
*** ijw_ has joined #openstack-ironic17:02
*** ociuhandu has joined #openstack-ironic17:02
gudrutisarne_wiebalck: o/17:02
*** openstackgerrit has joined #openstack-ironic17:05
openstackgerritIlya Etingof proposed x/sushy-oem-idrac master: Summon Zuul  https://review.opendev.org/69752717:05
TheJuliaetingof: best patch title ever.17:09
*** goldyfruit___ has joined #openstack-ironic17:12
*** goldyfruit_ has quit IRC17:15
dtantsurlol17:18
dtantsuretingof: don't forget to populate .gitreview if it wasn't done for you17:19
* etingof does not have +2 powers over that oem beast17:19
etingofyeah17:19
dtantsuretingof: you need to ask infra to bootstrap the core team17:19
dtantsurwith your humble person :)17:19
dtantsuron this positive note, I wish you good night17:19
*** dtantsur is now known as dtantsur|afk17:19
etingofI thought that's part of the work flow...17:19
dtantsur|afknope17:20
dtantsur|afkat least when I last did it, it was an explicit step17:20
*** lennyb has quit IRC17:20
* dtantsur|afk goes for real17:20
etingofwhom should I ping exactly?17:20
clarkbits explicit because there isn't a good way for our automation to know who to seed that group with. Though there are plans to change that in the hopefully near future17:21
openstackgerritIlya Etingof proposed x/sushy-oem-idrac master: Add basic CI jobs and .gitreview  https://review.opendev.org/69752917:22
clarkbetingof: I've updated the groups17:23
clarkbyou should be able to self manage them now17:24
etingofclarkb, ah, thanks! I already went to infra17:24
clarkbwhere? I didn't see the request? in any case its done17:24
openstackgerritMerged x/sushy-oem-idrac master: Summon Zuul  https://review.opendev.org/69752717:25
etingofclarkb, I did not hit Enter yet ;)17:25
etingofit took me a while to ask promptly17:25
*** lennyb has joined #openstack-ironic17:26
*** lennyb has quit IRC17:32
*** lennyb has joined #openstack-ironic17:37
*** gudrutis2 has joined #openstack-ironic17:38
*** gudrutis2 has quit IRC17:39
*** lennyb has quit IRC17:41
*** gudrutis2 has joined #openstack-ironic17:42
*** lucasagomes has quit IRC17:43
*** lennyb has joined #openstack-ironic17:46
*** derekh has quit IRC18:00
openstackgerritMerged openstack/ironic-lib master: mdns: default to listing on both V4 and V6 interfaces  https://review.opendev.org/69746718:02
*** lennyb has quit IRC18:06
*** rloo has joined #openstack-ironic18:08
*** ociuhandu has quit IRC18:17
*** bfournie has quit IRC18:25
*** alexmcleod has quit IRC18:33
*** Goneri has quit IRC18:35
*** Goneri has joined #openstack-ironic18:44
*** lennyb has joined #openstack-ironic18:53
*** dsneddon has joined #openstack-ironic18:59
*** ociuhandu has joined #openstack-ironic19:07
*** ijw_ has quit IRC19:17
*** dsneddon has quit IRC19:21
*** dsneddon has joined #openstack-ironic19:22
*** bfournie has joined #openstack-ironic19:28
*** ociuhandu has quit IRC19:31
*** rcernin has joined #openstack-ironic20:02
*** igordc has quit IRC20:05
*** dsneddon has quit IRC20:09
*** dsneddon has joined #openstack-ironic20:10
*** rcernin has quit IRC20:18
*** bobmel has joined #openstack-ironic20:23
*** bobmel has quit IRC20:23
*** bobmel has joined #openstack-ironic20:23
*** bobmel has quit IRC20:29
*** bobmel has joined #openstack-ironic20:46
*** mrda has quit IRC21:09
*** jtomasek has quit IRC21:12
*** dsneddon has quit IRC21:18
*** dsneddon has joined #openstack-ironic21:18
*** mrda has joined #openstack-ironic21:22
*** gudrutis has quit IRC21:24
*** bobmel has quit IRC21:30
*** bobmel has joined #openstack-ironic21:32
*** bobmel has quit IRC21:32
*** bobmel has joined #openstack-ironic21:32
*** dsneddon has quit IRC21:33
*** dsneddon has joined #openstack-ironic21:33
*** Goneri has quit IRC21:38
*** dsneddon has quit IRC21:39
*** dsneddon has joined #openstack-ironic21:40
*** ociuhandu has joined #openstack-ironic21:41
*** persia has quit IRC21:43
*** persia has joined #openstack-ironic21:45
*** ijw has joined #openstack-ironic21:46
*** mrda has quit IRC21:53
*** mrda has joined #openstack-ironic21:57
*** goldyfruit___ has quit IRC22:05
*** ociuhandu has quit IRC22:20
*** rcernin has joined #openstack-ironic22:28
openstackgerritJulia Kreger proposed openstack/ironic master: WIP: pre-shared agent token  https://review.opendev.org/69260922:38
openstackgerritJulia Kreger proposed openstack/ironic master: WIP: Enable agent_token for virtual media boot  https://review.opendev.org/69758422:38
openstackgerritJulia Kreger proposed openstack/ironic master: WIP: Send our token back to the agent  https://review.opendev.org/69758522:39
*** ijw has quit IRC23:04
*** dsneddon has quit IRC23:05
*** dsneddon has joined #openstack-ironic23:13
*** gudrutis2 has quit IRC23:19
*** dsneddon has quit IRC23:23
*** sshnaidm is now known as sshnaidm|afk23:24
*** ociuhandu has joined #openstack-ironic23:30
*** ociuhandu has quit IRC23:35
*** ijw has joined #openstack-ironic23:36
*** cdearborn has quit IRC23:38
*** ijw has quit IRC23:40
*** dsneddon has joined #openstack-ironic23:51

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