| *** sabeen has joined #openstack-ironic | 00:03 | |
| *** snizar_ has quit IRC | 00:14 | |
| *** snizar has quit IRC | 00:14 | |
| *** sabeen has quit IRC | 00:21 | |
| *** sabeen has joined #openstack-ironic | 00:28 | |
| *** harlowja has joined #openstack-ironic | 00:29 | |
| *** wajdi has joined #openstack-ironic | 00:42 | |
| *** hoangcx has joined #openstack-ironic | 00:46 | |
| *** wajdi_ has joined #openstack-ironic | 00:47 | |
| *** wajdi has quit IRC | 00:50 | |
| *** jabroney1 has quit IRC | 00:58 | |
| *** chlong has quit IRC | 01:03 | |
| *** joprovost has joined #openstack-ironic | 01:07 | |
| *** Sukhdev_ has quit IRC | 01:10 | |
| *** syed_ has quit IRC | 01:12 | |
| *** snizar has joined #openstack-ironic | 01:14 | |
| *** snizar_ has joined #openstack-ironic | 01:14 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Update unit tests for neutron interface https://review.openstack.org/367038 | 01:19 |
|---|---|---|
| *** jvgrant has joined #openstack-ironic | 01:26 | |
| *** wuhg has joined #openstack-ironic | 01:31 | |
| *** jvgrant has quit IRC | 01:31 | |
| *** sdake has joined #openstack-ironic | 01:32 | |
| *** rloo has joined #openstack-ironic | 01:39 | |
| *** rloo has left #openstack-ironic | 01:39 | |
| *** openstack has joined #openstack-ironic | 01:44 | |
| *** sabeen has quit IRC | 01:44 | |
| *** sabeen has joined #openstack-ironic | 01:46 | |
| *** rloo has joined #openstack-ironic | 01:46 | |
| *** rloo has left #openstack-ironic | 01:46 | |
| *** baoli has joined #openstack-ironic | 01:50 | |
| *** thrash is now known as thrash|g0ne | 02:01 | |
| *** baoli has quit IRC | 02:08 | |
| *** tuanla_ has joined #openstack-ironic | 02:12 | |
| *** hoangcx has quit IRC | 02:16 | |
| *** chlong has joined #openstack-ironic | 02:19 | |
| *** hoangcx has joined #openstack-ironic | 02:19 | |
| *** snizar_ has quit IRC | 02:25 | |
| *** snizar has quit IRC | 02:25 | |
| *** baoli has joined #openstack-ironic | 02:37 | |
| *** vishwana_ is now known as vishwanathj_zzz | 02:38 | |
| *** yuanying has quit IRC | 02:47 | |
| *** wajdi_ has quit IRC | 02:48 | |
| *** hoangcx has quit IRC | 02:51 | |
| *** hoangcx has joined #openstack-ironic | 02:54 | |
| *** joprovost has quit IRC | 03:00 | |
| *** rbudden has quit IRC | 03:02 | |
| *** vikrant has joined #openstack-ironic | 03:09 | |
| *** hj-hpe has quit IRC | 03:17 | |
| *** mkam has joined #openstack-ironic | 03:20 | |
| *** hoangcx has quit IRC | 03:27 | |
| *** praneshp has joined #openstack-ironic | 03:28 | |
| *** vikrant has quit IRC | 03:33 | |
| *** mkam has quit IRC | 03:34 | |
| *** vikrant has joined #openstack-ironic | 03:34 | |
| *** hoangcx has joined #openstack-ironic | 03:35 | |
| openstackgerrit | Xavier proposed openstack/ironic: Add inspection feature for the OneView drivers https://review.openstack.org/367065 | 03:38 |
| *** yuanying has joined #openstack-ironic | 03:51 | |
| *** vmud213 has joined #openstack-ironic | 03:58 | |
| *** baoli has quit IRC | 04:01 | |
| *** sabeen has quit IRC | 04:18 | |
| *** amotoki has joined #openstack-ironic | 04:23 | |
| *** links has joined #openstack-ironic | 04:28 | |
| *** harlowja_at_home has joined #openstack-ironic | 04:29 | |
| *** vmud213 has quit IRC | 04:31 | |
| openstackgerrit | Ankit Kumar proposed openstack/proliantutils: Fix Inspection failure in Gen7 https://review.openstack.org/357134 | 04:32 |
| *** vmud213 has joined #openstack-ironic | 04:35 | |
| *** fragatina has joined #openstack-ironic | 04:41 | |
| *** vmud213 has quit IRC | 04:44 | |
| *** fragatina has quit IRC | 04:45 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Update unit tests for neutron interface https://review.openstack.org/367038 | 04:47 |
| *** vmud213 has joined #openstack-ironic | 04:47 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Update unit tests for neutron interface https://review.openstack.org/367038 | 04:49 |
| *** sdake has quit IRC | 04:56 | |
| *** moshele has joined #openstack-ironic | 04:58 | |
| *** sdake has joined #openstack-ironic | 05:00 | |
| *** jaosorior has joined #openstack-ironic | 05:10 | |
| *** chlong has quit IRC | 05:14 | |
| *** ChubYann has quit IRC | 05:23 | |
| *** phuongnh has joined #openstack-ironic | 05:29 | |
| *** chlong has joined #openstack-ironic | 05:30 | |
| *** links has quit IRC | 05:31 | |
| *** harlowja_at_home has quit IRC | 05:46 | |
| *** links has joined #openstack-ironic | 05:46 | |
| *** fragatina has joined #openstack-ironic | 05:52 | |
| *** sdake_ has joined #openstack-ironic | 05:56 | |
| *** fragatina has quit IRC | 05:56 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 05:57 |
| *** sdake has quit IRC | 05:58 | |
| *** baoli has joined #openstack-ironic | 06:13 | |
| *** chlong has quit IRC | 06:13 | |
| *** baoli has quit IRC | 06:17 | |
| *** e0ne has joined #openstack-ironic | 06:22 | |
| *** hoonetorg has quit IRC | 06:25 | |
| *** mjura has joined #openstack-ironic | 06:25 | |
| *** chlong has joined #openstack-ironic | 06:26 | |
| *** vmud213 has quit IRC | 06:40 | |
| *** vmud213 has joined #openstack-ironic | 06:41 | |
| *** vmud213 has quit IRC | 06:48 | |
| *** vmud213 has joined #openstack-ironic | 06:49 | |
| *** vmud213 has quit IRC | 06:54 | |
| *** hoangcx_ has joined #openstack-ironic | 06:54 | |
| *** vmud213 has joined #openstack-ironic | 06:55 | |
| *** hoangcx has quit IRC | 06:57 | |
| *** fxpester has joined #openstack-ironic | 06:57 | |
| *** tesseract- has joined #openstack-ironic | 07:04 | |
| *** e0ne has quit IRC | 07:22 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Refactor _test_build_pxe_config_options tests https://review.openstack.org/367142 | 07:27 |
| *** rbartal has joined #openstack-ironic | 07:28 | |
| *** ifarkas_afk is now known as ifarkas | 07:33 | |
| *** openstackgerrit has quit IRC | 07:33 | |
| *** openstackgerrit has joined #openstack-ironic | 07:34 | |
| *** praneshp has quit IRC | 07:36 | |
| *** dtantsur|afk is now known as dtantsur | 07:42 | |
| dtantsur | Morning Ironic | 07:42 |
| *** hoangcx_ has quit IRC | 07:43 | |
| dtantsur | aarefiev, hey! what's the situation with grenade now? I see you're trying some reverts in tempest. | 07:43 |
| aarefiev | dtantsur: morning, so the problem in paramiko, that patch fixes deadlocks in paramiko when data bunch transfered, but somehow causes hangs | 07:46 |
| dtantsur | damn... | 07:46 |
| dtantsur | so reverting it is not the best option as well, right? | 07:46 |
| aarefiev | I've ran few times w/o this patch and grenade pass all times | 07:47 |
| aarefiev | I mean with reverted | 07:47 |
| dtantsur | aarefiev, could you please bring it to the ML with [qa] tag? the time is pressing, and we need assistance from qa folks, I guess... | 07:47 |
| aarefiev | yeah, if we just revert it will fail on other jobs | 07:48 |
| dtantsur | aarefiev, now, the only thing I don't understand is why one we are affected by this problem | 07:50 |
| aarefiev | dtantsur: sure, one thing I don't get it why only grenade in inspector? | 07:50 |
| dtantsur | lol | 07:50 |
| aarefiev | yeah :) | 07:50 |
| dtantsur | aarefiev, is it possible that we have a different configuration? I think you've tried changing some timeouts.. | 07:50 |
| aarefiev | dtantsur: looks like it doesn't depend on timeouts | 07:51 |
| *** ohamada has joined #openstack-ironic | 07:51 | |
| dtantsur | aarefiev, also this patch merged successfully: https://review.openstack.org/#/c/365902/ Oo | 07:51 |
| patchbot | patch 365902 - ironic-inspector - TrivialFix: Remove cfg import unused (MERGED) | 07:51 |
| aarefiev | merged ^ | 07:51 |
| aarefiev | ahh | 07:51 |
| *** jpich has joined #openstack-ironic | 07:52 | |
| dtantsur | lemme update https://review.openstack.org/#/c/358779/ and see if it passes or fails | 07:52 |
| patchbot | patch 358779 - ironic-inspector - Log hook names as we run them | 07:52 |
| *** chlong has quit IRC | 07:52 | |
| aarefiev | I recheck patch with pci, lets see | 07:52 |
| dtantsur | aarefiev, this passed grenade too: https://review.openstack.org/#/c/365662/ please review | 07:54 |
| patchbot | patch 365662 - ironic-inspector - Set node to the error if reapply fails | 07:54 |
| dtantsur | I wonder if there is a problem with that patch in the end... | 07:54 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Log hook names as we run them https://review.openstack.org/358779 | 07:55 |
| dtantsur | also updated ^^^ | 07:55 |
| aarefiev | dtantsur: looking, I wondering if it was some network issue, but why affects only inspector | 07:55 |
| dtantsur | aarefiev, maybe something wrong with the default introspection network? | 07:56 |
| dtantsur | it does not tell much about why only this patch has such severe problems... | 07:56 |
| aarefiev | yeah, and test doesn't related to patch code | 07:57 |
| *** hoangcx has joined #openstack-ironic | 07:57 | |
| tangchen | morning all :) | 07:57 |
| dtantsur | morning tangchen | 07:58 |
| *** zzzeek has quit IRC | 08:00 | |
| tangchen | dtantsur: Hi Dmitry, would you please help to confirm this trivial fix ? https://review.openstack.org/#/c/357639/ | 08:00 |
| patchbot | patch 357639 - python-ironicclient - Support i18n for osc plugin help msg | 08:00 |
| *** zzzeek has joined #openstack-ironic | 08:00 | |
| dtantsur | tangchen, it's on my radar, but now I'm busy figuring out urgent things, so I might not get to it immediately | 08:00 |
| *** athomas has joined #openstack-ironic | 08:01 | |
| tangchen | dtantsur: Sure, please help to review when you are free. :) Thanks. | 08:01 |
| lucasagomes | morning all | 08:02 |
| tangchen | morning lucasagomes | 08:03 |
| dtantsur | hey lucasagomes | 08:05 |
| aarefiev | morning lucasagomes, tangchen | 08:05 |
| tangchen | Hi aarefiev | 08:05 |
| *** Romanenko_K has joined #openstack-ironic | 08:14 | |
| aarefiev | dtantsur: crap, and now it fails with request timeout o_0 http://logs.openstack.org/95/352295/9/check/gate-grenade-dsvm-ironic-inspector/b6e3aa7/console.html | 08:17 |
| *** sdake_ is now known as sdake | 08:17 | |
| *** vmud213 has quit IRC | 08:18 | |
| aarefiev | something changed | 08:18 |
| *** vmud213 has joined #openstack-ironic | 08:18 | |
| dtantsur | wut? | 08:19 |
| aarefiev | dtantsur: it failed with test timeout because ping hangs and now with requests timeout, whats going on? | 08:21 |
| dtantsur | no idea.. | 08:29 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: DO NOT MERGE: testing gate https://review.openstack.org/367171 | 08:34 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Add oslotest to test-requirements https://review.openstack.org/367206 | 08:35 |
| *** vmud213 has quit IRC | 08:37 | |
| *** vmud213 has joined #openstack-ironic | 08:38 | |
| *** jaosorior is now known as jaosorior_lunch | 08:40 | |
| *** snecklifter has quit IRC | 08:45 | |
| *** sborkows has joined #openstack-ironic | 08:46 | |
| *** moshele has quit IRC | 08:47 | |
| *** moshele has joined #openstack-ironic | 08:48 | |
| dtantsur | aarefiev, seems like grenade is completely down :( yeah, we need some external help, I guess | 08:54 |
| *** derekh has joined #openstack-ironic | 08:55 | |
| aarefiev | dtantsur: I'm writing letter to openstack-dev now | 08:56 |
| dtantsur | thanks! | 08:56 |
| sborkows | morning all | 08:57 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 08:57 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: DO NOT MERGE: testing gate https://review.openstack.org/367171 | 09:03 |
| dtantsur | morning sborkows | 09:09 |
| openstackgerrit | Imre Farkas proposed openstack/ironic: Fix DRAC failure during automated cleaning https://review.openstack.org/367229 | 09:10 |
| ifarkas | morning all | 09:10 |
| *** lucasagomes is now known as lucas-relocate | 09:21 | |
| *** mbound has joined #openstack-ironic | 09:30 | |
| dtantsur | morning ifarkas | 09:33 |
| *** vnogin has left #openstack-ironic | 09:37 | |
| *** yibo has quit IRC | 09:37 | |
| *** sambetts|afk is now known as sambetts | 09:44 | |
| *** e0ne has joined #openstack-ironic | 09:44 | |
| *** mgould|afk is now known as mgould | 09:48 | |
| mgould | morning aarefiev dtantsur jaosorior_lunch ifarkas lucas-relocate sambetts and everyone else! | 09:49 |
| dtantsur | hey mgould | 09:50 |
| sambetts | Morning all | 09:54 |
| dtantsur | morning sambetts | 09:54 |
| sambetts | o/ dtantsur :( about the grenade job | 09:55 |
| dtantsur | yeah... dunno what to do, let's see if somebody answers on the ML | 09:55 |
| *** yibo has joined #openstack-ironic | 09:58 | |
| openstackgerrit | Sam Betts proposed openstack/python-ironicclient: Add python API for node interface attach/detach https://review.openstack.org/364420 | 10:03 |
| openstackgerrit | Sam Betts proposed openstack/python-ironicclient: DO NOT MERGE Testing Client Changes with Nova https://review.openstack.org/366747 | 10:03 |
| *** lucas-relocate is now known as lucasagomes | 10:05 | |
| *** yibo has quit IRC | 10:05 | |
| lucasagomes | mgould, sambetts aarefiev *all* morning :-) | 10:05 |
| openstackgerrit | Merged openstack/ironic: Log a warning for unsupported drivers and interfaces https://review.openstack.org/366399 | 10:07 |
| *** hoangcx has quit IRC | 10:10 | |
| mgould | hey everyone, did we support whole-disk images in Kilo? | 10:28 |
| mgould | also, what's the deal with whole-disk images and EFI support? "Make sure your image has an EFIboot partition and it should Just Work"? | 10:29 |
| *** phuongnh has quit IRC | 10:30 | |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with Chassis. https://review.openstack.org/343796 | 10:32 |
| dtantsur | mgould, I guess that means that we don't need anything fancy for local boot | 10:39 |
| dtantsur | mgould, I'm pretty sure we supported whole-disk images, not sure if we supported them with pxe_* family though | 10:40 |
| mgould | dtantsur: thanks! | 10:45 |
| *** amotoki has quit IRC | 10:48 | |
| *** sturivnyi_ has joined #openstack-ironic | 10:50 | |
| *** jcoufal has quit IRC | 10:50 | |
| sturivnyi_ | Hi =) I have bunch of patches for review https://review.openstack.org/#/q/status:open+project:openstack/python-ironicclient+branch:master+topic:%22functional+CLI+tests%22. Would be grateful if you could review them all. | 10:55 |
| sturivnyi_ | This is functional CLI tests for python-ironicclient | 10:57 |
| *** baoli has joined #openstack-ironic | 10:57 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Abort cleaning and deployment if node is in maintenance https://review.openstack.org/366828 | 10:59 |
| dtantsur | jroll, this is pretty close now ^^^ wdyt? | 11:01 |
| *** baoli has quit IRC | 11:05 | |
| openstackgerrit | Aparna proposed openstack/proliantutils: Adds support in hpssa for SDD interface 'Solid State SATA' https://review.openstack.org/367287 | 11:09 |
| *** baoli has joined #openstack-ironic | 11:13 | |
| openstackgerrit | Sam Betts proposed openstack/ironic: Add Network Interface Attach and Detach APIs https://review.openstack.org/327046 | 11:19 |
| *** baoli has quit IRC | 11:19 | |
| * sambetts bbiab | 11:20 | |
| *** sambetts is now known as sambetts|afk | 11:20 | |
| *** baoli has joined #openstack-ironic | 11:24 | |
| TheJulia | Good morning everyone | 11:27 |
| sturivnyi_ | Morning <TheJulia>) | 11:28 |
| *** vmud213 has quit IRC | 11:28 | |
| TheJulia | mgould: I know a number of folks that have worked with whole disk images pre-built with EFI, and as long as the image is properly setup with an EFI partition and all, it seems to be \o/ for everyone that I know that uses it. | 11:29 |
| dtantsur | morning TheJulia | 11:30 |
| lucasagomes | TheJulia, good morning | 11:30 |
| *** baoli has quit IRC | 11:30 | |
| mgould | TheJulia: great, thanks! Also, morning :-) | 11:31 |
| TheJulia | mgould: the only headache that I've seen people run into is network cards that won't run without being booted via EFI. The deployment ended up having to use virtual media with an EFI boot config on it too. | 11:32 |
| TheJulia | But, I've only heard of that one case. | 11:32 |
| mgould | TheJulia: great, thanks | 11:34 |
| * mgould thinks back to the innocent happy days a year ago before he learned just how complicated booting computers could be... | 11:35 | |
| TheJulia | Ahhh the simpler computing times... | 11:36 |
| *** fragatina has joined #openstack-ironic | 11:36 | |
| mgould | :-) | 11:39 |
| mat128 | morning everyone | 11:39 |
| *** fragatina has quit IRC | 11:41 | |
| TheJulia | good morning mat128 | 11:42 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Grub2 by default for PXE + UEFI https://review.openstack.org/367306 | 11:42 |
| * lucasagomes cleaning some backlog | 11:42 | |
| *** baoli has joined #openstack-ironic | 11:43 | |
| *** jcoufal has joined #openstack-ironic | 11:46 | |
| TheJulia | lucasagomes: I posted feedback, super quick fix to the release notes and I think it will be good. Also left a nit on it that I don't have any strong feelings about and can be ignored if you want. | 11:47 |
| lucasagomes | TheJulia, thanks a lot, will take a look | 11:47 |
| lucasagomes | just gonna grab something to eat and I will fix it :-) | 11:48 |
| *** lucasagomes is now known as lucas-hungry | 11:48 | |
| openstackgerrit | Tang Chen proposed openstack/ironic: Prepare for functional tests https://review.openstack.org/349421 | 11:49 |
| TheJulia | food sounds delicious right now | 11:52 |
| *** links has quit IRC | 11:52 | |
| TheJulia | I just finished my first cup of coffee | 11:52 |
| *** jaosorior_lunch is now known as jaosorior | 11:56 | |
| *** thrash|g0ne is now known as thrash | 11:56 | |
| *** baoli_ has joined #openstack-ironic | 11:56 | |
| *** baoli has quit IRC | 12:00 | |
| *** sambetts|afk is now known as sambetts | 12:00 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: iPXE allow skip caching in standalone mode https://review.openstack.org/367321 | 12:02 |
| *** vikrant has quit IRC | 12:07 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Change $LOGFILE path in configuration devstack https://review.openstack.org/366518 | 12:07 |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Change $LOGFILE path in configuration devstack https://review.openstack.org/366518 | 12:08 |
| *** leitan has joined #openstack-ironic | 12:08 | |
| *** dprince has joined #openstack-ironic | 12:09 | |
| *** trown|outtypewww is now known as trown | 12:09 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Change $LOGFILE path in configuration devstack https://review.openstack.org/366518 | 12:10 |
| *** vnogin has joined #openstack-ironic | 12:11 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Grub2 by default for PXE + UEFI https://review.openstack.org/367306 | 12:11 |
| *** vnogin has left #openstack-ironic | 12:11 | |
| *** bfournie has joined #openstack-ironic | 12:12 | |
| *** wuhg has quit IRC | 12:21 | |
| *** xavierr has quit IRC | 12:21 | |
| openstackgerrit | Aparna proposed openstack/proliantutils: Adds support in hpssa for SDD interface 'Solid State SATA' https://review.openstack.org/367287 | 12:24 |
| *** rbudden has joined #openstack-ironic | 12:25 | |
| openstackgerrit | Tang Chen proposed openstack/ironic: Prepare for functional tests https://review.openstack.org/349421 | 12:27 |
| TheJulia | lucas-hungry: wouldn't it also make sense to update the sample config? (sorry!) | 12:37 |
| lucas-hungry | TheJulia, ++ totally | 12:38 |
| *** lucas-hungry is now known as lucasagomes | 12:38 | |
| lucasagomes | TheJulia, lemme update it :-) | 12:38 |
| * lucasagomes see a lot of unrelated changes in the sample :-O | 12:39 | |
| TheJulia | :( | 12:39 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Grub2 by default for PXE + UEFI https://review.openstack.org/367306 | 12:42 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Update ironic/ironic.conf.sample https://review.openstack.org/367342 | 12:42 |
| lucasagomes | done :-) | 12:42 |
| TheJulia | lucasagomes: Thank you :) | 12:43 |
| lucasagomes | thank YOU for reviewing it | 12:43 |
| *** amotoki has joined #openstack-ironic | 12:48 | |
| *** mmedvede has quit IRC | 12:49 | |
| *** jaosorior has quit IRC | 12:51 | |
| *** jaosorior has joined #openstack-ironic | 12:51 | |
| jroll | morning everyone | 12:57 |
| lucasagomes | jroll, g'morning | 12:57 |
| jroll | only here for a few, have an appt to run to shortly | 12:57 |
| jroll | dtantsur: I guess I'm fine with that change | 12:58 |
| jroll | not a huge fan but I won't block it | 12:58 |
| *** skazi has quit IRC | 12:58 | |
| *** rloo has joined #openstack-ironic | 12:58 | |
| TheJulia | Good morning jroll | 12:59 |
| jroll | morning lucasagomes, TheJulia :) | 12:59 |
| dtantsur | jroll, what are your concerns about it? also g'morning | 13:00 |
| jroll | dtantsur: nothing really, just changing what I'm used to. morning :) | 13:00 |
| dtantsur | heh, I see :) | 13:01 |
| dtantsur | I'll finish unit tests, do some local testing, then remove WIP | 13:01 |
| dtantsur | pas-ha, answering your question ^^^ | 13:01 |
| *** mjturek has left #openstack-ironic | 13:05 | |
| *** jcoufal has quit IRC | 13:05 | |
| *** mjturek has joined #openstack-ironic | 13:06 | |
| *** jcoufal has joined #openstack-ironic | 13:07 | |
| jroll | dtantsur: patch lgtm fwiw | 13:07 |
| *** tuanla_ has quit IRC | 13:07 | |
| *** ohamada has quit IRC | 13:08 | |
| *** ohamada has joined #openstack-ironic | 13:08 | |
| *** jheroux has joined #openstack-ironic | 13:09 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add basic tests for OSC plugin baremetal driver commands https://review.openstack.org/367359 | 13:11 |
| *** Goneri has joined #openstack-ironic | 13:11 | |
| *** skazi has joined #openstack-ironic | 13:11 | |
| *** jraju has joined #openstack-ironic | 13:12 | |
| *** cdearborn has joined #openstack-ironic | 13:12 | |
| TheJulia | dtantsur: I posted some stuff for the release note entries, just suggestions really. | 13:14 |
| TheJulia | stuff being comments on the revision | 13:15 |
| dtantsur | thanks! | 13:15 |
| *** mmedvede has joined #openstack-ironic | 13:17 | |
| *** sdake_ has joined #openstack-ironic | 13:20 | |
| openstackgerrit | Ankit Kumar proposed openstack/proliantutils: Fix Inspection failure in Gen7 https://review.openstack.org/357134 | 13:21 |
| *** sdake has quit IRC | 13:22 | |
| hwoarang | Hello does anyone know how to 'force' delete a broken node which is stuck in the 'inspecting' state? ironic node-delete does not seem able to do it. | 13:24 |
| TheJulia | hwoarang: I would think that it would eventually time out and once it has timed out I _think_ you could then delete it | 13:26 |
| dtantsur | hwoarang, it it in-band inspection with ironic-inspector or some OOB inspection? | 13:26 |
| hwoarang | in-band | 13:27 |
| hwoarang | but i may be tempted to drop the entry from the DB directly if it fails to change the state soonish | 13:27 |
| hwoarang | TheJulia: i need you for one more thing regarding bifrost | 13:27 |
| TheJulia | hwoarang: sure! :) | 13:28 |
| hwoarang | This change 5506f5e327c14a2008c6321a2e78a772c4b9a540 "Only set hostname on 127.0.0.1 if not present in /etc/hosts" broke some stuff for me | 13:28 |
| TheJulia | broke how so? | 13:28 |
| hwoarang | the reason is that in my host, the hostname in /etc/hosts is set in the local 10.X.X.X IP | 13:28 |
| hwoarang | however, epmd runs on 127.0.0.1 so when the rabbitmq-server task tries to start the service | 13:29 |
| openstackgerrit | Ankit Kumar proposed openstack/proliantutils: Fix Inspection failure in Gen7 https://review.openstack.org/357134 | 13:29 |
| hwoarang | it fails, because the hostname resolves to 10.X.X.X instead of 127.0.0.1 | 13:29 |
| dtantsur | hwoarang, you can abort in-band inspection via inspector command line | 13:29 |
| *** baoli_ has quit IRC | 13:29 | |
| dtantsur | hwoarang, something like 'openstack baremetal introspection abort UUID' | 13:29 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Add Network Interface Attach and Detach APIs https://review.openstack.org/327046 | 13:29 |
| TheJulia | hwoarang: Okay, I understand :( | 13:29 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Abort cleaning and deployment if node is in maintenance https://review.openstack.org/366828 | 13:30 |
| dtantsur | updated ^^^ | 13:30 |
| hwoarang | TheJulia: it all goes away when i add the hostname to 127.0.0.1 as it used to be. but clearly, according to the original gerrit patch, this break the case for others... | 13:30 |
| TheJulia | hwoarang: I thought the behavior had been fixed too :( | 13:32 |
| hwoarang | dtantsur: thanks let me try that | 13:33 |
| *** joprovost has joined #openstack-ironic | 13:33 | |
| hwoarang | TheJulia: tricky situation... | 13:33 |
| *** rbartal has quit IRC | 13:34 | |
| hwoarang | if we keep it like that then we need to be able to instruct epmd to listen on the correct address. and perhaps a few other bits and pieces which i forget about at the moment | 13:35 |
| TheJulia | I seem to remember though, that epmd essentially expects the hostname to resolve to 127.0.0.1 which actually should resolve to localhost, and that is why that was there | 13:35 |
| TheJulia | This has given me too many migraines :) | 13:36 |
| hwoarang | right | 13:37 |
| hwoarang | :( | 13:37 |
| hwoarang | sounds like a horrible hack has to be applied somewhere :) | 13:39 |
| *** mbound_ has joined #openstack-ironic | 13:39 | |
| *** mbound has quit IRC | 13:41 | |
| *** hshiina has joined #openstack-ironic | 13:44 | |
| hwoarang | TheJulia: is it ok if I reply to the old gerrit change: https://review.openstack.org/#/c/355778/ and discuss with the author how to solve both cases? | 13:50 |
| patchbot | patch 355778 - bifrost - Only set hostname on 127.0.0.1 if not present in /... (MERGED) | 13:50 |
| *** Administrator_ has joined #openstack-ironic | 13:51 | |
| *** wajdi has joined #openstack-ironic | 13:52 | |
| *** zhugaoxiao has quit IRC | 13:54 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Functional tests for portgroups in ironicclient https://review.openstack.org/297727 | 14:00 |
| *** anush has joined #openstack-ironic | 14:00 | |
| TheJulia | hwoarang: absolutely! | 14:01 |
| hwoarang | ok lets do that | 14:01 |
| TheJulia | hwoarang: this is something I need to ponder too :( | 14:01 |
| *** amotoki has quit IRC | 14:01 | |
| *** rajinir has joined #openstack-ironic | 14:02 | |
| *** amotoki has joined #openstack-ironic | 14:02 | |
| *** joprovost1 has joined #openstack-ironic | 14:02 | |
| * jroll is back from appt | 14:03 | |
| *** joprovost has quit IRC | 14:03 | |
| *** joprovost1 is now known as joprovost | 14:03 | |
| mat128 | lucasagomes: just saw your comment on the docs patch, would you mind reviewing the initial one? https://review.openstack.org/#/c/363294/ It needs to merge before the project-config change can be merged | 14:06 |
| patchbot | patch 363294 - ironic - Add a basic install guide | 14:06 |
| lucasagomes | mat128, hi there! | 14:06 |
| mat128 | o/ | 14:06 |
| lucasagomes | yes I will do, just finishing something first and then I will take a look :D | 14:06 |
| mat128 | lucasagomes: oh sure, I didn't mean it as a "look at this now", more like "if you still want to look at install-guide stuff" | 14:07 |
| *** sdake has joined #openstack-ironic | 14:10 | |
| *** sdake_ has quit IRC | 14:12 | |
| jlvillal | lucasagomes, On https://review.openstack.org/#/c/334967/ I'm not sure when devananda|travel will be back. | 14:12 |
| patchbot | patch 334967 - ironic - Fix non-deployed nodes due to short nodes.instance... | 14:12 |
| jlvillal | Do you know? | 14:12 |
| *** sabeen has joined #openstack-ironic | 14:13 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Fix breakage with oslo.config master https://review.openstack.org/367407 | 14:13 |
| *** sabeen1 has joined #openstack-ironic | 14:15 | |
| *** sabeen has quit IRC | 14:17 | |
| *** amotoki has quit IRC | 14:17 | |
| *** xavierr has joined #openstack-ironic | 14:17 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add volume_connectors and volume_targets to task https://review.openstack.org/366196 | 14:17 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: [WIP] Add storage_interface DB field and object https://review.openstack.org/348005 | 14:18 |
| *** vishwanathj_zzz is now known as vishwanathj | 14:18 | |
| TheJulia | jlvillal: I _suspect_ he will be back today | 14:18 |
| jlvillal | TheJulia, Cool :) | 14:19 |
| *** syed_ has joined #openstack-ironic | 14:19 | |
| *** hj-hpe has joined #openstack-ironic | 14:22 | |
| *** bnemec has quit IRC | 14:24 | |
| *** vmud213 has joined #openstack-ironic | 14:25 | |
| *** bnemec has joined #openstack-ironic | 14:27 | |
| *** amotoki has joined #openstack-ironic | 14:28 | |
| *** joprovost1 has joined #openstack-ironic | 14:29 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: [WIP] Add base cinder common interface https://review.openstack.org/355625 | 14:29 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: [WIP] Add storage_interface to base driver class https://review.openstack.org/348006 | 14:29 |
| *** moshele has quit IRC | 14:29 | |
| jroll | hrm, I worry about the online-ness of ALTER TABLE there | 14:29 |
| jroll | (on 334967) | 14:29 |
| TheJulia | yeah, table would be locked. :\ | 14:30 |
| jroll | okay, so it does nuke rolling upgrades | 14:30 |
| TheJulia | Well, depends on what you term rolling upgrades :) | 14:30 |
| *** joprovost has quit IRC | 14:30 | |
| *** joprovost1 is now known as joprovost | 14:30 | |
| TheJulia | AFAIK, selects and updates against the table would just hang until the alter is complete | 14:31 |
| jroll | online? :) | 14:31 |
| jroll | yeah | 14:31 |
| jroll | previous jobs with 1-hour alters have scarred me | 14:31 |
| TheJulia | I can't think of another way short of having an online data migration tool built into ironic that moves data to new tables and knows where to look | 14:31 |
| jroll | well, we could use a new column name | 14:31 |
| TheJulia | true | 14:32 |
| jroll | and translate it in the object layer | 14:32 |
| TheJulia | would be the perfect way to migrate it out of the nodes table then | 14:32 |
| TheJulia | out of curiosity, was that 1 hour alter with ironic? | 14:33 |
| *** amotoki has quit IRC | 14:33 | |
| jroll | TheJulia: I guess it's okay with 5.6+ http://dev.mysql.com/doc/refman/5.6/en/innodb-create-index-overview.html | 14:34 |
| TheJulia | actually, don't answer that :) | 14:34 |
| jroll | TheJulia: no, previous jobs, janky django app with x million row tables | 14:34 |
| jroll | with many columns | 14:34 |
| TheJulia | oh neat, so queries should continue to work according to that | 14:35 |
| jroll | assuming folks are running 5.6 | 14:36 |
| *** vmud213 has quit IRC | 14:36 | |
| TheJulia | yeah | 14:36 |
| jroll | trusty is 5.5 | 14:36 |
| jlvillal | jroll, I did ask him about rolling upgrades and he responded near the bottom | 14:36 |
| jroll | jlvillal: yes, I read that, and the answer was not sufficient for me | 14:36 |
| jroll | jlvillal: my question is, how much does it hurt to read-lock the nodes table in a busy ironic environment | 14:36 |
| jlvillal | Ah, okay. | 14:36 |
| * jlvillal doesn't know | 14:37 | |
| TheJulia | I can't imagine it being all that bad with the existing structure of the table, which is the only reason I feel moderately okay with the change | 14:37 |
| jroll | given we don't officially support rolling upgrades, it's probably fine | 14:37 |
| *** scarab_ has joined #openstack-ironic | 14:38 | |
| TheJulia | once that change is in place though, table schema changes could get very bad | 14:39 |
| * TheJulia is liking the alternate table idea more and more | 14:39 | |
| jroll | I'd love to have a key/value table for each of our json blobs | 14:39 |
| jroll | or use postgres json support :P | 14:39 |
| TheJulia | heh | 14:40 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Grub2 by default for PXE + UEFI https://review.openstack.org/367306 | 14:40 |
| jroll | JayF: I might like your deployer hat's opinion on this, please see the comments around upgrades and table locks and such https://review.openstack.org/#/c/334967/8 | 14:41 |
| patchbot | patch 334967 - ironic - Fix non-deployed nodes due to short nodes.instance... | 14:41 |
| *** mkam has joined #openstack-ironic | 14:41 | |
| TheJulia | jroll: or just move to a database that can be distributed super easily, and start building the concept of geographic locations in *ducks* | 14:42 |
| jroll | TheJulia: brb switching everything to riak | 14:42 |
| *** jvgrant has joined #openstack-ironic | 14:43 | |
| dtantsur | we need something that is Web Scale, MySQL is not Web Scale, hence all the problems | 14:43 |
| *** baoli has joined #openstack-ironic | 14:43 | |
| JayF | jroll: that's a huge ass field | 14:43 |
| * TheJulia awaits a link to the webscale video | 14:43 | |
| *** michauds has joined #openstack-ironic | 14:43 | |
| jroll | JayF: yeeeeees? | 14:44 |
| jroll | JayF: more curious about upgrade implications, like, do you think that will cause a significantly longer deploy than usual, etc | 14:44 |
| JayF | I don't know enough about dbs to really make a good statement on that | 14:45 |
| JayF | but I will say if someone puts anything near a gig of data in that field | 14:45 |
| JayF | performance will tank | 14:45 |
| JayF | it seems very unwise to leave that field essentially unbounded | 14:45 |
| jroll | dtantsur: obligatory https://github.com/dcramer/mangodb/ | 14:45 |
| jroll | JayF: well, it holds a configdrive, so 32mb is possible without trying hard | 14:46 |
| jroll | er, not quite because compression, but ya know | 14:46 |
| TheJulia | jroll: writes would also get blocked, no deployments, no heartbeats, until the table copy is finished. I just changed my vote to -1 on it | 14:46 |
| JayF | from TheJulia "My only fear is this is something that can never really be un-done, and could result in some very large database entries." | 14:46 |
| JayF | and I agree with that, a lot | 14:46 |
| JayF | sure, but there's a lot of room between 32mb and 4gb | 14:46 |
| jroll | TheJulia: well, we don't currently support running database migrations while services are up, is the argument for it | 14:46 |
| jroll | can longtext specify a size? | 14:46 |
| *** causten has quit IRC | 14:47 | |
| jroll | like, clearly someone has hit more than 64k | 14:47 |
| JayF | idk, dbs not my strongsuit | 14:47 |
| jroll | if they took the time to do this | 14:47 |
| JayF | but 64k is not that much, 4gb is way too much | 14:47 |
| JayF | there's gotta be a better middle ground | 14:47 |
| *** rpioso has joined #openstack-ironic | 14:47 | |
| jlvillal | We have a customer that has hit greater than 64KB. And they don't use swift... | 14:47 |
| jroll | yeah, I'm not sure if there is a MEDIUMTEXT | 14:47 |
| jroll | :P | 14:47 |
| TheJulia | jroll: I think so, but only artificially I think if I'm remembering what I read last month correctly. Realistically we break after 64MB, but even that is rather big and would be a very long database migration | 14:48 |
| jroll | TheJulia: 64KB | 14:48 |
| *** rbrud has joined #openstack-ironic | 14:48 | |
| dtantsur | 64KB is enough for everyone | 14:48 |
| TheJulia | I mean, if I supply a URL for the config drive instead of the config drive contents, it can't be larger than 64MB | 14:48 |
| * dtantsur ducks | 14:49 | |
| jroll | heh | 14:49 |
| *** jvgrant has quit IRC | 14:49 | |
| jroll | TIL there *is* a MEDIUMTEXT, which is max 16 MiB | 14:49 |
| dtantsur | but yeah, tripleo uses config drive pretty extensively, it's a miracle we never maxed 64 KB | 14:49 |
| JayF | For something like this, I'm also tempted to say like | 14:49 |
| dtantsur | hmm, 16 MiB is a huge improvement | 14:50 |
| TheJulia | well, they are compressed | 14:50 |
| JayF | WONTFIX | 14:50 |
| jlvillal | The configdrive I believe is sent via the REST API to the conductor. And then stored. If setup to use Swift, it will save the configdrive to Swift and instead store the URL | 14:50 |
| JayF | because it's just not an awesome pattern | 14:50 |
| JayF | I'd like to see someone run this patch | 14:50 |
| JayF | and successfully deploy a configdrive of triple-digit-mb size | 14:50 |
| *** mkam has quit IRC | 14:50 | |
| JayF | I think we're pretending making a a db field larger is all we'd have to do | 14:50 |
| JayF | Madasi: you around yet? I bet you'd have an interesting insight into this | 14:50 |
| dtantsur | well, I'd prefer to not store config drive in this way at all | 14:51 |
| jroll | well, we max at 32MB | 14:51 |
| lucasagomes | the only reason we keep the config drive in the node is because of rebuild | 14:51 |
| lucasagomes | but we shouldn't, cause nova can regeneate the configdrive at rebuild time | 14:51 |
| dtantsur | I'm fine with keeping it, I'm not fine with keeping it this way :) | 14:51 |
| TheJulia | ++ | 14:51 |
| JayF | lucasagomes: ah yeah, we were talking about that a few weeks back too | 14:51 |
| lucasagomes | so we should change our API/CLI to accept passing it | 14:51 |
| JayF | jroll: ^ I think we should WONTFIX, and the real fix is to stop storing configdrive altogether | 14:51 |
| TheJulia | lucasagomes: I think those changes got proposed | 14:51 |
| JayF | I think there's at least a bug for it | 14:52 |
| * lucasagomes looks | 14:52 | |
| JayF | https://bugs.launchpad.net/ironic/+bug/1575935 | 14:52 |
| openstack | Launchpad bug 1575935 in Ironic "[RFE] Rebuild should also accept a configdrive" [Wishlist,Confirmed] - Assigned to Stephane Miller (stephaneeee) | 14:52 |
| jroll | JayF: if this human wants to solve the problem quickly, wdyt about node.configdrive field (or in another table), make that larger to fix their bug, then when we real-fix it we can drop that column | 14:52 |
| lucasagomes | JayF, nice | 14:53 |
| JayF | jroll: how about that human installs swift and uses our reasonable option for using superlarge configdrives? or makes a smaller script to pull down his bigger one? | 14:53 |
| JayF | jroll: there's a million workarounds that don't involve us ballooning our db | 14:53 |
| *** baoli has quit IRC | 14:53 | |
| dtantsur | do we always default to swift if it's present? | 14:53 |
| jroll | JayF: temporarily ballooning* | 14:53 |
| JayF | dtantsur: no | 14:53 |
| jroll | we don't, because we've no way to tell if swift is there | 14:53 |
| *** baoli has joined #openstack-ironic | 14:54 | |
| dtantsur | aha, so this needs configuration as well? | 14:54 |
| jroll | right | 14:54 |
| jroll | also 'just install swift' isn't a very friendly answer :/ | 14:54 |
| lucasagomes | JayF, edited that bug to -rfe, it's just a bug | 14:54 |
| dtantsur | I'm asking because I remember config drives on nodes in tripleo, where we definitely have swift | 14:54 |
| lucasagomes | dtantsur, no, we default to saving it to the node | 14:55 |
| lucasagomes | we can ask ironic to save it in swift by changing a config | 14:55 |
| JayF | jroll: "make a db field 2^16 larger" is not a very friendly patch | 14:55 |
| dtantsur | if it's the default, I'm afraid we have to fix it | 14:55 |
| jroll | so what's the path to fixing this for real? make nova re-send the configdrive, and change all the code to pass it through? probably not much work yeah? (keep in mind we can't drop it from ironic for a cycle after the change lands in nova) | 14:56 |
| lucasagomes | someone knows stephane miller ? | 14:56 |
| jroll | lucasagomes: that's cinerama | 14:56 |
| jroll | I believe | 14:56 |
| lucasagomes | a-ha, right :-) | 14:56 |
| TheJulia | that is her | 14:56 |
| JayF | jroll: yep, exactly, and then in a later release ironic doesn't store configdrive at all | 14:56 |
| lucasagomes | cinerama, are you working on bug/1575935 ? | 14:56 |
| JayF | jroll: except during deployment, I think, we'd have to keep it somewhere | 14:56 |
| jroll | JayF: that's why I'm saying for now, add a table/column | 14:56 |
| jroll | oh you're right | 14:56 |
| jroll | blah | 14:56 |
| TheJulia | I don't think she is right now. She also likely will not be on for another hour | 14:57 |
| JayF | I mean, a different field, with a max length <4gb and I'm a little OK with it | 14:57 |
| *** dprince has quit IRC | 14:57 | |
| lucasagomes | TheJulia, thanks, yeah she might check the ping later. Left a comment on that bug as well | 14:57 |
| jroll | right, MEDIUMTEXT somewhere outside of instance_info | 14:57 |
| JayF | I'm very nonplussed about the idea os making instance_info that large | 14:57 |
| dtantsur | jroll++ | 14:57 |
| *** dprince has joined #openstack-ironic | 14:57 | |
| jroll | JayF: I agree, I never said such a thing :) | 14:57 |
| jroll | or I haven't in some time | 14:58 |
| JayF | jroll: a patch that has lots of +2 votes says that, lol | 14:58 |
| TheJulia | jroll: ++, also since the drive is compressed, one should be able to fix the maximum size into that field while deploying | 14:58 |
| * jroll puts up a vote | 14:58 | |
| dtantsur | TheJulia, unless they put something that does not compress well there :) | 14:58 |
| TheJulia | dtantsur: true... :( | 14:58 |
| dtantsur | anyway, whatever limit we put, we should fail early, on the API stage | 14:58 |
| TheJulia | but if it is tons of text! :) | 14:58 |
| dtantsur | yeah, most often it is tons of text, maybe some binaries | 14:59 |
| JayF | dtantsur: I guess another possible fix is: API rejects configdrives that are too large if not stored in swift, fails cleanly, documentation + configs say "if you need >Xkb use swift" | 14:59 |
| *** jraju has quit IRC | 14:59 | |
| jroll | I need a browser with readline support, too many ^W closing windows while I'm typing :( | 14:59 |
| TheJulia | How have we not had someone hitting this before now though... have we really managed to convince people not to put binaries in their config drives all along? | 14:59 |
| dtantsur | JayF, my problem with it is that you don't know until you hit it | 15:00 |
| JayF | TheJulia: Because having huge config scripts is a major antipattern? | 15:00 |
| dtantsur | JayF, so we're kind of moving the problem to users | 15:00 |
| JayF | TheJulia: the proper thing is to bootstrap a *real* config management system using cloud-init, not to have it be huge from all your configs | 15:00 |
| JayF | dtantsur: ^ IMO the user is moving the problem to us, trying to use a big configdrive | 15:00 |
| dtantsur | tripleo config drives are JSONs, they compress well | 15:00 |
| TheJulia | JayF: touché | 15:00 |
| dtantsur | JayF, well, at the very least we should document it and reject too big config drives on API stage | 15:01 |
| JayF | TheJulia: I'm pretty sure that's actually in cloud-init documentation, too | 15:01 |
| JayF | dtantsur: I agree with that, and I agree we shouldn't store configdrives long term (the bug posted above) | 15:01 |
| JayF | I'm unhappy but can deal with a separate field being made that's bigger, but I feel like we should be a little opinionated here | 15:02 |
| *** causten has joined #openstack-ironic | 15:02 | |
| JayF | and say this use case for configdrive is not awesome, and if you want it, you have to be configured differently than default | 15:02 |
| dtantsur | side question: which option do I set to enable config drives in Swift? | 15:02 |
| * jroll lets this conversation finish before he posts an opinion | 15:02 | |
| jroll | dtantsur: configdrive_use_swift | 15:02 |
| JayF | dtantsur: https://github.com/openstack/ironic/blob/master/etc/ironic/ironic.conf.sample#L689 | 15:02 |
| jroll | and a couple related ones | 15:02 |
| *** mkam has joined #openstack-ironic | 15:03 | |
| dtantsur | meh, it's not in puppet-ironic. will have to add then. thanks | 15:03 |
| lucasagomes | dtantsur, ++ since we always have swift that would be a good thing to have | 15:04 |
| dtantsur | will hack on it tomorrow, yeah. though I hope we'll never hit the limit there. | 15:04 |
| lucasagomes | dtantsur, right, I don't think we inject anything in the config drive | 15:05 |
| lucasagomes | right? so it's unlikely | 15:05 |
| dtantsur | lucasagomes, for tripleo? os-net-config settings, but that's a relatively small JSON | 15:05 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Add Network Interface Attach and Detach APIs https://review.openstack.org/327046 | 15:06 |
| lucasagomes | ah right, yeah those are pretty small | 15:06 |
| *** scarab_ has quit IRC | 15:10 | |
| *** jjohnson2 has joined #openstack-ironic | 15:13 | |
| *** fxpester has quit IRC | 15:14 | |
| openstackgerrit | Imre Farkas proposed openstack/ironic: DRAC: list unfinished jobs https://review.openstack.org/367451 | 15:14 |
| jlvillal | Our customer ran into too large of a configdrive due to the size of their team and all the SSH public keys they tried to put into the config drive | 15:16 |
| jroll | O_O | 15:16 |
| jlvillal | That's what they told me :) | 15:17 |
| * TheJulia seconds jroll's O_O | 15:17 | |
| sambetts | thats a lot of SSH keys... | 15:18 |
| *** mbound_ has quit IRC | 15:18 | |
| TheJulia | they really shouldn't compress well, but that is still a lot | 15:19 |
| sambetts | where they using the OpenStack keypairs thing? or some other way to get the SSH keys into the instances? | 15:19 |
| sambetts | s/where/were | 15:19 |
| TheJulia | I'm curious why they are setting that up front instead of using a disposable key pair that gets ripped out after the fact from the instance once it is up and running | 15:19 |
| *** praneshp has joined #openstack-ironic | 15:20 | |
| rloo | hi and morning everyone, sambetts, TheJulia, jlvillal, jroll, lucasagomes, dtantsur, JayF | 15:20 |
| JayF | o/ | 15:20 |
| TheJulia | good morning rloo | 15:20 |
| lucasagomes | rloo, morning | 15:20 |
| jroll | morning rloo :) | 15:20 |
| sambetts | hey rloo | 15:20 |
| *** sdake_ has joined #openstack-ironic | 15:21 | |
| *** sabeen1 has quit IRC | 15:21 | |
| *** sdake has quit IRC | 15:22 | |
| *** mjura has quit IRC | 15:24 | |
| dtantsur | morning rloo | 15:26 |
| *** sabeen has joined #openstack-ironic | 15:27 | |
| *** sborkows has quit IRC | 15:27 | |
| *** amotoki has joined #openstack-ironic | 15:29 | |
| *** dtantsur is now known as dtantsur|afk | 15:29 | |
| *** dprince has quit IRC | 15:30 | |
| *** dprince has joined #openstack-ironic | 15:31 | |
| *** hshiina has quit IRC | 15:31 | |
| *** amotoki has quit IRC | 15:34 | |
| openstackgerrit | Merged openstack/ironic: Update ironic/ironic.conf.sample https://review.openstack.org/367342 | 15:36 |
| *** dprince has quit IRC | 15:37 | |
| *** mgoddard_ has joined #openstack-ironic | 15:37 | |
| *** jvgrant has joined #openstack-ironic | 15:40 | |
| *** mgoddard has quit IRC | 15:40 | |
| *** mgoddard_ has quit IRC | 15:44 | |
| *** MattMan has quit IRC | 15:46 | |
| *** MattMan has joined #openstack-ironic | 15:47 | |
| * jlvillal wonders if we could have some sort of proposal bot job for the ironic.conf.sample ? | 15:50 | |
| *** vmud213 has joined #openstack-ironic | 15:56 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Update unit tests for neutron interface https://review.openstack.org/367038 | 15:56 |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with Chassis. https://review.openstack.org/343796 | 15:57 |
| *** mgoddard has joined #openstack-ironic | 15:58 | |
| openstackgerrit | Matthew Treinish proposed openstack/ironic-inspector: DNM: Test a tempest ssh patch https://review.openstack.org/367480 | 16:01 |
| mtreinish | dtantsur|afk, aarefiev: ^^^ | 16:02 |
| jlvillal | mtreinish: DNM: Desperately Needs Merging? ;) | 16:05 |
| mtreinish | haha | 16:05 |
| mtreinish | jlvillal: so the patch that's testing is doing terrible things in unit tests on my local machine (I should have run those before pushing it) | 16:07 |
| mtreinish | it's sitting at ~15GB of ram consumed and growing | 16:08 |
| jlvillal | mtreinish: That has never ever ever happened to me. Nope never ;) | 16:08 |
| * jlvillal too often runs the tox battery of tests after he pushes ... | 16:09 | |
| *** praneshp has quit IRC | 16:09 | |
| mtreinish | well it has stopped growing at 16.3G of ram used | 16:09 |
| *** rbrud has quit IRC | 16:10 | |
| JayF | I think there are better ways for inspector to determine how much ram a node has than to use it all up /s | 16:10 |
| sambetts | mtreinish: woah?! | 16:10 |
| TheJulia | JayF: lol | 16:11 |
| *** mbound has joined #openstack-ironic | 16:19 | |
| *** vsaienk0 has quit IRC | 16:19 | |
| *** tesseract- has quit IRC | 16:21 | |
| sambetts | jroll: What do you think about the possiblity of splitting the Ironic tempest API tests away from the end to end deployment test? I find the junk requests in the Ironic conductor/api logs generated by the API tests makes it hard to debug issues | 16:22 |
| *** trown is now known as trown|outtypewww | 16:23 | |
| jroll | sambetts: +1, I've been wanting to do that for a while now | 16:24 |
| *** mbound has quit IRC | 16:24 | |
| jroll | sambetts: also means we should be able to make tempest run parallel for scenario jobs | 16:24 |
| sambetts | both cases should be parallisable if they are split would be awesome | 16:25 |
| sambetts | which would be awesome* | 16:25 |
| sambetts | jroll: do you think infra will mind if we add another job per patch ? | 16:26 |
| jroll | sambetts: should be fine | 16:27 |
| sambetts | cool :) I'll add that to my todo list then | 16:27 |
| jroll | <3 | 16:27 |
| *** rama_y has joined #openstack-ironic | 16:29 | |
| *** dprince has joined #openstack-ironic | 16:31 | |
| *** amotoki has joined #openstack-ironic | 16:31 | |
| *** amotoki has quit IRC | 16:31 | |
| *** amotoki has joined #openstack-ironic | 16:31 | |
| *** fxpester has joined #openstack-ironic | 16:31 | |
| *** ametts has joined #openstack-ironic | 16:32 | |
| *** fragatina has joined #openstack-ironic | 16:37 | |
| *** fragatina has quit IRC | 16:41 | |
| *** rbrud has joined #openstack-ironic | 16:41 | |
| *** clenimar has joined #openstack-ironic | 16:43 | |
| mat128 | JayF: there are, but not for dtantsur: https://bugzilla.redhat.com/show_bug.cgi?id=1366309 | 16:44 |
| openstack | bugzilla.redhat.com bug 1366309 in openstack-ironic-inspector "ironic introspection fails on GCE node" [Unspecified,Closed: wontfix] - Assigned to dtantsur | 16:44 |
| lucasagomes | folks I'm calling it a day | 16:47 |
| lucasagomes | have a great evening all | 16:47 |
| mat128 | o/ Lucas | 16:48 |
| lucasagomes | see ye later :-) | 16:48 |
| *** lucasagomes is now known as lucas-dinner | 16:48 | |
| *** derekh has quit IRC | 16:49 | |
| *** devananda|travel is now known as devananda | 16:56 | |
| *** jheroux has quit IRC | 16:57 | |
| *** anush has quit IRC | 17:00 | |
| *** jaosorior has quit IRC | 17:01 | |
| *** ElCoyote_ has quit IRC | 17:02 | |
| *** ElCoyote_ has joined #openstack-ironic | 17:04 | |
| *** ohamada has quit IRC | 17:05 | |
| *** leitan has quit IRC | 17:06 | |
| *** vmud213 has quit IRC | 17:13 | |
| NobodyCam | Night lucas-dinner | 17:17 |
| NobodyCam | and Good Morning Ironic'ers :) | 17:17 |
| mat128 | morning NobodyCam | 17:18 |
| *** leitan has joined #openstack-ironic | 17:19 | |
| NobodyCam | morning mat128 :) | 17:19 |
| jlvillal | Good morning NobodyCam mat128 | 17:19 |
| NobodyCam | morning jlvillal :) | 17:20 |
| jlvillal | And everyone else! | 17:20 |
| *** michauds has quit IRC | 17:30 | |
| openstackgerrit | Sam Betts proposed openstack/ironic: Add Network Interface Attach and Detach APIs https://review.openstack.org/327046 | 17:32 |
| * sambetts -> home | 17:33 | |
| sambetts | Night everyone! | 17:33 |
| *** sambetts is now known as sambetts|afk | 17:34 | |
| NobodyCam | night sambetts|afk | 17:34 |
| sambetts|afk | o/ NobodyCam | 17:34 |
| NobodyCam | :) | 17:34 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add power state change notifications https://review.openstack.org/321865 | 17:34 |
| rloo | jlvillal: last chance, i'm going to forget about it after this. what did you want to know about https://review.openstack.org/#/c/364440/ | 17:35 |
| patchbot | patch 364440 - python-ironicclient - osc node power on & off commands | 17:35 |
| jlvillal | rloo: Sorry, my boss wants to meet now for a 1:1 | 17:36 |
| rloo | jlvillal: lucky you ;) | 17:36 |
| jlvillal | Woo hoo! :) | 17:36 |
| *** rama_y has quit IRC | 17:44 | |
| *** krtaylor has quit IRC | 17:49 | |
| *** jpich has quit IRC | 17:50 | |
| *** sabeen has quit IRC | 17:53 | |
| *** e0ne has quit IRC | 17:58 | |
| *** fragatina has joined #openstack-ironic | 18:01 | |
| *** ifarkas is now known as ifarkas_afk | 18:04 | |
| *** vishwanathj is now known as vishwanathj_ | 18:06 | |
| *** Romanenko_K has quit IRC | 18:07 | |
| *** ChubYann has joined #openstack-ironic | 18:14 | |
| *** rama_y has joined #openstack-ironic | 18:17 | |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add power state change notifications https://review.openstack.org/321865 | 18:20 |
| *** athomas has quit IRC | 18:21 | |
| rloo | jroll: do we care if http://docs.openstack.org/releasenotes/python-ironicclient/current-series.html won't exist after this patch: https://review.openstack.org/#/c/366980/? | 18:22 |
| patchbot | patch 366980 - python-ironicclient - standardize release note page names and ordering | 18:22 |
| rloo | jroll: I mean, I don't particularly care but I can fix if TheRoyalWe care :) | 18:23 |
| *** bfournie has quit IRC | 18:23 | |
| jroll | rloo: in theory I care, but not really | 18:24 |
| JayF | heh | 18:24 |
| JayF | the unrendered versino of that page on github | 18:24 |
| JayF | is higher in google than the rendered version when searching for "ironicclient release notes" | 18:24 |
| *** krtaylor has joined #openstack-ironic | 18:24 | |
| JayF | so I'd say it's likely OK :P not killing any major google karma or anything | 18:24 |
| rloo | jroll: ha ha. ok, then forget it. i also wonder if that page will actually disappear (there were issues before with pages not being removed, so will see) | 18:25 |
| jroll | yeah, doubt it will | 18:25 |
| jroll | they just write whatever the job writes out | 18:25 |
| jroll | s/write/publish | 18:26 |
| rloo | jroll: in which case, someone can submit another patch to link from the old to the new page. i'm going to +2 doug's patch. | 18:26 |
| jroll | rloo: sounds good | 18:26 |
| *** sdake_ is now known as sdake | 18:27 | |
| mat128 | JayF: not here https://www.dropbox.com/s/eijela6fmippv5v/Screen%20Shot%202016-09-08%20at%202.29.15%20PM.png?dl=0 | 18:29 |
| mat128 | weird | 18:29 |
| *** michauds has joined #openstack-ironic | 18:33 | |
| JayF | mat128: look for current-series | 18:33 |
| JayF | mat128: the current-series file is only in the end of your thing | 18:33 |
| mat128 | Ahhh | 18:34 |
| mat128 | right | 18:34 |
| mat128 | my bad :) | 18:34 |
| *** aNupoisc has joined #openstack-ironic | 18:34 | |
| *** mgoddard has quit IRC | 18:43 | |
| *** Sukhdev has joined #openstack-ironic | 18:51 | |
| mat128 | anyone here who wants to look at https://review.openstack.org/#/c/363294/ ? :) | 18:52 |
| patchbot | patch 363294 - ironic - Add a basic install guide | 18:52 |
| JayF | https://review.openstack.org/#/c/356811/ more docs that could use review love | 18:53 |
| patchbot | patch 356811 - ironic - Add metrics to administrator guide | 18:53 |
| *** david-lyle has quit IRC | 18:53 | |
| *** david-lyle has joined #openstack-ironic | 18:53 | |
| *** sabeen has joined #openstack-ironic | 18:54 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic-specs: [WIP] API Evolution https://review.openstack.org/367583 | 18:58 |
| devananda | jroll: even though we aren't having an api meeting for a couple weeks, I've got a draft up ^ . | 19:00 |
| *** jralph_ has joined #openstack-ironic | 19:00 | |
| devananda | more substance still to write in these (and some more specs to add) but I'd like to know what folks think of the format | 19:00 |
| patchbot | Error: I haven't seen substance. | 19:00 |
| *** vishwanathj_ is now known as vishwanathj | 19:00 | |
| *** mbound has joined #openstack-ironic | 19:00 | |
| devananda | breaking things down into separate specs like this, with one overall spec | 19:01 |
| jroll | devananda: awesome, thanks, will review later | 19:03 |
| * jroll probably done for the day | 19:03 | |
| *** dprince has quit IRC | 19:16 | |
| *** e0ne has joined #openstack-ironic | 19:21 | |
| *** moshele has joined #openstack-ironic | 19:25 | |
| *** mgoddard has joined #openstack-ironic | 19:36 | |
| *** mgould is now known as mgould|afk | 19:37 | |
| *** fxpester has quit IRC | 19:49 | |
| *** sdake_ has joined #openstack-ironic | 19:50 | |
| *** sdake has quit IRC | 19:52 | |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Add volume_connector table to save connector information https://review.openstack.org/200983 | 20:03 |
| *** moshele has quit IRC | 20:03 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 20:05 |
| openstackgerrit | Fatih Degirmenci proposed openstack/bifrost: Test OPNFV CI verify job. Please ignore. https://review.openstack.org/367608 | 20:11 |
| *** mgoddard has quit IRC | 20:16 | |
| *** e0ne has quit IRC | 20:17 | |
| *** e0ne has joined #openstack-ironic | 20:18 | |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Introduce new RPCs to support volume connector operation https://review.openstack.org/214585 | 20:28 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Add volume_targets table to save target information https://review.openstack.org/285218 | 20:29 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Introduce VolumeTarget object https://review.openstack.org/285219 | 20:30 |
| *** Goneri has quit IRC | 20:31 | |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Introduce new RPCs to support volume target operations https://review.openstack.org/285220 | 20:31 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add volume_connectors and volume_targets to task https://review.openstack.org/366196 | 20:35 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: [WIP] Add storage_interface DB field and object https://review.openstack.org/348005 | 20:36 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: [WIP] Add storage_interface to base driver class https://review.openstack.org/348006 | 20:36 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: [WIP] Add base cinder common interface https://review.openstack.org/355625 | 20:36 |
| *** hamzy has quit IRC | 20:38 | |
| *** hogepodge has joined #openstack-ironic | 20:39 | |
| hogepodge | hi, I've been playing around with ironic | 20:40 |
| hogepodge | does anyone have a sample inventory file that they've used for provisioning NUCs using AMT? | 20:40 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Introduce new RPCs to support volume target operations https://review.openstack.org/285220 | 20:41 |
| * hogepodge glances sideways at devananda | 20:41 | |
| TheJulia | hogepodge: using bifrost? | 20:41 |
| hogepodge | TheJulia: yes | 20:42 |
| TheJulia | hogepodge: You know, I finally got an AMT device that is supported by the AMT driver, and never tried it out :( | 20:42 |
| hogepodge | TheJulia: so I have auth-less ironic set up, and my management interfaces are active, I just don't have any solid docs on how to configure the amt driver | 20:42 |
| *** bfournie has joined #openstack-ironic | 20:43 | |
| david-lyle | hogepodge: I tried that out for a while and could never get it to work | 20:43 |
| hogepodge | david-lyle: well that makes me sad | 20:44 |
| david-lyle | but I'm no expert | 20:44 |
| TheJulia | the amt driver, and interface... can be problematic. | 20:44 |
| hogepodge | TheJulia: yes? | 20:44 |
| hogepodge | TheJulia: there are two drivers, right? | 20:44 |
| TheJulia | hogepodge: the driver_info parameters can be found https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/amt/common.py#L38 | 20:44 |
| TheJulia | hogepodge: so, only one in tree | 20:44 |
| hogepodge | TheJulia: bifrost installed pxe_amt automatically | 20:45 |
| TheJulia | the other ?two? are out of tree. The main question is what version is the AMT controller in your NUC? | 20:45 |
| hogepodge | but it looked like agent_amt was also an option? | 20:45 |
| TheJulia | hogepodge: so agent_amt does appear to be an option if enabled in the config file. Mainly it has to do with method of deployment differences. There is no difference as far as I'm aware with regards to the hardware supported | 20:46 |
| openstackgerrit | Merged openstack/python-ironicclient: standardize release note page names and ordering https://review.openstack.org/366980 | 20:47 |
| *** absubram has joined #openstack-ironic | 20:48 | |
| hogepodge | TheJulia: AMT version 8.1.40 | 20:49 |
| TheJulia | hogepodge: so, according to http://docs.openstack.org/developer/ironic/drivers/amt.html that Might just work | 20:50 |
| TheJulia | hogepodge: so kind of like https://github.com/openstack/bifrost/blob/master/playbooks/inventory/baremetal.json.example#L31 lines 30-32, just amt_username, amt_password, amt_address | 20:51 |
| hogepodge | TheJulia: Ah, I see that. Thanks! I'll give it a try | 20:52 |
| rloo | hogepodge: i'd be interested if it works for you. I've been thinking about using a NUC, but in the past, I heard that the AMT part was flaky. Don't know if that has been improved or not. | 20:52 |
| hogepodge | TheJulia: hopefully I'll have 8 machines paved by end of day | 20:52 |
| hogepodge | rloo: I'm running on an ubuntu orange box that broke horribly after a recent update, and I wanted to give up on maas anyway, so this gave me the perfect opportunity | 20:53 |
| rloo | hogepodge: crossing my fingers :) | 20:53 |
| JayF | rloo: intel wants to give all ironic cores NUCs for testing? That sounds great! :P | 20:55 |
| * TheJulia fully supports this idea :) | 20:58 | |
| *** harlowja_ has joined #openstack-ironic | 20:59 | |
| *** causten has quit IRC | 20:59 | |
| rloo | JayF: Did you hear that? I didn't know. I have been thinking of getting something besides just my laptop... how ironic is it that i have no hardware? | 21:00 |
| JayF | I was mainly joking :P | 21:00 |
| rloo | JayF: :) | 21:00 |
| JayF | I don't have a hardware test lab for Ironic | 21:00 |
| *** harlowja has quit IRC | 21:00 | |
| JayF | I mean, I guess OnMetal has a hardware test lab | 21:00 |
| rloo | JayF: how do you try things out? | 21:00 |
| JayF | but we don't deploy devstack there generally or do end-to-end testing | 21:00 |
| rloo | JayF: yeah, you can use OnMetal, right? | 21:00 |
| JayF | rloo: devstack, mostly | 21:01 |
| JayF | rloo: unless it's a feature we already put in onmetal | 21:01 |
| rloo | JayF: but what do you run devstack on? | 21:01 |
| * jlvillal still trying to get a few old servers to use. Still has not had any luck. | 21:01 | |
| JayF | either a VM or an OnMetal server | 21:01 |
| JayF | I have a plan to buy a shiny new desktop computer and turn my old one into a devstack box :) | 21:01 |
| rloo | JayF: oh, that's a good plan. | 21:02 |
| rloo | JayF: I was only thinking of a nuc, in case I could figure out how to get Intel to give me one. or two :) | 21:02 |
| *** e0ne has quit IRC | 21:02 | |
| JayF | if you wanna contribute to project-jay-new-desktop, I could use an i7... | 21:03 |
| JayF | hehe | 21:03 |
| rloo | JayF: haha. I wish. | 21:03 |
| *** causten has joined #openstack-ironic | 21:05 | |
| *** jcoufal has quit IRC | 21:06 | |
| rloo | JayF, TheJulia: did either of you want to +A, or did you want more eyes on https://review.openstack.org/#/c/367306/? | 21:06 |
| patchbot | patch 367306 - ironic - Grub2 by default for PXE + UEFI | 21:06 |
| devananda | if folks are interested, I can write up a thing on running ironic in a VM on my laptop, to deploy images to a NUC on my LAN | 21:09 |
| devananda | but not right now - I need to run out to an appointment. bbiah | 21:09 |
| *** mmedvede has quit IRC | 21:09 | |
| *** devananda is now known as devananda|bbl | 21:09 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Add metrics to administrator guide https://review.openstack.org/356811 | 21:09 |
| JayF | rloo: wanna take another gander at that; I just responded to your comments ^ | 21:09 |
| jlvillal | Not sure if this is out of date or not. From mrda: http://lifelog.michaeldavies.org/2015/09/ironic-on-nuc-part-1-setting-things-up.html | 21:09 |
| rloo | devananda: ++ (leave it to him to entice us and then run away!) | 21:09 |
| rloo | JayF: sure, is jenkins fast today? I also saw another doc patch from you on the install guide. maybe i will get to it too. today. | 21:10 |
| JayF | rloo: I always find it fastish on docs-only changes, but mainly wanted you to look at the edits so maybe we could do a quick irc turnaround if you're still unhappy with it | 21:11 |
| JayF | rloo: re: install-guide, mat128 and I are working on migrating the install/admin guide from developer docs into the proper location. We might even get it done by newton release \o/ | 21:11 |
| rloo | JayF: OH. ok, in a sec. was looking up 'built-in function'. if we add a function to our code, it cannot be a built-in function, since we're adding it. | 21:12 |
| rloo | JayF: related to a patch I was jsut reviewing :) | 21:12 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Add metrics to administrator guide https://review.openstack.org/356811 | 21:13 |
| JayF | I reviewed myself and already fixed a nit :P | 21:13 |
| rloo | JayF: even better! | 21:13 |
| *** causten has quit IRC | 21:14 | |
| jlvillal | rloo: https://review.openstack.org/#/c/365491/ | 21:14 |
| patchbot | patch 365491 - ironic - Add __ne__ built-in function for API Version object | 21:14 |
| jlvillal | I just commented on that with a -1. | 21:15 |
| rloo | jlvillal: i just saw your comment | 21:15 |
| rloo | jlvillal: you sure about that? | 21:15 |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Allow user to get iPXE files https://review.openstack.org/367635 | 21:15 |
| rloo | jlvillal: i don't want to spend more time on it if i don't have to :) | 21:15 |
| jlvillal | rloo: That's what the docs say. | 21:15 |
| *** e0ne has joined #openstack-ironic | 21:15 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Add metrics to administrator guide https://review.openstack.org/356811 | 21:15 |
| jlvillal | rloo: I would like more documentation in the commit message to sway my opinion | 21:15 |
| *** cdearborn has quit IRC | 21:15 | |
| hogepodge | devananda|bbl: I am interested, hurry back! ;-) | 21:16 |
| jlvillal | rloo: I'll dig a bit more | 21:19 |
| *** ametts has quit IRC | 21:19 | |
| mrda | jlvillal: yes, that's out of date. I should update that for the OOT status of AMT now. (And I'd love to get an agent_ version working too, if I can find the time...) | 21:20 |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: [WIP] openSUSE support https://review.openstack.org/367637 | 21:20 |
| mrda | And good morning :) | 21:20 |
| jlvillal | Hi mrda | 21:20 |
| rloo | thx jlvillal | 21:20 |
| rloo | morning mrda | 21:20 |
| mrda | o/ | 21:20 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 21:23 |
| *** e0ne has quit IRC | 21:25 | |
| *** e0ne has joined #openstack-ironic | 21:26 | |
| *** e0ne has quit IRC | 21:26 | |
| *** wajdi_ has joined #openstack-ironic | 21:28 | |
| *** causten has joined #openstack-ironic | 21:29 | |
| jlvillal | rloo: Some code testing and looks the patch is correct. Cool :) My theory is that the default __ne__ compares the hash value. Learned something new. And think the functools docs should be updated. | 21:31 |
| *** wajdi has quit IRC | 21:31 | |
| rloo | jlvillal: thx for looking into it! | 21:31 |
| rloo | jlvillal: i was going to edit the commit and then +A. unless you want to. | 21:31 |
| *** ametts has joined #openstack-ironic | 21:31 | |
| jlvillal | rloo: I can do that. I want to put a link to the docs that they quoted | 21:32 |
| rloo | jlvillal: thx! | 21:32 |
| *** wajdi_ has quit IRC | 21:32 | |
| openstackgerrit | Fatih Degirmenci proposed openstack/bifrost: Test OPNFV CI verify job. Please ignore. https://review.openstack.org/367608 | 21:35 |
| cinerama | hi mrda | 21:37 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Add __ne__() function for API Version object https://review.openstack.org/365491 | 21:38 |
| jlvillal | rloo: ^^ | 21:38 |
| rloo | jlvillal: you have my permission to +2 and +A it all by yourself if you want | 21:40 |
| jlvillal | rloo: Done! | 21:40 |
| jlvillal | rloo: I hope you don't then complain about the commits message ;) | 21:40 |
| jlvillal | s/commits/commit/ | 21:41 |
| rloo | jlvillal: how could an edit'd commit by a core, be worse than what was there already? i think we should just do it to quickly improve things. | 21:41 |
| jlvillal | rloo: True | 21:42 |
| rloo | jlvillal: i'm trying to figure out how we can be more efficient yet not let quality suffer :) | 21:42 |
| jlvillal | Unless I haven't had my morning coffee. Then... | 21:42 |
| rloo | jlvillal: doesn't make sense to comment with the grammatical changes, and wait for the person to update it, blah blah. | 21:42 |
| jlvillal | +1 | 21:43 |
| rloo | jlvillal: having said that, i don't recall if that is a sanctioned 'thing' or if i had discussed with someone or if i had seen others doing it. | 21:43 |
| jlvillal | I think I've seen others do it. | 21:43 |
| rloo | jlvillal: i'll make a note to update some doc somewhere about it... | 21:44 |
| openstackgerrit | Merged openstack/ironic: Update unit tests for neutron interface https://review.openstack.org/367038 | 21:44 |
| openstackgerrit | Merged openstack/ironic: [api-ref] Remove temporary block in conf.py https://review.openstack.org/366473 | 21:45 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Update special methods to use (self, other) https://review.openstack.org/367643 | 21:46 |
| mrda | hey cinerama | 21:46 |
| openstackgerrit | Fatih Degirmenci proposed openstack/bifrost: Test OPNFV CI verify job. Please ignore. https://review.openstack.org/367608 | 21:47 |
| rloo | JayF: so I still don't understand what you mean by 'data points'. Is each timing metric that is emitted, considered a data point? If so, I don't see how you can say 100-500. It depends on the amount of activity going on. | 21:48 |
| JayF | rloo: so with a little bit of background: most systems that would back statsd, like graphite | 21:48 |
| rloo | JayF: if you do eg 'ironic node-show' for 1000 nodes, you'll get 1000 data points? | 21:49 |
| JayF | each distinctly named metric requires a certain amount of space on disk and processing time | 21:49 |
| JayF | the number of times that named metric is emitted doesn't matter | 21:49 |
| JayF | the number of distinctly named metrics do | 21:49 |
| rloo | JayF: ok, that's what I wanted to know. | 21:49 |
| JayF | so I'm trying to say, without getting into the implementation details of statsd+graphite, that ironic will create 100-500 whisper dbs on disk on your graphite server :) | 21:50 |
| JayF | maybe I should say "distinctly named" data points? | 21:50 |
| rloo | JayF: basically, there is a timing metric for each method, and there are 100-500 different methods that are timed? | 21:51 |
| mariojv | JayF: what patch set is this in reference to? | 21:51 |
| rloo | mariojv: https://review.openstack.org/#/c/356811/ | 21:51 |
| patchbot | patch 356811 - ironic - Add metrics to administrator guide | 21:51 |
| mariojv | thanks | 21:51 |
| JayF | mariojv: https://review.openstack.org/#/c/356811/ | 21:51 |
| patchbot | patch 356811 - ironic - Add metrics to administrator guide | 21:51 |
| JayF | rloo: pretty much, yeah | 21:52 |
| JayF | rloo: but it's also, if you exercise that code | 21:52 |
| jlvillal | JayF: mariojv: Would https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L2329-L2363 be a good candidate for a metric? | 21:52 |
| jlvillal | JayF: mariojv I was thinking of the case where it ends up using Swift. | 21:52 |
| rloo | JayF: do we really have > 100 timing metrics already? wow. | 21:52 |
| mariojv | jlvillal: i agree, think i commented there on some other patch set | 21:52 |
| JayF | jlvillal: probably so | 21:53 |
| JayF | rloo: yep, it doesn't take long | 21:53 |
| JayF | rloo: and in the world of metrics, that's not a crazy large number | 21:53 |
| rloo | JayF: cool. | 21:53 |
| *** mmedvede has joined #openstack-ironic | 21:53 | |
| jlvillal | mariojv: JayF Thanks | 21:54 |
| JayF | rloo: so wdyt? Should I change it to 'distinctly named' metrics, rewrite it differently? exclude it? leave it be? | 21:54 |
| mariojv | sort of tangentially related, i found this useful for estimating size of a whisper db (graphite storage backend) a while ago: https://gist.github.com/jjmaestro/5774063 | 21:55 |
| patchbot | (graphite a a ago: backend) db estimating for found https://gist.github.com/jjmaestro/5774063 i of of related, size storage tangentially this useful while whisper | 21:55 |
| rloo | JayF: should be changed. i'm thinking about it. | 21:55 |
| rloo | JayF: ok, i looked at the spec to refresh myself on terminology. so the 'metric name' is what distinguishes what you are talking about, right? | 21:56 |
| *** ametts has quit IRC | 21:57 | |
| JayF | rloo: well (prefix + name), but yeah | 21:57 |
| mariojv | i like "distinctly named", i think that makes it clear enough if other backends were added in the future too | 21:58 |
| mariojv | it seems generalizable enough to encompass the different types of metrics we may collect as well (time series, counter, etc) | 22:00 |
| rloo | JayF: ok, i suggested something but i'm not sure it is that great. But I think you know what I mean now :) | 22:01 |
| JayF | I might just put in the long form explanation | 22:02 |
| JayF | use distinctly named metrics, which in some backends may map to required disk capacity to store them | 22:02 |
| JayF | or something like that | 22:02 |
| rloo | JayF: I am mostly gone now. will pop in a bit, off/on this evening. | 22:02 |
| rloo | JayF: that seems good to me | 22:03 |
| JayF | It's fine, I'll be gone soon too and have tomorrow off :) | 22:03 |
| JayF | I just want to get these docs merged up, since all that's left is a little polish | 22:03 |
| *** hj-hpe has quit IRC | 22:04 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Add metrics to administrator guide https://review.openstack.org/356811 | 22:05 |
| *** mbound has quit IRC | 22:08 | |
| openstackgerrit | David Edery proposed openstack/ironic: Fix non-deployed nodes due to short nodes.instance_info https://review.openstack.org/334967 | 22:09 |
| *** dtantsur|afk has quit IRC | 22:12 | |
| *** baoli has quit IRC | 22:13 | |
| *** baoli has joined #openstack-ironic | 22:13 | |
| *** michauds has quit IRC | 22:14 | |
| *** dtantsur has joined #openstack-ironic | 22:14 | |
| *** absubram has quit IRC | 22:15 | |
| *** harlowja has joined #openstack-ironic | 22:18 | |
| openstackgerrit | Merged openstack/bifrost: Add possibility to set source branch for ironic-inspector https://review.openstack.org/363300 | 22:19 |
| *** mkam has quit IRC | 22:20 | |
| *** harlowja_ has quit IRC | 22:22 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 22:24 |
| *** sabeen has quit IRC | 22:31 | |
| *** sdake has joined #openstack-ironic | 22:34 | |
| *** sdake_ has quit IRC | 22:36 | |
| *** sai has quit IRC | 22:49 | |
| *** krtaylor has quit IRC | 22:51 | |
| *** sai has joined #openstack-ironic | 22:54 | |
| *** sdake has quit IRC | 22:59 | |
| *** Sukhdev has quit IRC | 23:00 | |
| *** rbrud has quit IRC | 23:02 | |
| *** leitan has quit IRC | 23:05 | |
| *** baoli has quit IRC | 23:06 | |
| hogepodge | TheJulia: ok, I can control the power state of the nucs using ironic, which is good. However, I'm stuck on the nucs finding the dhcp server to pxe boot from | 23:11 |
| *** sai has quit IRC | 23:11 | |
| *** rook has quit IRC | 23:11 | |
| *** lucas-dinner has quit IRC | 23:12 | |
| *** watanabe_isao has joined #openstack-ironic | 23:22 | |
| *** rook has joined #openstack-ironic | 23:26 | |
| *** sai has joined #openstack-ironic | 23:26 | |
| *** rook is now known as Guest90983 | 23:26 | |
| *** lucasagomes has joined #openstack-ironic | 23:26 | |
| TheJulia | hogepodge: using what bifrost installed? installed your own? | 23:45 |
| *** krtaylor has joined #openstack-ironic | 23:45 | |
| hogepodge | TheJulia: I installed bifrost from the provided instructions | 23:46 |
| hogepodge | TheJulia: I'm seeing the bootp/dhcp request from the node, but dnsmasq isn't responding | 23:47 |
| devananda|bbl | hogepodge: check for local firewall | 23:48 |
| *** devananda|bbl is now known as devananda | 23:48 | |
| TheJulia | ^^^ that and looking up the other likely possibility | 23:48 |
| hogepodge | devananda: yup, disabled. Got rid of it! :-D | 23:49 |
| TheJulia | hogepodge: check your /etc/dnsmasq.conf, make sure interface= is set to the right interface | 23:49 |
| hogepodge | TheJulia: yeah, br0, which is the local private network | 23:50 |
| devananda | TheJulia: is bifrost by default configuring dnsmasq with a whitelist, or a default response? | 23:50 |
| devananda | hogepodge: how is the NUC connected to your machine? | 23:50 |
| TheJulia | hogepodge: and you see the requests on br0? | 23:50 |
| devananda | also, are you tcpdump'ing on the host or the VM where bifrost is installed? | 23:50 |
| hogepodge | devananda: through a switch | 23:50 |
| devananda | hogepodge: any other DHCP servers on that network? | 23:51 |
| hogepodge | TheJulia: yeah, I'm seeing the packets arrive with tcpdump -i br0 | 23:51 |
| TheJulia | devananda: binds to specific interface, can operate via config if the macs are known via static mappings, otherwise does default range on the configured interface | 23:51 |
| hogepodge | devananda: I installed bifrost directly on a node. living large! | 23:52 |
| devananda | hogepodge: ah! I see | 23:52 |
| TheJulia | hogepodge: is br0 what has the ip address for that network bound to it? | 23:52 |
| devananda | TheJulia: that's what I thought, but I didn't recall the config name that changes it between static mappings / default range. | 23:52 |
| hogepodge | TheJulia: that could be it, where is that configured? | 23:52 |
| hogepodge | TheJulia: yeah | 23:52 |
| TheJulia | swapping that into brain now | 23:53 |
| TheJulia | inventory_dhcp and inventory_dhcp_static_ip | 23:53 |
| devananda | hogepodge: if bifrost is configured to do static dhcp mappings and it doesn't know the MAC of your NUC, that would explain the lack of response. check your dnsmasq config files. | 23:53 |
| TheJulia | as boolean settings upon install | 23:53 |
| TheJulia | The other possibility, is if it is not configured at all to do that, or was not installed that way... you dnsmasq could be doing something funky because it is a bridge interface, but that seems kind of remote in my mind | 23:55 |
| hogepodge | TheJulia: that was it, dnsmasq was serving on the wrong range | 23:55 |
| TheJulia | Well, that will do it :) | 23:56 |
| *** watanabe_isao has quit IRC | 23:56 | |
| openstackgerrit | Merged openstack/ironic: Add __ne__() function for API Version object https://review.openstack.org/365491 | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
