| *** amotoki has joined #openstack-ironic | 00:03 | |
| *** amotoki has quit IRC | 00:08 | |
| *** rama_y has quit IRC | 00:08 | |
| *** Sukhdev has quit IRC | 00:30 | |
| *** baoli has joined #openstack-ironic | 00:38 | |
| *** baoli has quit IRC | 00:39 | |
| *** Sukhdev has joined #openstack-ironic | 00:47 | |
| *** baoli has joined #openstack-ironic | 00:56 | |
| *** baoli has quit IRC | 00:57 | |
| *** links has joined #openstack-ironic | 00:57 | |
| *** spandhe has quit IRC | 01:03 | |
| *** phuongnh has joined #openstack-ironic | 01:03 | |
| *** Goneri has joined #openstack-ironic | 01:13 | |
| *** piet has quit IRC | 01:16 | |
| *** greghaynes has quit IRC | 01:16 | |
| *** amotoki has joined #openstack-ironic | 01:25 | |
| *** Goneri has quit IRC | 01:28 | |
| *** amotoki has quit IRC | 01:30 | |
| *** baoli has joined #openstack-ironic | 01:30 | |
| *** greghaynes has joined #openstack-ironic | 01:30 | |
| *** baoli has quit IRC | 01:31 | |
| *** yuikotakadamori has joined #openstack-ironic | 01:39 | |
| *** Sukhdev has quit IRC | 01:50 | |
| *** baoli has joined #openstack-ironic | 01:54 | |
| *** sdake has joined #openstack-ironic | 01:54 | |
| *** rajinir has quit IRC | 01:54 | |
| *** baoli has quit IRC | 01:55 | |
| *** sacharya has joined #openstack-ironic | 01:59 | |
| *** sabeen has quit IRC | 02:02 | |
| *** sdake has quit IRC | 02:04 | |
| *** PollyZ has joined #openstack-ironic | 02:04 | |
| *** spandhe has joined #openstack-ironic | 02:09 | |
| *** sacharya_ has joined #openstack-ironic | 02:12 | |
| *** sabeen has joined #openstack-ironic | 02:12 | |
| *** rama_y has joined #openstack-ironic | 02:14 | |
| *** sacharya has quit IRC | 02:15 | |
| *** jraju has joined #openstack-ironic | 02:34 | |
| *** links has quit IRC | 02:36 | |
| *** PollyZ has quit IRC | 02:41 | |
| *** baoli has joined #openstack-ironic | 02:41 | |
| *** baoli has quit IRC | 02:45 | |
| *** PollyZ has joined #openstack-ironic | 02:47 | |
| *** amotoki has joined #openstack-ironic | 02:47 | |
| *** jaybeale has joined #openstack-ironic | 02:49 | |
| *** yuanying has quit IRC | 02:50 | |
| *** amotoki has quit IRC | 02:51 | |
| *** PollyZ has quit IRC | 02:59 | |
| *** harlowja_at_home has joined #openstack-ironic | 03:00 | |
| *** jraju has quit IRC | 03:03 | |
| *** PollyZ has joined #openstack-ironic | 03:04 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Replace rootwrap_config with root_helper config https://review.openstack.org/334522 | 03:10 |
|---|---|---|
| *** piet has joined #openstack-ironic | 03:14 | |
| *** spandhe has quit IRC | 03:15 | |
| *** rloo has quit IRC | 03:16 | |
| *** harlowja_at_home has quit IRC | 03:20 | |
| *** harlowja_at_home has joined #openstack-ironic | 03:22 | |
| *** PollyZ has quit IRC | 03:24 | |
| *** PollyZ has joined #openstack-ironic | 03:25 | |
| *** jaybeale has quit IRC | 03:27 | |
| *** amotoki has joined #openstack-ironic | 03:36 | |
| *** piet has quit IRC | 03:37 | |
| *** MattMan has quit IRC | 03:40 | |
| *** MattMan has joined #openstack-ironic | 03:41 | |
| *** PollyZ has quit IRC | 03:47 | |
| *** tiendc has joined #openstack-ironic | 03:48 | |
| *** amotoki has quit IRC | 03:50 | |
| *** PollyZ has joined #openstack-ironic | 03:51 | |
| *** jaybeale has joined #openstack-ironic | 03:51 | |
| *** yuanying has joined #openstack-ironic | 03:53 | |
| *** jaybeale has quit IRC | 03:55 | |
| *** PollyZ has quit IRC | 03:57 | |
| *** PollyZ has joined #openstack-ironic | 03:57 | |
| *** appu103_ has joined #openstack-ironic | 03:57 | |
| *** amotoki has joined #openstack-ironic | 03:58 | |
| *** rama_y has quit IRC | 04:04 | |
| *** tiendc_ has joined #openstack-ironic | 04:09 | |
| *** tiendc has quit IRC | 04:12 | |
| *** piet has joined #openstack-ironic | 04:12 | |
| *** PollyZ has quit IRC | 04:13 | |
| *** PollyZ_ has joined #openstack-ironic | 04:13 | |
| *** PollyZ_ has quit IRC | 04:17 | |
| *** PollyZ has joined #openstack-ironic | 04:18 | |
| *** amotoki has quit IRC | 04:18 | |
| *** adu has joined #openstack-ironic | 04:22 | |
| *** Sukhdev has joined #openstack-ironic | 04:25 | |
| *** amotoki has joined #openstack-ironic | 04:26 | |
| *** PollyZ has quit IRC | 04:27 | |
| *** ayoung has quit IRC | 04:29 | |
| *** harlowja_at_home has quit IRC | 04:30 | |
| *** PollyZ has joined #openstack-ironic | 04:31 | |
| *** spandhe has joined #openstack-ironic | 04:39 | |
| *** PollyZ has quit IRC | 04:48 | |
| *** sdake has joined #openstack-ironic | 04:49 | |
| *** piet has quit IRC | 04:50 | |
| *** sdake_ has joined #openstack-ironic | 04:51 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic-specs: Enhance Driver Interface for Soft Power Off and Inject NMI https://review.openstack.org/186700 | 04:53 |
| *** sdake_ has quit IRC | 04:53 | |
| *** sdake has quit IRC | 04:54 | |
| *** moshele has joined #openstack-ironic | 05:06 | |
| *** jaybeale has joined #openstack-ironic | 05:08 | |
| *** vmud213 has joined #openstack-ironic | 05:10 | |
| *** edand has joined #openstack-ironic | 05:18 | |
| *** rama_y has joined #openstack-ironic | 05:22 | |
| *** sdake has joined #openstack-ironic | 05:24 | |
| *** marios has joined #openstack-ironic | 05:25 | |
| *** rama_y has quit IRC | 05:26 | |
| *** rama_y has joined #openstack-ironic | 05:27 | |
| *** rcernin has joined #openstack-ironic | 05:27 | |
| *** marios has quit IRC | 05:27 | |
| *** sacharya_ has quit IRC | 05:27 | |
| *** marios has joined #openstack-ironic | 05:27 | |
| *** sacharya has joined #openstack-ironic | 05:27 | |
| *** marios has quit IRC | 05:31 | |
| *** marios has joined #openstack-ironic | 05:31 | |
| *** marios has quit IRC | 05:32 | |
| *** marios has joined #openstack-ironic | 05:32 | |
| *** rcernin has quit IRC | 05:34 | |
| *** sdake_ has joined #openstack-ironic | 05:36 | |
| *** sdake has quit IRC | 05:37 | |
| *** adu has quit IRC | 05:38 | |
| openstackgerrit | Saravanan KR proposed openstack/ironic-python-agent: Set the kernel args to the nodes at bootloader install https://review.openstack.org/335052 | 05:39 |
| *** ChubYann has quit IRC | 05:40 | |
| openstackgerrit | Saravanan KR proposed openstack/ironic-python-agent: Set the kernel args to the nodes at bootloader install https://review.openstack.org/335052 | 05:41 |
| *** rcarrillocruz has quit IRC | 05:46 | |
| *** fragatina has joined #openstack-ironic | 05:46 | |
| *** hoangcx has joined #openstack-ironic | 05:47 | |
| *** Sukhdev has quit IRC | 05:47 | |
| *** dtantsur|afk is now known as dtantsur | 05:55 | |
| dtantsur | Morning Ironic | 05:55 |
| *** spandhe has quit IRC | 05:55 | |
| *** fragatina has quit IRC | 05:57 | |
| *** fragatina has joined #openstack-ironic | 05:57 | |
| *** rama_y has quit IRC | 06:02 | |
| vsaienko | Morning dtatnsur, and all Ironic'ers! | 06:08 |
| *** harlowja_at_home has joined #openstack-ironic | 06:08 | |
| *** rcernin has joined #openstack-ironic | 06:09 | |
| *** sabeen has quit IRC | 06:14 | |
| *** rook- has quit IRC | 06:17 | |
| *** yogi has joined #openstack-ironic | 06:18 | |
| dtantsur | morning vsaienko | 06:22 |
| *** rbartal has joined #openstack-ironic | 06:24 | |
| edand | good morning everybody :) | 06:28 |
| *** jaybeale has quit IRC | 06:29 | |
| *** sdake_ has quit IRC | 06:30 | |
| *** pcaruana has joined #openstack-ironic | 06:37 | |
| *** harlowja_at_home has quit IRC | 06:39 | |
| *** dtantsur is now known as dtantsur|brb | 06:43 | |
| openstackgerrit | Merged openstack/ironic: Add Ironic specs process to the code contribution guide https://review.openstack.org/330167 | 06:46 |
| *** sacharya has quit IRC | 06:51 | |
| *** ifarkas has joined #openstack-ironic | 07:15 | |
| *** tesseract- has joined #openstack-ironic | 07:15 | |
| *** hoangcx has quit IRC | 07:15 | |
| *** fragatina has quit IRC | 07:36 | |
| *** itamarl has joined #openstack-ironic | 07:41 | |
| *** ccamacho|away is now known as ccamacho | 07:43 | |
| *** penick_ has joined #openstack-ironic | 07:44 | |
| *** penick has quit IRC | 07:44 | |
| *** penick_ is now known as penick | 07:44 | |
| *** rcarrillocruz has joined #openstack-ironic | 07:45 | |
| *** fragatina has joined #openstack-ironic | 07:50 | |
| openstackgerrit | Yibo Cai proposed openstack/ironic: Support multi arch deployment https://review.openstack.org/322632 | 07:56 |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:00 | |
| *** derekh has joined #openstack-ironic | 08:04 | |
| *** athomas has joined #openstack-ironic | 08:09 | |
| aarefiev | morning! | 08:09 |
| *** mjura has quit IRC | 08:10 | |
| *** Romanenko_K has joined #openstack-ironic | 08:10 | |
| vdrok | morning ironic! | 08:10 |
| *** mjura has joined #openstack-ironic | 08:10 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 08:11 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 08:11 |
| alineb | morning! | 08:12 |
| *** snecklifter has joined #openstack-ironic | 08:13 | |
| gbraad | morning | 08:19 |
| *** Nisha_away has joined #openstack-ironic | 08:20 | |
| gbraad | TheJulia: haven't been able to test today. been working on another task | 08:20 |
| openstackgerrit | Saravanan KR proposed openstack/ironic-specs: Provide a mechanism to configure kernel boot parameters https://review.openstack.org/331564 | 08:20 |
| gbraad | dtantsur|brb: feeling better? ;-) | 08:21 |
| *** ohamada has joined #openstack-ironic | 08:26 | |
| *** lucas-afk is now known as lucasagomes | 08:30 | |
| lucasagomes | morning all | 08:30 |
| Nisha_away | Morning/afternoon All!!! | 08:31 |
| *** electrofelix has joined #openstack-ironic | 08:43 | |
| *** dedery has joined #openstack-ironic | 08:48 | |
| *** sborkows has joined #openstack-ironic | 08:49 | |
| Nisha_away | dtantsur|brb, lucasagomes | 08:49 |
| Nisha_away | lucasagomes, have u tried inband inspection? | 08:49 |
| lucasagomes | Nisha_away, inspector? yes | 08:50 |
| Nisha_away | yes | 08:50 |
| dtantsur|brb | gbraad, morning, not substantially.. I had to repeat the same trick with waking up early | 08:52 |
| dtantsur|brb | but thanks for asking | 08:52 |
| dtantsur|brb | morning lucasagomes, Nisha_away, alineb, aarefiev, vdrok | 08:52 |
| lucasagomes | dtantsur|brb, hi there | 08:52 |
| * dtantsur|brb will disappear again in a few minutes | 08:52 | |
| aarefiev | hi dtantsur|brb | 08:53 |
| *** ohamada has quit IRC | 08:55 | |
| *** ohamada has joined #openstack-ironic | 08:55 | |
| *** bapalm has quit IRC | 08:57 | |
| *** yuanying_ has joined #openstack-ironic | 08:58 | |
| *** Nisha_away is now known as Nisha1 | 08:58 | |
| alineb | o/ dtantsur|brb | 08:58 |
| *** ildikov_ has joined #openstack-ironic | 08:59 | |
| *** bcornec2 has joined #openstack-ironic | 08:59 | |
| *** bapalm has joined #openstack-ironic | 09:00 | |
| *** fragatina has quit IRC | 09:00 | |
| *** TheJulia_ has joined #openstack-ironic | 09:02 | |
| *** ionutbalutoiu_ has joined #openstack-ironic | 09:04 | |
| *** vsaienko1 has joined #openstack-ironic | 09:04 | |
| *** mgould|afk is now known as mgould | 09:04 | |
| mgould | morning Ironic | 09:04 |
| *** purp_too has joined #openstack-ironic | 09:04 | |
| mgould | morning Nisha1 lucasagomes dtantsur|brb aarefiev alineb gbraad | 09:04 |
| lucasagomes | mgould, hi there | 09:04 |
| Nisha1 | mgould, morning | 09:05 |
| aarefiev | morning mgould | 09:05 |
| *** jralph has joined #openstack-ironic | 09:05 | |
| *** alex_xu_ has joined #openstack-ironic | 09:06 | |
| *** joemonk- has joined #openstack-ironic | 09:08 | |
| *** ohamada has quit IRC | 09:08 | |
| *** yuanying has quit IRC | 09:08 | |
| *** alex_xu has quit IRC | 09:08 | |
| *** TheJulia has quit IRC | 09:08 | |
| *** vsaienko has quit IRC | 09:08 | |
| *** jralph_ has quit IRC | 09:08 | |
| *** ionutbalutoiu has quit IRC | 09:08 | |
| *** mordred has quit IRC | 09:08 | |
| *** hparekh has quit IRC | 09:08 | |
| *** purp has quit IRC | 09:08 | |
| *** joemonk has quit IRC | 09:08 | |
| *** ildikov has quit IRC | 09:08 | |
| *** bcornec has quit IRC | 09:08 | |
| *** konetzed has quit IRC | 09:08 | |
| *** ionutbalutoiu_ is now known as ionutbalutoiu | 09:08 | |
| *** tiendc_ has quit IRC | 09:09 | |
| *** ildikov_ is now known as ildikov | 09:09 | |
| *** hparekh has joined #openstack-ironic | 09:09 | |
| *** edand has quit IRC | 09:09 | |
| alineb | hi mgould | 09:09 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add CLI tests to check JSON response body https://review.openstack.org/291239 | 09:10 |
| vdrok | morning dtantsur|brb lucasagomes gbraad Nisha1 mgould and alineb | 09:10 |
| *** stendulker has joined #openstack-ironic | 09:10 | |
| lucasagomes | alineb, vdrok hey there | 09:10 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add CLI tests to check JSON response body https://review.openstack.org/291239 | 09:10 |
| *** mordred has joined #openstack-ironic | 09:13 | |
| *** ohamada has joined #openstack-ironic | 09:13 | |
| alineb | morning vdrok lucasagomes :) | 09:14 |
| *** baoli has joined #openstack-ironic | 09:22 | |
| *** konetzed has joined #openstack-ironic | 09:26 | |
| *** baoli has quit IRC | 09:26 | |
| Nisha1 | if inspector-dhcp service is not able to send/recv dhcp requests to/from server what can be the issue? | 09:27 |
| moshele | Nisha1: maybe firewall | 09:29 |
| Nisha1 | do i need to do explicit setting in inspector.conf for it? | 09:30 |
| mgould | [machines not physically connected, machines not in same L2 broadcast domain, firewall, dhcp server not running, dhcp client not running] seem like the obvious things to check, in that order | 09:31 |
| mgould | oh, [machine is not plugged in] | 09:31 |
| mgould | unlikely, but very embarrassing if it turns out to be the problem after a morning of debugging :-) | 09:32 |
| openstackgerrit | Merged openstack/ironic-lib: Remove releasenotes/* https://review.openstack.org/335668 | 09:32 |
| *** snecklifter has quit IRC | 09:34 | |
| *** hoangcx has joined #openstack-ironic | 09:34 | |
| *** konetzed has quit IRC | 09:37 | |
| mgould | morning vdrok | 09:42 |
| *** ohamada has quit IRC | 09:46 | |
| *** ohamada has joined #openstack-ironic | 09:47 | |
| *** dtantsur|brb is now known as dtantsur | 09:49 | |
| dtantsur | morning mgould | 09:49 |
| dtantsur | Nisha1, the problem is usually your network configuration. e.g. some switches do not allow DHCP/boot requests by default | 09:49 |
| *** ohamada has quit IRC | 09:51 | |
| Nisha1 | dtantsur, yesterday we used the same setup for ironic deploy using ironic-client. so i had neutron port etc everything created for it. the dhcp range given for neutron subnet and inspector-dhcp is same. Today when i used the same setup for inspection it was able to get DHCP requests | 09:52 |
| Nisha1 | and inspector kernel and ramdisk did booted up, IPA said that inspection is disabled hence skipping | 09:53 |
| *** snecklifter has joined #openstack-ironic | 09:54 | |
| Nisha1 | but as u advised no neutron things are required i removed neutron port, subnet and tried ironic inspection | 09:54 |
| Nisha1 | dtantsur, after this it again started failing for dhcp | 09:54 |
| dtantsur | correct, because neutron probably didn't pass required parameters | 09:54 |
| dtantsur | Nisha1, start with checking logs of ironic-inspector dhcp server (depends on how you installed it) | 09:55 |
| Nisha1 | its brought up devstack plugin | 09:55 |
| dtantsur | so wait, are you using devstack and vms? | 09:56 |
| Nisha1 | dtantsur, i just chnaged the default ip address range from 172.x.x.x to 10.y.y.y as 172.x.x.x is not reachable from our servers | 09:56 |
| Nisha1 | dtantsur, i am using devstack | 09:56 |
| Nisha1 | dtantsur, my devstack is up on a VM, trying inspection for a baremetal using "ironic node-set-provision-state $NODE inspect" | 09:57 |
| dtantsur | I don't think we support devstack with baremetal | 10:00 |
| dtantsur | I'm pretty sure you'll have to do some manual network configuration with ovs | 10:00 |
| Nisha1 | dtantsur, will inspector work if inspector-dhcp is not up but neutron is up and working | 10:01 |
| dtantsur | no, it won't it | 10:02 |
| dtantsur | we don't rely on neutron currently at all | 10:02 |
| Nisha1 | dtantsur, ok, when u initiate inspection through ironic, do u need ironci port to be created? | 10:03 |
| dtantsur | no | 10:03 |
| Nisha1 | as i understand deploy_kernel and deploy_ramdisk parameters are also not required | 10:03 |
| dtantsur | the most likely cause of your problems is that https://github.com/openstack/ironic-inspector/blob/master/devstack/plugin.sh#L255-L264 was never tested and probably does not work with hardware as opposed to VMs | 10:03 |
| dtantsur | it might work, but I have no clue. you may want to ask sambetts, he's much better in virtual networking | 10:04 |
| * dtantsur -> food | 10:04 | |
| Nisha1 | ok | 10:04 |
| *** fragatina has joined #openstack-ironic | 10:10 | |
| *** fragatina has quit IRC | 10:17 | |
| *** snecklifter has quit IRC | 10:18 | |
| *** thrash|g0ne is now known as thrash | 10:26 | |
| *** fragatina has joined #openstack-ironic | 10:28 | |
| *** hshiina has joined #openstack-ironic | 10:32 | |
| *** fragatina has quit IRC | 10:32 | |
| *** fragatina has joined #openstack-ironic | 10:33 | |
| *** hoangcx has quit IRC | 10:38 | |
| moshele | sambetts: can you please review the infiniband spec https://review.openstack.org/#/c/265635/ and https://review.openstack.org/#/c/279940/ ( removed the vendor name from port ) | 10:43 |
| *** _fortis has quit IRC | 10:43 | |
| *** Nisha1 has quit IRC | 10:45 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Implement new heartbeat for AgentDeploy https://review.openstack.org/333964 | 10:47 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Promote agent vendor passthru to core API https://review.openstack.org/330018 | 10:47 |
| *** thrash is now known as thrash|bbl | 10:49 | |
| openstackgerrit | Yibo Cai proposed openstack/ironic: Support multi arch deployment https://review.openstack.org/322632 | 10:51 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: WIP: Discovery tempest test https://review.openstack.org/323769 | 10:52 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Negative tests for testing chassis-create command https://review.openstack.org/322788 | 10:54 |
| yibo | vdrok, would you have another look at https://review.openstack.org/322632 when free. thanks. (no hurry) | 10:57 |
| *** stendulker has quit IRC | 10:57 | |
| *** yibo has quit IRC | 10:58 | |
| *** _fortis has joined #openstack-ironic | 10:58 | |
| *** phuongnh has quit IRC | 11:01 | |
| *** rook- has joined #openstack-ironic | 11:06 | |
| *** Nisha_away has joined #openstack-ironic | 11:13 | |
| openstackgerrit | Moshe Levi proposed openstack/ironic-python-agent: Add vendor id, vendor name and product_id to interface information https://review.openstack.org/279940 | 11:14 |
| openstackgerrit | Moshe Levi proposed openstack/ironic-python-agent: WIP Infiniband Support https://review.openstack.org/334816 | 11:14 |
| dtantsur | new agent API works, woohoo: https://review.openstack.org/334523 | 11:35 |
| *** ccamacho is now known as ccamacho|lunch | 11:39 | |
| openstackgerrit | Vasyl Saienko proposed openstack/python-ironicclient: Updates supporting ironic-neutron integration https://review.openstack.org/206144 | 11:41 |
| openstackgerrit | Vasyl Saienko proposed openstack/python-ironicclient: Add possibility to work with portgroups https://review.openstack.org/335964 | 11:41 |
| openstackgerrit | lokesh s proposed openstack/ironic: Add support for the audit middleware https://review.openstack.org/272658 | 11:41 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 11:43 |
| *** athomas has quit IRC | 11:44 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/334420 | 11:45 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to use FLAT network setup https://review.openstack.org/334382 | 11:45 |
| *** Nisha_away has quit IRC | 11:45 | |
| *** links has joined #openstack-ironic | 11:48 | |
| *** links has quit IRC | 11:48 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Tests for testing node-create command https://review.openstack.org/262055 | 11:48 |
| *** athomas has joined #openstack-ironic | 11:50 | |
| *** sdake has joined #openstack-ironic | 11:55 | |
| *** sdake_ has joined #openstack-ironic | 11:57 | |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient: Add create command to ironic client https://review.openstack.org/328955 | 11:59 |
| *** jcoufal has joined #openstack-ironic | 11:59 | |
| *** sdake has quit IRC | 12:01 | |
| *** mgoddard_ has joined #openstack-ironic | 12:01 | |
| *** fragatina has quit IRC | 12:01 | |
| *** lucasagomes is now known as lucas-hungry | 12:03 | |
| *** mgoddard has quit IRC | 12:04 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Negative tests for port-related commands https://review.openstack.org/259354 | 12:04 |
| *** Fdaisuke has quit IRC | 12:07 | |
| *** hshiina has quit IRC | 12:07 | |
| *** hshiina has joined #openstack-ironic | 12:07 | |
| *** Fdaisuke has joined #openstack-ironic | 12:07 | |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient: Add create command to ironic client https://review.openstack.org/328955 | 12:07 |
| *** raildo-afk is now known as raildo | 12:13 | |
| *** daemontool has joined #openstack-ironic | 12:14 | |
| *** vishwanathj has joined #openstack-ironic | 12:17 | |
| *** Nisha_away has joined #openstack-ironic | 12:23 | |
| *** hshiina has quit IRC | 12:24 | |
| *** Fdaisuke has quit IRC | 12:24 | |
| *** baoli_ has joined #openstack-ironic | 12:24 | |
| *** Fdaisuke has joined #openstack-ironic | 12:24 | |
| *** hshiina has joined #openstack-ironic | 12:24 | |
| Nisha_away | sambetts, Hi | 12:25 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: WIP: Discovery tempest test https://review.openstack.org/323769 | 12:28 |
| dtantsur | I see suspicious failures in ironic grenade, I hope we're not broken | 12:28 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Tests for testing node-create command https://review.openstack.org/262055 | 12:30 |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Replace rootwrap_config with root_helper config https://review.openstack.org/334522 | 12:30 |
| dtantsur | I don't like it at all: http://logs.openstack.org/67/327667/21/experimental/gate-grenade-dsvm-ironic-inspector-nv/72979c2/logs/new/screen-n-cpu.txt.gz?level=WARNING | 12:33 |
| *** fragatina has joined #openstack-ironic | 12:34 | |
| TheJulia_ | Good morning everyone | 12:35 |
| *** TheJulia_ is now known as TheJulia | 12:35 | |
| dtantsur | morning TheJulia | 12:35 |
| TheJulia | so as for noop, I think operators would recognize what it means after a few seconds | 12:37 |
| dtantsur | yep | 12:37 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Implement new heartbeat for AgentDeploy https://review.openstack.org/333964 | 12:38 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Promote agent vendor passthru to core API https://review.openstack.org/330018 | 12:38 |
| openstackgerrit | Vasyl Saienko proposed openstack/python-ironicclient: Updates supporting ironic-neutron integration https://review.openstack.org/206144 | 12:38 |
| *** mgoddard_ has quit IRC | 12:38 | |
| *** mgoddard has joined #openstack-ironic | 12:38 | |
| TheJulia | dtantsur: as long as they've had their coffee :) | 12:39 |
| dtantsur | I'd not operate anything before getting coffee :) | 12:39 |
| TheJulia | lol | 12:41 |
| *** fellypefca has joined #openstack-ironic | 12:41 | |
| dtantsur | grenade seems down for real.. | 12:44 |
| TheJulia | :( | 12:45 |
| mgould | morning TheJulia | 12:45 |
| * dtantsur -> #openstack-nova | 12:45 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Create flavor with minimal values https://review.openstack.org/335986 | 12:45 |
| *** ccamacho|lunch is now known as ccamacho | 12:46 | |
| vsaienko1 | morning TheJulia | 12:47 |
| mat128 | good morning TheJulia, dtantsur, mgould and vsaienko1 | 12:49 |
| *** Nisha_away has quit IRC | 12:50 | |
| dtantsur | morning mat128 | 12:51 |
| *** piet has joined #openstack-ironic | 12:52 | |
| vsaienko1 | morning mat128, mgould | 12:53 |
| TheJulia | good morning mat128 vsaienko1 mgould | 12:54 |
| *** thiagop has joined #openstack-ironic | 12:56 | |
| thiagop | Good morning, Ironicers | 12:57 |
| *** PollyZ has joined #openstack-ironic | 12:59 | |
| *** makowals has joined #openstack-ironic | 12:59 | |
| zigo | mat128: The point being that "master" at the time of the release of everything else is different from the master when Ironic released 6.0.0. | 12:59 |
| zigo | Between these 2 moments, fixtures switched from <2 to >3 | 12:59 |
| mat128 | hmm | 12:59 |
| mat128 | good point | 12:59 |
| zigo | Meaning that Newton b1 wouldn't work with Ironic 6.0.0. | 12:59 |
| mat128 | so you need other projects to tag their master, right? | 13:00 |
| openstackgerrit | Aparna proposed openstack/ironic-python-agent: Support clean step erase_devices for ProliantHardwareManager https://review.openstack.org/335997 | 13:00 |
| zigo | mat128: No, I need Ironic to keep on using what's in the requirements.txt repository at the time of the latest beta release. | 13:01 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: DO NOT MERGE: testing grenade issue https://review.openstack.org/335998 | 13:01 |
| zigo | Either that, or release in sync with everyone else (and be release-managed). | 13:01 |
| *** amotoki has quit IRC | 13:01 | |
| mat128 | zigo: doesnt the openstack proposal bot ensures our requirements.txt is in sync with the one of other projects? | 13:01 |
| mat128 | your only issue becomes when does the tagging happen | 13:01 |
| zigo | mat128: At a given moment, yes. | 13:01 |
| zigo | The problem is that Ironic isn't released with everyone else. | 13:02 |
| zigo | Time passes, and then the global-requirements.txt of master are incompatible with the latest beta. | 13:02 |
| dtantsur | fwiw ironic is release:managed | 13:02 |
| *** dedery has quit IRC | 13:02 | |
| zigo | dtantsur: Why doesn't it get released at the same time as the DefCore services then? | 13:02 |
| dtantsur | just things like "fixtures switched from <2 to >3" should not happen abruptly IMO | 13:02 |
| mat128 | dtantsur: that means we release with everyone else, right? | 13:02 |
| dtantsur | no | 13:03 |
| mat128 | erm let me rephrase that | 13:03 |
| dtantsur | being release:managed is not related to it. just pointing | 13:03 |
| mat128 | ah | 13:03 |
| dtantsur | there are different release cadences, ironic's is release-with-intermediates or something like that | 13:03 |
| mat128 | https://governance.openstack.org/reference/tags/release_cycle-with-intermediary.html | 13:04 |
| zigo | 6.0.0 was released on the 22nd of June, Nova was released on the 2nd of June. | 13:04 |
| zigo | Right. | 13:04 |
| zigo | Anyway, you got my point! :) | 13:04 |
| mat128 | I have to agree with dtantsur, incompatible version requirements (<2 switched to >3) shouldn't happen abruptly | 13:05 |
| mat128 | isnt that a test dep tho? | 13:05 |
| zigo | Right. | 13:05 |
| zigo | But distros (like Debian, for who I'm doing the packaging) are running unit tests at build time. | 13:05 |
| *** lucas-hungry is now known as lucasagomes | 13:06 | |
| zigo | So it becomes a build-depends: python-fixtures, which has to agree with the other packages. | 13:06 |
| zigo | Anyway, it's not a big deal, I just need to put on hold the upload of Newton Ironic until the other projects are releasing Newton b2. | 13:06 |
| dtantsur | even so, did we start using new fixtures right away? I don't think so | 13:06 |
| mat128 | dtantsur: I think if you declare it, the package maintainer can't just overrule you | 13:07 |
| zigo | It's possible that I could just ignore the version of fixtures ... | 13:07 |
| dtantsur | mat128, we can and do | 13:07 |
| zigo | I didn't try. | 13:07 |
| mat128 | ah | 13:07 |
| mat128 | how do you ensure it works correctly? unittests? | 13:07 |
| zigo | Yup. | 13:07 |
| *** amotoki has joined #openstack-ironic | 13:07 | |
| dtantsur | mat128, RH packaging removes requirements files, so that pbr does not try to use them. yes, unit tests | 13:07 |
| zigo | That's what happens when the package builds. | 13:07 |
| * zigo tries "fixing" the test-requirements.txt to see what happens. | 13:08 | |
| *** sdake has joined #openstack-ironic | 13:10 | |
| fellypefca | Does anyone knows where Ironic passees the local_link_information to Neutron, on the Ironic/Neutron Integration? | 13:11 |
| *** appu103_ has quit IRC | 13:11 | |
| fellypefca | I perform a deploy using the Integration, and we manually populate the fields on Ironic Port, but these informations aren't passed to Neutron. | 13:11 |
| dtantsur | this feature is not merged. do you use in-progress patches? | 13:11 |
| fellypefca | sambetts can you help me with this? | 13:11 |
| *** appu103_ has joined #openstack-ironic | 13:11 | |
| *** sdake_ has quit IRC | 13:13 | |
| *** trown|outtypewww is now known as trown | 13:13 | |
| *** appu103_ has quit IRC | 13:13 | |
| *** rloo has joined #openstack-ironic | 13:17 | |
| vsaienko1 | fellypefca: there are 2 chains (1 - devstack changes: https://review.openstack.org/#/c/296432/, another python changes https://review.openstack.org/#/c/317392). You can find examples of devstack configuration under the multitenancy job http://logs.openstack.org/32/296432/53/experimental/gate-tempest-dsvm-ironic-multitenant-network-nv/91b167e/ | 13:17 |
| thiagop | dtantsur: can you take that look on the dynamic allocation patch that you couldn't take because of the midcycle? :) | 13:18 |
| thiagop | dtantsur: if you're not too busy, of course | 13:18 |
| fellypefca | vsaienko1 Thanks! I will check that | 13:20 |
| vsaienko1 | grenade is unstable, I'm concerned about it | 13:21 |
| *** ametts has joined #openstack-ironic | 13:21 | |
| *** electrofelix has quit IRC | 13:25 | |
| TheJulia | rloo: Out of curiosity, does https://review.openstack.org/#/c/272658/58/doc/source/deploy/api-audit-support.rst satisfy your questions? | 13:28 |
| rloo | TheJulia: oh. i haven't had a chance to review that. was my only question that it was missing documentation? | 13:29 |
| TheJulia | rloo: more like, use related questions, where do things end up, etc | 13:29 |
| rloo | TheJulia: the first sentence doesn't seem right. Does it mean the user has to enable the keystone middleware library? | 13:30 |
| rloo | TheJulia: the problem is that we aren't technical writers. | 13:31 |
| TheJulia | more so turning it on in ironic | 13:31 |
| TheJulia | indeed, we are not | 13:31 |
| rloo | TheJulia: I'll quickly comment. Am I holding up the patch? | 13:31 |
| TheJulia | and our docs scare people | 13:31 |
| rloo | TheJulia: yeah, you mentioned that before. too much detail for them, right? | 13:32 |
| TheJulia | rloo: eh, I'd like to see it land at some point | 13:32 |
| rloo | TheJulia: so much to review :-( I'm having a hard time prioritizing. | 13:33 |
| TheJulia | rloo: yup, too much information for easy digestion | 13:33 |
| TheJulia | rloo: no worries :) | 13:33 |
| TheJulia | in the mean time, before the rest of my cohorts are awake... I need to go put water in the RV tank | 13:33 |
| *** electrofelix has joined #openstack-ironic | 13:34 | |
| dtantsur | thiagop, I | 13:34 |
| dtantsur | I'm tracking the grenade problem | 13:34 |
| thiagop | dtantsur: np | 13:34 |
| dtantsur | otherwise I can, if you have a link handy | 13:34 |
| *** sacharya has joined #openstack-ironic | 13:34 | |
| openstackgerrit | Michael Turek proposed openstack/ironic: Devstack: When appropriate, Skip setting cleaning_network_uuid https://review.openstack.org/335567 | 13:35 |
| thiagop | dtantsur: https://review.openstack.org/#/c/286192/ | 13:35 |
| *** tiendc has joined #openstack-ironic | 13:35 | |
| *** karimb has joined #openstack-ironic | 13:35 | |
| *** moshele has quit IRC | 13:37 | |
| vdrok | morning rloo and TheJulia | 13:37 |
| *** sacharya has quit IRC | 13:39 | |
| vdrok | dtantsur: there is also this in the logs - http://logs.openstack.org/58/272658/58/check/gate-grenade-dsvm-ironic/3436213/logs/new/screen-q-dhcp.txt.gz?level=ERROR | 13:39 |
| dtantsur | ouch | 13:40 |
| *** vmud213 has quit IRC | 13:40 | |
| *** karimb has quit IRC | 13:43 | |
| rloo | hi vdrok | 13:46 |
| *** ayoung has joined #openstack-ironic | 13:48 | |
| fellypefca | vsaienko1 But exactly where Ironic passees the local_link_information to Neutron, on the Ironic or Nova code? | 13:48 |
| vsaienko1 | fellypefca: ironic | 13:48 |
| vsaienko1 | fellypefca: https://review.openstack.org/#/c/317393/14/ironic/drivers/modules/network/neutron.py L174 | 13:49 |
| dtantsur | morning rloo | 13:49 |
| vsaienko1 | morning rloo | 13:49 |
| rloo | hi dtantsur, vsaienko1! | 13:50 |
| *** livelace has joined #openstack-ironic | 13:50 | |
| *** hkominos has quit IRC | 13:51 | |
| fellypefca | vsaienko1 thanks again :) | 13:51 |
| *** hkominos has joined #openstack-ironic | 13:52 | |
| *** krtaylor has joined #openstack-ironic | 13:52 | |
| hkominos | hello openstackers from all over the world. Another glorious day of fighting with ironic :P | 13:52 |
| dtantsur | grenade gate is broken, probably by nova. do not bother approving or rechecking patches for now | 13:52 |
| dtantsur | hello hkominos | 13:52 |
| thiagop | hkominos: noooo, not fighting. Taming. :) | 13:53 |
| thiagop | hkominos: good morning! | 13:53 |
| *** livelace has quit IRC | 13:53 | |
| hkominos | hahahhaha .true story. | 13:54 |
| hkominos | I think there should be a forum with all the error messages. Just what they mean :P | 13:54 |
| dtantsur | not the worst idea, to be honest | 13:55 |
| dtantsur | we should populate http://docs.openstack.org/developer/ironic/deploy/troubleshooting.html with more stuff | 13:55 |
| thiagop | dtantsur: that's a place where a wiki could be handy | 13:55 |
| *** karimb has joined #openstack-ironic | 13:55 | |
| *** livelace has joined #openstack-ironic | 13:56 | |
| dtantsur | I'm not sure. it's good to review such things | 13:56 |
| mat128 | thiagop: I have an internal "owners manual" that could be stripped out of proprietary information and shared | 13:57 |
| mgould | mat128: do it do it do it | 13:57 |
| hkominos | Can I also populate that doc? | 13:57 |
| dtantsur | ofc | 13:57 |
| hkominos | I could write all the error codes i have | 13:57 |
| hkominos | :P | 13:57 |
| mgould | morning hkominos | 13:57 |
| dtantsur | hkominos, would be nice. Like having a "common errors" section. | 13:57 |
| dtantsur | mat128++++++++ | 13:57 |
| thiagop | I personally think that passing thru a review process would discourage people to write things | 13:57 |
| dtantsur | it does not discourage them from contributing code after all :) | 13:58 |
| dtantsur | anyway, giving bad advice is usually worse than giving no advice | 13:58 |
| hkominos | Maybe some reviewing should happen | 13:58 |
| mgould | not convinced - it doesn't discourage them sufficiently to turn away all contributors | 13:58 |
| mgould | making things easier usually means that they happen more | 13:59 |
| thiagop | But that's necessary for code since it can include bugs or something. A troubleshoot manual/cases is just a pointer to what could be wrong | 13:59 |
| rloo | mat128, hkominos, dtantsur, etc: please enhance the developer docs! (or are they deployer docs? :)) | 13:59 |
| dtantsur | thiagop, wrong manual is not any better than wrong code | 13:59 |
| dtantsur | well, we have a wiki, some people can edit that. but I personally won't point people to docs that the team didn't review | 13:59 |
| mgould | turning off heavy CI for docs-only patches would be a win | 13:59 |
| dtantsur | mgould, we do that ;) | 14:00 |
| mat128 | mgould: thats easy with zuul | 14:00 |
| hkominos | but I can start on some errors that i have faced and maybe the more knowledgeable devs will add more stuf | 14:00 |
| mgould | oh, excellent | 14:00 |
| dtantsur | docs changes only pass -pep8 and -docs jobs | 14:00 |
| dtantsur | now 3rd party CI also runs on them, but it's not blocking anything | 14:00 |
| mgould | right, cool | 14:00 |
| mgould | so I should work out how we do that and propose the same change to tripleo | 14:00 |
| dtantsur | would be awesome hkominos. there is no need to cover everything, just things you've found to be confusing | 14:00 |
| mat128 | Fun fact, our manual contains things like "PLEASE CONSULT AN IRONIC EXPERT BEFORE GOING FORWARD. | 14:01 |
| mat128 | THIS FIX DOES NOT APPLY TO ALL CASES." | 14:01 |
| mat128 | lmao | 14:01 |
| mgould | oh dear | 14:01 |
| hkominos | hahaha | 14:01 |
| thiagop | lol | 14:01 |
| dtantsur | mgould, example: https://github.com/openstack-infra/project-config/blob/master/zuul/layout.yaml#L1236-L1243 | 14:01 |
| mat128 | the resolution is something like "edit the db directly" | 14:02 |
| dtantsur | there is a similar bit above for dsvm jobs | 14:02 |
| mgould | dtantsur: thanks! | 14:02 |
| dtantsur | #pixiesay Do not edit my database by hand -m angry | 14:02 |
| PixieBoots | ʕᗒᴥᗕʔ՞: Do not edit my database by hand | 14:02 |
| dtantsur | :) | 14:02 |
| mat128 | dtantsur: when you operate a cloud and your customers are angry, editing the db is better than nothing at all :) | 14:03 |
| mat128 | but with all our -wait and abort stuff, this is mostly unneeded now | 14:03 |
| hkominos | speaking of which how can I (a noob ) propose some changes to the docs ? | 14:04 |
| hkominos | Is there a seperate git repo for the documents or something? | 14:04 |
| dtantsur | mat128, I know that feeling :) | 14:04 |
| dtantsur | hkominos, there is a docs directory in the main repo | 14:04 |
| mat128 | hkominos: docs/ in the ironic tree | 14:04 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP: Test if Nova broke grenade https://review.openstack.org/336042 | 14:04 |
| dtantsur | it's RST files essentially | 14:04 |
| mat128 | apparently people can't read | 14:04 |
| dtantsur | hkominos, when you need to check your change, run 'tox -edocs', it will end up with files in docs/build/html | 14:05 |
| *** adu has joined #openstack-ironic | 14:05 | |
| mat128 | an operator added to our guide that if "Failed to invoke ata_erase, falling back to shred" appears, you have to act on it | 14:05 |
| mat128 | he apparently missed the "falling back to shred" part | 14:05 |
| thiagop | dtantsur: my idea wasn't a troubleshooting manual (since we already have one), but more like a table with [1]symptoms, [2]what could be and [3]how to solve. I have something similar downstream to log CI problems and it does help | 14:05 |
| thiagop | But... | 14:06 |
| thiagop | whatever works | 14:06 |
| thiagop | :) | 14:06 |
| hkominos | that was my idea as well | 14:06 |
| mat128 | thiagop: this is exactly what I have | 14:06 |
| mat128 | similar to your washing machine "troubleshooting" section in the owners manual | 14:06 |
| dtantsur | this is a reasonable part of troubleshooting manual fwiw | 14:06 |
| dtantsur | and I'd still prefer it to be reviewerd | 14:06 |
| thiagop | mine is for CI issues | 14:06 |
| dtantsur | for example: some time ago downstream people developed tradition on removing all ironic nodes on every problem | 14:07 |
| mat128 | if your cover ironic/nova errors, it ends up being very similar | 14:07 |
| mat128 | dtantsur: ouch | 14:07 |
| dtantsur | sometimes it screwed up things heavily | 14:07 |
| thiagop | wth... | 14:07 |
| dtantsur | so I'd prefer to sanity-check recommendations, even if a lot of people believe they're correct :) | 14:07 |
| *** sdake_ has joined #openstack-ironic | 14:07 | |
| mat128 | writing (and maintaining) a big table in rst seems tedious | 14:09 |
| dtantsur | oh yeah | 14:09 |
| mat128 | any recommendation? looking at snmp.rst and having text paragraphs in there will not be cool | 14:09 |
| dtantsur | dunno, I'd keep a lot of titles, then you can build a TOC in the beginning | 14:10 |
| mgould | mat128: just use paragraphs | 14:10 |
| mgould | title for the major symptom, then "Possible cause:" and "Action:" | 14:10 |
| mgould | bonus: works better on narrow screens | 14:10 |
| *** sdake has quit IRC | 14:11 | |
| mat128 | :D | 14:11 |
| mat128 | ok I'll try it | 14:11 |
| *** amotoki has quit IRC | 14:13 | |
| *** rbudden has joined #openstack-ironic | 14:13 | |
| *** rcernin has quit IRC | 14:14 | |
| *** wajdi has joined #openstack-ironic | 14:15 | |
| mat128 | trying rendering the first 2 troubleshooting steps | 14:16 |
| *** sdake_ is now known as sdake | 14:20 | |
| *** rbartal has quit IRC | 14:21 | |
| *** mjura has quit IRC | 14:23 | |
| mgould | dtantsur: https://review.openstack.org/#/c/336049/ | 14:23 |
| *** itamarl has quit IRC | 14:24 | |
| *** rcernin has joined #openstack-ironic | 14:29 | |
| *** jistr is now known as jistr|mtg | 14:31 | |
| *** adu has quit IRC | 14:33 | |
| *** PollyZ has quit IRC | 14:33 | |
| *** rbartal has joined #openstack-ironic | 14:33 | |
| *** krtaylor has quit IRC | 14:34 | |
| *** fellypefca has quit IRC | 14:35 | |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: (WIP) Document operators troubleshooting steps https://review.openstack.org/336055 | 14:35 |
| mat128 | rloo, hkominos, dtantsur, thiagop: ^ | 14:36 |
| mat128 | testing the waters, will add more step if thats something you like | 14:36 |
| dtantsur | mat128, have to run pretty soon, but quick comment: you have trailing whitespace there, marked as red squares in gerrit UI | 14:37 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: (WIP) Document operators troubleshooting steps https://review.openstack.org/336055 | 14:37 |
| mat128 | yeah, fixing them atm | 14:37 |
| mat128 | using an alternate editor that does not strip them out for me :( | 14:37 |
| *** PollyZ has joined #openstack-ironic | 14:38 | |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: (WIP) Document operators troubleshooting steps https://review.openstack.org/336055 | 14:38 |
| *** phuongnh has joined #openstack-ironic | 14:40 | |
| *** sacharya has joined #openstack-ironic | 14:40 | |
| rloo | mat128: looking... | 14:44 |
| *** sacharya has quit IRC | 14:45 | |
| *** rcernin has quit IRC | 14:46 | |
| *** pcaruana has quit IRC | 14:46 | |
| rloo | mat128: i thought you were all talking about a table or something with errors. I am fine with whatever format works for deployers/operators to find the info they want. | 14:46 |
| mat128 | mgould suggested paragraphs, a table seems tedious in rst | 14:47 |
| mat128 | unless you have good example to show me, all I can find is https://github.com/openstack/ironic/blob/master/doc/source/drivers/snmp.rst | 14:47 |
| mat128 | rloo: internally it looks like this https://www.dropbox.com/s/ledsqk0a90s6a7k/Screen%20Shot%202016-06-30%20at%2010.48.02%20AM.png?dl=0 | 14:48 |
| rloo | mat128: oh. that's nice. your table. | 14:48 |
| mat128 | http://docutils.sourceforge.net/docs/user/rst/quickref.html#tables | 14:49 |
| rloo | mat128: i'm not a doc person. i suppose if someone can figure out the table stuff, it'd be easy for people to copy/paste rows and add their stuff. but let's just get the info in and maybe someone will come along and prettyify it later. | 14:49 |
| mat128 | I guess.. I wanted it to be useful from day1 ;) | 14:50 |
| dtantsur | status update: we got some progress with the root cause of the grenade failure. seems a side effect of a new nova feature. | 14:50 |
| rloo | mat128: just add a bug and put 'low-hanging-fruit' tag on it :) | 14:50 |
| mat128 | I can try converting to table and see what it looks like | 14:50 |
| mat128 | heh | 14:50 |
| mat128 | dtantsur: which feature? | 14:50 |
| rloo | mat128: my only concern with a table is whether it might be too wide. otherwise, i like it :) | 14:50 |
| dtantsur | mat128, I didn't quite get it to be honest | 14:50 |
| dtantsur | not related to us really | 14:51 |
| rloo | dtantsur: thx. do we have an idea when it might be fixed? | 14:51 |
| dtantsur | folks seem to be hacking on it | 14:51 |
| dtantsur | we had to fix insufficient logging in nova first... | 14:52 |
| rloo | mat128: i am guessing dtantsur means that it seems nova-related, not ironic-related. (that's what i think/said in your patch) | 14:52 |
| mat128 | rloo: I was thinking dtantsur was talking about the grenade failure | 14:53 |
| mat128 | not my patch | 14:53 |
| rloo | mat128: oh. yeah, could be that too! :) | 14:53 |
| mat128 | rloo: you are right, the first entries are mostly about nova. The thing is my actual operators don't know if it's nova or ironic until they use that users guie | 14:54 |
| mat128 | similar to how your car's troubleshooting steps would not split a "not starting" issue under "electrical problems" or "starter" because it could as well be an out-of-fuel stiuation | 14:54 |
| rloo | mat128: you want to point your operators to one place to troubleshoot. what do they know? horizon? | 14:55 |
| mat128 | most of the time they use the CLI | 14:55 |
| *** hshiina has quit IRC | 14:55 | |
| rloo | mat128: you/we need an openstack troubleshooting thing. | 14:55 |
| mat128 | I guess.. yeah | 14:55 |
| mat128 | for generic issues | 14:55 |
| rloo | mat128: which cli then? soon it'll all be 'openstack ...'. but now? nova X, or ironic Y? | 14:55 |
| *** sdake_ has joined #openstack-ironic | 14:56 | |
| mat128 | given how patches for the latest metal stuff has just landed (tried it this morning), they are using the older CLIs | 14:56 |
| rloo | mat128: what 'metal stuff'? is this your downstream? | 14:56 |
| mat128 | (side note: I was surprised by ``openstack baremetal list``, I thought we had agreed on ``openstack node list`` | 14:56 |
| mat128 | rloo: ironic commits in OSC | 14:57 |
| rloo | mat128: for everything ironic related, it is 'openstack baremetal ...', | 14:57 |
| mat128 | but that differs from other projects | 14:57 |
| rloo | mat128: we had a big discussion about it, and it final word(s) are in the spec. | 14:57 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: DO NOT MERGE: testing grenade issue https://review.openstack.org/335998 | 14:57 |
| mat128 | they dont have openstack compute, etc. | 14:57 |
| mat128 | but yeah | 14:57 |
| mat128 | https://review.openstack.org/#/q/status:merged+project:openstack/python-ironicclient+branch:master+topic:osc-plugin | 14:57 |
| mat128 | all very new | 14:57 |
| jlvillal | dtantsur, I am trying this: https://review.openstack.org/336041 | 14:58 |
| *** skramaja is now known as skramaja|afk | 14:58 | |
| jlvillal | dtantsur, rloo: Good news is that Andrew Laski and Matt Riedmann both seem to be working on it :) | 14:58 |
| lucasagomes | rloo, some news on the log spec, I can't use "/" in the timestamp :-/ | 14:58 |
| rloo | mat128: yup http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/ironicclient-osc-plugin.html | 14:58 |
| dtantsur | jlvillal, it might help too, lets see if https://review.openstack.org/#/c/336069/ works around it | 14:58 |
| rloo | lucasagomes: ok. use something else then. | 14:58 |
| lucasagomes | rloo, linux filesystems won't accept '/' as part of the file name | 14:58 |
| lucasagomes | rloo, yeah, I will put a patch to update the spec soon | 14:59 |
| rloo | lucasagomes: sorry about that. | 14:59 |
| *** sdake has quit IRC | 14:59 | |
| lucasagomes | it's all good | 14:59 |
| *** sacharya has joined #openstack-ironic | 14:59 | |
| rloo | lucasagomes: btw, did you see my comments on the specs-ops stuff? | 14:59 |
| *** rcernin has joined #openstack-ironic | 14:59 | |
| lucasagomes | rloo, nop, have the link handy | 14:59 |
| lucasagomes | ? | 14:59 |
| rloo | lucasagomes: https://review.openstack.org/#/c/308398/ | 14:59 |
| rloo | lucasagomes: i can guess but want to be sure. what does <all-in> do? | 15:00 |
| dtantsur | jlvillal, see #openstack-nova. also good morning :) | 15:00 |
| rloo | lucasagomes: specifically, i was looking at the tests, and wondering why for the <all-in>, they test with str(values) | 15:00 |
| rloo | lucasagomes: is it because the user has to enter their list of strings, as a string itself? | 15:01 |
| lucasagomes | rloo, check if all elements is contained in a list/tuple | 15:01 |
| lucasagomes | afaik | 15:01 |
| * lucasagomes runs a test | 15:01 | |
| lucasagomes | rloo, apparently yeah, '<all-in> foo bar' etc | 15:01 |
| rloo | lucasagomes: and the values too, as input to the matcher? | 15:02 |
| *** krtaylor has joined #openstack-ironic | 15:02 | |
| lucasagomes | rloo, http://paste.openstack.org/show/524282/ | 15:03 |
| *** mat128 is now known as mat128|afk | 15:03 | |
| rloo | lucasagomes: but the unit tests are something like specs_matcher.match("['foo', 'bar']", '<all-in> foo bar') | 15:04 |
| mgould | yeah, all-in means "are all these values in the thing being tested", not the other way round | 15:04 |
| rloo | lucasagomes: the first arg is a string. | 15:04 |
| mgould | it's really confusing | 15:04 |
| dtantsur | #pixiesay pure awesomeness... -m meh | 15:04 |
| PixieBoots | ¯\_(ツ)_/¯: pure awesomeness... | 15:04 |
| rloo | mgould: especially since there is no documentation. took me a few secs to realize s== wasn't the same as ==... | 15:04 |
| mgould | yep | 15:04 |
| lucasagomes | rloo, yeah, strange works with string too | 15:05 |
| *** moorryan has joined #openstack-ironic | 15:05 | |
| * mgould repeats his point about how the language was crippled so it would be easy to write an ad-hoc parser for it | 15:05 | |
| lucasagomes | rloo, I think all it does is to use the "in" operator | 15:05 |
| *** thrash|bbl is now known as thrash | 15:05 | |
| lucasagomes | so it can be string or lists, dicts, tuples etc | 15:05 |
| rloo | lucasagomes: i think there might be a bug if it expects a string instead of a list. | 15:05 |
| lucasagomes | polymorphous :-) | 15:05 |
| thiagop | mat128|afk: +1 for table format | 15:05 |
| lucasagomes | rloo, I think it's does not matter | 15:05 |
| rloo | lucasagomes: on the other hand, i don't really care if the initial code has a bug. | 15:05 |
| lucasagomes | it uses the in operator in python | 15:05 |
| lucasagomes | '<all-in>': lambda x, y: all(val in x for val in y), | 15:05 |
| rloo | lucasagomes: try specs_matcher.match("['foo', 'bar']", '<all-in> f') | 15:06 |
| mgould | "RHS is a subset of LHS" | 15:06 |
| mat128|afk | thiagop: still not afk, will leave in a few seconds :), will try the table format but with 80 chars and paragrahs of text, that will be fun =) | 15:06 |
| lucasagomes | rloo, that would be a true | 15:06 |
| rloo | lucasagomes: yes, it is true. but 'f' is not in the first list, 'foo' is. | 15:07 |
| lucasagomes | rloo, it's probably used to find substrings too | 15:07 |
| rloo | lucasagomes: so should it be true? | 15:07 |
| lucasagomes | rloo, that is not a list, that's a string | 15:07 |
| lucasagomes | if you pass a list it will be false | 15:07 |
| rloo | lucasagomes: i know. | 15:07 |
| rloo | lucasagomes: what i don't know, if whether that was the intent or not. | 15:07 |
| lucasagomes | rloo, oh, that I don't know either :-/ we should check with nova | 15:08 |
| dtantsur | I have to go already folks, I'm leaving the gate in jlvillal's capable hands :) | 15:08 |
| dtantsur | see you tomorrow | 15:08 |
| rloo | bye dtantsur! | 15:08 |
| *** dtantsur is now known as dtantsur|afk | 15:08 | |
| lucasagomes | dtantsur|afk, see ya | 15:08 |
| lucasagomes | rloo, let's ask in the nova channel | 15:08 |
| rloo | lucasagomes: unfortunately?, I think we need to know what it was meant to do, to make sure that the new version does a similar (or not). | 15:08 |
| rloo | lucasagomes: do you want to ask? | 15:09 |
| lucasagomes | I can, but I need to do some checks first... like see if extra_specs does support lists | 15:09 |
| lucasagomes | or only strings to formulate a better question | 15:10 |
| *** sdake has joined #openstack-ironic | 15:10 | |
| rloo | lucasagomes: ok thx. I am missing that background too. Just too lazy to look into it. | 15:10 |
| rloo | lucasagomes: i looked at the original nova patch and didn't see any comments/anything useful to help me understand. | 15:10 |
| lucasagomes | rloo, yeah, I think even the nova folks don't understand these bits well | 15:11 |
| lucasagomes | rloo, http://docs.openstack.org/developer/nova/filter_scheduler.html#ComputeCapabilitiesFilter | 15:11 |
| lucasagomes | here's some more info, but not much more | 15:11 |
| * lucasagomes wonders if we should really use the nova syntax now | 15:11 | |
| *** sborkows has quit IRC | 15:11 | |
| jlvillal | dtantsur|afk, Good night! | 15:11 |
| lucasagomes | I mean, we want to make it consistent but it kinda sucks :-/ (specially that '=' == '>=' | 15:12 |
| lucasagomes | )* | 15:12 |
| jlvillal | lucasagomes, Can we have nova syntax plus alternatives? Or is that too ugly? | 15:12 |
| rloo | lucasagomes: yeah. so i don't think we can change the existing operators unless we don't want to be compatible with nova | 15:12 |
| lucasagomes | jlvillal, that was the initial idea to have nova and inspector syntax | 15:12 |
| *** piet has quit IRC | 15:12 | |
| *** sdake_ has quit IRC | 15:13 | |
| lucasagomes | but that's hard, cause nova is type-aware for some stuff | 15:13 |
| lucasagomes | e.g s== and == | 15:13 |
| lucasagomes | s== (for strings) | 15:13 |
| rloo | lucasagomes: and if oslo folks weren't going to replace that code with new code, i wouldn't care. however... since they are going to replace the code, i want to make sure it IS a replacement. | 15:13 |
| lucasagomes | both would map "eq" in inspector... I would really like to support either one or another | 15:13 |
| mgould | lucasagomes: that's not exactly type-awareness, it's "equality as strings" versus "equality as numbers" | 15:13 |
| lucasagomes | rloo, yeah that's unsure as well... if they replace will nova use it? | 15:14 |
| rloo | lucasagomes: i don't really care if nova will use it. we want to use it. | 15:14 |
| mgould | so 1 == 1.0 but 1 !s== 1.0 | 15:14 |
| rloo | lucasagomes: unless we just copy nova's into ironic-lib, like you did with your patch. but that seems wrong. | 15:14 |
| lucasagomes | mgould, yeah true | 15:14 |
| mgould | if we don't want to be compatible with nova, we should start with a language that isn't horrible IMHO | 15:14 |
| rloo | mgould: i suspect that is what lucasagomes meant by 'type aware'. | 15:14 |
| mgould | ah, OK | 15:15 |
| mgould | "type aware" to me means "values have types, and I'll do something appropriate based on that" | 15:15 |
| lucasagomes | mgould, is right | 15:16 |
| lucasagomes | but let's not get off topic with it | 15:16 |
| mgould | as opposed to "values do not have types, and you specify what type you want to treat them as in the operator" | 15:16 |
| mgould | sure | 15:16 |
| rloo | mgould: i think the problem is that the values come in as strings. | 15:16 |
| rloo | mgould: so either you have to indicate that the string you are entering should be treated as a number, or indicate the 'type' via the operator. | 15:17 |
| * mgould nods | 15:17 | |
| NobodyCam | Good morning Ironicers | 15:17 |
| mgould | morning NobodyCam | 15:17 |
| mgould | rloo: this is how Perl works, FWIW | 15:18 |
| NobodyCam | morning mgould | 15:18 |
| mgould | if our values-to-be-matched have types already, adopting operator-forces-type semantics might not be a great idea | 15:19 |
| rloo | lucasagomes: looking at the nova doc, it is clear that the extra_specs info will be a string, and it is in the extra specs that the "<operator> stuff" is specified. but is the value also a string? based on the unit tests, i am guessing yes. which means i think what i noticed is probably a bug. | 15:19 |
| rloo | morning NobodyCam | 15:19 |
| *** rcernin has quit IRC | 15:19 | |
| NobodyCam | morning rloo | 15:20 |
| NobodyCam | morning lucasagomes | 15:20 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to use FLAT network setup https://review.openstack.org/334382 | 15:21 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/334420 | 15:21 |
| lucasagomes | NobodyCam, morning | 15:22 |
| NobodyCam | :) | 15:23 |
| mgould | rloo: sounds plausible | 15:23 |
| *** jistr|mtg is now known as jistr | 15:23 | |
| lucasagomes | rloo, right... do you think we should stick to this nova syntax even tho it's pretty bad ? Nobody seems to like it :-/ | 15:23 |
| mgould | so we should ask in #openstack-nova if the value matched by <all-in> is a string? | 15:23 |
| lucasagomes | mgould, yeah | 15:23 |
| mgould | cool, I'll do so | 15:24 |
| lucasagomes | not sure who would be a good person to answer that in nova, maybe someone that works with the scheduler bits like sylvan | 15:24 |
| mgould | done | 15:25 |
| rloo | lucasagomes: i haven't been paying much attention. you want the operators for something... and we/ironic might want the operators for doing filtering, right? | 15:25 |
| lucasagomes | rloo, yeah I want operators for the root device hints | 15:25 |
| lucasagomes | rloo, to be able to do things like find me a disk which size is bigger than | 15:25 |
| lucasagomes | things like that | 15:26 |
| *** spandhe has joined #openstack-ironic | 15:26 | |
| *** sacharya_ has joined #openstack-ironic | 15:27 | |
| *** tesseract- has quit IRC | 15:27 | |
| *** spandhe_ has joined #openstack-ironic | 15:29 | |
| *** sacharya has quit IRC | 15:30 | |
| *** spandhe has quit IRC | 15:30 | |
| *** spandhe_ is now known as spandhe | 15:30 | |
| * mgould watches his question scroll off the top unanswered | 15:31 | |
| mgould | I need to get better at asking for help in IRC - any suggestions? | 15:31 |
| rloo | mgould: it also depends on who is around and who might know the answer, etc... | 15:31 |
| * mgould nods | 15:32 | |
| lucasagomes | mgould, maybe ask bauzas directly (that's sylvan the scheduler dude) | 15:32 |
| mgould | cool, thanks | 15:32 |
| rloo | lucasagomes: we seem to go around in circles over this. if we decide to provide some sort of search capability in ironic (which makes sense to me regardless of nova scheduling), what operators should we use? would it make sense to use nova's, or a diff set? | 15:33 |
| lucasagomes | rloo, honestly I have no idea... I pick the nova one because of consistency | 15:34 |
| lucasagomes | but in this case apparently nobody likes that syntax | 15:34 |
| lucasagomes | (with reason) so, I'm really divided | 15:34 |
| rloo | lucasagomes: yeah, consistency makes sense. nobody likes the syntax cuz it doesn't make sense. | 15:35 |
| lucasagomes | rloo, yeah, it's very confusing... I wonder if we, Ironic, should come up with something in ironic-lib and use between our projects | 15:35 |
| rloo | lucasagomes: how would operators use the feature you're doing? would those same operators also be using nova and maybe doing filtering using nova's operators? | 15:35 |
| lucasagomes | actually we don't need to come up with something, cause inspector already does it different (and it's fine) | 15:35 |
| lucasagomes | rloo, no, for the feature I want to use operators is an ironic only | 15:36 |
| lucasagomes | it's not about scheduling the nodes, but about finding which hard disk ironic should pick for the deployment | 15:36 |
| lucasagomes | in a multi disk node | 15:36 |
| rloo | lucasagomes: and that would usually be a different set of human operators, than ones that interact with nova? | 15:36 |
| lucasagomes | rloo, correct | 15:37 |
| lucasagomes | well | 15:37 |
| lucasagomes | sorry I misread | 15:37 |
| lucasagomes | it could be the same ones in principal, greater than, equal etc... | 15:37 |
| rloo | lucasagomes: so if they are different sets of people, then it doesn't matter much. but if were an operator and wanted to supply hints to ironic and wanted to do nova extra-specs filtering, i'd be very annoyed. | 15:37 |
| lucasagomes | they all can be used | 15:37 |
| lucasagomes | rloo, yeah, that's a good point | 15:38 |
| lucasagomes | the ones I want to use is only set by operators in ironic, it has nothing to do with the nova <-> ironic iteraction | 15:38 |
| lucasagomes | rloo, but still, if they were all the same it would be good | 15:39 |
| rloo | lucasagomes: it also seems to me that other projects may have/will have/ a need for such operators. would be nice if it was consistent. | 15:39 |
| lucasagomes | yeah | 15:39 |
| rloo | lucasagomes: eg, inspector did something. | 15:39 |
| lucasagomes | inspector could use it for e.g | 15:39 |
| lucasagomes | yeah | 15:39 |
| rloo | lucasagomes: gad. i hate spending time on these things. just want to *code* it and get on with things. but those darn operators... | 15:40 |
| lucasagomes | rloo, fwiw, ceilometer has it's own syntax too | 15:40 |
| rloo | lucasagomes: ah, see. what a mess. | 15:40 |
| lucasagomes | rloo, exactly, I honestly just wanted to code it | 15:40 |
| lucasagomes | heh | 15:40 |
| lucasagomes | rloo, maybe we should just use the inspector ones and be happy | 15:40 |
| mgould | rloo: <all-in> matches collections, see https://review.openstack.org/#/c/308398 | 15:40 |
| mgould | bah, c&p | 15:40 |
| mgould | https://github.com/openstack/nova/blob/master/nova/tests/unit/scheduler/filters/test_extra_specs_ops.py#L202-L235 | 15:40 |
| mgould | right, looks like the oslo.utils patch should include all the tests in that file | 15:41 |
| mgould | huh, looks like it does | 15:42 |
| rloo | mgould: yes, the oslo.utils patch doesn't include all these tests. but like i mentioned before, if you change one of the <all-in> reqs to be one character, that'll match, and i'm not sure it should. | 15:42 |
| rloo | mgould: i mean, it DOES include all those tests | 15:42 |
| mgould | oh, right, we call str(values) | 15:42 |
| mgould | how odd | 15:42 |
| rloo | mgould: so i think there is a bug in the original code. i don't think the intent is for 'e' to match "['aes', 'mmx', 'aux']" | 15:43 |
| mgould | rloo: do you want to jump in on openstack-nova | 15:43 |
| mgould | ? | 15:43 |
| rloo | mgould: not really. maybe i should file a bug and just continue my merry way. | 15:44 |
| *** piet has joined #openstack-ironic | 15:46 | |
| *** jaybeale has joined #openstack-ironic | 15:48 | |
| *** harlowja_at_home has joined #openstack-ironic | 15:48 | |
| vdrok | rloo: so, I don't know which one to use - network interface or network driver or network interface implementation | 15:49 |
| *** harlowja_at_home has quit IRC | 15:49 | |
| vdrok | which one do you like the most? :) | 15:49 |
| rloo | vdrok: OH. Personally, I don't like 'driver' because we use 'driver' for too many things. | 15:49 |
| *** jaybeale has quit IRC | 15:50 | |
| rloo | vdrok: we have the driver associated with a node, and the driver has interfaces, and one of those interfaces is a network interface. | 15:50 |
| rloo | what is a network driver? is that different from a network interface? | 15:50 |
| vdrok | rloo: meaning that we have to call 'flat' a network interface implementation? | 15:51 |
| rloo | vdrok: there *could* be a 'network driver' and a 'network interface' but if we use those terms, we should be clear what we are taking about. | 15:51 |
| rloo | vdrok: yes, I think of 'flat' as a network interface. | 15:51 |
| rloo | vdrok: that's how it is set up in setup.cfg, right? | 15:51 |
| vdrok | in the composition spec it's just interface everywhere | 15:51 |
| *** Nisha_away has joined #openstack-ironic | 15:51 | |
| rloo | vdrok: 'ipmi' is a power interface. or is it a power driver? | 15:52 |
| vdrok | rloo: yep, what I don't like here is that NetworkInterface is an abstract class - so eg flat is its implementation. maybe it's too much of detail and we can go with just interface, not interface implementation | 15:53 |
| rloo | vdrok: don't think we need to mention 'implementation'. | 15:54 |
| vdrok | rloo: ok, I'll change it then, thx | 15:54 |
| rloo | vdrok: although technically you are right. it is an implementation of ... But I'm not sure when people describe these things, they mention 'implementation'. | 15:54 |
| *** fragatina has quit IRC | 15:54 | |
| *** fragatina has joined #openstack-ironic | 15:55 | |
| vsaienko1 | Ironic'ers I'm thinking to setup saparate job that mirror grenade configuration but without upgrades, that launches smoke tests | 15:55 |
| vdrok | it is just a bit weird - we add network interface in the base class and then set network interface per node | 15:55 |
| vsaienko1 | it will allow catch errors, that related to concurrent builds wdt? | 15:56 |
| rloo | vdrok: it is going to be a similar thing for all X_interfaces. | 15:57 |
| vdrok | rloo: yup | 15:57 |
| rloo | vdrok: based on the driver composition stuff. | 15:57 |
| vdrok | so the spec is approved, I'll go with that :) | 15:57 |
| vdrok | vsaienko1: we have grenade voting on ironic, maybe we could add that job to other projects that we have jobs on, eg neutron and nova? | 15:58 |
| rloo | vdrok: i haven't quite figured out how to go from this network_interface to the driver composition stuff, since nodes will be using classic driver + .network_interface, but I'll let dtantsur|afk think about it. | 15:58 |
| vdrok | replacing the usual pxe_ssh job | 15:58 |
| *** mgoddard has quit IRC | 15:59 | |
| *** dims has quit IRC | 16:00 | |
| mgould | OK, ISTM that the problem rloo noticed is just a bug in the tests | 16:00 |
| rloo | mgould: ? | 16:01 |
| mgould | [for the specs_matcher] | 16:01 |
| rloo | mgould: what's the bug in the tests? | 16:01 |
| mgould | wrapping values with str | 16:01 |
| mgould | but I'm having trouble getting nova folks to understand the issue | 16:02 |
| rloo | mgould: heh. that was my original question. why are they wrapped in strs? but ALL the tests are strings. So I think the intent is that the matcher is used with strings. | 16:02 |
| mgould | this is not the impression I get from bauzas | 16:02 |
| rloo | mgould: OH. then the tests are wrong... | 16:02 |
| mgould | https://review.openstack.org/#/c/336094/ might shed some light | 16:03 |
| mgould | or rather, might help me explain that there is a problem | 16:03 |
| mariojv | has anyone ever had an issue with the limit parameter in this API call not working? http://docs.openstack.org/developer/ironic/webapi/v1.html#get--v1-nodes-detail | 16:03 |
| mariojv | specifically, i'm wondering if there's a maximum limit | 16:04 |
| mgould | rloo: http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2016-06-30.log.html#t2016-06-30T15:53:38 | 16:04 |
| vsaienko1 | vdrok, yeah I will be useful, but grenade is a bit slow... | 16:06 |
| *** dims has joined #openstack-ironic | 16:06 | |
| rloo | mgould: thx for the patch. might be useful to do the same test, w/o the str(), and show that it fails. | 16:06 |
| mgould | good idea | 16:06 |
| mgould | screw it, I'll just add them all now | 16:07 |
| rloo | mgould, i started to write up a bug, but can i just leave it for you to look after? i think if you write up a bug and link to your patch, that would help. | 16:07 |
| mgould | rloo: sure | 16:07 |
| rloo | mgould: if you add too many tests, it won't be clear/easy to see the problem. | 16:07 |
| vdrok | vsaienko1: what I mean is - to run only grenade on ironic (it already includes smoke twice and we have to run it, and ususally it's clear whether it's just a test failure or upgrade issue), but on other projects to have only smoke tests | 16:08 |
| mariojv | fyi i answered my own question; there is an upper limit as a default config option: https://github.com/openstack/ironic/blob/0d7cce3a79b1b0ddd1137886f12ee5942420e870/ironic/api/__init__.py#L27 | 16:08 |
| mariojv | perhaps that should be put in the API docs. | 16:08 |
| mgould | rloo: good point. I was thinking of adding (a) the '<all-in> f' test, (b) versions of the tests with the str() removed | 16:08 |
| mgould | but right now I want a build which passes when it shouldn't :-) | 16:09 |
| rloo | mariojv: we're moving to the api-ref docs, so maybe adding it there would help. | 16:09 |
| vdrok | mariojv: cool, didn't know that. yep, what rloo says | 16:09 |
| mariojv | i'll get on that, thanks | 16:09 |
| rloo | mgould: if you add (b), add them to the bottom then, so anyone that looks, can see the problem easily. removing the strs will probably still pass for all those existing tests. it is the single-char thing that is different. | 16:10 |
| mgould | rloo: yes, that was my plan | 16:11 |
| rloo | thx mgould | 16:11 |
| mgould | np | 16:11 |
| *** praneshp has quit IRC | 16:12 | |
| *** baoli_ has quit IRC | 16:12 | |
| *** baoli has joined #openstack-ironic | 16:13 | |
| *** PollyZ has quit IRC | 16:14 | |
| *** ifarkas has quit IRC | 16:14 | |
| *** fragatina has quit IRC | 16:15 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Collect deployment logs from IPA https://review.openstack.org/336102 | 16:15 |
| lucasagomes | btw folks, if you have some time... mind taking a peek at https://review.openstack.org/#/c/248832/ ? | 16:15 |
| lucasagomes | thanks | 16:15 |
| *** sdake_ has joined #openstack-ironic | 16:15 | |
| *** rama_y has joined #openstack-ironic | 16:16 | |
| * lucasagomes forgot the depends-on | 16:17 | |
| *** piet has quit IRC | 16:17 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Collect deployment logs from IPA https://review.openstack.org/336102 | 16:17 |
| *** sdake has quit IRC | 16:18 | |
| *** PollyZ has joined #openstack-ironic | 16:20 | |
| *** Nisha_brb has joined #openstack-ironic | 16:21 | |
| *** praneshp has joined #openstack-ironic | 16:22 | |
| *** Nisha_away has quit IRC | 16:23 | |
| *** sdake_ is now known as sdake | 16:23 | |
| *** jaybeale has joined #openstack-ironic | 16:25 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-specs: Small changes to collect system logs from IPA spec https://review.openstack.org/336108 | 16:27 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Collect deployment logs from IPA https://review.openstack.org/336102 | 16:29 |
| *** piet has joined #openstack-ironic | 16:30 | |
| *** trown is now known as trown|lunch | 16:30 | |
| *** baoli has quit IRC | 16:35 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add CLI tests to check JSON response body https://review.openstack.org/291239 | 16:36 |
| *** baoli has joined #openstack-ironic | 16:38 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add CLI tests to check JSON response body https://review.openstack.org/291239 | 16:41 |
| *** yogi has quit IRC | 16:41 | |
| *** baoli has quit IRC | 16:43 | |
| lucasagomes | folks calling it a day, have a great evening all | 16:50 |
| *** Romanenko_K has quit IRC | 16:50 | |
| *** lucasagomes is now known as lucas-afk | 16:50 | |
| *** Nisha_away has joined #openstack-ironic | 16:51 | |
| *** baoli has joined #openstack-ironic | 16:54 | |
| *** Nisha_brb has quit IRC | 16:55 | |
| *** daemontool_ has joined #openstack-ironic | 17:01 | |
| *** piet has quit IRC | 17:01 | |
| *** derekh has quit IRC | 17:02 | |
| *** spandhe has quit IRC | 17:04 | |
| *** daemontool has quit IRC | 17:04 | |
| alaski | hey everyone, there was a request in the nova channel to review https://review.openstack.org/#/c/328157 (serial console for Ironic) for FF. It would be helpful to have some Ironic folks review it as well since not many of us are up to date on the ironic side of this. | 17:06 |
| tiendc | alaski: thanks, I added Lucas and Ruby to the board | 17:07 |
| *** rbartal has quit IRC | 17:07 | |
| alaski | tiendc: cool. as you can see it's a bit busy today in Nova so it would help for us to not have to work out the dependencies here | 17:08 |
| tiendc | alaski: I know, as gmann commented in the patch, he wants to know nova team opinion | 17:11 |
| tiendc | alaski: can you give yours about that? | 17:12 |
| alaski | tiendc: if this works with ironic today then it's useful to try to get in now. if not I would rather wait until there's support in ironic for it. From the reviews in the commit message it seems that this doesn't work in ironic yet, but I want to make sure I'm understanding properly. | 17:14 |
| tiendc | alaski: you are right, thanks. I will ask for review from Lucas and Ruby to address the issue | 17:17 |
| *** PollyZ has quit IRC | 17:23 | |
| TheJulia | I think it ought to wait until we have support in ironic. It gives the wrong impression to a user that something should work when support doesn't completely exist yet. | 17:25 |
| TheJulia | I don't like being the person to obstruct something, but yeah... :\ | 17:26 |
| * mgould -> home; good night! | 17:26 | |
| *** mgould is now known as mgould|afk | 17:27 | |
| TheJulia | goodnight mgould|afk | 17:27 |
| *** spandhe has joined #openstack-ironic | 17:30 | |
| *** PollyZ has joined #openstack-ironic | 17:30 | |
| *** PollyZ has quit IRC | 17:35 | |
| *** penick_ has joined #openstack-ironic | 17:35 | |
| *** penick has quit IRC | 17:36 | |
| *** penick_ is now known as penick | 17:36 | |
| vdrok | good night everyone! | 17:37 |
| *** spandhe_ has joined #openstack-ironic | 17:38 | |
| *** spandhe has quit IRC | 17:39 | |
| *** spandhe_ is now known as spandhe | 17:39 | |
| *** jaypipes has joined #openstack-ironic | 17:40 | |
| *** trown|lunch is now known as trown | 17:40 | |
| *** athomas has quit IRC | 17:44 | |
| *** ChubYann has joined #openstack-ironic | 17:45 | |
| *** sdake has quit IRC | 17:45 | |
| *** karimb has quit IRC | 17:49 | |
| *** electrofelix has quit IRC | 17:55 | |
| *** Nisha_brb has joined #openstack-ironic | 17:58 | |
| *** Nisha_away has quit IRC | 17:59 | |
| *** Nisha_away has joined #openstack-ironic | 18:00 | |
| *** praneshp has quit IRC | 18:02 | |
| *** Nisha_brb has quit IRC | 18:03 | |
| *** Jabadia has joined #openstack-ironic | 18:04 | |
| *** tiendc has quit IRC | 18:05 | |
| openstackgerrit | Aparna proposed openstack/proliantutils: Add support for clean steps 'erase_devices' https://review.openstack.org/336148 | 18:08 |
| *** Nisha_brb has joined #openstack-ironic | 18:11 | |
| *** PollyZ has joined #openstack-ironic | 18:11 | |
| *** Nisha_away has quit IRC | 18:12 | |
| *** e0ne has joined #openstack-ironic | 18:13 | |
| *** Sukhdev has joined #openstack-ironic | 18:15 | |
| openstackgerrit | Merged openstack/ironic-lib: Add keyword arg 'log_stdout' to utils.execute() https://review.openstack.org/335211 | 18:16 |
| *** Sukhdev has quit IRC | 18:28 | |
| *** piet has joined #openstack-ironic | 18:31 | |
| *** e0ne has quit IRC | 18:33 | |
| *** thrash is now known as thrash|biab | 18:38 | |
| *** Nisha_away has joined #openstack-ironic | 18:43 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/336178 | 18:43 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/336179 | 18:43 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-lib: Updated from global requirements https://review.openstack.org/336180 | 18:43 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/336181 | 18:43 |
| *** e0ne has joined #openstack-ironic | 18:44 | |
| *** Nisha_brb has quit IRC | 18:45 | |
| *** Nisha_brb has joined #openstack-ironic | 18:45 | |
| *** Sukhdev has joined #openstack-ironic | 18:47 | |
| *** Nisha_away has quit IRC | 18:47 | |
| mariojv | TheJulia: agreed. maybe we should ask to have a Depends-On tag added | 18:47 |
| mariojv | there is a way to signal a cross-project dependency in gerrit iirc | 18:47 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add support for building ISO for deploy ramdisk https://review.openstack.org/264590 | 18:48 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironic-inspector-client: Updated from global requirements https://review.openstack.org/336228 | 18:49 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/336229 | 18:49 |
| TheJulia | mariojv: absolutely | 18:49 |
| TheJulia | downside is it might auto-merge at the wrong time too if it gets +A'ed and forgotten about | 18:50 |
| *** Sukhdev_ has joined #openstack-ironic | 18:51 | |
| *** Sukhdev has quit IRC | 18:51 | |
| *** e0ne has quit IRC | 18:54 | |
| *** e0ne_ has joined #openstack-ironic | 18:54 | |
| *** PollyZ has quit IRC | 18:57 | |
| *** Nisha_away has joined #openstack-ironic | 18:58 | |
| *** Nisha_brb has quit IRC | 18:58 | |
| *** PollyZ has joined #openstack-ironic | 19:03 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Config variable to configure [glance] section https://review.openstack.org/266803 | 19:03 |
| *** PollyZ has quit IRC | 19:07 | |
| *** Nisha_away has quit IRC | 19:07 | |
| *** Nisha_away has joined #openstack-ironic | 19:08 | |
| *** amotoki has joined #openstack-ironic | 19:14 | |
| *** Nisha_away has quit IRC | 19:15 | |
| *** Nisha_away has joined #openstack-ironic | 19:16 | |
| *** Jabadia has quit IRC | 19:24 | |
| *** sacharya_ has quit IRC | 19:26 | |
| *** sacharya has joined #openstack-ironic | 19:26 | |
| *** PollyZ has joined #openstack-ironic | 19:30 | |
| *** e0ne_ has quit IRC | 19:30 | |
| *** Nisha_brb has joined #openstack-ironic | 19:35 | |
| *** Nisha_away has quit IRC | 19:36 | |
| *** jralph has quit IRC | 19:39 | |
| *** jralph has joined #openstack-ironic | 19:39 | |
| *** hoonetorg has quit IRC | 19:40 | |
| *** daemontool_ has quit IRC | 19:46 | |
| *** Nisha_away has joined #openstack-ironic | 19:47 | |
| *** Nisha_brb has quit IRC | 19:49 | |
| *** piet has quit IRC | 19:51 | |
| *** nicodemos has left #openstack-ironic | 19:51 | |
| *** nicodemos has joined #openstack-ironic | 19:51 | |
| *** hoonetorg has joined #openstack-ironic | 19:52 | |
| *** dansmith is now known as unicorn_slayer | 19:59 | |
| *** fragatina has joined #openstack-ironic | 20:03 | |
| *** Nisha_away has quit IRC | 20:05 | |
| *** PollyZ has quit IRC | 20:05 | |
| *** livelace has quit IRC | 20:07 | |
| *** thrash|biab is now known as thrash | 20:12 | |
| *** PollyZ has joined #openstack-ironic | 20:13 | |
| *** unicorn_slayer is now known as dansmith | 20:15 | |
| *** PollyZ has quit IRC | 20:18 | |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: [WIP DO NOT MERGE] Force devstack VM to 1280M https://review.openstack.org/335543 | 20:18 |
| *** blakec has joined #openstack-ironic | 20:20 | |
| *** livelace has joined #openstack-ironic | 20:22 | |
| *** PollyZ has joined #openstack-ironic | 20:24 | |
| *** piet has joined #openstack-ironic | 20:28 | |
| *** livelace has quit IRC | 20:28 | |
| *** rbudden has quit IRC | 20:34 | |
| *** piet has quit IRC | 20:41 | |
| *** rcernin has joined #openstack-ironic | 20:41 | |
| *** blakec has quit IRC | 20:56 | |
| dansmith | FYI the fix to the bug blocking your gate is #5 in the merge queue | 20:59 |
| dansmith | also, sorry about that :) | 20:59 |
| rloo | dansmith: thx! (finally :)) | 20:59 |
| dansmith | well, sorry, if your tests were faster it might've merged sooner :) | 21:00 |
| rloo | dansmith: ha ha. if we didn't have tests, it would even be faster! | 21:00 |
| rloo | jlvillal: ^^ | 21:01 |
| dansmith | actually the problem is your obsession with hardware.. maybe you haven't heard of this new thing called "virtualization" ? | 21:01 |
| * dansmith runs | 21:01 | |
| * rloo catches up with dansmith in spain... grrr... | 21:01 | |
| dansmith | heh | 21:01 |
| * rloo wonders what a duel between hw and virtual would be like | 21:02 | |
| mrda | Morning Ironic | 21:02 |
| *** raildo is now known as raildo-afk | 21:02 | |
| rloo | morning mrda | 21:04 |
| mrda | hey rloo! | 21:04 |
| mariojv | hey mrda | 21:05 |
| mrda | hi mariojv | 21:05 |
| mrda | rloo: Just in case you were bored and itching to do a +2 +A, https://review.openstack.org/#/c/334387/ should be easy now :-) | 21:06 |
| rloo | mrda: oh. my brain is fried. did too many hard reviews this week. i have one *last* one (I hope it is the last) that I am working on now, and I think that might be all I can do today. | 21:07 |
| rloo | mrda: also, i think i'll leave the easy ones for others to review cuz I'm nice :) [Jeez, a great way to spin things...] | 21:08 |
| mrda | lol, no problems :) | 21:08 |
| *** ametts has quit IRC | 21:13 | |
| *** vishwanathj has quit IRC | 21:22 | |
| *** trown is now known as trown|outtypewww | 21:23 | |
| *** Sukhdev_ has quit IRC | 21:26 | |
| *** lucas-afk has quit IRC | 21:28 | |
| *** lucasagomes has joined #openstack-ironic | 21:28 | |
| *** rbudden has joined #openstack-ironic | 21:30 | |
| *** Sukhdev has joined #openstack-ironic | 21:31 | |
| *** Shalmazar has joined #openstack-ironic | 21:33 | |
| *** ayoung has quit IRC | 21:34 | |
| *** Madasi has quit IRC | 21:34 | |
| *** Shalmazar is now known as Madasi | 21:34 | |
| *** piet has joined #openstack-ironic | 21:38 | |
| *** HenryG_ has joined #openstack-ironic | 21:41 | |
| *** jroll|dupe has joined #openstack-ironic | 21:44 | |
| *** jroll|dupe has quit IRC | 21:44 | |
| *** jroll|dupe has joined #openstack-ironic | 21:44 | |
| *** hoonetorg has quit IRC | 21:45 | |
| *** thiagop has quit IRC | 21:45 | |
| *** mordred has quit IRC | 21:45 | |
| *** HenryG has quit IRC | 21:45 | |
| *** nicodemos has quit IRC | 21:45 | |
| *** swatson has quit IRC | 21:45 | |
| *** ppiela has quit IRC | 21:45 | |
| *** jroll has quit IRC | 21:45 | |
| *** ianw has quit IRC | 21:45 | |
| *** Guest40699 has quit IRC | 21:45 | |
| *** trown|outtypewww has quit IRC | 21:45 | |
| *** hamzy has quit IRC | 21:45 | |
| *** mkoderer has quit IRC | 21:45 | |
| *** jroll|dupe is now known as jroll | 21:45 | |
| *** fragatina has quit IRC | 21:47 | |
| *** wajdi_ has joined #openstack-ironic | 21:50 | |
| *** wajdi__ has joined #openstack-ironic | 21:51 | |
| *** wajdi_ has quit IRC | 21:51 | |
| *** HenryG_ is now known as HenryG | 21:52 | |
| *** sdake_ has joined #openstack-ironic | 21:53 | |
| *** sdake_ has quit IRC | 21:53 | |
| *** sdake_ has joined #openstack-ironic | 21:53 | |
| *** wajdi has quit IRC | 21:55 | |
| *** catintheroof has joined #openstack-ironic | 22:03 | |
| *** wajdi has joined #openstack-ironic | 22:05 | |
| *** wajdi__ has quit IRC | 22:08 | |
| *** wajdi has quit IRC | 22:13 | |
| *** piet has quit IRC | 22:16 | |
| *** Sukhdev_ has joined #openstack-ironic | 22:16 | |
| *** Sukhdev has quit IRC | 22:16 | |
| *** derekh has joined #openstack-ironic | 22:22 | |
| *** mordred has joined #openstack-ironic | 22:22 | |
| *** sdake_ has quit IRC | 22:33 | |
| *** piet has joined #openstack-ironic | 22:58 | |
| jlvillal | The gate should be fixed now. | 23:04 |
| *** rbudden has quit IRC | 23:06 | |
| *** amotoki has quit IRC | 23:08 | |
| *** ayoung has joined #openstack-ironic | 23:09 | |
| *** amotoki has joined #openstack-ironic | 23:10 | |
| *** piet has quit IRC | 23:10 | |
| *** sdake has joined #openstack-ironic | 23:11 | |
| *** sdake has quit IRC | 23:11 | |
| *** sacharya has quit IRC | 23:11 | |
| *** Sukhdev_ has quit IRC | 23:16 | |
| *** Sukhdev has joined #openstack-ironic | 23:18 | |
| *** catintheroof has quit IRC | 23:18 | |
| *** baoli has quit IRC | 23:42 | |
| *** amotoki has quit IRC | 23:44 | |
| *** derekh has quit IRC | 23:47 | |
| *** blakec has joined #openstack-ironic | 23:54 | |
| *** PollyZ has quit IRC | 23:54 | |
| *** phuongnh has quit IRC | 23:57 | |
| *** spandhe has quit IRC | 23:57 | |
| yuikotakadamori | jroll: is there? | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
