| openstackgerrit | Jesse J. Cook proposed openstack/ironic-specs: Update rescue spec https://review.openstack.org/407236 | 00:11 |
|---|---|---|
| *** [1]cdearborn has quit IRC | 00:21 | |
| *** penick has quit IRC | 00:31 | |
| openstackgerrit | Chris Krelle proposed openstack/bifrost: Adding staging driver support https://review.openstack.org/406401 | 00:31 |
| *** rajinir has quit IRC | 00:36 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 00:37 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Correct inspector keystone endpoint name https://review.openstack.org/407245 | 00:40 |
| *** Nisha_Agarwal has quit IRC | 00:41 | |
| *** soliosg has left #openstack-ironic | 00:49 | |
| *** SerenaFeng has joined #openstack-ironic | 00:55 | |
| *** penick has joined #openstack-ironic | 00:56 | |
| *** penick has quit IRC | 00:59 | |
| *** srobert has joined #openstack-ironic | 00:59 | |
| *** rpioso has quit IRC | 01:00 | |
| *** penick has joined #openstack-ironic | 01:01 | |
| *** rcernin has quit IRC | 01:01 | |
| *** fragatin_ has quit IRC | 01:02 | |
| *** fragatina has joined #openstack-ironic | 01:03 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM Testing tempest "smoke" 2 https://review.openstack.org/407258 | 01:03 |
| *** rama_y has quit IRC | 01:04 | |
| *** srobert has quit IRC | 01:08 | |
| openstackgerrit | Bob Fournier proposed openstack/ironic-inspector: Add plugin to process basic management LLDP TLVs https://review.openstack.org/406496 | 01:09 |
| *** srobert has joined #openstack-ironic | 01:09 | |
| *** Syed__ has quit IRC | 01:18 | |
| *** hw_wutianwei has joined #openstack-ironic | 01:24 | |
| *** zhangjl has joined #openstack-ironic | 01:33 | |
| *** shuquan has joined #openstack-ironic | 01:40 | |
| *** shuquan has quit IRC | 01:49 | |
| *** shuquan_ has joined #openstack-ironic | 01:50 | |
| *** penick has quit IRC | 01:59 | |
| *** yufei has joined #openstack-ironic | 02:01 | |
| *** shuquan_1 has joined #openstack-ironic | 02:01 | |
| *** shuquan_ has quit IRC | 02:03 | |
| openstackgerrit | Takanori Miyagishi proposed openstack/ironic: Remove period https://review.openstack.org/407281 | 02:11 |
| *** magic has joined #openstack-ironic | 02:12 | |
| *** SerenaFeng has quit IRC | 02:15 | |
| *** SerenaFeng has joined #openstack-ironic | 02:16 | |
| *** xiaoyang has quit IRC | 02:16 | |
| *** mtanino has quit IRC | 02:19 | |
| *** gcb has joined #openstack-ironic | 02:31 | |
| *** yuanying has quit IRC | 02:46 | |
| *** aNuposic has quit IRC | 02:48 | |
| *** rbudden has quit IRC | 03:27 | |
| *** baoli has joined #openstack-ironic | 03:33 | |
| *** vikrant has joined #openstack-ironic | 03:33 | |
| *** aparnav has joined #openstack-ironic | 03:50 | |
| *** links has joined #openstack-ironic | 03:52 | |
| *** zhangjl has quit IRC | 03:59 | |
| *** zhangjl has joined #openstack-ironic | 04:00 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 04:00 | |
| *** baoli has quit IRC | 04:03 | |
| *** IridiumScaffold has joined #openstack-ironic | 04:05 | |
| *** srobert has quit IRC | 04:09 | |
| *** IridiumScaffold has quit IRC | 04:09 | |
| *** SerenaFeng has quit IRC | 04:09 | |
| *** Romanenko_K has joined #openstack-ironic | 04:18 | |
| *** mnaser has quit IRC | 04:25 | |
| *** greghaynes has quit IRC | 04:26 | |
| *** afazekas has quit IRC | 04:26 | |
| *** alex_xu has joined #openstack-ironic | 04:49 | |
| *** alex_xu has quit IRC | 04:49 | |
| *** alex_xu has joined #openstack-ironic | 04:51 | |
| *** shuquan_1 has quit IRC | 04:53 | |
| *** yuanying has joined #openstack-ironic | 04:56 | |
| *** devlaps has joined #openstack-ironic | 04:58 | |
| *** afazekas has joined #openstack-ironic | 04:58 | |
| *** greghaynes has joined #openstack-ironic | 05:04 | |
| *** mnaser has joined #openstack-ironic | 05:07 | |
| *** deray has joined #openstack-ironic | 05:11 | |
| *** yufei has quit IRC | 05:13 | |
| openstackgerrit | Yucai Li proposed openstack/ironic: Improve the "Ironic behind mod wsgi" documentation https://review.openstack.org/407327 | 05:13 |
| *** Syed__ has joined #openstack-ironic | 05:13 | |
| deray | hi Ironicers .. g'morning | 05:15 |
| deray | I have a question regarding Ironic DB.. Is Ironic DB HA capable? | 05:16 |
| *** anteaya has quit IRC | 05:16 | |
| *** mnaser has quit IRC | 05:18 | |
| *** misty-cloud has joined #openstack-ironic | 05:18 | |
| *** mat128 has quit IRC | 05:19 | |
| *** rook has quit IRC | 05:22 | |
| *** mat128 has joined #openstack-ironic | 05:23 | |
| *** yufei_ has quit IRC | 05:25 | |
| *** yuanying_ has joined #openstack-ironic | 05:26 | |
| *** rook has joined #openstack-ironic | 05:28 | |
| *** rook is now known as Guest27303 | 05:28 | |
| *** anteaya has joined #openstack-ironic | 05:29 | |
| *** yuanying has quit IRC | 05:29 | |
| *** mnaser has joined #openstack-ironic | 05:30 | |
| *** yufei_ has joined #openstack-ironic | 05:34 | |
| *** yuanying has joined #openstack-ironic | 05:36 | |
| *** yuanying_ has quit IRC | 05:40 | |
| *** shuquan_ has joined #openstack-ironic | 05:40 | |
| *** SerenaFeng has joined #openstack-ironic | 05:41 | |
| *** yuanying_ has joined #openstack-ironic | 05:41 | |
| *** shuquan_ has quit IRC | 05:41 | |
| *** yuanying has quit IRC | 05:44 | |
| *** penick has joined #openstack-ironic | 05:50 | |
| *** e0ne has joined #openstack-ironic | 05:56 | |
| openstackgerrit | paresh sao proposed openstack/proliantutils: Support to disable iscsi boot option in iLO https://review.openstack.org/406706 | 06:02 |
| *** rcernin has joined #openstack-ironic | 06:34 | |
| openstackgerrit | paresh sao proposed openstack/proliantutils: Support to disable iscsi boot option in iLO https://review.openstack.org/406706 | 06:52 |
| *** rbartal has joined #openstack-ironic | 06:55 | |
| *** fxpester has joined #openstack-ironic | 06:59 | |
| *** zackf has quit IRC | 07:02 | |
| *** SerenaFeng has quit IRC | 07:02 | |
| *** e0ne has quit IRC | 07:04 | |
| *** SerenaFeng has joined #openstack-ironic | 07:06 | |
| *** mjura has joined #openstack-ironic | 07:06 | |
| *** ChubYann has quit IRC | 07:06 | |
| *** e0ne has joined #openstack-ironic | 07:11 | |
| *** rcernin has quit IRC | 07:12 | |
| *** yuanying_ has quit IRC | 07:13 | |
| *** e0ne has quit IRC | 07:13 | |
| *** mjura has quit IRC | 07:13 | |
| *** mjura has joined #openstack-ironic | 07:14 | |
| *** g3ek has quit IRC | 07:20 | |
| *** g3ek has joined #openstack-ironic | 07:21 | |
| *** mat128 has quit IRC | 07:22 | |
| *** mat128 has joined #openstack-ironic | 07:23 | |
| *** Guest66666 has quit IRC | 07:27 | |
| *** Guest66666 has joined #openstack-ironic | 07:29 | |
| *** alexpilo_ has joined #openstack-ironic | 07:29 | |
| *** alexpilotti has quit IRC | 07:30 | |
| *** mnaser has quit IRC | 07:30 | |
| *** rcernin has joined #openstack-ironic | 07:34 | |
| *** deray has quit IRC | 07:38 | |
| *** jtomasek has joined #openstack-ironic | 07:40 | |
| *** pcaruana has joined #openstack-ironic | 07:42 | |
| *** SerenaFeng has quit IRC | 07:46 | |
| *** mnaser has joined #openstack-ironic | 07:50 | |
| *** athomas has joined #openstack-ironic | 07:51 | |
| *** tuanluong has joined #openstack-ironic | 07:51 | |
| *** moshele has joined #openstack-ironic | 07:51 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic power interface for soft reboot and soft power off https://review.openstack.org/216730 | 07:53 |
| *** fragatina has quit IRC | 07:53 | |
| *** fragatina has joined #openstack-ironic | 07:54 | |
| *** SerenaFeng has joined #openstack-ironic | 07:55 | |
| *** d0ugal has joined #openstack-ironic | 07:57 | |
| *** d0ugal has quit IRC | 07:57 | |
| *** d0ugal has joined #openstack-ironic | 07:57 | |
| *** fragatina has quit IRC | 07:59 | |
| *** yuanying has joined #openstack-ironic | 08:01 | |
| *** SerenaFeng has quit IRC | 08:03 | |
| *** SerenaFeng has joined #openstack-ironic | 08:03 | |
| *** Syed__ has quit IRC | 08:07 | |
| *** mnaser has quit IRC | 08:15 | |
| *** ralonsoh has joined #openstack-ironic | 08:18 | |
| *** tuanluong has quit IRC | 08:29 | |
| *** mnaser has joined #openstack-ironic | 08:32 | |
| *** ohamada has joined #openstack-ironic | 08:38 | |
| *** priteau has joined #openstack-ironic | 08:42 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-python-agent: Use oslo-config-generator for sample config https://review.openstack.org/398989 | 08:42 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-python-agent: Configure and use SSL-related requests options https://review.openstack.org/398992 | 08:43 |
| pas-ha | morning Ironic :) | 08:44 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-python-agent: Use oslo-config-generator for sample config https://review.openstack.org/398989 | 08:56 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-python-agent: Configure and use SSL-related requests options https://review.openstack.org/398992 | 08:56 |
| *** tuanluong has joined #openstack-ironic | 08:57 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #openstack-ironic | 09:00 | |
| *** roeyc has joined #openstack-ironic | 09:02 | |
| *** zackf has joined #openstack-ironic | 09:14 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Fix takeover for AgentDeploy and PXEBoot https://review.openstack.org/404858 | 09:17 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Implement Driver API for dynamic boot config https://review.openstack.org/394398 | 09:17 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Allow not writing pxe configs to disk https://review.openstack.org/394397 | 09:17 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Convert iPXE boot config to proper Jinja template https://review.openstack.org/393462 | 09:17 |
| *** zackf has quit IRC | 09:19 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Serve boot configs from Ironic API https://review.openstack.org/369438 | 09:20 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Add RPC API to get the boot config https://review.openstack.org/394399 | 09:20 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: DNM test enabled dynamic iPXE https://review.openstack.org/391943 | 09:20 |
| *** strigazi_AFK is now known as strigazi | 09:28 | |
| *** daemontool has joined #openstack-ironic | 09:35 | |
| *** lucas-afk is now known as lucasagomes | 09:35 | |
| lucasagomes | morning all | 09:35 |
| *** derekh has joined #openstack-ironic | 09:40 | |
| *** misty-cloud has quit IRC | 09:46 | |
| *** misty-cloud has joined #openstack-ironic | 09:47 | |
| *** jpich has joined #openstack-ironic | 09:50 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: [WIP] Make _get_sensors_data concurrent https://review.openstack.org/407429 | 09:51 |
| *** SerenaFeng has quit IRC | 09:53 | |
| *** SerenaFeng has joined #openstack-ironic | 09:53 | |
| sergek | Morning Ironic | 09:58 |
| *** e0ne has joined #openstack-ironic | 09:58 | |
| lucasagomes | o/ | 10:03 |
| *** sborkows has joined #openstack-ironic | 10:05 | |
| *** abalfour has quit IRC | 10:08 | |
| *** abalfour has joined #openstack-ironic | 10:08 | |
| *** athomas has quit IRC | 10:09 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Catch unknown exceptions in validate driver ifaces https://review.openstack.org/406157 | 10:09 |
| *** SerenaFeng has quit IRC | 10:10 | |
| *** vikrant has quit IRC | 10:12 | |
| *** SerenaFeng has joined #openstack-ironic | 10:13 | |
| *** vikrant has joined #openstack-ironic | 10:13 | |
| *** sambetts|afk is now known as sambetts | 10:13 | |
| sambetts | jroll, mariojv: IMO unconfigure_tenant_networks shouldn't be removing the port from neutron, and I am changing that behaviour in my attach/detach patches because it is weird that unconfigure does the detach for you right now ...ewww | 10:15 |
| *** aweeks has quit IRC | 10:16 | |
| *** daemontool has quit IRC | 10:23 | |
| *** baoli has joined #openstack-ironic | 10:24 | |
| *** baoli has quit IRC | 10:28 | |
| mgould | morning Ironic! | 10:32 |
| mgould | morning lucasagomes sergek sambetts pas-ha strigazi galyna | 10:32 |
| *** athomas has joined #openstack-ironic | 10:35 | |
| fxpester | it`s just me who every time on `morning Ironic` start thinking a bit cruel things ? | 10:41 |
| *** ohamada has quit IRC | 10:44 | |
| *** ohamada has joined #openstack-ironic | 10:45 | |
| sergek | mgould: \o/ | 10:45 |
| *** e0ne has quit IRC | 10:45 | |
| lucasagomes | fxpester, heh I think so | 10:47 |
| lucasagomes | in Ironicland, any time of the day is morning heh | 10:47 |
| mgould | fxpester: "I wish all these cheery people would shut up", "it isn't morning here", or something else? | 10:48 |
| *** jkilpatr has quit IRC | 10:56 | |
| *** jkilpatr has joined #openstack-ironic | 10:57 | |
| fxpester | like setting up a bot to greet every timezone and every reconnecting person ^^ and asking random questions like `what error you got today ?`, `how long it will take to fix your code ?` and my lovely `why my node didn`t boot by pxe ?` | 10:59 |
| *** dtantsur|afk is now known as dtantsur | 11:01 | |
| dtantsur | Morning Ironic! | 11:01 |
| * dtantsur is a victim of an internet outage affecting half of Brno | 11:01 | |
| sambetts | :( | 11:02 |
| *** e0ne has joined #openstack-ironic | 11:03 | |
| sborkows | morning mgould, dtantsur | 11:04 |
| fxpester | morning all, yesterday I was building rhel7.2 image, and cause dib produces image with failed grub.conf it was unbootable, so I was fixing it by hands... but later I got need to make this image with 'stable interface names' I installed dib element with that name and... it didn`t work! after investigation I found a record in grub.conf that prevented it from working | 11:09 |
| fxpester | hope none of us will meet 'bug chains' at least for today ^^ | 11:10 |
| *** zhangjl has quit IRC | 11:11 | |
| *** daemontool has joined #openstack-ironic | 11:11 | |
| *** e0ne has quit IRC | 11:15 | |
| *** hshiina has joined #openstack-ironic | 11:15 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Catch unknown exceptions in validate driver ifaces https://review.openstack.org/406157 | 11:15 |
| openstackgerrit | Merged openstack/ironic: Add virtualpdu to ironic devstack plugin https://review.openstack.org/388154 | 11:16 |
| *** misty-cloud has quit IRC | 11:21 | |
| jroll | quiet morning y'all | 11:30 |
| charlledaniel | morning sergek , mgould, dtantsur, fxpester, jroll and all | 11:32 |
| sergek | o/ | 11:33 |
| dtantsur | morning jroll, charlledaniel, sborkows, sergek and everyone | 11:34 |
| jroll | heads up, all jobs going to xenial today | 11:34 |
| *** alexpilo_ has quit IRC | 11:36 | |
| *** alexpilotti has joined #openstack-ironic | 11:37 | |
| sambetts | \o/ yay | 11:40 |
| *** alexpilotti has quit IRC | 11:41 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru from OneView drivers https://review.openstack.org/397846 | 11:42 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove iBoot, WoL and AMT drivers https://review.openstack.org/397847 | 11:42 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru from iLO drvers https://review.openstack.org/397845 | 11:42 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru completely https://review.openstack.org/397848 | 11:42 |
| *** e0ne has joined #openstack-ironic | 11:43 | |
| *** zackf has joined #openstack-ironic | 11:46 | |
| *** alexpilotti has joined #openstack-ironic | 11:47 | |
| jroll | sambetts: I agree with you about unconfigure_tenant_networks, btw, we were chatting about it in terms of rescue and plan to make that two methods: unconfigure will disconnect the thing from the node, and another destroy or something removes the port | 11:48 |
| sambetts | jroll: nothing in Ironic should destroy the ports | 11:49 |
| sambetts | jroll: right now I class this as a bug | 11:49 |
| sambetts | jroll: unconfigure should disconnect the port which is what I've fixed in my attach/detach patch, detach should remove the configuration, and whoever created the port is responible for deleting it | 11:51 |
| sambetts | jroll: whether that be nova or a tenant | 11:51 |
| sambetts | jroll: in nova if you pass a --nic port-id= nova doesn't delete the port | 11:51 |
| jroll | sambetts: yeah, I agree | 11:51 |
| *** zackf has quit IRC | 11:51 | |
| sambetts | just disconnects it, which we break by deleteing it | 11:51 |
| *** alexpilotti has quit IRC | 11:51 | |
| jroll | yep | 11:52 |
| *** jkilpatr has quit IRC | 11:53 | |
| TheJulia | Good morning | 11:59 |
| lucasagomes | TheJulia, sambetts jroll morning | 12:00 |
| sambetts | o/ TheJulia, lucasagomes | 12:00 |
| lucasagomes | TheJulia, about boot from volume. At least the scenario #1 (with ipxe) it's possible to test on a full virtual environment right ? | 12:00 |
| TheJulia | lucasagomes: once the ipxe template generation code is hammered out, yes it should be possible, today I'm going to try and get the deploy noop logic in place | 12:02 |
| mgould | fxpester: the semi-standard name for that situation is "yak shaving": http://catb.org/jargon/html/Y/yak-shaving.html | 12:02 |
| mgould | also, sympathy :-( | 12:02 |
| mgould | relatedly, your "yak trace" is the chain of nested problems you're solving | 12:02 |
| mgould | occasionally, your yak trace will go circular: http://www.songsforteaching.com/folk/theresaholeinthebucket.php | 12:03 |
| * mgould was very disappointed not to meet any yaks in Kyrgyzstan | 12:04 | |
| TheJulia | heh | 12:04 |
| dtantsur | I wonder if nova expects a correct local_gb when the root disk is from cinder | 12:04 |
| dtantsur | and how to supply it.. | 12:04 |
| * dtantsur thinks about introspection of diskless nodes | 12:05 | |
| TheJulia | ohh... that is a good question regarding local_gb... i hadn't thought of that | 12:05 |
| mgould | we did meet some camels, though: https://www.dropbox.com/s/boxkt6r81gyfdoh/2016-08-16%2019.36.07.jpg?dl=0 | 12:05 |
| TheJulia | at least, thought in recent memory. | 12:05 |
| dtantsur | mgould, cool! | 12:05 |
| TheJulia | dtantsur: IPA afaik just needs to not halt if no disks are found | 12:06 |
| dtantsur | TheJulia, does it now? | 12:06 |
| TheJulia | It did last year if memory serves | 12:06 |
| dtantsur | TheJulia, I think ironic-inspector does fail, but not IPA | 12:07 |
| *** e0ne has quit IRC | 12:07 | |
| tuanluong | Morning ironic | 12:08 |
| TheJulia | dtantsur: hmm :( | 12:08 |
| *** tuanluong has quit IRC | 12:08 | |
| *** e0ne has joined #openstack-ironic | 12:09 | |
| mgould | morning tuanlong | 12:11 |
| dtantsur | TheJulia, does my proposal on https://bugs.launchpad.net/ironic-inspector/+bug/1554243 make sense? | 12:11 |
| openstack | Launchpad bug 1554243 in Ironic Inspector "[RFE] Support diskless nodes once Ironic gets such feature" [Wishlist,Triaged] | 12:11 |
| dtantsur | (see description) | 12:11 |
| TheJulia | makes perfect sense, I think the only thing that would also make sense is to set local_gb to - | 12:12 |
| TheJulia | err, 0 | 12:12 |
| dtantsur | this will probably prevent nova from scheduling on it.. but so will missing value, so yeah | 12:13 |
| TheJulia | we would need to check the scheduling code, because I think 0 would make sense to support, but that is just me | 12:13 |
| TheJulia | need to step away, bbiab | 12:13 |
| lucasagomes | TheJulia, gotcha! | 12:14 |
| sambetts | dtantsur, TheJulia: the nova scheduling problem will go away with resource classes | 12:14 |
| sambetts | because nova won;t be looking for disk_gb any more | 12:14 |
| dtantsur | jroll, sambetts, please review the resulting RFE: https://bugs.launchpad.net/ironic/+bug/1554243 | 12:14 |
| openstack | Launchpad bug 1554243 in Ironic Inspector "[RFE] In-band introspection of diskless nodes" [Wishlist,Triaged] | 12:14 |
| jroll | +1'd | 12:15 |
| jroll | meanwhile I am lost in a world of tempest not registering the options I think it is registering :( | 12:15 |
| *** milan has joined #openstack-ironic | 12:16 | |
| dtantsur | sigh.. | 12:16 |
| milan | morning Ironic! :) | 12:16 |
| dtantsur | milan, https://bugs.launchpad.net/ironic/+bug/1554243 and morning :) | 12:16 |
| openstack | Launchpad bug 1554243 in Ironic Inspector "[RFE] In-band introspection of diskless nodes" [Wishlist,Triaged] | 12:16 |
| jroll | hahaha | 12:16 |
| jroll | ughhh | 12:16 |
| milan | morning dtantsur :) | 12:16 |
| * milan looks | 12:16 | |
| lucasagomes | TheJulia, yeah I think IPA guess_root_device() method will raise an exception if no disk is found | 12:16 |
| jroll | do we not enable our tempest plugin on the old side of grenade? | 12:18 |
| dtantsur | I thought we do | 12:18 |
| sambetts | jroll: I'm not sure we do | 12:19 |
| sambetts | jroll: I think we run tempest smoke, not sure that uses our plugin | 12:19 |
| sambetts | jlvillal, or vdrok might know more though | 12:19 |
| jroll | right, I know that much, but not sure if it's enabled | 12:19 |
| jroll | it doesn't seem to be | 12:19 |
| jroll | :| | 12:19 |
| milan | dtantsur, I can babysit the bug on the inspector side since the states patch is sort of stale ^^^^^ | 12:20 |
| * jroll sighs | 12:20 | |
| mgould | morning jroll | 12:21 |
| dtantsur | milan, thanks! | 12:21 |
| milan | ....which reminds me of: mornig aarefiev, sambetts, dtantsur! mind reviewing https://review.openstack.org/#/c/348943/ ? ;) | 12:21 |
| patchbot | patch 348943 - ironic-inspector - Introducing node introspection state management | 12:21 |
| dtantsur | jroll, provided that we got buy-in from you, TheJulia and milan, do you think you can mark it rfe-approved? | 12:21 |
| sambetts | dtantsur: I'm +1 too for now, seems like the simplest approach, the alternative I was thinking of was making the thing that checks for disks or not an introspection rule which could be changed by the operator if their environment has diskless nodes | 12:23 |
| dtantsur | we can do both eventually. now I'm more worried about inspector compatibility with the BFV work | 12:24 |
| sambetts | dtantsur: how do you see this API paramter working for autodiscovery of diskless nodes, they will still fail right? | 12:24 |
| milan | :-/ | 12:25 |
| dtantsur | yep. this is probably why we should do both eventually :) | 12:25 |
| sambetts | dtantsur: and need to be somehow detected and reintrospected with the flag | 12:25 |
| dtantsur | essentially, "diskless" flag will be kept in NodeInfo.options | 12:25 |
| dtantsur | so, provided that our rules can modify options, this RFE actually enabled it | 12:26 |
| dtantsur | but I'm not sure how early we run rules | 12:26 |
| sambetts | last | 12:26 |
| patchbot | (last <text> [<text> ...]) -- Returns the last argument given. Useful when you'd like multiple nested commands to run, but only the output of the last one to be returned. | 12:26 |
| dtantsur | WUT? | 12:26 |
| sambetts | -_- | 12:26 |
| sambetts | patch bot... | 12:26 |
| milan | last sambetts | 12:26 |
| patchbot | sambetts | 12:26 |
| milan | :D | 12:27 |
| sambetts | -_- ffs... | 12:27 |
| milan | last sambetts dtantsur jroll | 12:27 |
| patchbot | jroll | 12:27 |
| milan | lol | 12:27 |
| dtantsur | I wonder if it knows more commands.. | 12:27 |
| dtantsur | patchbot: help | 12:27 |
| patchbot | (help <name>) -- Returns the description of the configuration variable <name>. | 12:27 |
| dtantsur | help | 12:27 |
| patchbot | (help <name>) -- Returns the description of the configuration variable <name>. | 12:27 |
| dtantsur | not so helpful | 12:27 |
| sambetts | lol | 12:27 |
| * dtantsur -> food | 12:27 | |
| *** SerenaFeng has quit IRC | 12:27 | |
| openstackgerrit | paresh sao proposed openstack/proliantutils: Adds 'UefiTarget' as a boot option https://review.openstack.org/407487 | 12:28 |
| *** athomas has quit IRC | 12:29 | |
| * mgould googles for "openstack patchbot commands" | 12:29 | |
| mgould | of *course* it's configured via a gigantic YAML file: https://git.openstack.org/cgit/openstack-infra/project-config/tree/gerritbot/channels.yaml | 12:29 |
| sambetts | lol | 12:29 |
| jroll | that isn't patchbot | 12:29 |
| jroll | patchbot is notmyname's thing | 12:29 |
| mgould | ah, OK | 12:30 |
| jroll | sambetts: jlvillal: tell me what I'm breaking :) https://review.openstack.org/407488 E | 12:30 |
| patchbot | patch 407488 - openstack-infra/project-config - Enable ironic tempest plugin in grenade job | 12:30 |
| jroll | ignore the E I guess | 12:30 |
| *** jkilpatr has joined #openstack-ironic | 12:30 | |
| *** lucasagomes is now known as lucas-hungry | 12:30 | |
| * mgould gives up on trying to find documentation for patchbot | 12:31 | |
| jroll | mgould: not docs but 'read the code' is typical around here anyway https://github.com/notmyname/Patches | 12:31 |
| mgould | jroll: assuming you can find the name of the repo :-) | 12:32 |
| * jroll did :P | 12:32 | |
| * mgould foolishly assumed it would be "patchbot" | 12:32 | |
| sambetts | jroll: so I'm a little confused what we're trying to do here, CONF.serivce_available.ironic has moved to our tempest plugins conf.py, is CONF.service_available.ironic used outside of our tempest plugin? | 12:35 |
| mgould | patchbot: list | 12:35 |
| patchbot | Error: The command "list" is available in the Config and User plugins. Please specify the plugin whose command you wish to call by using its name as a command before "list". | 12:35 |
| jroll | sambetts: yes, specifically line 813 here https://review.openstack.org/#/c/355586/22/tempest/scenario/manager.py | 12:35 |
| patchbot | patch 355586 - tempest - Removing baremetal tests from tempest tree | 12:35 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Add Virtual Network Interface REST APIs https://review.openstack.org/404240 | 12:36 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Add Virtual Network Interface RPC APIs https://review.openstack.org/404239 | 12:36 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Add Virtual Network Interface Driver APIs https://review.openstack.org/327046 | 12:36 |
| mgould | patchbot: Patches list | 12:36 |
| jroll | can we stop spamming patchbot junk in this channel please, PM the bot if needed | 12:36 |
| mgould | sorry | 12:36 |
| jroll | np | 12:36 |
| jroll | sambetts: or, well, line 813 is where it's breaking, there's a couple places it's still referencing | 12:37 |
| sambetts | jroll: hmmm... if its used outside of our plugin I'm not sure it should be in our plugin then right? because that means you have to install the Ironic plugin into every tempest install | 12:37 |
| jroll | sambetts: it uses getattr so won't break | 12:38 |
| jroll | sambetts: I agreed with qa folks to leave it there in a few places for now, one is a workaround for a bug, another is the nova baremetal compatibility extension, etc | 12:38 |
| jroll | but this gets 99% of the work done | 12:38 |
| *** aparnav has quit IRC | 12:38 | |
| jroll | work/move/whatever | 12:38 |
| sambetts | hmm, seems like a bit of a hack but ok :/ | 12:38 |
| jroll | tempest is a bit of a hack :) | 12:39 |
| sambetts | lol | 12:39 |
| jroll | it's mostly transitional code while we figure out how to deal with it | 12:39 |
| jroll | e.g. the one I pointed you at says: | 12:39 |
| jroll | # TODO(vsaienko) remove once bug: #1599836 is resolved. | 12:39 |
| openstack | bug 1599836 in OpenStack Compute (nova) "Booting Ironic instance, neutron port remains in DOWN state" [High,In progress] https://launchpad.net/bugs/1599836 - Assigned to Vasyl Saienko (vsaienko) | 12:39 |
| jroll | if getattr(CONF.service_available, 'ironic', False): | 12:39 |
| sambetts | so we basically need to enable our tempest plugin only to use the CONF variable during tempest smoke, not to run our own scenario tests | 12:40 |
| jroll | we didn't see the need to invent a new CONF.network.port_status_doesnt_actually_work there | 12:40 |
| jroll | sambetts: correct, and since our tests aren't tagged smoke, they won't run here | 12:40 |
| jroll | 'tagged' meaning @test.attr(type='smoke') | 12:41 |
| sambetts | jroll: the changes you've made in that ironic.yaml look more or less correct to me | 12:41 |
| jroll | ok, cool | 12:41 |
| jroll | project-config changes always scare me :/ | 12:41 |
| sambetts | yeah, /me can't wait until zuul can test them in CI | 12:41 |
| jroll | yerp | 12:42 |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add RPC version pinning https://review.openstack.org/407491 | 12:45 |
| jroll | sambetts: mind a +1 on that if it looks good? | 12:48 |
| sambetts | jroll: yeah, just taking a deeper look through the code to make sure, weirdly it seems like one of our grenade jobs specifies everything manually, and one of them built a custom builder to enable grenade on top on devstack-virtual-ironic... | 12:49 |
| jroll | sambetts: yeah, I'm not sure what's going on there | 12:49 |
| sambetts | your custom builder won't be required if they built both/all grenade jobs ontop of devstack-virtual-ironic, but not sure what the arch decision was there | 12:50 |
| jroll | yeah, probably cruft but I didn't want to do too much here | 12:51 |
| sambetts | yeah most likely, anyway the code you've written will certainly work based on the way the jobs are now, we'll probably have to do another clean up run of ironic.yaml when things are more settled | 12:51 |
| *** Goneri has joined #openstack-ironic | 12:54 | |
| jroll | nod | 12:54 |
| jroll | thanks | 12:54 |
| jroll | fwiw project-config is frozen until the xenial move is done :/ | 12:54 |
| sambetts | ah ok | 12:54 |
| *** bfournie has quit IRC | 13:00 | |
| *** Guest27303 is now known as rook | 13:02 | |
| *** vikrant has quit IRC | 13:03 | |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add RPC version pinning https://review.openstack.org/407491 | 13:03 |
| *** trown|outtypewww is now known as trown | 13:03 | |
| *** jheroux has joined #openstack-ironic | 13:11 | |
| *** fragatina has joined #openstack-ironic | 13:15 | |
| *** fragatin_ has joined #openstack-ironic | 13:15 | |
| *** rbudden has joined #openstack-ironic | 13:19 | |
| *** fragatina has quit IRC | 13:19 | |
| *** hamzy has quit IRC | 13:26 | |
| *** pmannidi has quit IRC | 13:27 | |
| *** srobert has joined #openstack-ironic | 13:27 | |
| *** srobert has quit IRC | 13:27 | |
| *** srobert has joined #openstack-ironic | 13:28 | |
| openstackgerrit | Merged openstack/ironic: Add volume_connectors and volume_targets to task https://review.openstack.org/366196 | 13:30 |
| *** bfournie has joined #openstack-ironic | 13:34 | |
| openstackgerrit | Moshe Levi proposed openstack/ironic-python-agent: Fix docsting in Mellanox _detect_hardware https://review.openstack.org/407510 | 13:50 |
| *** lucas-hungry is now known as lucasagomes | 13:50 | |
| *** fragatin_ has quit IRC | 13:52 | |
| *** fragatina has joined #openstack-ironic | 13:52 | |
| *** glonlas has joined #openstack-ironic | 13:55 | |
| openstackgerrit | Sam Betts proposed openstack/ironic: Rework Ironic devstack baremetal network simulation https://review.openstack.org/392959 | 13:56 |
| *** baoli has joined #openstack-ironic | 13:56 | |
| *** baoli has quit IRC | 13:58 | |
| *** baoli has joined #openstack-ironic | 13:58 | |
| *** baoli has quit IRC | 13:59 | |
| *** baoli has joined #openstack-ironic | 13:59 | |
| *** daemontool has quit IRC | 14:04 | |
| *** srobert has quit IRC | 14:08 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost: Add support for dnf as package manager https://review.openstack.org/407524 | 14:08 |
| *** rbartal has quit IRC | 14:08 | |
| *** rloo has joined #openstack-ironic | 14:09 | |
| sambetts | jroll: there seems to be a lot of junk printed out at the beginning of running the unit tests at the moment, any idea why? | 14:10 |
| jroll | sambetts: haven't seen that, link? | 14:10 |
| sambetts | jroll: http://logs.openstack.org/40/404240/3/check/gate-ironic-python27-db-ubuntu-xenial/6cf2b37/console.html | 14:11 |
| sambetts | jroll: literally like a thousand deprecation warnings | 14:11 |
| jroll | whaaaa | 14:12 |
| jroll | also some unreadable stuff, looks like it's stomping on itself | 14:12 |
| sambetts | yeah | 14:12 |
| *** lindycoder has joined #openstack-ironic | 14:13 | |
| rloo | hi and morning ironic'ers, sambetts, jroll. | 14:13 |
| jroll | hey rloo :) | 14:13 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Add Virtual Network Interface REST APIs https://review.openstack.org/404240 | 14:14 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Add Virtual Network Interface RPC APIs https://review.openstack.org/404239 | 14:14 |
| sambetts | hey rloo | 14:14 |
| rloo | is that the policy stuff, something about circular reference? | 14:14 |
| sambetts | rloo: in my test output? | 14:14 |
| rloo | sambetts: dunno, didn't look. but i recall seeing stuff about policy when running unit tests :) | 14:15 |
| sambetts | rloo: oh right, I thought you were looking at the test output from my recent update, I got a circular reference error which I think was caused by an error on my part which I've just fixed | 14:16 |
| sambetts | rloo: I have seen warnings about policy references appear before though | 14:16 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Support defining and loading hardware types https://review.openstack.org/336626 | 14:17 |
| *** zackf has joined #openstack-ironic | 14:18 | |
| *** [1]cdearborn has joined #openstack-ironic | 14:19 | |
| *** rbartal has joined #openstack-ironic | 14:19 | |
| jlvillal | sambetts, jroll: On the deprecation warnings. Might be because of this: https://github.com/openstack/ironic/commit/516f4341ff547fe251c75baf253d2e4074b401df | 14:19 |
| jroll | jlvillal: okay, that explains the new-ness, now we need to fix the actual warnings :) | 14:21 |
| sambetts | jlvillal: oh yeah, I've seen that change in other projects too, jroll++ | 14:21 |
| *** devlaps has quit IRC | 14:21 | |
| *** dprince has joined #openstack-ironic | 14:21 | |
| jroll | I'm most concerned about e.g. DeprecationWarning: Using class 'DbTestCase' (either directly or via inheritance) is deprecated | 14:22 |
| sambetts | hmm yeah, that might affect a lot of our test code | 14:22 |
| jroll | don't see debtcollector in testtools nor ironic | 14:23 |
| jroll | so no clue where that's coming from | 14:23 |
| jroll | oh, unittest2 maybe | 14:24 |
| sambetts | yeah, which we can't really fix I guess | 14:24 |
| jroll | which hasn't been touched in 17 months | 14:24 |
| jroll | :| | 14:24 |
| jroll | don't see a deprecation there, either, wat | 14:25 |
| sambetts | https://bugs.launchpad.net/ironic/+bug/1647735 | 14:25 |
| openstack | Launchpad bug 1647735 in Ironic "DeprecationWarning: Using class 'DbTestCase' (either directly or via inheritance) is deprecated" [Undecided,New] | 14:25 |
| sambetts | https://bugs.launchpad.net/ironic/+bug/1647733 | 14:26 |
| openstack | Launchpad bug 1647733 in Ironic "DeprecationWarning: Using class 'MySQLOpportunisticTestCase' (either directly or via inheritance) is deprecated" [Undecided,New] | 14:26 |
| jroll | thanks | 14:26 |
| jroll | those are probably the same bug btw :) | 14:26 |
| sambetts | we use both directly so I didn't know whether one or two made more sense, there are some deprecation warnings that we don't use directly but are the parent of MySQLOpportunisticTestCase for example | 14:27 |
| jroll | yeah, fair | 14:28 |
| jroll | dropped a comment that they're related | 14:28 |
| jroll | er, may be related | 14:29 |
| sambetts | cool :) | 14:29 |
| *** hshiina has quit IRC | 14:30 | |
| sambetts | jroll: it looks like DbTestCase might have just been moved | 14:31 |
| jroll | oh? | 14:31 |
| sambetts | jroll: from oslo_db/sqlalchemy/test_base.py to oslo_db/tests/sqlalchemy/base.py | 14:31 |
| sambetts | jroll: https://github.com/openstack/oslo.db/search?utf8=%E2%9C%93&q=DbTestCase | 14:31 |
| jroll | sambetts: oh, I thought we always defined that ourselves | 14:32 |
| jroll | ah, we use that in test_migrations | 14:33 |
| jroll | nice | 14:33 |
| sambetts | ah, its the same for the Oppertuistic ones | 14:34 |
| sambetts | so we need to fix our migrations tests some how | 14:34 |
| sambetts | sigh... I wish when someone did a deprecation like this there was a guide on how to upgrade somewhere... | 14:39 |
| *** links has quit IRC | 14:40 | |
| jroll | yeah, release notes would be nice | 14:40 |
| jroll | should just be able to move it though | 14:40 |
| sambetts | yeah, just trying it out locally now | 14:41 |
| *** srobert has joined #openstack-ironic | 14:42 | |
| *** sborkows has quit IRC | 14:45 | |
| *** jcoufal has joined #openstack-ironic | 14:52 | |
| sambetts | jroll: any idea why we use the sqlalchemy legacy facade? I notice another deprecation warning regarding switching to oslo_db.sqlalchemy.enginefacade ? | 14:54 |
| mariojv | cool, i'll review rescue spec today jcook | 14:54 |
| jroll | sambetts: because we never noticed it was deprecated? :) | 14:54 |
| sambetts | jroll: haha, ok I'll play around with that too | 14:54 |
| jroll | <3 | 14:54 |
| mariojv | sambetts: great! (about fixing unconfigure_tenant_networks so it doesn't remove ports from neutron) | 14:54 |
| sambetts | jroll: I'll see if I can push a fix all oslo_db deprecation warnings patch | 14:55 |
| mariojv | sambetts: so, after your patches, will i be able to run unconfigure_tenant_networks(); configure_tenant_networks() | 14:55 |
| jroll | sambetts: sounds good | 14:55 |
| mariojv | sambetts: and have the networks be set up just fine again? | 14:55 |
| sambetts | mariojv: theory is that should work | 14:55 |
| mariojv | \o/ | 14:55 |
| mariojv | i'll mark your patch as a dependency of mine and give it a good review then | 14:56 |
| sambetts | mariojv: https://review.openstack.org/#/c/327046/28/ironic/drivers/modules/network/neutron.py | 14:56 |
| patchbot | patch 327046 - ironic - Add Virtual Network Interface Driver APIs | 14:56 |
| sambetts | mariojv: I've sorted the unconfigure function now | 14:56 |
| sambetts | take a look | 14:56 |
| mariojv | one thing jroll JayF and i were talking about is that we might still want something for the current behavior (removing ports from neutron) | 14:57 |
| mariojv | maybe a separate call like destroy_tenant_ports | 14:57 |
| mariojv | idk | 14:57 |
| mariojv | looking | 14:57 |
| sambetts | mariojv: IMO and I think jroll agrees we shouldn't be deleting ports created outside of Ironic | 14:57 |
| mariojv | ah, so nova (or external service/person) created them, let that external service destroy? | 14:58 |
| mariojv | i think i can get on board with that | 14:58 |
| sambetts | yup | 14:58 |
| sambetts | mariojv: right now if you (as a user) create a port manually, and pass it into nova boot, with a VM nova doesn't delete it, but right now Ironic nukes it | 14:58 |
| mariojv | sambetts: yeah, the neutron piece looks like it should work | 14:59 |
| mariojv | neutron driver i mean | 14:59 |
| mariojv | sambetts: ok - and that behavior is going to be pushed onto nova eventually? | 14:59 |
| *** glonlas_ has joined #openstack-ironic | 15:00 | |
| mariojv | sambetts: one interesting bit is the flat network driver. it doesn't really matter *that* much whether vif_port_id stays on the port imo | 15:00 |
| *** anush has joined #openstack-ironic | 15:00 | |
| mariojv | sambetts: is nova / external service also going to be responsible for removing vif_port_id? | 15:01 |
| sambetts | mariojv: unless someone attaches the port without going through nova, and then was like dafuq my vif go | 15:01 |
| sambetts | mariojv: yes, it already is | 15:01 |
| mariojv | sambetts: ok - great | 15:01 |
| mariojv | sambetts: taking a look at the rest of the patch today, too | 15:01 |
| mariojv | sambetts: thanks! | 15:01 |
| sambetts | :D any reviews will be well received | 15:02 |
| openstackgerrit | Mark Hamzy proposed openstack/ironic-specs: Add initial specification for MoltenIron https://review.openstack.org/387418 | 15:02 |
| *** glonlas has quit IRC | 15:03 | |
| *** glonlas has joined #openstack-ironic | 15:04 | |
| *** hamzy has joined #openstack-ironic | 15:04 | |
| *** glonlas_ has quit IRC | 15:05 | |
| *** penick has quit IRC | 15:07 | |
| *** mjura_ has joined #openstack-ironic | 15:13 | |
| *** Syed__ has joined #openstack-ironic | 15:14 | |
| *** mjura has quit IRC | 15:15 | |
| *** glonlas has quit IRC | 15:18 | |
| *** tuan_luong has joined #openstack-ironic | 15:19 | |
| tuan_luong | Hello Ironic | 15:20 |
| *** mjura_ has quit IRC | 15:21 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Update help for 'provisioning_network' option https://review.openstack.org/407563 | 15:22 |
| *** glonlas has joined #openstack-ironic | 15:24 | |
| *** srobert has quit IRC | 15:26 | |
| *** srobert has joined #openstack-ironic | 15:26 | |
| *** rama_y has joined #openstack-ironic | 15:27 | |
| *** glonlas_ has joined #openstack-ironic | 15:28 | |
| *** glonlas has quit IRC | 15:31 | |
| *** cdearborn has joined #openstack-ironic | 15:31 | |
| *** [1]cdearborn has quit IRC | 15:32 | |
| *** mjura_ has joined #openstack-ironic | 15:33 | |
| *** glonlas_ has quit IRC | 15:34 | |
| *** glonlas has joined #openstack-ironic | 15:34 | |
| *** srobert has quit IRC | 15:36 | |
| *** anush has quit IRC | 15:37 | |
| *** srobert has joined #openstack-ironic | 15:37 | |
| lucasagomes | tuan_luong, morning | 15:37 |
| *** anush has joined #openstack-ironic | 15:38 | |
| *** srobert_ has joined #openstack-ironic | 15:38 | |
| *** srobert_ has quit IRC | 15:38 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add base cinder common interface https://review.openstack.org/355625 | 15:39 |
| *** srobert_ has joined #openstack-ironic | 15:39 | |
| *** srobert has quit IRC | 15:42 | |
| *** soliosg has joined #openstack-ironic | 15:42 | |
| *** misty-cloud has joined #openstack-ironic | 15:42 | |
| sambetts | TheJulia: QQ, does bifrost setup DHCP/DNSmasq for me? | 15:42 |
| TheJulia | sambetts: It does, but you can use your own if you so desire | 15:43 |
| *** misty-cloud has quit IRC | 15:43 | |
| *** rama_y has quit IRC | 15:43 | |
| sambetts | TheJulia: does it do it by default, and if it does should we see a separate dnsmasq instance running or do you use any existing ones? | 15:44 |
| sambetts | TheJulia: BTW we got passed the systemd issue, we reinstalled the server and it just started working magically... | 15:44 |
| TheJulia | sambetts: It does by default, it doesn't behave with multiple instances, the assumption is that the box is unused otherwise w/r/t dnsmasq | 15:45 |
| TheJulia | sambetts: That... is... freaky. | 15:45 |
| bradjones | yeah I'll try another install in the week doing *exactly* as I did before see if it's some weird edge case | 15:45 |
| bradjones | otherwise it seems it was just a broken install | 15:45 |
| *** vgadiraj has joined #openstack-ironic | 15:46 | |
| jlvillal | sambetts: How much of this code should stay? https://github.com/openstack-infra/devstack-gate/blob/master/devstack-vm-gate.sh#L396-L422 | 15:46 |
| jlvillal | Seems like a lot of that is done by our project-config templates. | 15:47 |
| jlvillal | Nothing urgent, but I noticed it when looking at a generated localrc from a gate job. | 15:47 |
| *** rbartal_ has joined #openstack-ironic | 15:49 | |
| sambetts | jlvillal: yeah, I personally dislike devstack-gate stuff, I think its just results in duplication, I would prefer we support whatever we can in our devstack plugin and set the options via the jobs | 15:49 |
| jlvillal | +1 | 15:49 |
| sambetts | TheJulia: does bifrost support non-ipxe pxe boot out of the box? or only servers with native ipxe? | 15:52 |
| TheJulia | sambetts: it supports pxe bootstrapping ipxe. It also has logic in in the dnsmasq config to bootstrap gpxe firmware too | 15:52 |
| tuan_luong | morning lucasagomes, sambetts, jlvillal TheJulia | 15:52 |
| *** fragatina has quit IRC | 15:53 | |
| TheJulia | sambetts: tl;dr native ipxe not required at all :) | 15:53 |
| sambetts | TheJulia: hmm ok, we're seeing PXE requests come into the server but its not responding to them, just trying to debug why, we're trying out inspection btw | 15:53 |
| TheJulia | sambetts: some people have had some weird issues with dnsmasq, but it expects the intended use interface to be defined upon install, which it binds dnsmasq to | 15:54 |
| sambetts | TheJulia: we're using a physical interface so it existed before we installed bifrost/started dnsmasq | 15:55 |
| TheJulia | sambetts: but did you tell the install playbook about those interfaces? | 15:55 |
| TheJulia | well, interface, it doesn't have a concept of multiple interfaces. Perhaps it should... | 15:56 |
| sambetts | Yeah, we updated the network_interface to point at the physical one, and the dnsmasq configuration has that interface in it | 15:56 |
| TheJulia | does the network addressing on the interface match dnsmasq's? | 15:57 |
| sambetts | TheJulia: the dhcp-range=<range> doesn't no | 15:57 |
| TheJulia | sambetts: dnsmasq won't respond if the range does not exist in the interface's address space | 15:58 |
| *** mtanino has joined #openstack-ironic | 15:58 | |
| TheJulia | :( | 15:58 |
| sambetts | TheJulia: ah ok, it seems to have configured a 192.168.200.* range even though our nic has a whole 172.x.x.x/20 range assigned to it | 15:59 |
| TheJulia | sambetts: yeah, that is likely someplace bifrost can be improved since we don't try to figure that out, nor do we check to see if the local config doesn't match with what the user has supplied. | 16:00 |
| *** rama_y has joined #openstack-ironic | 16:01 | |
| *** misty-cloud has joined #openstack-ironic | 16:02 | |
| mariojv | jcook: update to rescue spec (https://review.openstack.org/#/c/407236) lgtm, thanks for that | 16:04 |
| patchbot | patch 407236 - ironic-specs - Update rescue spec | 16:04 |
| *** rama_y has quit IRC | 16:04 | |
| *** HenryG has left #openstack-ironic | 16:05 | |
| *** devlaps has joined #openstack-ironic | 16:07 | |
| *** fxpester has quit IRC | 16:09 | |
| jcook | mariojv what is "this": IDK if it needs to be explicitly mentioned, but this needs to merge before we can merge the agent driver implementation: https://review.openstack.org/#/c/327046/ | 16:10 |
| patchbot | patch 327046 - ironic - Add Virtual Network Interface Driver APIs | 16:10 |
| jcook | mariojv the spec, the nova virt driver, something else? | 16:11 |
| *** dprince has quit IRC | 16:11 | |
| JayF | jcook: the linked patch he put in | 16:11 |
| mariojv | jcook: the part of the patch set that modifies the ironic network drivers | 16:11 |
| JayF | jcook: it's sambetts fixing the network problem for us | 16:11 |
| mariojv | yep | 16:11 |
| jcook | oh, I read that as the link was the agent driver implementation | 16:12 |
| jcook | I'll make those minor changes | 16:12 |
| *** jpich has quit IRC | 16:13 | |
| lindycoder | Hello folks, i have a little question about moving the gate-tempest-dsvm-ironic-ipa-partition-pxe_snmp-tinyipa-ubuntu-xenial-nv from experimental to official jobs (non-voting) | 16:15 |
| lindycoder | Hello folks, i have a little question about moving the gate-tempest-dsvm-ironic-ipa-partition-pxe_snmp-tinyipa-ubuntu-xenial-nv from experimental to official jobs (non-voting) | 16:15 |
| lindycoder | Right now in project-config i added a job directly (https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/projects.yaml#L5763) instead of using the matrix system, should i use it? | 16:15 |
| lindycoder | That would mean adding the driver to https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/projects.yaml#L5663 and mess with the exclude to avoid having 1000 jobs | 16:15 |
| lindycoder | (Or you don't care it's a question for openstack-infra?) | 16:15 |
| lindycoder | (And sorry for duplicated first message) | 16:15 |
| mariojv | ah, i should have put it as a top-level comment | 16:16 |
| *** daemontool has joined #openstack-ironic | 16:16 | |
| *** rcernin has quit IRC | 16:17 | |
| openstackgerrit | Jesse J. Cook proposed openstack/ironic-specs: Update rescue spec https://review.openstack.org/407236 | 16:17 |
| jcook | mariojv JayF made those minor changes ^ | 16:18 |
| *** pcaruana has quit IRC | 16:19 | |
| *** moshele has quit IRC | 16:22 | |
| *** yee37910 has joined #openstack-ironic | 16:25 | |
| *** yee379 has quit IRC | 16:25 | |
| *** aNuposic has joined #openstack-ironic | 16:25 | |
| mariojv | jcook: one comment about the states that i didn't notice before | 16:28 |
| *** ohamada has quit IRC | 16:30 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Minor documentation updates https://review.openstack.org/407591 | 16:31 |
| *** baoli has quit IRC | 16:32 | |
| NobodyCam | Good Morning Ironic'ers | 16:33 |
| TheJulia | Good morning | 16:33 |
| rloo | morning NobodyCam! | 16:34 |
| *** tuan_luong has quit IRC | 16:34 | |
| NobodyCam | Good Morning TheJulia and rloo :) | 16:34 |
| dtantsur | morning NobodyCam | 16:36 |
| dtantsur | morning rloo | 16:36 |
| NobodyCam | morning dtantsur :) | 16:37 |
| rloo | hi dtantsur | 16:37 |
| TheJulia | Why do we not have a good morning bot? | 16:37 |
| rloo | dtantsur: btw, i was wondering. how much more work (patches) do you think there will be, before driver composition is done? | 16:37 |
| dtantsur | rloo, in addition to what we have, we need: 1. conductor loading hardware types, 2. API changes, 3. port at least a few drivers | 16:38 |
| rloo | dtantsur: ok, so if we average 1 patch/week (seems like that is what is happening :)), we should be good to get this done in early Jan. | 16:39 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Make "enabled_drivers" config option value more resilient to failures https://review.openstack.org/407596 | 16:39 |
| lucasagomes | rbartal, ^ | 16:39 |
| *** vgadiraj has quit IRC | 16:40 | |
| dtantsur | rloo, yep :) also the next patch is probably the most complex and important | 16:40 |
| rloo | dtantsur: oh, thx for the heads up! :) | 16:40 |
| *** vgadiraj has joined #openstack-ironic | 16:41 | |
| *** absubram has quit IRC | 16:43 | |
| *** rama_y has joined #openstack-ironic | 16:53 | |
| *** vinsh has joined #openstack-ironic | 16:54 | |
| *** vgadiraj has quit IRC | 16:55 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 16:56 |
| *** misty-cloud has quit IRC | 16:59 | |
| *** mjura_ has quit IRC | 17:00 | |
| *** roeyc has quit IRC | 17:02 | |
| dtantsur | o/ | 17:06 |
| *** dtantsur is now known as dtantsur|afk | 17:06 | |
| NobodyCam | night dtantsur|afk | 17:06 |
| lucasagomes | dtantsur|afk, see ya | 17:07 |
| *** rama_y has quit IRC | 17:11 | |
| *** anush has quit IRC | 17:13 | |
| *** glonlas has quit IRC | 17:13 | |
| *** vgadiraj has joined #openstack-ironic | 17:15 | |
| *** anush has joined #openstack-ironic | 17:15 | |
| *** aNuposic has quit IRC | 17:18 | |
| *** e0ne has quit IRC | 17:19 | |
| *** rcernin has joined #openstack-ironic | 17:26 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Use IronicObject._from_db_object_list method https://review.openstack.org/407616 | 17:27 |
| *** snecklifter has quit IRC | 17:29 | |
| *** nicodemos has joined #openstack-ironic | 17:31 | |
| lucasagomes | have a great evening all o/ | 17:33 |
| *** lucasagomes is now known as lucas-afk | 17:33 | |
| jlvillal | vsaienk0: Regarding your comment: https://review.openstack.org/#/c/406376/3/devstack/lib/ironic | 17:34 |
| patchbot | patch 406376 - ironic - devstack: Set [compute]/fixed_network_name | 17:34 |
| jlvillal | vsaienk0: What do you think the best way to check that is? | 17:34 |
| jlvillal | vsaienk0: if [[ -n "${IRONIC_PROVISION_NETWORK_NAME}" ]] ??? | 17:35 |
| *** aweeks has joined #openstack-ironic | 17:35 | |
| *** baoli has joined #openstack-ironic | 17:36 | |
| *** milan has quit IRC | 17:37 | |
| rloo | bye lucas-afk | 17:38 |
| sambetts | jlvillal: you guys keep adding more stuff for me to remove/rework in my simulation refactor patch :/ :P | 17:38 |
| jlvillal | heh | 17:39 |
| JayF | should we work to land that first? | 17:43 |
| JayF | I know the OSA team wanted sambetts' linux bridge stuff for devstack in place so they could start gating ironic | 17:43 |
| *** Romanenko_K has quit IRC | 17:44 | |
| *** fragatina has joined #openstack-ironic | 17:48 | |
| *** aNuposic has joined #openstack-ironic | 17:48 | |
| jroll | lindycoder: didn't look you got an answer, still need help? | 17:52 |
| sambetts | I would like to see that happen, I have tested as many of the purmutations of things locally as possible, and things like multinode all work at least in my envionrment out of the box | 17:52 |
| *** rama_y has joined #openstack-ironic | 17:54 | |
| *** absubram has joined #openstack-ironic | 17:54 | |
| openstackgerrit | Sam Betts proposed openstack/ironic: Switch to new location for oslo.db test cases https://review.openstack.org/407625 | 17:56 |
| sambetts | jroll: ^ | 17:56 |
| *** baoli has quit IRC | 17:56 | |
| jroll | sambetts: sweet, thanks | 17:57 |
| * jroll queues | 17:57 | |
| *** baoli has joined #openstack-ironic | 17:57 | |
| jlvillal | sambetts: How can your devstack rework patch be made to pass the gate? | 17:57 |
| sambetts | jroll: I looked into the legacy engine warning, that too is related to our migrations code but I can't work out what the right path to go down is | 17:57 |
| jroll | sambetts: okay, no worries, is a bug filed? | 17:58 |
| JayF | jroll: yes | 17:58 |
| JayF | I just triaged it | 17:58 |
| JayF | lol | 17:58 |
| jroll | thanks | 17:58 |
| jroll | :D | 17:58 |
| jroll | so many variables in these job names, lol | 17:58 |
| sambetts | jlvillal: the ordinary tests should all pass out of the box, the grenade ones are going to be harder because they are very heavily hooked into the old simulation | 17:59 |
| sambetts | jlvillal: and in a similar way so are the multinode jobs | 17:59 |
| sambetts | jlvillal: and multitenant jobs | 17:59 |
| sambetts | :/ | 17:59 |
| sambetts | sigh ... I wish I had got this code up sooner | 18:00 |
| jlvillal | sambetts: yeah. I see 4 gate jobs fail and I think there must be work left to be done... | 18:00 |
| jlvillal | sambetts: Or some intricate dance has to be performed between project-config and ironic changes. | 18:00 |
| sambetts | jlvillal: most likely this ^ | 18:01 |
| sambetts | jlvillal: I might be able to get the grenade jobs working by just changing the grenade plugin | 18:01 |
| jlvillal | :( Thinks it will be painful. And prone to something going wrong. | 18:01 |
| *** alexpilotti has joined #openstack-ironic | 18:02 | |
| sambetts | jlvillal: it only get more painful the more and more we embed the hacky simulation into our testing | 18:02 |
| jlvillal | True... | 18:02 |
| sambetts | the testing shouldn't care about the simulation, and after my patch lands it won't need to | 18:02 |
| *** derekh has quit IRC | 18:05 | |
| sambetts | JayF: one more for you https://bugs.launchpad.net/ironic/+bug/1647808 | 18:05 |
| openstack | Launchpad bug 1647808 in Ironic "OsloDBDeprecationWarning: EngineFacade is deprecated; please use oslo_db.sqlalchemy.enginefacade" [Undecided,New] | 18:05 |
| jroll | huh, that's the one I was asking about before :P | 18:06 |
| *** ralonsoh has quit IRC | 18:06 | |
| *** alexpilotti has quit IRC | 18:06 | |
| JayF | Is there a reason, if one of us who work on the project file a bug | 18:06 |
| JayF | that they shouldn't just set importance and status to triaged when it's filed? | 18:07 |
| JayF | like is it intended to be a two person system, like your bug getting a "+2" or something? | 18:07 |
| jroll | no, there's no reason for that IMO | 18:07 |
| *** Romanenko_K has joined #openstack-ironic | 18:07 | |
| jroll | dmitry always asks people to do that | 18:07 |
| JayF | okay. Perfect. | 18:07 |
| sambetts | ah ok, I'll do that :) I always thought someone needed to confirm it | 18:10 |
| jroll | nah, just ask if you aren't sure | 18:10 |
| sambetts | cool :) | 18:10 |
| JayF | I already did it for that bug to be clear | 18:10 |
| sambetts | thanks | 18:11 |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Clean up object code https://review.openstack.org/407635 | 18:12 |
| *** anush has quit IRC | 18:15 | |
| *** Romanenko_K has quit IRC | 18:16 | |
| openstackgerrit | Anup Navare proposed openstack/ironic: Configdrive support for Ceph https://review.openstack.org/407639 | 18:21 |
| lindycoder | jroll, well i gave the matrix solution a shot : https://review.openstack.org/#/c/407604/ | 18:23 |
| patchbot | patch 407604 - openstack-infra/project-config - Make pxe_snmp jobs official and non-voting | 18:23 |
| jroll | lindycoder: that appears correct to me, but I'm not an expert :) | 18:24 |
| jroll | lindycoder: is all the code merged for that and the CI working? | 18:25 |
| *** mgould is now known as mgould|afk | 18:25 | |
| lindycoder | it does create a whole lot of jenkins jobs that are not referenced in the zuul layout but plenty of other parts the matrix are ignored aswell | 18:25 |
| lindycoder | jroll, yeah, that's the code that was required : https://review.openstack.org/#/c/388154/ | 18:26 |
| patchbot | patch 388154 - ironic - Add virtualpdu to ironic devstack plugin (MERGED) | 18:26 |
| *** rama_y has quit IRC | 18:26 | |
| jroll | lindycoder: awesome, +1 on 407604 then | 18:26 |
| lindycoder | Thanks | 18:26 |
| jroll | lindycoder: as an FYI, that repo is a bit frozen while infra moves all jobs to xenial, but after that we should be able to get it merged | 18:27 |
| *** krtaylor has quit IRC | 18:28 | |
| jroll | can I get an eye or two on this RFE? https://bugs.launchpad.net/ironic/+bug/1647813 | 18:28 |
| openstack | Launchpad bug 1647813 in Ironic "CI for using nova with resource classes and new-style flavors" [Wishlist,New] | 18:28 |
| *** glonlas has joined #openstack-ironic | 18:28 | |
| JayF | looking | 18:28 |
| lindycoder | well i got the experimental job merged not long ago, if they want me to remove all non-xenial i could do that | 18:29 |
| JayF | jroll: how can we have an RFE to test something that doesn't exist yet? | 18:29 |
| jroll | JayF: the code is proposed in nova, the goal is to be able to test it before it lands, then continue to test it | 18:30 |
| JayF | ah, okay | 18:30 |
| jroll | so create job, iterate on devstack plugin, iterate on nova code | 18:30 |
| jroll | also the flavor thing might be defined but there's other open questions | 18:30 |
| jroll | lindycoder: nah infra is on top of that | 18:30 |
| JayF | seems sane to me. For anyone but a core filing this RFE I'd probably ask for more details, but it seems pretty clear that the goal is to figure out the details | 18:30 |
| JayF | as much as anything else | 18:30 |
| jroll | JayF: yeah, I mean, I'm going to hack on it either way, if we want to wait for it to be solid before approving that's fine | 18:31 |
| JayF | jroll: also you should've marekd the bug as "Triaged" since you set importance :P | 18:31 |
| *** alexpilotti has joined #openstack-ironic | 18:31 | |
| jroll | yeah, couldn't remember if triaged or confirmed was the right thing for rfe | 18:31 |
| JayF | Triaged is always the right thing | 18:32 |
| jroll | k | 18:32 |
| JayF | if you don't want anyone using dmitry's dash to look at it again | 18:32 |
| * jroll mostly just wanted a bug up so I have a branch name :P | 18:32 | |
| *** jkilpatr_ has joined #openstack-ironic | 18:32 | |
| *** alexpilotti has quit IRC | 18:33 | |
| *** alexpilotti has joined #openstack-ironic | 18:33 | |
| aNuposic | Hi Folks, I have submitted RFE https://bugs.launchpad.net/ironic/+bug/1642719 can I get some inputs on it? | 18:33 |
| openstack | Launchpad bug 1642719 in Ironic "[RFE] RADOSGW authentication mechanism for storing config drive in Ceph" [Wishlist,In progress] - Assigned to Anup (anup-d-navare) | 18:33 |
| JayF | aNuposic: I'm OK with that being approved, but have reservations about adding a feature that we aren't going to be tempest testing. | 18:34 |
| JayF | aNuposic: assuming as a project we're OK with the testing plan, I'm OK with that rfe | 18:34 |
| JayF | jroll: ^ wdyt | 18:34 |
| *** jkilpatr has quit IRC | 18:34 | |
| *** penick has joined #openstack-ironic | 18:35 | |
| aNuposic | JayF: Ah. So do you think tempest tests would be required for this? | 18:35 |
| jroll | JayF: gotta give me a few | 18:35 |
| JayF | aNuposic: I think having something tested via tempest is the "default" | 18:35 |
| JayF | aNuposic: and we gotta think hard before adding an exception to that | 18:36 |
| *** penick_ has joined #openstack-ironic | 18:36 | |
| jroll | but tl;dr I know there's very minimal ceph testing in openstack, but it is possible | 18:36 |
| JayF | aNuposic: especially since historically we've broken support for untested things | 18:36 |
| jroll | can just install ceph in the gate and run with that | 18:36 |
| jroll | (YAY MORE JOB MATRIX EXPLOSION) | 18:36 |
| aNuposic | JayF: I see | 18:36 |
| aNuposic | jroll: let me see installing ceph into gate | 18:37 |
| *** rama_y has joined #openstack-ironic | 18:37 | |
| jroll | aNuposic: I'm not saying we should or should not do that; only that it's possible | 18:37 |
| JayF | aNuposic: that's a pretty deep rabbithole I think, just warning you :) | 18:37 |
| jroll | also that :P | 18:37 |
| JayF | aNuposic: but if you did a little research and put into the RFE information about how easy/hard it is to test ceph in the gate with ironic | 18:37 |
| JayF | aNuposic: it would be very informative to people approving the RFE | 18:38 |
| *** penick has quit IRC | 18:39 | |
| *** penick_ is now known as penick | 18:39 | |
| aNuposic | JayF: :) Cool. I will look investigate how can I put info on testing ceph into gate. Honestly I haven't played much with gates. This will be now good exercise for me :) | 18:40 |
| aNuposic | jroll: ^ | 18:40 |
| *** vsaienko has joined #openstack-ironic | 18:41 | |
| rloo | TheJulia: I see that KVM entry failed error; did you want me to add a link to the log, in our whiteboard? | 18:41 |
| vsaienko | jlvillal: could you please have look once have a time https://review.openstack.org/#/c/407502/ | 18:41 |
| patchbot | patch 407502 - openstack-dev/grenade - Use same tempest.conf for base and target | 18:41 |
| TheJulia | rloo: might as well, was it on OVH? | 18:42 |
| jlvillal | vsaienko: Will do! | 18:42 |
| rloo | TheJulia: yes. i think. | 18:42 |
| * TheJulia wonders if nested virt is just not an option on OVH | 18:42 | |
| vsaienko | jlvillal: thanks! https://review.openstack.org/#/c/368173/ multinode with smoke and grenade passing | 18:43 |
| patchbot | patch 368173 - ironic - Testing multinode stuff | 18:43 |
| JayF | TheJulia: the devstack plugin is supposed to test and find that | 18:43 |
| jlvillal | vsaienko: Sweet :) | 18:43 |
| TheJulia | JayF: Indeed :( | 18:43 |
| jlvillal | vsaienko: If you can respond to my question about: https://review.openstack.org/#/c/406376/3/devstack/lib/ironic | 18:43 |
| patchbot | patch 406376 - ironic - devstack: Set [compute]/fixed_network_name | 18:43 |
| jlvillal | vsaienko: It would be appreciated :) | 18:43 |
| JayF | Honestly, folks at the summit told us that nested kvm could be unstable even when available | 18:44 |
| TheJulia | woot | 18:44 |
| JayF | not in Barcelona, but Austin | 18:44 |
| *** vsaienko has quit IRC | 18:45 | |
| *** anush has joined #openstack-ironic | 18:47 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Allow using resource classes https://review.openstack.org/407651 | 18:47 |
| openstackgerrit | Mark Hamzy proposed openstack/ironic-specs: Add initial specification for MoltenIron https://review.openstack.org/387418 | 18:48 |
| *** penick has quit IRC | 18:48 | |
| *** vgadiraj has quit IRC | 18:49 | |
| jroll | hamzy: what's the spec for? | 18:49 |
| jlvillal | vsaienk0: Also on: https://review.openstack.org/#/c/406999/ I noticed it keeps failing. Do you think an intermittent issue or an issue with the patch? | 18:50 |
| patchbot | patch 406999 - tempest - Update create_networks() logic | 18:50 |
| hamzy | jroll, molteniron? | 18:50 |
| hamzy | not sure I understand your question | 18:50 |
| jroll | hamzy: I guess I'm curious why you're writing an ironic spec for it, we've already brought it into the ironic project, go hack :) | 18:51 |
| *** trown is now known as trown|lunch | 18:51 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Fix docsting in Mellanox _detect_hardware https://review.openstack.org/407510 | 18:51 |
| jroll | hamzy: I guess I don't mind you writing down design decisions there, just not sure who is qualified to review it | 18:51 |
| hamzy | jroll, that's that I said! :) but krtaylor said we should finish the spec even though it was approved | 18:51 |
| jroll | hamzy: my vote would be, if you want to write down design, just do it in the molteniron repo somewhere | 18:52 |
| rloo | jroll, hamzy: ++. as documentation instead of a spec. | 18:53 |
| jroll | :) | 18:54 |
| hamzy | sure... I can do that... less bureaucracy that way :) | 18:54 |
| *** krtaylor has joined #openstack-ironic | 18:54 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru from OneView drivers https://review.openstack.org/397846 | 18:54 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove iBoot, WoL and AMT drivers https://review.openstack.org/397847 | 18:54 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru from iLO drvers https://review.openstack.org/397845 | 18:54 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru completely https://review.openstack.org/397848 | 18:54 |
| *** rbartal_ has quit IRC | 18:55 | |
| hamzy | as a bit of history, this spec was started as part of the process to get molteniron accepted by the Ironic team... so I was cleaning up open things on my plate | 18:55 |
| jroll | yeah, that's why I was so curious when I saw it :P | 18:57 |
| *** rbartal has quit IRC | 18:57 | |
| jlvillal | vsaienk0: Will that grenade patch mean don't need https://review.openstack.org/#/c/380006/ ? | 18:59 |
| patchbot | patch 380006 - openstack-dev/devstack - Set fixed_network_name always when ironic is enabled. | 18:59 |
| *** vsaienko has joined #openstack-ironic | 19:01 | |
| hamzy | so jroll should I close the launchpad bug also? | 19:01 |
| jroll | hamzy: yes please | 19:02 |
| sambetts | Night all | 19:03 |
| *** sambetts is now known as sambetts|afk | 19:03 | |
| *** aNuposic has quit IRC | 19:03 | |
| *** Nisha_Agarwal has quit IRC | 19:04 | |
| jlvillal | sambetts|afk: Good night | 19:04 |
| hamzy | kill -9 1633540 | 19:06 |
| *** e0ne has joined #openstack-ironic | 19:07 | |
| *** aNuposic has joined #openstack-ironic | 19:10 | |
| *** vsaienko has quit IRC | 19:13 | |
| *** bapalm_ has quit IRC | 19:14 | |
| *** devlaps has quit IRC | 19:16 | |
| *** devlaps has joined #openstack-ironic | 19:16 | |
| *** bapalm has joined #openstack-ironic | 19:18 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: devstack: Set [compute]/fixed_network_name https://review.openstack.org/406376 | 19:20 |
| *** fragatina has quit IRC | 19:21 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 19:22 | |
| *** e0ne has quit IRC | 19:23 | |
| *** Sukhdev has joined #openstack-ironic | 19:23 | |
| *** AlexeyAbashkin has quit IRC | 19:23 | |
| jlvillal | vsaienk0: If you are around: https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L1704 | 19:24 |
| jlvillal | vsaienk0: Is that something that should be inside an 'if' statement? | 19:24 |
| *** yee37910 has quit IRC | 19:31 | |
| *** yee379 has joined #openstack-ironic | 19:31 | |
| *** Sukhdev has quit IRC | 19:31 | |
| *** Sukhdev has joined #openstack-ironic | 19:32 | |
| *** ChubYann has joined #openstack-ironic | 19:33 | |
| *** pleia2 has quit IRC | 19:34 | |
| *** morgabra has quit IRC | 19:34 | |
| *** pleia2 has joined #openstack-ironic | 19:34 | |
| *** morgabra has joined #openstack-ironic | 19:34 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: multi-TENANT with tempest "smoke" test helper https://review.openstack.org/406368 | 19:36 |
| *** vgadiraj has joined #openstack-ironic | 19:36 | |
| *** jkilpatr_ has quit IRC | 19:44 | |
| *** ijw has quit IRC | 19:49 | |
| *** aNuposic has quit IRC | 19:51 | |
| *** jkilpatr_ has joined #openstack-ironic | 19:57 | |
| *** vsaienko has joined #openstack-ironic | 19:57 | |
| *** moshele has joined #openstack-ironic | 19:59 | |
| *** Marx314 has joined #openstack-ironic | 20:05 | |
| *** trown|lunch is now known as trown | 20:10 | |
| *** devlaps has quit IRC | 20:10 | |
| *** vsaienko has quit IRC | 20:16 | |
| openstackgerrit | Bob Fournier proposed openstack/ironic-inspector: Add plugin to process basic management LLDP TLVs https://review.openstack.org/406496 | 20:21 |
| *** rajinir has joined #openstack-ironic | 20:25 | |
| *** aNuposic has joined #openstack-ironic | 20:28 | |
| *** glonlas_ has joined #openstack-ironic | 20:29 | |
| *** glonlas has quit IRC | 20:32 | |
| jroll | mtreinish: courtesy of NobodyCam https://github.com/NoBodyCam/ironic | 20:34 |
| jroll | check out the readme there | 20:34 |
| mtreinish | jroll: heh, I like that readme | 20:35 |
| jroll | :P | 20:36 |
| NobodyCam | huh | 20:36 |
| NobodyCam | lol | 20:36 |
| jroll | NobodyCam: your driver got a shoutout in the tc meeting :P | 20:36 |
| *** moshele has quit IRC | 20:36 | |
| NobodyCam | lol :) nice :) | 20:36 |
| mtreinish | NobodyCam: but that's cheating. I was hoping you'd actually have a network enabled coffee maker :p | 20:37 |
| NobodyCam | hahhaha | 20:37 |
| jroll | surprised he doesn't by now tbh | 20:38 |
| *** glonlas_ has quit IRC | 20:38 | |
| NobodyCam | #justSaying: https://www.amazon.com/Mr-Coffee-Wifi-Enabled-Coffeemaker-BVMC-PSTX91WE/dp/B00LUFSSWG | 20:39 |
| *** ijw_ has joined #openstack-ironic | 20:39 | |
| *** ijw_ has quit IRC | 20:39 | |
| jroll | :D | 20:39 |
| *** ijw_ has joined #openstack-ironic | 20:39 | |
| *** glonlas has joined #openstack-ironic | 20:39 | |
| NobodyCam | :p | 20:39 |
| *** Sukhdev has quit IRC | 20:41 | |
| *** daemontool_ has joined #openstack-ironic | 20:44 | |
| *** daemontool has quit IRC | 20:47 | |
| *** vsaienko has joined #openstack-ironic | 20:50 | |
| *** david-lyle has quit IRC | 20:55 | |
| *** glonlas_ has joined #openstack-ironic | 21:06 | |
| *** devlaps has joined #openstack-ironic | 21:06 | |
| *** glonlas_ has quit IRC | 21:07 | |
| *** glonlas_ has joined #openstack-ironic | 21:08 | |
| *** MattMan has quit IRC | 21:09 | |
| *** MattMan has joined #openstack-ironic | 21:09 | |
| *** fragatina has joined #openstack-ironic | 21:09 | |
| *** glonlas has quit IRC | 21:10 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Allow using resource classes https://review.openstack.org/407651 | 21:14 |
| rloo | hi jroll. wondering if you have thoughts about the rolling upgrade patches. This seems to be the first (and only one) so far, it adds a config. do we want to expose it before the rest of the code is working? https://review.openstack.org/#/c/407491 | 21:17 |
| patchbot | patch 407491 - ironic - Add RPC version pinning | 21:17 |
| rloo | jroll: am wondering if we want to -2 it until all/most is working. | 21:18 |
| *** glonlas_ has quit IRC | 21:19 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: devstack: Set [compute]/fixed_network_name https://review.openstack.org/406376 | 21:19 |
| jroll | rloo: yeah, probably, or at least until we can evaluate where the right places to pin are | 21:22 |
| rloo | jroll: ok. what are the odds that it will be done when you are away? you will be back before i am back so maybe you -2 it. | 21:23 |
| rloo | or maybe JayF. I think he is around more :) | 21:23 |
| jroll | rloo: I will? when am I back? | 21:23 |
| rloo | jroll: first week of jan, no? | 21:23 |
| jroll | er, when are you back? :P | 21:23 |
| rloo | 2nd week of Jan :D | 21:23 |
| jroll | rloo: TheJulia is taking almost no time off if you need someone that will be around | 21:23 |
| jroll | aha ok | 21:23 |
| rloo | oh, ok, TheJulia, would you mind -2'ing https://review.openstack.org/#/c/407491 | 21:24 |
| patchbot | patch 407491 - ironic - Add RPC version pinning | 21:24 |
| *** glonlas has joined #openstack-ironic | 21:25 | |
| TheJulia | rloo: sure | 21:25 |
| rloo | thx TheJulia! | 21:25 |
| rloo | TheJulia: you're around in Jan, right? :) | 21:26 |
| JayF | if someone wants a fairly painless IPA review... https://review.openstack.org/#/c/399857/ already has my +2 | 21:27 |
| patchbot | patch 399857 - ironic-python-agent - Skip API releated work if no api url configured | 21:27 |
| TheJulia | rloo: I will be :) | 21:28 |
| rloo | TheJulia: good! | 21:28 |
| rloo | JayF: there's a typo in that patch title :) | 21:29 |
| JayF | rloo: I know, I added a nit as soon as I saw the bot post it in here | 21:29 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: DNM/DNR: Testing tempest https://review.openstack.org/407715 | 21:29 |
| JayF | lol | 21:29 |
| rloo | JayF: that shouldn't stop it from being merged. but i. have. to. focus... | 21:29 |
| JayF | I don't have any particular like or dislike for that patch, just have seen it outstanding a while is only reason I posted it | 21:30 |
| JayF | retain focus if you wish :) | 21:30 |
| *** hamzy has quit IRC | 21:39 | |
| *** jtomasek has quit IRC | 21:39 | |
| *** chlong has joined #openstack-ironic | 21:43 | |
| jlvillal | JayF: TheJulia: What do you think of this as a commit message? | 21:44 |
| jlvillal | When a node is inspected the 'api_url' value is not provided to IPA via the PXE parameters. Update so a default value for 'api_url' is no longer provided. If 'api_url' is not provided skip the heartbeat and lookup steps. | 21:44 |
| jlvillal | The current commit message is confusing. At least for me. | 21:45 |
| JayF | jlvillal: your edit isn't accurate | 21:45 |
| JayF | jlvillal: this handles the !inspect-url !api-url cast | 21:45 |
| JayF | *case | 21:45 |
| jlvillal | JayF: I find the commit message even more confusing then :) | 21:45 |
| JayF | lol | 21:45 |
| *** jcoufal has quit IRC | 21:46 | |
| *** jcoufal has joined #openstack-ironic | 21:47 | |
| jlvillal | JayF: Can you suggest a better message? | 21:47 |
| JayF | If I wasn't simultaneously in a video meeting I could :) | 21:47 |
| jlvillal | Okay. Thanks :) | 21:47 |
| JayF | Give me a few and I can? | 21:47 |
| jlvillal | Thanks! | 21:47 |
| *** jcoufal has quit IRC | 21:48 | |
| *** alexpilo_ has joined #openstack-ironic | 21:51 | |
| TheJulia | It kind of makes sense to me, but admittedly, I'm weird. | 21:53 |
| *** alexpilotti has quit IRC | 21:54 | |
| *** daemontool_ has quit IRC | 21:55 | |
| *** daemontool_ has joined #openstack-ironic | 21:55 | |
| jlvillal | Good morning mrda | 22:00 |
| openstackgerrit | Merged openstack/bifrost: Allow installing Ansible from PyPI https://review.openstack.org/406321 | 22:01 |
| openstackgerrit | Merged openstack/bifrost: Add support for dnf as package manager https://review.openstack.org/407524 | 22:01 |
| mrda | jlvillal: how did you know I was here? :) | 22:01 |
| jlvillal | TheJulia: It could just be that I don't understand it enough :) | 22:01 |
| jlvillal | mrda: Because we have a meeting scheduled for now :) | 22:02 |
| jlvillal | I feel lonely in our chat room :( | 22:02 |
| TheJulia | :( | 22:03 |
| TheJulia | mrda: At least he can say a meeting and that he doesn't somehow have spies watching your every move :) | 22:03 |
| mrda | oh, I'm confused by timezones again | 22:04 |
| mrda | I'm on a call with jroll and mariojv about ironic :) | 22:04 |
| mrda | jlvillal: Can we do this in 30 minutes? | 22:04 |
| jlvillal | mrda: Uh. Sure. I do have to leave in about 50 minutes | 22:04 |
| TheJulia | Timezones may be the root of all evil. | 22:04 |
| openstackgerrit | Chris Krelle proposed openstack/bifrost: Adding staging driver support https://review.openstack.org/406401 | 22:06 |
| NobodyCam | morning mrda | 22:06 |
| *** chlong has quit IRC | 22:06 | |
| * mrda falls back to the defence of dying from the plague | 22:06 | |
| mrda | NobodyCam: o/ | 22:07 |
| *** krtaylor has quit IRC | 22:07 | |
| NobodyCam | :) | 22:07 |
| *** chlong has joined #openstack-ironic | 22:07 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Configure tempest for multitenancy/flat network https://review.openstack.org/406376 | 22:11 |
| *** chlong has quit IRC | 22:12 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Configure tempest for multitenancy/flat network https://review.openstack.org/406376 | 22:12 |
| *** Sukhdev has joined #openstack-ironic | 22:15 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent: Update mock variable name in unit tests https://review.openstack.org/407729 | 22:15 |
| Sukhdev | Hey folks "tox -egendocs" is not working for me - anybody has any insight? | 22:17 |
| *** lindycoder has quit IRC | 22:17 | |
| jroll | Sukhdev: tox -edocs | 22:17 |
| *** jkilpatr_ has quit IRC | 22:18 | |
| *** jkilpatr has joined #openstack-ironic | 22:18 | |
| Sukhdev | jroll : ah - thanks | 22:18 |
| *** bfournie has quit IRC | 22:18 | |
| jroll | np | 22:18 |
| *** srobert_ has quit IRC | 22:19 | |
| *** jkilpatr has quit IRC | 22:23 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: multi-TENANT with tempest "smoke" test helper https://review.openstack.org/406368 | 22:24 |
| *** david-lyle has joined #openstack-ironic | 22:24 | |
| *** trown is now known as trown|outtypewww | 22:29 | |
| *** jheroux has quit IRC | 22:30 | |
| *** glonlas has quit IRC | 22:30 | |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: [WIP] Agent driver rescue implementation https://review.openstack.org/400437 | 22:33 |
| *** daemontool_ has quit IRC | 22:33 | |
| *** daemontool_ has joined #openstack-ironic | 22:34 | |
| *** ijw_ has quit IRC | 22:37 | |
| openstackgerrit | Merged openstack/ironic: Update help for 'provisioning_network' option https://review.openstack.org/407563 | 22:37 |
| *** ijw has joined #openstack-ironic | 22:37 | |
| *** david-lyle has quit IRC | 22:41 | |
| *** david-lyle has joined #openstack-ironic | 22:41 | |
| *** ijw has quit IRC | 22:42 | |
| *** harlowja has quit IRC | 22:43 | |
| *** harlowja has joined #openstack-ironic | 22:43 | |
| *** cdearborn has quit IRC | 22:44 | |
| *** priteau has quit IRC | 22:45 | |
| *** david-lyle has quit IRC | 22:46 | |
| openstackgerrit | Merged openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/406850 | 22:47 |
| JayF | jlvillal: wdyt about the option I just posted? | 22:47 |
| *** bfournie has joined #openstack-ironic | 22:52 | |
| *** yolanda has quit IRC | 22:52 | |
| *** yolanda has joined #openstack-ironic | 22:54 | |
| *** Goneri has quit IRC | 22:54 | |
| jlvillal | JayF: That works for me from my quick scan | 22:56 |
| jlvillal | JayF: I have to leave now. Offsite to go to. | 22:56 |
| jlvillal | Bye everyone! | 22:57 |
| JayF | I was hoping we could edit that commit message and land it, but I guess not :/ | 22:57 |
| JayF | have fun at your offsite | 22:57 |
| NobodyCam | night jlvillal | 22:58 |
| *** ijw has joined #openstack-ironic | 23:01 | |
| *** rbudden has quit IRC | 23:02 | |
| *** rcernin has quit IRC | 23:02 | |
| *** yolanda has quit IRC | 23:04 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Allow using resource classes https://review.openstack.org/407651 | 23:10 |
| jroll | nice catch JayF | 23:10 |
| jroll | thanks | 23:10 |
| * jroll wips that this time | 23:10 | |
| JayF | jroll: np | 23:10 |
| jroll | night y'all | 23:10 |
| JayF | jroll: I know you didn't want reviews | 23:10 |
| JayF | jroll: but I was curious :P | 23:10 |
| jroll | ya | 23:10 |
| *** Goneri has joined #openstack-ironic | 23:11 | |
| *** yolanda has joined #openstack-ironic | 23:13 | |
| *** pmannidi has joined #openstack-ironic | 23:13 | |
| *** jtomasek has joined #openstack-ironic | 23:14 | |
| *** aNuposic has quit IRC | 23:14 | |
| *** jtomasek has quit IRC | 23:18 | |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui: Consolidate node last_error processing https://review.openstack.org/406184 | 23:19 |
| JayF | Whoa, a patch to ironic-ui | 23:19 |
| *** yolanda has quit IRC | 23:22 | |
| *** jkilpatr has joined #openstack-ironic | 23:26 | |
| ppiela | More to come | 23:28 |
| JayF | ppiela: awesome! I just hope there are folks around who are going to review them, lol | 23:28 |
| ppiela | JayF: me too | 23:29 |
| *** krtaylor has joined #openstack-ironic | 23:30 | |
| *** yolanda has joined #openstack-ironic | 23:35 | |
| *** vsaienko has quit IRC | 23:35 | |
| *** absubram has quit IRC | 23:42 | |
| *** yolanda has quit IRC | 23:43 | |
| *** vinsh has quit IRC | 23:46 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
