Monday, 2020-04-27

*** ociuhandu has quit IRC00:07
*** ociuhandu has joined #openstack-ironic00:13
*** ociuhandu has quit IRC00:18
*** kaifeng has joined #openstack-ironic00:45
*** Goneri has quit IRC00:59
*** Qianbiao has joined #openstack-ironic01:45
*** ociuhandu has joined #openstack-ironic02:40
*** ociuhandu has quit IRC02:50
*** threestrands has joined #openstack-ironic02:51
*** ociuhandu has joined #openstack-ironic02:55
*** k_mouza has joined #openstack-ironic02:57
*** ociuhandu has quit IRC03:00
*** k_mouza has quit IRC03:01
*** mkrai has joined #openstack-ironic03:19
*** ociuhandu has joined #openstack-ironic03:33
*** ociuhandu has quit IRC03:38
*** jdandrea has quit IRC03:38
*** ociuhandu has joined #openstack-ironic04:27
*** ociuhandu has quit IRC04:34
*** mkrai has quit IRC04:59
*** mkrai_ has joined #openstack-ironic04:59
*** yaawang_ has quit IRC05:13
*** yaawang_ has joined #openstack-ironic05:14
*** Qianbiao has quit IRC05:38
*** ociuhandu has joined #openstack-ironic06:02
*** ociuhandu has quit IRC06:06
*** Qianbiao has joined #openstack-ironic06:13
*** yolanda has joined #openstack-ironic06:16
iurygregorygood mornign Ironic o/06:23
arne_wiebalckGood morning iurygregory and ironic!06:36
Qianbiaogood morning :)06:40
*** lennyb has quit IRC06:54
*** ociuhandu has joined #openstack-ironic06:55
*** iurygregory has quit IRC07:09
*** iurygregory has joined #openstack-ironic07:10
*** mkrai_ has quit IRC07:11
iurygregoryarne_wiebalck, hey o/ sory was having my coffee =)07:11
*** ociuhandu has quit IRC07:13
*** ociuhandu has joined #openstack-ironic07:13
*** rpittau|afk is now known as rpittau07:22
rpittaugood morning ironic! o/07:22
rpittaummm looks like huawei ci is up again07:25
Qianbiaorpittau yes, my friend.07:26
*** ociuhandu has quit IRC07:26
rpittauhey Qianbiao, glad to see that :)07:26
QianbiaoI setup it last weekend.07:27
QianbiaoYeah. it's down because openstack.com -> opendev.07:27
Qianbiaoand opendev is block by china..07:28
*** ociuhandu has joined #openstack-ironic07:29
QianbiaoHello, anyone familiar with metal3?07:30
Qianbiaois there any docker image that could setup metal3 automate.07:30
*** sshnaidm|afk is now known as sshnaidm07:35
rpittauQianbiao: I'm not aware of such thing, although you can find comprehensive automation scripts in metal3-dev-env07:35
Qianbiaorpittau thanks will look at it.07:37
openstackgerritQianBiao Ng proposed openstack/ironic stable/train: Implements: Reactive HUAWEI ibmc driver  https://review.opendev.org/72333107:38
*** mkrai_ has joined #openstack-ironic07:39
iurygregorymorning rpittau o/07:39
rpittauhey iurygregory :)07:39
openstackgerritAija Jaunteva proposed openstack/ironic stable/train: DRAC: Added redfish management interface issue  https://review.opendev.org/72333307:47
*** yaawang_ has quit IRC07:49
*** yaawang_ has joined #openstack-ironic07:50
*** lucasagomes has joined #openstack-ironic07:54
*** yaawang_ has quit IRC07:54
*** yaawang_ has joined #openstack-ironic07:55
*** yaawang_ has quit IRC08:02
*** yaawang_ has joined #openstack-ironic08:03
rpittauQianbiao: about reinstating the ibmc drivers, I'm not sure we'll be able to add those back in ussuri, and probably not in stable branches. My suggestion is to bring the argument at the meeting and/or send open a thread in the mailing list.08:07
Qianbiaorpittau ok, when is the meeting open. how can i attend.08:09
rpittauthe meeting is on mondays, so we have one today, at 1500 UTC08:10
rpittauQianbiao: it's on this channel, more info here: https://wiki.openstack.org/wiki/Meetings/Ironic08:10
Qianbiaook08:10
Qianbiaorpittau it's about seven hours later right?08:12
rpittauQianbiao: yeah, it's in 7 hours08:12
rpittauprobably not the best time for you :/08:12
*** rcernin has quit IRC08:13
Qianbiao:) need more coffee.08:14
Qianbiao<rpittau> in this IRC channel directly?08:17
rpittauQianbiao: correct08:17
Qianbiaolol. i try to connect irc.freenode.net08:17
Qianbiaofind it seems the same chat.freenode.net08:17
*** threestrands has quit IRC08:20
*** tkajinam has quit IRC08:23
*** mkrai_ has quit IRC08:24
*** mkrai has joined #openstack-ironic08:25
*** mcarden has quit IRC08:29
*** logan_ has joined #openstack-ironic08:31
*** logan- has quit IRC08:32
*** alexmcleod has joined #openstack-ironic08:34
*** logan_ is now known as logan-08:35
*** derekh has joined #openstack-ironic08:38
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent master: Add function to calculate memory  https://review.opendev.org/72233408:46
iurygregoryI'm wondering here if there is a reason for the function _parse_deploy_info be a classmethod in redfish but not in irmc or ilo does anyone know ?08:47
*** k_mouza has joined #openstack-ironic08:55
*** k_mouza has quit IRC08:59
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent master: Fix TypeError with newer version of lshw  https://review.opendev.org/72233509:01
*** k_mouza has joined #openstack-ironic09:02
rpittauiurygregory: it really looks just like an implementation choice09:07
iurygregorygotcha09:07
rpittauin the redfish boot interface all methods are class methods, while in ilo some of them are independent, probably because ILO has multiple boot interfaces and therefore the need of having some shared methods09:08
iurygregorythanks rpittau =)09:15
openstackgerritMerged openstack/ironic-inspector master: Native zuulv3 grenade job for inspector  https://review.opendev.org/70399509:17
iurygregory\o/09:22
openstackgerritVerification of a change to openstack/ironic failed: In-band deploy steps: correctly wipe driver_internal_info  https://review.opendev.org/72291609:48
*** dtantsur|afk is now known as dtantsur09:49
dtantsurmorning ironic, happy Monday09:50
* dtantsur has finally replenished his stock of buckwheat \o/09:51
iurygregorymorning dtantsur o/09:51
dtantsurmgoddard: thank you for taking care of the deploy steps patches09:52
dtantsurpity we could not get it to show-caseable state in Ussuri.. but still great progress!09:52
mgoddarddtantsur: just a bit of rechecking :)09:52
mgoddarddtantsur: progress is good, thanks for pushing on it. CentOS 8 ate all my time :(09:53
dtantsurI can imagine. We had quite some fun with RHEL 8 as well, but it was in Stein timeframe downstream09:54
mgoddarddtantsur: best not to rush the split through anyway. Hopefully we can land early in V and give it some good testing09:54
dtantsurexactly09:55
dtantsurmaybe figure out how to simulate a 3rd party driver in the CI..09:55
dtantsurmgoddard: we should start preparing releases of the remaining projects today-tomorrow. will you have time for that?09:55
mgoddarddtantsur: I was thinking about testing. Would it be crazy to expose a run_command deploy step?09:56
mgoddardwe'd need to work out running steps more than once09:56
dtantsurmmmmmm09:56
*** yolanda has quit IRC09:56
mgoddardbut it could be used to e.g. echo foo > file at various points09:56
dtantsurI think you can request a step more than once via deploy templates09:56
mgoddardwe disabled it AFAIR, there were some subtle issues09:57
dtantsurbut yeah, it's an interesting idea. and will simplify plugging in custom hw manager.09:57
mgoddardcould be enabled with care09:57
dtantsuranother TODO item for V :)09:57
mgoddardyeah09:57
mgoddarddtantsur: on releases, yes I can make some time today09:58
dtantsurgreat! it seems that some patches are still going through the gate, but inspector may be ready09:59
mgoddarddtantsur: too late to get the deploy steps follow up in?09:59
dtantsurmgoddard: which one?09:59
dtantsur722916 must merge before we release IMO10:00
dtantsur(I'm not implying we must release right now, but rather start the countdown and track the remaining patches)10:00
mgoddardok, makes sense10:00
*** mkrai has quit IRC10:06
*** mkrai_ has joined #openstack-ironic10:07
rpittauso many rechecks :/10:08
*** yolanda has joined #openstack-ironic10:10
rpittauwe should also keep an eye on https://review.opendev.org/#/c/722027/ hopefully will merge on this run10:10
patchbotpatch 722027 - ironic - Fix agent_client handling of embedded errors - 3 patch sets10:10
*** mkrai_ has quit IRC10:10
*** mkrai has joined #openstack-ironic10:10
openstackgerritDerek Higgins proposed openstack/ironic master: WIP ipv6 jobs  https://review.opendev.org/61316910:20
*** rpittau is now known as rpittau|bbl10:32
openstackgerritVerification of a change to openstack/networking-baremetal failed: Convert networking-baremetal job to dib  https://review.opendev.org/71869710:34
*** ociuhandu has quit IRC10:42
*** ociuhandu has joined #openstack-ironic10:44
*** Qianbiao is now known as qianbiao|afk10:45
openstackgerritMerged openstack/ironic stable/rocky: Fix issue where server fails to reboot  https://review.opendev.org/72180311:00
*** ociuhandu has quit IRC11:03
*** ociuhandu has joined #openstack-ironic11:04
*** qianbiao|afk has quit IRC11:16
*** ociuhandu has quit IRC11:23
openstackgerritMerged openstack/ironic stable/stein: Make deploy step failure logging indicate the error  https://review.opendev.org/71768511:27
openstackgerritMerged openstack/ironic stable/stein: Fix issue where server fails to reboot  https://review.opendev.org/72179711:27
openstackgerritMerged openstack/ironic master: Upgrade flake8-import-order version to 0.17.1  https://review.opendev.org/72164611:27
*** Mahendra22 has joined #openstack-ironic11:37
mgoddardiurygregory, dtantsur, TheJulia: is ironic-tempest-plugin ready to release?11:43
mgoddardI saw TheJulia -1'd the patch11:43
mgoddard3 patches with a +2: https://review.opendev.org/#/q/project:openstack/ironic-tempest-plugin+status:open+branch:master11:43
iurygregorymgoddard, I don't think we do releases for ironic-tempest-plugin11:44
iurygregoryO.o11:44
iurygregorymaybe we create a tag only11:44
*** yolanda has quit IRC11:44
mgoddardiurygregory: it's released but not branched11:44
*** yolanda has joined #openstack-ironic11:44
iurygregorymgoddard, oh ok =) I thought by release you only mean create a stable branch =)11:45
iurygregorymgoddard, since we split standalone tests it's good to have the job voting I would say11:45
mgoddardiurygregory: you mean https://review.opendev.org/#/c/720675/2 ? Looks like it's non-voting11:46
patchbotpatch 720675 - ironic-tempest-plugin - Add standalone redfish jobs - 2 patch sets11:46
iurygregorymgoddard, yup11:46
iurygregoryand https://review.opendev.org/71204211:47
patchbotpatch 712042 - ironic - Add ironic-standalone-redfish - 10 patch sets11:47
*** qianbiao|afk has joined #openstack-ironic11:48
mgoddardiurygregory: doesn't look that that should block release though?11:49
iurygregorymgoddard, didn't she put the reason for the -1?11:49
mgoddardnope11:50
iurygregorymaybe we are waitting for something in other project11:50
iurygregorythat's all I can think atm11:50
mgoddardI'll wait for TheJulia11:50
iurygregoryyeah =)11:50
iurygregorysorry I couldn't help that much11:51
mgoddardLooks like inspector is good to go11:51
*** ociuhandu has joined #openstack-ironic11:55
mgoddardiurygregory: ironic-prometheus-exporter ready to release?11:59
iurygregorymgoddard, I would say we can merge the 3 remaining patches12:00
iurygregory2 are missing +A12:00
iurygregoryhttps://review.opendev.org/71768912:01
patchbotpatch 717689 - ironic-prometheus-exporter - Use hacking for pep8 tests - 5 patch sets12:01
iurygregoryhttps://review.opendev.org/71768412:01
patchbotpatch 717684 - ironic-prometheus-exporter - Add lower constraints jobs - 5 patch sets12:01
mgoddardiurygregory: aren't those really related to CI?12:01
mgoddardiurygregory: we're already past the official deadline12:01
iurygregoryone we are adding l-c12:01
mgoddardok12:01
iurygregoryI think would be good to have l-c12:01
mgoddardapproving :)12:02
iurygregoryCI is fast =)12:02
iurygregoryso we will we see Merged very soon12:02
mgoddardcool12:03
*** rh-jelabarre has joined #openstack-ironic12:05
*** ociuhandu has quit IRC12:06
*** ociuhandu has joined #openstack-ironic12:12
openstackgerritVerification of a change to openstack/ironic-inspector failed: Convert jobs to dib  https://review.opendev.org/71251612:23
iurygregory=(12:24
*** ociuhandu has quit IRC12:24
*** ociuhandu has joined #openstack-ironic12:24
*** ociuhandu has quit IRC12:35
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Switch to unittest mock  https://review.opendev.org/71797912:41
*** ociuhandu has joined #openstack-ironic12:41
*** hjensas is now known as hjensas|afk12:43
*** qianbiao|afk has quit IRC12:47
*** rpittau|bbl is now known as rpittau12:49
*** michchap has joined #openstack-ironic12:52
*** mkrai has quit IRC12:52
*** ociuhandu has quit IRC12:53
*** Goneri has joined #openstack-ironic12:54
dtantsurmgoddard: I would say all mandatory stuff has merged to tempest-plugin, but worth checking with Julia12:55
*** ociuhandu has joined #openstack-ironic13:00
dtantsurmgoddard: left a question on https://review.opendev.org/#/c/723399/113:00
patchbotpatch 723399 - releases - Release Ironic Python Agent 7.0.0 for Ussuri, bran... - 1 patch set13:00
*** Goneri has quit IRC13:01
TheJuliao/13:03
TheJuliaI feel like I've been missing, but yeah. :(13:04
dtantsurmorning TheJulia, how's your leg?13:04
TheJulia a little tingling, seems it was my old back injury for like 18 years ago acting up13:04
dtantsursounds like an improvement nonetheless!13:05
TheJuliayeah, I know how to handle my back being unhappy, my leg presenting with the way it did was just.. different13:05
dtantsurback on the release topic, we should consider including https://review.opendev.org/#/c/723225/13:06
patchbotpatch 723225 - ironic - Implements: Reactive HUAWEI ibmc driver - 2 patch sets13:06
*** qianbiao|afk has joined #openstack-ironic13:07
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent master: Fix TypeError with newer version of lshw  https://review.opendev.org/72233513:08
*** qianbiao|afk is now known as Qianbiao13:08
openstackgerritNoor Muhammad Malik proposed openstack/ironic master: Add PXE reset known issue to the docs  https://review.opendev.org/72255513:09
TheJuliaI don't see a vote for it, which makes me moderately hesitant. Qianbiao is here, which is awesome though. If this was the beginning of the cycle then I would say lets merge it, but being the end of the cycle and we need to release, I'm more hesitant. :(13:09
rpittauI'm not sure that will pass the cI tio be honest13:09
rpittaus/tio/to13:09
dtantsurI've seen the CI voting on patches, I need to double-check what it actually does (seems like bifrost)13:09
dtantsurQianbiao: two non-blocking requests re the iBMC CI (looking at http://116.66.187.11/builds/Ironic-Unsuri-CI/34/ironic-conductor.log.txt):13:10
dtantsur1) please enable debug logging for services13:11
dtantsur2) please change ibmc_client.connector logs to DEBUG13:11
Qianbiaodtantsur ok13:12
dtantsurTheJulia, rpittau, http://116.66.187.11/builds/Ironic-Unsuri-CI/34/ironic-conductor.log.txt has indication of passing the deployment and of the ibmc hardware type being used. I don't see reasons not to trust it.13:13
iurygregorynewbie question, we can only see ironic-standalone-redfish-train / stein if the stable branch has ironic-standalone-redfish job?  asking because when I added in https://review.opendev.org/#/c/720675/ i don't see the result O.o13:13
patchbotpatch 720675 - ironic-tempest-plugin - Add standalone redfish jobs - 2 patch sets13:13
TheJuliadtantsur: where did you see the job log?13:13
dtantsurTheJulia: plenty of runs on https://review.opendev.org/#/c/723225/ did you "Toggle CI"?13:14
patchbotpatch 723225 - ironic - Implements: Reactive HUAWEI ibmc driver - 2 patch sets13:14
iurygregoryyeah they start reporting in patches13:14
TheJuliaoh, there is now a "Toggle Extra CI" :(13:14
TheJuliaokay, I see it13:14
TheJuliaYeah, Qianbiao if you can revise the release note, I'm for merging prior to releasing13:15
QianbiaoTheJulia what should i update for the release note.13:17
TheJuliaQianbiao: dtantsur provided feedback in the review13:17
TheJuliaotherwise, we need to merge then re-edit and wait for that change to land13:17
Qianbiaochechking now.13:17
*** Goneri has joined #openstack-ironic13:18
Qianbiaodtantsur  Is this good:   Now that HUAWEI ironic 3rd party CI is back, the ``ibmc`` hardware type13:22
Qianbiao    driver is supported. We can reactive the ibmc driver.13:22
QianbiaoI am not a english speaker.. sorry fot that.13:22
dtantsurdon't worry, many of us are not13:22
TheJuliaQianbiao: that is perfectly okay :)13:22
dtantsur(I'm not either)13:22
dtantsurI'd replace "We can reactive" with "... has been activated again."13:22
dtantsurthe passive voice sounds a bit more official to me :)13:23
Qianbiao:) ok13:23
TheJuliaQianbiao: I was thinking the exact same thing as dtantsur13:23
*** dsneddon has quit IRC13:23
TheJuliaand I'm sadly a native english speaker/writer.13:23
TheJuliasilly languages13:23
QianbiaoIt's ok, i think it has express clear.13:23
openstackgerritRiccardo Pittau proposed openstack/virtualbmc master: Restore pep8 checking on import order  https://review.opendev.org/72116213:24
openstackgerritRiccardo Pittau proposed openstack/virtualbmc master: Set flake8-import-order version to 0.17.1  https://review.opendev.org/72165713:24
QianbiaoIs there any skill that could speed up `git review`?13:25
Qianbiaoit's very very slow for me.13:25
TheJuliamgoddard: replied to you on the ipa release changeset13:26
mgoddardTheJulia: thanks13:26
TheJuliagit review is typically fairly quick for me. I wonder what it is waiting on13:26
iurygregorygit review takes 30sec for me =(13:27
*** jdandrea has joined #openstack-ironic13:27
TheJuliahmm13:27
TheJuliaTypically about 15 for me13:27
*** rloo has joined #openstack-ironic13:27
QianbiaoTheJulia iurygregory sometimes up to 30 mins for me ...13:28
QianbiaoIt's chinese firewall again.. i think13:28
iurygregoryWOW13:29
* rpittau will stop complaining for general gerrit review slowness13:29
Qianbiaorpittau XD13:29
TheJuliaQianbiao: wow...13:29
TheJuliaQianbiao: are you using https or the gerrit native access?13:30
Qianbiaohttps13:30
openstackgerritQianBiao Ng proposed openstack/ironic master: Implements: Reactive HUAWEI ibmc driver  https://review.opendev.org/72322513:31
Qianbiaofinally...13:31
Qianbiaothe better news is .. i try to do things right to reduce `git review` times :)13:31
Qianbiao* the good news13:32
Qianbiaodtantsur i have set debug to true13:33
Qianbiaoibmcclient does not have much log, the API is pretty simple. I did not log many debug info for now.13:34
mgoddardTheJulia: are we good to go on ironic-tempest-plugin release?13:34
mgoddardhttps://review.opendev.org/#/c/721486/113:34
patchbotpatch 721486 - releases - Release ironic-tempest-plugin for ussuri - 1 patch set13:34
TheJuliamgoddard: thanks for the link13:35
dtantsurQianbiao: re ibmcclient logging: you're logging HTTP requests at INFO level. This ends up pretty verbose for operators, so it's recommended that HTTP requests logging is done as DEBUG.13:35
Qianbiaodtantsur ok, will fix it in next version.(with raid support feature)13:36
dtantsurthanks! it's not urgent, take your time.13:36
TheJuliamgoddard: changed the release tag value and +1'ed13:38
* TheJulia suspects that she needs to order a standing desk today13:39
mgoddardTheJulia: thanks13:39
*** tkajinam has joined #openstack-ironic13:42
openstackgerritJulia Kreger proposed openstack/ironic master: Implements: Reactive HUAWEI ibmc driver  https://review.opendev.org/72322513:42
TheJulialooks like we're waiting on ironic and the prometheus exporter, maybe13:44
dtantsursushy-tools and virtualbmc as well13:45
dtantsureven though they're not series-bound, it's good to have a reference version13:45
TheJuliayeah13:45
QianbiaoTheJulia dtantsur thanks for the review. May i cherry-pick the patch to stein?13:45
*** hjensas|afk is now known as hjensas13:46
* dtantsur thinks13:46
Qianbiao* to train13:46
*** tzumainn has joined #openstack-ironic13:46
* TheJulia also thinks13:46
TheJuliaI don't see why not. There is precedence for that being signaled on a stable branch13:47
dtantsurQianbiao: if your CI works on Train as well (I haven't checked), I see no problems with that.13:47
TheJulia+1 to dtantsur's statement13:47
Qianbiaook. it should works.13:47
openstackgerritQianBiao Ng proposed openstack/ironic stable/stein: Implements: Reactive HUAWEI ibmc driver  https://review.opendev.org/72353713:47
dtantsurQianbiao: this ^^^ doesn't have the removal of "supported = False" from the code, why is that?13:48
dtantsur(also it's stein, not train)13:48
Qianbiaoyes i want to cherry pick to train.13:49
Qianbiaohmm not sure..13:49
dtantsurQianbiao: if "supported = False" was not in Stein, you don't need to backport13:49
Qianbiaoyes, i know.13:49
QianbiaoI think i must have wrong operations.13:50
Qianbiaooh.. i didnot check ibmc.py..13:50
TheJuliaare we waiting on an actual patch to merge in sushy-tools and virtualbmc, or are we just thinking we should release them13:51
dtantsurokay, I think I get it13:51
dtantsurQianbiao: the ibmc hardware type was only deprecated in Ussuri in 14.0. You don't need any backports.13:51
dtantsurit was supported in Train and Stein officially13:51
iurygregoryonly sushy-tools-tempest-ironic-partition-redfish-src let us merge something >.<13:52
QianbiaoIs it so? i thought it's deprecated in train, will check now.13:52
dtantsurQianbiao: please double-check me, but it seems to be ussuri only13:52
iurygregorydon't look at  only look at https://zuul.openstack.org/builds?job_name=sushy-tools-tempest-ironic-partition-redfish-src13:53
Qianbiao<dtantsur> you are right.13:53
iurygregoryrpittau, you found anything about it? I think you were looking if I do remember13:53
* TheJulia wonders if there is a good reason to meet today13:54
dtantsurrelease readiness?13:55
* dtantsur -> tea, brb13:55
*** cdearborn has joined #openstack-ironic13:55
TheJuliayeah13:56
*** mkrai has joined #openstack-ironic13:57
TheJuliaso not quite our normal agenda then :)13:57
TheJuliao/ mkrai13:57
rpittauiurygregory: that's the issue with the non-existing mount point, it started failing when we changed teh default boot option to local13:58
iurygregoryoh we were using default_boot_option = netboot13:59
rpittauyep13:59
iurygregorywe missed this job >.<13:59
rpittauwell not only that one13:59
iurygregorydo you have a patch up?13:59
rpittauI was checking the jobs and preparing the patches13:59
rpittauyeah13:59
TheJulia:(13:59
rpittaulet me put that up now14:00
iurygregorylink pls14:00
rpittauI'll send the rest later14:00
rpittauhold on14:00
TheJuliaeasy to miss them sadly with how long that change had been up :(14:00
iurygregoryyeah14:00
openstackgerritRiccardo Pittau proposed openstack/sushy-tools master: Restore default netboot boot option  https://review.opendev.org/72354614:01
rpittau^^ there you go14:01
iurygregorytks!14:02
rpittauthe rest incoming14:04
openstackgerritRiccardo Pittau proposed openstack/python-ironicclient master: Restore default netboot boot option  https://review.opendev.org/72354714:04
*** mkrai has quit IRC14:05
*** mkrai_ has joined #openstack-ironic14:05
*** irclogbot_0 has joined #openstack-ironic14:08
iurygregorytime for the 3rd coffee brb14:08
openstackgerritRiccardo Pittau proposed openstack/sushy master: Restore default netboot boot option  https://review.opendev.org/72355414:09
TheJuliaiurygregory: working on making your atoms vibrate?14:10
openstackgerritRiccardo Pittau proposed openstack/virtualbmc master: Restore default netboot boot option  https://review.opendev.org/72355514:10
rpittauhttps://media.giphy.com/media/zWwngnPi6vWfe/giphy.gif14:12
*** irclogbot_0 has quit IRC14:12
TheJuliaarne_wiebalck: so re: efi, the capability is there, but efi firmware is supposed to store changes/data in nvram, not on disks, so the raid sets should basically be untouched14:13
TheJuliaheh14:14
iurygregoryTheJulia, hehehehe yeah14:14
*** Mahendra22 has quit IRC14:15
openstackgerritRiccardo Pittau proposed openstack/python-ironic-inspector-client master: Restore default netboot boot option  https://review.opendev.org/72355914:15
arne_wiebalckTheJulia: thanks. I think the use case is a runtime cache for UEFI variables ... but again, all I know so far is from blog posts :)14:17
iurygregoryin the past I had about 5 mugs during the day (2 mugs during dinner), but my gf said it's wrong and she complains a lot if I drink coffee in the dinner, so I'm trying to compensate during working hours XD14:18
*** k_mouza has quit IRC14:18
openstackgerritDerek Higgins proposed openstack/ironic master: WIP ipv6 jobs  https://review.opendev.org/61316914:18
iurygregoryand if I don't drink coffee I can easily have headache =(14:18
TheJuliaarne_wiebalck: that seems.. odd. UEFI variables should be runtime accessible14:21
TheJuliacaching to disk seems.. super odd14:21
TheJuliaAt least for linux. I think OSX does this14:21
dtantsuriurygregory: wow, that's serious14:22
dtantsurI only drink tea nowadays14:22
iurygregorydtantsur, yeah >.<14:22
*** irclogbot_1 has joined #openstack-ironic14:22
dtantsurheadache without coffee may signify a low blood pressure. not sure which conclusions to make out of it14:22
iurygregoryresult of 12hrs/day work to finish my master's =(14:22
* dtantsur has headache without enough carbohydrates14:23
dtantsuriurygregory: I hear you. Many of us lost some health there..14:23
arne_wiebalckTheJulia: what does runtime accessible mean here? The disk is not runtime accessible?14:23
iurygregorydtantsur, yeah =(14:23
iurygregorynow time to play with grenade multinode \o/14:24
dtantsurthat's a sophisticated toy to play with14:25
*** irclogbot_1 has quit IRC14:25
openstackgerritOpenStack Release Bot proposed openstack/ironic-python-agent stable/ussuri: Update .gitreview for stable/ussuri  https://review.opendev.org/72356214:25
openstackgerritOpenStack Release Bot proposed openstack/ironic-python-agent stable/ussuri: Update TOX_CONSTRAINTS_FILE for stable/ussuri  https://review.opendev.org/72356314:25
openstackgerritOpenStack Release Bot proposed openstack/ironic-python-agent master: Update master for stable/ussuri  https://review.opendev.org/72356414:25
openstackgerritOpenStack Release Bot proposed openstack/ironic-python-agent master: Add Python3 victoria unit tests  https://review.opendev.org/72356514:25
*** irclogbot_1 has joined #openstack-ironic14:26
TheJuliaarne_wiebalck: the OS should be able to get to get the variables. Any chance you have a link or two to blog posts?14:27
*** k_mouza has joined #openstack-ironic14:27
arne_wiebalckTheJulia: https://outflux.net/blog/archives/2018/04/19/uefi-booting-and-raid1/14:27
*** irclogbot_1 has quit IRC14:29
*** irclogbot_1 has joined #openstack-ironic14:30
openstackgerritMerged openstack/ironic master: In-band deploy steps: correctly wipe driver_internal_info  https://review.opendev.org/72291614:32
openstackgerritMerged openstack/ironic-prometheus-exporter master: Add lower constraints jobs  https://review.opendev.org/71768414:32
dtantsurgetting closer!14:33
TheJuliaarne_wiebalck: ouch :(14:34
openstackgerritOpenStack Release Bot proposed openstack/ironic-inspector stable/ussuri: Update .gitreview for stable/ussuri  https://review.opendev.org/72356814:34
openstackgerritOpenStack Release Bot proposed openstack/ironic-inspector stable/ussuri: Update TOX_CONSTRAINTS_FILE for stable/ussuri  https://review.opendev.org/72356914:34
openstackgerritOpenStack Release Bot proposed openstack/ironic-inspector master: Update master for stable/ussuri  https://review.opendev.org/72357014:34
openstackgerritOpenStack Release Bot proposed openstack/ironic-inspector master: Add Python3 victoria unit tests  https://review.opendev.org/72357114:34
*** irclogbot_1 has quit IRC14:35
openstackgerritMerged openstack/python-ironic-inspector-client master: Document the new standalone CLI  https://review.opendev.org/72003414:36
*** irclogbot_2 has joined #openstack-ironic14:36
*** irclogbot_2 has quit IRC14:39
*** irclogbot_3 has joined #openstack-ironic14:40
*** stendulker has joined #openstack-ironic14:44
openstackgerritRiccardo Pittau proposed openstack/ironic-prometheus-exporter master: Use hacking for pep8 tests  https://review.opendev.org/71768914:44
*** irclogbot_3 has quit IRC14:45
*** irclogbot_3 has joined #openstack-ironic14:46
openstackgerritRiccardo Pittau proposed openstack/ironic-prometheus-exporter master: Use hacking for pep8 tests  https://review.opendev.org/71768914:47
*** iurygregory has quit IRC14:47
*** iurygregory has joined #openstack-ironic14:48
*** kaifeng_ has joined #openstack-ironic14:49
*** beekneemech is now known as bnemec14:50
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-tempest-plugin master: Add standalone redfish jobs  https://review.opendev.org/72067514:51
*** irclogbot_3 has quit IRC14:51
*** irclogbot_2 has joined #openstack-ironic14:52
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client stable/ussuri: Document the new standalone CLI  https://review.opendev.org/72357514:53
*** tkajinam has quit IRC14:54
*** irclogbot_2 has quit IRC14:55
*** irclogbot_2 has joined #openstack-ironic14:56
*** irclogbot_2 has quit IRC14:59
dtantsurouch, we should probably finished and merge https://review.opendev.org/#/c/72078114:59
patchbotpatch 720781 - ironic - Don't break UEFI install with older IPAs - 1 patch set14:59
dtantsurarne_wiebalck: and updated on ^^?14:59
*** irclogbot_3 has joined #openstack-ironic15:00
iurygregoryo/15:00
TheJulia#startmeeting ironic15:00
TheJuliao/15:00
openstackMeeting started Mon Apr 27 15:00:39 2020 UTC and is due to finish in 60 minutes.  The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
dtantsuro/15:00
*** openstack changes topic to " (Meeting topic: ironic)"15:00
openstackThe meeting name has been set to 'ironic'15:00
cdearborno/15:00
erbarro/15:00
arne_wiebalckdtantsur: yes, we should merge this one and fix the other issue15:00
rpittauo/15:00
kaifeng_o/15:00
* iurygregory was too fast15:00
ajya\o15:00
iurygregoryo/15:00
arne_wiebalcko/15:00
rpiosoo/15:01
arne_wiebalckdtantsur: I think TheJulia was on it as well15:01
rlooo/15:01
stendulkero/15:01
TheJuliadtantsur: I can always revise arne_wiebalck's patch after the meeting15:01
TheJuliaGood morning everyone!15:01
TheJuliaOur agenda can be found on the wiki, as always.15:02
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting15:02
TheJuliaHopefully this will go quickly :)15:02
mgoddard\o15:02
TheJuliaSince I suspect we should skip status reports this week given it is basically release week15:02
TheJuliawell, last week was, but yeah. :)15:02
TheJulia#topic Announcements / Reminders15:03
*** openstack changes topic to "Announcements / Reminders (Meeting topic: ironic)"15:03
*** m1p has left #openstack-ironic15:03
*** irclogbot_3 has quit IRC15:03
TheJulia#info Final Ussuri release is due May 13th, we should expect devstack to possibly break us at any moment.15:03
TheJuliaIt seems like we're on track to be done maybe today or tomorrow?15:04
*** irclogbot_3 has joined #openstack-ironic15:04
TheJuliaSeems we also need to register our PTG attendance response, which I believe I already took care of.15:04
TheJuliaDoes anyone else have anything to announce or remind us of?15:05
iurygregoryTheJulia, do we have the slots defined?15:05
iurygregoryor this can be done after 28?15:05
TheJuliaiurygregory: we do not. I'm trying to remember if we got an etherpad going15:05
TheJuliaThat can be done after the 28th as I understand it15:06
dtantsur#link https://etherpad.opendev.org/p/Ironic-VictoriaPTG-Planning15:06
dtantsurthis ^^?15:06
iurygregoryhttp://lists.openstack.org/pipermail/openstack-discuss/2020-April/014423.html15:06
iurygregorydtantsur, yup15:06
*** k_mouza has quit IRC15:06
TheJuliaThat works!15:06
*** irclogbot_3 has quit IRC15:07
dtantsuroh, one more thing15:07
dtantsurIf you remember my flame war email about making ironic more independent, one of the follow-ups was to understand what exactly we need in terms of release cadence, etc15:08
*** irclogbot_0 has joined #openstack-ironic15:08
dtantsurI'd like to set up a call, ideally this week, to gather ideas and try to come up with a proposal15:08
iurygregory++ sounds good15:08
TheJulia++15:08
dtantsur#link https://doodle.com/poll/gtxzdhatabx8agy6 The doodle for release cadence discussion15:09
TheJuliadtantsur: thanks!15:09
dtantsurA small word of caution: this is not mandatory to attend. If you don't care much, you're welcome to skip it15:09
TheJulia++15:09
TheJuliaLooks like we didn't have any action items from our last meeting, except maybe SPUC II :)15:10
iurygregoryit's done ^15:10
iurygregoryhttp://lists.openstack.org/pipermail/openstack-discuss/2020-April/014410.html15:10
TheJuliaiurygregory: Is there an agreed upon date/time??15:10
TheJulia#link http://lists.openstack.org/pipermail/openstack-discuss/2020-April/014410.html15:10
iurygregoryTheJulia, so far only 5 people voted15:11
*** irclogbot_0 has quit IRC15:11
TheJuliaIndeed :(15:11
TheJulia#link https://doodle.com/poll/2q5zmv3g6uy2475e15:11
iurygregorywould be May 8 so far =) since we have holiday this week...15:11
dtantsuryeah, May 1st is a holiday for a lot of people15:12
TheJuliaAnyway, since we are basically at the release window, I guess we should skip ahead to priorities for the week?15:12
*** irclogbot_2 has joined #openstack-ironic15:12
dtantsur++15:12
TheJulia#topic Deciding on priorities for the coming week15:12
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)"15:12
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:13
*** k_mouza has joined #openstack-ironic15:13
TheJuliaStarting at line 13115:13
iurygregorycan we add  https://review.opendev.org/#/c/703098/ ? =)15:14
patchbotpatch 703098 - ironic - Native zuulv3 grenade job for ironic - 15 patch sets15:14
dtantsurCI changes are backportable15:14
dtantsurJFYI15:14
iurygregorygotcha15:15
TheJuliaI'd prefer to hold off until after we cut the branch and then just backport them15:15
*** irclogbot_2 has quit IRC15:15
iurygregorymakes sense to me15:15
TheJuliaanyone have the links handy for the stuff we're waiting on to merge?15:16
*** irclogbot_0 has joined #openstack-ironic15:16
iurygregorynope =(15:17
dtantsurI'm only aware of the patches that has already been added15:18
TheJuliaOkay, then we're in really good shape, ci permitting15:18
iurygregoryyup15:18
* TheJulia considers making an awful joke, but decides that it is a bad idea15:19
* dtantsur considers which sacrifice to make to the CI gods15:19
rpittauall the ci fixes are in the gate, I *think*15:19
TheJuliadtantsur: rubber chickens I believe15:19
*** irclogbot_0 has quit IRC15:19
dtantsur++15:19
TheJuliaDoes anyone have things for after the stable branch is cut?15:19
rpittauI'll recheck a bunch of stuff as soon as those are merged15:19
dtantsuras to awful jokes, that's what we have twitter for, no?15:19
TheJuliadtantsur: true15:20
TheJuliarpioso: Excellent! Thanks!15:20
*** irclogbot_3 has joined #openstack-ironic15:20
dtantsurhttps://review.opendev.org/703778 is for post-branch mering15:20
patchbotpatch 703778 - ironic - Add RPC objects for deployment API - 10 patch sets15:20
iurygregoryif we have patches that have changes in l-c they are ok to backport ?15:20
TheJulial-c?15:21
iurygregorylower-constraints15:21
iurygregory=)15:21
dtantsuriurygregory: l-c alone or also requirements?15:21
dtantsurlower-constraints.txt is not, to my best knowledge, a production artefact of ours15:21
dtantsurchanging requirements.txt is a whole different story15:21
iurygregorywell some required changes in requirements15:21
iurygregorye.g https://review.opendev.org/#/c/720790/415:22
patchbotpatch 720790 - ironic - Stop configuring install_command in tox. - 4 patch sets15:22
dtantsurchanging in requirements are only acceptable when we know for sure that the current lower bounds don't work at all15:22
dtantsur* changes15:22
dtantsur** on stable branches15:22
iurygregoryI only changed because the job said it was wrong XD15:22
*** irclogbot_3 has quit IRC15:23
TheJuliaSo, I suspect we should merge that prior to releasing ironic. We've already released basically everything else except sushy-tools/virtualbmc afaik15:23
dtantsurI've approved teh change15:23
TheJuliaso if the gods of CI are friendly, ~4 hours15:23
* TheJulia can live with that15:24
dtantsurIt would be a contentious change to backport15:24
TheJuliaYeah15:24
dtantsurTheJulia: with release mapping already in place, we can probably survive devstack and grenade branching15:24
*** irclogbot_2 has joined #openstack-ironic15:24
TheJuliadtantsur: if our branch is not there, we won't15:24
TheJuliagrenade will break once the mappings are changed15:24
dtantsurI *think* it works nowadays, although in a weird fashion15:24
dtantsurgrenade falls back to master on unknown branches15:25
TheJuliaoh! then that is good!15:25
dtantsurso it will be master->master testing15:25
TheJuliaokay15:25
iurygregoryyeah15:25
dtantsurwhich is bad, but fine if only for a brief momemnt15:25
TheJuliaI'm glad that has been changed15:25
dtantsurTheJulia: I *think* it has been the case for a while15:25
TheJuliaIs there anything else for the release?15:25
dtantsurwhat used to break us is missing release mapping on our side15:25
dtantsurI may be terribly wrong, of course, consult your lawyer before trusting me15:25
TheJuliadtantsur: eh, I guess it was ~3 cycles ago that we had such issues last15:26
TheJuliaI remember two cycles ago, we were early and were like "why has this not been changed yet?"15:26
TheJuliaAnyway!15:26
* dtantsur also suspects the release team will wait a reasonable amount of time15:26
TheJuliaI think we're good to move on15:26
dtantsur++15:26
iurygregory++15:26
TheJuliaWe seem to have no discussion topics, so we could stop by Baremetal SIG15:27
TheJuliaarne_wiebalck: ^^^15:27
TheJuliaor we could just go directly to Open Discussion15:27
rlooi believe dtantsur is correct wrt grenade15:27
QianbiaoHello, i got a question. If i submit "raid configuration support" patch after unsuri released, may it be cherry-picked to stable/unsuri ?15:27
arne_wiebalcknothing to see here either15:27
*** irclogbot_2 has quit IRC15:27
dtantsurQianbiao: features are generally not backported, sorry15:27
TheJuliaPass go, collect two code reviews and a get out of discussion free card! (Making a monopoly game reference)15:27
iurygregorynice reference15:28
*** irclogbot_2 has joined #openstack-ironic15:28
TheJuliaQianbiao: We can make an early Victoria cycle release once it merges15:28
*** mkrai_ has quit IRC15:28
Qianbiaodtantsur any chance that i commit it now?15:28
TheJuliaIt is not unheard of us to do such. :)15:28
QianbiaoTheJulia ok.15:28
openstackgerritMerged openstack/python-ironic-inspector-client stable/ussuri: Document the new standalone CLI  https://review.opendev.org/72357515:29
TheJuliaWell, with arne_wiebalck having chimed in, lets go to Open Discussion15:29
TheJulia#topic Open Discussion15:29
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)"15:29
TheJuliaHopefully everyone had a wonderful weekend!15:29
dtantsurThey become shorter and shorter, these weekends15:30
dtantsurhas anyone else noticed?15:30
iurygregorynope .-.15:30
rpittaudtantsur: kind of, yeah :/15:30
TheJuliaThey have :(15:31
*** irclogbot_2 has quit IRC15:31
kaifeng_anyone noticed the weird issue in the ML wrt the NCs and ICs?15:32
TheJuliakaifeng_: NCs? ICs? I'm guessing ML is mailing list?15:32
dtantsurI'm not even sure what NC and IC are..15:32
kaifeng_heh, actually i am not the inventor, NC = nova compute, IC = ironic conductor15:33
TheJuliaI've not seen anything, but pulling up the ml now15:34
* dtantsur used to n-cpu and ir-cond15:34
kaifeng_http://lists.openstack.org/pipermail/openstack-discuss/2020-April/014337.html15:34
dtantsurI wonder why it hasn't landed in my inbox..15:35
kaifeng_what makes me confused in the problem is an image is downloaded in one conductor host while pxe-ed from another conductor host15:35
dtantsurtake over takes some time15:36
kaifeng_i am not sure how that can happen, except there is a take over, but this is not mentioned in the text15:36
arne_wiebalckor does not work at all15:36
dtantsurlol15:36
dtantsurthis is.. concerning15:37
iurygregorylol15:37
rpittauoO15:37
iurygregorywould be a problem15:37
dtantsurcan we put CI testing of take over on our backlog please?15:37
dtantsurI recall fixing a lot of bugs with it at some point15:37
iurygregory++15:37
*** irclogbot_3 has joined #openstack-ironic15:37
arne_wiebalckI meant take over in Ironic and Nova, since he was changing both.15:37
TheJuliayeah, we've had to fix a number of things. I suspect we're missing some context.15:38
iurygregoryouch15:38
iurygregory>.< nova15:38
arne_wiebalckWe stopped doing this.15:38
dtantsurI don't think the nova's take over can affect the deploy process15:38
dtantsurit's only about resource tracker updates, everything else is re-balanced when going through ir-api15:38
kaifeng_because we will utilize mutiple n-cpu and ir-cond in production, this issue seems suspicious15:39
arne_wiebalckif multiple compute nodes claim an overlapping set of nodes, there will be a fine mess15:40
TheJuliaI think they may be expecting the hash rings in both ironic and nova to be calculated exactly the same way15:40
dtantsurit's a fair (and wrong) expectation15:40
TheJuliaindeed15:40
dtantsurbut I don't see how a discrepancy between the two hash rings could cause a node to be handled by two conductors15:40
dtantsurbut if they try to deploy while take over is in progress..15:41
dtantsurI don't think anybody has tried that. All bets are off.15:41
TheJuliathis is true. The two minutes in between the actions could be a lot of activity15:41
*** Qianbiao is now known as Qianbiao|afk15:42
*** Qianbiao|afk has quit IRC15:42
arne_wiebalckI don't think it is the interaction between the two hash rings, it is (or was) more a bug in the implementation in one of them.15:42
arne_wiebalckFWIU, the given nova-compute did not let go :)15:43
arne_wiebalckgiving15:43
TheJuliaI think we're going to need more in the way of logs to really understand what is occuring15:43
TheJuliait feels like there is more than one issue :\15:44
kaifeng_indeed, just the "unstable" scares15:44
arne_wiebalck"simple" take over works fine, we have added and removed Ironic controllers several times15:45
openstackgerritChristopher Dearborn proposed openstack/ironic master: Fix supported sushy-oem-idrac version  https://review.opendev.org/72358815:45
arne_wiebalck"simple" == I haven't tried to do that while mass-instantiating nodes15:45
TheJuliaI guess n-cpu and ir-cond is running on the same machine too15:45
TheJuliaI guess the question becomes can ir-cond be scaled first, and then api/n-cpu15:46
arne_wiebalckwe have these on different nodes15:46
* TheJulia wonders if this is also the latest stable queens or an older stable/queens release15:46
TheJuliaI seem to remember there were some queens timeframe fixes15:46
dtantsurnot unlikely15:47
dtantsuroh, https://review.opendev.org/#/c/723588/ should ideally merge before branching15:48
patchbotpatch 723588 - ironic - Fix supported sushy-oem-idrac version - 1 patch set15:48
* dtantsur has just noticed15:48
rpittau^ I was going to say :)15:48
dtantsur3 minutes to approval, I bet the fasted cdearborn has observed :D15:49
rpittauthis has been rebased also https://review.opendev.org/71768915:49
patchbotpatch 717689 - ironic-prometheus-exporter - Use hacking for pep8 tests - 7 patch sets15:49
* cdearborn almost fell out of my chair!15:49
* rpioso did :-)15:49
dtantsur:D15:50
dtantsuriurygregory: https://review.opendev.org/#/c/720793/ seemingly needs a rebase?15:50
patchbotpatch 720793 - ironic-prometheus-exporter - Stop configuring install_command in tox. - 4 patch sets15:50
iurygregorydtantsur, yeah just noticed15:50
TheJuliaugh. \o/15:50
iurygregoryI thought rpittau had everything done so I used his patch as base XD15:51
TheJuliaDoes anyone want to respond to the hash ring thread on the mailing list?15:51
dtantsurI responded with a couple of comments just to revive it15:51
TheJuliaokay15:51
dtantsurwe may need to ask for logs, not sure how doable it is15:51
TheJuliaYeah :\15:52
TheJuliais there anything else to discuss today?15:52
rpittaunot strictly related to release, but TheJulia when you have a minute I think this can be merged now https://review.opendev.org/70882715:53
patchbotpatch 708827 - ironic-python-agent (stable/train) - Rescan after restarting the md device - 2 patch sets15:53
*** sshnaidm is now known as sshnaidm|afk15:53
dtantsurre that release cadence doodle: please vote by ~ noon UTC tomorrow, as the most likely candidate now is on Wednesday15:54
TheJuliarpittau: thanks15:54
TheJuliadtantsur: Thanks!15:55
TheJuliaThanks everyone!15:55
dtantsurthank you TheJulia15:56
rpittauthank you TheJulia :)15:56
*** ociuhandu has quit IRC15:56
TheJuliaHave a wonderful week everyone!15:56
TheJulia#endmeeting15:57
*** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/ironic/ | Bugs: https://storyboard.openstack.org/#!/project_group/75 | Contributors are generally present between 6 AM and 12 AM UTC, If we do not answer, please feel free to pose questions to openstack-discuss mailing list."15:57
openstackMeeting ended Mon Apr 27 15:57:01 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:57
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-04-27-15.00.html15:57
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-04-27-15.00.txt15:57
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-04-27-15.00.log.html15:57
kaifeng_thank you TheJulia o/15:57
*** lucasagomes has quit IRC15:57
*** dsneddon has joined #openstack-ironic15:59
*** ociuhandu has joined #openstack-ironic16:00
openstackgerritMerged openstack/sushy-tools master: Restore default netboot boot option  https://review.opendev.org/72354616:00
TheJuliaso I think arne_wiebalck's change just needs a rebase on top of the fix raising an exception16:06
rpittauhave a great week everyone, talk tomorrow! o/16:07
*** rpittau is now known as rpittau|afk16:07
openstackgerritJulia Kreger proposed openstack/ironic master: Don't break UEFI install with older IPAs  https://review.opendev.org/72078116:09
TheJuliadtantsur: I don't think your -1 applies to ^^ anymore since we should now be raising an exception16:10
arne_wiebalckTheJulia: your changes will now make sure there is an exception in addition to the status code, no:16:12
arne_wiebalck?}16:12
* arne_wiebalck has fat fingers16:12
TheJuliaarne_wiebalck: TypeError16:12
TheJuliait looks for the text, not the status code since the 500 will get returned in the error status code for any error16:12
TheJuliaso more a "we know typeerror will indicate failure we know what the issue is, so we can handle it somewhat gracefully16:13
dtantsurTheJulia: yep, removed, will re-review in a bit16:14
TheJuliak16:14
arne_wiebalckso, there will be a TypeError when in the unknow param situation?16:14
arne_wiebalckunknown16:14
TheJuliayes16:14
arne_wiebalckok16:15
arne_wiebalckyou just pushed this?16:15
arne_wiebalckor do you want me to do it?16:15
TheJuliaarne_wiebalck: already rebased your change16:15
arne_wiebalckcool16:16
arne_wiebalckthanks16:16
arne_wiebalck ah, but the exception catch still needs change I see16:17
*** iurygregory has quit IRC16:17
TheJuliaarne_wiebalck: I don't think so16:17
* arne_wiebalck is confused16:17
TheJuliawe still emit the same exception that your patch was intended to use16:18
dtantsurmmm, my internet is gone, but IRC still works?16:18
* arne_wiebalck should go and the read the code ...16:19
TheJuliadtantsur: magic?!?16:19
dtantsurright?16:19
arne_wiebalckthat helps16:19
arne_wiebalckok, I understand now, sorry16:19
TheJuliaHas anyone seen https://www.hpe.com/us/en/newsroom/press-release/2020/03/hpe-and-industry-partners-simplify-5g-rollout-with-open-vendor-neutral-software-to-manage-it-infrastructure-deployments.html ?16:20
*** ociuhandu has quit IRC16:20
dtantsurI cannot ping anything outside of Germany, as it seems16:20
* dtantsur hasn't and won't be able to right now :(16:20
dtantsuroh, wait, hpe.com does open16:20
*** ociuhandu has joined #openstack-ironic16:21
*** k_mouza has quit IRC16:21
arne_wiebalckThe technical white paper: https://psnow.ext.hpe.com/doc/a50000782enw16:24
dtantsurmm, no, this subdomain won't open for me16:25
*** dsneddon has quit IRC16:26
*** alexmcleod has quit IRC16:26
*** vesper11 has quit IRC16:26
*** hoonetorg has quit IRC16:26
*** mjturek has quit IRC16:26
*** Blinkiz has quit IRC16:26
*** tonyb has quit IRC16:26
openstackgerritMerged openstack/sushy-tools master: Fix libvirt test for Python 3.8  https://review.opendev.org/72281516:26
*** irclogbot_3 has quit IRC16:28
*** k_mouza has joined #openstack-ironic16:29
*** vesper11 has joined #openstack-ironic16:29
*** irclogbot_0 has joined #openstack-ironic16:29
TheJulialooks like a distributed redfish wrapper being developed and will be released to LF during Q2 according to what I've read16:31
*** hoonetorg has joined #openstack-ironic16:31
openstackgerritMerged openstack/virtualbmc master: Restore default netboot boot option  https://review.opendev.org/72355516:31
dtantsurmm, I'd say this idea has been floating on the surface16:31
*** dsneddon has joined #openstack-ironic16:31
TheJuliafor some time16:32
openstackgerritMerged openstack/sushy master: Restore default netboot boot option  https://review.opendev.org/72355416:32
TheJuliawith an aggregator plugin system so you can use it for IPMI systems16:32
*** mjturek has joined #openstack-ironic16:32
*** Blinkiz has joined #openstack-ironic16:32
*** tonyb has joined #openstack-ironic16:32
TheJuliait feels like it is intended to be a consistency layer, but I checked DMTF's wips and not seeing anything that seems applicable16:33
openstackgerritNoor Muhammad Malik proposed openstack/ironic master: Add PXE reset known issue to the docs  https://review.opendev.org/72255516:35
openstackgerritDerek Higgins proposed openstack/ironic master: WIP ipv6 jobs  https://review.opendev.org/61316916:37
*** stendulker has quit IRC16:38
openstackgerritVerification of a change to openstack/ironic-inspector failed: No need to import print function anymore  https://review.opendev.org/72157616:39
*** alexmcleod has joined #openstack-ironic16:46
openstackgerritMerged openstack/python-ironic-inspector-client master: Restore default netboot boot option  https://review.opendev.org/72355916:47
*** k_mouza has quit IRC16:53
arne_wiebalckbye everyone o/16:57
*** k_mouza has joined #openstack-ironic16:57
*** derekh has quit IRC17:03
*** ociuhandu has quit IRC17:06
*** ociuhandu has joined #openstack-ironic17:06
*** k_mouza has quit IRC17:11
*** ociuhandu has quit IRC17:12
openstackgerritNoor Muhammad Malik proposed openstack/ironic master: Add PXE reset known issue to the docs  https://review.opendev.org/72255517:12
*** k_mouza has joined #openstack-ironic17:13
*** k_mouza has quit IRC17:13
*** dtantsur is now known as dtantsur|afk17:18
dtantsur|afkg'night17:18
*** yolanda has quit IRC17:18
*** mjturek has quit IRC17:26
*** Blinkiz has quit IRC17:26
*** tonyb has quit IRC17:26
*** alexmcleod has quit IRC17:26
*** dsneddon has quit IRC17:26
*** alexmcleod has joined #openstack-ironic17:27
*** mjturek has joined #openstack-ironic17:29
*** Blinkiz has joined #openstack-ironic17:29
*** tonyb has joined #openstack-ironic17:29
*** dsneddon has joined #openstack-ironic17:29
*** ociuhandu has joined #openstack-ironic17:42
*** hoonetorg has quit IRC18:02
*** kaifeng_ has quit IRC18:09
*** cdearborn has quit IRC18:12
*** dmellado has quit IRC18:17
*** ociuhandu has quit IRC18:20
*** ociuhandu has joined #openstack-ironic18:20
*** alexmcleod has quit IRC18:21
*** dmellado has joined #openstack-ironic18:24
*** dmellado has quit IRC18:25
*** alexmcleod has joined #openstack-ironic18:28
*** dmellado has joined #openstack-ironic18:33
*** iurygregory has joined #openstack-ironic18:34
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-prometheus-exporter master: Stop configuring install_command in tox.  https://review.opendev.org/72079318:42
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Increase timeout for zuulv3 grenade job  https://review.opendev.org/72364618:51
openstackgerritMerged openstack/ironic master: Log when IPA fallback occurs on bootloader install  https://review.opendev.org/72198818:52
openstackgerritMerged openstack/ironic master: Fix agent_client handling of embedded errors  https://review.opendev.org/72202718:52
openstackgerritMerged openstack/ironic-python-agent master: Update master for stable/ussuri  https://review.opendev.org/72356418:52
openstackgerritMerged openstack/ironic-python-agent stable/ussuri: Update .gitreview for stable/ussuri  https://review.opendev.org/72356218:52
openstackgerritMerged openstack/ironic-python-agent stable/ussuri: Update TOX_CONSTRAINTS_FILE for stable/ussuri  https://review.opendev.org/72356318:52
iurygregoryI'm loving the TIMED_OUT in ironic-inspector-grenade -.-'18:59
iurygregoryand seems we can't increase without increassing in grenade base job...18:59
* iurygregory talking with people in qa to see how we can fix this =)18:59
*** ociuhandu has quit IRC19:03
*** ociuhandu has joined #openstack-ironic19:06
*** ociuhandu has quit IRC19:20
*** ociuhandu has joined #openstack-ironic19:27
*** Lucas_Gray has joined #openstack-ironic19:41
*** Lucas_Gray has quit IRC19:42
*** jdandrea has quit IRC19:45
TheJuliaiurygregory: how long is it taking?19:46
*** Lucas_Gray has joined #openstack-ironic19:51
*** rloo has quit IRC19:55
*** rloo has joined #openstack-ironic19:56
*** jdandrea has joined #openstack-ironic19:56
*** rloo has quit IRC19:57
openstackgerritVerification of a change to openstack/ironic-inspector failed: Update .gitreview for stable/ussuri  https://review.opendev.org/72356819:57
openstackgerritVerification of a change to openstack/ironic-inspector failed: Update TOX_CONSTRAINTS_FILE for stable/ussuri  https://review.opendev.org/72356919:57
*** rloo has joined #openstack-ironic19:57
*** rloo has quit IRC20:00
*** rloo has joined #openstack-ironic20:01
openstackgerritMerged openstack/sushy-tools master: Add pep8 import order check  https://review.opendev.org/72165020:09
openstackgerritMerged openstack/ironic-inspector master: Update master for stable/ussuri  https://review.opendev.org/72357020:11
*** k_mouza has joined #openstack-ironic20:14
*** jdandrea has quit IRC20:15
*** alexmcleod has quit IRC20:18
*** jdandrea has joined #openstack-ironic20:19
*** k_mouza has quit IRC20:19
*** alexmcleod has joined #openstack-ironic20:27
*** ociuhandu has quit IRC20:57
*** ociuhandu has joined #openstack-ironic20:58
openstackgerritMerged openstack/sushy master: Add import order check  https://review.opendev.org/72164920:58
openstackgerritMerged openstack/python-ironicclient master: Restore default netboot boot option  https://review.opendev.org/72354721:04
*** Goneri has quit IRC21:05
*** igordc has joined #openstack-ironic21:07
*** ociuhandu has quit IRC21:11
*** cdearborn has joined #openstack-ironic21:14
*** rcernin has joined #openstack-ironic21:14
*** ociuhandu has joined #openstack-ironic21:19
*** rloo has quit IRC21:29
*** ociuhandu has quit IRC21:29
*** ociuhandu has joined #openstack-ironic21:39
*** rloo has joined #openstack-ironic21:44
*** rloo has quit IRC21:45
*** rloo has joined #openstack-ironic21:45
*** ociuhandu has quit IRC21:49
openstackgerritMerged openstack/ironic master: Stop configuring install_command in tox.  https://review.opendev.org/72079022:14
openstackgerritMerged openstack/ironic-prometheus-exporter master: Use hacking for pep8 tests  https://review.opendev.org/71768922:14
*** ociuhandu has joined #openstack-ironic22:18
*** alexmcleod has quit IRC22:20
*** Lucas_Gray has quit IRC22:21
*** alexmcleod has joined #openstack-ironic22:27
*** ociuhandu has quit IRC22:29
*** yaawang has joined #openstack-ironic22:32
*** yaawang_ has quit IRC22:33
*** tkajinam has joined #openstack-ironic22:49
*** tkajinam has quit IRC22:49
*** tkajinam has joined #openstack-ironic22:50
*** rloo has quit IRC22:51
openstackgerritMerged openstack/ironic-inspector master: Upgrade flake8-import-order version to 0.17.1  https://review.opendev.org/72164822:53
openstackgerritMerged openstack/ironic master: Fix supported sushy-oem-idrac version  https://review.opendev.org/72358822:53
openstackgerritMerged openstack/ironic-python-agent master: Add Python3 victoria unit tests  https://review.opendev.org/72356522:53
openstackgerritMerged openstack/ironic master: Implements: Reactive HUAWEI ibmc driver  https://review.opendev.org/72322522:53
openstackgerritVerification of a change to openstack/ironic failed: Don't break UEFI install with older IPAs  https://review.opendev.org/72078122:54
*** k_mouza has joined #openstack-ironic23:00
*** k_mouza has quit IRC23:04
*** alexmcleod_ has joined #openstack-ironic23:12
*** alexmcleod has quit IRC23:14
*** igordc has quit IRC23:14
*** k_mouza has joined #openstack-ironic23:50
*** k_mouza has quit IRC23:54

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