*** baoli has quit IRC | 00:00 | |
*** raddaoui has joined #openstack-ironic | 00:24 | |
*** chlong has quit IRC | 00:32 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Introduce libvirt dirver https://review.openstack.org/253096 | 00:33 |
---|---|---|
*** Sukhdev has joined #openstack-ironic | 00:45 | |
*** Sukhdev has quit IRC | 00:46 | |
*** chlong has joined #openstack-ironic | 00:49 | |
*** hoangcx has joined #openstack-ironic | 00:53 | |
*** dims has quit IRC | 01:03 | |
*** chlong has quit IRC | 01:09 | |
*** hoangcx has quit IRC | 01:15 | |
*** hoangcx has joined #openstack-ironic | 01:16 | |
*** chlong has joined #openstack-ironic | 01:23 | |
*** hoangcx has quit IRC | 01:27 | |
*** hoangcx has joined #openstack-ironic | 01:29 | |
*** yonglihe has joined #openstack-ironic | 01:34 | |
*** dims has joined #openstack-ironic | 01:37 | |
*** piet has joined #openstack-ironic | 01:51 | |
*** kan_ has joined #openstack-ironic | 02:05 | |
*** dims has quit IRC | 02:07 | |
*** piet has quit IRC | 02:11 | |
*** Sukhdev has joined #openstack-ironic | 02:21 | |
*** hoangcx has quit IRC | 02:25 | |
openstackgerrit | Kan proposed openstack/ironic: API to list nodes using the same driver https://review.openstack.org/264005 | 02:29 |
*** hoangcx has joined #openstack-ironic | 02:36 | |
*** hoangcx has quit IRC | 02:37 | |
*** hoangcx has joined #openstack-ironic | 02:37 | |
*** hoangcx has quit IRC | 02:38 | |
*** hoangcx has joined #openstack-ironic | 02:40 | |
openstackgerrit | Kan proposed openstack/python-ironicclient: Add CLI to list nodes using the same driver https://review.openstack.org/264007 | 02:40 |
*** hoangcx_ has joined #openstack-ironic | 02:42 | |
*** hoangcx has quit IRC | 02:44 | |
openstackgerrit | Tan Lin proposed openstack/ironic: Add xinetd and its configurs in Install Guide https://review.openstack.org/261429 | 03:02 |
openstackgerrit | Tan Lin proposed openstack/ironic: Add choices option to several options https://review.openstack.org/251732 | 03:34 |
*** vinm213 has joined #openstack-ironic | 03:37 | |
*** links has joined #openstack-ironic | 03:48 | |
*** Haomeng has quit IRC | 03:55 | |
*** hoangcx_ is now known as hoangcx | 03:56 | |
*** deray has joined #openstack-ironic | 04:07 | |
*** kalpase has joined #openstack-ironic | 04:33 | |
*** davidlenwell_ is now known as davidlenwell | 04:35 | |
*** coolsvap|away is now known as coolsvap | 04:41 | |
*** david-lyle has quit IRC | 05:00 | |
*** david-lyle has joined #openstack-ironic | 05:00 | |
*** piet has joined #openstack-ironic | 05:12 | |
*** vishwanathj has joined #openstack-ironic | 05:13 | |
*** teju has joined #openstack-ironic | 05:21 | |
*** deray has quit IRC | 05:26 | |
*** piet has quit IRC | 05:27 | |
*** deray has joined #openstack-ironic | 05:28 | |
teju | how to change provision state of a node from active to available | 05:30 |
teju | ? | 05:30 |
*** deray_ has joined #openstack-ironic | 05:39 | |
*** deray has quit IRC | 05:42 | |
*** raddaoui has quit IRC | 05:46 | |
*** stendulker has joined #openstack-ironic | 05:48 | |
*** vaishnavi has joined #openstack-ironic | 06:03 | |
lintan | teju, you can delete the instance then the node will become available again. | 06:05 |
lintan | ironic node-set-provision-state node deleted | 06:06 |
*** baoli_ has quit IRC | 06:15 | |
*** Jabbaqak has joined #openstack-ironic | 06:15 | |
Jabbaqak | Che etc za hren' | 06:16 |
*** raddaoui has joined #openstack-ironic | 06:17 | |
*** teju has quit IRC | 06:18 | |
*** Jabbaqak has left #openstack-ironic | 06:21 | |
vaishnavi | /msg NickServ VERIFY REGISTER vaishnavi rkytulxkevyu | 06:22 |
*** Haomeng has joined #openstack-ironic | 06:26 | |
*** vaishnavi_r has joined #openstack-ironic | 06:29 | |
*** ChubYann has quit IRC | 06:37 | |
*** ionutbalutoiu has quit IRC | 06:41 | |
*** bkumar has joined #openstack-ironic | 06:48 | |
*** ionutbalutoiu has joined #openstack-ironic | 06:51 | |
*** vaishnavi has quit IRC | 06:51 | |
*** yolanda has joined #openstack-ironic | 06:53 | |
openstackgerrit | Tan Lin proposed openstack/ironic: Wait longer for AMT firmware from deep sleep https://review.openstack.org/264106 | 06:55 |
*** raddaoui has quit IRC | 06:56 | |
openstackgerrit | Tan Lin proposed openstack/ironic: Wait longer for AMT firmware from deep sleep https://review.openstack.org/264106 | 06:57 |
*** Marga_ has joined #openstack-ironic | 07:01 | |
*** ionutbalutoiu has quit IRC | 07:03 | |
openstackgerrit | Vaishnavi proposed openstack/ironic: Add support for building ISO for deploy ramdisk https://review.openstack.org/264590 | 07:05 |
*** pas-ha has joined #openstack-ironic | 07:07 | |
pas-ha | morning Ironic | 07:10 |
*** athomas has joined #openstack-ironic | 07:14 | |
*** Marga_ has quit IRC | 07:17 | |
*** Marga_ has joined #openstack-ironic | 07:18 | |
*** raddaoui has joined #openstack-ironic | 07:25 | |
*** Marga__ has joined #openstack-ironic | 07:30 | |
*** Marga_ has quit IRC | 07:30 | |
*** ukalifon1 has joined #openstack-ironic | 07:33 | |
*** Marga__ has quit IRC | 07:35 | |
*** chlong has quit IRC | 07:39 | |
*** Marga_ has joined #openstack-ironic | 07:46 | |
*** Marga_ has quit IRC | 07:50 | |
*** raddaoui has quit IRC | 08:02 | |
*** Sukhdev has quit IRC | 08:08 | |
*** praneshp has quit IRC | 08:10 | |
*** ionutbalutoiu has joined #openstack-ironic | 08:19 | |
*** ankkumar5 has joined #openstack-ironic | 08:24 | |
*** vinm213 has quit IRC | 08:24 | |
*** deray_ has quit IRC | 08:28 | |
*** boris-42 has joined #openstack-ironic | 08:28 | |
*** ifarkas has joined #openstack-ironic | 08:31 | |
*** vinm213 has joined #openstack-ironic | 08:31 | |
*** openstackgerrit has quit IRC | 08:32 | |
*** openstackgerrit has joined #openstack-ironic | 08:33 | |
*** deray has joined #openstack-ironic | 08:34 | |
*** teju has joined #openstack-ironic | 08:53 | |
openstackgerrit | Vaishnavi proposed openstack/ironic: Add support for building ISO for deploy ramdisk https://review.openstack.org/264590 | 08:58 |
*** derekh has joined #openstack-ironic | 09:03 | |
*** athomas has quit IRC | 09:04 | |
*** mbound has joined #openstack-ironic | 09:05 | |
*** yolanda has quit IRC | 09:05 | |
*** yolanda has joined #openstack-ironic | 09:05 | |
*** e0ne has joined #openstack-ironic | 09:05 | |
*** athomas has joined #openstack-ironic | 09:08 | |
*** daemontool has joined #openstack-ironic | 09:14 | |
openstackgerrit | Tan Lin proposed openstack/ironic: Refactor install-guide to configure API/Conductor seperately https://review.openstack.org/261443 | 09:18 |
*** ndipanov has joined #openstack-ironic | 09:25 | |
*** daemontool has quit IRC | 09:25 | |
*** jistr has joined #openstack-ironic | 09:25 | |
*** daemontool has joined #openstack-ironic | 09:25 | |
*** ndipanov has quit IRC | 09:26 | |
*** daemontool has quit IRC | 09:27 | |
*** daemontool has joined #openstack-ironic | 09:29 | |
*** ionutbalutoiu has quit IRC | 09:37 | |
*** ionutbalutoiu has joined #openstack-ironic | 09:38 | |
*** aparnav has joined #openstack-ironic | 09:42 | |
openstackgerrit | Debayan Ray proposed openstack/proliantutils: common, ris, ribcl changes to support firmware update https://review.openstack.org/203543 | 09:44 |
*** ndipanov has joined #openstack-ironic | 09:44 | |
openstackgerrit | Imre Farkas proposed openstack/ironic: DRAC: switch to python-dracclient on vendor-passthru https://review.openstack.org/251294 | 09:48 |
openstackgerrit | Imre Farkas proposed openstack/ironic: DRAC: cleanup after switch to python-dracclient https://review.openstack.org/255310 | 09:48 |
openstackgerrit | vinay kumar muddu proposed openstack/ironic: Fix issues with uefi-ipxe booting https://review.openstack.org/265742 | 09:51 |
*** daemontool has quit IRC | 09:53 | |
*** daemontool has joined #openstack-ironic | 09:54 | |
*** daemontool has quit IRC | 09:55 | |
*** daemontool has joined #openstack-ironic | 09:56 | |
*** electrofelix has joined #openstack-ironic | 09:56 | |
*** daemontool has quit IRC | 09:57 | |
openstackgerrit | Imre Farkas proposed openstack/ironic: DRAC: switch to python-dracclient on vendor-passthru https://review.openstack.org/251294 | 09:58 |
openstackgerrit | Imre Farkas proposed openstack/ironic: DRAC: cleanup after switch to python-dracclient https://review.openstack.org/255310 | 09:58 |
*** daemontool has joined #openstack-ironic | 09:59 | |
*** ankkumar5 has left #openstack-ironic | 10:04 | |
*** ankkumar5 has joined #openstack-ironic | 10:05 | |
*** aarefiev has joined #openstack-ironic | 10:10 | |
*** daemontool_ has joined #openstack-ironic | 10:10 | |
openstackgerrit | vinay kumar muddu proposed openstack/ironic: Fix issues with uefi-ipxe booting https://review.openstack.org/265742 | 10:12 |
*** daemontool has quit IRC | 10:13 | |
openstackgerrit | Sam Betts proposed openstack/ironic: [WIP] Enable tinyipa for devstack Ironic https://review.openstack.org/259089 | 10:14 |
*** sambetts has joined #openstack-ironic | 10:14 | |
sambetts | morning all o/ | 10:15 |
*** yolanda has quit IRC | 10:15 | |
*** daemontool_ has quit IRC | 10:17 | |
*** daemontool_ has joined #openstack-ironic | 10:17 | |
*** yolanda has joined #openstack-ironic | 10:17 | |
deray | sambetts, g'morning o/ | 10:22 |
sambetts | o/ deray | 10:23 |
*** mgould has joined #openstack-ironic | 10:24 | |
*** dtantsur|afk is now known as dtantsur | 10:32 | |
dtantsur | Good monday morning folks :) | 10:33 |
mgould | morning Ironic! | 10:37 |
sambetts | o/ mgould dtantsur | 10:38 |
dtantsur | o/ | 10:47 |
mgould | sambetts dtantsur morning! | 10:48 |
*** Pespa has joined #openstack-ironic | 10:56 | |
*** hoangcx has quit IRC | 10:58 | |
*** stendulker has quit IRC | 11:08 | |
vdrok | good morning sambetts deray dtantsur and mgould ! | 11:11 |
deray | vdrok, g'morning :) | 11:12 |
*** bkumar has quit IRC | 11:12 | |
*** baoli has joined #openstack-ironic | 11:14 | |
mgould | vdrok deray, good morning! | 11:16 |
dtantsur | folks, our gate is broken right now, please wait before rechecking anything | 11:17 |
*** coolsvap is now known as coolsvap|away | 11:17 | |
deray | mgould, g'morning o/ | 11:18 |
*** dims_ has joined #openstack-ironic | 11:18 | |
*** lucas-dinner is now known as lucasagomes | 11:23 | |
lucasagomes | mgoddard, dtantsur deray good morning! | 11:23 |
dtantsur | morning lucasagomes | 11:23 |
deray | lucasagomes, g'morning :) | 11:23 |
dtantsur | lucasagomes, the deploy-ironic syntax error finally bites us :( | 11:23 |
lucasagomes | dtantsur, oh the patch wasn't approved yet? | 11:24 |
* lucasagomes looks | 11:24 | |
dtantsur | lucasagomes, nope, pinged #tripleo, but it's too early | 11:25 |
lucasagomes | :-( | 11:26 |
*** mbound has quit IRC | 11:28 | |
*** vinm213 has quit IRC | 11:29 | |
mgoddard | mgoddard | 11:29 |
mgoddard | doh, not what was intended :) | 11:29 |
mgoddard | morning lucasgomes | 11:30 |
lucasagomes | heh | 11:30 |
mgoddard | had a question for you guys actually | 11:30 |
mgoddard | I'm attending the Nova mid-cycle in a couple of weeks as it's in Bristol | 11:31 |
*** daemontool_ is now known as daemontool | 11:32 | |
mgoddard | Are there any Ironic-related Nova issues that need looking at? | 11:32 |
mgoddard | I could try to progress https://bugs.launchpad.net/ironic/+bug/1405131, but it really needs discussion more than anything | 11:34 |
openstack | Launchpad bug 1405131 in OpenStack Compute (nova) "Ports cannot be mapped to networks" [Low,In progress] - Assigned to Mark Goddard (mgoddard) | 11:34 |
lucasagomes | mgoddard, hmm yeah I believe that will be important, as well as the https://blueprints.launchpad.net/nova/+spec/ironic-networks-support | 11:35 |
deray | mgoddard, lucasagomes, could this be one of the items? https://review.openstack.org/#/c/211101/ | 11:36 |
*** jaypipes has joined #openstack-ironic | 11:37 | |
lucasagomes | deray, totally! | 11:37 |
mgoddard | deray, would love for that one to go through - we are supporting a custom implementation internally | 11:37 |
deray | lucasagomes, mgoddard okay .. | 11:38 |
mgoddard | like many spec reviews I'm subscribed to, it's gone quiet :( | 11:38 |
deray | mgoddard, ohkay.. | 11:39 |
*** sinval has joined #openstack-ironic | 11:39 | |
sinval | good morning folks o/ | 11:40 |
deray | are we planning to have a spec-less bluepritn for this one as well? is that in line? Satoru Moriya .. r u listening? | 11:40 |
deray | sinval, morning o/ | 11:41 |
deray | mgoddard, "we are supporting a custom implementation internally" - plz explain | 11:41 |
mgoddard | we needed Cinder/Ironic integration, so implemented it ourselves. We roughly followed the proposed data model, but used driver_info instead of the new DB tables | 11:43 |
mgoddard | We ended up with a new deploy driver which performs in-band iSCSI attachment via kernel command line arguments to Dracut | 11:44 |
mgoddard | Not ideal, but our hw doesn't support out of band iSCSI attachment | 11:44 |
openstackgerrit | Sam Betts proposed openstack/ironic-python-agent: Add tinyipa to IPA imagebuild directory https://review.openstack.org/234902 | 11:47 |
openstackgerrit | Sam Betts proposed openstack/ironic: [WIP] Enable tinyipa for devstack Ironic https://review.openstack.org/259089 | 11:48 |
mgoddard | I don't think there's anything confidential - we can provide source, but I don't know how useful it will be for the volume connection info work | 11:50 |
*** daemontool_ has joined #openstack-ironic | 11:53 | |
deray | mgoddard, okay .. should be fine. will let you know if I plan to look into the code. thx for the update | 11:54 |
*** daemontool has quit IRC | 11:55 | |
*** chlong has joined #openstack-ironic | 11:57 | |
*** ipukha has joined #openstack-ironic | 12:01 | |
*** mbound has joined #openstack-ironic | 12:01 | |
lucasagomes | anteaya, thanks for the ping! The patch was updated, we've moved the code from devstack to our gate that's why I abandoned the patch in devstack | 12:05 |
lucasagomes | to our tree* | 12:05 |
lucasagomes | anteaya, I've updated the project-config patch to point to the right dependent patch | 12:05 |
lucasagomes | dtantsur, re 257302 it was long time ago, I think I've tried setup_lib | 12:09 |
lucasagomes | but it didn't work | 12:09 |
lucasagomes | the right fix should be to merge https://review.openstack.org/#/c/219562/ | 12:09 |
dtantsur | yeah | 12:09 |
TheJulia | Good morning everyone | 12:10 |
dtantsur | morning TheJulia | 12:11 |
*** daemontool has joined #openstack-ironic | 12:12 | |
sinval | morning TheJulia | 12:13 |
*** daemontool_ has quit IRC | 12:14 | |
*** derekh has quit IRC | 12:15 | |
lucasagomes | TheJulia, sinval good morning! | 12:20 |
sinval | morning lucasagomes o/ | 12:21 |
*** raildo-afk is now known as raildo | 12:21 | |
*** vaishnavi_r has quit IRC | 12:24 | |
*** derekh has joined #openstack-ironic | 12:25 | |
TheJulia | mgoddard: it would be kind of awesome if that source made it at least into a review to spawn ideas. I'd really like to see that implemented, and I know there is a lot of interest from our userbase. | 12:26 |
*** HenryG has quit IRC | 12:27 | |
*** HenryG has joined #openstack-ironic | 12:27 | |
deray | TheJulia, second your thought | 12:30 |
TheJulia | and when I say a lot, I mean _a lot_ | 12:31 |
*** cgascoig has quit IRC | 12:35 | |
*** mkovacik has quit IRC | 12:36 | |
sinval | hey folks, quick question: is there any reason for a 4.3 driver versin does not work with 4.2? | 12:37 |
sinval | version* | 12:37 |
deray | dtantsur jroll jlvillal lucasagomes NobodyCam rloo TheJulia and all others.. if you have some free time then can you have a look at https://review.openstack.org/#/c/211019/? | 12:37 |
dtantsur | sinval, I'm not sure such compatibility is guaranteed, but I don't remember any reasons for it not to | 12:39 |
sinval | dtantsur: me too, and by reading the release notes I don't think that there is a reason, thanks | 12:47 |
*** trown|outttypeww is now known as trown | 12:52 | |
TheJulia | sinval: I seem to remember that was a break in the driver interface, but it may have been fixed before 4.2 was released. | 13:02 |
vdrok | morning lucasagomes sinval and TheJulia | 13:03 |
sinval | TheJulia: oh thanks, I'm going to check it | 13:04 |
sinval | morning vdrok | 13:04 |
ankkumar5 | hi vdrok, dtantsur can u look into the patch https://review.openstack.org/#/c/259921/3 | 13:06 |
dtantsur | ankkumar5, could you please fix all the -1's before asking more people to look at it? | 13:07 |
*** daemontool has quit IRC | 13:07 | |
ankkumar5 | dtantsur, ya i am working on that.. thank u | 13:08 |
*** daemontool has joined #openstack-ironic | 13:10 | |
*** links has quit IRC | 13:13 | |
lucasagomes | vdrok, hello there | 13:16 |
*** aparnav has quit IRC | 13:19 | |
zhenguo | morning lucasagomes, dtantsur, vdrok and all others | 13:20 |
*** erwan_taf has joined #openstack-ironic | 13:21 | |
dtantsur | o/ | 13:21 |
erwan_taf | o/ | 13:21 |
vdrok | morning :) | 13:21 |
*** daemontool_ has joined #openstack-ironic | 13:23 | |
*** dprince has joined #openstack-ironic | 13:26 | |
*** daemontool has quit IRC | 13:26 | |
openstackgerrit | Sam Betts proposed openstack/ironic-python-agent: Add tinyipa to IPA imagebuild directory https://review.openstack.org/234902 | 13:29 |
*** yolanda has quit IRC | 13:31 | |
*** yolanda has joined #openstack-ironic | 13:32 | |
mgoddard | TheJulia: that should not be a problem. I'll check that we're ok to release it | 13:33 |
sergek | o/ | 13:35 |
TheJulia | mgoddard: awesome! | 13:39 |
TheJulia | mgoddard: I would just put something in the commit message to let people know what to expect :) | 13:40 |
mgoddard | TheJulia: Will do. It's currently based on Kilo, so may need some work to port to Mitaka | 13:41 |
openstackgerrit | Sam Betts proposed openstack/ironic: [WIP] Enable tinyipa for devstack Ironic https://review.openstack.org/259089 | 13:43 |
mgoddard | TheJulia: just got the green light, I'll put a review together | 13:43 |
sambetts | lucasagomes: is there a way to enable the debug config option on IPA? | 13:50 |
jroll | sambetts: --debug on the ipa command line | 13:53 |
jroll | morning all | 13:53 |
sambetts | jroll: do I need to modify my ramdisk to make it do that? or can I do it via a kernel param? | 13:54 |
*** ankkumar5 has quit IRC | 13:54 | |
*** kan_ has quit IRC | 13:55 | |
jroll | sambetts: ramdisk afaik, maybe kernel param though | 13:59 |
jroll | nope https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/cmd/agent.py | 13:59 |
sambetts | :( | 13:59 |
jroll | patches welcome :P | 14:00 |
*** rloo has joined #openstack-ironic | 14:03 | |
lucasagomes | sambetts, sorry the delay, what jroll said | 14:04 |
lucasagomes | and morning :-) | 14:04 |
sambetts | jroll, lucasagomes :-P ah well | 14:04 |
* lucasagomes checks | 14:05 | |
*** [1]cdearborn has joined #openstack-ironic | 14:06 | |
*** krtaylor has quit IRC | 14:06 | |
NobodyCam | Good MOrning Ironicers | 14:12 |
mgould | NobodyCam, morning! | 14:14 |
NobodyCam | morning mgould :) | 14:15 |
dtantsur | morning jroll, NobodyCam | 14:17 |
jroll | \o | 14:18 |
NobodyCam | morning dtantsur, jroll, rloo, sambetts, vdrok, devananda, jlvillal and all others not listed here | 14:18 |
NobodyCam | :) | 14:18 |
sambetts | o/ NobodyCam | 14:18 |
vdrok | morning jroll and NobodyCam ! | 14:18 |
*** krtaylor has joined #openstack-ironic | 14:19 | |
NobodyCam | :) | 14:19 |
sinval | good morning NobodyCam | 14:19 |
sinval | and everyone else :) | 14:19 |
NobodyCam | morning sinval :) | 14:19 |
*** thrash|g0ne is now known as thrash | 14:22 | |
*** lucasagomes is now known as lucas-hungry | 14:23 | |
*** jdk has joined #openstack-ironic | 14:24 | |
jdk | I've been using bifrost/ironic for a month or more. I have a "best practices" type of question. I'm developing disk images quite regularly so I need to re-deploy often. | 14:26 |
rloo | Good morning NobodyCam, mgould, sambetts, vdrok, sinval, jdk, dtantsur, jroll, lucas-hungry, and everyone else! | 14:28 |
jdk | should I be updating instance_info using the command line client? what is the minimum inventory I need to provide as input to deploy-dynamic.yaml? | 14:28 |
sambetts | Hi rloo :D | 14:28 |
vdrok | morning rloo | 14:28 |
mgould | rloo, goot morning! | 14:29 |
mgould | *good | 14:29 |
mgould | and sinval jdk jroll and anyone else I've missed :-) | 14:29 |
dtantsur | morning rloo | 14:30 |
rloo | mgould: :) | 14:30 |
TheJulia | jdk: you can feed it in via the inventory file if you so desire | 14:30 |
*** piet has joined #openstack-ironic | 14:31 | |
TheJulia | jdk: minimum is a very good question. Driver, credentials for the driver... I think that would be it at a bare minimum | 14:31 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add Link-Local-Connection info to ironic port https://review.openstack.org/256365 | 14:32 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add configure_provision_network function https://review.openstack.org/256367 | 14:32 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update Ironic VM network connection https://review.openstack.org/256366 | 14:32 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add Ironic/Neutron integration documentation https://review.openstack.org/258596 | 14:32 |
*** bkumar has joined #openstack-ironic | 14:36 | |
jdk | good morning all | 14:36 |
jdk | TheJulia: thanks, I've been using a full json file with all the detail needed to enroll the hardware but then found it was becoming a pain to update it for re-deploy | 14:37 |
TheJulia | Yeah, if memory serves if no instance_info is defined, it just uses the defaults | 14:39 |
TheJulia | which can be overridden on the command line | 14:39 |
mgoddard | TheJulia, deray: Our Cinder/Ironic driver is up for review https://review.openstack.org/#/c/265856/ | 14:44 |
anteaya | lucas-hungry: thank you | 14:44 |
deray | mgoddard, thank you | 14:45 |
mgoddard | there is Nova driver code to go with it, which I can post, but let's just see what the interest is like first | 14:45 |
TheJulia | mgoddard: Awesome | 14:45 |
mgoddard | we're using it with the multi-attach Cinder work, and will be pointing it at a 128 node cluster shortly | 14:47 |
rloo | jroll, lucas-hungry: why did we have to go back to using 'ironic-tempest-plugin'? | 14:48 |
rloo | jroll, lucas-hungry: https://review.openstack.org/#/c/246161/ | 14:48 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Migrate Tempest tests into Ironic tree https://review.openstack.org/253982 | 14:49 |
*** vinm213 has joined #openstack-ironic | 14:49 | |
jroll | rloo: because we need to be able to match test names with a regex in the gate, and there wasn't any good identifiers without something like that | 14:50 |
jroll | rloo: see also https://review.openstack.org/#/c/265311/1/jenkins/jobs/ironic.yaml | 14:50 |
rloo | jroll: so sad. ok thx. | 14:51 |
jroll | devananda: lucas-hungry: just added a license file to 253982 to satisfy pep8 | 14:51 |
TheJulia | mgoddard: I think you may want to mark that rev as -1 workflow :) | 14:55 |
TheJulia | just so it is more clear :) | 14:55 |
openstackgerrit | Sam Betts proposed openstack/ironic-python-agent: Add tinyipa to IPA imagebuild directory https://review.openstack.org/234902 | 14:57 |
*** ukalifon1 has quit IRC | 14:58 | |
openstackgerrit | Sam Betts proposed openstack/ironic: [WIP] Enable tinyipa for devstack Ironic https://review.openstack.org/259089 | 14:59 |
*** deray has quit IRC | 14:59 | |
*** kalpase has quit IRC | 15:00 | |
jlvillal | Good morning NobodyCam. You wake up so early :) | 15:01 |
TheJulia | jlvillal: He has been up for... at least a half an hour :) | 15:01 |
jlvillal | Good morning TheJulia jroll rloo mgoddard derekh anteaya lucas-hungry dtantsur and everyone else :) | 15:01 |
mgoddard | TheJulia: I was looking for that button but couldn't find it. It doesn't appear for me in reply | 15:02 |
TheJulia | good morning jlvillal | 15:02 |
mgoddard | morning jlvilla1 | 15:02 |
jlvillal | TheJulia, That is so early ;) | 15:02 |
TheJulia | mgoddard: that... seems... a liottle weird... | 15:02 |
TheJulia | s/lio/li/ | 15:02 |
anteaya | morning jlvillal | 15:03 |
mgoddard | TheJulia: it does. | 15:04 |
dtantsur | morning jlvillal | 15:05 |
jlvillal | :) | 15:06 |
*** mkovacik has joined #openstack-ironic | 15:09 | |
dtantsur | folks, is there a bug tracker for ironic-webclient? I'd prefer not to track their bugs with ironic's.. | 15:14 |
dtantsur | krotscheck, betherly ^^ | 15:14 |
krotscheck | dtantsur: Urm... I think there's a launchpad group? | 15:16 |
krotscheck | Maybe? | 15:16 |
* krotscheck may not have set that up | 15:17 | |
dtantsur | krotscheck, could you please setup a bug tracker for it? and move https://bugs.launchpad.net/ironic/+bug/1531100 there? | 15:17 |
openstack | Launchpad bug 1531100 in Ironic "ironic-webclient not working if input api url ends with '/'" [Undecided,New] - Assigned to Haomeng,Wang (whaom) | 15:17 |
*** thrash has quit IRC | 15:17 | |
*** bkumar has quit IRC | 15:17 | |
krotscheck | dtantsur: That one merged already | 15:18 |
dtantsur | I guess so. it wasn't updated, because it is on a wrong project | 15:18 |
*** raddaoui has joined #openstack-ironic | 15:19 | |
*** thrash has joined #openstack-ironic | 15:20 | |
*** thrash has quit IRC | 15:20 | |
*** thrash has joined #openstack-ironic | 15:20 | |
*** lucas-hungry is now known as lucasagomes | 15:25 | |
lucasagomes | jlvillal, rloo morning | 15:26 |
lucasagomes | rloo, yeah cause of the regex in project-config | 15:26 |
lucasagomes | jroll, thanks! | 15:26 |
*** teju has left #openstack-ironic | 15:27 | |
rloo | lucasagomes: too bad, but that's life. thx. | 15:28 |
rloo | morning jlvillal | 15:28 |
jlvillal | :) | 15:28 |
*** piet has quit IRC | 15:29 | |
*** piet has joined #openstack-ironic | 15:29 | |
dtantsur | lucasagomes, is there something else to do for https://bugs.launchpad.net/ironic/+bug/1526518 ? | 15:33 |
openstack | Launchpad bug 1526518 in Ironic "Check the syntax of the bash files under devstack/" [Low,In progress] - Assigned to Lucas Alvares Gomes (lucasagomes) | 15:33 |
* lucasagomes looks | 15:33 | |
lucasagomes | dtantsur, I think we can close it now | 15:34 |
openstackgerrit | Anton Arefiev proposed openstack/python-ironicclient: Use requests lib in HTTPClient https://review.openstack.org/229286 | 15:35 |
vinm213 | Hello all | 15:39 |
vinm213 | GM | 15:39 |
vinm213 | I have created a bug related UEFI-IPXE https://bugs.launchpad.net/ironic/+bug/1532679 | 15:41 |
openstack | Launchpad bug 1532679 in Ironic "ipxe booting of node in uefi boot mode only boots deploy image and not the user image after reboot" [Undecided,In progress] - Assigned to vmud213 (vmud213) | 15:41 |
*** ukalifon has joined #openstack-ironic | 15:42 | |
devananda | morning, all | 15:42 |
vinm213 | can you confirm the bug? | 15:42 |
*** raddaoui has quit IRC | 15:43 | |
dtantsur | lucasagomes, ^^ | 15:43 |
lucasagomes | vinm213, thanks for the bug | 15:43 |
dtantsur | devananda, morning | 15:43 |
lucasagomes | vinm213, it seems valid, I've seem the patch fixing it too | 15:44 |
vinm213 | Thanks | 15:44 |
lucasagomes | or a patch fixing it* | 15:44 |
* lucasagomes finds the links | 15:44 | |
vinm213 | Can you please review it . | 15:44 |
lucasagomes | https://review.openstack.org/#/c/265742/ | 15:44 |
*** Sukhdev has joined #openstack-ironic | 15:44 | |
lucasagomes | vinm213, did already. It's missing unittests | 15:44 |
lucasagomes | other than that it LGTM | 15:45 |
jroll | \o devananda | 15:45 |
*** rpioso has joined #openstack-ironic | 15:45 | |
lucasagomes | devananda, morning | 15:45 |
vinm213 | ok. Sure will update the unit tests. | 15:46 |
lucasagomes | vinm213, thanks a lot | 15:47 |
*** ukalifon has quit IRC | 15:49 | |
vdrok | morning jlvillal and devananda | 15:51 |
*** mtanino has joined #openstack-ironic | 15:53 | |
*** lazy_prince has joined #openstack-ironic | 15:56 | |
devananda | dtantsur: wait, what? dib gate is not testing the proposed changes to dib? | 15:58 |
dtantsur | devananda, believe me or not, yes | 15:59 |
* dtantsur was *surprised* as well | 15:59 | |
devananda | :( | 15:59 |
devananda | wtf?? | 15:59 |
lucasagomes | devananda, no | 15:59 |
lucasagomes | remember when we had that problem with ubuntu changing the codename to the version number | 15:59 |
lucasagomes | ? | 15:59 |
lucasagomes | I've put some patches fixing it and adding a job to test dib from source | 16:00 |
lucasagomes | but they haven't got merged since | 16:00 |
devananda | have you pointed this out to anyone in -infra? | 16:00 |
lucasagomes | devananda, there's a change in devstack, ironic and project-config | 16:01 |
* lucasagomes find the patches | 16:01 | |
devananda | lucasagomes: I see them | 16:01 |
devananda | just very surprised | 16:01 |
dtantsur | patches are on the whiteboard | 16:01 |
lucasagomes | devananda, I kinda forgot because after ubuntu fixed the codiname vs version number the gate got fixed | 16:01 |
lucasagomes | but yeah, we should enable a job like that asap | 16:01 |
devananda | yup | 16:02 |
jroll | +1 | 16:05 |
dtantsur | devananda, jroll, pin for DIB 1.6.0 was approved | 16:06 |
jlvillal | vdrok, devananda Good morning | 16:06 |
*** ukalifon1 has joined #openstack-ironic | 16:06 | |
jroll | dtantsur: yay, thanks | 16:07 |
lucasagomes | dtantsur, fwiw tha workaround for the "pip install -e" works | 16:07 |
dtantsur | cool | 16:07 |
lucasagomes | so we can land w/o the proper fix and update it later (if the proper fix don't get in first) | 16:07 |
dtantsur | FYI: I plan to -1 all changes to deploy-ironic element, unless they're critical bug fixes | 16:07 |
dtantsur | lucasagomes, yeah | 16:08 |
dtantsur | lucasagomes, could you remind me a link? | 16:08 |
lucasagomes | dtantsur, https://review.openstack.org/#/c/257302/2/devstack/lib/ironic | 16:08 |
jroll | dtantsur: +1, kill it with fire | 16:09 |
jlvillal | :) | 16:10 |
*** ukalifon1 has quit IRC | 16:15 | |
*** daemontool_ has quit IRC | 16:21 | |
openstackgerrit | yolanda.robla proposed openstack/bifrost: Add inventory_dns feature to bifrost https://review.openstack.org/265630 | 16:23 |
*** deray has joined #openstack-ironic | 16:25 | |
*** lazy_prince has quit IRC | 16:25 | |
*** killer_prince has joined #openstack-ironic | 16:26 | |
*** lazy_prince has joined #openstack-ironic | 16:26 | |
*** lazy_prince has quit IRC | 16:26 | |
openstackgerrit | yolanda.robla proposed openstack/bifrost: Add inventory_dns feature to bifrost https://review.openstack.org/265630 | 16:26 |
openstackgerrit | yolanda.robla proposed openstack/bifrost: Add inventory_dns feature to bifrost https://review.openstack.org/265630 | 16:35 |
dtantsur | folks, is ATC registration for the summit open already? | 16:35 |
sambetts | dtantsur: I've not had an email with a code or anything yet | 16:37 |
dtantsur | ah, so it's not only me... c'mon, I need a visa invitation! | 16:37 |
lucasagomes | dtantsur, didn't get an email yet either | 16:38 |
dtantsur | meh.. | 16:38 |
* lucasagomes gotta update his waiver visa too | 16:38 | |
sambetts | lucasagomes: https://review.openstack.org/#/c/259089/ !!! Its passing!!! | 16:39 |
lucasagomes | w00t! | 16:40 |
lucasagomes | sambetts, wa | 16:40 |
lucasagomes | sambetts, what was it?* | 16:40 |
sambetts | lucasagomes: I had to add a udevadm settle, to the script because /dev wasn't getting updated fast enough | 16:41 |
sambetts | it was running the dd before /dev/vda2 had been created | 16:41 |
lucasagomes | sambetts, nice one | 16:41 |
dtantsur | sambetts, it's not the final idea of this patch, right? cause it removes building IPA from source :) | 16:41 |
NobodyCam | light (no) agenda today :) | 16:42 |
sambetts | dtantsur: what do you mean? its still building IPA from source | 16:42 |
dtantsur | sambetts, well, it breaks inspector gate (see the -nv job) | 16:42 |
dtantsur | so at quick look I got impression that it removes some IPA bits | 16:42 |
dtantsur | sambetts, ah no, it adds a git_clone command, so you should add IPA to all PROJECTS in project-config | 16:43 |
*** jdk has quit IRC | 16:43 | |
*** alaski has joined #openstack-ironic | 16:44 | |
sambetts | dtantsur: yeah, the devstack lib isn't in its final form, but the tiny core IPA patch is now working, lucasagomes I also managed to get it to build tgt and qemu-utils from source in the chroot build process instead of haven't them in the repo which is awesome :) | 16:45 |
lucasagomes | niceee | 16:45 |
*** piet has quit IRC | 16:48 | |
*** raddaoui has joined #openstack-ironic | 16:51 | |
*** yolanda has quit IRC | 16:55 | |
*** yolanda has joined #openstack-ironic | 16:55 | |
*** dims has joined #openstack-ironic | 16:56 | |
*** piet has joined #openstack-ironic | 16:56 | |
*** dims_ has quit IRC | 16:56 | |
*** mgoddard_ has joined #openstack-ironic | 16:59 | |
*** vishwana_ has joined #openstack-ironic | 17:00 | |
rloo | meeting now? | 17:00 |
*** mgoddard has quit IRC | 17:01 | |
jlvillal | [1]cdearborn, You are just a foot note :P | 17:01 |
*** vishwanathj has quit IRC | 17:03 | |
jlvillal | [1]cdearborn, I couldn't resist with [1] tacked on to the front of your name :) | 17:05 |
*** killer_prince has quit IRC | 17:10 | |
*** daemontool has joined #openstack-ironic | 17:10 | |
*** daemontool_ has joined #openstack-ironic | 17:12 | |
*** ifarkas has quit IRC | 17:15 | |
*** daemontool has quit IRC | 17:15 | |
openstackgerrit | yolanda.robla proposed openstack/bifrost: Add inventory_dns feature to bifrost https://review.openstack.org/265630 | 17:17 |
*** vinm213 has quit IRC | 17:17 | |
*** raddaoui has quit IRC | 17:25 | |
*** jaybeale has joined #openstack-ironic | 17:26 | |
*** raddaoui has joined #openstack-ironic | 17:30 | |
*** mbound has quit IRC | 17:40 | |
*** deray has quit IRC | 17:41 | |
*** vishwanathj has joined #openstack-ironic | 17:46 | |
*** trown is now known as trown|lunch | 17:47 | |
*** vishwana_ has quit IRC | 17:48 | |
*** lekha|away is now known as lekha | 17:48 | |
*** derekh has quit IRC | 17:51 | |
*** ionutbalutoiu has quit IRC | 17:52 | |
*** teju has joined #openstack-ironic | 17:53 | |
openstackgerrit | yolanda.robla proposed openstack/bifrost: Add inventory_dns feature to bifrost https://review.openstack.org/265630 | 17:54 |
*** penick has joined #openstack-ironic | 17:55 | |
jroll | I'm not sure I understand what ironic functional tests would test, fwiw | 17:59 |
lucasagomes | we can continue here | 17:59 |
jroll | api? | 17:59 |
jroll | we already have tempest to test the api | 17:59 |
*** ijw has joined #openstack-ironic | 17:59 | |
jlvillal | jroll, Yes I would think the API to start. | 17:59 |
jlvillal | Test communication from API to conductor. | 18:00 |
jlvillal | I would assume it would all be API calls. | 18:00 |
dtantsur | jroll, tempest requires devstack, my point is that lightweight functional testing would work just on a laptop with working python | 18:00 |
dtantsur | again, I'm basing my judgement on what inspector does | 18:00 |
sergek | jroll: my understanding is that functional test more close to real use case than unit ones | 18:00 |
*** e0ne has quit IRC | 18:01 | |
dtantsur | we mock ironicclient and keystoneclient and then call main(args) to launch the inspector service | 18:01 |
jroll | dtantsur: right, so we mock the backend of the api and poke at it and verify mocks? | 18:01 |
jroll | e.g. mock dbapi | 18:01 |
jroll | and rpcapi | 18:01 |
dtantsur | jroll, we use sqlite in temporary location | 18:01 |
dtantsur | so we do test it, including migrations | 18:01 |
*** raddaoui has quit IRC | 18:01 | |
dtantsur | we only mock 3rd party service clients (and don't use ramdisk, obviously) | 18:02 |
jroll | dtantsur: well, with conductor and oslo.msg that's a bit more complex | 18:02 |
jroll | but yeah, I see your point | 18:02 |
dtantsur | I'll think better about it tomorrow, and sum it up on the ML | 18:02 |
dtantsur | today I'm already too sleepy :) | 18:02 |
jroll | cool, ty | 18:02 |
sergek | dtantsur: ty | 18:02 |
dtantsur | good night everyone | 18:02 |
*** dtantsur is now known as dtantsur|afk | 18:03 | |
sergek | jroll: btw is there any magic behind tempest plugin? https://review.openstack.org/#/c/253982/14 | 18:03 |
lucasagomes | dtantsur|afk, night | 18:03 |
jroll | sergek: as long as ironic and tempest are both installed in the environment, it will be loaded correctly by tempest via entry_points | 18:04 |
sergek | I feel dumb or sleepy already :) | 18:04 |
jroll | sergek: see https://review.openstack.org/#/c/246161/11 and http://docs.openstack.org/developer/tempest/plugin.html | 18:04 |
jroll | bbiab | 18:04 |
sergek | jroll: ty. I'll continue tomorrow | 18:04 |
*** ijw has quit IRC | 18:04 | |
*** raddaoui has joined #openstack-ironic | 18:04 | |
jlvillal | sergek, Thanks for looking into functional testing. I will get back on it once I make progress on grenade. | 18:05 |
sergek | yeah. I cherry picked both of them | 18:05 |
jlvillal | sergek, Hopefully we can work together on it :) | 18:05 |
sergek | jlvillal: I only intended to add one more test and get lost :) | 18:05 |
jlvillal | :) | 18:06 |
sergek | jlvillal: definitely having the only ironic services is tempting. I'd give it a try | 18:06 |
sergek | jlvillal: Devstack is interesting is being a bit more "real" setup | 18:07 |
sergek | *as | 18:07 |
sergek | anyway I'd like to contribute in any of the option when I enforce them working in my env | 18:08 |
sergek | good night all o/ | 18:08 |
mgould | sergek, good night! | 18:08 |
sergek | mgould: :) | 18:09 |
*** SpamapS has quit IRC | 18:10 | |
*** athomas has quit IRC | 18:10 | |
*** raddaoui has quit IRC | 18:12 | |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Introducing API for aborting introspection https://review.openstack.org/265275 | 18:13 |
*** jistr has quit IRC | 18:15 | |
*** krtaylor has quit IRC | 18:17 | |
*** Sukhdev has quit IRC | 18:17 | |
*** mkovacik has quit IRC | 18:17 | |
*** athomas has joined #openstack-ironic | 18:17 | |
openstackgerrit | yolanda.robla proposed openstack/bifrost: Add inventory_dns feature to bifrost https://review.openstack.org/265630 | 18:19 |
*** Sukhdev has joined #openstack-ironic | 18:20 | |
openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Add OEM sensors for Lenovo Nextscale https://review.openstack.org/265974 | 18:21 |
*** teju has quit IRC | 18:22 | |
lucasagomes | NobodyCam, was you that was working on having common properties for credentials in Ironic? | 18:23 |
NobodyCam | me? | 18:23 |
lucasagomes | cause I thought about just adding something simple such as a "username", "password" and "address" properties in driver_info for the drivers in tree | 18:24 |
lucasagomes | NobodyCam, I thought you were looking at it, maybe I'm wrong | 18:24 |
NobodyCam | humm I was not looking at I can, but that suggestion does not sound to off base | 18:25 |
rloo | lucasagomes: I thought that was something we discussed several releases ago, but I could be wrong. | 18:25 |
lucasagomes | NobodyCam, oh ack, sorry for the confusion then | 18:26 |
NobodyCam | np :) | 18:26 |
lucasagomes | rloo, right, yeah... I was just wondering whether we want something simple as adding new common properties (like we did for deploy_ramdisk, deploy_kernel) | 18:26 |
lucasagomes | or we want to have something more complex like a different endpoint for credentials and so on | 18:26 |
lucasagomes | jroll, ^ do you have any opnion on it? | 18:27 |
lucasagomes | (for context, I'm just saying because in tripleo there are things like https://github.com/openstack/os-cloud-config/blob/master/os_cloud_config/nodes.py which is a PITA to maintain) | 18:27 |
NobodyCam | lucasagomes: how about for ssh driver would password contain a key? | 18:27 |
lucasagomes | NobodyCam, ssh accepts password too | 18:28 |
NobodyCam | ya | 18:28 |
*** Sukhdev has quit IRC | 18:28 | |
jroll | lucasagomes: I'm not opinionated, I don't think it's a very high priority | 18:28 |
jroll | but would be nice to have | 18:28 |
lucasagomes | if one wants to use a key or key content it can continue to use the ssh_* namespace because that's specific for that driver | 18:28 |
lucasagomes | jroll, it's not h prio at all | 18:28 |
jroll | we did talk about this before though | 18:28 |
lucasagomes | just asking so we can open an RFE about it | 18:28 |
*** krtaylor has joined #openstack-ironic | 18:28 | |
jroll | yeah, go for it | 18:29 |
NobodyCam | ++++ | 18:29 |
lucasagomes | and if we go for the just adding new properties, that should be a quick/easy patch | 18:29 |
lucasagomes | deprecate the namespaced ones in favor of the common ones | 18:29 |
lucasagomes | and that's all | 18:29 |
lucasagomes | remove it later... | 18:29 |
rloo | lucasagomes +. after reviewing patches that are high priority though :D | 18:29 |
lucasagomes | rloo, jroll NobodyCam cool, ok I will add an RFE for it | 18:30 |
lucasagomes | rloo, no rush reviewing it | 18:30 |
*** daemontool_ has quit IRC | 18:30 | |
lucasagomes | rush to review it* | 18:30 |
rloo | lucasagomes: no, i mean it'd be great if you had time to review the high priority patches ;) | 18:30 |
lucasagomes | ah, sure! | 18:31 |
rloo | lucasagomes: and we should try to get this spec approved: https://review.openstack.org/#/c/253605/ | 18:35 |
sambetts | night all o/ | 18:35 |
rloo | night sambetts | 18:35 |
lucasagomes | rloo, will review it | 18:35 |
lucasagomes | sambetts, see ya, g'night | 18:35 |
mjturek1 | hey all, I have a devstack patch that merged earlier than it should have (https://review.openstack.org/#/c/258079). I'm wondering might be related to the kilo tempest failures (https://bugs.launchpad.net/ironic/+bug/1531477). I have a revert in review for the original patch now (https://review.openstack.org/265969) | 18:35 |
openstack | Launchpad bug 1531477 in Ironic "Kilo dsvm gate is mostly broken (timeout when waiting for active state)" [High,Confirmed] | 18:35 |
*** ChubYann has joined #openstack-ironic | 18:36 | |
lucasagomes | rloo, ah that spec... yeah, I still gotta try the approach devananda commented there | 18:36 |
*** SpamapS has joined #openstack-ironic | 18:36 | |
lucasagomes | to use a prefix index | 18:36 |
*** electrofelix has quit IRC | 18:36 | |
rloo | lucasagomes: it'd be good to get that in mitaka but i'm worried that it might not happen. | 18:36 |
lucasagomes | yeah, same | 18:38 |
jroll | we still have a few months, we can totally get it in | 18:38 |
devananda | mjturek1: are you familiar with the depends-on flag in gerrit? | 18:38 |
mjturek1 | devananda: I was not :-\ | 18:39 |
rloo | jroll: i hope you're right. we don't seem to be moving faster than a snail's pace. (ok, i'll be more optimistic.) | 18:39 |
* mgould -> home; good night! | 18:39 | |
jroll | mjturek1: ouch | 18:39 |
mjturek1 | jroll: I know, very sorry :( | 18:39 |
*** mgould has quit IRC | 18:39 | |
jroll | rloo: I like to think we've been doing okay, except for during holidays | 18:39 |
jroll | but maybe I'm wrong | 18:40 |
jroll | mjturek1: it's fine | 18:40 |
rloo | jroll: i think holidays and gate/timeouts and a few other things. | 18:40 |
jroll | mjturek1: I don't know if I like the tempest patch either | 18:40 |
NobodyCam | night mgould | 18:40 |
rloo | jroll: mitaka-2 is coming up and we haven't released ironic since around mitaka-1. makes me sad that we changed to a new release thing but we so far seem to be following mitake schedule :-( | 18:41 |
jroll | rloo: mitaka schedule is one big release at end of cycle... | 18:43 |
jroll | that said | 18:43 |
jroll | m-1 through m-2 everyone was out on holiday | 18:43 |
jroll | we *have* gotten a lot of our priorities done / close to done | 18:43 |
mjturek1 | jroll: the timeout consolidation? Fair enough, it was requested by the tempest PTL so feel free to review | 18:43 |
*** praneshp has joined #openstack-ironic | 18:44 | |
jroll | mjturek1: heh, we're moving all of that code out anyway | 18:44 |
*** piet has quit IRC | 18:44 | |
jroll | mjturek1: https://review.openstack.org/#/c/265358/ | 18:45 |
*** piet has joined #openstack-ironic | 18:45 | |
*** sambetts has quit IRC | 18:45 | |
*** davideag_ has quit IRC | 18:46 | |
mjturek1 | jroll: ahh understood | 18:47 |
*** e0ne has joined #openstack-ironic | 18:49 | |
*** piet has quit IRC | 18:50 | |
*** davideagnello has joined #openstack-ironic | 18:50 | |
*** piet has joined #openstack-ironic | 18:50 | |
maurosr | jroll: but is there any other reason for you not liking mjturek1's patch? I mean is it just a matter of resubmit it on ironic's tree? | 18:51 |
jroll | maurosr: those are very different timeouts | 18:52 |
jroll | there's no reason power timeout should be 300s, for example | 18:52 |
mjturek1 | jroll: right makes sense | 18:52 |
maurosr | ok, specific timeout then | 18:53 |
jroll | I'm not saying it's wrong | 18:53 |
jroll | just something that should be discussed with the ironic team | 18:53 |
*** ijw has joined #openstack-ironic | 18:53 | |
*** Guest41804 is now known as mgagne | 18:54 | |
mjturek1 | jroll: okay, totally agree. I'll resubmit once the code moves to ironic | 18:55 |
*** mgagne is now known as Guest84419 | 18:55 | |
*** barra204 has joined #openstack-ironic | 18:55 | |
*** shakamunyi has quit IRC | 18:56 | |
jroll | mjturek1: cool, you can base it on top of 253982 | 18:56 |
mjturek1 | jroll: got it, will do | 18:56 |
jroll | thanks | 18:57 |
rloo | jroll: do you have next Monday (Jan 18, Martin Luther King Day) off? | 18:58 |
jroll | rloo: uhhhh | 18:59 |
jroll | not that I know of? | 18:59 |
rloo | jroll: ok, maybe just yahoo then. i was wondering if there would be a meeting, but i'll assume so. | 18:59 |
jroll | yeah, let's have one | 18:59 |
NobodyCam | rloo: we do have it listed as a holiday | 19:00 |
rloo | NobodyCam: lucky you :) I updated the meetings page with the next meeting being next week. There may be enough folks for a meeting. | 19:01 |
NobodyCam | :) | 19:02 |
*** trown|lunch is now known as trown | 19:02 | |
*** piet has quit IRC | 19:05 | |
*** piet has joined #openstack-ironic | 19:05 | |
sinval | hey folks, how a conductor is referenced by the nodes that it manages? | 19:11 |
*** shakamunyi has joined #openstack-ironic | 19:12 | |
*** barra204 has quit IRC | 19:12 | |
krotscheck | Looking through the ironic state machine (ironic.common.states), I see a few states that are declared, but not actually used. | 19:14 |
krotscheck | I'm guessing they're there for documentation purposes? | 19:14 |
krotscheck | Example: DEPLOYDONE | 19:14 |
devananda | krotscheck: such as? | 19:14 |
krotscheck | http://docs.openstack.org/developer/ironic/_modules/ironic/common/states.html | 19:14 |
devananda | ah | 19:14 |
devananda | there is an autogenerat[ed|able] graphic showing the paths in the state machine | 19:14 |
devananda | some of the states defined in states.py are internal and not exposed externally | 19:15 |
devananda | eg, DEPLOYDONE | 19:15 |
krotscheck | Ok, so not something I need to care about | 19:15 |
NobodyCam | sinval: you mean the hash ring? | 19:16 |
NobodyCam | https://github.com/openstack/ironic/blob/master/ironic/common/hash_ring.py | 19:16 |
krotscheck | Ditto on DELETED and REBUILD? | 19:16 |
devananda | krotscheck: http://docs.openstack.org/developer/ironic/_images/states.svg | 19:16 |
devananda | krotscheck: when you say "care about" do you mean "create input box for" for "might possibly be displayed" ? | 19:17 |
devananda | or somethng else? | 19:17 |
krotscheck | devananda: Might be displayed, and requires logic to determine what actions are allowed. | 19:17 |
krotscheck | (Even if said logic is "nothing's allowed") | 19:18 |
devananda | krotscheck: ok. as an aside, have you seen the spec that defines an API for ^ ? | 19:18 |
*** Guest84419 is now known as mgagne | 19:18 | |
*** mgagne has quit IRC | 19:18 | |
*** mgagne has joined #openstack-ironic | 19:18 | |
krotscheck | I heard about it and briefly read it, but from what I recall it wasn't what I needed. | 19:19 |
krotscheck | i.e. it didn't stick in my mind, ergo it's probably not what I'm looking for. | 19:19 |
sinval | NobodyCam: almost there, I mean how the node is marked, or receives the conductor id... something like that, because, the node itself has two attributes: reservation and conductor_affinity, and I thought that the reference was something like that... | 19:19 |
devananda | krotscheck: that's unfortunate. please review again -- it is expressly *for your use case* | 19:19 |
devananda | krotscheck: and please -1 if it in any way doesn't facilitate building a better UI | 19:19 |
krotscheck | This one? https://review.openstack.org/#/c/212132/ | 19:20 |
devananda | krotscheck: as far as reading the code to determine this, here is what you want https://github.com/openstack/ironic/blob/master/ironic/common/states.py#L215-L244 | 19:20 |
*** jdk has joined #openstack-ironic | 19:20 | |
krotscheck | Yeah, I'm already copying that. | 19:20 |
devananda | nope, not that spec | 19:20 |
*** garthb has joined #openstack-ironic | 19:20 | |
*** raddaoui has joined #openstack-ironic | 19:20 | |
sinval | NobodyCam: but I'll take a look at the hash ring code, I think that it will give me the answer :) thanks | 19:20 |
devananda | krotscheck: https://review.openstack.org/#/c/224022/ | 19:20 |
krotscheck | https://review.openstack.org/#/c/224022/ | 19:20 |
krotscheck | Ah | 19:20 |
krotscheck | Beat me to it | 19:20 |
devananda | sinval: reservation == "conductor places exclusive lock, does some work" | 19:22 |
*** marcusrafael has joined #openstack-ironic | 19:22 | |
*** raddaoui_ has joined #openstack-ironic | 19:22 | |
devananda | sinval: affinity == "last conductor that deployed node". if not equal to current mapping, then take_over() is required | 19:22 |
*** ipukha has quit IRC | 19:23 | |
sinval | devananda: oh, I see, so when both are null, any conductor can manage the node? | 19:23 |
*** [1]rpioso has joined #openstack-ironic | 19:24 | |
*** raddaoui has quit IRC | 19:25 | |
openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Add OEM sensors for Lenovo Nextscale https://review.openstack.org/265974 | 19:25 |
jdk | TheJulia: please take a look at http://paste.openstack.org/show/483470/ there is a question in there for you | 19:27 |
TheJulia | jdk: looking | 19:28 |
TheJulia | ahh | 19:29 |
*** rpioso has quit IRC | 19:30 | |
lucasagomes | sinval, there are different hash rings for different drivers | 19:32 |
lucasagomes | sinval, so only conductors loaded with the driver to manage that node would be hashed against that node | 19:33 |
TheJulia | jdk: is the instance info auto-populating? | 19:35 |
TheJulia | or are you populating it? | 19:35 |
jdk | TheJulia: in this case I populated it with the ironic node-update command | 19:36 |
TheJulia | jdk: so... the playbook is trying to make sure the default file name exists, it doesn't see that in this case, and the json does not come with pre-populated instance info which is why it checks tto see if the file name is there. Maybe try -e deploy_image_filename="edendisk-2016-01-11T17:34:26Z.qcow2" | 19:36 |
jdk | TheJulia: ok thanks, I'll try that | 19:37 |
TheJulia | on the ansible-playbook command line, that will reset the var and everything should find/automap accordingly. If that works, youshouldn't need need to populate instance_info by hand since it will do it for you, that is if the file can be found and checksum collected | 19:38 |
jdk | TheJulia: awesome, that worked. thanks a bunch | 19:41 |
sinval | lucasagomes, and, once that the conductor was selected how the node is referenced to it, my case is something like: "which nodes are managed by conductor X?" | 19:42 |
lucasagomes | sinval, we don't keep track of it, cause it's computed on-the-fly | 19:43 |
sinval | lucasagomes: cool, thanks | 19:43 |
lucasagomes | for every request, the api service will hash the node uuid and route it to the conductor that was returned from the hash ring module | 19:44 |
lucasagomes | sinval, not sure what changes would be needed in the hash ring to keep track of it tho, it should be possible do | 19:45 |
lucasagomes | it should be possible to do* | 19:45 |
sinval | lucasagomes: nice | 19:46 |
lucasagomes | sinval, yeah, it's called "consistent hash ring", there's many blog posts about it if you are interested | 19:47 |
krotscheck | devananda: Commented. Long story short: I need it to be able to provide me with the full state machine so that I can reconstruct it in browser memory. If I can't do that, then I have to send an API request for every row in a node list, and that's going to slow things way down. | 19:48 |
sinval | lucasagomes: I'll take a look | 19:48 |
lucasagomes | cool | 19:48 |
krotscheck | Whether that's done by describing the graph edges or nodes is fairly irellevant, though I do want some kind of indication if a state is volatile. | 19:48 |
krotscheck | devananda: With that in mind, I'm goign to assume that I can build a state machine from a JSON file that looks like the one provided. | 19:50 |
lucasagomes | folks I'm going to call it a day | 19:53 |
lucasagomes | have a great night! | 19:53 |
sinval | good night lucasagomes | 19:53 |
*** lucasagomes is now known as lucas-dinner | 19:53 | |
lucas-dinner | sinval, g'night! see ya | 19:53 |
*** pas-ha has quit IRC | 19:53 | |
*** sulgrove has joined #openstack-ironic | 19:59 | |
NobodyCam | night lucas-dinner | 20:02 |
*** jaybeale has quit IRC | 20:02 | |
*** raddaoui_ has quit IRC | 20:04 | |
openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Add OEM sensors for Lenovo Nextscale https://review.openstack.org/265974 | 20:10 |
devananda | krotscheck: sounds good | 20:11 |
*** raddaoui_ has joined #openstack-ironic | 20:15 | |
*** aginwala has joined #openstack-ironic | 20:23 | |
*** MattMan has quit IRC | 20:25 | |
*** MattMan has joined #openstack-ironic | 20:26 | |
*** raddaoui_ has quit IRC | 20:29 | |
*** Sukhdev has joined #openstack-ironic | 20:29 | |
krotscheck | devananda: So, that spec outlines states named things like "deployfail", while states.py names them "deploy failed". Which of the two is the one that'll come back from an API call? I'm assuming the latter. | 20:37 |
*** jaybeale has joined #openstack-ironic | 20:40 | |
adam_g | hi ironic people--maybe you have some insight here.. since the ironic api is an admin only API, is there a way to publish its endpoint into keystone in such a way that its only visible in the catalog to admin users? | 20:40 |
*** yolanda has quit IRC | 20:40 | |
*** aginwala has quit IRC | 20:41 | |
*** yolanda has joined #openstack-ironic | 20:41 | |
NobodyCam | adam_g: dont set a public end point? | 20:46 |
jlvillal | adam_g: Would that be a keystone thing? | 20:47 |
adam_g | NobodyCam, i wish, publicurl is required (at least to keystoneclient) | 20:48 |
TheJulia | That... seems like a bug | 20:48 |
adam_g | jlvillal, sure. figured i'd bother you all first, since ironics the first thing that popped into my head when i thought about admin-only APIs | 20:48 |
jlvillal | adam_g: Publish a fake keystone/internal? | 20:48 |
NobodyCam | hum.. | 20:49 |
jlvillal | adam_g: Publish a fake keystone public/internal? | 20:49 |
jlvillal | http://invalid.example.com/ :) | 20:49 |
adam_g | jlvillal, i was actually hoping to hide the endpoint entirely from non-admin tenants. there's no reason for them to know what admin-level stuff might also be deployed | 20:49 |
TheJulia | adam_g: so... ksc wants the public url by defualt why? | 20:49 |
* jlvillal has no idea if that would screw up keystone. | 20:49 | |
adam_g | but yea, ill go bug #openstack-keystone:) | 20:49 |
jlvillal | adam_g: Good luck! | 20:50 |
TheJulia | adam_g: I | 20:50 |
TheJulia | doh | 20:50 |
TheJulia | adam_g: I'm fairly sure we've not set a public endpoint places, but ksc is fairly new | 20:50 |
TheJulia | well, in terms of new code changes | 20:51 |
adam_g | keystone endpoint-create: error: argument --publicurl is required | 20:51 |
adam_g | keystoneclient is old bones | 20:51 |
TheJulia | I must be thinking of the library or middlewear that I dug through a few months back | 20:51 |
*** amotoki has joined #openstack-ironic | 20:52 | |
adam_g | yup, keystonemiddleware is newerish | 20:52 |
TheJulia | adam_g: have you thought about supplying an empty string? | 20:55 |
adam_g | TheJulia, i have but seems hacky. actually looks like ks v3 supports per-endpoint policies, i think thats what i need | 20:56 |
TheJulia | adam_g: interesting, didn't know that | 20:56 |
*** aginwala has joined #openstack-ironic | 20:57 | |
*** spandhe has joined #openstack-ironic | 21:03 | |
jroll | adam_g: our deployment is run by bad people and as such we just didn't put it in the catalog >.> | 21:04 |
*** ionutbalutoiu has joined #openstack-ironic | 21:06 | |
*** maratoid has quit IRC | 21:07 | |
*** aginwala has quit IRC | 21:17 | |
*** e0ne has quit IRC | 21:25 | |
*** garthb_ has joined #openstack-ironic | 21:27 | |
*** garthb has quit IRC | 21:28 | |
mrda | Morning Ironic | 21:34 |
TheJulia | Good morning mrda | 21:35 |
NobodyCam | morning mrda | 21:35 |
mrda | o/ | 21:35 |
*** ionutbalutoiu has quit IRC | 21:36 | |
*** dprince has quit IRC | 21:38 | |
*** aginwala has joined #openstack-ironic | 21:42 | |
*** Sukhdev has quit IRC | 21:48 | |
openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Added Node Transition mock service https://review.openstack.org/266050 | 21:48 |
*** marcusrafael has quit IRC | 21:51 | |
krotscheck | I _still_ can't get over the MRSA reference. | 21:51 |
krotscheck | Yeesh | 21:51 |
*** ndipanov has quit IRC | 21:51 | |
*** persia has quit IRC | 21:53 | |
*** persia has joined #openstack-ironic | 21:53 | |
jlvillal | mrda: Good morning | 21:54 |
*** Sukhdev has joined #openstack-ironic | 21:58 | |
mrda | o/ | 22:00 |
*** garthb__ has joined #openstack-ironic | 22:01 | |
jlvillal | Is our gate still broken? My last recheck failed :( | 22:03 |
*** garthb_ has quit IRC | 22:03 | |
* krotscheck wouldn't know, gerrit is throwing over-capacity errors for him. | 22:05 | |
-openstackstatus- NOTICE: Gerrit is restarting to resolve java memory issues | 22:05 | |
*** davidlenwell has quit IRC | 22:05 | |
NobodyCam | krotscheck: oh that ^^^^^ :) | 22:06 |
* krotscheck loves java | 22:06 | |
* krotscheck really loves java. | 22:06 | |
* krotscheck does NOT love java's memory footprint | 22:06 | |
NobodyCam | lol | 22:06 |
*** trown is now known as trown|outttypeww | 22:09 | |
*** izaakk has quit IRC | 22:13 | |
*** jdk has quit IRC | 22:17 | |
openstackgerrit | Merged openstack/pyghmi: Add OEM sensors for Lenovo Nextscale https://review.openstack.org/265974 | 22:19 |
openstackgerrit | John L. Villalovos proposed openstack/ironic-lib: WIP: Testing https://review.openstack.org/264577 | 22:22 |
*** aginwala has quit IRC | 22:25 | |
*** aginwala has joined #openstack-ironic | 22:28 | |
*** davidlenwell has joined #openstack-ironic | 22:29 | |
*** piet has quit IRC | 22:37 | |
*** praneshp has quit IRC | 22:45 | |
*** praneshp has joined #openstack-ironic | 22:46 | |
*** praneshp has quit IRC | 22:46 | |
Sukhdev | jroll : Hey Jim --ironic-api-version 1.14 (or latest) does not seem to work - do you know what is the exact syntax? | 22:48 |
*** athomas has quit IRC | 22:49 | |
Sukhdev | jroll : this is the error I get - ironic: error: unrecognized arguments: --ironic-api-version 1.14 | 22:49 |
jroll | Sukhdev: what version is your client? | 22:49 |
jroll | maybe it's just --api-version | 22:49 |
jroll | or use ironic --help | 22:50 |
*** harshs has joined #openstack-ironic | 22:50 | |
openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Added Node Transition mock service https://review.openstack.org/266050 | 22:52 |
Sukhdev | jroll : python-ironicclient==1.0.1.dev70 | 22:52 |
jroll | Sukhdev: yeah, use ironic --help | 22:53 |
Sukhdev | ironic version show 1.0.1 | 22:53 |
jroll | I mean | 22:54 |
Sukhdev | jroll : your original syntax was correct - that is that ironic --help shows | 22:54 |
jroll | ironic --help will show you the arguments | 22:54 |
jroll | ok | 22:54 |
jroll | can you paste your full command? | 22:54 |
Sukhdev | jroll : ironic port-update $port_id add local_link_connection/switch_info='Arista DCS-7050S-64-F' local_link_connection/switch_id='00:1c:73:10:3f:b0' local_link_connection/port_id='Ethernet2' --ironic-api-version 1.14 | 22:55 |
jroll | Sukhdev: move it to the beginning, ironic --ironic-api-version 1.14 port-update ... | 22:55 |
Sukhdev | jroll: oh I see - even before port-update keyword? | 22:56 |
jroll | yes | 22:56 |
jroll | syntax is: ironic [options] <command> [command options] | 22:56 |
Sukhdev | got it - thanks - let me give it a shot | 22:56 |
Sukhdev | jroll : I got past the syntax issue - but, still get version issue see the paste here - http://paste.openstack.org/show/483499/ | 23:00 |
jroll | O_o | 23:02 |
jroll | I'm not sure | 23:02 |
jroll | and I'm in the middle of three other things right now, sorry | 23:02 |
jroll | sounds like a busted patch :) | 23:02 |
Sukhdev | I am pulling the python-ironicclient repo and applying this patch on top of it - https://review.openstack.org/#/c/206144 | 23:04 |
jroll | I mean it sounds like a bad server patch | 23:04 |
jroll | but I'm sorry, I don't have time to debug this right now | 23:04 |
Sukhdev | any pointers that would help nail this down? | 23:05 |
jroll | not really, other than reading the api version code | 23:06 |
*** [1]cdearborn has quit IRC | 23:12 | |
*** [1]rpioso has quit IRC | 23:16 | |
*** sulgrove has quit IRC | 23:16 | |
*** amotoki has quit IRC | 23:25 | |
*** aginwala has quit IRC | 23:31 | |
*** aginwala has joined #openstack-ironic | 23:35 | |
devananda | Sukhdev: curl http://<URL>:6385/ | json_pp | 23:37 |
*** Sukhdev has quit IRC | 23:37 | |
devananda | Sukhdev: the root URL of Ironic will return the supported versions, even without auth. does that say 1.14 or something else? | 23:38 |
devananda | d'oh | 23:38 |
*** Sukhdev has joined #openstack-ironic | 23:40 | |
devananda | Sukhdev: curl http://<URL>:6385/ | json_pp | 23:43 |
devananda | Sukhdev: the root URL of Ironic will return the supported versions, even without auth. does that say 1.14 or something else? | 23:43 |
*** praneshp has joined #openstack-ironic | 23:48 | |
*** smoriya_ has joined #openstack-ironic | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!