| *** ociuhandu has joined #openstack-ironic | 00:21 | |
| *** ociuhandu has quit IRC | 00:25 | |
| *** threestrands has joined #openstack-ironic | 00:35 | |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Centralise imports of wsme types https://review.opendev.org/703898 | 00:43 |
|---|---|---|
| openstackgerrit | Steve Baker proposed openstack/ironic master: Don't use wsme test webapp for patch tests https://review.opendev.org/704485 | 00:43 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Do all serialization in the expose decorator https://review.opendev.org/704486 | 00:43 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Add validate decorator to expose https://review.opendev.org/704487 | 00:43 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Add function definition handling https://review.opendev.org/704488 | 00:43 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: WIP Add json parsing to args https://review.opendev.org/704489 | 00:43 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: WIP Add wsme core types, remove WSME https://review.opendev.org/704490 | 00:43 |
| *** Qianbiao has joined #openstack-ironic | 00:51 | |
| *** Qianbiao is now known as Qianbiao|afk | 00:52 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic-python-agent-builder master: [Trivial] Doc update on the stable-interface-names https://review.opendev.org/720872 | 00:54 |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic master: Fixes unusable Guru meditation report https://review.opendev.org/720870 | 01:03 |
| *** k_mouza has joined #openstack-ironic | 02:17 | |
| *** k_mouza has quit IRC | 02:22 | |
| *** ociuhandu has joined #openstack-ironic | 02:26 | |
| *** k_mouza has joined #openstack-ironic | 02:26 | |
| *** k_mouza has quit IRC | 02:33 | |
| *** ociuhandu has quit IRC | 02:36 | |
| *** Qianbiao|afk has quit IRC | 02:37 | |
| *** ociuhandu has joined #openstack-ironic | 02:37 | |
| *** k_mouza has joined #openstack-ironic | 02:38 | |
| *** ociuhandu has quit IRC | 02:42 | |
| *** k_mouza has quit IRC | 02:46 | |
| *** k_mouza has joined #openstack-ironic | 02:52 | |
| *** ociuhandu has joined #openstack-ironic | 02:55 | |
| *** jhesketh has joined #openstack-ironic | 02:56 | |
| openstackgerrit | Qitao proposed openstack/virtualbmc master: Use unittest.mock instead of third party mock https://review.opendev.org/721142 | 02:56 |
| *** k_mouza has quit IRC | 02:57 | |
| openstackgerrit | Qitao proposed openstack/ironic-inspector master: Use unittest.mock instead of third party mock https://review.opendev.org/721144 | 02:59 |
| *** k_mouza has joined #openstack-ironic | 03:01 | |
| openstackgerrit | Qitao proposed openstack/ironic master: Use unittest.mock instead of third party mock https://review.opendev.org/721153 | 03:04 |
| *** ociuhandu has quit IRC | 03:05 | |
| *** k_mouza has quit IRC | 03:08 | |
| *** k_mouza has joined #openstack-ironic | 03:15 | |
| *** mkrai has joined #openstack-ironic | 03:15 | |
| *** k_mouza has quit IRC | 03:20 | |
| *** k_mouza has joined #openstack-ironic | 03:22 | |
| openstackgerrit | Merged openstack/bifrost master: Use unittest.mock instead of third party mock https://review.opendev.org/720905 | 03:22 |
| openstackgerrit | Merged openstack/sushy master: Use unittest.mock instead of third party mock https://review.opendev.org/721017 | 03:22 |
| *** k_mouza has quit IRC | 03:28 | |
| *** k_mouza has joined #openstack-ironic | 03:31 | |
| *** k_mouza has quit IRC | 03:37 | |
| openstackgerrit | Kaifeng Wang proposed openstack/virtualbmc master: Restore pep8 checking on import order https://review.opendev.org/721162 | 03:38 |
| *** k_mouza has joined #openstack-ironic | 03:40 | |
| *** k_mouza has quit IRC | 03:45 | |
| *** ricolin has joined #openstack-ironic | 03:48 | |
| *** ociuhandu has joined #openstack-ironic | 03:48 | |
| *** ociuhandu has quit IRC | 03:53 | |
| *** k_mouza has joined #openstack-ironic | 04:02 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Use unittest.mock instead of third party mock https://review.opendev.org/720921 | 04:02 |
| openstackgerrit | Merged openstack/ironic-lib master: Use unittest.mock instead of third party mock https://review.opendev.org/720922 | 04:02 |
| *** k_mouza has quit IRC | 04:10 | |
| *** k_mouza has joined #openstack-ironic | 04:16 | |
| *** k_mouza has quit IRC | 04:20 | |
| *** k_mouza has joined #openstack-ironic | 04:22 | |
| *** k_mouza has quit IRC | 04:27 | |
| *** k_mouza has joined #openstack-ironic | 04:38 | |
| *** k_mouza has quit IRC | 04:43 | |
| *** mkrai has quit IRC | 04:44 | |
| *** mkrai has joined #openstack-ironic | 04:45 | |
| *** k_mouza has joined #openstack-ironic | 04:46 | |
| *** k_mouza has quit IRC | 04:50 | |
| *** k_mouza has joined #openstack-ironic | 04:56 | |
| *** k_mouza has quit IRC | 05:02 | |
| *** tkajinam has quit IRC | 05:12 | |
| *** k_mouza has joined #openstack-ironic | 05:13 | |
| *** tkajinam has joined #openstack-ironic | 05:13 | |
| *** k_mouza has quit IRC | 05:17 | |
| *** yaawang_ has quit IRC | 05:18 | |
| *** yaawang_ has joined #openstack-ironic | 05:20 | |
| *** k_mouza has joined #openstack-ironic | 05:23 | |
| *** k_mouza has quit IRC | 05:28 | |
| *** k_mouza has joined #openstack-ironic | 05:33 | |
| *** k_mouza has quit IRC | 05:38 | |
| *** k_mouza has joined #openstack-ironic | 05:41 | |
| *** mkrai has quit IRC | 05:41 | |
| *** mkrai has joined #openstack-ironic | 05:42 | |
| *** k_mouza has quit IRC | 05:45 | |
| *** k_mouza has joined #openstack-ironic | 05:51 | |
| openstackgerrit | Kaifeng Wang proposed openstack/virtualbmc master: Restore pep8 checking on import order https://review.opendev.org/721162 | 05:54 |
| *** k_mouza has quit IRC | 05:55 | |
| openstackgerrit | Merged openstack/virtualbmc master: Use unittest.mock instead of third party mock https://review.opendev.org/721026 | 06:00 |
| *** akahat|rover is now known as akahat|ruck | 06:02 | |
| *** jtomasek has joined #openstack-ironic | 06:04 | |
| *** Qianbiao|afk has joined #openstack-ironic | 06:06 | |
| *** k_mouza has joined #openstack-ironic | 06:11 | |
| *** k_mouza has quit IRC | 06:16 | |
| openstackgerrit | Andreas Jaeger proposed openstack/ironic-inspector master: Cleanup py27 support https://review.opendev.org/720551 | 06:18 |
| arne_wiebalck | Good morning, ironic! | 06:51 |
| *** k_mouza has joined #openstack-ironic | 06:56 | |
| iurygregory | good morning arne_wiebalck and Ironic o/ | 06:59 |
| *** ociuhandu has joined #openstack-ironic | 07:00 | |
| arne_wiebalck | hey iurygregory o/ | 07:01 |
| *** k_mouza has quit IRC | 07:01 | |
| iurygregory | o/ | 07:01 |
| *** dougsz has joined #openstack-ironic | 07:05 | |
| gudrutis2 | hi ironic o/ | 07:09 |
| Qianbiao|afk | hello, hello, hello. | 07:10 |
| *** Qianbiao|afk is now known as Qianbiao | 07:10 | |
| openstackgerrit | Pierre Riteau proposed openstack/tenks stable/1.0: Cap openstacksdk for compatibility with Python 2 https://review.opendev.org/721177 | 07:11 |
| *** rpittau|afk is now known as rpittau | 07:12 | |
| Qianbiao | I got a problem, when set "automated_clean=true", and there are no clean_steps in "node show", but clean_step still run, and finally, node failed with "clean waiting timeout" | 07:12 |
| rpittau | good morning ironic! o/ | 07:12 |
| *** hjensas has quit IRC | 07:13 | |
| Qianbiao | And even if a set a clean step manually, it seems that the clean step is not running(no log), and clean will finally failed(waiting timeout) | 07:14 |
| Qianbiao | any idea? | 07:15 |
| Qianbiao | Failed error log: Timeout reached while cleaning the node. Please check if the ramdisk responsible for the cleaning is running on the node. Failed on step {}. | 07:15 |
| *** hjensas has joined #openstack-ironic | 07:16 | |
| *** threestrands_ has joined #openstack-ironic | 07:18 | |
| *** threestrands_ has quit IRC | 07:18 | |
| *** threestrands has quit IRC | 07:21 | |
| *** k_mouza has joined #openstack-ironic | 07:31 | |
| iurygregory | morning rpittau o/ | 07:33 |
| rpittau | hey iurygregory :) | 07:34 |
| *** k_mouza has quit IRC | 07:36 | |
| openstackgerrit | Kaifeng Wang proposed openstack/virtualbmc master: Restore pep8 checking on import order https://review.opendev.org/721162 | 07:44 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-prometheus-exporter master: Stop configuring install_command in tox. https://review.opendev.org/720793 | 07:51 |
| *** lucasagomes has joined #openstack-ironic | 07:51 | |
| *** alexmcleod has joined #openstack-ironic | 07:51 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-ui master: Stop configuring install_command in tox. https://review.opendev.org/720813 | 08:03 |
| openstackgerrit | Riccardo Pittau proposed openstack/python-ironicclient master: Check import order https://review.opendev.org/721193 | 08:05 |
| *** ociuhandu has quit IRC | 08:07 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/networking-generic-switch master: Stop configuring install_command in tox. https://review.opendev.org/720808 | 08:09 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-ui master: Stop configuring install_command in tox. https://review.opendev.org/720813 | 08:13 |
| *** mkrai has quit IRC | 08:15 | |
| *** mkrai_ has joined #openstack-ironic | 08:15 | |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add `network_data` field to ironic node object https://review.opendev.org/687910 | 08:27 |
| etingof | dtantsur|afk, ^ network data field validation in the api code | 08:28 |
| openstackgerrit | Merged openstack/virtualbmc master: Stop configuring install_command in tox. https://review.opendev.org/720816 | 08:28 |
| openstackgerrit | Merged openstack/sushy-cli master: Stop configuring install_command in tox. https://review.opendev.org/720810 | 08:30 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add `get_node_network_data` method to NetworkInterface https://review.opendev.org/688184 | 08:31 |
| openstackgerrit | Merged openstack/bifrost master: Stop configuring install_command in tox. https://review.opendev.org/720809 | 08:31 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add `get_node_network_data` to non-Neutron NetworkInterface https://review.opendev.org/688266 | 08:31 |
| iurygregory | rpittau, did we test ironic-staging-drivers-all with dib? | 08:34 |
| rpittau | iurygregory: we're running that with dib | 08:36 |
| iurygregory | rpittau, I know, just wondering if we tested before switching | 08:36 |
| iurygregory | because it uses ironic-base so maybe we forgot to test | 08:36 |
| rpittau | iurygregory: I'm not sure what you mean, how can we run with dib now without having merged the change, therefore having tested that? :/ | 08:39 |
| iurygregory | rpittau, depends-On? | 08:40 |
| iurygregory | just wondering if the failure I got was related that's all =) | 08:40 |
| *** dsneddon has quit IRC | 08:41 | |
| rpittau | iurygregory: ironic-staging-drivers-all was running on tinyipa until 6 days ago, then was converted to dib https://review.opendev.org/#/c/718984/5 | 08:41 |
| patchbot | patch 718984 - x/ironic-staging-drivers - Switch to zuulv3 (MERGED) - 5 patch sets | 08:41 |
| *** dsneddon has joined #openstack-ironic | 08:41 | |
| iurygregory | rpittau, oh cool o/ | 08:41 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add `network_data` field to ironic node object https://review.opendev.org/687910 | 08:41 |
| * iurygregory needs more coffee I couldn't find the patch | 08:42 | |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add `get_node_network_data` method to NetworkInterface https://review.opendev.org/688184 | 08:42 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add `get_node_network_data` to non-Neutron NetworkInterface https://review.opendev.org/688266 | 08:43 |
| *** ociuhandu has joined #openstack-ironic | 08:43 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/networking-generic-switch master: Stop configuring install_command in tox. https://review.opendev.org/720808 | 08:44 |
| arne_wiebalck | Qianbiao: this is happens when the node to clean does not call back, in most cases I have seen the node does not boot | 08:44 |
| arne_wiebalck | Qianbiao: get a console on the node to clean and see if it comes up when you initiate the cleaning | 08:45 |
| Qianbiao | arne_wiebalck " node does not boot" means could not power on? | 08:46 |
| Qianbiao | i can see the node has power on in KVM. | 08:46 |
| arne_wiebalck | but does it boot? | 08:47 |
| Qianbiao | Hmm, do not know what does the "boot" means, finally, it will boot from drive and show a ubuntu login ui | 08:47 |
| arne_wiebalck | Qianbiao: the node could have power, but not boot, or it does not get on the network, or it does get on the network but cannot see the controller | 08:47 |
| *** ociuhandu has quit IRC | 08:48 | |
| arne_wiebalck | Qianbiao: aha! | 08:48 |
| openstackgerrit | Merged openstack/ironic-python-agent master: Stop configuring install_command in tox. https://review.opendev.org/720811 | 08:48 |
| arne_wiebalck | Qianbiao: it should boot from the network into the IPA image | 08:48 |
| Qianbiao | <arne_wiebalck> oh, it does not. | 08:48 |
| arne_wiebalck | Qianbiao: not from the local disk | 08:49 |
| Qianbiao | should it auto boot from network when exec clean step? i did nothing. | 08:49 |
| Qianbiao | btw, if i didnot config clean step, the whole deploy progress runs successfully. | 08:49 |
| arne_wiebalck | Qianbiao: not sure about the auto, but if it does not boot from the network, the cleaning will not start | 08:49 |
| Qianbiao | I mean will ironic auto boot from pxe? | 08:50 |
| Qianbiao | Ironic should control the node to boot from pxe i think. | 08:50 |
| arne_wiebalck | Qianbiao: right, so you need to have a look what it does when you try to clean | 08:51 |
| Qianbiao | ok will do now. | 08:51 |
| arne_wiebalck | Qianbiao: does it try to boot from the network (and fail) or does it not boot from the network at all | 08:51 |
| Qianbiao | ok. | 08:51 |
| Qianbiao | <arne_wiebalck> "get a console on the node to clean" | 08:52 |
| Qianbiao | ^^^ do u mean clean by manually | 08:52 |
| arne_wiebalck | Qianbiao: right :) | 08:52 |
| Qianbiao | node clean --clean-steps | 08:52 |
| Qianbiao | ok | 08:52 |
| arne_wiebalck | Qianbiao: well, doesn't matter | 08:52 |
| arne_wiebalck | Qianbiao: you'll need to check what the node does when ironic is instructed to clean it, manually or automatically | 08:53 |
| Qianbiao | arne_wiebalck thanks, you have helped me on several issues. :) | 08:53 |
| Qianbiao | ok, will check now. | 08:53 |
| arne_wiebalck | Qianbiao: ok, let us know how it goes | 08:56 |
| arne_wiebalck | Qianbiao: :) | 08:56 |
| *** k_mouza has joined #openstack-ironic | 09:00 | |
| *** ociuhandu has joined #openstack-ironic | 09:18 | |
| *** dtantsur|afk is now known as dtantsur | 09:21 | |
| dtantsur | morning ironic, happy Monday! | 09:22 |
| arne_wiebalck | hey dtantsur o/ | 09:23 |
| *** gregwork has quit IRC | 09:24 | |
| arne_wiebalck | ironic 101 question: ironic will set the way a node is booting explicitly, right? like, set to network boot for cleaning, to local disk after deployment etc.? | 09:25 |
| *** rcernin has quit IRC | 09:26 | |
| dtantsur | arne_wiebalck: yes | 09:27 |
| dtantsur | unless you're using the noop management interface | 09:27 |
| iurygregory | morning dtantsur | 09:27 |
| *** ricolin has quit IRC | 09:28 | |
| *** ociuhandu has quit IRC | 09:32 | |
| *** ociuhandu has joined #openstack-ironic | 09:33 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-ui master: Stop configuring install_command in tox. https://review.opendev.org/720813 | 09:35 |
| *** ociuhandu has quit IRC | 09:39 | |
| arne_wiebalck | dtantsur: thanks | 09:41 |
| arne_wiebalck | dtantsur: I don't, but in our infra all nodes are supposed to boot from the network all the time (and the answer determines how they boot), and this masks what Ironic would do :) | 09:42 |
| arne_wiebalck | dtantsur: hence I didn't know | 09:42 |
| dtantsur | ah :) | 09:42 |
| arne_wiebalck | dtantsur: thanks :) | 09:42 |
| dtantsur | oh man, the lower-constraints thing is sooo difficult :( | 09:46 |
| iurygregory | dtantsur, it's | 09:47 |
| iurygregory | specially when things locally works and CI does BOOM | 09:47 |
| dtantsur | yeah | 09:47 |
| iurygregory | and for ironic-ui we need to wait for a new release it seems | 09:47 |
| dtantsur | and apparently I have mislead the team re which dependencies have to be included :( | 09:48 |
| *** ricolin has joined #openstack-ironic | 09:48 | |
| *** ricolin_ has joined #openstack-ironic | 09:48 | |
| dtantsur | on a more positive note, I need reviews on https://review.opendev.org/#/c/718358/ please | 09:48 |
| patchbot | patch 718358 - ironic - Improve the command status checks in the agent's p... - 3 patch sets | 09:48 |
| dtantsur | and https://review.opendev.org/#/c/698768/ since it mostly lacks unit test updates | 09:49 |
| patchbot | patch 698768 - ironic - WIP: Support executing in-band deploy steps - 35 patch sets | 09:49 |
| dtantsur | mgoddard: if you could take a quick look today ^^^ | 09:49 |
| mgoddard | dtantsur: sure | 09:50 |
| mgoddard | looks surprisingly small :) | 09:50 |
| *** rcernin has joined #openstack-ironic | 09:50 | |
| *** mkrai has joined #openstack-ironic | 10:00 | |
| dtantsur | ikr? :) | 10:00 |
| dtantsur | there is one more missing bit which I'm working on right now. it will go into the 3rd patch. | 10:00 |
| *** mkrai_ has quit IRC | 10:02 | |
| *** ricolin_ has quit IRC | 10:02 | |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add missing and remove non-existing parameters. https://review.opendev.org/721218 | 10:02 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: WIP: Decompose core deploy step https://review.opendev.org/698309 | 10:07 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: WIP: Delay validating deploy templates until we get all steps https://review.opendev.org/721219 | 10:07 |
| openstackgerrit | Mark Goddard proposed openstack/networking-generic-switch master: Use unittest.mock instead of third party mock https://review.opendev.org/720952 | 10:08 |
| *** ociuhandu has joined #openstack-ironic | 10:08 | |
| openstackgerrit | Merged openstack/tenks stable/1.0: Cap openstacksdk for compatibility with Python 2 https://review.opendev.org/721177 | 10:13 |
| *** rpittau is now known as rpittau|bbl | 10:15 | |
| Qianbiao | hello, is there any console cmd could change node state from active to manageable again. | 10:18 |
| dtantsur | Qianbiao: you need to go through 'node undeploy' first to get to available | 10:19 |
| dtantsur | then you can do 'node manage' | 10:19 |
| *** ociuhandu has quit IRC | 10:22 | |
| * arne_wiebalck feels that dtantsur has a deja-vu | 10:23 | |
| Qianbiao | <dtantsur> thanks. | 10:23 |
| dtantsur | arne_wiebalck: I do indeed :) | 10:23 |
| dtantsur | mgoddard: could you remind me: when invoking raid.create_configuration via deploy templates, with pass the RAID configuration via step['args']? | 10:29 |
| dtantsur | s/with/we/ | 10:29 |
| dtantsur | mmm, seems so, judging by ilo. this needs fixing in IPA, sigh.. | 10:30 |
| mgoddard | dtantsur: correct | 10:34 |
| Qianbiao | <arne_wiebalck> thanks, it's because that we got another dhcp server in lan... so, when boot from pxe, it can not load correct ip. | 10:39 |
| dtantsur | arne_wiebalck: it seems that software RAID tests don't pass for me locally.. (in IPA) | 10:42 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Add raid.apply_configuration deploy step https://review.opendev.org/721236 | 10:50 |
| dtantsur | mgoddard: then we also need ^^ | 10:50 |
| dtantsur | and now I need to check if argsinfo is actually handled on the conductor side (I doubt it) | 10:50 |
| dtantsur | sigh | 10:51 |
| dtantsur | oh, it seems to actually, good | 10:51 |
| * etingof just found out how incredibly stinky the vmedia code has become... | 10:52 | |
| dtantsur | :D | 10:52 |
| * etingof spends half a day trying to untangle it | 10:53 | |
| etingof | but lost hope in the end | 10:53 |
| dtantsur | next cycle we could use a good refactoring of the vmedia code, software RAID code and deploy/clean steps | 10:53 |
| etingof | having spent 5 hours on this mess this morning, I'd rather rollback the ilo commit | 10:54 |
| *** derekh has joined #openstack-ironic | 10:54 | |
| etingof | but it's probably too late by now | 10:55 |
| etingof | I will try to move pieces around in hope to make it a bit more manageable | 10:56 |
| etingof | but really, it is not anymore | 10:56 |
| *** rachit7 has joined #openstack-ironic | 10:58 | |
| *** dougsz has quit IRC | 11:02 | |
| dtantsur | sigh | 11:04 |
| dtantsur | if it makes you feel better: deploy steps handling is also a huge mess and I'm actively working on making it worse :D | 11:08 |
| *** rachit7 has quit IRC | 11:15 | |
| etingof | we are all working hard towards making rewriting the whole ironic the only way out | 11:16 |
| *** dougsz has joined #openstack-ironic | 11:17 | |
| * etingof is being positive | 11:17 | |
| *** tkajinam has quit IRC | 11:21 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/networking-baremetal master: Stop configuring install_command in tox. https://review.opendev.org/720806 | 11:26 |
| *** hjensas has quit IRC | 11:28 | |
| *** dsneddon has quit IRC | 11:31 | |
| *** mkrai has quit IRC | 11:37 | |
| dtantsur | etingof: rewriting in go \o/ | 11:45 |
| etingof | let's do it in .NET to be on the same page with BMCs | 11:46 |
| *** rh-jelabarre has joined #openstack-ironic | 11:50 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Stop configuring install_command in tox. https://review.opendev.org/720790 | 11:53 |
| dtantsur | that's a lovely idea | 11:54 |
| *** ociuhandu has joined #openstack-ironic | 11:57 | |
| openstackgerrit | Merged openstack/ironic-python-agent-builder master: [Trivial] Doc update on the stable-interface-names https://review.opendev.org/720872 | 11:59 |
| *** ociuhandu has quit IRC | 12:02 | |
| *** rpittau|bbl is now known as rpittau | 12:03 | |
| arne_wiebalck | dtantsur: oh? | 12:04 |
| dtantsur | arne_wiebalck: on master: http://paste.openstack.org/show/792411/ | 12:06 |
| arne_wiebalck | dtantsur: hmm, will try to reproduce | 12:06 |
| dtantsur | for me it bisects to the initial UEFI patch | 12:08 |
| arne_wiebalck | dtantsur: the tests pass for me | 12:12 |
| dtantsur | yay for insufficient mocking somewhere? | 12:12 |
| arne_wiebalck | oh no | 12:12 |
| * arne_wiebalck remembers the last time he tried to chase such a bug down | 12:13 | |
| dtantsur | I'll look into it since I can reproduce | 12:15 |
| arne_wiebalck | dtantsur: the errors all due to the partition table type | 12:15 |
| dtantsur | right, msdos vs gpt | 12:15 |
| dtantsur | interesting | 12:15 |
| arne_wiebalck | the type comes from the node | 12:19 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Stop configuring install_command in tox. https://review.opendev.org/720791 | 12:20 |
| arne_wiebalck | dtantsur: is the node object reused and the boot type set to uefi due to the test which ran just before ... or sth like this? | 12:22 |
| dtantsur | yeah, maybe. will check in a few mins | 12:22 |
| arne_wiebalck | dtantsur: this would explain why the code tried gpt instead of bios | 12:22 |
| arne_wiebalck | dtantsur: ok, let me know if I can help | 12:23 |
| *** derekh has quit IRC | 12:24 | |
| *** Lucas_Gray has joined #openstack-ironic | 12:25 | |
| * etingof though six is gone in ironic, but apparently not quite -- https://github.com/openstack/ironic/blob/master/ironic/tests/unit/drivers/modules/test_virtual_media_base.py#L29 | 12:29 | |
| iurygregory | enr | 12:31 |
| iurygregory | this shouldn't be there at all ... | 12:32 |
| rpittau | how is taht there? | 12:32 |
| iurygregory | magic | 12:33 |
| rpittau | oh it sneaked in in a recent patch | 12:34 |
| etingof | also interesting, ilo seems to hijack generic redfish tests -- https://github.com/openstack/ironic/blob/master/ironic/tests/unit/drivers/modules/test_virtual_media_base.py#L42 | 12:34 |
| rpittau | https://review.opendev.org/#/c/677996/ | 12:34 |
| patchbot | patch 677996 - ironic - Generalize ISO building for virtual media driver (MERGED) - 16 patch sets | 12:34 |
| * etingof is going violent about that ^ patch | 12:35 | |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Fix vmedia test methods and remove `six` https://review.opendev.org/721256 | 12:39 |
| Qianbiao | hello, what should raid delete_configuration do? only delete volumes? should we delete spare drives meantime? | 12:39 |
| rpittau | etingof: I can support violence in this case :) | 12:41 |
| rpittau | but I feel responsible as I didn't review it :/ | 12:41 |
| *** Goneri has joined #openstack-ironic | 12:42 | |
| etingof | me too! I did review it and -1'ed it, but then I did not revisit it for some time so the spaghetti monster sneaked in it has taken over ironic | 12:43 |
| * dtantsur reminds people to review more and more often ;) | 12:45 | |
| dtantsur | Qianbiao: I'm not sure I understand the difference. What do you mean by delete drives? | 12:45 |
| Qianbiao | <dtantsur> raid configuration contains: 1. volumes(logical disk) config 2. spare drive config(will replace a physical disk of a volume if data is broken) | 12:47 |
| openstackgerrit | Verification of a change to openstack/ironic-inspector failed: Cleanup py27 support https://review.opendev.org/720551 | 12:47 |
| Qianbiao | drive is a physical disk. volume is logical disk. as i know | 12:47 |
| dtantsur | Qianbiao: then I'm not sure how you want to "delete spare drives"? they're physical, you cannot delete them. | 12:49 |
| Qianbiao | <dtantsur> delete the spare config. | 12:49 |
| Qianbiao | let me explain. | 12:49 |
| dtantsur | yeah, I guess I don't understand this concept | 12:50 |
| *** priteau has joined #openstack-ironic | 12:50 | |
| Qianbiao | we have 10 physical disks. and create a volume with RAID5 with disk 1,2,3. | 12:50 |
| Qianbiao | and then we set disk4 as a spare disk for this RAID volume. | 12:50 |
| Qianbiao | if any of disk 1,2,3 is broken(like disk1 we say). then disk 4 will auto replace disk1 and then rebuild data from disk,2,3 . | 12:51 |
| dtantsur | ah! nice, thank you | 12:52 |
| TheJulia | good morning everyone | 12:52 |
| dtantsur | morning TheJulia | 12:52 |
| * TheJulia rubs sleep from her eyes | 12:52 | |
| Qianbiao | and this RAID5 volume will still be good. | 12:52 |
| dtantsur | Qianbiao: if this configuration can be/is created with create_configuration, I think it should be deleted with delete_configuration | 12:52 |
| Qianbiao | create_configuration does support spare drive as i know for now. | 12:52 |
| Qianbiao | but iBMC API do support create hot-spare drive for RAID volumes. | 12:53 |
| Qianbiao | do u mean delete_configuration should only delete configurations created by "create_configuration" step | 12:54 |
| dtantsur | no, I don't mean this | 12:55 |
| dtantsur | let's approach this problem from another angle | 12:55 |
| dtantsur | Qianbiao: is there a use case for deleting RAID but leaving spare drive configuration intact? | 12:55 |
| dtantsur | and are there cases where it will end up in breakages? | 12:56 |
| Qianbiao | yes, a spare drive can be global. for any volume. | 12:56 |
| Qianbiao | no for second. spare drive config can be left as it is. | 12:57 |
| dtantsur | oh, so I can define a spare drive once, and it will be used for every RAID I build later on? | 12:57 |
| Qianbiao | just the spare drive can not be used then. | 12:57 |
| Qianbiao | yes. | 12:57 |
| dtantsur | Qianbiao: okay, then I think you should keep it intact and provide separate manual clean steps to manage it | 12:58 |
| Qianbiao | it's a global spare drive. any disk in a volume is broken, it will auto replace it. | 12:58 |
| Qianbiao | ok, thanks. dtantsur | 12:58 |
| * dtantsur likes this feature | 12:58 | |
| dtantsur | oh wow, the CI doesn't feel well | 12:59 |
| *** derekh has joined #openstack-ironic | 12:59 | |
| TheJulia | dtantsur: is ci in one of those "just go back to bed" days? | 12:59 |
| dtantsur | TheJulia: it is, yes | 12:59 |
| TheJulia | *sigh* | 12:59 |
| dtantsur | full red from retry failures and post failures | 12:59 |
| Qianbiao | = =... i like CI. | 12:59 |
| dtantsur | it seems that the infra is on it | 12:59 |
| * dtantsur double-checks | 13:00 | |
| iurygregory | for know we can keep trying recheck =) | 13:02 |
| iurygregory | doesn't seem to be things in our side for RETRY_LIMIT | 13:02 |
| * Qianbiao bye | 13:05 | |
| Qianbiao | oh, i find how to send action :) | 13:06 |
| * Qianbiao hello | 13:06 | |
| *** Lucas_Gray has quit IRC | 13:06 | |
| *** irclogbot_3 has quit IRC | 13:06 | |
| dtantsur | :) | 13:07 |
| dtantsur | iurygregory: seems something with log servers | 13:07 |
| *** irclogbot_0 has joined #openstack-ironic | 13:07 | |
| iurygregory | dtantsur, not the first time I would say =) | 13:07 |
| dtantsur | happens indeed | 13:07 |
| iurygregory | yeah | 13:08 |
| *** rloo has joined #openstack-ironic | 13:17 | |
| Qianbiao | hi, how can we simulate running clean steps for a node ? | 13:22 |
| Qianbiao | conductor.cleaning.do_node_clean(task, clean_steps=clean_steps) ? | 13:25 |
| * TheJulia tries to wake up | 13:26 | |
| dtantsur | Qianbiao: what exactly are you trying to do? | 13:28 |
| *** tzumainn has joined #openstack-ironic | 13:30 | |
| *** ociuhandu has joined #openstack-ironic | 13:30 | |
| *** ociuhandu has quit IRC | 13:35 | |
| Qianbiao | dtantsur simulate node cleaning in testcase. | 13:36 |
| dtantsur | Qianbiao: unit tests should only test a small region of code. if you're writing clean steps, test only them, not the whole procedure. | 13:37 |
| Qianbiao | call "task.driver.raid.delete_configuration" directly? | 13:38 |
| Qianbiao | then how can i know a step like {'step': "delete_configuration"} will call my raid.configuration exactly | 13:39 |
| *** hoonetorg has quit IRC | 13:40 | |
| Qianbiao | Oh, i see. ironic team has testcases for this situation. | 13:41 |
| Qianbiao | dtantsur thanks. :) | 13:41 |
| dtantsur | yep :) | 13:41 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-prometheus-exporter master: Add lower constraints jobs https://review.opendev.org/717684 | 13:48 |
| *** tkajinam has joined #openstack-ironic | 13:53 | |
| -openstackstatus- NOTICE: Zuul is temporarily offline; service should be restored in about 15 minutes. | 13:59 | |
| *** hoonetorg has joined #openstack-ironic | 13:59 | |
| *** ociuhandu has joined #openstack-ironic | 14:00 | |
| rpittau | guess we can go home.... oh wait | 14:01 |
| iurygregory | hehehe | 14:02 |
| iurygregory | recheck all the patches in 15 min | 14:03 |
| iurygregory | thank god no XD since Zuul should run the checks again if I do remember | 14:03 |
| *** mkrai has joined #openstack-ironic | 14:03 | |
| TheJulia | heh | 14:07 |
| *** Qianbiao has left #openstack-ironic | 14:15 | |
| *** Qianbiao has joined #openstack-ironic | 14:16 | |
| arne_wiebalck | erbarr: you around? | 14:22 |
| arne_wiebalck | erbarr: it's about the UEFI exception you see | 14:22 |
| erbarr | yea, what's up? | 14:22 |
| arne_wiebalck | erbarr: could you please post logs around this error? I think this is coming from the surrounding function. What my question was about was the exception the code tries to catch. | 14:23 |
| arne_wiebalck | erbarr: If another exception is raised, the patch is useless :) | 14:24 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add missing and remove non-existing parameters. https://review.opendev.org/721218 | 14:24 |
| erbarr | okay, so you want to see the full traceback or the full conductor log? | 14:25 |
| arne_wiebalck | let's start with the traceback :) | 14:27 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Fix vmedia test methods and remove `six` https://review.opendev.org/721256 | 14:27 |
| *** dsneddon has joined #openstack-ironic | 14:29 | |
| *** ociuhandu has quit IRC | 14:29 | |
| *** cdearborn has joined #openstack-ironic | 14:34 | |
| *** ociuhandu has joined #openstack-ironic | 14:35 | |
| dtantsur | arne_wiebalck: we have os.path.isdir('/sys/firmware/efi') on the test path :) my laptop uses UEFI, hence the difference. | 14:39 |
| arne_wiebalck | dtantsur: ouch | 14:39 |
| arne_wiebalck | dtantsur: this should be mocked, I guess | 14:40 |
| *** ociuhandu has quit IRC | 14:40 | |
| dtantsur | yeah | 14:40 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Mock get_node_boot_mode in software RAID unit tests https://review.opendev.org/721280 | 14:42 |
| dtantsur | arne_wiebalck: here we go ^^^ | 14:42 |
| arne_wiebalck | erbarr: is this a partition image you use? | 14:42 |
| arne_wiebalck | dtantsur: thanks! do you think there is value in duplicating these tests for UEFI/gpt? | 14:44 |
| dtantsur | arne_wiebalck: there are such tests, they mock os.path.isdir | 14:44 |
| arne_wiebalck | dtantsur: ah, great! | 14:44 |
| erbarr | arne_wiebalck: mmmmnn, i don't know, how can I find that out? | 14:46 |
| arne_wiebalck | erbarr: erm ... :) | 14:47 |
| * arne_wiebalck has never used a partition image | 14:47 | |
| arne_wiebalck | erbarr: it's just that the error comes from within a if which says not whole disk image | 14:47 |
| erbarr | arne_wiebalck: i'm seeing this IRONIC_AGENT_RAMDISK_URL=https://tarballs.openstack.org/ironic-python-agent/coreos/files/coreos_production_pxe_image-oem-master.cpio.gz | 14:49 |
| dtantsur | erbarr: FYI the coreos images are abandoned and no longer updated. but this is not a user image, it's a deploy ramdisk. | 14:50 |
| erbarr | dtantsur: thanks, yea I need to look into updating them, so what setting is a user image? | 14:52 |
| dtantsur | erbarr: I'm a bit out of context of what you're trying. if you're doing 'nova boot'/'openstack server create', you're passing an image there | 14:52 |
| arne_wiebalck | dtantsur: we established the other day that install_bootloader is called for wdi and uefi, remember? | 14:53 |
| dtantsur | yeah | 14:53 |
| arne_wiebalck | dtantsur: the traceback comes from inside an if which says 'not wdi' | 14:53 |
| * arne_wiebalck is confused | 14:53 | |
| dtantsur | well, install_bootloader is also always called for partition images | 14:54 |
| *** stendulker has joined #openstack-ironic | 14:54 | |
| dtantsur | (since they are never assumed to have a bootloader) | 14:54 |
| *** hjensas has joined #openstack-ironic | 14:54 | |
| arne_wiebalck | exactly, this is why I was asking whether this a partition image | 14:54 |
| dtantsur | "target_boot_mode == 'uefi' and software_raid" should cover it though | 14:55 |
| arne_wiebalck | that's true, it is not directly related with the problem, but uefi is not mandatory | 14:57 |
| arne_wiebalck | is it possible we do not get a AgentAPIError, but another exception? | 14:58 |
| dtantsur | I mean, https://review.opendev.org/#/c/720781/ does seem correct to me | 14:58 |
| patchbot | patch 720781 - ironic - Don't break UEFI install with older IPAs - 1 patch set | 14:58 |
| *** rh-jelabarre has quit IRC | 14:58 | |
| arne_wiebalck | yes, but it does not solve erbarr's issue it seems | 14:58 |
| dtantsur | erbarr: is there another part of the traceback? in python 3 the causing traceback is usually reported before the final one | 14:58 |
| dtantsur | oh, meeting in 2 mins, we'll have to continue later | 14:58 |
| arne_wiebalck | ok | 14:58 |
| * dtantsur considers this one of the best python 3 features | 14:59 | |
| erbarr | ok, i'll look during meeting and report afterwrds | 14:59 |
| *** tkajinam has quit IRC | 14:59 | |
| TheJulia | #startmeeting ironic | 15:00 |
| openstack | Meeting started Mon Apr 20 15:00:11 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 |
| TheJulia | o/ | 15:00 |
| *** openstack changes topic to " (Meeting topic: ironic)" | 15:00 | |
| iurygregory | o/ | 15:00 |
| openstack | The meeting name has been set to 'ironic' | 15:00 |
| rloo | hi dtantsur, wrt https://storyboard.openstack.org/#!/story/2006910 -- was it approved? (I saw a PR associated with it and was going to review it but... ? ) | 15:00 |
| etingof | \o | 15:00 |
| dtantsur | o/ | 15:00 |
| rloo | o/ | 15:00 |
| TheJulia | o/ | 15:00 |
| ajya | o/ | 15:00 |
| rpioso | \o | 15:00 |
| TheJulia | Greetings everyone! | 15:00 |
| dtantsur | rloo: I think we've become pretty relaxed with approvals. that being said, this work should wait for V to beging. | 15:00 |
| TheJulia | Our meeting agenda can be found at https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:00 |
| rajinir | hi | 15:01 |
| TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:01 |
| dtantsur | rloo: I think it was sort-of-approved-at-least-nobody-objected | 15:01 |
| rloo | dtantsur: ok :) | 15:01 |
| TheJulia | #topic Announcements / Reminder | 15:01 |
| *** openstack changes topic to "Announcements / Reminder (Meeting topic: ironic)" | 15:01 | |
| mgoddard | \o | 15:01 |
| TheJulia | #info This is the end of the ussuri cycle as we will likely need to be ready to cut our stable branch this week or MAYBE early next week at the absolute latest. | 15:02 |
| *** kaifeng has joined #openstack-ironic | 15:02 | |
| arne_wiebalck | o/ | 15:02 |
| kaifeng | o/ | 15:02 |
| cdearborn | o/ | 15:02 |
| TheJulia | #info TheJulia needs to understand who is is intending to attend the virtual PTG, we will discuss this more later in the meeeting | 15:02 |
| TheJulia | Does anyone else have anything else to announce or remind us of? | 15:03 |
| stendulker | o/ | 15:03 |
| dtantsur | I guess we're in sort of feature freeze now? | 15:03 |
| dtantsur | At least we'll need to freeze API/RPC at some point to be able to update the release mapping | 15:03 |
| rpittau | o/ | 15:04 |
| TheJulia | Essentially yes, so I'd like to propose we review and maybe merge any outstanding changes that would be good to merge that may impact API/RPC, so we can update the release mappings in the next couple of days | 15:04 |
| TheJulia | I'm intending to write a prelude later today and revise any confusing release notes | 15:04 |
| dtantsur | I don't think we have API changes that are realistic to merge in Ussuri | 15:04 |
| * rpittau following from cellphone as my isp decided to die | 15:04 | |
| TheJulia | looks like you blocked the one RPC change that I know of off the top of my head that made sense | 15:05 |
| dtantsur | heh :) | 15:05 |
| dtantsur | well, I see no big point in merging it without everything else | 15:05 |
| dtantsur | and delaying till V will get us a bigger margin of error | 15:06 |
| TheJulia | I'd love to see https://review.opendev.org/#/c/687910/ in, but *shrugs* | 15:06 |
| patchbot | patch 687910 - ironic - Add `network_data` field to ironic node object - 30 patch sets | 15:06 |
| dtantsur | I'd apply same argument. We have no chances of finishing the feature in Ussuri. | 15:06 |
| TheJulia | No way to know if we would be able to get the rest of the stack merged in time | 15:06 |
| TheJulia | yeah | 15:06 |
| dtantsur | So landing API that is not backed by anything... | 15:06 |
| TheJulia | yup | 15:06 |
| dtantsur | Traditionally, we have been trying to land API changes *last* | 15:07 |
| dtantsur | so that the backend is in place by the time they're exposed | 15:07 |
| dtantsur | etingof: ^^^ | 15:07 |
| mgoddard | +1 | 15:07 |
| TheJulia | Indeed, although tradition doesn't make sense in all cases | 15:07 |
| TheJulia | Anyway, we should move on | 15:07 |
| dtantsur | I should be worried that I'm becoming too conservative? :) | 15:08 |
| TheJulia | heh | 15:08 |
| iurygregory | I would say maybe =) | 15:08 |
| * dtantsur prepares to yell at cloud | 15:09 | |
| TheJulia | dtantsur: I'm afraid you've already been doign that | 15:09 |
| *** dougsz has quit IRC | 15:09 | |
| rpittau | Considering the ci status also I doubt anything new will land in time | 15:09 |
| TheJulia | we just didn't issue you a cane to shake at the cloud | 15:09 |
| TheJulia | lol | 15:09 |
| *** rh-jelabarre has joined #openstack-ironic | 15:09 | |
| TheJulia | Anyway, moving onward | 15:09 |
| dtantsur | yeah, where's my cane kids? | 15:09 |
| TheJulia | #topic review action items from the prior week | 15:09 |
| openstackgerrit | Merged openstack/ironic-ui master: Stop configuring install_command in tox. https://review.opendev.org/720813 | 15:09 |
| *** openstack changes topic to "review action items from the prior week (Meeting topic: ironic)" | 15:09 | |
| TheJulia | Well, it was more like two weeks ago | 15:10 |
| * iurygregory joins dtantsur to yell | 15:10 | |
| TheJulia | and iurygregory I believe you sent an email regarding cancelling the meeting, so I think were good! | 15:10 |
| iurygregory | TheJulia, yup | 15:10 |
| TheJulia | #topic Review subteam status reports | 15:10 |
| *** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)" | 15:10 | |
| TheJulia | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 15:10 |
| TheJulia | Line 248 | 15:11 |
| TheJulia | so deploy steps work going to next cycle correct? | 15:11 |
| dtantsur | not fully, I hope | 15:12 |
| TheJulia | I saw there was an outstanding uefi raid patch, but it seems like its done | 15:12 |
| dtantsur | at least one patch has to land now | 15:12 |
| dtantsur | #link https://review.opendev.org/#/c/721236/ is technically breaking | 15:12 |
| patchbot | patch 721236 - ironic-python-agent - Add raid.apply_configuration deploy step - 1 patch set | 15:12 |
| TheJulia | Well, I guess it is a good time to merge that then | 15:13 |
| * TheJulia adds to list | 15:14 | |
| arne_wiebalck | TheJulia: this one? https://review.opendev.org/#/c/720781 | 15:14 |
| patchbot | patch 720781 - ironic - Don't break UEFI install with older IPAs - 1 patch set | 15:14 |
| TheJulia | seems reasonable | 15:14 |
| TheJulia | interesting | 15:15 |
| TheJulia | Is anyone else aware of patches that must merge? | 15:16 |
| TheJulia | tzumainn: all done w/r/t multitenancy? | 15:16 |
| tzumainn | TheJulia, yep! as far as I know :) | 15:17 |
| TheJulia | we're keeping six for sushy-tools for the time being right? | 15:17 |
| dtantsur | tzumainn: you owe us a demo video ;) | 15:17 |
| TheJulia | +++++ | 15:17 |
| TheJulia | +1,000,000 | 15:17 |
| tzumainn | dtantsur, hahaha, I can do that | 15:18 |
| dtantsur | that would be amazing (and you can show it on your yearly performance review!) | 15:18 |
| rpittau | TheJulia: re sushy-tools six, yes, for now | 15:18 |
| iurygregory | we can drop already no? =) | 15:19 |
| TheJulia | On a plus side, looks like we got a number of awesome things merged this year | 15:19 |
| TheJulia | well | 15:20 |
| TheJulia | cycle | 15:20 |
| dtantsur | we need to make sure we consume sushy-tools from releases on stable branches | 15:20 |
| dtantsur | same change I did for vbmc | 15:20 |
| rpittau | I'd rather wait for the next cycle to be honest | 15:20 |
| TheJulia | I'm good with proceeding whenever you all are ready. | 15:20 |
| dtantsur | it's low priority either way, let's concentrate on the release for now | 15:20 |
| rpittau | yep | 15:20 |
| iurygregory | ok, I will check the stable branches to make sure we are using sushy-tools from releases | 15:21 |
| * TheJulia feels like we drifted into priority setting anyway | 15:21 | |
| dtantsur | iurygregory: it boils down to removing it from required-projects/$PROJECTS | 15:21 |
| dtantsur | I think. I hope. | 15:21 |
| iurygregory | dtantsur, I hope too =) let's see | 15:22 |
| *** mkrai has quit IRC | 15:22 | |
| *** mkrai has joined #openstack-ironic | 15:22 | |
| TheJulia | Are we good to proceed? | 15:23 |
| iurygregory | I think so | 15:23 |
| TheJulia | #topic Priorities for the week | 15:24 |
| *** openstack changes topic to "Priorities for the week (Meeting topic: ironic)" | 15:24 | |
| TheJulia | So I believe the priorities are kind of simple. The must land patches, prelude, and release mapping update. | 15:24 |
| TheJulia | Do we want to change up CI dramatically right before we release? | 15:25 |
| TheJulia | iurygregory: this is in regards to your job changes | 15:25 |
| iurygregory | TheJulia, you mean the l-c changes? | 15:26 |
| dtantsur | I guess the standalone job? | 15:26 |
| iurygregory | oh | 15:26 |
| TheJulia | iurygregory: standalone chagnes | 15:26 |
| iurygregory | well we merged the change in ironic-tempest-plugin | 15:27 |
| iurygregory | if we don't mind not testing that it's fine? | 15:27 |
| TheJulia | I guess we do release the plugin... | 15:27 |
| iurygregory | but I'm assuming that since we split we could add a job to check | 15:27 |
| iurygregory | https://review.opendev.org/720675 this adds the job in ironic | 15:28 |
| patchbot | patch 720675 - ironic-tempest-plugin - Add standalone redfish jobs - 1 patch set | 15:28 |
| iurygregory | ops in tempest-plugin | 15:28 |
| iurygregory | and https://review.opendev.org/712042 in ironic | 15:28 |
| patchbot | patch 712042 - ironic - Add ironic-standalone-redfish - 7 patch sets | 15:28 |
| *** yolanda has joined #openstack-ironic | 15:28 | |
| TheJulia | check job non-voting and then make it gate/voting after the stable branch is cut? | 15:28 |
| iurygregory | ++ sure =) | 15:29 |
| iurygregory | will update | 15:29 |
| dtantsur | ++ | 15:29 |
| rpittau | TheJulia: about CI, we should also land the last dib conversion patches | 15:29 |
| TheJulia | rpittau: links? | 15:29 |
| TheJulia | or topic link I guess :) | 15:30 |
| rpittau | https://review.opendev.org/#/q/topic:ironic-convert-dib+(status:open+OR+status:merged) | 15:30 |
| rpittau | the -1 are because of the recent ci issues | 15:30 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Add ironic-standalone-redfish https://review.opendev.org/712042 | 15:30 |
| TheJulia | okay, I guess we're just going to have to wait and see on those. | 15:31 |
| rpittau | yep | 15:31 |
| TheJulia | I think it is if they make it awesome, if not it is still okay | 15:31 |
| TheJulia | Anything else? | 15:31 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-tempest-plugin master: Add standalone redfish jobs https://review.opendev.org/720675 | 15:32 |
| TheJulia | I guess not, Are we good to proceed to discussion | 15:34 |
| iurygregory | ++ | 15:34 |
| TheJulia | #topic Discussion | 15:35 |
| *** openstack changes topic to "Discussion (Meeting topic: ironic)" | 15:35 | |
| TheJulia | First topic, is PTG Attendance. I would like to get an idea of those that will be interested in attending and what time zones would be best for themselves. | 15:35 |
| * dtantsur Central Europe | 15:36 | |
| TheJulia | or that they can cover. I suspect, after all these years, I could kind of guess at least at the later question, but wanted to put it out there for everyone. | 15:36 |
| rpittau | Central Europe | 15:36 |
| * arne_wiebalck Central Europe | 15:36 | |
| rloo | eastern time zone | 15:37 |
| mgoddard | Europe | 15:37 |
| * iurygregory Central Europe | 15:37 | |
| cdearborn | eastern | 15:38 |
| TheJulia | I guess another question and moderately unrelated is are there any topics that people are interested with regards to the scientific sig, edge working group, etc? | 15:38 |
| dtantsur | a status check with the edge group may be useful | 15:39 |
| dtantsur | there may be some discussions with ansible SIG re our modules | 15:39 |
| dtantsur | not sure if it's interesting for everyone though | 15:40 |
| iurygregory | not sure if there will be a discussion about Goals for V... | 15:40 |
| TheJulia | I've been going back and forth in email with Ilidko and they are hoping to have some ironic discussion around ironic | 15:40 |
| TheJulia | iurygregory: what do you mean? | 15:40 |
| *** gyee has joined #openstack-ironic | 15:40 | |
| TheJulia | I mean, aside from https://review.opendev.org/720100 :) | 15:40 |
| patchbot | patch 720100 - ironic-specs - WIP - Victoria Cycle Priorit(y|ies) - 1 patch set | 15:40 |
| dtantsur | iurygregory: there certainly will, but here's a thing to keep in mind | 15:40 |
| dtantsur | we no longer have to fit our internal topics into the same week | 15:41 |
| dtantsur | it's pretty expected that teams may set up discussions outside of it | 15:41 |
| iurygregory | TheJulia, just to *avoid* overlap if possible | 15:41 |
| dtantsur | at least that's the impression I've got | 15:41 |
| iurygregory | dtantsur, yeah | 15:41 |
| dtantsur | the week should be dedicated to questions requiring a wider audience | 15:41 |
| dtantsur | like cross-project interactions, reaching out to the ops community, and so on | 15:42 |
| TheJulia | same, although I have concerns about requirements gathering and the feedback loop being disjointed | 15:42 |
| dtantsur | we can have something even before AND after | 15:42 |
| TheJulia | I kind of like that idea, even if it is just an hour sync-up or call to discuss priorities again after opendev | 15:42 |
| dtantsur | I'm just pointing out that we have more flexibility and we should use it | 15:42 |
| TheJulia | Agreed | 15:43 |
| dtantsur | rather than only suffer from downsides :) | 15:43 |
| TheJulia | ++ | 15:43 |
| dtantsur | the week of the PTG will be cramped. many teams will meet in the same EU-US timeslot | 15:43 |
| dtantsur | apparently | 15:43 |
| TheJulia | Yeah :\ | 15:43 |
| dtantsur | 3pm UTC or something like that | 15:43 |
| * dtantsur returns the mic to Julia | 15:44 | |
| TheJulia | :) | 15:44 |
| TheJulia | Oh, I'm about to give it right back :) | 15:44 |
| dtantsur | oh? | 15:44 |
| TheJulia | Well, that discussion answered the questions I needed input on, I'll see what I can do! | 15:45 |
| dtantsur | ah, right. I think we chatted about http://lists.openstack.org/pipermail/openstack-discuss/2020-April/013757.html previously already? | 15:45 |
| dtantsur | we may talk about potential follow-ups though | 15:45 |
| TheJulia | Yes, and there has been some additional discussion. That being said I think the victoria cycle priorities draft is a good place to continue the discussion amongst the contributors | 15:46 |
| dtantsur | ++ | 15:46 |
| *** hoonetorg has quit IRC | 15:46 | |
| -openstackstatus- NOTICE: Gerrit will be restarted to correct a misconfiguration which caused some git mirrors to have outdated references. | 15:47 | |
| dtantsur | of immediate follow-ups I see: a potential web site, rethinking release schedule, reducing (hard) dependencies, making the docs less openstack-centric | 15:47 |
| dtantsur | but we probably should shake off Ussuri first | 15:47 |
| TheJulia | ++ There is some additional discussion occuring but I can't comment on it... yet :) | 15:48 |
| TheJulia | ++ | 15:48 |
| * dtantsur is intrigued :) | 15:48 | |
| TheJulia | dtantsur: attend open board meetings, 'tis all I can say | 15:48 |
| rloo | definitely docs less openstack-centric. regardless of anything else :) | 15:48 |
| dtantsur | TheJulia: oh nope :D | 15:49 |
| rloo | or more docs about standalone | 15:49 |
| dtantsur | ++ | 15:49 |
| dtantsur | we have one page that is not ideally structured | 15:49 |
| dtantsur | we *may* even use 'baremetal' instead of 'openstack baremetal' | 15:49 |
| * dtantsur will compose a list of potential actions that don't involve drastic political changes | 15:50 | |
| TheJulia | dtantsur: +1,000,000 | 15:50 |
| rloo | seems like a reorg of this content: https://docs.openstack.org/ironic/latest/, some things will be common to both openstack & standalone, others won't. | 15:50 |
| dtantsur | yeah | 15:50 |
| TheJulia | ++ | 15:50 |
| iurygregory | ++ | 15:50 |
| TheJulia | Anyway, docs and topic presentation dives deep into perceptions and it is one of those discussions _should_ likely save for a social hour or soemthing | 15:51 |
| TheJulia | Speaking of which, we should do another SPUC! | 15:51 |
| dtantsur | totally | 15:51 |
| TheJulia | Anyway, I guess we're good to stop over at the Baremetal SIG if arne_wiebalck has anything he wants to talk about, and then move to open discussion. | 15:52 |
| dtantsur | +++ | 15:52 |
| iurygregory | II SPUC \o/ | 15:52 |
| arne_wiebalck | nothing from me, please have a look at the white paper and contribute or review :) | 15:52 |
| TheJulia | ++ | 15:52 |
| TheJulia | More text keeps flowing into it, it is kind of awesome! | 15:52 |
| TheJulia | Moving to open discussion then! | 15:53 |
| TheJulia | #topic Open Discussion | 15:53 |
| *** openstack changes topic to "Open Discussion (Meeting topic: ironic)" | 15:53 | |
| *** ianychoi_ is now known as ianychoi | 15:53 | |
| TheJulia | I did add some philosopical/whimsical thoughts to the end of the white paper and went through the authorship history | 15:54 |
| *** hjensas has quit IRC | 15:54 | |
| cdearborn | hey folks, we have a patch that we think is going to be important to get in: https://review.opendev.org/693328 | 15:54 |
| patchbot | patch 693328 - ironic - DRAC : idrac-redfish inspect updates pxe port - 11 patch sets | 15:54 |
| TheJulia | at least, what was visible :( \o/ for anonymous edits | 15:54 |
| arne_wiebalck | TheJulia: thanks | 15:54 |
| TheJulia | cdearborn: I'm a little hesitant to merge features until you guys merge the sushy-oem-idrac changes and cut a release | 15:54 |
| kaifeng | i have a quick question wrt the adopt process, does anyone know why it's validating instance_info/image_source ? | 15:54 |
| dtantsur | cdearborn: https://review.opendev.org/#/c/710327/ needs to be merged AND released first | 15:54 |
| patchbot | patch 710327 - x/sushy-oem-idrac - Export System Configuration - 2 patch sets | 15:54 |
| TheJulia | kaifeng: it doesn't have to | 15:55 |
| dtantsur | kaifeng: to be able to rebuild | 15:55 |
| cdearborn | TheJulia, dtantsur, good point | 15:55 |
| TheJulia | kaifeng: but yes, as dtantsur just pointed out, so rebuild could be supported. If you want to create a knob for it, I think it would be easy to merge that | 15:55 |
| cdearborn | I wanted to point out that the redfish driver doesn't set pxe_enabled on the ports in ironic, which will cause major problems with nova if more than 1 port is enabled on a baremetal server | 15:56 |
| dtantsur | kaifeng: generally, the fewer special cases, the easier it is to reason about a state of a node | 15:56 |
| TheJulia | and by knob, I mean configuration parameter because I've had the same question asked elsewhere recently but in a more subtle way | 15:56 |
| TheJulia | Rebuild is also not really widely used afaik | 15:56 |
| dtantsur | TheJulia: I'm soft -1 on a new parameter. if we think it's better without the check - let's just remove it. | 15:56 |
| kaifeng | oh, we have some case that adopt existing nodes, then perform routine tasks like power control, or even monitoring | 15:56 |
| dtantsur | yeah, we overestimated the importance of rebuild back then | 15:56 |
| TheJulia | dtantsur: I think that is reasonable | 15:56 |
| TheJulia | so lets remove the requirement \o/ | 15:57 |
| dtantsur | cdearborn: good point :( I wonder if etingof has any plans about it | 15:57 |
| TheJulia | 3 minute warning :) | 15:57 |
| * etingof can't read that quickly | 15:57 | |
| cdearborn | dtantsur, unfortunately, i don't think there is a generic way to get the pxe ports via redfish, so not sure if there's anything that can be done in the redfish driver itself | 15:58 |
| * arne_wiebalck would love an ironic-aware openstack server rebuild | 15:58 | |
| dtantsur | arne_wiebalck: it is ironic-aware Oo | 15:58 |
| TheJulia | etingof: no worries | 15:58 |
| dtantsur | I mean.. we even tested it in the CI at some point | 15:58 |
| * etingof has a silly question | 15:58 | |
| arne_wiebalck | dtantsur: including s/w RAID | 15:58 |
| * kaifeng is in the same both with etingof | 15:58 | |
| dtantsur | arne_wiebalck: oh. let's discuss afterwards | 15:58 |
| kaifeng | *boat | 15:58 |
| etingof | is it too late to revert anything? | 15:58 |
| etingof | I noticed a patch lately that has quite some issues to follow up. I've worked on it the whole today, but now I think it would be way easier to revert than to follow up.... | 15:59 |
| dtantsur | etingof: never too late, depends on what and why | 15:59 |
| etingof | this patch https://review.opendev.org/#/c/677996/16 | 15:59 |
| patchbot | patch 677996 - ironic - Generalize ISO building for virtual media driver (MERGED) - 16 patch sets | 15:59 |
| dtantsur | a storyboard story is a good start | 15:59 |
| dtantsur | let's make sure we document reasons outside of IRC | 15:59 |
| etingof | has multiple problems | 15:59 |
| *** hoonetorg has joined #openstack-ironic | 16:00 | |
| etingof | a story on the discovered problems? | 16:00 |
| TheJulia | Nothing is perfect and the intent was in the right direction | 16:00 |
| dtantsur | etingof: yes | 16:00 |
| TheJulia | ++ lets detail why | 16:00 |
| etingof | fixing these problems will be quite a time sink | 16:00 |
| etingof | okay | 16:00 |
| kaifeng | TheJulia, dtantsur: thanks, will think more about it :) | 16:01 |
| TheJulia | Thanks everyone! | 16:02 |
| TheJulia | One last question before I end the meeting | 16:02 |
| TheJulia | Do we want to have another SPUC this friday? | 16:02 |
| TheJulia | or the following week? | 16:02 |
| dtantsur | can be this week | 16:03 |
| cdearborn | SPUC? | 16:03 |
| dtantsur | cdearborn: Sanity Preserving Un-Conference | 16:03 |
| iurygregory | 2nd edition | 16:03 |
| dtantsur | i.e. an ironic happy hour | 16:03 |
| cdearborn | ahhh - lol - +3 | 16:03 |
| * dtantsur hopes he remembered the abbreviation right | 16:03 | |
| TheJulia | close enough :) | 16:04 |
| TheJulia | The idea is get people to talk about things they are interested in and there may be coffee, tea.... or other tasty beverage | 16:04 |
| iurygregory | s/Preserving/Preservation | 16:04 |
| TheJulia | Oh, and we did silly hats | 16:04 |
| dtantsur | we need something new silly for this round :) | 16:04 |
| TheJulia | JayF totally won with his video overlay silly hat of silliness | 16:04 |
| dtantsur | oh yeah :D | 16:04 |
| iurygregory | ++ | 16:04 |
| iurygregory | best hat | 16:05 |
| * TheJulia delegates the silliness topic | 16:05 | |
| TheJulia | best hat could work! | 16:05 |
| TheJulia | Most well dressed? | 16:05 |
| dtantsur | oh, I'm going to lose :D | 16:05 |
| rpittau | heh I lose before even starting | 16:05 |
| * dtantsur will bet on TheJulia | 16:05 | |
| iurygregory | 3 | 16:06 |
| dtantsur | TheJulia: we all know you have awesome dresses! | 16:06 |
| TheJulia | dtantsur: this is true... | 16:06 |
| TheJulia | Well, you guys decide on something! | 16:06 |
| TheJulia | and with that, thanks everyone! | 16:06 |
| TheJulia | Have a wonderful week! | 16:07 |
| dtantsur | thank you TheJulia | 16:07 |
| TheJulia | #endmeeting | 16:07 |
| *** 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." | 16:07 | |
| openstack | Meeting ended Mon Apr 20 16:07:05 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:07 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-04-20-15.00.html | 16:07 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-04-20-15.00.txt | 16:07 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-04-20-15.00.log.html | 16:07 |
| rpittau | thanks! | 16:07 |
| rpittau | and bye! :) | 16:07 |
| *** rpittau is now known as rpittau|afk | 16:07 | |
| dtantsur | arne_wiebalck: software RAID and rebuild, what exactly is wrong about it? | 16:07 |
| dtantsur | you mean, rebuilding the RAID as well? | 16:08 |
| arne_wiebalck | exactly | 16:08 |
| dtantsur | mmm, I have a feeling it goes against the idea of RAID in ironic.. although I do see a point in it. | 16:08 |
| dtantsur | like, a user has screwed up their RAID and wants to rebuild? | 16:08 |
| JayF | TheJulia: I mean, that's kinda cheating. Someone with a physical hat has to be the winner lol | 16:08 |
| arne_wiebalck | the way we handle this now is to delete and recreate the instance | 16:08 |
| dtantsur | arne_wiebalck: deleting doesn't tear down RAID, does it? | 16:09 |
| iurygregory | JayF, you have a good point I would say lol | 16:09 |
| iurygregory | we could have virtual and real hat lol | 16:09 |
| *** lucasagomes has quit IRC | 16:09 | |
| arne_wiebalck | dtantsur: deleting removes everything, no? | 16:09 |
| dtantsur | arne_wiebalck: no? I think we explicitly made it so that cleaning doesn't tear down RAID? | 16:10 |
| TheJulia | raids may not survive with preserve_ephemeral.... | 16:10 |
| TheJulia | with the way it is written | 16:10 |
| dtantsur | TheJulia: preserve_ephemeral is for partition images? | 16:10 |
| dtantsur | which aren't really supported with software RAID | 16:10 |
| TheJulia | dtantsur: true | 16:10 |
| dtantsur | (although may just work) | 16:10 |
| arne_wiebalck | dtantsur: ah, true we have this in our local h/w manager | 16:10 |
| dtantsur | you cheaters! :) | 16:10 |
| TheJulia | heh | 16:11 |
| arne_wiebalck | :-D | 16:11 |
| TheJulia | figures | 16:11 |
| dtantsur | arne_wiebalck: so, you tear down RAID on each cleaning? | 16:11 |
| arne_wiebalck | we have nodes with RAID-0, the moment a disk breaks, the node is gone | 16:11 |
| arne_wiebalck | dtantsur: yes | 16:11 |
| dtantsur | and when do you build it again? | 16:11 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/sushy-tools master: Stop configuring install_command in tox. https://review.opendev.org/720807 | 16:12 |
| arne_wiebalck | on provide, with cleaning :) | 16:12 |
| arne_wiebalck | delete/create | 16:12 |
| dtantsur | aaaahh. oooh | 16:12 |
| dtantsur | this makes sense.. do you plan to upstream this mode of operation? | 16:12 |
| arne_wiebalck | this way, the cycle gives me back a node with a re-configured RAID | 16:12 |
| arne_wiebalck | dtantsur: I was suggesting this! | 16:13 |
| arne_wiebalck | dtantsur: was rejected at the time | 16:13 |
| dtantsur | I guess we wanted to get the MVP out first | 16:13 |
| arne_wiebalck | dtantsur: I think the main idea (which I was supporting) was to make it work the same way as h/w RAID. | 16:14 |
| dtantsur | it could be handled the same way as wiping hard drives: via a priority override (0 by default) | 16:14 |
| dtantsur | I'd not mind hardware RAID being able to do the same.. but that's a whole different story | 16:14 |
| arne_wiebalck | dtantsur: what I mean was that the reason to make it not automatic was to make it symmetric with h/w RAID | 16:15 |
| dtantsur | and we'll get a similar mode of operation with deploy templates | 16:16 |
| arne_wiebalck | dtantsur: but even if this is done upon cleaning, it does not fulfill the nova rebuild wish | 16:16 |
| dtantsur | (once the in-band steps are finished... sigh.....) | 16:16 |
| dtantsur | right | 16:16 |
| arne_wiebalck | an in-place heal and redeploy | 16:16 |
| dtantsur | let me open a huge can of worms on this lovely day: | 16:16 |
| dtantsur | mgoddard, TheJulia, should we re-run any deploy steps on rebuld? | 16:16 |
| *** yolanda has quit IRC | 16:17 | |
| arne_wiebalck | uh! | 16:17 |
| dtantsur | aka how is rebuild going to work when we stop special-case deploy.deploy? | 16:17 |
| dtantsur | hehe | 16:17 |
| dtantsur | I'd expect us to re-run all deploy steps.. which I don't think we do now | 16:17 |
| * dtantsur brains boiling | 16:18 | |
| arne_wiebalck | and with s/w RAID moving to in-band deploy steps ... | 16:18 |
| dtantsur | ... you can get your feature for free | 16:18 |
| arne_wiebalck | exactly | 16:18 |
| dtantsur | oh man, I'm going to spend the whole V cycle on deploy steps | 16:19 |
| * arne_wiebalck calls it a day and lets dtantsur sort out the remaining details | 16:19 | |
| dtantsur | \o/ | 16:19 |
| arne_wiebalck | :-D | 16:19 |
| dtantsur | is it drinks time already? | 16:19 |
| arne_wiebalck | erbarr: did you manage to get more backtraces by any chance? | 16:20 |
| arne_wiebalck | erbarr: I think we would need to find out which exception agent_client sees ... if that is not AgentAPIError, this would explain a lot | 16:21 |
| arne_wiebalck | erbarr: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent_client.py#L302 | 16:22 |
| TheJulia | dtantsur: well, ideally rebuild is just a redeploy to the same machine. rebuild just re-invokes deploy | 16:23 |
| dtantsur | I guess you're right and I don't need more code for that | 16:23 |
| dtantsur | that's a relief | 16:23 |
| arne_wiebalck | erbarr: this is where the error comes back from the deploy ramdisk and the retry should happen | 16:23 |
| *** stendulker has quit IRC | 16:24 | |
| arne_wiebalck | TheJulia: dtantsur: rebuild is supposed to keep certain aspects constant, while the delete/re-instantiate cycle we do to mimic this in our deployment does not | 16:25 |
| *** fdegir has joined #openstack-ironic | 16:25 | |
| dtantsur | arne_wiebalck: which aspects do you have in mind? you mean, from scheduling perspective? | 16:26 |
| dtantsur | or specifically in ironic? | 16:26 |
| arne_wiebalck | dtantsur: like the exact same physical machine | 16:26 |
| arne_wiebalck | dtantsur: it should conserve IP address | 16:26 |
| dtantsur | right, so from nova perspective. I see. | 16:26 |
| arne_wiebalck | dtantsur: yes | 16:27 |
| arne_wiebalck | dtantsur: anyway, no big deal, we have some machinery in place, but it would be nice none the less | 16:27 |
| arne_wiebalck | dtantsur: to have a "nova rebuild" rebuild the instance from ground up | 16:28 |
| dtantsur | right | 16:29 |
| dtantsur | I guess that's where deploy templates will come handy indeed | 16:29 |
| arne_wiebalck | yes, never thought of them as a solution to this problem | 16:29 |
| arne_wiebalck | s/problem/feature wish/ | 16:29 |
| *** yolanda has joined #openstack-ironic | 16:29 | |
| * TheJulia loves migraine | 16:29 | |
| TheJulia | migraines | 16:30 |
| erbarr | arne_wiebalck: i didn't see any other tracebacks, just the same error on the bootloader in different places | 16:30 |
| arne_wiebalck | TheJulia: get some rest! | 16:30 |
| dtantsur | erbarr: could you at least fetch the preceding logs? | 16:30 |
| * TheJulia makes a pouty look | 16:31 | |
| dtantsur | TheJulia: sorry to hear, I suffered most of yesterday. get some rest indeed! | 16:31 |
| TheJulia | just started to realize I'm starting to see the visual effects | 16:31 |
| dtantsur | not good, resting becomes mandatory | 16:32 |
| arne_wiebalck | erbarr: or instrument the line I pointed to above to make sure we get the exception we expect | 16:32 |
| *** dtantsur is now known as dtantsur|afk | 16:32 | |
| dtantsur|afk | g'night folks | 16:32 |
| arne_wiebalck | bye dtantsur|afk | 16:32 |
| TheJulia | o. | 16:33 |
| TheJulia | o/ | 16:33 |
| erbarr | dtantsur: sure, i'll look for other logs, have a good night | 16:33 |
| *** yaawang has joined #openstack-ironic | 16:33 | |
| erbarr | arne_wiebalck: let me look at the api log | 16:33 |
| *** yaawang_ has quit IRC | 16:33 | |
| openstackgerrit | Derek Higgins proposed openstack/ironic master: WIP ipv6 jobs https://review.opendev.org/613169 | 16:34 |
| arne_wiebalck | bye everyone o/ | 16:35 |
| *** ociuhandu has joined #openstack-ironic | 16:36 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add ironic 15.0 release mapping https://review.opendev.org/721301 | 16:44 |
| * etingof noted a few things on the unhappy patch -- https://storyboard.openstack.org/#!/story/2007576 | 16:45 | |
| *** k_mouza has quit IRC | 16:46 | |
| TheJulia | etingof: please issue the revert when you have a minute | 16:47 |
| openstackgerrit | Merged openstack/networking-generic-switch master: Use unittest.mock instead of third party mock https://review.opendev.org/720952 | 16:50 |
| etingof | TheJulia, just to confirm, are you proposing me pressing that button now? | 16:51 |
| TheJulia | etingof: yes please | 16:54 |
| etingof | happily! | 16:54 |
| TheJulia | etingof: just update hte commit message of the revert to point to the story and provide high lvel details | 16:54 |
| TheJulia | Thanks! | 16:54 |
| *** ociuhandu has quit IRC | 16:56 | |
| TheJulia | ls | 16:57 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Revert "Generalize ISO building for virtual media driver" https://review.opendev.org/721309 | 16:57 |
| etingof | done ^ | 16:58 |
| TheJulia | etingof: left two comments on the commit message | 17:01 |
| *** derekh has quit IRC | 17:03 | |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Revert "Generalize ISO building for virtual media driver" https://review.opendev.org/721309 | 17:05 |
| openstackgerrit | Aija Jaunteva proposed openstack/ironic master: Fix RAID configuration with idrac-wsman interface https://review.opendev.org/720702 | 17:06 |
| *** ociuhandu has joined #openstack-ironic | 17:10 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Ironic 15.0 prelude https://review.opendev.org/721315 | 17:16 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Prepare release notes/docs for 15.0 release https://review.opendev.org/721316 | 17:16 |
| *** priteau has quit IRC | 17:19 | |
| *** ociuhandu has quit IRC | 17:22 | |
| *** yolanda has quit IRC | 17:26 | |
| openstackgerrit | Verification of a change to openstack/ironic-inspector failed: Add BasePhysnetHook https://review.opendev.org/720010 | 17:26 |
| openstackgerrit | Verification of a change to openstack/ironic-inspector failed: Port physical network CIDR map hook https://review.opendev.org/718594 | 17:26 |
| *** ricolin has quit IRC | 17:29 | |
| *** portdirect has quit IRC | 17:30 | |
| *** portdirect has joined #openstack-ironic | 17:30 | |
| openstackgerrit | Merged openstack/ironic stable/train: Document deploy_boot_mode and boot_option for standalone deployments https://review.opendev.org/720156 | 17:53 |
| *** ociuhandu has joined #openstack-ironic | 18:15 | |
| *** ociuhandu has quit IRC | 18:20 | |
| *** ociuhandu has joined #openstack-ironic | 18:20 | |
| *** kaifeng has quit IRC | 18:21 | |
| *** mkrai has quit IRC | 18:24 | |
| *** mkrai_ has joined #openstack-ironic | 18:24 | |
| openstackgerrit | Aija Jaunteva proposed openstack/ironic master: Fix SpanLength calculation for DRAC RAID configuration https://review.opendev.org/721326 | 18:41 |
| *** yolanda has joined #openstack-ironic | 18:46 | |
| *** yolanda has quit IRC | 18:46 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic stable/train: Fix SpanLength calculation for DRAC RAID configuration https://review.opendev.org/721327 | 18:47 |
| openstackgerrit | Merged openstack/ironic master: Update iDRAC doc about soft power off timeout https://review.opendev.org/718761 | 18:47 |
| *** k_mouza has joined #openstack-ironic | 18:48 | |
| *** k_mouza has quit IRC | 18:52 | |
| *** ociuhandu has quit IRC | 18:57 | |
| *** ociuhandu has joined #openstack-ironic | 18:58 | |
| *** ociuhandu has quit IRC | 19:03 | |
| *** jmccrory has quit IRC | 19:08 | |
| *** jmccrory has joined #openstack-ironic | 19:09 | |
| *** rloo has quit IRC | 19:09 | |
| *** rloo has joined #openstack-ironic | 19:09 | |
| *** rloo has quit IRC | 19:10 | |
| *** rloo has joined #openstack-ironic | 19:10 | |
| *** Lucas_Gray has joined #openstack-ironic | 19:18 | |
| *** Lucas_Gray has quit IRC | 19:21 | |
| *** hjensas has joined #openstack-ironic | 19:23 | |
| *** Lucas_Gray has joined #openstack-ironic | 19:24 | |
| dhellmann | Am I right in reading ironic_lib.utils.find_devices_by_hints() that root device hints are applied in a non-deterministic order? It seems to loop over a dictionary at https://opendev.org/openstack/ironic-lib/src/branch/master/ironic_lib/utils.py#L367 | 19:31 |
| dhellmann | hmm, though that's inside the loop over devices | 19:33 |
| dhellmann | so maybe it's the first device to match all of the rules? | 19:33 |
| dhellmann | s/rules/hints/ | 19:33 |
| dhellmann | what I'm trying to work out is the precedence ordering of the hints, and it seems maybe they all have the same precedence. | 19:34 |
| dhellmann | for example, if you specify an HCTL and a size and the device at that address isn't at least that big, provisioning would fail? | 19:34 |
| openstackgerrit | Christopher Dearborn proposed openstack/ironic master: DRAC: Added redfish management interface issue https://review.opendev.org/720076 | 19:35 |
| *** Lucas_Gray has quit IRC | 19:41 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Fixes unusable Guru meditation report https://review.opendev.org/720870 | 19:42 |
| *** Lucas_Gray has joined #openstack-ironic | 19:45 | |
| *** ociuhandu has joined #openstack-ironic | 19:50 | |
| *** ociuhandu has quit IRC | 19:57 | |
| dhellmann | ah, yes, there's a note on https://docs.openstack.org/ironic/pike/install/include/root-device-hints.html about needing to match all hints | 19:59 |
| *** ociuhandu has joined #openstack-ironic | 20:10 | |
| *** k_mouza has joined #openstack-ironic | 20:15 | |
| *** k_mouza has quit IRC | 20:20 | |
| *** ociuhandu has quit IRC | 20:30 | |
| TheJulia | matching and smallest of the matching if memory serves at all. | 20:34 |
| *** rloo has quit IRC | 20:46 | |
| *** rloo has joined #openstack-ironic | 20:47 | |
| *** rloo has quit IRC | 20:49 | |
| *** rloo has joined #openstack-ironic | 20:55 | |
| *** ociuhandu has joined #openstack-ironic | 21:03 | |
| *** ociuhandu has quit IRC | 21:08 | |
| *** ociuhandu has joined #openstack-ironic | 22:04 | |
| *** rcernin has quit IRC | 22:05 | |
| *** rcernin has joined #openstack-ironic | 22:06 | |
| *** Lucas_Gray has quit IRC | 22:13 | |
| *** ociuhandu has quit IRC | 22:18 | |
| *** ociuhandu has joined #openstack-ironic | 22:18 | |
| *** ociuhandu has quit IRC | 22:23 | |
| *** mrda has quit IRC | 22:31 | |
| *** mrda has joined #openstack-ironic | 22:33 | |
| TheJulia | cdearborn: the duplicate you identifed is a different branch target | 22:35 |
| *** rloo has quit IRC | 22:41 | |
| *** tkajinam has joined #openstack-ironic | 22:43 | |
| *** jtomasek has quit IRC | 22:43 | |
| *** Lucas_Gray has joined #openstack-ironic | 23:04 | |
| *** threestrands has joined #openstack-ironic | 23:18 | |
| openstackgerrit | Derek Higgins proposed openstack/ironic master: WIP ipv6 jobs https://review.opendev.org/613169 | 23:33 |
| *** Lucas_Gray has quit IRC | 23:34 | |
| *** bdodd has quit IRC | 23:34 | |
| *** bdodd has joined #openstack-ironic | 23:37 | |
| *** hjensas has quit IRC | 23:49 | |
| *** hjensas has joined #openstack-ironic | 23:51 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!