| *** hoangcx has joined #openstack-ironic | 00:01 | |
| *** hoangcx2 has quit IRC | 00:04 | |
| *** fragatina has joined #openstack-ironic | 00:43 | |
| *** sabeen has joined #openstack-ironic | 00:49 | |
| *** hoangcx has quit IRC | 00:59 | |
| *** hoangcx has joined #openstack-ironic | 01:03 | |
| *** fragatin_ has joined #openstack-ironic | 01:43 | |
| openstackgerrit | Tang Chen proposed openstack/python-ironicclient: Use osc_lib instead of cliff https://review.openstack.org/334803 | 01:45 |
|---|---|---|
| *** fragatina has quit IRC | 01:47 | |
| *** mmedvede_ has joined #openstack-ironic | 01:54 | |
| *** baoli has joined #openstack-ironic | 02:07 | |
| openstackgerrit | Tang Chen proposed openstack/python-ironicclient: Use osc_lib instead of cliff https://review.openstack.org/334803 | 02:18 |
| openstackgerrit | Tang Chen proposed openstack/python-ironicclient: Use osc-lib instead of openstackclient https://review.openstack.org/333311 | 02:18 |
| openstackgerrit | Naohiro Tamura proposed openstack/python-ironicclient: Add a new OSC command for Inject NMI https://review.openstack.org/357661 | 02:29 |
| *** yuanying has quit IRC | 02:46 | |
| *** yuanying has joined #openstack-ironic | 02:50 | |
| *** jcoufal has joined #openstack-ironic | 02:51 | |
| *** baoli has quit IRC | 02:55 | |
| *** baoli has joined #openstack-ironic | 02:59 | |
| *** baoli has quit IRC | 02:59 | |
| openstackgerrit | Naohiro Tamura proposed openstack/python-ironicclient: Support soft reboot and soft power off with timeout for OSC https://review.openstack.org/357627 | 03:05 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/358285 | 03:08 |
| *** xuao has joined #openstack-ironic | 03:08 | |
| *** krtaylor has quit IRC | 03:10 | |
| *** baoli has joined #openstack-ironic | 03:12 | |
| *** jcoufal has quit IRC | 03:16 | |
| *** baoli has quit IRC | 03:16 | |
| *** jcoufal has joined #openstack-ironic | 03:18 | |
| *** jcoufal has quit IRC | 03:20 | |
| *** vishwanathj has joined #openstack-ironic | 03:28 | |
| *** vikrant has joined #openstack-ironic | 03:39 | |
| *** yogi has joined #openstack-ironic | 03:41 | |
| *** yuanying has quit IRC | 03:45 | |
| *** yuanying has joined #openstack-ironic | 03:48 | |
| *** vmud213 has joined #openstack-ironic | 03:58 | |
| *** yibo has quit IRC | 04:02 | |
| *** yibo has joined #openstack-ironic | 04:03 | |
| *** vishwanathj has quit IRC | 04:11 | |
| *** yibo has quit IRC | 04:16 | |
| *** yibo has joined #openstack-ironic | 04:16 | |
| *** michauds has joined #openstack-ironic | 04:34 | |
| *** david-lyle has quit IRC | 04:37 | |
| *** hoangcx has quit IRC | 04:38 | |
| *** sabeen has quit IRC | 04:49 | |
| *** krtaylor has joined #openstack-ironic | 04:50 | |
| openstackgerrit | Tuan proposed openstack/ironic-inspector: Remove import Objects instead of Modules https://review.openstack.org/358398 | 04:53 |
| *** pcaruana has quit IRC | 04:55 | |
| *** vsaienko has joined #openstack-ironic | 04:58 | |
| *** amotoki has joined #openstack-ironic | 05:11 | |
| *** fragatina has joined #openstack-ironic | 05:20 | |
| *** fragatin_ has quit IRC | 05:20 | |
| openstackgerrit | Naohiro Tamura proposed openstack/python-ironicclient: Support soft reboot and soft power off with timeout for OSC https://review.openstack.org/357627 | 05:20 |
| *** aswadr_ has joined #openstack-ironic | 05:22 | |
| *** zhangjl has joined #openstack-ironic | 05:24 | |
| *** pester has quit IRC | 05:25 | |
| *** pester has joined #openstack-ironic | 05:26 | |
| *** jtomasek has joined #openstack-ironic | 05:28 | |
| *** hoangcx has joined #openstack-ironic | 05:28 | |
| openstackgerrit | Yibo Cai proposed openstack/ironic: Support multi arch deployment https://review.openstack.org/322632 | 05:30 |
| *** vsaienko has quit IRC | 05:33 | |
| *** links has joined #openstack-ironic | 05:38 | |
| *** links has quit IRC | 05:38 | |
| *** eglute has quit IRC | 05:46 | |
| *** eglute has joined #openstack-ironic | 05:47 | |
| *** eglute has quit IRC | 05:48 | |
| *** baoli has joined #openstack-ironic | 05:49 | |
| *** eglute has joined #openstack-ironic | 05:49 | |
| openstackgerrit | Tuan proposed openstack/ironic: Using assertIsNone() is preferred over assertEqual() https://review.openstack.org/358405 | 05:49 |
| *** eglute has quit IRC | 05:49 | |
| *** eglute has joined #openstack-ironic | 05:50 | |
| *** eglute has quit IRC | 05:51 | |
| *** baoli has quit IRC | 05:53 | |
| *** eglute has joined #openstack-ironic | 05:54 | |
| *** eglute has quit IRC | 05:54 | |
| *** eglute has joined #openstack-ironic | 06:05 | |
| *** Lord_Phoenix has joined #openstack-ironic | 06:11 | |
| *** rbartal has joined #openstack-ironic | 06:13 | |
| *** michauds has quit IRC | 06:15 | |
| *** baoli has joined #openstack-ironic | 06:16 | |
| *** baoli has quit IRC | 06:16 | |
| *** mjura has joined #openstack-ironic | 06:17 | |
| *** mjura has quit IRC | 06:22 | |
| *** mjura has joined #openstack-ironic | 06:23 | |
| *** ohamada has joined #openstack-ironic | 06:45 | |
| *** rcernin has joined #openstack-ironic | 06:51 | |
| *** pcaruana has joined #openstack-ironic | 06:52 | |
| *** appu103_ has joined #openstack-ironic | 07:04 | |
| *** tesseract- has joined #openstack-ironic | 07:09 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Remove neutron client workarounds https://review.openstack.org/354588 | 07:12 |
| *** milan has joined #openstack-ironic | 07:18 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic-specs: Collect Ironic agent ramdisk logs https://review.openstack.org/337223 | 07:21 |
| *** zhangyufei has joined #openstack-ironic | 07:25 | |
| *** rcernin has quit IRC | 07:25 | |
| *** jpich has joined #openstack-ironic | 07:30 | |
| *** vsaienko has joined #openstack-ironic | 07:33 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/ironic-python-agent: Trust certificate based on fingerprint https://review.openstack.org/358457 | 07:44 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/ironic-python-agent: Trust certificate based on fingerprint https://review.openstack.org/358457 | 07:47 |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/ironic-python-agent: Trust certificate based on fingerprint https://review.openstack.org/358457 | 07:47 |
| *** aparnav has joined #openstack-ironic | 07:56 | |
| *** appu103_ has quit IRC | 07:56 | |
| *** ifarkas_afk is now known as ifarkas | 07:56 | |
| openstackgerrit | Aparna proposed openstack/proliantutils: Add hpssa support only in raid mode https://review.openstack.org/347978 | 07:59 |
| *** ccamacho has joined #openstack-ironic | 07:59 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:01 | |
| *** athomas has joined #openstack-ironic | 08:10 | |
| *** moshele has joined #openstack-ironic | 08:16 | |
| *** derekh has joined #openstack-ironic | 08:16 | |
| *** aparnav has quit IRC | 08:23 | |
| *** appu103_ has joined #openstack-ironic | 08:24 | |
| *** aparnav has joined #openstack-ironic | 08:24 | |
| *** appu103_ has quit IRC | 08:24 | |
| *** jaosorior has joined #openstack-ironic | 08:31 | |
| *** vsaienko has quit IRC | 08:33 | |
| *** lucas-afk is now known as lucasagomes | 08:34 | |
| *** yuanying has quit IRC | 08:34 | |
| *** vsaienko has joined #openstack-ironic | 08:35 | |
| *** yuanying has joined #openstack-ironic | 08:35 | |
| *** vmud213 has quit IRC | 08:37 | |
| *** vmud213 has joined #openstack-ironic | 08:38 | |
| openstackgerrit | Markos Chandras proposed openstack/bifrost: Split --syntax-check and --list-tasks steps in test-bifrost.sh https://review.openstack.org/358477 | 08:42 |
| *** yuanying has quit IRC | 08:47 | |
| *** yuanying has joined #openstack-ironic | 08:49 | |
| *** tiendc has joined #openstack-ironic | 08:50 | |
| *** Goneri has joined #openstack-ironic | 08:51 | |
| openstackgerrit | Tuan proposed openstack/ironic-inspector: Import modules instead of objects https://review.openstack.org/358398 | 08:53 |
| *** vsaienko has quit IRC | 08:57 | |
| *** baoli has joined #openstack-ironic | 09:00 | |
| *** vnogin has joined #openstack-ironic | 09:01 | |
| *** aarefiev_ is now known as aarefiev | 09:06 | |
| *** ohamada has quit IRC | 09:06 | |
| aarefiev | morning! | 09:06 |
| *** ohamada has joined #openstack-ironic | 09:06 | |
| *** baoli has quit IRC | 09:06 | |
| *** ohamada has quit IRC | 09:06 | |
| *** ohamada_ has joined #openstack-ironic | 09:06 | |
| *** ohamada_ has quit IRC | 09:07 | |
| *** ohamada_ has joined #openstack-ironic | 09:07 | |
| *** ohamada_ has quit IRC | 09:07 | |
| sambetts | Morning all | 09:21 |
| *** skazi has joined #openstack-ironic | 09:22 | |
| skazi | can someone take a look at: https://review.openstack.org/#/c/354685/ ? | 09:23 |
| skazi | am I right that those settings are not needed anymore in current version? | 09:24 |
| *** athomas has quit IRC | 09:29 | |
| *** athomas has joined #openstack-ironic | 09:34 | |
| vmud213 | morning sambetts, aarefiev and everyone else | 09:34 |
| lucasagomes | aarefiev, sambetts vmud213 morning | 09:36 |
| *** dtantsur|afk is now known as dtantsur | 09:37 | |
| vmud213 | morning lucasagomes,dtantsur | 09:37 |
| dtantsur | Morning Ironic, morning lucasagomes, aarefiev, sambetts, vmud213 | 09:37 |
| lucasagomes | hi there | 09:37 |
| *** sborkows has joined #openstack-ironic | 09:45 | |
| aarefiev | morning sambetts, vmud213, lucasagomes, dtantsur | 09:49 |
| *** e0ne has joined #openstack-ironic | 09:54 | |
| milan | morning aarefiev dtantsur lucasagomes sambetts skazi vmud213! :) | 09:56 |
| milan | and morning Ironic! | 09:57 |
| dtantsur | hey milan | 09:57 |
| vmud213 | morning milan :) | 09:57 |
| milan | dtantsur, o/ review time? :P | 09:57 |
| tangchen | milan, lucasagomes, dtantsur: morning guys. :) | 09:57 |
| dtantsur | tee/coffee first :) | 09:57 |
| dtantsur | morning tangchen | 09:57 |
| milan | morning tangchen! :) | 09:57 |
| *** sborkows has quit IRC | 09:57 | |
| milan | dtantsur, +1 that! :D | 09:57 |
| *** sborkows has joined #openstack-ironic | 09:58 | |
| * dtantsur got some grey hair trying to make tftp work in tripleo overcloud :( | 09:58 | |
| lucasagomes | dtantsur, T.T | 09:58 |
| sambetts | dtantsur: :( | 09:58 |
| lucasagomes | selinux ? | 09:58 |
| milan | ouch, you'd better pull that one out before it spreads ;) | 09:58 |
| dtantsur | lucasagomes, disabled | 09:58 |
| lucasagomes | dtantsur, :-O | 09:59 |
| dtantsur | derek is looking now, so I'll wait for now :) | 09:59 |
| milan | p.s. states patch is worth one as well ;) | 09:59 |
| * lucasagomes pictures dan walsh weeping in his head | 09:59 | |
| dtantsur | +988, -83 wow that's a patch Oo | 09:59 |
| milan | Oops! :D | 10:00 |
| dtantsur | milan, do you think it's possible to split it? | 10:00 |
| dtantsur | (I'm not saying it's necessary is, just asking) | 10:00 |
| milan | dtantsur, not quite I'm afraid, at least I don't see how right now | 10:00 |
| dtantsur | okie, I'll get to it this afternoon | 10:00 |
| milan | dtantsur, thanks, much appreciate that! | 10:01 |
| milan | I think sambetts and aarefiev wait for your opinion as well | 10:01 |
| * milan relocates | 10:03 | |
| *** milan has quit IRC | 10:04 | |
| *** Goneri has quit IRC | 10:07 | |
| sborkows | dtantsur: You reviewed my patch about week ago: https://review.openstack.org/#/c/352295/ and I do not understand how can I enable again the IPA collector, because I cannot recall that I have disabled it earlier. What did you mean? | 10:12 |
| openstackgerrit | Aparna proposed openstack/proliantutils: Raise exception when HPSSA controller not available https://review.openstack.org/347791 | 10:13 |
| dtantsur | sborkows, just bring back this: https://review.openstack.org/#/c/352295/2/devstack/plugin.sh | 10:13 |
| dtantsur | my comment there was actually incorrect, and nobody noticed :) | 10:14 |
| dtantsur | sorry for that | 10:14 |
| *** chihhsin has quit IRC | 10:15 | |
| *** Goneri has joined #openstack-ironic | 10:18 | |
| sborkows | dtantsur: ok, thanks for replay, I will soon update a patch. | 10:19 |
| *** zhangjl1 has joined #openstack-ironic | 10:20 | |
| vdrok | morning all, happy monday! :) | 10:21 |
| *** zhangjl has quit IRC | 10:22 | |
| openstackgerrit | Szymon Borkowski proposed openstack/ironic-inspector: Add PCI devices plugin to inspector https://review.openstack.org/352295 | 10:26 |
| dtantsur | morning vdrok | 10:27 |
| vdrok | good morning dtantsur | 10:27 |
| *** chihhsin has joined #openstack-ironic | 10:28 | |
| *** tiendc has quit IRC | 10:29 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node provision state change notification https://review.openstack.org/348437 | 10:31 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add create and delete notifications for ironic resources https://review.openstack.org/356541 | 10:32 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add update notifications for ironic resources https://review.openstack.org/357799 | 10:32 |
| *** narasimha_SV has joined #openstack-ironic | 10:35 | |
| *** aparnav has quit IRC | 10:51 | |
| *** ipukha has joined #openstack-ironic | 10:58 | |
| *** milan has joined #openstack-ironic | 10:59 | |
| *** hoangcx has quit IRC | 11:00 | |
| *** openstackgerrit has quit IRC | 11:03 | |
| *** zhangyufei has quit IRC | 11:03 | |
| *** openstackgerrit has joined #openstack-ironic | 11:04 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Replace DB API call to object's method in iLO drivers https://review.openstack.org/347403 | 11:05 |
| *** aparnav has joined #openstack-ironic | 11:13 | |
| dtantsur | short walk, brb | 11:14 |
| *** dtantsur is now known as dtantsur|brb | 11:14 | |
| *** appu1034 has joined #openstack-ironic | 11:14 | |
| *** amotoki has quit IRC | 11:15 | |
| *** amotoki has joined #openstack-ironic | 11:15 | |
| *** fragatina has quit IRC | 11:15 | |
| *** fragatina has joined #openstack-ironic | 11:16 | |
| *** dhellmann has quit IRC | 11:17 | |
| *** trown has quit IRC | 11:17 | |
| *** dhellmann has joined #openstack-ironic | 11:17 | |
| *** aparnav has quit IRC | 11:18 | |
| *** trown has joined #openstack-ironic | 11:19 | |
| openstackgerrit | avnish proposed openstack/ironic-inspector: Use upper constraints for all jobs in tox.ini https://review.openstack.org/358564 | 11:25 |
| openstackgerrit | Merged openstack/ironic: Using assertIsNone() is preferred over assertEqual() https://review.openstack.org/358405 | 11:25 |
| openstackgerrit | Tang Chen proposed openstack/ironic: Move "server_profile_template_uri" to REQUIRED_ON_PROPERTIES https://review.openstack.org/347127 | 11:26 |
| *** vmud213 has quit IRC | 11:29 | |
| *** appu1034 has quit IRC | 11:31 | |
| *** thrash|g0ne is now known as thrash | 11:31 | |
| *** moshele has quit IRC | 11:35 | |
| *** openstackstatus has quit IRC | 11:36 | |
| *** moshele has joined #openstack-ironic | 11:37 | |
| *** openstackstatus has joined #openstack-ironic | 11:38 | |
| *** ChanServ sets mode: +v openstackstatus | 11:38 | |
| jroll | morning everyone | 11:38 |
| TheJulia | Good morning | 11:38 |
| *** dprince has joined #openstack-ironic | 11:43 | |
| mat128|afk | good morning jroll, TheJulia and everyone else | 11:44 |
| *** mat128|afk is now known as mat128 | 11:44 | |
| *** tiendc has joined #openstack-ironic | 11:45 | |
| TheJulia | vsaienk0: good idea on maintenance state for node selection, I'll make that change once coffee has woken me up in a little bit. I also replied to your question as to why another method. | 11:47 |
| *** jaosorior has quit IRC | 11:50 | |
| *** jaosorior has joined #openstack-ironic | 11:50 | |
| vdrok | morning jroll TheJulia and mat128 | 11:50 |
| vsaienk0 | devananda: as it was mentioned by JayF https://review.openstack.org/#/c/228496/ is already merged. Regarding developer documentation http://docs.openstack.org/developer/ironic/dev/ironic-multitenant-networking.html the guide seems to be correct. Could you please tell what is missing in the developer documentation regarding multitenancy? | 11:51 |
| tangchen | vdork: morning Vladyslav, thanks for approving the patch. :) | 11:51 |
| *** tiendc has quit IRC | 11:53 | |
| jroll | vsaienk0: maybe https://review.openstack.org/#/c/345557/2 ? :) | 11:54 |
| *** skramaja has joined #openstack-ironic | 11:58 | |
| *** e0ne has quit IRC | 11:59 | |
| *** e0ne has joined #openstack-ironic | 12:00 | |
| *** baoli has joined #openstack-ironic | 12:00 | |
| *** baoli_ has joined #openstack-ironic | 12:01 | |
| *** baoli has quit IRC | 12:05 | |
| *** zhangjl1 has left #openstack-ironic | 12:06 | |
| *** moshele has quit IRC | 12:07 | |
| *** moshele has joined #openstack-ironic | 12:08 | |
| *** alex_xu has quit IRC | 12:08 | |
| lucasagomes | jroll, morning, what time you are thinking about releasing the ironic-lib ? | 12:09 |
| *** alex_xu has joined #openstack-ironic | 12:09 | |
| lucasagomes | cause I'm testing the operators thing this morning changing IPA and Ironic and the code in ironic-lib seems good | 12:10 |
| lucasagomes | I just need to update the commit message to include the bug # cause I forgot | 12:10 |
| * lucasagomes will do it now | 12:10 | |
| *** vsaienko has joined #openstack-ironic | 12:10 | |
| jroll | lucasagomes: whenever it's ready this week, dmitry can release it | 12:11 |
| lucasagomes | jroll, cool | 12:11 |
| skazi | can anyone confirm that this is correct: https://review.openstack.org/#/c/354685/ ? | 12:12 |
| skazi | I need this for my deployment script (to avoid unnecessary changes in the config files) | 12:13 |
| tangchen | lucasagomes: Hi Lucas, this one again. I have updated the patch to solve vdrok's comment. And vdrok has helped to approve it. So begging for another +2 if you are OK with it. https://review.openstack.org/#/c/347127/ | 12:16 |
| lucasagomes | skazi, that lines came from https://review.openstack.org/#/c/156092/ you may want to add the author if this patch in your review | 12:16 |
| lucasagomes | tangchen, will take a look in a few | 12:16 |
| tangchen | lucasagomes: Thanks. :) | 12:17 |
| vsaienk0 | Ironic'ers: once you have a time, please review https://review.openstack.org/#/c/343684/. The spec describes Ironic-Neutron notification mechanism. PoC that passed multitenancy CI is https://review.openstack.org/#/c/357995/. I want to discuss the spec on todays meeting. | 12:17 |
| skazi | lucasagomes, I tracked the config back to the original commit | 12:17 |
| skazi | which was quite some time ago | 12:17 |
| skazi | those changes were just migration of settings when the syntax changes | 12:17 |
| skazi | // changed | 12:18 |
| skazi | the original lines were from somewhere around 2014 | 12:18 |
| lucasagomes | tangchen, the only change was that deprecated message? | 12:18 |
| skazi | but I'll add the authors to the review | 12:18 |
| skazi | maybe it will help :) | 12:18 |
| lucasagomes | tangchen, +a | 12:19 |
| tangchen | lucasagomes: Yes | 12:19 |
| tangchen | Thanks. :) | 12:19 |
| lucasagomes | skazi, cool | 12:20 |
| lucasagomes | skazi, I will take a look at the patch after lunch | 12:20 |
| *** lucasagomes is now known as lucas-hungry | 12:20 | |
| skazi | lucasagomes, thx | 12:20 |
| *** e0ne has quit IRC | 12:21 | |
| *** e0ne has joined #openstack-ironic | 12:21 | |
| * TheJulia gets more coffee | 12:24 | |
| *** MattMan has quit IRC | 12:26 | |
| *** MattMan has joined #openstack-ironic | 12:26 | |
| *** jaosorior is now known as jaosorior_brb | 12:26 | |
| *** e0ne has quit IRC | 12:31 | |
| *** e0ne has joined #openstack-ironic | 12:32 | |
| *** bfournie has quit IRC | 12:38 | |
| *** xuao has quit IRC | 12:39 | |
| *** jcoufal has joined #openstack-ironic | 12:41 | |
| *** Lord_Phoenix has quit IRC | 12:50 | |
| *** rloo has joined #openstack-ironic | 12:52 | |
| *** e0ne has quit IRC | 12:54 | |
| *** e0ne has joined #openstack-ironic | 12:55 | |
| *** ccamacho is now known as ccamacho|lunch | 12:56 | |
| *** pester has quit IRC | 12:57 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client: Fix functional tests broken by the latest refactoring https://review.openstack.org/357795 | 12:58 |
| *** clenimar has joined #openstack-ironic | 12:58 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Replace DB API call to object's method in iLO drivers https://review.openstack.org/347403 | 12:58 |
| *** skramaja has left #openstack-ironic | 12:59 | |
| *** mgagne_ has quit IRC | 13:00 | |
| *** mgagne_ has joined #openstack-ironic | 13:00 | |
| *** mgoddard has joined #openstack-ironic | 13:00 | |
| *** eset has quit IRC | 13:10 | |
| openstackgerrit | Ricardo Araújo Santos proposed openstack/ironic: Adds instructions to perform nodes migration https://review.openstack.org/357454 | 13:12 |
| openstackgerrit | Ricardo Araújo Santos proposed openstack/ironic: Adds instructions to perform nodes migration https://review.openstack.org/357454 | 13:12 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Replace DB API call to object's method in iLO drivers https://review.openstack.org/347403 | 13:13 |
| *** vikrant has quit IRC | 13:14 | |
| *** bfournie has joined #openstack-ironic | 13:16 | |
| *** dtantsur|brb is now known as dtantsur | 13:16 | |
| *** ppiela has joined #openstack-ironic | 13:16 | |
| *** lucas-hungry is now known as lucasagomes | 13:18 | |
| *** rloo has quit IRC | 13:20 | |
| *** rloo has joined #openstack-ironic | 13:20 | |
| *** superdan is now known as dansmith | 13:21 | |
| *** adreznec has joined #openstack-ironic | 13:23 | |
| *** adreznec has quit IRC | 13:25 | |
| *** adreznec has joined #openstack-ironic | 13:26 | |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with Chassis. https://review.openstack.org/343796 | 13:27 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-lib: Add match_root_device_hints() to the utils.py module https://review.openstack.org/358000 | 13:30 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-lib: Extend parse_root_device_hints to support operators https://review.openstack.org/348953 | 13:30 |
| *** rbudden has joined #openstack-ironic | 13:30 | |
| *** yuriyz__ has joined #openstack-ironic | 13:33 | |
| *** vnogin_ has joined #openstack-ironic | 13:33 | |
| *** aarefiev_ has joined #openstack-ironic | 13:33 | |
| *** e0ne_ has joined #openstack-ironic | 13:34 | |
| *** vsaienk01 has joined #openstack-ironic | 13:36 | |
| *** ipukha1 has joined #openstack-ironic | 13:36 | |
| *** aarefiev has quit IRC | 13:36 | |
| *** e0ne has quit IRC | 13:36 | |
| *** vsaienk0 has quit IRC | 13:36 | |
| *** hjanssen-hpe has joined #openstack-ironic | 13:37 | |
| *** hpe-hj has joined #openstack-ironic | 13:37 | |
| *** yuriyz_ has quit IRC | 13:37 | |
| *** vnogin has quit IRC | 13:37 | |
| *** ipukha has quit IRC | 13:37 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Add functional test for wait_for_finish https://review.openstack.org/357804 | 13:38 |
| *** cdearborn has joined #openstack-ironic | 13:38 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Add functional tests for CLI https://review.openstack.org/357864 | 13:38 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Increase verbosity for functional tests https://review.openstack.org/358650 | 13:40 |
| *** vishwanathj has joined #openstack-ironic | 13:44 | |
| *** sabeen has joined #openstack-ironic | 13:44 | |
| *** wajdi has joined #openstack-ironic | 13:45 | |
| *** sdake has joined #openstack-ironic | 13:46 | |
| openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/358285 | 13:47 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Substrate fixes and changes for adoption testing https://review.openstack.org/344974 | 13:47 |
| *** ccamacho|lunch is now known as ccamacho | 13:50 | |
| *** sdake_ has joined #openstack-ironic | 13:50 | |
| xavierr | good morning Ironic | 13:50 |
| *** ipukha1 has quit IRC | 13:51 | |
| *** sdake has quit IRC | 13:53 | |
| jroll | if anyone has time for an easy-ish review, this is needed for the next os-api-ref release: https://review.openstack.org/#/c/357922/2 | 13:54 |
| *** ipukha has joined #openstack-ironic | 13:57 | |
| dtantsur | morning jroll | 13:57 |
| jroll | hey dtantsur, fyi I gave you "access" to release libraries this week (told release team) | 13:57 |
| dtantsur | cool | 13:58 |
| dtantsur | I guess we're waiting for lucasagomes' root device hints stuff, then release ironic-lib? | 13:58 |
| jroll | should probably get wsman and drac clients released, too | 13:58 |
| jroll | yeah | 13:58 |
| jroll | oh and the metrics docs patch for ironic-lib | 13:58 |
| * lucasagomes needs to release staging drivers as well | 13:59 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-lib: Add match_root_device_hints() to the utils.py module https://review.openstack.org/358000 | 13:59 |
| jroll | lucasagomes: if it's not in g-r, you can release at any time, jfyi | 13:59 |
| dtantsur | jroll, docs patch is nice to have, but not requires for a release. still will try to get it in | 13:59 |
| lucasagomes | jroll, yeah, it's independent | 13:59 |
| jroll | dtantsur: indeed, has my +2 already | 13:59 |
| *** mgoddard_ has joined #openstack-ironic | 13:59 | |
| dtantsur | yeah, I think staging drivers can release any time, but it still worth releasing | 13:59 |
| jroll | +1 | 13:59 |
| lucasagomes | but the conversation just reminded me that I have to do it :D so I added to my todo list here | 14:00 |
| dtantsur | heh | 14:00 |
| jroll | :) | 14:00 |
| * dtantsur urgently hacks on ironic-inspector-client gate improvements :) | 14:00 | |
| *** anush_ has joined #openstack-ironic | 14:00 | |
| xavierr | dtantsur: I need to apply a configuration in the machine before inspection and remove that configuration right after, however the inspection is started by an asynchronous call from ironic via eventlet and there is no callbacks to plug the code to apply and remove those configurations. Any clue how to execute these steps (apply->inspect->remove)? | 14:01 |
| *** vsaienko has quit IRC | 14:01 | |
| dtantsur | xavierr, maybe watching introspection status? | 14:01 |
| *** mgoddard has quit IRC | 14:03 | |
| xavierr | dtantsur: please, could you clarify a bit more? | 14:04 |
| sambetts | jroll: I'm working on my interface attach/detach stuff, and it turns out the conductor manager UT framework doesn't handle the dynamicly built driver stuff yet, so that a fun one :-P | 14:04 |
| *** causten has joined #openstack-ironic | 14:05 | |
| dtantsur | xavierr, disregard that; it will work, but I think we should switch https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/inspector.py#L113 to use our thread pool, then you'll get a Future object back | 14:05 |
| openstackgerrit | Merged openstack/ironic: Get ready for os-api-ref sphinx theme change https://review.openstack.org/357922 | 14:05 |
| dtantsur | except for, I'm not sure we can get access to the conductor, hmmm | 14:06 |
| dtantsur | (to access its thread pool) | 14:06 |
| sborkows | dtantsur: Can you look at https://review.openstack.org/#/c/352295/4/ironic_inspector/plugins/pci_devices.py ? I think we cannot call that option 'aliases' as this is the single entry, which can be stated in file many times | 14:09 |
| *** jcoufal has quit IRC | 14:10 | |
| dtantsur | sborkows, ok, then "alias". I'm mostly concerned with repeating "pci" | 14:10 |
| *** cinerama has quit IRC | 14:11 | |
| *** jcoufal has joined #openstack-ironic | 14:11 | |
| xavierr | dtantsur: so when you say 'use our thread pool' you mean create task (from task_manager) instead use eventlet? | 14:12 |
| dtantsur | xavierr, I mean somehow using https://github.com/openstack/ironic/blob/master/ironic/conductor/base_manager.py#L268 | 14:13 |
| dtantsur | (maybe it's not a great idea, but it will allow us to attach a callback to the returned Future object) | 14:13 |
| *** syed_ has joined #openstack-ironic | 14:17 | |
| xavierr | dtantsur: oh, I see. in the worst case, could we implement a periodic in our implementation of inspect interface to check if the introspector process is over and remove the configuration (Server Profile)? | 14:17 |
| dtantsur | xavierr, you definitely can | 14:18 |
| *** sdake_ is now known as sdake | 14:18 | |
| *** mtanino has joined #openstack-ironic | 14:19 | |
| xavierr | dtantsur: another question, for implement a inspect interface to a driver would be necessary a spec, rfe? | 14:19 |
| dtantsur | xavierr, RFE; it might get approved without a spec | 14:19 |
| xavierr | ok dtantsur :) | 14:20 |
| openstackgerrit | Merged openstack/ironic-inspector: Modify the SQL's string from double quote to single quote. https://review.openstack.org/347795 | 14:21 |
| xavierr | dtantsur: would be great if we hed a 'before_inspect' and 'after_inspect' to inspect interface? would this be a good idea? | 14:21 |
| xavierr | had* | 14:21 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Modify the SQL's string from double quote to single quote. https://review.openstack.org/358673 | 14:21 |
| dtantsur | xavierr, I'm not sure I can imagine how it is going to look.. | 14:22 |
| xavierr | np dtantsur, thank you | 14:23 |
| *** ifarkas has left #openstack-ironic | 14:23 | |
| *** ifarkas has joined #openstack-ironic | 14:23 | |
| lucasagomes | jroll, dtantsur T.T found a problem with the specs matcher at oslo utils | 14:28 |
| lucasagomes | jroll, dtantsur http://paste.openstack.org/show/562224/ | 14:28 |
| *** thiagop has joined #openstack-ironic | 14:28 | |
| lucasagomes | it does not handle spaces >.< | 14:28 |
| lucasagomes | disk models could contain spaces in the string | 14:28 |
| thiagop | Good morning, Ironicers | 14:28 |
| * lucasagomes cries | 14:28 | |
| jroll | lucasagomes: does nova's handle spaces? | 14:28 |
| lucasagomes | jroll, apparently not, will test with the old version of specs matcher | 14:28 |
| lucasagomes | 1 sec lemme see | 14:28 |
| jroll | lucasagomes: or what if you wrap in quotes or something? :/ | 14:29 |
| lucasagomes | jroll, same, tried as well | 14:29 |
| dtantsur | morning thiagop | 14:29 |
| dtantsur | lucasagomes, oh damn... | 14:29 |
| lucasagomes | In [33]: grammar.parseString('s== "foo bar"') | 14:29 |
| lucasagomes | Out[33]: (['s==', '"foo'], {}) | 14:29 |
| jroll | ouch | 14:29 |
| dtantsur | jroll, we require specs for new drivers, don't we (re https://bugs.launchpad.net/ironic/+bug/1613198)? | 14:29 |
| openstack | Launchpad bug 1613198 in Ironic "[RFE] Xclarity boot driver" [Wishlist,In progress] - Assigned to Andy Yan (yanchao3) | 14:29 |
| jroll | dtantsur: it's been a while :) I think yes | 14:30 |
| * dtantsur writes a comment | 14:30 | |
| thiagop | jroll dtantsur we did wrote a spec :) | 14:31 |
| thiagop | did write* | 14:31 |
| lucasagomes | jroll, well I guess it's not broken then because the old version of nova also does not handle it | 14:31 |
| lucasagomes | wonder how can ironic users use it, it's possible to use the "<in>" operator but still... it's a bit misleading | 14:32 |
| lucasagomes | s/bit// | 14:32 |
| jroll | lucasagomes: yeah, idk | 14:33 |
| rloo | lucasagomes: can't you "just" add a patch to fix it in the oslo library? | 14:34 |
| *** cinerama has joined #openstack-ironic | 14:34 | |
| rloo | morning everyone! | 14:34 |
| jroll | hey rloo :) | 14:35 |
| lucasagomes | rloo, looking into it, tho, I'm unsure about the right way to fix it because the code in nova (which oslo.utils derived from) also doesn't support spaces | 14:35 |
| rloo | lucasagomes: is it because nova didn't have a need to support spaces, or because spaces would break nova's use case? | 14:36 |
| dtantsur | jroll, I'd like to bulk rfe-approved all these "[RFE] Metrics for XXX driver" requests created by JayF. objections? | 14:36 |
| dtantsur | morning rloo | 14:36 |
| jroll | dtantsur: +1 | 14:36 |
| lucasagomes | rloo, idk, I literally just found the problem | 14:36 |
| rloo | morning dtantsur | 14:36 |
| rloo | dtantsur: btw, do you know where you saw that osc doesn't support '-'? (https://review.openstack.org/#/c/357505/) | 14:37 |
| rloo | dtantsur: i looked in osc doc but didn't see it. might have missed it though. | 14:37 |
| dtantsur | rloo, I think I've figured it out myself. I think it's related to how they use entry points. I've also never seen a single OSC command using dashes. | 14:38 |
| rloo | dtantsur: oh. cuz i coded it and it worked. but i can change it. | 14:38 |
| dtantsur | oh, interesting.. I still think they tend to use spaces | 14:39 |
| rloo | dtantsur: i looked at some of the existing osc commands. they don't seem as complicated as ours. or maybe we've made ours complicated. dunno. | 14:39 |
| rloo | dtantsur: so you'd prefer 'openstack baremetal node boot device set' vs 'openstack baremetal node bootdevice set' ? | 14:39 |
| dtantsur | good question... I think we tend to go with spaces so far | 14:40 |
| rloo | dtantsur: we haven't used spaces for a noun or an object | 14:40 |
| rloo | dtantsur: eg we have 'configdrive' not 'config drive' | 14:40 |
| dtantsur | mmmm | 14:41 |
| *** tiendc has joined #openstack-ironic | 14:42 | |
| * lucasagomes think about a way to work-around the problem... perhaps using urlib.parse.quote() to normalize the strings | 14:44 | |
| openstackgerrit | Szymon Borkowski proposed openstack/ironic-inspector: Add PCI devices plugin to inspector https://review.openstack.org/352295 | 14:47 |
| *** vishwanathj has quit IRC | 14:53 | |
| *** vishwanathj has joined #openstack-ironic | 14:53 | |
| * jroll afk for a while | 14:55 | |
| *** sborkows has quit IRC | 14:59 | |
| *** jaosorior_brb is now known as jaosorior | 15:00 | |
| rloo | dtantsur: i was wrong, it is --config-drive, there is no command with 'configdrive' in it. | 15:04 |
| dtantsur | I see, yeah, makes sense | 15:05 |
| dtantsur | we probably have it in the old "ironic" tool | 15:05 |
| rloo | dtantsur: i can't keep any of it straight. Just tell me what to do :D | 15:06 |
| rloo | dtantsur: maybe we ought to reach out to the osc folks... | 15:06 |
| dtantsur | I vote for "openstack baremetal node boot device set" | 15:06 |
| dtantsur | yeah, +1 | 15:06 |
| rloo | dtantsur: ok | 15:06 |
| *** pester has joined #openstack-ironic | 15:09 | |
| *** mjura has quit IRC | 15:09 | |
| *** fxpester has quit IRC | 15:11 | |
| *** mgoddard has joined #openstack-ironic | 15:14 | |
| *** mgoddard_ has quit IRC | 15:15 | |
| *** mjturek2 has quit IRC | 15:15 | |
| *** causten_ has joined #openstack-ironic | 15:17 | |
| *** causten has quit IRC | 15:20 | |
| *** narasimha_SV_ has joined #openstack-ironic | 15:21 | |
| vdrok | good morning xavierr thiagop and rloo | 15:22 |
| narasimha_SV_ | while pxe booting int the ironic compute logs I am getting below warning | 15:22 |
| narasimha_SV_ | WARNING stevedore.named [req-129c0140-6c83-4bf4-b709-43e0416eae2f bd3c57c095d14c08b3b15cde9724dd09 bfcc2f804f1847e0a3554f44136401f2 - - -] Could not load instance_network_info | 15:22 |
| rloo | hi vdrok | 15:22 |
| narasimha_SV_ | by which I am unable to get the network | 15:22 |
| narasimha_SV_ | is this kind of issue with some neutron net settings | 15:22 |
| narasimha_SV_ | http://paste.openstack.org/show/562239/ | 15:24 |
| *** causten has joined #openstack-ironic | 15:24 | |
| narasimha_SV_ | these are the commands which I used to create the networks required for Ironic as per the documentation | 15:24 |
| narasimha_SV_ | Am I missing anything here ??? | 15:24 |
| thiagop | vdrok: o/ | 15:26 |
| thiagop | rloo: morning! | 15:26 |
| rloo | hi thiagop | 15:26 |
| *** narasimha_SV_ has quit IRC | 15:26 | |
| *** causten_ has quit IRC | 15:27 | |
| *** Administrator__ has quit IRC | 15:36 | |
| *** ifarkas is now known as ifarkas_afk | 15:37 | |
| *** Administrator__ has joined #openstack-ironic | 15:37 | |
| vdrok | narasimha_SV: hm, instance_network_info is a kind of hook in nova, and it seems that stevedore can not find it as it is not in setup.cfg? | 15:37 |
| vdrok | narasimha_SV: you'd better ask in nova channel i think, it does not look like something ironic-related | 15:37 |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Tests for testing node-create command https://review.openstack.org/262055 | 15:37 |
| *** harlowja_at_home has joined #openstack-ironic | 15:40 | |
| * devananda waves from NYC | 15:42 | |
| jlvillal | Good morning everyone | 15:42 |
| *** baoli_ has quit IRC | 15:43 | |
| dtantsur | morning devananda, jlvillal | 15:43 |
| vdrok | good morning devananda and jlvillal :) | 15:44 |
| *** aNupoisc has joined #openstack-ironic | 15:45 | |
| jlvillal | Privyet dtantsur vdrok :) | 15:46 |
| dtantsur | :) | 15:46 |
| *** moshele has quit IRC | 15:47 | |
| *** hshiina has joined #openstack-ironic | 15:47 | |
| openstackgerrit | Thiago Paiva Brito proposed openstack/ironic: Bring configurations from tempest to ironic_tempest_plugin https://review.openstack.org/355591 | 15:47 |
| openstackgerrit | Merged openstack/python-ironicclient: Add docs target to tox.ini https://review.openstack.org/358011 | 15:48 |
| *** michauds has joined #openstack-ironic | 15:50 | |
| thiagop | morning devananda jlvillal | 15:50 |
| *** milan has quit IRC | 15:51 | |
| jlvillal | :) | 15:52 |
| *** Sukhdev has joined #openstack-ironic | 15:52 | |
| devananda | I'll be hopping around cafe's today. might miss some of the neutron meeting, but should make the ironic meeting | 15:52 |
| devananda | shameless plug for my api-ref and doc updates that still need to land: https://review.openstack.org/353117 and https://review.openstack.org/356788 | 15:54 |
| *** sdake has quit IRC | 15:54 | |
| *** sdake has joined #openstack-ironic | 15:55 | |
| *** mjturek1 has joined #openstack-ironic | 15:59 | |
| *** mgoddard_ has joined #openstack-ironic | 16:01 | |
| *** kirankv has quit IRC | 16:02 | |
| *** bradjones has quit IRC | 16:03 | |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add power state change notifications https://review.openstack.org/321865 | 16:04 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add notification base classes and docs https://review.openstack.org/298461 | 16:04 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Switch Inspector interface to pass keystoneauth sessions https://review.openstack.org/357963 | 16:04 |
| *** mgoddard has quit IRC | 16:04 | |
| *** vmud213 has joined #openstack-ironic | 16:04 | |
| *** lazy_prince has quit IRC | 16:05 | |
| *** kirankv has joined #openstack-ironic | 16:05 | |
| *** bradjones has joined #openstack-ironic | 16:06 | |
| *** bradjones has quit IRC | 16:06 | |
| *** bradjones has joined #openstack-ironic | 16:06 | |
| *** praneshp has joined #openstack-ironic | 16:06 | |
| *** praneshp has quit IRC | 16:06 | |
| *** e0ne_ has quit IRC | 16:07 | |
| vdrok | mariojv: around? | 16:07 |
| mariojv | vdrok: hi, yes | 16:07 |
| *** lazy_prince has joined #openstack-ironic | 16:07 | |
| *** xavierr has quit IRC | 16:09 | |
| mariojv | vdrok: did you want to chat about the notifications patch? | 16:09 |
| vdrok | mariojv: I'm really sorry but I think I missed one more place in your patch :) https://review.openstack.org/#/c/298461/26/ironic/objects/notification.py@144 | 16:09 |
| mariojv | ooh, that could re-raise the exception, right? | 16:09 |
| vdrok | mariojv: do you think it's worth checking that key is in self.fields? | 16:09 |
| mariojv | vdrok: yes | 16:09 |
| *** anush_ has quit IRC | 16:10 | |
| mariojv | vdrok: i don't think that would even raise the exception if it's nullable | 16:10 |
| mariojv | vdrok: so perhaps it would be better to just remove that line? | 16:10 |
| *** anush_ has joined #openstack-ironic | 16:11 | |
| vdrok | mariojv: yeah, might be, if we can't set what we want something went wrong | 16:11 |
| vdrok | and we raise anyway whether it's nullable or not | 16:12 |
| vdrok | yep, that makes sense to me | 16:12 |
| *** causten has quit IRC | 16:12 | |
| mariojv | ok, i will remove the line | 16:13 |
| *** maurosr has quit IRC | 16:13 | |
| *** causten has joined #openstack-ironic | 16:13 | |
| mariojv | running tests first then updating it | 16:14 |
| *** maurosr has joined #openstack-ironic | 16:14 | |
| *** jpich has quit IRC | 16:16 | |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add power state change notifications https://review.openstack.org/321865 | 16:18 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add notification base classes and docs https://review.openstack.org/298461 | 16:18 |
| mariojv | vdrok: updated | 16:18 |
| *** hoangcx has joined #openstack-ironic | 16:19 | |
| *** fragatina has quit IRC | 16:19 | |
| vdrok | mariojv: thx, looking :) | 16:19 |
| *** fragatina has joined #openstack-ironic | 16:19 | |
| NobodyCam | Good Morning Ironic'ers :) | 16:21 |
| dtantsur | morning NobodyCam | 16:21 |
| NobodyCam | hey Hey dtantsur Good morning ... Did you have a good weekend | 16:22 |
| dtantsur | great Saturday, awful Sunday :) how does it count? | 16:22 |
| vdrok | morning NobodyCam ! | 16:23 |
| NobodyCam | lol :( average outs to a average weekend :p | 16:23 |
| devananda | g'morning, NobodyCam ! | 16:23 |
| NobodyCam | morning vdrok | 16:23 |
| NobodyCam | morning devananda :) | 16:23 |
| dtantsur | so, it was average :) and for you? | 16:24 |
| NobodyCam | about the same... to average days.. nothing supper exciting here | 16:24 |
| lucasagomes | NobodyCam, morning | 16:24 |
| NobodyCam | rewired the RV entertainment system :p | 16:24 |
| NobodyCam | morning lucasagomes | 16:25 |
| dtantsur | sounds like some fun :D | 16:25 |
| NobodyCam | lol :) | 16:25 |
| *** causten has quit IRC | 16:25 | |
| *** tesseract- has quit IRC | 16:28 | |
| TheJulia | NobodyCam: I did something slightly crazier... got a jeep wrangler as a tow vehicale for the RV :) The re-wiring stuff is still sitting in my amazon cart though :) | 16:29 |
| *** pcaruana has quit IRC | 16:29 | |
| NobodyCam | TheJulia: ++++ those make great toads | 16:30 |
| *** vmud213 has quit IRC | 16:32 | |
| *** lucasagomes is now known as lucas-afk | 16:32 | |
| * lucas-afk will be back for the meeting | 16:32 | |
| *** vmud213 has joined #openstack-ironic | 16:33 | |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Add tests for testing node-delete command https://review.openstack.org/251458 | 16:33 |
| *** rbrud has joined #openstack-ironic | 16:35 | |
| *** athomas has quit IRC | 16:36 | |
| *** harlowja_at_home has quit IRC | 16:37 | |
| *** jmccrory_away is now known as jmccrory | 16:37 | |
| *** vmud213 has quit IRC | 16:43 | |
| *** Goneri has quit IRC | 16:44 | |
| *** vmud213 has joined #openstack-ironic | 16:45 | |
| *** mgoddard_ has quit IRC | 16:46 | |
| openstackgerrit | Sam Betts proposed openstack/ironic: [WIP] Add Network Interface Attach and Detach APIs https://review.openstack.org/327046 | 16:46 |
| *** fragatina has quit IRC | 16:47 | |
| sambetts | mat128: ^ a much more complete version | 16:49 |
| *** mgoddard has joined #openstack-ironic | 16:49 | |
| *** aNupoisc has quit IRC | 16:50 | |
| *** Goneri has joined #openstack-ironic | 16:50 | |
| openstackgerrit | Rodion Promyshlennikov proposed openstack/python-ironicclient: Add baremetal port delete command to OSC plugin https://review.openstack.org/346075 | 16:51 |
| Sukhdev | sambetts : can you tell me those two config steps that you mentioned - I can test inspector feature today | 16:51 |
| Sukhdev | sambetts : other than ironic config, I do not mess with any other configs :-) | 16:52 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Remove neutron client workarounds https://review.openstack.org/354588 | 16:52 |
| Sukhdev | sambetts : or better yet, if you do not mind shoot me something in email at sukhdev@arista.com | 16:53 |
| *** rpioso has joined #openstack-ironic | 16:53 | |
| *** dims has quit IRC | 16:54 | |
| *** rama_y has joined #openstack-ironic | 16:54 | |
| *** jaosorior has quit IRC | 16:55 | |
| *** derekh has quit IRC | 16:55 | |
| *** dims has joined #openstack-ironic | 16:56 | |
| *** lucas-afk is now known as lucasagomes | 16:58 | |
| jroll | meeting time \o/ | 16:59 |
| *** tiendc has quit IRC | 17:01 | |
| vgadiraj | is there a link to the meeting? | 17:04 |
| *** vmud213 has quit IRC | 17:05 | |
| mat128 | vgadiraj: #openstack-meeting-3 | 17:05 |
| mat128 | on irc | 17:05 |
| *** amotoki has quit IRC | 17:06 | |
| *** baoli has joined #openstack-ironic | 17:07 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Log hook names as we run them https://review.openstack.org/358779 | 17:08 |
| NobodyCam | :) | 17:09 |
| *** fragatina has joined #openstack-ironic | 17:09 | |
| *** xavierr has joined #openstack-ironic | 17:18 | |
| *** xavierr has left #openstack-ironic | 17:21 | |
| *** xavierr has joined #openstack-ironic | 17:21 | |
| *** aNupoisc has joined #openstack-ironic | 17:23 | |
| *** e0ne has joined #openstack-ironic | 17:26 | |
| *** rajinir has joined #openstack-ironic | 17:28 | |
| *** hshiina has quit IRC | 17:30 | |
| *** jhooker has quit IRC | 17:30 | |
| *** yogi has quit IRC | 17:30 | |
| *** jhooker has joined #openstack-ironic | 17:31 | |
| *** aNupoisc has quit IRC | 17:31 | |
| *** phschwartz has quit IRC | 17:31 | |
| *** fragatina has quit IRC | 17:33 | |
| *** fragatina has joined #openstack-ironic | 17:33 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add tempest tests for node Adoption https://review.openstack.org/344975 | 17:34 |
| *** sabeen has quit IRC | 17:41 | |
| jroll | vgadiraj: btw, meeting info is here https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 17:41 |
| *** sabeen has joined #openstack-ironic | 17:42 | |
| lucasagomes | calling it a day | 17:42 |
| lucasagomes | have a great evening all! | 17:43 |
| thiagop | good night lucasagomes | 17:43 |
| lucasagomes | jroll, enjoy the holidays | 17:43 |
| * vdrok leaves too | 17:43 | |
| jroll | thanks lucasagomes | 17:43 |
| vdrok | jroll: have a good rest! :) | 17:43 |
| *** lucasagomes is now known as lucas-dinner | 17:43 | |
| rloo | bye lucasagomes, vdrok | 17:43 |
| jroll | see you all on thursday :D | 17:43 |
| vdrok | good night everyone | 17:43 |
| lucas-dinner | see y'all :-) | 17:43 |
| *** e0ne has quit IRC | 17:43 | |
| *** sambetts is now known as sambetts|afk | 17:43 | |
| devananda | g'night vdrik, lucas-dinner ! | 17:44 |
| NobodyCam | night vdrok lucas-dinner jroll | 17:44 |
| thiagop | night vdrok | 17:44 |
| *** fellypefca has joined #openstack-ironic | 17:44 | |
| dtantsur | g'night folks | 17:45 |
| *** sabeen has quit IRC | 17:45 | |
| thiagop | night dtantsur | 17:45 |
| *** e0ne has joined #openstack-ironic | 17:46 | |
| *** Goneri has quit IRC | 17:50 | |
| NobodyCam | night dtantsur | 17:55 |
| *** narasimha_SV has quit IRC | 17:56 | |
| *** bnemec-pto is now known as bnemec | 17:56 | |
| *** e0ne has quit IRC | 17:57 | |
| fellypefca | Hey everyone, does anyone knows devstack runs with python 2.7.12? | 17:59 |
| NobodyCam | I would assume it does. | 17:59 |
| *** Goneri has joined #openstack-ironic | 18:01 | |
| *** phschwartz has joined #openstack-ironic | 18:01 | |
| xavierr | it should works with any python 2.7.x | 18:03 |
| *** moshele has joined #openstack-ironic | 18:03 | |
| *** moshele has quit IRC | 18:08 | |
| *** dtantsur is now known as dtantsur|afk | 18:11 | |
| *** e0ne has joined #openstack-ironic | 18:14 | |
| thiagop | devananda: answered your review, thanks. | 18:17 |
| thiagop | devananda: I'd appreciate your feedback before sending the next patchset | 18:18 |
| *** causten has joined #openstack-ironic | 18:19 | |
| *** e0ne has quit IRC | 18:25 | |
| *** sdake has quit IRC | 18:29 | |
| *** e0ne has joined #openstack-ironic | 18:30 | |
| *** hjanssen-hpe has quit IRC | 18:31 | |
| *** hpe-hj has quit IRC | 18:31 | |
| *** anush_ has quit IRC | 18:31 | |
| *** vsaienko has joined #openstack-ironic | 18:32 | |
| *** sabeen has joined #openstack-ironic | 18:35 | |
| devananda | thiagop: "enabled by default" -- is there a way to see if the user specified this vs. if it is merely default? | 18:35 |
| devananda | I agree thta we should do what this patch does by default | 18:35 |
| devananda | but I'd lke to be able to tell users that set "ipv6_enabled=true" in their config file that they've requested something invalid | 18:35 |
| thiagop | devananda: my thought was to echo that message and carry on, you think we should break stack? | 18:40 |
| *** e0ne has quit IRC | 18:40 | |
| thiagop | devananda: just a remainder, tempest is the last service to be installed (just prior to ironic). Users may have lost ~20-25min just to see things break.. | 18:41 |
| devananda | thiagop: good point. echo and carry on is fine for this stage | 18:42 |
| devananda | thiagop: perhaps in the ironic/devstack/ironic plugin, as we're installing or confguring things (that happens earlier in the process than tempest) we could look for invalid devtack configs and error out | 18:42 |
| devananda | that would clearly be a different patch than your current one :) | 18:43 |
| devananda | I need to step away and find lunch. will bbl! | 18:43 |
| openstackgerrit | Merged openstack/python-ironicclient: Add baremetal port set command to OSC plugin https://review.openstack.org/347287 | 18:45 |
| *** mjturek1 has quit IRC | 18:51 | |
| *** moshele has joined #openstack-ironic | 18:51 | |
| *** Sukhdev has quit IRC | 18:55 | |
| *** e0ne has joined #openstack-ironic | 18:56 | |
| *** Goneri has quit IRC | 19:01 | |
| *** Goneri has joined #openstack-ironic | 19:16 | |
| *** e0ne has quit IRC | 19:18 | |
| *** e0ne_ has joined #openstack-ironic | 19:18 | |
| openstackgerrit | Merged openstack/ironic: Move "server_profile_template_uri" to REQUIRED_ON_PROPERTIES https://review.openstack.org/347127 | 19:20 |
| mariojv | morgabra: you might be interested in reviewing https://review.openstack.org/#/c/277853 | 19:20 |
| openstackgerrit | Ruby Loo proposed openstack/ironic-specs: Changes to some OSC commands https://review.openstack.org/357505 | 19:22 |
| openstackgerrit | Merged openstack/ironic: Replace DB API call to object's method in iLO drivers https://review.openstack.org/347403 | 19:23 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Substrate fixes and changes for adoption testing https://review.openstack.org/344974 | 19:27 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add tempest tests for node Adoption https://review.openstack.org/344975 | 19:27 |
| *** causten has quit IRC | 19:28 | |
| *** causten has joined #openstack-ironic | 19:28 | |
| openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Refactor of models to_oneview_dict and init method https://review.openstack.org/331673 | 19:30 |
| *** Sukhdev has joined #openstack-ironic | 19:31 | |
| *** e0ne_ has quit IRC | 19:35 | |
| *** mjturek1 has joined #openstack-ironic | 19:39 | |
| *** rcernin has joined #openstack-ironic | 19:41 | |
| *** fellypefca has quit IRC | 19:42 | |
| *** sdake has joined #openstack-ironic | 19:42 | |
| *** mjturek1 has quit IRC | 19:43 | |
| thiagop | devananda: maybe something like https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L661 ?? | 19:47 |
| thiagop | devananda: in fact, earlier than this is #L191 | 19:48 |
| *** bfournie has quit IRC | 19:53 | |
| *** bnemec has quit IRC | 19:57 | |
| *** rbrud has quit IRC | 19:57 | |
| *** mjturek1 has joined #openstack-ironic | 19:57 | |
| *** causten has quit IRC | 20:03 | |
| *** causten_ has joined #openstack-ironic | 20:04 | |
| *** wajdi_ has joined #openstack-ironic | 20:13 | |
| *** wajdi__ has joined #openstack-ironic | 20:16 | |
| *** wajdi has quit IRC | 20:16 | |
| *** dprince has quit IRC | 20:17 | |
| *** wajdi_ has quit IRC | 20:20 | |
| *** wajdi__ has quit IRC | 20:20 | |
| *** sdake has quit IRC | 20:21 | |
| *** moshele has quit IRC | 20:24 | |
| *** jcoufal_ has joined #openstack-ironic | 20:27 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Add node serial console documentation https://review.openstack.org/293872 | 20:30 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Move console documentation to separate file https://review.openstack.org/358844 | 20:30 |
| *** jcoufal has quit IRC | 20:31 | |
| *** bfournie has joined #openstack-ironic | 20:32 | |
| *** vsaienko has quit IRC | 20:34 | |
| *** sdake has joined #openstack-ironic | 20:35 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Add node serial console documentation https://review.openstack.org/293872 | 20:40 |
| *** vsaienko has joined #openstack-ironic | 20:40 | |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic-python-agent: Enforce upper-constraints when building CoreOS image https://review.openstack.org/358855 | 20:50 |
| *** vsaienko has quit IRC | 20:56 | |
| *** baoli has quit IRC | 21:28 | |
| *** aNupoisc has joined #openstack-ironic | 21:28 | |
| *** adnavare has joined #openstack-ironic | 21:28 | |
| thiagop | Calling it a day | 21:35 |
| thiagop | see you tomorrow | 21:35 |
| *** thiagop has quit IRC | 21:35 | |
| mrda | Morning Ironic | 21:55 |
| *** cdearborn has quit IRC | 22:12 | |
| *** causten_ has quit IRC | 22:20 | |
| *** rbudden has quit IRC | 22:22 | |
| *** rajinir has quit IRC | 22:25 | |
| TheJulia | goodnight everyone | 22:29 |
| *** bnemec has joined #openstack-ironic | 22:37 | |
| *** mpsairam has joined #openstack-ironic | 22:39 | |
| jlvillal | TheJulia: Good night | 22:40 |
| *** fxpester has joined #openstack-ironic | 22:44 | |
| *** pester has quit IRC | 22:48 | |
| *** jcoufal_ has quit IRC | 22:51 | |
| *** Sukhdev has quit IRC | 22:51 | |
| tonyb | jroll: we shoudl chat about that constraints/dockerfile review See if we can muster someone on the ironic side to run with it | 23:02 |
| NobodyCam | morning mrda | 23:02 |
| NobodyCam | hey jlvillal belated morning to you too :) | 23:03 |
| openstackgerrit | Merged openstack/python-ironicclient: Add --wait flag for provision actions and wait_for_provision_state function https://review.openstack.org/328191 | 23:03 |
| mrda | hey NobodyCam | 23:04 |
| NobodyCam | :) | 23:05 |
| tonyb | mat128: I guess you're on the hook to fix that thing. Feel free to grab me if you need help with the constraints stuff | 23:05 |
| jlvillal | Hey NobodyCam. Good afternoon to you :) | 23:14 |
| NobodyCam | :) | 23:15 |
| *** sdake has quit IRC | 23:16 | |
| jlvillal | NobodyCam: If you want (what I think is) an easy review :) https://review.openstack.org/358844 | 23:17 |
| *** michauds has quit IRC | 23:17 | |
| jlvillal | tonyb: I think mat128 has proposed a patch: https://review.openstack.org/#/c/358855/ | 23:19 |
| *** wajdi has joined #openstack-ironic | 23:20 | |
| *** rbudden has joined #openstack-ironic | 23:22 | |
| *** rpioso has quit IRC | 23:31 | |
| *** sdake has joined #openstack-ironic | 23:33 | |
| *** baoli has joined #openstack-ironic | 23:35 | |
| tonyb | jlvillal: Thanks I added a comment | 23:37 |
| jlvillal | tonyb: Thanks. I commented on your comment ;) | 23:39 |
| *** adnavare has quit IRC | 23:42 | |
| *** aNupoisc has quit IRC | 23:42 | |
| *** hoangcx2 has joined #openstack-ironic | 23:46 | |
| *** hoangcx has quit IRC | 23:48 | |
| tonyb | jlvillal: /me creates some terrible sed magic | 23:48 |
| jlvillal | tonyb: heh | 23:48 |
| tonyb | jlvillal: of course that's only part of the full gate solution but it's getting closer. | 23:49 |
| jlvillal | tonyb: Maybe there should be a CONSTRAINTS_URL file or something? | 23:49 |
| tonyb | jlvillal: not ironic but Can you take another look at: https://review.openstack.org/#/c/353008/ | 23:50 |
| jlvillal | tonyb: And everyone could use it. But maybe tox.ini can't :( | 23:50 |
| tonyb | jlvillal: Yeah tox wouldn't work | 23:50 |
| jlvillal | tonyb: Oh. I think Ryan's internship ended. | 23:51 |
| tonyb | jlvillal: I agree that getting the correct file in the gate is a little messy | 23:51 |
| tonyb | jlvillal: boo :( | 23:51 |
| jlvillal | tonyb: We had some high-school interns for the summer. And I haven't seen them recently. | 23:52 |
| *** rama_y has quit IRC | 23:53 | |
| tonyb | jlvillal: Oh well. We'll see what happens with that review then :/ | 23:53 |
| *** rama_y has joined #openstack-ironic | 23:53 | |
| jlvillal | tonyb: yeah. I'm not sure. | 23:54 |
| *** sdake has quit IRC | 23:55 | |
| tonyb | jlvillal: if it's functionally okay I'll land it then he'll be ATC at least. | 23:55 |
| *** mtanin___ has joined #openstack-ironic | 23:58 | |
| *** rama_y has quit IRC | 23:58 | |
| *** mtanino has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!