| *** wajdi_ has joined #openstack-ironic | 00:06 | |
| *** ppiela has quit IRC | 00:09 | |
| *** wajdi has quit IRC | 00:10 | |
| *** wajdi_ has quit IRC | 00:15 | |
| *** rbudden has quit IRC | 00:27 | |
| *** mtanino_ has joined #openstack-ironic | 00:33 | |
| *** mtanino has quit IRC | 00:34 | |
| *** jmccrory has quit IRC | 00:37 | |
| *** jmccrory_ has joined #openstack-ironic | 00:37 | |
| *** rm_work has quit IRC | 00:38 | |
| *** rm_work has joined #openstack-ironic | 00:38 | |
| *** jmccrory_ is now known as jmccrory | 00:39 | |
| *** sasha2 has quit IRC | 00:40 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/ironic: Move baremetal tempest config setting from devstack https://review.openstack.org/408404 | 00:47 |
|---|---|---|
| *** SerenaFeng has joined #openstack-ironic | 00:54 | |
| *** SerenaFeng has quit IRC | 01:02 | |
| *** rama_y has quit IRC | 01:03 | |
| *** phuongnh has joined #openstack-ironic | 01:05 | |
| openstackgerrit | Merged openstack/python-ironicclient: Simplify heading capitalization. https://review.openstack.org/414997 | 01:10 |
| *** lindycoder has joined #openstack-ironic | 01:11 | |
| *** hoangcx has joined #openstack-ironic | 01:21 | |
| *** hw_wutianwei has joined #openstack-ironic | 01:23 | |
| *** hoangcx_ has joined #openstack-ironic | 01:23 | |
| *** rbudden has joined #openstack-ironic | 01:30 | |
| *** sasha21 has joined #openstack-ironic | 01:32 | |
| openstackgerrit | Anh Tran proposed openstack/ironic: [PoC] Add fake fields in Node object and RPC https://review.openstack.org/415462 | 01:34 |
| *** Syed__ has quit IRC | 01:35 | |
| *** dyasny has quit IRC | 01:44 | |
| *** dyasny has joined #openstack-ironic | 01:45 | |
| *** mtanino has joined #openstack-ironic | 02:05 | |
| *** mtanino_ has quit IRC | 02:07 | |
| *** hoangcx has quit IRC | 02:08 | |
| *** hoangcx_ has quit IRC | 02:08 | |
| *** lindycoder has quit IRC | 02:10 | |
| *** ijw has joined #openstack-ironic | 02:16 | |
| *** ijw_ has quit IRC | 02:18 | |
| *** zhangjl has joined #openstack-ironic | 02:20 | |
| *** phuongnh has quit IRC | 02:21 | |
| *** phuongnh has joined #openstack-ironic | 02:22 | |
| *** lennyb has quit IRC | 02:23 | |
| *** yonglihe has joined #openstack-ironic | 02:23 | |
| *** lennyb has joined #openstack-ironic | 02:24 | |
| *** rbudden has quit IRC | 02:25 | |
| openstackgerrit | Merged openstack/ironic: Remove greenlet useless requirement https://review.openstack.org/417499 | 02:36 |
| *** mtanino has quit IRC | 02:42 | |
| *** awiddersheim has quit IRC | 02:46 | |
| *** gcb has joined #openstack-ironic | 02:50 | |
| *** dsneddon has quit IRC | 02:51 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add Cinder storage driver https://review.openstack.org/366197 | 02:55 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add base cinder common interface https://review.openstack.org/355625 | 02:55 |
| *** causten has joined #openstack-ironic | 03:03 | |
| *** xuao has joined #openstack-ironic | 03:06 | |
| *** rbudden has joined #openstack-ironic | 03:14 | |
| *** wajdi has joined #openstack-ironic | 03:14 | |
| *** lindycoder has joined #openstack-ironic | 03:15 | |
| *** mtanino has joined #openstack-ironic | 03:19 | |
| *** vikrant has joined #openstack-ironic | 03:23 | |
| *** mtanino has quit IRC | 03:29 | |
| *** gcb has quit IRC | 03:36 | |
| *** gcb has joined #openstack-ironic | 03:37 | |
| *** mtanino has joined #openstack-ironic | 03:43 | |
| *** awiddersheim has joined #openstack-ironic | 03:47 | |
| *** awiddersheim has quit IRC | 03:51 | |
| *** links has joined #openstack-ironic | 03:54 | |
| *** aparnav has joined #openstack-ironic | 04:02 | |
| *** lindycoder has quit IRC | 04:06 | |
| *** lindycoder has joined #openstack-ironic | 04:07 | |
| *** links has quit IRC | 04:21 | |
| *** links has joined #openstack-ironic | 04:22 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Ipmitool power driver for soft reboot and soft power off https://review.openstack.org/216738 | 04:29 |
| *** lindycoder has quit IRC | 04:31 | |
| *** causten has quit IRC | 04:33 | |
| *** causten has joined #openstack-ironic | 04:34 | |
| *** zhangjl has quit IRC | 04:38 | |
| *** causten has quit IRC | 04:38 | |
| *** zhangjl has joined #openstack-ironic | 04:39 | |
| *** zhangjl has quit IRC | 04:42 | |
| *** zhangjl has joined #openstack-ironic | 04:44 | |
| *** mtanino has quit IRC | 04:46 | |
| *** ijw_ has joined #openstack-ironic | 04:54 | |
| *** ijw has quit IRC | 04:57 | |
| *** sacharya has quit IRC | 04:58 | |
| *** causten has joined #openstack-ironic | 05:20 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 05:27 | |
| *** rama_y has joined #openstack-ironic | 05:35 | |
| *** zhangjl1 has joined #openstack-ironic | 05:37 | |
| *** zhangjl has quit IRC | 05:39 | |
| *** zhangjl1 has quit IRC | 05:42 | |
| *** moshele has joined #openstack-ironic | 05:47 | |
| *** awiddersheim has joined #openstack-ironic | 05:49 | |
| *** awiddersheim has quit IRC | 05:55 | |
| *** moshele has quit IRC | 05:56 | |
| *** sacharya has joined #openstack-ironic | 05:59 | |
| *** sacharya has quit IRC | 06:03 | |
| *** rama_y has quit IRC | 06:03 | |
| *** tuanluong has joined #openstack-ironic | 06:04 | |
| *** zzzeek has quit IRC | 06:06 | |
| *** zhangjl has joined #openstack-ironic | 06:10 | |
| *** zzzeek has joined #openstack-ironic | 06:10 | |
| *** AlexeyAbashkin has quit IRC | 06:24 | |
| *** pcaruana has quit IRC | 06:26 | |
| *** rbudden has quit IRC | 06:34 | |
| openstackgerrit | Karthik S proposed openstack/ironic-inspector-specs: Spec for retrieving NUMA node information https://review.openstack.org/396147 | 06:43 |
| *** fxpester has joined #openstack-ironic | 06:44 | |
| *** mjura has joined #openstack-ironic | 06:44 | |
| *** mjura_ has joined #openstack-ironic | 06:45 | |
| *** mjura_ has quit IRC | 06:46 | |
| *** mjura has quit IRC | 06:46 | |
| *** mjura has joined #openstack-ironic | 06:46 | |
| *** rbartal has joined #openstack-ironic | 06:55 | |
| *** karthiks has joined #openstack-ironic | 06:59 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic management I/F for Inject NMI https://review.openstack.org/348191 | 07:07 |
| *** AlexeyAbashkin has joined #openstack-ironic | 07:09 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Ipmitool management driver for Inject NMI https://review.openstack.org/348721 | 07:11 |
| *** tesseract has joined #openstack-ironic | 07:13 | |
| *** rcernin has joined #openstack-ironic | 07:17 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Ipmitool power driver for soft reboot and soft power off https://review.openstack.org/216738 | 07:19 |
| *** makowals has quit IRC | 07:20 | |
| *** amotoki has joined #openstack-ironic | 07:24 | |
| *** makowals has joined #openstack-ironic | 07:26 | |
| *** ChubYann has quit IRC | 07:26 | |
| *** hoonetorg has quit IRC | 07:48 | |
| *** awiddersheim has joined #openstack-ironic | 07:52 | |
| *** awiddersheim has quit IRC | 07:57 | |
| *** sacharya has joined #openstack-ironic | 08:00 | |
| *** karthiks has quit IRC | 08:00 | |
| *** karthiks has joined #openstack-ironic | 08:01 | |
| *** sacharya has quit IRC | 08:05 | |
| *** pcaruana has joined #openstack-ironic | 08:09 | |
| *** e0ne has joined #openstack-ironic | 08:11 | |
| *** ralonsoh has joined #openstack-ironic | 08:17 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add Virtual Network Interface REST APIs https://review.openstack.org/404240 | 08:19 |
| *** strigazi_AFK is now known as strigazi | 08:26 | |
| *** xek has joined #openstack-ironic | 08:27 | |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add new dbsync command online_data_migrations with first migration https://review.openstack.org/408556 | 08:28 |
| *** kthia has joined #openstack-ironic | 08:28 | |
| *** ccamacho has joined #openstack-ironic | 08:37 | |
| *** e0ne has quit IRC | 08:40 | |
| *** e0ne has joined #openstack-ironic | 08:47 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Catch error when wrong temp_url were generated https://review.openstack.org/370875 | 08:50 |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils: snmp inspection for disk size https://review.openstack.org/394858 | 08:52 |
| *** awiddersheim has joined #openstack-ironic | 08:53 | |
| *** dtantsur has quit IRC | 08:53 | |
| *** rcernin has quit IRC | 08:54 | |
| *** tesseract has quit IRC | 08:54 | |
| *** priteau has joined #openstack-ironic | 08:55 | |
| *** pcaruana has quit IRC | 08:55 | |
| *** moshele has joined #openstack-ironic | 08:57 | |
| *** awiddersheim has quit IRC | 08:57 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: TMP: Verify attach/detach https://review.openstack.org/414160 | 08:58 |
| *** zzzeek has quit IRC | 09:00 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Retry to connect to RPC if failed https://review.openstack.org/376462 | 09:00 |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Rolling upgrade procedure documentation https://review.openstack.org/416565 | 09:01 |
| *** dtantsur has joined #openstack-ironic | 09:01 | |
| *** zzzeek has joined #openstack-ironic | 09:02 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to session for glance client https://review.openstack.org/410767 | 09:03 |
| *** athomas has joined #openstack-ironic | 09:04 | |
| *** moshele has quit IRC | 09:05 | |
| *** tesseract has joined #openstack-ironic | 09:07 | |
| *** rcernin has joined #openstack-ironic | 09:08 | |
| *** pcaruana has joined #openstack-ironic | 09:08 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic-specs: Ironic deployment steps https://review.openstack.org/412523 | 09:09 |
| *** derekh has joined #openstack-ironic | 09:14 | |
| *** lucas-afk is now known as lucasagomes | 09:18 | |
| lucasagomes | morning all | 09:19 |
| pas-ha | morning Ironic :) | 09:24 |
| *** ccamacho has quit IRC | 09:27 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Move default image logic from DevStack to Ironic https://review.openstack.org/414227 | 09:30 |
| *** yee379 has quit IRC | 09:30 | |
| *** yee379 has joined #openstack-ironic | 09:30 | |
| *** lindycoder has joined #openstack-ironic | 09:32 | |
| *** moshele has joined #openstack-ironic | 09:32 | |
| *** dtantsur has quit IRC | 09:34 | |
| *** rcernin has quit IRC | 09:34 | |
| *** wznoinsk has quit IRC | 09:35 | |
| *** mmedvede has quit IRC | 09:35 | |
| tuanluong | morning ironic | 09:35 |
| *** wznoinsk has joined #openstack-ironic | 09:35 | |
| *** pcaruana has quit IRC | 09:35 | |
| *** tesseract has quit IRC | 09:35 | |
| tuanluong | morning lucasagomes, pas-ha | 09:35 |
| *** mmedvede has joined #openstack-ironic | 09:36 | |
| openstackgerrit | Merged openstack/bifrost: Fixes to allow services startup after reboot https://review.openstack.org/414600 | 09:37 |
| *** hamzy has quit IRC | 09:37 | |
| *** vsaienk0 has quit IRC | 09:37 | |
| *** slagle has quit IRC | 09:37 | |
| *** hecliunyx has quit IRC | 09:37 | |
| *** bapalm has quit IRC | 09:37 | |
| *** rybridges2 has quit IRC | 09:37 | |
| *** stevemar has quit IRC | 09:37 | |
| *** kencjohnston has quit IRC | 09:37 | |
| *** hecliunyx has joined #openstack-ironic | 09:37 | |
| *** stevemar has joined #openstack-ironic | 09:37 | |
| *** rybridges2 has joined #openstack-ironic | 09:37 | |
| *** slagle has joined #openstack-ironic | 09:37 | |
| *** kencjohnston has joined #openstack-ironic | 09:37 | |
| *** vsaienk0 has joined #openstack-ironic | 09:37 | |
| *** hamzy has joined #openstack-ironic | 09:37 | |
| *** bapalm has joined #openstack-ironic | 09:38 | |
| *** hogepodge has quit IRC | 09:38 | |
| *** yuriyz has quit IRC | 09:38 | |
| *** alineb has quit IRC | 09:38 | |
| *** strigazi has quit IRC | 09:38 | |
| *** SpamapS has quit IRC | 09:38 | |
| *** hogepodge has joined #openstack-ironic | 09:39 | |
| *** strigazi has joined #openstack-ironic | 09:39 | |
| *** openstack has joined #openstack-ironic | 14:28 | |
| jroll | np, thank you! | 14:29 |
| *** joanna has joined #openstack-ironic | 14:31 | |
| jroll | https://twitter.com/alt_kia/status/818609521928998912 <- sigh, thanks intel | 14:32 |
| jroll | tl;dr jtag over usb 3 on skylake | 14:32 |
| *** mpotdar has joined #openstack-ironic | 14:32 | |
| openstackgerrit | Jesse J. Cook proposed openstack/ironic: Add API and conductor methods for [un]rescue https://review.openstack.org/350831 | 14:33 |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Add tests for Payloads with SCHEMAs https://review.openstack.org/418148 | 14:34 |
| *** wajdi has joined #openstack-ironic | 14:35 | |
| *** wajdi_ has joined #openstack-ironic | 14:37 | |
| *** sacharya has joined #openstack-ironic | 14:37 | |
| lucasagomes | jroll, ew, ugly... tho, once one have access to the hardware there's very little that can be done really | 14:37 |
| dtantsur | well, there is still a slight difference between "get my hands on a keyboard without the owner looking" and "quickly insert a small USB device to get machine completely owned" | 14:38 |
| jroll | ^^ | 14:38 |
| *** wajdi has quit IRC | 14:40 | |
| *** fxpester has quit IRC | 14:40 | |
| lucasagomes | right, I would need to read more about jtags to actually see if that would be the case | 14:40 |
| lucasagomes | but in theory, if possible, yeah pretty nasty | 14:41 |
| jroll | well, it gives you kernel/hardware level access | 14:42 |
| jroll | is that enough? :) | 14:42 |
| *** makowals_ has joined #openstack-ironic | 14:42 | |
| *** makowals has quit IRC | 14:44 | |
| *** makowal__ has joined #openstack-ironic | 14:44 | |
| lucasagomes | jroll, yup, that's f'd | 14:45 |
| * lucasagomes was reading the full news at http://www.itnews.com.au/news/intel-debugger-interface-open-to-hacking-via-usb-446889 | 14:46 | |
| openstackgerrit | Dirk Mueller proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/409080 | 14:47 |
| *** makowals_ has quit IRC | 14:47 | |
| *** ralonsoh_ has joined #openstack-ironic | 14:47 | |
| openstackgerrit | Merged openstack/pyghmi: Refactor Lenovo OEM support https://review.openstack.org/416718 | 14:48 |
| *** absubram has joined #openstack-ironic | 14:49 | |
| *** ralonsoh__ has quit IRC | 14:51 | |
| *** tuan_luong has joined #openstack-ironic | 14:53 | |
| *** makowal__ has quit IRC | 14:53 | |
| *** makowals has joined #openstack-ironic | 14:53 | |
| *** lennyb has joined #openstack-ironic | 14:53 | |
| *** links has quit IRC | 14:54 | |
| milan | morning xavierr ! :) | 14:57 |
| *** karthiks has quit IRC | 14:57 | |
| milan | and morning jroll! :) | 14:58 |
| *** jaosorior has joined #openstack-ironic | 15:00 | |
| jroll | \o milan | 15:01 |
| *** Guest90743 is now known as rook | 15:08 | |
| *** karthiks has joined #openstack-ironic | 15:10 | |
| *** steveb has joined #openstack-ironic | 15:11 | |
| *** steveb is now known as Guest23380 | 15:12 | |
| *** Guest23380 has quit IRC | 15:12 | |
| *** steveb811 has joined #openstack-ironic | 15:13 | |
| steveb811 | Hello all, quick question for you; does ironic now support more network types other than the "flat network" option? | 15:13 |
| sambetts | steveb811: as of Newton there is support for other neutron networks, provided you have ml2 support for your hardware | 15:14 |
| *** AlexeyAbashkin has quit IRC | 15:15 | |
| sambetts | steveb811: http://docs.openstack.org/developer/ironic/deploy/multitenancy.html | 15:15 |
| steveb811 | sambets: thank you | 15:15 |
| openstackgerrit | Jesse J. Cook proposed openstack/ironic: Add rescuewait timeout perodic task https://review.openstack.org/353156 | 15:16 |
| *** AlexeyAbashkin has joined #openstack-ironic | 15:16 | |
| *** tuan_luong has quit IRC | 15:17 | |
| *** AlexeyAbashkin has quit IRC | 15:19 | |
| *** steveb811 has quit IRC | 15:19 | |
| vdrok | morning jroll xavierr and rloo ! | 15:23 |
| JayF | lucasagomes: ty for the link, landing that | 15:23 |
| lucasagomes | JayF, cool, ty | 15:23 |
| rloo | hi vdrok! | 15:23 |
| vdrok | and morning JayF :) | 15:23 |
| openstackgerrit | Aline Bousquet proposed openstack/python-ironicclient: Fix multiple ports deletion https://review.openstack.org/418445 | 15:24 |
| *** wajdi has joined #openstack-ironic | 15:24 | |
| *** aparnav has quit IRC | 15:25 | |
| sambetts | dtantsur: is that ^ the expected behaviour? I thought stopping on the first failing request was the expected behaviour | 15:26 |
| *** wajdi_ has quit IRC | 15:27 | |
| dtantsur | sambetts, I think we ended up trying to delete everything. otherwise the outcome depends on listing order, and anyway users I've met prefer this approach. | 15:28 |
| NobodyCam | Good morning ironic'ers | 15:28 |
| amarao | Where I should report bugs in documentation at http://docs.openstack.org/developer/ironic/deploy/multitenancy.html? | 15:28 |
| *** trown is now known as trown|brb | 15:29 | |
| sambetts | dtantsur: ah ok, with a patch like it difficult to know if someone created the bug report and then push the patch without any review, from the gerrit page there is no way to know if the bug is accepted | 15:29 |
| sambetts | valid | 15:29 |
| vdrok | morning NobodyCam | 15:29 |
| sambetts | amarao: https://bugs.launchpad.net/ironic | 15:30 |
| jroll | amarao: https://bugs.launchpad.net/ironic | 15:30 |
| jroll | jinx | 15:30 |
| sambetts | lol | 15:30 |
| vdrok | sambetts: yeah, this is the behaviour we have for nodes for a while already | 15:30 |
| NobodyCam | Morning vdrok | 15:30 |
| amarao | (this one: cleaning_network and provisioning_network should be cleaning_network_uuid, provisioning_network_uuid otherwise there is an exception: Driver NeutronNetwork could not be loaded. Reason: The following [neutron] group configuration options are incorrect, they must be valid UUIDs: cleaning_network_uuid=None, provisioning_network_uuid=None) | 15:30 |
| *** wajdi has quit IRC | 15:30 | |
| dtantsur | amarao, no, these are the right options | 15:31 |
| dtantsur | you should check non-master version of docs. which ironic are you using? | 15:31 |
| dtantsur | * which version of ironic? | 15:31 |
| amarao | newton | 15:31 |
| *** mjura has quit IRC | 15:31 | |
| dtantsur | amarao, here you go: http://docs.openstack.org/developer/ironic/newton/deploy/multitenancy.html | 15:32 |
| dtantsur | notice "newton" after "ironic" in the URL | 15:32 |
| amarao | Oh, thanks. | 15:32 |
| *** aparnav has joined #openstack-ironic | 15:32 | |
| * dtantsur wonders how to make this difference more apparent | 15:32 | |
| sambetts | dtantsur: yeah... thats horrible... is there a way to put in in big letters in the title? | 15:33 |
| amarao | Normal openstack docs has version written on background: http://docs.openstack.org/mitaka/install-guide-ubuntu/ | 15:33 |
| dtantsur | hah, a nice watermark | 15:33 |
| sambetts | haha | 15:33 |
| dtantsur | yeah, this makes a lot of sense, if somebody capable of CSS/Sphinx/whatever can figure it out | 15:33 |
| amarao | And they have nice 'report bug in the docs' link at the top of the page. I really used it when I deployed openstack. | 15:33 |
| TheJulia | Good morning | 15:33 |
| JayF | dtantsur: amarao: In the new install guide, I believe we have a watermark too | 15:34 |
| openstackgerrit | Merged openstack/ironic: Update regenerate-samples.sh api-ref script https://review.openstack.org/414933 | 15:35 |
| dtantsur | morning TheJulia | 15:35 |
| *** links has joined #openstack-ironic | 15:35 | |
| vdrok | good morning TheJulia | 15:35 |
| TheJulia | Anyone know if there were any grenade issues last night? | 15:36 |
| NobodyCam | Good morning TheJulia | 15:37 |
| dtantsur | I've seen a few random failures. Most not on in grenade, but in random jobs on IPA. | 15:37 |
| *** trown|brb is now known as trown | 15:39 | |
| TheJulia | Okay, I've rechecked one that failed oddly with nova attempting to destroy resources during grenade... completely unrelated to the change | 15:41 |
| *** aslezil has joined #openstack-ironic | 15:41 | |
| *** hw_wutianwei has quit IRC | 15:42 | |
| jroll | TheJulia: yeah, we have intermittent timeouts on nova delete at the moment | 15:43 |
| jroll | there's a grenade patch out to increase it from 30 to 45 seconds | 15:43 |
| JayF | has that still not landed? :( | 15:43 |
| TheJulia | I thought that patch landed.. | 15:43 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Update tox envs list https://review.openstack.org/415250 | 15:43 |
| JayF | no core reviews at all, awesome | 15:43 |
| JayF | I'll try to track folks down | 15:43 |
| JayF | https://review.openstack.org/#/c/417406/ fwiw | 15:44 |
| patchbot | patch 417406 - openstack-dev/grenade - Increase instance deletion timeout after upgrade | 15:44 |
| *** mtanino has joined #openstack-ironic | 15:44 | |
| TheJulia | fun, only three cores for grenade | 15:49 |
| *** amotoki has quit IRC | 15:50 | |
| *** Syed__ has joined #openstack-ironic | 15:51 | |
| lucasagomes | yolanda, hi there, any luck on the configdrive partition thingy ? | 15:53 |
| yolanda | hi lucasagomes , i'm debugging that | 15:54 |
| yolanda | i have a tripleo deployed, and i'm updating the code for it | 15:54 |
| JayF | re: grenade fix, https://review.openstack.org/#/c/418455/ is the new patch made up based on feedback from those cores, we have 2x+2 | 15:57 |
| patchbot | patch 418455 - openstack-dev/grenade - Replace now unneeded timeout with --wait on OSC | 15:57 |
| JayF | I presume they're waiting on CI to +A | 15:57 |
| yolanda | lucasagomes, i am trying to use e2label, composing the partition with device + part['number'], after i list the partitions | 15:58 |
| *** rbudden has joined #openstack-ironic | 15:58 | |
| lucasagomes | yolanda, cool, yeah I think that should be the way to go | 15:58 |
| JayF | wait, is the configdrive partition an ext partition? | 15:58 |
| JayF | If so, that's not valid. | 15:58 |
| *** rama_y has joined #openstack-ironic | 15:58 | |
| JayF | Configdrive partitions have to be ISO or vfat per the standard | 15:59 |
| jlvillal | Good morning Ironic :) | 15:59 |
| JayF | and I think e2label only works on ext* filesystems | 15:59 |
| *** rama_y has quit IRC | 15:59 | |
| NobodyCam | morning jlvillal | 15:59 |
| jlvillal | Hi NobodyCam :) | 16:00 |
| yolanda | JayF , in my case it's an ext partition, but of course we need to add max compatibility | 16:00 |
| yolanda | so what shall be the best way? | 16:00 |
| NobodyCam | :) hey hey jlvillal got a free minute to take a look at https://review.openstack.org/#/c/413249/ | 16:00 |
| patchbot | patch 413249 - ironic-specs - Intel Node Manager sensor collection support | 16:00 |
| jlvillal | NobodyCam: I'm in a meeting for the next hour. Lovely way to start the day :) | 16:01 |
| NobodyCam | ugh | 16:01 |
| NobodyCam | +++ | 16:01 |
| JayF | yolanda: I'm just saying, in general, I don't think the code that handles it is different, but cloud-init specifically requires a configdrive to be in ISO (ro) or vfat (rw) formats. The use of ext is unlikely to be the source of the bug, but it's a notable difference from anything we ever test. | 16:01 |
| jroll | JayF: we create an ext2 partition in ironic-lib, fwiw | 16:02 |
| *** amotoki has joined #openstack-ironic | 16:02 | |
| lucasagomes | JayF, I thought e2label would work with ISO or vFAT (tho I have to test it) | 16:02 |
| mariojv | hm, cloud-init documents JayF's requirement as correct http://cloudinit.readthedocs.io/en/latest/topics/datasources/configdrive.html#version-2 | 16:02 |
| *** rpioso has joined #openstack-ironic | 16:02 | |
| JayF | jroll: egad! That's really bad! | 16:02 |
| jroll | ¯\_(ツ)_/¯ | 16:03 |
| JayF | jroll: that will just simply not work at all with windows images | 16:03 |
| jroll | ooo, true | 16:03 |
| JayF | yeah, that's a High/Critical level bug | 16:03 |
| yolanda | mm | 16:03 |
| yolanda | e2label will display or change the filesystem label on the ext2, ext3, or ext4 filesystem located on device. | 16:03 |
| yolanda | that's bad | 16:03 |
| JayF | because it means any !linux images aren't going to work | 16:03 |
| yolanda | JayF, lucasagomes , what about lsblk -o label on the device? | 16:03 |
| dtantsur | jroll, I simple driver-comp-related change, if you have a minute: https://review.openstack.org/#/c/418364/1 | 16:04 |
| patchbot | patch 418364 - ironic - Correct logging of loaded drivers/hardware types/i... | 16:04 |
| jroll | JayF: fyi, the code: https://github.com/openstack/ironic-lib/blob/master/ironic_lib/disk_utils.py#L759 | 16:04 |
| lucasagomes | JayF, you correct, it doesn't yeah | 16:04 |
| *** mtanino has quit IRC | 16:05 | |
| jroll | dtantsur: nice, +2 | 16:05 |
| lucasagomes | yolanda, seems to work with vfat and exts yeah | 16:07 |
| lucasagomes | you can use -P as well to make it easier to parse perhaps | 16:07 |
| lucasagomes | $ sudo lsblk -Po label /dev/loop1p1 | 16:07 |
| lucasagomes | LABEL="test-label" | 16:07 |
| yolanda | ok let me try that one | 16:08 |
| lucasagomes | yolanda, that case as well you may even want to check all partitions at the same time ? Like http://paste.openstack.org/show/594436/ | 16:08 |
| yolanda | yep, iterate through the result, and get the one with config-2 label | 16:09 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-lib: Create configdrive partitions as vfat https://review.openstack.org/418458 | 16:09 |
| JayF | ^^ the fix for the bug https://bugs.launchpad.net/ironic-lib/+bug/1655389 we just identified | 16:10 |
| openstack | Launchpad bug 1655389 in ironic-lib "disk utils creates an ext2 configdrive, only compatible with linux" [High,In progress] - Assigned to Jay Faulkner (jason-oldos) | 16:10 |
| aslezil | git diff | 16:10 |
| JayF | aslezil: No files changed. | 16:10 |
| *** soliosg_ has joined #openstack-ironic | 16:10 | |
| aslezil | sorry | 16:10 |
| JayF | :) | 16:10 |
| *** vgadiraj has joined #openstack-ironic | 16:12 | |
| *** yarkot has joined #openstack-ironic | 16:13 | |
| *** alezil has joined #openstack-ironic | 16:14 | |
| JayF | So is there a reason, even for some of the !configdrive partitions, we're using ext*2* | 16:14 |
| lucasagomes | JayF, not sure, you mean things like ephemeral partition and so on ? | 16:17 |
| JayF | yes | 16:17 |
| JayF | I just noticied in that test you pointed out, that we were creating larger partitions as ext2 | 16:17 |
| JayF | which is really not a reccomended thing to do (have large, non-journaled filesystems) | 16:17 |
| lucasagomes | yeah, it's kinda odd in ironic that ephemeral is actually a partition (I guess nova it's another disk right ?) | 16:18 |
| TheJulia | lucasagomes: afaik yes it is another disk | 16:18 |
| JayF | I mean, what it "is" in real life doesn't matter, but it shows up to the VM as another disk | 16:18 |
| *** rcernin has quit IRC | 16:18 | |
| JayF | mariojv: no release notes in ironic-lib; we'll release note the version bump in ironic/ipa | 16:19 |
| JayF | mariojv: or more likely, we won't, because that won't require a special commit i don't think | 16:19 |
| mariojv | ah, ok | 16:19 |
| lucasagomes | JayF, right... I don't know off the top of my head the reason about the fs choices there | 16:19 |
| *** aparna has joined #openstack-ironic | 16:19 | |
| JayF | I didn't know if there were reasons, related to how iscsi does things, that we did it that way | 16:19 |
| lucasagomes | JayF, but yeah, for another OSs (aka windows) ext* might be a problem | 16:20 |
| JayF | not just that, but for linux | 16:20 |
| *** wajdi has joined #openstack-ironic | 16:20 | |
| JayF | if someone doesn't reformat it as a better fs | 16:20 |
| JayF | you can end up with those hours-long fscks that were common before journaled filesystems were in common use | 16:20 |
| dtantsur | going a bit earlier today, see you tomorrow | 16:20 |
| *** dtantsur is now known as dtantsur|afk | 16:20 | |
| *** baoli has quit IRC | 16:21 | |
| lucasagomes | JayF, ++ but I thought only configdrive used ext2... lemme see the code, /me thought ephemeral was ext3 or ext4 | 16:21 |
| vdrok | good night dtantsur|afk | 16:21 |
| *** rama_y has joined #openstack-ironic | 16:22 | |
| JayF | lucasagomes: you may be right, i think I'm misreading this test | 16:22 |
| lucasagomes | JayF, #default_ephemeral_format = ext4 | 16:22 |
| lucasagomes | yeah, it comes from this ironic config | 16:22 |
| *** vinsh has quit IRC | 16:22 | |
| JayF | awesome | 16:22 |
| JayF | I wonder if we should make configdrive configurable too, eventually | 16:22 |
| *** aparnav has quit IRC | 16:22 | |
| JayF | vfat vs iso | 16:22 |
| JayF | I know downstream, we liked that it was read only, so we liked ISO | 16:23 |
| lucasagomes | JayF, it's configurable via nova right ? | 16:23 |
| JayF | oooooooh, that's a good point | 16:23 |
| JayF | this whole thing is completely for moot | 16:23 |
| JayF | not that we shouldn't fix it, but we don't use the filesystem parted creates | 16:23 |
| lucasagomes | yeah | 16:23 |
| lucasagomes | I think we create a FS so that we can label it | 16:23 |
| lucasagomes | and find it later | 16:23 |
| JayF | that makes a hell of a lot more sense | 16:23 |
| lucasagomes | not sure... but, urgh it's ugly | 16:24 |
| *** amotoki has quit IRC | 16:24 | |
| JayF | yeah, you have to create the fs to label it | 16:24 |
| JayF | well, there are partition labels, but that's not what cloud-init looks for | 16:24 |
| openstackgerrit | Merged openstack/bifrost: Support to wait for node deployment to complete https://review.openstack.org/254245 | 16:24 |
| lucasagomes | JayF, yeah that's a grey area when it comes to MBR vs GPT | 16:24 |
| lucasagomes | so fs labels are easier I guess | 16:25 |
| *** txengr266873468 has quit IRC | 16:25 | |
| jlvillal | lucasagomes: Cool, socket library calls :) | 16:25 |
| alezil | cd | 16:25 |
| jroll | ~ | 16:25 |
| lucasagomes | jlvillal, heh I think it's better. But I would leave it to another patch if possible | 16:26 |
| lucasagomes | sounds unrelated to that change, unless derekh wants to do it in the same patch :-) | 16:26 |
| JayF | lucasagomes: jlvillal: fwiw that's how I did the partial v6 support in certbot | 16:26 |
| jlvillal | lucasagomes: Probably good. It was just getting confusing that giant line of splits and strips and indexing... | 16:26 |
| JayF | lucasagomes: jlvillal: I agree we should do that in a separate patch, and using the socket lib is a good way to decide :) | 16:26 |
| lucasagomes | jlvillal, yeah it's ugly | 16:26 |
| lucasagomes | JayF, ++ | 16:26 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-lib: Create configdrive partitions as vfat https://review.openstack.org/418458 | 16:27 |
| JayF | ^^ unit tests passed locally | 16:27 |
| JayF | although I'll update the bug to be less... dramatic | 16:27 |
| JayF | and lower priority | 16:27 |
| *** jcoufal has quit IRC | 16:27 | |
| jlvillal | JayF: Nice grenade change. Why didn't I think of --wait??? I use it for some of my stuff. | 16:28 |
| lucasagomes | heh | 16:28 |
| JayF | It was dtroyer's idea :) | 16:28 |
| jlvillal | :) | 16:28 |
| derekh | lucasagomes: jlvillal so I'll keep the split as is in the current patch? | 16:29 |
| *** absubram has quit IRC | 16:29 | |
| jlvillal | derekh: Okay, though I'll admit the trailing backslash annoys me ;) | 16:29 |
| *** absubram has joined #openstack-ironic | 16:29 | |
| lucasagomes | derekh, IMHO, split | 16:29 |
| jlvillal | derekh: Will you do the follow on patch for socket library? | 16:29 |
| derekh | jlvillal: yup, will do, | 16:30 |
| jlvillal | derekh: And did you see my comment that the one strip is not needed because of the 'split' after? | 16:30 |
| jlvillal | derekh: Might be able to fit it all on one line. | 16:30 |
| derekh | jlvillal: will take a look now and update the unit tests | 16:30 |
| jlvillal | derekh: Thanks! | 16:30 |
| lucasagomes | JayF, actually, apparently "vfat" is not a valid fstype for parted | 16:32 |
| lucasagomes | https://www.gnu.org/software/parted/manual/html_node/mkpart.html | 16:32 |
| lucasagomes | sudo parted -a optimal -s -- /dev/loop1 mkpart primary vfat 0 25 | 16:33 |
| lucasagomes | parted: invalid token: vfat | 16:33 |
| lucasagomes | fat32 is what you are looking for here | 16:33 |
| JayF | that's silly | 16:33 |
| JayF | why does it have a different name for the filesystem than the linux kernel? | 16:33 |
| JayF | I'll fix it :) | 16:33 |
| lucasagomes | JayF, yeah, consistency is not a thing :-) | 16:34 |
| *** Goneri has quit IRC | 16:35 | |
| *** jcoufal has joined #openstack-ironic | 16:35 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 16:36 | |
| *** amotoki has joined #openstack-ironic | 16:37 | |
| JayF | I should've GNU better than to assume | 16:37 |
| *** amarao has quit IRC | 16:37 | |
| jroll | jlvillal: soliosg: matt treinish wrote this up, it may be helpful to you: https://etherpad.openstack.org/p/tempest-separate-plugin | 16:39 |
| jroll | it lgtm, honestly | 16:39 |
| yolanda | agh, forgot to add lsblk to filters :( | 16:39 |
| lucasagomes | >.<' | 16:40 |
| jroll | jlvillal: see also #openstack-dev | 16:41 |
| *** zackf has joined #openstack-ironic | 16:41 | |
| rloo | sambetts, vsaienk0: would be good if you could update https://review.openstack.org/#/c/404240/ today/soon, it is close... | 16:43 |
| patchbot | patch 404240 - ironic - Add Virtual Network Interface REST APIs | 16:43 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-lib: Create configdrive partitions as vfat https://review.openstack.org/418458 | 16:43 |
| vdrok | rloo: I'll do that in a bit | 16:43 |
| rloo | thx vdrok -- but would it mean you can't +2 it? | 16:44 |
| jroll | rloo: I'm going to review it today as well so if he can't +2 it's fine | 16:44 |
| *** Goneri has joined #openstack-ironic | 16:44 | |
| *** causten has quit IRC | 16:45 | |
| rloo | jroll: sweet, ok vdrok, update away :) | 16:45 |
| *** rbartal has quit IRC | 16:45 | |
| vdrok | rloo: those things that are pointed out do not seem to require significant changes, I can leave myself out of co-authors :) | 16:45 |
| vdrok | but maybe that's cheating :D | 16:45 |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Rolling upgrade procedure documentation https://review.openstack.org/416565 | 16:45 |
| rloo | vdrok: I think it is a bit of cheating. if we had to get it in today i'd say go for it but we don't. we're good since we have another reviewer lined up :D | 16:46 |
| rloo | vdrok: 'go for it' == +2 :) | 16:47 |
| lucasagomes | JayF, I think you are replacing one wrong "vfat" in that patch | 16:47 |
| JayF | egad | 16:47 |
| JayF | *facepalm* | 16:47 |
| * JayF shakes his fist angrily at regular expressions | 16:48 | |
| JayF | that's what I get for trying to do standup + update the patch at the same time | 16:48 |
| mat128 | lucasagomes, JayF: maybe it should be extracted in self-explanatory variables :) | 16:48 |
| JayF | if you don't know what vfat/fat32 are you probably shouldn't be in this code :) | 16:49 |
| JayF | this issue was simply inattention on my part, tbh | 16:49 |
| mat128 | JayF: I meant KERNEL_FS_NAME and PARTED_FS_NAME | 16:49 |
| *** causten has joined #openstack-ironic | 16:49 | |
| lucasagomes | mat128, that's just the filesystem type, vfat == fat32 is virtually the same | 16:50 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-lib: Create configdrive partitions as vfat https://review.openstack.org/418458 | 16:50 |
| * mat128 shrugs | 16:50 | |
| mat128 | I know | 16:50 |
| mat128 | lol | 16:50 |
| mat128 | I was saying, extracting it in 2 variables prevents mass find-replace issues | 16:50 |
| lucasagomes | oh, right on... yeah, actually, I really would like to use some library for the disk partitioning. Something like blivet or even parted (have to see if it still need root access) | 16:51 |
| lucasagomes | that would simplify the code big time | 16:52 |
| *** links has quit IRC | 16:52 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Fix take over for ACTIVE nodes in PXEBoot https://review.openstack.org/404858 | 16:54 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Fix take over of ACTIVE nodes in AgentDeploy https://review.openstack.org/408667 | 16:59 |
| yolanda | lucasagomes, it worked | 17:00 |
| yolanda | http://paste.openstack.org/show/594449/ | 17:00 |
| *** hamzy has quit IRC | 17:00 | |
| lucasagomes | yolanda, w00t! | 17:00 |
| yolanda | i'm just thinking in searching for LABEL="config-2" in the output | 17:00 |
| yolanda | as we don't need info of the partition, just check if exists | 17:01 |
| lucasagomes | yolanda, yeah sounds simple and good | 17:01 |
| *** strigazi is now known as strigazi_AFK | 17:02 | |
| lucasagomes | yolanda, ty for fixing it btw | 17:02 |
| yolanda | ah, no, it needs the partition | 17:02 |
| lucasagomes | yolanda, you can pass -o name,label | 17:02 |
| yolanda | yep, and just parse and retrieve the partition name | 17:03 |
| lucasagomes | yup | 17:03 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add Virtual Network Interface REST APIs https://review.openstack.org/404240 | 17:10 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Make _send_sensors_data concurrent https://review.openstack.org/407429 | 17:11 |
| *** rbudden has quit IRC | 17:13 | |
| *** david-lyle has joined #openstack-ironic | 17:13 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: TMP: Verify attach/detach https://review.openstack.org/414160 | 17:14 |
| *** vishwanathj has joined #openstack-ironic | 17:14 | |
| lucasagomes | folks, if you have some time mind taking a look at https://review.openstack.org/#/c/414227/ ? | 17:14 |
| patchbot | patch 414227 - ironic - Move default image logic from DevStack to Ironic | 17:14 |
| * lucasagomes wants to get it merged soon to continue with the UEFi stuff | 17:15 | |
| *** e0ne has quit IRC | 17:20 | |
| *** trown is now known as trown|lunch | 17:20 | |
| *** deray has joined #openstack-ironic | 17:20 | |
| *** deray_ has joined #openstack-ironic | 17:20 | |
| yolanda | lucasagomes, so i need to test and will send a patch | 17:20 |
| *** deray_ has quit IRC | 17:20 | |
| yolanda | it will also need a change to enable filter... that will come from puppet i guess? | 17:21 |
| yolanda | or what is populating that filters? | 17:21 |
| lucasagomes | yolanda, the rootwrap filters ? It's just a file in the ironic tree (or ironic-lib, have to look) | 17:21 |
| *** morgabra_ is now known as morgabra | 17:21 | |
| *** morgabra has joined #openstack-ironic | 17:21 | |
| yolanda | ah | 17:21 |
| deray | harlowja, hi | 17:21 |
| yolanda | better | 17:21 |
| lucasagomes | yolanda, yeah should be straight fwd | 17:22 |
| yolanda | k, i see it | 17:22 |
| * yolanda is learning a lot about Ironic | 17:22 | |
| lucasagomes | :-) | 17:23 |
| *** ijw_ has quit IRC | 17:23 | |
| *** pcaruana has quit IRC | 17:25 | |
| yolanda | taking a break while it deploys | 17:25 |
| lucasagomes | cool, I'm calling it a day | 17:27 |
| lucasagomes | have a great evening all | 17:27 |
| *** lucasagomes is now known as lucas-afk | 17:27 | |
| sambetts | night lucas-afk | 17:27 |
| *** krtaylor has quit IRC | 17:28 | |
| openstackgerrit | Joanna Taryma proposed openstack/ironic-specs: Clean invalid conflict response usages and unneccessary retries https://review.openstack.org/414194 | 17:31 |
| *** tesseract has quit IRC | 17:31 | |
| openstackgerrit | Jesse J. Cook proposed openstack/ironic: Add rescuewait timeout perodic task https://review.openstack.org/353156 | 17:32 |
| vdrok | good night lucas-afk | 17:32 |
| *** rama_y has quit IRC | 17:33 | |
| *** baoli has joined #openstack-ironic | 17:33 | |
| *** ccamacho has quit IRC | 17:34 | |
| deray | Hello all Ironicers !! | 17:34 |
| NobodyCam | Good Morning deray | 17:35 |
| *** vishwanathj has quit IRC | 17:35 | |
| NobodyCam | night lucas-afk | 17:35 |
| deray | NobodyCam, g'morning :) | 17:35 |
| NobodyCam | :) | 17:35 |
| deray | I have a question regarding the ``futurist`` library .. | 17:36 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Serve boot configs from Ironic API https://review.openstack.org/369438 | 17:36 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Implement Driver API for dynamic boot config https://review.openstack.org/394398 | 17:36 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Add RPC API to get the boot config https://review.openstack.org/394399 | 17:36 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Allow not writing pxe configs to disk https://review.openstack.org/394397 | 17:36 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Convert iPXE boot config to proper Jinja template https://review.openstack.org/393462 | 17:36 |
| deray | I am planning to use the futurist's periodics.PeriodicWorker | 17:37 |
| *** vishwanathj has joined #openstack-ironic | 17:38 | |
| deray | I am planning to add a list of periodic callables to the worker and start .. and my strategy for ending the periodicity of each periodic func is .. when it meets some condition | 17:40 |
| *** lekha has quit IRC | 17:40 | |
| deray | but here only I am stuck .. I want to get a way by which I can keep reducing the worker.callables based on each one of their condition dynamically | 17:41 |
| sambetts | deray: If I understand you correctly, you want a periodic task that can remove itself from the pool of periodic tasks when it finishes? | 17:45 |
| *** athomas has quit IRC | 17:45 | |
| deray | sambetts, I couldn't have described it more finely :) | 17:45 |
| *** vishwanathj has quit IRC | 17:46 | |
| deray | sambetts, moreover .. as a matter of fact, all my periodic tasks are essentially the same callable with different params | 17:47 |
| *** dsneddon_afk is now known as dsneddon | 17:48 | |
| sambetts | deray: it seems like periodic tasks don't fit the use case very well, can I ask why you need the periodic to remove itself?, normally a periodic is a task that runs at set intervals forever | 17:48 |
| sambetts | deray: do the callables need to be called over and over again until they meet a condition? | 17:53 |
| deray | sambetts, I have a task at hand wherein I have to run multiple (may be short or long running tasks) on the baremetal and status of running each task is saved as part of some ilo variable.. Now I need to poll from my code for the status (constrained here of not having any other good mechanism of getting the status, e.g. eventing or any querying api) | 17:55 |
| deray | sambetts, yes exactly | 17:55 |
| openstackgerrit | Derek Higgins proposed openstack/ironic-python-agent: Advertise the correct address when using IPv6 https://review.openstack.org/411817 | 17:56 |
| openstackgerrit | Derek Higgins proposed openstack/ironic-python-agent: iscsi: Listen for connections on both IPv4 and IPv6 ports https://review.openstack.org/411819 | 17:56 |
| openstackgerrit | Derek Higgins proposed openstack/ironic-python-agent: Agent: Listen for connections on both IPv4 and IPv6 ports https://review.openstack.org/411818 | 17:56 |
| *** Nisha_Agarwal has quit IRC | 17:57 | |
| deray | sambetts, and as such when I get the status of each task (be it failed/success) I want to stop polling for that task .. and keep on continuing with the polling of other tasks | 17:57 |
| deray | whose status results are yet to be written .. | 17:57 |
| deray | Altho I can write my own handcrafted logic to achieve this .. but the executor functionalities like futurist.ThreadPoolExecutor of PeriodicWorker is very catch for me .. | 17:58 |
| deray | s/catch/catchy | 17:59 |
| *** derekh has quit IRC | 17:59 | |
| *** ijw has joined #openstack-ironic | 18:01 | |
| *** athomas has joined #openstack-ironic | 18:01 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 18:02 | |
| *** krtaylor has joined #openstack-ironic | 18:02 | |
| deray | sambetts, how I wish I could have an API like ``remove`` just like the ``add`` here: https://github.com/openstack/futurist/blob/master/futurist/periodics.py#L771 | 18:03 |
| sambetts | deray: so I understand the use case much better now, does the rest of your ironic code need to wait for polling for the status to finish and get the result?? | 18:03 |
| *** ralonsoh_ has quit IRC | 18:04 | |
| deray | sambetts, yes .. I need to wait and keep polling for all the other not-yet-got-status-tasks to be done | 18:05 |
| deray | sambetts, essentially as mentioned earlier all my periodic tasks are essentially the same callable, e.g. ``is_ilo_blob_updated_with_task_status`` with different params - task1_status_string, task2_status_string etc. | 18:07 |
| alezil | root | 18:07 |
| JayF | alezil: password: | 18:09 |
| *** AlexeyAbashkin has joined #openstack-ironic | 18:09 | |
| sambetts | deray: ok, so I have a suggestion which I think fits in better with your use case, the idea would be that you summit tasks against the conductors green thread worker pool, and then use http://docs.openstack.org/developer/futurist/api.html#futurist.waiters.wait_for_all to wait at the point you need to wait to get the results | 18:11 |
| sambetts | deray: the trick lays in the code in the callable, esentially you write a "while True" loop inside it, have it check for the status, and if its what you need then you return the result, and if its not you do an green sleep to release the thread's resources for others to run | 18:14 |
| *** Nisha_Agarwal has quit IRC | 18:16 | |
| *** rama_y has joined #openstack-ironic | 18:18 | |
| deray | sambetts, give me some time to assimilate your suggestion .. btw I am planning to use this as part of proliantutils, our vendor driver code, and with Thread for each periodic task and __not__ Green thread | 18:18 |
| sambetts | deray: ah ok, I thought you were using this directly in the Ironic driver, if its in the library then its a bit different, I would create a new futurist.ThreadPoolExecutor, and then summit the tasks to it and use the wait_for_all to wait for them all to finish and collect the results | 18:22 |
| deray | sambetts, okay .. is there any way I can run those tasks at certain predefined interval? | 18:24 |
| *** rbudden has joined #openstack-ironic | 18:25 | |
| *** amoralej is now known as amoralej|off | 18:26 | |
| deray | sambetts, as such I am **not** planning to wait till all of them finishes.. I am planning to make each status-checking jobs pure-play asynchronous jobs which will invoke some ``cb`` ( task.trigger(task.name + '-error') || task.trigger(task.name + '-success') ) once done | 18:27 |
| *** milan has quit IRC | 18:28 | |
| *** e0ne has joined #openstack-ironic | 18:39 | |
| *** [1]cdearborn has joined #openstack-ironic | 18:42 | |
| *** cdearborn has quit IRC | 18:42 | |
| *** trown|lunch is now known as trown | 18:44 | |
| *** bdeschenes has joined #openstack-ironic | 18:46 | |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui: Create base-port module to support create and update operations https://review.openstack.org/418520 | 18:51 |
| *** dyasny has quit IRC | 19:00 | |
| deray | sambetts, just to reiterate my question on futurist.ThreadPoolExecutor, Is there any way I can run the thread tasks at certain periodic interval? is ``time.sleep(periodic_interval)`` the answer? | 19:03 |
| *** d0ugal has quit IRC | 19:04 | |
| *** aparna has quit IRC | 19:13 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic-lib: Fix retrieval of config-2 existing partition https://review.openstack.org/417022 | 19:13 |
| *** dsneddon is now known as dsneddon_afk | 19:13 | |
| *** sacharya_ has joined #openstack-ironic | 19:14 | |
| *** sacharya has quit IRC | 19:16 | |
| *** moshele has joined #openstack-ironic | 19:18 | |
| *** e0ne has quit IRC | 19:19 | |
| *** ppiela has joined #openstack-ironic | 19:19 | |
| *** d0ugal has joined #openstack-ironic | 19:20 | |
| *** milan has joined #openstack-ironic | 19:20 | |
| *** strigazi_AFK is now known as strigazi | 19:21 | |
| *** deray_ has joined #openstack-ironic | 19:22 | |
| *** deray_ has quit IRC | 19:25 | |
| *** deray_ has joined #openstack-ironic | 19:25 | |
| *** deray has quit IRC | 19:26 | |
| *** deray has joined #openstack-ironic | 19:26 | |
| *** deray_ has joined #openstack-ironic | 19:26 | |
| *** deray_ has quit IRC | 19:26 | |
| deray | sambetts, thank you :) I believe I can use your suggestion of employing futurist.ThreadPoolExecutor and waiter.wait_for_all to finally put the close in of the multiple asynchronous tasks | 19:29 |
| vdrok | if anyone wondering about failing multitenant jobs, the reason is https://github.com/openstack-dev/devstack/commit/eede9ddb1d0f0970672f02bcdfb33923603e65e7 | 19:30 |
| vdrok | supposed fixes are https://review.openstack.org/418548 and https://review.openstack.org/418542 | 19:30 |
| patchbot | patch 418548 - networking-generic-switch - Remove deprecated Q_PLUGIN_EXTRA_CONF_* vars | 19:30 |
| patchbot | patch 418542 - openstack-infra/project-config - Remove deprecated neutron options | 19:30 |
| jroll | oh that's fun | 19:30 |
| jroll | thanks vdrok | 19:31 |
| vdrok | me and pas-ha may leave soon, if we won't be able to fix it today, some eyes on that would be appreciated :) | 19:31 |
| *** strigazi is now known as strigazi_AFK | 19:32 | |
| *** pcaruana has joined #openstack-ironic | 19:37 | |
| *** ijw has quit IRC | 19:37 | |
| *** dyasny has joined #openstack-ironic | 19:38 | |
| deray | TheJulia, hi | 19:43 |
| deray | TheJulia, will you be able to have a relook at: https://review.openstack.org/#/c/207337/ anytime soon? | 19:45 |
| patchbot | patch 207337 - ironic-specs - EFI iSCSI boot for ProLiant hardware | 19:45 |
| *** AlexeyAbashkin has quit IRC | 19:46 | |
| *** dsneddon_afk has quit IRC | 19:46 | |
| *** anish has quit IRC | 19:46 | |
| *** anish has joined #openstack-ironic | 19:46 | |
| *** mguz has joined #openstack-ironic | 19:52 | |
| *** mrtenio has quit IRC | 19:53 | |
| mguz | hi all. using ironic in liberty. looking for a way to boot specific servers in a pool | 19:53 |
| mguz | don't want to create a unique flavor for each server as we have 100s of servers | 19:54 |
| mguz | i thought it might work using fixed_ips tied to mac-addresses in neutron, but it doesn't seem to | 19:55 |
| *** vinsh has joined #openstack-ironic | 19:56 | |
| *** vinsh has quit IRC | 19:57 | |
| *** ijw has joined #openstack-ironic | 19:57 | |
| *** shobhar_ has joined #openstack-ironic | 19:57 | |
| *** makowals has quit IRC | 19:59 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 20:05 | |
| *** ChubYann has joined #openstack-ironic | 20:05 | |
| mat128 | mguz: your nodes can have a specific capability that flavors will match. It doesnt have to be exact specs. | 20:13 |
| *** sacharya_ has quit IRC | 20:14 | |
| *** sacharya has joined #openstack-ironic | 20:15 | |
| mguz | @mat128: how would that allow me to select a specific baremetal node to boot? | 20:16 |
| mat128 | mguz: if you want to point to a specific one, you can use the availability zone hack | 20:16 |
| mat128 | mguz: http://docs.openstack.org/admin-guide/cli-nova-specify-host.html | 20:16 |
| mguz | is there a document on that i can read? | 20:16 |
| mguz | thanks | 20:16 |
| *** deray has quit IRC | 20:23 | |
| *** AlexeyAbashkin has quit IRC | 20:27 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 20:50 | |
| *** bfournie has quit IRC | 20:51 | |
| *** bfournie has joined #openstack-ironic | 20:54 | |
| jlvillal | Looks like the grenade fix should land soon :) I hope that helps most of our issues. | 21:02 |
| *** sambetts is now known as sambetts|afk | 21:02 | |
| *** Goneri has quit IRC | 21:02 | |
| *** dyasny has quit IRC | 21:02 | |
| rloo | jroll: wrt attach/detach, it is Tuesday. I'd like if we could get that ironic code done by end of this week, as well as have nova code updated/ready by then. Wondering if we might want to meet tomorrow AM to go over/review. The big issue now (for me) is the CLI part: https://review.openstack.org/#/c/364420/ | 21:02 |
| patchbot | patch 364420 - python-ironicclient - Add interface attach/detach support | 21:02 |
| rloo | sambetts|afk, vsaienk0, vdrok: ^^ if you're around. I will try to catch you tomorrow. | 21:03 |
| jroll | rloo: interesting | 21:04 |
| jroll | rloo: should be an easy fix, change up vif-id to vif-info or something | 21:05 |
| rloo | jroll: fix that patch, or fix it later if we want vif-info? | 21:05 |
| jroll | rloo: that patch | 21:05 |
| jroll | only because backwards compat imo | 21:05 |
| rloo | jroll: i can see the case now where users just want to specify the id | 21:06 |
| jroll | yeah | 21:06 |
| * jroll looks at schema again | 21:06 | |
| rloo | jroll: ok, that is fine with me too. to fix now and do json. i think we shoudl have spent a bit more time wrt what vif-info might be. | 21:06 |
| jroll | hrm, it requires an id | 21:06 |
| jroll | maybe it's fine now | 21:06 |
| TheJulia | NobodyCam: re-reviewed that spec you pinged me about. | 21:06 |
| rloo | yeah, the id is the only thing we use now. we don't save the vif-info :-( | 21:06 |
| jroll | or what if... | 21:06 |
| NobodyCam | TheJulia: awesome thank you :) | 21:07 |
| jroll | --vif:id foo --vif:other_thing bar | 21:07 |
| jroll | and those are all top level keys | 21:07 |
| *** pas-ha_ has joined #openstack-ironic | 21:07 | |
| jroll | nova does a thing like that, --nic:network_id, --nic:port_id, etc | 21:07 |
| jroll | something like that | 21:07 |
| *** pas-ha has quit IRC | 21:08 | |
| *** pas-ha_ is now known as pas-ha | 21:08 | |
| rloo | jroll: OH, is that OSC? | 21:08 |
| *** eglute has quit IRC | 21:08 | |
| *** eglute has joined #openstack-ironic | 21:08 | |
| jroll | rloo: novaclient | 21:08 |
| jroll | idk what they do for osc | 21:08 |
| *** jjohnson2 has quit IRC | 21:09 | |
| rloo | jroll: i'm more worried about osc since that is the future. | 21:09 |
| rloo | jroll: for python API, i'm guessing we want vif-info as a dict? | 21:09 |
| *** jjohnson2 has joined #openstack-ironic | 21:09 | |
| jlvillal | rloo: Would 'vif-id' be the same as 'vif-uuid' ? | 21:10 |
| jlvillal | If there was such a thing. Basically is it a UUID | 21:10 |
| rloo | jlvillal: no, not the same. a VIF ID can be a uuid or a name/string. | 21:10 |
| jroll | rloo: probably, or kwargs | 21:10 |
| jlvillal | rloo: Thanks | 21:11 |
| rloo | jlvillal: i think it was coded in ironic to use the same code as a node's ident | 21:11 |
| jlvillal | makes sense | 21:11 |
| jlvillal | rloo: I was just noticing this comment https://review.openstack.org/#/c/364420/7/ironicclient/v1/resource_fields.py | 21:11 |
| patchbot | patch 364420 - python-ironicclient - Add interface attach/detach support | 21:11 |
| rloo | jlvillal: i hadn't even gotten that far in the review :) | 21:12 |
| rloo | jlvillal: it should be s/id/vif_id/ I think but need to wait til we figure out the API/CLIs first :) | 21:13 |
| jlvillal | rloo: Okay | 21:13 |
| rloo | jlvillal: the api-ref should describe what a vif id is. there's a patch for that which i haven't yet looked at :) | 21:14 |
| *** zackf has quit IRC | 21:16 | |
| *** dyasny has joined #openstack-ironic | 21:19 | |
| *** srobert has joined #openstack-ironic | 21:22 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: WIP: API changes for driver-composition https://review.openstack.org/417970 | 21:22 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Expose default interface calculation from driver_factory https://review.openstack.org/418573 | 21:22 |
| TheJulia | Is it just me, or are the multimode jobs broken? | 21:24 |
| rloo | TheJulia: broken. | 21:24 |
| rloo | TheJulia: vdrok mentioned it earlier today in irc. | 21:24 |
| TheJulia | Ahh, I didn't see. Errand and then a meeting :( | 21:24 |
| rloo | TheJulia: 14:30 vdrok: if anyone wondering about failing multitenant jobs, the reason is https://github.com/openstack-dev/devstack/commit/eede9ddb1d0f0970672f02bcdfb33923603e65e7 | 21:25 |
| rloo | TheJulia: maybe we need to update the whiteboard | 21:25 |
| rloo | TheJulia: i haven't really been paying attention | 21:25 |
| *** srobert_ has quit IRC | 21:25 | |
| * TheJulia updates the whiteboard | 21:25 | |
| rloo | thx TheJulia | 21:25 |
| *** zackf has joined #openstack-ironic | 21:26 | |
| jlvillal | rloo: Is there a proposed fix? | 21:26 |
| jlvillal | Ah I see: supposed fixes are https://review.openstack.org/418548 and https://review.openstack.org/418542 | 21:27 |
| patchbot | patch 418548 - networking-generic-switch - Remove deprecated Q_PLUGIN_EXTRA_CONF_* vars | 21:27 |
| patchbot | patch 418542 - openstack-infra/project-config - Remove deprecated neutron options | 21:27 |
| TheJulia | looks like the networking-generic-switch one is abandoned :\ | 21:28 |
| jlvillal | TheJulia: huh? | 21:28 |
| vdrok | TheJulia: this one is not https://review.openstack.org/418548 | 21:28 |
| patchbot | patch 418548 - networking-generic-switch - Remove deprecated Q_PLUGIN_EXTRA_CONF_* vars | 21:28 |
| vdrok | trying to figure out why one of the jobs failed | 21:28 |
| TheJulia | okay, cool | 21:29 |
| jlvillal | vdrok: Thanks for staying up late! :) | 21:29 |
| vdrok | morning jlvillal :) | 21:29 |
| jlvillal | 11:29PM to be exact. Ouch :( | 21:29 |
| * jlvillal hopes vdrok gets to work after 2PM ;) | 21:29 | |
| vdrok | heh, nope unfortunately :) that's kind of our bad (generic switch) that we've missed this deprecation | 21:31 |
| *** AlexeyAbashkin has quit IRC | 21:31 | |
| *** rbudden has quit IRC | 21:31 | |
| jlvillal | vdrok: Hard to keep up with all the changes/information | 21:31 |
| rloo | jroll: wrt rolling upgrades, i added you to https://review.openstack.org/#/c/407491/. I'm concerned about the release_mappings.py and how easy they will be to maintain (see my comments) | 21:32 |
| patchbot | patch 407491 - ironic - Add RPC and object version pinning | 21:32 |
| jroll | rloo: ok, thanks | 21:33 |
| * jroll stars | 21:33 | |
| rloo | jlvillal: ^^ if you're interested | 21:33 |
| jlvillal | rloo: thanks | 21:33 |
| vdrok | ok, seems like generic switch does not do ks v3 in its exercise.py | 21:40 |
| *** AlexeyAbashkin has joined #openstack-ironic | 21:41 | |
| *** AlexeyAbashkin has quit IRC | 21:47 | |
| *** trown is now known as trown|outtypewww | 21:47 | |
| *** krtaylor has quit IRC | 21:47 | |
| *** pcaruana has quit IRC | 21:48 | |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add docs about creating release note when metrics change https://review.openstack.org/418589 | 21:49 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: WIP: API changes for driver-composition https://review.openstack.org/417970 | 21:57 |
| jroll | so much code | 21:58 |
| *** rama_y has quit IRC | 21:59 | |
| *** dprince has quit IRC | 22:01 | |
| *** shobhar_ has quit IRC | 22:03 | |
| rloo | jroll: you mean, you're making more work for us by putting up large patches? :) | 22:04 |
| jroll | rloo: not that large! | 22:04 |
| jroll | +257-28 | 22:05 |
| jroll | I've been splitting where I can but I don't want 3 API versions for one feature :/ | 22:05 |
| rloo | jroll: it isn't the quantity, it is the quality :) | 22:05 |
| rloo | jroll: i wouldn't mind one patch with 1000 lines of code, if i just had to review once and approve! | 22:05 |
| jroll | rloo: quality guaranteed here :D | 22:06 |
| * jroll will have it in WIP until it's ready | 22:06 | |
| rloo | jroll: rrrrrrrrrright | 22:06 |
| *** priteau has quit IRC | 22:08 | |
| jlvillal | JayF: Probably https://review.openstack.org/418455 should be abandoned now | 22:08 |
| patchbot | patch 418455 - openstack-dev/grenade - Replace now unneeded timeout with --wait on OSC | 22:08 |
| jlvillal | Since other similar patch has merged | 22:09 |
| *** milan has quit IRC | 22:11 | |
| *** Goneri has joined #openstack-ironic | 22:13 | |
| vdrok | woo, generic switch works now! fixing a couple of nits | 22:16 |
| *** Sukhdev has joined #openstack-ironic | 22:19 | |
| *** moshele has quit IRC | 22:20 | |
| vdrok | jroll: would it be better to depend on project-config change or leave the empty values here https://review.openstack.org/#/c/418548/3/devstack/plugin.sh for deprecated options, if there is a need for working gate right now? | 22:20 |
| patchbot | patch 418548 - networking-generic-switch - Remove deprecated Q_PLUGIN_EXTRA_CONF_* vars | 22:20 |
| vdrok | Just don't know if any infra cores are around to merge https://review.openstack.org/418542 | 22:23 |
| patchbot | patch 418542 - openstack-infra/project-config - Remove deprecated neutron options | 22:23 |
| JayF | vdrok: depends-on project-config doesn't really work | 22:26 |
| vdrok | JayF: well, that's just to ensure that we remove them from the job before that gets merged, if I'll remove those empty options before project config fix the gates will still be broken | 22:26 |
| JayF | vdrok: I'm OK w/filing a bug and removing them in a followup | 22:32 |
| JayF | vdrok: they won't instabreak when project-config lands, right? It's just an ugly thing to cleanup? | 22:32 |
| vdrok | JayF: it seem so, yeah | 22:32 |
| vdrok | ok, doing now | 22:32 |
| *** jheroux has quit IRC | 22:33 | |
| *** krtaylor has joined #openstack-ironic | 22:33 | |
| *** jcoufal has quit IRC | 22:36 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Enable OSProfiler support in Ironic https://review.openstack.org/358142 | 22:37 |
| vdrok | patch on the way to merge, gate should be in an hour or so | 22:39 |
| vdrok | good night everyone | 22:39 |
| joanna | good night, vdrok :) | 22:39 |
| NobodyCam | night vdrok | 22:40 |
| JayF | ty vdrok nn | 22:42 |
| *** AnxiousGarlic has quit IRC | 22:47 | |
| openstackgerrit | Jesse J. Cook proposed openstack/ironic: Add rescuewait timeout periodic task https://review.openstack.org/353156 | 22:50 |
| * jroll is over today, see y'all tomorrow | 22:54 | |
| jroll | PROMISE that I will review attach/detach stuff then. too many meetings today. sigh. | 22:54 |
| *** bfournie has quit IRC | 22:54 | |
| openstackgerrit | Merged openstack/ironic: Add docs about creating release note when metrics change https://review.openstack.org/418589 | 22:54 |
| jroll | only 30 minutes tomorrow :) | 22:54 |
| jroll | nn | 22:54 |
| *** srobert has quit IRC | 22:57 | |
| TheJulia | goodnight jroll | 22:57 |
| *** Sukhdev has quit IRC | 22:58 | |
| *** wajdi has quit IRC | 22:59 | |
| NobodyCam | night jroll | 23:07 |
| *** rbudden has joined #openstack-ironic | 23:16 | |
| *** aweeks has quit IRC | 23:21 | |
| *** BadCub has quit IRC | 23:22 | |
| *** sergek has quit IRC | 23:22 | |
| *** cppforlife_ has quit IRC | 23:23 | |
| *** BadCub has joined #openstack-ironic | 23:24 | |
| *** [1]cdearborn has quit IRC | 23:26 | |
| *** bfournie has joined #openstack-ironic | 23:27 | |
| *** jaosorior has quit IRC | 23:27 | |
| *** shobhar_ has joined #openstack-ironic | 23:28 | |
| *** siva has joined #openstack-ironic | 23:32 | |
| *** shobhar_ has quit IRC | 23:34 | |
| *** sergek has joined #openstack-ironic | 23:38 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!