| stevemar | notmorgan: :) | 00:08 |
|---|---|---|
| stevemar | notmorgan: that's like putting up the bat signal | 00:08 |
| jroll | notmorgan: who'd you expect :P | 00:14 |
| *** mbound has quit IRC | 00:16 | |
| *** syed_ has quit IRC | 00:22 | |
| *** vishwana_ has quit IRC | 00:48 | |
| *** hoangcx has joined #openstack-ironic | 00:51 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: HOME directory is deleted when run stack.sh and clean.sh in devstack https://review.openstack.org/366518 | 00:54 |
| *** wajdi has joined #openstack-ironic | 00:55 | |
| *** rloo has quit IRC | 00:58 | |
| *** rbudden has joined #openstack-ironic | 01:03 | |
| *** pmannidi has joined #openstack-ironic | 01:13 | |
| *** rbudden has quit IRC | 01:13 | |
| *** chlong_ has joined #openstack-ironic | 01:14 | |
| *** mbound has joined #openstack-ironic | 01:16 | |
| *** michauds has joined #openstack-ironic | 01:18 | |
| *** mbound has quit IRC | 01:22 | |
| *** wajdi has quit IRC | 01:25 | |
| *** wajdi has joined #openstack-ironic | 01:25 | |
| *** Sukhdev has quit IRC | 01:28 | |
| *** clenimar_ has joined #openstack-ironic | 01:31 | |
| *** syed_ has joined #openstack-ironic | 01:32 | |
| *** chlong_ has quit IRC | 01:49 | |
| *** chlong_ has joined #openstack-ironic | 02:06 | |
| *** wajdi has quit IRC | 02:45 | |
| *** vishwanathj has joined #openstack-ironic | 02:58 | |
| *** thrash is now known as thrash|g0ne | 03:02 | |
| *** pmannidi has quit IRC | 03:19 | |
| *** sdake has joined #openstack-ironic | 03:31 | |
| *** syed_ has quit IRC | 03:42 | |
| *** fxpester has joined #openstack-ironic | 03:55 | |
| *** michauds has quit IRC | 03:56 | |
| *** baoli_ has quit IRC | 03:57 | |
| *** pmannidi has joined #openstack-ironic | 04:04 | |
| *** aswadr_ has joined #openstack-ironic | 04:09 | |
| *** vishwanathj is now known as vishwanathj_zzz | 04:26 | |
| *** moshele has joined #openstack-ironic | 04:33 | |
| *** wajdi has joined #openstack-ironic | 04:56 | |
| *** jtomasek has quit IRC | 05:08 | |
| *** sdake_ has joined #openstack-ironic | 05:10 | |
| *** sdake has quit IRC | 05:12 | |
| *** amitkqed has quit IRC | 05:16 | |
| *** amitkqed has joined #openstack-ironic | 05:16 | |
| *** ChubYann has quit IRC | 05:16 | |
| *** jaosorior has joined #openstack-ironic | 05:21 | |
| *** _vsaienko has quit IRC | 05:32 | |
| *** penick has joined #openstack-ironic | 05:39 | |
| *** mjura has joined #openstack-ironic | 05:57 | |
| openstackgerrit | Himanshu Kumar proposed openstack/python-ironicclient: Added CLI links in the readme file https://review.openstack.org/369157 | 06:00 |
| openstackgerrit | Himanshu Kumar proposed openstack/python-ironicclient: Added CLI links in the readme file https://review.openstack.org/369157 | 06:02 |
| *** anush__ has joined #openstack-ironic | 06:04 | |
| *** mjura has quit IRC | 06:05 | |
| *** mjura has joined #openstack-ironic | 06:05 | |
| *** hoangcx has quit IRC | 06:05 | |
| *** hoangcx has joined #openstack-ironic | 06:06 | |
| anish | ifarkas_afk: you around ? | 06:08 |
| *** _vsaienko has joined #openstack-ironic | 06:14 | |
| *** rcernin has joined #openstack-ironic | 06:19 | |
| *** penick has quit IRC | 06:24 | |
| *** fragatina has quit IRC | 06:25 | |
| *** fragatina has joined #openstack-ironic | 06:26 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: HOME directory is deleted when run stack.sh and clean.sh in devstack https://review.openstack.org/366518 | 06:28 |
| *** fragatin_ has joined #openstack-ironic | 06:29 | |
| *** fragatina has quit IRC | 06:30 | |
| pas-ha | morning Ironic | 06:32 |
| *** fragatin_ has quit IRC | 06:33 | |
| *** fxpester has joined #openstack-ironic | 06:33 | |
| *** athomas has joined #openstack-ironic | 06:40 | |
| *** rbartal has joined #openstack-ironic | 06:47 | |
| *** jcoufal has joined #openstack-ironic | 06:50 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update iptables rules and services IPs for multinode https://review.openstack.org/368648 | 06:59 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add devstack setup_vxlan_network() https://review.openstack.org/368647 | 06:59 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Skip some steps for multinode case https://review.openstack.org/368646 | 06:59 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update ironic node names for multinode case https://review.openstack.org/368645 | 06:59 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update devstack provision net config for multihost https://review.openstack.org/368644 | 06:59 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 06:59 |
| *** pcaruana has joined #openstack-ironic | 07:02 | |
| *** tesseract- has joined #openstack-ironic | 07:08 | |
| *** jtomasek has joined #openstack-ironic | 07:10 | |
| *** yarkot has quit IRC | 07:11 | |
| *** aslezil has quit IRC | 07:11 | |
| *** manjeets- has quit IRC | 07:11 | |
| *** anush_ has quit IRC | 07:11 | |
| *** vgadiraj has quit IRC | 07:11 | |
| *** tuanla_ has joined #openstack-ironic | 07:12 | |
| *** dprince has joined #openstack-ironic | 07:13 | |
| *** sdake has joined #openstack-ironic | 07:19 | |
| *** ccamacho has joined #openstack-ironic | 07:19 | |
| *** sdake_ has quit IRC | 07:20 | |
| *** ifarkas_afk is now known as ifarkas | 07:24 | |
| ifarkas | morning all | 07:27 |
| ifarkas | anish, yeah, I am here | 07:27 |
| *** anush_ has joined #openstack-ironic | 07:28 | |
| *** aslezil has joined #openstack-ironic | 07:28 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic-inspector: Add translation marker to help of config opts https://review.openstack.org/365888 | 07:30 |
| *** vgadiraj has joined #openstack-ironic | 07:33 | |
| *** Romanenko_K has joined #openstack-ironic | 07:39 | |
| tangchen | morning all :) | 07:44 |
| *** pmannidi has quit IRC | 07:48 | |
| *** openstackgerrit has quit IRC | 07:48 | |
| *** openstackgerrit has joined #openstack-ironic | 07:49 | |
| *** dtantsur|afk is now known as dtantsur | 07:53 | |
| dtantsur | Morning Ironic, morning pas-ha, ifarkas, tangchen | 07:54 |
| *** ohamada has joined #openstack-ironic | 07:55 | |
| tuanla_ | Hello all, | 07:56 |
| tangchen | dtantsur: morning | 07:56 |
| tangchen | dtantsur: Hi Dmitry, would you please help to confirm this patch ? I have fixed all the comments, I think. | 07:59 |
| dtantsur | which exactly patch do you have in mind? | 07:59 |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:00 | |
| *** e0ne has joined #openstack-ironic | 08:00 | |
| tangchen | dtantsur: Sorry....https://review.openstack.org/#/c/349421/ | 08:01 |
| patchbot | patch 349421 - ironic - Prepare for functional tests | 08:01 |
| tuanla_ | Hello, does anyone know how can access devstack channel IRC | 08:02 |
| dtantsur | ok, this is a big one, I will try to get to it later today | 08:02 |
| dtantsur | tuanla_, there is no dedicated channel, depending on your question you might want to talk to #openstack-qa or #openstack-infra | 08:02 |
| tangchen | dtantsur: Thank you very much :) | 08:02 |
| tuanla_ | dtantsur: Thanks you | 08:03 |
| *** tuanla_ has quit IRC | 08:03 | |
| *** jpich has joined #openstack-ironic | 08:03 | |
| tangchen | dtantsur: BTW, this one is easier. Please take a look at it when you are free. :) https://review.openstack.org/#/c/357639/ | 08:04 |
| patchbot | patch 357639 - python-ironicclient - Support i18n for osc plugin help msg | 08:04 |
| *** tuanla_ has joined #openstack-ironic | 08:04 | |
| *** mbound has joined #openstack-ironic | 08:05 | |
| dtantsur | adding to my list as well | 08:07 |
| tangchen | dtantsur: Thanks. | 08:08 |
| *** sdake has quit IRC | 08:10 | |
| *** mgoddard has joined #openstack-ironic | 08:18 | |
| *** mkoderer has quit IRC | 08:20 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Fix breakage with oslo.config master https://review.openstack.org/367407 | 08:20 |
| openstackgerrit | Imre Farkas proposed openstack/ironic: Fix DRAC failure during automated cleaning https://review.openstack.org/367229 | 08:21 |
| *** mkoderer has joined #openstack-ironic | 08:21 | |
| *** electrofelix has joined #openstack-ironic | 08:22 | |
| *** lucas-dinner is now known as lucasagomes | 08:23 | |
| lucasagomes | morning all | 08:23 |
| tangchen | morning lucasagomes | 08:25 |
| *** amotoki has joined #openstack-ironic | 08:32 | |
| *** zigo_ is now known as zigo | 08:34 | |
| *** milan has joined #openstack-ironic | 08:40 | |
| tuanla_ | Hello ironic team, does anyone successfull using agent_ipmitool deploy whole-disk-image | 08:42 |
| milan | Good morning Ironic! The Sun is shinning here in Brno and we've got a lovely 21 centigrade, what's it like for you? ;) | 08:43 |
| *** jistr is now known as jistr|mtgs | 08:43 | |
| milan | morning tuanla_! | 08:43 |
| tangchen | milan: morning | 08:43 |
| milan | morning tangchen! | 08:43 |
| tuanla_ | morning milan, tangchen | 08:44 |
| tangchen | tuanla_: morning | 08:44 |
| *** amotoki has quit IRC | 08:44 | |
| tangchen | milan: I'd like to ask you for this favor again. https://review.openstack.org/#/c/349421/ | 08:45 |
| patchbot | patch 349421 - ironic - Prepare for functional tests | 08:45 |
| tangchen | milan: I have fixed all the comments. | 08:45 |
| milan | tangchen, lemme see | 08:45 |
| tangchen | milan: Please. Thanks | 08:45 |
| dtantsur | morning lucasagomes, milan | 08:49 |
| openstackgerrit | Zhou Ya proposed openstack/ironic-python-agent: Get pci device's numa_node info when collecting pci devices info This is the commit of https://blueprints.launchpad.net/ironic-python-agent/+spec/get-pci-device-numa-node bp https://review.openstack.org/369245 | 08:49 |
| milan | morning dtantsur lucasagomes :) | 08:49 |
| aarefiev | morning all! | 08:51 |
| dtantsur | hey aarefiev | 08:53 |
| *** dtantsur is now known as dtantsur|bbl | 08:53 | |
| milan | morning aarefiev! :) | 08:53 |
| openstackgerrit | Nguyen Hung Phuong proposed openstack/ironic-specs: Update document to use Newton, Mitaka as the release name https://review.openstack.org/368545 | 08:56 |
| *** anush__ has quit IRC | 09:04 | |
| *** amotoki has joined #openstack-ironic | 09:05 | |
| *** jaosorior is now known as jaosorior_lunch | 09:05 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Tempest: add auto-discovery test https://review.openstack.org/369261 | 09:11 |
| sambetts|afk | Morning all | 09:16 |
| *** sambetts|afk is now known as sambetts | 09:16 | |
| lucasagomes | sambetts, morning | 09:17 |
| tuanla_ | Hello sambetts, Did you try to deploy whole-disk-image using agent_ipmitool | 09:18 |
| sambetts | tuanla_: Our CI tests that | 09:19 |
| sambetts | The gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-nv job | 09:19 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Tempest: add auto-discovery test https://review.openstack.org/369261 | 09:20 |
| aarefiev | dtantsur: ugly, but works^ | 09:22 |
| aarefiev | morning sambetts | 09:22 |
| sambetts | o/ aarefiev | 09:22 |
| *** KrisK19 has joined #openstack-ironic | 09:23 | |
| openstackgerrit | Imre Farkas proposed openstack/ironic: Fix nits on DRAC OOB inspection patch https://review.openstack.org/366785 | 09:24 |
| *** fragatina has joined #openstack-ironic | 09:26 | |
| tuanla_ | yub sambetts. I will test whole-disk-image in physical machine using agent_ipmitool | 09:26 |
| openstackgerrit | Ukesh Kumar proposed openstack/python-ironicclient: Extend OSC "node list" cmd to fetch nodes without instance UUID https://review.openstack.org/369279 | 09:29 |
| *** fragatina has quit IRC | 09:30 | |
| *** tuanla_ has quit IRC | 09:31 | |
| milan | morning sambetts! :) | 09:31 |
| KrisK19 | Hi, we have Mitaka release of ironic implementation, did anyone observed issue with rebuild. First rebuild is working fine, second one is getting error (it seems like not correct target phisical server is pick, the one which have alredy other instance) | 09:32 |
| KrisK19 | if any operators are here please check rebuild twice of server | 09:32 |
| *** tuanla_ has joined #openstack-ironic | 09:33 | |
| *** derekh has joined #openstack-ironic | 09:34 | |
| *** tczekajlo has joined #openstack-ironic | 09:35 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Ironic conductor to honor the interval values for periodic tasks https://review.openstack.org/369000 | 09:36 |
| sambetts | lucasagomes: wasn't there a bug recently highlighted with rebuild? I remember a conversation between JayF and jroll about it | 09:36 |
| lucasagomes | sambetts, it was, but it was about the configuration driver | 09:37 |
| *** amotoki has quit IRC | 09:37 | |
| sambetts | oh :/ | 09:37 |
| lucasagomes | tl;dr the nova ironic driver should recreate the config drive at rebuild instead of having ironic to keep it in the node/swift | 09:37 |
| lucasagomes | not sure if that relates to KrisK19 problems, apparently it does not | 09:38 |
| sambetts | yup, I see that one on launchpad, I don't see any others relating to rebuild though | 09:38 |
| lucasagomes | KrisK19, btw can you paste the error ? | 09:38 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Ironic conductor to honor the interval values for periodic tasks https://review.openstack.org/369000 | 09:43 |
| KrisK19 | Sep 13 10:11:17 ironic nova-compute[127233]: 2016-09-13 10:11:17.019 127233 ERROR nova.compute.manager [req-ef3b5f20-614f-4a58-af32-412ec1de4fa6 add537e9cc554c57807a1dd5474f48e8 a21370dc14f34197979140a9d68599e6 - - -] [instance: 06378490-18f6-4c60-bc29-5e807f3379fe] Setting instance vm_state to ERROR | 09:44 |
| KrisK19 | . | 09:44 |
| KrisK19 | Sep 13 10:11:17 ironic nova-compute[127233]: 2016-09-13 10:11:17.019 127233 ERROR nova.compute.manager [instance: 06378490-18f6-4c60-bc29-5e807f3379fe] File "/usr/lib/python2.7/dist-packages/ironicclient/common/http.py", line 189, in wrapper | 09:44 |
| KrisK19 | Sep 13 10:11:17 ironic nova-compute[127233]: 2016-09-13 10:11:17.019 127233 ERROR nova.compute.manager [instance: 06378490-18f6-4c60-bc29-5e807f3379fe] return func(self, url, method, **kwargs) | 09:44 |
| KrisK19 | Sep 13 10:11:17 ironic nova-compute[127233]: 2016-09-13 10:11:17.019 127233 ERROR nova.compute.manager [instance: 06378490-18f6-4c60-bc29-5e807f3379fe] File "/usr/lib/python2.7/dist-packages/ironicclient/common/http.py", line 528, in _http_request | 09:44 |
| KrisK19 | Sep 13 10:11:17 ironic nova-compute[127233]: 2016-09-13 10:11:17.019 127233 ERROR nova.compute.manager [instance: 06378490-18f6-4c60-bc29-5e807f3379fe] error_json.get('debuginfo'), method, url) | 09:44 |
| KrisK19 | Sep 13 10:11:17 ironic nova-compute[127233]: 2016-09-13 10:11:17.019 127233 ERROR nova.compute.manager [instance: 06378490-18f6-4c60-bc29-5e807f3379fe] Conflict: Node 03781ac2-ca7a-483b-a954-8c7b97fcb1de is associated with instance a7432593-80ba-4099-bbfe-5b3966762092. (HTTP 409) | 09:44 |
| KrisK19 | Sep 13 10:11:17 ironic nova-compute[127233]: 2016-09-13 10:11:17.019 127233 ERROR nova.compute.manager [instance: 06378490-18f6-4c60-bc29-5e807f3379fe] | 09:44 |
| KrisK19 | more or less like that | 09:44 |
| patchbot | Error: I haven't seen or. | 09:44 |
| *** amotoki has joined #openstack-ironic | 09:46 | |
| KrisK19 | it work for us with Liberty (we still have one deployment) Mitaka seems like have that issue, if anyone could check at env with Mitaka how it works would be great | 09:48 |
| KrisK19 | at ironi-conductor.log: | 09:48 |
| KrisK19 | 2016-09-13 11:07:00.486 127870 ERROR oslo_messaging._drivers.common [req-e10e0692-c1ba-4754-959c-d95513165a20 - - - - -] ['Traceback (most recent call last):\n', ' File "/usr/lib/python2.7/dist-packages/oslo_messaging/rpc/dispatcher.py", line 138, in _dispatch_and_reply\n incoming.message))\n', ' File "/usr/lib/python2.7/dist-packages/oslo_messaging/rpc/dispatcher.py", line 185, in _dispatch\n return self._do_dispatch(endpoint, | 09:48 |
| patchbot | Error: Missing "]". You may want to quote your arguments with double quotes in order to prevent extra brackets from being evaluated as nested commands. | 09:48 |
| KrisK19 | method, ctxt, args)\n', ' File "/usr/lib/python2.7/dist-packages/oslo_messaging/rpc/dispatcher.py", line 127, in _do_dispatch\n result = func(ctxt, **new_args)\n', ' File "/usr/lib/python2.7/dist-packages/oslo_messaging/rpc/server.py", line 150, in inner\n return func(*args, **kwargs)\n', ' File "/usr/lib/python2.7/dist-packages/ironic/conductor/manager.py", line 228, in update_node\n node_obj.save()\n', ' File | 09:48 |
| KrisK19 | "/usr/lib/python2.7/dist-packages/ironic/objects/node.py", line 340, in save\n self.dbapi.update_node(self.uuid, updates)\n', ' File "/usr/lib/python2.7/dist-packages/ironic/db/sqlalchemy/api.py", line 388, in update_node\n return self._do_update_node(node_id, values)\n', ' File "/usr/lib/python2.7/dist-packages/ironic/db/sqlalchemy/api.py", line 413, in _do_update_node\n node=ref.uuid, instance=ref.instance_uuid)\n', | 09:48 |
| KrisK19 | 'NodeAssociated: Node 03781ac2-ca7a-483b-a954-8c7b97fcb1de is associated with instance a7432593-80ba-4099-bbfe-5b3966762092.\n'] | 09:48 |
| patchbot | Error: Spurious "]". You may want to quote your arguments with double quotes in order to prevent extra brackets from being evaluated as nested commands. | 09:48 |
| sambetts | KrisK19: please use http://paste.openstack.org/ instead of filling the IRC with logs | 09:48 |
| KrisK19 | sure | 09:48 |
| *** hoangcx has quit IRC | 09:52 | |
| lucasagomes | KrisK19, yeah please, as sambetts said :-) | 09:54 |
| vsaienk0 | sambetts: there is no secgroup support int networking-generic-switch yet, so it will just ignore applying secgrouprule | 09:55 |
| sambetts | vsaienk0: ah right, so thats how we're not broken by it | 09:56 |
| *** mgoddard_ has joined #openstack-ironic | 09:56 | |
| *** amotoki has quit IRC | 09:56 | |
| *** vishwana_ has joined #openstack-ironic | 09:56 | |
| vsaienk0 | Morning sambetts, lucasagomes, dtantsur, vdrok, aarefiev, milan and all Ironic'ers! | 09:56 |
| lucasagomes | vsaienk0, morning :D | 09:56 |
| milan | morning vsaienk0! :) | 09:56 |
| aarefiev | morning vsaienk0 | 09:56 |
| sambetts | vsaienk0: so as long as the ml2 driver devs don't apply secgrouprules for baremetal vnics then we're ok right now, and we need to document adding TFTP and http to the default sec groups for the provisioning network for those who do | 09:57 |
| *** vishwanathj_zzz has quit IRC | 09:58 | |
| vsaienk0 | sambetts: For me it is better to do not apply any security group by default even default group. | 09:59 |
| *** mgould|afk is now known as mgould | 09:59 | |
| mgould | morning everyone! | 09:59 |
| *** mgoddard has quit IRC | 10:00 | |
| vsaienk0 | sambetts: we should apply secgroup only when it is specified in config I think. | 10:00 |
| sambetts | vsaienk0: sure, but we either need to document disabling network security for the provisioning network, or editing it in neutron to allow for provisioning traffic, for cases when ML2 driver support it already | 10:00 |
| sambetts | vsaienk0: if you don't specify a sec gruop neutron will automatically put the default one the provisioning network, unless we make Ironic disable network security in newton | 10:02 |
| vsaienk0 | sambetts: so Neutron will apply default group event without https://review.openstack.org/#/c/361451/ when ML2 supports secgroups right? | 10:04 |
| patchbot | patch 361451 - ironic - Add support for Security Groups for baremetal servers | 10:04 |
| *** jaosorior_lunch is now known as jaosorior | 10:04 | |
| sambetts | vsaienk0: yes, which is a problem because the default sec groups are too restrictive to allow provisioning to succeed | 10:04 |
| vsaienk0 | sambetts: what if we change this behavior by https://review.openstack.org/#/c/361451/ to do not apply any security group by default? Because default security group will not work either | 10:06 |
| patchbot | patch 361451 - ironic - Add support for Security Groups for baremetal servers | 10:06 |
| *** amotoki has joined #openstack-ironic | 10:07 | |
| sambetts | vsaienk0: we decided in the meeting yesterday that patch will not make it in Newton | 10:07 |
| sambetts | vsaienk0: so we have to solve it without that patch for newton | 10:07 |
| vsaienk0 | sambetts: so the problem is that arista or might be other vendors who added security support are not working in Newton because of this? | 10:09 |
| sambetts | vsaienk0: yup | 10:09 |
| vsaienk0 | and there is no way in Ironic to disable/or change secgroup for provisioning | 10:09 |
| sambetts | exactly | 10:09 |
| vsaienk0 | sambetts: I see, it looks like a problem :) | 10:09 |
| vsaienk0 | sambetts: is it possible to solve it by ML2 config option? for example to do not apply secgrouprules for ironic-provisioning network? | 10:10 |
| *** mikal has quit IRC | 10:10 | |
| sambetts | vsaienk0: you can disable network security for the provisioning network yes | 10:11 |
| vsaienk0 | because I doubt that we can solve it in ironic, without secgoup patch | 10:11 |
| openstackgerrit | Szymon Borkowski proposed openstack/ironic-specs: WIP: Add VCA driver to Ironic https://review.openstack.org/369309 | 10:11 |
| sambetts | so we need to document either disabling network security for that network, or document change the default rules | 10:12 |
| *** mikal has joined #openstack-ironic | 10:12 | |
| vsaienk0 | sambetts: right, sounds reasonable | 10:14 |
| *** mikal has quit IRC | 10:14 | |
| *** mikal has joined #openstack-ironic | 10:16 | |
| *** permalac has joined #openstack-ironic | 10:17 | |
| *** thrash|g0ne is now known as thrash | 10:18 | |
| permalac | Hello guys. I'm having heavy problems with UEFI , I'm using RH Director 7 , which is the Kilo equivalent, and Uefi boot looks like it's not supported. | 10:19 |
| permalac | Is there any way to put a DVD with all the packages, and without the PXE start the installation without the bootstrap , or something alike? | 10:20 |
| lucasagomes | permalac, right, that sounds correct since I don't think UEFI was fully supported in kilo | 10:20 |
| lucasagomes | (that was the first official release of openstack Ironic btw) | 10:21 |
| permalac | I know the machines will work, I do not need ironic to do a bootstrap of high end machines, if I set the machines to leggacy boot they boot and everything goes nice, but the vendor does not provide support if UEFI is not enabled, (for RAS features and many nice things ) | 10:22 |
| permalac | so , I would like to deploy the compute node without using the pxe provided by ironic, but make the machines managed by ironic. | 10:22 |
| permalac | like providing the kickstart file to a DVD booted manually or something alike. | 10:22 |
| lucasagomes | permalac, right, so ironic does not install an OS like kickstart it does image based deployment | 10:23 |
| permalac | lucasagomes, we tried with liberty and pxe boot also fails with UEFI, and redhat still says that is not supported. | 10:23 |
| lucasagomes | one way to avoid PXE boot is to use virtual media, but I'm afraid few drivers support it even today | 10:23 |
| permalac | can I do the image based deployment by hand ? | 10:23 |
| lucasagomes | ilO IRMC | 10:23 |
| lucasagomes | at Kilo, probably only iLO | 10:23 |
| *** mkoderer has quit IRC | 10:25 | |
| lucasagomes | permalac, in a way yes, it's possible to use some driver like fake_ipmitool that will pxe the boot process (you could them create an ISO that will boot the deploy ramdisk and kernel) | 10:25 |
| lucasagomes | but that's not encouraged nor supported | 10:25 |
| lucasagomes | that will fake the pxe process* | 10:26 |
| permalac | lucasagomes, you mean, deploy on a virtual machine, and then dd the disk of the virtual machine to an image and that image to the physical machine. | 10:26 |
| lucasagomes | permalac, wait I was assuming you already had the image and just wanted to avoid PXE | 10:27 |
| mgould | permalac: "virtual media" in this context means "your BMC pretends to be a CD-ROM drive" | 10:27 |
| lucasagomes | permalac, so if you have the image already, you could avoid the pxe by using a driver such as fake_ipmitool and booting the deploy ramdisk from an ISO | 10:28 |
| permalac | lucasagomes, I only need to avoid the pxe , could you elaborate more on that way to avoid it ? | 10:29 |
| permalac | or just some keywords to look on google will do. | 10:30 |
| *** mgoddard_ has quit IRC | 10:30 | |
| *** mgoddard has joined #openstack-ironic | 10:30 | |
| permalac | any ramdisk will do or I need to get the one that ironic is using? | 10:30 |
| *** ifarkas has quit IRC | 10:33 | |
| *** ifarkas has joined #openstack-ironic | 10:33 | |
| KrisK19 | sambetts: as you wish logs attached: http://paste.openstack.org/show/573562/ http://paste.openstack.org/show/573563/ | 10:34 |
| lucasagomes | permalac, right, so, there is two types of ramdisk in Ironic. You have the ramdisk of the image you are deploying and you have a special ramdisk which we refer to as "deploy ramdisk" | 10:34 |
| lucasagomes | the deploy ramdisk contains code that helps ironic to deploy the image onto the hard driver of the target machine | 10:35 |
| lucasagomes | permalac, https://github.com/openstack/ironic-python-agent is what we use today | 10:36 |
| lucasagomes | what is tricky here is that back in Kilo we had two versions of the code for the deploy ramdisk | 10:36 |
| lucasagomes | one being that ironic-python-agent and the other this one: https://github.com/openstack/diskimage-builder/tree/master/elements/deploy-ironic | 10:37 |
| lucasagomes | since you are using OSP 7 you will be using https://github.com/openstack/diskimage-builder/tree/master/elements/deploy-ironic | 10:37 |
| lucasagomes | now I don't wanna extend much into the details because what you are doing is highly discouraged | 10:38 |
| permalac | :D I love that. | 10:38 |
| lucasagomes | heh | 10:38 |
| lucasagomes | so yeah, you can fake the PXE process by using a driver such as fake_ipmitool | 10:38 |
| lucasagomes | then in order to boot the deploy ramdisk w/o the PXE process you can create an ISO which contains such deploy ramdisk and kernel | 10:39 |
| lucasagomes | and boot it on the machine while the node is in "deploy wait" state | 10:39 |
| permalac | I understand why is discouraged, but we have to walk the path. Thanks for putting so much light on it. | 10:39 |
| lucasagomes | and Ironic should take it over from there and continue the deploy process as normal | 10:39 |
| mgould | lucasagomes: will permalac also have to rebuild the image-to-be-deployed with a UEFI partition? | 10:40 |
| lucasagomes | permalac, no worries, it's ok... It's just that it's a ball of wax :-) | 10:40 |
| lucasagomes | or can or worms, idk how you would call it :D | 10:40 |
| mgould | a ball of wax containing worms | 10:41 |
| lucasagomes | mgould, yeah, either the image could be a whole disk image containing all partitions | 10:41 |
| permalac | worms armed with nails. | 10:41 |
| permalac | :) | 10:41 |
| lucasagomes | or we need to check if setting the capabilities to boot_mode:uefi will actually create the ESP partition | 10:41 |
| lucasagomes | (I know there's some plumbing work in Kilo to support UEFI, but I don't know at which stage it is) | 10:42 |
| permalac | since the ramdisk that I need to use to build the ISO has some code that helps ironic to follow up with the deployment, from where I sould take the ramdisk ? Sould I take it when the node is in "deploy wait "state already? | 10:43 |
| lucasagomes | permalac, you can create the ramdisk using a tool called diskimage-builder | 10:43 |
| lucasagomes | https://github.com/openstack/diskimage-builder | 10:43 |
| lucasagomes | If I recall correctly, the command will be something like: ramdisk-image-create -o <name> deploy-ironic fedora | 10:44 |
| *** dtantsur|bbl is now known as dtantsur | 10:44 | |
| lucasagomes | to create a fedora based ramdisk with that code | 10:44 |
| dtantsur | morning sambetts, vsaienk0 | 10:45 |
| mgould | morning dtantsur | 10:46 |
| dtantsur | hey mgould | 10:46 |
| dtantsur | aarefiev, you have an interesting definition of "works" ;) http://logs.openstack.org/61/369261/2/check/gate-tempest-dsvm-ironic-inspector-nv/e12e5ad/console.html#_2016-09-13_09_51_26_229987 | 10:47 |
| permalac | lucasagomes, thanks for all this guidelines. I will look at it, test it and hopelly succed. ;) | 10:47 |
| lucasagomes | permalac, no problem | 10:47 |
| dtantsur | aarefiev, left a quick comment, still reviewing | 10:48 |
| aarefiev | dtantsur: https://review.openstack.org/#/c/369304/ | 10:49 |
| patchbot | patch 369304 - openstack-infra/project-config - Enable inspector-auto-discovery on tempest discove... | 10:49 |
| aarefiev | yeah, I'll fix basic one, didn't test it | 10:50 |
| lucasagomes | permalac, out of curiosity, what's the problem with PXE itself and UEFI ? | 10:50 |
| *** permalac has quit IRC | 10:51 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic-inspector: Add translation marker to deprecated_reason of config opts https://review.openstack.org/369333 | 10:53 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic-inspector: Add translation marker to deprecated_reason of config opts https://review.openstack.org/369333 | 10:53 |
| dtantsur | aarefiev, thanks! finished reviewing, left some more comments | 10:55 |
| aarefiev | dtantsur: about introspection_start, how do you propose to start node w/o it? | 10:57 |
| *** ifarkas has quit IRC | 10:59 | |
| *** wznoinsk has quit IRC | 11:05 | |
| *** dprince has quit IRC | 11:09 | |
| dtantsur | aarefiev, node.set_power_state | 11:15 |
| aarefiev | dtantsur: but it's blacklisted | 11:16 |
| dtantsur | oh, correct. hmmm | 11:17 |
| *** wznoinsk has joined #openstack-ironic | 11:18 | |
| dtantsur | aarefiev, why do you need start_vm_by_node then btw? | 11:18 |
| aarefiev | dtantsur: I'll clean up, it's not used in current approach | 11:18 |
| dtantsur | aarefiev, well, ok, leave start_introspection, but please put a comment why we need it there | 11:19 |
| aarefiev | dtantsur: there is one, line 127 | 11:20 |
| aarefiev | I can elaborate more on this | 11:20 |
| dtantsur | ugh, maybe I'm too sleepy, lemme check | 11:20 |
| aarefiev | no, it just moved on top ( | 11:21 |
| dtantsur | aarefiev, yeah, specify why we use start_introspection instead of set_power_state. and clean up the virsh bits to avoid confusion | 11:21 |
| aarefiev | I'll just remove not used methods for now | 11:22 |
| openstackgerrit | Merged openstack/ironic-inspector: Add translation marker to help of config opts https://review.openstack.org/365888 | 11:22 |
| dtantsur | thanks | 11:22 |
| aarefiev | thank you sir | 11:22 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Fix breakage with oslo.config master https://review.openstack.org/367407 | 11:24 |
| *** ccamacho is now known as ccamacho|lunch | 11:25 | |
| *** clenimar_ has quit IRC | 11:27 | |
| jroll | morning everyone | 11:27 |
| dtantsur | morning jroll | 11:27 |
| jroll | sambetts | so we need to document either disabling network security for that network, or document change the default rules <- or maybe both? :) | 11:28 |
| jroll | who signed up for that btw? | 11:28 |
| jroll | hey dtantsur :) | 11:28 |
| lucasagomes | jroll, morning | 11:29 |
| jroll | morning lucasagomes | 11:29 |
| *** lucasagomes is now known as lucas-hungry | 11:30 | |
| sambetts | jroll: yeah, I think both would be good, I'm not sure who signed up to do it | 11:30 |
| jroll | ok | 11:30 |
| *** baoli has joined #openstack-ironic | 11:31 | |
| *** baoli has quit IRC | 11:31 | |
| *** baoli has joined #openstack-ironic | 11:31 | |
| *** joprovost has joined #openstack-ironic | 11:33 | |
| * jroll does it | 11:33 | |
| sambetts | thanks jroll :D | 11:33 |
| *** baoli_ has joined #openstack-ironic | 11:34 | |
| *** tczekajlo has quit IRC | 11:35 | |
| *** bfournie has quit IRC | 11:36 | |
| jroll | do we allow configuring the iscsi port? | 11:36 |
| sambetts | jroll: yup | 11:37 |
| *** baoli has quit IRC | 11:37 | |
| *** trown|outtypewww is now known as trown | 11:40 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add CRUD notifications for ironic resources https://review.openstack.org/356541 | 11:40 |
| *** ElCoyote_ has quit IRC | 11:41 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add a note about security groups in install guide https://review.openstack.org/369385 | 11:42 |
| jroll | there we go | 11:42 |
| sambetts | jroll: awesome :) first glance it looks good | 11:42 |
| jroll | cool, thanks :) | 11:43 |
| jroll | I'm not 100% about the rendering yet | 11:43 |
| sambetts | jroll: wow there are a ton of warnings thrown by tox -e docs | 11:46 |
| *** dprince has joined #openstack-ironic | 11:47 | |
| milan | vdrok, could you please check my comment on tangchen 's patch so I don't recommend something you wouldn't agree to later: https://review.openstack.org/#/c/349421/16 https://review.openstack.org/#/c/349421/14 Thanks a lot! | 11:47 |
| patchbot | patch 349421 - ironic - Prepare for functional tests | 11:47 |
| patchbot | patch 349421 - ironic - Prepare for functional tests | 11:47 |
| *** amotoki has quit IRC | 11:49 | |
| jroll | sambetts: yeah, warnerrors is broken in current PBR and just doesn't work :( | 11:49 |
| jroll | we keep fixing them and then more pop up | 11:49 |
| sambetts | jroll: rendering doesn't look too bad to me, the only thing that bugs me a little is the lack of space between the text and the list, and the styling of the list, but those are down to the theme right? | 11:50 |
| sambetts | jroll: the toctree warnings? | 11:50 |
| jroll | sambetts: yes, the toctree errors | 11:50 |
| jroll | not sure about the space/style, probably the theme? | 11:50 |
| * jroll too lazy to render locally right now, will wait for zuul | 11:50 | |
| sambetts | :-P | 11:51 |
| dtantsur | jroll, should we add "deprecate unsupported drivers" to "must have" on https://trello.com/b/ROTxmGIc/ironic-newton-priorities? | 11:53 |
| jroll | dtantsur: it's part of the third party CI card, I haven't pushed code yet so left it in todo, but if you prefer it in must have we can do that | 11:54 |
| dtantsur | jroll, if we don't deprecate them now, we can't remove them in ocata, I guess... | 11:54 |
| * jroll moves it | 11:54 | |
| jroll | yeah | 11:54 |
| jroll | I've just been stuck in internal meetings for the last week :/ | 11:55 |
| jroll | going to do it this week, for sure | 11:55 |
| vdrok | Hey milan, I'm on pto but will try to tonight | 11:56 |
| milan | hey vdrok, didn't know, sorry to bother | 11:57 |
| *** jpich has quit IRC | 11:57 | |
| jroll | dtantsur: I approved multinode CI RFE, btw https://bugs.launchpad.net/ironic/+bug/1622653 | 11:57 |
| openstack | Launchpad bug 1622653 in Ironic "[RFE] Setup multinode job" [Undecided,New] | 11:57 |
| * jroll not doing it in secret, just seems obvious | 11:57 | |
| dtantsur | ack. please also Confirmed/Wishlist ;) | 11:59 |
| jroll | ah right, sorry :) | 11:59 |
| jroll | done | 11:59 |
| dtantsur | lucas-hungry, we need a release notes job on ironic-staging-drivers.. | 12:03 |
| dtantsur | jroll, so, wdyt about https://bugs.launchpad.net/ironic/+bug/1621530 ? should we try to get it in? | 12:04 |
| openstack | Launchpad bug 1621530 in Ironic "[RFE] Inband inspection for the OneView drivers" [Wishlist,In progress] - Assigned to Xavier (marcusrafael) | 12:04 |
| jroll | dtantsur: it's probably fine, other than time constraints :) | 12:05 |
| dtantsur | it lacks rfe-approved, hence I don't review the patch | 12:05 |
| jroll | in other words, I don't think we need to block it, but I'm not going to ask people to review it | 12:05 |
| jroll | ah, should we approve it? | 12:05 |
| jroll | seems fine | 12:05 |
| *** wznoinsk has quit IRC | 12:06 | |
| jroll | I'll approve it if you're +1 on it | 12:06 |
| dtantsur | yeah, the RFE looks fine | 12:07 |
| jroll | ok, done | 12:07 |
| *** bfournie has joined #openstack-ironic | 12:11 | |
| *** tuanla_ has quit IRC | 12:11 | |
| *** wznoinsk has joined #openstack-ironic | 12:13 | |
| *** leitan has joined #openstack-ironic | 12:24 | |
| openstackgerrit | Cao ShuFeng proposed openstack/ironic: [WIP] sanboot featuer of agent_ssh driver https://review.openstack.org/369411 | 12:24 |
| *** sborkows has joined #openstack-ironic | 12:28 | |
| openstackgerrit | Cao ShuFeng proposed openstack/ironic: [WIP] sanboot featuer of agent_ssh driver https://review.openstack.org/369411 | 12:28 |
| *** joprovost has quit IRC | 12:29 | |
| *** leitan has quit IRC | 12:30 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent: DO NOT MERGE: testing gate https://review.openstack.org/369420 | 12:31 |
| *** permalac has joined #openstack-ironic | 12:33 | |
| *** fxpester has quit IRC | 12:33 | |
| *** lucas-hungry is now known as lucasagomes | 12:34 | |
| lucasagomes | dtantsur, oh ++ do we have reno support already ? | 12:34 |
| dtantsur | lucasagomes, we have some release notes there, so dunno :) | 12:34 |
| lucasagomes | I will check :D | 12:34 |
| lucasagomes | dtantsur, we do have reno support, I will take a look at the job | 12:38 |
| *** ccamacho|lunch is now known as ccamacho | 12:40 | |
| *** jpich has joined #openstack-ironic | 12:44 | |
| openstackgerrit | Merged openstack/ironic: Fix nits on DRAC OOB inspection patch https://review.openstack.org/366785 | 12:46 |
| *** Goneri has joined #openstack-ironic | 12:46 | |
| *** trown is now known as trown|brb | 12:47 | |
| *** mjturek has quit IRC | 12:50 | |
| lucasagomes | jroll, dtantsur I'm trying to find here, you if release notes is only for official projects ? | 12:53 |
| dtantsur | worth asking on #infra imo | 12:53 |
| milan | aarefiev, w/r inspector discovery tempest test, which patch is preferred: https://review.openstack.org/#/c/323769/ or https://review.openstack.org/#/c/369261/? | 12:53 |
| patchbot | patch 323769 - ironic-inspector - WIP: Discovery tempest test | 12:53 |
| patchbot | patch 369261 - ironic-inspector - Tempest: add auto-discovery test | 12:53 |
| *** wajdi has quit IRC | 12:53 | |
| permalac | lucasagomes, sorry, I got dropped out for a while. | 12:53 |
| lucasagomes | permalac, no worries, I just asked what is the problem with PXE in UEFI mode ? | 12:54 |
| permalac | lucasagomes, the problem with UEFI on this machines (Bullion S4 ) is unknown, we are debugging it with the vendor. | 12:54 |
| permalac | looks like the pxe is too slow or something alike. | 12:55 |
| aarefiev | milan: I've forgot about that one, thanks | 12:55 |
| milan | aarefiev, np, so I guess the later is what needs more attention ;) | 12:55 |
| *** ElCoyote_ has joined #openstack-ironic | 12:56 | |
| aarefiev | yeah, abandoned first one | 12:56 |
| milan | ah, just saw it | 12:56 |
| lucasagomes | permalac, hmm | 12:56 |
| *** mmedvede has quit IRC | 12:56 | |
| *** jaosorior has quit IRC | 12:59 | |
| *** jaosorior has joined #openstack-ironic | 13:00 | |
| *** trown|brb is now known as trown | 13:01 | |
| lucasagomes | dtantsur, yeah it's only for official projects. But it can be done in a way wich the release notes appear in the docs | 13:01 |
| dtantsur | lucasagomes++ | 13:02 |
| *** rloo has joined #openstack-ironic | 13:02 | |
| lucasagomes | need a bit more work, I will open a bug with the links I got and fix it soon | 13:02 |
| sambetts | lucasagomes: both docs.openstack.org and releases and release notes are only for offical projects :( | 13:03 |
| lucasagomes | sambetts, yeah | 13:03 |
| lucasagomes | I use rtfd btw | 13:04 |
| dtantsur | ++ | 13:04 |
| *** sdake_ has joined #openstack-ironic | 13:04 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: WIP iPXE config endpoint https://review.openstack.org/369438 | 13:06 |
| *** eset has quit IRC | 13:08 | |
| *** sdake_ is now known as sdake | 13:10 | |
| rloo | hi and morning everyone, lucasagomes, sambetts, dtantsur | 13:11 |
| dtantsur | morning rloo | 13:11 |
| lucasagomes | rloo, hi there :D | 13:12 |
| mat128 | o/ | 13:12 |
| lucasagomes | rloo, btw, thanks for the comment in the periodic task thing. I've changed the approach there | 13:12 |
| lucasagomes | mat128, hi there | 13:12 |
| rloo | dtantsur: wrt https://review.openstack.org/#/c/352745/, I forgot/didn't think of 3rd party ci. we just want CI working, and/or do we want their CI testing/exercising the patch? | 13:12 |
| patchbot | patch 352745 - ironic - Timing metrics: iLO drivers | 13:12 |
| milan | morning rloo mat128! :) | 13:12 |
| rloo | thx lucasagomes; on my list to do in the next few minutes | 13:13 |
| rloo | hi milan, mat128 | 13:13 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Abort cleaning and deployment if node is in maintenance https://review.openstack.org/366828 | 13:14 |
| lucasagomes | rloo, sure no rush :D | 13:15 |
| *** lindycoder has joined #openstack-ironic | 13:15 | |
| *** mmedvede has joined #openstack-ironic | 13:16 | |
| *** jheroux has joined #openstack-ironic | 13:18 | |
| dtantsur | rloo, at this stage I'd prefer at least one job to pass on every related patch | 13:18 |
| dtantsur | but I'm not sure how well the iLO CI feels now | 13:18 |
| rloo | dtantsur: we should come up with a general policy or I'm going to forget. | 13:19 |
| openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Fix multiple console object behavior https://review.openstack.org/369026 | 13:19 |
| dtantsur | rloo, we can ask jroll to give us such policy ;) | 13:20 |
| rloo | dtantsur: yeah, make the ptl do some work :) jroll ^^ for patches to vendor drivers, do we wait for their CI to work before +A'ing? | 13:20 |
| mgould | rloo: I kinda feel that CI which doesn't exercise the code under test is a waste of cycles | 13:23 |
| *** joprovost has joined #openstack-ironic | 13:24 | |
| rloo | mgould: i agree with you. that's the case now with our existing CI I suspect. (that we probably don't exercise all our code) eg, do we test the metrics? | 13:25 |
| mgould | good question | 13:25 |
| *** amotoki has joined #openstack-ironic | 13:26 | |
| mgould | but I really meant the contrapositive: tests which can't possibly be affected by the patch being tested are a waste of time at best and a source of false alarms at worst | 13:26 |
| mgould | it's very hard to identify such tests given the way OpenStack works, of course :-( | 13:26 |
| mgould | but yes, <100% coverage is a Bad Thing | 13:27 |
| rloo | mgould: i'm not a tester but ... how do you know that a test can't possibly be affected by the patch. most of us could probably say "it won't", but ah, when tests fail... :) | 13:28 |
| mgould | [these two statements only *appear* to be contradictory :-)] | 13:28 |
| mgould | rloo: therein lies the problem | 13:28 |
| mgould | for a unified codebase, you can look at line coverage | 13:28 |
| *** amotoki has quit IRC | 13:28 | |
| rloo | mgould: so i'd rather focus on writing really good code, cuz that seems an easier problem than 100% test coverage :) | 13:29 |
| mgould | [I believe Google do this for their internal test sytem] | 13:29 |
| mgould | if test X doesn't exercise any lines affected by patch P, don't run it when testing P | 13:29 |
| lucasagomes | can someone take a quick look at https://review.openstack.org/#/c/367306/ ? It has 2+2 already | 13:29 |
| patchbot | patch 367306 - ironic - Grub2 by default for PXE + UEFI | 13:29 |
| rloo | lucasagomes: looking, but nothing is quick if it touches the install guide... | 13:31 |
| mgould | rloo: the hypothesis fuzz-testing library has 100% *branch* coverage: http://hypothesis.readthedocs.io/en/latest/supported.html | 13:32 |
| mgould | but it's pretty small :-) | 13:32 |
| *** cdearborn has joined #openstack-ironic | 13:32 | |
| jroll | rloo: dtantsur: that seems like a nice policy, and a nice way to get vendors to make their CI reliable. if they want to go around that policy, they can set supported=False in their driver :) | 13:32 |
| lucasagomes | rloo, hah good point :P | 13:33 |
| rloo | jroll: is that something we should bring up at our meeting/vote/whatever? | 13:33 |
| lucasagomes | luckily I'm mostly moving things around, but yeah there's always window for improvements in the docs apparently | 13:33 |
| jroll | rloo: probably just ML is fine | 13:33 |
| rloo | mgould: would be interesting to see how that works with ironic | 13:34 |
| rloo | jroll: will you be the bearer of the bad news? | 13:34 |
| mgould | yeah | 13:34 |
| jroll | rloo: I can do that | 13:34 |
| rloo | jroll: thx! | 13:34 |
| jroll | rloo: or should I just propose a docs patch and email about it, and people can discuss there? | 13:35 |
| * jroll wonders what neutron/cinder do, goes off to talk to them | 13:35 | |
| jroll | I guess neutron is all different repos | 13:35 |
| rloo | jroll: one thing i was wondering. we have a spec/process for 3rd party CI. It didn't mention this and I'd have to review the spec but maybe we shouldn't be modifying it before newton ends, for something to take effect "now". Is that fair to them? (Just wondering) | 13:36 |
| rloo | jroll: or did we already mention that we wouldn't +A their stuff w/o CI? | 13:37 |
| mgould | rloo: my guess would be "the interactions of shell scripts/installers/config files/services/etc is complex enough that we couldn't make reverse-dependency detection work reliably for functional tests, and our unit tests run fast enough that it's not worth the bother" | 13:37 |
| rloo | mgould: sigh. yup, I suspect you're right. Might make it interesting for some intern to investigate/unravel | 13:38 |
| dtantsur | rloo, fwiw I care less about policy, more about not breaking the driver at this stage of the cycle by a non-critical patch | 13:39 |
| mgould | heh, that would be a baptism by fire | 13:39 |
| jroll | rloo: I don't recall if we told people that | 13:39 |
| rloo | dtantsur: that's a metric patch, if metrics isn't turned on, it won't break anything. | 13:39 |
| dtantsur | well, unless there is a typo somewhere.. or decorator is applied in an incorrect way.. | 13:40 |
| rloo | jroll: maybe we should just focus on what happens to vendor drivers *after* newton then. | 13:40 |
| jroll | rloo: yeah, maybe | 13:40 |
| jroll | I do agree we should focus on newton | 13:40 |
| rloo | dtantsur: OH. Can we just ask them if they tested it? :) | 13:40 |
| rloo | dtantsur: I mean, who has tested the existing metrics stuff? | 13:41 |
| dtantsur | I didn't, but at least our CI made sure we haven't broken everything | 13:41 |
| rloo | jroll: yeah, 2 weeks or so left :) | 13:41 |
| jroll | the existing stuff gets the code path run, other than the statsd bits | 13:41 |
| rloo | dtantsur: true. i thought i saw other metric patches to drivers, being approved but maybe their CIs were working | 13:42 |
| rloo | dtantsur: ok, lets see what ilo sez. i'll keep it in mind when reviewing other driver patches. which i rarely do, I knew i shouldn't have looked at that one ;) | 13:42 |
| jroll | let's handle "require CI to approve" case by case right now | 13:42 |
| rloo | jroll: ++ | 13:43 |
| jroll | it's pretty easy to test this though, just make sure it loads properly | 13:43 |
| jroll | that should make sure there's no typos and such | 13:43 |
| rloo | jroll: but who's going to test? not me. gotta go to the next patch to review :) | 13:44 |
| jroll | totally fair :) | 13:44 |
| jroll | just pointing it out | 13:44 |
| rloo | jroll: true. so i'd be fine if they said they tested it. | 13:45 |
| openstackgerrit | Michael Turek proposed openstack/python-ironicclient: Add --wait to OSC provisioning commands https://review.openstack.org/354806 | 13:45 |
| rloo | lucasagomes: wrt https://review.openstack.org/#/c/367306/, why do we want to document using elilo if it isn't being actively supported/developed? | 13:52 |
| patchbot | patch 367306 - ironic - Grub2 by default for PXE + UEFI | 13:52 |
| strigazi | Hi, I just created a devstack environment from master with pxe_ssh and when I create an instance, ir-cond gives Error in deploy of node 08066702-5163-443b-8dc5-cfb1175b21a0: Root partition is too small for requested image. Image virtual size: 24 MB, Root size: 0 MB | 13:53 |
| lucasagomes | rloo, just to be safe, we can't just remove the ilo templates because someone might be using it | 13:53 |
| lucasagomes | rloo, so I thought I would still document it while we have it in tree | 13:53 |
| rloo | lucasagomes: oh. ok. going to suggest some changes in wording then. | 13:53 |
| lucasagomes | but that said, maybe I should add some deprecation warning for people using elilo | 13:53 |
| lucasagomes | the problem is to figure out whether it's being use or not, we only can do some string matching to see if the work "elilo" is present, but that's not great | 13:54 |
| lucasagomes | rloo, ack thanks | 13:54 |
| *** sdake has quit IRC | 13:58 | |
| *** wznoinsk has quit IRC | 13:59 | |
| *** amotoki has joined #openstack-ironic | 13:59 | |
| *** anush__ has joined #openstack-ironic | 14:00 | |
| *** rloo has quit IRC | 14:01 | |
| *** joprovost1 has joined #openstack-ironic | 14:02 | |
| strigazi | Hi, I just created a devstack environment from master with pxe_ssh and when I create an instance, ir-cond gives Error in deploy of node 08066702-5163-443b-8dc5-cfb1175b21a0: Root partition is too small for requested image. Image virtual size: 24 MB, Root size: 0 MB. | 14:02 |
| *** rloo has joined #openstack-ironic | 14:02 | |
| *** joprovost has quit IRC | 14:03 | |
| *** joprovost1 is now known as joprovost | 14:03 | |
| lucasagomes | strigazi, check the properties/local_gb of the node | 14:05 |
| lucasagomes | what value do you have there ? | 14:05 |
| lucasagomes | (ironic node-show <uuid/name>) | 14:05 |
| strigazi | ok | 14:05 |
| *** hoangcx has joined #openstack-ironic | 14:07 | |
| *** wznoinsk has joined #openstack-ironic | 14:10 | |
| *** sdake has joined #openstack-ironic | 14:12 | |
| rloo | lucasagomes: wrt https://review.openstack.org/#/c/369000/, do you know the chain of imports from ironic.api.app -> ironic.conductor.manager? we should fix that if possible | 14:12 |
| patchbot | patch 369000 - ironic - Ironic conductor to honor the interval values for ... | 14:12 |
| lucasagomes | rloo, I can follow that and get the exact path (I think I've it in some notes, will check) | 14:13 |
| lucasagomes | rloo, but in any case I don't think the imports are wrong to be honest | 14:14 |
| rloo | lucasagomes: well, at first glance, why should ironic.api.app need something that needs something from conductor.manager? | 14:14 |
| lucasagomes | IMO the code should be resilient to such failures | 14:14 |
| rloo | lucasagomes: that's a good point. | 14:15 |
| rloo | lucasagomes: but if code is poorly written/organized, it means adding more stuff to make it resilient. | 14:16 |
| lucasagomes | rloo, so when i was looking, apparently objects/base.py in some point imports rpcapi | 14:16 |
| lucasagomes | which imports conductor/manager.py | 14:16 |
| lucasagomes | see L97 from objects/base.py | 14:16 |
| rloo | lucasagomes: with a FIXME in L96 | 14:17 |
| lucasagomes | rloo, yeah, because that's an odd from the python decorators, it's evaluated at import time and not at instantiation time (in case the method decorated belongs to a class) | 14:17 |
| rloo | lucasagomes: so it seems like we have two import issues now, instead of the one from objects/base.py | 14:18 |
| lucasagomes | so we know it before hand, that's why I think that in general passing things like "spacing=CONF.blah" in a decorator is bad form | 14:18 |
| lucasagomes | rloo, yeah | 14:18 |
| jroll | lucasagomes: I feel like I've seen a different approach in nova, which is run all of them every minute, but then check last run time inside the task or something | 14:18 |
| *** wajdi has joined #openstack-ironic | 14:19 | |
| lucasagomes | jroll, hmm I gotta look at nova, I'm not sure about the internals and also nova uses the openstack periodic tasks instead of futurist | 14:20 |
| *** moshele has quit IRC | 14:20 | |
| lucasagomes | AFAIK | 14:20 |
| * lucasagomes looks | 14:20 | |
| *** dtantsur is now known as dtantsur|mtg | 14:20 | |
| jroll | lucasagomes: yeah, it might also be only for certain tasks | 14:20 |
| strigazi | lucasagomes: When you answered me I was already stacking again, now it works | 14:20 |
| lucasagomes | strigazi, cool :-) | 14:21 |
| strigazi | lucasagomes: btw this time I set IRONIC_VM_SPECS_RAM=2048 | 14:21 |
| lucasagomes | jroll, so what the last verison of my code does is just move that import which is causing manager.py to be imported earlier to inside the scope that it's gonna be used | 14:21 |
| strigazi | before was 1024 | 14:21 |
| lucasagomes | jroll, and add an assert in the code as a safeguard to prevent we from introducing a similar error in the future | 14:22 |
| lucasagomes | maybe ordering the imports as rloo suggests is another way, but will probably require moving code around to different modules | 14:22 |
| sambetts | the VM specs RAM requirement is differnt depending on the RAMDISK used, if you are using coreos one you need more RAM, if you're using tinyIPA you can use less | 14:23 |
| lucasagomes | which can/need to be done anyway, but idk how easy/big work that would be | 14:23 |
| rloo | lucasagomes, jroll: I'm fine with the change as a 'temporary' thing. but it bothers me because it seems to me that it might be due to the import in objects/base. and if that's the case, we're not addressing the real issue. | 14:24 |
| jroll | rloo: that assert verifies the fix, no? | 14:25 |
| rloo | jroll: I suspect lucasagomes' patch works/fixes the current problem. | 14:25 |
| lucasagomes | rloo, ++ two things I think we need to do here: 1. fix these import issues 2. stop passing "spacing=CONF..." in decorators, make it more resilent by not relying on import time evaluation things | 14:25 |
| rloo | jroll: but I'm not convinced that is the right way to fix it | 14:26 |
| jroll | rloo: right, so you're worried about the future | 14:26 |
| jroll | right | 14:26 |
| rloo | jroll: right, i'm always worried about the future :) | 14:26 |
| jroll | rloo: have you seen lucas' oslo.concurrency rfe/patch/idea? | 14:26 |
| jroll | dunno what stage it's at | 14:26 |
| rloo | jroll: eg, if we had fixed the first FIXME in objects/base, this current problem might not have been a problem | 14:26 |
| lucasagomes | jroll, looking at implmenting it in futurist now | 14:26 |
| jroll | er right, futurist | 14:27 |
| rloo | jroll: nope, didn't know about lucas' idea in oslo.concurrency. Oh, yeah, the futurist one. | 14:27 |
| *** rama_y has joined #openstack-ironic | 14:27 | |
| rloo | jroll: that too, only addresses the issue with the CONF blah blah. it still doesn't address the import issue. | 14:27 |
| lucasagomes | rloo, btw testing that patch is easy, add a bogus value to the some interval value (like sync_power_state_interval=-99) and try to start the conductor | 14:27 |
| lucasagomes | it should start fine w/o the patch, and fail with that patch | 14:27 |
| rloo | lucasagomes: I tested your first version and that worked too. | 14:27 |
| lucasagomes | ah cool | 14:28 |
| rloo | lucasagomes: i'm not questioning (yet) that your patch fixes the issue; I suspect it will. | 14:28 |
| jroll | rloo: what's the import issue? that the api loads conductor.manager? | 14:28 |
| rloo | lucasagomes: I'm just questioning whether that is the right approach | 14:28 |
| lucasagomes | rloo, gotcha | 14:28 |
| rloo | jroll: yes, the chain of imports from ironic.api.app ends up importing ironic.conductor.manager, and it seems to be objects.base that is doing that | 14:29 |
| rloo | jroll: and objects.base already has a FIXME to indicate that it shouldn't be doing that. | 14:30 |
| jroll | rloo: why is that an issue? | 14:30 |
| jroll | well | 14:30 |
| sambetts | tbh there is a problem using CONF.XXXXXX at any point during module import time, instead of at runtime, because you can't guarentee the config file is loaded yet | 14:30 |
| rloo | sambetts: yes, that is another issue | 14:30 |
| jroll | rloo: the fixme is about doing it in __init__ instead of the top, not about not importing it | 14:30 |
| rloo | jroll: but are we fixing *this* issue cuz it cropped up cuz we didn't fix the first issue in https://github.com/openstack/ironic/blob/master/ironic/objects/base.py#L97 | 14:31 |
| sambetts | ++ we should fix the cause of the cyclical import | 14:32 |
| jroll | rloo: objects.base would be importing that in any case | 14:32 |
| * jroll is confused | 14:32 | |
| * lucasagomes is in a meeting with dtantsur|mtg but is paying attention here too | 14:33 | |
| jroll | sambetts: side question, both UCS and CIMC have CI running, right? | 14:33 |
| sambetts | jroll: yup, they are separate jobs under the Cisco Ironic CI | 14:34 |
| jroll | thought so, thanks | 14:34 |
| rloo | jroll: my point is that i think someone ought to spend some time unravelling this and understanding whether/why we have that cyclical import error and if it can be fixed or not. cuz that import may have caused this current issue and could potentially cause some other issue in the future. and we're addressing this current issue which is fine, but not looking at the root cause perhaps. | 14:34 |
| jroll | rloo: okay, I see your point now | 14:37 |
| jroll | still think it's fine for now | 14:37 |
| jroll | fyi, db9ddd39d3f3d854d38e4b05655ebddd98f1a727 was the commit that introduced this | 14:37 |
| sambetts | so looking at the cyclical import I hope it should be pretty straight forward to fix, I'll see what I can do | 14:38 |
| jroll | added ironic.objects.base import to conductor.manager | 14:38 |
| *** mkam has joined #openstack-ironic | 14:38 | |
| jroll | sambetts: yeah, I think just move IronicObjectIndirectionAPI to another file yeah? | 14:38 |
| *** syed_ has joined #openstack-ironic | 14:38 | |
| sambetts | jroll: yeah thats what I'm thinking | 14:39 |
| jroll | (mebbe, dunno) | 14:39 |
| sambetts | jroll: just trying it now | 14:39 |
| rloo | jroll, sambetts: thx. if it proves to be not so simple, then we can go with lucas' solution and open a bug to address that later. | 14:41 |
| *** joprovost1 has joined #openstack-ironic | 14:42 | |
| *** joprovost has quit IRC | 14:43 | |
| *** joprovost1 is now known as joprovost | 14:43 | |
| *** amotoki has quit IRC | 14:44 | |
| *** amotoki has joined #openstack-ironic | 14:44 | |
| jroll | agree | 14:45 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Mark untested drivers as unsupported https://review.openstack.org/369503 | 14:45 |
| jroll | here's the fun patch ^^ | 14:45 |
| * rloo wonders why jroll seems to have so much fun | 14:46 | |
| jroll | rloo: gotta make work fun or else I'd be miserable | 14:46 |
| * jroll loves deleting code, this is a dependency to do so | 14:46 | |
| rloo | jroll: ha ha. so for that patch, how did you get the list of those drivers? (so I can verify you got the right ones) | 14:46 |
| *** mkam has quit IRC | 14:47 | |
| jroll | rloo: I scrolled through each file in ironic/drivers/ :P | 14:47 |
| jroll | rloo: or do you mean check the CI status? | 14:47 |
| *** mkam has joined #openstack-ironic | 14:47 | |
| rloo | jroll: and compared them against ..? CI status? | 14:47 |
| jroll | there's this, which is helpful http://ci-watch.tintri.com/project?project=ironic&time=7+days | 14:47 |
| jroll | but it's mostly in my head by now | 14:47 |
| *** mgoddard_ has joined #openstack-ironic | 14:47 | |
| jroll | for example, oneview and iRMC CI are both currently in maintenance | 14:48 |
| jroll | iirc | 14:48 |
| rloo | jroll: ok. is that something that kurt should also review? it needs to be apparent/obvious for those 3rd party driver/owners | 14:48 |
| jroll | no CIs look especially happy, but I know who is trying :( | 14:48 |
| jroll | rloo: maybe? | 14:49 |
| jroll | and I'll email the list so folks know what's being changed here | 14:49 |
| rloo | jroll: ok. i think the commit msg should indicate how you came up with them. if folks ask (in the future) :) | 14:50 |
| jroll | rloo: ehhh, it's really mostly in my head though | 14:51 |
| *** mgoddard has quit IRC | 14:51 | |
| rloo | jroll: that doesn't seem right. i mean it is ok if it is in your head, but it needs to be documented somewhere that people can see (besides this patch) :) | 14:51 |
| jroll | rloo: there's no definitive way to see for sure, is the problem | 14:52 |
| jroll | ci-watch is useful | 14:52 |
| *** amotoki has quit IRC | 14:52 | |
| jroll | but only covers 7 days | 14:52 |
| *** eset has joined #openstack-ironic | 14:52 | |
| *** eset has joined #openstack-ironic | 14:52 | |
| jroll | also need to keep an eye on gerrit and such | 14:52 |
| *** sborkows has quit IRC | 14:52 | |
| rloo | jroll: what's that third party CI/drivers link? shouldn't the supported ones be listed there? | 14:53 |
| jroll | rloo: a CI can be listed there, and running, but not meeting our expectations | 14:54 |
| *** dprince has quit IRC | 14:54 | |
| jroll | (most are not right now, but cutting some slack, we will tighten that up in the future) | 14:54 |
| rloo | jroll: for the first pass, we can at least unsupport those that are not listed there | 14:54 |
| jroll | rloo: yeah, I believe my patch matches that | 14:56 |
| rloo | jroll: this page: https://wiki.openstack.org/wiki/ThirdPartySystems | 14:56 |
| jroll | but I didn't double check | 14:56 |
| rloo | jroll: ok, if it matches that, just mention in the commit. | 14:56 |
| rloo | jroll: or i can do it if you want. when i review your patch. later today. | 14:57 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Import install and configure into new install guide https://review.openstack.org/367988 | 14:57 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Import Bare Metal service overview into install-guide https://review.openstack.org/366329 | 14:57 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Import "Configure integration with other OpenStack components" https://review.openstack.org/369510 | 14:57 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Mark untested drivers as unsupported https://review.openstack.org/369503 | 14:57 |
| jroll | rloo: done | 14:57 |
| rloo | jroll: thx :) | 14:57 |
| jroll | np | 14:57 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic-python-agent: Use ironic-lib to create configdrive https://review.openstack.org/296466 | 14:58 |
| mat128 | jroll: when I'm done integrating virtualPDU in devstack, can we just toggle that back to "True" ? | 14:58 |
| mat128 | re: snmp_power | 14:58 |
| jroll | mat128: yep | 14:58 |
| jroll | mat128: unless it takes more than three months, then we may drop it and you'll need to re-propose :) | 14:58 |
| jroll | but we could wait to drop it if it's clear that's coming | 14:58 |
| mat128 | jroll: I have something that works locally, but I have fighting priorities between this, the docs stuff and downstream priorities | 14:59 |
| mat128 | ;) | 14:59 |
| jroll | no worries :) | 14:59 |
| jroll | minimum three months deprecation to remove a thing | 14:59 |
| mat128 | jroll: https://github.com/internap/virtualpdu | 15:00 |
| mat128 | yeah | 15:00 |
| jlvillal | Good morning Ironic :) | 15:00 |
| mat128 | good morning jlvillal | 15:00 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Remove cyclic import between rpcapi and objects.base https://review.openstack.org/369513 | 15:00 |
| sambetts | jroll, rloo ^^^ | 15:00 |
| *** mjura has quit IRC | 15:01 | |
| jroll | sambetts: that works locally? does it also fix lucas' bug? | 15:01 |
| jlvillal | mat128: :) | 15:01 |
| *** mat128 is now known as mat128|mtg | 15:01 | |
| rloo | thx sambetts. on my list to look at (in meeting now) | 15:02 |
| *** jaosorior has quit IRC | 15:02 | |
| sambetts | jroll: it passes all the UTs so the cycle is gone I think, however I don't currently have a running setup to test the config file | 15:02 |
| *** rama_y has quit IRC | 15:02 | |
| jroll | ok, cool, lucasagomes ^^ | 15:03 |
| jroll | if you have some time and could test that | 15:03 |
| lucasagomes | sambetts, jroll nice! I will give it a go after the meeting | 15:03 |
| jroll | thanks :) | 15:03 |
| * jroll just jumped into a meeting as well | 15:03 | |
| sambetts | tbh I still think even with that fix that we probably shouldn't be using the CONF varibles at module load time, we run into this with the NodeObject defaults too | 15:04 |
| *** cdearborn has quit IRC | 15:04 | |
| openstackgerrit | Michael Turek proposed openstack/python-ironicclient: Refactor provision state so all actions can use inherited take_action https://review.openstack.org/364552 | 15:04 |
| Romanenko_K | Hello, rloo! Please look into my comment in LP1621565 if you have some time. | 15:05 |
| rloo | Romanenko_K: that's the email you sent me? will try, my TODO list seems to be growin' | 15:07 |
| jroll | sambetts: yeah, +1, it'll just happen again | 15:07 |
| Romanenko_K | yes, that's is | 15:07 |
| *** rpioso has joined #openstack-ironic | 15:08 | |
| *** fxpester has joined #openstack-ironic | 15:08 | |
| *** trown has quit IRC | 15:10 | |
| *** purp has quit IRC | 15:11 | |
| *** mtreinish has quit IRC | 15:12 | |
| *** trown has joined #openstack-ironic | 15:12 | |
| *** mkam has quit IRC | 15:12 | |
| *** mkam has joined #openstack-ironic | 15:13 | |
| *** mtreinish has joined #openstack-ironic | 15:14 | |
| *** purp has joined #openstack-ironic | 15:15 | |
| JayF | mat128|mtg: any advice on the next place to start with the install guide? Wanna make sure I know what you're working on before I jump in | 15:15 |
| *** moshele has joined #openstack-ironic | 15:16 | |
| jroll | JayF: mat128|mtg: that reminds me, for ongoing install guide changes, are you folks taking care of picking those up or? | 15:17 |
| JayF | jroll: what is your question? | 15:18 |
| JayF | I don't understand your question | 15:18 |
| jroll | JayF: so, you guys are moving content, right | 15:18 |
| JayF | yes, bit by bit | 15:19 |
| jroll | for a patch like this https://review.openstack.org/#/c/369385/ | 15:19 |
| patchbot | patch 369385 - ironic - Add a note about security groups in install guide | 15:19 |
| JayF | from the old install guide to the new | 15:19 |
| JayF | if it's not conflicting, we haven't moved that section yet, and it's fine | 15:19 |
| jroll | do you expect the contributor to handle putting it in both or will you pick it up? | 15:19 |
| JayF | if that person was updating a section that we had moved, they'd put it only in the new place | 15:19 |
| jroll | oh, you're moving it bit by bit? | 15:19 |
| JayF | yes | 15:19 |
| jroll | won't that be pretty confusing while it's on going? | 15:20 |
| *** mgoddard_ has quit IRC | 15:20 | |
| *** mgoddard has joined #openstack-ironic | 15:20 | |
| jroll | (for readers) | 15:20 |
| JayF | not really, we're maintaining links back to the place that has the info in both sides (install guide and dev docs) | 15:20 |
| JayF | and it shouldn't take longer than a week or two | 15:20 |
| jroll | hmm, ok | 15:20 |
| JayF | this is the path we talked to you about on vidyo | 15:20 |
| JayF | you suggested it iirc | 15:20 |
| JayF | lol | 15:20 |
| jroll | I don't think I suggested bit by bit | 15:22 |
| jroll | just move the whole thing in one patch | 15:22 |
| jroll | and then make it pretty | 15:22 |
| JayF | jroll: https://review.openstack.org/#/c/369510/ is an example of one of the patches | 15:25 |
| patchbot | patch 369510 - ironic - Import "Configure integration with other OpenStack... | 15:25 |
| JayF | jroll: mat128|mtg seems to have 3 like that up already | 15:25 |
| jroll | JayF: yeah, I understand what you're doing | 15:25 |
| *** rbrud has joined #openstack-ironic | 15:26 | |
| jroll | I'd rather move it in one shot, and then break it into sections and such | 15:26 |
| JayF | That's going to be a giant patch that'll conflict with everything | 15:26 |
| JayF | and I don't think some sections directly map over | 15:26 |
| jroll | then we land it quickly :) | 15:26 |
| jroll | if it's the same content it won't be controversial | 15:26 |
| JayF | but it can't be | 15:26 |
| jroll | "directly map over" to what? | 15:26 |
| JayF | for instance, in the install guide, there are separate install pages for each distribution | 15:26 |
| jroll | why? | 15:26 |
| JayF | we will have to do editorial work to split those instructions into the different install pages | 15:27 |
| jroll | is that a requirement to begin the work? | 15:27 |
| JayF | I believe so, it's in the cookiecutter and appears to be central to the design of the install guide | 15:27 |
| jroll | ugh | 15:27 |
| JayF | like if you look at other install guides | 15:27 |
| JayF | exactly | 15:27 |
| JayF | that's why we were doing it bit by bit, so the pieces that go across no change have no change, then we can do the ones that need a little more translation | 15:27 |
| jroll | this is going to be super weird while it's in progress | 15:28 |
| JayF | I don't think it's that bad, we're maintaining links around to everywhere we need to | 15:28 |
| jroll | yeah, are the themes the same and such though? | 15:29 |
| sambetts | nope | 15:29 |
| JayF | http://docs.openstack.org/project-install-guide/baremetal/draft/install-ubuntu.html#configure-the-bare-metal-service-for-cleaning is an example of it in practice | 15:29 |
| sambetts | one using oslosphix and the other uses openstackdoctheme | 15:30 |
| sambetts | or something | 15:30 |
| JayF | I am going to dig today as to why it's going under draft/ as well, before we land any of those from mat | 15:30 |
| jroll | right, so it'll feel weird jumping back and forth | 15:31 |
| JayF | sure; it'll be weird for one or two weeks. In exchange we get a proper install guide merged with less pain. I think it's a good trade off. | 15:32 |
| *** krtaylor has joined #openstack-ironic | 15:32 | |
| *** dtantsur|mtg is now known as dtantsur | 15:33 | |
| jroll | yeah, I hope we can finish it that quickly | 15:33 |
| jroll | I feel like this is a nice sprint for between cutting newton and summit | 15:33 |
| JayF | I think I need to figure out a couple of things, mainly: does stable/* matter for install guide, and why are we showing up under /draft/ | 15:34 |
| lucasagomes | sambetts, apparently the patch doesn't fix the bug | 15:34 |
| jroll | I don't think stable matters yet | 15:34 |
| jroll | yeah | 15:35 |
| sambetts | lucasagomes: yeah, I wasn't sure it would, but it removes the cyclical import anyway which is nice | 15:35 |
| lucasagomes | sambetts, ++ | 15:36 |
| *** rbartal has quit IRC | 15:37 | |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Add volume_connector table to save connector information https://review.openstack.org/200983 | 15:38 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Introduce VolumeConnector object https://review.openstack.org/214584 | 15:38 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Introduce new RPCs to support volume connector operation https://review.openstack.org/214585 | 15:38 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Add volume_targets table to save target information https://review.openstack.org/285218 | 15:39 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Introduce VolumeTarget object https://review.openstack.org/285219 | 15:39 |
| JayF | mat128|mtg: when you're free of your meeting, we should chat a bit about install guide more | 15:39 |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Introduce new RPCs to support volume target operations https://review.openstack.org/285220 | 15:39 |
| *** athomas has quit IRC | 15:41 | |
| *** aNuposic has joined #openstack-ironic | 15:42 | |
| JayF | aNuposic: https://github.com/openstack/ironic/blob/705e96c92e133aa09434a1b41b0529bc84e03660/ironic/common/glance_service/v2/image_service.py#L150 | 15:43 |
| *** jtomasek has quit IRC | 15:44 | |
| *** hoangcx has quit IRC | 15:45 | |
| sambetts | lucasagomes: have you made sure that the periodics setup by the different driver modules aren't affected my the same thing?? | 15:47 |
| lucasagomes | sambetts, not really, I've tested only the periodics from the ConductorManager | 15:47 |
| lucasagomes | sambetts, lemme try it out | 15:47 |
| *** MattMan has quit IRC | 15:48 | |
| *** MattMan has joined #openstack-ironic | 15:48 | |
| jlvillal | aNuposic: Also: https://github.com/openstack/ironic/commit/7d6eb162377a6c3c4803d9516705f01a8dd195b3 | 15:48 |
| aNuposic | jlvillal: hey | 15:48 |
| JayF | jroll: ^ aNuposic is Anup from OSIC, you missed meeting him at standup but he's going to be working with us | 15:49 |
| jroll | aNuposic: \o welcome | 15:49 |
| *** ccamacho is now known as ccamacho|afk | 15:50 | |
| sambetts | o/ aNuposic | 15:50 |
| aNuposic | JayF: jroll: THanks | 15:50 |
| *** jcoufal has quit IRC | 15:51 | |
| *** hoangcx has joined #openstack-ironic | 15:55 | |
| lucasagomes | sambetts, http://paste.openstack.org/show/573659/ | 15:56 |
| lucasagomes | sambetts, missing oneview I don't have it enabled | 15:56 |
| lucasagomes | but should be good | 15:56 |
| *** jistr|mtgs is now known as jistr | 15:57 | |
| sambetts | lucasagomes: so they suffer the same fate? | 15:57 |
| sambetts | or no? | 15:57 |
| lucasagomes | sambetts, oh no, the drivers AFAICT does error out even without the patch | 15:58 |
| lucasagomes | sambetts, 1 min lemme revert the patch and check | 15:58 |
| *** vishwana_ is now known as vishwanathj | 15:58 | |
| lucasagomes | sambetts, yeah drivers are good, they fail even w/o that patch | 15:58 |
| lucasagomes | that patch == #369000 | 15:59 |
| jlvillal | lucasagomes: http://paste.openstack.org/show/573663/ What do you think? | 16:00 |
| * jroll grabs quick lunch | 16:00 | |
| jlvillal | jroll: Bon appetit | 16:01 |
| lucasagomes | jlvillal, nice! +1 much more precise | 16:01 |
| jlvillal | lucasagomes: Okay I can push it | 16:01 |
| lucasagomes | jlvillal, I will re-add the +2 once you update it :D | 16:01 |
| *** dtantsur is now known as dtantsur|afk | 16:02 | |
| lucasagomes | probably we can land it straight away since it's only the commit message being updated (for something better) | 16:02 |
| mgould | jlvillal: LGTM | 16:02 |
| jlvillal | lucasagomes: I pushed it, but the bot didn't say so https://review.openstack.org/#/c/366518/ | 16:03 |
| patchbot | patch 366518 - ironic - Docs: Add warning and add setting of $LOGFILE | 16:03 |
| *** moshele has quit IRC | 16:03 | |
| jlvillal | lucasagomes: I made a tiny bit more changes than just the commit message. | 16:03 |
| jlvillal | lucasagomes: I also changed IRONIC_VM_LOG_DIR too | 16:04 |
| lucasagomes | jlvillal, yeah I see the warnings there | 16:04 |
| jlvillal | And the comments | 16:04 |
| lucasagomes | lgtm :-) | 16:04 |
| jlvillal | Thanks :) | 16:04 |
| rloo | Romanenko_K: replied | 16:04 |
| lucasagomes | but that said, I hope nobody is running devstack on their main installation | 16:04 |
| jlvillal | heh | 16:05 |
| sambetts | lucasagomes: /me wonders if we should do something like http://paste.openstack.org/show/573664/ | 16:05 |
| lucasagomes | sambetts, doesn't look bad :D tho I think I still prefer the spacing_func= approach | 16:06 |
| * jlvillal found this interesting https://code.facebook.com/posts/365787980419535/scalable-and-secure-access-with-ssh/?_fb_noscript=1 | 16:07 | |
| sambetts | lucasagomes: yeah, one advantage of doing it like in my paste is that we only have CONF.conductor.sync_power_state_interval in one place instead of all over the file | 16:07 |
| lucasagomes | sambetts, we only have it in place, I think /me checks | 16:08 |
| lucasagomes | sambetts, yeah it's only used in that decorator (for now at least) | 16:08 |
| JayF | mariojv: I just landed https://review.openstack.org/#/c/364422 -- can you push a fixup patch once it merges fixing that one link rloo pointed out? | 16:09 |
| patchbot | patch 364422 - ironic-specs - Update notifications spec | 16:09 |
| *** ohamada has quit IRC | 16:09 | |
| sambetts | lucasagomes: right, but in 4 places in the file you have to change if we need to change the config for what ever reason | 16:09 |
| lucasagomes | sambetts, hmm I think I don't follow, what you mean ? | 16:13 |
| lucasagomes | sorry my brain is really slow today heh | 16:13 |
| lucasagomes | finland, france, switzerland, spain and UK are in #2 place | 16:14 |
| lucasagomes | wrong channel | 16:14 |
| *** adnavare has joined #openstack-ironic | 16:14 | |
| mgould | heh | 16:14 |
| mgould | jlvillal: that's very interesting, thanks! | 16:14 |
| mariojv | sure thanks JayF | 16:15 |
| mgould | I love that FB perform statistical analysis on their server-login data :-) | 16:15 |
| *** rbrud has quit IRC | 16:16 | |
| lucasagomes | sambetts, +2 the patch removing the cyclic dependency | 16:16 |
| sambetts | lucasagomes: I mean that instead of have 4 functions and therefore 4 decorators all using the config option, whereas in my paste you would only have 1 place where the config option is used because the function only needs to use the periodics.periodic() function once | 16:16 |
| lucasagomes | sambetts, oh gotcha, you centralize all the decorators | 16:16 |
| lucasagomes | true | 16:16 |
| *** e0ne has quit IRC | 16:17 | |
| *** aNuposic has quit IRC | 16:17 | |
| lucasagomes | it can also be confusing because by only looking at the function you don't know whether it's decorated or not | 16:17 |
| lucasagomes | you would have to check that other method to see the decorators being applied | 16:17 |
| *** trown is now known as trown|lunch | 16:18 | |
| lucasagomes | sambetts, maybe we should propose both approachs and see what reviewers think ? | 16:19 |
| lucasagomes | I'm actually good with any | 16:19 |
| lucasagomes | as long as we fix that problem | 16:19 |
| *** permalac has quit IRC | 16:20 | |
| *** fragatina has joined #openstack-ironic | 16:28 | |
| *** ChubYann has joined #openstack-ironic | 16:28 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client: TrivialFix: Remove logging import unused https://review.openstack.org/367788 | 16:29 |
| sambetts | lucasagomes: just looking at the futurist code, they store the spacing on the function itself in f._periodic_spacing, so it is technically possible to update it after the decorator has been used... however ... I am wary of that approach | 16:30 |
| *** milan has quit IRC | 16:31 | |
| *** fragatina has quit IRC | 16:32 | |
| *** derekh has quit IRC | 16:32 | |
| *** _vsaienko has joined #openstack-ironic | 16:32 | |
| lucasagomes | sambetts, is that a function or just an attribute ? | 16:33 |
| * lucasagomes opens the futurist code | 16:33 | |
| *** fragatina has joined #openstack-ironic | 16:33 | |
| sambetts | lucasagomes: just an attribute https://github.com/openstack/futurist/blob/master/futurist/periodics.py#L150 | 16:34 |
| *** hoangcx has quit IRC | 16:34 | |
| lucasagomes | sambetts, ah right | 16:34 |
| sambetts | lucasagomes: I'm thinking for the SIGHUP case if we wanted to update it | 16:34 |
| lucasagomes | so <func>._periodic_spacing is where it's stored | 16:34 |
| sambetts | yup | 16:34 |
| lucasagomes | sambetts, heh yeah we could update it, but not sure if we should since it's private | 16:35 |
| lucasagomes | I was thinking of having _periodic_spacing to be a pointer to another function | 16:35 |
| lucasagomes | and futurist would do <func>._periodic_spacing() whereever it's needed | 16:35 |
| NobodyCam | Good Morning Ironic'ers :) | 16:35 |
| sambetts | yeah... thats one way or the alternative is having an update_spacing function in futurist which takes a periodic function and just updates that attribute | 16:36 |
| *** xavierr_afk is now known as xavierr | 16:37 | |
| xavierr | me.say_good_morning('ironicers') | 16:37 |
| NobodyCam | hehehe :) | 16:38 |
| NobodyCam | morning xavierr :) | 16:38 |
| xavierr | hey NobodyCam :D | 16:38 |
| sambetts | then we could have a function in Ironic which just does for <list of functions> update_spacing(function, CONF....) but I like your way a little better I think, although it depends how often it'll get re-evaulated | 16:38 |
| lucasagomes | NobodyCam, hi there | 16:39 |
| lucasagomes | sambetts, indeed | 16:39 |
| NobodyCam | but shouldn't that be self.printf("Good Morning %s") % "Ironic'ers" | 16:39 |
| NobodyCam | morning lucasagomes :) | 16:39 |
| *** _vsaienko has quit IRC | 16:40 | |
| xavierr | NobodyCam: self.printf("Good Morning %s" % "Ironic'ers")? | 16:42 |
| *** hoangcx has joined #openstack-ironic | 16:42 | |
| lucasagomes | xavierr, s/self.printf/print | 16:43 |
| NobodyCam | lol !!! +++ | 16:43 |
| JayF | I prefer the dot matrix printer functions | 16:44 |
| JayF | they make a nice crunching noise for each character | 16:44 |
| xavierr | lucasagomes: it could be a custom self.printf method :P | 16:44 |
| xavierr | hahaha | 16:44 |
| jroll | print('<%s> good morning ironic' % self.irc_nick) | 16:44 |
| lucasagomes | xavierr, haha right | 16:44 |
| xavierr | ;) | 16:44 |
| lucasagomes | xavierr, but it's not in the class scope here, so no self | 16:45 |
| * lucasagomes thinks we are getting too far | 16:45 | |
| *** fragatina has quit IRC | 16:45 | |
| lucasagomes | jroll, ++ PixeBoots should do that | 16:45 |
| xavierr | jroll: +2 to your print | 16:45 |
| * lucasagomes writes an RFE #butnot | 16:45 | |
| *** fragatina has joined #openstack-ironic | 16:45 | |
| jroll | heh | 16:45 |
| *** tesseract- has quit IRC | 16:46 | |
| *** hoangcx has quit IRC | 16:47 | |
| xavierr | ok, so let's stop here or someone will ask to add a release note to that print function | 16:47 |
| xavierr | dtantsur|afk: arround? | 16:47 |
| *** _vsaienko has joined #openstack-ironic | 16:47 | |
| * xavierr needed to to talk to him :( | 16:48 | |
| *** penick has joined #openstack-ironic | 16:48 | |
| JayF | he's usually gone by this time | 16:48 |
| lucasagomes | yeah, it's almost 7pm for him now | 16:49 |
| xavierr | :( | 16:49 |
| xavierr | aarefiev: arround? | 16:49 |
| *** _vsaienko has quit IRC | 16:50 | |
| *** adnavare has quit IRC | 16:51 | |
| *** ChubYann has quit IRC | 16:52 | |
| *** electrofelix has quit IRC | 16:55 | |
| lucasagomes | folks, I'm calling it a day | 16:55 |
| lucasagomes | have a great evening all, talk to you tomorrow | 16:56 |
| xavierr | hey lucasagomes, have a good evening :) | 16:56 |
| *** lucasagomes is now known as lucas-dinner | 16:56 | |
| lucas-dinner | :-) see ya | 16:56 |
| *** rbrud has joined #openstack-ironic | 16:57 | |
| NobodyCam | night lucas-dinner | 16:58 |
| mgould | lucas-dinner: o/ | 16:59 |
| *** pcaruana has quit IRC | 16:59 | |
| NobodyCam | morning mgould :) | 17:01 |
| mgould | morning NobodyCam | 17:02 |
| NobodyCam | :) | 17:02 |
| *** rcernin has quit IRC | 17:03 | |
| *** xek has quit IRC | 17:03 | |
| openstackgerrit | Merged openstack/ironic-inspector: Add translation marker to deprecated_reason of config opts https://review.openstack.org/369333 | 17:03 |
| jlvillal | JayF: If you have time to look at the RFE I did for the configdrive database change. https://bugs.launchpad.net/ironic/+bug/1596421 | 17:03 |
| openstack | Launchpad bug 1596421 in Ironic "RFE: Increase size of data base entry for instance_info to allow configdrives larger than 64KB" [Medium,In progress] - Assigned to David Edery (david-edery) | 17:03 |
| JayF | jlvillal: I'm not convinced that needs an rfe though | 17:04 |
| * mgould has posted a few more mountaineering photos, if anyone's interested: https://twitter.com/AndyMilesExped | 17:04 | |
| JayF | jlvillal: pointed my brain at that patch this morning and punted | 17:04 |
| JayF | jroll: ^ do you have opinions? | 17:04 |
| jroll | what doesn't need an rfe? | 17:05 |
| jlvillal | JayF: I made an RFE, since we don't seem to have consensus | 17:05 |
| jlvillal | jroll: https://bugs.launchpad.net/ironic/+bug/1596421 | 17:05 |
| openstack | Launchpad bug 1596421 in Ironic "RFE: Increase size of data base entry for instance_info to allow configdrives larger than 64KiB" [Medium,In progress] - Assigned to David Edery (david-edery) | 17:05 |
| jroll | mmm | 17:05 |
| jroll | I mean, it's totally a bug | 17:06 |
| * mgould -> home; good night! | 17:06 | |
| jroll | I'm not really opinionated whether we should tag it rfe or not | 17:06 |
| *** mgould is now known as mgould|afk | 17:06 | |
| jlvillal | I can switch it back. But I remember people not being thrilled with the idea of changing the database. | 17:07 |
| jlvillal | Switch it back to just being a bug, that is. | 17:07 |
| jroll | I really don't think it matters personally | 17:07 |
| NobodyCam | might mgould|afk ... morning jlvillal jroll JayF :) | 17:07 |
| jroll | there's no effective difference other than extra process around rfes | 17:07 |
| jroll | morning NobodyCam | 17:07 |
| JayF | I'm wondering if that patch is even that controversial anymore now that it's MEDIUMTEXT | 17:07 |
| JayF | gm NobodyCam | 17:07 |
| jlvillal | JayF: What do you think of the idea for a config option to maintain the current 64KiB limit, so operators have to explicitly increase the size. | 17:08 |
| jlvillal | JayF: overkill? good idea? something else? :) | 17:08 |
| NobodyCam | :) | 17:08 |
| jlvillal | hey NobodyCam :) | 17:08 |
| JayF | jlvillal: I am as -1 as possible to a config option | 17:09 |
| JayF | jlvillal: because you shouldn't change DB scheme based on options | 17:09 |
| jlvillal | JayF: Not sure what you mean by changing the DB scheme by option. | 17:09 |
| xavierr | is gerrit down? | 17:10 |
| jlvillal | JayF: I'm not saying to change the DB scheme. It would still be MEDIUMTEXT | 17:10 |
| -openstackstatus- NOTICE: The Gerrit service on review.openstack.org is being restarted now to address current performance problems, but should return to a working state within a few minutes | 17:10 | |
| jlvillal | JayF: I'm saying a config option to check the configdrive size and if it comes in larger than the config option to reject it. | 17:10 |
| JayF | jlvillal: we should add that anyway, but that's an RFE, not related to the bug imo | 17:11 |
| JayF | jlvillal: well, kinda related, but less important and we shouldn't block one on the otehr | 17:11 |
| jroll | can we have a config option to get rid of all the silly config options we have? | 17:11 |
| jlvillal | heh | 17:11 |
| *** lindycoder has quit IRC | 17:12 | |
| *** fragatina has quit IRC | 17:13 | |
| *** ijw has joined #openstack-ironic | 17:13 | |
| *** hoangcx has joined #openstack-ironic | 17:13 | |
| * sambetts -> homer | 17:14 | |
| sambetts | home* | 17:14 |
| sambetts | Night all o/ | 17:14 |
| NobodyCam | night sambetts | 17:14 |
| *** sambetts is now known as sambetts|afk | 17:14 | |
| sambetts|afk | cya NobodyCam | 17:14 |
| *** yarkot has joined #openstack-ironic | 17:14 | |
| *** yarkot has quit IRC | 17:14 | |
| JayF | o/ | 17:15 |
| *** e0ne has joined #openstack-ironic | 17:17 | |
| *** daemontool has joined #openstack-ironic | 17:18 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Negative tests for testing actions with Chassis. https://review.openstack.org/343796 | 17:22 |
| *** xek has joined #openstack-ironic | 17:24 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add basic tests for OSC plugin baremetal port commands https://review.openstack.org/365692 | 17:26 |
| *** adnavare has joined #openstack-ironic | 17:27 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add basic tests for OSC plugin baremetal node commands https://review.openstack.org/301687 | 17:27 |
| *** KrisK19 has quit IRC | 17:29 | |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic: Onetime boot when set_boot_device isn't persistent https://review.openstack.org/340596 | 17:33 |
| *** _vsaienko has joined #openstack-ironic | 17:34 | |
| *** trown|lunch is now known as trown | 17:35 | |
| *** daemontool has quit IRC | 17:36 | |
| *** rbudden has joined #openstack-ironic | 17:36 | |
| *** _vsaienko has quit IRC | 17:42 | |
| *** aswadr_ has quit IRC | 17:42 | |
| *** Romanenko_K has quit IRC | 17:42 | |
| *** mbound has quit IRC | 17:42 | |
| *** krtaylor has quit IRC | 17:48 | |
| *** hoangcx has quit IRC | 17:51 | |
| *** hoangcx has joined #openstack-ironic | 17:51 | |
| *** amotoki has joined #openstack-ironic | 17:53 | |
| *** yarkot has joined #openstack-ironic | 17:56 | |
| *** _vsaienko has joined #openstack-ironic | 17:57 | |
| *** amotoki has quit IRC | 17:58 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Add config option to specify maximum size for configdrive https://review.openstack.org/369617 | 18:05 |
| JayF | rloo: +1 on namespace everything (just saw your comment on that merged spec) | 18:07 |
| rloo | JayF: :) | 18:07 |
| rloo | JayF: I'm glad Balazs took a look. Makes me feel more comfortable that it is similar to nova's approach | 18:08 |
| rloo | JayF: a cross-project spec for notifications would be good. | 18:09 |
| JayF | I mean, nova/ironic doing them | 18:09 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Add config option to specify maximum size for configdrive https://review.openstack.org/369617 | 18:09 |
| JayF | if a third project starts, we might have a good reason for it | 18:09 |
| rloo | JayF: i had the feeling there was at least one other project with notifications, but didn't look into it. | 18:09 |
| *** fragatina has joined #openstack-ironic | 18:10 | |
| *** KrisK19 has joined #openstack-ironic | 18:11 | |
| *** _vsaienko has quit IRC | 18:11 | |
| *** _vsaienko has joined #openstack-ironic | 18:11 | |
| *** rcernin has joined #openstack-ironic | 18:12 | |
| *** Sukhdev has joined #openstack-ironic | 18:14 | |
| *** mkam has quit IRC | 18:20 | |
| *** jpich has quit IRC | 18:23 | |
| *** david-lyle has quit IRC | 18:23 | |
| *** _vsaienko has quit IRC | 18:24 | |
| *** hoangcx has quit IRC | 18:26 | |
| *** hoangcx has joined #openstack-ironic | 18:26 | |
| *** david-lyle has joined #openstack-ironic | 18:26 | |
| *** _vsaienko has joined #openstack-ironic | 18:29 | |
| *** penick has quit IRC | 18:32 | |
| *** mkam has joined #openstack-ironic | 18:33 | |
| *** _vsaienko has quit IRC | 18:35 | |
| *** _vsaienko has joined #openstack-ironic | 18:35 | |
| *** pcaruana has joined #openstack-ironic | 18:36 | |
| *** moshele has joined #openstack-ironic | 18:36 | |
| *** fxpester has quit IRC | 18:36 | |
| *** KrisK19 has quit IRC | 18:38 | |
| *** penick has joined #openstack-ironic | 18:42 | |
| anish | anyone who has experience with python-dracclient around ? want to propose some changes, but have a few questions before that | 18:42 |
| mariojv | a cross-project spec would be interesting, if other projects start using notifications | 18:42 |
| mariojv | i think the people who work on Searchlight (notification search service iirc) would be interested in that | 18:43 |
| mariojv | they had some comments about standardization on the power state notification patch set. i think their concerns were addressed in yuriyz's spec for CRUD notifications | 18:43 |
| *** sahil has joined #openstack-ironic | 18:43 | |
| sahil | hello | 18:43 |
| sahil | is capetella here ? | 18:43 |
| *** yarkot has quit IRC | 18:44 | |
| jroll | sahil: I don't see anyone with that name here, maybe you can ask your question and see if someone else can answer? | 18:44 |
| jroll | anish: go ahead and ask your questions, we'll try to answer them | 18:45 |
| sahil | ok, so as you know jroll I have been trying to boot a baremetal node using ironic-devstack in a vm | 18:46 |
| sahil | I have been able to configure it correctly enough that the baremetal node is getting an ip address from the neutron made dhcp server | 18:46 |
| sahil | but it cannot locate the tftp server | 18:46 |
| sahil | on doing tcp dump, on the interface br-ironic, this was the message | 18:47 |
| sahil | ARP, Ethernet (len 6), IPv4 (len 4), Request who-has ironstack03 tell 192.168.1.29, length 46 | 18:47 |
| sahil | so to check what the deal was | 18:47 |
| sahil | I tried to add ironstack03 to the ip address of devstack VM a | 18:47 |
| sahil | and pinged from the host | 18:47 |
| sahil | I am not getting any response | 18:48 |
| sahil | at the host side | 18:48 |
| anish | looking at how the attribute values are populated via wsman, only name, current_value and pending_value are really populated | 18:48 |
| sahil | but on the VM side tcpdump showed this | 18:48 |
| anish | however 'AttributeName' is not unique if I extend this to SystemView of iDracCardView | 18:48 |
| sahil | Request who-has ironstack03 tell 192.168.1.11, length 28 | 18:48 |
| sahil | this is the ip of the gateway/host on which the VM is running | 18:48 |
| anish | so I need to extend it to FQDD#GroupID#AttributeName | 18:48 |
| sahil | I have disabled the iptables for the time being | 18:49 |
| anish | or at least GroupID#AttributeName | 18:49 |
| sahil | on the devstack VM | 18:49 |
| anish | not sure how many things might break because of that | 18:49 |
| *** KrisK19 has joined #openstack-ironic | 18:49 | |
| anish | though BIOS Attributes (which seems to be the only thing being used right now) do not suffer from this problem | 18:49 |
| anish | * SystemView or iDracCardView | 18:49 |
| sahil | how do I get past this error ? How do I make devstack resolve a name to correct IP address ? | 18:50 |
| anish | I'm trying to figure out if it's okay to make the idrac/system stuff behave differently from bios stuff, or change everything to the same fashion | 18:50 |
| *** moshele has quit IRC | 18:50 | |
| anish | any suggestions ? | 18:50 |
| jroll | sahil: huh, I'm somewhat of a network noob. but I thought ARP requests could only be for IP addresses, not hostnames, seems odd, maybe broken dhcp/pxe config... lemme look around a minute | 18:51 |
| jroll | anish: you're right, that does need a dracclient expert. you'll probably want ifarkas but he doesn't seem to be around. lucas-dinner might also be helpful but is done for the day... maybe email the -dev list and see if they catch it (or try daytime europe time) | 18:52 |
| *** _vsaienko has quit IRC | 18:53 | |
| anish | jroll: thanks ! this would be the ironic-dev list or does dracclient have it's own ? | 18:53 |
| sahil | no even if I do a ping to the devstack VM from the host it is running on | 18:54 |
| sahil | produces the same message in tcp dump | 18:54 |
| sahil | but no response on the host side | 18:54 |
| *** hoangcx has quit IRC | 18:54 | |
| jroll | anish: openstack-dev with [ironic] in the subject http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev | 18:54 |
| anish | thanks again ! | 18:54 |
| jroll | np :) | 18:54 |
| jroll | sahil: ah, I see, hrm | 18:55 |
| * jroll has no idea | 18:56 | |
| sahil | jroll: funny thing is I can ping to the gateway (which is a bridge on the host) from my devstack VM and get a response | 18:58 |
| sahil | but from host when I ping the VM no response | 18:59 |
| jroll | yeah, that's very odd | 18:59 |
| jroll | sahil: what do you get with `hostname -f` in the vm? | 18:59 |
| sahil | I figured out what is the problem | 19:00 |
| sahil | may need help to fix it | 19:00 |
| KrisK19 | Is anyone using ironic to have one flavor with one phisical disk and (X% roofs, Y% ephemeral, Z% swap)? | 19:01 |
| sahil | jroll: so it seems when I provided the gateway in the local.conf file | 19:01 |
| KrisK19 | GPT or UEFI it is not important | 19:01 |
| sahil | devstack used that info to configure an interface in the qrouter-<id> namespace | 19:01 |
| sahil | now I have two interfaces with same network ip address | 19:01 |
| jroll | oh my | 19:01 |
| sahil | no wonder I was getting arp time outs | 19:01 |
| jroll | so you can configure different network ranges and such | 19:02 |
| *** ssl_ has joined #openstack-ironic | 19:02 | |
| * jroll finds things | 19:02 | |
| sahil | jroll: in my env I have configured a fixed VLAN on which this baremetal nodes are available | 19:02 |
| sahil | I decided the subnet for the same | 19:02 |
| ssl_ | i'm testing the ironic. one node stuck at provisioning state "clean failed", how do i fix it? or reset? | 19:03 |
| jroll | sahil: this might be what you want? https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L267-L279 | 19:03 |
| jroll | otherwise I've no idea | 19:03 |
| jroll | it might be something with neutron's devstack setup | 19:04 |
| jroll | KrisK19: so you want to define those by percent rather than MB? I don't think that's possible | 19:04 |
| sahil | nope that is specific configuration about the provisioning network | 19:04 |
| sahil | let me try something, I will report here again | 19:04 |
| KrisK19 | jroll: it could be MB as well | 19:04 |
| KrisK19 | as flavor have that information plus scheduler must have specific values to match nodes | 19:05 |
| KrisK19 | it is jost like 10 GB 10 GB 200GB ( one disk ) | 19:06 |
| KrisK19 | *just | 19:06 |
| *** e0ne has quit IRC | 19:06 | |
| rloo | mario, wrt notifications, i was looking at nova's (http://docs.openstack.org/developer/nova/notifications.html#existing-versioned-notifications) and look at the payloads | 19:06 |
| rloo | mariojv: ^^ | 19:06 |
| jroll | KrisK19: yes, that's supported then, as long as you use "partition images" which have a separate root/kernel/initrd. the flavor is matched with the sum of root/ephemeral/swap against total disk for the ironic node (node.properties['local_gb']) | 19:07 |
| mariojv | oh, that's a nice table | 19:07 |
| *** ijw has quit IRC | 19:07 | |
| rloo | mariojv: they provide most of the instance info in all the payloads for instance events | 19:07 |
| rloo | mariojv: yeah, the table/doc is nice. we, too, should do that :) | 19:07 |
| jroll | ssl_: the ironic state machine is here, you want something like ironic node-set-provision-state $uuid manage, and same for 'provide' http://docs.openstack.org/developer/ironic/dev/states.html | 19:08 |
| KrisK19 | yeah as I thought, cos we have “whole image” and this is what according to code could cuse issue | 19:08 |
| mariojv | i'm not opposed to adding more information to payloads if people find it useful | 19:08 |
| jroll | ssl_: you might look at logs for why cleaning failed, too | 19:08 |
| KrisK19 | but my question was more like is anyone is using that and it is working fine | 19:08 |
| rloo | mariojv: i hate to say it, but we should look, maybe model our code similar to theirs | 19:08 |
| mariojv | rloo: i totally modeled all my code off of theirs :) | 19:08 |
| rloo | mariojv: ha ha | 19:08 |
| mariojv | the payload is specified on an individual notification basis, though | 19:09 |
| jroll | KrisK19: yeah, you can still put swap/ephemeral/root in the flavor, it will still be matched, but won't be partitioned to those sizes | 19:09 |
| mariojv | so it's not hard to update, say, the power state notificatinon patch to add more fields | 19:09 |
| rloo | mariojv: right, they have payloads that inherit from base payload, then add more fields to it | 19:09 |
| jroll | KrisK19: we don't use the disk filter, but we do use whole disk images and have root/ephemeral/swap in the flavor, and it doesn't break anythign | 19:09 |
| rloo | mariojv: so I am thinking we can do a similar thing. have a NodePayload, then a NodeActionPayload (if 'Action' makes sense for us). I'm not quite sure what Action vs Update is for, for nova. | 19:10 |
| mariojv | rloo: oh, i see - so in terms of inheritance base payload class -> NodePayload -> more specific things | 19:10 |
| rloo | mariojv: nova's InstanceActionPayload and InstanceUpdatePayload are both based on InstancePayload. | 19:11 |
| mariojv | ok; i'll take a look at that piece of the code this week when i have some time | 19:11 |
| mariojv | i think it was just the base classes (the one InstancePayload presumably inherits from) when i started this | 19:11 |
| rloo | mariojv: thx. we (the royal we) would like to see if we can get this in for newton. | 19:12 |
| mariojv | alright | 19:12 |
| rloo | mariojv: but if we can't, we can't. | 19:12 |
| mariojv | when's the last day for newton? | 19:12 |
| ssl_ | jroll I tried to reset the state, it throws: The requested action "active" can not be performed on node "xxxx" while it is in state "clean failed". | 19:12 |
| rloo | mariojv: i think it means the power patch has to be in pretty good shape by the end of this week. or it might mean it has to be merged by then. jroll is the boss. | 19:12 |
| mariojv | ok | 19:12 |
| jroll | ssl_: right, you need to follow the path, from clean failed you can only call 'manage' | 19:13 |
| mariojv | so, besides whatever comments are already there, to land this: 1. update doc to match nova's formatting 2. update payload to inherit from NodePayload base class with more field | 19:13 |
| mariojv | *fields | 19:13 |
| mariojv | maybe switch the order on those in terms of priority | 19:13 |
| jroll | ssl_: then it'll be in manageable, and you can use 'provide' to go through cleaning to available | 19:13 |
| rloo | mariojv: final week for newton candidates is week of sept 26 but i think we'd like to get ironic done sooner. | 19:13 |
| jroll | ssl_: (this is confusing, sorry :( ) | 19:13 |
| rloo | mariojv: i'm not too worried about the doc, we can prettyify that later. as long as the/some info is there. i'm more concerned with getting the code 'right'. | 19:14 |
| mariojv | ok | 19:14 |
| rloo | mariojv: which means i should play with it and see if i like the power sync stuff. i think that's the last thing that is bothering me... | 19:15 |
| rloo | mariojv: or we leave out the power-sync for now... | 19:15 |
| rloo | mariojv: btw, what kind of comments have you gotten from searchlight? | 19:16 |
| *** david-lyle has quit IRC | 19:16 | |
| *** rbudden has quit IRC | 19:16 | |
| *** rpioso has quit IRC | 19:16 | |
| *** anush_ has quit IRC | 19:16 | |
| *** ppiela has quit IRC | 19:16 | |
| *** Daviey_ has quit IRC | 19:16 | |
| *** persia has quit IRC | 19:16 | |
| *** Fdaisuke has quit IRC | 19:16 | |
| *** aignatov has quit IRC | 19:16 | |
| *** Nakato has quit IRC | 19:16 | |
| *** jlvillal has quit IRC | 19:16 | |
| *** yarkot1 has quit IRC | 19:16 | |
| *** htruta has quit IRC | 19:16 | |
| *** aarefiev has quit IRC | 19:16 | |
| *** alex_xu has quit IRC | 19:16 | |
| *** thrash has quit IRC | 19:16 | |
| *** lynxman has quit IRC | 19:16 | |
| *** sinh_ has quit IRC | 19:16 | |
| *** Sukhdev has quit IRC | 19:16 | |
| *** anush__ has quit IRC | 19:16 | |
| *** amitkqed has quit IRC | 19:16 | |
| *** hoonetorg has quit IRC | 19:16 | |
| *** hj-hpe has quit IRC | 19:16 | |
| *** devananda has quit IRC | 19:16 | |
| *** clenimar has quit IRC | 19:16 | |
| *** bnemec has quit IRC | 19:16 | |
| *** patchbot has quit IRC | 19:16 | |
| *** ianw has quit IRC | 19:16 | |
| *** cmyster has quit IRC | 19:16 | |
| *** penick has quit IRC | 19:17 | |
| *** MattMan has quit IRC | 19:17 | |
| *** trown has quit IRC | 19:17 | |
| *** davidlenwell has quit IRC | 19:17 | |
| *** pas-ha has quit IRC | 19:17 | |
| *** anush has quit IRC | 19:17 | |
| *** rm_work has quit IRC | 19:17 | |
| *** hogepodge has quit IRC | 19:17 | |
| *** clif_h has quit IRC | 19:17 | |
| *** patrickeast has quit IRC | 19:17 | |
| *** rbradfor has quit IRC | 19:17 | |
| *** adam_g has quit IRC | 19:17 | |
| *** francoblanco has quit IRC | 19:17 | |
| *** JayF has quit IRC | 19:17 | |
| *** yuriyz has quit IRC | 19:17 | |
| *** dhellmann has quit IRC | 19:17 | |
| *** pleia2 has quit IRC | 19:17 | |
| *** odyssey4me has quit IRC | 19:17 | |
| *** alaski has quit IRC | 19:17 | |
| *** Guest66666 has quit IRC | 19:17 | |
| *** yhvh has quit IRC | 19:17 | |
| *** joemonk has quit IRC | 19:17 | |
| *** ionutbalutoiu has quit IRC | 19:17 | |
| *** SpamapS has quit IRC | 19:17 | |
| *** logan- has quit IRC | 19:17 | |
| *** baoli_ has quit IRC | 19:17 | |
| *** ccamacho|afk has quit IRC | 19:17 | |
| *** zigo has quit IRC | 19:17 | |
| *** vsaienk0 has quit IRC | 19:17 | |
| *** yuanying has quit IRC | 19:17 | |
| *** serverascode has quit IRC | 19:17 | |
| *** notmorgan has quit IRC | 19:17 | |
| *** hamzy has quit IRC | 19:17 | |
| *** HenryG has quit IRC | 19:17 | |
| *** tdurakov has quit IRC | 19:17 | |
| *** kozhukalov has quit IRC | 19:17 | |
| *** boris-42 has quit IRC | 19:17 | |
| *** natorious has quit IRC | 19:17 | |
| *** dstepanenko_ has quit IRC | 19:17 | |
| *** adreznec has quit IRC | 19:17 | |
| *** mgagne has quit IRC | 19:17 | |
| *** aslezil has quit IRC | 19:17 | |
| *** d0ugal has quit IRC | 19:17 | |
| *** ptx0 has quit IRC | 19:17 | |
| *** lucas-dinner has quit IRC | 19:17 | |
| *** jralph_ has quit IRC | 19:17 | |
| *** nicodemos has quit IRC | 19:17 | |
| *** cinerama has quit IRC | 19:17 | |
| *** lennyb has quit IRC | 19:17 | |
| *** anteaya has quit IRC | 19:17 | |
| *** sambetts|afk has quit IRC | 19:17 | |
| *** stevebaker has quit IRC | 19:17 | |
| *** Guest40699 has quit IRC | 19:17 | |
| *** lekha has quit IRC | 19:17 | |
| *** agordeev has quit IRC | 19:17 | |
| *** BadCub has quit IRC | 19:17 | |
| *** yonglihe has quit IRC | 19:17 | |
| *** Mr_T has quit IRC | 19:17 | |
| *** jhesketh has quit IRC | 19:17 | |
| *** ssl_ has quit IRC | 19:17 | |
| *** sahil has quit IRC | 19:17 | |
| *** dtantsur|afk has quit IRC | 19:17 | |
| *** mgoddard has quit IRC | 19:17 | |
| *** purp has quit IRC | 19:17 | |
| *** rloo has quit IRC | 19:17 | |
| *** causten has quit IRC | 19:17 | |
| *** vinsh has quit IRC | 19:17 | |
| *** mgould|afk has quit IRC | 19:17 | |
| *** zhugaoxiao has quit IRC | 19:17 | |
| *** melwitt has quit IRC | 19:17 | |
| *** hwoarang has quit IRC | 19:17 | |
| *** mgkwill has quit IRC | 19:17 | |
| *** sergek has quit IRC | 19:17 | |
| *** betherly has quit IRC | 19:17 | |
| *** vdrok has quit IRC | 19:17 | |
| *** ildikov has quit IRC | 19:17 | |
| *** morgabra has quit IRC | 19:17 | |
| *** phschwartz has quit IRC | 19:17 | |
| *** UForgotten has quit IRC | 19:17 | |
| *** mikal has quit IRC | 19:17 | |
| *** mag009 has quit IRC | 19:17 | |
| *** dansmith has quit IRC | 19:17 | |
| *** harlowja has quit IRC | 19:17 | |
| *** chihhsin has quit IRC | 19:17 | |
| *** xavierr has quit IRC | 19:17 | |
| *** jmccrory has quit IRC | 19:17 | |
| *** rcernin has quit IRC | 19:17 | |
| *** mtreinish has quit IRC | 19:17 | |
| *** syed_ has quit IRC | 19:17 | |
| *** AlexeyAbashkin has quit IRC | 19:17 | |
| *** jistr has quit IRC | 19:17 | |
| *** amit213 has quit IRC | 19:17 | |
| *** yibo has quit IRC | 19:17 | |
| *** evgenyl has quit IRC | 19:17 | |
| *** mariojv has quit IRC | 19:17 | |
| *** romcheg has quit IRC | 19:17 | |
| *** fungi has quit IRC | 19:17 | |
| *** kun_huang has quit IRC | 19:17 | |
| *** cppforlife_ has quit IRC | 19:17 | |
| *** adnavare has quit IRC | 19:17 | |
| *** kirankv has quit IRC | 19:17 | |
| *** ishant|away has quit IRC | 19:17 | |
| *** lazy_prince has quit IRC | 19:17 | |
| *** slagle has quit IRC | 19:17 | |
| *** jcook has quit IRC | 19:18 | |
| *** eglute has quit IRC | 19:18 | |
| *** alineb has quit IRC | 19:18 | |
| *** gmmaha has quit IRC | 19:18 | |
| *** rbrud has quit IRC | 19:18 | |
| *** eset has quit IRC | 19:18 | |
| *** wajdi has quit IRC | 19:18 | |
| *** wznoinsk has quit IRC | 19:18 | |
| *** Goneri has quit IRC | 19:18 | |
| *** zzzeek has quit IRC | 19:18 | |
| *** chlong_ has quit IRC | 19:18 | |
| *** igordcard has quit IRC | 19:18 | |
| *** skazi has quit IRC | 19:18 | |
| *** bapalm has quit IRC | 19:18 | |
| *** hecliunyx has quit IRC | 19:18 | |
| *** zhenguo has quit IRC | 19:18 | |
| *** dims has quit IRC | 19:18 | |
| *** wendar has quit IRC | 19:18 | |
| *** mmedvede has quit IRC | 19:18 | |
| *** gabor_antal_ has quit IRC | 19:18 | |
| *** mat128|mtg has quit IRC | 19:18 | |
| *** rook has quit IRC | 19:18 | |
| *** stevemar has quit IRC | 19:18 | |
| *** bradjones has quit IRC | 19:18 | |
| *** jrist has quit IRC | 19:18 | |
| *** pshige______ has quit IRC | 19:18 | |
| *** sirushti has quit IRC | 19:18 | |
| *** xhku has quit IRC | 19:18 | |
| *** mrda has quit IRC | 19:18 | |
| *** tonyb has quit IRC | 19:18 | |
| *** lukl_ has quit IRC | 19:18 | |
| *** pcaruana has quit IRC | 19:18 | |
| *** mkam has quit IRC | 19:18 | |
| *** jheroux has quit IRC | 19:18 | |
| *** jjohnson2 has quit IRC | 19:18 | |
| *** bcornec has quit IRC | 19:18 | |
| *** gabriel-bezerra has quit IRC | 19:18 | |
| *** Ng has quit IRC | 19:18 | |
| *** izaakk has quit IRC | 19:18 | |
| *** TheJulia has quit IRC | 19:18 | |
| *** kragniz has quit IRC | 19:18 | |
| *** fyxim has quit IRC | 19:18 | |
| *** NobodyCam has quit IRC | 19:18 | |
| *** crinkle has quit IRC | 19:18 | |
| *** greghaynes has quit IRC | 19:18 | |
| *** johnthetubaguy has quit IRC | 19:18 | |
| *** KrisK19 has quit IRC | 19:18 | |
| *** fragatina has quit IRC | 19:18 | |
| *** joprovost has quit IRC | 19:18 | |
| *** openstackgerrit has quit IRC | 19:18 | |
| *** vgadiraj has quit IRC | 19:18 | |
| *** jvgrant has quit IRC | 19:18 | |
| *** lifeless has quit IRC | 19:18 | |
| *** sturivnyi_ has quit IRC | 19:18 | |
| *** mrtenio has quit IRC | 19:18 | |
| *** ricardoas has quit IRC | 19:18 | |
| *** rybridges1 has quit IRC | 19:18 | |
| *** marios has quit IRC | 19:18 | |
| *** jroll has quit IRC | 19:18 | |
| *** anish has quit IRC | 19:18 | |
| *** EmilienM has quit IRC | 19:18 | |
| *** marlinc has quit IRC | 19:18 | |
| *** hecliunyx has joined #openstack-ironic | 19:18 | |
| *** tonyb has joined #openstack-ironic | 19:21 | |
| *** lukl has joined #openstack-ironic | 19:22 | |
| *** patchbot has joined #openstack-ironic | 19:22 | |
| *** zzzeek has joined #openstack-ironic | 19:22 | |
| *** johnthetubaguy has joined #openstack-ironic | 19:22 | |
| *** skazi has joined #openstack-ironic | 19:22 | |
| *** Sukhdev has joined #openstack-ironic | 19:22 | |
| *** trown has joined #openstack-ironic | 19:22 | |
| *** sinh_ has joined #openstack-ironic | 19:22 | |
| *** lynxman has joined #openstack-ironic | 19:22 | |
| *** thrash has joined #openstack-ironic | 19:22 | |
| *** alex_xu has joined #openstack-ironic | 19:22 | |
| *** aarefiev has joined #openstack-ironic | 19:22 | |
| *** htruta has joined #openstack-ironic | 19:22 | |
| *** yarkot1 has joined #openstack-ironic | 19:22 | |
| *** jlvillal has joined #openstack-ironic | 19:22 | |
| *** Nakato has joined #openstack-ironic | 19:22 | |
| *** aignatov has joined #openstack-ironic | 19:22 | |
| *** Fdaisuke has joined #openstack-ironic | 19:22 | |
| *** persia has joined #openstack-ironic | 19:22 | |
| *** Daviey_ has joined #openstack-ironic | 19:22 | |
| *** ppiela has joined #openstack-ironic | 19:22 | |
| *** anush_ has joined #openstack-ironic | 19:22 | |
| *** rpioso has joined #openstack-ironic | 19:22 | |
| *** rbudden has joined #openstack-ironic | 19:22 | |
| *** dtantsur has joined #openstack-ironic | 19:22 | |
| *** mkam has joined #openstack-ironic | 19:22 | |
| *** chlong_ has joined #openstack-ironic | 19:22 | |
| *** bnemec has joined #openstack-ironic | 19:22 | |
| *** sirushti has joined #openstack-ironic | 19:22 | |
| *** pcaruana has joined #openstack-ironic | 19:22 | |
| *** kragniz has joined #openstack-ironic | 19:22 | |
| *** UForgotten has joined #openstack-ironic | 19:22 | |
| *** phschwartz has joined #openstack-ironic | 19:22 | |
| *** morgabra has joined #openstack-ironic | 19:22 | |
| *** vdrok has joined #openstack-ironic | 19:22 | |
| *** betherly has joined #openstack-ironic | 19:22 | |
| *** sergek has joined #openstack-ironic | 19:22 | |
| *** hwoarang has joined #openstack-ironic | 19:22 | |
| *** melwitt has joined #openstack-ironic | 19:22 | |
| *** zhugaoxiao has joined #openstack-ironic | 19:22 | |
| *** mgould|afk has joined #openstack-ironic | 19:22 | |
| *** causten has joined #openstack-ironic | 19:22 | |
| *** vinsh has joined #openstack-ironic | 19:22 | |
| *** rloo has joined #openstack-ironic | 19:22 | |
| *** purp has joined #openstack-ironic | 19:22 | |
| *** mgoddard has joined #openstack-ironic | 19:22 | |
| *** xhku_ has joined #openstack-ironic | 19:22 | |
| *** yarkot has joined #openstack-ironic | 19:22 | |
| *** Goneri has joined #openstack-ironic | 19:22 | |
| *** gabriel-bezerra has joined #openstack-ironic | 19:22 | |
| *** cppforlife_ has joined #openstack-ironic | 19:22 | |
| *** kun_huang has joined #openstack-ironic | 19:22 | |
| *** fungi has joined #openstack-ironic | 19:22 | |
| *** romcheg has joined #openstack-ironic | 19:22 | |
| *** mariojv has joined #openstack-ironic | 19:22 | |
| *** yibo has joined #openstack-ironic | 19:22 | |
| *** jistr has joined #openstack-ironic | 19:22 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 19:22 | |
| *** syed_ has joined #openstack-ironic | 19:22 | |
| *** mtreinish has joined #openstack-ironic | 19:22 | |
| *** rcernin has joined #openstack-ironic | 19:22 | |
| *** wajdi has joined #openstack-ironic | 19:22 | |
| *** jrist has joined #openstack-ironic | 19:22 | |
| *** hj-hpe has joined #openstack-ironic | 19:22 | |
| *** gabor_antal has joined #openstack-ironic | 19:22 | |
| *** david-lyle has joined #openstack-ironic | 19:22 | |
| *** Mr_T has joined #openstack-ironic | 19:22 | |
| *** rbrud has joined #openstack-ironic | 19:22 | |
| *** ianw has joined #openstack-ironic | 19:22 | |
| *** wendar has joined #openstack-ironic | 19:22 | |
| *** clenimar has joined #openstack-ironic | 19:22 | |
| *** baoli_ has joined #openstack-ironic | 19:22 | |
| *** ccamacho|afk has joined #openstack-ironic | 19:22 | |
| *** zigo has joined #openstack-ironic | 19:22 | |
| *** vsaienk0 has joined #openstack-ironic | 19:22 | |
| *** yuanying has joined #openstack-ironic | 19:22 | |
| *** notmorgan has joined #openstack-ironic | 19:22 | |
| *** hamzy has joined #openstack-ironic | 19:22 | |
| *** HenryG has joined #openstack-ironic | 19:22 | |
| *** tdurakov has joined #openstack-ironic | 19:22 | |
| *** kozhukalov has joined #openstack-ironic | 19:22 | |
| *** boris-42 has joined #openstack-ironic | 19:22 | |
| *** natorious has joined #openstack-ironic | 19:22 | |
| *** dstepanenko_ has joined #openstack-ironic | 19:22 | |
| *** adreznec has joined #openstack-ironic | 19:22 | |
| *** mgagne has joined #openstack-ironic | 19:22 | |
| *** amitkqed has joined #openstack-ironic | 19:22 | |
| *** wznoinsk has joined #openstack-ironic | 19:22 | |
| *** stevemar_ has joined #openstack-ironic | 19:22 | |
| *** adnavare has joined #openstack-ironic | 19:22 | |
| *** ishant|away has joined #openstack-ironic | 19:22 | |
| *** kirankv has joined #openstack-ironic | 19:22 | |
| *** lazy_prince has joined #openstack-ironic | 19:22 | |
| *** slagle has joined #openstack-ironic | 19:22 | |
| *** jcook has joined #openstack-ironic | 19:22 | |
| *** eglute has joined #openstack-ironic | 19:22 | |
| *** alineb has joined #openstack-ironic | 19:22 | |
| *** gmmaha has joined #openstack-ironic | 19:22 | |
| *** bcornec has joined #openstack-ironic | 19:22 | |
| *** bapalm has joined #openstack-ironic | 19:22 | |
| *** crinkle has joined #openstack-ironic | 19:22 | |
| *** mat128 has joined #openstack-ironic | 19:22 | |
| *** jjohnson2 has joined #openstack-ironic | 19:22 | |
| *** mrda has joined #openstack-ironic | 19:22 | |
| *** jhesketh has joined #openstack-ironic | 19:22 | |
| *** penick has joined #openstack-ironic | 19:22 | |
| *** MattMan has joined #openstack-ironic | 19:22 | |
| *** davidlenwell has joined #openstack-ironic | 19:22 | |
| *** pas-ha has joined #openstack-ironic | 19:22 | |
| *** anush has joined #openstack-ironic | 19:22 | |
| *** rm_work has joined #openstack-ironic | 19:22 | |
| *** hogepodge has joined #openstack-ironic | 19:22 | |
| *** clif_h has joined #openstack-ironic | 19:22 | |
| *** patrickeast has joined #openstack-ironic | 19:22 | |
| *** rbradfor has joined #openstack-ironic | 19:22 | |
| *** adam_g has joined #openstack-ironic | 19:22 | |
| *** francoblanco has joined #openstack-ironic | 19:22 | |
| *** JayF has joined #openstack-ironic | 19:22 | |
| *** yuriyz has joined #openstack-ironic | 19:22 | |
| *** dhellmann has joined #openstack-ironic | 19:22 | |
| *** pleia2 has joined #openstack-ironic | 19:22 | |
| *** odyssey4me has joined #openstack-ironic | 19:22 | |
| *** alaski has joined #openstack-ironic | 19:22 | |
| *** Guest66666 has joined #openstack-ironic | 19:22 | |
| *** yhvh has joined #openstack-ironic | 19:22 | |
| *** joemonk has joined #openstack-ironic | 19:22 | |
| *** ionutbalutoiu has joined #openstack-ironic | 19:22 | |
| *** SpamapS has joined #openstack-ironic | 19:22 | |
| *** logan- has joined #openstack-ironic | 19:22 | |
| *** devananda has joined #openstack-ironic | 19:22 | |
| *** rook has joined #openstack-ironic | 19:23 | |
| *** igordcard has joined #openstack-ironic | 19:23 | |
| *** jheroux has joined #openstack-ironic | 19:23 | |
| *** cmyster has joined #openstack-ironic | 19:23 | |
| *** KrisK19 has joined #openstack-ironic | 19:23 | |
| *** fragatina has joined #openstack-ironic | 19:23 | |
| *** joprovost has joined #openstack-ironic | 19:23 | |
| *** openstackgerrit has joined #openstack-ironic | 19:23 | |
| *** vgadiraj has joined #openstack-ironic | 19:23 | |
| *** jvgrant has joined #openstack-ironic | 19:23 | |
| *** lifeless has joined #openstack-ironic | 19:23 | |
| *** sturivnyi_ has joined #openstack-ironic | 19:23 | |
| *** ricardoas has joined #openstack-ironic | 19:23 | |
| *** mrtenio has joined #openstack-ironic | 19:23 | |
| *** rybridges1 has joined #openstack-ironic | 19:23 | |
| *** anish has joined #openstack-ironic | 19:23 | |
| *** marios has joined #openstack-ironic | 19:23 | |
| *** jroll has joined #openstack-ironic | 19:23 | |
| *** EmilienM has joined #openstack-ironic | 19:23 | |
| *** hoonetorg has joined #openstack-ironic | 19:23 | |
| *** dims has joined #openstack-ironic | 19:23 | |
| *** bradjones has joined #openstack-ironic | 19:23 | |
| *** mikal has joined #openstack-ironic | 19:23 | |
| *** mag009 has joined #openstack-ironic | 19:23 | |
| *** dansmith has joined #openstack-ironic | 19:23 | |
| *** aslezil has joined #openstack-ironic | 19:23 | |
| *** d0ugal has joined #openstack-ironic | 19:23 | |
| *** lennyb has joined #openstack-ironic | 19:23 | |
| *** lucas-dinner has joined #openstack-ironic | 19:23 | |
| *** jralph_ has joined #openstack-ironic | 19:23 | |
| *** nicodemos has joined #openstack-ironic | 19:23 | |
| *** cinerama has joined #openstack-ironic | 19:23 | |
| *** anteaya has joined #openstack-ironic | 19:23 | |
| *** sambetts|afk has joined #openstack-ironic | 19:23 | |
| *** stevebaker has joined #openstack-ironic | 19:23 | |
| *** Guest40699 has joined #openstack-ironic | 19:23 | |
| *** hoangcx has joined #openstack-ironic | 19:24 | |
| *** ptx0 has joined #openstack-ironic | 19:24 | |
| *** mmedvede has joined #openstack-ironic | 19:24 | |
| *** greghaynes has joined #openstack-ironic | 19:24 | |
| *** pshige______ has joined #openstack-ironic | 19:24 | |
| *** harlowja has joined #openstack-ironic | 19:24 | |
| *** chihhsin has joined #openstack-ironic | 19:24 | |
| *** xavierr has joined #openstack-ironic | 19:24 | |
| *** jmccrory has joined #openstack-ironic | 19:24 | |
| *** dtantsur has quit IRC | 19:24 | |
| *** dtantsur has joined #openstack-ironic | 19:24 | |
| *** kozhukalov has quit IRC | 19:24 | |
| *** boris-42 has quit IRC | 19:24 | |
| *** trown has quit IRC | 19:24 | |
| *** trown has joined #openstack-ironic | 19:24 | |
| *** rook has quit IRC | 19:24 | |
| *** rook has joined #openstack-ironic | 19:24 | |
| *** cmyster has quit IRC | 19:24 | |
| *** cmyster has joined #openstack-ironic | 19:24 | |
| *** jheroux has quit IRC | 19:24 | |
| *** jheroux has joined #openstack-ironic | 19:24 | |
| *** patrickeast has quit IRC | 19:24 | |
| *** ptx0 has quit IRC | 19:24 | |
| *** rook is now known as Guest46013 | 19:24 | |
| *** ptx0_ has joined #openstack-ironic | 19:24 | |
| *** bradjones has quit IRC | 19:24 | |
| *** bradjones has joined #openstack-ironic | 19:24 | |
| *** marlinc has joined #openstack-ironic | 19:24 | |
| *** cmyster is now known as Guest90031 | 19:24 | |
| *** eset has joined #openstack-ironic | 19:24 | |
| *** eset has joined #openstack-ironic | 19:24 | |
| *** anush__ has joined #openstack-ironic | 19:24 | |
| *** hecliunyx has quit IRC | 19:24 | |
| *** ChubYann has joined #openstack-ironic | 19:26 | |
| *** TheJulia has joined #openstack-ironic | 19:28 | |
| KrisK19 | anyone had expiriance with Ironic and disks like 6TB with GPT | 19:29 |
| rloo | JayF, mariojv: Ok, let me think about the power_sync. | 19:30 |
| openstackgerrit | Ricardo Araújo Santos proposed openstack/ironic: Fixes parameters validation in SSH power manager. https://review.openstack.org/366150 | 19:31 |
| *** fyxim has joined #openstack-ironic | 19:33 | |
| *** bfournie has quit IRC | 19:34 | |
| *** bfournie has joined #openstack-ironic | 19:34 | |
| *** bfournie has quit IRC | 19:34 | |
| *** KrisK19 has quit IRC | 19:35 | |
| openstackgerrit | Ricardo Araújo Santos proposed openstack/ironic: Fixes parameters validation in SSH power manager. https://review.openstack.org/366150 | 19:36 |
| *** NobodyCam has joined #openstack-ironic | 19:41 | |
| openstackgerrit | Merged openstack/ironic-specs: Update document to use Newton, Mitaka as the release name https://review.openstack.org/368545 | 19:41 |
| *** jrist has quit IRC | 19:43 | |
| *** jrist has joined #openstack-ironic | 19:43 | |
| *** hecliunyx has joined #openstack-ironic | 19:44 | |
| JayF | https://review.openstack.org/#/c/366828 anyone wanna look at this before I land it? | 19:45 |
| patchbot | patch 366828 - ironic - Abort cleaning and deployment if node is in mainte... | 19:45 |
| *** lekha has joined #openstack-ironic | 19:46 | |
| *** ijw has joined #openstack-ironic | 19:46 | |
| *** ildikov has joined #openstack-ironic | 19:47 | |
| *** yonglihe has joined #openstack-ironic | 19:47 | |
| *** clenimar_ has joined #openstack-ironic | 19:49 | |
| *** hoangcx has quit IRC | 19:49 | |
| *** hoangcx has joined #openstack-ironic | 19:50 | |
| *** agordeev has joined #openstack-ironic | 19:53 | |
| *** serverascode has joined #openstack-ironic | 19:53 | |
| *** ijw has quit IRC | 19:54 | |
| *** ijw has joined #openstack-ironic | 19:58 | |
| *** ijw has quit IRC | 19:58 | |
| *** ijw has joined #openstack-ironic | 19:58 | |
| *** amit213 has joined #openstack-ironic | 19:59 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Add config option to specify maximum size for configdrive https://review.openstack.org/369617 | 20:01 |
| *** hoangcx has quit IRC | 20:02 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add a note about security groups in install guide https://review.openstack.org/369385 | 20:02 |
| *** BadCub has joined #openstack-ironic | 20:02 | |
| *** kozhukalov has joined #openstack-ironic | 20:02 | |
| *** evgenyl has joined #openstack-ironic | 20:02 | |
| openstackgerrit | Xavier proposed openstack/ironic: Add inspection feature for the OneView drivers https://review.openstack.org/367065 | 20:03 |
| *** mgkwill has joined #openstack-ironic | 20:03 | |
| *** zhenguo has joined #openstack-ironic | 20:05 | |
| *** mkam has quit IRC | 20:05 | |
| *** patrickeast has joined #openstack-ironic | 20:06 | |
| *** izaakk has joined #openstack-ironic | 20:07 | |
| xavierr | jroll, I don't know why but now that worked. I droped that two new drivers and I'm only create_if_enabled. | 20:07 |
| *** bfournie has joined #openstack-ironic | 20:07 | |
| xavierr | jroll: https://review.openstack.org/#/c/367065/4/ironic/drivers/oneview.py | 20:07 |
| patchbot | patch 367065 - ironic - Add inspection feature for the OneView drivers | 20:08 |
| jroll | \o/ thanks xavierr | 20:08 |
| xavierr | I don't know what I was doing wrong... :( | 20:08 |
| xavierr | ty jroll!! | 20:08 |
| jroll | np :) | 20:10 |
| *** pcaruana has quit IRC | 20:11 | |
| xavierr | jroll, are you too busy to a review? if so, np.. :) | 20:11 |
| jroll | xavierr: yes, sorry | 20:12 |
| xavierr | ok, thank you anyways ;) | 20:12 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Add config option to specify maximum size for configdrive https://review.openstack.org/369617 | 20:16 |
| *** mkam has joined #openstack-ironic | 20:18 | |
| *** boris-42 has joined #openstack-ironic | 20:18 | |
| *** hoangcx has joined #openstack-ironic | 20:19 | |
| *** hoangcx has quit IRC | 20:20 | |
| *** hoangcx has joined #openstack-ironic | 20:21 | |
| xavierr | jlvillal: anything missing in 369617? you -2 it... | 20:23 |
| jlvillal | xavierr: I had put a -2 on it (my own patch). | 20:24 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Mark untested drivers as unsupported https://review.openstack.org/369503 | 20:24 |
| jlvillal | xavierr: But I just removed that because I made it depend on the other patch. | 20:24 |
| jroll | that should be ready | 20:24 |
| *** jabroney has joined #openstack-ironic | 20:24 | |
| *** sahil has joined #openstack-ironic | 20:25 | |
| xavierr | jlvillal: oh, ok John :) | 20:26 |
| sahil | hello jroll, | 20:26 |
| sahil | are you here | 20:26 |
| *** sahil has quit IRC | 20:27 | |
| jlvillal | jroll: On the deprecated driver. Should the release note say something like: ... and therefore deprecated. Some or all of these drivers could be removed in the XXXXX cycle. ??? | 20:27 |
| jlvillal | Sorry if that has already been discussed | 20:27 |
| jroll | jlvillal: hasn't been discussed, I could update | 20:28 |
| jlvillal | jroll: Thanks. I'll leave a comment if you like. | 20:29 |
| JayF | If we don't say when they're going to be removed, does it still start the "timer" for deprecation? | 20:29 |
| jroll | I believe so? dunno | 20:29 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Mark untested drivers as unsupported https://review.openstack.org/369503 | 20:29 |
| jroll | jlvillal: ^ | 20:29 |
| * jlvillal thinks jroll is so fast! | 20:30 | |
| jroll | :D | 20:30 |
| *** baoli_ has quit IRC | 20:31 | |
| *** joprovost has quit IRC | 20:32 | |
| xavierr | I'm reviewing a patch for ssh driver, should I tell the owner to include in the realease note that the driver will be deprecated? | 20:33 |
| jroll | no, that's already in my release note | 20:33 |
| JayF | But that is a good questions | 20:33 |
| JayF | should cores continue to review/merge patches to deprecated drivers? | 20:33 |
| openstackgerrit | Ricardo Araújo Santos proposed openstack/ironic: Fixes parameters validation in SSH power manager. https://review.openstack.org/366150 | 20:34 |
| xavierr | ^^^ thats the patch a was saying btw :D | 20:34 |
| * xavierr thinks it was in the perfect time | 20:35 | |
| *** trown is now known as trown|outtypewww | 20:40 | |
| ricardoas | xavierr: afffff | 20:40 |
| xavierr | ricardoas: hi there | 20:41 |
| jroll | JayF: guess it depends on the patch | 20:41 |
| jroll | bug fixes are still nice :) | 20:41 |
| jroll | most will probably get picked up by ironic-staging-drivers | 20:41 |
| xavierr | jroll: is ironic-staging-drivers a tag on launchpad? | 20:43 |
| jroll | xavierr: it's a whole project | 20:43 |
| jroll | xavierr: it is *not* managed by the ironic team (but rather a few ironic contributors) https://github.com/openstack/ironic-staging-drivers | 20:44 |
| xavierr | jroll: ohhh, I heard about it... but I | 20:46 |
| xavierr | thought is was no real | 20:46 |
| xavierr | s/no/not/ | 20:47 |
| jroll | oh, it is very real | 20:47 |
| JayF | there are drivers in there already in use by ironic developers | 20:48 |
| xavierr | really good to know :) | 20:48 |
| *** Ng has joined #openstack-ironic | 20:56 | |
| *** clenimar_ has quit IRC | 20:57 | |
| *** harlowja has quit IRC | 21:04 | |
| *** shobhar_ has joined #openstack-ironic | 21:04 | |
| *** jheroux has quit IRC | 21:09 | |
| mrda | Morning all | 21:14 |
| xavierr | hey mrda o/ | 21:17 |
| mrda | o/ | 21:17 |
| *** bnemec has quit IRC | 21:20 | |
| *** anush__ has quit IRC | 21:21 | |
| *** trown|outtypewww has quit IRC | 21:21 | |
| *** shobhar_ has quit IRC | 21:21 | |
| *** anush__ has joined #openstack-ironic | 21:22 | |
| *** mmedvede has quit IRC | 21:22 | |
| *** trown has joined #openstack-ironic | 21:23 | |
| *** fragatin_ has joined #openstack-ironic | 21:24 | |
| *** bnemec has joined #openstack-ironic | 21:24 | |
| *** ptx0_ has quit IRC | 21:25 | |
| *** ptx0 has joined #openstack-ironic | 21:25 | |
| *** ptx0 has joined #openstack-ironic | 21:25 | |
| *** lucas-dinner has quit IRC | 21:26 | |
| NobodyCam | morning mrda | 21:27 |
| *** fragatina has quit IRC | 21:27 | |
| *** fragatin_ has quit IRC | 21:27 | |
| mrda | o/ | 21:27 |
| *** zhenguo has quit IRC | 21:27 | |
| *** kozhukalov has quit IRC | 21:27 | |
| *** BadCub has quit IRC | 21:27 | |
| *** amit213 has quit IRC | 21:27 | |
| *** TheJulia has quit IRC | 21:27 | |
| *** pshige______ has quit IRC | 21:27 | |
| *** greghaynes has quit IRC | 21:27 | |
| *** chihhsin has quit IRC | 21:27 | |
| *** xavierr has quit IRC | 21:28 | |
| *** jmccrory has quit IRC | 21:28 | |
| *** jmccrory_ has joined #openstack-ironic | 21:28 | |
| *** chihhsin has joined #openstack-ironic | 21:28 | |
| *** greghaynes has joined #openstack-ironic | 21:28 | |
| *** xavierr has joined #openstack-ironic | 21:28 | |
| *** dhellmann has quit IRC | 21:28 | |
| *** pshige______ has joined #openstack-ironic | 21:28 | |
| *** sdake_ has joined #openstack-ironic | 21:28 | |
| *** fragatina has joined #openstack-ironic | 21:29 | |
| *** jmccrory_ is now known as jmccrory | 21:29 | |
| *** lucasagomes has joined #openstack-ironic | 21:29 | |
| *** shobhar_ has joined #openstack-ironic | 21:29 | |
| *** dhellmann has joined #openstack-ironic | 21:30 | |
| *** sdake has quit IRC | 21:30 | |
| *** rama_y has joined #openstack-ironic | 21:31 | |
| *** TheJulia has joined #openstack-ironic | 21:33 | |
| *** fragatina has quit IRC | 21:33 | |
| *** hoangcx3 has joined #openstack-ironic | 21:33 | |
| *** BadCub has joined #openstack-ironic | 21:33 | |
| *** hoangcx has quit IRC | 21:33 | |
| *** kozhukalov has joined #openstack-ironic | 21:34 | |
| *** amit213 has joined #openstack-ironic | 21:34 | |
| *** lucasagomes has quit IRC | 21:34 | |
| *** zhenguo has joined #openstack-ironic | 21:35 | |
| *** lucasagomes has joined #openstack-ironic | 21:35 | |
| *** sdake_ is now known as sdake | 21:36 | |
| *** ijw has quit IRC | 21:37 | |
| *** hoangcx3 has quit IRC | 21:38 | |
| openstackgerrit | Xavier proposed openstack/python-oneviewclient: Ensure values under validation have same data type https://review.openstack.org/369722 | 21:39 |
| *** fragatina has joined #openstack-ironic | 21:39 | |
| *** fragatina has quit IRC | 21:39 | |
| *** fragatina has joined #openstack-ironic | 21:40 | |
| *** lucasagomes has quit IRC | 21:40 | |
| *** lucasagomes has joined #openstack-ironic | 21:41 | |
| *** Guest46013 has quit IRC | 21:42 | |
| *** rook has joined #openstack-ironic | 21:42 | |
| *** rook is now known as Guest734 | 21:42 | |
| *** sai has quit IRC | 21:43 | |
| *** sai has joined #openstack-ironic | 21:43 | |
| *** fragatina has quit IRC | 21:45 | |
| *** hoangcx has joined #openstack-ironic | 21:45 | |
| jlvillal | Hi mrda | 21:45 |
| mrda | hey jlvillal | 21:45 |
| *** lucasagomes has quit IRC | 21:46 | |
| *** rama_y has quit IRC | 21:47 | |
| *** lucasagomes has joined #openstack-ironic | 21:48 | |
| *** ijw has joined #openstack-ironic | 21:51 | |
| *** ijw has quit IRC | 21:52 | |
| *** lucasagomes has quit IRC | 21:52 | |
| openstackgerrit | Merged openstack/ironic: Remove cyclic import between rpcapi and objects.base https://review.openstack.org/369513 | 21:53 |
| *** hoangcx has quit IRC | 21:53 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Separate WSGIService from RPCService https://review.openstack.org/369724 | 21:54 |
| *** hoangcx has joined #openstack-ironic | 21:55 | |
| *** fragatina has joined #openstack-ironic | 21:56 | |
| *** lucasagomes has joined #openstack-ironic | 21:57 | |
| *** fragatina has quit IRC | 21:57 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Add config option to specify maximum size for configdrive https://review.openstack.org/369617 | 21:58 |
| *** ijw has joined #openstack-ironic | 21:58 | |
| *** ijw has quit IRC | 21:58 | |
| *** hoangcx has quit IRC | 22:00 | |
| *** wajdi_ has joined #openstack-ironic | 22:01 | |
| *** hoangcx has joined #openstack-ironic | 22:02 | |
| *** lucasagomes has quit IRC | 22:02 | |
| *** lucasagomes has joined #openstack-ironic | 22:02 | |
| *** harlowja has joined #openstack-ironic | 22:03 | |
| *** wajdi has quit IRC | 22:04 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Add config option to specify maximum size for configdrive https://review.openstack.org/369617 | 22:04 |
| *** wajdi_ has quit IRC | 22:05 | |
| *** mkam has quit IRC | 22:06 | |
| *** hoangcx has quit IRC | 22:06 | |
| *** lucasagomes has quit IRC | 22:07 | |
| *** hoangcx has joined #openstack-ironic | 22:08 | |
| *** lucasagomes has joined #openstack-ironic | 22:10 | |
| mat128 | JayF: about https://review.openstack.org/#/c/366329/ | 22:13 |
| patchbot | patch 366329 - ironic - Import Bare Metal service overview into install-guide | 22:13 |
| mat128 | JayF: do you think we should improve while importing or just import + fix as different commit? | 22:13 |
| JayF | mat128: I only pointed those out because they seemed particularly egregious | 22:14 |
| mat128 | JayF: I'm fine either way, and the former is probably less work than the latter | 22:14 |
| JayF | mat128: but I didn't realize it was a verbatim copy over | 22:14 |
| JayF | mat128: I'll change my vote | 22:14 |
| mat128 | besides formatting, it's verbatim | 22:14 |
| JayF | flipped my vote | 22:14 |
| mat128 | :) | 22:14 |
| mat128 | JayF: and about the .txt or ignored-rst | 22:14 |
| mat128 | the more I think about it the more I think they should be .rst in an ignored subfolder | 22:15 |
| mat128 | thoughts? | 22:15 |
| *** lucasagomes has quit IRC | 22:15 | |
| mat128 | that would go against Swift and other projects that switched to .txt, but would make more sense IMO | 22:15 |
| JayF | I mean, I sometimes feel like as a core I'm supposed to be opinionated about this stuff | 22:15 |
| JayF | but I really couldn't care less | 22:15 |
| mat128 | ok, no problem :) | 22:16 |
| JayF | about implementation of how to make the docs build ignore our included files | 22:16 |
| JayF | lol | 22:16 |
| mat128 | yeeah... lol | 22:16 |
| *** lucasagomes has joined #openstack-ironic | 22:16 | |
| JayF | the advantage of a folder would be syntax detection based on extension would work | 22:16 |
| mat128 | having a "couldn't care less about this little detail" is a valid opinion | 22:16 |
| JayF | there's no really disadvantage | 22:16 |
| JayF | but *shrug* | 22:16 |
| mat128 | yeah | 22:16 |
| mat128 | it used to be .rst in the "root" | 22:16 |
| mat128 | and the conf.py excluded_files kept growing | 22:16 |
| mat128 | my other idea was to simply ignore the toctree stuff, but couldn't figure out how | 22:17 |
| mat128 | because we're running sphinx in a "warnings are errors" mode | 22:17 |
| JayF | I think an include/*.rst is a good idea | 22:17 |
| mat128 | good | 22:17 |
| mat128 | will switch that | 22:17 |
| JayF | but if you can't get it to work on the first or second try | 22:17 |
| JayF | I'd give up on it | 22:17 |
| JayF | lol | 22:17 |
| mat128 | JayF: it's super easy | 22:17 |
| JayF | I always prefer working over nonworking, lol | 22:17 |
| mat128 | I went from 0 sphinx/rst knowledge to "I can almost remember the syntax for special macros" in a few days | 22:18 |
| mat128 | :)P | 22:18 |
| JayF | Dude, like my first year working on Ironic | 22:18 |
| JayF | I fixed all the doc warnings in IPA and Ironic | 22:18 |
| JayF | and learned way more than I ever wanted to know about how docs build | 22:18 |
| mat128 | haha | 22:18 |
| JayF | it's actually really sweet, but a lot of weird stuff to understand | 22:18 |
| mat128 | the other thing I learned | 22:18 |
| mat128 | about our particular doc builds | 22:18 |
| mat128 | when the install guide lives in the openstack-manuals project | 22:19 |
| mat128 | you can leverage the "only" macro | 22:19 |
| mat128 | so you can do like | 22:19 |
| mat128 | only deb: bleh | 22:19 |
| *** ijw has joined #openstack-ironic | 22:19 | |
| mat128 | so that would make the platform-dependent commands a little prettier | 22:19 |
| JayF | but you can't do it in the distributed build? | 22:19 |
| JayF | why not/ | 22:19 |
| mat128 | but it seems we lost that feature (because we arent building with tags) with the "install-guide in your own project" | 22:19 |
| mat128 | not saying it can't be done | 22:19 |
| mat128 | but the build scripts need to be adjusted | 22:20 |
| JayF | building with tags? | 22:20 |
| mat128 | to pass a tag | 22:20 |
| mat128 | and probably make 3 different builds | 22:20 |
| mat128 | right now it's one build with 3+ pages | 22:20 |
| *** fragatina has joined #openstack-ironic | 22:20 | |
| *** lucasagomes has quit IRC | 22:21 | |
| mat128 | JayF: https://review.openstack.org/#/c/369510/ < found that "missing" file, it was configure-integrations.txt | 22:21 |
| patchbot | patch 369510 - ironic - Import "Configure integration with other OpenStack... | 22:21 |
| mat128 | JayF: already part of the changeset | 22:22 |
| JayF | mat128: I'm saying, don't you need a *.rst file in there too? | 22:23 |
| JayF | mat128: to use the include files you amde | 22:23 |
| JayF | you add lots of .txt files to include somewhere | 22:23 |
| JayF | but don't include them anywhere | 22:23 |
| mat128 | configure-integration.txt now includes them | 22:23 |
| JayF | so that patchset appears invalid | 22:23 |
| JayF | configure-integration.txt is included from other places, though? | 22:23 |
| *** lucasagomes has joined #openstack-ironic | 22:23 | |
| mat128 | from a previou commit, yes | 22:23 |
| JayF | ah, nested includes, fun | 22:23 |
| mat128 | it used to tell the reader to go to our previous page | 22:23 |
| mat128 | yeah | 22:23 |
| mat128 | well | 22:24 |
| *** ijw has quit IRC | 22:24 | |
| JayF | mat128: can you lmk an "area" of the install guide you're not working on | 22:24 |
| JayF | mat128: so I can push a patch or two too? | 22:24 |
| mat128 | seeing our current 2-3k lines files | 22:24 |
| mat128 | I'd rather have smaller files covering only 1 thing | 22:24 |
| mat128 | all I have done so far is pushed | 22:24 |
| mat128 | so, Image requirements and later isn't converted | 22:24 |
| mat128 | but I was thinking we should re-organize it | 22:24 |
| mat128 | http://docs.openstack.org/developer/ironic/deploy/install-guide.html | 22:24 |
| mat128 | right now we have | 22:24 |
| *** fragatina has quit IRC | 22:24 | |
| mat128 | Installing / configuring Ironic | 22:24 |
| mat128 | and configuring integration with other openstack services | 22:25 |
| mat128 | perhaps we should have a section like configuring optional features? | 22:25 |
| JayF | I kinda agree with you | 22:25 |
| mat128 | that would cover special drivers and other things? | 22:25 |
| mat128 | and optional "infra" configs could be done together | 22:26 |
| mat128 | like running -api under mod_wsgi | 22:26 |
| JayF | I mean, there's kinda a weird thing here, right? | 22:26 |
| mat128 | and https everything | 22:26 |
| mat128 | yes | 22:26 |
| JayF | I'm thinking the key example of what you're talkign about | 22:26 |
| JayF | is the drivers, right? | 22:26 |
| JayF | I think the driver doc stuff needs to be in the install guide | 22:26 |
| JayF | mainly because then it gets versioned for "free" based on release | 22:26 |
| mat128 | the drivers should all be almost equal | 22:26 |
| mat128 | so our guide should be more like | 22:26 |
| JayF | (/draft/ is master, /newton/ will be built from stable/newton) | 22:26 |
| mat128 | yes | 22:26 |
| mat128 | Imagine I have only heard of Ironic | 22:27 |
| mat128 | and I heard great things, I want to try it | 22:27 |
| mat128 | see what it does | 22:27 |
| mat128 | I probably have hardware already | 22:27 |
| mat128 | and I'd like to try it | 22:27 |
| mat128 | running the first few steps can lead me to an empty ironic node-list | 22:27 |
| mat128 | but what do I do next? How do I choose a driver? | 22:27 |
| JayF | I'd honestly save that thinking for ocata | 22:27 |
| mat128 | oh | 22:27 |
| mat128 | ok then :) | 22:27 |
| JayF | mainly because we are on a time crunch | 22:28 |
| mat128 | yeah | 22:28 |
| JayF | to get an install guide that makes sense before newton is branched | 22:28 |
| mat128 | yeah | 22:28 |
| mat128 | ok | 22:28 |
| mat128 | config drive | 22:28 |
| patchbot | Error: 'supybot.drive' is not a valid configuration variable. | 22:28 |
| mat128 | http://docs.openstack.org/developer/ironic/deploy/install-guide.html#enabling-the-configuration-drive-configdrive | 22:28 |
| JayF | and ocata will hopefully have the hardware type changes (driver comp) | 22:28 |
| mat128 | yeah | 22:28 |
| JayF | which makes that a LOT more reasonable to understand | 22:28 |
| mat128 | agreed | 22:28 |
| *** lucasagomes has quit IRC | 22:28 | |
| mat128 | theres a paragraph down there about building/downloading a deploy ramdisk | 22:29 |
| *** lucasagomes has joined #openstack-ironic | 22:29 | |
| JayF | I mean, to put this in perspective | 22:29 |
| JayF | jroll: was suggesting earlier we forklift faster | 22:30 |
| mat128 | so almost verbatim import? | 22:30 |
| JayF | maybe that's what we should do? get as much content moved as possible, even if it's a little jumbled | 22:30 |
| JayF | then work to fix it from there | 22:30 |
| mat128 | ok | 22:30 |
| JayF | then we get an install guide at all for newton | 22:30 |
| JayF | and start on making it a *good* install guide for ocata | 22:30 |
| mat128 | sold :) | 22:31 |
| mat128 | so I'm at "Image requirements" | 22:32 |
| mat128 | wanna do it? | 22:32 |
| mat128 | I'll change the txt stuff | 22:32 |
| *** lucasagomes has quit IRC | 22:36 | |
| *** ijw has joined #openstack-ironic | 22:36 | |
| JayF | How about I migrate over all the driver shenanigans | 22:38 |
| JayF | that's pretty self-contained | 22:38 |
| JayF | and a large quantity of stuff | 22:38 |
| mat128 | JayF: good idea | 22:38 |
| mat128 | JayF: http://docs.openstack.org/developer/ironic/deploy/install-guide.html#setup-the-drivers-for-the-bare-metal-service | 22:38 |
| mat128 | good | 22:38 |
| *** ijw has quit IRC | 22:38 | |
| *** lucasagomes has joined #openstack-ironic | 22:38 | |
| *** ijw has joined #openstack-ironic | 22:39 | |
| JayF | I'll jump on that t'morrow | 22:39 |
| openstackgerrit | Xavier proposed openstack/ironic: OneView driver docs explaining hardware inspection https://review.openstack.org/369737 | 22:39 |
| *** ijw has quit IRC | 22:41 | |
| *** sai has quit IRC | 22:43 | |
| *** sai has joined #openstack-ironic | 22:44 | |
| xavierr | have a good night all | 22:45 |
| *** xavierr is now known as xavierr|afk | 22:45 | |
| mat128 | cya xavierr|afk | 22:45 |
| xavierr|afk | heey mat128 o/ | 22:46 |
| *** lucasagomes has quit IRC | 22:46 | |
| xavierr|afk | mat128: talk to you tomorrow :) | 22:47 |
| mat128 | yeah, have a good night | 22:47 |
| *** rbrud has quit IRC | 22:47 | |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Import install and configure into new install guide https://review.openstack.org/367988 | 22:47 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Import Bare Metal service overview into install-guide https://review.openstack.org/366329 | 22:47 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Import "Configure integration with other OpenStack components" https://review.openstack.org/369510 | 22:47 |
| * mat128 feels like Vasyl when this happens ^ | 22:48 | |
| mat128 | :) | 22:48 |
| *** lucasagomes has joined #openstack-ironic | 22:49 | |
| *** lucasagomes has quit IRC | 22:54 | |
| *** fragatina has joined #openstack-ironic | 22:55 | |
| *** lucasagomes has joined #openstack-ironic | 22:56 | |
| *** sai has quit IRC | 22:56 | |
| *** Goneri has quit IRC | 22:57 | |
| *** sdake_ has joined #openstack-ironic | 22:57 | |
| *** fragatina has quit IRC | 23:00 | |
| *** sai has joined #openstack-ironic | 23:00 | |
| *** sdake has quit IRC | 23:00 | |
| *** fragatina has joined #openstack-ironic | 23:03 | |
| *** lucasagomes has quit IRC | 23:05 | |
| *** lucasagomes has joined #openstack-ironic | 23:06 | |
| *** fragatina has quit IRC | 23:07 | |
| *** lucasagomes has quit IRC | 23:10 | |
| *** fragatina has joined #openstack-ironic | 23:10 | |
| *** lucasagomes has joined #openstack-ironic | 23:16 | |
| *** joprovost has joined #openstack-ironic | 23:17 | |
| *** joprovost has quit IRC | 23:19 | |
| *** Goneri has joined #openstack-ironic | 23:22 | |
| *** pmannidi has joined #openstack-ironic | 23:23 | |
| *** rcernin has quit IRC | 23:23 | |
| *** lucasagomes has quit IRC | 23:23 | |
| *** ijw has joined #openstack-ironic | 23:24 | |
| *** lucasagomes has joined #openstack-ironic | 23:27 | |
| *** rpioso has quit IRC | 23:28 | |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Import "Image requirements" https://review.openstack.org/369754 | 23:28 |
| *** rbudden has quit IRC | 23:31 | |
| *** lucasagomes has quit IRC | 23:31 | |
| *** Goneri has quit IRC | 23:31 | |
| *** lucasagomes has joined #openstack-ironic | 23:33 | |
| *** sai has quit IRC | 23:34 | |
| mat128 | JayF: pep8 and sphinx have their own way of evaluating path when including | 23:35 |
| mat128 | tl;dr, including from an included file can't pass both checks | 23:35 |
| JayF | mat128: can you somehow do a more absolute path? | 23:36 |
| JayF | mat128: or can we fix pep8 somehow? | 23:36 |
| mat128 | well, thinking about it and your previous comment re: missing a file / include | 23:36 |
| mat128 | It's probably a bad idea to have nested includes | 23:36 |
| mat128 | for clarity | 23:36 |
| JayF | I'd buy that | 23:36 |
| mat128 | so i'm making them all "top level" with only root .rst including files from include | 23:36 |
| JayF | I sorta thought that at first | 23:36 |
| JayF | but looked at it and thought it was useful enough | 23:37 |
| mat128 | well | 23:37 |
| mat128 | it is, but I don't want to mess too much with this | 23:37 |
| mat128 | and it's less obvious when you need a diagram to find out where your file ends up | 23:37 |
| mat128 | ;) | 23:37 |
| JayF | I feel like there's a fine line between "we should implement it another way" and bikeshedding, aand sometimes I find as a core most folks will just do what I say rather than defending themselves if they I'm wrong :( | 23:37 |
| mat128 | we can always refactor the files later | 23:37 |
| *** lucasagomes has quit IRC | 23:38 | |
| *** sai has joined #openstack-ironic | 23:38 | |
| mat128 | the "intimidated by the core member" phenomenon | 23:38 |
| mat128 | lol | 23:38 |
| *** ijw has quit IRC | 23:38 | |
| mat128 | contributors and cores have different values, the occasional contributor probably only wants his bug fixed | 23:39 |
| mat128 | where as the core will have to live with that code | 23:39 |
| *** ijw has joined #openstack-ironic | 23:39 | |
| mat128 | if the cores are too rigid, the project will not get much traction | 23:39 |
| mat128 | it's always abalance | 23:39 |
| JayF | I mean, I guess there was an <opinion></opinion> tag in gerrit | 23:39 |
| JayF | because sometimes I'm like "I would've done it this way" but don't have confidence it's better/worse/etc | 23:39 |
| *** lucasagomes has joined #openstack-ironic | 23:40 | |
| mat128 | JayF: in that case it's probably not a case of -2 either | 23:42 |
| *** hoangcx2 has joined #openstack-ironic | 23:44 | |
| *** hoangcx has quit IRC | 23:45 | |
| * mat128 is getting better at interactive rebasing | 23:47 | |
| *** shobhar_ has quit IRC | 23:47 | |
| JayF | I'm about to head out for today o/ see you good folks tomorrow | 23:53 |
| *** ijw has quit IRC | 23:54 | |
| mat128 | have a good one | 23:54 |
| mat128 | see you tmw | 23:54 |
| *** anush__ has quit IRC | 23:56 | |
| *** sai has quit IRC | 23:56 | |
| *** sai has joined #openstack-ironic | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!