| jroll | Shrews: rebased ^ | 00:01 |
|---|---|---|
| *** blamar has quit IRC | 00:08 | |
| *** ellenh has quit IRC | 00:15 | |
| *** blamar has joined #openstack-ironic | 00:18 | |
| *** ellenh has joined #openstack-ironic | 00:19 | |
| *** matsuhashi has joined #openstack-ironic | 00:25 | |
| *** yfujioka has joined #openstack-ironic | 00:30 | |
| *** ellenh has quit IRC | 00:38 | |
| *** ellenh has joined #openstack-ironic | 00:38 | |
| *** matsuhashi has quit IRC | 00:40 | |
| *** matsuhashi has joined #openstack-ironic | 00:40 | |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Add Node decommission in conductor https://review.openstack.org/102984 | 00:43 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Add Node decommission in conductor https://review.openstack.org/102984 | 00:43 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Add Node decommission in conductor https://review.openstack.org/102984 | 00:50 |
| *** matsuhashi has quit IRC | 00:51 | |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Add node decomission interfaces https://review.openstack.org/102953 | 00:51 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Add Node decommission interfaces https://review.openstack.org/102953 | 00:51 |
| *** matsuhashi has joined #openstack-ironic | 00:51 | |
| *** matsuhashi has quit IRC | 00:55 | |
| *** achanda has quit IRC | 01:05 | |
| *** openstackgerrit has quit IRC | 01:06 | |
| *** openstackgerrit has joined #openstack-ironic | 01:07 | |
| *** achanda has joined #openstack-ironic | 01:13 | |
| *** achanda has quit IRC | 01:13 | |
| *** matsuhashi has joined #openstack-ironic | 01:21 | |
| *** matsuhashi has quit IRC | 01:27 | |
| *** hemna has quit IRC | 01:31 | |
| openstackgerrit | Ghe Rivero proposed a change to openstack/ironic: Update image_source when available https://review.openstack.org/102988 | 01:34 |
| *** lazy_prince has quit IRC | 01:52 | |
| *** nosnos has joined #openstack-ironic | 01:52 | |
| *** GheRivero is now known as GheRivero_afk | 01:56 | |
| openstackgerrit | A change was merged to openstack/ironic: Fix 'fake' driver unable to finish a deploy https://review.openstack.org/102962 | 02:29 |
| openstackgerrit | Adam Gandelman proposed a change to openstack/ironic: pep8 cleanup of Nova code https://review.openstack.org/103001 | 02:34 |
| *** eghobo has quit IRC | 02:38 | |
| *** harlowja is now known as harlowja_away | 02:53 | |
| *** rloo has quit IRC | 03:01 | |
| *** geekyogi has joined #openstack-ironic | 03:31 | |
| *** achanda has joined #openstack-ironic | 03:39 | |
| *** killer_prince has joined #openstack-ironic | 03:43 | |
| *** killer_prince is now known as lazy_prince | 03:43 | |
| *** nosnos has quit IRC | 03:49 | |
| *** nosnos has joined #openstack-ironic | 03:49 | |
| openstackgerrit | Nisha Agarwal proposed a change to openstack/ironic-specs: Discover node properties at node-create/node-update https://review.openstack.org/100951 | 03:49 |
| *** nosnos has quit IRC | 03:50 | |
| *** nosnos has joined #openstack-ironic | 03:50 | |
| *** nosnos has quit IRC | 03:55 | |
| *** eghobo has joined #openstack-ironic | 03:55 | |
| *** amitpp has joined #openstack-ironic | 03:59 | |
| openstackgerrit | Nisha Agarwal proposed a change to openstack/ironic-specs: Discover node properties for iLO driver https://review.openstack.org/103007 | 04:09 |
| *** nosnos has joined #openstack-ironic | 04:13 | |
| *** ellenh has quit IRC | 04:13 | |
| openstackgerrit | Nisha Agarwal proposed a change to openstack/ironic-specs: Discover node properties for iLO driver https://review.openstack.org/103007 | 04:14 |
| *** rameshg87 has joined #openstack-ironic | 04:14 | |
| *** ellenh has joined #openstack-ironic | 04:14 | |
| *** eghobo has quit IRC | 04:16 | |
| *** lazy_prince is now known as killer_prince | 04:20 | |
| *** ccrouch has joined #openstack-ironic | 04:41 | |
| *** ccrouch has quit IRC | 04:46 | |
| *** eghobo has joined #openstack-ironic | 04:46 | |
| *** achanda has quit IRC | 04:52 | |
| openstackgerrit | Nisha Agarwal proposed a change to openstack/ironic-specs: Discover node properties at node-create/node-update https://review.openstack.org/100951 | 04:52 |
| *** achanda has joined #openstack-ironic | 04:53 | |
| *** achanda has quit IRC | 04:54 | |
| *** Shiv has joined #openstack-ironic | 04:55 | |
| openstackgerrit | Nisha Agarwal proposed a change to openstack/ironic-specs: Discover node properties for iLO driver https://review.openstack.org/103007 | 04:56 |
| *** Shiv has quit IRC | 04:59 | |
| *** rakesh_hs has joined #openstack-ironic | 05:04 | |
| *** yfujioka has left #openstack-ironic | 05:04 | |
| *** pradipta` is now known as pradipta | 05:13 | |
| *** ajc_ has joined #openstack-ironic | 05:15 | |
| *** killer_prince is now known as lazy_prince | 05:16 | |
| *** lazy_prince is now known as killer_prince | 05:21 | |
| *** coolsvap|afk is now known as coolsvap | 05:22 | |
| *** bmahalakshmi has joined #openstack-ironic | 05:23 | |
| openstackgerrit | Nisha Agarwal proposed a change to openstack/ironic-specs: Discover node properties at node-create/node-update https://review.openstack.org/100951 | 05:28 |
| openstackgerrit | Nisha Agarwal proposed a change to openstack/ironic-specs: Discover node properties for iLO driver https://review.openstack.org/103007 | 05:34 |
| *** eghobo has quit IRC | 05:38 | |
| *** eghobo has joined #openstack-ironic | 05:38 | |
| *** Nisha has joined #openstack-ironic | 05:39 | |
| *** bvivek has joined #openstack-ironic | 05:41 | |
| *** killer_prince is now known as lazy_prince | 05:42 | |
| *** geekyogi has quit IRC | 05:57 | |
| *** sabah has joined #openstack-ironic | 06:05 | |
| *** chen12 has joined #openstack-ironic | 06:06 | |
| *** eguz has joined #openstack-ironic | 06:06 | |
| *** eghobo has quit IRC | 06:10 | |
| *** radsy has quit IRC | 06:11 | |
| chen12 | hello, anyone here ??? | 06:12 |
| chen12 | I'm trying to install/using ironic. | 06:12 |
| chen12 | I have followed steps here :http://docs.openstack.org/developer/ironic/dev/dev-quickstart.html#deploying-ironic-with-devstack | 06:12 |
| chen12 | While, after I run command "nova boot --flavor baremetal --image $image test-instance" | 06:12 |
| chen12 | My instance stays in BUILD and spawning for a really long time, and become ERROR at end. | 06:12 |
| chen12 | This is my log for compute.log: | 06:12 |
| chen12 | http://paste.openstack.org/ | 06:12 |
| chen12 | Looks like it is waiting ironic node to become ACTIVE. | 06:12 |
| chen12 | This is my ironic-conductor service log: | 06:12 |
| chen12 | http://paste.openstack.org/show/85043/ | 06:12 |
| chen12 | No obvious error message can be find, but there is a warning: | 06:12 |
| chen12 | WARNING ironic.common.neutron [-] No VIFs found for node c9caca46-64a7-44d5-b6db-554077b2473a when attempting to update Neutron DHCP BOOT options. | 06:12 |
| chen12 | Is this warning caused instance can't start ?? | 06:12 |
| chen12 | I also check neutron-server, no IP has been allocated at all. | 06:12 |
| chen12 | Anyone know why this is happened ??? | 06:13 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Updated from global requirements https://review.openstack.org/102126 | 06:32 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/88722 | 06:32 |
| *** geekyogi has joined #openstack-ironic | 06:34 | |
| *** bvivek has quit IRC | 06:35 | |
| *** xuhaiwei_ has joined #openstack-ironic | 06:35 | |
| xuhaiwei_ | when I tried to provision a baremetal machine, I got this error "Node "overcloud-compute-node1" with MAC address [u'6c626dcbf5ed'] not found." | 06:37 |
| xuhaiwei_ | it seems this error happens when trying set power on, but as far as I know the IPMI use ip to restart os, but why MAC is used here? | 06:38 |
| *** foexle_ has joined #openstack-ironic | 06:43 | |
| *** nosnos has quit IRC | 06:44 | |
| *** nosnos has joined #openstack-ironic | 06:44 | |
| *** rwsu has quit IRC | 06:53 | |
| *** pradipta is now known as pradipta_away | 06:54 | |
| *** jcoufal has joined #openstack-ironic | 06:54 | |
| *** bvivek has joined #openstack-ironic | 07:05 | |
| *** mrda is now known as mrda-weekend | 07:06 | |
| *** sabah has quit IRC | 07:06 | |
| *** viktors|afk is now known as viktors | 07:13 | |
| *** eguz has quit IRC | 07:15 | |
| *** lynxman has quit IRC | 07:17 | |
| openstackgerrit | Adam Gandelman proposed a change to openstack/ironic: Wait for Neutron port updates when using SSHPower https://review.openstack.org/91719 | 07:17 |
| openstackgerrit | Sirushti Murugesan proposed a change to openstack/ironic-specs: Whole Disk Image Support for PXE Deploy Driver https://review.openstack.org/97150 | 07:24 |
| *** lynxman has joined #openstack-ironic | 07:31 | |
| *** bvivek has quit IRC | 07:36 | |
| *** GheRivero_afk is now known as GheRivero | 07:38 | |
| *** takadayuiko has joined #openstack-ironic | 07:43 | |
| *** pcrews has quit IRC | 07:49 | |
| *** romcheg has joined #openstack-ironic | 07:49 | |
| chen12 | hello, anyone here ??? | 07:53 |
| chen12 | I'm trying to install/using ironic. | 07:53 |
| chen12 | I have followed steps here :http://docs.openstack.org/developer/ironic/dev/dev-quickstart.html#deploying-ironic-with-devstack | 07:53 |
| chen12 | While, after I run command "nova boot --flavor baremetal --image $image test-instance" | 07:53 |
| chen12 | My instance stays in BUILD and spawning for a really long time, and become ERROR at end. | 07:53 |
| chen12 | This is my log for compute.log: | 07:53 |
| chen12 | http://paste.openstack.org/ | 07:53 |
| chen12 | Looks like it is waiting ironic node to become ACTIVE. | 07:53 |
| chen12 | This is my ironic-conductor service log: | 07:53 |
| chen12 | http://paste.openstack.org/show/85043/ | 07:53 |
| chen12 | No obvious error message can be find, but there is a warning: | 07:53 |
| chen12 | WARNING ironic.common.neutron [-] No VIFs found for node c9caca46-64a7-44d5-b6db-554077b2473a when attempting to update Neutron DHCP BOOT options. | 07:53 |
| chen12 | Is this warning caused instance can't start ?? | 07:54 |
| chen12 | I also check neutron-server, no IP has been allocated at all. | 07:54 |
| chen12 | Anyone know why this is happened ??? | 07:54 |
| *** athomas has joined #openstack-ironic | 08:00 | |
| *** nosnos has quit IRC | 08:07 | |
| *** bvivek has joined #openstack-ironic | 08:08 | |
| *** rwsu has joined #openstack-ironic | 08:08 | |
| *** rwsu has quit IRC | 08:08 | |
| *** ifarkas has joined #openstack-ironic | 08:08 | |
| *** martyntaylor has joined #openstack-ironic | 08:12 | |
| *** martyntaylor has left #openstack-ironic | 08:12 | |
| openstackgerrit | Sirushti Murugesan proposed a change to openstack/ironic-specs: Whole Disk Image Support for PXE Deploy Driver https://review.openstack.org/97150 | 08:39 |
| *** ifarkas has quit IRC | 08:47 | |
| *** lucasagomes has joined #openstack-ironic | 08:47 | |
| *** russell_h has quit IRC | 08:53 | |
| *** lazy_prince has quit IRC | 08:58 | |
| *** igordcard has joined #openstack-ironic | 08:58 | |
| *** martyntaylor has joined #openstack-ironic | 09:05 | |
| *** martyntaylor has left #openstack-ironic | 09:09 | |
| *** martyntaylor has joined #openstack-ironic | 09:14 | |
| openstackgerrit | Shivanand Tendulker proposed a change to openstack/ironic-specs: Firmware setting design spec. https://review.openstack.org/101122 | 09:14 |
| *** pelix has joined #openstack-ironic | 09:15 | |
| *** rakesh_hs has quit IRC | 09:16 | |
| openstackgerrit | Shivanand Tendulker proposed a change to openstack/ironic-specs: Firmware setting design spec. https://review.openstack.org/101122 | 09:21 |
| *** sabah has joined #openstack-ironic | 09:22 | |
| openstackgerrit | Shivanand Tendulker proposed a change to openstack/ironic-specs: Firmware setting design spec. https://review.openstack.org/101122 | 09:29 |
| openstackgerrit | Shivanand Tendulker proposed a change to openstack/ironic-specs: Design spec for iLO driver for firmware settings https://review.openstack.org/103065 | 09:29 |
| rameshg87 | chen12: have you enrolled the ports for the particular node ? | 09:30 |
| *** takadayuiko has quit IRC | 09:36 | |
| *** nosnos has joined #openstack-ironic | 09:42 | |
| *** nosnos has quit IRC | 09:44 | |
| *** nosnos has joined #openstack-ironic | 09:45 | |
| *** athomas has quit IRC | 09:45 | |
| *** nosnos has quit IRC | 09:49 | |
| *** athomas has joined #openstack-ironic | 09:52 | |
| foexle_ | hi guys, is there any way to resolve this issue? => NodeAssociated: Node 4b9ae9b7-233e-4b98-81e7-510111fe24f7 is associated with instance 76080f5e-0897-4bcb-b7d6-e36dc279e541. | 09:56 |
| foexle_ | this instance is not longer available | 09:56 |
| foexle_ | if i'm deleting the instance_uuid in the db | 09:56 |
| foexle_ | i'll get 400 Expected a uuid but received asdasd. (HTTP 400) | 09:57 |
| *** romcheg1 has joined #openstack-ironic | 09:58 | |
| *** romcheg has quit IRC | 09:59 | |
| *** killer_prince has joined #openstack-ironic | 10:02 | |
| *** killer_prince is now known as lazy_prince | 10:02 | |
| *** ifarkas has joined #openstack-ironic | 10:02 | |
| *** xuhaiwei_ has quit IRC | 10:05 | |
| *** Nisha has quit IRC | 10:06 | |
| *** romcheg has joined #openstack-ironic | 10:08 | |
| *** rakesh_hs has joined #openstack-ironic | 10:08 | |
| *** romcheg1 has quit IRC | 10:09 | |
| openstackgerrit | Haomeng,Wang proposed a change to openstack/ironic-specs: Send sensor data to Ceilometer https://review.openstack.org/102435 | 10:16 |
| lucasagomes | foexle_, did you tear down the node? | 10:24 |
| foexle_ | lucasagomes: yep but same error i've solved this instance_uuid=null => db row update | 10:27 |
| lucasagomes | foexle_, hmm right, that's bad we should be able to recover from that... you have any idea of how to reproduce that? | 10:28 |
| romcheg | Morning all! | 10:28 |
| lucasagomes | foexle_, you had this node deployed and then you tried to tear it down and got that error? | 10:28 |
| lucasagomes | romcheg, morning | 10:28 |
| *** coolsvap is now known as coolsvap|afk | 10:28 | |
| foexle_ | lucasagomes: no :) the whole system is not runnning. I'm trying to install and configure by hand ironic. atm i'm trying to boot an instance so a node but without success. I've deleted the instance by nova client it was delete then but ironic have the assiciation to the delted instance | 10:31 |
| foexle_ | lucasagomes: so it was a deadlock | 10:31 |
| foexle_ | romcheg: morning | 10:34 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add the remaining unittests to the ClientWrapper class https://review.openstack.org/92416 | 10:34 |
| openstackgerrit | Ramakrishnan G proposed a change to openstack/ironic-specs: iLO Power Driver for Ironic https://review.openstack.org/97455 | 10:41 |
| *** dtantsur|afk is now known as dtantsur | 10:42 | |
| dtantsur | g'afternoon Ironic! | 10:44 |
| *** faizan has joined #openstack-ironic | 10:56 | |
| *** rakesh_hs has quit IRC | 10:58 | |
| *** Mikhail_D_ltp has joined #openstack-ironic | 10:59 | |
| *** rakesh_hs has joined #openstack-ironic | 10:59 | |
| dtantsur | Need more folks (especially specs-core) to review Ceilometer spec https://review.openstack.org/#/c/102435/ ! | 11:00 |
| dtantsur | ifarkas, lucasagomes ^^^ | 11:00 |
| *** Mikhail_D_ltp has quit IRC | 11:05 | |
| *** rameshg87 has left #openstack-ironic | 11:11 | |
| *** romcheg has quit IRC | 11:14 | |
| openstackgerrit | Dmitry Tantsur proposed a change to openstack/ironic-specs: Generic Hardware Discovery Bits https://review.openstack.org/102565 | 11:17 |
| *** bvivek has quit IRC | 11:25 | |
| foexle_ | is there maybe any with a hint ? ... I'm despair of that error -.- .... it occurs if i'm booting an insance | 11:25 |
| foexle_ | sry for all questions .... but installing without devstack ... is really crazy :) | 11:26 |
| *** martyntaylor has quit IRC | 11:27 | |
| *** romcheg has joined #openstack-ironic | 11:27 | |
| foexle_ | ups .... http://paste.openstack.org/show/85050/ => log -.- forgot | 11:27 |
| dtantsur | foexle_, do you have an up-to-date python-ironicclient? | 11:29 |
| dtantsur | I mean, both should be relatively new, log seems to be related to one recent change | 11:30 |
| *** lazy_prince has quit IRC | 11:30 | |
| *** martyntaylor has joined #openstack-ironic | 11:31 | |
| foexle_ | dtantsur: ahhh good hint .... could be the package of ubuntu repo on the comp node, i'll check and update | 11:37 |
| foexle_ | dtantsur: many thanks | 11:37 |
| foexle_ | dtantsur: was the newest :( => master branch | 11:38 |
| foexle_ | and ironic python package too | 11:38 |
| dtantsur | foexle, your Ubuntu packages are from master? really? | 11:40 |
| *** bmahalakshmi has quit IRC | 11:40 | |
| dtantsur | could you double-check? | 11:40 |
| foexle_ | dtantsur: no i've checked out the git repo and installing manually | 11:41 |
| foexle_ | with setup.py | 11:41 |
| dtantsur | foexle_, /usr/lib/python2.7/dist-packages/ironic/ <-- this is for downstream packages, it doesn't look like something you installed via setup.py | 11:42 |
| dtantsur | I mean, "/dist-packages/" | 11:42 |
| foexle_ | mean ironicclient i'll take a look on the dist package | 11:42 |
| *** martyntaylor has quit IRC | 11:43 | |
| *** martyntaylor has joined #openstack-ironic | 11:45 | |
| chen12 | rameshg87 hi, are you still there ?? | 11:48 |
| *** faizan has quit IRC | 11:48 | |
| chen12 | rameshg87 sorry,I'm really new to ironic.... | 11:48 |
| chen12 | rameshg87 just follwoed the guide.. | 11:48 |
| foexle_ | dtantsur: the setup.py installs to => creating /usr/local/lib/python2.7/dist-packages/ironic/api | 11:49 |
| *** shardy has joined #openstack-ironic | 11:49 | |
| foexle_ | python setup.py build && python setup.py install | 11:49 |
| dtantsur | foexle, note the /local/ | 11:49 |
| dtantsur | foexle, afaik /usr/local/lib/python2.7/dist-packages/ is not expected by python | 11:50 |
| dtantsur | it should be /usr/local/lib/python2.7/site-packages/ | 11:50 |
| *** lucasagomes is now known as lucas-lunch | 11:50 | |
| dtantsur | or even /usr/lib/python2.7/site-packages/ | 11:50 |
| *** bvivek has joined #openstack-ironic | 11:51 | |
| *** lazy_prince has joined #openstack-ironic | 11:54 | |
| dtantsur | Any IPA folks around now? | 11:55 |
| dtantsur | Or anyone can answer: is it reasonable to store nested structures in Node.extra? Or is it better to flatten? | 11:55 |
| dtantsur | I remember IPA has some considerations on it... | 11:55 |
| dtantsur | jroll, JayF, when you're around, could you please answer ^^^ | 11:56 |
| *** ccrouch has joined #openstack-ironic | 12:01 | |
| *** ajc_ has quit IRC | 12:03 | |
| foexle_ | dtantsur: all right now checked all ironic libs and requirements are compiled .... but the error still exists. Is there maybe the nova or oslo libs too old ? | 12:08 |
| *** jdob has joined #openstack-ironic | 12:08 | |
| dtantsur | foexle, can you show me a new traceback, where we can see using of site-packages? | 12:08 |
| dtantsur | foexle_, ^^^ | 12:08 |
| foexle_ | they are not installing to site-packages. they install and load the libs from local/lib/python2.7/dist-packages/ | 12:09 |
| Shrews | morning all | 12:10 |
| foexle_ | hi Shrews | 12:10 |
| openstackgerrit | Dmitry Tantsur proposed a change to openstack/ironic-specs: Generic Hardware Discovery Bits https://review.openstack.org/102565 | 12:10 |
| foexle_ | dtantsur: i mean i'm installing directly with setup.py not from the ubuntu repos | 12:12 |
| foexle_ | dtantsur: all ubuntu ironic packages are deinstalled | 12:13 |
| dtantsur | foexle_, can I see the new traceback just to be sure? | 12:14 |
| foexle_ | maybe a version conflict with nova ? because i'm using the ubuntu icehouse packages | 12:14 |
| foexle_ | of course | 12:14 |
| dtantsur | Shrews, morning! | 12:14 |
| foexle_ | dtantsur: http://paste.openstack.org/show/85060/ | 12:15 |
| dtantsur | lemme see... | 12:15 |
| foexle_ | => file "/usr/local/lib/python2.7/dist-packages/ironic/nova/virt/ironic/driver.py" | 12:15 |
| *** amitpp has quit IRC | 12:16 | |
| foexle_ | i've made a diff from git repo and this file and it's the same | 12:16 |
| *** martyntaylor1 has joined #openstack-ironic | 12:16 | |
| foexle_ | md5 is equal too | 12:17 |
| dtantsur | foexle_, ok, I suspect incompatibility between Icehouse Nova and Juno Ironic | 12:18 |
| foexle_ | cry | 12:18 |
| foexle_ | .D | 12:18 |
| dtantsur | foexle_, you see, Nova passes string to instance_exists(), but Ironic expects a dict/structure. In master Nova also passes dict/structure | 12:18 |
| dtantsur | foexle_, well, it will be hard, until we merge our driver into nova, as we use non-public API, which do not have back-comp guarantees. | 12:19 |
| foexle_ | dtantsur: i see ... hmmm bad problem :) | 12:19 |
| foexle_ | dtantsur: dont worry | 12:19 |
| foexle_ | dtantsur: incubation project and so on :) | 12:19 |
| dtantsur | yeah :) | 12:20 |
| *** martyntaylor has quit IRC | 12:20 | |
| *** ifarkas has quit IRC | 12:21 | |
| foexle_ | dtantsur: i ran on icehouse branch of ironic in some bugs ... and i thought ok lets try master branch :) | 12:21 |
| foexle_ | dtantsur: all right many thanks for your time! I'll try next week devstack :). The goal is to have a undercloud for tribleO .... i hope I've more luck with devstack | 12:24 |
| *** lazy_prince has quit IRC | 12:25 | |
| dtantsur | foexle_, did you try devtest? | 12:26 |
| dtantsur | foexle_, it's the instrument of creating underclouds | 12:27 |
| chen12 | hello, anyone here ??? | 12:28 |
| chen12 | I'm trying to install/using ironic. | 12:28 |
| chen12 | I have followed steps here :http://docs.openstack.org/developer/ironic/dev/dev-quickstart.html#deploying-ironic-with-devstack | 12:28 |
| chen12 | While, after I run command "nova boot --flavor baremetal --image $image test-instance" | 12:28 |
| chen12 | My instance stays in BUILD and spawning for a really long time, and become ERROR at end. | 12:28 |
| chen12 | This is my ironic-conductor service log: | 12:28 |
| chen12 | http://paste.openstack.org/show/85043/ | 12:28 |
| chen12 | No obvious error message can be find, but there is a warning: | 12:28 |
| chen12 | WARNING ironic.common.neutron [-] No VIFs found for node c9caca46-64a7-44d5-b6db-554077b2473a when attempting to update Neutron DHCP BOOT options. | 12:28 |
| chen12 | Based on the warning information above, | 12:28 |
| chen12 | I changed to run command with nic paramter: | 12:28 |
| chen12 | nova boot --flavor baremetal --image cirros-0.3.2-x86_64-uec --nic net-id=b957c908-39bc-4de5-b620-addfd5618680 test | 12:28 |
| chen12 | Now I can get IP address by command “nova list”: | 12:28 |
| chen12 | And the warning in ironic-conductor seems gone. | 12:28 |
| chen12 | But, the instance still stays under BUILD status. | 12:28 |
| chen12 | Then, I use VNCviewer directly connected to the instance to check the instance's status. | 12:28 |
| chen12 | I get : | 12:28 |
| chen12 | ...... | 12:28 |
| chen12 | net0:10.1.0.4/255.255.255.0 gw 10.1.0.1 | 12:28 |
| chen12 | Next server:192.168.50.1 | 12:28 |
| chen12 | Filename: pxelinux.0 | 12:28 |
| chen12 | tftp://192.168.50.1/pxelinux.0 ................ Connection time out (http://ipxe.org/4c126035) | 12:28 |
| chen12 | No more network devices | 12:28 |
| chen12 | No bootable device. | 12:28 |
| chen12 | Anyone know why this is happened ??? | 12:28 |
| foexle_ | dtantsur: good hint i'll check thank you ! | 12:29 |
| *** martyntaylor1 has quit IRC | 12:29 | |
| *** martyntaylor has joined #openstack-ironic | 12:29 | |
| *** blamar has quit IRC | 12:30 | |
| *** blamar has joined #openstack-ironic | 12:31 | |
| openstackgerrit | Sandhya Balakrishnan proposed a change to openstack/ironic: Updates Ironic Guide with deployment information https://review.openstack.org/100787 | 12:31 |
| *** sabah has quit IRC | 12:39 | |
| *** lucas-lunch is now known as lucasagomes | 12:44 | |
| lucasagomes | chen12, seems neutron didn't started the dnsmasq service to answer the DHCP requests | 12:46 |
| lucasagomes | chen12, can you check if there's any dnsmaq service running and if the neutron port status == DOWN ? | 12:46 |
| chen12 | lucasagomes why you would say that ? I have dnsmaq service running, and isn't the instance already get IP address 10.1.0.4? | 12:47 |
| lucasagomes | Shrews, ping re 100957 | 12:47 |
| Shrews | lucasagomes: hi | 12:47 |
| chen12 | lucasagomes The output of the instance console (from VNCviewer) http://paste.openstack.org/show/85065/ | 12:48 |
| lucasagomes | Shrews, idk if I got the suggestion there, ur suggestion looks same but with more code | 12:48 |
| Shrews | lucasagomes: sorry about that. let me try again here.... :) | 12:49 |
| lucasagomes | Shrews, no it's cool | 12:49 |
| *** athomas has quit IRC | 12:49 | |
| lucasagomes | Shrews, cause in that hook you don't pass the exception it expects to handle | 12:49 |
| lucasagomes | Shrews, the exception raised will be passed to the handler method as the first parameter | 12:50 |
| chen12 | lucasagomes what I don't understand is: 192.168.50.1 is the physical machine's IP I'm woking on, while the 10.1.0.4 is the IP neutron dispatched. | 12:50 |
| *** Mikhail_D_ltp has joined #openstack-ironic | 12:50 | |
| Shrews | lucasagomes: in my example, the exception is passed to handle_error() | 12:50 |
| chen12 | lucasagomes How this two IP address work together ???? | 12:50 |
| Shrews | lucasagomes: so the idea is, what does your solution look like if you want to handle multiple types of exceptions? | 12:51 |
| lucasagomes | Shrews, yeah, but that's the same on the original, the exception is passed as the first argument to the method u set as hook | 12:51 |
| * lucasagomes maybe I'm missing something | 12:51 | |
| lucasagomes | Shrews, https://review.openstack.org/#/c/100958/3/ironic/conductor/manager.py | 12:51 |
| lucasagomes | Shrews, "if isinstance(e, exception.NoFreeConductorWorker):" | 12:52 |
| lucasagomes | you can add more elif isinstance(e, <exception) to hand multiple ones | 12:52 |
| Shrews | lucasagomes: right. but this method will need to grow very large for multiple exceptions, and maybe take lots of arguments | 12:52 |
| chen12 | but, yes, neutron port-show the neutron port status is down : http://paste.openstack.org/show/85067/ | 12:52 |
| lucasagomes | chen12, right, as part of the PXE boot process, it will send a DHCP request | 12:53 |
| Shrews | lucasagomes: with what i propose, the handle_error() should be small and can dispatch to private methods to handle each exception type | 12:53 |
| lucasagomes | that's the ip ur getting there | 12:53 |
| lucasagomes | chen12, the other IP was allocated to the instance (which is not deployed yet) | 12:53 |
| chen12 | lucasagomes the neutron port-show the neutron port status is down : http://paste.openstack.org/show/85067/ | 12:53 |
| Shrews | lucasagomes: and attributes can be set in the class to pass the data for the arguments | 12:54 |
| jroll | dtantsur: the json field thing ironic uses in the databases doesn't do nested structures :) | 12:54 |
| chen12 | lucasagomes and the dnsmasq process: http://paste.openstack.org/show/85066/ | 12:54 |
| jroll | morning ironic :) | 12:54 |
| dtantsur | jroll, morning and thanks for clarification! Do you have some spec already on how to store the inventory? | 12:54 |
| *** athomas has joined #openstack-ironic | 12:55 | |
| * dtantsur wants to copy-paste, doesn't want to think! | 12:55 | |
| Shrews | lucasagomes: your way will totally work, but my suggestion is just a more object-oriented way to handle it, and is a common design pattern for this type of thing | 12:55 |
| lucasagomes | Shrews, right... ok, what I don't get is that it's also possible with the other hook... you can create a class with the arguments and then you pass <Class>.handle_error to the handler | 12:55 |
| chen12 | lucasagomes I'm really really confused here. | 12:55 |
| lucasagomes | Shrews, that will work the same | 12:55 |
| lucasagomes | Shrews, alright, :) thanks for the suggestion I will take a look | 12:55 |
| jroll | dtantsur: lol spec :) | 12:55 |
| jroll | dtantsur: I... could write one. I'd prefer to just write code | 12:55 |
| * jroll finds said code | 12:55 | |
| dtantsur | jroll, code is nice too :) | 12:56 |
| dtantsur | jroll, and please your review, the generic spec is close to be finished :) | 12:56 |
| *** killer_prince has joined #openstack-ironic | 12:56 | |
| *** killer_prince is now known as lazy_prince | 12:56 | |
| jroll | dtantsur: line 225 and on https://review.openstack.org/#/c/84795/98/ironic/drivers/modules/agent_utils.py | 12:57 |
| jroll | dtantsur: there's a bug with lists at the leaf of the structure, in unflatten_dict. need to dig into | 12:57 |
| chen12 | lucasagomes How the instance reach tftp ? I think this is my issue now. Because it is said "tftp://192.168.50.1/pxelinux.0 ................ Connection time out (http://ipxe.org/4c126035)" | 12:57 |
| jroll | dtantsur: my review on the discovery thing? | 12:57 |
| dtantsur | jroll, yep, please! | 12:57 |
| jroll | dtantsur: will do. have some things this morning but will review by like lunch time today :) | 12:58 |
| dtantsur | ack, thanks! | 12:58 |
| lucasagomes | chen12, the DHCP server should send the pxe image to the node | 13:00 |
| dtantsur | jroll, btw, got a concern on review about messing with other's field in Node.extra. Can we include any kind of prefix for keys? like "inventory:iterfaces/.../..."? | 13:00 |
| lucasagomes | chen12, idk when/how that dnsmasq was started on ur env | 13:00 |
| jroll | dtantsur: yeah, I think this code already does: inventory/interfaces/... | 13:01 |
| lucasagomes | chen12, but, seems Ironic failed to update the dhcp options to serve the DHCP correctly | 13:01 |
| * lucasagomes grabs the link | 13:01 | |
| Shrews | lucasagomes: passing <Class>.handle_error would work too. i'm just offering a way that *I* think is more readable/maintable. but my feelings will not be hurt if you choose to ignore my suggestion :) | 13:02 |
| jroll | dtantsur: if that's not how it works today, I'm happy to fix. this code still needs its own patch anyway :) | 13:02 |
| dtantsur | jroll, ok, I'll put that to my spec | 13:02 |
| lucasagomes | chen12, https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/pxe.py#L509 | 13:02 |
| chen12 | lucasagomes what you mean "idk when/how that dnsmasq was started on ur env " ??? I think I know my dnsmasq process that is dispatch IP address for 10.1.0.4 i started by neutron-dhcp-agent | 13:02 |
| chen12 | lucasagomes do you mean there shoudl be another dnsmasq ? | 13:03 |
| lucasagomes | Shrews, no it's fine I will take a look... I'd like to see other people opnion on that | 13:03 |
| chen12 | lucasagomes then what IP range should it work with ? | 13:03 |
| jroll | dtantsur: honestly, I think I'd rather have full support for nested structures in the json field we use, with this function | 13:03 |
| lucasagomes | Shrews, I just think that the old way is more flexible | 13:03 |
| jroll | dtantsur: but I think I'll get this function merged and then propose this later | 13:04 |
| dtantsur | yeah, we can fix whatever we want later | 13:04 |
| lucasagomes | Shrews, cause it's generic enough, all it needs is a executable method... Doesn't need to be an instance of a specific class etc | 13:04 |
| dtantsur | let's start with the minimum :) | 13:04 |
| jroll | dtantsur: now that I think about it, I think how it works today is node.extra['inventory'] = flatten_dict({'interfaces': ...}) | 13:04 |
| jroll | but yeah, agree :) | 13:05 |
| dtantsur | jroll, so you actually have nested dicts, no? | 13:05 |
| jroll | I gotta run for now but will review your spec this morning, thanks! | 13:05 |
| chen12 | lucasagomes pxe_bootfile_name=pxelinux.0 | 13:05 |
| jroll | dtantsur: yes... just not in the db | 13:05 |
| dtantsur | thanks, see you later | 13:06 |
| chen12 | lucasagomes what is neutron.update_neutron do ? | 13:06 |
| Shrews | lucasagomes: it comes down to a preference of using OO methodology or functional programming, i think. yes, i'd like to see other's opinion too | 13:06 |
| jroll | dtantsur: I retract my last statement. it ends up as {'inventory/interfaces/0/mac_addr': '00:11:22:33:44:55', ...} | 13:06 |
| jroll | :) | 13:06 |
| dtantsur | ok! | 13:07 |
| * jroll drinks more coffee before talking any more | 13:07 | |
| jroll | dtantsur: hmm, reading comments. wdyt about just adding an 'inventory' or 'hardware' field in the db? | 13:08 |
| lucasagomes | chen12, so it's suppose to tell neutron (which will configure dnsmasq) to send the PXE boot image (pxelinux.0) as part of the DHCP request answer | 13:08 |
| jroll | dtantsur: that comes dangerously close to the line of 'ironic is not an inventory manager' but I think devananda is on board with what we're doing here :) | 13:09 |
| lucasagomes | chen12, http://blogging.dragon.org.uk/howto-setup-a-pxe-server-with-dnsmasq/ | 13:09 |
| dtantsur | jroll, hmmm... | 13:09 |
| dtantsur | jroll, let us ask devananda, what he likes more: prefix keys in `extras` or separate field (I'm in favor of the latter) | 13:10 |
| jroll | dtantsur: yeah, I'd like a new field. we just used extra at first because it was specific to us | 13:11 |
| lucasagomes | Shrews, +1 | 13:11 |
| jroll | dtantsur: left a note. should really go now, thanks :) | 13:12 |
| chen12 | lucasagomes there is a warning in method neutron.update_neutron, and looks like the warning I get before I add --net parameter for nova boot | 13:12 |
| lucasagomes | chen12, right, I'm also struggling with neutron on my env :( trying to learn how to debug it | 13:13 |
| *** foexle has quit IRC | 13:14 | |
| chen12 | lucasagomes https://github.com/openstack/ironic/blob/master/ironic/common/neutron.py#135 | 13:14 |
| lucasagomes | chen12, but in ur case it looks like that neutron didn't configured the dnsmasq process to serve as a PXE server, and the reason seems that it's because the port is DOWN | 13:14 |
| lucasagomes | now why the port is DOWN idk, that's the problem I've on my env as well | 13:14 |
| lucasagomes | chen12, https://github.com/openstack/ironic/blob/master/ironic/nova/virt/ironic/driver.py#L653 | 13:16 |
| *** foexle has joined #openstack-ironic | 13:16 | |
| lucasagomes | this method should have added the VIF, do a ironic node-show <node_uuid> | 13:16 |
| lucasagomes | check if you have a "vif_port_id" parameter in the extra field | 13:17 |
| *** martyntaylor1 has joined #openstack-ironic | 13:20 | |
| chen12 | lucasagomes what is "extra field" ? didn't found vif_port_id | 13:21 |
| lucasagomes | chen12, it's a attribute of the node resource | 13:22 |
| chen12 | lucasagomes ironic port-show has this infomation | 13:22 |
| lucasagomes | chen12, that's the node metadata | 13:22 |
| lucasagomes | chen12, oh my bad, yeah the port should have it | 13:23 |
| *** martyntaylor1 has quit IRC | 13:23 | |
| *** martyntaylor has quit IRC | 13:23 | |
| *** martyntaylor has joined #openstack-ironic | 13:23 | |
| dtantsur | jroll, when you're back: I'm going to expect "ip_address" (may be null) for NIC, expect frequency in MHz, memory and hard driver in MB, ok? | 13:24 |
| chen12 | lucasagomes The method spawn is the method that doing all these work, right ????https://github.com/openstack/ironic/blob/master/ironic/nova/virt/ironic/driver.py#L414 | 13:24 |
| *** rakesh_hs has quit IRC | 13:24 | |
| chen12 | lucasagomes how this method "call" deploy ???? https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/pxe.py#L509 | 13:26 |
| openstackgerrit | Dmitry Tantsur proposed a change to openstack/ironic-specs: Generic Hardware Discovery Bits https://review.openstack.org/102565 | 13:26 |
| lucasagomes | chen12, right, so the spawn() method is part of the Nova Ironic Driver | 13:27 |
| lucasagomes | chen12, the spawn() method will set up the Ironic node (set the values coming from flavor, associate the node with the nova instance that will be deployed etc) | 13:28 |
| lucasagomes | and then it triggers the deployment here: https://github.com/openstack/ironic/blob/master/ironic/nova/virt/ironic/driver.py#L462 | 13:28 |
| lucasagomes | chen12, are u familiar with the Ironic architecture ? | 13:32 |
| chen12 | lucasagomes no,actually, today is my first day install ironic | 13:33 |
| lucasagomes | chen12, right... our docs is not very complete really, but it might worth to u to watch a presentation about it | 13:33 |
| lucasagomes | to understand it's componets etc | 13:33 |
| lucasagomes | the iteraction with nova and all | 13:33 |
| chen12 | lucasagomes sure, can you send me the link of the presentation ? | 13:34 |
| lucasagomes | chen12, devananda is the PTL and he does many presentations about Ironic, some slides you can find on his slideshare http://www.slideshare.net/devananda1 | 13:34 |
| lucasagomes | chen12, there's videos of those presentation on youtube as well | 13:35 |
| lucasagomes | chen12, you can check that out | 13:35 |
| chen12 | lucasagomes sure! Thanks for the infomation | 13:35 |
| lucasagomes | chen12, here thre's a bit of information that might help : http://docs.openstack.org/developer/ironic/deploy/user-guide.html | 13:35 |
| chen12 | lucasagomes I have read this part already. :) | 13:36 |
| lucasagomes | ^^ ok | 13:36 |
| chen12 | lucasagomes this line should communicate with neutron-server ,rght ?? | 13:38 |
| chen12 | lucasagomes https://github.com/openstack/ironic/blob/master/ironic/common/neutron.py#148 | 13:38 |
| lucasagomes | chen12, yes | 13:39 |
| lucasagomes | chen12, that will send the extra DHCP options to neutron (with the tftp server ip, the PXE boot image etc) | 13:39 |
| chen12 | lucasagomes may be I should check neutron-server's log too. | 13:39 |
| *** Shiv has joined #openstack-ironic | 13:39 | |
| *** bvivek has quit IRC | 13:41 | |
| chen12 | lucasagomes Time is late here... I have to leave now.. nice talking to you. Gona to work next week... | 13:49 |
| lucasagomes | chen12, right, no problem... sorry I dunno neutron very well to help u more with it | 13:49 |
| lucasagomes | chen12, trying to learn it as well | 13:49 |
| chen12 | lucasagomes I have also send my issue through mailing list, although no one answer me yet, :( But if you have any idea, would you email me ? | 13:50 |
| lucasagomes | chen12, have a great weekend | 13:50 |
| lucasagomes | chen12, did you send it already? (didn't take a look at the maillist lately) | 13:50 |
| chen12 | lucasagomes titled with " [ironic]Failed to boot ironic instance " | 13:50 |
| chen12 | lucasagomes a good thing to know I'm not alone. :) | 13:51 |
| *** linggao has joined #openstack-ironic | 13:51 | |
| lucasagomes | chen12, yeah :) I bet there's more ppl suffering with the same problem heh | 13:53 |
| lucasagomes | neutron is complicated and hard to debug and figure out what's going on | 13:53 |
| *** blamar has quit IRC | 13:56 | |
| *** pcrews has joined #openstack-ironic | 13:57 | |
| Shiv | Hi lucasagomes | 13:57 |
| *** chen12 has quit IRC | 13:59 | |
| lucasagomes | Shiv, hi there | 14:00 |
| Shiv | I have raised a review related to firmware settings https://review.openstack.org/#/c/101122 . Can you please review the same | 14:01 |
| Shiv | A design spec review | 14:02 |
| lucasagomes | Shiv, nice! I probably won't have enough time to check it out today :( mind if I take a look early next week? | 14:04 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add/Update docstrings in the Nova Ironic Driver https://review.openstack.org/97536 | 14:04 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Nova Ironic driver get_info() to return memory stats in KBytes https://review.openstack.org/103136 | 14:04 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Update Nova Ironic Driver destroy() method https://review.openstack.org/103137 | 14:04 |
| lucasagomes | rloo ^ hope I addressed all ur comments | 14:05 |
| Shiv | It should be fine. Thank you for considering my request. | 14:05 |
| lucasagomes | Shiv, np, thank you for submitting it! | 14:06 |
| jroll | dtantsur: works for me :) | 14:09 |
| dtantsur | cool! | 14:10 |
| openstackgerrit | linggao proposed a change to openstack/ironic: Fix exception handling in console https://review.openstack.org/102318 | 14:14 |
| dtantsur | Shiv, left my comments | 14:15 |
| openstackgerrit | linggao proposed a change to openstack/ironic: Fix exception handling in console https://review.openstack.org/102318 | 14:16 |
| *** geekyogi has quit IRC | 14:18 | |
| linggao | Hi lucasagomes | 14:22 |
| NobodyCam | good morning Ironic ...TGIF | 14:24 |
| linggao | good morning NobodyCam dtantsur lucasagomes Shiv jroll | 14:25 |
| lucasagomes | linggao, NobodyCam morning | 14:25 |
| lucasagomes | linggao, hi there | 14:25 |
| dtantsur | NobodyCam, linggao, morning! | 14:25 |
| dtantsur | and TGIF, yeah | 14:25 |
| dtantsur | :) | 14:26 |
| matty_dubs | Howdy NobodyCam, dtantsur, linggao, et al. | 14:26 |
| linggao | lucasagomes, if a dependency path is merged, how do you remove the dependency from a patch? | 14:26 |
| linggao | path->patch | 14:26 |
| NobodyCam | morning linggao dtantsur and matty_dubs | 14:26 |
| lifeless | linggao: you don't need to | 14:26 |
| dtantsur | ++ | 14:26 |
| NobodyCam | morning lifeless | 14:26 |
| linggao | morning lifeless | 14:26 |
| lifeless | linggao: but if the patch is outdated, just pushing up your rebased version will clear it up | 14:26 |
| lifeless | morning all :) | 14:27 |
| dtantsur | morning lifeless :) | 14:27 |
| lucasagomes | linggao, don't need :) git review won't send it if it's already merged | 14:27 |
| linggao | thanks lifeless and lucasagomes | 14:27 |
| *** rwsu has joined #openstack-ironic | 14:27 | |
| lucasagomes | morning lifeless | 14:27 |
| NobodyCam | morning lucasagomes :) | 14:28 |
| *** russell_h has joined #openstack-ironic | 14:30 | |
| *** russell_h has quit IRC | 14:30 | |
| *** russell_h has joined #openstack-ironic | 14:30 | |
| *** jgrimm has joined #openstack-ironic | 14:33 | |
| NobodyCam | has any one seen this error: i'm not seeing how the patch would cause such an error: http://logs.openstack.org/48/102948/2/check/gate-ironic-python26/eea0393/console.html#_2014-06-26_23_20_47_774 | 14:33 |
| linggao | lucasagomes, (and lifeless) I still do not quite know how to work on two patches, one depends on the other. | 14:37 |
| linggao | lucasagomes, lifeless: here is a flow I use: brant A, branch B dependes on A. (different order from yesterday) | 14:37 |
| linggao | git review -d "chang-id of A". This will create patch "review/A" | 14:37 |
| linggao | git checkout B | 14:37 |
| linggao | make changes in branch B and commit | 14:37 |
| linggao | git rebase "review/A" | 14:37 |
| linggao | git review | 14:37 |
| linggao | git checkout A | 14:37 |
| linggao | make changes in A and commit | 14:37 |
| linggao | git rebase master | 14:37 |
| linggao | git review | 14:37 |
| linggao | git checkout B | 14:37 |
| linggao | make changes in B and commot | 14:37 |
| linggao | now how do I rebase? | 14:37 |
| lucasagomes | linggao, right... forget having more than one branch for the dependencies | 14:37 |
| lucasagomes | linggao, you can have only one branch with the whole serie of patches | 14:38 |
| lucasagomes | patch A->B->C | 14:38 |
| linggao | lucasagomes, just one dependency. | 14:38 |
| linggao | B depends on A | 14:38 |
| lucasagomes | linggao, right, ok... just an example let's have 3 | 14:38 |
| lucasagomes | linggao, so u've ur branch and the path A B anc C is there... C depends on B and B depends on A | 14:39 |
| lucasagomes | in case you want to edit B let's say | 14:39 |
| lucasagomes | you can do a git rebase -i <hash of branch A/hash of the previous patch> | 14:39 |
| linggao | lucasagomes, ok | 14:39 |
| lucasagomes | not "branch A" sorry... "patch A" | 14:39 |
| *** comstud is now known as bearhands | 14:40 | |
| lucasagomes | git will show you a text editor with the patches you can edit, so you change it from "pick" to "edit" and save | 14:40 |
| lucasagomes | git will now point to the patch u marked to edit | 14:40 |
| lucasagomes | so you do all the changes, the you "git add" all the files you changed there, then you "git commit --amend", and "git rebase --continue" | 14:40 |
| lucasagomes | then you can do a "git review" | 14:41 |
| lucasagomes | that will send to gerrit patch B and C (because C depends on the new B that u just edited) | 14:41 |
| lucasagomes | linggao, I think that's an easier way to work with dependencies without dependending on multiple branches (linear dependencies) | 14:41 |
| lucasagomes | linggao, that way you can also mark multiple patches to be edited, and git will iteract between then from the oldest to the newest | 14:43 |
| lucasagomes | and you can edit one by one in order | 14:43 |
| lucasagomes | at the end you do ur git review and all the patches affected will be sent to gerrit | 14:43 |
| lucasagomes | lifeless, I assume u work in a similar way ^ ? | 14:44 |
| dtantsur | NobodyCam, what's the patch? | 14:45 |
| *** rameshg87 has joined #openstack-ironic | 14:46 | |
| linggao | lucasagomes, I'll save your instructions and try it. | 14:46 |
| linggao | thanks | 14:46 |
| lucasagomes | linggao, ack, yeah create like a new git repo | 14:46 |
| lucasagomes | and play a bit with "git rebase -i" | 14:47 |
| rameshg87 | dtantsur, need your review on the ilo power design spec https://review.openstack.org/#/c/97455/ | 14:47 |
| mgagne | What does node decommission mean? Is it about removing the node from the inventory pool? | 14:47 |
| rameshg87 | dtantsur, please have a look at it :-) | 14:47 |
| dtantsur | rameshg87, I'll try to :) | 14:48 |
| rameshg87 | NobodyCam, request your review as well on the ilo power design spechttps://review.openstack.org/#/c/97455/ | 14:48 |
| dtantsur | rameshg87, please note, that it is yet to be decided, which specs are going into J | 14:49 |
| dtantsur | devananda, wanted to talk to us today on it | 14:49 |
| *** faizan has joined #openstack-ironic | 14:49 | |
| rameshg87 | dtantsur, okay | 14:51 |
| NobodyCam | dtantsur: https://review.openstack.org/#/c/102948/2 | 14:51 |
| * dtantsur will look in 5 minutes, brb | 14:52 | |
| NobodyCam | rameshg87: let refill my coffee then I take another look | 14:52 |
| rameshg87 | thanks NobodyCam | 14:52 |
| Shiv | dtantsur, have addressed your initial comments for firmware spec review. Please have a look https://review.openstack.org/#/c/101122 | 14:56 |
| dtantsur | Shiv, I'll try to :) I'm seriously occupied today though | 14:57 |
| Shiv | dtantsur, ok. | 14:57 |
| Shiv | NobodyCam, can you please review design spec for firmware settings feature https://review.openstack.org/#/c/101122 | 14:58 |
| *** faizan has quit IRC | 14:59 | |
| Shiv | jroll, can you please review design spec for firmware settings feature https://review.openstack.org/#/c/101122 | 14:59 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: PEP fixes for the Nova driver https://review.openstack.org/103162 | 15:00 |
| NobodyCam | rameshg87: The commented | 15:01 |
| *** viktors has left #openstack-ironic | 15:01 | |
| devananda | morning, all | 15:01 |
| NobodyCam | good morning devananda :) | 15:01 |
| dtantsur | devananda, morning | 15:02 |
| NobodyCam | how was the flight back | 15:02 |
| rameshg87 | NobodyCam, thanks, let me look into it | 15:03 |
| jroll | Shiv: will do | 15:05 |
| jroll | morning again, everybody :) | 15:05 |
| * devananda just proposed the nova driver to nova's tree here: https://review.openstack.org/#/c/103164/ | 15:06 | |
| Shrews | devananda: w00t!! | 15:06 |
| lifeless | lucasagomes: yeah I work in a stack whenever things are even slightly related | 15:06 |
| Shiv | Thank you jroll | 15:06 |
| jroll | devananda: nice! now I can use it as a dependency :P | 15:07 |
| lucasagomes | devananda, morning | 15:07 |
| lucasagomes | devananda, w00ts! | 15:07 |
| devananda | jroll: it's a chain of 5 patches, fwiw | 15:07 |
| rameshg87 | NobodyCam, just wondering if this answers your question. users can install ssl certificate on their ilo. the ribcl client used works irrespective of any certificate installed on the ilo. | 15:07 |
| devananda | jroll: we'll see if nova core wants it split up ay more | 15:07 |
| rameshg87 | Nobody, the ribcl client (any the ilo driver) will still need to input the username/password for the communication | 15:08 |
| jroll | devananda: yep, I see that. going to wait a couple days for initial review, I'm sure my stuff will get clobbered if I put something up now | 15:08 |
| lucasagomes | devananda, FYI, I updated the patch adding/updating docstring for the methods in the driver | 15:08 |
| rameshg87 | NobodyCam, does that answer your question ? | 15:08 |
| devananda | lucasagomes: ok. i'll probably need to set up a script to sync things back and forth -- haven't done that yet | 15:09 |
| lucasagomes | devananda, ack, yeah that sounds useful indeed | 15:09 |
| jroll | JayF: you should review this, too :) https://review.openstack.org/#/c/101122 | 15:10 |
| Shiv | lifeless, can you please review design spec for firmware settings feature https://review.openstack.org/#/c/101122 | 15:11 |
| *** rameshg87 has left #openstack-ironic | 15:15 | |
| *** mdorman has joined #openstack-ironic | 15:16 | |
| dtantsur | devananda, are we having a discussion on what specs to land for J today? | 15:17 |
| NobodyCam | Shiv: commented | 15:17 |
| devananda | dtantsur: that'd be great | 15:18 |
| devananda | dtantsur: how much longer are you around? | 15:18 |
| dtantsur | devananda, up to 5 hours, preferably 3-4 | 15:19 |
| dtantsur | devananda, also need to relocate home from office, it's gonna take ~30 minutes | 15:19 |
| linggao | hmm, no idea why my patch failed the Jenkins varification. It works in my sandbox. Could anyone help? https://review.openstack.org/#/c/102318/4 | 15:20 |
| NobodyCam | rameshg87 so the client would not have a need to install anything to use a self-singed cert? | 15:21 |
| dtantsur | NobodyCam, commented on a patch, tl;dr comma only works for LOG.smth, use % for exceptions | 15:21 |
| *** openstackgerrit has quit IRC | 15:21 | |
| devananda | dtantsur: how about in an hour? | 15:22 |
| devananda | lucasagomes: an hour from now work for you? | 15:22 |
| dtantsur | devananda, works for me. If everyone confirms, I start relocation right now | 15:22 |
| lucasagomes | devananda, a bit tight but yeah (it will be ~17:30 here) | 15:22 |
| lucasagomes | but yah | 15:22 |
| jroll | this is... interesting: http://logs.openstack.org/18/102318/4/check/check-tempest-dsvm-virtual-ironic/dc6776a/console.html | 15:22 |
| lucasagomes | yeah* | 15:22 |
| jroll | that's linggao's jenkins failure ^ | 15:22 |
| devananda | lucasagomes: later? sooner? | 15:22 |
| lucasagomes | devananda, if possible to do a bit sooner the better | 15:23 |
| lucasagomes | devananda, but if not, no problem I'll join anyway :) | 15:23 |
| devananda | jroll, linggao: that's a known bug in eventlet .... urggh. I dont think we've hit that in the gate before | 15:23 |
| devananda | https://bugs.launchpad.net/ironic/+bug/1321787 | 15:23 |
| jroll | too many known bugs in eventlet :( | 15:23 |
| dtantsur | ... and it does not support Python 3! | 15:24 |
| jroll | ^^ | 15:24 |
| jroll | imo that's just another known bug | 15:24 |
| NobodyCam | dtantsur: ahh .. TY good catch! | 15:24 |
| jroll | doesn't support pypy either afaik | 15:24 |
| devananda | http://logs.openstack.org/18/102318/4/check/check-tempest-dsvm-virtual-ironic/dc6776a/logs/screen-ir-cond.txt.gz?level=ERROR | 15:24 |
| jroll | s/afaik/last I checked/ | 15:24 |
| devananda | anyone want to set up an elastic recheck for that ? | 15:25 |
| jroll | I don't mind doing it if you point me in the right direction | 15:25 |
| devananda | jroll: git://git.openstack.org/openstack-infra/elastic-recheck | 15:25 |
| devananda | jroll: https://wiki.openstack.org/wiki/ElasticRecheck | 15:26 |
| jroll | thanks | 15:26 |
| devananda | cheers | 15:26 |
| linggao | devananda, the patch worked yesterday with set 2. | 15:26 |
| jroll | give me a bit on that | 15:26 |
| devananda | linggao: yes. that is a non-deterministic error. please "recheck bug 1321787" | 15:27 |
| jroll | linggao: that looks like a probably intermittent failure. you can comment with 'recheck bug #1321787' to run tests again | 15:27 |
| * devananda steps afk for a bit | 15:27 | |
| dtantsur | devananda, lucasagomes, what did we decide about meeting? | 15:27 |
| linggao | devananda, jroll, will do. thanks | 15:27 |
| jroll | linggao: no problem :) | 15:28 |
| dtantsur | ok, I'm gonna relocate, see you in 30 minutes | 15:28 |
| dtantsur | devananda, when you're back, could you please comment on https://review.openstack.org/#/c/102565/ and also what you thing about adding special field for hardware inventory instead of using Node.extras | 15:29 |
| devananda | dtantsur: ironic should not become a CMDB | 15:30 |
| jroll | devananda: either way the same data is being stored | 15:30 |
| jroll | devananda: unless you mean to say that data should only be put in 'properties' as basic cpu/ram/disk info | 15:31 |
| devananda | to paraphrase, and then i really do need to step away | 15:31 |
| devananda | CMDBs have a lot of additional functional requirements that have no relationship to "provision this image on that server" | 15:32 |
| Shiv | NobodyCam, thank you. Will look into it. | 15:32 |
| devananda | while Ironic may "own" the hardware interface (eg, IPMI/iLO/IPA) a full schema to represent inventory data (and search it, and audit it, and retain historical data, and relate to ticketing systems, and ...) is out of scope | 15:33 |
| devananda | try to think about how it works if Ironic has a designated location to store inventory data which can not be indexed or queried | 15:33 |
| devananda | that doesn't fulfil the needs of "find any node with firmware rev X and start the process (human or automated) to update it to rev Y" | 15:34 |
| devananda | which is a common req of CMDBs | 15:34 |
| * devananda really steps away now | 15:34 | |
| *** igordcard has quit IRC | 15:39 | |
| *** openstackgerrit has joined #openstack-ironic | 15:45 | |
| jroll | Shiv: reviewed :) | 15:46 |
| Shiv | jroll: thank you | 15:46 |
| jroll | don't thank me until you've seen the comments ;) | 15:47 |
| jroll | (joking) | 15:47 |
| Shiv | :) | 15:47 |
| NobodyCam | brb | 15:49 |
| *** martyntaylor has quit IRC | 15:49 | |
| *** amitpp has joined #openstack-ironic | 15:51 | |
| *** matty_dubs is now known as matty_dubs|lunch | 15:54 | |
| Shiv | dtansur, jroll : wanted to discuss few comments on firmware settings design spec. Can we do it now? | 15:54 |
| Shiv | I mean dtantsur | 15:55 |
| NobodyCam | :0p | 15:58 |
| jroll | Shiv: dtantsur is in transit and then is discussing features for juno with some folks, but I can talk now | 15:58 |
| Shiv | thanks. | 15:59 |
| *** klindgren has quit IRC | 15:59 | |
| Shiv | this is regarding only get/set_firmware_settings instead of big list | 15:59 |
| *** klindgren has joined #openstack-ironic | 15:59 | |
| jroll | sure | 16:00 |
| jroll | I guess my question is: is the big list of functions just shortcuts for e.g. get_firmware_settings(node, 'boot_device') ? | 16:00 |
| Shiv | Actually get/set_firmware settings would cover all settings. But for some standard settings like boot related if we provide the get_* methods, that would enforce the standarddization of names for these settings | 16:01 |
| Shiv | The standard names for these settings would help user when he uses different drivers that implement these APIs. Atleast for some standard settings he could have common names | 16:02 |
| Shiv | And that is the reason have suggested on get_* methods for them and not separate set_* methods. | 16:02 |
| jroll | hmm | 16:03 |
| jroll | not the worst idea | 16:03 |
| Shiv | get_firmware_settings() would provide all the settings that vendor would like to support. Common plus vendor specific ones | 16:03 |
| *** eghobo has joined #openstack-ironic | 16:04 | |
| jroll | Shiv: maybe another idea is have a standard name you can pass to get_firmware_settings(), that is an alias to the actual name | 16:04 |
| NobodyCam | bbt..brb | 16:04 |
| dtantsur | Shiv, I'm still not convinced we should have that many methods | 16:04 |
| *** eghobo has quit IRC | 16:04 | |
| dtantsur | Shiv, and we'll anyway have to document standard options for generic get/set | 16:05 |
| *** eghobo has joined #openstack-ironic | 16:05 | |
| jroll | Shiv: like, a driver can have a dictionary like {'boot_device': 'weird_vendor_boot_device_name'} | 16:05 |
| dtantsur | Shiv, having 2 ways of setting the same parameter is even worse | 16:05 |
| jroll | and user can just pass 'boot_device' and it will resolve it | 16:05 |
| jroll | +1 dtantsur | 16:05 |
| Shiv | dtantsur: have removed all the set_* methods. Only one method to set. set_firmware_settings() | 16:06 |
| dtantsur | devananda, what we want is to avoid cluttering Node.extras. New attribute will just as well be storing JSON | 16:06 |
| jroll | dtantsur: I have a feeling deva doesn't want to store this detailed data at all :) | 16:07 |
| Shiv | We could have standard names to be passed to get_firmware settings(), but I am not sure if that can be enforced programattically.or that is to be handled through documentation | 16:08 |
| dtantsur | Shiv, the same for `get`. It's just too many new methods for not-too-complicated thing | 16:08 |
| dtantsur | Shiv, through both, I guess. | 16:08 |
| Shiv | jroll, dtantsur We could have standard names to be passed to get_firmware settings(), but I am not sure if that can be enforced programattically.or that is to be handled through documentation | 16:09 |
| jroll | Shiv: both :) | 16:09 |
| dtantsur | ++ | 16:09 |
| Shiv | jroll, dtantsur: ok. I will check how we can do this programatically. I think you mean the filtered list passed to get_firmware settings could have some key names which could be marked as standard ones | 16:12 |
| *** jcoufal has quit IRC | 16:12 | |
| jroll | Shiv: indeed | 16:14 |
| Shiv | jroll, dtantsur: Regarding patch failure scenario. Since each of the settings is independent of each other, partial patch would get applied. | 16:14 |
| jroll | Shiv: the user should be told so | 16:15 |
| Shiv | jroll: Ok. through http code? I'll check the http code that is used for partail failure of operation. | 16:16 |
| jroll | Shiv: makes sense to me, plus maybe a response with what failed and what was ok | 16:16 |
| jroll | Shiv: although I'd almost say a partial patch should not be applied, for "safety" | 16:16 |
| jroll | JayF: I know this is a bad time for you, but you should read this conversation and the matching spec: https://review.openstack.org/#/c/101122 | 16:17 |
| JayF | yeah I saw it, it's in a tab | 16:17 |
| jroll | awesome | 16:17 |
| Shiv | jroll : ok. I will surely read this. | 16:17 |
| jroll | Shiv: read what? that was for JayF, he's been working on some bios things on our team :) | 16:18 |
| JayF | I think the BIOS things have been working on me | 16:18 |
| jroll | lol | 16:18 |
| Shiv | ok. I thought you provided another link for me :) | 16:19 |
| Shiv | jroll: my bad. | 16:19 |
| jroll | no worries :) | 16:19 |
| dtantsur | lucasagomes, hopefully I addressed your comments | 16:19 |
| lucasagomes | dtantsur, ack | 16:19 |
| Shiv | jroll: regarding your comment ' What new module? Would these methods not just be implemented in existing drivers?' | 16:19 |
| openstackgerrit | Divakar Padiyar Nandavar proposed a change to openstack/ironic: Fix the section name in CONTRIBUTING.rst https://review.openstack.org/103189 | 16:20 |
| dtantsur | lucasagomes, please have a look. I guess there's some misinterpretation, of what this spec is for | 16:20 |
| Shiv | jroll: i meant new methods in the ManagementInterface | 16:20 |
| jroll | Shiv: I figured, just wanted to be sure :) please fix | 16:20 |
| lucasagomes | dtantsur, will do, finishing something here quickly and then I will take a look | 16:21 |
| Shiv | jroll: regarding 'reboot' node after applying firmware settings. Dropped it purposefully, as the user may want to do more maintenance related actions. And there is a separate interface available to reboot the server | 16:22 |
| jroll | Shiv: I think most often, people will want to reboot immediately. but I don't run a datacenter, so I might be wrong. | 16:22 |
| jroll | Shiv: it's fine either way, just an idea. would like to hear what others think | 16:23 |
| Shiv | jroll: ok | 16:23 |
| dtantsur | I'm ok with leaving reboot separate | 16:23 |
| Shiv | jroll: Regarding representation of the settings values. "As mentioned above, I think this should be a list e.g. ['Boot0008', 'Boot0009', 'Boot000A', 'Boot000D']" | 16:25 |
| Shiv | jroll: Was trying to represent it as key/value pairs and hence proposed it as '.' separated values for the settings that have multiple values. | 16:26 |
| Shiv | jroll: this was more from simplification in mind as not all settings have multiple values. | 16:26 |
| jroll | I don't see a problem with using lists | 16:27 |
| jroll | that's what lists are for | 16:27 |
| jroll | I think dtantsur mentioned this first | 16:27 |
| Shiv | jroll: supported_boot_ devices could be list, whereas current_boot_device a single value | 16:27 |
| jroll | which is fine | 16:27 |
| *** romcheg has quit IRC | 16:27 | |
| Shiv | jroll: Or possibly it could represented by having all the setting values as list. it should cover for both types of settings, one that has single or multiple values. | 16:30 |
| jroll | Shiv: meh. I'm fine with mixing strings and lists. you might be able to convince someone else, but idk. | 16:31 |
| *** romcheg has joined #openstack-ironic | 16:32 | |
| Shiv | jroll, I was worried on getting resistance to mix strings and lists. I did not see any instance of the same. That is the reason I proposing key/value format | 16:32 |
| *** geekyogi has joined #openstack-ironic | 16:33 | |
| jroll | really? hmm, I would think otherwise | 16:33 |
| Shiv | jroll: ok | 16:33 |
| * devananda is semi around, semi in other meetings | 16:35 | |
| NobodyCam | :) | 16:35 |
| NobodyCam | glance image-list | 16:36 |
| NobodyCam | doh.. ww | 16:36 |
| Shiv | jroll: i have covered all your major comments. I will work on other comments and get back to you. | 16:36 |
| jroll | Shiv: excellent, thanks :) | 16:36 |
| Shiv | jroll: Thank you for the quick review and patience to respond to my follow-up queries | 16:37 |
| jroll | no problem | 16:37 |
| Shiv | dtantsur: Thank you for the quick review and patience to respond to my follow-up queries | 16:37 |
| dtantsur | np :) | 16:38 |
| *** ellenh_ has joined #openstack-ironic | 16:45 | |
| dtantsur | devananda, sorry for pinging you, but are we having a discussion? That is a bit late for me and lucasagomes :) | 16:52 |
| lucasagomes | +1 | 16:52 |
| devananda | dtantsur: sorry -- yes. here | 16:52 |
| devananda | i was pulled into a TC meeting ... | 16:52 |
| devananda | so of the priorities we laid out at the summit | 16:53 |
| lucasagomes | devananda, right, are we having a hangout about it ? | 16:53 |
| lucasagomes | or IRC? | 16:54 |
| lucasagomes | or both | 16:54 |
| devananda | hangout is fine :) | 16:54 |
| dtantsur | oh, time to get properly dressed :D | 16:54 |
| lucasagomes | lol | 16:54 |
| lucasagomes | ok I will start it | 16:54 |
| *** jbjohnso has joined #openstack-ironic | 16:55 | |
| jroll | devananda: I'm happy to join this, if it would be helpful | 16:55 |
| lucasagomes | jroll, sure | 16:56 |
| lucasagomes | jroll, dont have ur gtalk tho, | 16:56 |
| jroll | jimrollenhagen@gmail | 16:56 |
| *** harlowja_away is now known as harlowja | 16:56 | |
| jroll | give me about two minutes | 16:56 |
| devananda | NobodyCam: around? want to join? | 16:57 |
| lucasagomes | jroll, added | 16:57 |
| jroll | thanks | 16:57 |
| * jroll grabs coffee | 16:57 | |
| *** matty_dubs|lunch is now known as matty_dubs | 16:57 | |
| lucasagomes | jroll, https://plus.google.com/hangouts/_/gwgqmct5doa7cezm5lkyanbdjaa?hl=en | 16:57 |
| *** Shiv has quit IRC | 17:07 | |
| *** athomas has quit IRC | 17:08 | |
| *** rameshg87 has joined #openstack-ironic | 17:13 | |
| openstackgerrit | Kyle Stevenson proposed a change to openstack/ironic-python-agent: Add support for passing no configdrive https://review.openstack.org/103195 | 17:15 |
| rameshg87 | NobodyCam, are you there ? | 17:16 |
| rameshg87 | NobodyCam, the user wouldn't need to do anything to install a self-signed certificate | 17:16 |
| jroll | devananda: https://review.openstack.org/#/c/101920/ and https://review.openstack.org/#/c/102563/ | 17:16 |
| rameshg87 | devananda, request your review on ilo power spec https://review.openstack.org/#/c/97455/ | 17:17 |
| *** achanda has joined #openstack-ironic | 17:17 | |
| *** romcheg has quit IRC | 17:17 | |
| rameshg87 | dtantsur, i still didn't get your time for ilo power review spec. :-( - https://review.openstack.org/#/c/97455/ | 17:20 |
| *** rameshg87 has left #openstack-ironic | 17:24 | |
| *** achanda_ has joined #openstack-ironic | 17:33 | |
| matty_dubs | How many of you folks are staying at the 'recommended' hotel (Larkspur Landing) for the meetup? | 17:33 |
| *** overlayer has joined #openstack-ironic | 17:35 | |
| *** achanda has quit IRC | 17:36 | |
| openstackgerrit | Kyle Stevenson proposed a change to openstack/ironic-python-agent: Add support for passing no configdrive https://review.openstack.org/103195 | 17:39 |
| *** pelix has quit IRC | 17:44 | |
| *** Nisha has joined #openstack-ironic | 17:51 | |
| Nisha | dtantsur, HI | 17:51 |
| dtantsur | Hi! | 17:52 |
| Nisha | dtantsur: Thanks for the review. I was gng thru the comments | 17:53 |
| Nisha | dtantsur: one common comment i see is this | 17:53 |
| Nisha | please change this dict to a format used by IPA | 17:53 |
| Nisha | dtantsur: I have a ques, rather concern | 17:54 |
| Nisha | dtantsur: Why we want generic stuff to be similar to what IPA has today? That should be generic and should not be tied up with one driver | 17:54 |
| *** rameshg87 has joined #openstack-ironic | 17:55 | |
| dtantsur | Nisha, you can always suggest better ideas :) | 17:55 |
| Nisha | dtantsur: I see most of the comments saying that do as IPA do it , which is suggested in another spec raised by you | 17:56 |
| Nisha | :) | 17:56 |
| dtantsur | IPA has quite good extensible format that is _already_ working (and it wasn't written by me) | 17:56 |
| openstackgerrit | Alex Weeks proposed a change to openstack/ironic: Add statsd metrics support (not ready) https://review.openstack.org/103202 | 17:56 |
| dtantsur | I only took the only present and working format and put it to a generic spec | 17:56 |
| dtantsur | don't give me credit, I didn't deserve it :) | 17:57 |
| rameshg87 | dtantsur, i am back to bug again. sorry. the ilo power spec review :-) -- https://review.openstack.org/#/c/97455/ | 17:58 |
| dtantsur | rameshg87, I remember. I'm on a meeting and is terribly out of time | 17:58 |
| rameshg87 | dtantsur, was the meeting on hangout now to decide the features going into juno ? | 17:58 |
| dtantsur | yep | 17:58 |
| rameshg87 | dtantsur, thanks, please have a look when you get time :-) | 17:58 |
| rameshg87 | dtantsur, is there some meeting notes available. | 17:59 |
| dtantsur | rameshg87, there will be | 17:59 |
| rameshg87 | thanks dtantsur | 18:00 |
| *** rameshg87 has left #openstack-ironic | 18:00 | |
| Nisha | dtantsur: one comment is "What should they do, if they can't implement it?" ..... What do you expect there | 18:00 |
| Nisha | ? | 18:00 |
| Nisha | dtantsur: i think this is a comment which will apply to any feature who implements a new abstract function | 18:01 |
| dtantsur | Nisha, yes and we ask it every time | 18:01 |
| Nisha | dtantsur: I think this will apply to the new abstract functions proposed by you too :) | 18:02 |
| dtantsur | Nisha, sure, and I've written what to do | 18:03 |
| *** harlowja is now known as harlowja_away | 18:04 | |
| openstackgerrit | Kyle Stevenson proposed a change to openstack/ironic-python-agent: Add support for passing no configdrive https://review.openstack.org/103195 | 18:07 |
| *** Penick has joined #openstack-ironic | 18:15 | |
| *** harlowja_away is now known as harlowja | 18:15 | |
| dtantsur | ok, based on out chat I am altering discovery spec: dropping storing inventory, dropping find_node (leaving only match by MAC) | 18:17 |
| lucasagomes | dtantsur, +1 | 18:17 |
| dtantsur | the less text - the better :) | 18:17 |
| openstackgerrit | Dmitry Tantsur proposed a change to openstack/ironic-specs: Generic Hardware Discovery Bits https://review.openstack.org/102565 | 18:19 |
| dtantsur | lucasagomes, jroll, NobodyCam, devananda ^^^ | 18:19 |
| lucasagomes | that was... quick | 18:20 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic-specs: Swift Temporary URLs Spec https://review.openstack.org/98904 | 18:20 |
| dtantsur | folks, you have the last chance to make _me_ update it, very soon I hand it over to lucasagomes and jroll and go on PTO :) | 18:20 |
| jroll | super quick | 18:21 |
| jroll | dtantsur: go ahead, it's beer o'clock for you :) | 18:21 |
| dtantsur | well, deleting is faster than writing ;) | 18:21 |
| jroll | lol | 18:22 |
| dtantsur | I want to have one last look and the compiled HTML and than will go | 18:23 |
| jroll | ok :) | 18:23 |
| * jroll lunches | 18:23 | |
| dtantsur | In the meanwhile, I guess it's quite funny that Deva's patch to Nova adding empty __init__.py failed 2 checks, including Jenkins :) | 18:24 |
| jroll | . | 18:24 |
| *** kylestev has joined #openstack-ironic | 18:24 | |
| jroll | thats lovely | 18:24 |
| devananda | hah! | 18:24 |
| devananda | dtantsur: looks like random failure | 18:25 |
| devananda | the rate of those in openstack is far too high | 18:26 |
| NobodyCam | brb | 18:26 |
| devananda | horizon failed to start in the grenade test | 18:27 |
| *** kylestev has quit IRC | 18:28 | |
| *** kylestev has joined #openstack-ironic | 18:29 | |
| dtantsur | In the meanwhile, are we having a meeting on Monday? | 18:30 |
| dtantsur | devananda, ^^^ | 18:30 |
| *** kylestev has quit IRC | 18:31 | |
| devananda | dtantsur: sure. any reason not to? | 18:31 |
| *** foexle_ has quit IRC | 18:31 | |
| dtantsur | just asking. it's the only thing I'm going to do on Monday regarding work :) | 18:31 |
| devananda | :) | 18:32 |
| *** kylestev has joined #openstack-ironic | 18:32 | |
| *** klindgren has quit IRC | 18:33 | |
| *** ellenh_ has quit IRC | 18:35 | |
| lucasagomes | alright, done for today... g'night everybody, enjoy the weekend! | 18:37 |
| *** achanda_ has quit IRC | 18:37 | |
| matty_dubs | Adios lucasagomes | 18:38 |
| matty_dubs | dtantsur: Is Monday a holiday over there? | 18:38 |
| dtantsur | matty_dubs, no, it's my PTO | 18:38 |
| *** lucasagomes is now known as lucas-dinner | 18:38 | |
| dtantsur | g'night, lucas-dinner! | 18:39 |
| NobodyCam | night lucas-dinner | 18:39 |
| dtantsur | Ok! See some of you on Monday on IRC, see some of you in Paris, see the other in 2 weeks! | 18:41 |
| matty_dubs | Have fun, dtantsur! | 18:41 |
| *** dtantsur is now known as dtantsur|pto | 18:43 | |
| *** amitpp has quit IRC | 18:59 | |
| NobodyCam | have a good time dtantsur|pto | 19:00 |
| NobodyCam | ahh and I had a question about 102565 too | 19:01 |
| *** ellenh_ has joined #openstack-ironic | 19:08 | |
| matty_dubs | I missed the point where the _LI() HELPER WAS INTRODUCED?EXPLAINED | 19:18 |
| matty_dubs | Whoa sorry all caps | 19:18 |
| matty_dubs | shift key got stuck | 19:18 |
| matty_dubs | But I see discussion on using, e.g., LOG.info(_LI('message')) -- and attempting to google is turning up stuff about the <li> HTML tag. Can anyone steer me in the right direction? | 19:20 |
| *** openstackgerrit has quit IRC | 19:20 | |
| *** openstackgerrit has joined #openstack-ironic | 19:21 | |
| ellenh_ | matty_dubs: there’s an explanation here: http://docs.openstack.org/developer/oslo.i18n/guidelines.html#choosing-a-marker-function | 19:22 |
| matty_dubs | ellenh_: Thanks! | 19:22 |
| openstackgerrit | David Shrewsbury proposed a change to openstack/ironic-specs: Make the REST API fully asynchronous https://review.openstack.org/94923 | 19:25 |
| Shrews | devananda: Our spec format is missing a section for RPC API impact. | 19:26 |
| *** ellenh has quit IRC | 19:27 | |
| *** ellenh_ is now known as ellenh | 19:27 | |
| *** ellenh_ has joined #openstack-ironic | 19:27 | |
| devananda | Shrews: huh. good point. want to add it? | 19:29 |
| Shrews | devananda: sure | 19:29 |
| jroll | new ATC incoming: kylestev :) | 19:32 |
| jroll | devananda: ^ another intern working with us this summer | 19:33 |
| jroll | just +A'd his first patch | 19:33 |
| matty_dubs | Woo-hoo! Welcome, kylestev! | 19:34 |
| * devananda lunches, bbiafh | 19:35 | |
| openstackgerrit | A change was merged to openstack/ironic-python-agent: Add support for passing no configdrive https://review.openstack.org/103195 | 19:36 |
| *** Nisha has quit IRC | 19:39 | |
| openstackgerrit | David Shrewsbury proposed a change to openstack/ironic-specs: Add a section for RPC API impact https://review.openstack.org/103225 | 19:40 |
| *** datajerk1 has joined #openstack-ironic | 19:41 | |
| *** datajerk has quit IRC | 19:43 | |
| *** devananda has quit IRC | 19:43 | |
| *** mordred has quit IRC | 19:43 | |
| *** devananda has joined #openstack-ironic | 19:43 | |
| *** mordred has joined #openstack-ironic | 19:43 | |
| Shrews | kylestev: welcome to ironic | 19:44 |
| openstackgerrit | Adam Gandelman proposed a change to openstack/ironic: Update nova rebuild to account for new image https://review.openstack.org/103227 | 19:46 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic-specs: Support Decommissioning of Nodes https://review.openstack.org/102685 | 19:46 |
| openstackgerrit | David Shrewsbury proposed a change to openstack/ironic-specs: Make the REST API fully asynchronous https://review.openstack.org/94923 | 19:48 |
| openstackgerrit | Adam Gandelman proposed a change to openstack/ironic: Update nova rebuild to account for new image https://review.openstack.org/103227 | 19:50 |
| *** jgrimm has quit IRC | 19:58 | |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Add Node decommission in conductor https://review.openstack.org/102984 | 19:59 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Add Node decommission in conductor https://review.openstack.org/102984 | 19:59 |
| *** wanyen has joined #openstack-ironic | 20:01 | |
| NobodyCam | brb | 20:03 |
| kylestev | jroll: matty_dubs: Shrews: Thanks! Glad to be here :) | 20:19 |
| jroll | :) | 20:19 |
| jroll | kylestev: now you can go to the paris summit for free! | 20:19 |
| jroll | <fineprint>airfare, lodging, and food not included </fineprint> | 20:19 |
| kylestev | ;) | 20:19 |
| kylestev | I’ve been needing a reason to get my passport anywayts | 20:20 |
| kylestev | anyways* | 20:20 |
| jroll | lol | 20:20 |
| *** wanyen has quit IRC | 20:21 | |
| *** wanyen has joined #openstack-ironic | 20:21 | |
| wanyen | hi, we are looking for more reviewers for our iLO design specs, esecially iLO Power Driver https://review.openstack.org/#/c/97755/ and iLO Virtual Media Deploy driver https://review.openstack.org/#/c/97744/. | 20:29 |
| JoshNang | wanyen: i think that ilo power driver link might be wrong | 20:30 |
| wanyen | These two drivers are carry-over from icehouse. We have made substantial changes in virtual media driver to address icehouse review comments regarding performance | 20:30 |
| wanyen | okay. let me check teh link for power driver | 20:31 |
| JoshNang | ilo power driver: dhttps://review.openstack.org/#/c/97455/ | 20:32 |
| wanyen | iLO power driver link is https://review.openstack.org/#/c/97455/ | 20:32 |
| JoshNang | :) | 20:32 |
| wanyen | JoshNang: TY. | 20:33 |
| wanyen | We need reviewers to review these specs. We have the code ready but we wont' be able to submit until the specs are approved. | 20:34 |
| jroll | wanyen: you can submit code :) | 20:35 |
| jroll | wanyen: just, code cannot merge before spec is done :) | 20:35 |
| wanyen | jroll- good to know. We will do that then. | 20:36 |
| wanyen | please review these specs. The earlier we can get approval the better chance we get to land in hopefully Juno2. | 20:36 |
| jroll | yep, they will be reviewed :) | 20:37 |
| wanyen | jroll- thanks! | 20:37 |
| *** linggao has quit IRC | 20:38 | |
| NobodyCam | reviewing specs is very draining | 20:40 |
| jroll | ^^ | 20:41 |
| NobodyCam | lol so is re-living VBA scripts from the past ... lol | 20:42 |
| NobodyCam | :-p | 20:42 |
| JoshNang | so the ilo power driver requires proliantutils. does that need to go into the global requirements file? or will it just be mocked out and noted in documentation that you need it to test? | 20:42 |
| JoshNang | *need it to deploy/use | 20:43 |
| jroll | the latter | 20:43 |
| JoshNang | kk makes sense | 20:43 |
| *** coolsvapl has joined #openstack-ironic | 20:51 | |
| *** coolsvap|afk has quit IRC | 20:52 | |
| *** jbjohnso has quit IRC | 21:03 | |
| devananda | still it needs to go in global reqs, even if it's not in ironic/requirements.txt | 21:05 |
| devananda | JoshNang, jroll ^ | 21:06 |
| *** jdob has quit IRC | 21:06 | |
| devananda | so that packagers know to look there for the dependencies | 21:06 |
| jroll | oh huh | 21:06 |
| jroll | I see | 21:06 |
| JoshNang | ahh gotcha | 21:07 |
| jroll | JoshNang: I'm inclined to think passing a "swift+https://..." URL into the temp url function should produce an "https://..." URL | 21:12 |
| JoshNang | jroll: that's reasonable | 21:14 |
| * jroll will fix | 21:15 | |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic: Factor out TFTPImageCache https://review.openstack.org/100734 | 21:15 |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic: Factor out deploy info from PXE driver https://review.openstack.org/100735 | 21:16 |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic: Adding swift temp url support https://review.openstack.org/81391 | 21:16 |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic: Add ironic-python-agent deploy driver https://review.openstack.org/101020 | 21:16 |
| jroll | JoshNang: oops, can you edit the topic for https://review.openstack.org/#/c/81391/ | 21:16 |
| jroll | apparently I can't via web ui | 21:16 |
| JoshNang | heh i love the blocks of jroll changes that come with every rebase | 21:16 |
| jroll | >.> | 21:17 |
| JoshNang | one | 21:17 |
| JoshNang | *done. and fwiw i need to rework that patch to use the python-swiftclient patch (once its merged) | 21:18 |
| jroll | right | 21:19 |
| jroll | do you want to fix your patch to remove the 'swift+' then? | 21:20 |
| JoshNang | sure | 21:20 |
| jroll | cool | 21:21 |
| jroll | just the swiftclient patch, to be cleart | 21:21 |
| jroll | -t | 21:21 |
| JoshNang | the swiftclient only deals with the path portion | 21:21 |
| jroll | ahhh | 21:21 |
| jroll | right | 21:21 |
| JoshNang | so do you want me to change the ironic patch then? | 21:22 |
| *** matty_dubs is now known as matty_dubs|gone | 21:22 | |
| wanyen | Hi Deva, so far we only get one review comment from Dmitry to re-factor UEFI boot design spec into multiple specs. Can you take a look at the spec https://review.openstack.org/#/c/99850/ and give us suggestion on how to refactor? Any commetns regarding the content of the spec are most welcome so we can incoporate them into the re-factor specs. | 21:23 |
| jroll | JoshNang: nah, I'll hit it next time I rebase :) | 21:24 |
| JoshNang | heh k | 21:24 |
| *** zigo_ has joined #openstack-ironic | 21:24 | |
| *** NobodyCa1 has joined #openstack-ironic | 21:27 | |
| *** NobodyCam has quit IRC | 21:31 | |
| *** bearhands has quit IRC | 21:31 | |
| *** mrda-weekend has quit IRC | 21:31 | |
| *** wendar has quit IRC | 21:31 | |
| *** mgagne has quit IRC | 21:31 | |
| *** agordeev has quit IRC | 21:31 | |
| *** adam_g has quit IRC | 21:31 | |
| *** rainya has quit IRC | 21:31 | |
| *** stevebaker has quit IRC | 21:31 | |
| *** zigo has quit IRC | 21:31 | |
| *** morgabra has quit IRC | 21:31 | |
| *** antonym has quit IRC | 21:31 | |
| devananda | wanyen: not likely that i'll have time today | 21:31 |
| wanyen | <deva> as long as you can review it and provide feedback in the next few days will be fine. | 21:35 |
| *** agordeev has joined #openstack-ironic | 21:37 | |
| wanyen | JoshNang: ty for the review comments for iLO power and virtual media drivers. | 21:47 |
| *** rwsu has quit IRC | 21:49 | |
| wanyen | singingoff. Thanks! | 21:51 |
| *** wanyen has quit IRC | 21:51 | |
| JoshNang | wanyen: np! | 21:51 |
| *** mdorman has quit IRC | 21:58 | |
| *** ccrouch has quit IRC | 22:02 | |
| *** geekyogi has quit IRC | 22:03 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Updated from global requirements https://review.openstack.org/102126 | 22:08 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/88722 | 22:08 |
| *** jbjohnso has joined #openstack-ironic | 22:12 | |
| *** Penick has quit IRC | 22:13 | |
| *** ccrouch has joined #openstack-ironic | 22:17 | |
| *** adam_g has joined #openstack-ironic | 22:21 | |
| *** adam_g has quit IRC | 22:21 | |
| *** adam_g has joined #openstack-ironic | 22:21 | |
| *** ellenh_ has quit IRC | 22:25 | |
| *** ellenh_ has joined #openstack-ironic | 22:26 | |
| *** ellenh has quit IRC | 22:26 | |
| *** ellenh_ is now known as ellenh | 22:26 | |
| *** ellenh has quit IRC | 22:27 | |
| *** ellenh_ has joined #openstack-ironic | 22:31 | |
| *** ellenh_ has quit IRC | 22:31 | |
| NobodyCa1 | question. | 22:34 |
| *** NobodyCa1 is now known as NobodyCam | 22:34 | |
| NobodyCam | where should use cases go? in the spec, blueprint, an etherpad, or "Use Cases we don't need any Use Cases" | 22:35 |
| devananda | NobodyCam: spec | 22:37 |
| devananda | if it's really needed | 22:37 |
| jroll | NobodyCam: I'd say the 'problem description' section of the spec | 22:39 |
| jroll | like 'this is a problem because we have use case a, b, and c | 22:40 |
| jroll | ' | 22:40 |
| *** ellenh has joined #openstack-ironic | 22:40 | |
| NobodyCam | devananda: jroll: looking at https://review.openstack.org/#/c/99850/1/specs/juno/uefi-boot-for-ironic.rst | 22:41 |
| NobodyCam | line 63 thru 12x | 22:42 |
| NobodyCam | s/12x/129/ | 22:42 |
| jroll | well | 22:42 |
| jroll | those aren't use cases | 22:42 |
| jroll | but conditions where it may be used | 22:42 |
| NobodyCam | spec seems wrong place | 22:42 |
| jroll | I think the 'use cases' specified there are more of like, 'possible code paths' | 22:43 |
| NobodyCam | my initial thought was in BP | 22:43 |
| jroll | know what I mean? | 22:43 |
| NobodyCam | jroll: YES I do | 22:43 |
| jroll | :) | 22:43 |
| *** Mikhail_D_ltp has quit IRC | 22:43 | |
| NobodyCam | but I didn't quite know kow to comment on it? | 22:44 |
| jroll | yeah | 22:44 |
| jroll | it's also a bit verbose | 22:44 |
| devananda | so the term "use case" is misapplied there | 22:44 |
| jroll | I think I would remove the bullets | 22:44 |
| devananda | but I think the content is valid | 22:44 |
| jroll | and leave it in proposed change, like "ways uefi mode can be specified: config option, flavor, image property' | 22:45 |
| devananda | Note: User requested boot mode can be passed to ironic node by setting46 "requested_boot_mode" in driver_info field by nova ironic virt driver.47 | 22:45 |
| devananda | that ^ is not acceptable | 22:45 |
| devananda | nova virt driver should not be modifying driver_info | 22:46 |
| devananda | only instance info | 22:46 |
| jroll | +1 | 22:46 |
| NobodyCam | lol I have question on the conf vaules that line, | 22:46 |
| NobodyCam | and others | 22:46 |
| jroll | I think we're ignoring the other problems in this spec at the moment :P | 22:46 |
| NobodyCam | ack tackle a few at a time | 22:47 |
| devananda | jroll: such as? (i haven't really read it yet) | 22:47 |
| NobodyCam | to me that is almost instance info... | 22:49 |
| *** jbjohnso has quit IRC | 22:49 | |
| jroll | devananda: I've only skimmed in the last 5 minutes, but things like what you mentioned just now | 22:49 |
| NobodyCam | not driver... driver should support both / all | 22:50 |
| devananda | jroll, NobodyCam: comments posted on that spec | 23:05 |
| NobodyCam | :) | 23:06 |
| * devananda steps afk again | 23:08 | |
| *** lucas-dinner has quit IRC | 23:09 | |
| kylestev | jroll: got a task for me? | 23:16 |
| jroll | kylestev: go enjoy your weekend? :) | 23:16 |
| kylestev | haha | 23:16 |
| jroll | kylestev: otherwise... get devstack running in a vm | 23:16 |
| jroll | but I'm about to drop off | 23:16 |
| kylestev | got that done | 23:16 |
| kylestev | ah | 23:16 |
| jroll | I can find something if you want | 23:16 |
| NobodyCam | kylestev: what kind of vm? | 23:16 |
| jroll | maybe NobodyCam has some low-hanging fruit :P | 23:16 |
| kylestev | NobodyCam: vagrant w/ ubuntu | 23:17 |
| jroll | vmware | 23:17 |
| kylestev | yeah | 23:17 |
| jroll | ... why | 23:17 |
| kylestev | i forgot the provider | 23:17 |
| * jroll is skeptical | 23:17 | |
| NobodyCam | ahh I have fought (and lost) to nested kvm attempt this week | 23:17 |
| jroll | oh heh | 23:17 |
| jroll | yeah, we've been mostly testing on servers for that reason :P | 23:18 |
| jroll | NobodyCam: you're just doing kvm on kvm? | 23:18 |
| kylestev | i can work on my tech talk. just killing time until my dad gets up here from sjc. | 23:19 |
| jroll | ah | 23:19 |
| jroll | yeah, write a talk :) | 23:19 |
| NobodyCam | jroll: ya, devtest crashes the vm launching seed, | 23:20 |
| jroll | NobodyCam: do we have docs on running devtest? | 23:20 |
| lifeless | jroll: devtest is all docs :) | 23:21 |
| NobodyCam | http://docs.openstack.org/developer/tripleo-incubator/devtest.html | 23:21 |
| NobodyCam | not we but us | 23:21 |
| NobodyCam | lol | 23:21 |
| jroll | heh | 23:21 |
| jroll | I've been meaning to get this running | 23:22 |
| jroll | and/or tempest | 23:22 |
| NobodyCam | its much to close to the weekend | 23:22 |
| NobodyCam | :-p | 23:22 |
| jroll | oh yes | 23:23 |
| jroll | not today :) | 23:23 |
| lifeless | NobodyCam: thanks, was just looking that up | 23:23 |
| *** ccrouch has quit IRC | 23:27 | |
| NobodyCam | lol | 23:28 |
| * NobodyCam has that link in his link bar. lol :-p | 23:28 | |
| NobodyCam | lifeless: I was just looking over my reviews and found. https://review.openstack.org/#/c/98050/ which seem to have dropped the ball on keeping up with. I will fix up and repost. later tonight or tomorrow :) | 23:33 |
| NobodyCam | s/which/which I/ | 23:33 |
| lifeless | thanks | 23:34 |
| *** openstackgerrit has quit IRC | 23:37 | |
| *** rwsu has joined #openstack-ironic | 23:54 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!