| TheJulia | mat128: super choppy audio? | 00:02 |
|---|---|---|
| mat128 | TheJulia: https://www.dropbox.com/s/mswqy25ag0egyti/2016-06-22%4020.02.19-call-7777.mp3?dl=0 | 00:02 |
| *** yuikotakadamori has joined #openstack-ironic | 00:03 | |
| * TheJulia tosses wet cats at pbx.openstack.org | 00:03 | |
| *** baoli has joined #openstack-ironic | 00:06 | |
| *** baoli has quit IRC | 00:06 | |
| *** baoli has joined #openstack-ironic | 00:14 | |
| *** baoli has quit IRC | 00:15 | |
| mat128 | #info The 0000 UTC session has ended. | 00:17 |
| *** sabeen has joined #openstack-ironic | 00:19 | |
| *** mbound has quit IRC | 00:20 | |
| *** Nisha_brb has joined #openstack-ironic | 00:23 | |
| *** Nisha_away has quit IRC | 00:25 | |
| openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/328537 | 00:26 |
| *** hshiina has joined #openstack-ironic | 00:27 | |
| *** achanda_ has quit IRC | 00:27 | |
| *** Nisha_brb has quit IRC | 00:29 | |
| *** baoli has joined #openstack-ironic | 00:33 | |
| *** baoli has quit IRC | 00:36 | |
| openstackgerrit | Gerard Braad proposed openstack/ironic-webclient: Correct markup of README.md https://review.openstack.org/332707 | 00:41 |
| *** baoli has joined #openstack-ironic | 00:41 | |
| *** baoli has quit IRC | 00:42 | |
| openstackgerrit | Merged openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/332368 | 00:44 |
| *** spandhe has quit IRC | 00:45 | |
| *** hoangcx has joined #openstack-ironic | 00:45 | |
| *** YiboCai has joined #openstack-ironic | 00:54 | |
| *** baoli has joined #openstack-ironic | 00:54 | |
| *** YiboCai has quit IRC | 00:54 | |
| *** phuongnh has joined #openstack-ironic | 00:57 | |
| *** fragatina has joined #openstack-ironic | 00:59 | |
| *** yibo has joined #openstack-ironic | 01:01 | |
| *** baoli_ has joined #openstack-ironic | 01:02 | |
| *** spandhe has joined #openstack-ironic | 01:05 | |
| *** baoli has quit IRC | 01:05 | |
| *** fragatina has quit IRC | 01:07 | |
| *** fragatina has joined #openstack-ironic | 01:08 | |
| *** yibo has quit IRC | 01:11 | |
| *** yibo has joined #openstack-ironic | 01:13 | |
| *** yibo has quit IRC | 01:15 | |
| *** yibo has joined #openstack-ironic | 01:15 | |
| *** spandhe has quit IRC | 01:16 | |
| *** smoriya has joined #openstack-ironic | 01:17 | |
| *** PollyZ has joined #openstack-ironic | 01:19 | |
| *** PollyZ_ has joined #openstack-ironic | 01:24 | |
| *** joprovost has joined #openstack-ironic | 01:25 | |
| *** PollyZ has quit IRC | 01:26 | |
| *** hshiina has quit IRC | 01:34 | |
| *** baoli_ has quit IRC | 01:43 | |
| *** baoli has joined #openstack-ironic | 01:44 | |
| openstackgerrit | Yuiko Takada Mori proposed openstack/ironic: IPMITool: add IPMISocatConsole class https://review.openstack.org/293873 | 01:47 |
| openstackgerrit | Yuiko Takada Mori proposed openstack/ironic: [WIP]IPMITool: add IPMISocatConsole class https://review.openstack.org/293873 | 01:51 |
| *** sacharya has joined #openstack-ironic | 01:59 | |
| *** sacharya has quit IRC | 02:03 | |
| *** joprovost has quit IRC | 02:04 | |
| *** sacharya has joined #openstack-ironic | 02:04 | |
| *** thrash is now known as thrash|g0ne | 02:12 | |
| *** spandhe has joined #openstack-ironic | 02:15 | |
| *** rajinir has quit IRC | 02:24 | |
| *** greghaynes has quit IRC | 02:27 | |
| *** spandhe_ has joined #openstack-ironic | 02:27 | |
| *** spandhe has quit IRC | 02:28 | |
| *** spandhe_ is now known as spandhe | 02:28 | |
| *** rloo has quit IRC | 02:28 | |
| *** greghaynes has joined #openstack-ironic | 02:38 | |
| *** achanda has joined #openstack-ironic | 03:03 | |
| *** ijw has quit IRC | 03:07 | |
| *** achanda has quit IRC | 03:08 | |
| openstackgerrit | Merged openstack/python-ironicclient: Add test for chassis-node-list https://review.openstack.org/299482 | 03:10 |
| *** baoli has quit IRC | 03:19 | |
| *** baoli has joined #openstack-ironic | 03:23 | |
| *** stevemar has joined #openstack-ironic | 03:36 | |
| *** links has joined #openstack-ironic | 03:51 | |
| *** Nisha_brb has joined #openstack-ironic | 03:52 | |
| *** ijw has joined #openstack-ironic | 03:54 | |
| *** appu103_ has joined #openstack-ironic | 04:04 | |
| *** baoli has quit IRC | 04:10 | |
| *** ijw has quit IRC | 04:12 | |
| *** achanda has joined #openstack-ironic | 04:18 | |
| *** yogi has joined #openstack-ironic | 04:21 | |
| *** amotoki has joined #openstack-ironic | 04:25 | |
| *** Sukhdev has joined #openstack-ironic | 04:28 | |
| *** PollyZ_ has quit IRC | 04:28 | |
| *** sacharya has quit IRC | 04:30 | |
| *** appu103_ is now known as aparnav | 04:49 | |
| *** aparnav has quit IRC | 04:54 | |
| *** Sukhdev has quit IRC | 04:55 | |
| *** aparnav has joined #openstack-ironic | 04:55 | |
| *** _vsaienko has joined #openstack-ironic | 04:56 | |
| *** aparnav has quit IRC | 04:57 | |
| *** aparnav has joined #openstack-ironic | 04:58 | |
| *** ChubYann has quit IRC | 05:26 | |
| *** ijw has joined #openstack-ironic | 05:33 | |
| *** yogi has quit IRC | 05:39 | |
| *** yogi has joined #openstack-ironic | 05:39 | |
| *** yolanda has joined #openstack-ironic | 05:56 | |
| openstackgerrit | Yuiko Takada Mori proposed openstack/ironic-specs: Nova compatible serial console support https://review.openstack.org/319505 | 06:13 |
| *** rcernin has joined #openstack-ironic | 06:15 | |
| *** moshele has joined #openstack-ironic | 06:25 | |
| *** _vsaienko has quit IRC | 06:27 | |
| *** _vsaienko has joined #openstack-ironic | 06:30 | |
| *** yolanda has quit IRC | 06:30 | |
| *** _vsaienko has quit IRC | 06:33 | |
| *** sdake has joined #openstack-ironic | 06:34 | |
| *** rbartal has joined #openstack-ironic | 06:34 | |
| *** _vsaienko has joined #openstack-ironic | 06:36 | |
| *** livelace has joined #openstack-ironic | 06:40 | |
| *** sabeen has quit IRC | 06:46 | |
| *** _vsaienko has quit IRC | 06:46 | |
| *** _vsaienko has joined #openstack-ironic | 06:47 | |
| *** ijw has quit IRC | 06:48 | |
| openstackgerrit | Saravanan KR proposed openstack/ironic-specs: Provide a mechanism to configure kernel boot parameters https://review.openstack.org/331564 | 06:48 |
| *** hoangcx has quit IRC | 06:49 | |
| *** hoangcx has joined #openstack-ironic | 06:53 | |
| *** achanda has quit IRC | 06:56 | |
| *** _vsaienko has quit IRC | 06:56 | |
| *** fragatina has quit IRC | 06:57 | |
| *** fragatina has joined #openstack-ironic | 06:58 | |
| *** _vsaienko has joined #openstack-ironic | 06:59 | |
| *** achanda has joined #openstack-ironic | 07:01 | |
| *** rcernin has quit IRC | 07:02 | |
| *** _vsaienko has quit IRC | 07:04 | |
| *** _vsaienko has joined #openstack-ironic | 07:05 | |
| *** achanda has quit IRC | 07:06 | |
| *** tesseract- has joined #openstack-ironic | 07:13 | |
| *** tesseract- has quit IRC | 07:14 | |
| *** tesseract- has joined #openstack-ironic | 07:14 | |
| *** _vsaienko has quit IRC | 07:15 | |
| *** arturas has joined #openstack-ironic | 07:15 | |
| *** _vsaienko has joined #openstack-ironic | 07:15 | |
| *** rcernin has joined #openstack-ironic | 07:17 | |
| *** sdake_ has joined #openstack-ironic | 07:21 | |
| *** _vsaienko has quit IRC | 07:21 | |
| *** sdake has quit IRC | 07:24 | |
| *** e0ne has joined #openstack-ironic | 07:24 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: WIP: Discovery tempest test https://review.openstack.org/323769 | 07:25 |
| *** _vsaienko has joined #openstack-ironic | 07:25 | |
| *** watanabe_isao has joined #openstack-ironic | 07:29 | |
| *** watanabe_isao has quit IRC | 07:43 | |
| *** _vsaienko has quit IRC | 07:48 | |
| pas-ha | morning Ironic | 07:49 |
| *** _vsaienko has joined #openstack-ironic | 07:51 | |
| *** spandhe has quit IRC | 07:52 | |
| *** mbound has joined #openstack-ironic | 07:53 | |
| *** ifarkas has joined #openstack-ironic | 07:53 | |
| *** itamarl has joined #openstack-ironic | 07:54 | |
| ifarkas | morning all! | 07:57 |
| *** zzzeek has quit IRC | 08:00 | |
| *** itamarl has quit IRC | 08:00 | |
| *** itamarl has joined #openstack-ironic | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:01 | |
| _milan_ | morning pas-ha ifarkas aarefiev :) | 08:02 |
| pas-ha | morning ifarkas _milan_ | 08:02 |
| _milan_ | #pixiesay Morning Ironic! -m flexing | 08:02 |
| *** jtomasek_ has joined #openstack-ironic | 08:02 | |
| aarefiev | morning _milan_ pas-ha ifarkas | 08:02 |
| _milan_ | lucas-afk, poor pixie is still down | 08:02 |
| *** yolanda has joined #openstack-ironic | 08:04 | |
| * _milan_ has new gained a new bash skill: setsid https://review.openstack.org/#/c/333155/ | 08:05 | |
| _milan_ | grenade for Inspector moves on a bit again :) | 08:05 |
| *** _vs_ has joined #openstack-ironic | 08:07 | |
| *** _vsaienko has quit IRC | 08:07 | |
| _milan_ | http://paste.openstack.org/show/521589/ | 08:07 |
| *** e0ne has quit IRC | 08:11 | |
| *** hoangcx has quit IRC | 08:13 | |
| *** ohamada has joined #openstack-ironic | 08:15 | |
| aarefiev | _milan_: \o/ good work :) | 08:16 |
| *** _vs_ has quit IRC | 08:16 | |
| _milan_ | aarefiev, thanks :D | 08:17 |
| *** kromanenko_ has joined #openstack-ironic | 08:17 | |
| *** Romanenko_K has joined #openstack-ironic | 08:17 | |
| *** kromanenko__ has joined #openstack-ironic | 08:17 | |
| aarefiev | _milan_: btw do guys have any progress on ha? | 08:17 |
| *** e0ne has joined #openstack-ironic | 08:18 | |
| _milan_ | aarefiev, +1 on the devstack patches are much welcome ;) | 08:18 |
| _milan_ | aarefiev, unfortunately that's blocked by grenade :-/ | 08:18 |
| *** hoangcx has joined #openstack-ironic | 08:18 | |
| _milan_ | sambetts|afk, wouldn't approve otherwise ;) | 08:18 |
| *** mbound has quit IRC | 08:19 | |
| *** jtomasek_ has quit IRC | 08:20 | |
| aarefiev | _milan_: yeah, just wondering, I don't see any progress on queries for ha | 08:20 |
| *** _vsaienko has joined #openstack-ironic | 08:20 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: [WIP]Introducing Inspector Grenade testing https://review.openstack.org/327667 | 08:21 |
| _milan_ | aarefiev, right. It's going to be me and dtantsur|afk to start implementing I guess, help much appreciated | 08:22 |
| aarefiev | _milan_: get it, I'll take a look how I can help | 08:22 |
| _milan_ | aarefiev, thanks! | 08:23 |
| *** _vsaienko has quit IRC | 08:23 | |
| aarefiev | _milan_: thanks you :) | 08:23 |
| _milan_ | :) | 08:23 |
| *** yuikotakadamori has quit IRC | 08:25 | |
| *** alineb has joined #openstack-ironic | 08:25 | |
| alineb | morning all | 08:26 |
| *** _vsaienko has joined #openstack-ironic | 08:27 | |
| *** lucas-afk is now known as lucasagomes | 08:29 | |
| lucasagomes | morning all | 08:29 |
| aarefiev | morning lucasagomes | 08:30 |
| *** PixieBoots has joined #openstack-ironic | 08:31 | |
| lucasagomes | #pixiesay hey _milan_ :-) | 08:31 |
| *** daemontool has joined #openstack-ironic | 08:31 | |
| PixieBoots | ʕ•͡ᴥ•ʔ: hey _milan_ :-) | 08:31 |
| *** vmud213 has joined #openstack-ironic | 08:32 | |
| _milan_ | lucasagomes, morning | 08:32 |
| vmud213 | morning all | 08:32 |
| _milan_ | #pixiesay morning Ironic! -m flexing | 08:32 |
| PixieBoots | ᕙʕ⇀ᴥ⇀ʔᕗ: morning Ironic! | 08:32 |
| _milan_ | :D | 08:32 |
| _milan_ | at last, it's alive! :D | 08:32 |
| _milan_ | vmud213, morning | 08:32 |
| _milan_ | alineb, morning | 08:33 |
| vmud213 | _milan_ o\ :) | 08:33 |
| *** itamarl_ has joined #openstack-ironic | 08:33 | |
| lucasagomes | yup | 08:34 |
| alineb | o/ _milan_ vmud213 aarefiev lucasagomes | 08:34 |
| vmud213 | morning alineb | 08:35 |
| *** itamarl has quit IRC | 08:36 | |
| *** itamarl_ is now known as itamarl | 08:36 | |
| sambetts|afk | Morning all | 08:36 |
| *** sambetts|afk is now known as sambetts | 08:36 | |
| sambetts | _milan_: what wouldn't I approve ;) | 08:37 |
| _milan_ | sambetts, HA inspector without Grenade Inspector :D | 08:37 |
| _milan_ | morning sambetts :D | 08:37 |
| sambetts | _milan_: Ah yes :-P | 08:37 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/bifrost: Updated from global requirements https://review.openstack.org/332275 | 08:38 |
| _milan_ | sambetts, https://review.openstack.org/#/c/333155/ reviews welcome then ;) | 08:38 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/333177 | 08:40 |
| vdrok | morning ironic! | 08:40 |
| _milan_ | morning vdrok! :) | 08:41 |
| _milan_ | ehm | 08:41 |
| _milan_ | #pixiesay morning vdrok! -m happy | 08:41 |
| PixieBoots | ʕ^ᴥ^ʔ: morning vdrok! | 08:41 |
| vdrok | good morning _milan_ | 08:41 |
| vdrok | :) | 08:41 |
| _milan_ | thanks to lucasagomes pixie is back | 08:41 |
| vdrok | \o/ | 08:42 |
| vmud213 | morning vdrok | 08:43 |
| *** derekh has joined #openstack-ironic | 08:44 | |
| vdrok | good morning vmud213 | 08:45 |
| *** rcernin has quit IRC | 08:50 | |
| *** _vs_ has joined #openstack-ironic | 08:51 | |
| *** _vsaienko has quit IRC | 08:51 | |
| *** amotoki has quit IRC | 08:53 | |
| *** arturas has quit IRC | 08:55 | |
| sergek | morning vdrok , vmud213 _milan_ , PixieBoots , sambetts | 08:55 |
| vdrok | morning sergek ! | 08:55 |
| _milan_ | morning sergek :) | 08:55 |
| sergek | vdrok: o/ | 08:55 |
| vmud213 | morning sergek | 08:55 |
| sergek | vmud213: \o | 08:55 |
| sergek | \o/ | 08:56 |
| *** arturas has joined #openstack-ironic | 08:56 | |
| *** _vs_ has quit IRC | 08:56 | |
| *** dtantsur|afk is now known as dtantsur | 08:59 | |
| dtantsur | Morning Ironic! How was the midcycle for you? | 08:59 |
| *** yolanda has quit IRC | 09:01 | |
| dtantsur | lucasagomes, I'm looking for a brave person to approve https://review.openstack.org/#/c/188370/ after getting 2x +2 ;) | 09:01 |
| dtantsur | sambetts, do you feel like your comments were resolved ^^^? | 09:01 |
| *** rcernin has joined #openstack-ironic | 09:02 | |
| *** achanda has joined #openstack-ironic | 09:03 | |
| sambetts | dtantsur: Re: the validation thing or Re: the InvalidInterface class thing? | 09:03 |
| *** mgould|afk is now known as mgould | 09:03 | |
| mgould | morning Ironic! | 09:03 |
| dtantsur | sambetts, re everything :) | 09:04 |
| dtantsur | morning mgould | 09:04 |
| dtantsur | sambetts, and do you feel that we can approve what we have and then start thinking how to follow-up with the details? | 09:04 |
| sambetts | dtantsur: I think we've beaten the dead horse enough and big ideas are there and we just need to get it done :) | 09:05 |
| *** amotoki has joined #openstack-ironic | 09:05 | |
| sambetts | +1 from me | 09:05 |
| sambetts | I don't have +2 on specs | 09:06 |
| *** achanda has quit IRC | 09:08 | |
| dtantsur | I know, I just wanted to make sure you don't have objections :) | 09:08 |
| *** dmk0202 has joined #openstack-ironic | 09:09 | |
| dtantsur | I hope lucasagomes just approves that, cause I can no longer rewrite this thing every time a new reviewer comes >_< | 09:09 |
| sambetts | Nope, my comment about the Invalid... class thing was the only comment I come come up with when I re-read it ysterday | 09:09 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Added operator documentation for ironic portgroups https://review.openstack.org/228496 | 09:10 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add multitenancy-related fields to port API object https://review.openstack.org/206244 | 09:10 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Create common neutron module https://review.openstack.org/317390 | 09:10 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add network_interface node field DB, OBJ https://review.openstack.org/317391 | 09:10 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add 'neutron' network driver https://review.openstack.org/317393 | 09:10 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 09:10 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add network_interface API https://review.openstack.org/317392 | 09:10 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add network_drivers https://review.openstack.org/285852 | 09:10 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add portgroups to support LAG interfaces - API https://review.openstack.org/332177 | 09:10 |
| vdrok | morning dtantsur and mgould | 09:11 |
| dtantsur | morning vdrok | 09:11 |
| *** hkominos has quit IRC | 09:11 | |
| *** hkominos has joined #openstack-ironic | 09:12 | |
| hkominos | good morning all | 09:12 |
| hkominos | another glorious day of fighting ironic | 09:12 |
| *** electrofelix has joined #openstack-ironic | 09:20 | |
| dtantsur | morning hkominos, good luck | 09:24 |
| *** arturas has quit IRC | 09:24 | |
| *** zhenguo_ has joined #openstack-ironic | 09:25 | |
| mgould | morning vdrok hkominos | 09:26 |
| hkominos | morning mgould | 09:27 |
| zhenguo_ | does some ever seen this error "location is outside of the device /dev/sdg" when using pxe_ipmitool driver? | 09:29 |
| *** caoshufeng_ has joined #openstack-ironic | 09:29 | |
| *** athomas has joined #openstack-ironic | 09:30 | |
| *** amotoki has quit IRC | 09:30 | |
| *** amotoki has joined #openstack-ironic | 09:32 | |
| openstackgerrit | Cao ShuFeng proposed openstack/ironic: Auto generate ironic.conf.sample file https://review.openstack.org/333210 | 09:32 |
| *** amotoki has quit IRC | 09:33 | |
| _milan_ | morning dtantsur mgould :) | 09:35 |
| dtantsur | hey _milan_ | 09:35 |
| yibo | vdrok: would you help review https://review.openstack.org/#/c/322632/ ? thanks. | 09:35 |
| openstackgerrit | Cao ShuFeng proposed openstack/ironic: Auto generate ironic.conf.sample file https://review.openstack.org/333210 | 09:37 |
| *** amotoki has joined #openstack-ironic | 09:39 | |
| * _milan_ relocates | 09:39 | |
| *** _milan_ has quit IRC | 09:39 | |
| *** sdake_ has quit IRC | 09:42 | |
| *** Nisha_brb has quit IRC | 09:43 | |
| *** amotoki has quit IRC | 09:50 | |
| *** pcaruana has joined #openstack-ironic | 09:51 | |
| openstackgerrit | Sam Betts proposed openstack/ironic: Fail early if ramdisk type is dib, and not building https://review.openstack.org/328246 | 09:56 |
| *** fragatina has quit IRC | 09:56 | |
| *** amotoki has joined #openstack-ironic | 09:58 | |
| *** daemontool has quit IRC | 10:00 | |
| *** foexle has joined #openstack-ironic | 10:01 | |
| *** sdake has joined #openstack-ironic | 10:01 | |
| *** itamarl has quit IRC | 10:12 | |
| *** hoangcx has quit IRC | 10:14 | |
| *** dtantsur is now known as dtantsur|bbl | 10:17 | |
| vdrok | hi yibo, yep, will do | 10:20 |
| *** caoshufeng_ has quit IRC | 10:23 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for openstackclient plugin https://review.openstack.org/301687 | 10:24 |
| *** itamarl has joined #openstack-ironic | 10:25 | |
| *** sdake_ has joined #openstack-ironic | 10:28 | |
| *** dmk0202 has quit IRC | 10:29 | |
| *** thrash|g0ne is now known as thrash | 10:31 | |
| *** dedery has joined #openstack-ironic | 10:32 | |
| *** sdake has quit IRC | 10:32 | |
| *** phuongnh has quit IRC | 10:35 | |
| *** moshele has quit IRC | 10:37 | |
| *** milan has joined #openstack-ironic | 10:39 | |
| *** moshele has joined #openstack-ironic | 10:40 | |
| aarefiev | milan: heh we have tempest job in mitaka https://review.openstack.org/#/c/332408/ | 10:40 |
| milan | aarefiev, wow, and it's failing :D | 10:41 |
| aarefiev | milan: yeah because no tests :) | 10:41 |
| aarefiev | fail | 10:41 |
| * milan checks | 10:41 | |
| milan | yeah, that seems correct ;) | 10:42 |
| aarefiev | I guess I forgot to exclude mitaka branch | 10:42 |
| milan | ah, I thought we could backport | 10:43 |
| milan | dtantsur|bbl, wdyt ^ | 10:43 |
| milan | or not worth it? | 10:43 |
| aarefiev | milan: I don't see any reason | 10:43 |
| aarefiev | to backport | 10:44 |
| milan | hmmm, unless we want some tempest sanity for grenade (there's none atm) | 10:44 |
| milan | but cool w/ me | 10:44 |
| milan | less work work :) | 10:44 |
| milan | aarefiev, I wish infra merged this: https://review.openstack.org/#/c/331001/ | 10:47 |
| milan | so we can check experimental the grenade | 10:47 |
| milan | stuff | 10:47 |
| *** derekh has quit IRC | 10:53 | |
| aarefiev | milan: does this template run inspector test? | 10:54 |
| jroll | dtantsur|bbl: I went ahead and approved driver comp \o/ | 10:54 |
| *** jistr is now known as jistr|mtg | 10:56 | |
| milan | aarefiev, good question | 10:57 |
| milan | my fake gate env didn't reach it that far to be able to tell | 10:57 |
| milan | mind my screenlog from the morning | 10:57 |
| milan | morning jroll :) | 10:57 |
| jroll | morning milan, aarefiev, vdrok, dtantsur|bbl, everyone else :) | 10:58 |
| vdrok | morning jroll :) | 10:58 |
| milan | jroll, \o/ merge comp spec :D | 10:58 |
| aarefiev | milan: are there more blockers? | 10:59 |
| aarefiev | morning jroll | 10:59 |
| openstackgerrit | Merged openstack/ironic-specs: The driver composition reform https://review.openstack.org/188370 | 11:00 |
| milan | aarefiev, I plan to push the fake gate env test a bit more to see. | 11:00 |
| aarefiev | milan: cool | 11:00 |
| milan | atm there's this thing with command not found in the upgrade database part | 11:00 |
| milan | but maybe there are more | 11:00 |
| milan | aarefiev, I'd say that tempest in grenade should run all plugins (including inspector's) unless one specifies a filter | 11:01 |
| milan | but I might be wrong | 11:01 |
| milan | the template might work as it is if I'm right | 11:02 |
| *** moshele has quit IRC | 11:02 | |
| *** dtantsur|bbl is now known as dtantsur | 11:03 | |
| dtantsur | jroll, morning and OMG THANKS! | 11:03 |
| *** moshele has joined #openstack-ironic | 11:03 | |
| jroll | dtantsur: :) | 11:03 |
| aarefiev | milan: make sense , but dunno | 11:03 |
| jroll | dtantsur: if people don't like it, we can edit it | 11:03 |
| dtantsur | yes, I think we can continue polishing the details. but I can also code in parallel | 11:04 |
| milan | jroll, or not :D | 11:04 |
| *** achanda has joined #openstack-ironic | 11:05 | |
| *** moshele has quit IRC | 11:08 | |
| *** hshiina has joined #openstack-ironic | 11:09 | |
| *** achanda has quit IRC | 11:09 | |
| hkominos | hej again ironicers. IS there a reason why one whould choose to use partition images instead of full disk images? | 11:12 |
| *** vmud213 has quit IRC | 11:15 | |
| *** lucasagomes is now known as lucas-hungry | 11:19 | |
| dtantsur | jroll, a question I didn't think about: should the new heartbeat API validate the deploy interface? | 11:20 |
| jroll | dtantsur: why? | 11:21 |
| dtantsur | dunno, but that's how most of methods work | 11:21 |
| *** sdake_ has quit IRC | 11:21 | |
| dtantsur | so I wonder if some implementations could rely on validation... | 11:21 |
| dtantsur | on the other hand, we'll be reporting a failure to the ramdisk, which is mostly useless | 11:22 |
| dtantsur | just thinking aloud :) | 11:22 |
| lazy_prince | quick question: why is ironic making a request to get a keystone token when using v3 as below: | 11:23 |
| lazy_prince | curl -i https://10.1.1.20:5000/v3/tokens -X POST -H "User-Agent: python-neutronclient" -d '{"auth": {"tenantName": "services", "passwordCredentials": {"username": "ironic", "password": "REDACTED"}}}' | 11:23 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add test for 'baremetal port create' command https://review.openstack.org/332880 | 11:23 |
| dtantsur | lazy_prince, judging by the user agent, I suspect it's neutronclient, not ironic | 11:23 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Introduce new driver call and RPC for heartbeat https://review.openstack.org/332890 | 11:24 |
| lazy_prince | dtantsur: right but its being invoked by ironic. | 11:24 |
| lazy_prince | here is relevant log: http://paste.openstack.org/show/521653/ | 11:24 |
| dtantsur | lazy_prince, correct. is there any problems with it? | 11:24 |
| openstackgerrit | Merged openstack/ironic-python-agent: Replace assertRaisesRegexp with assertRaisesRegex https://review.openstack.org/332250 | 11:25 |
| lazy_prince | yes.. it results in 404 | 11:25 |
| dtantsur | lazy_prince, hmm, it reminds me something.. when do you face this problem? | 11:25 |
| lazy_prince | cleaning port delete.. | 11:25 |
| lazy_prince | https://bugs.launchpad.net/ironic/+bug/1590408 | 11:26 |
| openstack | Launchpad bug 1590408 in Ironic "Node cleaning fails with 'Failed to tear down from cleaning for node UUID'" [Undecided,Incomplete] | 11:26 |
| *** ekarlso has quit IRC | 11:26 | |
| dtantsur | ah, yeah, the bug rbartal faced | 11:26 |
| jroll | dtantsur: yeah, I don't think heartbeat needs to validate deploy | 11:27 |
| vdrok | lazy_prince: it should be /v3/auth/tokens | 11:27 |
| * jroll will think about it more though | 11:27 | |
| dtantsur | vdrok, lazy_prince, I suspect a bug in neutronclient, but need to investigate | 11:27 |
| lazy_prince | for me, main concern is I am using keystone v3 apis and its failing to obtain a token.. | 11:27 |
| dtantsur | lazy_prince, what's your version of neutronclient? | 11:27 |
| *** daemontool has joined #openstack-ironic | 11:28 | |
| lazy_prince | python-neutronclient==4.1.3 | 11:29 |
| dtantsur | ok | 11:29 |
| vdrok | lazy_prince: right, so it seems that the endpoint is set incorrectly, what is the value of CONF.keystone_authtoken.auth_uri ? | 11:30 |
| vdrok | https://github.com/openstack/ironic/blob/master/ironic/dhcp/neutron.py#L89 | 11:30 |
| lazy_prince | https://10.1.1.20:5000/v3 | 11:30 |
| lazy_prince | vdrok: can you help me fix the endpoint..? | 11:31 |
| vdrok | lazy_prince: yeah, looking at the code, I don't remember right now what it should include | 11:32 |
| lazy_prince | vdrok: i have tried with https://10.1.1.20:5000/v3 and https://10.1.1.20:5000 | 11:33 |
| *** dmk0202 has joined #openstack-ironic | 11:33 | |
| lazy_prince | tried with https://10.1.1.20:5000/v3/auth too but that resulted in 400 error.. | 11:33 |
| dtantsur | lazy_prince, looks like it should be /v3/auth: https://github.com/openstack/python-neutronclient/blob/master/neutronclient/client.py#L217 | 11:36 |
| dtantsur | but it might break other users of our keystone_token parameters | 11:36 |
| vdrok | lazy_prince: yeah, what dtantsur says is correct | 11:37 |
| lazy_prince | vdrok: {"error": {"message": "Expecting to find identity in auth - the server could not comply with the request since it is either malformed or otherwise incorrect. The client is assumed to be in error.", "code": 400, "title": "Bad Request"}} | 11:37 |
| vdrok | lazy_prince: we don't pass domains in case of v3 to neutron client | 11:38 |
| vdrok | that is the problem | 11:38 |
| lazy_prince | this is the error I get when i use /v3/auth | 11:38 |
| vdrok | without domain it gives 400 | 11:38 |
| *** dedery has quit IRC | 11:38 | |
| dtantsur | lazy_prince, vdrok, I wonder if https://review.openstack.org/236982 will fix it | 11:39 |
| lazy_prince | so does that mean keystone v3 api is not supported by ironic yet... | 11:39 |
| dtantsur | pas-ha, wdyt ^^? | 11:39 |
| dtantsur | OR we might need to create a small backportable fix, because apparently the issue is in mitaka too | 11:40 |
| dtantsur | (it's probably everywhere, but we didn't use v3 actively before mitaka) | 11:40 |
| lazy_prince | well.. I am using mitaka... | 11:40 |
| vdrok | dtantsur: yeah, that should fix the things | 11:40 |
| dtantsur | I wonder if we can fix it by prepending /auth if the auth_url contains /v3 | 11:42 |
| dtantsur | lazy_prince, will you be able to test a patch if I put it up? | 11:42 |
| lazy_prince | yup... I can do that.. | 11:42 |
| lazy_prince | but put it for mitaka so that it would make merge simple for me.. | 11:43 |
| vdrok | dtantsur: huh, there are no domain parameters in client constructor | 11:43 |
| vdrok | https://github.com/openstack/python-neutronclient/blob/master/neutronclient/v2_0/client.py#L181 | 11:43 |
| *** fragatina has joined #openstack-ironic | 11:43 | |
| vdrok | so the only way seems to be using sessions | 11:43 |
| dtantsur | I'm not sure I get it, but lemme try | 11:44 |
| lazy_prince | vdrok: so does it mean, mitaka will never work with ks v3 apis.. | 11:44 |
| vdrok | lazy_prince: it seems to me it will only if we backport the whole keystoneauth thing to mitaka... | 11:45 |
| lazy_prince | do we have any eta by which it will merge and by which backport will finish..? | 11:46 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] A simple workaround for Keyston v3 support in ironic.dhcp.neutron https://review.openstack.org/333276 | 11:46 |
| dtantsur | lazy_prince, lets try this ^^^ | 11:46 |
| dtantsur | I think it'll apply cleanly to mitaka | 11:46 |
| dtantsur | vdrok, I'm not sure I get it, why can't we just fix the auth_url? | 11:46 |
| lazy_prince | dtantsur: it wont work.. | 11:47 |
| dtantsur | why? | 11:47 |
| vdrok | dtantsur: because if v3 is used user_domain_id and project_domain_id are required | 11:47 |
| vdrok | and there are no such arguments in neutron client constructor | 11:47 |
| dtantsur | hmm | 11:47 |
| vdrok | and the way to solve it is to create a keystoneauth session with all the parameters | 11:48 |
| vdrok | and pass it into neutron client constructor | 11:48 |
| openstackgerrit | Sofiia Andriichenko proposed openstack/python-ironicclient: Negative tests for testing chassis-create command https://review.openstack.org/322788 | 11:48 |
| openstackgerrit | Sofiia Andriichenko proposed openstack/python-ironicclient: Negative tests for testing actions with Chassis https://review.openstack.org/301757 | 11:48 |
| vdrok | i'm trying this locally now | 11:48 |
| lazy_prince | i tried CONF.keystone_authtoken.auth_uri with /v3/auth and it did not work.. it gave me {"error": {"message": "Expecting to find identity in auth - the server could not comply with the request since it is either malformed or otherwise incorrect. The client is assumed to be in error.", "code": 400, "title": "Bad Request"}} | 11:49 |
| dtantsur | lazy_prince, was this failure in ironic.dhcp.neutron or somewhere before? | 11:49 |
| lazy_prince | same place where i got earlier issue... | 11:50 |
| dtantsur | ok, abandoning this one | 11:50 |
| pas-ha | a better solution is this chain https://review.openstack.org/#/q/topic:use-ksa+project:openstack/ironic | 11:51 |
| *** ekarlso has joined #openstack-ironic | 11:51 | |
| dtantsur | pas-ha, yes, but how do suggest we backport it to mitaka? | 11:51 |
| pas-ha | ouch, that would not be that easy | 11:51 |
| dtantsur | that's the problem :) | 11:51 |
| dtantsur | I'm not saying we should not merge your patch chain ofc | 11:52 |
| dtantsur | I'm just trying to think of something we could backport | 11:52 |
| pas-ha | mainly due to the fact that we reuse keystone_authtoken for building clients | 11:52 |
| dtantsur | yeah, that's a Bad Idea (tm) | 11:52 |
| * dtantsur -> quick lunch, brb | 11:52 | |
| openstackgerrit | Aparna proposed openstack/proliantutils: Sort values of raid_config for 'share_physical_disks' https://review.openstack.org/326319 | 11:53 |
| lazy_prince | so i would take it that as of now, ironic does not fully support ks v3 api... can we document this at-least somewhere..? | 11:53 |
| lazy_prince | (scratch fully) | 11:53 |
| sambetts | hmm I thought devstack used ks v3 and our gate isn't broken | 11:54 |
| lazy_prince | sambetts: yup.. but code says otherwise.. :) | 11:54 |
| *** daemontool_ has joined #openstack-ironic | 11:54 | |
| lazy_prince | may be devstack still uses v2.0 ks apis for ironic | 11:55 |
| *** daemontool has quit IRC | 11:55 | |
| vdrok | sambetts: I think it has v3 but does not disable v2 | 11:56 |
| lazy_prince | https://review.openstack.org/#/c/297312/15/devstack/lib/ironic#l434 | 11:56 |
| lazy_prince | you can see devstack still uses v2.0 auth uri for ironic.. | 11:57 |
| jroll | devstack still uses v2 by default | 11:57 |
| jroll | everywhere | 11:57 |
| lazy_prince | morning jroll | 11:57 |
| jroll | some folks are working on v3 by default, they're supposed to put a job on ironic for it, dunno what happened to that | 11:57 |
| jroll | hey lazy_prince :) | 11:57 |
| *** xavierr has joined #openstack-ironic | 11:58 | |
| *** dprince has joined #openstack-ironic | 12:01 | |
| pas-ha | sambetts: yep, there was a patch from dims trying to move devstack to Kv3 only, and Ironic broke :( that was an original starter for my patches | 12:02 |
| sambetts | ah :( | 12:02 |
| vdrok | lazy_prince: and another thing, just /v3 without /auth is correct :) | 12:02 |
| lazy_prince | vdrok: thats what i thought and started with.. :) | 12:03 |
| lazy_prince | but then the failures confused me and i stared with other variants | 12:04 |
| lucas-hungry | folks, I'm feeling a bit under the weather today | 12:04 |
| lucas-hungry | I'll take the afternoon out | 12:04 |
| *** lucas-hungry is now known as lucas-afk | 12:04 | |
| vdrok | lucas-afk: get better! | 12:04 |
| lucas-afk | cheers | 12:04 |
| *** achanda has joined #openstack-ironic | 12:06 | |
| *** fragatina has quit IRC | 12:07 | |
| *** achanda has quit IRC | 12:10 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add test for 'baremetal port create' command https://review.openstack.org/332880 | 12:14 |
| xavierr | bom dia Ironic (good morning in portuguese) :) | 12:15 |
| xavierr | lucas-afk hi there, take care! | 12:15 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Warn on legacy auth options on conductor start https://review.openstack.org/326660 | 12:20 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Update devstack plugin with new auth options https://review.openstack.org/297312 | 12:20 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Migrate to using keystoneauth Sessions https://review.openstack.org/236982 | 12:20 |
| openstackgerrit | Tang Chen proposed openstack/python-ironicclient: Use osc-lib instead of openstackclient https://review.openstack.org/333311 | 12:21 |
| *** jcoufal has joined #openstack-ironic | 12:21 | |
| *** fragatina has joined #openstack-ironic | 12:22 | |
| *** amotoki has quit IRC | 12:23 | |
| TheJulia | Good morning | 12:24 |
| *** jcoufal_ has joined #openstack-ironic | 12:26 | |
| TheJulia | dtantsur: re https://bugs.launchpad.net/ironic/+bug/1590408, it looks like neutronclient is completely unaware of what to do so it just defaults ot taking the auth uri and appending /tokens to it | 12:28 |
| openstack | Launchpad bug 1590408 in Ironic "Node cleaning fails with 'Failed to tear down from cleaning for node UUID'" [High,Confirmed] | 12:28 |
| *** jcoufal has quit IRC | 12:29 | |
| openstackgerrit | Sofiia Andriichenko proposed openstack/python-ironicclient: Add test for 'node-show --field' command https://review.openstack.org/300064 | 12:31 |
| *** jjohnson2_ has joined #openstack-ironic | 12:31 | |
| *** thiagop-afk has joined #openstack-ironic | 12:31 | |
| thiagop-afk | Good morning, Ironic | 12:32 |
| *** thiagop-afk is now known as thiagop | 12:32 | |
| vdrok | morning TheJulia and thiagop | 12:32 |
| lazy_prince | TheJulia: I guess, we will hit the same issue when using swift... | 12:33 |
| lazy_prince | so if we fix, we fix both.. | 12:33 |
| thiagop | And great congratulations to dtantsur on the approval of the driver composition spec. A fight well fought with infinite patience and good will. | 12:33 |
| *** trown|outtypewww is now known as trown | 12:34 | |
| TheJulia | \o/ | 12:34 |
| *** M00nr41n has quit IRC | 12:37 | |
| *** ccamacho is now known as ccamacho|lunch | 12:42 | |
| *** ametts has joined #openstack-ironic | 12:43 | |
| *** sdake has joined #openstack-ironic | 12:48 | |
| *** fragatina has quit IRC | 12:51 | |
| *** smoriya has quit IRC | 12:55 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: WIP: Discovery tempest test https://review.openstack.org/323769 | 12:56 |
| *** aparnav has quit IRC | 12:57 | |
| *** jvgrant has joined #openstack-ironic | 12:59 | |
| *** baoli has joined #openstack-ironic | 13:01 | |
| *** cdearborn has joined #openstack-ironic | 13:03 | |
| *** rloo has joined #openstack-ironic | 13:03 | |
| *** hshiina has quit IRC | 13:03 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Negative tests for testing actions with Chassis https://review.openstack.org/301757 | 13:04 |
| dtantsur | jroll, as you're one of assignees on the agent vendor API thing, could you please review https://review.openstack.org/330938 and https://review.openstack.org/332890 ? :) | 13:05 |
| *** fragatina has joined #openstack-ironic | 13:05 | |
| jroll | dtantsur: yeah, I'll try | 13:07 |
| dtantsur | thnx | 13:07 |
| rloo | hi and morning all! | 13:15 |
| rloo | dtantsur: i'd like a followup to the reform spec. I think (for me) it is important to mention that when it mentions an interface becoming invalid, it only (in that spec) means disabled. | 13:16 |
| *** rbudden has joined #openstack-ironic | 13:16 | |
| dtantsur | rloo, morning, will do | 13:16 |
| rloo | dtantsur: OH. just reading jroll's comment. invalid also if not supported. sigh. | 13:16 |
| rloo | dtantsur, jroll: don't really want to deal with it now, but really, if a hw vendor says an interface is supported, how can they no longer support it? that's a breaking change. | 13:17 |
| dtantsur | rloo, imagine 3rd party vendors. we can't affect them | 13:17 |
| rloo | dtantsur: so do we do a microversion update what that happens? | 13:18 |
| rloo | s/what/when/ | 13:18 |
| dtantsur | rloo, we can't. we don't have drivers covered by microversions | 13:18 |
| dtantsur | we don't even have control over out-of-tree drivers | 13:18 |
| jroll | rloo: yeah, for in tree things we'd have some deprecation | 13:19 |
| jroll | we can't do anything about out of tree | 13:19 |
| rloo | dtantsur: this is intree. out-of-tree don't have supported anything, do they? that is in code. | 13:19 |
| jroll | even with deprecation, the code still needs to handle it | 13:19 |
| jroll | (if the operator doesn't follow deprecation warnings) | 13:19 |
| dtantsur | rloo, sorry? imagine an out-of-tree hw type removes an interface people use. we'll have to cope with it somehow. | 13:19 |
| rloo | dtantsur: no, we aren't responsible for out-of-tree hw types. | 13:20 |
| jroll | I think nulling task.driver.foo is reasonable, and any code we write should expect that | 13:20 |
| rloo | dtantsur: how do they hook in their outoftree drivers? they'd have to pull in their classes, etc. | 13:20 |
| jroll | rloo: but the internal code needs to handle things that out of tree drivers do | 13:20 |
| rloo | dtantsur: we'd have to cope with it wrt ironic not falling down and dying i guess, but we don't have to deal with supporting their nonsupport. | 13:20 |
| dtantsur | rloo, define a hardware type, create an entry point for it. we can have 3rd party drivers by design | 13:21 |
| sambetts | I suggest a InvalidInterface object that implements all functions a no-op except validate, I don't want to have if task.driver.<interface> is not None: everywhere | 13:21 |
| dtantsur | rloo, yeah, by "cope" I mean "don't use HTTP 500 on every request touching this node" | 13:21 |
| dtantsur | sambetts, how is try.. except NotImplementedError.. better? | 13:21 |
| sambetts | I never said a notimplementederror I said a Noop | 13:22 |
| dtantsur | I hope you don't mean that InvalidInterface will just no-op every call... | 13:22 |
| dtantsur | oh | 13:22 |
| dtantsur | so the node will be silently broken? | 13:22 |
| jroll | that's equivalent to: if task.driver.foo is None: task.driver.foo = FakeFoo() | 13:22 |
| jroll | :P | 13:22 |
| sambetts | no because it fails to validate | 13:22 |
| sambetts | because of def validate | 13:23 |
| rloo | dtantsur, sambetts: I think/hope that when dtantsur codes it, it will become more obvious what to do/how to handle. deciding on having an InvalidInterface seems premature (and doesn't actually make sense to me right now.) | 13:23 |
| jroll | do people often run 'ironic node-validate'? | 13:23 |
| jroll | I don't think I've ever run that downstream | 13:23 |
| sambetts | jroll: no but things like nova do | 13:23 |
| jroll | sure | 13:23 |
| jroll | I think either way it needs some though | 13:23 |
| jroll | t | 13:23 |
| dtantsur | I think we should make sure that API requests and periodic tasks are not processed when the interface fails to validate | 13:24 |
| dtantsur | then we won't have to decide what to do about None | 13:24 |
| jroll | if e.g. task.driver.deploy.foo() is always guarded by a validate call, no need to check if task.driver.derploy is None | 13:24 |
| jroll | yeah | 13:24 |
| sambetts | exactly, I don't think there is a need to have a if interfacs is None thing, when we can just use the existing validate function check | 13:25 |
| jroll | right, which means there's also no need for InvalidInterface, right? :) | 13:25 |
| *** _vs_ has joined #openstack-ironic | 13:25 | |
| sambetts | no because it'll have to call validate on something right? | 13:25 |
| sambetts | so that we can get a resonable message in the validate response | 13:26 |
| jroll | well | 13:26 |
| jroll | we could have the null checks there | 13:26 |
| sambetts | unless your expecting a try: except TypeError | 13:26 |
| rloo | i think (maybe) what sambetts is saying (or at least I am thinking) is that there may not even be a class/interface associated with the node.x_interface value, if eg a 3rdparty driver removed that class. | 13:27 |
| sambetts | jroll: we'd need null checks everywhere task.driver.<interface> is called | 13:27 |
| jroll | sambetts: not if those are guarded by validate calls | 13:27 |
| jroll | validate_interface(task.driver.deploy); task.driver.deploy.foo() | 13:27 |
| sambetts | I'm thinking more like things like sync power state, do we validate(task.driver.power) in there/ | 13:28 |
| sambetts | ? | 13:28 |
| jroll | sambetts: also, current validation code already checks if it's None | 13:28 |
| jroll | yeah | 13:28 |
| dtantsur | sambetts, we need validate calls everywhere task.driver.XXX is called | 13:28 |
| openstackgerrit | Sofiia Andriichenko proposed openstack/python-ironicclient: Negative tests for testing chassis-create command https://review.openstack.org/322788 | 13:28 |
| dtantsur | i.e. if the deploy interface calls boot interface, deploy validate must check boot validate | 13:29 |
| dtantsur | I'm not sure if we do it now, but it's worth doing | 13:29 |
| jroll | sambetts: so the way we call it needs to change, but we do it already https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L2371 | 13:29 |
| sambetts | maybe just have a bulk interface validate at the point we build the task.driver object? | 13:29 |
| jroll | hm, maybe | 13:29 |
| jroll | not sure the implications there | 13:30 |
| jroll | anyway, I'm not sure we can really figure this out without playing with the code, honestly | 13:30 |
| sambetts | I was just thinking a InvalidInterface with all the functions passing except validate would just work out of the box now with our current checls | 13:30 |
| jroll | well, it'd work in that it wouldn't throw random AttributeErrors, but instead just do nothing and pretend it did something | 13:31 |
| jroll | I'm not sure which is better :) | 13:31 |
| hkominos | well i run node-validate for sure :P | 13:31 |
| *** jistr|mtg is now known as jistr | 13:31 | |
| hkominos | jroll | 13:31 |
| dtantsur | I'm thinking that we could have InvalidInterface that raise something meaninful on every call | 13:31 |
| jroll | hkominos: :) | 13:31 |
| *** david-lyle_ has joined #openstack-ironic | 13:32 | |
| dtantsur | I've changed tripleo scripts to run validate as well, previously it wasn't used there | 13:32 |
| dtantsur | inspector has always run it | 13:32 |
| sambetts | dtantsur: at that point we might as well just catch it at task.driver build time and always error out early | 13:32 |
| *** catintheroof has joined #openstack-ironic | 13:32 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Functional tests for portgroups in ironicclient https://review.openstack.org/297727 | 13:32 |
| *** baoli_ has joined #openstack-ironic | 13:33 | |
| dtantsur | sambetts, that will make it impossible to update such node, no? | 13:33 |
| jroll | whomp | 13:33 |
| jroll | I was just going to say it should work, but that's correct | 13:33 |
| *** ccamacho|lunch is now known as ccamacho | 13:33 | |
| sambetts | dtantsur: interesting, if we're updating the interfaces shouldn't we wait till after that to build task.driver? | 13:33 |
| dtantsur | also a node with invalid inspect interface can still (arguably!) sync power states | 13:33 |
| dtantsur | sambetts, I'll check how our update works now | 13:34 |
| *** joprovost has joined #openstack-ironic | 13:34 | |
| jroll | our update uses the new driver to figure out the conductor to send to | 13:34 |
| *** rbudden has quit IRC | 13:34 | |
| *** yolanda has joined #openstack-ironic | 13:34 | |
| *** piet has joined #openstack-ironic | 13:34 | |
| *** baoli has quit IRC | 13:35 | |
| sambetts | so we have to wait until after the update to build the task.driver anyway | 13:35 |
| *** achanda has joined #openstack-ironic | 13:35 | |
| dtantsur | https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L117-L120 | 13:35 |
| sambetts | right? otherwise it'll go to the wrong conductor? *mind melt* | 13:35 |
| dtantsur | we don't build tasks on API side | 13:35 |
| dtantsur | we just take the "driver" field and send the request | 13:35 |
| dtantsur | then the conductor builds the task | 13:36 |
| *** david-lyle has quit IRC | 13:36 | |
| dtantsur | see the code above | 13:36 |
| sambetts | ah this comes back to the whole API -> DB or Conductor -> DB | 13:36 |
| sambetts | thing | 13:36 |
| jroll | also related https://github.com/openstack/ironic/blob/master/ironic/api/controllers/v1/node.py#L1301-L1312 | 13:36 |
| jroll | to find which conductor the request goes to | 13:36 |
| dtantsur | yeah | 13:36 |
| jroll | it calculates that with the new driver | 13:37 |
| dtantsur | so as it's written now, we call task manager before updating the interfaces | 13:37 |
| dtantsur | but! we don't use a task object at all, we just use locking | 13:37 |
| dtantsur | so it seems solvable in theory | 13:37 |
| jroll | right, so we can't blow up in acquire() | 13:38 |
| sambetts | hmm yeah, implemention of this is going to be fun XD | 13:38 |
| dtantsur | I'm still not entirely convinced that something like a broken inspect interface should make the whole node unusable (incl. power syncs) | 13:38 |
| jroll | or maybe acquire gets a validate_driver=True | 13:38 |
| jroll | and that one spot overrides to False | 13:38 |
| dtantsur | I was thinking about something like that, but see above | 13:38 |
| jroll | right | 13:38 |
| * jroll suddenly gets worried about putting his name on this one :D | 13:40 | |
| sambetts | I can't put it together in my head right now, probably much easier to work out once we've got code up to see how much of everything we touch :-P | 13:40 |
| dtantsur | jroll, lol, too late to retreat :D | 13:40 |
| jroll | sambetts: indeed | 13:40 |
| dtantsur | sambetts, that's what rloo suggests and I agree with her. lets get something up and get back to a planning board if we're not pleased with how it shapes | 13:41 |
| jroll | maybe this ends in a complete rewrite of task management :) | 13:41 |
| sambetts | dtantsur: +1 | 13:41 |
| dtantsur | jroll, with tooz on the radar it may happen sooner or later anyway | 13:41 |
| rloo | i have complete faith that dtantsur will come up with the right solution :) | 13:41 |
| sambetts | oooooo yeah thats going to be super interesting | 13:41 |
| dtantsur | thanks :) | 13:41 |
| sambetts | owl power! | 13:41 |
| dtantsur | \o/ | 13:42 |
| dtantsur | #pixiesay owl power! | 13:42 |
| PixieBoots | ʕ•͡ᴥ•ʔ: owl power! | 13:42 |
| jroll | dtantsur: quick, let's ninja edit the spec to use a DLM here | 13:42 |
| dtantsur | LOL | 13:42 |
| dtantsur | it will make the spec so much simpler for sure | 13:42 |
| dtantsur | </sarcasm> | 13:42 |
| hkominos | sorry to bother you ironicers but i got a quick question.Does the deploy disk give a specific name to the HDD on the machines? and if so how do i find tha name? | 13:43 |
| dtantsur | hkominos, not sure what you mean by "give name" here | 13:44 |
| jroll | hkominos: like a partition label? | 13:44 |
| hkominos | yes | 13:44 |
| hkominos | I wantto deploy a full disk image | 13:45 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add test for 'baremetal port create' command https://review.openstack.org/332880 | 13:45 |
| hkominos | and i think i need to set some metadata to the image so that ironic can pick them up and format the disk correct? | 13:45 |
| jroll | ironic doesn't format the disk when using full disk images | 13:45 |
| jroll | it just dd's the entire image to the block device for the disk | 13:46 |
| *** achanda has quit IRC | 13:46 | |
| hkominos | ok. So it has to know for example its size right? | 13:46 |
| *** foexle has quit IRC | 13:46 | |
| hkominos | and what is the block devices name | 13:46 |
| jroll | yes, that's a glance property | 13:46 |
| jroll | for which block device, by default it chooses the smallest disk that is larger than 4gb | 13:46 |
| jroll | but you can override it with hints | 13:46 |
| * jroll finds doc | 13:47 | |
| jroll | hkominos: http://docs.openstack.org/developer/ironic/deploy/install-guide.html#specifying-the-disk-for-deployment | 13:47 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add sanity tests for baremetal power state commands https://review.openstack.org/333361 | 13:48 |
| hkominos | excellent. I have only one disk though so i would assume that it is always sda? | 13:49 |
| jroll | probably :) anyway ironic will find the disk just fine without specifying it | 13:50 |
| *** PollyZ has joined #openstack-ironic | 13:50 | |
| hkominos | the point is that it does not. | 13:50 |
| jroll | oh? :/ | 13:50 |
| hkominos | or at least it does not find the partition correctly. | 13:50 |
| jroll | got any logs or something? | 13:51 |
| hkominos | or i am doing something wrong (much much more probable) | 13:51 |
| jroll | hehe | 13:51 |
| jroll | yeah, curious what's going on there | 13:51 |
| hkominos | before i show the error message I should specify that i am using the fuel_Ipmi driver which in order to work and give hints to the deploy image you specify metadata in the user image | 13:52 |
| hkominos | like so | 13:52 |
| hkominos | https://github.com/openstack/fuel-agent/tree/master/contrib/ironic/ironic-fa-deploy | 13:52 |
| hkominos | --property fuel_disk_info='[{"name": "sda", "extra": [], "free_space": 11000, "type": "disk", "id": "vda", "size": 11000, "volumes": [{"mount": "/", "type": "partition", "file_system": "ext4", "size": 10000}]}]' | 13:52 |
| hkominos | more specificaly | 13:52 |
| jroll | oh, that's completely different, and not something we support :/ | 13:53 |
| * jroll has no clue how the fuel driver or the fuel agent work | 13:53 | |
| jroll | maybe try asking the fuel folks for help? | 13:53 |
| *** _vsaienko has joined #openstack-ironic | 13:55 | |
| *** _vs_ has quit IRC | 13:55 | |
| hkominos | Cant find them atm. + i am just trying to understand ironic. | 13:55 |
| hkominos | So i would assume that it works in a very similar way | 13:55 |
| *** yolanda has quit IRC | 13:56 | |
| jroll | well, they use a different deploy driver and agent, so how it finds the disk may be different | 13:56 |
| jroll | this seems like it might be where they find the disk https://github.com/openstack/fuel-agent/blob/master/fuel_agent/drivers/nailgun.py | 13:58 |
| jroll | but it's hard to tell without reading it all | 13:59 |
| jroll | sorry, I'd help if I could but don't think I can :( | 13:59 |
| * jroll bbiab | 13:59 | |
| *** rbradfor has joined #openstack-ironic | 14:00 | |
| hkominos | You help quite a lot. I am sure there are similarities so i might find the problem. the question is this. I think that it finds the disks. What it does not find is a filesystem. (cant find ext4 filesystem is the error) . So I was wondering if that meant that it cant find the disk or it is something else | 14:01 |
| rloo | TheJulia: did you or someone resolve this? L607: https://review.openstack.org/#/c/307225/5/ironic/drivers/modules/irmc/boot.py | 14:02 |
| *** rbradfor has quit IRC | 14:02 | |
| *** joprovost1 has joined #openstack-ironic | 14:03 | |
| openstackgerrit | Merged openstack/ironic-webclient: Correct markup of README.md https://review.openstack.org/332707 | 14:03 |
| openstackgerrit | Merged openstack/ironic-webclient: Set IP and port using environment variables https://review.openstack.org/332706 | 14:03 |
| jlvillal | dtantsur, Do you know why for PCI it was decided to use 'product_id" instead of 'device_id'? https://review.openstack.org/#/c/326318/8/ironic_python_agent/inspector.py | 14:04 |
| jlvillal | I am used to PCI being VENDOR_ID and DEVICE_ID. | 14:04 |
| *** joprovost has quit IRC | 14:04 | |
| *** joprovost1 is now known as joprovost | 14:04 | |
| *** achanda has joined #openstack-ironic | 14:05 | |
| jlvillal | Line 432 in the patch | 14:05 |
| *** rbradfor has joined #openstack-ironic | 14:05 | |
| jlvillal | sambetts, Same question to you :) ^^^ | 14:06 |
| sambetts | jlvillal: not sure, not much experience with PCI stuff like that | 14:07 |
| *** achanda has quit IRC | 14:07 | |
| jlvillal | sambetts, https://en.wikipedia.org/wiki/PCI_configuration_space#Standardized_registers | 14:10 |
| *** rbartal has quit IRC | 14:12 | |
| *** sdake has quit IRC | 14:13 | |
| *** sabeen has joined #openstack-ironic | 14:13 | |
| sambetts | jlvillal: I like standards :) | 14:15 |
| *** sdake has joined #openstack-ironic | 14:16 | |
| dtantsur | jlvillal, no idea, I don't have a strong opinion there | 14:16 |
| dtantsur | jlvillal, we can amend the spec if needed | 14:16 |
| *** baoli_ has quit IRC | 14:19 | |
| *** rook has joined #openstack-ironic | 14:19 | |
| vdrok | hkominos: fuel_agent does not support whole disk images, only partition images | 14:21 |
| rook | morning JayF -- i hear you might be looking at adding metrics to Ironic -- i am interested in tracking the time it takes for ironic to deploy a instnace. | 14:21 |
| *** baoli has joined #openstack-ironic | 14:21 | |
| vdrok | you can switch to agent_ipmitool for whole disk | 14:21 |
| *** piet has quit IRC | 14:22 | |
| *** amotoki has joined #openstack-ironic | 14:23 | |
| rloo | rook, JayF et al have some patches for metrics; what you want might be covered by L423: https://review.openstack.org/#/c/301923/2/ironic/api/controllers/v1/node.py | 14:25 |
| rloo | rook: actually, no it won't. because it only times the request. not the background time to actually do the deploy. | 14:26 |
| *** foexle has joined #openstack-ironic | 14:26 | |
| rloo | dtantsur, jlvillal: wrt trello cards (cuz i'm too lazy to read up on how to use trello). are 'members' of a card the folks that are the contacts/leads/whatever? | 14:27 |
| dtantsur | rloo, yes. everyone directly involved | 14:27 |
| jroll | rloo: with most people that use trello, they use that for either assignees, or people add themselves so they get notifications when the card changes | 14:27 |
| rloo | dtantsur: ok thx. i had put down 'contacts' or something, but i'll change that to using members if they have trello accounts. and if i can figure out how to do it :) | 14:27 |
| jroll | we can of course use 'members' however we like | 14:28 |
| jroll | every core except TheJulia, yuriyz, and haomeng are on trello now | 14:28 |
| rloo | jroll: OH. well, can we use 'members' to be the contacts? if we do that, is there some other way for folks to get notifications? | 14:28 |
| jroll | rloo: sure, we totally can, I'm good with that | 14:29 |
| jroll | other than that, not sure, let me have a look | 14:29 |
| jroll | rloo: oh, folks can push 'subscribe' for that | 14:29 |
| rloo | jroll: ok, then we are good with using 'members' as owners/leads/... | 14:30 |
| dtantsur | +1 | 14:30 |
| jroll | rloo: ++ | 14:30 |
| TheJulia | jroll: you can add me if you want, my username in trello is thejulia | 14:32 |
| jroll | TheJulia: done \o/ | 14:32 |
| *** amotoki has quit IRC | 14:32 | |
| *** _vsaienko has quit IRC | 14:35 | |
| hkominos | vdrok :thx. Speaking of which i found https://bugs.launchpad.net/mos/+bug/1568879 . Is it valid?should the json be vda everywhere? | 14:35 |
| openstack | Launchpad bug 1568879 in Mirantis OpenStack "User guide Ironic section improvement" [Undecided,New] - Assigned to Fuel Documentation Team (fuel-docs) | 14:35 |
| rloo | jroll: qq. I believe JayF is hiding til he gets the keystone policy spec done. Is he also leading the rescue stuff? | 14:35 |
| jroll | rloo: he is indeed | 14:36 |
| *** _vsaienko has joined #openstack-ironic | 14:36 | |
| rloo | jroll: thx | 14:37 |
| TheJulia | rloo: *sigh* lost track of that one. | 14:37 |
| jroll | welcome | 14:37 |
| *** ayoung has quit IRC | 14:37 | |
| *** jistr is now known as jistr|mtg | 14:38 | |
| yuriyz | hi Ironic jroll please add @yuriyzveryanskyy to trello members | 14:38 |
| vdrok | hkominos: yep, it should be the same disk name everywhere | 14:38 |
| hkominos | omg . I wild manual appeared!! | 14:38 |
| dtantsur | jroll, I think we can move driver composition to "code patches". while no patches are bound to it specifically, the agent vendor API can probably count, wdyt? | 14:39 |
| jroll | dtantsur: yeah, go for it | 14:40 |
| rloo | TheJulia: I was going to approve but wanted to make sure it was ok; i didn't want to read through the chain of comments. | 14:40 |
| jroll | yuriyz: done, thanks! | 14:40 |
| *** amotoki has joined #openstack-ironic | 14:40 | |
| rloo | dtantsur: isn't the agent vendor part of another rfe? | 14:41 |
| *** links has quit IRC | 14:41 | |
| *** pcaruana has quit IRC | 14:41 | |
| dtantsur | rloo, it is technically. but logically I'm rushing with it due to the driver composition | 14:41 |
| jroll | yeah, it's a dependency for driver comp | 14:42 |
| dtantsur | otherwise we won't be able to separate agent deploy from passthru | 14:42 |
| rloo | dtantsur: wondering if we should just add a separate card for the agent vendor stuff then. | 14:42 |
| jroll | separate card does seem better | 14:42 |
| dtantsur | I don't mind either way, I just don't want to spend too much time on cards :) | 14:42 |
| dtantsur | feel free to fix it as you feel appropriate | 14:42 |
| *** jrist has quit IRC | 14:43 | |
| *** joprovost has quit IRC | 14:43 | |
| rloo | dtantsur: i'll add a card. am going through trello now anyway. | 14:43 |
| dtantsur | thnx | 14:43 |
| rloo | jlvillal: grenade is voting now, right? | 14:43 |
| *** baoli has quit IRC | 14:43 | |
| dtantsur | it is | 14:44 |
| * rloo moves 'make grenade voting' card to Done! | 14:44 | |
| jroll | \o/ | 14:44 |
| *** _vsaienko has quit IRC | 14:44 | |
| jlvillal | rloo, yes | 14:45 |
| *** _vsaienko has joined #openstack-ironic | 14:45 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add sanity tests for baremetal power state commands https://review.openstack.org/333361 | 14:46 |
| mgould | rloo: yay! | 14:47 |
| *** _vsaienko has quit IRC | 14:47 | |
| TheJulia | rloo: I _think_ it is okay, but the back and forth has really been with different contextual perspectives :\ | 14:47 |
| *** baoli has joined #openstack-ironic | 14:48 | |
| rloo | TheJulia: which is why i don't want to get into it :) I suspect the comment may be incorrect wrt conductor takeover based on what nao says but that can be fixed later if it is incorrect. | 14:48 |
| *** foexle has quit IRC | 14:49 | |
| TheJulia | rloo: hmmm looking again | 14:49 |
| *** joprovost has joined #openstack-ironic | 14:50 | |
| rloo | vdrok: if you have a minute, would you take a look at the description; I took it from your comment but not sure I got it right: https://trello.com/c/0xRRc77z/12-ci-test-agent-driver-with-partition-images | 14:50 |
| *** _vsaienko has joined #openstack-ironic | 14:51 | |
| vdrok | rloo: yep, it's correct :) | 14:51 |
| rloo | vdrok: so 330738 is the -nv partition images job that you mention in the first sentence? | 14:52 |
| openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/333177 | 14:52 |
| *** baoli has quit IRC | 14:52 | |
| vdrok | rloo: those are on IPA and there is one in ironic too with almost the same name. they all need that change to devstack plugin to do what they should | 14:53 |
| rloo | vdrok: could you list the patches needed to address that? or a link to list of patches | 14:53 |
| jroll | sambetts: mind putting your eyes on this? https://review.openstack.org/#/c/327009/7 | 14:54 |
| jroll | morgabra: ^ you might also be interested | 14:54 |
| vdrok | rloo: OK, I'll change the description a bit, thanks | 14:54 |
| *** baoli has joined #openstack-ironic | 14:54 | |
| jroll | johnthetubaguy: if I wanted to test that stack harder, would you recommend I base that on https://review.openstack.org/#/c/332712/2 or something else in the middle? | 14:54 |
| *** jrist has joined #openstack-ironic | 14:56 | |
| sambetts | jroll: hmmm interesting, /me still wants to nuke Ironic setting mac addresses like that in Nova | 14:56 |
| *** sacharya has joined #openstack-ironic | 14:57 | |
| jroll | sambetts: yeah, I think we could still do that later | 14:57 |
| TheJulia | rloo: I think the comment is correct, but I've also walked the four ways that method can get called too :( | 14:58 |
| rloo | TheJulia: ok. going to approve. thx. | 14:59 |
| *** david-lyle_ is now known as david-lyle | 14:59 | |
| sambetts | jroll: yeah, whats interesting about this to me is it basically does what I wanted to do for mac addresses except that it does it in Nova still, e.g. create port with generated mac address, and update it later with the mac address you want. the only difference is that I want to do it way later like post provisioning later | 14:59 |
| *** ijw has joined #openstack-ironic | 15:01 | |
| jlvillal | sambetts: TheJulia mat128: Ready to chat about multi-node Grenade? | 15:03 |
| mat128 | oh it's right now | 15:03 |
| JayF | sambetts: you can't do it post-provisioning, you have to have the neutron object setup properly when you generate the configdrive | 15:03 |
| mat128 | jlvillal: I'm supposed to have a meeting downstream, but we can start | 15:03 |
| *** amotoki has quit IRC | 15:03 | |
| JayF | sambetts: b/c you have to generate the configdrive w/the physical mac | 15:03 |
| mat128 | jlvillal: room 7777? | 15:04 |
| sambetts | JayF: thats someelse I want to work on ;) | 15:04 |
| sambetts | something* | 15:04 |
| jlvillal | mat128: That works for me | 15:04 |
| JayF | sambetts: howso? Want to remove mac from configdrive? | 15:04 |
| mat128 | ok, joining | 15:04 |
| *** amit213 has quit IRC | 15:04 | |
| TheJulia | joining in two minutes | 15:04 |
| jlvillal | Let me find the dial-in. I was doing SIP from home. But that doesn't work inside the firewall at work :( | 15:04 |
| sambetts | JayF: no, I want to do late generation of config drive plus injection or something I havn't worked it all out yet | 15:05 |
| JayF | hmm. | 15:05 |
| mat128 | sambetts: injection :( | 15:05 |
| sambetts | mat128: nova already does it | 15:05 |
| JayF | yeah I'm not keen at all on Ironic modifying the configdrive created by nova | 15:05 |
| JayF | because today that is one place where we are not different from nova whatsoever | 15:05 |
| mat128 | sambetts, JayF: maybe Ironic should generate it? | 15:05 |
| mat128 | just a thought | 15:05 |
| sambetts | that was my plan | 15:05 |
| JayF | I'd be extremely -1 to that for ^ above reasons | 15:05 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Promote agent vendor passthru to core API https://review.openstack.org/330018 | 15:05 |
| mat128 | JayF: yeah.. | 15:06 |
| *** fragatina has quit IRC | 15:06 | |
| JayF | That's 100% on the "nova" side of the driver interface. | 15:06 |
| sambetts | need to sketch things out in my head first | 15:06 |
| jroll | -2 for injecting files | 15:06 |
| JayF | As someoen who runs a "mixed cloud", I don't wanna play "how is this configdrive different" | 15:06 |
| jroll | generating configdrive, maybe | 15:06 |
| * jroll oslo.configdrive | 15:06 | |
| sambetts | +100 | 15:06 |
| JayF | if it's using the same code/methods from nova, maybe | 15:07 |
| sambetts | jroll: we'll need it for standalone + glean mode anyway | 15:07 |
| jroll | it relies on lots of nova metadata, but we could figure something out | 15:07 |
| JayF | just seems like we should be less land-grabby | 15:07 |
| jroll | sambetts: yeah, I'd like non-nova users to not need to generate a configdrive | 15:07 |
| openstackgerrit | Merged openstack/ironic: Create common neutron module https://review.openstack.org/317390 | 15:07 |
| JayF | rather than eating up more and more of what nova would do :/ | 15:07 |
| jroll | to be clear, I'm not totally in support of doing configdrive ourselves | 15:07 |
| jroll | but it's an interesting idea I'd like to explore | 15:07 |
| mat128 | jroll: I prefer it to injecting stuff afterwards | 15:08 |
| *** amotoki has joined #openstack-ironic | 15:08 | |
| jroll | mat128: oh, totally | 15:08 |
| sambetts | I need to sketch it out, I also kinda want to look into a brick like thing for networking | 15:09 |
| jroll | sambetts: os-vif? :) | 15:09 |
| jroll | it's a thing already | 15:09 |
| jroll | using that in ironic is... interesting | 15:09 |
| jroll | idk enough details on it to have an opinion there | 15:09 |
| sambetts | me neither | 15:09 |
| sambetts | i need to do more digging for pizza box | 15:10 |
| jlvillal | sambetts: mat128 TheJulia : I'm over in #openstack-meeting-6 | 15:10 |
| sambetts | e.g. in my setup I can do post provisioning mac update because I don't need to configure anything in the host | 15:11 |
| mat128 | TheJulia: can you join #openstack-meeting-6 | 15:11 |
| jroll | jlvillal: we shouldn't be creating channels that start with #openstack- that aren't managed by infra :/ | 15:13 |
| jroll | if y'all are on a phone bridge and just dropping random links, feel free to do that here | 15:14 |
| jroll | or start a channel beginning with ## | 15:14 |
| jlvillal | jroll: Okay | 15:14 |
| TheJulia | ++ | 15:14 |
| *** sdake_ has joined #openstack-ironic | 15:14 | |
| mat128 | oh ok | 15:14 |
| mat128 | ironic stuff crashing on multiple private network http://paste.openstack.org/show/521692/ | 15:15 |
| sambetts | 16:11 < jlvillal> https://etherpad.openstack.org/p/ironic-newton-grenade-whiteboard | 15:15 |
| sambetts | 16:12 -!- Channel #openstack-meeting-6 created Tue May 17 23:49:57 2016 | 15:15 |
| jlvillal | TheJulia: sambetts mat128 : https://etherpad.openstack.org/p/ironic-newton-grenade-whiteboard | 15:15 |
| sambetts | 16:12 < mat128> ironic stuff crashing on multiple private network http://paste.openstack.org/show/521692/ | 15:15 |
| sambetts | 16:13 -!- Irssi: Join to #openstack-meeting-6 was synced in 95 secs | 15:15 |
| sambetts | 16:14 < sambetts> My local conf for real BM in third party CI http://paste.openstack.org/show/521216/ | 15:15 |
| jroll | ..... | 15:15 |
| sambetts | 16:14 < mat128> localconf for "master": http://paste.openstack.org/show/521693/ | 15:15 |
| sambetts | 16:14 < jlvillal> Okay, jroll wants us to drop the links over in #openstack-ironic | 15:15 |
| jroll | oh I see | 15:15 |
| jroll | heh | 15:15 |
| mat128 | localconf for "slave": http://paste.openstack.org/show/521694/ | 15:15 |
| *** baoli has quit IRC | 15:15 | |
| *** sdake has quit IRC | 15:16 | |
| *** amotoki has quit IRC | 15:18 | |
| *** lintan_ has joined #openstack-ironic | 15:18 | |
| *** hshiina has joined #openstack-ironic | 15:18 | |
| mat128 | https://review.openstack.org/#/c/332614/ | 15:19 |
| *** sdake_ has quit IRC | 15:20 | |
| *** baoli has joined #openstack-ironic | 15:23 | |
| *** baoli has quit IRC | 15:23 | |
| *** penick has quit IRC | 15:24 | |
| *** baoli has joined #openstack-ironic | 15:25 | |
| jroll | hey friends. after today, I'll be on ETO june 24-27. at openstack leadership training thing june 28-30, mostly afk. on ETO again july 1-5. if something comes up that needs my ack (release requests, etc), feel free to redirect to dtantsur, rloo, or devananda. point people at this irc log if they don't believe you. :) | 15:25 |
| mat128 | jroll: please prove you are the real jroll =) | 15:26 |
| jroll | mat128: /whois jroll :D | 15:26 |
| rloo | jroll: so when are you available? might be easier. | 15:26 |
| jroll | rloo: july 6. | 15:26 |
| thiagop | this whois thing actually works? | 15:26 |
| jroll | rloo: feel free to hit me on hangouts, though, if you *really* need me | 15:26 |
| *** ijw has quit IRC | 15:27 | |
| jroll | thiagop: yes, it's an irc command | 15:27 |
| mat128 | jroll: but it doesnt really prove anything ;) | 15:27 |
| rloo | jroll: gotcha. no worries, ironic will still be here when you get back. i think the only emergency would be some security thing. | 15:27 |
| thiagop | cool | 15:27 |
| jroll | you can see "is logged in as jroll" which doesn't mean much other than someone knows my freenode login password | 15:27 |
| mat128 | :P | 15:27 |
| mat128 | I think you proved it well enough | 15:27 |
| jroll | rloo: devananda is security liaison, but yes I'd like to know about that | 15:27 |
| * rloo is looking forward to the new, improved leadership skills of jroll in July. | 15:28 | |
| jroll | mat128: I can send you a gpg signed email if you prefer :P | 15:28 |
| rloo | but how do we know mat128 is mat128? | 15:28 |
| thiagop | openstack leadership trainning? | 15:28 |
| dtantsur | lol | 15:28 |
| thiagop | that would be cool to watch | 15:28 |
| dtantsur | jroll will become our openstack leader :D | 15:28 |
| thiagop | and/or participate | 15:28 |
| jlvillal | Want to have Ironic API get upgraded and a not have Ironic Conductor be upgraded. | 15:28 |
| rloo | thiagop: i think they opened it up to everyone, maybe with a waiting list, dunno. | 15:29 |
| thiagop | is it something promoted by the foundation? | 15:29 |
| rloo | thiagop: yup. i think initially it was for TC, but of course, people complained. | 15:29 |
| jroll | thiagop: it's something the foundation is trying out and possibly doing more of based on feedback https://etherpad.openstack.org/p/Leadershiptraining | 15:29 |
| rloo | thiagop: there was some thread about it. you have to subscribe to openstack-dev if you haven't and like drama etc. | 15:30 |
| jroll | rloo: people didn't complain, but given it's a first try, they wanted as many TC-type of people as possible to get good feedback | 15:30 |
| jroll | and they had 20 spots, and not 20 (ex-)TC people that could go | 15:30 |
| rloo | jroll: well, maybe not complain. whine about wanting to attend even if they weren't TC? :) | 15:30 |
| jlvillal | jroll: So on doing the upgrade process. Is it upgrade Conductor or API first? | 15:30 |
| rloo | jlvillal: conductor | 15:31 |
| mat128 | jlvillal, TheJulia, sambetts: https://review.openstack.org/#/c/299245/ | 15:31 |
| jroll | jlvillal: see the spec :) | 15:31 |
| jlvillal | rloo: thanks | 15:31 |
| jroll | rloo: whine, heh | 15:31 |
| mat128 | L123 | 15:31 |
| *** sdake has joined #openstack-ironic | 15:31 | |
| thiagop | lots of known names on that list. Maybe I'll subscribe on a next session. Leadership is an area I'm building right now... | 15:32 |
| rloo | jroll: so for monday's meeting, dtantsur? who? can chair. And I think we skip the following Monday (July 4)? Or we can decide on Monday whether to skip or not. | 15:33 |
| jroll | thiagop: yeah, if this goes well, sounds like they're interested in doing more | 15:33 |
| jroll | rloo: yeah, any of you three can chair | 15:33 |
| dtantsur | rloo, why skip? due to being close to midcycle? | 15:33 |
| jroll | dtantsur: july 4 is US holiday | 15:33 |
| rloo | dtantsur: July 4 is US holiday. | 15:33 |
| dtantsur | ah | 15:33 |
| jroll | jinx | 15:33 |
| rloo | heh | 15:34 |
| dtantsur | well, I can lead for non-US folks | 15:34 |
| thiagop | jroll: I'd like a debrief if you don't mind. Know what people discussed. :) | 15:34 |
| rloo | dtantsur: but if enough non-US folks, we can still have. | 15:34 |
| *** sdake_ has joined #openstack-ironic | 15:34 | |
| dtantsur | I'll check before starting; there may be enough folks | 15:34 |
| jroll | thiagop: you'll be interested in this thread, it talks a bit about that http://lists.openstack.org/pipermail/openstack-dev/2016-May/096071.html | 15:34 |
| rloo | dtantsur: I'll put it as an item for Mon meeting so we don't forget. | 15:35 |
| jroll | thiagop: I can give you some ideas | 15:35 |
| JayF | In the same vein as jroll's comment; I'm going to be moving and will not be around 6/29 - 7/12 or so while moving. | 15:35 |
| * thiagop reads | 15:35 | |
| jroll | er, thiagop I can give you my take on it, is what I meant | 15:35 |
| * rloo wonders if we shoudl shut down ironic for the next 2 weeks :) | 15:35 | |
| mat128 | rloo: updated https://review.openstack.org/#/c/332614/ based on your comments, if you don't mind giving it a look | 15:35 |
| jroll | rloo: if having JayF and I away means a full shutdown, we have big problems :P | 15:35 |
| dtantsur | rloo, jroll, actually July 4 is not a working day for me too, so I'm fine with skipping it, but I'm also fine with showing up :) | 15:35 |
| thiagop | jroll: great | 15:35 |
| rloo | mat128: ok. | 15:35 |
| rloo | jroll: HA HA. any excuse for a vacation! | 15:36 |
| jroll | lol | 15:36 |
| jroll | dtantsur: nice | 15:36 |
| *** PollyZ has quit IRC | 15:36 | |
| *** PollyZ has joined #openstack-ironic | 15:36 | |
| *** sdake has quit IRC | 15:36 | |
| rloo | jroll: we decided to review priorities when you were back, right? am going to remove from agenda. | 15:37 |
| *** ayoung has joined #openstack-ironic | 15:37 | |
| jroll | rloo: yeah, I'll try to think about it and kickstart with what I think | 15:37 |
| rloo | jroll: k, updated agenda. | 15:38 |
| jroll | thanks! | 15:38 |
| mat128 | localconf for "master": http://paste.openstack.org/show/521693/ "slave": http://paste.openstack.org/show/521694/ | 15:40 |
| *** sukhdev has joined #openstack-ironic | 15:41 | |
| *** lintan_ has quit IRC | 15:43 | |
| vdrok | jroll: sambetts rloo I'm still thinking about the flat network provider and requiring cleaning_network_uuid for the conductor to start - https://review.openstack.org/#/c/285852/57/ironic/drivers/modules/network/flat.py@38 . We need to have 'flat' in enabled_network_interfaces, which will break bifrost, so I see 2 options here: 1. remove the raise from init | 15:44 |
| vdrok | and put it into (add,remove)_cleaning_network methods, just log the warning in init; 2. do the CONF.set_defaults(enabled_network_interfaces=['none']) if CONF.dhcp.dhcp_provider == 'none' | 15:44 |
| jroll | vdrok: hrm, I don't mind 2 | 15:45 |
| jroll | the other alternative is fix bifrost to set that option before we land the code :) | 15:46 |
| cinerama | ^ TheJulia | 15:46 |
| jroll | but I think I like option 2 | 15:46 |
| *** dmk0202 has quit IRC | 15:46 | |
| vdrok | jroll: the only problem with this is that in sample config it'll say [none, flat] | 15:46 |
| jroll | vdrok: yeah, it gets weird | 15:46 |
| jroll | hmm | 15:46 |
| vdrok | tho I can just mention it in description somehow | 15:46 |
| jroll | vdrok: I think I'd rather do reno upgrade notes + fix bifrost before landing to set enabled_network_interfaces=flat | 15:48 |
| cinerama | we really should get that templatized config patch landed | 15:48 |
| TheJulia | I like #2, we could set enabled_network_interfaces to none as well in bifrost, likely should, although that will mean that we will still have a breaking config change for stand-alone users who don't use bifrost | 15:48 |
| yuriyz | vdrok maybe do not check this param in the init for flat driver and use behavior like now https://github.com/openstack/ironic/blob/master/ironic/dhcp/neutron.py#L278-L280 | 15:48 |
| mat128 | sambetts, jlvillalhttps://trello.com/c/GAlhSzLm | 15:48 |
| mat128 | jlvillal: https://trello.com/c/GAlhSzLm | 15:48 |
| vdrok | yuriyz: yeah, this is basically option 1 :) | 15:49 |
| jroll | well, we made that error because we shouldn't be loading interfaces that cannot be used | 15:49 |
| *** itamarl has quit IRC | 15:49 | |
| vdrok | OK then, I'll leave it as is for now with only reno, please comment if you disagree | 15:53 |
| *** athomas has quit IRC | 15:53 | |
| openstackgerrit | Merged openstack/ironic-inspector: devstack/plugin.sh: use screen_stop https://review.openstack.org/332408 | 15:53 |
| yuriyz | vdrok 2 is not full solution because I can use neutron provider and do not use cleaning (auto or manual) at all | 15:53 |
| vdrok | yuriyz: well, yes, but we just can not possibly know if someone will use cleaning or not, this will be in the release note for sure | 15:55 |
| rloo | vdrok: can you do 1 with addition of checking CONF.dhcp.dhcp_provider != None? | 15:56 |
| yuriyz | rloo +1 | 15:56 |
| vdrok | rloo: that would mean that the network_interface can be updated to flat afterwards without requiring cleaning_network_uuid | 15:56 |
| vdrok | or | 15:57 |
| vdrok | wait | 15:57 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-inspector-specs: Update PCI spec to use 'device_id' instead of 'product_id' https://review.openstack.org/333458 | 15:57 |
| *** spandhe has joined #openstack-ironic | 15:57 | |
| *** ijw has joined #openstack-ironic | 15:57 | |
| sambetts | https://review.openstack.org/#/c/332490/ | 15:57 |
| rloo | vdrok: so I'm not up-to-speed on the breaking change. even if we can do it so it doesn't break, do we have a deprecation plan or whatever? | 15:58 |
| * rloo in meetings for next 2ish hours. | 15:59 | |
| vdrok | rloo: for cleaning functions in out-of-tree dhcp providers, we just start to ignore them I think :( | 16:00 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: [WIP]Introducing Inspector Grenade testing https://review.openstack.org/327667 | 16:00 |
| rloo | jroll: before you take off, if you have a change to look at the serial console spec quickly, would be great. not sure what the nova deadline is for the nova part of the work. | 16:01 |
| rloo | jroll: https://review.openstack.org/#/c/319505/ | 16:01 |
| jroll | rloo: I'll try to get to it today, but can't make promises | 16:01 |
| * jroll dealing with performance review type of stuff :/ | 16:01 | |
| rloo | jroll: yup, understood. oh, performance review stuff is more important! you want me to write up something nice about you? :) | 16:02 |
| *** spandhe has quit IRC | 16:02 | |
| jroll | rloo: hehe, I will later this year actually | 16:03 |
| * dtantsur passed the performance review already :) | 16:03 | |
| rloo | jroll: ok, let me know :) | 16:03 |
| rloo | if dtantsur didn't pass, it'd make me wonder about RH... | 16:04 |
| dtantsur | hehe | 16:04 |
| *** e0ne has quit IRC | 16:05 | |
| *** milan has quit IRC | 16:08 | |
| *** sacharya has quit IRC | 16:12 | |
| *** sacharya_ has joined #openstack-ironic | 16:12 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add sanity tests for baremetal power state commands https://review.openstack.org/333361 | 16:13 |
| dtantsur | g'night | 16:14 |
| * dtantsur -> home + beers | 16:14 | |
| sambetts | night dtantsur | 16:14 |
| *** dtantsur is now known as dtantsur|afk | 16:14 | |
| sambetts | enjoy! | 16:14 |
| openstackgerrit | Merged openstack/ironic: Deployment vmedia operations to run when cleaning https://review.openstack.org/307225 | 16:16 |
| *** tesseract- has quit IRC | 16:17 | |
| *** Romanenko_K has quit IRC | 16:17 | |
| *** kromanenko_ has quit IRC | 16:18 | |
| *** kromanenko__ has quit IRC | 16:18 | |
| *** yogi has quit IRC | 16:19 | |
| vdrok | rloo: so yeah, if we have "if dhcp_provider != none and not cleaning_net: raise" in __init__, that would mean that we can set dhcp provider to none, create a node with flat network interface and the driver will be loaded fine | 16:20 |
| *** sdake_ is now known as sdake | 16:21 | |
| sambetts | vdrok, rloo: we want to migrate the standalone users off that network interface though, because it is technically a neutron interface | 16:22 |
| hkominos | quick final question for today ironicers. for a partioned image, the bootstrap_image (Kernel,inititramfs etc) are not the same as the User image kernel and initramfs | 16:22 |
| jroll | hkominos: correct | 16:23 |
| jroll | (assuming fuel doesn't change this assumption) | 16:23 |
| *** amotoki has joined #openstack-ironic | 16:24 | |
| *** ifarkas has quit IRC | 16:25 | |
| vdrok | hkominos: yes, correct | 16:26 |
| *** hshiina has quit IRC | 16:29 | |
| * thiagop wants beer too but will drive 2hs today.. | 16:30 | |
| *** cameron_C has quit IRC | 16:34 | |
| *** _vsaienko has quit IRC | 16:37 | |
| *** _vsaienko has joined #openstack-ironic | 16:40 | |
| *** jistr|mtg is now known as jistr | 16:42 | |
| *** jvgrant has quit IRC | 16:50 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add network_interface node field DB, OBJ https://review.openstack.org/317391 | 16:54 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add 'neutron' network driver https://review.openstack.org/317393 | 16:54 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 16:54 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add network_interface API https://review.openstack.org/317392 | 16:54 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add network_drivers https://review.openstack.org/285852 | 16:54 |
| *** penick has joined #openstack-ironic | 16:55 | |
| *** fragatina has joined #openstack-ironic | 16:58 | |
| jroll | devananda: JayF: y'all will be interested in this patch series https://review.openstack.org/#/q/topic:bp/policy-in-code | 17:00 |
| openstackgerrit | Polly Zhou proposed openstack/ironic: For the unit tests for drivers that test internal.parse_driver_info methods, check the actual value of the information instead of only checking if information is returned https://review.openstack.org/333495 | 17:00 |
| JayF | PollyZ: ^ you might want to read https://wiki.openstack.org/wiki/GitCommitMessages then adjust your commit message | 17:01 |
| *** joprovost has quit IRC | 17:01 | |
| JayF | https://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure being the specific section that's most relevant | 17:02 |
| *** M00nr41n has joined #openstack-ironic | 17:02 | |
| *** daemontool_ has quit IRC | 17:06 | |
| JayF | jroll: https://bugs.launchpad.net/ironic/+bug/1595625 | 17:06 |
| openstack | Launchpad bug 1595625 in Ironic "[RFE] Ability to run manual cleaning with automated clean steps" [Wishlist,New] | 17:06 |
| jroll | JayF++ | 17:07 |
| jroll | rloo might have opinions on that | 17:07 |
| JayF | rloo: ^ you might find that interesting. This is a useful downstream case we uncovered for manual cleaning, would love your input on it. tl;dr we wanna be able to "QC" our own cleaning by sending machines through manual cleaning, but without specifying the steps, and instead using the steps that would be run for automated cleaning | 17:07 |
| rloo | in meeting, will try to look later today. If not, next week. | 17:08 |
| mat128 | JayF: similar usecase downstream, but what prevents you from simply going through automatic cleaning and making the node available? | 17:08 |
| *** ijw has quit IRC | 17:09 | |
| jroll | mat128: well, if you don't want to make it available immediately | 17:09 |
| JayF | mat128: two things: 1) a desire to online entire cabinets/sets of gear atomically | 17:09 |
| *** M00nr41n has quit IRC | 17:09 | |
| JayF | mat128: i.e. if my automated cleaning says half the nodes are cabled wrong vs what the node says (we check this in cleaning), I don't want that maintenance to have to happen while the other half are available | 17:09 |
| mat128 | jroll, JayF: won't you have to go through cleaning again when trying to make them "available" ? | 17:10 |
| JayF | mat128: also for QC the actual process itself; have a box in manageable, "dirty" it, pass it through manual cleaning w/automated steps, check resulting node in managable for messyness | 17:10 |
| JayF | mat128: yep, that's not a bad thing at all imo | 17:10 |
| mat128 | JayF: OK I get it then | 17:10 |
| JayF | mat128: our cleaning is idempotent. We do ~1500 QC builds a day so we don't have any problem with cleaning nodes over and over | 17:10 |
| mat128 | JayF: remember, I'm shredding dollar bills | 17:11 |
| JayF | we shredded our dollar bills on people making better cleaning steps | 17:11 |
| JayF | rather than on new hard disks | 17:11 |
| JayF | lol | 17:11 |
| mat128 | haha | 17:11 |
| jroll | lol | 17:11 |
| mat128 | We had a one-off BIOS having the unlock option, but our provider hasnt been very good at making it a mainstream feature | 17:11 |
| jroll | :( | 17:12 |
| mat128 | Supermicro... | 17:12 |
| JayF | Yeah; we don't really talk much about the deals we make with vendors to make it more secure than it would be otherwise. | 17:12 |
| mat128 | we could work it out on our latest gen stuff, but we also have to take care of the older stuff | 17:13 |
| JayF | But I will say we have gotten HDDs with firmwares that permit ISE *without* requiring the security lockout first | 17:13 |
| JayF | so we completely punt on the problem of the security lockout on HDDs on that particular device | 17:13 |
| mat128 | JayF: interesting | 17:13 |
| mat128 | hmm | 17:13 |
| JayF | We have a whole group here doing that work though, working with vendors, etc | 17:13 |
| mat128 | You are definitely operating at a different scale, wish we had that leverage | 17:14 |
| mat128 | I mean, if we do, I don't see it or the people in charge aren't hearing our concerns | 17:14 |
| mat128 | plus all the hardware that was put in place before all those requirements | 17:15 |
| mat128 | I read somewhere that the securty lockout stuff was put in place to prevent a virus from wiping your drives | 17:17 |
| mat128 | so much for nothing, nowadays they just encrypt your data and ask for a ransom | 17:17 |
| *** ChubYann has joined #openstack-ironic | 17:17 | |
| *** devananda has quit IRC | 17:17 | |
| *** mgould is now known as mgould|afk | 17:19 | |
| PollyZ | JayF: working on it. Sorry for that | 17:20 |
| JayF | PollyZ: no problem, that's what reviews are for | 17:21 |
| *** M00nr41n has joined #openstack-ironic | 17:22 | |
| *** raildo is now known as raildo-afk | 17:22 | |
| *** sdake has quit IRC | 17:22 | |
| *** PollyZ has quit IRC | 17:25 | |
| *** trown is now known as trown|lunch | 17:25 | |
| *** ayoung has quit IRC | 17:28 | |
| *** ohamada has quit IRC | 17:29 | |
| *** PollyZ has joined #openstack-ironic | 17:32 | |
| openstackgerrit | Merged openstack/python-oneviewclient: Updated from global requirements https://review.openstack.org/332374 | 17:33 |
| *** alineb has quit IRC | 17:36 | |
| *** thiagop has quit IRC | 17:36 | |
| *** gabriel-bezerra has quit IRC | 17:37 | |
| *** thiagop has joined #openstack-ironic | 17:37 | |
| *** rcernin has quit IRC | 17:38 | |
| *** electrofelix has quit IRC | 17:38 | |
| * jlvillal just learned about https://zealdocs.org/ | 17:38 | |
| *** gabriel-bezerra has joined #openstack-ironic | 17:39 | |
| *** _vsaienko has quit IRC | 17:39 | |
| *** _vsaienko has joined #openstack-ironic | 17:42 | |
| *** sukhdev has quit IRC | 17:43 | |
| *** ijw has joined #openstack-ironic | 17:47 | |
| *** e0ne has joined #openstack-ironic | 17:54 | |
| rloo | JayF: that seems fine to me. i think from the API/CLI point of view, just add a flag to indicate use-auto-clean-steps or something like that. | 17:55 |
| vdrok | good night everyone! | 17:56 |
| rloo | night vdrok | 17:56 |
| rloo | JayF: you just want a shortcut from actually getting the list of auto clean steps and then entering them in the request to do manual cleaning. | 17:56 |
| jroll | rloo: yeah, especially given the clean steps API doesn't work if you haven't booted an agent | 17:57 |
| rloo | JayF: and, uhh, I don't think I ever finished the code/command to get the list of clean steps. | 17:57 |
| jroll | or maybe it isn't even done | 17:57 |
| jroll | yeah that :P | 17:57 |
| *** sabeen has quit IRC | 17:57 | |
| rloo | on my list but really close to the bottom. guess i got tired of the cleaning stuff... or it brings back bad memories or somethin' :) | 17:57 |
| jroll | hehe | 17:58 |
| jroll | if nobody has been asking where it's at, maybe it isn't terribly useful? | 17:58 |
| jroll | or people just aren't using that heavily yet | 17:58 |
| rloo | jroll: i suspect folks aren't using cleaning that much yet, or if they do, they know what they are doing. | 17:58 |
| jroll | ya | 17:58 |
| rloo | jroll: it's the part about getting the list from the agent that i don't like. i'll get to it in this cycle though. ... maybe... :) | 17:59 |
| JayF | I explicitly don't want it to be a client shortcut to feed it in | 17:59 |
| JayF | I'd much, much prefer Ironic to do the same actions it would for automated cleaning, just with a different target state | 17:59 |
| JayF | to make it as "pure" of an experience as possible for QE purposes | 17:59 |
| rloo | JayF: ? | 17:59 |
| jroll | I don't think rloo meant that it would be a shortcut to make the client fetch it and pass it in the request | 17:59 |
| jroll | but rather, a shortcut API so that you don't need to do that | 18:00 |
| rloo | JayF: yeah, what jroll said. | 18:00 |
| JayF | gotcha | 18:00 |
| rloo | JayF: so I don't think it needs a spec. just flush it out a bit in the bug. but that's my opinion. not sure how you were thinking of doing it. | 18:00 |
| JayF | that's what I'd prefer | 18:01 |
| *** spandhe has joined #openstack-ironic | 18:01 | |
| JayF | Question though; do you think it should be a specific argument to the existing command | 18:01 |
| rloo | JayF: that's why 'flush it out a bit' so I know you aren't going to do anything complicated/convoluted :) | 18:01 |
| JayF | or the absense of arguments means to use the "default automated" cleanign steps | 18:01 |
| rloo | JayF: as opposed to? | 18:01 |
| rloo | JayF: nope, should not assume that absence means use automated clean steps. | 18:02 |
| rloo | JayF: it is a manual clean. | 18:02 |
| JayF | That was my only general question about it, I think I can work with that | 18:02 |
| rloo | JayF: it needs to know what kind of cleaning to do. No assumptions. | 18:02 |
| JayF | I'll flesh it out a little more in the bug | 18:02 |
| rloo | JayF: ok. | 18:02 |
| JayF | thanks for the vote of confidence | 18:02 |
| JayF | I'm super excited, this is kinda obvious and solves several problems for us | 18:03 |
| rloo | yw. I can always take my vote back, but til then, enjoy it :D | 18:03 |
| JayF | :P | 18:03 |
| *** PollyZ has quit IRC | 18:03 | |
| *** PollyZ has joined #openstack-ironic | 18:03 | |
| rloo | JayF: there are some 'odd'/gotcha stuff wrt target states and auto vs manual cleaning, so not sure how easy the coding will be, but hopefully it will be easy. | 18:04 |
| mat128 | JayF: any news on getting a newer CoreOS in master? We've seen a problem with that super old ixgbe module where if the network card was initialized with a newer driver, it refuses (unless cold reboot) to load with an older module | 18:04 |
| mat128 | JayF: at that point I think I will suggest we switch to TinyIPA downstream :) | 18:04 |
| *** PollyZ has quit IRC | 18:05 | |
| *** e0ne has quit IRC | 18:07 | |
| JayF | mat128: I haven't put much focus on it; if you wanted to put some effort in that direction, the piece of info we need is what size VM would be required to get coreos working | 18:07 |
| JayF | mat128: then https://review.openstack.org/#/c/327994/ updated to actually work + have the correct ram set in a coreos experimental job | 18:08 |
| *** fragatina has quit IRC | 18:08 | |
| JayF | mat128: get that job passing on https://review.openstack.org/#/c/327891/ (without the depends-on) | 18:08 |
| JayF | mat128: then change the "real" job to be that size and get 327891 merged | 18:09 |
| *** PollyZ has joined #openstack-ironic | 18:09 | |
| JayF | mat128: there's absolutely nothing today, afaik, taht prevents using a newer coreos in a downstream image. In fact, I recently had a deploy work on latest alpha coreos as part of some testing I was doing for a bug | 18:09 |
| JayF | mat128: just the memory requirements have balooned to the level that it's difficult to get it tested | 18:09 |
| * JayF going to update the bug with ^ that info | 18:10 | |
| *** e0ne has joined #openstack-ironic | 18:11 | |
| krtaylor | jroll, I added you to the stackalytics driver completeness patch - no rush -> https://review.openstack.org/#/c/330270/ | 18:11 |
| *** sdake has joined #openstack-ironic | 18:11 | |
| jroll | krtaylor: \o/ | 18:11 |
| jroll | thanks | 18:11 |
| krtaylor | theres a few contacts to clean up, but thats about it | 18:12 |
| *** PollyZ has quit IRC | 18:12 | |
| jroll | cool | 18:13 |
| *** e0ne has quit IRC | 18:13 | |
| jroll | krtaylor: I also want to check out e.g. pxe_ipmitool and make sure they're pointed at upstream CI, etc | 18:13 |
| JayF | mat128: https://bugs.launchpad.net/ironic-python-agent/+bug/1581673 updated with all that info in a little more parsable format | 18:14 |
| openstack | Launchpad bug 1581673 in ironic-python-agent "Shipped CoreOS image is out of date" [High,In progress] | 18:14 |
| *** PollyZ has joined #openstack-ironic | 18:22 | |
| *** jvgrant has joined #openstack-ironic | 18:27 | |
| *** Sukhdev has joined #openstack-ironic | 18:27 | |
| *** M00nr41n has quit IRC | 18:29 | |
| *** trown|lunch is now known as trown | 18:37 | |
| *** PollyZ has quit IRC | 18:39 | |
| *** sabeen has joined #openstack-ironic | 18:42 | |
| mat128 | JayF: This is what we're going to do short term, I will take that patch and make it pass through our downstream tests | 18:43 |
| openstackgerrit | Merged openstack/pyghmi: Implement Python3 compatibility https://review.openstack.org/313012 | 18:43 |
| mat128 | at the same time I will be able to determine how much ram is needed | 18:43 |
| JayF | mat128: just remember we use cirros in the gate | 18:43 |
| JayF | mat128: so overall ram needed might be less there | 18:43 |
| JayF | mat128: I suspect it'll be around 1.5-2GB, but that's just an educated guess | 18:44 |
| *** ijw has quit IRC | 18:47 | |
| mat128 | JayF: thanks. I will report my findings in the bug | 18:48 |
| JayF | please do update those patches if you do | 18:48 |
| *** ijw has joined #openstack-ironic | 18:48 | |
| mat128 | will do | 18:48 |
| JayF | most of it is straightforward, I just have to multitask less hard so I get things that have to be done-done before i move | 18:48 |
| mat128 | I'll take care of it :) | 18:49 |
| mat128 | National holiday tomorrow, but next week I'll submit new revisions | 18:49 |
| *** alexpilotti has joined #openstack-ironic | 18:49 | |
| *** PollyZ has joined #openstack-ironic | 18:49 | |
| rloo | mat128: oh, happy St Jean B. | 18:50 |
| mat128 | thanks! | 18:50 |
| mat128 | didnt think anyone knew about it | 18:50 |
| rloo | mat128: crazy quebec'ers. | 18:50 |
| mat128 | haha | 18:50 |
| rloo | mat128: je viens de montreal | 18:50 |
| rloo | mat128: if that is correct. am rusty. | 18:51 |
| mat128 | rloo: ah cool :) | 18:51 |
| mat128 | rloo: that was perfect :P | 18:51 |
| rloo | mat128: :) | 18:51 |
| *** Sukhdev has quit IRC | 18:51 | |
| JayF | Happy ^ that day! | 18:52 |
| * sambetts could never place your accent and now knows why | 18:52 | |
| mat128 | sambetts: rloo's? | 18:52 |
| sambetts | yours :-P | 18:52 |
| mat128 | Ahhh | 18:52 |
| *** ccamacho is now known as ccamacho|Zzz | 18:52 | |
| rloo | sambetts: not quite French, eh? | 18:52 |
| sambetts | living ~40 miles from Franch, I'm used the french accent but it wasn't quite right :-P | 18:53 |
| sambetts | France* | 18:53 |
| sambetts | man its too late for me already... | 18:53 |
| sambetts | my brain is melted | 18:53 |
| rloo | sambetts: go away | 18:53 |
| sambetts | hahaha | 18:53 |
| sambetts | might have to go get a beer in a minute | 18:54 |
| sambetts | (probably not rloo idea of relaxing though) | 18:54 |
| *** Sukhdev has joined #openstack-ironic | 18:54 | |
| sambetts | rloo's | 18:54 |
| rloo | sambetts: Oh, it is my idea of relaxing for you :) | 18:55 |
| mat128 | jroll: should I send the midcycle summary directly to the ML? | 18:55 |
| sambetts | rloo: whats your drink of choice then if beers off the table? \ | 18:55 |
| jroll | mat128: yeah, you totally could if you think it's good :) | 18:56 |
| rloo | sambetts: I'm a tea drinker, even though I'm not British. G&T if it has to be alcohol for the amusement of my friends. | 18:56 |
| jroll | mat128: I can proofread if you want, I could add to it on the ML if you want, I could just read and enjoy | 18:56 |
| jroll | :) | 18:56 |
| sambetts | rloo: good choices hehe, dtantsur|afk is also a non-British tea drinker, although I don't think he's into the traditional black leaf + milk we have here | 18:59 |
| rloo | sambetts: i drink my tea straight (no milk or sugar) :D | 18:59 |
| rloo | sambetts: now run along | 19:00 |
| sambetts | heh yes :-P | 19:00 |
| * sambetts -> food then bed | 19:01 | |
| sambetts | Night all o/ cya in the morning | 19:01 |
| *** sambetts is now known as sambetts|afk | 19:01 | |
| *** amit213 has joined #openstack-ironic | 19:01 | |
| *** _vsaienko has quit IRC | 19:02 | |
| *** _vsaienko has joined #openstack-ironic | 19:09 | |
| mat128 | jroll: sent! enjoy | 19:09 |
| mat128 | I'm leaving until monday | 19:09 |
| mat128 | well | 19:09 |
| mat128 | holiday and everything :) | 19:09 |
| jroll | mat128: <3 | 19:10 |
| jroll | have a good one! | 19:11 |
| mat128 | you too | 19:12 |
| mat128 | alright | 19:12 |
| mat128 | stepping away for real now :) | 19:12 |
| *** mat128 is now known as mat128|away | 19:12 | |
| *** e0ne has joined #openstack-ironic | 19:21 | |
| openstackgerrit | Thiago Paiva Brito proposed openstack/ironic: [WIP] Custom power sync https://review.openstack.org/332978 | 19:22 |
| *** htruta is now known as henrique | 19:22 | |
| *** henrique is now known as Guest78419 | 19:23 | |
| *** Guest78419 is now known as htruta | 19:23 | |
| *** PollyZ has quit IRC | 19:27 | |
| *** PollyZ has joined #openstack-ironic | 19:29 | |
| *** ayoung has joined #openstack-ironic | 19:31 | |
| *** _vsaienko has quit IRC | 19:31 | |
| *** e0ne has quit IRC | 19:31 | |
| *** MattMan has quit IRC | 19:34 | |
| *** MattMan has joined #openstack-ironic | 19:34 | |
| *** _vsaienko has joined #openstack-ironic | 19:34 | |
| *** e0ne has joined #openstack-ironic | 19:34 | |
| *** PollyZ has quit IRC | 19:35 | |
| *** PollyZ has joined #openstack-ironic | 19:37 | |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Unify test playbooks https://review.openstack.org/327912 | 19:44 |
| *** e0ne has quit IRC | 19:50 | |
| *** derekh has joined #openstack-ironic | 19:50 | |
| *** e0ne has joined #openstack-ironic | 19:54 | |
| *** fragatina has joined #openstack-ironic | 19:59 | |
| *** penick has quit IRC | 20:01 | |
| *** penick has joined #openstack-ironic | 20:01 | |
| *** ayoung has quit IRC | 20:04 | |
| *** rajinir has joined #openstack-ironic | 20:08 | |
| *** jcoufal_ has quit IRC | 20:09 | |
| *** e0ne has quit IRC | 20:10 | |
| *** livelace has quit IRC | 20:11 | |
| *** fragatina has quit IRC | 20:11 | |
| *** ayoung has joined #openstack-ironic | 20:18 | |
| *** PollyZ has quit IRC | 20:25 | |
| *** e0ne has joined #openstack-ironic | 20:27 | |
| *** sabeen has quit IRC | 20:27 | |
| *** rbradfor has quit IRC | 20:27 | |
| *** cdearborn has quit IRC | 20:27 | |
| *** nicodemos has quit IRC | 20:27 | |
| *** notmorgan has quit IRC | 20:27 | |
| *** NobodyCam has quit IRC | 20:27 | |
| *** yhvh has quit IRC | 20:27 | |
| *** aignatov has quit IRC | 20:27 | |
| *** mrda has quit IRC | 20:27 | |
| *** trown has quit IRC | 20:27 | |
| *** zigo has quit IRC | 20:27 | |
| *** patrickeast has quit IRC | 20:27 | |
| *** mgagne has quit IRC | 20:27 | |
| *** morgabra has quit IRC | 20:27 | |
| *** phschwartz has quit IRC | 20:27 | |
| *** mat128|away has quit IRC | 20:27 | |
| *** _fortis has quit IRC | 20:27 | |
| *** natorious has quit IRC | 20:27 | |
| *** dutsmoc has quit IRC | 20:27 | |
| *** odyssey4me has quit IRC | 20:27 | |
| *** mat128 has joined #openstack-ironic | 20:27 | |
| *** phschwartz has joined #openstack-ironic | 20:27 | |
| *** aignatov has joined #openstack-ironic | 20:27 | |
| *** mrda has joined #openstack-ironic | 20:27 | |
| *** sabeen has joined #openstack-ironic | 20:27 | |
| *** nicodemos has joined #openstack-ironic | 20:27 | |
| *** cdearborn has joined #openstack-ironic | 20:28 | |
| *** notmorgan has joined #openstack-ironic | 20:28 | |
| *** morgabra has joined #openstack-ironic | 20:28 | |
| *** rbradfor has joined #openstack-ironic | 20:28 | |
| *** yhvh has joined #openstack-ironic | 20:28 | |
| *** odyssey4me has joined #openstack-ironic | 20:28 | |
| *** trown has joined #openstack-ironic | 20:28 | |
| *** zigo has joined #openstack-ironic | 20:29 | |
| *** natorious has joined #openstack-ironic | 20:30 | |
| *** NobodyCam has joined #openstack-ironic | 20:30 | |
| *** mgagne has joined #openstack-ironic | 20:30 | |
| *** mgagne is now known as Guest20454 | 20:30 | |
| *** patrickeast has joined #openstack-ironic | 20:31 | |
| *** fyxim has quit IRC | 20:31 | |
| *** comstud has joined #openstack-ironic | 20:32 | |
| *** fyxim has joined #openstack-ironic | 20:35 | |
| *** alineb has joined #openstack-ironic | 20:36 | |
| *** sabeen has quit IRC | 20:36 | |
| *** sabeen has joined #openstack-ironic | 20:39 | |
| *** rbudden has joined #openstack-ironic | 20:40 | |
| *** _vsaienko has quit IRC | 20:40 | |
| *** _vsaienko has joined #openstack-ironic | 20:44 | |
| *** jvgrant has quit IRC | 20:46 | |
| jroll | outta here, see y'all in a week or two | 20:46 |
| jroll | \o | 20:46 |
| *** hecliunyx has joined #openstack-ironic | 20:47 | |
| jlvillal | jroll: Good luck and have fun! | 20:48 |
| *** adu has joined #openstack-ironic | 20:50 | |
| *** ayoung has quit IRC | 20:51 | |
| *** e0ne has quit IRC | 20:51 | |
| *** ElCoyote_ has left #openstack-ironic | 20:56 | |
| *** jjohnson2_ has quit IRC | 20:58 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Replace dict.get(key) in api & conductor tests https://review.openstack.org/330830 | 21:00 |
| *** sacharya has joined #openstack-ironic | 21:01 | |
| *** alexpilotti has quit IRC | 21:02 | |
| *** penick has quit IRC | 21:03 | |
| *** sacharya_ has quit IRC | 21:05 | |
| *** foexle has joined #openstack-ironic | 21:07 | |
| *** _vsaienko has quit IRC | 21:08 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Replace dict.get(key) in api & conductor tests https://review.openstack.org/330830 | 21:09 |
| *** _vsaienko has joined #openstack-ironic | 21:10 | |
| *** trown is now known as trown|outtypewww | 21:10 | |
| *** catintheroof has quit IRC | 21:21 | |
| *** _vsaienko has quit IRC | 21:24 | |
| *** adu has quit IRC | 21:26 | |
| *** rloo has quit IRC | 21:26 | |
| *** rloo has joined #openstack-ironic | 21:27 | |
| *** alexpilotti has joined #openstack-ironic | 21:33 | |
| *** sabeen has quit IRC | 21:35 | |
| *** alexpilotti has quit IRC | 21:38 | |
| *** sdake has quit IRC | 21:38 | |
| *** sdake has joined #openstack-ironic | 21:42 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic-python-agent: Replace dict.get(key) with dict[key] in tests https://review.openstack.org/333600 | 21:43 |
| *** baoli has quit IRC | 21:57 | |
| *** ametts has quit IRC | 22:02 | |
| *** cdearborn has quit IRC | 22:03 | |
| *** e0ne has joined #openstack-ironic | 22:05 | |
| *** e0ne has quit IRC | 22:05 | |
| *** derekh has quit IRC | 22:11 | |
| *** davidlenwell has quit IRC | 22:12 | |
| *** davidlenwell has joined #openstack-ironic | 22:13 | |
| *** penick has joined #openstack-ironic | 22:25 | |
| *** rbudden has quit IRC | 22:28 | |
| *** Sukhdev has quit IRC | 22:29 | |
| *** Sukhdev has joined #openstack-ironic | 22:30 | |
| *** aNupoisc has joined #openstack-ironic | 22:32 | |
| *** Sukhdev has quit IRC | 22:35 | |
| *** Sukhdev has joined #openstack-ironic | 22:35 | |
| *** Sukhdev has quit IRC | 22:40 | |
| *** rbudden has joined #openstack-ironic | 22:44 | |
| *** thrash is now known as thrash|g0ne | 22:54 | |
| *** Sukhdev has joined #openstack-ironic | 22:54 | |
| *** fragatina has joined #openstack-ironic | 22:57 | |
| *** Sukhdev has quit IRC | 22:59 | |
| *** fragatina has quit IRC | 23:03 | |
| *** Sukhdev has joined #openstack-ironic | 23:03 | |
| *** alexpilotti has joined #openstack-ironic | 23:03 | |
| *** fragatina has joined #openstack-ironic | 23:03 | |
| *** Sukhdev has quit IRC | 23:07 | |
| *** Sukhdev has joined #openstack-ironic | 23:11 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Add Ironic specs process to the code contribution guide https://review.openstack.org/330167 | 23:16 |
| *** rcernin has joined #openstack-ironic | 23:16 | |
| *** dprince has quit IRC | 23:19 | |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Unify test playbooks https://review.openstack.org/327912 | 23:21 |
| *** foexle has quit IRC | 23:26 | |
| *** fragatina has quit IRC | 23:30 | |
| *** sdake_ has joined #openstack-ironic | 23:31 | |
| *** sdake has quit IRC | 23:35 | |
| *** adu has joined #openstack-ironic | 23:40 | |
| *** amotoki has quit IRC | 23:48 | |
| *** sdake_ has quit IRC | 23:53 | |
| *** daemontool has joined #openstack-ironic | 23:53 | |
| *** alexpilotti has quit IRC | 23:56 | |
| *** sdake has joined #openstack-ironic | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!