| *** jvgrant has quit IRC | 00:00 | |
| *** testing1234 has quit IRC | 00:03 | |
| *** kragniz1 has joined #openstack-ironic | 00:03 | |
| *** hoangcx2 has quit IRC | 00:16 | |
| *** hoangcx has joined #openstack-ironic | 00:16 | |
| *** baoli has joined #openstack-ironic | 00:17 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/354453 | 00:19 |
|---|---|---|
| *** joprovost has quit IRC | 00:30 | |
| *** rajinir has quit IRC | 00:35 | |
| *** aNupoisc has quit IRC | 00:37 | |
| *** hamzy has joined #openstack-ironic | 00:38 | |
| *** syed_ has quit IRC | 00:42 | |
| *** sdake has joined #openstack-ironic | 00:46 | |
| openstackgerrit | Merged openstack/ironic: Remove unused [pxe]disk_devices option https://review.openstack.org/354308 | 00:51 |
| *** davidlenwell has quit IRC | 00:53 | |
| *** davidlenwell has joined #openstack-ironic | 01:05 | |
| *** xuao has joined #openstack-ironic | 01:06 | |
| *** vishwanathj has quit IRC | 01:10 | |
| *** baoli has quit IRC | 01:13 | |
| *** jaypipes has quit IRC | 01:15 | |
| openstackgerrit | jxiaobin proposed openstack/ironic-python-agent: Deactivate LVM before write image to disk https://review.openstack.org/354440 | 01:26 |
| *** baoli has joined #openstack-ironic | 01:29 | |
| *** vishwanathj has joined #openstack-ironic | 01:32 | |
| *** zhangjl has joined #openstack-ironic | 01:33 | |
| openstackgerrit | Teng Fei proposed openstack/ironic-inspector: Modify the SQL's string from double quote to single quote. https://review.openstack.org/347795 | 01:33 |
| *** bvandewa_ has quit IRC | 01:46 | |
| *** sdake has quit IRC | 02:01 | |
| *** PollyZ has joined #openstack-ironic | 02:04 | |
| openstackgerrit | Tang Chen proposed openstack/ironic: Split API root unit and functional tests https://review.openstack.org/348965 | 02:05 |
| openstackgerrit | Tang Chen proposed openstack/ironic: Move api driver tests to functional dir https://review.openstack.org/350365 | 02:05 |
| openstackgerrit | Tang Chen proposed openstack/ironic: Prepare for functional tests https://review.openstack.org/349421 | 02:05 |
| *** hoangcx has quit IRC | 02:07 | |
| *** sdake has joined #openstack-ironic | 02:13 | |
| *** bvandewa has joined #openstack-ironic | 02:17 | |
| *** sdake has quit IRC | 02:34 | |
| *** hoangcx has joined #openstack-ironic | 02:40 | |
| *** rloo has quit IRC | 02:42 | |
| *** bvandewa has quit IRC | 02:50 | |
| *** bvandewa has joined #openstack-ironic | 02:52 | |
| *** sdake has joined #openstack-ironic | 02:54 | |
| *** betherly has quit IRC | 02:55 | |
| *** francoblanco has quit IRC | 02:55 | |
| *** tdurakov has quit IRC | 02:56 | |
| *** ijw has quit IRC | 02:59 | |
| *** betherly has joined #openstack-ironic | 02:59 | |
| *** francoblanco has joined #openstack-ironic | 02:59 | |
| *** tdurakov has joined #openstack-ironic | 03:00 | |
| *** amotoki has joined #openstack-ironic | 03:02 | |
| *** hoangcx has quit IRC | 03:03 | |
| *** bvandewa has quit IRC | 03:06 | |
| *** bvandewa has joined #openstack-ironic | 03:10 | |
| *** PollyZ has quit IRC | 03:10 | |
| *** amotoki has quit IRC | 03:11 | |
| *** PollyZ has joined #openstack-ironic | 03:11 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic-python-agent: Minor updates to metrics documentation https://review.openstack.org/354490 | 03:13 |
| *** amotoki has joined #openstack-ironic | 03:22 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic-inspector: Add API for listing all introspection statuses https://review.openstack.org/344921 | 03:23 |
| *** sdake has quit IRC | 03:24 | |
| *** amotoki has quit IRC | 03:31 | |
| *** bvandewa has quit IRC | 03:32 | |
| *** links has joined #openstack-ironic | 03:33 | |
| *** amotoki has joined #openstack-ironic | 03:35 | |
| *** baoli has quit IRC | 03:43 | |
| *** hoangcx has joined #openstack-ironic | 03:51 | |
| *** vmud213 has joined #openstack-ironic | 03:52 | |
| *** sdake has joined #openstack-ironic | 03:54 | |
| *** hoangcx has quit IRC | 03:57 | |
| *** amotoki has quit IRC | 04:01 | |
| *** vishwanathj has quit IRC | 04:06 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic-inspector: Allow using introspection data in condition's 'value' field https://review.openstack.org/345535 | 04:07 |
| *** rbudden has joined #openstack-ironic | 04:08 | |
| *** Sukhdev has joined #openstack-ironic | 04:09 | |
| *** amotoki has joined #openstack-ironic | 04:09 | |
| *** amotoki has quit IRC | 04:10 | |
| *** amotoki has joined #openstack-ironic | 04:11 | |
| *** PollyZ has quit IRC | 04:11 | |
| *** psachin has joined #openstack-ironic | 04:15 | |
| *** fragatina has quit IRC | 04:16 | |
| *** PollyZ has joined #openstack-ironic | 04:20 | |
| *** vishwanathj has joined #openstack-ironic | 04:26 | |
| *** vishwanathj has quit IRC | 04:26 | |
| openstackgerrit | Yuiko Takada Mori proposed openstack/ironic: Add node serial console documentation https://review.openstack.org/293872 | 04:33 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/354453 | 04:34 |
| *** hshiina has joined #openstack-ironic | 04:37 | |
| *** PollyZ has quit IRC | 04:41 | |
| *** Sukhdev has quit IRC | 04:54 | |
| *** fragatina has joined #openstack-ironic | 04:58 | |
| *** fragatina has quit IRC | 05:03 | |
| *** hoangcx has joined #openstack-ironic | 05:03 | |
| *** hoangcx has quit IRC | 05:06 | |
| *** vishwanathj has joined #openstack-ironic | 05:29 | |
| *** vishwanathj has quit IRC | 05:29 | |
| *** sdake has quit IRC | 05:32 | |
| *** vmud213 has quit IRC | 05:33 | |
| *** amotoki_ has joined #openstack-ironic | 05:40 | |
| *** amotoki has quit IRC | 05:43 | |
| *** rama_y has joined #openstack-ironic | 05:44 | |
| *** rcernin has quit IRC | 05:51 | |
| *** zhangjl1 has joined #openstack-ironic | 05:55 | |
| *** zhangjl has quit IRC | 05:57 | |
| *** xuao has quit IRC | 06:01 | |
| *** rbudden has quit IRC | 06:03 | |
| *** vmud213 has joined #openstack-ironic | 06:03 | |
| *** sabeen has quit IRC | 06:04 | |
| *** bvandewa has joined #openstack-ironic | 06:12 | |
| *** vmud213 has quit IRC | 06:13 | |
| *** vmud213 has joined #openstack-ironic | 06:14 | |
| *** rcernin has joined #openstack-ironic | 06:21 | |
| *** bvandewa has quit IRC | 06:22 | |
| *** e0ne has joined #openstack-ironic | 06:25 | |
| *** e0ne has quit IRC | 06:40 | |
| *** e0ne has joined #openstack-ironic | 06:43 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic-inspector: Add API for listing all introspection statuses https://review.openstack.org/344921 | 06:49 |
| *** tesseract- has joined #openstack-ironic | 06:59 | |
| *** e0ne has quit IRC | 07:04 | |
| *** daemontool has joined #openstack-ironic | 07:06 | |
| openstackgerrit | lokesh s proposed openstack/ironic: Adding follow-up patch for audit middleware https://review.openstack.org/350399 | 07:09 |
| *** e0ne has joined #openstack-ironic | 07:13 | |
| *** bvandewa has joined #openstack-ironic | 07:19 | |
| *** e0ne has quit IRC | 07:22 | |
| *** fxpester has joined #openstack-ironic | 07:39 | |
| *** abehl has joined #openstack-ironic | 07:40 | |
| *** abehl has quit IRC | 07:41 | |
| *** JieLi has joined #openstack-ironic | 07:41 | |
| *** amoralej|off is now known as amoralej | 07:44 | |
| *** ifarkas_afk is now known as ifarkas | 07:46 | |
| *** jpich has joined #openstack-ironic | 07:49 | |
| *** ohamada has joined #openstack-ironic | 07:49 | |
| *** xuao has joined #openstack-ironic | 07:49 | |
| *** milan has joined #openstack-ironic | 07:50 | |
| *** xuao has quit IRC | 07:51 | |
| *** xuao has joined #openstack-ironic | 07:51 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:01 | |
| *** Goneri has joined #openstack-ironic | 08:04 | |
| *** athomas has joined #openstack-ironic | 08:07 | |
| *** Romanenko_K has joined #openstack-ironic | 08:08 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 08:10 |
| *** tangchen has joined #openstack-ironic | 08:10 | |
| vdrok | morning ironic! | 08:10 |
| *** xuao has quit IRC | 08:14 | |
| *** lucas-dinner is now known as lucasagomes | 08:15 | |
| lucasagomes | vdrok, morning | 08:15 |
| vdrok | morning lucasagomes :) | 08:16 |
| *** xuao has joined #openstack-ironic | 08:16 | |
| xuao | morning~ Ironic! | 08:17 |
| milan | morning Ironic! :) | 08:17 |
| milan | morning lucasagomes vdrok xuao | 08:17 |
| milan | :) | 08:17 |
| *** zhangjl has joined #openstack-ironic | 08:17 | |
| ifarkas | morning all! | 08:18 |
| vdrok | morning xuao milan and ifarkas | 08:18 |
| milan | morning ifarkas! :) | 08:18 |
| *** zhangjl1 has quit IRC | 08:19 | |
| tangchen | lucasagomes: Hi Lucas, I have fixed the ironic functional configuration in project-config, and now, the three patches below are able to run the experimental functional tests successfully. | 08:19 |
| tangchen | https://review.openstack.org/#/c/349421/ | 08:19 |
| tangchen | https://review.openstack.org/#/c/348965/ | 08:19 |
| tangchen | https://review.openstack.org/#/c/350365/ | 08:19 |
| tangchen | lucasagomes: Please help to review them. :) | 08:19 |
| lucasagomes | tangchen, cool! Will do | 08:20 |
| * lucasagomes adds to the list | 08:20 | |
| tangchen | lucasagomes; Thank you very much | 08:20 |
| *** amitkqed has quit IRC | 08:23 | |
| *** amitkqed has joined #openstack-ironic | 08:24 | |
| *** stendulker has joined #openstack-ironic | 08:25 | |
| *** xuao has quit IRC | 08:28 | |
| *** electrofelix has joined #openstack-ironic | 08:28 | |
| *** e0ne has joined #openstack-ironic | 08:30 | |
| *** JieLi has quit IRC | 08:31 | |
| *** bfournie1 has joined #openstack-ironic | 08:31 | |
| *** bfournie has quit IRC | 08:33 | |
| aarefiev | morning1 | 08:34 |
| *** kozhukalov_ has joined #openstack-ironic | 08:35 | |
| lucasagomes | dtantsur|afk, when you around, apparently the pxe_ssh job from the stable branch is broken ? http://logs.openstack.org/87/354187/1/check/gate-tempest-dsvm-ironic-pxe_ssh/2f66ef1/logs/devstacklog.txt.gz#_2016-08-11_17_19_37_178 | 08:36 |
| *** bfournie1 has quit IRC | 08:36 | |
| pas-ha | morning all | 08:37 |
| *** Goneri has quit IRC | 08:38 | |
| *** kozhukalov has quit IRC | 08:38 | |
| *** kozhukalov_ is now known as kozhukalov | 08:38 | |
| zhangjl | morning all | 08:41 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Remove neutron client workarounds https://review.openstack.org/354588 | 08:45 |
| *** dtantsur|afk is now known as dtantsur | 08:52 | |
| *** d0ugal has quit IRC | 08:52 | |
| dtantsur | Morning Ironic, lucasagomes, aarefiev, pas-ha, zhangjl, milan, vdrok and everyone else :) | 08:52 |
| dtantsur | lucasagomes, ugh | 08:52 |
| milan | morning dtantsur! :) | 08:52 |
| *** Goneri has joined #openstack-ironic | 08:53 | |
| milan | morning aarefiev, pas-ha, zhangjl! :) | 08:53 |
| dtantsur | lucasagomes, I wonder if it's the result of the recent jobs refactoring (conducted by sambetts|afk iirc) | 08:53 |
| vdrok | morning aarefiev pas-ha zhangjl and dtantsur | 08:53 |
| dtantsur | lucasagomes, we need to set IRONIC_BUILD_DEPLOY_RAMDISK for old ramdisk jobs | 08:54 |
| lucasagomes | dtantsur, it could be it... It seems to be some options to devstack that are messed up | 08:54 |
| lucasagomes | dtantsur, the only problem is that, I don't know if we can actually test changes to project-config using the Depends-On tag or similar mechanisms | 08:55 |
| lucasagomes | we probably needs to reproduce in a vm locally | 08:55 |
| dtantsur | lucasagomes, no, we can't | 08:55 |
| *** hshiina has quit IRC | 08:55 | |
| dtantsur | only guess :( | 08:55 |
| lucasagomes | :-/ | 08:55 |
| *** athomas has quit IRC | 08:58 | |
| *** athomas has joined #openstack-ironic | 09:03 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add "erase_devices_metadata_priority" config option https://review.openstack.org/344179 | 09:04 |
| *** sambetts|afk is now known as sambetts | 09:04 | |
| *** xuao has joined #openstack-ironic | 09:04 | |
| sambetts | Morning lucasagomes dtantsur | 09:04 |
| lucasagomes | morning sambetts | 09:04 |
| dtantsur | morning sambetts | 09:04 |
| *** tangchen has quit IRC | 09:04 | |
| sambetts | I hope I didn't break the stable job :/ but its quite possible, the refactor so big many options moved around | 09:05 |
| *** tangchen has joined #openstack-ironic | 09:05 | |
| sambetts | lucasagomes, dtantsur: It might be JayF's patch change the default of BUILD_DEPLOY_RAMDISK from true to false, thats affect things if thats the part thats missing | 09:08 |
| dtantsur | true.. | 09:08 |
| sambetts | but we have stable prebuilt ramdisks now | 09:08 |
| lucasagomes | sambetts, indeed... perhaps we just need to set it explicitly for the stable jobs? | 09:08 |
| dtantsur | sambetts, pxe_ssh is the old ramdisk, we don't prebuild it | 09:08 |
| dtantsur | lucasagomes, s/stable jobs/old ramdisk jobs/ | 09:08 |
| sambetts | ooh right, yeah the pxe_ssh job is a weird one | 09:08 |
| dtantsur | which we have only on stable, true :) | 09:09 |
| dtantsur | other than that, folks, do you think we can put some effort into reviewing ironic-lib and ironicclient? their deadlines are coming pretty fast | 09:10 |
| dtantsur | (faster than ironic and inspector themselves) | 09:10 |
| sambetts | is there anything in particular to focus on ? | 09:10 |
| sambetts | or just in general? | 09:11 |
| sambetts | I've not kept up with the changes going into ironic-lib | 09:11 |
| dtantsur | my personal wish (and it's on the priority list for this cycle) is finishing the OSC plugin | 09:12 |
| dtantsur | I think folks have all patches up, we just need to verify and review them | 09:12 |
| sambetts | oh yeah that would be awesome | 09:12 |
| lucasagomes | ++ will take a look today | 09:14 |
| * lucasagomes too many patches to review T.T | 09:14 | |
| dtantsur | ++ | 09:15 |
| dtantsur | lucasagomes, do you think you could do a quick sanity check of my ironic-installed-by-tripleo patch https://review.openstack.org/#/c/341433/ ? no need to understand THT, just take a look at the options I'm setting there | 09:16 |
| lucasagomes | dtantsur, sure, I'm just finishing one review and will look at it right after | 09:17 |
| *** amotoki has joined #openstack-ironic | 09:20 | |
| sambetts | dtantsur, lucasagomes: https://review.openstack.org/354608 | 09:21 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Remove neutron client workarounds https://review.openstack.org/354588 | 09:21 |
| openstackgerrit | Ankit Kumar proposed openstack/proliantutils: Fix log message for exception in Gen7 https://review.openstack.org/353860 | 09:22 |
| *** amotoki_ has quit IRC | 09:23 | |
| *** stendulker has quit IRC | 09:27 | |
| lucasagomes | dtantsur, done, looks alright apart from that neutron url thing | 09:28 |
| lucasagomes | I thought it would come from the keystone catalog | 09:29 |
| dtantsur | lucasagomes, well, we have this option for some reason, and we set it for undercloud too... | 09:29 |
| dtantsur | lucasagomes, other services have similar configurations, so really no idea | 09:30 |
| lucasagomes | heh cool | 09:30 |
| lucasagomes | I didn't vote, just commented because I don't really understand heat templates | 09:30 |
| dtantsur | no problem, thanks. just wanted to be sure I'm not doing something entirely stupid from ironic standpoint | 09:31 |
| lucasagomes | heh no, looks fine for me :D | 09:32 |
| openstackgerrit | Ricardo Carrillo Cruz proposed openstack/bifrost: Set OS_AUTH_TOKEN to dummy string, instead of empty space https://review.openstack.org/354615 | 09:36 |
| *** rama_y has quit IRC | 09:37 | |
| *** amitkqed has quit IRC | 09:40 | |
| *** amitkqed has joined #openstack-ironic | 09:41 | |
| *** amotoki_ has joined #openstack-ironic | 09:41 | |
| *** amotoki has quit IRC | 09:44 | |
| openstackgerrit | Ricardo Carrillo Cruz proposed openstack/bifrost: Initialize 'auth' to empty dict on bifrost_configdrives_dynamic https://review.openstack.org/354620 | 09:45 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Introducing node introspection state management https://review.openstack.org/348943 | 09:45 |
| *** mgould|afk is now known as mgould | 09:47 | |
| mgould | morning Ironic! | 09:47 |
| sambetts | Morning mgould | 09:48 |
| milan | morning mgould! :) | 09:51 |
| mgould | morning sambetts milan | 09:53 |
| milan | aarefiev, hi, I think I may have fixed the invalid state transition processing->processing case from yesterday: https://review.openstack.org/#/c/348943/11 I'll be replacing the enum_ with plain classes and add a function test automating my manual error tests later. | 09:56 |
| milan | dtantsur, sambetts, ifarkas FYI ^ | 09:56 |
| milan | * the introspection state patch | 09:56 |
| openstackgerrit | Ricardo Carrillo Cruz proposed openstack/bifrost: Remove 'auth' fact initialization from bifrost-deploy-nodes-dynamic https://review.openstack.org/354626 | 10:00 |
| aarefiev | milan: great, I'll take a look, thank you | 10:00 |
| milan | aarefiev, thanks for the reviews! | 10:01 |
| milan | dtantsur, sambetts w/r the introspection state patch, do you guys prefer aarefiev 's suggestion to move the fsm logic from the node_info object into custom module? | 10:11 |
| * dtantsur hasn't followed, sorry | 10:12 | |
| dtantsur | maybe it makes sense | 10:12 |
| * milan would like to limit rewrites here | 10:12 | |
| milan | dtantsur, please, check the node_cache/NodeInfo couple of transition methods | 10:13 |
| * milan would appreciate other opinions on this | 10:13 | |
| dtantsur | sure, when I have a minute :/ probably I'll have some time today actually | 10:13 |
| dtantsur | lucasagomes, could you please rereview https://review.openstack.org/#/c/353450/ ? it'll make some downstream folks happy | 10:14 |
| milan | dtantsur, thx! | 10:14 |
| *** amotoki_ has quit IRC | 10:15 | |
| * lucasagomes was having is 1-1 | 10:18 | |
| lucasagomes | dtantsur, will take a look, need a cup of coffee first | 10:18 |
| lucasagomes | dtantsur, I will retest it in my env as well | 10:18 |
| lucasagomes | I've it handy | 10:18 |
| dtantsur | yes please! | 10:19 |
| *** dtantsur is now known as dtantsur|brb | 10:20 | |
| *** vmud213 has quit IRC | 10:24 | |
| *** vmud213 has joined #openstack-ironic | 10:24 | |
| sambetts | oh wow there are a lot of open patchs for ironicclient | 10:26 |
| *** amotoki has joined #openstack-ironic | 10:27 | |
| *** xuao has quit IRC | 10:27 | |
| *** xuao has joined #openstack-ironic | 10:29 | |
| *** amitkqed has quit IRC | 10:35 | |
| *** amitkqed has joined #openstack-ironic | 10:36 | |
| *** xuao has quit IRC | 10:38 | |
| *** zhangjl has quit IRC | 10:39 | |
| *** daemontool has quit IRC | 10:39 | |
| *** daemontool has joined #openstack-ironic | 10:41 | |
| *** daemontool_ has joined #openstack-ironic | 10:47 | |
| *** daemontool has quit IRC | 10:47 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 10:51 |
| *** kragniz1 has quit IRC | 10:52 | |
| *** kragniz1 has joined #openstack-ironic | 10:53 | |
| rcarrillocruz | hi folks, is it normal that i perform an 'ironic node-set-provision state deleted' and after cleaning now the node is in clean wait for more than 10min? | 10:57 |
| *** bvandewa has quit IRC | 10:58 | |
| sambetts | rcarrillocruz: depending on which clean steps you have enabled cleaning can take a long time, e.g. things like shredding the disks | 10:58 |
| rcarrillocruz | thing is i have 'clean_nodes = false' on ironic.conf, i don't even understand why it went to cleaning it... | 10:58 |
| *** kragniz1 has quit IRC | 10:59 | |
| rcarrillocruz | where can i look tho for those clean steps ? ironic.conf or some other file | 10:59 |
| rcarrillocruz | ? | 10:59 |
| *** Ng has joined #openstack-ironic | 10:59 | |
| jroll | morning folks! I'm not really here today, just wanted to say hi :) and if there's anything needed quickly let me know | 10:59 |
| sambetts | jroll: hi | 10:59 |
| jroll | rcarrillocruz: what version? clean_nodes moved to automated_clean | 11:00 |
| jroll | deprecated in mitaka http://docs.openstack.org/releasenotes/ironic/mitaka.html#deprecation-notes | 11:00 |
| jroll | and removed in 6.0.0 | 11:00 |
| rcarrillocruz | ironic==6.0.1.dev295 | 11:00 |
| rcarrillocruz | hmm, so this is probably an old conf | 11:00 |
| rcarrillocruz | not touched by the upgrade | 11:00 |
| jroll | http://docs.openstack.org/releasenotes/ironic/current-series.html#id7 "Remove the deprecated “[conductor]/clean_nodes” option. Configuration files should instead use the “[conductor]/automated_clean” option." | 11:01 |
| jroll | yeah, gotta read release notes and check for deprecation warnings | 11:01 |
| rcarrillocruz | it's the bifrost controller on the infracloud btw... so it may be i have an old ironic.conf , despite having run the install playbook to have an updated ironic and reqs | 11:01 |
| rcarrillocruz | thx | 11:01 |
| jroll | ooo | 11:01 |
| * jroll wonders if bifrost missed that :( | 11:01 | |
| rcarrillocruz | i better remove all ironic related stuff and run install afresh... i suspect some missed upgrade from bifrost too | 11:02 |
| sambetts | rcarrillocruz: btw I think you can view which clean steps are running/ are going to run by looking at the node, via ironic node-show | 11:03 |
| rcarrillocruz | ah | 11:03 |
| rcarrillocruz | a DB side conf thing | 11:03 |
| rcarrillocruz | k, thx | 11:03 |
| rcarrillocruz | hmm, clean_step: {} | 11:03 |
| rcarrillocruz | i'm gonna nuke *everything*, even the ironic DB | 11:04 |
| Ng | if policy.json has show_password:! is there any way to get IPMI passwords out of Ironic? | 11:04 |
| jroll | Ng: via the database. you can also set policy to allow that for a certain role | 11:07 |
| jroll | or tenant, or whatever | 11:07 |
| Ng | jroll: policy for a certain role/tenant sounds perfect, thanks! | 11:07 |
| jroll | e.g. "show_password": "role:ipmi_admin" | 11:08 |
| jroll | no problem :) | 11:08 |
| jroll | Ng: in case you missed it, we just released 6.1.0 with a more fine-grained policy for read/write per API resource :) | 11:08 |
| *** joprovost has joined #openstack-ironic | 11:08 | |
| Ng | jroll: I had missed that. very nice! | 11:09 |
| rcarrillocruz | oh, there a redeploy playbook now in bifrost, nice! | 11:09 |
| *** joprovost has joined #openstack-ironic | 11:10 | |
| mat128 | good morning | 11:12 |
| mgould | morning mat128 | 11:13 |
| sambetts | o/ mat128 | 11:13 |
| *** kragniz1 has joined #openstack-ironic | 11:15 | |
| * sambetts -> lunch | 11:16 | |
| *** sambetts is now known as sambetts|lunch | 11:16 | |
| *** thrash|g0ne is now known as thrash | 11:18 | |
| *** kragniz1 has quit IRC | 11:18 | |
| vdrok | morning sambetts|lunch mgould jroll and mat128 | 11:18 |
| mgould | morning vdrok | 11:18 |
| *** kragniz1 has joined #openstack-ironic | 11:19 | |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Add tests for testing node-delete command https://review.openstack.org/251458 | 11:20 |
| milan | morning jroll, mat128, vdrok! :) | 11:21 |
| vdrok | gm milan :) | 11:21 |
| *** amotoki has quit IRC | 11:30 | |
| lucasagomes | dtantsur|brb, +a on the update port thing, I've tested it locally and works http://paste.openstack.org/show/555954/ | 11:31 |
| *** dtantsur|brb is now known as dtantsur | 11:34 | |
| dtantsur | cool, thanks | 11:34 |
| dtantsur | morning jroll, vdrok | 11:34 |
| *** kragniz is now known as Guest66666 | 11:36 | |
| *** kragniz1 is now known as kragniz | 11:36 | |
| vdrok | milan: dtantsur two good mornings from you today, is it because of friday is two times better than other days? :) | 11:36 |
| dtantsur | of course :) | 11:37 |
| * dtantsur needs a bot to keep track of good mornings | 11:37 | |
| milan | vdrok, TGIF! :D | 11:37 |
| *** jistr has quit IRC | 11:39 | |
| rcarrillocruz | TheJulia: shouldn't create_ipa_image have a default ? imho it should be defaulted to false | 11:39 |
| rcarrillocruz | the bootstrap task file from bifrost-ironic-install fails , as there's no value defined for it | 11:40 |
| rcarrillocruz | yet download_ipa does have a default | 11:40 |
| rcarrillocruz | (defaults to yes) | 11:40 |
| *** dansmith has quit IRC | 11:40 | |
| *** amotoki has joined #openstack-ironic | 11:41 | |
| *** dansmith has joined #openstack-ironic | 11:41 | |
| *** dansmith is now known as Guest13957 | 11:42 | |
| *** narasimha_SV has joined #openstack-ironic | 11:42 | |
| narasimha_SV | when I executed the nova boot command to get pxe boot of baremental using pxe_ssh I am getting no valid host found error | 11:43 |
| narasimha_SV | http://paste.openstack.org/show/555959/ | 11:43 |
| narasimha_SV | all the information about my issue is in above link | 11:43 |
| narasimha_SV | please help me in fixing this | 11:43 |
| mgould | narasimha_SV: have you tried following the steps at http://docs.openstack.org/developer/ironic/deploy/troubleshooting.html ? | 11:44 |
| sambetts|lunch | narasimha_SV: the schedular log info you've included is to late in the log, there will be an earlier log which contains the real failure | 11:45 |
| sambetts|lunch | narasimha_SV: can you paste us more of the log | 11:46 |
| *** sambetts|lunch is now known as sambetts | 11:46 | |
| sambetts | narasimha_SV: we'd also need a full ironic node-show | 11:48 |
| sambetts | of your Ironic node | 11:48 |
| narasimha_SV | http://paste.openstack.org/show/555963/ | 11:52 |
| narasimha_SV | i have checked all those things mentioned in troubleshooting link | 11:53 |
| narasimha_SV | all are good | 11:53 |
| narasimha_SV | when I execute nova boot command I am getting no valid host error | 11:53 |
| narasimha_SV | in scheduler there are no logs | 11:54 |
| sambetts | are there any error logs in the Ironic conductor or API log, if a node fails to deploy for any reason it can also result in a no valid host error | 11:54 |
| *** vmud213 has quit IRC | 11:54 | |
| sambetts | but nova will retry scheduling if its first node fails until it fines a node that is successful | 11:55 |
| sambetts | s/but/because | 11:55 |
| sambetts | and if it can't find another node to try then it throws the node valid host found error | 11:56 |
| jroll | it's also important to check nova-compute configs - reserved_host_memory_mb=0 and ram_allocation_ratio=1.0 | 11:57 |
| jroll | though I think we'd see that in hypervisor-stats | 11:57 |
| * jroll is with the deploy failure theory | 11:57 | |
| sambetts | nova has definatly tried to put the instance on the node at least once, because the instace_info is populated | 11:58 |
| sambetts | but I think there must be an error message in the conductors logs somewhere about this | 11:58 |
| jroll | ah, yeah | 11:59 |
| dtantsur | jroll, do you think https://review.openstack.org/353450 is backportable? | 12:00 |
| *** lucasagomes is now known as lucas-hungry | 12:00 | |
| jroll | dtantsur: looks like it | 12:01 |
| *** ccamacho is now known as ccamacho|lunch | 12:02 | |
| openstackgerrit | Jacek Tomasiak proposed openstack/ironic: Removed unneeded vlan settings from neutron config https://review.openstack.org/354685 | 12:04 |
| narasimha_SV | jroll,sambetts: PortBindingFailed: Binding failed for port 14ac413f-159a-4c7d-ac72-7620f671a52b | 12:04 |
| narasimha_SV | i am getting this issue in compute logs | 12:04 |
| narasimha_SV | let me check this | 12:05 |
| *** alaski is now known as lascii | 12:05 | |
| *** bfournie has joined #openstack-ironic | 12:07 | |
| *** amotoki has quit IRC | 12:10 | |
| *** bfournie has quit IRC | 12:12 | |
| *** ohamada has quit IRC | 12:13 | |
| * dtantsur -> lunch | 12:14 | |
| *** ohamada has joined #openstack-ironic | 12:15 | |
| *** amotoki has joined #openstack-ironic | 12:16 | |
| rcarrillocruz | hmm, the install playbook checks for create_ipa_image, but does not for download_ipa | 12:21 |
| *** MattMan has quit IRC | 12:22 | |
| *** MattMan has joined #openstack-ironic | 12:22 | |
| *** amoralej is now known as amoralej|lunch | 12:23 | |
| *** jistr has joined #openstack-ironic | 12:24 | |
| TheJulia | Good morning | 12:29 |
| mat128 | morning TheJulia | 12:29 |
| mgould | morning TheJulia | 12:29 |
| *** amotoki has quit IRC | 12:29 | |
| *** jistr has quit IRC | 12:31 | |
| TheJulia | rcarrillocruz: I believe we went back through put in |bool for everything to process as defaults, so I think the default is required, although I also suspect that the default may need toward building an IPA image for a user a tis point. | 12:31 |
| *** amotoki has joined #openstack-ironic | 12:31 | |
| TheJulia | good morning mat128 mgould | 12:31 |
| *** jistr has joined #openstack-ironic | 12:32 | |
| *** adreznec has quit IRC | 12:33 | |
| *** chlong|mtg has quit IRC | 12:33 | |
| dtantsur | morning TheJulia | 12:35 |
| *** adreznec has joined #openstack-ironic | 12:35 | |
| *** amotoki has quit IRC | 12:35 | |
| *** baoli has joined #openstack-ironic | 12:36 | |
| *** baoli_ has joined #openstack-ironic | 12:37 | |
| *** baoli has quit IRC | 12:41 | |
| *** jistr has quit IRC | 12:41 | |
| *** amotoki has joined #openstack-ironic | 12:47 | |
| *** bfournie has joined #openstack-ironic | 12:51 | |
| *** links has quit IRC | 12:53 | |
| vdrok | morning TheJulia | 12:55 |
| xavierr | morning Ironic | 12:57 |
| *** rbudden has joined #openstack-ironic | 12:57 | |
| xavierr | morning rcarrillocruz TheJulia mat128 dtantsur vdrok | 12:57 |
| *** amoralej|lunch is now known as amoralej | 12:59 | |
| *** lucas-hungry is now known as lucasagomes | 13:00 | |
| *** baoli_ has quit IRC | 13:02 | |
| vdrok | morning xavierr ! | 13:02 |
| *** jistr has joined #openstack-ironic | 13:03 | |
| *** daemontool_ has quit IRC | 13:05 | |
| lucasagomes | gate seems a bit flaky today | 13:06 |
| lucasagomes | 2016-08-12 11:52:08.513893 | [Zuul] standard output/error still open after child exited | 13:06 |
| *** psachin has quit IRC | 13:08 | |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Tests for testing node-create command https://review.openstack.org/262055 | 13:12 |
| *** jistr has quit IRC | 13:13 | |
| *** fragatina has joined #openstack-ironic | 13:13 | |
| *** ccamacho|lunch is now known as ccamacho | 13:14 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Remove neutron client workarounds https://review.openstack.org/354588 | 13:14 |
| *** sdake has joined #openstack-ironic | 13:14 | |
| *** jcoufal has joined #openstack-ironic | 13:15 | |
| *** fragatina has quit IRC | 13:18 | |
| *** dprince has joined #openstack-ironic | 13:18 | |
| *** dprince has quit IRC | 13:19 | |
| *** [1]cdearborn has joined #openstack-ironic | 13:20 | |
| *** dprince has joined #openstack-ironic | 13:20 | |
| rcarrillocruz | morning xavierr | 13:24 |
| rcarrillocruz | TheJulia: so yeah, given that the install playbook use case is to create an ipa | 13:25 |
| rcarrillocruz | i believe it is sane to have the default to true | 13:25 |
| rcarrillocruz | so a user does not require to pass it | 13:25 |
| rcarrillocruz | for a better experience for someone that just starting to use bifrost | 13:25 |
| *** sean-k-mooney has joined #openstack-ironic | 13:29 | |
| *** rama_y has joined #openstack-ironic | 13:29 | |
| TheJulia | rcarrillocruz: the downside is then a standard/typical/initial user is consuming a distribution via dib that might be broken for a few hours for any number of reasons :( | 13:30 |
| rcarrillocruz | yeah, i mean, if you're ok with it i can work a patch that switches do download an ipa, instead of building one ? | 13:30 |
| TheJulia | Well, by default it should be downloading by default | 13:31 |
| sean-k-mooney | hi, has anyone had an issues with the ipa image that is pulled down by bifrost lately? | 13:31 |
| * TheJulia clearly needs more coffee | 13:31 | |
| rcarrillocruz | that's why i intially thought it may be good to have create_ipa_image false, since the download_ipa is true on the role by default | 13:31 |
| rcarrillocruz | but then the install does not have loging for dwnloading ipa, just for creating it | 13:32 |
| TheJulia | sean-k-mooney: What are you seeing? | 13:32 |
| TheJulia | rcarrillocruz: oh, okay, I see what your talking about now | 13:32 |
| rcarrillocruz | https://github.com/openstack/bifrost/blob/master/playbooks/install.yaml | 13:32 |
| rcarrillocruz | the when three | 13:32 |
| rcarrillocruz | there's no alternative to just pull a known built one from wherever you keep them ( i assume from tarballs.openstack.org/ci ? ) | 13:33 |
| sean-k-mooney | TheJulia: the server is pxe booting the image but one of the services fails to start so the deploy eventully times out in the waiting_for_callback phase | 13:33 |
| openstackgerrit | Merged openstack/ironic: Fix updating port MAC address for active nodes https://review.openstack.org/353450 | 13:33 |
| TheJulia | sean-k-mooney: physical hardware or VMs? | 13:33 |
| sean-k-mooney | physical. my vm setups is still working but i have not pulled a new ipa image in quite a while | 13:34 |
| TheJulia | rcarrillocruz: https://github.com/openstack/bifrost/blob/master/playbooks/roles/bifrost-ironic-install/tasks/download_ipa_image.yml ? | 13:34 |
| TheJulia | sean-k-mooney: CoreOS or TinyIPA? | 13:35 |
| sean-k-mooney | TheJulia: i was just using bifrst defaults so whatever that pulls down. ill check | 13:36 |
| TheJulia | sean-k-mooney: That would be CoreOS | 13:36 |
| sean-k-mooney | ah ok | 13:36 |
| TheJulia | sean-k-mooney: What do you see on the screen/console for the baremetal node? | 13:36 |
| sean-k-mooney | ill kick oc another deploy and see if i can get one | 13:37 |
| sean-k-mooney | /oc/off | 13:37 |
| TheJulia | It might be worth to try tinyipa, but we need to know what your encountering with coreos | 13:38 |
| sean-k-mooney | TheJulia: sure ill try that after i get the log output. i tried geting bifrost to build it too but hit other issues with the ironic python agent service already existing and i could not build it. | 13:39 |
| TheJulia | oh... lovely | 13:39 |
| TheJulia | :( | 13:39 |
| sean-k-mooney | TheJulia: well that may have been my fault | 13:40 |
| TheJulia | cinerama: are you aware of any dib issues with building IPA presently? | 13:40 |
| TheJulia | sean-k-mooney: building an image without a cache or can be... problematic if you loose connectivity | 13:40 |
| *** sabeen has joined #openstack-ironic | 13:41 | |
| sean-k-mooney | TheJulia: only set -e create_ipa_image=true when i run the install playbook it completed fine but if i tried to change the dib os or add extra element things broke | 13:41 |
| TheJulia | Ahhhh | 13:42 |
| TheJulia | I feel like this is just more fuel on the fire to build other images that we know work, but everyone has some special requirement too :( | 13:42 |
| * TheJulia wonders if some sort of matrix would work | 13:43 | |
| sean-k-mooney | TheJulia: TheJulia well i was trying to set a username and password on the image using the devuser image so i could take a look at what going on | 13:43 |
| TheJulia | ahhh! | 13:44 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Fix updating port MAC address for active nodes https://review.openstack.org/354735 | 13:44 |
| TheJulia | well, you said the OS as well, that was likely the issue :\ | 13:44 |
| dtantsur | jroll, a backport of the port update patch: https://review.openstack.org/#/c/354735/ | 13:45 |
| * dtantsur has no idea if the stable gate works right now | 13:45 | |
| sean-k-mooney | TheJulia: yep ill try the default image again and let you know how that goes | 13:45 |
| *** PollyZ has joined #openstack-ironic | 13:45 | |
| *** sdake has quit IRC | 13:46 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [pxe] https://review.openstack.org/309612 | 13:49 |
| *** adu has joined #openstack-ironic | 13:49 | |
| *** Guest13957 is now known as dansmith | 13:50 | |
| *** dansmith is now known as Guest37890 | 13:50 | |
| *** d0ugal has joined #openstack-ironic | 13:51 | |
| *** Guest37890 is now known as dansmith | 13:51 | |
| *** hpe-hj has joined #openstack-ironic | 13:53 | |
| *** hjanssen-hpe has joined #openstack-ironic | 13:53 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [iscsi] https://review.openstack.org/309615 | 13:53 |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize Config Options - patch merge, cleanup https://review.openstack.org/330160 | 13:54 |
| openstackgerrit | Imre Farkas proposed openstack/ironic: DRAC RAID configuration https://review.openstack.org/176319 | 13:54 |
| *** e0ne has quit IRC | 13:54 | |
| *** e0ne has joined #openstack-ironic | 13:55 | |
| rcarrillocruz | TheJulia: so what are your thoughts, you think we should have the install as a basic 'get me started quick' and thus i should replace the create_ipa for download_ipa? | 13:56 |
| rcarrillocruz | role defaults don't follow the logic of the playbook here | 13:56 |
| *** vishwanathj has joined #openstack-ironic | 13:56 | |
| *** sdake has joined #openstack-ironic | 13:56 | |
| *** dansmith is now known as superdan | 13:57 | |
| TheJulia | Kind of, or move the downloading IPA step to be adjacent to image building. Quick starting would be best in my mind, it gets most users going.... although some still need to build their own IPA images for various drivers or troubleshooting needs. | 13:57 |
| rcarrillocruz | k | 13:59 |
| rcarrillocruz | will work a patch | 13:59 |
| *** anush_ has joined #openstack-ironic | 14:00 | |
| *** joprovost1 has joined #openstack-ironic | 14:00 | |
| rcarrillocruz | ah nice, i see there's alrady a default to pull a sane ipa | 14:00 |
| rcarrillocruz | ipa_kernel_upstream_url: http://tarballs.openstack.org/ironic-python-agent/coreos/files/coreos_production_pxe.vmlinuz | 14:00 |
| rcarrillocruz | thx | 14:00 |
| *** joprovost has quit IRC | 14:01 | |
| *** joprovost1 is now known as joprovost | 14:01 | |
| *** baoli has joined #openstack-ironic | 14:05 | |
| TheJulia | rcarrillocruz: My only worry at the moment, is if coreos is being a less sane option. Seems more and more issues are being encountered | 14:06 |
| rcarrillocruz | and if ipa is built, is it against debian ? | 14:06 |
| TheJulia | by default with bifrost, yes | 14:06 |
| rcarrillocruz | gotcha | 14:07 |
| *** bnemec is now known as beekneemech | 14:07 | |
| lucasagomes | ifarkas, around? There's a couple of unaddressed comments from vdrok here: https://review.openstack.org/#/c/176319/32..33/ironic/drivers/modules/drac/raid.py | 14:07 |
| vdrok | lucasagomes: yup, ifarkas said he'll fix it :) | 14:07 |
| lucasagomes | vdrok, ifarkas oh sorry then | 14:07 |
| ifarkas | lucasagomes, yeah, check my last comment :-) | 14:07 |
| lucasagomes | cause I was looking at the last patch-set | 14:08 |
| lucasagomes | ack | 14:08 |
| lucasagomes | lemme know and I will re-review the change too :-) | 14:08 |
| openstackgerrit | Miles Gould proposed openstack/python-ironicclient: Document updating nested node attributes with CLI https://review.openstack.org/354742 | 14:08 |
| ifarkas | lucasagomes, will do, thanks! | 14:08 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add save_after_refresh test https://review.openstack.org/354744 | 14:11 |
| vsaienk0 | I''m wondered if anyone saw this problem ^^ | 14:15 |
| lucasagomes | dtantsur, if you have a time today or next week, mind taking a look at https://review.openstack.org/#/c/273684/ ? I'll work on a subsequent patch enabling iboot in the project | 14:17 |
| *** xavierr has quit IRC | 14:18 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add save_after_refresh test https://review.openstack.org/354744 | 14:18 |
| *** xavierr has joined #openstack-ironic | 14:18 | |
| dtantsur | lucasagomes, sure, lemme finish with OSC patches first | 14:19 |
| lucasagomes | dtantsur, yeah no rush at all | 14:19 |
| *** adu has quit IRC | 14:22 | |
| wznoinsk | hi all | 14:25 |
| *** xavierr has quit IRC | 14:26 | |
| *** xavierr has joined #openstack-ironic | 14:26 | |
| lucasagomes | wznoinsk, hello | 14:27 |
| wznoinsk | a long shot but maybe someone saw it before: http://paste.openstack.org/show/556066/ <- ironic_api, 10.237.223.28 is my physical ironic node with ironic python agent started trying to run that query, same behavior with both pxe_ipmtitool and agent_ipmitool deploy drivers | 14:27 |
| lucasagomes | wznoinsk, apparently the IPA version is newer than the ironic services | 14:28 |
| lucasagomes | wznoinsk, are you running master on both? Or a mix of mitaka + master IPA ? | 14:29 |
| lucasagomes | IPA = the deploy ramdisk | 14:29 |
| wznoinsk | that is actually possible, running stable/mitaka of kolla -> ironic and the IPA image was build yesterday with no version constraints | 14:29 |
| lucasagomes | wznoinsk, right, yeah that's my guess | 14:29 |
| lucasagomes | that endpoint IPA is trying to ping is quite new | 14:29 |
| lucasagomes | and won't be present in mitaka | 14:29 |
| lucasagomes | wznoinsk, can you possible build another ramdisk running IPA mitaka version ? | 14:30 |
| wznoinsk | actually I might have played around with endpoints, let me check | 14:30 |
| *** causten has joined #openstack-ironic | 14:31 | |
| wznoinsk | lucasagomes: I actually put the auth_url = http://10.237.222.18:35357 | 14:31 |
| wznoinsk | instead of :5000 in ironic_api-> keystone_authtoken as part of my testing, i've changed prject/user_domain_id -> _name and added Region | 14:31 |
| lucasagomes | wznoinsk, right, the /lookup endpoint doesn't have any auth AFAICT (/me will check ironic's api code) | 14:32 |
| sambetts | lucasagomes: I think it depends on the policy now | 14:32 |
| lucasagomes | https://github.com/openstack/ironic/blob/f0360dcc16b60f3a89db3e779e8109282f2792cf/ironic/api/config.py#L32-L34 | 14:33 |
| lucasagomes | sambetts, maybe | 14:33 |
| wznoinsk | lucasagomes: IPA would read endpoints information from ironic_api is my guess, the original (v2 - alike) didn't work for me either | 14:33 |
| lucasagomes | wznoinsk, what happened is that, I'm not sure how familiar you are with the vendor_passthru endpoint in Ironic | 14:33 |
| lucasagomes | basically the /lookup and /heartbeat endpoints where part of vendor passthru | 14:34 |
| wznoinsk | lucasagomes: we can assume I'm not ... my first week playing with Ironic | 14:34 |
| lucasagomes | but since now we only have 1 ramdisk supported (the IPA one) we've promote those endpoints to the main API | 14:34 |
| lucasagomes | so the newer ramdisk is trying to use these new endpoints | 14:34 |
| lucasagomes | which are not present in the old releases (e.g mitaka) | 14:35 |
| wznoinsk | you mean it's looking for lookup? instead of lookup/ | 14:35 |
| wznoinsk | ? | 14:35 |
| *** baoli has quit IRC | 14:36 | |
| *** baoli has joined #openstack-ironic | 14:36 | |
| lucasagomes | wznoinsk, I mean it's looking for v1/lookup (the newer endpoint) instead of v1/drivers/<driver_name>/vendor_passhtru/lookup | 14:36 |
| lucasagomes | wznoinsk, see http://docs.openstack.org/developer/ironic/dev/vendor-passthru.html | 14:37 |
| sambetts | dtantsur: re ^ I thought that IPA would try only try to use the new endpoint if it exists or it'll fall back to the old one? | 14:37 |
| dtantsur | sambetts, it is supposed to fall back | 14:37 |
| wznoinsk | lucasagomes: ok, I guess I'll recreate the IPA ramdisk in the meantime | 14:37 |
| sambetts | hmm :/ | 14:37 |
| lucasagomes | wznoinsk, yeah, try to build a mitaka version please | 14:38 |
| wznoinsk | the error 401 is not authorized, hence I was playing around with v2/v3 keystone endpoionts | 14:38 |
| lucasagomes | wznoinsk, right... that's resonable, but I think those endpoints are "public" | 14:38 |
| lucasagomes | so I discarded keystone from my theory here | 14:38 |
| *** jistr has joined #openstack-ironic | 14:39 | |
| lucasagomes | wznoinsk, anyway, lemme know if the mitaka version works for you | 14:39 |
| lucasagomes | if not we can dig more into the problem | 14:39 |
| *** PollyZ has quit IRC | 14:39 | |
| * dtantsur loves the new 'openstack baremetal create' command! | 14:40 | |
| *** joprovost has quit IRC | 14:41 | |
| sambetts | \o/ | 14:43 |
| jlvillal | lucasagomes, What do you think about this? https://bugs.launchpad.net/bugs/1612476 | 14:43 |
| openstack | Launchpad bug 1612476 in Ironic "Do not auto expire config drive in swift" [Undecided,New] - Assigned to xiaobin (jxiaobin) | 14:43 |
| sambetts | dtantsur: /me is looking forward to not having to type out set-provision-state all the time | 14:43 |
| jlvillal | lucasagomes, I'm not sure if that is a bug or not. I think it is working as you designed it. | 14:44 |
| *** joprovost has joined #openstack-ironic | 14:44 | |
| lucasagomes | jlvillal, yeah that doesn't sound like a bug for me either | 14:44 |
| lucasagomes | I mean, the timeout in swift should match the timeout of the deploy in ironic | 14:45 |
| lucasagomes | if that's not the case then it's a bug | 14:45 |
| jlvillal | That is what I thought | 14:45 |
| lucasagomes | but, not deleting it from swift and keeping it there forever is a security problem | 14:45 |
| lucasagomes | configdrive does contain secrets, I don't think we should do that | 14:45 |
| jlvillal | Yep | 14:45 |
| dtantsur | sambetts++ | 14:46 |
| dtantsur | morning jlvillal | 14:46 |
| lucasagomes | jlvillal, morning yeah :D | 14:46 |
| dtantsur | ++ for not keeping configdrive forever | 14:46 |
| jlvillal | Good morning dtantsur lucasagomes sambetts and everyone else :) | 14:47 |
| sambetts | o/ jlvillal | 14:47 |
| dtantsur | hmm, though rebuild poses an interesting problem | 14:48 |
| *** michauds has joined #openstack-ironic | 14:50 | |
| dtantsur | jlvillal, after reading a bit more: we should expire the temporary URL, but not the config drive itself | 14:50 |
| dtantsur | lucasagomes, jlvillal, do you agree with my comments on https://bugs.launchpad.net/ironic/+bug/1612476 ? | 14:51 |
| openstack | Launchpad bug 1612476 in Ironic "Do not auto expire config drive in swift" [Undecided,New] - Assigned to xiaobin (jxiaobin) | 14:51 |
| sambetts | I thought that was what made the temp URLs, temp URLs? | 14:51 |
| dtantsur | so, we expire both the object and the temporary URL | 14:52 |
| dtantsur | the former seems incorrect | 14:52 |
| * jlvillal doesn't know enough about Swift to understand the difference. If the URL goes away how is the object still accessible? | 14:53 | |
| sambetts | hmm, maybe? but once we've provisioned the node will we need it again? I guess if we rebuild/ | 14:53 |
| sambetts | 14:53 | |
| sambetts | ? | 14:53 |
| sambetts | jlvillal: because you can ask swift to generate a new URL | 14:53 |
| jroll | uh, we broke new IPA + mitaka ironic?? | 14:53 |
| jroll | lucasagomes: ^ | 14:53 |
| jlvillal | sambetts, Using what as the argument? The old URL? | 14:53 |
| jroll | that should work, it should fall back | 14:54 |
| dtantsur | jlvillal, using container and object name | 14:54 |
| dtantsur | which we know, as we set them | 14:54 |
| dtantsur | jroll, WUT? | 14:54 |
| jlvillal | dtantsur, Okay. Thanks. | 14:54 |
| jroll | dtantsur: idk, reading scrollback | 14:54 |
| sambetts | dtantsur, jroll: the URL fallback logic should mean its not broken | 14:54 |
| dtantsur | ah, ok | 14:55 |
| lucasagomes | jroll, that's my theory. I'm looking at the code | 14:55 |
| sambetts | but if thats not working | 14:55 |
| jroll | code first checks api version and 404s if not high enough | 14:55 |
| jroll | er wait, this would be mitaka code | 14:55 |
| jroll | dtantsur: this is the error though http://paste.openstack.org/show/556066/ | 14:55 |
| lucasagomes | the error wznoinsk is a 401 | 14:56 |
| sambetts | jroll: right, but it should then fall back to the old URL if that fails as I understand it | 14:56 |
| jroll | oh you know what... | 14:56 |
| jroll | ironic checks auth before it routes | 14:56 |
| lucasagomes | https://github.com/openstack/ironic-python-agent/blob/87b2b82478fafe521efb6881dab0a278bbc60d72/ironic_python_agent/ironic_api_client.py#L155-L158 | 14:56 |
| dtantsur | sambetts, jroll, whoops, we don't fall back on 401 | 14:56 |
| * jlvillal for some reason thought jroll signed off for the weekend last night | 14:56 | |
| lucasagomes | jroll, yeah | 14:56 |
| jlvillal | :) | 14:56 |
| sambetts | dtantsur: oh | 14:56 |
| jroll | and in mitaka, we don't have /v1/lookup as auth-less | 14:56 |
| dtantsur | damn, I haven't thought about it. but it means the fix is trivial. wanna me to submit a patch? | 14:56 |
| jroll | jlvillal: this is what happens when we break things :D | 14:56 |
| jroll | dtantsur: yes please :) | 14:56 |
| jroll | I'm hoping to release IPA next week | 14:56 |
| dtantsur | I don't have a mitaka env to test it though | 14:57 |
| dtantsur | or do I? | 14:57 |
| jroll | I can test it next week | 14:57 |
| jroll | maybe we do need that ipa-src against mitaka job after all >.> | 14:57 |
| dtantsur | I think we do | 14:57 |
| sambetts | jroll: that should be covered by my spec adding IPA versioning I think | 14:57 |
| jroll | well, ipa master + ironic mitaka | 14:57 |
| dtantsur | do we have a bug # for it? | 14:58 |
| *** Romanenko_K has quit IRC | 14:58 | |
| jroll | I don't know | 14:58 |
| lucasagomes | dtantsur, apparently there's a flag in IPA to trigger one or another ? https://github.com/openstack/ironic-python-agent/blob/87b2b82478fafe521efb6881dab0a278bbc60d72/ironic_python_agent/ironic_api_client.py#L155-L158 | 14:58 |
| jroll | wznoinsk: did you file a bug for this? | 14:58 |
| jroll | or lucasagomes ^ | 14:58 |
| lucasagomes | nop, I'm investigating | 14:58 |
| sambetts | jroll, dtantsur: https://bugs.launchpad.net/ironic-python-agent/+bug/1602265 | 14:58 |
| openstack | Launchpad bug 1602265 in ironic-python-agent "[RFE] IPA API Versioning" [Wishlist,Confirmed] - Assigned to Sam Betts (sambetts) | 14:58 |
| wznoinsk | jroll: not yet, wasn't sure whether that's my misconfig first | 14:58 |
| dtantsur | lucasagomes, yes, but we don't fall back on HTTP 401 | 14:58 |
| jroll | lucasagomes: here's the fallback https://github.com/openstack/ironic-python-agent/blob/87b2b82478fafe521efb6881dab0a278bbc60d72/ironic_python_agent/ironic_api_client.py#L125 | 14:58 |
| *** rajinir has joined #openstack-ironic | 14:58 | |
| lucasagomes | jroll, a-ha yeah, we should check it out | 14:59 |
| lucasagomes | check 401 I mean | 14:59 |
| jroll | wznoinsk: it's supposed to work but doesn't :( | 14:59 |
| lucasagomes | dtantsur, you want me to take a look at it since I was doing it already? | 14:59 |
| lucasagomes | I can spawn a mitaka env for the tests | 14:59 |
| jroll | I can test it monday | 14:59 |
| jroll | or maybe wznoinsk has time to today :) | 15:00 |
| lucasagomes | ack, lemme open a bug then | 15:00 |
| lucasagomes | jroll, true | 15:00 |
| *** ijw has joined #openstack-ironic | 15:00 | |
| dtantsur | lucasagomes, feel free to handle it, if you feel like | 15:00 |
| * lucasagomes opens a bug | 15:00 | |
| jroll | thank you guys | 15:00 |
| lucasagomes | dtantsur, ack | 15:00 |
| dtantsur | thanks lucasagomes | 15:00 |
| wznoinsk | jroll: I'll test using mitaka IPA, kind of trying to get this setup working | 15:00 |
| * dtantsur is sad | 15:00 | |
| jroll | wznoinsk: okay :) | 15:00 |
| jroll | dtantsur: not your fault honestly | 15:01 |
| openstackgerrit | Vasyl Saienko proposed openstack/python-ironicclient: Add create external event functuonality https://review.openstack.org/354770 | 15:01 |
| xavierr | hey wznoinsk | 15:01 |
| dtantsur | well, it is my fault, though maybe not only mine ;) | 15:02 |
| jroll | dtantsur: +2'd that backport for you :)( | 15:02 |
| jroll | :) | 15:02 |
| wznoinsk | xavierr: hi | 15:02 |
| dtantsur | jroll, thnx | 15:02 |
| xavierr | wznoinsk: I was facing the same 401 error | 15:02 |
| * jroll goes back to doing whatever not work things he was doing | 15:02 | |
| dtantsur | oh, so we have Dell CI back (even though failing right now) | 15:02 |
| lucasagomes | jroll, if you have time mind taking a look at https://review.openstack.org/#/c/344179/ as well? Because I would like to get it merged before we release IPA to be honest | 15:02 |
| wznoinsk | xavierr: did that story end well? | 15:03 |
| lucasagomes | otherwise IPA will enable that cleaning step to wipe the disks metadata by default | 15:03 |
| xavierr | wznoinsk: yeap, try it using ipa-coreos-master.tar.gz | 15:03 |
| lucasagomes | and operators won't be able to disable it, unless they change the IPA's code | 15:03 |
| lucasagomes | which sucks | 15:03 |
| *** michauds has quit IRC | 15:03 | |
| *** sabeen has quit IRC | 15:04 | |
| xavierr | wznoinsk: tell me if it works for you | 15:05 |
| wznoinsk | xavierr: ok, I'm getting stable/mitaka IPA ready, then will try yours | 15:05 |
| xavierr | wznoinsk: go go go | 15:05 |
| dtantsur | lucasagomes, we definitely need a framework to change priorities for random steps. I've already shot down one request based on that, and honestly I feel like doing it again... | 15:06 |
| dtantsur | lucasagomes, though in your case it's at least not a 3rd party step | 15:06 |
| jroll | lucasagomes: fair enough, +2 | 15:06 |
| jroll | dtantsur: ++ | 15:06 |
| lucasagomes | dtantsur, yeah I totally dislike having steps on two sides | 15:06 |
| lucasagomes | I think we talked about it later | 15:06 |
| lucasagomes | I think all steps should be described in ironic-conductor, even if they are implemented in IPA | 15:06 |
| * lucasagomes envision something like ironic/clean-step.d/ | 15:07 | |
| * jroll actually goes now | 15:07 | |
| dtantsur | lucasagomes, yeah... I guess we don't have time to specify it right here and now for your case, but I'd prefer we think about it before we introduce the next clean step | 15:07 |
| xavierr | wznoinsk: https://drive.google.com/open?id=0Bx3iHZG5D0neckhLb2hqTFZsTDQ | 15:08 |
| lucasagomes | dtantsur, right, yeah it's too late. And in any case we will need a spec | 15:08 |
| lucasagomes | and I bet it's going to be hard to make it backward compatible | 15:08 |
| dtantsur | inspector folks, sambetts, aarefiev, ifarkas, do you think we can make our grenade job voting? | 15:08 |
| mat128 | dtantsur: are you fixing https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/ironic_api_client.py#L125 ? | 15:08 |
| dtantsur | mat128, lucasagomes, not me. but yes, we need to fix this bit | 15:09 |
| mat128 | ok I can fix it, should we check any 4xx code? | 15:09 |
| ifarkas | dtantsur, looks stable enough, so why not | 15:09 |
| dtantsur | mat128, I was thinking about it; not sure though | 15:11 |
| dtantsur | lucasagomes, +A | 15:11 |
| mat128 | ok, I'll submit a bug | 15:11 |
| mat128 | then we can resolve it | 15:11 |
| mat128 | only check 401? | 15:11 |
| dtantsur | 401 and 404, I think | 15:11 |
| mat128 | it could be something else, based on policy | 15:11 |
| mat128 | ok | 15:11 |
| lucasagomes | mat128, I was about to fix it | 15:12 |
| lucasagomes | just opened a bug | 15:12 |
| mat128 | ah | 15:12 |
| lucasagomes | https://bugs.launchpad.net/ironic-python-agent/+bug/1612696 | 15:12 |
| openstack | Launchpad bug 1612696 in ironic-python-agent "Lookup does not fallback on 401" [Critical,New] - Assigned to Lucas Alvares Gomes (lucasagomes) | 15:12 |
| mat128 | ok then | 15:12 |
| mat128 | :) | 15:12 |
| mat128 | saves me typing | 15:12 |
| mat128 | ah you mentioned it already in the scrollback | 15:13 |
| * mat128 fails at reading sometimes | 15:13 | |
| *** praneshp has quit IRC | 15:14 | |
| lucasagomes | yeah, I was looking at the code while talking to wznoinsk already | 15:15 |
| *** michauds has joined #openstack-ironic | 15:18 | |
| *** pcaruana has quit IRC | 15:23 | |
| *** jistr is now known as jistr|call | 15:23 | |
| *** e0ne has quit IRC | 15:26 | |
| *** praneshp has joined #openstack-ironic | 15:27 | |
| rcarrillocruz | jroll , TheJulia : is this https://github.com/openstack/bifrost/blob/master/playbooks/roles/bifrost-ironic-install/tasks/bootstrap.yml#L32 really needed? | 15:29 |
| rcarrillocruz | it breaks local fqdn resolution on the bifrost machine | 15:29 |
| rcarrillocruz | which prevents from correctly running puppet | 15:29 |
| *** sabeen has joined #openstack-ironic | 15:29 | |
| rcarrillocruz | since puppet uses fqdn to check what classes belong to a given server | 15:29 |
| rcarrillocruz | and even, is it *really* needed rabbitmq in bifrost? | 15:33 |
| *** ifarkas is now known as ifarkas_afk | 15:33 | |
| *** sdake_ has joined #openstack-ironic | 15:39 | |
| *** sdake has quit IRC | 15:41 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Fallback to the old /lookup endpoint on 401 https://review.openstack.org/354802 | 15:42 |
| lucasagomes | dtantsur, mat128 jroll wznoinsk ^ need tests yet | 15:42 |
| *** jistr|call is now known as jistr | 15:42 | |
| mat128 | lucasagomes: commented, `import requests` left over? | 15:43 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient: Add --wait flag for provision actions and wait_for_provision_state function https://review.openstack.org/328191 | 15:44 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient: [WIP] Add --wait to OSC provisioning commands https://review.openstack.org/354806 | 15:44 |
| dtantsur | the first of these patches could use some reviews ^^^ | 15:44 |
| *** jpich has quit IRC | 15:45 | |
| lucasagomes | mat128, it's being used on L352 and L355 | 15:45 |
| openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/354453 | 15:46 |
| mat128 | AHhhh | 15:46 |
| * mat128 fails | 15:46 | |
| *** michauds has quit IRC | 15:46 | |
| mat128 | was hardcoded | 15:46 |
| mat128 | got it | 15:46 |
| lucasagomes | mat128, heh it's just more readable (IMO) to use the error names instead of the codes | 15:47 |
| lucasagomes | we do it in the code as well | 15:47 |
| mat128 | yes | 15:47 |
| mat128 | I agree, I just didnt see it | 15:47 |
| *** rcernin has quit IRC | 15:47 | |
| openstackgerrit | Merged openstack/python-ironicclient: Remove unused variables assignments in OSC plugin unit tests https://review.openstack.org/353558 | 15:48 |
| openstackgerrit | Merged openstack/python-ironicclient: Deprecate -l option for port creation in OSC plugin https://review.openstack.org/351836 | 15:49 |
| openstackgerrit | Merged openstack/python-ironicclient: Trivial: remove redundant parentheses https://review.openstack.org/353592 | 15:49 |
| openstackgerrit | Merged openstack/python-ironicclient: Add baremetal port unset command to OSC plugin https://review.openstack.org/347402 | 15:49 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow to define vms with independent naems https://review.openstack.org/354813 | 15:53 |
| lucasagomes | mat128, :D happens | 15:54 |
| * lucasagomes hands mat128 a cup of coffee | 15:54 | |
| mat128 | brb I'll go make myself a third coffee :) | 15:54 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow to define vms with independent names https://review.openstack.org/354813 | 15:54 |
| *** krtaylor has quit IRC | 15:59 | |
| *** sdake_ has quit IRC | 15:59 | |
| *** ksumit has joined #openstack-ironic | 16:00 | |
| *** michauds has joined #openstack-ironic | 16:02 | |
| jlvillal | dtantsur: Another random gertty question :) When you check out a patch, how does it know where to check it out? Not sure if you use that feature. | 16:03 |
| dtantsur | jlvillal, it used the directory where is keeps all the git repos. it's in your gertty configuration file | 16:04 |
| jlvillal | dtantsur: Thanks, I'll look at that file again :) | 16:04 |
| dtantsur | for me it's ~/Gertty or something like that, I don't mix it with my usual ~/Projects, so it could be handy | 16:04 |
| openstackgerrit | David Lenwell proposed openstack/ironic: Add REST API for volume connector and volume target operation https://review.openstack.org/214586 | 16:04 |
| jlvillal | dtantsur: Ah, I see it has a default value which is nothing like where I keep my stuff :) | 16:05 |
| jlvillal | dtantsur: Thanks! | 16:05 |
| dtantsur | heh | 16:05 |
| *** wajdi has joined #openstack-ironic | 16:07 | |
| *** praneshp has quit IRC | 16:08 | |
| *** PollyZ has joined #openstack-ironic | 16:10 | |
| NobodyCam | Good Morning Ironicers | 16:12 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic-python-agent: Use CoreOS 1068.9.0 https://review.openstack.org/354821 | 16:13 |
| mat128 | morning NobodyCam | 16:13 |
| lucasagomes | NobodyCam, morning | 16:13 |
| wznoinsk | lucasagomes: when build deploy ramdisk with export DIB_REPOREF_ironic_agent=stable/mitaka | 16:13 |
| wznoinsk | it helped with my problem, I see agent now reported some details to ironic but it's not deploying the user image/complains about conflict error sending hearbeat | 16:13 |
| * lucasagomes is having trouble setting up devstack on stable/mitaka | 16:14 | |
| aarefiev | dtantsur: +1 from me, looks pretty stable, I've seen problems only with tempest basic | 16:14 |
| lucasagomes | wznoinsk, do you have the output handy there? | 16:14 |
| mat128 | lucasagomes: thank you :) https://www.dropbox.com/s/a8otaxkx8mm927y/2016-08-12%2016.13.39.jpg?dl=0 | 16:14 |
| lucasagomes | mat128, nice one :D | 16:14 |
| NobodyCam | morning mat128 lucasagomes dtantsur jlvillal jroll mgould xavierr and all others not listed here :) | 16:15 |
| *** adu has joined #openstack-ironic | 16:15 | |
| NobodyCam | oh and ofc TGIF | 16:15 |
| lucasagomes | NobodyCam, ++ | 16:15 |
| NobodyCam | :) | 16:16 |
| wznoinsk | http://paste.openstack.org/show/556130/ lucasagomes | 16:16 |
| lucasagomes | urgh, someone knows how to deploy devstack on mitaka? I'm setting all projects *_BRANCH=stable/mitaka | 16:16 |
| lucasagomes | but it's failing :-( | 16:16 |
| xavierr | hey NobodyCam o/ | 16:17 |
| lucasagomes | wznoinsk, I see, it should retry the heartbeat request tho | 16:17 |
| lucasagomes | wznoinsk, anything on the ironic-api logs? | 16:17 |
| lucasagomes | or ironic-conductor? | 16:17 |
| vdrok | lucasagomes: I think you just need to do git checkout stable/mitaka in devstack dir | 16:17 |
| vdrok | that's it | 16:17 |
| lucasagomes | vdrok, oh really? ok lemme give it a go | 16:18 |
| lucasagomes | thanks :D | 16:18 |
| vdrok | morning jlvillal and NobodyCam | 16:18 |
| jlvillal | Good morning NobodyCam vdrok :) | 16:18 |
| mgould | morning NobodyCam vdrok | 16:18 |
| vdrok | hey mgould :) | 16:19 |
| dtantsur | morning NobodyCam | 16:19 |
| *** clenimar_ has joined #openstack-ironic | 16:20 | |
| wznoinsk | lucasagomes: http://paste.openstack.org/show/556132/ - not sure tho whether it's locked by my deployment task or my deployment task is being blocked by it | 16:20 |
| lucasagomes | wznoinsk, hmm what's the provision_state of the node? | 16:21 |
| lucasagomes | "wait call-back" ? | 16:21 |
| *** narasimha_SV has quit IRC | 16:22 | |
| wznoinsk | http://paste.openstack.org/show/556134/ | 16:22 |
| wznoinsk | but 'deploying' will be before it enters wait call-back and after (I think)? | 16:22 |
| lucasagomes | wznoinsk, correct, deploying basically means the conductor is doing something with the node | 16:23 |
| lucasagomes | wait call-back means the conductor is waiting on an external service (in this case the deploy ramdisk) | 16:23 |
| *** ksumit has quit IRC | 16:23 | |
| lucasagomes | wznoinsk, so it switches between one and another depending on the task | 16:23 |
| lucasagomes | wznoinsk, can you restart the ironic-conductor please? That will force it to clean the reservations | 16:23 |
| *** ksumit has joined #openstack-ironic | 16:24 | |
| lucasagomes | tho I don't think it will solve your problem once you try to deploy again | 16:24 |
| NobodyCam | :) morning vdrok :) | 16:24 |
| lucasagomes | reservation: silpixa00385229.ir.intel.com | 16:24 |
| wznoinsk | yes, I guess that was placed when IPA reported back to ironic conductor to silpixa00385229.ir.intel.com | 16:25 |
| xavierr | hey dtantsur | 16:25 |
| wznoinsk | ok, will deploy again, node was treated with power state off | 16:25 |
| dtantsur | hi xavierr | 16:26 |
| *** clenimar_ has quit IRC | 16:26 | |
| xavierr | dtantsur: I'm trying to enable inspection (to do in-band) on OneView drivers | 16:26 |
| xavierr | dtantsur: I have a devstack up and running ironic-inspector | 16:27 |
| NobodyCam | hey dtantsur I had a quick question about inspector. can I access the port object in a rule? I wanted to add some info to port.extra kinda like: {"action": "set-attribute", "path": "port.extra/reserved", "value": "true"} | 16:27 |
| TheJulia | rcarrillocruz: it may not be needed any more, but it was required to even do a stand-alone install of rabbitmq a while back | 16:27 |
| xavierr | dtantsur: and a have made these changes http://paste.openstack.org/show/556135/ | 16:28 |
| NobodyCam | morning TheJulia | 16:28 |
| dtantsur | NobodyCam, hmm, I don't think we support setting port attributes now.. e.g. in your example, which exactly port are you trying to update? | 16:28 |
| xavierr | dtantsur: but the node is going from manageable to cleaning without doing inspection :/ | 16:28 |
| xavierr | dtantsur: any idea? | 16:29 |
| TheJulia | good morning NobodyCam | 16:29 |
| NobodyCam | when I discover a node. I want to tag the ports for that node. with extra data like pxe and such. | 16:29 |
| dtantsur | xavierr, inspection does not happen automatically, you need to explicitly run it with provision verb "active" | 16:29 |
| dtantsur | NobodyCam, all ports with the same data? | 16:30 |
| *** fragatina has joined #openstack-ironic | 16:30 | |
| *** PollyZ has quit IRC | 16:31 | |
| NobodyCam | dtantsur: nope.. I need to register all port but tag (in port.extra) things like the pie port | 16:31 |
| NobodyCam | pie = pxe | 16:31 |
| *** fragatin_ has joined #openstack-ironic | 16:31 | |
| *** athomas has quit IRC | 16:32 | |
| openstackgerrit | Vasyl Saienko proposed openstack/python-ironicclient: Add create external event functionality https://review.openstack.org/354770 | 16:32 |
| xavierr | so, if I have a node in manageable state and do a provide (via API) it won't do inspection? | 16:32 |
| xavierr | dtantsur: ^ | 16:33 |
| dtantsur | xavierr, correct | 16:33 |
| xavierr | ok dtantsur, thanks! o/ | 16:34 |
| dtantsur | NobodyCam, I think it justifies the whole plugin (just make sure it is not covered by sambetts' local_link_information plugin) | 16:34 |
| *** xavierr is now known as xavierr_afk | 16:34 | |
| NobodyCam | ack :) | 16:34 |
| wznoinsk | lucasagomes: http://paste.openstack.org/show/556136/ can it be it just waits for the command? | 16:34 |
| *** fragatina has quit IRC | 16:35 | |
| NobodyCam | I will start to dig in to that later today or maybe even monday depending on how the day goes :) | 16:35 |
| *** ohamada has quit IRC | 16:35 | |
| *** tesseract- has quit IRC | 16:35 | |
| lucasagomes | wznoinsk, yay | 16:36 |
| lucasagomes | wznoinsk, yeah IPA will wait for ironic to instruct it what to do next | 16:36 |
| *** electrofelix has quit IRC | 16:37 | |
| wznoinsk | lucasagomes: should it start deploying user image? maybe it's waiting for the exclusive lock (as per the paste above) | 16:37 |
| *** PollyZ has joined #openstack-ironic | 16:37 | |
| lucasagomes | wznoinsk, what driver are you using? | 16:37 |
| wznoinsk | pxe_ipmitool | 16:37 |
| lucasagomes | wznoinsk, right | 16:37 |
| lucasagomes | wznoinsk, is the node in deploying state again? | 16:37 |
| wznoinsk | yes | 16:38 |
| dtantsur | meh, quite a few things seem broken on stable/mitaka... | 16:38 |
| sambetts | :( | 16:38 |
| wznoinsk | I'm deploying using kolla, may be some not-quite-tested configuraiton | 16:39 |
| wznoinsk | lucasagomes: have to go now, leave a msg if you have some ideas, I'll get back to it later | 16:39 |
| wznoinsk | thanks for your help so far | 16:39 |
| lucasagomes | wznoinsk, cool, no worries | 16:40 |
| lucasagomes | I'm going pretty soon too | 16:40 |
| lucasagomes | it's friday and almost 6pm :-) | 16:40 |
| lucasagomes | wznoinsk, but next week we sort that out | 16:40 |
| dtantsur | nearly 7pm for me, yeah, time to go :) | 16:40 |
| lucasagomes | dtantsur, go! | 16:40 |
| lucasagomes | dtantsur, have a great evening, enjoy the weekend | 16:40 |
| dtantsur | have a great weekend! | 16:40 |
| dtantsur | g'night lucasagomes | 16:40 |
| *** dtantsur is now known as dtantsur|afk | 16:41 | |
| vdrok | night dtantsur|afk | 16:42 |
| *** aNupoisc has joined #openstack-ironic | 16:43 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 16:44 |
| devananda | morning, all | 16:45 |
| vdrok | morning devananda | 16:45 |
| mgould | morning devananda | 16:46 |
| *** PollyZ_ has joined #openstack-ironic | 16:47 | |
| jlvillal | dtantsur|afk: Good night | 16:47 |
| *** PollyZ has quit IRC | 16:47 | |
| openstackgerrit | Merged openstack/ironic: Add "erase_devices_metadata_priority" config option https://review.openstack.org/344179 | 16:47 |
| jlvillal | morning devananda | 16:47 |
| *** [1]cdearborn has quit IRC | 16:49 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Fix object save after refresh failure https://review.openstack.org/354744 | 16:50 |
| TheJulia | good morning devananda | 16:52 |
| vdrok | devananda: lucasagomes jlvillal TheJulia if you have a bit of time, could you please check https://review.openstack.org/354744 ? | 16:52 |
| devananda | jroll: re "baremetal" vs "bare metal" what is your opinion on when we should separate the words vs. not? | 16:53 |
| *** PollyZ_ has quit IRC | 16:54 | |
| lucasagomes | vdrok, I'm about to call it a day | 16:54 |
| lucasagomes | it's ok if I look at it next week? | 16:54 |
| vdrok | lucasagomes: yeah sure :) | 16:54 |
| vdrok | enjoy the weekend :) | 16:55 |
| lucasagomes | cool | 16:55 |
| lucasagomes | that said, have a great eveinng all | 16:55 |
| lucasagomes | talk to you all soon | 16:55 |
| lucasagomes | vdrok, you too | 16:55 |
| vdrok | night lucasagomes | 16:55 |
| *** lucasagomes is now known as lucas-dinner | 16:55 | |
| *** e0ne has joined #openstack-ironic | 16:58 | |
| *** anush_ has quit IRC | 17:00 | |
| NobodyCam | night lucas-dinner | 17:01 |
| NobodyCam | morning devananda | 17:02 |
| sambetts | ome | 17:04 |
| * sambetts -> home | 17:04 | |
| *** causten has quit IRC | 17:04 | |
| sambetts | Night all, have a great weekend! | 17:04 |
| *** sambetts is now known as sambetts|afk | 17:04 | |
| *** causten has joined #openstack-ironic | 17:05 | |
| NobodyCam | night sambetts|afk | 17:06 |
| *** causten has quit IRC | 17:09 | |
| *** sabeen has quit IRC | 17:14 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Fix object save after refresh failure https://review.openstack.org/354744 | 17:16 |
| *** baoli has quit IRC | 17:19 | |
| *** ksumit has quit IRC | 17:20 | |
| *** krtaylor has joined #openstack-ironic | 17:24 | |
| jlvillal | vdrok: I don't understand the object stuff enough to be a good reviewer for that patch. I added tanlin to the patch as he knows a lot about objects. | 17:25 |
| vdrok | jlvillal: thx, me neither, that's why I wanted some eyes :) | 17:26 |
| jlvillal | vdrok: heh. Maybe superdan but I know he is busy with the Nova stuff. | 17:26 |
| superdan | vdrok: jlvillal: https://github.com/openstack/nova/blob/master/nova/objects/instance.py#L795 | 17:27 |
| vdrok | superdan: ah, great! :) | 17:27 |
| *** amoralej is now known as amoralej|off | 17:27 | |
| vdrok | thanks | 17:28 |
| jlvillal | Thanks superdan you are .... super ;) | 17:28 |
| superdan | today at least.. ;) | 17:28 |
| jlvillal | heh | 17:28 |
| *** sabeen has joined #openstack-ironic | 17:30 | |
| *** e0ne has quit IRC | 17:31 | |
| *** bvandewa has joined #openstack-ironic | 17:32 | |
| vdrok | time to leave, good night everyone! | 17:35 |
| jlvillal | vdrok: Dos ve dos! | 17:35 |
| xavierr_afk | good night vdrok | 17:37 |
| *** xavierr_afk is now known as xavierr | 17:37 | |
| *** bvandewa has quit IRC | 17:39 | |
| *** bvandewa has joined #openstack-ironic | 17:39 | |
| *** michauds has quit IRC | 17:39 | |
| *** ijw has quit IRC | 17:40 | |
| *** e0ne has joined #openstack-ironic | 17:42 | |
| *** sambetts|afk has quit IRC | 17:43 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 17:44 |
| *** sambetts has joined #openstack-ironic | 17:45 | |
| mgould | I'm going on holiday for the next three weeks: see you all when I get back! | 17:48 |
| mgould | follow https://twitter.com/andymilesexped for occasional updates, if you're interested | 17:49 |
| devananda | mgould: enjoy! | 17:49 |
| mgould | devananda: thanks! | 17:49 |
| *** mgould has quit IRC | 17:50 | |
| *** michauds has joined #openstack-ironic | 17:51 | |
| *** sabeen has quit IRC | 17:52 | |
| *** devananda has quit IRC | 17:54 | |
| *** krtaylor has quit IRC | 17:54 | |
| cinerama | mgould: wow best of luck | 17:56 |
| *** [1]cdearborn has joined #openstack-ironic | 17:56 | |
| *** fxpester_home has joined #openstack-ironic | 18:01 | |
| *** fragatin_ has quit IRC | 18:02 | |
| *** pester has quit IRC | 18:04 | |
| *** joprovost1 has joined #openstack-ironic | 18:08 | |
| *** joprovost has quit IRC | 18:08 | |
| *** joprovost1 is now known as joprovost | 18:08 | |
| *** bvandewa has quit IRC | 18:09 | |
| *** rama_y has quit IRC | 18:13 | |
| *** adu has quit IRC | 18:14 | |
| *** adu has joined #openstack-ironic | 18:15 | |
| *** baoli has joined #openstack-ironic | 18:17 | |
| *** adu has quit IRC | 18:17 | |
| *** bvandewa has joined #openstack-ironic | 18:18 | |
| *** sdake has joined #openstack-ironic | 18:21 | |
| *** amcguign has joined #openstack-ironic | 18:25 | |
| *** devananda has joined #openstack-ironic | 18:26 | |
| *** amcguign has quit IRC | 18:26 | |
| openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Declare strings explicitly as binary https://review.openstack.org/354892 | 18:27 |
| openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Implement identify workaround for ThinkServer SD350 https://review.openstack.org/354894 | 18:29 |
| *** krtaylor has joined #openstack-ironic | 18:39 | |
| *** amotoki has quit IRC | 18:42 | |
| *** rama_y has joined #openstack-ironic | 18:44 | |
| *** bvandewa has quit IRC | 18:44 | |
| *** sdake has quit IRC | 18:45 | |
| *** ksumit has joined #openstack-ironic | 18:51 | |
| *** mgoddard_ has joined #openstack-ironic | 18:54 | |
| *** mgoddard has quit IRC | 18:57 | |
| *** mgoddard_ has quit IRC | 18:59 | |
| *** e0ne has quit IRC | 19:01 | |
| *** fragatina has joined #openstack-ironic | 19:01 | |
| *** krtaylor has quit IRC | 19:07 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 19:07 |
| *** krtaylor has joined #openstack-ironic | 19:08 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Introducing node introspection state management https://review.openstack.org/348943 | 19:13 |
| apughazh | This patch needs a workflow +1, could someone please take a look? | 19:17 |
| apughazh | https://review.openstack.org/#/c/354136/4 | 19:17 |
| *** milan has quit IRC | 19:20 | |
| *** ksumit has quit IRC | 19:20 | |
| *** ksumit has joined #openstack-ironic | 19:22 | |
| *** ksumit has quit IRC | 19:22 | |
| *** ksumit has joined #openstack-ironic | 19:22 | |
| *** ksumit has quit IRC | 19:24 | |
| *** aNupoisc has quit IRC | 19:25 | |
| *** rajinir has quit IRC | 19:25 | |
| *** fragatina has quit IRC | 19:28 | |
| *** fragatina has joined #openstack-ironic | 19:29 | |
| *** fragatina has quit IRC | 19:29 | |
| cinerama | jlvillal: do you have a link handy for your devstack/vagrant/ci stuff? | 19:34 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 19:36 |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Enable support for OSprofiler in Ironic https://review.openstack.org/337987 | 19:37 |
| *** joprovost has quit IRC | 19:49 | |
| TheJulia | cinerama: https://github.com/JohnVillalovos/devstack-gate-test might be what your looking for | 20:08 |
| cinerama | yes! thanks | 20:08 |
| TheJulia | apughazh: Looking | 20:11 |
| apughazh | TheJulia: Thanks! | 20:12 |
| TheJulia | apughazh: sorry | 20:17 |
| *** dprince has quit IRC | 20:25 | |
| apughazh | TheJulia: no prob | 20:25 |
| apughazh | TheJulia: I just posted a comment, let me know if it sounds good | 20:26 |
| *** rajinir has joined #openstack-ironic | 20:27 | |
| TheJulia | apughazh: Ahh, I see the discussion on rev2 | 20:29 |
| TheJulia | apughazh: That sounds good, that at least sets the context of system wide instead of "maybe libvirt and openvswitch" which is exactly what my brain thought when I read the sentence. | 20:29 |
| apughazh | TheJulia: Ok, I'll put it in! | 20:31 |
| openstackgerrit | Akilan Pughazhendi proposed openstack/ironic: Doc: Recommend users to update their systems https://review.openstack.org/354136 | 20:32 |
| *** sdake has joined #openstack-ironic | 20:32 | |
| TheJulia | apughazh: Awesome, thank you, +2 posted | 20:33 |
| TheJulia | apughazh: Again, sorry for the additional headache | 20:33 |
| apughazh | TheJulia: no problem, thanks for the input! | 20:34 |
| *** sabeen has joined #openstack-ironic | 20:40 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add test for ironic port-list command https://review.openstack.org/329068 | 20:40 |
| *** aNupoisc has joined #openstack-ironic | 20:48 | |
| jlvillal | cinerama: What TheJulia said. Sorry I was out renewing my passport... | 20:48 |
| -openstackstatus- NOTICE: The Gerrit service on review.openstack.org is restarting for a scheduled upgrade, but should return to service momentarily: http://lists.openstack.org/pipermail/openstack-dev/2016-August/101394.html | 20:49 | |
| xavierr | see you monday guys! | 20:52 |
| xavierr | have a nice weekend | 20:52 |
| *** xavierr is now known as xavierr_afk | 20:52 | |
| -openstackstatus- NOTICE: The Mediawiki service at wiki.openstack.org will be offline from 21:00 UTC until approximately 23:00 UTC for a planned upgrade http://lists.openstack.org/pipermail/openstack-dev/2016-August/101395.html | 20:57 | |
| *** ChanServ changes topic to "The Mediawiki service at wiki.openstack.org will be offline from 21:00 UTC until approximately 23:00 UTC for a planned upgrade http://lists.openstack.org/pipermail/openstack-dev/2016-August/101395.html" | 20:57 | |
| *** wajdi_ has joined #openstack-ironic | 21:08 | |
| *** wajdi has quit IRC | 21:08 | |
| *** fragatina has joined #openstack-ironic | 21:11 | |
| *** wajdi has joined #openstack-ironic | 21:13 | |
| *** rbudden has quit IRC | 21:15 | |
| *** wajdi_ has quit IRC | 21:16 | |
| *** wajdi has quit IRC | 21:19 | |
| *** fragatina has quit IRC | 21:19 | |
| *** sdake has quit IRC | 21:23 | |
| *** lascii is now known as alaski | 21:23 | |
| *** rama_y has quit IRC | 21:31 | |
| *** adu has joined #openstack-ironic | 21:38 | |
| *** adu has quit IRC | 21:44 | |
| *** jistr has quit IRC | 21:47 | |
| *** adu has joined #openstack-ironic | 21:48 | |
| *** ppiela has quit IRC | 21:48 | |
| *** fragatina has joined #openstack-ironic | 21:48 | |
| *** fragatina has quit IRC | 21:50 | |
| *** fragatina has joined #openstack-ironic | 21:50 | |
| *** jistr has joined #openstack-ironic | 21:51 | |
| *** ijw has joined #openstack-ironic | 22:00 | |
| *** jistr has quit IRC | 22:00 | |
| *** RomanenkoK has joined #openstack-ironic | 22:08 | |
| *** jistr has joined #openstack-ironic | 22:09 | |
| *** michauds has quit IRC | 22:11 | |
| *** beekneemech is now known as bnemec-pto | 22:12 | |
| jroll | devananda: honestly "baremetal" and "bare metal" are synonyms to me, I do think we should try to be consistent, and I prefer the latter. that said, spaces make it weird for code things, and there's already service catalog / OSC / etc, and we should probably leave that alone. so maybe the former in user-facing code, and the latter in english? (I believe URLs should be closer to english) | 22:13 |
| *** bfournie has quit IRC | 22:14 | |
| *** [1]cdearborn has quit IRC | 22:16 | |
| *** lynxman has quit IRC | 22:18 | |
| *** jistr has quit IRC | 22:19 | |
| *** jistr has joined #openstack-ironic | 22:25 | |
| *** rama_y has joined #openstack-ironic | 22:27 | |
| *** aNupoisc has quit IRC | 22:27 | |
| *** ijw has quit IRC | 22:28 | |
| *** adu has quit IRC | 22:43 | |
| *** lynxman has joined #openstack-ironic | 22:43 | |
| -openstackstatus- NOTICE: ok https://wiki.openstack.org/ is now running Mediawiki 1.27.0; please let us know in #openstack-infra if anything seems wrong | 23:00 | |
| *** ChanServ changes topic to "ok https://wiki.openstack.org/ is now running Mediawiki 1.27.0; please let us know in #openstack-infra if anything seems wrong" | 23:00 | |
| *** vishwanathj has quit IRC | 23:02 | |
| *** ChanServ changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic" | 23:06 | |
| -openstackstatus- NOTICE: https://wiki.openstack.org/ is now running Mediawiki 1.27.0; please let us know in #openstack-infra if anything seems wrong | 23:06 | |
| *** bfournie has joined #openstack-ironic | 23:13 | |
| *** Goneri has quit IRC | 23:26 | |
| *** RomanenkoK has quit IRC | 23:33 | |
| *** rama_y has quit IRC | 23:33 | |
| *** sdake has joined #openstack-ironic | 23:33 | |
| *** rama_y has joined #openstack-ironic | 23:33 | |
| *** ijw has joined #openstack-ironic | 23:34 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP: Just curious https://review.openstack.org/355112 | 23:36 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP: Testing https://review.openstack.org/355113 | 23:38 |
| *** ijw has quit IRC | 23:41 | |
| *** ijw has joined #openstack-ironic | 23:46 | |
| *** rama_y has quit IRC | 23:47 | |
| *** baoli has quit IRC | 23:56 | |
| *** hparekh has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
