| *** dsneddon has quit IRC | 00:03 | |
| *** Goneri has quit IRC | 00:06 | |
| *** rcernin has quit IRC | 00:30 | |
| *** dsneddon has joined #openstack-ironic | 00:33 | |
| *** dsneddon has quit IRC | 00:38 | |
| *** kaifeng has joined #openstack-ironic | 00:51 | |
| *** dsneddon has joined #openstack-ironic | 01:09 | |
| *** dsneddon has quit IRC | 01:14 | |
| *** dsneddon has joined #openstack-ironic | 01:45 | |
| *** rcernin has joined #openstack-ironic | 01:45 | |
| *** dsneddon has quit IRC | 01:49 | |
| *** gregoryo has joined #openstack-ironic | 01:50 | |
| *** dsneddon has joined #openstack-ironic | 02:23 | |
| *** dsneddon has quit IRC | 02:28 | |
| *** dsneddon has joined #openstack-ironic | 03:01 | |
| *** dsneddon has quit IRC | 03:06 | |
| *** mkrai has joined #openstack-ironic | 03:18 | |
| *** rh-jelabarre has joined #openstack-ironic | 03:27 | |
| *** dsneddon has joined #openstack-ironic | 03:40 | |
| *** dsneddon has quit IRC | 03:45 | |
| *** gkadam has joined #openstack-ironic | 03:57 | |
| *** pradip_k has joined #openstack-ironic | 04:15 | |
| *** dsneddon has joined #openstack-ironic | 04:16 | |
| *** dsneddon has quit IRC | 04:21 | |
| *** mroh has joined #openstack-ironic | 04:37 | |
| *** dsneddon has joined #openstack-ironic | 04:55 | |
| *** pradip_k has quit IRC | 05:00 | |
| *** dsneddon has quit IRC | 05:00 | |
| *** ricolin has joined #openstack-ironic | 05:08 | |
| *** yaawang has quit IRC | 05:22 | |
| *** yaawang has joined #openstack-ironic | 05:24 | |
| *** dsneddon has joined #openstack-ironic | 05:31 | |
| openstackgerrit | M V P Nitesh proposed openstack/ironic-lib master: Returns root_uuid and efi_sys_partuuid for UEFI boot mode nodes https://review.opendev.org/669830 | 05:32 |
|---|---|---|
| *** dsneddon has quit IRC | 05:36 | |
| *** pradip_k has joined #openstack-ironic | 05:46 | |
| *** jtomasek has joined #openstack-ironic | 06:02 | |
| *** e0ne has joined #openstack-ironic | 06:09 | |
| *** dsneddon has joined #openstack-ironic | 06:10 | |
| *** dsneddon has quit IRC | 06:15 | |
| *** e0ne has quit IRC | 06:18 | |
| *** jtomasek has quit IRC | 06:22 | |
| arne_wiebalck | Good morning, ironic! | 06:38 |
| *** jtomasek has joined #openstack-ironic | 06:40 | |
| *** rachit7 has joined #openstack-ironic | 06:41 | |
| *** dsneddon has joined #openstack-ironic | 06:46 | |
| *** dsneddon has quit IRC | 06:50 | |
| *** yaawang has quit IRC | 06:51 | |
| *** rcernin has quit IRC | 07:02 | |
| *** jawad_axd has joined #openstack-ironic | 07:09 | |
| kaifeng | morning arne_wiebalck o/ | 07:09 |
| *** trident has quit IRC | 07:10 | |
| arne_wiebalck | hey kaifeng o/ | 07:10 |
| *** tesseract has joined #openstack-ironic | 07:13 | |
| *** dsneddon has joined #openstack-ironic | 07:18 | |
| *** yaawang has joined #openstack-ironic | 07:18 | |
| *** hamzy_ has quit IRC | 07:19 | |
| *** trident has joined #openstack-ironic | 07:21 | |
| *** dsneddon has quit IRC | 07:23 | |
| openstackgerrit | Ankit Kumar proposed openstack/ironic master: DNM Patch : Testing the HPEProliant CI https://review.opendev.org/534137 | 07:26 |
| *** hamzy_ has joined #openstack-ironic | 07:29 | |
| *** bobmel has joined #openstack-ironic | 07:43 | |
| *** pradip_k has quit IRC | 07:46 | |
| *** pcaruana has joined #openstack-ironic | 07:49 | |
| *** dsneddon has joined #openstack-ironic | 07:53 | |
| *** rpittau|afk is now known as rpittau | 07:55 | |
| rpittau | good morning ironic! o/ | 07:55 |
| * rpittau is back | 07:55 | |
| kaifeng | hey rpittau o/ | 07:58 |
| *** dsneddon has quit IRC | 07:58 | |
| rpittau | hi kaifeng :) | 07:58 |
| rachit7 | It seems zull checks is not working | 07:59 |
| kaifeng | rachit7: there is a ironic job has run 6 hours, don't know why there is a "3. attempt" there, there is a similar case in a ipa gate job, seems related with ironic-standalone | 08:07 |
| *** e0ne has joined #openstack-ironic | 08:19 | |
| *** alexmcleod has joined #openstack-ironic | 08:20 | |
| openstackgerrit | raphael.glon proposed openstack/ironic master: Minor: change a misleading InvalidState error message https://review.opendev.org/680687 | 08:25 |
| *** dsneddon has joined #openstack-ironic | 08:25 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Remove support for CoreOS images https://review.opendev.org/679064 | 08:33 |
| openstackgerrit | Merged openstack/ironic-inspector master: Adds job to run inspector in non-standalone mode https://review.opendev.org/675030 | 08:33 |
| *** dsneddon has quit IRC | 08:33 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Update non-standalone job to use uwsgi https://review.opendev.org/675724 | 08:33 |
| *** stendulker has joined #openstack-ironic | 08:37 | |
| *** kaifeng has quit IRC | 08:39 | |
| *** kaifeng has joined #openstack-ironic | 08:40 | |
| *** derekh has joined #openstack-ironic | 08:43 | |
| *** dougsz has joined #openstack-ironic | 08:44 | |
| *** bobmel has quit IRC | 08:44 | |
| *** bobmel has joined #openstack-ironic | 08:45 | |
| *** gregoryo has quit IRC | 08:48 | |
| mgoddard | stendulker: hi | 08:50 |
| stendulker | mgoddard: Hi | 08:51 |
| mgoddard | Thanks for finishing some of my deploy steps work :) | 08:51 |
| mgoddard | I'm now testing the Dell patch and found a problem | 08:51 |
| stendulker | oh... What is it? | 08:52 |
| mgoddard | stendulker: it's this: https://review.opendev.org/#/c/676152/2/ironic/drivers/modules/agent_base_vendor.py | 08:52 |
| patchbot | patch 676152 - ironic (stable/stein) - Asynchronous out of band deploy steps fails to exe... (MERGED) - 2 patch sets | 08:52 |
| mgoddard | for steps that use a periodic job to poll for completion (like idrac), the heartbeat now resumes too early | 08:53 |
| mgoddard | I could fix that with another flag | 08:53 |
| mgoddard | but there was another problem when the deploy was resumed in that way | 08:54 |
| mgoddard | http://paste.openstack.org/show/773975/ | 08:54 |
| stendulker | why image related error? | 08:55 |
| mgoddard | I can download images normally - a deploy without a RAID step is successful | 08:55 |
| mgoddard | I think it might be due to the context, because the request comes from an agent heartbeat rather than a user request | 08:56 |
| stendulker | but for async steps it worked properly | 08:57 |
| stendulker | have tested with both redfish and ilo drivers | 08:57 |
| mgoddard | using nova? | 08:58 |
| stendulker | not with nova, in standalone | 08:59 |
| mgoddard | and keystone | 08:59 |
| mgoddard | ok, I think that's the difference | 08:59 |
| mgoddard | noauth | 08:59 |
| stendulker | why it should matter at this later stage. The required data is available in instance_info | 09:01 |
| rachit7 | kaifeng : Observing since last week | 09:02 |
| mgoddard | stendulker: it's downloading the image | 09:03 |
| *** dsneddon has joined #openstack-ironic | 09:04 | |
| stendulker | but error says, its not able to find image, not auth failure... | 09:04 |
| stendulker | or may be message is generic one here | 09:05 |
| *** bobmel has quit IRC | 09:05 | |
| *** bobmel has joined #openstack-ironic | 09:06 | |
| *** hamzy_ has quit IRC | 09:07 | |
| mgoddard | stendulker: could be that the ramdisk's admin token does not have access to the glance image | 09:07 |
| *** ociuhandu has joined #openstack-ironic | 09:08 | |
| stendulker | mgoddard: ok | 09:08 |
| *** dsneddon has quit IRC | 09:09 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client stable/train: Update .gitreview for stable/train https://review.opendev.org/680825 | 09:09 |
| *** ociuhandu has quit IRC | 09:10 | |
| *** ociuhandu has joined #openstack-ironic | 09:10 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client stable/train: Update TOX/UPPER_CONSTRAINTS_FILE for stable/train https://review.opendev.org/680826 | 09:10 |
| openstackgerrit | Merged openstack/python-ironic-inspector-client master: Update master for stable/train https://review.opendev.org/680827 | 09:10 |
| mgoddard | stendulker: maybe we need a context.get_admin_context()? | 09:11 |
| *** ociuhandu has quit IRC | 09:14 | |
| *** ociuhandu_ has joined #openstack-ironic | 09:14 | |
| stendulker | mgoddard: where do we do that? | 09:16 |
| mgoddard | stendulker: good question | 09:16 |
| *** ociuhandu_ has quit IRC | 09:18 | |
| mgoddard | stendulker: possibly in ironic/api/middleware/auth_token.py, or ironic/api/controllers/v1/ramdisk.py | 09:19 |
| mgoddard | stendulker: do you have an environment to test with nova enabled? | 09:20 |
| stendulker | unfortunately no... | 09:20 |
| *** tssurya has joined #openstack-ironic | 09:22 | |
| mgoddard | stendulker: ok, let me try get_admin_context | 09:23 |
| stendulker | mgoddard: ok, thank you. | 09:24 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic-prometheus-exporter master: Add Redfish metrics support https://review.opendev.org/680731 | 09:24 |
| *** ociuhandu has joined #openstack-ironic | 09:27 | |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: fix emulator: remove image on eject and update devices https://review.opendev.org/677449 | 09:29 |
| *** kaifeng has quit IRC | 09:35 | |
| *** stendulker has quit IRC | 09:36 | |
| *** dsneddon has joined #openstack-ironic | 09:40 | |
| *** dsneddon has quit IRC | 09:45 | |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Store virtual media device changes persistently https://review.opendev.org/677449 | 09:47 |
| etingof | dtantsur, one more useful fix to sushy-tools ^ if it's not too late | 09:47 |
| *** jawad_axd has quit IRC | 09:51 | |
| *** ricolin_ has joined #openstack-ironic | 09:55 | |
| *** ricolin has quit IRC | 09:57 | |
| *** jawad_axd has joined #openstack-ironic | 10:05 | |
| *** rcernin has joined #openstack-ironic | 10:08 | |
| *** jawad_axd has quit IRC | 10:10 | |
| *** dsneddon has joined #openstack-ironic | 10:11 | |
| *** ricolin_ has quit IRC | 10:15 | |
| *** dsneddon has quit IRC | 10:16 | |
| *** iurygregory has joined #openstack-ironic | 10:34 | |
| iurygregory | good morning all o/ | 10:34 |
| rpittau | jey iurygregory :) | 10:38 |
| iurygregory | rpittau, o/ | 10:39 |
| *** ociuhandu has quit IRC | 10:45 | |
| *** dsneddon has joined #openstack-ironic | 10:51 | |
| *** mkrai_ has joined #openstack-ironic | 10:53 | |
| *** ricolin_ has joined #openstack-ironic | 10:53 | |
| *** dsneddon has quit IRC | 10:56 | |
| *** mkrai has quit IRC | 10:56 | |
| *** rpittau is now known as rpittau|bbl | 11:12 | |
| *** ricolin_ is now known as ricolin | 11:15 | |
| *** ociuhandu has joined #openstack-ironic | 11:17 | |
| *** ociuhandu has quit IRC | 11:22 | |
| *** dsneddon has joined #openstack-ironic | 11:24 | |
| *** dsneddon has quit IRC | 11:29 | |
| *** ociuhandu has joined #openstack-ironic | 11:41 | |
| *** bfournie has quit IRC | 11:47 | |
| dtantsur | (late Monday) morning ironic! | 11:49 |
| mkrai_ | morning dtantsur iurygregory | 11:52 |
| goldyfruit | Morning dtantsur mkrai_ | 11:55 |
| mkrai_ | good morning goldyfruit | 11:55 |
| mkrai_ | Gate is broken with a unit test failure tests.unit.common.test_raid.ValidateRaidConfigurationTestCase.test_validate_configuration_custom_schema | 11:56 |
| dtantsur | awesome... | 11:56 |
| *** goldyfruit has quit IRC | 11:56 | |
| dtantsur | jsonschema update? | 11:56 |
| dtantsur | I think it finally hit global-requirements this weekend | 11:57 |
| dtantsur | mkrai_: ^^ | 11:57 |
| dtantsur | any ideas for a fix already? | 11:57 |
| mkrai_ | dtantsur, working on it. Will let you know soon | 11:57 |
| dtantsur | many thanks! | 11:57 |
| *** dsneddon has joined #openstack-ironic | 11:58 | |
| iurygregory | mkrai_, dtantsur morning =D | 12:01 |
| dtantsur | etingof: morning! how close are we to switching the redfish CI job to using virtual media? | 12:01 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-prometheus-exporter master: Support for metrics description https://review.opendev.org/675345 | 12:02 |
| etingof | dtantsur, o/ all necessary bits seem to be in place, hopefully that's only a matter of devstack configuration... | 12:03 |
| dtantsur | etingof: it would be ideal to get it in train | 12:03 |
| etingof | dtantsur, is it tied to ironic release cut? | 12:03 |
| *** dsneddon has quit IRC | 12:03 | |
| dtantsur | etingof: you bug fix seems to be in conflict with https://review.opendev.org/#/c/679456/ could you check? | 12:04 |
| patchbot | patch 679456 - sushy-tools - Fix non-functional driver configuration on Py2 - 1 patch set | 12:04 |
| dtantsur | hmm, no, not this one | 12:04 |
| etingof | dtantsur, I thought I resolved the conflict this morning... | 12:04 |
| dtantsur | ouch, it's this very patch I'm thinking of | 12:04 |
| * dtantsur slaps himself to wake up | 12:05 | |
| *** hamzy_ has joined #openstack-ironic | 12:11 | |
| *** iurygregory has quit IRC | 12:15 | |
| *** jawad_axd has joined #openstack-ironic | 12:21 | |
| *** iurygregory has joined #openstack-ironic | 12:21 | |
| *** ociuhandu has quit IRC | 12:21 | |
| iurygregory | etingof, about the metrics description you can follow https://review.opendev.org/#/c/675345 and add the idea in your patch for redfish already | 12:22 |
| patchbot | patch 675345 - ironic-prometheus-exporter - Support for metrics description - 6 patch sets | 12:22 |
| etingof | iurygregory, yeah, I'm adding a follow up for redfish based on ^ | 12:22 |
| *** rcernin has quit IRC | 12:22 | |
| iurygregory | etingof++ | 12:22 |
| mkrai_ | dtantsur, I think I found the issue. Doing a final test | 12:23 |
| iurygregory | I will review the redfish patch in a few | 12:23 |
| iurygregory | =) | 12:23 |
| openstackgerrit | Merged openstack/metalsmith stable/train: Build pdf doc https://review.opendev.org/680791 | 12:23 |
| openstackgerrit | Madhuri Kumari proposed openstack/ironic master: Fix gate failure related to jsonschema https://review.opendev.org/680987 | 12:25 |
| *** jawad_axd has quit IRC | 12:25 | |
| mkrai_ | dtantsur, https://review.opendev.org/#/c/680987/1 | 12:26 |
| patchbot | patch 680987 - ironic - Fix gate failure related to jsonschema - 1 patch set | 12:26 |
| *** dsneddon has joined #openstack-ironic | 12:31 | |
| etingof | iurygregory, btw, we have metrics for temperature on the per-location basis (baremetal_inlet_temp), but for location metrics for fans fall under one metric (baremetal_fun_rpm). would it make sense for redfish to expand the latter as it's done for the former? | 12:34 |
| *** dsneddon has quit IRC | 12:36 | |
| *** bfournie has joined #openstack-ironic | 12:36 | |
| iurygregory | etingof, let me see if I understood correct | 12:36 |
| iurygregory | you have temperature metrics under baremetal_fan_rpm? | 12:36 |
| iurygregory | O.o | 12:36 |
| etingof | well, that's what we have for ipmi -- https://review.opendev.org/#/c/675345/6/ironic_prometheus_exporter/parsers/metrics_information/ipmi.json | 12:37 |
| patchbot | patch 675345 - ironic-prometheus-exporter - Support for metrics description - 6 patch sets | 12:37 |
| etingof | iurygregory, for redfish I can produce exactly the same metrics or try to unify them like I proposed above | 12:38 |
| iurygregory | etingof, so you would like to re-use the metrics description? | 12:39 |
| etingof | iurygregory, e.g. baremetal_fan_rpm would become baremetal_fan_cpu_rpm, baremetal_fan_chassis_rpm, ... | 12:39 |
| iurygregory | hummm | 12:39 |
| *** jawad_axd has joined #openstack-ironic | 12:39 | |
| etingof | iurygregory, no, perhaps I will do separate json for refdish descriptions - they are different | 12:40 |
| etingof | iurygregory, I am not sure if we should unify metrics names across ipmi and redfish | 12:40 |
| iurygregory | so, if I understood you have fan metrics for cpu component, for chassis etc | 12:40 |
| iurygregory | right? | 12:41 |
| *** ociuhandu has joined #openstack-ironic | 12:43 | |
| dtantsur | mkrai_: thanks! | 12:44 |
| etingof | iurygregory, yes | 12:44 |
| *** jawad_axd has quit IRC | 12:44 | |
| iurygregory | etingof, for me would make sense to add a lable to the baremetal_fan_rpm metric | 12:45 |
| iurygregory | saying where it comes from | 12:45 |
| *** jawad_axd has joined #openstack-ironic | 12:45 | |
| *** Goneri has joined #openstack-ironic | 12:47 | |
| *** ociuhandu has quit IRC | 12:48 | |
| *** jawad_axd has quit IRC | 12:50 | |
| *** jawad_ax_ has joined #openstack-ironic | 12:50 | |
| *** gkadam has quit IRC | 12:55 | |
| ricolin | Hi, I just send heat ironic resource implementation up for review last Friday, so if anyone can give their review will be awesome | 12:55 |
| ricolin | https://review.opendev.org/#/q/topic:story/2006403+(status:open+OR+status:merged) | 12:55 |
| *** rpittau|bbl is now known as rpittau | 12:55 | |
| * rpittau is drowining in emails | 12:57 | |
| dtantsur | rpittau: could you approve https://review.opendev.org/#/c/666789/ please? needed for the release | 12:57 |
| patchbot | patch 666789 - python-ironicclient - Build pdf doc - 3 patch sets | 12:57 |
| dtantsur | and good morning | 12:57 |
| dtantsur | hi ricolin | 12:58 |
| ricolin | dtantsur, I change a bit of implementation from what we discussed in https://etherpad.openstack.org/p/ironic-plus-heat | 12:58 |
| dtantsur | okie, I'll put on my review list | 12:58 |
| ricolin | dtantsur, morning | 12:58 |
| rpittau | dtantsur: hi! adding that to my list :) | 12:58 |
| rpittau | that was smaller than expected, approved | 13:02 |
| mkrai_ | dtantsur, welcome :) | 13:03 |
| *** dsneddon has joined #openstack-ironic | 13:03 | |
| etingof | iurygregory, are you saying that we should collapse all per-component metrics (cpu, power ...) for fan RPM under one metric baremetal_fan_rpm? | 13:07 |
| *** dsneddon has quit IRC | 13:08 | |
| iurygregory | etingof, correct | 13:12 |
| *** bobmel has quit IRC | 13:15 | |
| *** e0ne has quit IRC | 13:19 | |
| *** ociuhandu has joined #openstack-ironic | 13:34 | |
| *** Goneri has quit IRC | 13:34 | |
| *** dsneddon has joined #openstack-ironic | 13:34 | |
| openstackgerrit | Ilya Etingof proposed openstack/ironic-prometheus-exporter master: Add metrics descriptions https://review.opendev.org/681007 | 13:36 |
| etingof | iurygregory, ^ | 13:36 |
| *** ociuhandu has quit IRC | 13:39 | |
| *** dsneddon has quit IRC | 13:39 | |
| *** tzumainn has joined #openstack-ironic | 13:43 | |
| dtantsur | rpittau: could you also put https://review.opendev.org/#/c/680652/ on your queue? This will hopefully unblock IPA-builder gate. | 13:49 |
| patchbot | patch 680652 - ironic - CI: reduce the sizes of disks in the standalone job - 1 patch set | 13:49 |
| dtantsur | (ignore the test failures, the fix for them is on its way) | 13:49 |
| *** e0ne has joined #openstack-ironic | 13:50 | |
| rpittau | dtantsur: yeah, I see | 13:51 |
| iurygregory | etingof, ack | 13:52 |
| *** beekneemech is now known as bnemec | 13:54 | |
| *** priteau has joined #openstack-ironic | 13:54 | |
| *** rloo has joined #openstack-ironic | 13:57 | |
| *** bobmel has joined #openstack-ironic | 14:00 | |
| *** bobmel has quit IRC | 14:06 | |
| *** cdearborn has joined #openstack-ironic | 14:06 | |
| *** bdodd has joined #openstack-ironic | 14:08 | |
| *** jawad_ax_ has quit IRC | 14:12 | |
| *** jawad_axd has joined #openstack-ironic | 14:13 | |
| *** ociuhandu has joined #openstack-ironic | 14:13 | |
| *** jawad_axd has quit IRC | 14:13 | |
| *** dsneddon has joined #openstack-ironic | 14:14 | |
| *** Goneri has joined #openstack-ironic | 14:15 | |
| *** ociuhandu has quit IRC | 14:17 | |
| *** dsneddon has quit IRC | 14:19 | |
| *** belmoreira has joined #openstack-ironic | 14:27 | |
| openstackgerrit | Radoslaw Smigielski proposed openstack/bifrost master: Add quotes around ansible ipv4_gateway variable https://review.opendev.org/681035 | 14:34 |
| openstackgerrit | Merged openstack/python-ironicclient master: Build pdf doc https://review.opendev.org/666789 | 14:34 |
| openstackgerrit | Merged openstack/ironic master: Fix gate failure related to jsonschema https://review.opendev.org/680987 | 14:34 |
| *** ociuhandu has joined #openstack-ironic | 14:35 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: Abort waiting early if reached a failure state https://review.opendev.org/678877 | 14:35 |
| *** ociuhandu has quit IRC | 14:39 | |
| openstackgerrit | Merged openstack/ironic-python-agent-builder master: Build pdf doc https://review.opendev.org/665577 | 14:42 |
| openstackgerrit | Merged openstack/ironic-inspector-specs master: Build pdf doc https://review.opendev.org/665290 | 14:45 |
| openstackgerrit | Merged openstack/ironic-prometheus-exporter master: Support for metrics description https://review.opendev.org/675345 | 14:49 |
| *** belmoreira has quit IRC | 14:50 | |
| openstackgerrit | Merged openstack/ironic-specs master: Build pdf doc https://review.opendev.org/665285 | 14:51 |
| *** ociuhandu has joined #openstack-ironic | 14:52 | |
| openstackgerrit | Rachit Kapadia proposed openstack/ironic master: DRAC: Drives conversion from JBOD to RAID https://review.opendev.org/679093 | 14:53 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic-prometheus-exporter master: Move timestamp registry to a sharable 'header' module https://review.opendev.org/680432 | 14:53 |
| *** dsneddon has joined #openstack-ironic | 14:54 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Adds CI job that uses DIB build images https://review.opendev.org/676239 | 14:54 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic-prometheus-exporter master: Add Redfish metrics support https://review.opendev.org/680731 | 14:54 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic-prometheus-exporter master: Add metrics descriptions https://review.opendev.org/681007 | 14:54 |
| *** belmoreira has joined #openstack-ironic | 14:56 | |
| *** kaifeng has joined #openstack-ironic | 14:56 | |
| openstackgerrit | Ilya Etingof proposed openstack/ironic-prometheus-exporter master: Add metrics descriptions https://review.opendev.org/681007 | 14:58 |
| *** dsneddon has quit IRC | 14:59 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Remove support for CoreOS images https://review.opendev.org/680404 | 14:59 |
| *** rachit7 has quit IRC | 15:00 | |
| dtantsur | #startmeeting ironic | 15:00 |
| openstack | Meeting started Mon Sep 9 15:00:10 2019 UTC and is due to finish in 60 minutes. The chair is dtantsur. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| *** openstack changes topic to " (Meeting topic: ironic)" | 15:00 | |
| openstack | The meeting name has been set to 'ironic' | 15:00 |
| dtantsur | heya! who's here for the most ironic meeting in the openstack world? | 15:00 |
| rpittau | o/ | 15:00 |
| etingof | o/ | 15:00 |
| cdearborn | o/ | 15:00 |
| rloo | o/ | 15:00 |
| kaifeng | o/ | 15:00 |
| *** jroll has joined #openstack-ironic | 15:00 | |
| rpioso | o/ | 15:01 |
| dtantsur | #link https://wiki.openstack.org/wiki/Meetings/Ironic is our agenda (quite empty) | 15:01 |
| dtantsur | #topic Announcements / Reminder | 15:01 |
| *** openstack changes topic to "Announcements / Reminder (Meeting topic: ironic)" | 15:01 | |
| dtantsur | #info Non-client libraries (metalsmith, ironic-lib and sushy) have been finalized for Train | 15:01 |
| dtantsur | all new development there will target U | 15:02 |
| dtantsur | (bug fixes are, of course, welcome) | 15:02 |
| dtantsur | #info Client libraries release deadline this week | 15:02 |
| iurygregory | o/ | 15:02 |
| dtantsur | actually, python-ironic-inspector-client has been released as well, and I'm about to process python-ironicclient | 15:02 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic-prometheus-exporter master: Add metrics descriptions https://review.opendev.org/681007 | 15:02 |
| dtantsur | last but not least: | 15:02 |
| mgoddard | o/ | 15:03 |
| dtantsur | #info Soft feature freeze begins on Thursday | 15:03 |
| dtantsur | I'd like us to slow down on new features this week, so that we can branch stable/train for all projects in one-two weeks | 15:03 |
| dtantsur | #link https://releases.openstack.org/train/index.html | 15:03 |
| dtantsur | #undo | 15:04 |
| openstack | Removing item from minutes: #link https://releases.openstack.org/train/index.html | 15:04 |
| dtantsur | #link https://releases.openstack.org/train/schedule.html | 15:04 |
| dtantsur | We must absolutely have stable/train by RC1 deadline, which is the week of Sep 23rd | 15:04 |
| dtantsur | better next week, so please plan accordingly | 15:05 |
| iurygregory | ack | 15:05 |
| dtantsur | if you're planning on feature work for train, let's raise it during the discussion section of this meeting | 15:05 |
| dtantsur | anything else to announce? any questions about the deadlines? | 15:05 |
| dtantsur | #topic Review subteam status reports (capped at ten minutes) | 15:07 |
| *** openstack changes topic to "Review subteam status reports (capped at ten minutes) (Meeting topic: ironic)" | 15:07 | |
| dtantsur | #link https://etherpad.openstack.org/p/IronicWhiteBoard around line 217 | 15:07 |
| *** gyee has joined #openstack-ironic | 15:07 | |
| *** goldyfruit has joined #openstack-ironic | 15:08 | |
| rloo | dtantsur: let's be explicit, and mention where we are punting to U | 15:08 |
| rloo | eg, deploy steps. ? | 15:09 |
| dtantsur | we could do it the other way around and mention what is NOT punted :) | 15:09 |
| rloo | dtantsur: true, either way. | 15:09 |
| rloo | dtantsur: historically, i think we've mentioned the stuff we've punted, but as long as it is clear to people :) | 15:09 |
| dtantsur | so yes, deploy steps are unlikely to make it, except for the vendor implementation patches (e.g. iDrac RAID) | 15:09 |
| rloo | idrac raid is WIP in the etherpad | 15:10 |
| rloo | i guess what i'd like to know is for deploy steps, there are 4 PRs; which do we think we can get into train? | 15:10 |
| dtantsur | most vendor PRs are in good shape, they're likely to make it | 15:11 |
| mgoddard | I'm working on the iDRAC RAID deploy step | 15:11 |
| rloo | well, it'd help if the status was updated; am doing that now... | 15:11 |
| mgoddard | punting other deploy steps work :( | 15:11 |
| mgoddard | sure | 15:11 |
| openstackgerrit | Richard G. Pioso proposed openstack/ironic master: Add idrac HW type support of Redfish interfaces https://review.opendev.org/681046 | 15:12 |
| rpioso | There's one ^^^ | 15:12 |
| rloo | so for deploy steps, mgoddard is working on idrac raid. what about deploy steps for ilo mgt? | 15:12 |
| rloo | shiv doesn't seem to be here. | 15:13 |
| dtantsur | rloo: stendulker has patches, I think they're in a good shape and close to approval | 15:13 |
| rloo | dtantsur: ok, i see you've looked at it. | 15:13 |
| dtantsur | yep | 15:13 |
| dtantsur | Indicators API remains unclear | 15:13 |
| dtantsur | this is a big chunk of work that has been largely unreviewed | 15:14 |
| dtantsur | opinions? etingof, will you have cycles to iterate on it this week? will anybody have time for reviewing it? | 15:15 |
| etingof | dtantsur, most certainly! | 15:15 |
| rpittau | if it's a priority I can make some time to review them | 15:15 |
| rloo | etingof: how close do you think that work is? minor changes or .. ? | 15:16 |
| dtantsur | what worries me is that the last time I reviewed we ended up changing the API design :) | 15:16 |
| etingof | rloo, it works in devstack! | 15:16 |
| dtantsur | \o/ ship it | 15:16 |
| rpittau | etingof: I see 3 patches mentioned in the etherpad, is that all? | 15:17 |
| rpittau | and lol | 15:17 |
| etingof | rpittau, yes, other relevant ones have been merged | 15:17 |
| rloo | if etingof is avail to iterate and there are people willing to review, then i'm fine with it in train. we can see where we're at next Monday. | 15:18 |
| * etingof will do his best | 15:18 | |
| dtantsur | do we have a *non Red Hat* person to review it? | 15:18 |
| rloo | wrt software raid, it is done but there is a comment at L324, 'Finished for Train, small fixes are still being posted'. If there are small fixes, could someone provide the links please? | 15:19 |
| * rpioso suggests expanding our cores' wardrobe | 15:19 | |
| rloo | dtantsur: i can try later this week, but no promises | 15:19 |
| * rpittau still doesn't have a Red Hat :/ | 15:19 | |
| dtantsur | thanks rloo | 15:19 |
| dtantsur | I'll add the sw raid links | 15:19 |
| * iurygregory is sad for rpittau | 15:20 | |
| rloo | rpittau: don't worry, most of us don't have red hats :D | 15:20 |
| rpittau | :D | 15:20 |
| * dtantsur has probably lost his already | 15:21 | |
| dtantsur | anyway, are we ready to move on? | 15:21 |
| rpioso | I updated the idrac vendor priority. It's the change above. We would like to see it land in Train. | 15:21 |
| dtantsur | k | 15:21 |
| dtantsur | #topic Deciding on priorities for the coming week | 15:22 |
| rloo | tssurya: when you are avail, would you please let us know if there are any more PRs related to power state callbacks to Nova. It is done except for a release note update: https://review.opendev.org/#/c/676400/, yes? | 15:22 |
| patchbot | patch 676400 - ironic - Follow-up to power sync reno - 1 patch set | 15:22 |
| *** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)" | 15:22 | |
| *** jawad_axd has joined #openstack-ironic | 15:22 | |
| dtantsur | oh, not ready yet? | 15:22 |
| rloo | dtantsur: almost ready i think. just want to verify | 15:22 |
| rloo | so ipv6 is not going to get done, right? | 15:22 |
| tssurya | rloo: yea that's correct | 15:23 |
| tssurya | all done except thta follow-up | 15:23 |
| dtantsur | rloo: quite unlikely, yes. but since it's CI work, it's not affected by deadlines. | 15:23 |
| rloo | tssurya: thx for confirming! | 15:23 |
| tssurya | that* | 15:23 |
| dtantsur | (unless it turns into a feature work, of course) | 15:23 |
| tssurya | thanks to all of you for the review efforts and time | 15:23 |
| rloo | dtantsur: ok. and the doc generation stuff? is that affected by deadline? | 15:23 |
| rloo | L356 ^^ | 15:24 |
| dtantsur | rloo: nope. and it's fine to backport. | 15:24 |
| kaifeng | the docs job is basically done except a few patches in the review list | 15:25 |
| rloo | good to know dtantsur! | 15:25 |
| dtantsur | kaifeng: could you add them to the priority list? | 15:25 |
| dtantsur | etingof: same request for indicator patches ^^ | 15:25 |
| kaifeng | dtantsur: will do | 15:25 |
| etingof | dtantsur, ok | 15:25 |
| etingof | how about reviewing the latest ironic-prometheus-exporter additions? -- https://review.opendev.org/680731 | 15:26 |
| patchbot | patch 680731 - ironic-prometheus-exporter - Add Redfish metrics support - 3 patch sets | 15:26 |
| *** jawad_axd has quit IRC | 15:26 | |
| dtantsur | we probably have to, yes. if you plan on making it into train. | 15:26 |
| *** bobmel has joined #openstack-ironic | 15:26 | |
| etingof | dtantsur, I'd like to | 15:27 |
| kaifeng | dtantsur: added at L168, molteniron and virtualpdf are not listed, they don't have doc job so I guess it's not required. | 15:29 |
| dtantsur | kaifeng: thanks! | 15:29 |
| *** dsneddon has joined #openstack-ironic | 15:29 | |
| dtantsur | etingof: when listing patches, could you add a short summary before each? | 15:30 |
| etingof | dtantsur, ack | 15:30 |
| *** tssurya has quit IRC | 15:30 | |
| etingof | we also have a couple of easy fixes for sushy-tools, those are good to merge because then the whole thing actually works | 15:31 |
| *** ociuhandu has quit IRC | 15:31 | |
| *** hjensas|afk is now known as hjensas | 15:31 | |
| rloo | mgoddard: dtantsur: this client PR, is that something we want discussed/done in train? https://review.opendev.org/#/c/675940/ | 15:31 |
| dtantsur | I'd delay them, we already have a huge review backlog | 15:31 |
| patchbot | patch 675940 - python-ironicclient - Fix AttributeError in negotiate_version - 1 patch set | 15:31 |
| dtantsur | rloo: ideally, although it can (and should) be backported | 15:32 |
| rloo | dtantsur: mgoddard: do you two know what the solution is? (I just skimmed the comments) | 15:32 |
| dtantsur | I don't | 15:33 |
| mgoddard | rloo: it would be nice, although as dtantsur says it's a bug so can be backported | 15:33 |
| dtantsur | I'd make it a priority for next week | 15:33 |
| dtantsur | and wrap up feature work this week | 15:33 |
| rloo | ok, i don't think it'll make it in this week then. the client deadline is this week, right? | 15:33 |
| dtantsur | yep | 15:33 |
| dtantsur | I actually wanted to request a release today | 15:34 |
| rloo | dtantsur: ok, let's try to fix that soon and backport it. i don't see anything other PRs for client that might need addressing right now, so a release today is fine. | 15:34 |
| rloo | dtantsur: we're bumping the major version or ? cuz we deleted ironic CLI, right? | 15:35 |
| dtantsur | right | 15:35 |
| rloo | dtantsur: yay. although i worry that we'll have broken people that didn't know/didn't get around to switching to openstackclient. | 15:35 |
| dtantsur | it would happen whenever we pull the trigger :) | 15:36 |
| rloo | dtantsur: yup. so even though i worry... pull the trigger anyway! | 15:36 |
| rloo | i'm good with the weekly priorities :) | 15:36 |
| dtantsur | now there are talks of migrating for python API to openstacksdk.. | 15:36 |
| dtantsur | anyway | 15:36 |
| dtantsur | any other comments on the priorities? anything to add or (better!) remove? | 15:36 |
| rloo | dtantsur: oh, that reminds me. the two PRs at L143 & 144 | 15:37 |
| rloo | dtantsur: do we want to get them into train? if so we mgiht want to discuss L143, and someone needs to continue L144 | 15:37 |
| *** jawad_axd has joined #openstack-ironic | 15:38 | |
| dtantsur | I'd love to get 143 in, it's a constant source of complaints from the customers | 15:38 |
| rloo | dtantsur: yeah, me too... | 15:38 |
| kaifeng | wrt 144, there is a similar patch already gets in, proposed by mkrai | 15:39 |
| dtantsur | rloo: would be great if you could check my responses for L143 | 15:39 |
| dtantsur | kaifeng: link handy? | 15:39 |
| rloo | dtantsur: ok, will discuss with you. prob tomorrow though. | 15:40 |
| kaifeng | dtantsur: https://review.opendev.org/#/c/677649/ it's doing the opposite | 15:40 |
| patchbot | patch 677649 - ironic - Allow deleting unbound ports on active node (MERGED) - 3 patch sets | 15:40 |
| dtantsur | rloo: tl;dr I'm purposely disallowing "provide" action for nodes in maintenance | 15:41 |
| dtantsur | kaifeng: not exactly the opposite, although I am somewhat confused by these two patches.. | 15:42 |
| dtantsur | I have to check with a fresh head tomorrow | 15:42 |
| dtantsur | we can leave them on the list, so that we don't forget to follow-up/discuss, wdyt? | 15:42 |
| rpittau | makes sense | 15:42 |
| rloo | i think mkrai's addresses a subset of the port deletion/with-or-without-vif problem. | 15:42 |
| rloo | but yeah, leave on the list. | 15:43 |
| dtantsur | okay. moving on? | 15:43 |
| mkrai_ | mkrai_, I can take a look at the L144 patch tomorrow | 15:43 |
| dtantsur | thanks mkrai_ | 15:43 |
| dtantsur | #topic Open discussion | 15:43 |
| *** openstack changes topic to "Open discussion (Meeting topic: ironic)" | 15:43 | |
| dtantsur | anything? I guess we've already discussed the feature freeze | 15:44 |
| * kaifeng was cheated by the double negative commit message | 15:44 | |
| dtantsur | heh | 15:44 |
| rloo | double negatives are tricky; we should try to avoid them! | 15:45 |
| rloo | crickets? | 15:46 |
| dtantsur | apparently :) | 15:46 |
| dtantsur | it's still sort of summer, right? | 15:46 |
| etingof | +15 over here | 15:46 |
| dtantsur | sort of | 15:46 |
| rpittau | crickets and dragonflies everywhere | 15:46 |
| dtantsur | thanks everyone, let's make a rocking train! :) | 15:46 |
| iurygregory | not in Brazil XD | 15:46 |
| dtantsur | #endmeeting | 15:46 |
| *** 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:46 | |
| openstack | Meeting ended Mon Sep 9 15:46:58 2019 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:47 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-09-09-15.00.html | 15:47 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-09-09-15.00.txt | 15:47 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-09-09-15.00.log.html | 15:47 |
| *** mkrai_ has quit IRC | 15:48 | |
| rpittau | goodnight! o/ | 15:54 |
| *** rpittau is now known as rpittau|afk | 15:54 | |
| *** igordc has joined #openstack-ironic | 15:54 | |
| openstackgerrit | Merged openstack/ironic-lib master: Build pdf doc https://review.opendev.org/666785 | 15:58 |
| *** jawad_axd is now known as JAX | 16:02 | |
| *** JAX is now known as JAX_ | 16:03 | |
| *** belmoreira has quit IRC | 16:04 | |
| *** JAX_ is now known as JAX | 16:06 | |
| *** tesseract has quit IRC | 16:07 | |
| *** igordc has quit IRC | 16:09 | |
| *** e0ne has quit IRC | 16:10 | |
| *** kaifeng has quit IRC | 16:10 | |
| *** JAX has quit IRC | 16:12 | |
| *** jawad_axd has joined #openstack-ironic | 16:12 | |
| *** alexmcleod has quit IRC | 16:13 | |
| *** jawad_axd has quit IRC | 16:14 | |
| *** dtantsur is now known as dtantsur|afk | 16:23 | |
| dtantsur|afk | o/ | 16:23 |
| *** dsneddon has quit IRC | 16:32 | |
| *** bobmel has quit IRC | 16:36 | |
| arne_wiebalck | bye everyone o/ | 16:40 |
| *** jawad_axd has joined #openstack-ironic | 16:45 | |
| *** ociuhandu has joined #openstack-ironic | 16:48 | |
| *** bdodd has quit IRC | 16:56 | |
| openstackgerrit | Tzu-Mainn Chen proposed openstack/ironic-specs master: Expose node owner information to oslo.policy checks https://review.opendev.org/681066 | 16:57 |
| *** trident has quit IRC | 16:58 | |
| *** derekh has quit IRC | 17:00 | |
| *** dougsz has quit IRC | 17:00 | |
| *** dsneddon has joined #openstack-ironic | 17:05 | |
| *** trident has joined #openstack-ironic | 17:06 | |
| *** dsneddon has quit IRC | 17:10 | |
| *** bobmel has joined #openstack-ironic | 17:21 | |
| *** bobmel has quit IRC | 17:25 | |
| *** ociuhandu has quit IRC | 17:34 | |
| *** ociuhandu has joined #openstack-ironic | 17:36 | |
| *** ricolin has quit IRC | 17:40 | |
| *** ociuhandu has quit IRC | 17:40 | |
| *** dsneddon has joined #openstack-ironic | 17:44 | |
| *** dsneddon has quit IRC | 17:49 | |
| *** goldyfruit_ has joined #openstack-ironic | 17:51 | |
| *** goldyfruit has quit IRC | 17:53 | |
| *** ociuhandu has joined #openstack-ironic | 17:53 | |
| *** e0ne has joined #openstack-ironic | 18:12 | |
| *** hjensas has quit IRC | 18:14 | |
| *** jtomasek_ has joined #openstack-ironic | 18:17 | |
| openstackgerrit | Radoslaw Smigielski proposed openstack/bifrost master: Add quotes around ansible ipv4_gateway variable https://review.opendev.org/681035 | 18:17 |
| *** hjensas has joined #openstack-ironic | 18:18 | |
| *** jtomasek has quit IRC | 18:19 | |
| *** dsneddon has joined #openstack-ironic | 18:21 | |
| *** priteau has quit IRC | 18:23 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Read in non-blocking fashion when starting console https://review.opendev.org/681087 | 18:24 |
| *** e0ne has quit IRC | 18:34 | |
| *** ociuhandu has quit IRC | 18:41 | |
| *** vesper11 has quit IRC | 18:52 | |
| *** clarkb has joined #openstack-ironic | 18:55 | |
| clarkb | FYI I've just filed https://storyboard.openstack.org/#!/story/2006520 after investigating why ironic zuul jobs are hitting retry limit failures | 18:56 |
| *** e0ne has joined #openstack-ironic | 19:03 | |
| *** e0ne has quit IRC | 19:09 | |
| *** e0ne has joined #openstack-ironic | 19:10 | |
| *** goldyfruit_ has quit IRC | 19:15 | |
| *** henriqueof1 has joined #openstack-ironic | 19:18 | |
| *** henriqueof1 has quit IRC | 19:20 | |
| *** henriqueof has joined #openstack-ironic | 19:20 | |
| *** bobmel has joined #openstack-ironic | 19:22 | |
| *** bobmel has quit IRC | 19:27 | |
| *** bnemec has quit IRC | 19:27 | |
| *** bnemec has joined #openstack-ironic | 19:32 | |
| *** goldyfruit has joined #openstack-ironic | 19:33 | |
| *** dsneddon has quit IRC | 19:34 | |
| *** dsneddon has joined #openstack-ironic | 19:35 | |
| *** vesper11 has joined #openstack-ironic | 19:41 | |
| *** bobmel has joined #openstack-ironic | 19:45 | |
| *** ociuhandu has joined #openstack-ironic | 19:51 | |
| *** igordc has joined #openstack-ironic | 20:00 | |
| *** jtomasek_ has quit IRC | 20:01 | |
| *** ociuhandu has quit IRC | 20:03 | |
| *** ociuhandu has joined #openstack-ironic | 20:03 | |
| *** Lucas_Gray has joined #openstack-ironic | 20:11 | |
| *** trident has quit IRC | 20:15 | |
| *** ociuhandu has quit IRC | 20:22 | |
| *** ociuhandu has joined #openstack-ironic | 20:22 | |
| *** iurygregory has quit IRC | 20:23 | |
| *** ociuhandu has quit IRC | 20:27 | |
| *** trident has joined #openstack-ironic | 20:27 | |
| *** trident has quit IRC | 20:32 | |
| *** patchbot has quit IRC | 20:32 | |
| *** trident has joined #openstack-ironic | 20:33 | |
| *** patchbot has joined #openstack-ironic | 20:42 | |
| *** ociuhandu has joined #openstack-ironic | 20:44 | |
| *** jawad_axd has quit IRC | 21:01 | |
| *** ociuhandu has quit IRC | 21:09 | |
| *** Goneri has quit IRC | 21:14 | |
| *** bfournie has quit IRC | 21:27 | |
| *** e0ne has quit IRC | 21:40 | |
| *** goldyfruit has quit IRC | 22:01 | |
| *** goldyfruit has joined #openstack-ironic | 22:08 | |
| *** ociuhandu has joined #openstack-ironic | 22:10 | |
| *** ociuhandu has quit IRC | 22:15 | |
| *** goldyfruit_ has joined #openstack-ironic | 22:21 | |
| *** goldyfruit has quit IRC | 22:24 | |
| *** EmilienM is now known as little_script | 22:39 | |
| *** little_script is now known as EmilienM | 22:42 | |
| *** goldyfruit_ has quit IRC | 22:51 | |
| *** Lucas_Gray has quit IRC | 22:57 | |
| *** rcernin has joined #openstack-ironic | 22:59 | |
| *** bfournie has joined #openstack-ironic | 23:02 | |
| *** Lucas_Gray has joined #openstack-ironic | 23:14 | |
| *** threestrands has joined #openstack-ironic | 23:14 | |
| *** Lucas_Gray has quit IRC | 23:38 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!