*** praneshp has joined #openstack-ironic | 00:07 | |
jlvillal | TheJulia, https://www.manning.com/books/irresistible-apis | 00:09 |
---|---|---|
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: [WIP]Introducing Inspector Grenade testing https://review.openstack.org/327667 | 00:16 |
*** sabeen1 has quit IRC | 00:16 | |
*** edand has joined #openstack-ironic | 00:17 | |
*** amotoki has joined #openstack-ironic | 00:17 | |
*** edand has quit IRC | 00:21 | |
mat128 | http://open-zfs.org/wiki/Feature_Flags | 00:22 |
*** amotoki has quit IRC | 00:22 | |
*** xavierrr has joined #openstack-ironic | 00:22 | |
*** sabeen has joined #openstack-ironic | 00:26 | |
*** Nisha_away has quit IRC | 00:28 | |
openstackgerrit | Clenimar Filemon proposed openstack/python-ironic-inspector-client: Switch to keystoneauth https://review.openstack.org/331893 | 00:30 |
*** rloo has quit IRC | 00:38 | |
*** yogi has joined #openstack-ironic | 00:43 | |
*** sdake_ has quit IRC | 00:44 | |
mrda | . | 00:47 |
jlvillal | FYI: We are talking in the mid-cycle call now. | 00:49 |
jlvillal | mrda, ^^^ :) | 00:49 |
jlvillal | https://etherpad.openstack.org/p/ironic-newton-midcycle | 00:49 |
*** hoangcx has joined #openstack-ironic | 00:52 | |
*** rajinir has quit IRC | 00:54 | |
*** spandhe has quit IRC | 00:58 | |
*** phuongnh has joined #openstack-ironic | 01:00 | |
jroll | jlvillal: GRE :) | 01:01 |
* jroll here now btw | 01:01 | |
jlvillal | jroll, Thanks! | 01:02 |
jroll | this seems relevant: https://github.com/openstack-infra/devstack-gate/blob/3ef48cec70c771242d743467768f22d1ecb55ecb/devstack-vm-gate.sh#L89 | 01:04 |
openstackgerrit | Merged openstack/ironic: Replace dict.get(key) in drivers unit tests https://review.openstack.org/330852 | 01:05 |
xavierrr | coding and listening to you guys :D | 01:05 |
jlvillal | xavierrr, stalker! ;) | 01:06 |
xavierrr | hahahaha | 01:06 |
xavierrr | yeap | 01:06 |
jlvillal | Making some notes on the multi-node stuff in: https://etherpad.openstack.org/p/ironic-newton-grenade-whiteboard | 01:06 |
xavierrr | to much to study before joining a meeting with you haha | 01:06 |
jlvillal | feel free to add content | 01:06 |
*** rama_y has quit IRC | 01:08 | |
jcook | clif_h should this be a bug instead of a blueprint (not sure ironic uses blueprints): https://blueprints.launchpad.net/nova/+spec/auth-based-on-image-metadata | 01:08 |
jlvillal | TheJulia, https://github.com/openstack/ironic/blob/master/devstack/upgrade/resources.sh#L35 | 01:09 |
jlvillal | TheJulia, That is for Grenade though | 01:09 |
jcook | clif_h reviewed your spec: https://review.openstack.org/#/c/326073/1 | 01:09 |
jlvillal | https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L660 | 01:10 |
jroll | jcook: it's a nova feature, so it should be a bp :P | 01:11 |
jcook | (-‸ლ) | 01:11 |
jcook | clif_h disregard, apparently my neurons have given up on me for the day and I should close my laptop lid | 01:12 |
mrda | Just to let y'all know, the midcycle has finished up for today | 01:19 |
*** amotoki has joined #openstack-ironic | 01:20 | |
TheJulia | mrda: thank you :) | 01:22 |
openstackgerrit | Ruby Loo proposed openstack/ironic: Fix test in test_agent_client.py https://review.openstack.org/331904 | 01:22 |
*** amotoki has quit IRC | 01:24 | |
*** joprovost has joined #openstack-ironic | 01:34 | |
*** amotoki has joined #openstack-ironic | 01:40 | |
*** sacharya has joined #openstack-ironic | 01:59 | |
*** sacharya has quit IRC | 01:59 | |
*** sacharya has joined #openstack-ironic | 02:00 | |
*** amotoki has quit IRC | 02:00 | |
*** zhs has joined #openstack-ironic | 02:10 | |
*** fragatina has quit IRC | 02:14 | |
*** fragatina has joined #openstack-ironic | 02:15 | |
*** amotoki has joined #openstack-ironic | 02:16 | |
*** xavierrr has quit IRC | 02:31 | |
*** zhs_s has joined #openstack-ironic | 02:34 | |
*** PollyZ has joined #openstack-ironic | 02:35 | |
*** zhs has quit IRC | 02:37 | |
*** PollyZ has quit IRC | 02:38 | |
*** PollyZ has joined #openstack-ironic | 02:40 | |
*** PollyZ has quit IRC | 02:44 | |
*** yuanying has quit IRC | 02:51 | |
*** zhs_ss has joined #openstack-ironic | 02:53 | |
*** stevemar has quit IRC | 02:54 | |
*** yuanying has joined #openstack-ironic | 02:54 | |
*** zhs_s has quit IRC | 02:56 | |
*** PollyZ has joined #openstack-ironic | 03:02 | |
*** zhs_ss has quit IRC | 03:12 | |
*** spandhe has joined #openstack-ironic | 03:15 | |
*** joprovost has quit IRC | 03:16 | |
*** rama_y has joined #openstack-ironic | 03:17 | |
*** spandhe_ has joined #openstack-ironic | 03:22 | |
*** spandhe has quit IRC | 03:23 | |
*** spandhe_ is now known as spandhe | 03:23 | |
*** jaybeale has joined #openstack-ironic | 03:23 | |
*** jaybeale has quit IRC | 03:24 | |
*** jaybeale has joined #openstack-ironic | 03:24 | |
*** jaybeale has quit IRC | 03:25 | |
*** jaybeale has joined #openstack-ironic | 03:25 | |
*** sabeen has quit IRC | 03:31 | |
*** sabeen has joined #openstack-ironic | 03:35 | |
*** rama_y has quit IRC | 03:35 | |
*** spandhe has quit IRC | 03:35 | |
*** sabeen has quit IRC | 03:36 | |
*** ijw has quit IRC | 03:54 | |
*** spandhe has joined #openstack-ironic | 04:02 | |
*** yuanying has quit IRC | 04:02 | |
*** yuanying has joined #openstack-ironic | 04:03 | |
*** PollyZ has quit IRC | 04:04 | |
*** Nisha_away has joined #openstack-ironic | 04:04 | |
*** jaybeale has quit IRC | 04:09 | |
*** jaybeale has joined #openstack-ironic | 04:10 | |
*** jaybeale has quit IRC | 04:15 | |
*** links has joined #openstack-ironic | 04:17 | |
*** yuikotakadamori has joined #openstack-ironic | 04:20 | |
*** yuikotakadamori has quit IRC | 04:31 | |
mat128 | jlvillal: multinode grenade stuff: https://review.openstack.org/#/c/331936/ for CI setup, still testing locally but if neutron plays well it wont be too hard. There is also a pending question for Tan Lin in https://review.openstack.org/#/c/299245/ concerning which cases to cover (only API, only conductor or hybrid). Neutron has mutlinode grenade setup and it doesnt seem clear which they are upgrading (havent checked further than the | 04:33 |
mat128 | jenkins jobs) | 04:33 |
mat128 | (first time I get a 2-part message.. bed time I guess) | 04:34 |
mat128 | we simply need to hook ourselves onto "central-data-br" instead of "br-int-1" (https://raw.githubusercontent.com/openstack/neutron/master/doc/source/devref/images/fullstack_multinode_simulation.png) | 04:36 |
* mat128 signs off for some sleep =) | 04:37 | |
*** fragatina has quit IRC | 04:47 | |
*** fragatina has joined #openstack-ironic | 04:48 | |
*** fragatina has quit IRC | 04:50 | |
*** yogi has quit IRC | 04:51 | |
*** fragatina has joined #openstack-ironic | 04:51 | |
*** rama_y has joined #openstack-ironic | 04:56 | |
*** moshele has joined #openstack-ironic | 04:59 | |
*** moshele has quit IRC | 05:10 | |
*** amotoki has quit IRC | 05:11 | |
*** amotoki has joined #openstack-ironic | 05:14 | |
*** ChubYann has quit IRC | 05:19 | |
*** itamarl has joined #openstack-ironic | 05:37 | |
*** spandhe_ has joined #openstack-ironic | 05:51 | |
*** spandhe has quit IRC | 05:52 | |
*** spandhe_ is now known as spandhe | 05:52 | |
*** edand has joined #openstack-ironic | 05:54 | |
*** yolanda has joined #openstack-ironic | 05:55 | |
*** rcernin has joined #openstack-ironic | 05:56 | |
*** rama_y has quit IRC | 05:57 | |
*** jtomasek has joined #openstack-ironic | 06:02 | |
*** skramaja has joined #openstack-ironic | 06:07 | |
*** moshele has joined #openstack-ironic | 06:09 | |
*** spandhe has quit IRC | 06:15 | |
*** hoangcx has quit IRC | 06:24 | |
*** praneshp has quit IRC | 06:26 | |
*** praneshp has joined #openstack-ironic | 06:28 | |
*** yolanda has quit IRC | 06:32 | |
*** lucasagomes has quit IRC | 06:44 | |
*** PixieBoots has quit IRC | 06:44 | |
*** yuikotakadamori has joined #openstack-ironic | 06:44 | |
openstackgerrit | Bharath kumar proposed openstack/ironic: Dummy patch: Do not merge https://review.openstack.org/331968 | 06:45 |
*** edand has quit IRC | 06:50 | |
openstackgerrit | Moshe Levi proposed openstack/ironic: Add missing translation hint to IronicException https://review.openstack.org/331969 | 06:50 |
*** edand has joined #openstack-ironic | 06:51 | |
*** lucasagomes has joined #openstack-ironic | 06:51 | |
*** pcaruana has joined #openstack-ironic | 06:52 | |
*** praneshp has quit IRC | 06:59 | |
*** rbartal has joined #openstack-ironic | 06:59 | |
openstackgerrit | Yuiko Takada Mori proposed openstack/ironic-specs: Nova compatible serial console support https://review.openstack.org/319505 | 07:13 |
*** sacharya has quit IRC | 07:13 | |
*** hoangcx has joined #openstack-ironic | 07:14 | |
*** chlong|rhce_trng has quit IRC | 07:27 | |
*** kromanenko__ has joined #openstack-ironic | 07:29 | |
*** kromanenko_ has joined #openstack-ironic | 07:29 | |
*** Romanenko_K has joined #openstack-ironic | 07:29 | |
pas-ha | morning Ironic | 07:34 |
*** athomas has joined #openstack-ironic | 07:35 | |
openstackgerrit | Moshe Levi proposed openstack/ironic: Replace assertRaisesRegexp with assertRaisesRegex https://review.openstack.org/331995 | 07:40 |
*** lucasagomes has quit IRC | 07:46 | |
*** lucasagomes has joined #openstack-ironic | 07:47 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-ironic | 08:03 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 08:10 |
*** e0ne has joined #openstack-ironic | 08:11 | |
*** sacharya has joined #openstack-ironic | 08:14 | |
*** snecklifter has joined #openstack-ironic | 08:17 | |
*** dmk0202 has joined #openstack-ironic | 08:18 | |
*** mgoddard has joined #openstack-ironic | 08:19 | |
*** sacharya has quit IRC | 08:19 | |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: [WIP]Introducing Inspector Grenade testing https://review.openstack.org/327667 | 08:23 |
openstackgerrit | Moshe Levi proposed openstack/ironic: Replace assertRaisesRegexp with assertRaisesRegex https://review.openstack.org/331995 | 08:27 |
lucasagomes | morning all | 08:33 |
*** vmud213 has joined #openstack-ironic | 08:35 | |
*** snecklifter has quit IRC | 08:36 | |
*** moshele has quit IRC | 08:38 | |
openstackgerrit | Aline Bousquet proposed openstack/python-ironicclient: Fix error returned by ironic --json node-validate https://review.openstack.org/303964 | 08:38 |
*** yolanda has joined #openstack-ironic | 08:39 | |
*** edand has quit IRC | 08:40 | |
openstackgerrit | Szymon Borkowski proposed openstack/ironic-python-agent: Add PCI devices collector to inspector https://review.openstack.org/326318 | 08:40 |
*** moshele has joined #openstack-ironic | 08:40 | |
*** mgould|afk is now known as mgould | 08:41 | |
mgould | morning Ironic! | 08:41 |
*** dmk0202 has quit IRC | 08:42 | |
*** derekh has joined #openstack-ironic | 08:44 | |
openstackgerrit | Saravanan KR proposed openstack/ironic-specs: Provide a mechanism to configure kernel boot parameters https://review.openstack.org/331564 | 08:45 |
milan | morning lucasagomes mgould and all :) | 08:59 |
milan | yuikotakadamori, morning :) | 08:59 |
*** karthi has joined #openstack-ironic | 09:01 | |
karthi | Hi Rolks | 09:01 |
karthi | * Folks | 09:01 |
hkominos | morning | 09:02 |
openstackgerrit | Saravanan KR proposed openstack/ironic-specs: Provide a mechanism to configure kernel boot parameters https://review.openstack.org/331564 | 09:03 |
milan | karthi, hkominos morning | 09:03 |
hkominos | another glorious day where i will be trying to deploy an ironic node! | 09:03 |
*** ohamada has joined #openstack-ironic | 09:05 | |
karthi | guys, i am trying to create a ironic node with network-interface perameter | 09:06 |
*** livelace has joined #openstack-ironic | 09:06 | |
karthi | But it is not creating the node it is saying "Requested API is not supported" | 09:07 |
karthi | Posted the output here http://paste.openstack.org/show/520899/ | 09:07 |
*** bharathk has joined #openstack-ironic | 09:15 | |
*** openstackgerrit has quit IRC | 09:18 | |
*** openstackgerrit has joined #openstack-ironic | 09:19 | |
*** moshele has quit IRC | 09:21 | |
*** amotoki has quit IRC | 09:24 | |
*** ifarkas has joined #openstack-ironic | 09:28 | |
*** itamarl has quit IRC | 09:30 | |
*** itamarl has joined #openstack-ironic | 09:30 | |
ifarkas | morning all! | 09:34 |
milan | morning ifarkas | 09:35 |
*** sacharya has joined #openstack-ironic | 09:37 | |
*** sacharya has quit IRC | 09:41 | |
*** moshele has joined #openstack-ironic | 09:44 | |
hkominos | Hello openstackers. Does anybody know with which networks must the ILO interface communicate (VLAN tagging case) | 09:53 |
openstackgerrit | Moshe Levi proposed openstack/ironic-specs: Adding InfiniBand Support to ironic https://review.openstack.org/265635 | 09:56 |
openstackgerrit | Derek Higgins proposed openstack/pyghmi: Convert data tuple to a list https://review.openstack.org/332046 | 09:58 |
sergek | morning All :) | 09:59 |
hkominos | morning sergek | 10:04 |
*** snecklifter has joined #openstack-ironic | 10:05 | |
sergek | hkominos: p/ | 10:06 |
*** yolanda has quit IRC | 10:08 | |
*** karthi has quit IRC | 10:10 | |
*** hkominos has quit IRC | 10:10 | |
*** hoangcx has quit IRC | 10:15 | |
*** amotoki has joined #openstack-ironic | 10:17 | |
*** athomas has quit IRC | 10:20 | |
*** athomas has joined #openstack-ironic | 10:21 | |
sambetts|afk | Morning all o/ | 10:25 |
*** sambetts|afk is now known as sambetts | 10:25 | |
milan | morning sambetts | 10:25 |
*** phuongnh has quit IRC | 10:30 | |
mgould | morning sambetts sergek milan ifarkas | 10:30 |
*** snecklifter has quit IRC | 10:31 | |
milan | hey mgould | 10:31 |
sergek | mgould: o/ | 10:33 |
vdrok | late morning everyone | 10:33 |
milan | morning vdrok sergek | 10:34 |
milan | how are things, Ironic? ;) | 10:34 |
vdrok | morning milan mgould and sambetts :) | 10:34 |
vdrok | milan: too much heat here in ukraine :) | 10:35 |
milan | vdrok, concrete melting point? ;) | 10:36 |
vdrok | brain melting point for sure, +30C now, +32C tomorrow | 10:37 |
milan | vdrok, you'd better take PTO and visit some water ;) Nothing like a Pina Colada on a beach | 10:38 |
vdrok | milan: heh, it will be PTO till september then :) | 10:38 |
milan | vdrok, I envy :D | 10:39 |
mgould | morning vdrok | 10:40 |
mgould | vdrok: yikes | 10:40 |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: [WIP]Introducing Inspector Grenade testing https://review.openstack.org/327667 | 10:41 |
* milan just had a second face-palm this morning ^^^ | 10:42 | |
milan | ./unstack.sh && grenade again | 10:43 |
sambetts | morning milan mgould vdrok | 10:45 |
milan | sambetts, o/ :) | 10:46 |
*** chlong|rhce_trng has joined #openstack-ironic | 10:50 | |
openstackgerrit | Merged openstack/ironic: Replace assertRaisesRegexp with assertRaisesRegex https://review.openstack.org/331995 | 10:54 |
*** vmud213 has quit IRC | 11:01 | |
*** makowals has joined #openstack-ironic | 11:03 | |
*** lucasagomes is now known as lucas-afk | 11:06 | |
*** thrash|g0ne is now known as thrash | 11:18 | |
jiangfei | livelace ping | 11:19 |
TheJulia | Good morning | 11:19 |
jiangfei | morning | 11:19 |
*** snecklifter has joined #openstack-ironic | 11:20 | |
livelace | jiangfei, pong | 11:20 |
TheJulia | vdrok: It could always be worse, NobodyCam sent me a picture of the tempature yesterday and it read 54C... | 11:20 |
vdrok | morning TheJulia, omg :( | 11:21 |
jiangfei | i have question on step 2: Reset disk controller to default HDD | 11:22 |
*** amotoki has quit IRC | 11:22 | |
jiangfei | livelace, i install windows-server 2008 on vm, but i can't boot from this image. | 11:23 |
jiangfei | livelace, may be i miss the step 2, but i don't known how ? can u help | 11:25 |
livelace | jiangfei, Maybe, I will try this very soon. Now I understand Ironic and will try work with windows too | 11:25 |
*** thrash has quit IRC | 11:25 | |
*** Nisha_away has quit IRC | 11:26 | |
jiangfei | thank you. ironic node can active ,but can't start the windows from node. | 11:27 |
jiangfei | windows Blue Screen of Death | 11:28 |
TheJulia | jiangfei: I've never done it myself, but if your using a whole disk image, it realistically should just work... oh | 11:28 |
*** thrash has joined #openstack-ironic | 11:28 | |
*** thrash has quit IRC | 11:28 | |
*** thrash has joined #openstack-ironic | 11:28 | |
TheJulia | drivers most likely? | 11:28 |
*** ohamada has quit IRC | 11:29 | |
jiangfei | my ironic node use a whole disk image in node's drvier | 11:29 |
*** ohamada has joined #openstack-ironic | 11:29 | |
livelace | jiangfei, https://www.raymond.cc/blog/move-windows-xp-hard-drive-or-change-motherboard-without-getting-blue-screen-of-death/ | 11:33 |
livelace | Like this | 11:33 |
livelace | The same procedure need to do with windows 2008 | 11:34 |
jiangfei | livelace, thank you very much. | 11:34 |
*** links has quit IRC | 11:35 | |
*** makowals has quit IRC | 11:38 | |
*** makowals has joined #openstack-ironic | 11:39 | |
*** makowals has quit IRC | 11:39 | |
*** makowals has joined #openstack-ironic | 11:40 | |
*** ccamacho is now known as ccamacho|lunch | 11:42 | |
*** links has joined #openstack-ironic | 11:47 | |
*** jvgrant has joined #openstack-ironic | 11:57 | |
rbartal | Hi all, how can I check which ipa version I got after installing ironic? is there an cli or util for it? | 12:01 |
*** dtantsur|afk is now known as dtantsur | 12:01 | |
dtantsur | Morning/afternoon Ironic | 12:01 |
sambetts | morning dtantsur | 12:02 |
rbartal | dtantsur, good morning | 12:04 |
openstackgerrit | Sam Betts proposed openstack/ironic-inspector: Add generic LLDP processing hook https://review.openstack.org/321082 | 12:06 |
*** raildo-afk is now known as raildo | 12:07 | |
*** jvgrant has quit IRC | 12:07 | |
*** dprince has joined #openstack-ironic | 12:07 | |
*** yolanda has joined #openstack-ironic | 12:08 | |
* dtantsur dives into updating the driver composition spec | 12:08 | |
*** krtaylor has quit IRC | 12:11 | |
*** krtaylor has joined #openstack-ironic | 12:15 | |
openstackgerrit | Bharath kumar proposed openstack/ironic: adding back the inspect_ports as an optional parameter https://review.openstack.org/325230 | 12:16 |
*** ifarkas has quit IRC | 12:18 | |
*** ifarkas has joined #openstack-ironic | 12:19 | |
xavierr | good morning Ironic | 12:21 |
xavierr | morning vdrok TheJulia jiangfei rbartal dtantsur | 12:21 |
vdrok | morning xavierr and dtantsur | 12:21 |
rbartal | good morning xavierr | 12:22 |
*** ccamacho|lunch is now known as ccamacho | 12:25 | |
dtantsur | morning xavierr, vdrok | 12:26 |
rbartal | dtantsur, continuing our dissociation from yesterday about create ironic resource from file (1588339) here is a file I use with tripleO http://paste.openstack.org/show/520929/ to create nodes | 12:29 |
rbartal | dtantsur, I may be missing some info here but I think the format it not that 'weird' | 12:30 |
*** jjohnson2 has joined #openstack-ironic | 12:31 | |
dtantsur | I'm sorry, I don't see point in arguing about that. This format is not compatible with Ironic's one and will not be for obvious reasons. | 12:33 |
*** links has quit IRC | 12:33 | |
rbartal | dtantsur, just tried to understand the differences | 12:36 |
*** marios_ is now known as marios | 12:37 | |
*** daemontool has joined #openstack-ironic | 12:40 | |
*** rbartal has left #openstack-ironic | 12:43 | |
*** baoli has joined #openstack-ironic | 12:43 | |
openstackgerrit | Merged openstack/pyghmi: Convert data tuple to a list https://review.openstack.org/332046 | 12:46 |
*** links has joined #openstack-ironic | 12:46 | |
*** trown|outtypewww is now known as trown | 12:47 | |
openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Add support for PyCA cryptography https://review.openstack.org/331828 | 12:47 |
*** baoli_ has joined #openstack-ironic | 12:49 | |
openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Add support for PyCA cryptography https://review.openstack.org/331828 | 12:50 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-specs: The driver composition reform https://review.openstack.org/188370 | 12:50 |
dtantsur | sambetts, lucas-afk and everyone online ^^^ | 12:50 |
dtantsur | I think it got simpler after we've removed the hardware_type field | 12:50 |
sambetts | dtantsur: \o/ simpler == better | 12:51 |
sambetts | still a 600+ line spec though heh | 12:51 |
*** snecklifter has quit IRC | 12:51 | |
openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Implement Python3 compatibility https://review.openstack.org/313012 | 12:51 |
dtantsur | sambetts, well... it could be 200 lines, but then everyone started asking a question on every line :) | 12:52 |
*** baoli has quit IRC | 12:52 | |
sambetts | ;) hahaha yup best to cover all the bases | 12:52 |
*** hkominos has joined #openstack-ironic | 12:54 | |
hkominos | hello ironicers. I am getting a very weird and serious error in my irronic deployment. I am trying to an ironic node. The node is getting a pxe message and tryies to install some OS: After a while the machine reboots. | 12:55 |
hkominos | However It never comes backonline and in fact the machine gets killed. It cannot even gets by a post message in the bios | 12:55 |
*** nico-afk is now known as nicodemos | 12:55 | |
hkominos | (second server i 'destroy' this way) | 12:55 |
*** jcoufal has joined #openstack-ironic | 12:56 | |
*** cdearborn has joined #openstack-ironic | 12:57 | |
*** edand has joined #openstack-ironic | 13:01 | |
*** snecklifter has joined #openstack-ironic | 13:02 | |
sambetts | hkominos: does the server give you an error message in the post output? | 13:04 |
hkominos | nope. | 13:04 |
hkominos | it just does not even come up. | 13:04 |
hkominos | There is a possible problem with the server overheating right now and i am also looking into that . However i Would like to know if IRonic interacts with more low level stuff and could potentially mess with the boot process | 13:05 |
hkominos | the ROMS etc | 13:05 |
sambetts | Unless you've configured custom cleaning steps that touch a machines ROMs Ironic won't touch anything like that | 13:08 |
jroll | morning everyone | 13:10 |
sambetts | morning jroll | 13:10 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add test for ironic port-list command https://review.openstack.org/329068 | 13:10 |
dtantsur | morning jroll. I've updated the driver composition spec, should be a nice addition to your first cup of coffee :D | 13:12 |
jroll | dtantsur: second! | 13:12 |
jroll | :) | 13:12 |
vdrok | morning jroll | 13:12 |
jroll | mat128: good fine re https://raw.githubusercontent.com/openstack/neutron/master/doc/source/devref/images/fullstack_multinode_simulation.png | 13:12 |
*** hkominos has quit IRC | 13:12 | |
jroll | s/fine/find/ | 13:12 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add test for ironic node-list command https://review.openstack.org/329056 | 13:14 |
*** afaranha has quit IRC | 13:15 | |
*** snecklifter has quit IRC | 13:16 | |
milan | jroll, dtantsur morning :) | 13:18 |
dtantsur | morning milan | 13:19 |
mgould | morning jroll dtantsur | 13:20 |
*** rloo has joined #openstack-ironic | 13:22 | |
*** links has quit IRC | 13:23 | |
*** joprovost has joined #openstack-ironic | 13:24 | |
*** e0ne has quit IRC | 13:27 | |
*** e0ne has joined #openstack-ironic | 13:31 | |
openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Implement Python3 compatibility https://review.openstack.org/313012 | 13:33 |
openstackgerrit | Merged openstack/python-ironic-inspector-client: Switch to keystoneauth https://review.openstack.org/331893 | 13:35 |
jroll | ^^ nice | 13:35 |
*** sdake has joined #openstack-ironic | 13:36 | |
*** derekh has quit IRC | 13:36 | |
mat128 | morning jroll dtantsur milan | 13:39 |
dtantsur | morning mgould, mat128 | 13:39 |
mat128 | jroll: yeah, found that looking at the multinode stuff for neutron | 13:40 |
milan | morning mat128 :) | 13:40 |
*** e0ne has quit IRC | 13:41 | |
*** yolanda has quit IRC | 13:43 | |
*** PollyZ has joined #openstack-ironic | 13:46 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 13:47 |
openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Add support for PyCA cryptography https://review.openstack.org/331828 | 13:47 |
*** openstackgerrit has quit IRC | 13:48 | |
jroll | dims: I can't tell you how much I appreciate you running those tests <3 | 13:48 |
dims | jroll : thanks :) | 13:48 |
*** openstackgerrit has joined #openstack-ironic | 13:48 | |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add test for ironic driver-list command https://review.openstack.org/332150 | 13:50 |
*** bharathk has quit IRC | 13:50 | |
*** e0ne has joined #openstack-ironic | 13:51 | |
*** rbartal has joined #openstack-ironic | 13:53 | |
jroll | dtantsur: +2 again, thanks | 13:53 |
jroll | bbiab, going to eat before we start (in 65 minutes) | 13:54 |
*** ametts has joined #openstack-ironic | 13:55 | |
dtantsur | \o/ | 13:56 |
*** thrash is now known as thrash|biab | 13:57 | |
*** rbartal has quit IRC | 13:59 | |
milan | dtantsur, devstack question O:-) is there any reason for not using devstack/functions-common:scree_stop here: https://github.com/openstack/ironic-inspector/blob/master/devstack/plugin.sh#L85 ? Kinda breaks my grenade stuff ;) | 13:59 |
dtantsur | milan, there may be a lot of reasons, 2 most probably are: 1. I didn't know about it; 2. it didn't exist back then :) | 13:59 |
milan | dtantsur, lol, OK thx | 14:00 |
* milan assembling a pull request | 14:00 | |
*** joprovost1 has joined #openstack-ironic | 14:00 | |
dtantsur | cool | 14:00 |
dtantsur | milan, just make sure it tears down cleanly, as it's not tested in gate | 14:01 |
milan | dtantsur, I'll see in my grenade log, best I can do I guess | 14:02 |
*** joprovost has quit IRC | 14:02 | |
*** joprovost1 is now known as joprovost | 14:02 | |
*** PollyZ_ has joined #openstack-ironic | 14:02 | |
milan | dtantsur, actually, better to make it a separate patch from my grenade stuff, correct? | 14:02 |
milan | (smaller, faster to land) | 14:03 |
dtantsur | yes | 14:03 |
* milan cool on it | 14:03 | |
dtantsur | jroll, JayF, wanna see the IPA lldp patch https://review.openstack.org/#/c/320584/ before it gets approved? | 14:03 |
*** jaybeale has joined #openstack-ironic | 14:03 | |
*** PollyZ_ has quit IRC | 14:05 | |
*** PollyZ has quit IRC | 14:05 | |
*** PollyZ has joined #openstack-ironic | 14:07 | |
rloo | hi and morning everyone, milan, dtantsur, jroll, mat128 | 14:07 |
mat128 | hi rloo | 14:07 |
milan | morning rloo :) | 14:08 |
dtantsur | morning rloo! the driver composition spec is ready for your attention :) | 14:08 |
rloo | dtantsur: wrt driver composition, you write that the node.driver field can accept dynamic drivers. | 14:08 |
rloo | dtantsur: am reading it now :) | 14:08 |
dtantsur | true | 14:08 |
rloo | dtantsur: but your defn of dynamic driver is | 14:08 |
rloo | a driver created at run time with links to interfaces generated based on information in the node record.' | 14:08 |
* milan hides behind inspector's devstack issues | 14:08 | |
rloo | dtantsur: isn't it the value for a hardware type? | 14:08 |
dtantsur | rloo, I should have said hardware types probably... | 14:08 |
* dtantsur is too tired of this text :D | 14:08 | |
dtantsur | please read "hardware types" there | 14:09 |
rloo | dtantsur: ok. wanted to be sure. i'll point it out. | 14:09 |
*** jaybeale has quit IRC | 14:09 | |
mat128 | hardware types == dynamic drivers :) | 14:09 |
rloo | dtantsur: i am sorry, i know, it is hard to rewrite again and again. thank you for doing it! | 14:09 |
dtantsur | mat128, roughly, yes. but the point rloo makes is valid nonetheless | 14:09 |
dtantsur | rloo, no problem, as soon as we finish it reasonable soon :) | 14:10 |
mat128 | dtantsur: yup | 14:10 |
NobodyCam | good morning Ironicers | 14:11 |
vdrok | morning rloo and NobodyCam | 14:12 |
rloo | hi vdrok, morning NobodyCam | 14:12 |
*** moshele has quit IRC | 14:14 | |
NobodyCam | morning vdrok and rloo :) | 14:14 |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: devstack/plugin.sh: use screen_stop https://review.openstack.org/332164 | 14:17 |
milan | morning NobodyCam :) | 14:17 |
dtantsur | morning NobodyCam | 14:18 |
NobodyCam | morning milan and dtantsur :) | 14:18 |
openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Implement Python3 compatibility https://review.openstack.org/313012 | 14:22 |
mgould | morning NobodyCam | 14:22 |
NobodyCam | mgould: morning :) | 14:22 |
*** joprovost has quit IRC | 14:23 | |
*** rajinir has joined #openstack-ironic | 14:25 | |
rajinir | thiagop: thanks, let me know | 14:26 |
*** itamarl has quit IRC | 14:29 | |
*** snecklifter has joined #openstack-ironic | 14:29 | |
*** joprovost has joined #openstack-ironic | 14:30 | |
rloo | dtantsur: wrt driver composition, line 396. 'Extend the ``driver-list`` command with ``--no-classic`` and ``--no-dynamic``' | 14:34 |
rloo | dtantsur: the REST API is 'classic' and 'dynamic' | 14:34 |
rloo | dtantsur: why the difference? | 14:34 |
dtantsur | rloo, for usability purpose. or we'll have to have --classic=0 and --dynamic=0 to follow API | 14:35 |
dtantsur | I feel like flags work a bit different from API parameters | 14:35 |
rloo | dtantsur: seems confusing. | 14:35 |
rloo | dtantsur: why not have --classic and --dynamic default to 1 | 14:35 |
dtantsur | because --classic=0 is ugly and not how people do it | 14:36 |
rloo | dtantsur: but having 'no-X' is also ugly. | 14:36 |
dtantsur | rloo, well, that's pretty standard anyway | 14:36 |
rloo | dtantsur: it is? | 14:36 |
dtantsur | imagine ./configure scripts :) they usually have tons of --no- --with- and --without | 14:37 |
dtantsur | on the other hand I can't think of any script using an explicit value for binary. can you? | 14:37 |
dtantsur | s/binary/boolean/ ofc | 14:37 |
rloo | dtantsur: i'm not concerned with scripts, i'm concerned with the CLI. | 14:37 |
dtantsur | sorry, I don't get the difference. aren't scripts CLI? | 14:38 |
dtantsur | anyway, could you please show me an example of your approach anywhere? | 14:38 |
rloo | dtantsur: i don't want to discuss what scripts are. | 14:38 |
rloo | dtantsur: i looked through our OSC, and we don't have any 'no-X' options. | 14:38 |
rloo | dtantsur: will take a look later wrt OSC ones. | 14:38 |
dtantsur | maybe, but I don't see any reasonable alternatives | 14:39 |
dtantsur | i.e. I'm not pleased with it as well, just dunno what would be better | 14:39 |
rloo | dtantsur: maybe it should be --classic-only, --dynamic-only. not sure. we can change it later. | 14:40 |
mat128 | rloo, dtantsur: spec could be open to actual wording, simply saying "will allow filtering classic and/or dynamic drivers" | 14:40 |
mat128 | err that didnt sound too well | 14:41 |
rloo | mat128: yup. this is API related, so it is better to put down something than nothing. | 14:41 |
mat128 | spec could avoid mentioning actual wording, leaving that open to discussion later | 14:41 |
dtantsur | for CLI - maybe, but not for API, I guess | 14:41 |
mat128 | correct | 14:41 |
rloo | mat128: this is dmitry, so it is fine, but i get push back when i suggest changes at the code-patch-level. | 14:41 |
rloo | mat128: I mean, dtantsur is reasonable and i think is fine if we decide to change later, but other folks aren't always so happy. | 14:41 |
mat128 | IMO, we should be open to better suggestions down the road even if it deviates from original plan | 14:42 |
dtantsur | specs can be changed, that's true :) | 14:42 |
mat128 | yeah I understood it that way | 14:42 |
mat128 | APi could return a list of drivers and a type | 14:42 |
mat128 | filtering done client side | 14:42 |
rloo | mat128: i agree with you. | 14:42 |
dtantsur | mat128, +100. but rloo is right: not everyone is so easy on changing their apporved specs | 14:42 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Added operator documentation for ironic portgroups https://review.openstack.org/228496 | 14:42 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add multitenancy-related fields to port API object https://review.openstack.org/206244 | 14:42 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Create common neutron module https://review.openstack.org/317390 | 14:42 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add network_interface node field DB, OBJ https://review.openstack.org/317391 | 14:42 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add 'neutron' network driver https://review.openstack.org/317393 | 14:42 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 14:42 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add network_interface API https://review.openstack.org/317392 | 14:42 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add network_drivers https://review.openstack.org/285852 | 14:42 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add portgroups to support LAG interfaces - API https://review.openstack.org/332177 | 14:42 |
dtantsur | boom! | 14:42 |
dtantsur | :) | 14:42 |
rloo | mat128: i mean, we do ask for changes later and they do make them, they just aren't happy to do so. | 14:42 |
*** kromanenko_ has quit IRC | 14:42 | |
*** Romanenko_K has quit IRC | 14:42 | |
dtantsur | mat128, filtering on client side is a bit exclusive towards people not using ironicclient | 14:42 |
*** kromanenko__ has quit IRC | 14:43 | |
rloo | dtantsur, mat128: also, i was thinking, when we deprecate classic, we'll have to deprecate the API stuff too. | 14:43 |
dtantsur | do we have a plan to deprecate chassis? | 14:43 |
rloo | mat128, dtantsur: yeah, I don't think we want to put the filtering burden on our users. | 14:43 |
dtantsur | I thought it's something about V2 ;) | 14:43 |
rloo | dtantsur: have we decided to deprecate chassis? ha ha. | 14:43 |
dtantsur | rloo, back to the topic: I'm fine with both the current idea and the -only postfix; I'd prefer this to be done in a follow-up if I don't receive any more critical comments | 14:44 |
dtantsur | this is because this API part is probably the least important bit of the whole spec :) | 14:45 |
rloo | dtantsur: gotcha. | 14:45 |
vdrok | huh, is gerrit alive? | 14:45 |
TheJulia | Part of a page just loaded for me | 14:46 |
vdrok | I can't get further than Loading Gerrit Code Review ... | 14:47 |
*** yolanda has joined #openstack-ironic | 14:47 | |
vdrok | oh, working now | 14:47 |
*** lucas-afk is now known as lucasagomes | 14:48 | |
* lucasagomes is not sure about deprecating API bits... IIRC we won't get rid of anything already exposed by the api | 14:48 | |
*** praneshp has joined #openstack-ironic | 14:49 | |
jroll | yeah, I don't see what we would need to deprecate in the API | 14:49 |
jroll | it just won't return any classic drivers, because they won't be loaded | 14:49 |
*** thrash|biab is now known as thrash | 14:51 | |
dtantsur | rloo, thanks for all the comments. I'll check them and try to follow-up on everything needing clean up. | 14:54 |
dtantsur | NobodyCam, TheJulia, wanna check the driver composition too? | 14:55 |
rloo | dtantsur: yw. nothing serious, i think by tomorrow, we should +A unless there are objections. | 14:55 |
NobodyCam | will do :) just getting coffee all set up | 14:55 |
TheJulia | dtantsur: already loaded in my web browser :) | 14:55 |
rloo | jroll, etc. I have a meeting, will join midcycle in 35 min if not sooner. don't have too much fun w/o me :) | 14:55 |
jroll | rloo: enjoy :) | 14:56 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 14:56 |
*** hshiina has joined #openstack-ironic | 14:57 | |
vdrok | I need to get some food, will be ~30 min late too | 14:57 |
openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Implement Python3 compatibility https://review.openstack.org/313012 | 14:58 |
openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Implement Python3 compatibility https://review.openstack.org/313012 | 14:58 |
*** yuriyz|2 has joined #openstack-ironic | 14:59 | |
*** makowals has quit IRC | 15:00 | |
jroll | sambetts: around yet? :) | 15:00 |
jlvillal | I have a conflicting meeting for the next 30 minutes :( | 15:00 |
*** rloo has quit IRC | 15:00 | |
*** athomas has quit IRC | 15:01 | |
sambetts | jroll: just finishing up a call | 15:01 |
*** skramaja has quit IRC | 15:01 | |
jroll | sambetts: cool, I'm going to have you lead the first topic | 15:01 |
*** rloo has joined #openstack-ironic | 15:01 | |
*** daemontool has quit IRC | 15:02 | |
jroll | jlvillal: tell them you're away at the ironic midcycle :) | 15:02 |
*** jaybeale has joined #openstack-ironic | 15:02 | |
* jroll is treating this as travel | 15:02 | |
mat128 | 60hz buzz? | 15:02 |
jroll | yep | 15:02 |
JayF | that's pretty awful | 15:03 |
dtantsur | ++ I usually ask people to assume I'm actually travelling | 15:03 |
*** rpioso has joined #openstack-ironic | 15:04 | |
jroll | brb | 15:04 |
cdearborn | still getting the 60 hz buzz? | 15:05 |
sambetts | nope | 15:05 |
cdearborn | great - was wondering if it was my cordless phone | 15:05 |
*** piet_ has joined #openstack-ironic | 15:06 | |
mrda | . | 15:06 |
mat128 | jroll: got a link? | 15:07 |
jroll | mat128: I'll post it when it's open | 15:07 |
mat128 | oh, ok | 15:07 |
*** edand has quit IRC | 15:07 | |
*** moshele has joined #openstack-ironic | 15:08 | |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Mask password on agent lookup according to policy https://review.openstack.org/332195 | 15:10 |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Mask password on agent lookup according to policy https://review.openstack.org/332196 | 15:10 |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Mask password on agent lookup according to policy https://review.openstack.org/332197 | 15:11 |
*** PollyZ has quit IRC | 15:11 | |
*** PollyZ has joined #openstack-ironic | 15:11 | |
*** PollyZ has quit IRC | 15:11 | |
jroll | dtantsur: ^ reno coming as follow up | 15:11 |
dtantsur | on it | 15:11 |
dtantsur | jroll, devananda, I don't quite like that we don't have 'cherry picked from' lines in the commit messages | 15:12 |
jroll | dtantsur: agree | 15:12 |
dtantsur | devananda, is it possible to quickly fix please? assuming we do land the same commit in master | 15:13 |
jroll | I'll fix it | 15:13 |
dtantsur | ack | 15:13 |
dtantsur | jroll, also +A on https://review.openstack.org/#/c/332195/ ? | 15:13 |
devananda | ah - instead of the same commit-id. yeah. | 15:13 |
jroll | in addition to the same change-id | 15:13 |
dtantsur | in addtition | 15:14 |
devananda | just edit the commit message on the last two | 15:14 |
devananda | uh, well | 15:14 |
devananda | we can't do that | 15:14 |
dtantsur | why? | 15:14 |
devananda | cherry-picked-from commit SHA is not known until it actually merges in master | 15:14 |
devananda | gerrit could merge something else | 15:14 |
jroll | that sha is what will be merged unless the patch is modified | 15:14 |
dtantsur | devananda, I don't think so. I think it's the same as you have locally | 15:14 |
devananda | no | 15:14 |
jroll | we use the commit, not the merge commit | 15:14 |
devananda | right | 15:14 |
jroll | for "cherry-picked from" | 15:14 |
dtantsur | the merge patch will be different, but gerrit does not rebase anything | 15:14 |
jroll | ^ | 15:14 |
devananda | ... | 15:15 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Mask password on agent lookup according to policy https://review.openstack.org/332196 | 15:15 |
lucasagomes | devananda, if you add "-x" to the git cherry-pick comamnd it will add that line for you | 15:15 |
devananda | lucasagomes: it's not cherry picked | 15:15 |
devananda | actually | 15:15 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Mask password on agent lookup according to policy https://review.openstack.org/332197 | 15:15 |
*** snecklifter has quit IRC | 15:15 | |
devananda | jroll: these are subtly different patches | 15:16 |
jroll | devananda: why aren't they cherry-picked? | 15:16 |
devananda | as you know, I had to make small changes to get it to merge | 15:16 |
jroll | they were when we were iterating them | 15:16 |
*** PollyZ has joined #openstack-ironic | 15:16 | |
jroll | right, cherry-picked but had conflicts | 15:16 |
dtantsur | devananda, that counts as a merge conflict | 15:16 |
devananda | right | 15:16 |
jroll | which is still cherry-picked | 15:16 |
devananda | eh. ok | 15:16 |
lucasagomes | yeah still cherry picked | 15:16 |
devananda | words :) | 15:16 |
lucasagomes | are hard :D | 15:16 |
dtantsur | ok, the goal of this rule is for people to be able to find an original commit on master :) | 15:16 |
jroll | ok, I'll +A when these pass tests and submit releases | 15:17 |
*** jjohnson2 has quit IRC | 15:17 | |
dtantsur | thanks jroll, devananda for tracking it | 15:17 |
jroll | oh wait, reno | 15:17 |
* jroll waits | 15:17 | |
devananda | reno coming ... | 15:17 |
dtantsur | separate patches? | 15:17 |
*** jjohnson2 has joined #openstack-ironic | 15:19 | |
dtantsur | devananda, I think we need reno's in the same patches FWIW | 15:21 |
jroll | I can squash once they're posted | 15:22 |
jroll | either way | 15:22 |
dtantsur | ah ok, +1 | 15:22 |
devananda | jroll, dtantsur: https://gist.github.com/devananda/a668521889cc130e573ecf3224d6290c | 15:22 |
devananda | if that looks good, I'll squash and repost now | 15:22 |
jroll | devananda: +1 | 15:23 |
devananda | I didn't want to hold up the sec patch if folks disagreed on wording in the reno ... | 15:23 |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Clear target_power_state on conductor startup https://review.openstack.org/310090 | 15:23 |
jroll | heh, yeah | 15:23 |
devananda | since we didn't think to review that ahead of time | 15:23 |
devananda | but if you both like ^ I'll just push it | 15:23 |
jroll | yeah, it's a bummer :( | 15:23 |
jroll | I'm good with it | 15:23 |
TheJulia | devananda: +1 | 15:23 |
devananda | dtantsur: ? | 15:23 |
dtantsur | devananda, it mentions "this request" before the actual request, but it's fine with me | 15:24 |
devananda | jroll: reno question - can it have the same file name on different branches? | 15:24 |
jroll | devananda: yes | 15:24 |
dtantsur | devananda, it must | 15:24 |
*** piet_ has quit IRC | 15:24 | |
devananda | k. thought so but hadn't done it before | 15:24 |
jroll | devananda: it's what makes backports work :) | 15:24 |
lucasagomes | devananda, gist text looks good | 15:25 |
devananda | jroll: ah, of course | 15:25 |
*** e0ne has quit IRC | 15:25 | |
devananda | dtantsur: ah. I can reverse those two sentences then | 15:26 |
*** e0ne has joined #openstack-ironic | 15:26 | |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Mask password on agent lookup according to policy https://review.openstack.org/332195 | 15:27 |
* dtantsur has a cat limiting his review abilities | 15:27 | |
*** Nisha_away has joined #openstack-ironic | 15:27 | |
devananda | cherry pick lines change now ... | 15:29 |
dtantsur | yep | 15:29 |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Mask password on agent lookup according to policy https://review.openstack.org/332196 | 15:30 |
TheJulia | heh, whitespace | 15:30 |
*** ohamada has quit IRC | 15:30 | |
*** ohamada has joined #openstack-ironic | 15:30 | |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Mask password on agent lookup according to policy https://review.openstack.org/332197 | 15:30 |
*** penick has quit IRC | 15:31 | |
* devananda starts to get ready for dockercon day two | 15:32 | |
* devananda really wishes he could both be at dockercon and the midcycle | 15:32 | |
jroll | devananda: everything is approved, thanks so much <3 | 15:32 |
dtantsur | jroll, thanks for approving, the f***ing cat tried to prevent me from using the mouse by scratching me >_< | 15:33 |
jroll | lol | 15:33 |
jroll | the cat is in for a bad time if she gets on my desk :) | 15:33 |
devananda | heh | 15:33 |
TheJulia | speaking of cats... mine has decided she HAS to be on my desk for the first time in months | 15:34 |
dtantsur | that's not quite my cat, that's the problem. *my* cat would have a lot of troubles | 15:34 |
dtantsur | :D | 15:34 |
jlvillal | Meeting finished and now I am lurking... :) | 15:34 |
*** e0ne has quit IRC | 15:34 | |
*** pcaruana has quit IRC | 15:35 | |
TheJulia | stable ipa vs master ironic | 15:36 |
*** lintan_ has joined #openstack-ironic | 15:36 | |
TheJulia | oh wait, now my brain is broken | 15:36 |
* dtantsur could not follow | 15:36 | |
JayF | Grenade tests stable/mitaka IPA vs master Ironic | 15:37 |
JayF | we need a job to test master IPA vs stable/mitaka Ironic | 15:37 |
JayF | to guarantee N-1 support in IPA | 15:37 |
JayF | dtantsur: ^ | 15:37 |
*** piet has joined #openstack-ironic | 15:37 | |
mat128 | JayF: added in etherpad, feel free to update | 15:38 |
*** rbudden has joined #openstack-ironic | 15:38 | |
dtantsur | JayF, or just run grenade? | 15:38 |
JayF | dtantsur: grenade doesn't upgrade IPA. | 15:38 |
*** amotoki has joined #openstack-ironic | 15:38 | |
JayF | dtantsur: the grenade job does stable/mitaka Ironic + stable/mitaka IPA, only upgrades Ironic to master, leaves IPA at stable/mitaka | 15:38 |
dtantsur | aha, true | 15:38 |
dtantsur | which is not a realistic scenario imo | 15:38 |
JayF | so the only combo missed is IPA master vs Ironic stable | 15:38 |
dtantsur | ok, got it | 15:39 |
JayF | I mean, I think it's possibly realistic too | 15:39 |
*** jjohnson2_ has joined #openstack-ironic | 15:39 | |
JayF | like I said in chat, we've been deploying Ironic + IPA almost entirely indepedendently the whole existance of OnMetal | 15:39 |
mat128 | same here | 15:39 |
mat128 | IPA always in advance | 15:39 |
JayF | I think right now I actually have some envs with newer IPA and some with older (don't ask, lol) | 15:40 |
mat128 | lol | 15:40 |
JayF | but given we don't use paritition images, we don't use a lot of code in IPA at all so it doesn't neccessarily mean that much | 15:40 |
dtantsur | I see | 15:40 |
dtantsur | yeah, ideally we should have 2 upgrade jobs: with and without upgrading IPA.. | 15:41 |
*** jjohnson2 has quit IRC | 15:42 | |
openstackgerrit | Moshe Levi proposed openstack/ironic-inspector: Replace assertRaisesRegexp with assertRaisesRegex https://review.openstack.org/332218 | 15:43 |
jjohnson2_ | lucasagomes, fyi, review.openstack.org/#/c/313012/ is looking promising | 15:44 |
*** snecklifter has joined #openstack-ironic | 15:45 | |
lintan_ | Ironic is using number version, but IPA only using name version? | 15:47 |
*** sukhdev has joined #openstack-ironic | 15:47 | |
jroll | lintan_: both use numbered versions, both use named versions | 15:48 |
jroll | lintan_: if we test named versions, everything in between those named versions should work | 15:48 |
*** amotoki has quit IRC | 15:48 | |
lintan_ | oh | 15:48 |
lintan_ | thanks | 15:48 |
jroll | :) | 15:48 |
lucasagomes | jjohnson2_, w00t ! | 15:49 |
lucasagomes | thanks for that | 15:49 |
jlvillal | jjohnson2_, Any reason to not use the 'six' library? | 15:50 |
*** snecklifter has quit IRC | 15:51 | |
jlvillal | jjohnson2_, https://pythonhosted.org/six/ | 15:51 |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add description filed to ports - DB https://review.openstack.org/324644 | 15:51 |
jroll | man, zuul is nice and backed up today :( | 15:53 |
*** sacharya has joined #openstack-ironic | 15:53 | |
rloo | jroll: good thing we're in midcycle and not using zuul that much :) | 15:54 |
jroll | rloo: except our patches for a CVE :( | 15:54 |
rloo | jroll: OH. sigh. | 15:54 |
zigo | Is proliantutils 2.1.8 compatible with Py3 these days? | 15:54 |
jroll | rloo: in case you missed it, https://review.openstack.org/#/q/topic:fix-1572796 | 15:54 |
*** dmk0202 has joined #openstack-ironic | 15:54 | |
zigo | I'd like to make a new package with py3 support if it does. | 15:54 |
JayF | zigo: that's a vendor library, I'm unsure :/ | 15:54 |
zigo | Cheers. | 15:55 |
jroll | zigo: /me looking | 15:55 |
zigo | There's no metadata for supported py version. | 15:55 |
jroll | zigo: they run unit tests on 3.4 https://review.openstack.org/#/c/326319/ | 15:55 |
zigo | Ah, thanks. | 15:55 |
jroll | zigo: that's as close as I can get you, otherwise bug Nisha_away or maybe NobodyCam / TheJulia could find out easily | 15:56 |
zigo | ok | 15:56 |
zigo | Thanks a lot. | 15:56 |
jroll | yeah, np | 15:56 |
Nisha_away | jroll, whats this about | 15:57 |
jroll | Nisha_away: "Is proliantutils 2.1.8 compatible with Py3 these days?" | 15:57 |
Nisha_away | yes | 15:57 |
jroll | zigo: affirmative :): | 15:57 |
Nisha_away | its py3 compatible | 15:57 |
jroll | :) rather | 15:57 |
JayF | Nisha_away: it sounds like you're missing the supported python version metadata though, according to zigo. May want to add a patch to do that. | 15:57 |
Nisha_away | we run py3 gate for proliantutils | 15:57 |
Nisha_away | JayF, zigo please add on some more details... | 15:58 |
zigo | Nisha_away: Any plan on testing for py3.5 as well? | 15:58 |
zigo | Debian Sid and Xenial are both using Py3.5 | 15:58 |
Nisha_away | we can add that | 15:59 |
zigo | That's be nice. | 15:59 |
Nisha_away | as of now it runs gate for py3.4 | 15:59 |
rloo | taking break for 15 minutes | 15:59 |
Nisha_away | zigo, could you add enhancement bug for proliantutils | 15:59 |
*** bharath has joined #openstack-ironic | 15:59 | |
Nisha_away | we will take it fwd | 15:59 |
*** Nisha_away is now known as Nisha1 | 16:00 | |
*** yolanda has quit IRC | 16:04 | |
*** snecklifter has joined #openstack-ironic | 16:04 | |
-openstackstatus- NOTICE: Gerrit is being restarted now to apply an emergency security-related configuration change | 16:05 | |
*** piet has quit IRC | 16:09 | |
jroll | have to run out for 20 minutes suddenly, maybe less, dtantsur can you intro/moderate the raid discussion? | 16:11 |
dtantsur | jroll, will do | 16:12 |
*** jvgrant has joined #openstack-ironic | 16:12 | |
*** praneshp has quit IRC | 16:14 | |
rloo | starting now, line 150: https://etherpad.openstack.org/p/ironic-newton-midcycle | 16:16 |
*** trown is now known as trown|lunch | 16:17 | |
*** jaybeale has quit IRC | 16:17 | |
*** jaybeale has joined #openstack-ironic | 16:18 | |
*** piet has joined #openstack-ironic | 16:18 | |
*** lintan_ has quit IRC | 16:19 | |
*** Sukhdev_ has joined #openstack-ironic | 16:20 | |
*** Sukhdev_ has quit IRC | 16:21 | |
*** Sukhdev_ has joined #openstack-ironic | 16:22 | |
*** dmk0202 has quit IRC | 16:22 | |
*** lintan_ has joined #openstack-ironic | 16:24 | |
lintan_ | how many drivers support raid config? I think only ilo support right? | 16:26 |
cdearborn | drac supports as part of vendor passthru | 16:27 |
*** edand has joined #openstack-ironic | 16:27 | |
ifarkas | drac is in-progress and I think ucs is also in-progress | 16:27 |
lintan_ | oh, thanks :) | 16:28 |
TheJulia | lintan_: the hpe hardware support for raid through manual cleaning is essentially driver independent | 16:28 |
TheJulia | lintan_: works via ipmi and ilo | 16:28 |
jlvillal | I also don't like calling it cleaning. But will remain quiet on audio :) | 16:28 |
TheJulia | yeah, this is not cleaning, it is deployment preparation | 16:29 |
lintan_ | cool, TheJulia | 16:29 |
*** amotoki has joined #openstack-ironic | 16:30 | |
*** links has joined #openstack-ironic | 16:30 | |
TheJulia | lintan_: it is done via a hardware manager | 16:31 |
*** e0ne has joined #openstack-ironic | 16:32 | |
cdearborn | the bios and raid config that need to be applied depends on what role you want the node to take: controller, compute, storage | 16:33 |
cdearborn | and you only know that just prior to deployment | 16:33 |
cdearborn | not when a node is being deleted | 16:33 |
openstackgerrit | Moshe Levi proposed openstack/ironic-inspector: Replace assertRaisesRegexp with assertRaisesRegex https://review.openstack.org/332218 | 16:33 |
*** mgould is now known as mgould|afk | 16:34 | |
TheJulia | cdearborn: regardless of role, if I want to deploy joe's lab machine that is not known by anything else, but joe needs raid 50. | 16:36 |
TheJulia | "joe" being a random name | 16:36 |
jlvillal | TheJulia, Joe's last name is Sixpack ;) | 16:37 |
TheJulia | lol | 16:37 |
TheJulia | I'm not sure Joe could drink a Sixpack unless joe was waiting for the raid array to complete initial sync/zero out ;) | 16:38 |
*** david-lyle has quit IRC | 16:38 | |
jlvillal | heh | 16:38 |
mat128 | TheJulia: haha | 16:38 |
*** baoli_ has quit IRC | 16:38 | |
*** david-lyle has joined #openstack-ironic | 16:38 | |
TheJulia | At least, conceptually that is based upon watching the ilo hardware manager setup raid config which was pretty much a momentary thing once IPA heartbeated | 16:41 |
jroll | sorry, I'm back now :| | 16:44 |
jroll | more like 40 minutes | 16:44 |
rloo | jroll: yup, we were timing you! | 16:45 |
rloo | jroll: we're talking now about how to get raid info from user -> nova -> ironic | 16:46 |
rloo | jroll: line 158 of etherpad | 16:46 |
jroll | yep | 16:46 |
rloo | jroll: we've all agreed that we want this feature :) | 16:46 |
jroll | shocking! | 16:47 |
* jroll lets dog out quickly | 16:47 | |
*** piet has quit IRC | 16:47 | |
rloo | jroll: i just couldn't remember why people said 'no' before. i think it was only cuz of the amount of time it would take to deploy an instance. | 16:47 |
jroll | rloo: that was a primary argument | 16:48 |
rloo | jroll: do you recall the other arguments? I mean, assuming user is willing to wait... | 16:48 |
*** fragatina has quit IRC | 16:48 | |
jroll | rloo: dunno, probably nova things | 16:48 |
rloo | jroll: ok, then we're good. "just" need to deal with nova. ha ha. | 16:48 |
jroll | :) | 16:49 |
jjohnson2_ | jlvillal, basically, the things I was dealing with didn't seem enough to pull in another dependency (every additional dependency causes some non technical work in my area) | 16:50 |
jjohnson2_ | jlvillal, (on why not do six, I was at lunch so delayed response) | 16:50 |
JayF | I remember it being punted on because 1) we wanted the simpler case first (pre-raid-config) + 2) avoiding the big mess of dynamic JIT hardware config | 16:50 |
jlvillal | jjohnson2_, Okay. I'm just used to using six as that is what we do in all of our projects. | 16:50 |
*** sacharya has quit IRC | 16:51 | |
* devananda is sort of here | 16:51 | |
jjohnson2_ | jlvillal, also in a weird place on other fronts (e.g. we still have customers demanding python 2.6 support for this library) | 16:51 |
*** praneshp has joined #openstack-ironic | 16:52 | |
rloo | devananda: take a look at etherpad, from line 170 I guess https://etherpad.openstack.org/p/ironic-newton-midcycle | 16:52 |
*** catintheroof has joined #openstack-ironic | 16:53 | |
*** piet has joined #openstack-ironic | 16:53 | |
jlvillal | jjohnson2_, I think the six library works with Python 2.5 and newer. Not 100% sure on that. | 16:53 |
jlvillal | But I see the docs mention Python 2.5 | 16:53 |
TheJulia | I think they would always need to be operator defined | 16:53 |
dtantsur | jroll, the liberty patch failed due to horizon problem.. asking qa folks to fast-track it | 16:53 |
jjohnson2_ | jlvillal, yeah, not directly related, just another example of things seemingly not the case in openstack in general | 16:54 |
*** yuriyz|2 has quit IRC | 16:54 | |
jlvillal | jjohnson2_, Understood :) | 16:54 |
openstackgerrit | Moshe Levi proposed openstack/ironic-python-agent: tox: Update flake8 to ignore imagebuild folder https://review.openstack.org/332247 | 16:54 |
jjohnson2_ | jlvillal, for example, it passed my tests on 3.4 and 2.7, but failed 2.6, so I had to do more.... hurray | 16:54 |
TheJulia | sambetts: exactly what you were saying why I'm saying it would always need to be operator defined | 16:55 |
jjohnson2_ | the move to cryptography from pycrypto is more tricky than the python 3 compatibility, simplest answer means I keep supporting pycrypto for 2.6 | 16:56 |
jlvillal | jjohnson2_, :( | 16:56 |
openstackgerrit | Moshe Levi proposed openstack/ironic-python-agent: Replace assertRaisesRegexp with assertRaisesRegex https://review.openstack.org/332250 | 16:57 |
*** amotoki has quit IRC | 16:57 | |
jjohnson2_ | but when dealing with 1,500 systems, the encryption itself accounts for about 17% of the CPU time, so going to cryptography might be a nice boost | 16:57 |
jjohnson2_ | I love it when we have a freshly deployed pre-production system, some leeway to measure and optimize | 16:58 |
openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Implement Python3 compatibility https://review.openstack.org/313012 | 17:00 |
dtantsur | lucasagomes, jroll, the horizon fix was approved for liberty | 17:02 |
lucasagomes | w00t! | 17:02 |
*** e0ne has quit IRC | 17:02 | |
jroll | dtantsur: horizon fix? | 17:02 |
dtantsur | jroll, "can't bind port" | 17:03 |
jroll | dtantsur: oh, turns out it was a zuul problem, not horizon :) | 17:03 |
dtantsur | jroll, aka https://review.openstack.org/#/c/330846/ | 17:03 |
*** penick has joined #openstack-ironic | 17:03 | |
dtantsur | well, ok :) I'm still referring to it like that | 17:03 |
jroll | dtantsur: they had also switched zuul's port so it was fixed ahead of time, but glad that landed | 17:03 |
JayF | dtantsur: mat128: http://specs.openstack.org/openstack/ironic-specs/specs/backlog/exposing-hardware-capabilities.html | 17:04 |
lintan_ | One more question, do we need for raid API anymore? | 17:05 |
* JayF brb | 17:05 | |
jroll | lintan_: I think so, that allows operators to configure a complex raid ahead of time | 17:05 |
lintan_ | but the raid config is passing from nova to instance_info in current solution | 17:07 |
lintan_ | this is kind of duplicate | 17:07 |
jroll | kind of, yes | 17:07 |
*** dprince has quit IRC | 17:07 | |
rloo | lintan_: yes, for RAID to do predeployment | 17:08 |
rloo | we aren't getting rid of configuring RAID in manual cleaning, right? | 17:08 |
jroll | right | 17:08 |
rloo | some people might still want to do RAID before making them available. | 17:08 |
lucasagomes | rloo, right, I think we can not | 17:08 |
lintan_ | but node already have raid field | 17:10 |
rloo | lintan_: yes, true. i would like whoever is putting together the spec, to take these in consideration. | 17:11 |
rloo | i don't think we can write the spec and get it right, in 60 min :) | 17:11 |
rloo | break for 17 min. | 17:13 |
sambetts | mat128: I'm still a little confused by the nova metadata thing you were talking about in your comment, is there a nova boot parameter I'm missing that does that or is it a custom thing you have downstream? | 17:15 |
mat128 | nova boot --metadata key=value | 17:15 |
mat128 | we just read a magic key | 17:16 |
mat128 | in theory | 17:16 |
dtantsur | jroll, did they change the zuul port? cause our cve just failed with the same failure to restart horizon... | 17:16 |
mat128 | metadata is something for the tenant to use on their own | 17:16 |
dtantsur | s/horizon/apache/ ofc | 17:16 |
JayF | dtantsur: what branch? | 17:16 |
mat128 | but other than a scheduler hint, we didnt see any other way of feeding data to nova | 17:16 |
dtantsur | JayF, liberty | 17:16 |
JayF | dtantsur: I think our fix only very recently landed there, if it has at all | 17:16 |
JayF | dtantsur: it's possible the old value is cached, can you link the logs/ | 17:17 |
JayF | or the patch | 17:17 |
dtantsur | JayF, our was just approved, but jroll says they changed it on zuul side.. | 17:17 |
mat128 | sambetts: not sure if this is clear enough :( | 17:17 |
mat128 | I'll check if the doc is public yet | 17:17 |
dtantsur | ok, taking a break for real anyway | 17:17 |
JayF | the zuul side change, I was told, will take week+ to take effect | 17:17 |
JayF | that's why we changed it on our side too | 17:17 |
*** piet has quit IRC | 17:17 | |
dtantsur | ah, got it | 17:17 |
*** jvgrant has quit IRC | 17:17 | |
JayF | so until like, friday, just assume infra is still breaking us unless that change is in place | 17:17 |
sambetts | mat128: ah ok I see it now, http://docs.openstack.org/cli-reference/nova.html the nova boot cli ref lists it as --meta key=v | 17:17 |
sambetts | mat128: have you been following the spec I have up for the networking stuff? | 17:18 |
jroll | dtantsur: wat O_O | 17:18 |
jroll | oh right | 17:18 |
jroll | whomp whomp | 17:18 |
mat128 | sambetts: the neutron integration stuff? | 17:18 |
sambetts | mat128: yeah, the spec for VLAN Aware BMs | 17:19 |
mat128 | oh, attach/detach?% | 17:19 |
sambetts | and that | 17:19 |
mat128 | yeah | 17:19 |
jroll | man grenade is loooooong | 17:19 |
mat128 | following it all | 17:19 |
mat128 | but we already do all of this | 17:19 |
sambetts | mat128: cool, hopefully it'll help you carry less code downstream :) | 17:19 |
mat128 | sambetts: yeah :P | 17:19 |
mat128 | and less ugly | 17:20 |
mat128 | right now attach/detach I think is driver_passthru | 17:20 |
sambetts | mat128: I did exactly the same http://git.openstack.org/cgit/openstack/cisco-ironic-contrib/tree/cisco_ironic_contrib/nova/driver.py#n33 | 17:20 |
sambetts | mat128: I created an add_vnic and delete_vnic vendor passthru | 17:21 |
*** PollyZ has quit IRC | 17:21 | |
mat128 | ours a bit longer than this | 17:22 |
mat128 | hmm | 17:22 |
mat128 | ah I see | 17:22 |
mat128 | we have len(vif) != len(pif) | 17:22 |
mat128 | I wanted to ask yesterday | 17:22 |
mat128 | deriving lacp from vif > pif doesnt sound like a good idea | 17:22 |
mat128 | I am offering my customers bonding even if they only have 1 vif | 17:22 |
sambetts | mat128: that is also supported | 17:23 |
sambetts | we don't derive LACP from number of vifs we only detrive trunks | 17:23 |
mat128 | ah | 17:23 |
mat128 | interesting | 17:23 |
*** electrofelix has quit IRC | 17:23 | |
mat128 | but pif/2 when you're doing LACP? | 17:23 |
sambetts | bonds are all defined in portgroups on the ironic side | 17:23 |
mat128 | oic | 17:24 |
mat128 | so they are "one port" | 17:24 |
sambetts | yup | 17:24 |
*** sdake_ has joined #openstack-ironic | 17:24 | |
mat128 | interesting | 17:24 |
sambetts | so if you setup all your pifs into portgroups we'll decide if we need to configure trunks based on vif > len(portgroups) | 17:25 |
*** PollyZ has joined #openstack-ironic | 17:25 | |
mat128 | yeah, got it | 17:25 |
*** rbradfor has quit IRC | 17:25 | |
jroll | sambetts: what happens when you want to add a vif later, everything gets reshuffled or? | 17:26 |
mat128 | oh man | 17:26 |
sambetts | jroll: currently round robin I guess | 17:26 |
mat128 | jroll: good point | 17:26 |
jroll | sambetts: I mean | 17:26 |
jroll | if it isn't trunked | 17:26 |
mat128 | attaching makes it a trunk | 17:26 |
jroll | and then you add a vif, and it suddenly needs to be trunked | 17:26 |
mat128 | you lose connectivity | 17:27 |
*** sdake has quit IRC | 17:27 | |
sambetts | we'd need an agent to configure the networking inside the instance | 17:27 |
mat128 | we do trunking during all active-backup and lacp scenarios | 17:27 |
jroll | besides the configuration | 17:27 |
mat128 | sambetts: ah | 17:27 |
jroll | what do we do control plane side, shuffle things around so that it becomes a trunk? | 17:27 |
sambetts | I guess we'd have too, although I like the idea of making all ports a trunk, and just setting the native to the first vif | 17:28 |
*** trown|lunch is now known as trown | 17:28 | |
mat128 | sambetts: what happens when you detach that vif? | 17:28 |
mat128 | in all trunk scenarios we have no native | 17:28 |
*** moshele has quit IRC | 17:28 | |
sambetts | mat128: native becomes free, and then the next vif to attach becomes native | 17:28 |
jroll | sambetts: yeah, I like the everything is trunk model too | 17:29 |
*** sacharya has joined #openstack-ironic | 17:29 | |
mat128 | interesting | 17:29 |
sambetts | its down to the neutron drivers to handle all of this though | 17:29 |
*** ohamada has quit IRC | 17:29 | |
sambetts | its down to us to make sure the instance is configured right | 17:29 |
mat128 | sambetts: thats hard, if neutron decides but nova has to generate the right thing | 17:30 |
*** PollyZ has quit IRC | 17:30 | |
openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Refactored to_oneview_dict and init methods https://review.openstack.org/331673 | 17:30 |
sambetts | mat128: thats why we don't support on the fly right now because we can't change things in the instance right now | 17:30 |
sambetts | mat128: we'd need an agent in the instance that ironic can talk to to change the networking | 17:31 |
mat128 | sambetts: we do support it, but the customer has to change their own config | 17:31 |
mat128 | sambetts: we're agent-less | 17:31 |
sambetts | mat128: kinda, right now nova attach-port is a no-op | 17:31 |
rloo | midcycle starting | 17:31 |
sambetts | mat128: needs digging into :D | 17:31 |
rloo | line 221 https://etherpad.openstack.org/p/ironic-newton-midcycle | 17:31 |
rloo | software RAID support in IPA | 17:32 |
*** dprince has joined #openstack-ironic | 17:32 | |
JayF | dtantsur: we use the crazy hybrid stuff (Intel RSTe raid)... main use case is you can setup raid using linux/md tooling that works on windows | 17:32 |
jroll | mat128: yeah, that's how we do volume-attach right now, put iscsi details in metadata and they need to configure it | 17:32 |
*** ayoung has quit IRC | 17:33 | |
mat128 | jroll: totally acceptable for an MVP / desire to run agent-less | 17:33 |
jroll | mat128: yeah, it's a support nightmare though :( | 17:33 |
mat128 | oh | 17:34 |
sambetts | mat128: the difficulty with networking though is that a tenant might not be able to reach the node to configure it | 17:34 |
*** baoli has joined #openstack-ironic | 17:34 | |
sambetts | mat128: unless OOB serial console | 17:34 |
*** Sukhdev_ has quit IRC | 17:34 | |
mat128 | sambetts: if you lock yourself out: https://review.openstack.org/#/c/306074/ | 17:34 |
mat128 | ;)_ | 17:34 |
jroll | mat128: a big part being, if you delete or shut down an instance while the volume is attached, you end up with a half open socket in cinder and it's just gross | 17:34 |
jlvillal | mat128, Regarding: https://review.openstack.org/#/c/331936/ I left a comment. | 17:38 |
mat128 | jlvillal: I agree :) | 17:38 |
openstackgerrit | Merged openstack/ironic-inspector: devstack/plugin.sh: use screen_stop https://review.openstack.org/332164 | 17:38 |
openstackgerrit | Aparna proposed openstack/proliantutils: Sort values of raid_config for 'share_physical_disks' https://review.openstack.org/326319 | 17:39 |
*** pcaruana has joined #openstack-ironic | 17:40 | |
*** sacharya has quit IRC | 17:41 | |
JayF | https://gist.github.com/jayofdoom/247f50bdff458945fb26b1f857aec7cb | 17:41 |
mrda | thanks JayF | 17:42 |
rloo | JayF: if you wouldn't mind, would you please lower your volume? | 17:42 |
JayF | my outgoing volume? | 17:42 |
jroll | rloo: have you met jay? :) | 17:42 |
rloo | JayF: yup | 17:42 |
JayF | if I have a slider for it I will | 17:43 |
rloo | jroll: YES, so he is EXTRA LOUD! :) | 17:43 |
openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Refactor of models class to_oneview_dict and init methods https://review.openstack.org/331673 | 17:43 |
JayF | personal volume controls have been broken snice a young age :P | 17:43 |
jroll | JayF: yes, that's better | 17:43 |
*** rcernin has quit IRC | 17:43 | |
rloo | JayF: much better thx! | 17:43 |
jroll | ty | 17:43 |
rloo | now JayF seems almost normal ;) | 17:43 |
JayF | you sure you've actaully met me? :P | 17:44 |
mrda | I thought the volume control was binary for JayF? :) | 17:44 |
*** piet has joined #openstack-ironic | 17:45 | |
*** jaybeale has quit IRC | 17:45 | |
JayF | mrda: luckily for you I'm transitioning through technology that can slow it down :) | 17:45 |
JayF | s/slow/quiet/ | 17:46 |
openstackgerrit | Merged openstack/ironic: Mask password on agent lookup according to policy https://review.openstack.org/332196 | 17:48 |
*** livelace has quit IRC | 17:49 | |
*** PollyZ has joined #openstack-ironic | 17:52 | |
JayF | in windows it's usually called 'dynamic disks' not raid | 17:55 |
JayF | and I don't think you can boot off of them at all | 17:55 |
mat128 | iirc you could | 17:55 |
*** spandhe has joined #openstack-ironic | 17:55 | |
mat128 | hmm | 17:55 |
JayF | My windows knowledge is covered in dust | 17:55 |
mat128 | but I think it left a tiny boot part in the front | 17:55 |
mat128 | :P | 17:55 |
JayF | so if that's newerish I'm not sure | 17:55 |
sambetts | Spanned volume? http://www.howtogeek.com/howto/36504/how-to-create-a-software-raid-array-in-windows-7/ :/ | 17:55 |
mat128 | you had to configure it after booting off the first sidk | 17:55 |
dtantsur | for UEFI we'll also need "a tiny boot part" fwiw | 17:56 |
JayF | sambetts: most consumer windows now don't support 'dynamic disks' which are the 'real' server raid support for windows | 17:56 |
mat128 | what I remember about dynamic disks, if you convert, you can't go back :D | 17:56 |
vdrok | mat128: this happened to me too :) | 17:57 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/bifrost: Updated from global requirements https://review.openstack.org/332275 | 17:57 |
*** hshiina has quit IRC | 17:57 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/332295 | 17:59 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/332296 | 17:59 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/332297 | 17:59 |
*** ijw has joined #openstack-ironic | 18:00 | |
*** Nisha1 has quit IRC | 18:01 | |
JayF | FWIW you can trigger evaluate_hardware_support() off items in node as well | 18:02 |
JayF | so you could have a tag that says "always_deploy_as_software" in the node object somewhere, find it in hardware support and react accordingly | 18:03 |
dtantsur | https://github.com/openstack/ironic/blob/69d4fd93533ed473bbbfb55a6d0433d3718b6474/ironic/drivers/modules/agent.py#L561 | 18:03 |
JayF | despite the name evaluate_hardware_support does not have to strictly make decisions based on hardware | 18:03 |
mat128 | http://docs.openstack.org/developer/ironic/deploy/raid.html | 18:04 |
mat128 | JayF: I like this idea, putting it in the node | 18:04 |
mat128 | not sure how it fits with driver composition tho | 18:04 |
JayF | yeah, I mean, you can do all sorts of custom logic in hardware managers because the node gets passed in | 18:05 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironic-inspector-client: Updated from global requirements https://review.openstack.org/332367 | 18:05 |
*** ayoung has joined #openstack-ironic | 18:05 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/332368 | 18:05 |
JayF | you could probably implement a RaidInterface that just calls out to hardware managers :) | 18:05 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-oneviewclient: Updated from global requirements https://review.openstack.org/332374 | 18:05 |
sambetts | I would see it as a operator choice e.g. node.raid_interface = AgentRAID or CIMCRAID or ILORAID | 18:06 |
mat128 | but AgentRAID is "inband raid" more than agent | 18:06 |
mat128 | CIMC is oob, right? | 18:06 |
jroll | AgentRAID exists today, it calls the agent | 18:06 |
*** edand has quit IRC | 18:06 | |
jroll | but upstream agent doesn't have a clean step to do this | 18:06 |
JayF | https://github.com/jayofdoom/ipa-example-hardware-managers | 18:06 |
mat128 | ooh | 18:07 |
mat128 | if we were to land mdraid, LSI and Adaptec support, where would they fit? | 18:07 |
* dtantsur has no clue about these | 18:08 | |
JayF | all could go upstream | 18:08 |
mat128 | yes, but where | 18:08 |
mat128 | in the same hw manager? | 18:08 |
JayF | no, new managers for each | 18:08 |
mat128 | and "determine support" would just bail out | 18:08 |
mat128 | hmm | 18:08 |
mat128 | and if I have all of these in my pool | 18:09 |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: devstack/plugin.sh: use screen_stop https://review.openstack.org/332408 | 18:09 |
mat128 | I write a manager that uses all of them? | 18:09 |
JayF | no | 18:09 |
JayF | the managers automatically load/unload based on available hardware | 18:09 |
mat128 | ah | 18:09 |
mat128 | problem solved then | 18:09 |
JayF | if you haven't read the example managers I linked, it answers all of this | 18:09 |
mat128 | mdraid module has lower priority | 18:09 |
JayF | the example_device.py being the relevant example here | 18:09 |
JayF | exactly | 18:09 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP: For as_dict() functions return a copy.deepcopy() https://review.openstack.org/332410 | 18:11 |
*** Nisha has joined #openstack-ironic | 18:12 | |
JayF | #agreed raid is hard | 18:13 |
JayF | lol | 18:13 |
*** Nisha_away has joined #openstack-ironic | 18:14 | |
mat128 | example 1 http://docs.openstack.org/developer/ironic/deploy/raid.html | 18:15 |
*** Nisha has quit IRC | 18:17 | |
*** Nisha_away has quit IRC | 18:19 | |
*** e0ne has joined #openstack-ironic | 18:19 | |
*** pcaruana has quit IRC | 18:22 | |
rloo | https://review.openstack.org/200496 ? volume connection info spec | 18:22 |
TheJulia | rloo: jroll is talking about driver composition at the moment | 18:23 |
*** sukhdev has quit IRC | 18:23 | |
rloo | TheJulia: thx. took me a few to figure that out! | 18:23 |
rloo | break for 20 min. Talk to you soon. | 18:25 |
*** ifarkas has quit IRC | 18:26 | |
lintan_ | Good night, guys | 18:26 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add test to verify ironic multitenancy https://review.openstack.org/269157 | 18:26 |
*** lintan_ has quit IRC | 18:26 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 18:27 |
mrda | I'm going to have to give up, can't stay awake. See you tomorrow. | 18:27 |
dtantsur | see you mrda, thanks for coming! | 18:29 |
dtantsur | good night mrda and lintan | 18:29 |
TheJulia | goodnight! | 18:29 |
dtantsur | rloo, do you like deva's suggestion about using "type" for filtering drivers? lile ?type=dynamic in API and --type=dynamic in CLI? | 18:32 |
dtantsur | (if missing, return everything) | 18:33 |
*** _milan_ has joined #openstack-ironic | 18:35 | |
jlvillal | So am I being foolish on: https://review.openstack.org/#/c/332195/2/ironic/drivers/modules/agent_base_vendor.py | 18:35 |
*** links has quit IRC | 18:36 | |
rloo | dtantsur: yeah, type is better. something other than type would even be better, but i can't think of anything now :) | 18:37 |
rloo | dtantsur: will look through his comments today. | 18:37 |
*** milan has quit IRC | 18:37 | |
rloo | night mrda and lintan. thx for attending! | 18:37 |
dtantsur | yes please | 18:37 |
* dtantsur is tired of this spec and probably said it several times already :) | 18:37 | |
rloo | jlvillal: it might not be 'ipmi_password', could be other '*password*' I think that .mask_password looks for several strings. | 18:39 |
rloo | jlvillal: but i guess a comment would help :) | 18:39 |
*** ChubYann has joined #openstack-ironic | 18:39 | |
rloo | jlvillal: i believe we/ironic already does a similar thing in code elsewhere. | 18:39 |
jroll | jlvillal: it returns a string '{"foo": "bar"}' is why, and yes we have the same code elsewhere | 18:40 |
dtantsur | yeah, the problem is that we also can have ilo_password, etc | 18:40 |
jroll | and also covers many things besides ipmi_password | 18:40 |
jlvillal | Yeah, I'm filling a bug report on oslo.utils as mask_password and mask_dict_password work differently. Really mask_dict_password should do what we want, but doesn't | 18:40 |
*** PollyZ has quit IRC | 18:40 | |
jroll | oslo now has differently. Really mask_dict_password should do what we want, but doesn't | 18:40 |
jroll | er | 18:40 |
jroll | ... | 18:40 |
jroll | mask_dict_password is what we really want. it didn't exist at the time | 18:40 |
jroll | I've been waiting for this code to drop to change it in a few places | 18:40 |
jroll | (though I don't see why it doesn't do what we want?) | 18:41 |
* rloo wonders why she made so many comments in that volume connection spec. now to go through them. sigh. | 18:41 | |
jlvillal | jroll, Because it checks to see if 'ipmi_password' is in the list of strings | 18:41 |
jlvillal | jroll, While mask_password() checks to see if each string in the list is in the field name. | 18:41 |
*** sacharya has joined #openstack-ironic | 18:41 | |
jlvillal | So maks_password does subset checking. mask_dict_password() does exact matching | 18:42 |
jroll | aha | 18:42 |
jroll | maybe I won't change it then :) | 18:42 |
jlvillal | jroll, https://bugs.launchpad.net/oslo.utils/+bug/1594935 | 18:42 |
openstack | Launchpad bug 1594935 in oslo.utils "mask_dict_password() is inconsistent with mask_password()" [Undecided,New] | 18:42 |
jroll | fwiw, I tried my damndest to make an RCE out of this literal_eval and couldn't | 18:42 |
jroll | it passes through some amount of json parsing before getting here, which is safe | 18:43 |
jroll | so, still tech debt but not a huge deal imo | 18:43 |
mat128 | jroll: RCE? | 18:43 |
jroll | remote code execution | 18:43 |
* jlvillal thanks mat128 for asking that :) | 18:43 | |
*** piet has quit IRC | 18:43 | |
mat128 | ahh | 18:43 |
mat128 | Google wasn't helpful | 18:44 |
*** PollyZ has joined #openstack-ironic | 18:44 | |
mat128 | even wikipedia; https://en.wikipedia.org/wiki/RCE | 18:44 |
mat128 | ;) | 18:44 |
mat128 | oy I think | 18:45 |
mat128 | hoy | 18:45 |
jroll | oi! | 18:45 |
jroll | OI OI OI | 18:45 |
* jroll https://en.wikipedia.org/wiki/Oi! | 18:46 | |
jroll | that ! is part of the link btw | 18:46 |
jlvillal | dtantsur, Howdy Y'all | 18:46 |
dtantsur | ah! | 18:46 |
mat128 | jroll: I see we're probably on the same IRC client ;) | 18:46 |
dtantsur | or rather: ой ;) | 18:46 |
TheJulia | Y'all ready to start discussing this spec? | 18:46 |
dtantsur | jlvillal, ^^ | 18:46 |
mat128 | https://review.openstack.org/#/c/200496/39/specs/approved/volume-connection-information.rst | 18:47 |
*** sacharya has quit IRC | 18:47 | |
*** jvgrant has joined #openstack-ironic | 18:47 | |
*** htruta` is now known as htruta | 18:48 | |
*** catintheroof has quit IRC | 18:51 | |
*** catintheroof has joined #openstack-ironic | 18:51 | |
sambetts | brb | 18:52 |
openstackgerrit | Merged openstack/ironic: Centralize config options - [conductor] https://review.openstack.org/301399 | 18:52 |
*** ayoung has quit IRC | 18:54 | |
*** PollyZ_ has joined #openstack-ironic | 18:55 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic-specs: Volume connection information for Ironic nodes https://review.openstack.org/200496 | 18:56 |
*** sdake has joined #openstack-ironic | 18:56 | |
*** piet has joined #openstack-ironic | 18:56 | |
*** PollyZ has quit IRC | 18:56 | |
TheJulia | jroll: line 779 rdb_root | 18:57 |
jroll | TheJulia: /me shakes fist at rbd vs rdb | 18:58 |
TheJulia | heh | 18:58 |
mat128 | boot from volume spec, all we had to correct are hyphens | 18:58 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic-specs: Volume connection information for Ironic nodes https://review.openstack.org/200496 | 18:58 |
mat128 | :) | 18:58 |
*** ayoung has joined #openstack-ironic | 18:59 | |
*** sdake_ has quit IRC | 18:59 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [iboot] https://review.openstack.org/301856 | 19:00 |
*** ayoung has quit IRC | 19:00 | |
jlvillal | sheep vote :D | 19:00 |
mat128 | https://review.openstack.org/#/c/294995/ | 19:01 |
rloo | can you hear me? | 19:02 |
rloo | i guess not. | 19:02 |
mat128 | rloo: no | 19:02 |
rloo | i got muted hmm... | 19:02 |
jroll | HRM | 19:02 |
rloo | i'll dial in again. | 19:02 |
rloo | connector_id | 19:03 |
mat128 | From now on, jroll will be known as the guy with unlookupable acronyms (https://www.google.ca/search?q=HRM&oq=HRM&aqs=chrome..69i57j0l5.886j0j7&sourceid=chrome&ie=UTF-8#q=define:+HRM) | 19:03 |
mat128 | jlvillal: ^ | 19:03 |
mat128 | ;) | 19:03 |
jlvillal | mat128, heh | 19:03 |
rloo | line 80 | 19:03 |
TheJulia | rloo: pondering | 19:03 |
TheJulia | kind of in the no opinion boat | 19:04 |
openstackgerrit | Bharath kumar proposed openstack/ironic: adding back the inspect_ports as an optional parameter https://review.openstack.org/325230 | 19:04 |
jroll | mat128: I'm a huge fan of TLAs | 19:05 |
mat128 | got that one :P | 19:06 |
mat128 | https://en.wikipedia.org/wiki/Three-letter_acronym | 19:06 |
mat128 | jroll: ever worked in military systems? | 19:06 |
mat128 | 3-4 letter acronyms are the norm :P | 19:06 |
jroll | mat128: general motors close enough? | 19:06 |
mat128 | probably | 19:06 |
jroll | :D | 19:06 |
jroll | sorry, GM* | 19:06 |
dtantsur | lol | 19:07 |
pas-ha | has the sound become better? | 19:07 |
jroll | pas-ha: yes | 19:07 |
jlvillal | If you are on the call, please mute! | 19:07 |
pas-ha | in fact I was muted | 19:07 |
pas-ha | smth really wrong with my setup then ^( | 19:07 |
jroll | :/ | 19:07 |
jlvillal | I think it got quieter :) | 19:08 |
rloo | DONE | 19:08 |
TheJulia | \o/ | 19:08 |
mat128 | yeah | 19:08 |
rloo | next..... TheJulia's spec! | 19:08 |
TheJulia | yay! | 19:08 |
mat128 | https://review.openstack.org/#/c/294995/ | 19:08 |
jlvillal | Unrelated note: Proposed patch to fix make_dict_password() https://review.openstack.org/#/c/332438/ | 19:09 |
* pas-ha sighs | 19:09 | |
pas-ha | ok then, 10 PM here, will leave you guys | 19:09 |
pas-ha | will try tomorrow from other machine | 19:10 |
jlvillal | pas-ha, Paka paka! Thanks :) | 19:10 |
rloo | bye pas-ha | 19:10 |
JayF | TheJulia: was multitasking, heard my name, and missed context | 19:10 |
pas-ha | jlvillal: lol :) | 19:10 |
JayF | TheJulia: like being called on by the teacher while sleeping | 19:10 |
*** bharath has quit IRC | 19:11 | |
openstackgerrit | Merged openstack/ironic: Centralize config options - [database] https://review.openstack.org/301815 | 19:12 |
*** Sukhdev has joined #openstack-ironic | 19:13 | |
* TheJulia does not want anyone to travel back in time 7 days | 19:16 | |
mat128 | why is this so? :P | 19:16 |
jlvillal | Too late! | 19:16 |
TheJulia | doh! | 19:16 |
mat128 | why 7 days? | 19:16 |
*** fragatina has joined #openstack-ironic | 19:16 | |
* jlvillal bought 1000 shares of LinkedIn :) | 19:16 | |
jroll | jlvillal: just now? O_o | 19:18 |
TheJulia | mat128: https://en.wikipedia.org/wiki/Seven_Days_(TV_series) | 19:20 |
jlvillal | jroll, 7 days ago. TheJulia said don't go back in time 7 days and I said I did ;) | 19:20 |
mat128 | ahh | 19:20 |
jroll | jlvillal: 7 days wouldn't get you much, 11 days looks better :P | 19:21 |
* vdrok is not able to review anything anymore | 19:21 | |
vdrok | good night everyone! | 19:21 |
TheJulia | goodnight vdrok | 19:21 |
jroll | \o vdrok | 19:21 |
* jroll brb | 19:21 | |
jlvillal | jroll, Oh darn. I thought Microsoft had announced the purchase about six days ago. | 19:21 |
*** edand has joined #openstack-ironic | 19:22 | |
*** jvgrant has quit IRC | 19:24 | |
rloo | jlvillal: don't give up your day job | 19:25 |
jlvillal | rloo, :P | 19:26 |
lucasagomes | jlvillal, you serious or kidding about the shares? | 19:27 |
*** kromanenko_ has joined #openstack-ironic | 19:27 | |
*** kromanenko__ has joined #openstack-ironic | 19:27 | |
*** Romanenko_K has joined #openstack-ironic | 19:27 | |
* lucasagomes is intrigued | 19:27 | |
jlvillal | lucasagomes, joking! | 19:27 |
lucasagomes | ah right heh | 19:27 |
TheJulia | before anyone travels through time, can I get some sort of temporal displacement device, I need to make sure I don't repeat driving for 9 hours on sunday | 19:27 |
*** jcoufal_ has joined #openstack-ironic | 19:27 | |
*** e0ne_ has joined #openstack-ironic | 19:28 | |
*** e0ne_ has quit IRC | 19:28 | |
*** piet has quit IRC | 19:29 | |
*** e0ne has quit IRC | 19:30 | |
*** david-lyle has quit IRC | 19:30 | |
*** jcoufal has quit IRC | 19:30 | |
*** vsaienko has quit IRC | 19:30 | |
*** cinerama has quit IRC | 19:30 | |
*** caiobo-afk has quit IRC | 19:30 | |
*** aarefiev has quit IRC | 19:30 | |
*** gabriel-bezerra has quit IRC | 19:30 | |
*** hoonetorg has quit IRC | 19:31 | |
*** tonyb has quit IRC | 19:31 | |
* TheJulia suspects there has been a fiber cut someplace in the world | 19:31 | |
*** sukhdev_ has joined #openstack-ironic | 19:31 | |
JayF | http://internetpulse.keynote.com/ | 19:32 |
*** tonyb has joined #openstack-ironic | 19:32 | |
JayF | L3 to cogent and verizon is hurting | 19:32 |
*** MattMan has quit IRC | 19:33 | |
*** MattMan has joined #openstack-ironic | 19:33 | |
*** cinerama has joined #openstack-ironic | 19:34 | |
jlvillal | Off topic. Do we want the 90 minute Grenade job to be also be a 'gate' job. Currently it is only a 'check' job. | 19:35 |
*** rbudden has quit IRC | 19:35 | |
jlvillal | Never mind | 19:35 |
jlvillal | It already is. Duh | 19:35 |
mat128 | was about to suggest that they should | 19:35 |
* jlvillal can't read a Zuul dashbord. | 19:35 | |
mat128 | otherwise you're never running it against latest code | 19:35 |
*** ayoung has joined #openstack-ironic | 19:40 | |
openstackgerrit | Merged openstack/ironic: Centralize config options - [dhcp] https://review.openstack.org/301830 | 19:40 |
openstackgerrit | Merged openstack/ironic: Mask password on agent lookup according to policy https://review.openstack.org/332195 | 19:40 |
*** david-lyle has joined #openstack-ironic | 19:41 | |
TheJulia | JayF: yeouch, nearly 8% packet loss on one of the paths that is measuring | 19:41 |
*** caiobo-afk has joined #openstack-ironic | 19:42 | |
dtantsur | jlvillal, mat128, we are not allowed to have voting check jobs that are not in gate | 19:42 |
dtantsur | (allowed by infra) | 19:42 |
mat128 | excellent | 19:43 |
jlvillal | dtantsur, thanks | 19:43 |
mat128 | from a consistency/integrity standpoint :) | 19:43 |
*** gabriel-bezerra has joined #openstack-ironic | 19:43 | |
*** aarefiev has joined #openstack-ironic | 19:43 | |
*** hoonetorg has joined #openstack-ironic | 19:43 | |
TheJulia | JayF: or take migraine meds | 19:43 |
*** vsaienko has joined #openstack-ironic | 19:44 | |
rloo | i would just cancel tonight's, but I am not going to be there. You could do bug triage :) | 19:45 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add test to verify ironic multitenancy https://review.openstack.org/269157 | 19:45 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 19:46 |
mat128 | for whoever reads scrollback, I'm going to be there at 0000 UTC for the next session | 19:47 |
dtantsur | aaaaaaaaaaaaand I'm calling it a day :) | 19:48 |
dtantsur | please, please, please review the driver composition spec and all the comments to it | 19:49 |
lucasagomes | ++ | 19:49 |
lucasagomes | have a great evening all | 19:49 |
dtantsur | I really hope to land it tomorrow | 19:49 |
dtantsur | g'night everyone, g'night lucasagomes | 19:49 |
rloo | bye dtantsur, lucasagomes. a demain. | 19:49 |
JayF | You know, a thought I had after how effective the mid-cycle is at unsticking specs | 19:49 |
*** edand has quit IRC | 19:49 | |
JayF | would anyone be interested in doing a regular, over audio/video, review jam for specs? | 19:49 |
* rloo worries about JayF's thought | 19:49 | |
dtantsur | true | 19:49 |
JayF | like weekly or semi-weekly | 19:49 |
jroll | yes, was going to suggest that | 19:50 |
dtantsur | JayF++, probably 1-2 times a month | 19:50 |
mat128 | JayF: sure | 19:50 |
rloo | JayF, jroll: i actually was thinking that it would be worthwhile to do these meetings sort of *whenever* there was something to discuss. | 19:50 |
rloo | instead of waiting for a midcycle | 19:50 |
JayF | whenever is kinda like eventually | 19:50 |
mat128 | just send a doodle to the list ;) | 19:50 |
JayF | it's hard to ever come around | 19:50 |
TheJulia | JayF: ++ | 19:50 |
JayF | somethign that's scheduled and cancelled if unneeded, the momentum helps | 19:51 |
rloo | JayF: well, whenever would be when someone suggests it. | 19:51 |
*** dtantsur is now known as dtantsur|afk | 19:51 | |
rloo | JayF: eg, you had some sort of meeting to talk about keystone policy support and i assume people joined that were interested in that topic? | 19:51 |
rloo | JayF: how did that work for you? | 19:51 |
mat128 | rloo: we were 4 and JayF wrote the spec after our discussion | 19:51 |
*** PollyZ_ has quit IRC | 19:51 | |
dtantsur|afk | it's easier to dedicate time if it's regular | 19:51 |
rloo | ok, so it was productive, right? | 19:52 |
JayF | well I didn't really write much of the spec | 19:52 |
JayF | yet actaully | 19:52 |
dtantsur|afk | it's also easier to communicate downstream that you're busy every Nth of the month, for example | 19:52 |
openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Extend python-oneviewclient to Network https://review.openstack.org/294559 | 19:52 |
* dtantsur|afk is really afk now | 19:52 | |
mat128 | rloo: IMO, yes | 19:52 |
rloo | the problem with picking a regular time/date, is that you will be ostracizing? certain folks. | 19:52 |
JayF | It was mostly productive. I probably would try to have more of a framework in place before doing it again | 19:52 |
mat128 | rloo: just like the weekly meeting | 19:52 |
JayF | i.e. not going in pre-spec full-on | 19:52 |
jroll | JayF: re IPA releases, a few things: 1) there is indeed a liberty branch, 2) this patch isn't on mitaka nor liberty (even in review), 3) master release is here: https://review.openstack.org/332455 | 19:52 |
JayF | I'd have a spec with todos or whatever in it | 19:52 |
JayF | jroll: eek, I can cherry pick it back | 19:53 |
rloo | mat128: right, the meeting time isn't conducive to some people attending it. | 19:53 |
jroll | JayF: cool, thanks | 19:53 |
rloo | mat128: so it seems like it makes more sense to say, hey, i want to discuss Foo. Who is interested in Foo and lets find a time that works for us. | 19:53 |
mat128 | rloo: but you risk never "calling out" those sessions | 19:53 |
mat128 | rloo: I say let's try it | 19:53 |
mat128 | the worst that can happen is that we learn from it | 19:54 |
mat128 | best case, we get stuff done< | 19:54 |
rloo | mat128: try me. I can call it out if I wanted to. And I would think that other cores would too. JayF did! | 19:54 |
JayF | I am going to activate on doing an informal review jam, with a preferred focus on stuck specs. Participation optional. | 19:54 |
mat128 | +1 to this | 19:55 |
rloo | my concern is that if we pick a time when 'most' folks can attend, folks in asia/australia/etc will be more inconvenienced. | 19:55 |
JayF | I mean, IMO, they can feel free to have one at a convienient time for them too? | 19:55 |
mat128 | rloo: so more points for doing it ad-hoc, but Dimtry's point was "harder to block schedule time for it" | 19:55 |
JayF | There's nothing special about saying "I'm going to work on specs from x PM to y PM, feel free to join me in sip:blah" | 19:55 |
mat128 | JayF: I guess | 19:55 |
rloo | JayF: well, it seems like if there aren't enough cores/regulars/vocals, it isn't that useful. | 19:55 |
JayF | anyone can do it :) | 19:55 |
mat128 | jlvillal, sambetts, myself and maybe TheJulia are going to be discussing multinode grenade thursday 0800 PST | 19:56 |
mat128 | there, done :) | 19:56 |
rloo | mat128: maybe send out email then. Would reach a wider audience i hope. | 19:57 |
mat128 | John was leading this tho, but I'm sure he will read it from the scrollback | 19:57 |
rloo | jlvillal: ^^ | 19:57 |
mat128 | I kinda just jumped in the topic | 19:57 |
*** PollyZ has joined #openstack-ironic | 19:58 | |
mat128 | alright, away until right before 0000 UTC | 19:59 |
openstackgerrit | Merged openstack/ironic-specs: Volume connection information for Ironic nodes https://review.openstack.org/200496 | 20:01 |
TheJulia | also away for the next few hours | 20:01 |
jlvillal | rloo, mat128 Yep. We are meeting on Thursday 8AM Pacific time | 20:01 |
jlvillal | To discuss multi-node grenade | 20:01 |
*** PollyZ has quit IRC | 20:02 | |
jlvillal | rloo, I'm not sure about wanting a giant group of people. Is that wrong??? | 20:02 |
rloo | jlvillal: no, it isn't wrong. i doubt many people would show up, or if they did, how much they'd contribute. It was in light of the discussion we were having. | 20:03 |
*** jjohnson2_ has quit IRC | 20:03 | |
rloo | jlvillal: i wonder how inclusive we have been. but whatever works to get the work done. | 20:03 |
*** jjohnson2 has joined #openstack-ironic | 20:07 | |
*** PollyZ has joined #openstack-ironic | 20:11 | |
*** Sukhdev has quit IRC | 20:13 | |
*** sukhdev_ is now known as sukhdev | 20:13 | |
*** rcernin has joined #openstack-ironic | 20:13 | |
*** ijw has quit IRC | 20:14 | |
*** moshele has joined #openstack-ironic | 20:15 | |
*** PollyZ has quit IRC | 20:16 | |
openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Extend python-oneviewclient to UplinkSet https://review.openstack.org/295981 | 20:21 |
*** PollyZ has joined #openstack-ironic | 20:25 | |
*** ijw has joined #openstack-ironic | 20:26 | |
*** PollyZ has quit IRC | 20:29 | |
*** moshele has quit IRC | 20:33 | |
*** moshele has joined #openstack-ironic | 20:34 | |
*** moshele has quit IRC | 20:35 | |
*** moshele has joined #openstack-ironic | 20:36 | |
*** ayoung has quit IRC | 20:38 | |
*** PollyZ has joined #openstack-ironic | 20:38 | |
openstackgerrit | Merged openstack/ironic: Mask password on agent lookup according to policy https://review.openstack.org/332197 | 20:39 |
*** PollyZ has quit IRC | 20:43 | |
*** jjohnson2 has quit IRC | 20:43 | |
JayF | jroll: https://review.openstack.org/#/c/332471/ | 20:52 |
JayF | also, apparently stable/mitaka commits for ipa aren't coming in here | 20:52 |
jroll | yeah, probably something missed in irc bot config | 20:52 |
rloo | i thought jlvillal had a patch to fix that. maybe it hasn't merged yet. | 20:53 |
*** dprince has quit IRC | 20:53 | |
jroll | rloo: seems to be fixed for ironic | 20:53 |
openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Fix unwanted unmounting of /usr/share/oem https://review.openstack.org/332472 | 20:54 |
JayF | but stable/liberty works | 20:54 |
JayF | *shrug* | 20:54 |
JayF | jroll: ^ that actually needs a review with brain involved | 20:55 |
jroll | lol | 20:55 |
JayF | jroll: it didn't cleanly cherry pick and more changes got pulled in | 20:55 |
JayF | but the changes are good stuff that should've been backported too, I think | 20:55 |
JayF | so... yeah | 20:55 |
jroll | JayF: cherry-pick pulled all of those in? | 20:55 |
JayF | cherry-pick + I think a bad rebase by me | 20:55 |
JayF | but that's the end result I'd prefer over just adding the one mask | 20:56 |
jlvillal | JayF, rloo, I did update it: https://review.openstack.org/#/c/330867/ For the bot announcements | 20:56 |
jlvillal | It got merged. | 20:56 |
JayF | so IDK if technical correctness or git correctness is better | 20:56 |
*** Sukhdev_ has joined #openstack-ironic | 20:56 | |
rloo | jlvillal: thx. I have no idea how I knew about it but I did :) | 20:57 |
jroll | JayF: commented, liberty is security support right now :/ | 20:58 |
jroll | and you'll need devananda or dtantsur|afk for the +A on the mitaka | 20:59 |
openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Fix unwanted unmounting of /usr/share/oem https://review.openstack.org/332472 | 20:59 |
JayF | jroll: ^ | 20:59 |
openstackgerrit | Moshe Levi proposed openstack/ironic-python-agent: Replace assertRaisesRegexp with assertRaisesRegex https://review.openstack.org/332250 | 20:59 |
jlvillal | jroll, Speaking of security. When I searched for the CVE for the masked password. It said, no information yet. | 20:59 |
*** rbudden has joined #openstack-ironic | 20:59 | |
jlvillal | As in, the information was still to be released. | 20:59 |
jroll | JayF: done | 20:59 |
jroll | jlvillal: hrm, not sure who is responsible for that, got a quick link? | 21:00 |
JayF | devananda: tonyb: would appreciate if one of you found time to land https://review.openstack.org/#/c/332471/ + https://review.openstack.org/#/c/332472/ -- security fixes for IPA | 21:00 |
rloo | why announce it before the fix actually lands? | 21:00 |
JayF | rloo: IDK if that was pointed at me or john, but for the IPA one, we unembargoed it after research showed it was non-exploitable | 21:01 |
jroll | rloo: because that's the process | 21:01 |
jroll | rloo: everything (bug, emails, patches proposed) happens at the same time https://security.openstack.org/vmt-process.html | 21:01 |
jroll | patches are included in the announcement, so people can apply without them merging | 21:01 |
rloo | jroll: Oh. Interesting. I guess if the patches are included, it is fine. | 21:02 |
jroll | yep | 21:02 |
jlvillal | jroll, http://www.cve.mitre.org/cgi-bin/cvename.cgi?name=2016-4985 | 21:02 |
jlvillal | jroll, https://review.openstack.org/#/c/332197/3/releasenotes/notes/fix-cve-2016-4985-b62abae577025365.yaml | 21:02 |
*** moshele has quit IRC | 21:02 | |
jroll | jlvillal: thanks | 21:03 |
*** moshele has joined #openstack-ironic | 21:03 | |
*** moshele has quit IRC | 21:03 | |
*** raildo is now known as raildo-afk | 21:04 | |
*** PollyZ has joined #openstack-ironic | 21:06 | |
*** yolanda has joined #openstack-ironic | 21:06 | |
*** trown is now known as trown|outtypewww | 21:07 | |
*** baoli has quit IRC | 21:11 | |
*** baoli has joined #openstack-ironic | 21:12 | |
*** ijw has quit IRC | 21:23 | |
*** ccamacho has quit IRC | 21:24 | |
*** jcoufal_ has quit IRC | 21:30 | |
*** baoli has quit IRC | 21:30 | |
*** fragatina has quit IRC | 21:33 | |
*** ayoung has joined #openstack-ironic | 21:41 | |
*** rcernin has quit IRC | 21:41 | |
JayF | I just -1'd https://review.openstack.org/#/c/320584 because it removes two fields from the NetworkInterface class on IPA hardware module | 21:44 |
*** sacharya has joined #openstack-ironic | 21:44 | |
JayF | I'm afraid it could break backwards compat but could be convinced it doesn't matter ... but I'm not sure. | 21:44 |
rloo | JayF: thx. I wondered about that (and asked). Thought it was sufficient to document it cuz it apparently wasn't used. | 21:45 |
JayF | I mean, someone downstream could be using it | 21:45 |
sambetts | JayF: those fields where never used, and where commented as such, so will we really break anyone? Do we need a deprecation period for a field and will be and has always been None? | 21:45 |
JayF | and completely removing it could break code ... if it wasn't properly using .get() and if it populated themselves | 21:45 |
JayF | sambetts: I don't know, I'm asking the question here because I feel like it oculd be a stretch | 21:45 |
sambetts | JayF: do we support people using those fields? | 21:46 |
JayF | We support pluggable hardware managers. and list_hardware_info is overrideable. | 21:46 |
JayF | We can't assume that just because we don't populate it that nobody does. | 21:46 |
sambetts | JayF: is NetworkInterface specifc to the generic hardware manager? | 21:46 |
JayF | But I agree that it's a bit of a giant stretch | 21:46 |
JayF | no | 21:46 |
JayF | it's in the hardware module | 21:46 |
JayF | and populated by list_hardware_info, which can be overridden by a hardware manager | 21:46 |
JayF | *that class is instanced into list_hardware_info, and used to describe network interfaces, and those fields could've been populated by a hardware manager | 21:47 |
* JayF not sure the proper terms to use for that | 21:47 | |
* sambetts hates people adding stuff like this :'( I would have totally -1'd a patch adding fields for no reason | 21:47 | |
JayF | I mean, this dates way, way, way back | 21:47 |
JayF | was probably in the code when it went from teeth-agent -> IPA | 21:47 |
jroll | # TODO(russellhaering): Pull these from LLDP | 21:48 |
JayF | I'm OK with us saying "we're OK breaking this" | 21:48 |
jroll | lol | 21:48 |
JayF | I just want us to make that decision explicitly | 21:48 |
jroll | I mean, deprecate and remove next cycle? | 21:48 |
JayF | jroll: how do you even detect that's being read to log it for deprecation? | 21:48 |
jroll | JayF: override __getattr__ and __setattr__ | 21:48 |
JayF | jroll: I'm sorta on the side of just dump it, but wanted to make sure we knew what we were doing | 21:48 |
*** chlong|rhce_trng has quit IRC | 21:48 | |
*** sacharya has quit IRC | 21:49 | |
sambetts | we've added a release note for it if thats any condolence | 21:49 |
JayF | I mean, if the consensus is drop it, I'll land the patch now | 21:49 |
JayF | I'm like, 90% sure I have the only downstream hwm | 21:49 |
JayF | and we don't use it | 21:49 |
sambetts | heh | 21:49 |
JayF | but that assumption only gets worse and worse as time goes by | 21:50 |
jroll | I mean, I don't agree we should drop it | 21:50 |
jroll | when there's a clear path to deprecating it | 21:50 |
*** catintheroof has quit IRC | 21:51 | |
jroll | (hacky, but clear) | 21:51 |
* sambetts can put up a fresh patch when I'm more awake deprecating it | 21:51 | |
sambetts | I'll leave it as None though, I don't plan to populate it with anything, if thats ok? | 21:51 |
JayF | oh absolutely | 21:52 |
rloo | oh, I think we should populate it now that we are grabbing that info. | 21:52 |
* rloo is joking | 21:52 | |
* sambetts nearly started crying | 21:52 | |
jroll | sambetts: yeah, that's fine | 21:52 |
rloo | no worries sambetts. guess I should have +A'd before JayF saw it :) | 21:53 |
JayF | I mean, I don't have a strong opinion that we should keep it | 21:53 |
JayF | I do have a strong opinion that we should provide a stable interface for downstream hwms | 21:53 |
* JayF feels bad man for -1'ing over something like that | 21:54 | |
*** ametts has quit IRC | 21:54 | |
*** mbound has joined #openstack-ironic | 21:55 | |
jroll | JayF: I would've done it if I saw it first, don't feel bad | 21:55 |
sambetts | it wasn't like I wasn't expecting it, I just tried it the other way first because it was a weird "we've reserved these fields" senario | 21:56 |
openstackgerrit | Ruby Loo proposed openstack/python-ironicclient: Move methods to utils.py https://review.openstack.org/332489 | 21:56 |
JayF | I think that's maybe the last case of something like that in IPA | 21:57 |
jroll | hope so :P | 21:57 |
JayF | I might breeze through hardware module and see if there's anything else we might want to "proatively deprecate" from the beforetime | 21:58 |
sambetts | I hope so :-p /me has got his ban hammer out | 21:58 |
sambetts | anyways its like 23:00 and my beer is getting warm over there so I'll see everyone tomorrow o/ | 22:01 |
sambetts | jlvillal: https://review.openstack.org/#/c/332490/1 | 22:02 |
jroll | why'd you leave your beer over there? :) | 22:02 |
jroll | \o night sam | 22:02 |
sambetts | heh good point :-P | 22:02 |
jlvillal | sambetts, You should be going to bed! :) | 22:02 |
*** sambetts is now known as sambetts|afk | 22:02 | |
jlvillal | sambetts|afk, You and mat128 are doing the same stuff :) | 22:02 |
sambetts|afk | XD | 22:02 |
sambetts|afk | we'll work it out on Thursday I guess | 22:03 |
jlvillal | sambetts|afk, Needs to be experimental though | 22:03 |
sambetts|afk | jlvillal: yeah good point | 22:03 |
sambetts|afk | jlvillal: I'll fix in the morning | 22:03 |
sambetts|afk | :D | 22:03 |
sambetts|afk | night all o/ | 22:03 |
jlvillal | sambetts|afk, Ciao | 22:03 |
* jlvillal hopes sambetts|afk votes "No on Brexit" :) | 22:04 | |
* jlvillal is a British citizen but doesn't have a vote since he doesn't live in the UK | 22:04 | |
*** jrist has quit IRC | 22:06 | |
*** thrash is now known as thrash|g0ne | 22:07 | |
*** Sukhdev_ has quit IRC | 22:16 | |
*** Sukhdev_ has joined #openstack-ironic | 22:20 | |
tonyb | JayF: np, I'll look at them after the school run | 22:22 |
JayF | thanks! | 22:22 |
tonyb | JayF: actually they're so simple ..... mostly done | 22:24 |
tonyb | JayF: As I said in my comment. I'll +W the liberty version once mitaka merges | 22:25 |
JayF | tonyb: curious why the wait? Not complaining just wanna understand | 22:25 |
*** ijw has joined #openstack-ironic | 22:27 | |
*** rbudden has quit IRC | 22:27 | |
jroll | JayF: I'll release those in the morning btw | 22:28 |
JayF | thanks | 22:28 |
tonyb | JayF: because I'm a stickler for the rules. Nothing non n-1 that isn't on n. I've seen things get +W togther andf hen fail to merge on n and noone ever looks to see that | 22:28 |
JayF | tonyb: gotcha, that's the rule I wasn't aware of, makes perfect sense though | 22:28 |
jroll | dtantsur|afk: sorry I didn't get back to driver comp spec today, I'll do it in the morning, got tied up with surprise meetings and this CVE :/ | 22:28 |
jroll | jlvillal: fyi, I emailed the oss-security list with the announcement, they will release the CVE at some point after they see that | 22:29 |
* jroll finds beer and goes outside, night everyone | 22:29 | |
tonyb | JayF: I have a poll running on the merge event so as long as that happens today I'll do the liberty version | 22:29 |
tonyb | jroll: good plan! | 22:29 |
JayF | yeah, as long as it's landed for jroll to release in the morning | 22:29 |
JayF | I seriously was just curious about why you were waiting, always want to know why so I can understand :) | 22:30 |
tonyb | JayF: cool. Always better to ask than sit and wonder. | 22:31 |
*** rcarrillocruz has quit IRC | 22:39 | |
*** rcarrillocruz has joined #openstack-ironic | 22:41 | |
*** rpioso has quit IRC | 22:48 | |
*** sukhdev has quit IRC | 22:53 | |
*** chlong has joined #openstack-ironic | 23:02 | |
*** praneshp has quit IRC | 23:02 | |
openstackgerrit | Merged openstack/ironic-python-agent: Fix unwanted unmounting of /usr/share/oem https://review.openstack.org/332471 | 23:09 |
*** Nisha has joined #openstack-ironic | 23:09 | |
*** jrist has joined #openstack-ironic | 23:12 | |
openstackgerrit | Merged openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/332297 | 23:17 |
*** snecklifter has quit IRC | 23:24 | |
openstackgerrit | Moshe Levi proposed openstack/ironic-python-agent: Add vendor id and vendor name to interface information https://review.openstack.org/279940 | 23:26 |
*** snecklifter has joined #openstack-ironic | 23:33 | |
*** PollyZ has quit IRC | 23:37 | |
mrda | Morning all | 23:44 |
*** sacharya has joined #openstack-ironic | 23:45 | |
*** sacharya has quit IRC | 23:49 | |
mat128 | hi mrda | 23:55 |
mat128 | pstn line is busy :( | 23:57 |
mrda | I'm not going to join | 23:57 |
mrda | I've got day jobn responsibilities to deal with :) | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!