Tuesday, 2020-03-03

« Monday, 2020-03-02 Index Wednesday, 2020-03-04 »
*** threestrands has joined #openstack-ironic00:03
*** bfournie has joined #openstack-ironic00:17
*** diurnalist has quit IRC00:26
*** k_mouza has joined #openstack-ironic00:45
*** k_mouza has quit IRC00:49
*** aedc has quit IRC00:50
*** k_mouza has joined #openstack-ironic00:55
*** aedc has joined #openstack-ironic00:58
*** k_mouza has quit IRC01:00
*** erbarr has quit IRC01:09
*** yolanda has quit IRC01:15
*** ijw has quit IRC01:25
*** cdearborn has quit IRC01:52
*** gyee has quit IRC02:08
*** rloo has quit IRC02:15
*** Goneri has quit IRC02:41
*** mkrai has joined #openstack-ironic02:51
*** rcernin_ has joined #openstack-ironic02:52
*** rcernin has quit IRC02:55
*** dsneddon has quit IRC03:15
*** mkrai has quit IRC03:15
*** mkrai has joined #openstack-ironic03:21
*** jdandrea has joined #openstack-ironic03:34
*** jdandrea has quit IRC03:37
*** ricolin has quit IRC04:15
*** subhamrai03 has joined #openstack-ironic04:22
*** ociuhandu has joined #openstack-ironic04:55
*** ociuhandu has quit IRC04:59
*** subhamrai03 has quit IRC05:04
*** subhamrai03 has joined #openstack-ironic05:22
*** rcernin_ has quit IRC05:24
*** rachit7 has joined #openstack-ironic05:33
*** vesper has quit IRC05:34
*** vesper11 has joined #openstack-ironic05:37
*** diurnalist has joined #openstack-ironic05:42
*** diurnalist has quit IRC05:47
openstackgerritMerged openstack/sushy master: Fix incorrect usage of assertRaisesRegex in unit tests  https://review.opendev.org/71080205:55
openstackgerritM V P Nitesh proposed openstack/ironic-python-agent master: Fixes error handling if efibootmgr is not present in ramdisk  https://review.opendev.org/70910706:01
*** igordc has joined #openstack-ironic06:09
*** igordc has quit IRC06:13
*** yedongcan has joined #openstack-ironic06:15
*** rcernin has joined #openstack-ironic07:13
*** subhamrai03 has quit IRC07:14
*** hwoarang has quit IRC07:18
arne_wiebalckGood morning, ironic!07:20
*** yolanda has joined #openstack-ironic07:21
*** mkrai has quit IRC07:21
*** hwoarang has joined #openstack-ironic07:21
*** ijw has joined #openstack-ironic07:30
openstackgerritMaël Kimmerlin proposed openstack/ironic master: Redfish: Add root_prefix to Sushy  https://review.opendev.org/70734207:30
*** subhamrai03 has joined #openstack-ironic07:32
*** ricolin has joined #openstack-ironic07:33
*** mkrai has joined #openstack-ironic07:35
openstackgerritHarald Jensås proposed openstack/ironic master: Additional IP addresses to IPv6 stateful ports  https://review.opendev.org/70000207:37
*** subhamrai03 has quit IRC08:06
*** iurygregory has joined #openstack-ironic08:26
iurygregorygood morning Ironic o/08:26
*** hwoarang has quit IRC08:27
*** hwoarang has joined #openstack-ironic08:28
*** tesseract has joined #openstack-ironic08:30
*** iurygregory has quit IRC08:36
*** rpittau|afk is now known as rpittau08:41
rpittaugood morning ironic! o/08:41
*** iurygregory has joined #openstack-ironic08:41
*** rcernin has quit IRC08:41
*** FlorianFa has quit IRC09:05
*** ociuhandu has joined #openstack-ironic09:06
*** alexmcleod has joined #openstack-ironic09:07
*** threestrands has quit IRC09:09
*** ociuhandu has quit IRC09:12
*** ociuhandu has joined #openstack-ironic09:15
*** k_mouza has joined #openstack-ironic09:21
*** tesseract has quit IRC09:25
*** lucasagomes has joined #openstack-ironic09:25
*** tesseract has joined #openstack-ironic09:26
*** iurygregory has quit IRC09:27
*** Lucas_Gray has joined #openstack-ironic09:29
openstackgerritRiccardo Pittau proposed openstack/ironic master: Default IRONIC_RAMDISK_TYPE to dib  https://review.opendev.org/71027809:36
*** ociuhandu has quit IRC09:37
*** ijw has quit IRC09:43
*** ijw_ has joined #openstack-ironic09:43
*** iurygregory has joined #openstack-ironic09:43
*** derekh has joined #openstack-ironic09:43
*** ociuhandu has joined #openstack-ironic09:45
*** ociuhandu has quit IRC09:58
*** khansa has joined #openstack-ironic10:01
*** dtantsur|afk is now known as dtantsur10:03
dtantsurmorning ironic10:03
*** rcernin has joined #openstack-ironic10:04
iurygregorymorning dtantsur10:08
*** ociuhandu has joined #openstack-ironic10:25
*** ociuhandu has quit IRC10:27
*** ociuhandu has joined #openstack-ironic10:27
*** k_mouza has quit IRC10:27
*** rcernin has quit IRC10:37
openstackgerritDmitry Tantsur proposed openstack/ironic master: Docs: split away user image building and highlight whole disk images  https://review.opendev.org/71096610:38
*** rcernin has joined #openstack-ironic10:38
dtantsurbased on field feedback ^^^10:38
*** ociuhandu has quit IRC10:39
*** k_mouza has joined #openstack-ironic10:47
*** ajya[m] has quit IRC10:56
*** mkowalski[b] has quit IRC10:56
*** openstackstatus has quit IRC10:59
*** Lucas_Gray has quit IRC11:01
etingofsomehow, Depends-On sushy no longer works, it seems...11:02
dtantsuretingof: it never had, as with all libraries11:06
etingofhmmm, how did I make that work...?11:08
etingofwhen I have ironic patch depending on a sushy patch11:09
dtantsuretingof: temporary adding sushy to required-projects?11:11
openstackgerritVerification of a change to openstack/ironic-python-agent failed: Fixes error handling if efibootmgr is not present in ramdisk  https://review.opendev.org/70910711:11
*** ociuhandu has joined #openstack-ironic11:12
openstackgerritDmitry Tantsur proposed openstack/sushy stable/train: Fix incorrect usage of assertRaisesRegex in unit tests  https://review.opendev.org/71097911:13
*** rachit7 has quit IRC11:15
*** rpittau is now known as rpittau|bbl11:17
mgoddardhi dtantsur11:17
mgoddardwdyt about https://review.opendev.org/70984911:18
patchbotpatch 709849 - releases - Switch networking-baremetal to cycle-with-rc - 1 patch set11:18
mgoddardI don't know why they care so much :)11:18
mgoddardsurely the intermediary should be optional?11:19
*** ajya[m] has joined #openstack-ironic11:20
dtantsurmgoddard: I don't see sense in cycle-with-rc, to be honest..11:21
dtantsurbut people apparently think it's useful. I have been refusing these requests while I was the liaison.11:21
dtantsurNow I guess it's yours and TheJulia's call :)11:22
mgoddarddid you refuse by making an intermediate release, or just saying no?11:22
dtantsurmgoddard: by saying no :) But I don't have any opinion, I think the whole discussion is a bit pointless..11:23
TheJuliaI need coffee for this11:23
iurygregorydouble coffee11:23
dtantsurfor me cycle-with-rc is a degraded version of cycle-with-intermediary11:23
mgoddard+111:23
iurygregorybecause 1 it's never enough11:23
* TheJulia is just waking up11:23
mgoddardmaybe from release team perspective it's just easier to have everything the same so they push for it?11:23
dtantsurTheJulia: I'd suggest delaying waking up11:24
dtantsurmgoddard: it's exactly the reason11:24
TheJuliaBy the way, the using multitenant ironic for elastic secure Infrastructure has 21 people in a BOF last night11:24
dtantsurwow11:24
TheJuliaTons of interest!11:24
dtantsurTheJulia: and I saw another person in our telegram chat trying standalone ironic (hence the doc patch above)11:25
dtantsur\o/11:25
dtantsurmgoddard: you can always ask them so that we don't play a broken phone11:26
TheJuliaThe release team definitely wants everyone the same11:27
TheJuliaReality is that it doesn't really work of your not a core project11:27
TheJuliaAnd I realize how that sounds, but it has basically been enshrined in code and testing patterns.11:28
dtantsurexactly11:29
TheJuliaAnd nobody wants to change that or shake that up. Well, zaneb maybe... I promised him I would read his proposal this week11:29
dtantsurmmm, there is a proposal?11:29
TheJuliaOpenstack/ideas repo11:30
* dtantsur is ready to do anything that will distract him from filing expenses... sigh....11:30
TheJuliaLol11:30
TheJuliaI need to do that... When I get home.11:30
iurygregoryfiling expenses is funny XD11:30
dtantsur"funny". type 2 fun :)11:31
iurygregoryit doesn't take too much time since I'm using the app on the phone hehehe11:31
TheJuliaI have never had it line things up properly so I don't bother with the app11:32
*** mkrai has quit IRC11:33
*** mkowalski[b] has joined #openstack-ironic11:43
*** mkrai has joined #openstack-ironic11:43
*** trident has quit IRC11:46
*** trident has joined #openstack-ironic11:49
*** Lucas_Gray has joined #openstack-ironic11:50
*** rcernin has quit IRC11:52
*** Lucas_Gray has quit IRC11:55
*** khansa has quit IRC12:10
*** subhamrai03 has joined #openstack-ironic12:10
*** mkrai has quit IRC12:12
*** Lucas_Gray has joined #openstack-ironic12:20
openstackgerritIlya Etingof proposed openstack/ironic master: Improve `redfish` set-boot-mode implementation  https://review.opendev.org/71084612:23
openstackgerritDmitry Tantsur proposed openstack/ironic master: Docs: split away user image building and highlight whole disk images  https://review.opendev.org/71096612:25
*** mkrai has joined #openstack-ironic12:26
*** Lucas_Gray has quit IRC12:31
dtantsurTheJulia: could you check https://review.opendev.org/#/c/710766/ please? It seems that we can recheck it into merging, and that will unblock stable/rocky for the final release.12:34
patchbotpatch 710766 - ironic (stable/rocky) - [stable] consume virtualbmc from pip packages - 1 patch set12:34
*** aedc has quit IRC12:34
*** aedc has joined #openstack-ironic12:35
*** jcoufal has joined #openstack-ironic12:35
*** bfournie has quit IRC12:44
*** jcoufal has quit IRC12:48
*** aedc has quit IRC12:50
*** aedc has joined #openstack-ironic12:51
*** khansa has joined #openstack-ironic12:53
dtantsuralso, could someone please review https://review.opendev.org/#/c/708681/ ? it's mostly moving code around12:56
patchbotpatch 708681 - ironic - Refactoring: move generic agent clean step functio... - 2 patch sets12:56
*** dougsz has joined #openstack-ironic13:00
dtantsurI'll probably have to rebase it, sigh13:02
openstackgerritDmitry Tantsur proposed openstack/ironic master: Refactoring: move generic agent clean step functions to agent_base  https://review.opendev.org/70868113:03
openstackgerritDmitry Tantsur proposed openstack/ironic master: Refactoring: move iSCSI deploy code to iscsi_deploy.py  https://review.opendev.org/70869713:03
openstackgerritDmitry Tantsur proposed openstack/ironic master: Provide a foundation for purely out-of-band deploy/clean steps execution  https://review.opendev.org/70863813:04
iurygregoryonly moving code around XD 3 patches13:04
*** rpittau|bbl is now known as rpittau13:04
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: In-band deploy steps  https://review.opendev.org/69876813:05
*** ociuhandu has quit IRC13:06
*** mkrai has quit IRC13:06
*** ociuhandu has joined #openstack-ironic13:07
*** Lucas_Gray has joined #openstack-ironic13:11
*** ociuhandu has quit IRC13:12
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: In-band deploy steps  https://review.opendev.org/69876813:13
*** subhamrai03 has quit IRC13:14
*** subhamrai03 has joined #openstack-ironic13:16
openstackgerritDmitry Tantsur proposed openstack/ironic master: Add RPC objects for deployment API  https://review.opendev.org/70377813:17
openstackgerritDmitry Tantsur proposed openstack/ironic master: [WIP] Conductor changes for deployment API  https://review.opendev.org/70736213:17
*** jcoufal has joined #openstack-ironic13:19
*** subhamrai03 has quit IRC13:22
*** mkrai has joined #openstack-ironic13:28
*** k_mouza has quit IRC13:33
*** jdandrea has joined #openstack-ironic13:33
*** mkrai has quit IRC13:33
dtantsurrpittau: if you have a minute: https://review.opendev.org/#/c/710766/13:36
patchbotpatch 710766 - ironic (stable/rocky) - [stable] consume virtualbmc from pip packages - 1 patch set13:36
rpittauoh it's passing now, good13:37
dtantsuryup13:37
openstackgerritDmitry Tantsur proposed openstack/ironic stable/rocky: tell reno to ignore the kilo branch  https://review.opendev.org/70678613:38
*** bfournie has joined #openstack-ironic13:41
openstackgerritDmitry Tantsur proposed openstack/ironic stable/queens: [stable] consume virtualbmc from pip packages  https://review.opendev.org/71100513:41
dtantsurlet's try to revive queens the same way13:41
TheJuliaoooh ahh13:44
iurygregoryseems like we are playing RPG XD13:45
* dtantsur is curious what emotions TheJulia is trying to express :)13:45
dtantsuriurygregory: wait, we are not? they lied on their job posting???13:45
iurygregorydtantsur, they lied only a little bit XD13:46
dtantsurthat's a relief :)13:46
openstackgerritIlya Etingof proposed openstack/sushy master: Decouple `set_system_boot_source` parameters  https://review.opendev.org/71075113:49
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Set abstract for ironic-base  https://review.opendev.org/71101613:56
*** rloo has joined #openstack-ironic13:56
TheJuliareviving, I keep getting distracted by mnansur and fungi13:57
* TheJulia should turn and glare at them13:57
* fungi distracts TheJulia some more13:58
*** zigo has quit IRC13:58
openstackgerritIlya Etingof proposed openstack/ironic master: Improve `redfish` set-boot-mode implementation  https://review.opendev.org/71084613:59
dtantsurrpittau: do you remember the background behind https://review.opendev.org/#/c/690537/ ? I wonder if something like that could help with gmann's tempest-plugin patch?14:03
patchbotpatch 690537 - ironic - Using loop instead of with_X (MERGED) - 7 patch sets14:03
dtantsuralthough I have no clue why the same problem doesn't affect stein14:03
rpittaummmm14:04
rpittauif I remember correctly, it's a matter of compatibility with recent Ansible versions and basically just the recommended way to handle loops14:05
iurygregorydtantsur, this is to avoid ansible getting /dev/sdb14:05
rpittauyeah, it was giving some weird errors when dealing with multiple choices14:06
dtantsuriurygregory: riiiiight. this may explain what we see in the tempest-plugin patch...14:06
iurygregorybecause the instance was supposed to boot using /dev/sda14:06
rpittauyep14:06
iurygregorydtantsur, yeah!14:06
iurygregoryI was about to say in the review14:06
dtantsurso just backport it - and happiness, except that we cannot backport it >_<14:06
iurygregoryD:14:06
dtantsurwhich is pretty stupid since ansible 2.4 is ancient14:07
rpittauso sadness? should I finish the bottle of wine I've opened at lunch?14:07
dtantsurrpittau: you probably should14:08
iurygregoryrpittau, please do14:08
iurygregorymaybe we can do the ugly way I think14:08
iurygregorysicne we can't bump =X14:08
dtantsurTheJulia: wdyt about backporting https://review.opendev.org/#/c/690537/7/driver-requirements.txt to train, given that the oldest supported ansible is 2.7?14:08
patchbotpatch 690537 - ironic - Using loop instead of with_X (MERGED) - 7 patch sets14:08
dtantsurI can prepare a backport with a convincing enough release note if you're fine14:09
rpittauif I'm not mistaken bifrost also support only from 2.6 in train, I guess that was kind of a forgotten thing14:11
rpittauI'm tempted to bump current supported to 2.7 in ironic too14:11
dtantsurrpittau: we should probably start habit of bumping it to the oldest supported release14:13
rpittaudtantsur: totally agreed14:13
dtantsurto avoid situations like this one when we have a technically breaking change even though no sane person should use 2.4 at this point14:13
* rpittau sips white wine and nods14:13
* dtantsur is jealous14:13
*** yedongcan has left #openstack-ironic14:15
openstackgerritDmitry Tantsur proposed openstack/ironic stable/train: Using loop instead of with_X  https://review.opendev.org/71102014:16
dtantsursomething like ^^14:17
openstackgerritRiccardo Pittau proposed openstack/ironic master: [WIP] Bump minimum supported ansible version to 2.7  https://review.opendev.org/71102114:17
*** k_mouza has joined #openstack-ironic14:17
openstackgerritDmitry Tantsur proposed openstack/ironic stable/train: DNM Test standalone job with Python 3  https://review.opendev.org/71102214:18
iurygregorydtantsur, ++14:20
*** Goneri has joined #openstack-ironic14:21
dtantsurI cannot find the exact 2.4 EOL date, but presumably when 2.7 was released, i.e. in 2018, which is before train14:22
dtantsur(they support 3 releases)14:22
gmanndtantsur: thanks for checking py2 drop patch. will you make py3 job running on train along with py2- https://review.opendev.org/#/c/711022/114:25
patchbotpatch 711022 - ironic (stable/train) - DNM Test standalone job with Python 3 - 1 patch set14:25
rpittaudtantsur: October 9, 201814:25
dtantsurgmann: that's not the intention, I'm merely testing the patch14:25
gmannbecause if we run train job on py3 on master gate then we should do on train gate also14:25
gmannif anything non-py3 things goes in train14:26
dtantsurgmann: I'm not sure why. Were we supposed to switch the CI to Python 3 in Train?14:26
*** iurygregory has quit IRC14:26
rpittaudtantsur: latest 2.4 release was 2.4.6 (2018-07-05)14:27
dtantsuryeah, looks acceptable to drop it14:27
gmanndtantsur: not required but I am worried on if an backport break train on py3 which can break train job on master gate of ironic-tempest-plugin as that run under py3. But i think that can be handled on pinning plugins like neutron does.14:28
dtantsurProbably. One problem at a time..14:28
*** iurygregory has joined #openstack-ironic14:28
gmannok14:28
* TheJulia gets too much email14:28
iurygregorydtantsur, don't you need depends-On?14:29
dtantsuriurygregory: where?14:29
iurygregorydtantsur, nvm just noticed it's in chain XD14:30
dtantsurTheJulia: use the button with a trash bin on it :)14:30
*** tkajinam has quit IRC14:30
iurygregoryi thought you would be testing in ironic-tempest-plugin...14:30
dtantsurOr delete a random half of them (you don't need to answer unlucky people)14:30
*** tkajinam has joined #openstack-ironic14:31
dtantsuriurygregory: doesn't matter much, it's the same job. I'm not sure depends-on will work correctly given how the tempest-plugin CI is configured.14:31
iurygregorygotcha14:31
*** zigo has joined #openstack-ironic14:32
*** tzumainn has joined #openstack-ironic14:38
*** ijw_ has quit IRC14:41
iurygregorydtantsur, I think at the mid-cycle you said something about renaming agent_ipmitool in job names, would be to direct since it's the deploy?15:00
openstackgerritIlya Etingof proposed openstack/sushy master: Add `set_system_boot_options` method  https://review.opendev.org/71075115:00
*** ociuhandu has joined #openstack-ironic15:03
dtantsuriurygregory: yep. something like ipmi-direct, we should have similar names around already15:07
rpittauthe ipmitool direct job is not working with dib :/15:09
iurygregorydtantsur, ack15:09
*** tkajinam has quit IRC15:09
iurygregoryrpittau, D:15:09
dtantsurrpittau: I'm pretty sure I've seen a successful run15:10
dtantsurwhere is that?15:10
rpittaudtantsur: https://review.opendev.org/#/c/710278/15:10
patchbotpatch 710278 - ironic - Default IRONIC_RAMDISK_TYPE to dib - 11 patch sets15:10
dtantsurrpittau: the streaming jobs do seem to succeed, hmm15:11
* rpittau is not sure will survive the change of contexts today15:11
dtantsurrpittau: welcome to my world :D15:13
* dtantsur is in two discussions and one meeting atm15:13
iurygregoryrpittau, don't we need to set IRONIC_RAMDISK_TYPE to dib?15:26
iurygregoryby default it's tinyipa15:26
rpittauiurygregory: it's set to dib15:27
iurygregoryjust found it15:27
rpittauiurygregory: https://review.opendev.org/#/c/710278/11/devstack/lib/ironic15:27
patchbotpatch 710278 - ironic - Default IRONIC_RAMDISK_TYPE to dib - 11 patch sets15:27
rpittauok15:27
iurygregoryso we are using centtos8 in everything with dib right?15:29
rpittauiurygregory: yep15:36
*** Goneri has quit IRC15:36
iurygregoryrpittau, cool =)15:37
rpittauall the jobs are working so we could actually publish ubuntu, fedora, debian, and centos7 too :P15:37
iurygregoryrpittau, you want to use all the openstack infra? XD15:38
TheJuliadtantsur: +2'ed https://review.opendev.org/#/c/711020/115:38
patchbotpatch 711020 - ironic (stable/train) - Using loop instead of with_X - 1 patch set15:38
dtantsurthx15:39
*** ociuhandu has quit IRC15:39
TheJuliadtantsur: for https://review.opendev.org/#/c/711005/, maybe we just drop the multinode/grenade jobs. I think we had to do that the last time we wanted to do a last minute backport to the last branch15:43
patchbotpatch 711005 - ironic (stable/queens) - [stable] consume virtualbmc from pip packages - 1 patch set15:43
dtantsurTheJulia: this is on my list, yes15:43
*** tesseract has quit IRC15:43
TheJuliaI can propose the change to drop the tests15:44
dtantsurTheJulia: you may have to squash it with the vbmc change. but maybe worth giving it a try first?15:47
TheJuliaoh... that is a super good point15:48
TheJuliayeah, maybe15:48
TheJuliaWe could also just drop the jobs and turn them back on in the vbmc change15:48
TheJulia. o ( filing under likely a bad idea )15:49
TheJuliaactually, we want todo this to fix rocky for final release?15:52
TheJulialets drop the tests in rocky15:52
TheJuliaspecifically grenade if that is what is breaking15:52
*** tesseract has joined #openstack-ironic15:53
dtantsurTheJulia: rocky is fine with https://review.opendev.org/#/c/710766/15:53
patchbotpatch 710766 - ironic (stable/rocky) - [stable] consume virtualbmc from pip packages - 1 patch set15:53
dtantsurbut we should drop grenade there for sure since it's testing EM->EM15:53
dtantsurwhich is not going to work for long :)15:53
TheJuliaI'm talking with fungi and that is bascially the consensus15:54
TheJulianuke grenade job on the oldest EM branch15:54
dtantsurTheJulia: if you want to torture your head, you can also stare at https://review.opendev.org/#/c/706785/15:54
patchbotpatch 706785 - ironic (stable/stein) - tell reno to ignore the kilo branch - 1 patch set15:54
dtantsurgrenade fails consistently and weirdly15:54
openstackgerritJulia Kreger proposed openstack/ironic stable/queens: Remove grenade jobs from queens branch  https://review.opendev.org/71104315:57
fungiyeah, stable/queens is not a supported stable branch, it's under extended maintenance. testing upgrades from it to stable/rocky is questionable, not to mention rocky is past due to enter em itself15:57
TheJuliaone of two dropping the em grenade jobs15:58
* TheJulia goes and gets coffee15:58
fungicoffeecoffeecoffee15:59
TheJuliaIs there a coffee song? Could we start one at the table?16:03
openstackgerritJulia Kreger proposed openstack/ironic stable/rocky: Remove rocky grenade jobs  https://review.opendev.org/71104516:05
TheJuliathe academic and research folks would <3 if we had some support for linuxbios.16:09
rpittaucoreboot ?16:10
iurygregorylol16:11
iurygregorycoreboot nice name16:11
rpittauhttps://en.wikipedia.org/wiki/Coreboot16:12
iurygregoryit's a rabbit.. didn't we kill the rabbits?16:12
iurygregoryops not we but dtantsur XD16:13
rpittaupoor rabbits :(16:13
iurygregoryfor this one yeah =(16:13
* dtantsur gets his gun16:14
dtantsurTheJulia: what exactly we don't support now? does it use some fancy flavor of PXE?16:14
TheJuliaI suspect the hope is for agent in the bios16:15
TheJulia*boom*16:15
TheJuliaI feel like I had a similar discusison with the LANL folks16:16
dtantsuragent in the bios?16:16
TheJulialike ipa, but I'm not solid on the details. I need to circle back to the person during a break16:17
*** diurnalist has joined #openstack-ironic16:17
dtantsursounds like a recipe for troubles..16:18
TheJuliapossibly16:22
TheJuliamore a note than anything16:23
* TheJulia sticks her brain into IPA unit testing16:23
*** gyee has joined #openstack-ironic16:27
iurygregory1.28 is quite old right? =)16:28
iurygregory"# TODO(vsaienko) switch to get_node_vifs() when all stable releases supports Ironic API 1.28"16:30
dtantsuriurygregory: https://docs.openstack.org/ironic/latest/contributor/webapi-version-history.html says Ocata16:30
dtantsurso yeah, it's ancient16:30
iurygregoryso we can remove the check in tempest-plugin XD16:30
* dtantsur is worried that he participated in releases now considered ancient16:30
iurygregoryI said old not ancient.. /me is safe16:31
dtantsurthese modern kids..16:31
iurygregorynext year I'll be 30 .-.16:32
dtantsurI'll welcome you to the club then :D16:33
iurygregoryThanks =)16:33
TheJuliaheh16:33
iurygregoryIf I remove my beard people say I look like a kid16:34
* TheJulia sighs at unit test that passes when it should fail16:34
dtantsuriurygregory: that's why I don't have a beard :D16:37
iurygregorydtantsur, hehehehe16:37
*** jdandrea has quit IRC16:37
*** subhamkrai has joined #openstack-ironic16:37
*** jdandrea has joined #openstack-ironic16:38
* rpittau feeling old16:39
etingofiurygregory, should grow unix beard16:41
iurygregoryetingof, would be nice16:42
etingofrpittau try thinking of age in base 1616:45
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-tempest-plugin master: Refactor validate_ports function  https://review.opendev.org/71105416:46
iurygregoryholy moly, we have some "Remove this when all stable releases support these API versions... XD16:48
dtantsursomebody has to do 'grep TODO' periodically. should we make it a TODO item? ;)16:53
iurygregorydtantsur, I'm doing this atm for the ironic-tempest XD16:53
dtantsurnice :)16:56
iurygregorydtantsur, some TODO are in Queens time frame, we should wait till ironic doesn't have stable/queens or it's ok to change things?16:57
dtantsuriurygregory: depends on the case? we shouldn't outright break queens at this point, I think.16:58
iurygregorye.g https://github.com/openstack/ironic-tempest-plugin/blob/2ebfc15bf4424bf9fa32f5cfeb641b384c76b6c9/ironic_tempest_plugin/tests/scenario/test_baremetal_basic_ops.py#L16316:58
*** tesseract has quit IRC16:58
iurygregoryhttps://github.com/openstack/ironic-tempest-plugin/blob/0cbe05eff6b1618e0cba700e246680f91e029832/ironic_tempest_plugin/config.py#L21616:58
dtantsuriurygregory: we can assume 1.21 to be present. defaulting to queens will break pike.. which I'm not sure is alive anyway.16:59
dtantsuryeah, the 2nd example should probably stay until we retire queens completely17:00
*** lucasagomes has quit IRC17:00
iurygregoryOcata Pike Queens are in Unmaintained17:00
rpittauiurygregory: I think they're still in EM17:01
iurygregoryrpittau, not according to https://releases.openstack.org/17:01
rpittauI'm looking at that :)17:02
iurygregoryoh nvm17:02
rpittauthey're in EM17:02
iurygregoryit's the Next Phase =)17:02
* iurygregory was too fast17:02
*** subhamkrai has quit IRC17:03
dtantsurI'd vote for killing pike and ocata for ironic projects. It's been a while since anything merged there, and I'm not sure who here is ready to keep them alive.17:03
rpittauI guess ocata and pike are gone anyway17:03
dtantsurQueens is of some interest for Red Hat, so it's in a better position17:03
iurygregoryyeah17:04
* iurygregory won't change things that say "Queens" in the tempest plugin XD17:04
dtantsur++17:04
dtantsurI guess the final call is up to TheJulia17:04
* TheJulia blinks17:05
TheJuliawhat?!?17:05
* TheJulia reads17:05
dtantsurTheJulia: I'm essentially proposing to stop maintaining Ocata and Pike17:05
TheJulia+100017:05
dtantsurto avoid ending up with 4 EM releases to support17:05
TheJuliait is in EM, there is no reason to17:05
TheJuliathe branches can remain there17:05
dtantsurTheJulia: I think branches are killed when EM is over, replaced with tags like ocata-eol17:06
TheJuliabut we shouldb't really be trying to put effort into those... period.17:06
*** k_mouza has quit IRC17:06
TheJuliadtantsur: I'd have to look it up at this point, but once in EM it up to the project team17:06
dtantsurTheJulia: it's up to us when to end them17:07
TheJuliadefine end them17:07
iurygregorykill with fire XD17:07
dtantsurTheJulia: replace a branch with a tag and stop accepting patches therefore17:07
dtantsuriurygregory++17:07
iurygregoryhehehe17:08
TheJuliaI'd put it up to a vote, tbh17:08
* iurygregory gtg time to do grocery shopping 17:08
dtantsurTheJulia: I'll put on the next meeting agenda17:08
iurygregorywe can bring this up in our meeting  on monday =)17:08
dtantsuradded17:09
iurygregorybye ppl o/17:09
dtantsur\o17:10
TheJuliaexcellent17:10
rpittaubye everyone, good night! o/17:10
*** rpittau is now known as rpittau|afk17:10
*** bdodd has quit IRC17:12
dtantsurI think I'll go as well17:13
*** dtantsur is now known as dtantsur|afk17:13
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Agent token support  https://review.opendev.org/69261417:19
TheJuliadtantsur|afk: ^^^^ was painful17:19
*** ociuhandu has joined #openstack-ironic17:21
*** diurnalist has quit IRC17:21
* etingof realized that pep8 still has something in common with the punch cards17:24
*** diurnalist has joined #openstack-ironic17:26
dtantsur|afkTheJulia: ugh :( will check tomorrow17:26
*** derekh has quit IRC17:28
*** ociuhandu_ has joined #openstack-ironic17:40
*** ociuhandu has quit IRC17:43
*** ociuhandu_ has quit IRC17:44
openstackgerritVerification of a change to openstack/ironic failed: [stable] consume virtualbmc from pip packages  https://review.opendev.org/71076617:46
openstackgerritVerification of a change to openstack/ironic failed: tell reno to ignore the kilo branch  https://review.opendev.org/70678617:46
rpiosoGood morning, ironic17:52
* rpioso feels like it's the evening.17:52
*** igordc has joined #openstack-ironic17:59
*** diurnalist has quit IRC18:01
*** Lucas_Gray has quit IRC18:12
*** k_mouza has joined #openstack-ironic18:13
*** khansa has quit IRC18:14
*** jcoufal has quit IRC18:16
*** dsneddon has joined #openstack-ironic18:17
*** jcoufal has joined #openstack-ironic18:17
*** k_mouza has quit IRC18:18
*** diurnalist has joined #openstack-ironic18:28
*** jcoufal_ has joined #openstack-ironic18:31
*** diurnalist has quit IRC18:33
*** jcoufal has quit IRC18:34
*** diurnalist has joined #openstack-ironic19:25
*** ociuhandu has joined #openstack-ironic20:10
*** aedc has quit IRC20:11
*** aedc has joined #openstack-ironic20:11
*** ociuhandu has quit IRC20:15
TheJuliaDefinitely the evening20:17
TheJuliaalthough my body says "please go sleep meow"20:17
*** trident has quit IRC20:23
*** trident has joined #openstack-ironic20:26
*** aedc_ has joined #openstack-ironic20:27
*** aedc has quit IRC20:27
*** dougsz has quit IRC20:44
*** Goneri has joined #openstack-ironic20:47
*** diurnalist has quit IRC20:56
*** k_mouza has joined #openstack-ironic20:58
*** diurnalist has joined #openstack-ironic21:00
*** jcoufal_ has quit IRC21:02
*** k_mouza has quit IRC21:02
*** igordc has quit IRC21:08
*** igordc has joined #openstack-ironic21:08
*** igordc has quit IRC21:10
*** TxGirlGeek has joined #openstack-ironic21:11
*** igordc has joined #openstack-ironic21:12
*** igordc has quit IRC21:12
*** ijw has joined #openstack-ironic21:20
*** Lucas_Gray has joined #openstack-ironic21:22
*** k_mouza has joined #openstack-ironic21:32
*** k_mouza has quit IRC21:41
*** Lucas_Gray has quit IRC21:54
*** rcernin has joined #openstack-ironic21:56
*** bfournie has quit IRC22:31
openstackgerritIlya Etingof proposed openstack/ironic master: Improve `redfish` set-boot-device behaviour  https://review.opendev.org/71111822:35
*** tkajinam has joined #openstack-ironic22:48
*** kozhukalov has joined #openstack-ironic22:51
*** k_mouza has joined #openstack-ironic22:57
*** k_mouza has quit IRC22:58
*** k_mouza has joined #openstack-ironic22:58
*** bfournie has joined #openstack-ironic23:24
*** k_mouza has quit IRC23:57
« Monday, 2020-03-02 Index Wednesday, 2020-03-04 »

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