| *** rpioso has quit IRC | 00:00 | |
| mrda | jlvillal: Donkey Kong Jnr was a much better game :) | 00:02 |
|---|---|---|
| NobodyCam | morning mrda | 00:02 |
| jlvillal | mrda: Morning! | 00:02 |
| mrda | hey NobodyCam and jlvillal and etc | 00:02 |
| NobodyCam | :) | 00:03 |
| * mrda has been stuck in a tangled web of meetings all morning | 00:03 | |
| NobodyCam | ++++ | 00:03 |
| *** aginwala has joined #openstack-ironic | 00:03 | |
| *** moshele has quit IRC | 00:12 | |
| *** raddaoui has joined #openstack-ironic | 00:26 | |
| *** raddaoui has quit IRC | 00:32 | |
| *** garthb__ has quit IRC | 00:35 | |
| *** Sukhdev has joined #openstack-ironic | 00:38 | |
| *** garthb__ has joined #openstack-ironic | 00:39 | |
| *** dims has quit IRC | 00:45 | |
| *** Sukhdev has quit IRC | 00:46 | |
| *** rajinir has quit IRC | 01:05 | |
| *** amotoki has joined #openstack-ironic | 01:09 | |
| *** aginwala has quit IRC | 01:12 | |
| *** jasondotstar has quit IRC | 01:24 | |
| *** dims has joined #openstack-ironic | 01:24 | |
| *** jasondotstar has joined #openstack-ironic | 01:29 | |
| *** hoangcx has joined #openstack-ironic | 01:29 | |
| *** aginwala has joined #openstack-ironic | 01:30 | |
| *** _zouyee has joined #openstack-ironic | 01:31 | |
| *** __zouyee has joined #openstack-ironic | 01:31 | |
| *** Sukhdev has joined #openstack-ironic | 01:32 | |
| *** garthb__ has quit IRC | 01:34 | |
| *** shuquan has joined #openstack-ironic | 01:36 | |
| *** aginwala has quit IRC | 01:44 | |
| *** aginwala has joined #openstack-ironic | 01:47 | |
| *** yonglihe has joined #openstack-ironic | 01:48 | |
| *** mtanino has quit IRC | 01:49 | |
| *** chenke_ has quit IRC | 01:54 | |
| *** kan_ has joined #openstack-ironic | 01:55 | |
| *** smoriya__ has joined #openstack-ironic | 01:57 | |
| *** shuquan has quit IRC | 01:57 | |
| *** Sukhdev has quit IRC | 01:58 | |
| *** smoriya_ has quit IRC | 01:59 | |
| *** shuquan has joined #openstack-ironic | 02:01 | |
| *** rloo has quit IRC | 02:08 | |
| *** piet has joined #openstack-ironic | 02:08 | |
| * jlvillal is shocked that he has got to the point of reproducing the grenade tempest failures locally. Shocked yet happy :) | 02:09 | |
| *** ijw has quit IRC | 02:12 | |
| *** hanchao has joined #openstack-ironic | 02:16 | |
| *** boris-42 has joined #openstack-ironic | 02:18 | |
| *** piet has quit IRC | 02:22 | |
| *** aginwala has quit IRC | 02:22 | |
| openstackgerrit | Haomeng,Wang proposed openstack/ironic-specs: Add node name regexp filter to API https://review.openstack.org/266688 | 02:24 |
| *** aginwala has joined #openstack-ironic | 02:27 | |
| *** dims has quit IRC | 02:29 | |
| *** Haomeng has quit IRC | 02:46 | |
| *** shuquan has quit IRC | 02:48 | |
| *** shuquan has joined #openstack-ironic | 02:51 | |
| *** davideag_ has joined #openstack-ironic | 02:51 | |
| *** crinkle_ has joined #openstack-ironic | 02:52 | |
| *** amotoki has quit IRC | 02:52 | |
| *** _fortis has quit IRC | 02:52 | |
| *** ekarlso has quit IRC | 02:52 | |
| *** davideagnello has quit IRC | 02:52 | |
| *** maurosr has quit IRC | 02:52 | |
| *** mikal has quit IRC | 02:52 | |
| *** ChubYann has quit IRC | 02:52 | |
| *** crinkle has quit IRC | 02:52 | |
| *** ChubYann has joined #openstack-ironic | 02:53 | |
| *** aarefiev has quit IRC | 02:53 | |
| *** vsaienko has quit IRC | 02:53 | |
| *** mikal has joined #openstack-ironic | 02:53 | |
| *** hoangcx has quit IRC | 02:53 | |
| *** aarefiev has joined #openstack-ironic | 02:53 | |
| *** amotoki has joined #openstack-ironic | 02:53 | |
| *** maurosr has joined #openstack-ironic | 02:53 | |
| *** vsaienko has joined #openstack-ironic | 02:54 | |
| *** shuquan has quit IRC | 02:55 | |
| *** hoangcx has joined #openstack-ironic | 02:58 | |
| *** shuquan has joined #openstack-ironic | 02:58 | |
| *** dims has joined #openstack-ironic | 02:58 | |
| *** dims_ has joined #openstack-ironic | 03:01 | |
| *** baoli has joined #openstack-ironic | 03:02 | |
| *** dims has quit IRC | 03:03 | |
| *** spandhe has quit IRC | 03:05 | |
| *** _fortis has joined #openstack-ironic | 03:05 | |
| *** raddaoui has joined #openstack-ironic | 03:05 | |
| *** ekarlso has joined #openstack-ironic | 03:06 | |
| *** crinkle_ is now known as crinkle | 03:08 | |
| *** aginwala has quit IRC | 03:09 | |
| *** raddaoui has quit IRC | 03:12 | |
| *** aginwala has joined #openstack-ironic | 03:14 | |
| *** aginwala has quit IRC | 03:16 | |
| *** links has joined #openstack-ironic | 03:17 | |
| *** shuquan has quit IRC | 03:22 | |
| *** harshs has quit IRC | 03:24 | |
| *** shuquan has joined #openstack-ironic | 03:25 | |
| *** daemontool has quit IRC | 03:30 | |
| *** yuanying_ has quit IRC | 03:34 | |
| *** coolsvap|away is now known as coolsvap | 03:36 | |
| *** jaybeale has quit IRC | 03:36 | |
| *** shuquan has quit IRC | 03:39 | |
| *** vinm213 has joined #openstack-ironic | 03:40 | |
| *** thrash is now known as thrash|g0ne | 03:41 | |
| *** vinm213 has quit IRC | 03:42 | |
| *** shuquan has joined #openstack-ironic | 03:42 | |
| *** yuanying has joined #openstack-ironic | 03:46 | |
| *** yuanying_ has joined #openstack-ironic | 03:56 | |
| *** yuanying has quit IRC | 03:59 | |
| *** yuanying_ has quit IRC | 04:00 | |
| *** yuanying has joined #openstack-ironic | 04:06 | |
| *** yuanying has quit IRC | 04:07 | |
| *** yuanying_ has joined #openstack-ironic | 04:07 | |
| *** raddaoui has joined #openstack-ironic | 04:12 | |
| *** spandhe has joined #openstack-ironic | 04:14 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic-lib: Config drive support for whole disk images for iscsi based deploy https://review.openstack.org/230924 | 04:28 |
| *** Marga_ has quit IRC | 04:28 | |
| *** dims_ has quit IRC | 04:30 | |
| *** Sukhdev has joined #openstack-ironic | 04:33 | |
| *** Marga_ has joined #openstack-ironic | 04:44 | |
| *** Marga_ has quit IRC | 04:48 | |
| *** alexpilotti has joined #openstack-ironic | 04:54 | |
| *** shuquan has quit IRC | 04:58 | |
| *** alexpilotti has quit IRC | 04:58 | |
| *** raddaoui has quit IRC | 05:01 | |
| openstackgerrit | vinay kumar muddu proposed openstack/ironic: Fix issues with uefi-ipxe booting https://review.openstack.org/265742 | 05:10 |
| *** pradipta has joined #openstack-ironic | 05:17 | |
| *** baoli has quit IRC | 05:22 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Update typos https://review.openstack.org/266691 | 05:25 |
| *** yolanda has quit IRC | 05:27 | |
| *** alexpilotti has joined #openstack-ironic | 05:30 | |
| *** stendulker has joined #openstack-ironic | 05:34 | |
| *** alexpilotti has quit IRC | 05:35 | |
| *** spandhe_ has joined #openstack-ironic | 05:42 | |
| *** harshs has joined #openstack-ironic | 05:43 | |
| *** spandhe has quit IRC | 05:43 | |
| *** spandhe_ is now known as spandhe | 05:43 | |
| *** moshele has joined #openstack-ironic | 05:50 | |
| *** shuquan has joined #openstack-ironic | 05:56 | |
| *** harshs has quit IRC | 06:08 | |
| *** shuquan has quit IRC | 06:08 | |
| *** shuquan has joined #openstack-ironic | 06:10 | |
| *** alexpilotti has joined #openstack-ironic | 06:11 | |
| openstackgerrit | caoyue proposed openstack/ironic: use oslo.utils.reflection extract the class name https://review.openstack.org/267324 | 06:11 |
| *** aginwala has joined #openstack-ironic | 06:14 | |
| *** alexpilotti has quit IRC | 06:15 | |
| openstackgerrit | Kan proposed openstack/ironic: Enable removing name when updating node https://review.openstack.org/267327 | 06:18 |
| *** Haomeng has joined #openstack-ironic | 06:20 | |
| *** aginwala_ has joined #openstack-ironic | 06:23 | |
| *** aginwala has quit IRC | 06:26 | |
| *** aginwala has joined #openstack-ironic | 06:27 | |
| *** aginwala_ has quit IRC | 06:30 | |
| *** ChubYann has quit IRC | 06:31 | |
| openstackgerrit | caoyue proposed openstack/ironic: Add debug testenv in tox https://review.openstack.org/266720 | 06:38 |
| *** deray has joined #openstack-ironic | 06:41 | |
| *** aginwala has quit IRC | 06:43 | |
| *** aginwala has joined #openstack-ironic | 06:43 | |
| *** kalpase has joined #openstack-ironic | 06:48 | |
| *** spandhe has quit IRC | 06:49 | |
| *** spandhe has joined #openstack-ironic | 06:50 | |
| *** spandhe has quit IRC | 06:50 | |
| *** spandhe has joined #openstack-ironic | 06:51 | |
| *** amotoki_ has joined #openstack-ironic | 06:52 | |
| *** amotoki_ has quit IRC | 06:52 | |
| *** hoangcx has quit IRC | 06:53 | |
| *** hoangcx has joined #openstack-ironic | 06:57 | |
| *** shuquan has quit IRC | 06:57 | |
| *** vishwana_ is now known as vishwanathj | 07:00 | |
| *** vishwanathj is now known as vishwanathj_good | 07:01 | |
| *** vishwanathj_good is now known as vishwanathj_slee | 07:01 | |
| *** vishwanathj_slee is now known as vishwanathj_nap | 07:01 | |
| *** vishwanathj_nap is now known as vishwanathj_away | 07:01 | |
| *** daemontool has joined #openstack-ironic | 07:14 | |
| *** shuquan has joined #openstack-ironic | 07:18 | |
| openstackgerrit | Vaishnavi proposed openstack/ironic: Addressing review comments https://review.openstack.org/267351 | 07:21 |
| *** daemontool has quit IRC | 07:22 | |
| *** ukalifon has joined #openstack-ironic | 07:24 | |
| *** erwan_taf has quit IRC | 07:24 | |
| *** ijw has joined #openstack-ironic | 07:33 | |
| openstackgerrit | Yuiko Takada proposed openstack/ironic: Migrate Tempest tests into Ironic tree https://review.openstack.org/253982 | 07:33 |
| *** aginwala has quit IRC | 07:34 | |
| openstackgerrit | Haomeng,Wang proposed openstack/python-ironicclient: continue to delete next node if failed with previous one https://review.openstack.org/262393 | 07:36 |
| *** spandhe has quit IRC | 07:39 | |
| *** kalpase has quit IRC | 07:47 | |
| *** Sukhdev has quit IRC | 07:49 | |
| *** yolanda has joined #openstack-ironic | 07:51 | |
| *** kalpase has joined #openstack-ironic | 07:55 | |
| openstackgerrit | Kan proposed openstack/ironic: Enable removing name when updating node https://review.openstack.org/267327 | 07:58 |
| openstackgerrit | Merged openstack/ironic: Clean up unneeded deprecated_group https://review.openstack.org/266752 | 08:00 |
| *** shuquan has quit IRC | 08:03 | |
| *** ionutbalutoiu has joined #openstack-ironic | 08:04 | |
| *** hoangcx has quit IRC | 08:05 | |
| *** hoangcx has joined #openstack-ironic | 08:06 | |
| *** shuquan has joined #openstack-ironic | 08:06 | |
| *** shuquan has joined #openstack-ironic | 08:09 | |
| *** shuquan has quit IRC | 08:22 | |
| *** shuquan has joined #openstack-ironic | 08:25 | |
| *** dnikishov has joined #openstack-ironic | 08:27 | |
| *** shuquan has quit IRC | 08:27 | |
| openstackgerrit | Vaishnavi proposed openstack/ironic: Add support for dib based agent ramdisk in lib/ironic https://review.openstack.org/264579 | 08:30 |
| openstackgerrit | Tan Lin proposed openstack/ironic: Refactor install-guide to configure API/Conductor seperately https://review.openstack.org/261443 | 08:30 |
| *** shuquan has joined #openstack-ironic | 08:32 | |
| mkovacik | harlowja, cool, you have any estimate/plan to implement this semantics? | 08:32 |
| *** shuquan has quit IRC | 08:35 | |
| *** shuquan has joined #openstack-ironic | 08:36 | |
| *** ijw has quit IRC | 08:40 | |
| openstackgerrit | Merged openstack/ironic-inspector: Track node identification during the whole processing https://review.openstack.org/265240 | 08:43 |
| *** coolsvap is now known as coolsvap|away | 08:44 | |
| *** mbound has joined #openstack-ironic | 08:47 | |
| *** e0ne has joined #openstack-ironic | 08:48 | |
| *** derekh has joined #openstack-ironic | 08:57 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Clarify that size in root device hints and local_gb are often different https://review.openstack.org/266442 | 08:59 |
| *** ifarkas has joined #openstack-ironic | 09:00 | |
| openstackgerrit | Tan Lin proposed openstack/ironic: Add xinetd and its TFTP configuration in Install Guide https://review.openstack.org/261429 | 09:01 |
| *** jistr has joined #openstack-ironic | 09:03 | |
| mkovacik | harlowja, there's however a limitation; the listener's high-availability; how to plumb it to the client without a queue | 09:06 |
| mkovacik | harlowja, I'll mention that in the pull-request reply | 09:07 |
| *** mkovacik has quit IRC | 09:08 | |
| *** UForgotten has quit IRC | 09:11 | |
| openstackgerrit | Vaishnavi proposed openstack/ironic: [IRONIC] Config variable to configure [glance] section https://review.openstack.org/266803 | 09:11 |
| *** ndipanov has joined #openstack-ironic | 09:12 | |
| *** ionutbalutoiu has quit IRC | 09:15 | |
| aarefiev | morning all! | 09:15 |
| *** shuquan has quit IRC | 09:15 | |
| *** ionutbalutoiu has joined #openstack-ironic | 09:15 | |
| *** shuquan has joined #openstack-ironic | 09:17 | |
| *** ig0r_ has joined #openstack-ironic | 09:17 | |
| *** shuquan has quit IRC | 09:17 | |
| *** jamielennox is now known as jamielennox|away | 09:18 | |
| *** shuquan has joined #openstack-ironic | 09:19 | |
| *** bigjools has quit IRC | 09:20 | |
| *** bigjools has joined #openstack-ironic | 09:21 | |
| *** daemontool has joined #openstack-ironic | 09:23 | |
| *** mgoddard has joined #openstack-ironic | 09:24 | |
| *** erwan_taf has joined #openstack-ironic | 09:25 | |
| *** Pespa has quit IRC | 09:33 | |
| *** __zouyee has quit IRC | 09:37 | |
| *** MattMan has joined #openstack-ironic | 09:38 | |
| *** mbound has quit IRC | 09:39 | |
| *** MattMan has joined #openstack-ironic | 09:40 | |
| *** MattMan has quit IRC | 09:40 | |
| *** MattMan has joined #openstack-ironic | 09:40 | |
| *** ijw has joined #openstack-ironic | 09:40 | |
| *** ig0r_ has quit IRC | 09:41 | |
| *** MattMan has quit IRC | 09:42 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Normalize the tag data structure https://review.openstack.org/267434 | 09:43 |
| *** ijw has quit IRC | 09:45 | |
| *** mbound has joined #openstack-ironic | 09:46 | |
| *** shuquan_ has joined #openstack-ironic | 09:47 | |
| *** shuquan has quit IRC | 09:48 | |
| *** shuquan_ is now known as shuquan | 09:48 | |
| openstackgerrit | Debayan Ray proposed openstack/ironic: Firmware update(iLO) as manual cleaning step https://review.openstack.org/211019 | 09:48 |
| *** mkovacik has joined #openstack-ironic | 09:55 | |
| *** MattMan has joined #openstack-ironic | 09:57 | |
| *** ijw has joined #openstack-ironic | 10:00 | |
| vdrok | morning ironic and aarefiev ! | 10:01 |
| aarefiev | vdrok: g'morning | 10:02 |
| *** pradipta has quit IRC | 10:03 | |
| *** _zouyee has quit IRC | 10:03 | |
| *** yolanda has quit IRC | 10:03 | |
| *** ijw has quit IRC | 10:05 | |
| *** shuquan has quit IRC | 10:07 | |
| *** _degorenko|afk is now known as degorenko | 10:07 | |
| *** yolanda has joined #openstack-ironic | 10:08 | |
| deray | late g'morning to Ironicers! .. o/ to vdrok aarefiev and all others | 10:11 |
| aarefiev | deray: morning | 10:12 |
| deray | aarefiev, its kinda evening for me now :) o/ again | 10:13 |
| vdrok | morning deray | 10:13 |
| vdrok | or evening then :) | 10:13 |
| deray | vdrok, aarefiev :) | 10:13 |
| aarefiev | deray: good evening so :) | 10:13 |
| *** athomas has quit IRC | 10:14 | |
| deray | I am planning to put a checksum functionality for my firmware update feature (iLO). | 10:15 |
| deray | Got to see this: https://github.com/openstack/ironic/blob/69c33f7ed5004afd4fd1589f1aed0e498845a952/ironic/common/utils.py#L393 in ironic but no any place where it is getting used. Shoudl I go ahead and use it? | 10:16 |
| openstackgerrit | Tan Lin proposed openstack/ironic: Add choices option to several options https://review.openstack.org/251732 | 10:16 |
| vdrok | deray, if it works for you, why not? :) | 10:18 |
| deray | vdrok, cool .. | 10:19 |
| *** sambetts has joined #openstack-ironic | 10:19 | |
| sambetts | Morning all o/ | 10:19 |
| deray | vdrok, others, is that a good option to pass on the checksum string as part of the manual cleaning step dictionary args? | 10:20 |
| deray | sambetts, g'morning | 10:20 |
| * deray just wondering if passing checksum string as an open string to the manual clean step is a security issue or not. | 10:22 | |
| sambetts | o/ deray | 10:22 |
| *** athomas has joined #openstack-ironic | 10:23 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Introducing API for aborting introspection https://review.openstack.org/265275 | 10:24 |
| *** hoangcx has quit IRC | 10:24 | |
| deray | sambetts, :) | 10:26 |
| vdrok | deray, image checksum is passed to agent in plain text - https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/extensions/standby.py#L168-L176 | 10:27 |
| vdrok | but i'm not sure if the firmware file is the same case | 10:27 |
| deray | vdrok, thx .. having a close look at it | 10:28 |
| *** teju has joined #openstack-ironic | 10:29 | |
| *** ionutbalutoiu has quit IRC | 10:33 | |
| *** electrofelix has joined #openstack-ironic | 10:34 | |
| *** ionutbalutoiu has joined #openstack-ironic | 10:34 | |
| deray | vdrok, gr8.. it should ideally work for firmware image type as well. Only issue I have with the algorithm (md5), may be its better if we use the sha-1. | 10:34 |
| deray | *have is with the algo | 10:34 |
| vdrok | deray, just propose what you think makes sense, people will review :) | 10:35 |
| deray | vdrok, whatsovever, nice to have a reference from ironic_python_agent | 10:36 |
| deray | vdrok, sure | 10:36 |
| openstackgerrit | Tan Lin proposed openstack/ironic: Add possible values for config options https://review.openstack.org/250132 | 10:36 |
| openstackgerrit | Tan Lin proposed openstack/ironic: Add possible values for config options https://review.openstack.org/250132 | 10:37 |
| *** electrofelix has quit IRC | 10:38 | |
| *** baoli has joined #openstack-ironic | 10:42 | |
| *** mgould has joined #openstack-ironic | 10:43 | |
| *** ionutbalutoiu has quit IRC | 10:45 | |
| *** ionutbalutoiu has joined #openstack-ironic | 10:45 | |
| mkovacik | guys, I could use some reviews on https://review.openstack.org/#/c/265275 ; should be ready to merge. Thanks a lot! | 10:47 |
| openstackgerrit | Moshe Levi proposed openstack/ironic-inspector: Adding InfiniBand Support https://review.openstack.org/264257 | 10:54 |
| mgould | morning Ironic! | 10:58 |
| mkovacik | mgould, o/ | 11:01 |
| vdrok | morning mkovacik mgould and sambetts | 11:01 |
| *** ijw has joined #openstack-ironic | 11:02 | |
| sambetts | o/ vdrok mkovacik mgould | 11:02 |
| mkovacik | o/ sambetts, vdrok :) | 11:02 |
| *** _zouyee has joined #openstack-ironic | 11:04 | |
| *** dims has joined #openstack-ironic | 11:04 | |
| *** mgould has quit IRC | 11:05 | |
| *** ijw has quit IRC | 11:07 | |
| *** amotoki_ has joined #openstack-ironic | 11:09 | |
| *** alexpilotti has joined #openstack-ironic | 11:12 | |
| openstackgerrit | Vaishnavi proposed openstack/ironic: Add support for building ISO for deploy ramdisk https://review.openstack.org/264590 | 11:14 |
| *** deray has quit IRC | 11:15 | |
| *** electrofelix has joined #openstack-ironic | 11:15 | |
| *** lucas-dinner is now known as lucasagomes | 11:18 | |
| *** mgould has joined #openstack-ironic | 11:19 | |
| openstackgerrit | Vaishnavi proposed openstack/ironic: Add support for dib based agent ramdisk in lib/ironic https://review.openstack.org/264579 | 11:20 |
| *** Pespa has joined #openstack-ironic | 11:23 | |
| *** stendulker_ has joined #openstack-ironic | 11:24 | |
| *** stendulker has quit IRC | 11:28 | |
| *** Marga_ has joined #openstack-ironic | 11:29 | |
| *** vdrok has quit IRC | 11:29 | |
| *** stendulker_ has quit IRC | 11:29 | |
| *** mgould has quit IRC | 11:32 | |
| *** mgould has joined #openstack-ironic | 11:33 | |
| *** Marga_ has quit IRC | 11:34 | |
| *** d0ugal has quit IRC | 11:40 | |
| *** vaishnavi has joined #openstack-ironic | 11:45 | |
| *** baoli has quit IRC | 11:50 | |
| *** amotoki_ has quit IRC | 11:50 | |
| *** ionutbalutoiu has quit IRC | 11:52 | |
| *** ionutbalutoiu has joined #openstack-ironic | 11:53 | |
| *** d0ugal has joined #openstack-ironic | 11:58 | |
| lucasagomes | morning all | 11:59 |
| mgould | lucasagomes, morning! | 12:01 |
| *** ijw has joined #openstack-ironic | 12:04 | |
| *** alexpilotti has quit IRC | 12:04 | |
| *** ijw has quit IRC | 12:09 | |
| sambetts | Hey lucasagomes o/ I've trade one devstack issue for another :( now I'm seeing "ImportError: No module named ironicclient.osc.plugin" | 12:14 |
| *** vaishnavi has quit IRC | 12:15 | |
| *** mgould has quit IRC | 12:19 | |
| *** mgould has joined #openstack-ironic | 12:20 | |
| *** shuquan has joined #openstack-ironic | 12:21 | |
| *** raildo-afk is now known as raildo | 12:22 | |
| openstackgerrit | Rui Chen proposed openstack/python-ironicclient: Fix Resource.__eq__ mismatch semantics of object equal https://review.openstack.org/251836 | 12:23 |
| lucasagomes | sambetts, hi there | 12:27 |
| lucasagomes | sambetts, ouch, where you seem that? | 12:27 |
| *** sinval-afk is now known as sinval | 12:27 | |
| sinval | good morning | 12:27 |
| *** shuquan has quit IRC | 12:28 | |
| sambetts | lucasagomes: I've managed to fix that issue now, I think the python package install went weird so I uninstalled it and reinstalled it and now its working, however now I'm back to the Could not determine a suitable URL for the plugin error from openstack image create, when it tries to upload the cirros image | 12:28 |
| *** trown|outttypeww is now known as trown | 12:30 | |
| lucasagomes | sambetts, I think I'm a bit out of context, are you working on having the devstack/lib/ironic to use OSC instead of ironicclient ? | 12:31 |
| sambetts | lucasagomes: oh sorry, not I'm just trying to devstack :( | 12:31 |
| sambetts | no * | 12:31 |
| sambetts | lucasagomes: this is the first time I'm devstacking using the ironic plugin instead of the enable_service thing | 12:32 |
| lucasagomes | oh right | 12:38 |
| *** baoli has joined #openstack-ironic | 12:41 | |
| * lucasagomes should try that too | 12:47 | |
| TheJulia | good morning | 12:47 |
| *** smoriya__ has quit IRC | 12:48 | |
| lucasagomes | TheJulia, morning | 12:50 |
| *** degorenko is now known as _degorenko|afk | 12:51 | |
| *** yolanda has quit IRC | 13:00 | |
| NobodyCam | good morning Ironicers | 13:03 |
| *** ijw has joined #openstack-ironic | 13:05 | |
| *** hanchao has quit IRC | 13:06 | |
| *** jcoufal has joined #openstack-ironic | 13:07 | |
| *** ijw has quit IRC | 13:09 | |
| *** pas-ha has quit IRC | 13:10 | |
| sinval | morning NobodyCam | 13:10 |
| NobodyCam | morning sinval ... lol well almost it's like 5 am here | 13:10 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Normalize the tag data structure https://review.openstack.org/267434 | 13:10 |
| *** _zouyee has quit IRC | 13:12 | |
| sinval | NobodyCam: omg, go grab some coffee to keep you alive :) | 13:14 |
| mgould | TheJulia, NobodyCam sinval morning! | 13:14 |
| NobodyCam | lol already started :) | 13:15 |
| sinval | morning mgould and TheJulia | 13:15 |
| *** pas-ha has joined #openstack-ironic | 13:15 | |
| NobodyCam | morning mgould | 13:15 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Fix issues with uefi-ipxe booting https://review.openstack.org/265742 | 13:16 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Pin Ansible version prior to the 2.0.0.1-1 release https://review.openstack.org/266946 | 13:16 |
| *** dprince has joined #openstack-ironic | 13:17 | |
| *** thrash|g0ne is now known as thrash | 13:18 | |
| *** mtanino has joined #openstack-ironic | 13:19 | |
| lucasagomes | sinval, NobodyCam good morning | 13:19 |
| *** yolanda has joined #openstack-ironic | 13:20 | |
| sinval | lucasagomes, o/ | 13:20 |
| NobodyCam | morning lucasagomes o/ | 13:21 |
| *** _degorenko|afk is now known as degorenko | 13:30 | |
| *** vdrok- is now known as vdrok | 13:35 | |
| vdrok | morning lucasagomes sinval TheJulia NobodyCam ! :) | 13:36 |
| NobodyCam | Imorning vdrok :) | 13:36 |
| lucasagomes | vdrok, hi there, morning | 13:36 |
| sinval | vdrok, o/ | 13:37 |
| *** ijw has joined #openstack-ironic | 14:05 | |
| *** ijw has quit IRC | 14:10 | |
| sambetts | s | 14:11 |
| sambetts | dtantsur|afk: Did you see the Shovel anouncment on | 14:12 |
| sambetts | dtantsur|afk: the mailer? | 14:12 |
| *** [3]cdearborn has joined #openstack-ironic | 14:13 | |
| *** rloo has joined #openstack-ironic | 14:15 | |
| *** kan_ has quit IRC | 14:18 | |
| *** lucasagomes is now known as lucas-hungry | 14:20 | |
| *** dims has quit IRC | 14:26 | |
| *** teju has quit IRC | 14:28 | |
| *** dims has joined #openstack-ironic | 14:30 | |
| *** kalpase has left #openstack-ironic | 14:38 | |
| *** [3]cdearborn has quit IRC | 14:39 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Normalize the tag data structure https://review.openstack.org/267434 | 14:39 |
| *** jistr has quit IRC | 14:40 | |
| *** jistr has joined #openstack-ironic | 14:41 | |
| *** dprince has quit IRC | 14:42 | |
| *** sinval is now known as sinval-afk | 14:51 | |
| *** ionutbalutoiu has quit IRC | 14:52 | |
| *** links has quit IRC | 14:52 | |
| *** ionutbalutoiu has joined #openstack-ironic | 14:52 | |
| *** Sukhdev has joined #openstack-ironic | 14:55 | |
| *** Haomeng has quit IRC | 15:03 | |
| *** ijw has joined #openstack-ironic | 15:05 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Allow rerunning introspection on stored data https://review.openstack.org/267637 | 15:08 |
| *** jcoufal has quit IRC | 15:11 | |
| *** piet has joined #openstack-ironic | 15:12 | |
| *** dprince has joined #openstack-ironic | 15:17 | |
| *** rajinir has joined #openstack-ironic | 15:18 | |
| NobodyCam | oh meetings why do you plage me | 15:22 |
| *** spandhe has joined #openstack-ironic | 15:23 | |
| *** raddaoui has joined #openstack-ironic | 15:23 | |
| *** lucas-hungry is now known as lucasagomes | 15:24 | |
| jlvillal | NobodyCam, That's what happens when you are important :) | 15:26 |
| NobodyCam | lol | 15:26 |
| NobodyCam | :p | 15:26 |
| jlvillal | sambetts, Did you see my Docker IPA patch? :) | 15:26 |
| sambetts | jlvillal: I saw it go up but I've not had a chance to look through it yet | 15:27 |
| jlvillal | sambetts, Fair enough. Thanks :) | 15:27 |
| jlvillal | sambetts, If we had Docker 1.9 (or maybe it is 2.0) there is a more elegant method using ARG available. But we don't have that in Ubuntu 14.04 | 15:28 |
| NobodyCam | JayF: around yet? | 15:28 |
| *** spandhe has quit IRC | 15:30 | |
| *** mgoddard has quit IRC | 15:30 | |
| *** mgoddard has joined #openstack-ironic | 15:30 | |
| *** jcoufal has joined #openstack-ironic | 15:31 | |
| openstackgerrit | Imre Farkas proposed openstack/ironic: [WIP] DRAC: RAID configuration on vendor-passthru https://review.openstack.org/176319 | 15:33 |
| *** Sukhdev has quit IRC | 15:40 | |
| *** Sukhdev has joined #openstack-ironic | 15:40 | |
| *** piet has quit IRC | 15:42 | |
| *** piet has joined #openstack-ironic | 15:43 | |
| sambetts | jlvillal: It certainly feels a little hacky doesn't it :( | 15:44 |
| devananda | g'monring, all | 15:45 |
| NobodyCam | morning devananda :) | 15:45 |
| mgould | devananda, morning! | 15:45 |
| NobodyCam | how Tx? | 15:45 |
| *** Sukhdev has quit IRC | 15:46 | |
| devananda | it's Dallas | 15:49 |
| devananda | I have yet to find decent coffee anywhere | 15:49 |
| NobodyCam | oh man | 15:50 |
| sambetts | devananda: that doesn't sound good for coffee at the summit :-P | 15:52 |
| mgould | sambetts, AIUI Austin is culturally very different from the rest of Texas | 15:52 |
| mgould | so they *probably* have good coffee there :-) | 15:53 |
| sambetts | mgould: I've only over stopped over in the airport in Tx so it'll be my first time experiencing the real Texas haha | 15:54 |
| * NobodyCam notes he can (most) always find a starbucks for coffee :) | 15:55 | |
| *** Sukhdev has joined #openstack-ironic | 15:55 | |
| lucasagomes | devananda, morning | 15:56 |
| *** vishwanathj_away is now known as vishwanathj | 15:57 | |
| mgould | sambetts, I'm not sure Austin is "the real Texas" :-) https://en.wikipedia.org/wiki/Keep_Austin_Weird | 16:00 |
| NobodyCam | mgould: nice :) | 16:01 |
| sambetts | mgould: haha looking forward to it :D | 16:01 |
| * mgould likes the building with an aircraft tailplane sticking out, but raises you https://en.wikipedia.org/wiki/The_Headington_Shark | 16:02 | |
| * lucasagomes dunno what to expect from austin/texas in general | 16:02 | |
| mgould | all my knowledge of Texas comes from watching King Of The Hill :-/ | 16:02 |
| lucasagomes | heh I haven't seem that | 16:02 |
| jlvillal | sambetts, Yeah, sort of hacky :( | 16:03 |
| jlvillal | sambetts, If you have ideas on how to make it less so that would be great. | 16:03 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Introducing API for aborting introspection https://review.openstack.org/265275 | 16:05 |
| sambetts | jlvillal: I wonder if we could get rid of the need to copy and replace files if we just made proxy export the variables and then made the commands "RUN do proxy.sh && <the real command>" | 16:05 |
| *** mjturek1 has quit IRC | 16:05 | |
| sambetts | jlvillal: RUN proxy.sh && <real the command> | 16:05 |
| jlvillal | sambetts, It doesn't stay in the environment. | 16:05 |
| jlvillal | sambetts, Maybe 'source proxy.inc && command' ???? | 16:06 |
| jlvillal | But then I'm not sure which is more hacky. | 16:06 |
| jlvillal | And not sure if that would actually work. | 16:06 |
| sambetts | jlvillal: it wouldn't stay between run statements, but if RUN does /bin/sh -c "proxy.sh && <the command>" then for each RUN you'd only need to to proxy.sh once | 16:07 |
| jlvillal | sambetts, But would still have to create that proxy file though. | 16:07 |
| sambetts | jlvillal: Yeah we would but we wouldn't have to copy and then replace it with an existing one, it could just be an empty file because its running next to the other command rather than running it itself with $* | 16:08 |
| jlvillal | sambetts, In my quick test that didn't work. | 16:09 |
| sambetts | :( | 16:09 |
| *** mjturek1 has joined #openstack-ironic | 16:09 | |
| jlvillal | sambetts, I don't quite understand why wouldn't have to copy and replace. | 16:10 |
| jlvillal | But maybe it doesn't matter since the test failed... | 16:11 |
| vdrok | morning jlvillal and devananda | 16:11 |
| jlvillal | vdrok, Morning! | 16:12 |
| *** harshs has joined #openstack-ironic | 16:12 | |
| jlvillal | vdrok, Did you see my question a day or two ago here about your tempest patch? | 16:12 |
| vdrok | jlvillal, yep, I guess I answered | 16:12 |
| jlvillal | vdrok, I was wondering if you are still working on it :) | 16:12 |
| sambetts | jlvillal: the only reason we're having the copy and paste the file is because we need to replace the temp file we create with a file that contains "$*" but if we got rid of that then we could just do rm proxy.sh && touch proxy.sh | 16:12 |
| vdrok | jlvillal, i'm not working on that atm, because that is needed only for full tempest right now | 16:13 |
| vdrok | and it takes too much time | 16:13 |
| vdrok | like 4 hours | 16:13 |
| sambetts | jlvillal: I guess by copy and paste I'm referring to your mv command | 16:13 |
| sambetts | s | 16:13 |
| jlvillal | vdrok, Understood. Yeah it takes a long time, as I am discovering. | 16:13 |
| vdrok | so first thing to do is multinode devstack | 16:13 |
| vdrok | so that tests can run in parallel | 16:13 |
| vdrok | and I'm not good at it :( | 16:14 |
| jlvillal | sambetts, But if there are proxy settings, wouldn't we still need to copy/paste? | 16:14 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Introducing API for aborting introspection https://review.openstack.org/265275 | 16:14 |
| jlvillal | I guess I could dynamically create the proxy.sh file but I was wanting to make sure it worked for people who ran 'docker build' directly | 16:15 |
| jlvillal | sambetts, I'm not sure who those people are :) | 16:15 |
| sambetts | :-P | 16:15 |
| openstackgerrit | Ruby Loo proposed openstack/ironic-lib: Remove unused packages from requirements https://review.openstack.org/267213 | 16:16 |
| jlvillal | vdrok, Thanks for the info. I had my patch to tempest to disable the floating ip check for Ironic. | 16:16 |
| vdrok | jlvillal, ah, I saw that | 16:16 |
| jlvillal | vdrok, I'm still not quite understanding how floating IPs work or if they do work, with Ironic | 16:16 |
| vdrok | jlvillal, iirc that test requires multiple "baremetal" nodes and fails if there is only 1 | 16:16 |
| jlvillal | jroll, Had -1 my patch because he said he tested and floating IP do work with Ironic. So either that test should work or I should have a different reason why it shouldn't work :) | 16:17 |
| jlvillal | vdrok, Ah that might be a better reason :) | 16:17 |
| mkovacik | sambetts, I've fixed the logs & rebased the merge; when ever you're ready ;) thx for the review! | 16:17 |
| jlvillal | Maybe I only need to change the description of why we shouldn't run that test. To the correct reason. | 16:17 |
| jlvillal | vdrok, Thanks for your help :) | 16:18 |
| mkovacik | * the pull request | 16:18 |
| sambetts | jlvillal: what if we wrote the environment variables into /etc/profie file in the container? then we wouldn't have to run proxy.sh every time because they would become permanently set for the container, and then we remove them afterwards? | 16:18 |
| vdrok | jlvillal, where did I help? :) np | 16:18 |
| jlvillal | sambetts, Okay. 1) How do we get them there. 2) Does doing '/bin/sh -c' pull them in? | 16:19 |
| jlvillal | vdrok, You gave me a better description. I think. | 16:19 |
| vdrok | jlvillal, :) | 16:20 |
| jlvillal | sambetts, env -i /bin/sh -c env | sort does not give me great hope for that idea. | 16:21 |
| *** erwan_taf has quit IRC | 16:21 | |
| * jlvillal feels like he is being negative and keeps shooting down ideas :( | 16:22 | |
| *** garthb__ has joined #openstack-ironic | 16:23 | |
| sambetts | jlvillal: we'd still need a script like proxy.sh that we'd have to generate, but in that we'd just have something like echo 'export https_proxy=192.168.4.5' >> /etc/profile | 16:24 |
| jlvillal | Okay | 16:25 |
| *** ukalifon has quit IRC | 16:27 | |
| sambetts | jlvillal: :( you have to add -l to the /bin/sh command to make it read from /etc/profile | 16:27 |
| *** ukalifon has joined #openstack-ironic | 16:28 | |
| jlvillal | sambetts, Okay. That's not how docker does the RUN command though. | 16:28 |
| sambetts | jlvillal: yeah :( | 16:28 |
| jlvillal | sambetts, Thanks for the ideas! :) | 16:29 |
| * jlvillal goes to get ready to walk to work... | 16:30 | |
| *** harshs_ has joined #openstack-ironic | 16:32 | |
| *** ukalifon has quit IRC | 16:32 | |
| *** spandhe has joined #openstack-ironic | 16:32 | |
| *** harshs has quit IRC | 16:33 | |
| *** harshs has joined #openstack-ironic | 16:35 | |
| *** piet has quit IRC | 16:36 | |
| *** _zouyee has joined #openstack-ironic | 16:36 | |
| *** mkovacik_ has joined #openstack-ironic | 16:37 | |
| *** harshs_ has quit IRC | 16:37 | |
| *** mkovacik has quit IRC | 16:38 | |
| *** piet has joined #openstack-ironic | 16:38 | |
| jroll | morning y'all | 16:45 |
| devananda | morning, yo | 16:45 |
| *** mgoddard_ has joined #openstack-ironic | 16:45 | |
| jroll | :) | 16:45 |
| JayF | NobodyCam: now I am, sup? | 16:45 |
| jroll | devananda: what do you say we land some networking code today? | 16:45 |
| *** dprince has quit IRC | 16:45 | |
| jlvillal | jroll, devananda Would you say grenade is testing "rolling" upgrades? | 16:45 |
| NobodyCam | hummm can I remember | 16:45 |
| *** dprince has joined #openstack-ironic | 16:46 | |
| jroll | devananda: or at least +1 the devstack vars so we can drop the experimental job in | 16:46 |
| * jlvillal was asked about "rolling upgrades" today and didn't have a clear answer on what that mean. | 16:46 | |
| vdrok | morning jroll and JayF | 16:46 |
| jlvillal | I think "rolling upgrade" is in place | 16:46 |
| JayF | morning | 16:46 |
| jroll | jlvillal: no, grenade is bring everything down, upgrade, bring everything up. so an upgrade with downtime. | 16:46 |
| jlvillal | As opposed to bringing up a new OpenStack setup and then copying data from old to new. | 16:46 |
| *** pas-ha has quit IRC | 16:46 | |
| jlvillal | jroll, So "rolling" implies no downtime? | 16:47 |
| openstackgerrit | Imre Farkas proposed openstack/ironic: DRAC: switch to python-dracclient on vendor-passthru https://review.openstack.org/251294 | 16:47 |
| openstackgerrit | Imre Farkas proposed openstack/ironic: DRAC: cleanup after switch to python-dracclient https://review.openstack.org/255310 | 16:47 |
| jroll | jlvillal: nova has "partial grenade" jobs, which do rolling upgrades. it basically runs nova with an older conductor | 16:47 |
| jroll | er, older compute | 16:47 |
| jroll | jlvillal: yeah, rolling == no downtime | 16:47 |
| jroll | jlvillal: so in reality, you'd upgrade one ironic-conductor at a time, then upgrade one ironic-api at a time (managing them in load balancer as you go) | 16:47 |
| jroll | and users would not see downtime | 16:48 |
| jroll | in devstack, we'd test that it is possible to do so by running old ironic-api with new ironic-conductor | 16:48 |
| jlvillal | jroll, Thanks. That helps a lot. | 16:48 |
| *** mgoddard has quit IRC | 16:48 | |
| jroll | well, running old both, then upgrading conductor and running tests again | 16:48 |
| jroll | cool :) | 16:48 |
| jroll | you're welcome | 16:48 |
| jlvillal | jroll, There is a 'partial ironic' grenade job too | 16:49 |
| jlvillal | Doesn't work, but it exists :) | 16:49 |
| jroll | jlvillal: yeah, that's what would test rolling | 16:49 |
| jroll | so in theory since neither job works, we don't support upgrading at all :D | 16:49 |
| jroll | we can assert that we support upgrades when grenade works, and rolling upgrades when partial works. | 16:49 |
| jroll | oh just thought of something, 'rolling upgrade' is AKA 'online upgrade' | 16:50 |
| *** dprince has quit IRC | 16:50 | |
| jlvillal | That seems like a better term and easier to know what it is. | 16:51 |
| devananda | jroll: also, online upgrade of ironic != online upgrade of cloud, where we upgrade ironic before nova | 16:51 |
| devananda | both are useful things to test | 16:51 |
| jroll | devananda: right, that would be a heck of a test to write :) | 16:51 |
| jroll | upgrade ir-cond, upgrade ir-api, upgrade n-cond, upgrade n-api, upgrade n-* | 16:51 |
| jroll | fun! | 16:51 |
| *** dprince has joined #openstack-ironic | 16:51 | |
| jroll | I don't believe in the philosophy that all the cloud should upgrade at once, personally | 16:52 |
| jroll | and I want to work toward making that work well | 16:52 |
| jroll | (hint: it already works pretty okay) | 16:52 |
| jroll | (you just have to pay attention to lots of things) | 16:52 |
| openstackgerrit | Sam Betts proposed openstack/ironic-python-agent: Add tinyipa to IPA imagebuild directory https://review.openstack.org/234902 | 16:54 |
| *** mbound has quit IRC | 16:58 | |
| jroll | sambetts: out of curiousity, how much ram does tinyipa need? | 16:59 |
| *** mgoddard_ has quit IRC | 16:59 | |
| jroll | to fully boot and run | 16:59 |
| * jroll wondering how much parallelism we could get here | 17:00 | |
| *** mgoddard has joined #openstack-ironic | 17:00 | |
| *** Sukhdev has quit IRC | 17:00 | |
| *** alexpilotti has joined #openstack-ironic | 17:01 | |
| *** spandhe has quit IRC | 17:01 | |
| NobodyCam | JayF: oh ya, I just wanted to ask if your use both the debug lines for debugging. ie. "Clean steps before deduplication" and "Returning clean steps".... i'm thought / concern is that method may be a bit chatty in the log? | 17:02 |
| NobodyCam | JayF: also sorry for hte slow reply | 17:03 |
| NobodyCam | on a call | 17:03 |
| JayF | NobodyCam: if you look I added that logging as a result of reviews | 17:03 |
| JayF | NobodyCam: but it's a debug log and I don't think logging at DEBUG is bad to be chatty | 17:03 |
| JayF | NobodyCam: i.e. if I'm building a new hardware manager and my clean step is disappearing, that's 100% key to diagnosis | 17:03 |
| NobodyCam | ack... just wanted to check in on that | 17:04 |
| *** raddaoui has quit IRC | 17:04 | |
| sambetts | jroll: currently booting it locally on a 512MB ram virtual box vm | 17:04 |
| jroll | sambetts: I'd love to try at 256 and 128, that could get us a decent amount of boxes | 17:05 |
| sambetts | jroll: just trying 256 now | 17:05 |
| jroll | <3 | 17:05 |
| NobodyCam | JayF: +2 :) | 17:05 |
| JayF | woo yu | 17:06 |
| JayF | ty | 17:06 |
| *** raildo is now known as raildo-afk | 17:12 | |
| *** sinval-afk is now known as sinval | 17:13 | |
| *** alexpilotti has quit IRC | 17:13 | |
| jroll | patch your ssh http://undeadly.org/cgi?action=article&sid=20160114142733 | 17:14 |
| *** alexpilotti has joined #openstack-ironic | 17:14 | |
| *** alexpilo_ has joined #openstack-ironic | 17:16 | |
| *** alexpilotti has quit IRC | 17:16 | |
| *** harshs has quit IRC | 17:17 | |
| mgould | jroll, p_q | 17:17 |
| *** harshs has joined #openstack-ironic | 17:17 | |
| jroll | mhm. | 17:17 |
| devananda | jroll: ouch | 17:18 |
| sambetts | jroll: 256 booted but couldn't run IPA, 384 managed to run IPA (this was on a build that installed the python at runtime), I'm just making a build with everything pre-installed in the ramdisk to see if it'll boot with less | 17:19 |
| jroll | sambetts: cool, good to know | 17:19 |
| *** raildo-afk is now known as raildo | 17:20 | |
| sambetts | JayF: I managed to get rid of the binarys in the tinyipa patch, it now manages to successfully build everything thats required in a chroot during the build process | 17:21 |
| *** raddaoui has joined #openstack-ironic | 17:21 | |
| JayF | \o/ | 17:22 |
| lucasagomes | jroll, devananda rloo (others) when you guys have time https://review.openstack.org/#/c/267723/ | 17:28 |
| *** jistr has quit IRC | 17:29 | |
| *** harshs has quit IRC | 17:29 | |
| sambetts | jroll: as I found before installing all the python requirements at build time actually leads to a greater ram requirement for the ramdisk for some reason, 512mb is the lowest I had that version booting | 17:34 |
| rloo | lucasagomes: thx. will try to look today.it should go in specs/approved, with a soft link in specs/not-implemented :) | 17:35 |
| jroll | sambetts: oh right, because ramdisk | 17:35 |
| lucasagomes | rloo, oh, ok I will change it | 17:35 |
| lucasagomes | rloo, thanks | 17:35 |
| sambetts | jroll: yeah :( | 17:35 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-specs: Split the node's capabilities to its own table https://review.openstack.org/267723 | 17:37 |
| * lucasagomes fixed | 17:37 | |
| *** sambetts has quit IRC | 17:40 | |
| *** e0ne has quit IRC | 17:40 | |
| *** moshele has quit IRC | 17:44 | |
| *** alexpilo_ has quit IRC | 17:45 | |
| *** alexpilotti has joined #openstack-ironic | 17:47 | |
| *** Pespa has quit IRC | 17:49 | |
| *** alexpilotti has quit IRC | 17:51 | |
| *** alexpilotti has joined #openstack-ironic | 17:52 | |
| *** mkovacik_ has quit IRC | 17:56 | |
| *** mkovacik_ has joined #openstack-ironic | 17:56 | |
| *** mkovacik_ has quit IRC | 17:56 | |
| *** trown is now known as trown|lunch | 17:56 | |
| *** raddaoui has quit IRC | 17:57 | |
| *** raddaoui has joined #openstack-ironic | 17:58 | |
| *** ndipanov has quit IRC | 18:00 | |
| *** daemontool has quit IRC | 18:01 | |
| *** alexpilotti has quit IRC | 18:02 | |
| *** alexpilotti has joined #openstack-ironic | 18:04 | |
| *** derekh has quit IRC | 18:05 | |
| *** praneshp has quit IRC | 18:07 | |
| *** vishwanathj is now known as vishwanathj_ | 18:18 | |
| *** alexpilotti has quit IRC | 18:19 | |
| *** alexpilotti has joined #openstack-ironic | 18:20 | |
| *** raddaoui has quit IRC | 18:21 | |
| * mgould -> home; good night! | 18:22 | |
| *** mgould has quit IRC | 18:22 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-specs: Split the node's capabilities to its own table https://review.openstack.org/267723 | 18:24 |
| *** _zouyee has quit IRC | 18:27 | |
| *** electrofelix has quit IRC | 18:31 | |
| *** trown|lunch is now known as trown | 18:33 | |
| *** alexpilotti has quit IRC | 18:33 | |
| *** praneshp has joined #openstack-ironic | 18:33 | |
| *** spandhe has joined #openstack-ironic | 18:34 | |
| *** alexpilotti has joined #openstack-ironic | 18:34 | |
| *** moshele has joined #openstack-ironic | 18:36 | |
| jroll | FYI ATC codes for austin summit went out last night | 18:37 |
| *** alexpilotti has quit IRC | 18:40 | |
| vdrok | good night! | 18:40 |
| *** harshs has joined #openstack-ironic | 18:41 | |
| TheJulia | I was looking at hotels this morning, seems the hotels are already a little full. :( | 18:46 |
| *** alexpilotti has joined #openstack-ironic | 18:47 | |
| *** e0ne has joined #openstack-ironic | 18:48 | |
| jroll | airbnb! | 18:50 |
| *** ifarkas has quit IRC | 18:50 | |
| *** Marga_ has joined #openstack-ironic | 18:52 | |
| *** degorenko is now known as _degorenko|afk | 18:54 | |
| rloo | jlvillal: dumb question, for https://review.openstack.org/#/c/255834/, why do you need to mock? what is calling those functions? | 18:55 |
| *** spandhe has quit IRC | 19:05 | |
| *** jaypipes has quit IRC | 19:07 | |
| * lucasagomes registered for austin already! | 19:07 | |
| lucasagomes | rloo, I believe they mock it to make it prediticable | 19:12 |
| lucasagomes | rloo, otherwise that functions would return a different name depending on the host machine | 19:12 |
| rloo | lucasagomes: when are they used? | 19:12 |
| lucasagomes | that I don't know :-) /me checks the whole thing | 19:13 |
| * NobodyCam has a hotel for austin but has not yet regeistered or booked flights | 19:13 | |
| NobodyCam | :p | 19:13 |
| jroll | rloo++ I don't think those are doing anything | 19:14 |
| lucasagomes | would be good the bug ticket to actually show where it fails | 19:16 |
| lucasagomes | but yeah, looking at the code doesn't seem to be used anywhere | 19:16 |
| *** spandhe has joined #openstack-ironic | 19:17 | |
| rloo | jroll: , lucasagomes, i'll -1 until we hear from jlvillal. those functions had been used before but he removed them. | 19:18 |
| lucasagomes | cool | 19:18 |
| jroll | right | 19:18 |
| jroll | thanks | 19:18 |
| lucasagomes | rloo, https://github.com/openstack/ironic/blob/master/ironic/common/service.py#L45 | 19:25 |
| jlvillal | rloo: reading backscroll | 19:25 |
| lucasagomes | I wonder if the config from other libs may also affect it | 19:25 |
| lucasagomes | rabbitmq may want to use the hostname | 19:25 |
| jroll | oh yep, that'll do it | 19:25 |
| lucasagomes | oslo.messaging* | 19:25 |
| jlvillal | What was happening is if the name of the host was set to 'localhost' then the config file would use the name 'ironic' | 19:25 |
| jlvillal | So the generated config file would be different from a system with a hostname of 'localhost' compared to a system with the name 'foo' | 19:26 |
| jroll | jlvillal: well, so the bug fix is intended to use the real hostname of the system, right? | 19:26 |
| jroll | (in all cases) | 19:26 |
| jroll | or is intended to use 'ironic' in all cases | 19:27 |
| jlvillal | jroll: No, to ignore the real hostname when generating the config file. | 19:27 |
| jroll | I conjecture that both are bad | 19:27 |
| lucasagomes | yeah both has side effects | 19:28 |
| jroll | this is a thing that deployers might use to generate a config | 19:28 |
| lucasagomes | but I prefer to mock that name otherwise the config will always differ dependning on who generates it | 19:28 |
| jlvillal | The patch makes it generate the exact same output as a current system that is not named 'localhost' | 19:28 |
| jlvillal | So for 99% of people they generate the normal config | 19:29 |
| jroll | oh, that's a good point, it's already "broken" in that way | 19:29 |
| jroll | ok fair enuogh | 19:29 |
| jlvillal | But when someones fires up a VM named localhost, currently the config file gets changed to different values for a few items | 19:29 |
| jlvillal | If they run the config generation inside the VM named localhost | 19:29 |
| openstackgerrit | Merged openstack/bifrost: Pin Ansible version prior to the 2.0.0.1-1 release https://review.openstack.org/266946 | 19:30 |
| jroll | so 'ironic' is intended to be used in all cases, right? | 19:30 |
| jlvillal | Personally I think it should be 'localhost' but I am duplicating the current output | 19:31 |
| rloo | i thought 'localhost' | 19:31 |
| *** praneshp has quit IRC | 19:31 | |
| rloo | at least it reflects/indicates that it is a host. 'ironic' makes no sense. | 19:31 |
| jroll | ironic seems to be current intention of the script in master | 19:31 |
| jroll | and also the output here | 19:31 |
| jroll | so I'm inclined to call this good | 19:32 |
| jroll | and if y'all want to change it later, that's up to you | 19:32 |
| jlvillal | rloo: Agreed, but I was trying to duplicate current generation so it would be the same on both a system named 'localhost' and a system not named 'localhost' | 19:32 |
| rloo | so what is making the calls to those mocked functions | 19:32 |
| * lucasagomes is ok with the name | 19:32 | |
| jroll | rloo, https://github.com/openstack/ironic/blob/master/ironic/common/service.py#L45 | 19:32 |
| jroll | for example | 19:32 |
| jlvillal | rloo: Some of the libraries call those functions. | 19:32 |
| rloo | jroll: ahh. | 19:32 |
| * jroll +2 | 19:33 | |
| *** praneshp has joined #openstack-ironic | 19:33 | |
| rloo | sorry, so is it supposed to show host=ironic or host=localhost? | 19:33 |
| jroll | ironic | 19:34 |
| rloo | why 'ironic'. and jlvillal, why duplicate the current output if the current output is incorrect? | 19:35 |
| lucasagomes | it's just a deterministic placeholder | 19:35 |
| jlvillal | rloo: Why 'ironic' I don't know. Someone did that. I think 'localhost' would be better but I thought that would be better for a follow-on patch if there was consensus | 19:35 |
| rloo | jlvillal: the bug is that it shouldn't be 'ironic', it should be 'localhost'. | 19:36 |
| jlvillal | rloo: I thought my bug was that the output changed between systems named 'localhost' and not named 'localhost' :) | 19:36 |
| rloo | jlvillal: in my opinion. folks keep generating that config, and it toggles between localhost & ironic depending on who generates it and reviewers just +2 it w/o noticing. | 19:36 |
| jlvillal | rloo: Your suggestion might be the true bug. | 19:36 |
| jlvillal | rloo: This patch fixes it so it doesn't toggle | 19:37 |
| rloo | jlvillal: think about it. how would 'ironic' help anyone as a default value? | 19:37 |
| jlvillal | rloo: I'm not arguing. I was just doing one piece at a time. | 19:37 |
| jlvillal | If there is consensus I am happy to change it to localhost. | 19:37 |
| jlvillal | :) | 19:37 |
| rloo | jlvillal: i think it should be localhost. am in meeting and i have to actually participate so bye for now :) | 19:38 |
| *** piet has quit IRC | 19:38 | |
| jlvillal | rloo: Okay. How about a follow on patch. I'll do that. | 19:38 |
| lucasagomes | neither ironic nor localhost is the real default (because the default is to call that socket.get<hostname,fqdname>() ) | 19:38 |
| lucasagomes | it's just a placeholder | 19:38 |
| jlvillal | lucasagomes: But localhost is sort of the same thing. | 19:38 |
| lucasagomes | same thing as calling socket.gethostname() ? | 19:39 |
| jlvillal | lucasagomes: Same thing as the packets should get to the same point. | 19:39 |
| jlvillal | If DNS setup correctly. No guarantee though. | 19:40 |
| lucasagomes | right | 19:40 |
| lucasagomes | yeah, idk, I'm good with any name | 19:40 |
| lucasagomes | ironic, localhost,foo | 19:41 |
| jlvillal | :) | 19:41 |
| *** ChubYann has joined #openstack-ironic | 19:41 | |
| *** Sukhdev has joined #openstack-ironic | 19:45 | |
| *** erwan_taf has joined #openstack-ironic | 19:48 | |
| *** ijw has quit IRC | 19:54 | |
| lucasagomes | I'm going to call it a day :-) | 19:54 |
| lucasagomes | have a great night everyone | 19:54 |
| *** lucasagomes is now known as lucas-dinner | 19:55 | |
| *** alexpilotti has quit IRC | 19:56 | |
| *** alexpilotti has joined #openstack-ironic | 19:56 | |
| rloo | jlvillal: sorry, going back to your patch 255834. If you don't mock, but remove the code that you removed, then what happens if the function returns 'localhost'? | 19:56 |
| rloo | jlvillal: the config is 'localhost'? and if the function returns 'myfqdn', the config is 'myfqdn'? | 19:57 |
| *** piet has joined #openstack-ironic | 19:58 | |
| jlvillal | rloo: then the output will be whatever is the name of the system. | 19:58 |
| jlvillal | The default value will change depending which system it is run on. | 19:58 |
| rloo | jlvillal: and is that what we want, or do we want it to always be 'localhost' or 'ironic'? | 19:58 |
| rloo | jlvillal: it seems to me that if we are generating for the .sample, we want some placeholder with something in the help to indicate what it should be. | 19:59 |
| jlvillal | rloo: So it is slightly confusing about my patch because it isn't changing things to make it be 'ironic'. It is actually trying to prevent values of 'localhost' from becoming 'ironic' | 19:59 |
| jlvillal | rloo: I just added a comment showing a diff | 19:59 |
| jlvillal | If run on a VM named 'localhost' what happens currently without my patch. | 19:59 |
| rloo | jlvillal: my question is what happens with your change but w/o your mock. | 19:59 |
| rloo | jlvillal: i agree that the code w/o your patch is incorrect | 20:00 |
| rloo | jlvillal: i just don't understand why we need the mocks in your patch. | 20:00 |
| jlvillal | Then the config values now that are '=ironic' will be come '=FQDN' or '=<hostname>' | 20:00 |
| jlvillal | As in the FQDN of the machine that the genconfig is run on. or the hostname of the machine that the genconfig is run on | 20:01 |
| rloo | jlvillal: ok. so what you're saying is we don't want =FQDN or =hostname, we want something that is always the same, regardless of who generates the config file | 20:01 |
| jlvillal | rloo: Yes, so it doesn't change everytime someone updates the config file. | 20:01 |
| rloo | jlvillal: ok, that's what i want too. but i think the value should be 'localhost', not 'ironic'. | 20:01 |
| jlvillal | My FQDN is dork.example.com and yours is rockstar.example.com | 20:02 |
| jlvillal | So then when I generate it values might be =dork.example.com | 20:02 |
| jlvillal | And you do it, it becomes: = rockstar.example.com | 20:02 |
| jlvillal | rloo: I agree | 20:02 |
| jlvillal | Do you think I should do it in a follow-up patch or in this patch? | 20:02 |
| rloo | jlvillal: so the reason for the mock, is to be able to always have the same value for the config. | 20:02 |
| jlvillal | rloo: Yep | 20:02 |
| rloo | jlvillal: i would prefer if you just fixed it in this patch; i don't see the need for a subsequent patch. | 20:03 |
| jlvillal | rloo: Okay. I will push out a new one with a regenerated config file. | 20:03 |
| jlvillal | rloo: Thanks :) | 20:03 |
| rloo | jlvillal: your current patch is confusing to me cuz it doesn't seem to address what i think is the bug. | 20:03 |
| jlvillal | rloo: I guess there are two bugs. One what my patch talks about and the other that we use 'ironic' as a default hostname which is wonky | 20:04 |
| rloo | jlvillal: well, your bug hardcodes the value to be 'ironic', and if we +2 that, it indicates that yes, this is what we want. but i don't think that's what we want. | 20:04 |
| jlvillal | rloo: Agreed. Working on new patch. Should have it in a few minutes | 20:05 |
| rloo | jlvillal: i think the problem is that you are assuming that we wanted 'ironic' as the default hostname, and i think that is the bug, it shouldn't be 'ironic' but folks have been doing that. | 20:05 |
| rloo | thx jlvillal! | 20:05 |
| jlvillal | Yep. Thank you for the feedback. | 20:05 |
| rloo | jlvillal: it would also be useful if you added more comment above the mocks, saying that some config defaults call those functions to generate the default value but we don't want those. | 20:08 |
| *** piet has quit IRC | 20:09 | |
| jlvillal | rloo: Good idea. | 20:09 |
| rloo | jlvillal: thx! | 20:10 |
| *** penick has joined #openstack-ironic | 20:18 | |
| *** Marga_ has quit IRC | 20:18 | |
| jlvillal | rloo: So it will change the default for this: https://github.com/openstack/ironic/blob/master/etc/ironic/ironic.conf.sample#L123-L128 | 20:18 |
| jlvillal | rloo: And this: https://github.com/openstack/ironic/blob/master/etc/ironic/ironic.conf.sample#L311-L314 | 20:18 |
| rloo | jlvillal: as it should! yay! :) | 20:18 |
| *** Marga_ has joined #openstack-ironic | 20:18 | |
| jlvillal | rloo: Another option would be to set it to: <FQDN> and <HOSTNAME> to try to imply they are derived values. | 20:19 |
| jlvillal | At the moment I did 'localhost' | 20:19 |
| jroll | ok now I'm confused. how does this result in localhost? @mock.patch.object(socket, 'getfqdn', lambda: 'ironic') | 20:20 |
| rloo | it has 'always' been 'localhost' (toggled with the incorrect-in-my-opinion 'ironic') | 20:20 |
| *** raddaoui has joined #openstack-ironic | 20:20 | |
| rloo | jroll: it doesn't. i think jlvillal has to change that to s/ironic/localhost/ | 20:20 |
| jroll | I agree | 20:21 |
| jroll | vagrant@vagrant-ubuntu-trusty-64:~/ironic$ grep 'host=' etc/ironic/ironic.conf.sample | 20:21 |
| jroll | #host=ironic | 20:21 |
| jroll | after re-generating with this patch | 20:21 |
| jlvillal | jroll: That is preserving how it was. What it was fixing was other values would get changed from '=localhost' to '=ironic' when run on a system named 'localhost' | 20:21 |
| jlvillal | Before the patch | 20:22 |
| jlvillal | The lines that were deleted would change the value. | 20:22 |
| *** penick has quit IRC | 20:22 | |
| *** praneshp has quit IRC | 20:23 | |
| jroll | jlvillal: right, se the current patch makes default values 'ironic' | 20:23 |
| jroll | but reading the conversation here, the intent is to be 'localhost' | 20:23 |
| openstackgerrit | Merged openstack/bifrost: Modify detection of iso_gen_utility https://review.openstack.org/264374 | 20:23 |
| jroll | is that correct? | 20:23 |
| jlvillal | jroll: Not exactly making the default values 'ironic'. They already were 'ironic' for two values. My patch that is posted doesn't change that. | 20:23 |
| jroll | making configs that use gethostname() 'ironic', I mean | 20:24 |
| jlvillal | What it does is change where a value that was set to '=localhost' would get changed to '=ironic' when run on a system named 'localhost' | 20:24 |
| jroll | well | 20:24 |
| jroll | it mocks gethostname() to return 'ironic', currently | 20:24 |
| jroll | making them always =ironic | 20:24 |
| jlvillal | New patch will change it so that the values that are set to '=ironic' will now get changed to '=localhost'. In progress | 20:24 |
| jroll | AIUI | 20:24 |
| jroll | ok | 20:24 |
| jroll | we've spent way too much time on this patch :( | 20:24 |
| openstackgerrit | Arun S A G proposed openstack/ironic-specs: Add spec for deploying OS tarballs https://review.openstack.org/248968 | 20:25 |
| jlvillal | True | 20:25 |
| rloo | jroll: here, let me identify what the bug is. We want to generate a config file, where config values that are hosts, should not reflect the user's host where the generated the config, but be the value 'localhost'. | 20:25 |
| openstackgerrit | Merged openstack/bifrost: Use "state=touch" instead of running touch https://review.openstack.org/264498 | 20:25 |
| jroll | rloo: right, that's what I've gathered here | 20:25 |
| *** piet has joined #openstack-ironic | 20:26 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Fix issue where system hostname can impact genconfig https://review.openstack.org/255834 | 20:26 |
| jlvillal | jroll: rloo ^^^ | 20:26 |
| rloo | jroll: :) | 20:26 |
| * jlvillal prays to the gate gods for a successful gate run, even though his code is never run by the gate... | 20:27 | |
| jroll | okay, that seems right | 20:27 |
| *** mbound has joined #openstack-ironic | 20:28 | |
| *** erwan_taf has quit IRC | 20:30 | |
| rloo | jlvillal: with the way the gate has been going, it make take a few days for that patch to make it past the non-pearly gates. | 20:30 |
| jlvillal | rloo, jroll: I added a comment to the config changes to show the two places that the patch causes a change. | 20:30 |
| jlvillal | rloo: Unfortunately you are likely correct. Last patch took 8 rechecks :( | 20:30 |
| rloo | jlvillal: thx! | 20:30 |
| rloo | jlvillal: i stopped doing rechecks. am going to wait til next week or the week after. | 20:31 |
| jlvillal | TheJulia: Reading the doc bifrost/readme.html and it talks about: Change network_interface to match the interface that will need to service DHCP requests. | 20:34 |
| jlvillal | So we have a BMC network. Is that the network that this is talking about? | 20:34 |
| jlvillal | Or is this the real network | 20:35 |
| jlvillal | ? | 20:35 |
| jlvillal | As in the network that they connect to access the LAN and possibly the Internet. | 20:35 |
| *** praneshp has joined #openstack-ironic | 20:36 | |
| jlvillal | Assumes it is the 'real network' :) | 20:38 |
| *** toure has quit IRC | 20:38 | |
| *** penick has joined #openstack-ironic | 20:38 | |
| *** trown has quit IRC | 20:39 | |
| *** thrash has quit IRC | 20:39 | |
| *** thrash has joined #openstack-ironic | 20:41 | |
| *** thrash has quit IRC | 20:41 | |
| *** thrash has joined #openstack-ironic | 20:41 | |
| *** bigjools has quit IRC | 20:42 | |
| *** bigjools has joined #openstack-ironic | 20:46 | |
| rloo | JayF: you around to delete the comment quickly? https://review.openstack.org/#/c/189969/16 | 20:47 |
| *** alexpilotti has quit IRC | 20:50 | |
| *** alexpilotti has joined #openstack-ironic | 20:51 | |
| *** Sukhdev has quit IRC | 20:55 | |
| *** mkovacik_ has joined #openstack-ironic | 20:59 | |
| *** dprince has quit IRC | 21:01 | |
| JayF | yup will fix rloo | 21:03 |
| JayF | rloo: as soon as tests run locally will have that fix up | 21:06 |
| rloo | JayF: thx. glad to see you're testing, even if only removing a comment line :) | 21:06 |
| JayF | lol I always test | 21:06 |
| JayF | with that change in particular I wrote all the tests before even thinking about code | 21:06 |
| rloo | JayF: whoa. you're my new hero! | 21:07 |
| JayF | haha, You should see the hundreds of lines of downstream hwmgr tests I wrote that exposed all these bugs | 21:08 |
| *** raildo is now known as raildo-afk | 21:09 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Allow hardware managers to override clean step priority https://review.openstack.org/189969 | 21:13 |
| JayF | rloo: NobodyCam ^ minor changes to comments / making the test thing a local var as requested | 21:13 |
| *** mbound has quit IRC | 21:14 | |
| *** mbound has joined #openstack-ironic | 21:15 | |
| *** harlowja has quit IRC | 21:27 | |
| *** praneshp has joined #openstack-ironic | 21:30 | |
| *** harshs has quit IRC | 21:31 | |
| *** bigjools has quit IRC | 21:34 | |
| *** piet has quit IRC | 21:35 | |
| *** bigjools has joined #openstack-ironic | 21:37 | |
| *** bigjools has quit IRC | 21:37 | |
| *** bigjools has joined #openstack-ironic | 21:37 | |
| *** shakamunyi has quit IRC | 21:40 | |
| *** piet has joined #openstack-ironic | 21:41 | |
| *** dims has quit IRC | 21:42 | |
| *** dims_ has joined #openstack-ironic | 21:42 | |
| *** jamielennox|away is now known as jamielennox | 21:49 | |
| *** Sukhdev has joined #openstack-ironic | 21:55 | |
| *** alexpilotti has quit IRC | 21:59 | |
| *** alexpilotti has joined #openstack-ironic | 21:59 | |
| *** alexpilotti has quit IRC | 22:15 | |
| *** e0ne has quit IRC | 22:15 | |
| *** alexpilotti has joined #openstack-ironic | 22:19 | |
| *** harshs has joined #openstack-ironic | 22:25 | |
| *** jcoufal has quit IRC | 22:28 | |
| *** ionutbalutoiu has quit IRC | 22:33 | |
| *** ionutbalutoiu has joined #openstack-ironic | 22:34 | |
| *** [1]cdearborn has joined #openstack-ironic | 22:34 | |
| *** ijw has joined #openstack-ironic | 22:36 | |
| *** piet has quit IRC | 22:46 | |
| *** harlowja has joined #openstack-ironic | 22:50 | |
| harlowja | mkovacik_ estimate/plan is whatever we figure out together ;) | 22:51 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent: Add support for proxy servers during image build https://review.openstack.org/267219 | 22:52 |
| *** alexpilotti has quit IRC | 22:52 | |
| *** alexpilotti has joined #openstack-ironic | 22:52 | |
| harlowja | mkovacik_ to me its something we can dig-deeper on and figure out the plan and time and all that | 22:52 |
| *** dims_ has quit IRC | 22:54 | |
| *** ijw has quit IRC | 22:54 | |
| *** piet has joined #openstack-ironic | 22:54 | |
| *** alexpilotti has quit IRC | 23:01 | |
| *** alexpilotti has joined #openstack-ironic | 23:04 | |
| *** e0ne has joined #openstack-ironic | 23:06 | |
| *** ionutbalutoiu has quit IRC | 23:11 | |
| *** ionutbalutoiu has joined #openstack-ironic | 23:11 | |
| *** alexpilotti has quit IRC | 23:13 | |
| *** alexpilotti has joined #openstack-ironic | 23:14 | |
| *** alexpilotti has quit IRC | 23:24 | |
| *** alexpilotti has joined #openstack-ironic | 23:24 | |
| *** dims has joined #openstack-ironic | 23:27 | |
| *** [1]cdearborn has quit IRC | 23:39 | |
| *** smoriya_ has joined #openstack-ironic | 23:47 | |
| *** e0ne has quit IRC | 23:49 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!