| openstackgerrit | Merged openstack/ironic master: Fix AttributeError in check allowed port fields https://review.opendev.org/718127 | 00:04 |
|---|---|---|
| *** rloo has quit IRC | 00:38 | |
| *** cdearborn has quit IRC | 00:50 | |
| openstackgerrit | shenxindi proposed openstack/ironic master: Release reservation when stoping the ironic-conductor service https://review.opendev.org/718342 | 01:03 |
| *** Qianbiao has joined #openstack-ironic | 01:44 | |
| Qianbiao | good morning. | 01:45 |
| *** Goneri has quit IRC | 01:59 | |
| *** ociuhandu has quit IRC | 02:31 | |
| *** jdandrea has quit IRC | 02:40 | |
| *** mkrai has joined #openstack-ironic | 02:45 | |
| *** yaawang has quit IRC | 02:55 | |
| *** yaawang has joined #openstack-ironic | 02:56 | |
| openstackgerrit | Harald Jensås proposed openstack/ironic-inspector master: Port physical network hook https://review.opendev.org/718594 | 03:00 |
| openstackgerrit | Harald Jensås proposed openstack/ironic-inspector master: Port physical network hook https://review.opendev.org/718594 | 03:02 |
| *** ociuhandu has joined #openstack-ironic | 03:03 | |
| *** rh-jelabarre has quit IRC | 03:12 | |
| *** ociuhandu has quit IRC | 03:32 | |
| *** mkrai_ has joined #openstack-ironic | 03:36 | |
| *** mkrai has quit IRC | 03:37 | |
| openstackgerrit | Verification of a change to openstack/ironic-inspector failed: Set default boot option to netboot for jobs using cirros https://review.opendev.org/716613 | 04:21 |
| *** k_mouza has joined #openstack-ironic | 04:32 | |
| *** k_mouza has quit IRC | 04:37 | |
| *** yaawang has quit IRC | 04:40 | |
| *** yaawang has joined #openstack-ironic | 04:41 | |
| *** logan- has quit IRC | 04:45 | |
| *** logan_ has joined #openstack-ironic | 04:47 | |
| *** logan_ is now known as logan- | 04:48 | |
| openstackgerrit | Ankit Kumar proposed openstack/ironic master: Generalize ISO building for virtual media driver https://review.opendev.org/677996 | 05:02 |
| *** Qianbiao has quit IRC | 05:29 | |
| *** ociuhandu has joined #openstack-ironic | 05:31 | |
| *** mkrai has joined #openstack-ironic | 05:38 | |
| *** mkrai_ has quit IRC | 05:40 | |
| *** ociuhandu has quit IRC | 06:01 | |
| gudrutis2 | Morning 0/ | 06:32 |
| *** ociuhandu has joined #openstack-ironic | 06:42 | |
| openstackgerrit | Rabi Mishra proposed openstack/ironic master: DNM Test patch https://review.opendev.org/718602 | 06:46 |
| arne_wiebalck | Good morning, ironic! | 06:49 |
| *** jtomasek has joined #openstack-ironic | 06:57 | |
| *** jtomasek has quit IRC | 06:59 | |
| *** ricolin has quit IRC | 06:59 | |
| *** jtomasek has joined #openstack-ironic | 07:00 | |
| *** dtantsur|afk is now known as dtantsur | 07:06 | |
| dtantsur | morning ironic | 07:09 |
| dtantsur | folks, could someone maybe review https://review.opendev.org/#/c/698770/ please? it's actually simple, moving code around and copy-paste-with-sed. | 07:10 |
| patchbot | patch 698770 - ironic-python-agent - Add an ability to run in-band deploy steps - 10 patch sets | 07:10 |
| *** iurygregory has joined #openstack-ironic | 07:11 | |
| iurygregory | good morning Ironic! | 07:12 |
| dtantsur | iurygregory, rpittau|afk, does any of you know why we still default to 384M of RAM in ironic-base? I think it breaks the SNMP job. | 07:13 |
| dtantsur | I can update the latter, but I wonder if the base value is still valid at all. | 07:13 |
| iurygregory | dtantsur, I was wondering this yesterday also | 07:14 |
| iurygregory | maybe 512 would be something better | 07:14 |
| dtantsur | iurygregory: the devstack plugin defaults to DIB, we need something that matches | 07:14 |
| dtantsur | 512 won't help | 07:14 |
| iurygregory | truth, sorry need more coffee XD | 07:15 |
| iurygregory | 2048? | 07:15 |
| iurygregory | I saw this for most of the dib jobs... | 07:16 |
| dtantsur | yeah | 07:18 |
| rpittau|afk | I thought I increased that, didn't I? No, I didn't.... | 07:18 |
| *** rpittau|afk is now known as rpittau | 07:18 | |
| dtantsur | and lower it for the jobs that still use tinyipa | 07:18 |
| rpittau | good morning ironic! o/ | 07:18 |
| dtantsur | o/ | 07:18 |
| iurygregory | rpittau, I think you added a value in each job | 07:18 |
| iurygregory | 2048 or more =) | 07:18 |
| rpittau | iurygregory: yeah, the thought of increasing it in the base job didn't leave my brain | 07:19 |
| iurygregory | but the default we didn't change maybe because we would need to change a lot of the tinyipa jobs to have small value | 07:19 |
| iurygregory | I can take care in few minutes if no one has started up to you guys =) | 07:20 |
| dtantsur | I haven't at least, still waking up | 07:20 |
| rpittau | ^ same | 07:20 |
| iurygregory | ok I'm starting here =) | 07:21 |
| iurygregory | 4GB for disk is ok right? | 07:24 |
| dtantsur | should be fine, it's still cirros | 07:26 |
| dtantsur | also, what on earth is happening to the inspector CI? | 07:27 |
| rpittau | dtantsur: that is something I noticed yesterday but didn't have time to check | 07:29 |
| *** ricolin has joined #openstack-ironic | 07:44 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Change default ram value https://review.opendev.org/718607 | 07:53 |
| * iurygregory now checks the other projects | 07:53 | |
| *** threestrands has quit IRC | 08:00 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/networking-baremetal master: Set ram value for tinyipa job https://review.opendev.org/715411 | 08:02 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-python-agent master: Change default ram for tinyipa jobs https://review.opendev.org/718611 | 08:08 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Change default ram for tinyipa jobs https://review.opendev.org/718613 | 08:13 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/python-ironicclient master: Set default ram for tinyipa job https://review.opendev.org/718614 | 08:17 |
| *** lucasagomes has joined #openstack-ironic | 08:18 | |
| dtantsur | iurygregory: cannot easily spot in the file, did you update the multinode jobs to the lower RAM? | 08:21 |
| iurygregory | dtantsur, it was set already | 08:21 |
| dtantsur | nice | 08:21 |
| iurygregory | https://github.com/openstack/ironic/blob/master/playbooks/legacy/grenade-dsvm-ironic-multinode-multitenant/run.yaml#L95 =) | 08:22 |
| openstackgerrit | Riccardo Pittau proposed openstack/python-ironicclient master: Convert ironicclient-tempest to dib https://review.opendev.org/716979 | 08:24 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/python-ironic-inspector-client master: Update python-ironic-inspector-client-tempest https://review.opendev.org/718616 | 08:24 |
| *** alexmcleod has joined #openstack-ironic | 08:28 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/python-ironic-inspector-client master: Remove python3 from job name https://review.opendev.org/718616 | 08:31 |
| iurygregory | rpittau, sushy is necessary or we are moving to dib? sushy-tempest-ironic-partition-redfish-src | 08:39 |
| rpittau | iurygregory: it's in the list for dib | 08:42 |
| rpittau | actually I'm going to add the patch | 08:42 |
| iurygregory | rpittau, ok so I won't push the one to change the ram in sushy | 08:42 |
| rpittau | iurygregory: yeah, I will just depend the change on the ironic one | 08:42 |
| openstackgerrit | Riccardo Pittau proposed openstack/sushy master: Convert ironicclient-tempest to dib https://review.opendev.org/718617 | 08:44 |
| openstackgerrit | Riccardo Pittau proposed openstack/sushy master: Convert sushy tempest to dib https://review.opendev.org/718617 | 08:45 |
| * rpittau would not mind some sushi | 08:45 | |
| iurygregory | kkkkkkkkkkkkkkkkkk | 08:45 |
| iurygregory | I was about to say WHAT? when I saw the commit message | 08:45 |
| rpittau | I blame the inventor of copy-paste :) | 08:46 |
| *** ociuhandu has quit IRC | 08:46 | |
| iurygregory | ++ | 08:46 |
| *** ociuhandu has joined #openstack-ironic | 08:46 | |
| openstackgerrit | Riccardo Pittau proposed openstack/sushy-tools master: Convert sushy-tools tempest to dib https://review.opendev.org/718618 | 08:51 |
| openstackgerrit | Riccardo Pittau proposed openstack/virtualbmc master: Convert virtualbmc tempest to dib https://review.opendev.org/718620 | 08:54 |
| openstackgerrit | Merged openstack/ironic-specs master: Cleanup py27 support https://review.opendev.org/718365 | 08:55 |
| *** derekh has joined #openstack-ironic | 08:56 | |
| *** tkajinam has quit IRC | 09:03 | |
| dtantsur | mgoddard: hey, could you request a stable/train ironic-ui release? apparently we forgot to release https://review.opendev.org/#/c/706783/ :( | 09:04 |
| patchbot | patch 706783 - ironic-ui (stable/train) - Repair creating an ironicclient instance (MERGED) - 1 patch set | 09:04 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add `network_data` field to ironic node object https://review.opendev.org/687910 | 09:13 |
| dtantsur | mgoddard: or I can do it around right now, don't worry | 09:14 |
| mgoddard | hey dtantsur, I'll propose one | 09:15 |
| dtantsur | mgoddard: ah, great, thanks! | 09:15 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Convert jobs to dib https://review.opendev.org/718627 | 09:25 |
| dtantsur | the CI seems largely non-working at this point :( | 09:26 |
| rpittau | dtantsur: any specific issue I can help checking ? | 09:26 |
| dtantsur | looking at https://review.opendev.org/#/c/718142/ it fails different jobs each time | 09:26 |
| patchbot | patch 718142 - ironic - Refactor AgentBase.heartbeat and process_next_step - 1 patch set | 09:26 |
| dtantsur | since the patch is just moving code around, I doubt that it's related | 09:26 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Convert jobs to dib https://review.opendev.org/718627 | 09:27 |
| dtantsur | it's the 2nd time I see https://zuul.opendev.org/t/openstack/build/638527be7aba44b59bb37c71dffe4a4a/log/logs/screen-n-cpu.txt#15859 though | 09:27 |
| rpittau | mmm wow | 09:28 |
| dtantsur | another job has connection errors from IPA https://zuul.opendev.org/t/openstack/build/46846d3f824344dba31c7538423569b7/log/controller/logs/ironic-bm-logs/node-0_console_2020-04-09-08:06:19_log.txt | 09:29 |
| rpittau | I saw grenade failing in a lot of other places too | 09:29 |
| dtantsur | so it's probably an increased rate of random failures | 09:29 |
| dtantsur | I raised it #openstack-nova, let's see what they say | 09:29 |
| *** k_mouza has joined #openstack-ironic | 09:30 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Add an ability to run in-band deploy steps https://review.opendev.org/698770 | 09:31 |
| openstackgerrit | Merged openstack/ironic master: Add sushy-cli to client libraries release list https://review.opendev.org/718465 | 09:31 |
| openstackgerrit | Merged openstack/ironic master: Added node multitenancy doc https://review.opendev.org/718174 | 09:31 |
| iurygregory | Guru Meditation LOL | 09:32 |
| iurygregory | I never heard about it | 09:32 |
| rpittau | dtantsur: same thins here https://zuul.opendev.org/t/openstack/build/9cf1bdee1e0d4489997e0702363b893c/log/logs/screen-n-cpu.txt#14441 | 09:32 |
| dtantsur | iurygregory: we support it as well: https://docs.openstack.org/ironic/latest/admin/gmr.html :) | 09:33 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Validate node `network_data` field contents https://review.opendev.org/714614 | 09:33 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add `get_node_network_data` method to NetworkInterface https://review.opendev.org/688184 | 09:33 |
| iurygregory | dtantsur, the name is funny | 09:33 |
| dtantsur | it is | 09:33 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add `get_node_network_data` to non-Neutron NetworkInterface https://review.opendev.org/688266 | 09:34 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Make redfish boot interface to use `network_data` https://review.opendev.org/688272 | 09:34 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add `get_node_network_data` to Neutron NetworkInterface https://review.opendev.org/688896 | 09:34 |
| *** k_mouza has quit IRC | 09:35 | |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add L3 boot section to the docs https://review.opendev.org/689844 | 09:35 |
| etingof | dates back to Commodore Amiga | 09:36 |
| rpittau | https://en.wikipedia.org/wiki/Guru_Meditation :) | 09:36 |
| rpittau | I always feared it in consoles! | 09:36 |
| dtantsur | overall, the situation doesn't look so bad http://zuul.openstack.org/builds?job_name=ironic-grenade-dsvm | 09:37 |
| dtantsur | we probably just have too many failures elsewhere | 09:37 |
| rpittau | right, the impression was worse than the stats | 09:38 |
| dtantsur | that's actually bad. if we had a bug in nova, we could nag them to fix it. | 09:39 |
| dtantsur | now we're probably just screwed :) | 09:39 |
| iurygregory | maybe something on the cloud provider side? | 09:40 |
| iurygregory | *we are just unlucky* | 09:40 |
| dtantsur | maybe? | 09:40 |
| iurygregory | I prefer the benefit of the doubt XD | 09:41 |
| *** ociuhandu has quit IRC | 09:45 | |
| *** k_mouza has joined #openstack-ironic | 09:50 | |
| *** ociuhandu has joined #openstack-ironic | 09:58 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Support burning configdrive into boot ISO https://review.opendev.org/687250 | 10:00 |
| *** rpittau is now known as rpittau|bbl | 10:13 | |
| *** k_mouza has quit IRC | 10:35 | |
| *** k_mouza has joined #openstack-ironic | 10:40 | |
| *** ociuhandu has quit IRC | 10:50 | |
| *** ociuhandu has joined #openstack-ironic | 10:51 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client master: Remove python3 from job name https://review.opendev.org/718616 | 10:53 |
| kaifeng | dtantsur: the gmr may not work as expected, it seems we setup signal handler in the conductor which overrides the sigusr2 used by gmr | 10:55 |
| *** ociuhandu has quit IRC | 10:57 | |
| * etingof has an impression that Python does not grok UNIX signals well | 10:57 | |
| dtantsur | kaifeng: ouch | 10:58 |
| dtantsur | etingof: it seems to be fine mostly, although you have to be careful with eventlet | 10:58 |
| etingof | "seems" is the key ;) somewhat depends on luck | 10:59 |
| etingof | e.g. https://stackoverflow.com/questions/52291768/python-and-signal-handlers | 10:59 |
| etingof | arguably, eventlet might be even more robust to signal interrupts in comparison with threads | 11:00 |
| *** mgoddard has quit IRC | 11:02 | |
| openstackgerrit | Verification of a change to openstack/ironic-python-agent failed: Remove unused version parameter in version header function https://review.opendev.org/718378 | 11:04 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add `network_data` field to ironic node object https://review.opendev.org/687910 | 11:04 |
| openstackgerrit | Merged openstack/tenks master: Fix issue with VirtualBMC client path https://review.opendev.org/718404 | 11:16 |
| *** k_mouza has quit IRC | 11:28 | |
| *** k_mouza has joined #openstack-ironic | 11:28 | |
| *** ociuhandu has joined #openstack-ironic | 11:32 | |
| *** k_mouza has quit IRC | 11:33 | |
| *** yedongcan has joined #openstack-ironic | 11:37 | |
| *** rh-jelabarre has joined #openstack-ironic | 11:41 | |
| *** ociuhandu has quit IRC | 11:48 | |
| *** ociuhandu has joined #openstack-ironic | 11:49 | |
| *** k_mouza has joined #openstack-ironic | 11:51 | |
| *** k_mouza has quit IRC | 11:51 | |
| *** k_mouza has joined #openstack-ironic | 11:52 | |
| *** ociuhandu has quit IRC | 11:55 | |
| *** rpittau|bbl is now known as rpittau | 11:57 | |
| *** mkrai has quit IRC | 12:12 | |
| *** igordc has quit IRC | 12:18 | |
| *** Qianbiao has joined #openstack-ironic | 12:21 | |
| *** rloo has joined #openstack-ironic | 12:23 | |
| *** ociuhandu has joined #openstack-ironic | 12:27 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Move logic for raid start sector to raid_utils https://review.opendev.org/718489 | 12:31 |
| gudrutis2 | Have anyone had issues with set-upping devstack recently? On newly setuped VM I get error that nodes are not available ( they stuck in 'clean wait'). http://paste.openstack.org/show/791872/ http://paste.openstack.org/show/791873/ . The configuration was working before. | 12:40 |
| *** iurygregory has quit IRC | 12:43 | |
| *** iurygregory has joined #openstack-ironic | 12:44 | |
| *** ociuhandu has quit IRC | 12:55 | |
| *** ociuhandu has joined #openstack-ironic | 12:55 | |
| dtantsur | gudrutis2: no idea, but we see all sorts of networking issues in the CI now | 12:56 |
| dtantsur | oh wait | 12:56 |
| dtantsur | gudrutis2: we've changed the default image type to DIB | 12:56 |
| dtantsur | it requires at least 2G RAM on the testing nodes | 12:57 |
| dtantsur | check you configuration | 12:57 |
| iurygregory | probably we need to update docs for local tests | 12:57 |
| dtantsur | very likely | 12:58 |
| *** cdearborn has joined #openstack-ironic | 12:59 | |
| *** ociuhandu has quit IRC | 13:00 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent master: Move logic to calculate raid sectors to raid_utils https://review.opendev.org/718695 | 13:04 |
| *** ociuhandu has joined #openstack-ironic | 13:04 | |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add `get_node_network_data` method to NetworkInterface https://review.opendev.org/688184 | 13:12 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add `get_node_network_data` to non-Neutron NetworkInterface https://review.opendev.org/688266 | 13:13 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Make redfish boot interface to use `network_data` https://review.opendev.org/688272 | 13:13 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add `get_node_network_data` to Neutron NetworkInterface https://review.opendev.org/688896 | 13:13 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add L3 boot section to the docs https://review.opendev.org/689844 | 13:13 |
| openstackgerrit | Riccardo Pittau proposed openstack/networking-baremetal master: Convert networking-baremetal job to dib https://review.opendev.org/718697 | 13:14 |
| openstackgerrit | Riccardo Pittau proposed openstack/python-ironic-inspector-client master: Convert job to dib https://review.opendev.org/718698 | 13:16 |
| Qianbiao | hello, i am running stein on ubuntu 18. When start inspector, i got an error. http://paste.openstack.org/show/791877/ | 13:18 |
| Qianbiao | any hint will be nice. | 13:19 |
| openstackgerrit | Merged openstack/ironic-inspector master: Set default boot option to netboot for jobs using cirros https://review.opendev.org/716613 | 13:27 |
| openstackgerrit | Verification of a change to openstack/ironic failed: Improve `redfish` set-boot-device behaviour https://review.opendev.org/711118 | 13:27 |
| gudrutis2 | dtantsur: how to set RAM amount in the configuration local.conf? | 13:28 |
| rpittau | gudrutis2: you need to change IRONIC_VM_SPECS_RAM | 13:28 |
| *** ociuhandu has quit IRC | 13:44 | |
| *** ociuhandu has joined #openstack-ironic | 13:44 | |
| *** tzumainn has joined #openstack-ironic | 13:46 | |
| *** ociuhandu has quit IRC | 13:47 | |
| *** ociuhandu has joined #openstack-ironic | 13:48 | |
| *** priteau has joined #openstack-ironic | 13:49 | |
| *** dougsz has joined #openstack-ironic | 13:49 | |
| tzumainn | TheJulia, hi! the node multitenancy doc merged; I think you mentioned there was something I should update after that happens... ? | 13:50 |
| Qianbiao | rpittau remember the eventlet bug i post yestoday. log is here: http://paste.openstack.org/show/791879/ | 13:52 |
| rpittau | tzumainn: that would be the cycle highligths, but I think the link was already added -> https://docs.openstack.org/ironic/latest/admin/node-multitenancy.html | 13:53 |
| tzumainn | rpittau, ah, okay - thanks! | 13:53 |
| *** rcernin has quit IRC | 13:54 | |
| rpittau | Qianbiao: I believe you have some issues with python libraries and dependencies on that system | 13:55 |
| Qianbiao | <rpittau> donot think so, eventlet!=0.18.3,!=0.20.1,>=0.18.2 # MIT this is defined in requirements.txt | 13:57 |
| Qianbiao | but i am sure, there are not only mokey patch bugs in eventlet except 0.20.1 and 0.18.3 | 13:57 |
| dtantsur | tzumainn: yep, I've updated the highlights with the link | 14:03 |
| tzumainn | dtantsur, thanks! | 14:03 |
| Qianbiao | <rpittau> you may try eventlet==0.20.0, it will be auto install on ubuntu 18.04 | 14:04 |
| rpittau | etingof: do we still need this https://review.opendev.org/708495 ? | 14:05 |
| patchbot | patch 708495 - sushy-tools - Fix attribute error in `set_boot_image()` - 4 patch sets | 14:05 |
| etingof | yes | 14:05 |
| etingof | it's a silly and trivial typo | 14:05 |
| *** Qianbiao has quit IRC | 14:09 | |
| rpioso | Good morning, ironic :-) | 14:11 |
| openstackgerrit | Ilya Etingof proposed openstack/python-ironicclient master: Add `network_data` ironic node attribute support https://review.opendev.org/688299 | 14:12 |
| openstackgerrit | Ilya Etingof proposed openstack/python-ironicclient master: Add `network_data` ironic node attribute support https://review.opendev.org/688299 | 14:13 |
| *** tkajinam has joined #openstack-ironic | 14:13 | |
| *** Goneri has joined #openstack-ironic | 14:14 | |
| *** ociuhandu has quit IRC | 14:25 | |
| *** ociuhandu has joined #openstack-ironic | 14:25 | |
| *** mgoddard has joined #openstack-ironic | 14:26 | |
| openstackgerrit | Merged openstack/ironic master: Change default ram value https://review.opendev.org/718607 | 14:30 |
| *** ociuhandu has quit IRC | 14:30 | |
| gudrutis2 | rpittau: thanks, it did the trick | 14:36 |
| rpittau | gudrutis2: good :) | 14:36 |
| *** Qianbiao has joined #openstack-ironic | 14:43 | |
| arne_wiebalck | rpittau: should this RAM update be added to our docs? | 14:44 |
| Qianbiao | hello ironic. | 14:45 |
| rpittau | arne_wiebalck: we'll probably have to change some things in the docs as now we default the main ci job to dib centos8 image, so RAM change included, yes | 14:46 |
| *** ociuhandu has joined #openstack-ironic | 14:47 | |
| Qianbiao | Hi, need help with ironic-inspector service start. it shows could not connect to ironic. But I am using no auth. | 14:47 |
| arne_wiebalck | rpittau: I was thinking of the devstack local.conf | 14:47 |
| Qianbiao | And I checked the code stack, it shows if auth type is none, it does will raise error. | 14:47 |
| Qianbiao | log is here: http://paste.openstack.org/show/791877/ | 14:48 |
| iurygregory | arne_wiebalck, we should =) | 14:48 |
| rpittau | arne_wiebalck: in dev-quickstart ? maybe | 14:49 |
| iurygregory | probably update to either match the defaults (dib etc) or using tinyipa | 14:49 |
| iurygregory | dev-quickstart ++ | 14:49 |
| *** ociuhandu has quit IRC | 14:50 | |
| *** ociuhandu has joined #openstack-ironic | 14:51 | |
| rpittau | I'm going to update the jobs description too :) | 14:53 |
| openstackgerrit | Merged openstack/ironic master: Support burning configdrive into boot ISO https://review.opendev.org/687250 | 14:53 |
| dtantsur | arne_wiebalck: btw I've written a bit more text to "Usage pattern", please check when you have a minute | 14:54 |
| dtantsur | (this is re the whitepaper) | 14:54 |
| arne_wiebalck | dtantsur: will do, thanks! | 14:54 |
| arne_wiebalck | Qianbiao: which version is this? | 14:56 |
| Qianbiao | <arne_wiebalck> stein | 14:56 |
| openstackgerrit | Christopher Dearborn proposed openstack/ironic stable/train: Include iDRAC specific Redfish library in train https://review.opendev.org/718721 | 14:59 |
| Qianbiao | @ arne_wiebalck may it a compatibility issue with ironicclient lib? | 14:59 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Update jobs description https://review.opendev.org/718722 | 14:59 |
| *** yedongcan has left #openstack-ironic | 15:01 | |
| cdearborn | hey folks, we have some people working on OSP16 (train based), and we've found that sushy-oem-idrac is not included in the container running ironic-conductor | 15:02 |
| dtantsur | cdearborn: I'm not sure anybody has implemented that | 15:02 |
| dtantsur | I mean, the TripleO/Kolla change | 15:02 |
| cdearborn | I talked with the kolla folks, and they include everything in driver-requirements.txt in the container, but sushy-oem-idrac is missing in train | 15:02 |
| cdearborn | so: https://review.opendev.org/#/c/718721/ | 15:03 |
| patchbot | patch 718721 - ironic (stable/train) - Include iDRAC specific Redfish library in train - 1 patch set | 15:03 |
| dtantsur | strictly speaking, the relevant ironic code doesn't exist in train | 15:03 |
| dtantsur | so this patch is a no-go, unless I'm missing something | 15:03 |
| dtantsur | cdearborn: ^^ | 15:03 |
| dtantsur | yeah, idrac-redfish-virtual-media appeared in ussuri | 15:04 |
| rpittau | cdearborn: I confirm, that would not be usable in train | 15:04 |
| arne_wiebalck | Qianbiao: I have no idea to be honest ... so maybe :) | 15:04 |
| arne_wiebalck | Qianbiao: I am running on Stein as well. | 15:04 |
| Qianbiao | dtantsur i found u update python-ironicclient | 15:05 |
| *** gyee has joined #openstack-ironic | 15:05 | |
| Qianbiao | and it seems it will break the code and cause my issue | 15:05 |
| Qianbiao | do u have a time look at it? | 15:05 |
| Qianbiao | dtantsur the commit id is e8914a7ef9eb582441a7067f2663e19209f32822 | 15:06 |
| *** rh-jelabarre has quit IRC | 15:06 | |
| cdearborn | dtantsur, rpittau, ok - thx | 15:06 |
| dtantsur | Qianbiao: could you elaborate?\ | 15:06 |
| dtantsur | this patch is not supposed to cause behavior changes | 15:07 |
| arne_wiebalck | Qianbiao: I have ironic-inspector-8.2.2, ironicclient-2.7.3, ironic-lib-2.16.3, inspector-client-3.5.0 | 15:07 |
| Qianbiao | when run ironic-inspector(stein) with [ironic]no auth | 15:07 |
| Qianbiao | dtantsur will cause issue: http://paste.openstack.org/show/791877/ | 15:08 |
| Qianbiao | <arne_wiebalck> try latest python-ironicclient(4.0.0) | 15:08 |
| dtantsur | note that my patch doesn't cause a problem, it merely provides you with an error message | 15:09 |
| Qianbiao | Hmmm.. but the code in stein does not pass session or token when it's noauth | 15:09 |
| Qianbiao | it means it will always raise error? | 15:09 |
| dtantsur | Qianbiao: you seem to be using train ironicclient with stein inspector | 15:10 |
| dtantsur | I'd recommend against that | 15:10 |
| dtantsur | I'll see if we can fix ironic-inspector in stein | 15:10 |
| Qianbiao | ：0 | 15:10 |
| Qianbiao | i am not sure about the version mapping.. | 15:10 |
| Qianbiao | but i check the requirments.txt in stein. | 15:10 |
| Qianbiao | it only asks: python-ironicclient>=2.3.0 | 15:11 |
| Qianbiao | so, when we install a fresh ironic-inspector, it will always download the latest? | 15:12 |
| rpittau | Qianbiao: in theory you're not supposed to install anything that is more recent than what's defined in upper-constraints | 15:12 |
| dtantsur | Qianbiao: that's why we usually recommend using upper-constraints https://github.com/openstack/requirements/blob/stable/stein/upper-constraints.txt | 15:12 |
| dtantsur | Qianbiao: if you file a bug on storyboard, I can look into fixing ironic-inspector to become forward-compatible | 15:13 |
| *** ociuhandu has quit IRC | 15:13 | |
| Qianbiao | i see, it seems it my fault.. | 15:13 |
| Qianbiao | i forget the upper-constraints.txt | 15:13 |
| dtantsur | I'd fix it anyway, even though using the constraints is recommended | 15:14 |
| Qianbiao | But that should still be an issue i think, if user install with apt install ironic-inspector, it will install latest still? | 15:14 |
| rpittau | Qianbiao: that really depends on dependencies at system level | 15:15 |
| *** ociuhandu has joined #openstack-ironic | 15:15 | |
| Qianbiao | i see.. | 15:16 |
| Qianbiao | <rpittau> so, eventlet is the same reason? | 15:16 |
| rpittau | Qianbiao: most likely | 15:18 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector stable/stein: [WIP] [stable-only] Make ironicclient creation forward-compatible https://review.opendev.org/718728 | 15:20 |
| *** ociuhandu has quit IRC | 15:20 | |
| dtantsur | Qianbiao: something like ^^^ (needs updating unit-tests) | 15:20 |
| Qianbiao | thanks, i will try fix my env tomorrow. | 15:21 |
| Qianbiao | see u. | 15:21 |
| *** Qianbiao is now known as Qianbiao|leave | 15:21 | |
| *** rh-jelabarre has joined #openstack-ironic | 15:21 | |
| dtantsur | note that most of us are out tomorrow and on Monday | 15:22 |
| Qianbiao|leave | sure, see u next moday then. | 15:22 |
| *** Qianbiao|leave has quit IRC | 15:22 | |
| arne_wiebalck | Kind reminder for everyone interested in the next whitepaper session to mark their availabilities: https://doodle.com/poll/kyd5vvgmg68vxduv | 15:31 |
| dtantsur | etingof: ^^ | 15:32 |
| etingof | ack | 15:32 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/networking-baremetal master: Use openstacksdk for ironic connection https://review.opendev.org/683971 | 15:33 |
| iurygregory | rpittau, I fixed the conflict to see how it goes ^ | 15:33 |
| *** ociuhandu has joined #openstack-ironic | 15:37 | |
| dtantsur | arne_wiebalck: out of curiosity, did you folks end up using the active nodes introspection in prod? | 15:38 |
| rpittau | iurygregory: we're not using mock anymore | 15:38 |
| iurygregory | ops the rebase went wrong | 15:40 |
| iurygregory | for some reason didn't show conflicts in the test-requirements | 15:40 |
| arne_wiebalck | dtantsur: still ongoing, we have agreed on which data we want (and have filed or are preparing the corresponding changes for python-hardware), need to get this into the IPA, and we are working on a translator of this data from S3 into the inventory system we picked | 15:40 |
| arne_wiebalck | dtantsur: this is all for standard inspection for now | 15:41 |
| arne_wiebalck | dtantsur: only afterwards we will be looking into active node inspection | 15:41 |
| dtantsur | ah, I see. okay | 15:41 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/networking-baremetal master: Use openstacksdk for ironic connection https://review.opendev.org/683971 | 15:42 |
| arne_wiebalck | dtantsur: but we tried already and it works, but I cannot make any comments on the scalability, for instance | 15:42 |
| dtantsur | we'll see :) | 15:43 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector stable/stein: [stable-only] Make ironicclient initialization future-proof https://review.opendev.org/718728 | 15:43 |
| iurygregory | rpittau, sorry now it's updated properly | 15:45 |
| *** tkajinam has quit IRC | 15:49 | |
| rpittau | iurygregory: np, thanks | 15:50 |
| openstackgerrit | Merged openstack/ironic-python-agent master: Remove unused version parameter in version header function https://review.opendev.org/718378 | 15:52 |
| *** lucasagomes has quit IRC | 15:53 | |
| arne_wiebalck | dtantsur: two (hopefully quick to review) doc patches for your attention https://review.opendev.org/#/c/701926/ and https://review.opendev.org/#/c/718485/ | 15:57 |
| patchbot | patch 701926 - ironic - [doc] Images need some metadata for software RAID - 3 patch sets | 15:57 |
| patchbot | patch 718485 - ironic - [doc] Remove the device selection limitation for S... - 1 patch set | 15:57 |
| rpittau | bye all, talk tomorrow or tuesday! o/ | 15:59 |
| *** rpittau is now known as rpittau|afk | 15:59 | |
| *** iurygregory has quit IRC | 16:03 | |
| *** diurnalist has joined #openstack-ironic | 16:09 | |
| *** ociuhandu has quit IRC | 16:10 | |
| *** ociuhandu has joined #openstack-ironic | 16:28 | |
| *** ociuhandu has quit IRC | 16:38 | |
| dtantsur | Easter holidays are ahead, see you on Tuesday! | 16:46 |
| *** dtantsur is now known as dtantsur|afk | 16:46 | |
| *** k_mouza has quit IRC | 16:49 | |
| *** alexmcleod has quit IRC | 16:55 | |
| *** hjensas has quit IRC | 17:00 | |
| *** derekh has quit IRC | 17:01 | |
| *** mgoddard has quit IRC | 17:03 | |
| *** ricolin has quit IRC | 17:04 | |
| *** diurnalist has quit IRC | 17:05 | |
| *** dougsz has quit IRC | 17:07 | |
| *** dougsz has joined #openstack-ironic | 17:07 | |
| *** dougsz has quit IRC | 17:12 | |
| *** ociuhandu has joined #openstack-ironic | 17:19 | |
| *** priteau has quit IRC | 17:22 | |
| *** ociuhandu has quit IRC | 17:25 | |
| openstackgerrit | Aija Jaunteva proposed openstack/ironic master: Update iDRAC doc about soft power off timeout https://review.opendev.org/718761 | 17:46 |
| *** k_mouza has joined #openstack-ironic | 18:05 | |
| *** k_mouza has quit IRC | 18:05 | |
| *** ociuhandu has joined #openstack-ironic | 18:07 | |
| *** ociuhandu has quit IRC | 18:20 | |
| arne_wiebalck | Bye everyone, have a good break, see you on Tuesday o/ | 18:22 |
| *** ociuhandu has joined #openstack-ironic | 18:22 | |
| *** ociuhandu has quit IRC | 18:27 | |
| cdearborn | hey folks, we've been working on a couple of things for ussuri, but they haven't landed yet. was wondering if we need to request a feature freeze exception, and if so, how to do that? | 18:42 |
| cdearborn | one of the items is implementing cleaning steps in the iDRAC driver to: reset the iDRAC, clear the job queue, and put the iDRAC into a good state, all using redfish | 18:43 |
| cdearborn | we already have cleaning steps to do these things using wsman. the idea is to bring redfish support up to parity with wsman support in this area | 18:43 |
| cdearborn | the second change could be considered a bug fix. it is to set pxe_enabled during inspection when using redfish and the iDRAC driver. | 18:46 |
| *** mgoddard has joined #openstack-ironic | 18:46 | |
| cdearborn | the generic redfish driver does not set pxe_enabled, and works by only creating ports in ironic for enabled interfaces. | 18:46 |
| cdearborn | this is basically a poor man's version of setting pxe_enabled. this approach may be fine for dev but will not work when a node is connected to multiple networks with multiple enabled ports | 18:47 |
| *** dougsz has joined #openstack-ironic | 18:53 | |
| *** ociuhandu has joined #openstack-ironic | 19:20 | |
| *** Lucas_Gray has joined #openstack-ironic | 19:20 | |
| *** Lucas_Gray has quit IRC | 19:26 | |
| *** Lucas_Gray has joined #openstack-ironic | 19:27 | |
| *** ociuhandu has quit IRC | 19:50 | |
| *** Lucas_Gray has quit IRC | 19:53 | |
| *** priteau has joined #openstack-ironic | 19:54 | |
| *** priteau has quit IRC | 20:05 | |
| *** dougsz has quit IRC | 20:10 | |
| openstackgerrit | Christopher Dearborn proposed openstack/ironic master: Fix issue where server fails to reboot https://review.opendev.org/714491 | 20:14 |
| *** dougsz has joined #openstack-ironic | 20:22 | |
| *** Lucas_Gray has joined #openstack-ironic | 20:23 | |
| openstackgerrit | Christopher Dearborn proposed openstack/ironic master: Fix issue where server fails to reboot https://review.opendev.org/714491 | 20:24 |
| *** markguz_ has joined #openstack-ironic | 20:26 | |
| openstackgerrit | Christopher Dearborn proposed openstack/ironic master: Fix issue where server fails to reboot https://review.opendev.org/714491 | 20:27 |
| *** dougsz has quit IRC | 20:27 | |
| *** Lucas_Gray has quit IRC | 20:28 | |
| markguz_ | Hi i'm getting an error http://paste.openstack.org/show/791902/ when spinning up a system. it's failing on creating the configdrive | 20:34 |
| markguz_ | I'm running stable/train | 20:35 |
| markguz_ | my config file is here http://paste.openstack.org/show/791901/ | 20:35 |
| markguz_ | i'm completely stumped. I know swift is working ok, glance is working fine. as usual i seem to be great at having problems that no one else does. Google didn't help a bit | 20:37 |
| *** jtomasek has quit IRC | 20:41 | |
| openstackgerrit | Hang Yang proposed openstack/ironic-lib master: Convert str to bytes for py3 socket compatibility https://review.opendev.org/718795 | 20:55 |
| markguz_ | it seems to fail when trying to create the temp file. | 20:57 |
| JayF | markguz_: I suspect it's failing due to py3 compat, the same class of issue Hang fixed just now in ironic-lib | 20:58 |
| markguz_ | JayF: is there a patch ? | 20:59 |
| markguz_ | oh.. | 20:59 |
| markguz_ | duh | 20:59 |
| markguz_ | Hang... it's just there.. | 20:59 |
| JayF | not for your specific issue, but hte patch above Hang just submitted is for a similar issue elsewhere in the code | 20:59 |
| markguz_ | oh ic. | 20:59 |
| JayF | (at least according to my hypothesis given your err) | 20:59 |
| markguz_ | well i did install it with python3 | 21:00 |
| *** ociuhandu has joined #openstack-ironic | 21:01 | |
| markguz_ | i had a feeling it might be a bug. | 21:01 |
| markguz_ | i'll see if i can't fix it | 21:01 |
| *** ociuhandu has quit IRC | 21:06 | |
| markguz_ | JayF: changing fileobj.write(configdrive) to fileobj.write(configdrive.encode()) line 3719 of ironic/conductor/manager.py fixed it | 21:07 |
| JayF | markguz_: If you're setup to contribute to openstack (or want an excuse to) that's a great first-time bugfix, alternatively, you can just put that info in a bug | 21:08 |
| JayF | nice job getting yourself fixed up though! | 21:08 |
| *** Lucas_Gray has joined #openstack-ironic | 21:09 | |
| markguz_ | JayF: ok i'll look into that getting set up to contribute. thanks! | 21:11 |
| JayF | https://docs.openstack.org/contributors/code-and-documentation/quick-start.html should be a good place to get started; glhf! | 21:11 |
| *** Lucas_Gray has quit IRC | 21:17 | |
| *** hyang has joined #openstack-ironic | 21:18 | |
| *** Lucas_Gray has joined #openstack-ironic | 21:18 | |
| *** Lucas_Gray has quit IRC | 21:38 | |
| *** Lucas_Gray has joined #openstack-ironic | 21:39 | |
| *** hyang has quit IRC | 22:19 | |
| *** k_mouza has joined #openstack-ironic | 22:33 | |
| openstackgerrit | Christopher Dearborn proposed openstack/ironic master: Fix issue where server fails to reboot https://review.opendev.org/714491 | 22:49 |
| *** tkajinam has joined #openstack-ironic | 22:53 | |
| *** hjensas has joined #openstack-ironic | 23:00 | |
| *** Lucas_Gray has quit IRC | 23:01 | |
| *** Lucas_Gray has joined #openstack-ironic | 23:02 | |
| *** k_mouza has quit IRC | 23:12 | |
| *** Lucas_Gray has quit IRC | 23:18 | |
| *** rloo has quit IRC | 23:55 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!