*** ociuhandu has quit IRC | 00:07 | |
*** ociuhandu has joined #openstack-ironic | 00:13 | |
*** ociuhandu has quit IRC | 00:18 | |
*** kaifeng has joined #openstack-ironic | 00:45 | |
*** Goneri has quit IRC | 00:59 | |
*** Qianbiao has joined #openstack-ironic | 01:45 | |
*** ociuhandu has joined #openstack-ironic | 02:40 | |
*** ociuhandu has quit IRC | 02:50 | |
*** threestrands has joined #openstack-ironic | 02:51 | |
*** ociuhandu has joined #openstack-ironic | 02:55 | |
*** k_mouza has joined #openstack-ironic | 02:57 | |
*** ociuhandu has quit IRC | 03:00 | |
*** k_mouza has quit IRC | 03:01 | |
*** mkrai has joined #openstack-ironic | 03:19 | |
*** ociuhandu has joined #openstack-ironic | 03:33 | |
*** ociuhandu has quit IRC | 03:38 | |
*** jdandrea has quit IRC | 03:38 | |
*** ociuhandu has joined #openstack-ironic | 04:27 | |
*** ociuhandu has quit IRC | 04:34 | |
*** mkrai has quit IRC | 04:59 | |
*** mkrai_ has joined #openstack-ironic | 04:59 | |
*** yaawang_ has quit IRC | 05:13 | |
*** yaawang_ has joined #openstack-ironic | 05:14 | |
*** Qianbiao has quit IRC | 05:38 | |
*** ociuhandu has joined #openstack-ironic | 06:02 | |
*** ociuhandu has quit IRC | 06:06 | |
*** Qianbiao has joined #openstack-ironic | 06:13 | |
*** yolanda has joined #openstack-ironic | 06:16 | |
iurygregory | good mornign Ironic o/ | 06:23 |
---|---|---|
arne_wiebalck | Good morning iurygregory and ironic! | 06:36 |
Qianbiao | good morning :) | 06:40 |
*** lennyb has quit IRC | 06:54 | |
*** ociuhandu has joined #openstack-ironic | 06:55 | |
*** iurygregory has quit IRC | 07:09 | |
*** iurygregory has joined #openstack-ironic | 07:10 | |
*** mkrai_ has quit IRC | 07:11 | |
iurygregory | arne_wiebalck, hey o/ sory was having my coffee =) | 07:11 |
*** ociuhandu has quit IRC | 07:13 | |
*** ociuhandu has joined #openstack-ironic | 07:13 | |
*** rpittau|afk is now known as rpittau | 07:22 | |
rpittau | good morning ironic! o/ | 07:22 |
rpittau | mmm looks like huawei ci is up again | 07:25 |
Qianbiao | rpittau yes, my friend. | 07:26 |
*** ociuhandu has quit IRC | 07:26 | |
rpittau | hey Qianbiao, glad to see that :) | 07:26 |
Qianbiao | I setup it last weekend. | 07:27 |
Qianbiao | Yeah. it's down because openstack.com -> opendev. | 07:27 |
Qianbiao | and opendev is block by china.. | 07:28 |
*** ociuhandu has joined #openstack-ironic | 07:29 | |
Qianbiao | Hello, anyone familiar with metal3? | 07:30 |
Qianbiao | is there any docker image that could setup metal3 automate. | 07:30 |
*** sshnaidm|afk is now known as sshnaidm | 07:35 | |
rpittau | Qianbiao: I'm not aware of such thing, although you can find comprehensive automation scripts in metal3-dev-env | 07:35 |
Qianbiao | rpittau thanks will look at it. | 07:37 |
openstackgerrit | QianBiao Ng proposed openstack/ironic stable/train: Implements: Reactive HUAWEI ibmc driver https://review.opendev.org/723331 | 07:38 |
*** mkrai_ has joined #openstack-ironic | 07:39 | |
iurygregory | morning rpittau o/ | 07:39 |
rpittau | hey iurygregory :) | 07:39 |
openstackgerrit | Aija Jaunteva proposed openstack/ironic stable/train: DRAC: Added redfish management interface issue https://review.opendev.org/723333 | 07:47 |
*** yaawang_ has quit IRC | 07:49 | |
*** yaawang_ has joined #openstack-ironic | 07:50 | |
*** lucasagomes has joined #openstack-ironic | 07:54 | |
*** yaawang_ has quit IRC | 07:54 | |
*** yaawang_ has joined #openstack-ironic | 07:55 | |
*** yaawang_ has quit IRC | 08:02 | |
*** yaawang_ has joined #openstack-ironic | 08:03 | |
rpittau | Qianbiao: 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 |
Qianbiao | rpittau ok, when is the meeting open. how can i attend. | 08:09 |
rpittau | the meeting is on mondays, so we have one today, at 1500 UTC | 08:10 |
rpittau | Qianbiao: it's on this channel, more info here: https://wiki.openstack.org/wiki/Meetings/Ironic | 08:10 |
Qianbiao | ok | 08:10 |
Qianbiao | rpittau it's about seven hours later right? | 08:12 |
rpittau | Qianbiao: yeah, it's in 7 hours | 08:12 |
rpittau | probably not the best time for you :/ | 08:12 |
*** rcernin has quit IRC | 08:13 | |
Qianbiao | :) need more coffee. | 08:14 |
Qianbiao | <rpittau> in this IRC channel directly? | 08:17 |
rpittau | Qianbiao: correct | 08:17 |
Qianbiao | lol. i try to connect irc.freenode.net | 08:17 |
Qianbiao | find it seems the same chat.freenode.net | 08:17 |
*** threestrands has quit IRC | 08:20 | |
*** tkajinam has quit IRC | 08:23 | |
*** mkrai_ has quit IRC | 08:24 | |
*** mkrai has joined #openstack-ironic | 08:25 | |
*** mcarden has quit IRC | 08:29 | |
*** logan_ has joined #openstack-ironic | 08:31 | |
*** logan- has quit IRC | 08:32 | |
*** alexmcleod has joined #openstack-ironic | 08:34 | |
*** logan_ is now known as logan- | 08:35 | |
*** derekh has joined #openstack-ironic | 08:38 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Add function to calculate memory https://review.opendev.org/722334 | 08:46 |
iurygregory | I'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-ironic | 08:55 | |
*** k_mouza has quit IRC | 08:59 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Fix TypeError with newer version of lshw https://review.opendev.org/722335 | 09:01 |
*** k_mouza has joined #openstack-ironic | 09:02 | |
rpittau | iurygregory: it really looks just like an implementation choice | 09:07 |
iurygregory | gotcha | 09:07 |
rpittau | in 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 methods | 09:08 |
iurygregory | thanks rpittau =) | 09:15 |
openstackgerrit | Merged openstack/ironic-inspector master: Native zuulv3 grenade job for inspector https://review.opendev.org/703995 | 09:17 |
iurygregory | \o/ | 09:22 |
openstackgerrit | Verification of a change to openstack/ironic failed: In-band deploy steps: correctly wipe driver_internal_info https://review.opendev.org/722916 | 09:48 |
*** dtantsur|afk is now known as dtantsur | 09:49 | |
dtantsur | morning ironic, happy Monday | 09:50 |
* dtantsur has finally replenished his stock of buckwheat \o/ | 09:51 | |
iurygregory | morning dtantsur o/ | 09:51 |
dtantsur | mgoddard: thank you for taking care of the deploy steps patches | 09:52 |
dtantsur | pity we could not get it to show-caseable state in Ussuri.. but still great progress! | 09:52 |
mgoddard | dtantsur: just a bit of rechecking :) | 09:52 |
mgoddard | dtantsur: progress is good, thanks for pushing on it. CentOS 8 ate all my time :( | 09:53 |
dtantsur | I can imagine. We had quite some fun with RHEL 8 as well, but it was in Stein timeframe downstream | 09:54 |
mgoddard | dtantsur: best not to rush the split through anyway. Hopefully we can land early in V and give it some good testing | 09:54 |
dtantsur | exactly | 09:55 |
dtantsur | maybe figure out how to simulate a 3rd party driver in the CI.. | 09:55 |
dtantsur | mgoddard: we should start preparing releases of the remaining projects today-tomorrow. will you have time for that? | 09:55 |
mgoddard | dtantsur: I was thinking about testing. Would it be crazy to expose a run_command deploy step? | 09:56 |
mgoddard | we'd need to work out running steps more than once | 09:56 |
dtantsur | mmmmmm | 09:56 |
*** yolanda has quit IRC | 09:56 | |
mgoddard | but it could be used to e.g. echo foo > file at various points | 09:56 |
dtantsur | I think you can request a step more than once via deploy templates | 09:56 |
mgoddard | we disabled it AFAIR, there were some subtle issues | 09:57 |
dtantsur | but yeah, it's an interesting idea. and will simplify plugging in custom hw manager. | 09:57 |
mgoddard | could be enabled with care | 09:57 |
dtantsur | another TODO item for V :) | 09:57 |
mgoddard | yeah | 09:57 |
mgoddard | dtantsur: on releases, yes I can make some time today | 09:58 |
dtantsur | great! it seems that some patches are still going through the gate, but inspector may be ready | 09:59 |
mgoddard | dtantsur: too late to get the deploy steps follow up in? | 09:59 |
dtantsur | mgoddard: which one? | 09:59 |
dtantsur | 722916 must merge before we release IMO | 10:00 |
dtantsur | (I'm not implying we must release right now, but rather start the countdown and track the remaining patches) | 10:00 |
mgoddard | ok, makes sense | 10:00 |
*** mkrai has quit IRC | 10:06 | |
*** mkrai_ has joined #openstack-ironic | 10:07 | |
rpittau | so many rechecks :/ | 10:08 |
*** yolanda has joined #openstack-ironic | 10:10 | |
rpittau | we should also keep an eye on https://review.opendev.org/#/c/722027/ hopefully will merge on this run | 10:10 |
patchbot | patch 722027 - ironic - Fix agent_client handling of embedded errors - 3 patch sets | 10:10 |
*** mkrai_ has quit IRC | 10:10 | |
*** mkrai has joined #openstack-ironic | 10:10 | |
openstackgerrit | Derek Higgins proposed openstack/ironic master: WIP ipv6 jobs https://review.opendev.org/613169 | 10:20 |
*** rpittau is now known as rpittau|bbl | 10:32 | |
openstackgerrit | Verification of a change to openstack/networking-baremetal failed: Convert networking-baremetal job to dib https://review.opendev.org/718697 | 10:34 |
*** ociuhandu has quit IRC | 10:42 | |
*** ociuhandu has joined #openstack-ironic | 10:44 | |
*** Qianbiao is now known as qianbiao|afk | 10:45 | |
openstackgerrit | Merged openstack/ironic stable/rocky: Fix issue where server fails to reboot https://review.opendev.org/721803 | 11:00 |
*** ociuhandu has quit IRC | 11:03 | |
*** ociuhandu has joined #openstack-ironic | 11:04 | |
*** qianbiao|afk has quit IRC | 11:16 | |
*** ociuhandu has quit IRC | 11:23 | |
openstackgerrit | Merged openstack/ironic stable/stein: Make deploy step failure logging indicate the error https://review.opendev.org/717685 | 11:27 |
openstackgerrit | Merged openstack/ironic stable/stein: Fix issue where server fails to reboot https://review.opendev.org/721797 | 11:27 |
openstackgerrit | Merged openstack/ironic master: Upgrade flake8-import-order version to 0.17.1 https://review.opendev.org/721646 | 11:27 |
*** Mahendra22 has joined #openstack-ironic | 11:37 | |
mgoddard | iurygregory, dtantsur, TheJulia: is ironic-tempest-plugin ready to release? | 11:43 |
mgoddard | I saw TheJulia -1'd the patch | 11:43 |
mgoddard | 3 patches with a +2: https://review.opendev.org/#/q/project:openstack/ironic-tempest-plugin+status:open+branch:master | 11:43 |
iurygregory | mgoddard, I don't think we do releases for ironic-tempest-plugin | 11:44 |
iurygregory | O.o | 11:44 |
iurygregory | maybe we create a tag only | 11:44 |
*** yolanda has quit IRC | 11:44 | |
mgoddard | iurygregory: it's released but not branched | 11:44 |
*** yolanda has joined #openstack-ironic | 11:44 | |
iurygregory | mgoddard, oh ok =) I thought by release you only mean create a stable branch =) | 11:45 |
iurygregory | mgoddard, since we split standalone tests it's good to have the job voting I would say | 11:45 |
mgoddard | iurygregory: you mean https://review.opendev.org/#/c/720675/2 ? Looks like it's non-voting | 11:46 |
patchbot | patch 720675 - ironic-tempest-plugin - Add standalone redfish jobs - 2 patch sets | 11:46 |
iurygregory | mgoddard, yup | 11:46 |
iurygregory | and https://review.opendev.org/712042 | 11:47 |
patchbot | patch 712042 - ironic - Add ironic-standalone-redfish - 10 patch sets | 11:47 |
*** qianbiao|afk has joined #openstack-ironic | 11:48 | |
mgoddard | iurygregory: doesn't look that that should block release though? | 11:49 |
iurygregory | mgoddard, didn't she put the reason for the -1? | 11:49 |
mgoddard | nope | 11:50 |
iurygregory | maybe we are waitting for something in other project | 11:50 |
iurygregory | that's all I can think atm | 11:50 |
mgoddard | I'll wait for TheJulia | 11:50 |
iurygregory | yeah =) | 11:50 |
iurygregory | sorry I couldn't help that much | 11:51 |
mgoddard | Looks like inspector is good to go | 11:51 |
*** ociuhandu has joined #openstack-ironic | 11:55 | |
mgoddard | iurygregory: ironic-prometheus-exporter ready to release? | 11:59 |
iurygregory | mgoddard, I would say we can merge the 3 remaining patches | 12:00 |
iurygregory | 2 are missing +A | 12:00 |
iurygregory | https://review.opendev.org/717689 | 12:01 |
patchbot | patch 717689 - ironic-prometheus-exporter - Use hacking for pep8 tests - 5 patch sets | 12:01 |
iurygregory | https://review.opendev.org/717684 | 12:01 |
patchbot | patch 717684 - ironic-prometheus-exporter - Add lower constraints jobs - 5 patch sets | 12:01 |
mgoddard | iurygregory: aren't those really related to CI? | 12:01 |
mgoddard | iurygregory: we're already past the official deadline | 12:01 |
iurygregory | one we are adding l-c | 12:01 |
mgoddard | ok | 12:01 |
iurygregory | I think would be good to have l-c | 12:01 |
mgoddard | approving :) | 12:02 |
iurygregory | CI is fast =) | 12:02 |
iurygregory | so we will we see Merged very soon | 12:02 |
mgoddard | cool | 12:03 |
*** rh-jelabarre has joined #openstack-ironic | 12:05 | |
*** ociuhandu has quit IRC | 12:06 | |
*** ociuhandu has joined #openstack-ironic | 12:12 | |
openstackgerrit | Verification of a change to openstack/ironic-inspector failed: Convert jobs to dib https://review.opendev.org/712516 | 12:23 |
iurygregory | =( | 12:24 |
*** ociuhandu has quit IRC | 12:24 | |
*** ociuhandu has joined #openstack-ironic | 12:24 | |
*** ociuhandu has quit IRC | 12:35 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Switch to unittest mock https://review.opendev.org/717979 | 12:41 |
*** ociuhandu has joined #openstack-ironic | 12:41 | |
*** hjensas is now known as hjensas|afk | 12:43 | |
*** qianbiao|afk has quit IRC | 12:47 | |
*** rpittau|bbl is now known as rpittau | 12:49 | |
*** michchap has joined #openstack-ironic | 12:52 | |
*** mkrai has quit IRC | 12:52 | |
*** ociuhandu has quit IRC | 12:53 | |
*** Goneri has joined #openstack-ironic | 12:54 | |
dtantsur | mgoddard: I would say all mandatory stuff has merged to tempest-plugin, but worth checking with Julia | 12:55 |
*** ociuhandu has joined #openstack-ironic | 13:00 | |
dtantsur | mgoddard: left a question on https://review.opendev.org/#/c/723399/1 | 13:00 |
patchbot | patch 723399 - releases - Release Ironic Python Agent 7.0.0 for Ussuri, bran... - 1 patch set | 13:00 |
*** Goneri has quit IRC | 13:01 | |
TheJulia | o/ | 13:03 |
TheJulia | I feel like I've been missing, but yeah. :( | 13:04 |
dtantsur | morning TheJulia, how's your leg? | 13:04 |
TheJulia | a little tingling, seems it was my old back injury for like 18 years ago acting up | 13:04 |
dtantsur | sounds like an improvement nonetheless! | 13:05 |
TheJulia | yeah, I know how to handle my back being unhappy, my leg presenting with the way it did was just.. different | 13:05 |
dtantsur | back on the release topic, we should consider including https://review.opendev.org/#/c/723225/ | 13:06 |
patchbot | patch 723225 - ironic - Implements: Reactive HUAWEI ibmc driver - 2 patch sets | 13:06 |
*** qianbiao|afk has joined #openstack-ironic | 13:07 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Fix TypeError with newer version of lshw https://review.opendev.org/722335 | 13:08 |
*** qianbiao|afk is now known as Qianbiao | 13:08 | |
openstackgerrit | Noor Muhammad Malik proposed openstack/ironic master: Add PXE reset known issue to the docs https://review.opendev.org/722555 | 13:09 |
TheJulia | I 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 |
rpittau | I'm not sure that will pass the cI tio be honest | 13:09 |
rpittau | s/tio/to | 13:09 |
dtantsur | I've seen the CI voting on patches, I need to double-check what it actually does (seems like bifrost) | 13:09 |
dtantsur | Qianbiao: 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 |
dtantsur | 1) please enable debug logging for services | 13:11 |
dtantsur | 2) please change ibmc_client.connector logs to DEBUG | 13:11 |
Qianbiao | dtantsur ok | 13:12 |
dtantsur | TheJulia, 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 |
iurygregory | newbie 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.o | 13:13 |
patchbot | patch 720675 - ironic-tempest-plugin - Add standalone redfish jobs - 2 patch sets | 13:13 |
TheJulia | dtantsur: where did you see the job log? | 13:13 |
dtantsur | TheJulia: plenty of runs on https://review.opendev.org/#/c/723225/ did you "Toggle CI"? | 13:14 |
patchbot | patch 723225 - ironic - Implements: Reactive HUAWEI ibmc driver - 2 patch sets | 13:14 |
iurygregory | yeah they start reporting in patches | 13:14 |
TheJulia | oh, there is now a "Toggle Extra CI" :( | 13:14 |
TheJulia | okay, I see it | 13:14 |
TheJulia | Yeah, Qianbiao if you can revise the release note, I'm for merging prior to releasing | 13:15 |
Qianbiao | TheJulia what should i update for the release note. | 13:17 |
TheJulia | Qianbiao: dtantsur provided feedback in the review | 13:17 |
TheJulia | otherwise, we need to merge then re-edit and wait for that change to land | 13:17 |
Qianbiao | chechking now. | 13:17 |
*** Goneri has joined #openstack-ironic | 13:18 | |
Qianbiao | dtantsur Is this good: Now that HUAWEI ironic 3rd party CI is back, the ``ibmc`` hardware type | 13:22 |
Qianbiao | driver is supported. We can reactive the ibmc driver. | 13:22 |
Qianbiao | I am not a english speaker.. sorry fot that. | 13:22 |
dtantsur | don't worry, many of us are not | 13:22 |
TheJulia | Qianbiao: that is perfectly okay :) | 13:22 |
dtantsur | (I'm not either) | 13:22 |
dtantsur | I'd replace "We can reactive" with "... has been activated again." | 13:22 |
dtantsur | the passive voice sounds a bit more official to me :) | 13:23 |
Qianbiao | :) ok | 13:23 |
TheJulia | Qianbiao: I was thinking the exact same thing as dtantsur | 13:23 |
*** dsneddon has quit IRC | 13:23 | |
TheJulia | and I'm sadly a native english speaker/writer. | 13:23 |
TheJulia | silly languages | 13:23 |
Qianbiao | It's ok, i think it has express clear. | 13:23 |
openstackgerrit | Riccardo Pittau proposed openstack/virtualbmc master: Restore pep8 checking on import order https://review.opendev.org/721162 | 13:24 |
openstackgerrit | Riccardo Pittau proposed openstack/virtualbmc master: Set flake8-import-order version to 0.17.1 https://review.opendev.org/721657 | 13:24 |
Qianbiao | Is there any skill that could speed up `git review`? | 13:25 |
Qianbiao | it's very very slow for me. | 13:25 |
TheJulia | mgoddard: replied to you on the ipa release changeset | 13:26 |
mgoddard | TheJulia: thanks | 13:26 |
TheJulia | git review is typically fairly quick for me. I wonder what it is waiting on | 13:26 |
iurygregory | git review takes 30sec for me =( | 13:27 |
*** jdandrea has joined #openstack-ironic | 13:27 | |
TheJulia | hmm | 13:27 |
TheJulia | Typically about 15 for me | 13:27 |
*** rloo has joined #openstack-ironic | 13:27 | |
Qianbiao | TheJulia iurygregory sometimes up to 30 mins for me ... | 13:28 |
Qianbiao | It's chinese firewall again.. i think | 13:28 |
iurygregory | WOW | 13:29 |
* rpittau will stop complaining for general gerrit review slowness | 13:29 | |
Qianbiao | rpittau XD | 13:29 |
TheJulia | Qianbiao: wow... | 13:29 |
TheJulia | Qianbiao: are you using https or the gerrit native access? | 13:30 |
Qianbiao | https | 13:30 |
openstackgerrit | QianBiao Ng proposed openstack/ironic master: Implements: Reactive HUAWEI ibmc driver https://review.opendev.org/723225 | 13:31 |
Qianbiao | finally... | 13:31 |
Qianbiao | the better news is .. i try to do things right to reduce `git review` times :) | 13:31 |
Qianbiao | * the good news | 13:32 |
Qianbiao | dtantsur i have set debug to true | 13:33 |
Qianbiao | ibmcclient does not have much log, the API is pretty simple. I did not log many debug info for now. | 13:34 |
mgoddard | TheJulia: are we good to go on ironic-tempest-plugin release? | 13:34 |
mgoddard | https://review.opendev.org/#/c/721486/1 | 13:34 |
patchbot | patch 721486 - releases - Release ironic-tempest-plugin for ussuri - 1 patch set | 13:34 |
TheJulia | mgoddard: thanks for the link | 13:35 |
dtantsur | Qianbiao: 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 |
Qianbiao | dtantsur ok, will fix it in next version.(with raid support feature) | 13:36 |
dtantsur | thanks! it's not urgent, take your time. | 13:36 |
TheJulia | mgoddard: changed the release tag value and +1'ed | 13:38 |
* TheJulia suspects that she needs to order a standing desk today | 13:39 | |
mgoddard | TheJulia: thanks | 13:39 |
*** tkajinam has joined #openstack-ironic | 13:42 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Implements: Reactive HUAWEI ibmc driver https://review.opendev.org/723225 | 13:42 |
TheJulia | looks like we're waiting on ironic and the prometheus exporter, maybe | 13:44 |
dtantsur | sushy-tools and virtualbmc as well | 13:45 |
dtantsur | even though they're not series-bound, it's good to have a reference version | 13:45 |
TheJulia | yeah | 13:45 |
Qianbiao | TheJulia dtantsur thanks for the review. May i cherry-pick the patch to stein? | 13:45 |
*** hjensas|afk is now known as hjensas | 13:46 | |
* dtantsur thinks | 13:46 | |
Qianbiao | * to train | 13:46 |
*** tzumainn has joined #openstack-ironic | 13:46 | |
* TheJulia also thinks | 13:46 | |
TheJulia | I don't see why not. There is precedence for that being signaled on a stable branch | 13:47 |
dtantsur | Qianbiao: 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 statement | 13:47 |
Qianbiao | ok. it should works. | 13:47 |
openstackgerrit | QianBiao Ng proposed openstack/ironic stable/stein: Implements: Reactive HUAWEI ibmc driver https://review.opendev.org/723537 | 13:47 |
dtantsur | Qianbiao: 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 |
Qianbiao | yes i want to cherry pick to train. | 13:49 |
Qianbiao | hmm not sure.. | 13:49 |
dtantsur | Qianbiao: if "supported = False" was not in Stein, you don't need to backport | 13:49 |
Qianbiao | yes, i know. | 13:49 |
Qianbiao | I think i must have wrong operations. | 13:50 |
Qianbiao | oh.. i didnot check ibmc.py.. | 13:50 |
TheJulia | are we waiting on an actual patch to merge in sushy-tools and virtualbmc, or are we just thinking we should release them | 13:51 |
dtantsur | okay, I think I get it | 13:51 |
dtantsur | Qianbiao: the ibmc hardware type was only deprecated in Ussuri in 14.0. You don't need any backports. | 13:51 |
dtantsur | it was supported in Train and Stein officially | 13:51 |
iurygregory | only sushy-tools-tempest-ironic-partition-redfish-src let us merge something >.< | 13:52 |
Qianbiao | Is it so? i thought it's deprecated in train, will check now. | 13:52 |
dtantsur | Qianbiao: please double-check me, but it seems to be ussuri only | 13:52 |
iurygregory | don't look at only look at https://zuul.openstack.org/builds?job_name=sushy-tools-tempest-ironic-partition-redfish-src | 13:53 |
Qianbiao | <dtantsur> you are right. | 13:53 |
iurygregory | rpittau, you found anything about it? I think you were looking if I do remember | 13:53 |
* TheJulia wonders if there is a good reason to meet today | 13:54 | |
dtantsur | release readiness? | 13:55 |
* dtantsur -> tea, brb | 13:55 | |
*** cdearborn has joined #openstack-ironic | 13:55 | |
TheJulia | yeah | 13:56 |
*** mkrai has joined #openstack-ironic | 13:57 | |
TheJulia | so not quite our normal agenda then :) | 13:57 |
TheJulia | o/ mkrai | 13:57 |
rpittau | iurygregory: that's the issue with the non-existing mount point, it started failing when we changed teh default boot option to local | 13:58 |
iurygregory | oh we were using default_boot_option = netboot | 13:59 |
rpittau | yep | 13:59 |
iurygregory | we missed this job >.< | 13:59 |
rpittau | well not only that one | 13:59 |
iurygregory | do you have a patch up? | 13:59 |
rpittau | I was checking the jobs and preparing the patches | 13:59 |
rpittau | yeah | 13:59 |
TheJulia | :( | 13:59 |
rpittau | let me put that up now | 14:00 |
iurygregory | link pls | 14:00 |
rpittau | I'll send the rest later | 14:00 |
rpittau | hold on | 14:00 |
TheJulia | easy to miss them sadly with how long that change had been up :( | 14:00 |
iurygregory | yeah | 14:00 |
openstackgerrit | Riccardo Pittau proposed openstack/sushy-tools master: Restore default netboot boot option https://review.opendev.org/723546 | 14:01 |
rpittau | ^^ there you go | 14:01 |
iurygregory | tks! | 14:02 |
rpittau | the rest incoming | 14:04 |
openstackgerrit | Riccardo Pittau proposed openstack/python-ironicclient master: Restore default netboot boot option https://review.opendev.org/723547 | 14:04 |
*** mkrai has quit IRC | 14:05 | |
*** mkrai_ has joined #openstack-ironic | 14:05 | |
*** irclogbot_0 has joined #openstack-ironic | 14:08 | |
iurygregory | time for the 3rd coffee brb | 14:08 |
openstackgerrit | Riccardo Pittau proposed openstack/sushy master: Restore default netboot boot option https://review.opendev.org/723554 | 14:09 |
TheJulia | iurygregory: working on making your atoms vibrate? | 14:10 |
openstackgerrit | Riccardo Pittau proposed openstack/virtualbmc master: Restore default netboot boot option https://review.opendev.org/723555 | 14:10 |
rpittau | https://media.giphy.com/media/zWwngnPi6vWfe/giphy.gif | 14:12 |
*** irclogbot_0 has quit IRC | 14:12 | |
TheJulia | arne_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 untouched | 14:13 |
TheJulia | heh | 14:14 |
iurygregory | TheJulia, hehehehe yeah | 14:14 |
*** Mahendra22 has quit IRC | 14:15 | |
openstackgerrit | Riccardo Pittau proposed openstack/python-ironic-inspector-client master: Restore default netboot boot option https://review.opendev.org/723559 | 14:15 |
arne_wiebalck | TheJulia: 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 |
iurygregory | in 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 XD | 14:18 |
*** k_mouza has quit IRC | 14:18 | |
openstackgerrit | Derek Higgins proposed openstack/ironic master: WIP ipv6 jobs https://review.opendev.org/613169 | 14:18 |
iurygregory | and if I don't drink coffee I can easily have headache =( | 14:18 |
TheJulia | arne_wiebalck: that seems.. odd. UEFI variables should be runtime accessible | 14:21 |
TheJulia | caching to disk seems.. super odd | 14:21 |
TheJulia | At least for linux. I think OSX does this | 14:21 |
dtantsur | iurygregory: wow, that's serious | 14:22 |
dtantsur | I only drink tea nowadays | 14:22 |
iurygregory | dtantsur, yeah >.< | 14:22 |
*** irclogbot_1 has joined #openstack-ironic | 14:22 | |
dtantsur | headache without coffee may signify a low blood pressure. not sure which conclusions to make out of it | 14:22 |
iurygregory | result of 12hrs/day work to finish my master's =( | 14:22 |
* dtantsur has headache without enough carbohydrates | 14:23 | |
dtantsur | iurygregory: I hear you. Many of us lost some health there.. | 14:23 |
arne_wiebalck | TheJulia: what does runtime accessible mean here? The disk is not runtime accessible? | 14:23 |
iurygregory | dtantsur, yeah =( | 14:23 |
iurygregory | now time to play with grenade multinode \o/ | 14:24 |
dtantsur | that's a sophisticated toy to play with | 14:25 |
*** irclogbot_1 has quit IRC | 14:25 | |
openstackgerrit | OpenStack Release Bot proposed openstack/ironic-python-agent stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/723562 | 14:25 |
openstackgerrit | OpenStack Release Bot proposed openstack/ironic-python-agent stable/ussuri: Update TOX_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/723563 | 14:25 |
openstackgerrit | OpenStack Release Bot proposed openstack/ironic-python-agent master: Update master for stable/ussuri https://review.opendev.org/723564 | 14:25 |
openstackgerrit | OpenStack Release Bot proposed openstack/ironic-python-agent master: Add Python3 victoria unit tests https://review.opendev.org/723565 | 14:25 |
*** irclogbot_1 has joined #openstack-ironic | 14:26 | |
TheJulia | arne_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-ironic | 14:27 | |
arne_wiebalck | TheJulia: https://outflux.net/blog/archives/2018/04/19/uefi-booting-and-raid1/ | 14:27 |
*** irclogbot_1 has quit IRC | 14:29 | |
*** irclogbot_1 has joined #openstack-ironic | 14:30 | |
openstackgerrit | Merged openstack/ironic master: In-band deploy steps: correctly wipe driver_internal_info https://review.opendev.org/722916 | 14:32 |
openstackgerrit | Merged openstack/ironic-prometheus-exporter master: Add lower constraints jobs https://review.opendev.org/717684 | 14:32 |
dtantsur | getting closer! | 14:33 |
TheJulia | arne_wiebalck: ouch :( | 14:34 |
openstackgerrit | OpenStack Release Bot proposed openstack/ironic-inspector stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/723568 | 14:34 |
openstackgerrit | OpenStack Release Bot proposed openstack/ironic-inspector stable/ussuri: Update TOX_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/723569 | 14:34 |
openstackgerrit | OpenStack Release Bot proposed openstack/ironic-inspector master: Update master for stable/ussuri https://review.opendev.org/723570 | 14:34 |
openstackgerrit | OpenStack Release Bot proposed openstack/ironic-inspector master: Add Python3 victoria unit tests https://review.opendev.org/723571 | 14:34 |
*** irclogbot_1 has quit IRC | 14:35 | |
openstackgerrit | Merged openstack/python-ironic-inspector-client master: Document the new standalone CLI https://review.opendev.org/720034 | 14:36 |
*** irclogbot_2 has joined #openstack-ironic | 14:36 | |
*** irclogbot_2 has quit IRC | 14:39 | |
*** irclogbot_3 has joined #openstack-ironic | 14:40 | |
*** stendulker has joined #openstack-ironic | 14:44 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-prometheus-exporter master: Use hacking for pep8 tests https://review.opendev.org/717689 | 14:44 |
*** irclogbot_3 has quit IRC | 14:45 | |
*** irclogbot_3 has joined #openstack-ironic | 14:46 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-prometheus-exporter master: Use hacking for pep8 tests https://review.opendev.org/717689 | 14:47 |
*** iurygregory has quit IRC | 14:47 | |
*** iurygregory has joined #openstack-ironic | 14:48 | |
*** kaifeng_ has joined #openstack-ironic | 14:49 | |
*** beekneemech is now known as bnemec | 14:50 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-tempest-plugin master: Add standalone redfish jobs https://review.opendev.org/720675 | 14:51 |
*** irclogbot_3 has quit IRC | 14:51 | |
*** irclogbot_2 has joined #openstack-ironic | 14:52 | |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client stable/ussuri: Document the new standalone CLI https://review.opendev.org/723575 | 14:53 |
*** tkajinam has quit IRC | 14:54 | |
*** irclogbot_2 has quit IRC | 14:55 | |
*** irclogbot_2 has joined #openstack-ironic | 14:56 | |
*** irclogbot_2 has quit IRC | 14:59 | |
dtantsur | ouch, we should probably finished and merge https://review.opendev.org/#/c/720781 | 14:59 |
patchbot | patch 720781 - ironic - Don't break UEFI install with older IPAs - 1 patch set | 14:59 |
dtantsur | arne_wiebalck: and updated on ^^? | 14:59 |
*** irclogbot_3 has joined #openstack-ironic | 15:00 | |
iurygregory | o/ | 15:00 |
TheJulia | #startmeeting ironic | 15:00 |
TheJulia | o/ | 15:00 |
openstack | Meeting 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 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
dtantsur | o/ | 15:00 |
*** openstack changes topic to " (Meeting topic: ironic)" | 15:00 | |
openstack | The meeting name has been set to 'ironic' | 15:00 |
cdearborn | o/ | 15:00 |
erbarr | o/ | 15:00 |
arne_wiebalck | dtantsur: yes, we should merge this one and fix the other issue | 15:00 |
rpittau | o/ | 15:00 |
kaifeng_ | o/ | 15:00 |
* iurygregory was too fast | 15:00 | |
ajya | \o | 15:00 |
iurygregory | o/ | 15:00 |
arne_wiebalck | o/ | 15:00 |
rpioso | o/ | 15:01 |
arne_wiebalck | dtantsur: I think TheJulia was on it as well | 15:01 |
rloo | o/ | 15:01 |
stendulker | o/ | 15:01 |
TheJulia | dtantsur: I can always revise arne_wiebalck's patch after the meeting | 15:01 |
TheJulia | Good morning everyone! | 15:01 |
TheJulia | Our agenda can be found on the wiki, as always. | 15:02 |
TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:02 |
TheJulia | Hopefully this will go quickly :) | 15:02 |
mgoddard | \o | 15:02 |
TheJulia | Since I suspect we should skip status reports this week given it is basically release week | 15:02 |
TheJulia | well, last week was, but yeah. :) | 15:02 |
TheJulia | #topic Announcements / Reminders | 15:03 |
*** openstack changes topic to "Announcements / Reminders (Meeting topic: ironic)" | 15:03 | |
*** m1p has left #openstack-ironic | 15:03 | |
*** irclogbot_3 has quit IRC | 15:03 | |
TheJulia | #info Final Ussuri release is due May 13th, we should expect devstack to possibly break us at any moment. | 15:03 |
TheJulia | It seems like we're on track to be done maybe today or tomorrow? | 15:04 |
*** irclogbot_3 has joined #openstack-ironic | 15:04 | |
TheJulia | Seems we also need to register our PTG attendance response, which I believe I already took care of. | 15:04 |
TheJulia | Does anyone else have anything to announce or remind us of? | 15:05 |
iurygregory | TheJulia, do we have the slots defined? | 15:05 |
iurygregory | or this can be done after 28? | 15:05 |
TheJulia | iurygregory: we do not. I'm trying to remember if we got an etherpad going | 15:05 |
TheJulia | That can be done after the 28th as I understand it | 15:06 |
dtantsur | #link https://etherpad.opendev.org/p/Ironic-VictoriaPTG-Planning | 15:06 |
dtantsur | this ^^? | 15:06 |
iurygregory | http://lists.openstack.org/pipermail/openstack-discuss/2020-April/014423.html | 15:06 |
iurygregory | dtantsur, yup | 15:06 |
*** k_mouza has quit IRC | 15:06 | |
TheJulia | That works! | 15:06 |
*** irclogbot_3 has quit IRC | 15:07 | |
dtantsur | oh, one more thing | 15:07 |
dtantsur | If 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, etc | 15:08 |
*** irclogbot_0 has joined #openstack-ironic | 15:08 | |
dtantsur | I'd like to set up a call, ideally this week, to gather ideas and try to come up with a proposal | 15:08 |
iurygregory | ++ sounds good | 15:08 |
TheJulia | ++ | 15:08 |
dtantsur | #link https://doodle.com/poll/gtxzdhatabx8agy6 The doodle for release cadence discussion | 15:09 |
TheJulia | dtantsur: thanks! | 15:09 |
dtantsur | A small word of caution: this is not mandatory to attend. If you don't care much, you're welcome to skip it | 15:09 |
TheJulia | ++ | 15:09 |
TheJulia | Looks like we didn't have any action items from our last meeting, except maybe SPUC II :) | 15:10 |
iurygregory | it's done ^ | 15:10 |
iurygregory | http://lists.openstack.org/pipermail/openstack-discuss/2020-April/014410.html | 15:10 |
TheJulia | iurygregory: Is there an agreed upon date/time?? | 15:10 |
TheJulia | #link http://lists.openstack.org/pipermail/openstack-discuss/2020-April/014410.html | 15:10 |
iurygregory | TheJulia, so far only 5 people voted | 15:11 |
*** irclogbot_0 has quit IRC | 15:11 | |
TheJulia | Indeed :( | 15:11 |
TheJulia | #link https://doodle.com/poll/2q5zmv3g6uy2475e | 15:11 |
iurygregory | would be May 8 so far =) since we have holiday this week... | 15:11 |
dtantsur | yeah, May 1st is a holiday for a lot of people | 15:12 |
TheJulia | Anyway, 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-ironic | 15:12 | |
dtantsur | ++ | 15:12 |
TheJulia | #topic Deciding on priorities for the coming week | 15:12 |
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)" | 15:12 | |
TheJulia | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 15:13 |
*** k_mouza has joined #openstack-ironic | 15:13 | |
TheJulia | Starting at line 131 | 15:13 |
iurygregory | can we add https://review.opendev.org/#/c/703098/ ? =) | 15:14 |
patchbot | patch 703098 - ironic - Native zuulv3 grenade job for ironic - 15 patch sets | 15:14 |
dtantsur | CI changes are backportable | 15:14 |
dtantsur | JFYI | 15:14 |
iurygregory | gotcha | 15:15 |
TheJulia | I'd prefer to hold off until after we cut the branch and then just backport them | 15:15 |
*** irclogbot_2 has quit IRC | 15:15 | |
iurygregory | makes sense to me | 15:15 |
TheJulia | anyone have the links handy for the stuff we're waiting on to merge? | 15:16 |
*** irclogbot_0 has joined #openstack-ironic | 15:16 | |
iurygregory | nope =( | 15:17 |
dtantsur | I'm only aware of the patches that has already been added | 15:18 |
TheJulia | Okay, then we're in really good shape, ci permitting | 15:18 |
iurygregory | yup | 15:18 |
* TheJulia considers making an awful joke, but decides that it is a bad idea | 15:19 | |
* dtantsur considers which sacrifice to make to the CI gods | 15:19 | |
rpittau | all the ci fixes are in the gate, I *think* | 15:19 |
TheJulia | dtantsur: rubber chickens I believe | 15:19 |
*** irclogbot_0 has quit IRC | 15:19 | |
dtantsur | ++ | 15:19 |
TheJulia | Does anyone have things for after the stable branch is cut? | 15:19 |
rpittau | I'll recheck a bunch of stuff as soon as those are merged | 15:19 |
dtantsur | as to awful jokes, that's what we have twitter for, no? | 15:19 |
TheJulia | dtantsur: true | 15:20 |
TheJulia | rpioso: Excellent! Thanks! | 15:20 |
*** irclogbot_3 has joined #openstack-ironic | 15:20 | |
dtantsur | https://review.opendev.org/703778 is for post-branch mering | 15:20 |
patchbot | patch 703778 - ironic - Add RPC objects for deployment API - 10 patch sets | 15:20 |
iurygregory | if we have patches that have changes in l-c they are ok to backport ? | 15:20 |
TheJulia | l-c? | 15:21 |
iurygregory | lower-constraints | 15:21 |
iurygregory | =) | 15:21 |
dtantsur | iurygregory: l-c alone or also requirements? | 15:21 |
dtantsur | lower-constraints.txt is not, to my best knowledge, a production artefact of ours | 15:21 |
dtantsur | changing requirements.txt is a whole different story | 15:21 |
iurygregory | well some required changes in requirements | 15:21 |
iurygregory | e.g https://review.opendev.org/#/c/720790/4 | 15:22 |
patchbot | patch 720790 - ironic - Stop configuring install_command in tox. - 4 patch sets | 15:22 |
dtantsur | changing in requirements are only acceptable when we know for sure that the current lower bounds don't work at all | 15:22 |
dtantsur | * changes | 15:22 |
dtantsur | ** on stable branches | 15:22 |
iurygregory | I only changed because the job said it was wrong XD | 15:22 |
*** irclogbot_3 has quit IRC | 15:23 | |
TheJulia | So, I suspect we should merge that prior to releasing ironic. We've already released basically everything else except sushy-tools/virtualbmc afaik | 15:23 |
dtantsur | I've approved teh change | 15:23 |
TheJulia | so if the gods of CI are friendly, ~4 hours | 15:23 |
* TheJulia can live with that | 15:24 | |
dtantsur | It would be a contentious change to backport | 15:24 |
TheJulia | Yeah | 15:24 |
dtantsur | TheJulia: with release mapping already in place, we can probably survive devstack and grenade branching | 15:24 |
*** irclogbot_2 has joined #openstack-ironic | 15:24 | |
TheJulia | dtantsur: if our branch is not there, we won't | 15:24 |
TheJulia | grenade will break once the mappings are changed | 15:24 |
dtantsur | I *think* it works nowadays, although in a weird fashion | 15:24 |
dtantsur | grenade falls back to master on unknown branches | 15:25 |
TheJulia | oh! then that is good! | 15:25 |
dtantsur | so it will be master->master testing | 15:25 |
TheJulia | okay | 15:25 |
iurygregory | yeah | 15:25 |
dtantsur | which is bad, but fine if only for a brief momemnt | 15:25 |
TheJulia | I'm glad that has been changed | 15:25 |
dtantsur | TheJulia: I *think* it has been the case for a while | 15:25 |
TheJulia | Is there anything else for the release? | 15:25 |
dtantsur | what used to break us is missing release mapping on our side | 15:25 |
dtantsur | I may be terribly wrong, of course, consult your lawyer before trusting me | 15:25 |
TheJulia | dtantsur: eh, I guess it was ~3 cycles ago that we had such issues last | 15:26 |
TheJulia | I remember two cycles ago, we were early and were like "why has this not been changed yet?" | 15:26 |
TheJulia | Anyway! | 15:26 |
* dtantsur also suspects the release team will wait a reasonable amount of time | 15:26 | |
TheJulia | I think we're good to move on | 15:26 |
dtantsur | ++ | 15:26 |
iurygregory | ++ | 15:26 |
TheJulia | We seem to have no discussion topics, so we could stop by Baremetal SIG | 15:27 |
TheJulia | arne_wiebalck: ^^^ | 15:27 |
TheJulia | or we could just go directly to Open Discussion | 15:27 |
rloo | i believe dtantsur is correct wrt grenade | 15:27 |
Qianbiao | Hello, 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_wiebalck | nothing to see here either | 15:27 |
*** irclogbot_2 has quit IRC | 15:27 | |
dtantsur | Qianbiao: features are generally not backported, sorry | 15:27 |
TheJulia | Pass go, collect two code reviews and a get out of discussion free card! (Making a monopoly game reference) | 15:27 |
iurygregory | nice reference | 15:28 |
*** irclogbot_2 has joined #openstack-ironic | 15:28 | |
TheJulia | Qianbiao: We can make an early Victoria cycle release once it merges | 15:28 |
*** mkrai_ has quit IRC | 15:28 | |
Qianbiao | dtantsur any chance that i commit it now? | 15:28 |
TheJulia | It is not unheard of us to do such. :) | 15:28 |
Qianbiao | TheJulia ok. | 15:28 |
openstackgerrit | Merged openstack/python-ironic-inspector-client stable/ussuri: Document the new standalone CLI https://review.opendev.org/723575 | 15:29 |
TheJulia | Well, with arne_wiebalck having chimed in, lets go to Open Discussion | 15:29 |
TheJulia | #topic Open Discussion | 15:29 |
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)" | 15:29 | |
TheJulia | Hopefully everyone had a wonderful weekend! | 15:29 |
dtantsur | They become shorter and shorter, these weekends | 15:30 |
dtantsur | has anyone else noticed? | 15:30 |
iurygregory | nope .-. | 15:30 |
rpittau | dtantsur: kind of, yeah :/ | 15:30 |
TheJulia | They have :( | 15:31 |
*** irclogbot_2 has quit IRC | 15:31 | |
kaifeng_ | anyone noticed the weird issue in the ML wrt the NCs and ICs? | 15:32 |
TheJulia | kaifeng_: NCs? ICs? I'm guessing ML is mailing list? | 15:32 |
dtantsur | I'm not even sure what NC and IC are.. | 15:32 |
kaifeng_ | heh, actually i am not the inventor, NC = nova compute, IC = ironic conductor | 15:33 |
TheJulia | I've not seen anything, but pulling up the ml now | 15:34 |
* dtantsur used to n-cpu and ir-cond | 15:34 | |
kaifeng_ | http://lists.openstack.org/pipermail/openstack-discuss/2020-April/014337.html | 15:34 |
dtantsur | I 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 host | 15:35 |
dtantsur | take over takes some time | 15:36 |
kaifeng_ | i am not sure how that can happen, except there is a take over, but this is not mentioned in the text | 15:36 |
arne_wiebalck | or does not work at all | 15:36 |
dtantsur | lol | 15:36 |
dtantsur | this is.. concerning | 15:37 |
iurygregory | lol | 15:37 |
rpittau | oO | 15:37 |
iurygregory | would be a problem | 15:37 |
dtantsur | can we put CI testing of take over on our backlog please? | 15:37 |
dtantsur | I recall fixing a lot of bugs with it at some point | 15:37 |
iurygregory | ++ | 15:37 |
*** irclogbot_3 has joined #openstack-ironic | 15:37 | |
arne_wiebalck | I meant take over in Ironic and Nova, since he was changing both. | 15:37 |
TheJulia | yeah, we've had to fix a number of things. I suspect we're missing some context. | 15:38 |
iurygregory | ouch | 15:38 |
iurygregory | >.< nova | 15:38 |
arne_wiebalck | We stopped doing this. | 15:38 |
dtantsur | I don't think the nova's take over can affect the deploy process | 15:38 |
dtantsur | it's only about resource tracker updates, everything else is re-balanced when going through ir-api | 15:38 |
kaifeng_ | because we will utilize mutiple n-cpu and ir-cond in production, this issue seems suspicious | 15:39 |
arne_wiebalck | if multiple compute nodes claim an overlapping set of nodes, there will be a fine mess | 15:40 |
TheJulia | I think they may be expecting the hash rings in both ironic and nova to be calculated exactly the same way | 15:40 |
dtantsur | it's a fair (and wrong) expectation | 15:40 |
TheJulia | indeed | 15:40 |
dtantsur | but I don't see how a discrepancy between the two hash rings could cause a node to be handled by two conductors | 15:40 |
dtantsur | but if they try to deploy while take over is in progress.. | 15:41 |
dtantsur | I don't think anybody has tried that. All bets are off. | 15:41 |
TheJulia | this is true. The two minutes in between the actions could be a lot of activity | 15:41 |
*** Qianbiao is now known as Qianbiao|afk | 15:42 | |
*** Qianbiao|afk has quit IRC | 15:42 | |
arne_wiebalck | I 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_wiebalck | FWIU, the given nova-compute did not let go :) | 15:43 |
arne_wiebalck | giving | 15:43 |
TheJulia | I think we're going to need more in the way of logs to really understand what is occuring | 15:43 |
TheJulia | it feels like there is more than one issue :\ | 15:44 |
kaifeng_ | indeed, just the "unstable" scares | 15:44 |
arne_wiebalck | "simple" take over works fine, we have added and removed Ironic controllers several times | 15:45 |
openstackgerrit | Christopher Dearborn proposed openstack/ironic master: Fix supported sushy-oem-idrac version https://review.opendev.org/723588 | 15:45 |
arne_wiebalck | "simple" == I haven't tried to do that while mass-instantiating nodes | 15:45 |
TheJulia | I guess n-cpu and ir-cond is running on the same machine too | 15:45 |
TheJulia | I guess the question becomes can ir-cond be scaled first, and then api/n-cpu | 15:46 |
arne_wiebalck | we have these on different nodes | 15:46 |
* TheJulia wonders if this is also the latest stable queens or an older stable/queens release | 15:46 | |
TheJulia | I seem to remember there were some queens timeframe fixes | 15:46 |
dtantsur | not unlikely | 15:47 |
dtantsur | oh, https://review.opendev.org/#/c/723588/ should ideally merge before branching | 15:48 |
patchbot | patch 723588 - ironic - Fix supported sushy-oem-idrac version - 1 patch set | 15:48 |
* dtantsur has just noticed | 15:48 | |
rpittau | ^ I was going to say :) | 15:48 |
dtantsur | 3 minutes to approval, I bet the fasted cdearborn has observed :D | 15:49 |
rpittau | this has been rebased also https://review.opendev.org/717689 | 15:49 |
patchbot | patch 717689 - ironic-prometheus-exporter - Use hacking for pep8 tests - 7 patch sets | 15:49 |
* cdearborn almost fell out of my chair! | 15:49 | |
* rpioso did :-) | 15:49 | |
dtantsur | :D | 15:50 |
dtantsur | iurygregory: https://review.opendev.org/#/c/720793/ seemingly needs a rebase? | 15:50 |
patchbot | patch 720793 - ironic-prometheus-exporter - Stop configuring install_command in tox. - 4 patch sets | 15:50 |
iurygregory | dtantsur, yeah just noticed | 15:50 |
TheJulia | ugh. \o/ | 15:50 |
iurygregory | I thought rpittau had everything done so I used his patch as base XD | 15:51 |
TheJulia | Does anyone want to respond to the hash ring thread on the mailing list? | 15:51 |
dtantsur | I responded with a couple of comments just to revive it | 15:51 |
TheJulia | okay | 15:51 |
dtantsur | we may need to ask for logs, not sure how doable it is | 15:51 |
TheJulia | Yeah :\ | 15:52 |
TheJulia | is there anything else to discuss today? | 15:52 |
rpittau | not strictly related to release, but TheJulia when you have a minute I think this can be merged now https://review.opendev.org/708827 | 15:53 |
patchbot | patch 708827 - ironic-python-agent (stable/train) - Rescan after restarting the md device - 2 patch sets | 15:53 |
*** sshnaidm is now known as sshnaidm|afk | 15:53 | |
dtantsur | re that release cadence doodle: please vote by ~ noon UTC tomorrow, as the most likely candidate now is on Wednesday | 15:54 |
TheJulia | rpittau: thanks | 15:54 |
TheJulia | dtantsur: Thanks! | 15:55 |
TheJulia | Thanks everyone! | 15:55 |
dtantsur | thank you TheJulia | 15:56 |
rpittau | thank you TheJulia :) | 15:56 |
*** ociuhandu has quit IRC | 15:56 | |
TheJulia | Have a wonderful week everyone! | 15:56 |
TheJulia | #endmeeting | 15: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 | |
openstack | Meeting ended Mon Apr 27 15:57:01 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:57 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-04-27-15.00.html | 15:57 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-04-27-15.00.txt | 15:57 |
openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-04-27-15.00.log.html | 15:57 |
kaifeng_ | thank you TheJulia o/ | 15:57 |
*** lucasagomes has quit IRC | 15:57 | |
*** dsneddon has joined #openstack-ironic | 15:59 | |
*** ociuhandu has joined #openstack-ironic | 16:00 | |
openstackgerrit | Merged openstack/sushy-tools master: Restore default netboot boot option https://review.opendev.org/723546 | 16:00 |
TheJulia | so I think arne_wiebalck's change just needs a rebase on top of the fix raising an exception | 16:06 |
rpittau | have a great week everyone, talk tomorrow! o/ | 16:07 |
*** rpittau is now known as rpittau|afk | 16:07 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Don't break UEFI install with older IPAs https://review.opendev.org/720781 | 16:09 |
TheJulia | dtantsur: I don't think your -1 applies to ^^ anymore since we should now be raising an exception | 16:10 |
arne_wiebalck | TheJulia: 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 fingers | 16:12 | |
TheJulia | arne_wiebalck: TypeError | 16:12 |
TheJulia | it looks for the text, not the status code since the 500 will get returned in the error status code for any error | 16:12 |
TheJulia | so more a "we know typeerror will indicate failure we know what the issue is, so we can handle it somewhat gracefully | 16:13 |
dtantsur | TheJulia: yep, removed, will re-review in a bit | 16:14 |
TheJulia | k | 16:14 |
arne_wiebalck | so, there will be a TypeError when in the unknow param situation? | 16:14 |
arne_wiebalck | unknown | 16:14 |
TheJulia | yes | 16:14 |
arne_wiebalck | ok | 16:15 |
arne_wiebalck | you just pushed this? | 16:15 |
arne_wiebalck | or do you want me to do it? | 16:15 |
TheJulia | arne_wiebalck: already rebased your change | 16:15 |
arne_wiebalck | cool | 16:16 |
arne_wiebalck | thanks | 16:16 |
arne_wiebalck | ah, but the exception catch still needs change I see | 16:17 |
*** iurygregory has quit IRC | 16:17 | |
TheJulia | arne_wiebalck: I don't think so | 16:17 |
* arne_wiebalck is confused | 16:17 | |
TheJulia | we still emit the same exception that your patch was intended to use | 16:18 |
dtantsur | mmm, my internet is gone, but IRC still works? | 16:18 |
* arne_wiebalck should go and the read the code ... | 16:19 | |
TheJulia | dtantsur: magic?!? | 16:19 |
dtantsur | right? | 16:19 |
arne_wiebalck | that helps | 16:19 |
arne_wiebalck | ok, I understand now, sorry | 16:19 |
TheJulia | Has 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 IRC | 16:20 | |
dtantsur | I cannot ping anything outside of Germany, as it seems | 16:20 |
* dtantsur hasn't and won't be able to right now :( | 16:20 | |
dtantsur | oh, wait, hpe.com does open | 16:20 |
*** ociuhandu has joined #openstack-ironic | 16:21 | |
*** k_mouza has quit IRC | 16:21 | |
arne_wiebalck | The technical white paper: https://psnow.ext.hpe.com/doc/a50000782enw | 16:24 |
dtantsur | mm, no, this subdomain won't open for me | 16:25 |
*** dsneddon has quit IRC | 16:26 | |
*** alexmcleod has quit IRC | 16:26 | |
*** vesper11 has quit IRC | 16:26 | |
*** hoonetorg has quit IRC | 16:26 | |
*** mjturek has quit IRC | 16:26 | |
*** Blinkiz has quit IRC | 16:26 | |
*** tonyb has quit IRC | 16:26 | |
openstackgerrit | Merged openstack/sushy-tools master: Fix libvirt test for Python 3.8 https://review.opendev.org/722815 | 16:26 |
*** irclogbot_3 has quit IRC | 16:28 | |
*** k_mouza has joined #openstack-ironic | 16:29 | |
*** vesper11 has joined #openstack-ironic | 16:29 | |
*** irclogbot_0 has joined #openstack-ironic | 16:29 | |
TheJulia | looks like a distributed redfish wrapper being developed and will be released to LF during Q2 according to what I've read | 16:31 |
*** hoonetorg has joined #openstack-ironic | 16:31 | |
openstackgerrit | Merged openstack/virtualbmc master: Restore default netboot boot option https://review.opendev.org/723555 | 16:31 |
dtantsur | mm, I'd say this idea has been floating on the surface | 16:31 |
*** dsneddon has joined #openstack-ironic | 16:31 | |
TheJulia | for some time | 16:32 |
openstackgerrit | Merged openstack/sushy master: Restore default netboot boot option https://review.opendev.org/723554 | 16:32 |
TheJulia | with an aggregator plugin system so you can use it for IPMI systems | 16:32 |
*** mjturek has joined #openstack-ironic | 16:32 | |
*** Blinkiz has joined #openstack-ironic | 16:32 | |
*** tonyb has joined #openstack-ironic | 16:32 | |
TheJulia | it feels like it is intended to be a consistency layer, but I checked DMTF's wips and not seeing anything that seems applicable | 16:33 |
openstackgerrit | Noor Muhammad Malik proposed openstack/ironic master: Add PXE reset known issue to the docs https://review.opendev.org/722555 | 16:35 |
openstackgerrit | Derek Higgins proposed openstack/ironic master: WIP ipv6 jobs https://review.opendev.org/613169 | 16:37 |
*** stendulker has quit IRC | 16:38 | |
openstackgerrit | Verification of a change to openstack/ironic-inspector failed: No need to import print function anymore https://review.opendev.org/721576 | 16:39 |
*** alexmcleod has joined #openstack-ironic | 16:46 | |
openstackgerrit | Merged openstack/python-ironic-inspector-client master: Restore default netboot boot option https://review.opendev.org/723559 | 16:47 |
*** k_mouza has quit IRC | 16:53 | |
arne_wiebalck | bye everyone o/ | 16:57 |
*** k_mouza has joined #openstack-ironic | 16:57 | |
*** derekh has quit IRC | 17:03 | |
*** ociuhandu has quit IRC | 17:06 | |
*** ociuhandu has joined #openstack-ironic | 17:06 | |
*** k_mouza has quit IRC | 17:11 | |
*** ociuhandu has quit IRC | 17:12 | |
openstackgerrit | Noor Muhammad Malik proposed openstack/ironic master: Add PXE reset known issue to the docs https://review.opendev.org/722555 | 17:12 |
*** k_mouza has joined #openstack-ironic | 17:13 | |
*** k_mouza has quit IRC | 17:13 | |
*** dtantsur is now known as dtantsur|afk | 17:18 | |
dtantsur|afk | g'night | 17:18 |
*** yolanda has quit IRC | 17:18 | |
*** mjturek has quit IRC | 17:26 | |
*** Blinkiz has quit IRC | 17:26 | |
*** tonyb has quit IRC | 17:26 | |
*** alexmcleod has quit IRC | 17:26 | |
*** dsneddon has quit IRC | 17:26 | |
*** alexmcleod has joined #openstack-ironic | 17:27 | |
*** mjturek has joined #openstack-ironic | 17:29 | |
*** Blinkiz has joined #openstack-ironic | 17:29 | |
*** tonyb has joined #openstack-ironic | 17:29 | |
*** dsneddon has joined #openstack-ironic | 17:29 | |
*** ociuhandu has joined #openstack-ironic | 17:42 | |
*** hoonetorg has quit IRC | 18:02 | |
*** kaifeng_ has quit IRC | 18:09 | |
*** cdearborn has quit IRC | 18:12 | |
*** dmellado has quit IRC | 18:17 | |
*** ociuhandu has quit IRC | 18:20 | |
*** ociuhandu has joined #openstack-ironic | 18:20 | |
*** alexmcleod has quit IRC | 18:21 | |
*** dmellado has joined #openstack-ironic | 18:24 | |
*** dmellado has quit IRC | 18:25 | |
*** alexmcleod has joined #openstack-ironic | 18:28 | |
*** dmellado has joined #openstack-ironic | 18:33 | |
*** iurygregory has joined #openstack-ironic | 18:34 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-prometheus-exporter master: Stop configuring install_command in tox. https://review.opendev.org/720793 | 18:42 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Increase timeout for zuulv3 grenade job https://review.opendev.org/723646 | 18:51 |
openstackgerrit | Merged openstack/ironic master: Log when IPA fallback occurs on bootloader install https://review.opendev.org/721988 | 18:52 |
openstackgerrit | Merged openstack/ironic master: Fix agent_client handling of embedded errors https://review.opendev.org/722027 | 18:52 |
openstackgerrit | Merged openstack/ironic-python-agent master: Update master for stable/ussuri https://review.opendev.org/723564 | 18:52 |
openstackgerrit | Merged openstack/ironic-python-agent stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/723562 | 18:52 |
openstackgerrit | Merged openstack/ironic-python-agent stable/ussuri: Update TOX_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/723563 | 18:52 |
iurygregory | I'm loving the TIMED_OUT in ironic-inspector-grenade -.-' | 18:59 |
iurygregory | and 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 IRC | 19:03 | |
*** ociuhandu has joined #openstack-ironic | 19:06 | |
*** ociuhandu has quit IRC | 19:20 | |
*** ociuhandu has joined #openstack-ironic | 19:27 | |
*** Lucas_Gray has joined #openstack-ironic | 19:41 | |
*** Lucas_Gray has quit IRC | 19:42 | |
*** jdandrea has quit IRC | 19:45 | |
TheJulia | iurygregory: how long is it taking? | 19:46 |
*** Lucas_Gray has joined #openstack-ironic | 19:51 | |
*** rloo has quit IRC | 19:55 | |
*** rloo has joined #openstack-ironic | 19:56 | |
*** jdandrea has joined #openstack-ironic | 19:56 | |
*** rloo has quit IRC | 19:57 | |
openstackgerrit | Verification of a change to openstack/ironic-inspector failed: Update .gitreview for stable/ussuri https://review.opendev.org/723568 | 19:57 |
openstackgerrit | Verification of a change to openstack/ironic-inspector failed: Update TOX_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/723569 | 19:57 |
*** rloo has joined #openstack-ironic | 19:57 | |
*** rloo has quit IRC | 20:00 | |
*** rloo has joined #openstack-ironic | 20:01 | |
openstackgerrit | Merged openstack/sushy-tools master: Add pep8 import order check https://review.opendev.org/721650 | 20:09 |
openstackgerrit | Merged openstack/ironic-inspector master: Update master for stable/ussuri https://review.opendev.org/723570 | 20:11 |
*** k_mouza has joined #openstack-ironic | 20:14 | |
*** jdandrea has quit IRC | 20:15 | |
*** alexmcleod has quit IRC | 20:18 | |
*** jdandrea has joined #openstack-ironic | 20:19 | |
*** k_mouza has quit IRC | 20:19 | |
*** alexmcleod has joined #openstack-ironic | 20:27 | |
*** ociuhandu has quit IRC | 20:57 | |
*** ociuhandu has joined #openstack-ironic | 20:58 | |
openstackgerrit | Merged openstack/sushy master: Add import order check https://review.opendev.org/721649 | 20:58 |
openstackgerrit | Merged openstack/python-ironicclient master: Restore default netboot boot option https://review.opendev.org/723547 | 21:04 |
*** Goneri has quit IRC | 21:05 | |
*** igordc has joined #openstack-ironic | 21:07 | |
*** ociuhandu has quit IRC | 21:11 | |
*** cdearborn has joined #openstack-ironic | 21:14 | |
*** rcernin has joined #openstack-ironic | 21:14 | |
*** ociuhandu has joined #openstack-ironic | 21:19 | |
*** rloo has quit IRC | 21:29 | |
*** ociuhandu has quit IRC | 21:29 | |
*** ociuhandu has joined #openstack-ironic | 21:39 | |
*** rloo has joined #openstack-ironic | 21:44 | |
*** rloo has quit IRC | 21:45 | |
*** rloo has joined #openstack-ironic | 21:45 | |
*** ociuhandu has quit IRC | 21:49 | |
openstackgerrit | Merged openstack/ironic master: Stop configuring install_command in tox. https://review.opendev.org/720790 | 22:14 |
openstackgerrit | Merged openstack/ironic-prometheus-exporter master: Use hacking for pep8 tests https://review.opendev.org/717689 | 22:14 |
*** ociuhandu has joined #openstack-ironic | 22:18 | |
*** alexmcleod has quit IRC | 22:20 | |
*** Lucas_Gray has quit IRC | 22:21 | |
*** alexmcleod has joined #openstack-ironic | 22:27 | |
*** ociuhandu has quit IRC | 22:29 | |
*** yaawang has joined #openstack-ironic | 22:32 | |
*** yaawang_ has quit IRC | 22:33 | |
*** tkajinam has joined #openstack-ironic | 22:49 | |
*** tkajinam has quit IRC | 22:49 | |
*** tkajinam has joined #openstack-ironic | 22:50 | |
*** rloo has quit IRC | 22:51 | |
openstackgerrit | Merged openstack/ironic-inspector master: Upgrade flake8-import-order version to 0.17.1 https://review.opendev.org/721648 | 22:53 |
openstackgerrit | Merged openstack/ironic master: Fix supported sushy-oem-idrac version https://review.opendev.org/723588 | 22:53 |
openstackgerrit | Merged openstack/ironic-python-agent master: Add Python3 victoria unit tests https://review.opendev.org/723565 | 22:53 |
openstackgerrit | Merged openstack/ironic master: Implements: Reactive HUAWEI ibmc driver https://review.opendev.org/723225 | 22:53 |
openstackgerrit | Verification of a change to openstack/ironic failed: Don't break UEFI install with older IPAs https://review.opendev.org/720781 | 22:54 |
*** k_mouza has joined #openstack-ironic | 23:00 | |
*** k_mouza has quit IRC | 23:04 | |
*** alexmcleod_ has joined #openstack-ironic | 23:12 | |
*** alexmcleod has quit IRC | 23:14 | |
*** igordc has quit IRC | 23:14 | |
*** k_mouza has joined #openstack-ironic | 23:50 | |
*** k_mouza has quit IRC | 23:54 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!