Tuesday, 2016-06-21

*** praneshp has joined #openstack-ironic00:07
jlvillalTheJulia, https://www.manning.com/books/irresistible-apis00:09
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: [WIP]Introducing Inspector Grenade testing  https://review.openstack.org/32766700:16
*** sabeen1 has quit IRC00:16
*** edand has joined #openstack-ironic00:17
*** amotoki has joined #openstack-ironic00:17
*** edand has quit IRC00:21
mat128http://open-zfs.org/wiki/Feature_Flags00:22
*** amotoki has quit IRC00:22
*** xavierrr has joined #openstack-ironic00:22
*** sabeen has joined #openstack-ironic00:26
*** Nisha_away has quit IRC00:28
openstackgerritClenimar Filemon proposed openstack/python-ironic-inspector-client: Switch to keystoneauth  https://review.openstack.org/33189300:30
*** rloo has quit IRC00:38
*** yogi has joined #openstack-ironic00:43
*** sdake_ has quit IRC00:44
mrda.00:47
jlvillalFYI: We are talking in the mid-cycle call now.00:49
jlvillalmrda, ^^^ :)00:49
jlvillalhttps://etherpad.openstack.org/p/ironic-newton-midcycle00:49
*** hoangcx has joined #openstack-ironic00:52
*** rajinir has quit IRC00:54
*** spandhe has quit IRC00:58
*** phuongnh has joined #openstack-ironic01:00
jrolljlvillal: GRE :)01:01
* jroll here now btw01:01
jlvillaljroll, Thanks!01:02
jrollthis seems relevant: https://github.com/openstack-infra/devstack-gate/blob/3ef48cec70c771242d743467768f22d1ecb55ecb/devstack-vm-gate.sh#L8901:04
openstackgerritMerged openstack/ironic: Replace dict.get(key) in drivers unit tests  https://review.openstack.org/33085201:05
xavierrrcoding and listening to you guys :D01:05
jlvillalxavierrr, stalker! ;)01:06
xavierrrhahahaha01:06
xavierrryeap01:06
jlvillalMaking some notes on the multi-node stuff in: https://etherpad.openstack.org/p/ironic-newton-grenade-whiteboard01:06
xavierrrto much to study before joining a meeting with you haha01:06
jlvillalfeel free to add content01:06
*** rama_y has quit IRC01:08
jcookclif_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-metadata01:08
jlvillalTheJulia, https://github.com/openstack/ironic/blob/master/devstack/upgrade/resources.sh#L3501:09
jlvillalTheJulia, That is for Grenade though01:09
jcookclif_h reviewed your spec: https://review.openstack.org/#/c/326073/101:09
jlvillalhttps://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L66001:10
jrolljcook: it's a nova feature, so it should be a bp :P01:11
jcook(-‸ლ)01:11
jcookclif_h disregard, apparently my neurons have given up on me for the day and I should close my laptop lid01:12
mrdaJust to let y'all know, the midcycle has finished up for today01:19
*** amotoki has joined #openstack-ironic01:20
TheJuliamrda: thank you :)01:22
openstackgerritRuby Loo proposed openstack/ironic: Fix test in test_agent_client.py  https://review.openstack.org/33190401:22
*** amotoki has quit IRC01:24
*** joprovost has joined #openstack-ironic01:34
*** amotoki has joined #openstack-ironic01:40
*** sacharya has joined #openstack-ironic01:59
*** sacharya has quit IRC01:59
*** sacharya has joined #openstack-ironic02:00
*** amotoki has quit IRC02:00
*** zhs has joined #openstack-ironic02:10
*** fragatina has quit IRC02:14
*** fragatina has joined #openstack-ironic02:15
*** amotoki has joined #openstack-ironic02:16
*** xavierrr has quit IRC02:31
*** zhs_s has joined #openstack-ironic02:34
*** PollyZ has joined #openstack-ironic02:35
*** zhs has quit IRC02:37
*** PollyZ has quit IRC02:38
*** PollyZ has joined #openstack-ironic02:40
*** PollyZ has quit IRC02:44
*** yuanying has quit IRC02:51
*** zhs_ss has joined #openstack-ironic02:53
*** stevemar has quit IRC02:54
*** yuanying has joined #openstack-ironic02:54
*** zhs_s has quit IRC02:56
*** PollyZ has joined #openstack-ironic03:02
*** zhs_ss has quit IRC03:12
*** spandhe has joined #openstack-ironic03:15
*** joprovost has quit IRC03:16
*** rama_y has joined #openstack-ironic03:17
*** spandhe_ has joined #openstack-ironic03:22
*** spandhe has quit IRC03:23
*** spandhe_ is now known as spandhe03:23
*** jaybeale has joined #openstack-ironic03:23
*** jaybeale has quit IRC03:24
*** jaybeale has joined #openstack-ironic03:24
*** jaybeale has quit IRC03:25
*** jaybeale has joined #openstack-ironic03:25
*** sabeen has quit IRC03:31
*** sabeen has joined #openstack-ironic03:35
*** rama_y has quit IRC03:35
*** spandhe has quit IRC03:35
*** sabeen has quit IRC03:36
*** ijw has quit IRC03:54
*** spandhe has joined #openstack-ironic04:02
*** yuanying has quit IRC04:02
*** yuanying has joined #openstack-ironic04:03
*** PollyZ has quit IRC04:04
*** Nisha_away has joined #openstack-ironic04:04
*** jaybeale has quit IRC04:09
*** jaybeale has joined #openstack-ironic04:10
*** jaybeale has quit IRC04:15
*** links has joined #openstack-ironic04:17
*** yuikotakadamori has joined #openstack-ironic04:20
*** yuikotakadamori has quit IRC04:31
mat128jlvillal: 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 the04:33
mat128 jenkins jobs)04:33
mat128(first time I get a 2-part message.. bed time I guess)04:34
mat128we 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 IRC04:47
*** fragatina has joined #openstack-ironic04:48
*** fragatina has quit IRC04:50
*** yogi has quit IRC04:51
*** fragatina has joined #openstack-ironic04:51
*** rama_y has joined #openstack-ironic04:56
*** moshele has joined #openstack-ironic04:59
*** moshele has quit IRC05:10
*** amotoki has quit IRC05:11
*** amotoki has joined #openstack-ironic05:14
*** ChubYann has quit IRC05:19
*** itamarl has joined #openstack-ironic05:37
*** spandhe_ has joined #openstack-ironic05:51
*** spandhe has quit IRC05:52
*** spandhe_ is now known as spandhe05:52
*** edand has joined #openstack-ironic05:54
*** yolanda has joined #openstack-ironic05:55
*** rcernin has joined #openstack-ironic05:56
*** rama_y has quit IRC05:57
*** jtomasek has joined #openstack-ironic06:02
*** skramaja has joined #openstack-ironic06:07
*** moshele has joined #openstack-ironic06:09
*** spandhe has quit IRC06:15
*** hoangcx has quit IRC06:24
*** praneshp has quit IRC06:26
*** praneshp has joined #openstack-ironic06:28
*** yolanda has quit IRC06:32
*** lucasagomes has quit IRC06:44
*** PixieBoots has quit IRC06:44
*** yuikotakadamori has joined #openstack-ironic06:44
openstackgerritBharath kumar proposed openstack/ironic: Dummy patch: Do not merge  https://review.openstack.org/33196806:45
*** edand has quit IRC06:50
openstackgerritMoshe Levi proposed openstack/ironic: Add missing translation hint to IronicException  https://review.openstack.org/33196906:50
*** edand has joined #openstack-ironic06:51
*** lucasagomes has joined #openstack-ironic06:51
*** pcaruana has joined #openstack-ironic06:52
*** praneshp has quit IRC06:59
*** rbartal has joined #openstack-ironic06:59
openstackgerritYuiko Takada Mori proposed openstack/ironic-specs: Nova compatible serial console support  https://review.openstack.org/31950507:13
*** sacharya has quit IRC07:13
*** hoangcx has joined #openstack-ironic07:14
*** chlong|rhce_trng has quit IRC07:27
*** kromanenko__ has joined #openstack-ironic07:29
*** kromanenko_ has joined #openstack-ironic07:29
*** Romanenko_K has joined #openstack-ironic07:29
pas-hamorning Ironic07:34
*** athomas has joined #openstack-ironic07:35
openstackgerritMoshe Levi proposed openstack/ironic: Replace assertRaisesRegexp with assertRaisesRegex  https://review.openstack.org/33199507:40
*** lucasagomes has quit IRC07:46
*** lucasagomes has joined #openstack-ironic07:47
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:03
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c  https://review.openstack.org/31844008:10
*** e0ne has joined #openstack-ironic08:11
*** sacharya has joined #openstack-ironic08:14
*** snecklifter has joined #openstack-ironic08:17
*** dmk0202 has joined #openstack-ironic08:18
*** mgoddard has joined #openstack-ironic08:19
*** sacharya has quit IRC08:19
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: [WIP]Introducing Inspector Grenade testing  https://review.openstack.org/32766708:23
openstackgerritMoshe Levi proposed openstack/ironic: Replace assertRaisesRegexp with assertRaisesRegex  https://review.openstack.org/33199508:27
lucasagomesmorning all08:33
*** vmud213 has joined #openstack-ironic08:35
*** snecklifter has quit IRC08:36
*** moshele has quit IRC08:38
openstackgerritAline Bousquet proposed openstack/python-ironicclient: Fix error returned by ironic --json node-validate  https://review.openstack.org/30396408:38
*** yolanda has joined #openstack-ironic08:39
*** edand has quit IRC08:40
openstackgerritSzymon Borkowski proposed openstack/ironic-python-agent: Add PCI devices collector to inspector  https://review.openstack.org/32631808:40
*** moshele has joined #openstack-ironic08:40
*** mgould|afk is now known as mgould08:41
mgouldmorning Ironic!08:41
*** dmk0202 has quit IRC08:42
*** derekh has joined #openstack-ironic08:44
openstackgerritSaravanan KR proposed openstack/ironic-specs: Provide a mechanism to configure kernel boot parameters  https://review.openstack.org/33156408:45
milanmorning lucasagomes mgould and all :)08:59
milanyuikotakadamori, morning :)08:59
*** karthi has joined #openstack-ironic09:01
karthiHi Rolks09:01
karthi* Folks09:01
hkominosmorning09:02
openstackgerritSaravanan KR proposed openstack/ironic-specs: Provide a mechanism to configure kernel boot parameters  https://review.openstack.org/33156409:03
milankarthi, hkominos morning09:03
hkominosanother glorious day where i will be trying to deploy an ironic node!09:03
*** ohamada has joined #openstack-ironic09:05
karthiguys, i am trying to create a ironic node with network-interface perameter09:06
*** livelace has joined #openstack-ironic09:06
karthiBut it  is not creating the node it is saying "Requested API is not supported"09:07
karthiPosted the output here http://paste.openstack.org/show/520899/09:07
*** bharathk has joined #openstack-ironic09:15
*** openstackgerrit has quit IRC09:18
*** openstackgerrit has joined #openstack-ironic09:19
*** moshele has quit IRC09:21
*** amotoki has quit IRC09:24
*** ifarkas has joined #openstack-ironic09:28
*** itamarl has quit IRC09:30
*** itamarl has joined #openstack-ironic09:30
ifarkasmorning all!09:34
milanmorning ifarkas09:35
*** sacharya has joined #openstack-ironic09:37
*** sacharya has quit IRC09:41
*** moshele has joined #openstack-ironic09:44
hkominosHello openstackers. Does anybody know with which networks must the ILO interface communicate (VLAN tagging case)09:53
openstackgerritMoshe Levi proposed openstack/ironic-specs: Adding InfiniBand Support to ironic  https://review.openstack.org/26563509:56
openstackgerritDerek Higgins proposed openstack/pyghmi: Convert data tuple to a list  https://review.openstack.org/33204609:58
sergekmorning All :)09:59
hkominosmorning sergek10:04
*** snecklifter has joined #openstack-ironic10:05
sergekhkominos: p/10:06
*** yolanda has quit IRC10:08
*** karthi has quit IRC10:10
*** hkominos has quit IRC10:10
*** hoangcx has quit IRC10:15
*** amotoki has joined #openstack-ironic10:17
*** athomas has quit IRC10:20
*** athomas has joined #openstack-ironic10:21
sambetts|afkMorning all o/10:25
*** sambetts|afk is now known as sambetts10:25
milanmorning sambetts10:25
*** phuongnh has quit IRC10:30
mgouldmorning sambetts sergek milan ifarkas10:30
*** snecklifter has quit IRC10:31
milanhey mgould10:31
sergekmgould: o/10:33
vdroklate morning everyone10:33
milanmorning vdrok sergek10:34
milanhow are things, Ironic? ;)10:34
vdrokmorning milan mgould and sambetts :)10:34
vdrokmilan: too much heat here in ukraine :)10:35
milanvdrok, concrete melting point? ;)10:36
vdrokbrain melting point for sure, +30C now, +32C tomorrow10:37
milanvdrok, you'd better take PTO and visit some water ;) Nothing like a Pina Colada on a beach10:38
vdrokmilan: heh, it will be PTO till september then :)10:38
milanvdrok, I envy :D10:39
mgouldmorning vdrok10:40
mgouldvdrok: yikes10:40
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: [WIP]Introducing Inspector Grenade testing  https://review.openstack.org/32766710:41
* milan just had a second face-palm this morning ^^^10:42
milan./unstack.sh && grenade again10:43
sambettsmorning milan mgould vdrok10:45
milansambetts, o/ :)10:46
*** chlong|rhce_trng has joined #openstack-ironic10:50
openstackgerritMerged openstack/ironic: Replace assertRaisesRegexp with assertRaisesRegex  https://review.openstack.org/33199510:54
*** vmud213 has quit IRC11:01
*** makowals has joined #openstack-ironic11:03
*** lucasagomes is now known as lucas-afk11:06
*** thrash|g0ne is now known as thrash11:18
jiangfei livelace ping11:19
TheJuliaGood morning11:19
jiangfeimorning11:19
*** snecklifter has joined #openstack-ironic11:20
livelacejiangfei, pong11:20
TheJuliavdrok: It could always be worse, NobodyCam sent me a picture of the tempature yesterday and it read 54C...11:20
vdrokmorning TheJulia, omg :(11:21
jiangfei i have question  on  step 2:    Reset disk controller to default HDD11:22
*** amotoki has quit IRC11:22
jiangfei  livelace, i  install windows-server 2008 on vm, but i can't boot from this image.11:23
jiangfeilivelace, may be i miss the step 2, but i don't known  how ? can u help11:25
livelacejiangfei, Maybe, I will try this very soon. Now I understand Ironic and will try work with windows too11:25
*** thrash has quit IRC11:25
*** Nisha_away has quit IRC11:26
jiangfei thank you.  ironic node can active ,but can't start the windows from node.11:27
jiangfei  windows Blue Screen of Death11:28
TheJuliajiangfei: I've never done it myself, but if your using a whole disk image, it realistically should just work... oh11:28
*** thrash has joined #openstack-ironic11:28
*** thrash has quit IRC11:28
*** thrash has joined #openstack-ironic11:28
TheJuliadrivers most likely?11:28
*** ohamada has quit IRC11:29
jiangfeimy ironic node use  a whole disk image in node's drvier11:29
*** ohamada has joined #openstack-ironic11:29
livelacejiangfei, https://www.raymond.cc/blog/move-windows-xp-hard-drive-or-change-motherboard-without-getting-blue-screen-of-death/11:33
livelaceLike this11:33
livelaceThe same procedure need to do with windows 200811:34
jiangfei livelace, thank you very much.11:34
*** links has quit IRC11:35
*** makowals has quit IRC11:38
*** makowals has joined #openstack-ironic11:39
*** makowals has quit IRC11:39
*** makowals has joined #openstack-ironic11:40
*** ccamacho is now known as ccamacho|lunch11:42
*** links has joined #openstack-ironic11:47
*** jvgrant has joined #openstack-ironic11:57
rbartalHi 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 dtantsur12:01
dtantsurMorning/afternoon Ironic12:01
sambettsmorning dtantsur12:02
rbartaldtantsur,  good morning12:04
openstackgerritSam Betts proposed openstack/ironic-inspector: Add generic LLDP processing hook  https://review.openstack.org/32108212:06
*** raildo-afk is now known as raildo12:07
*** jvgrant has quit IRC12:07
*** dprince has joined #openstack-ironic12:07
*** yolanda has joined #openstack-ironic12:08
* dtantsur dives into updating the driver composition spec12:08
*** krtaylor has quit IRC12:11
*** krtaylor has joined #openstack-ironic12:15
openstackgerritBharath kumar proposed openstack/ironic: adding back the inspect_ports as an optional parameter  https://review.openstack.org/32523012:16
*** ifarkas has quit IRC12:18
*** ifarkas has joined #openstack-ironic12:19
xavierrgood morning Ironic12:21
xavierrmorning vdrok TheJulia jiangfei rbartal dtantsur12:21
vdrokmorning xavierr and dtantsur12:21
rbartalgood morning xavierr12:22
*** ccamacho|lunch is now known as ccamacho12:25
dtantsurmorning xavierr, vdrok12:26
rbartaldtantsur, 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 nodes12:29
rbartaldtantsur, I may be missing some info here but I think the format it not that 'weird'12:30
*** jjohnson2 has joined #openstack-ironic12:31
dtantsurI'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 IRC12:33
rbartaldtantsur,  just tried to understand the differences12:36
*** marios_ is now known as marios12:37
*** daemontool has joined #openstack-ironic12:40
*** rbartal has left #openstack-ironic12:43
*** baoli has joined #openstack-ironic12:43
openstackgerritMerged openstack/pyghmi: Convert data tuple to a list  https://review.openstack.org/33204612:46
*** links has joined #openstack-ironic12:46
*** trown|outtypewww is now known as trown12:47
openstackgerritJarrod Johnson proposed openstack/pyghmi: Add support for PyCA cryptography  https://review.openstack.org/33182812:47
*** baoli_ has joined #openstack-ironic12:49
openstackgerritJarrod Johnson proposed openstack/pyghmi: Add support for PyCA cryptography  https://review.openstack.org/33182812:50
openstackgerritDmitry Tantsur proposed openstack/ironic-specs: The driver composition reform  https://review.openstack.org/18837012:50
dtantsursambetts, lucas-afk and everyone online ^^^12:50
dtantsurI think it got simpler after we've removed the hardware_type field12:50
sambettsdtantsur: \o/ simpler == better12:51
sambettsstill a 600+ line spec though heh12:51
*** snecklifter has quit IRC12:51
openstackgerritJarrod Johnson proposed openstack/pyghmi: Implement Python3 compatibility  https://review.openstack.org/31301212:51
dtantsursambetts, well... it could be 200 lines, but then everyone started asking a question on every line :)12:52
*** baoli has quit IRC12:52
sambetts;) hahaha yup best to cover all the bases12:52
*** hkominos has joined #openstack-ironic12:54
hkominoshello 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
hkominosHowever It never comes backonline and in fact the machine gets killed. It cannot even gets by a post message in the bios12:55
*** nico-afk is now known as nicodemos12:55
hkominos(second server i 'destroy' this way)12:55
*** jcoufal has joined #openstack-ironic12:56
*** cdearborn has joined #openstack-ironic12:57
*** edand has joined #openstack-ironic13:01
*** snecklifter has joined #openstack-ironic13:02
sambettshkominos: does the server give you an error message in the post output?13:04
hkominosnope.13:04
hkominosit just does not even come up.13:04
hkominosThere 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 process13:05
hkominosthe ROMS etc13:05
sambettsUnless you've configured custom cleaning steps that touch a machines ROMs Ironic won't touch anything like that13:08
jrollmorning everyone13:10
sambettsmorning jroll13:10
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add test for ironic port-list command  https://review.openstack.org/32906813:10
dtantsurmorning jroll. I've updated the driver composition spec, should be a nice addition to your first cup of coffee :D13:12
jrolldtantsur: second!13:12
jroll:)13:12
vdrokmorning jroll13:12
jrollmat128: good fine re https://raw.githubusercontent.com/openstack/neutron/master/doc/source/devref/images/fullstack_multinode_simulation.png13:12
*** hkominos has quit IRC13:12
jrolls/fine/find/13:12
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add test for ironic node-list command  https://review.openstack.org/32905613:14
*** afaranha has quit IRC13:15
*** snecklifter has quit IRC13:16
milanjroll, dtantsur morning :)13:18
dtantsurmorning milan13:19
mgouldmorning jroll dtantsur13:20
*** rloo has joined #openstack-ironic13:22
*** links has quit IRC13:23
*** joprovost has joined #openstack-ironic13:24
*** e0ne has quit IRC13:27
*** e0ne has joined #openstack-ironic13:31
openstackgerritJarrod Johnson proposed openstack/pyghmi: Implement Python3 compatibility  https://review.openstack.org/31301213:33
openstackgerritMerged openstack/python-ironic-inspector-client: Switch to keystoneauth  https://review.openstack.org/33189313:35
jroll^^ nice13:35
*** sdake has joined #openstack-ironic13:36
*** derekh has quit IRC13:36
mat128morning jroll dtantsur milan13:39
dtantsurmorning mgould, mat12813:39
mat128jroll: yeah, found that looking at the multinode stuff for neutron13:40
milanmorning mat128 :)13:40
*** e0ne has quit IRC13:41
*** yolanda has quit IRC13:43
*** PollyZ has joined #openstack-ironic13:46
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c  https://review.openstack.org/31844013:47
openstackgerritJarrod Johnson proposed openstack/pyghmi: Add support for PyCA cryptography  https://review.openstack.org/33182813:47
*** openstackgerrit has quit IRC13:48
jrolldims: I can't tell you how much I appreciate you running those tests <313:48
dimsjroll : thanks :)13:48
*** openstackgerrit has joined #openstack-ironic13:48
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add test for ironic driver-list command  https://review.openstack.org/33215013:50
*** bharathk has quit IRC13:50
*** e0ne has joined #openstack-ironic13:51
*** rbartal has joined #openstack-ironic13:53
jrolldtantsur: +2 again, thanks13:53
jrollbbiab, going to eat before we start (in 65 minutes)13:54
*** ametts has joined #openstack-ironic13:55
dtantsur\o/13:56
*** thrash is now known as thrash|biab13:57
*** rbartal has quit IRC13:59
milandtantsur, 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
dtantsurmilan, 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
milandtantsur, lol, OK thx14:00
* milan assembling a pull request14:00
*** joprovost1 has joined #openstack-ironic14:00
dtantsurcool14:00
dtantsurmilan, just make sure it tears down cleanly, as it's not tested in gate14:01
milandtantsur, I'll see in my grenade log, best I can do I guess14:02
*** joprovost has quit IRC14:02
*** joprovost1 is now known as joprovost14:02
*** PollyZ_ has joined #openstack-ironic14:02
milandtantsur, actually, better to make it a separate patch from my grenade stuff, correct?14:02
milan(smaller, faster to land)14:03
dtantsuryes14:03
* milan cool on it14:03
dtantsurjroll, JayF, wanna see the IPA lldp patch https://review.openstack.org/#/c/320584/ before it gets approved?14:03
*** jaybeale has joined #openstack-ironic14:03
*** PollyZ_ has quit IRC14:05
*** PollyZ has quit IRC14:05
*** PollyZ has joined #openstack-ironic14:07
rloohi and morning everyone, milan, dtantsur, jroll, mat12814:07
mat128hi rloo14:07
milanmorning rloo :)14:08
dtantsurmorning rloo! the driver composition spec is ready for your attention :)14:08
rloodtantsur: wrt driver composition, you write that the node.driver field can accept dynamic drivers.14:08
rloodtantsur: am reading it now :)14:08
dtantsurtrue14:08
rloodtantsur: but your defn of dynamic driver is14:08
rlooa 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 issues14:08
rloodtantsur: isn't it the value for a hardware type?14:08
dtantsurrloo, I should have said hardware types probably...14:08
* dtantsur is too tired of this text :D14:08
dtantsurplease read "hardware types" there14:09
rloodtantsur: ok. wanted to be sure. i'll point it out.14:09
*** jaybeale has quit IRC14:09
mat128hardware types == dynamic drivers :)14:09
rloodtantsur: i am sorry, i know, it is hard to rewrite again and again. thank you for doing it!14:09
dtantsurmat128, roughly, yes. but the point rloo makes is valid nonetheless14:09
dtantsurrloo, no problem, as soon as we finish it reasonable soon :)14:10
mat128dtantsur: yup14:10
NobodyCamgood morning Ironicers14:11
vdrokmorning rloo and NobodyCam14:12
rloohi vdrok, morning NobodyCam14:12
*** moshele has quit IRC14:14
NobodyCammorning vdrok and rloo :)14:14
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: devstack/plugin.sh: use screen_stop  https://review.openstack.org/33216414:17
milanmorning NobodyCam :)14:17
dtantsurmorning NobodyCam14:18
NobodyCammorning milan and dtantsur :)14:18
openstackgerritJarrod Johnson proposed openstack/pyghmi: Implement Python3 compatibility  https://review.openstack.org/31301214:22
mgouldmorning NobodyCam14:22
NobodyCammgould: morning :)14:22
*** joprovost has quit IRC14:23
*** rajinir has joined #openstack-ironic14:25
rajinirthiagop: thanks, let me know14:26
*** itamarl has quit IRC14:29
*** snecklifter has joined #openstack-ironic14:29
*** joprovost has joined #openstack-ironic14:30
rloodtantsur: wrt driver composition, line 396. 'Extend the ``driver-list`` command with ``--no-classic`` and ``--no-dynamic``'14:34
rloodtantsur: the REST API is 'classic' and 'dynamic'14:34
rloodtantsur: why the difference?14:34
dtantsurrloo, for usability purpose. or we'll have to have --classic=0 and --dynamic=0 to follow API14:35
dtantsurI feel like flags work a bit different from API parameters14:35
rloodtantsur: seems confusing.14:35
rloodtantsur: why not have --classic and --dynamic default to 114:35
dtantsurbecause --classic=0 is ugly and not how people do it14:36
rloodtantsur: but having 'no-X' is also ugly.14:36
dtantsurrloo, well, that's pretty standard anyway14:36
rloodtantsur: it is?14:36
dtantsurimagine ./configure scripts :) they usually have tons of --no- --with- and --without14:37
dtantsuron the other hand I can't think of any script using an explicit value for binary. can you?14:37
dtantsurs/binary/boolean/ ofc14:37
rloodtantsur: i'm not concerned with scripts, i'm concerned with the CLI.14:37
dtantsursorry, I don't get the difference. aren't scripts CLI?14:38
dtantsuranyway, could you please show me an example of your approach anywhere?14:38
rloodtantsur: i don't want to discuss what scripts are.14:38
rloodtantsur: i looked through our OSC, and we don't have any 'no-X' options.14:38
rloodtantsur: will take a look later wrt OSC ones.14:38
dtantsurmaybe, but I don't see any reasonable alternatives14:39
dtantsuri.e. I'm not pleased with it as well, just dunno what would be better14:39
rloodtantsur: maybe it should be --classic-only, --dynamic-only. not sure. we can change it later.14:40
mat128rloo, dtantsur: spec could be open to actual wording, simply saying "will allow filtering classic and/or dynamic drivers"14:40
mat128err that didnt sound too well14:41
rloomat128: yup. this is API related, so it is better to put down something than nothing.14:41
mat128spec could avoid mentioning actual wording, leaving that open to discussion later14:41
dtantsurfor CLI - maybe, but not for API, I guess14:41
mat128correct14:41
rloomat128: this is dmitry, so it is fine, but i get push back when i suggest changes at the code-patch-level.14:41
rloomat128: 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
mat128IMO, we should be open to better suggestions down the road even if it deviates from original plan14:42
dtantsurspecs can be changed, that's true :)14:42
mat128yeah I understood it that way14:42
mat128APi could return a list of drivers and a type14:42
mat128filtering done client side14:42
rloomat128: i agree with you.14:42
dtantsurmat128, +100. but rloo is right: not everyone is so easy on changing their apporved specs14:42
openstackgerritVladyslav Drok proposed openstack/ironic: Added operator documentation for ironic portgroups  https://review.openstack.org/22849614:42
openstackgerritVladyslav Drok proposed openstack/ironic: Add multitenancy-related fields to port API object  https://review.openstack.org/20624414:42
openstackgerritVladyslav Drok proposed openstack/ironic: Create common neutron module  https://review.openstack.org/31739014:42
openstackgerritVladyslav Drok proposed openstack/ironic: Add network_interface node field DB, OBJ  https://review.openstack.org/31739114:42
openstackgerritVladyslav Drok proposed openstack/ironic: Add 'neutron' network driver  https://review.openstack.org/31739314:42
openstackgerritVladyslav Drok proposed openstack/ironic: Update the deploy drivers with network flipping logic  https://review.openstack.org/21326214:42
openstackgerritVladyslav Drok proposed openstack/ironic: Add network_interface API  https://review.openstack.org/31739214:42
openstackgerritVladyslav Drok proposed openstack/ironic: Add network_drivers  https://review.openstack.org/28585214:42
openstackgerritVladyslav Drok proposed openstack/ironic: Add portgroups to support LAG interfaces - API  https://review.openstack.org/33217714:42
dtantsurboom!14:42
dtantsur:)14:42
rloomat128:  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 IRC14:42
*** Romanenko_K has quit IRC14:42
dtantsurmat128, filtering on client side is a bit exclusive towards people not using ironicclient14:42
*** kromanenko__ has quit IRC14:43
rloodtantsur, mat128: also, i was thinking, when we deprecate classic, we'll have to deprecate the API stuff too.14:43
dtantsurdo we have a plan to deprecate chassis?14:43
rloomat128, dtantsur: yeah, I don't think we want to put the filtering burden on our users.14:43
dtantsurI thought it's something about V2 ;)14:43
rloodtantsur: have we decided to deprecate chassis? ha ha.14:43
dtantsurrloo, 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 comments14:44
dtantsurthis is because this API part is probably the least important bit of the whole spec :)14:45
rloodtantsur: gotcha.14:45
vdrokhuh, is gerrit alive?14:45
TheJuliaPart of a page just loaded for me14:46
vdrokI can't get further than Loading Gerrit Code Review ...14:47
*** yolanda has joined #openstack-ironic14:47
vdrokoh, working now14:47
*** lucas-afk is now known as lucasagomes14:48
* lucasagomes is not sure about deprecating API bits... IIRC we won't get rid of anything already exposed by the api14:48
*** praneshp has joined #openstack-ironic14:49
jrollyeah, I don't see what we would need to deprecate in the API14:49
jrollit just won't return any classic drivers, because they won't be loaded14:49
*** thrash|biab is now known as thrash14:51
dtantsurrloo, thanks for all the comments. I'll check them and try to follow-up on everything needing clean up.14:54
dtantsurNobodyCam, TheJulia, wanna check the driver composition too?14:55
rloodtantsur: yw. nothing serious, i think by tomorrow, we should +A unless there are objections.14:55
NobodyCamwill do :) just getting coffee all set up14:55
TheJuliadtantsur: already loaded in my web browser :)14:55
rloojroll, 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
jrollrloo: enjoy :)14:56
openstackgerritVladyslav Drok proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643214:56
*** hshiina has joined #openstack-ironic14:57
vdrokI need to get some food, will be ~30 min late too14:57
openstackgerritJarrod Johnson proposed openstack/pyghmi: Implement Python3 compatibility  https://review.openstack.org/31301214:58
openstackgerritJarrod Johnson proposed openstack/pyghmi: Implement Python3 compatibility  https://review.openstack.org/31301214:58
*** yuriyz|2 has joined #openstack-ironic14:59
*** makowals has quit IRC15:00
jrollsambetts: around yet? :)15:00
jlvillalI have a conflicting meeting for the next 30 minutes :(15:00
*** rloo has quit IRC15:00
*** athomas has quit IRC15:01
sambettsjroll: just finishing up a call15:01
*** skramaja has quit IRC15:01
jrollsambetts: cool, I'm going to have you lead the first topic15:01
*** rloo has joined #openstack-ironic15:01
*** daemontool has quit IRC15:02
jrolljlvillal: tell them you're away at the ironic midcycle :)15:02
*** jaybeale has joined #openstack-ironic15:02
* jroll is treating this as travel15:02
mat12860hz buzz?15:02
jrollyep15:02
JayFthat's pretty awful15:03
dtantsur++ I usually ask people to assume I'm actually travelling15:03
*** rpioso has joined #openstack-ironic15:04
jrollbrb15:04
cdearbornstill getting the 60 hz buzz?15:05
sambettsnope15:05
cdearborngreat - was wondering if it was my cordless phone15:05
*** piet_ has joined #openstack-ironic15:06
mrda.15:06
mat128jroll: got a link?15:07
jrollmat128: I'll post it when it's open15:07
mat128oh, ok15:07
*** edand has quit IRC15:07
*** moshele has joined #openstack-ironic15:08
openstackgerritDevananda van der Veen proposed openstack/ironic: Mask password on agent lookup according to policy  https://review.openstack.org/33219515:10
openstackgerritDevananda van der Veen proposed openstack/ironic: Mask password on agent lookup according to policy  https://review.openstack.org/33219615:10
openstackgerritDevananda van der Veen proposed openstack/ironic: Mask password on agent lookup according to policy  https://review.openstack.org/33219715:11
*** PollyZ has quit IRC15:11
*** PollyZ has joined #openstack-ironic15:11
*** PollyZ has quit IRC15:11
jrolldtantsur: ^ reno coming as follow up15:11
dtantsuron it15:11
dtantsurjroll, devananda, I don't quite like that we don't have 'cherry picked from' lines in the commit messages15:12
jrolldtantsur: agree15:12
dtantsurdevananda, is it possible to quickly fix please? assuming we do land the same commit in master15:13
jrollI'll fix it15:13
dtantsurack15:13
dtantsurjroll, also +A on https://review.openstack.org/#/c/332195/ ?15:13
devanandaah - instead of the same commit-id. yeah.15:13
jrollin addition to the same change-id15:13
dtantsurin addtition15:14
devanandajust edit the commit message on the last two15:14
devanandauh, well15:14
devanandawe can't do that15:14
dtantsurwhy?15:14
devanandacherry-picked-from commit SHA is not known until it actually merges in master15:14
devanandagerrit could merge something else15:14
jrollthat sha is what will be merged unless the patch is modified15:14
dtantsurdevananda, I don't think so. I think it's the same as you have locally15:14
devanandano15:14
jrollwe use the commit, not the merge commit15:14
devanandaright15:14
jrollfor "cherry-picked from"15:14
dtantsurthe merge patch will be different, but gerrit does not rebase anything15:14
jroll^15:14
devananda...15:15
openstackgerritJim Rollenhagen proposed openstack/ironic: Mask password on agent lookup according to policy  https://review.openstack.org/33219615:15
lucasagomesdevananda, if you add "-x" to the git cherry-pick comamnd it will add that line for you15:15
devanandalucasagomes: it's not cherry picked15:15
devanandaactually15:15
openstackgerritJim Rollenhagen proposed openstack/ironic: Mask password on agent lookup according to policy  https://review.openstack.org/33219715:15
*** snecklifter has quit IRC15:15
devanandajroll: these are subtly different patches15:16
jrolldevananda: why aren't they cherry-picked?15:16
devanandaas you know, I had to make small changes to get it to merge15:16
jrollthey were when we were iterating them15:16
*** PollyZ has joined #openstack-ironic15:16
jrollright, cherry-picked but had conflicts15:16
dtantsurdevananda, that counts as a merge conflict15:16
devanandaright15:16
jrollwhich is still cherry-picked15:16
devanandaeh. ok15:16
lucasagomesyeah still cherry picked15:16
devanandawords :)15:16
lucasagomesare hard :D15:16
dtantsurok, the goal of this rule is for people to be able to find an original commit on master :)15:16
jrollok, I'll +A when these pass tests and submit releases15:17
*** jjohnson2 has quit IRC15:17
dtantsurthanks jroll, devananda for tracking it15:17
jrolloh wait, reno15:17
* jroll waits15:17
devanandareno coming ...15:17
dtantsurseparate patches?15:17
*** jjohnson2 has joined #openstack-ironic15:19
dtantsurdevananda, I think we need reno's in the same patches FWIW15:21
jrollI can squash once they're posted15:22
jrolleither way15:22
dtantsurah ok, +115:22
devanandajroll, dtantsur: https://gist.github.com/devananda/a668521889cc130e573ecf3224d6290c15:22
devanandaif that looks good, I'll squash and repost now15:22
jrolldevananda: +115:23
devanandaI didn't want to hold up the sec patch if folks disagreed on wording in the reno ...15:23
openstackgerritZhenguo Niu proposed openstack/ironic: Clear target_power_state on conductor startup  https://review.openstack.org/31009015:23
jrollheh, yeah15:23
devanandasince we didn't think to review that ahead of time15:23
devanandabut if you both like ^ I'll just push it15:23
jrollyeah, it's a bummer :(15:23
jrollI'm good with it15:23
TheJuliadevananda: +115:23
devanandadtantsur: ?15:23
dtantsurdevananda, it mentions "this request" before the actual request, but it's fine with me15:24
devanandajroll: reno question - can it have the same file name on different branches?15:24
jrolldevananda: yes15:24
dtantsurdevananda, it must15:24
*** piet_ has quit IRC15:24
devanandak. thought so but hadn't done it before15:24
jrolldevananda: it's what makes backports work :)15:24
lucasagomesdevananda, gist text looks good15:25
devanandajroll: ah, of course15:25
*** e0ne has quit IRC15:25
devanandadtantsur: ah. I can reverse those two sentences then15:26
*** e0ne has joined #openstack-ironic15:26
openstackgerritDevananda van der Veen proposed openstack/ironic: Mask password on agent lookup according to policy  https://review.openstack.org/33219515:27
* dtantsur has a cat limiting his review abilities15:27
*** Nisha_away has joined #openstack-ironic15:27
devanandacherry pick lines change now ...15:29
dtantsuryep15:29
openstackgerritDevananda van der Veen proposed openstack/ironic: Mask password on agent lookup according to policy  https://review.openstack.org/33219615:30
TheJuliaheh, whitespace15:30
*** ohamada has quit IRC15:30
*** ohamada has joined #openstack-ironic15:30
openstackgerritDevananda van der Veen proposed openstack/ironic: Mask password on agent lookup according to policy  https://review.openstack.org/33219715:30
*** penick has quit IRC15:31
* devananda starts to get ready for dockercon day two15:32
* devananda really wishes he could both be at dockercon and the midcycle15:32
jrolldevananda: everything is approved, thanks so much <315:32
dtantsurjroll, thanks for approving, the f***ing cat tried to prevent me from using the mouse by scratching me >_<15:33
jrolllol15:33
jrollthe cat is in for a bad time if she gets on my desk :)15:33
devanandaheh15:33
TheJuliaspeaking of cats... mine has decided she HAS to be on my desk for the first time in months15:34
dtantsurthat's not quite my cat, that's the problem. *my* cat would have a lot of troubles15:34
dtantsur:D15:34
jlvillalMeeting finished and now I am lurking... :)15:34
*** e0ne has quit IRC15:34
*** pcaruana has quit IRC15:35
TheJuliastable ipa vs master ironic15:36
*** lintan_ has joined #openstack-ironic15:36
TheJuliaoh wait, now my brain is broken15:36
* dtantsur could not follow15:36
JayFGrenade tests stable/mitaka IPA vs master Ironic15:37
JayFwe need a job to test master IPA vs stable/mitaka Ironic15:37
JayFto guarantee N-1 support in IPA15:37
JayFdtantsur: ^15:37
*** piet has joined #openstack-ironic15:37
mat128JayF: added in etherpad, feel free to update15:38
*** rbudden has joined #openstack-ironic15:38
dtantsurJayF, or just run grenade?15:38
JayFdtantsur: grenade doesn't upgrade IPA.15:38
*** amotoki has joined #openstack-ironic15:38
JayFdtantsur: the grenade job does stable/mitaka Ironic + stable/mitaka IPA, only upgrades Ironic to master, leaves IPA at stable/mitaka15:38
dtantsuraha, true15:38
dtantsurwhich is not a realistic scenario imo15:38
JayFso the only combo missed is IPA master vs Ironic stable15:38
dtantsurok, got it15:39
JayFI mean, I think it's possibly realistic too15:39
*** jjohnson2_ has joined #openstack-ironic15:39
JayFlike I said in chat, we've been deploying Ironic + IPA almost entirely indepedendently the whole existance of OnMetal15:39
mat128same here15:39
mat128IPA always in advance15:39
JayFI think right now I actually have some envs with newer IPA and some with older (don't ask, lol)15:40
mat128lol15:40
JayFbut 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 much15:40
dtantsurI see15:40
dtantsuryeah, ideally we should have 2 upgrade jobs: with and without upgrading IPA..15:41
*** jjohnson2 has quit IRC15:42
openstackgerritMoshe Levi proposed openstack/ironic-inspector: Replace assertRaisesRegexp with assertRaisesRegex  https://review.openstack.org/33221815:43
jjohnson2_lucasagomes, fyi, review.openstack.org/#/c/313012/ is looking promising15:44
*** snecklifter has joined #openstack-ironic15:45
lintan_Ironic is using number version, but IPA only using name version?15:47
*** sukhdev has joined #openstack-ironic15:47
jrolllintan_: both use numbered versions, both use named versions15:48
jrolllintan_: if we test named versions, everything in between those named versions should work15:48
*** amotoki has quit IRC15:48
lintan_oh15:48
lintan_thanks15:48
jroll:)15:48
lucasagomesjjohnson2_, w00t !15:49
lucasagomesthanks for that15:49
jlvillaljjohnson2_, Any reason to not use the 'six' library?15:50
*** snecklifter has quit IRC15:51
jlvillaljjohnson2_, https://pythonhosted.org/six/15:51
openstackgerritZhenguo Niu proposed openstack/ironic: Add description filed to ports - DB  https://review.openstack.org/32464415:51
jrollman, zuul is nice and backed up today :(15:53
*** sacharya has joined #openstack-ironic15:53
rloojroll: good thing we're in midcycle and not using zuul that much :)15:54
jrollrloo: except our patches for a CVE :(15:54
rloojroll: OH. sigh.15:54
zigoIs proliantutils 2.1.8 compatible with Py3 these days?15:54
jrollrloo: in case you missed it, https://review.openstack.org/#/q/topic:fix-157279615:54
*** dmk0202 has joined #openstack-ironic15:54
zigoI'd like to make a new package with py3 support if it does.15:54
JayFzigo: that's a vendor library, I'm unsure :/15:54
zigoCheers.15:55
jrollzigo: /me looking15:55
zigoThere's no metadata for supported py version.15:55
jrollzigo: they run unit tests on 3.4 https://review.openstack.org/#/c/326319/15:55
zigoAh, thanks.15:55
jrollzigo: that's as close as I can get you, otherwise bug Nisha_away or maybe NobodyCam / TheJulia could find out easily15:56
zigook15:56
zigoThanks a lot.15:56
jrollyeah, np15:56
Nisha_awayjroll, whats this about15:57
jrollNisha_away: "Is proliantutils 2.1.8 compatible with Py3 these days?"15:57
Nisha_awayyes15:57
jrollzigo: affirmative :):15:57
Nisha_awayits py3 compatible15:57
jroll:) rather15:57
JayFNisha_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_awaywe run py3 gate for proliantutils15:57
Nisha_awayJayF, zigo please add on some more details...15:58
zigoNisha_away: Any plan on testing for py3.5 as well?15:58
zigoDebian Sid and Xenial are both using Py3.515:58
Nisha_awaywe can add that15:59
zigoThat's be nice.15:59
Nisha_awayas of now it runs gate for py3.415:59
rlootaking break for 15 minutes15:59
Nisha_awayzigo, could you add enhancement bug for proliantutils15:59
*** bharath has joined #openstack-ironic15:59
Nisha_awaywe will take it fwd15:59
*** Nisha_away is now known as Nisha116:00
*** yolanda has quit IRC16:04
*** snecklifter has joined #openstack-ironic16:04
-openstackstatus- NOTICE: Gerrit is being restarted now to apply an emergency security-related configuration change16:05
*** piet has quit IRC16:09
jrollhave to run out for 20 minutes suddenly, maybe less, dtantsur can you intro/moderate the raid discussion?16:11
dtantsurjroll, will do16:12
*** jvgrant has joined #openstack-ironic16:12
*** praneshp has quit IRC16:14
rloostarting now, line 150: https://etherpad.openstack.org/p/ironic-newton-midcycle16:16
*** trown is now known as trown|lunch16:17
*** jaybeale has quit IRC16:17
*** jaybeale has joined #openstack-ironic16:18
*** piet has joined #openstack-ironic16:18
*** lintan_ has quit IRC16:19
*** Sukhdev_ has joined #openstack-ironic16:20
*** Sukhdev_ has quit IRC16:21
*** Sukhdev_ has joined #openstack-ironic16:22
*** dmk0202 has quit IRC16:22
*** lintan_ has joined #openstack-ironic16:24
lintan_how many drivers support raid config? I think only ilo support right?16:26
cdearborndrac supports as part of vendor passthru16:27
*** edand has joined #openstack-ironic16:27
ifarkasdrac is in-progress and I think ucs is also in-progress16:27
lintan_oh, thanks :)16:28
TheJulialintan_: the hpe hardware support for raid through manual cleaning is essentially driver independent16:28
TheJulialintan_: works via ipmi and ilo16:28
jlvillalI also don't like calling it cleaning. But will remain quiet on audio :)16:28
TheJuliayeah, this is not cleaning, it is deployment preparation16:29
lintan_cool, TheJulia16:29
*** amotoki has joined #openstack-ironic16:30
*** links has joined #openstack-ironic16:30
TheJulialintan_: it is done via a hardware manager16:31
*** e0ne has joined #openstack-ironic16:32
cdearbornthe bios and raid config that need to be applied depends on what role you want the node to take: controller, compute, storage16:33
cdearbornand you only know that just prior to deployment16:33
cdearbornnot when a node is being deleted16:33
openstackgerritMoshe Levi proposed openstack/ironic-inspector: Replace assertRaisesRegexp with assertRaisesRegex  https://review.openstack.org/33221816:33
*** mgould is now known as mgould|afk16:34
TheJuliacdearborn: 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 name16:36
jlvillalTheJulia, Joe's last name is Sixpack ;)16:37
TheJulialol16:37
TheJuliaI'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 IRC16:38
jlvillalheh16:38
mat128TheJulia: haha16:38
*** baoli_ has quit IRC16:38
*** david-lyle has joined #openstack-ironic16:38
TheJuliaAt least, conceptually that is based upon watching the ilo hardware manager setup raid config which was pretty much a momentary thing once IPA heartbeated16:41
jrollsorry, I'm back now :|16:44
jrollmore like 40 minutes16:44
rloojroll: yup, we were timing you!16:45
rloojroll: we're talking now about how to get raid info from user -> nova -> ironic16:46
rloojroll: line 158 of etherpad16:46
jrollyep16:46
rloojroll: we've all agreed that we want this feature :)16:46
jrollshocking!16:47
* jroll lets dog out quickly16:47
*** piet has quit IRC16:47
rloojroll: 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
jrollrloo: that was a primary argument16:48
rloojroll: do you recall the other arguments? I mean, assuming user is willing to wait...16:48
*** fragatina has quit IRC16:48
jrollrloo: dunno, probably nova things16:48
rloojroll: 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
JayFI 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 config16:50
jlvillaljjohnson2_, Okay. I'm just used to using six as that is what we do in all of our projects.16:50
*** sacharya has quit IRC16:51
* devananda is sort of here16: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-ironic16:52
rloodevananda: take a look at etherpad, from line 170 I guess https://etherpad.openstack.org/p/ironic-newton-midcycle16:52
*** catintheroof has joined #openstack-ironic16:53
*** piet has joined #openstack-ironic16:53
jlvillaljjohnson2_, I think the six library works with Python 2.5 and newer. Not 100% sure on that.16:53
jlvillalBut I see the docs mention Python 2.516:53
TheJuliaI think they would always need to be operator defined16:53
dtantsurjroll, the liberty patch failed due to horizon problem.. asking qa folks to fast-track it16:53
jjohnson2_jlvillal, yeah, not directly related, just another example of things seemingly not the case in openstack in general16:54
*** yuriyz|2 has quit IRC16:54
jlvillaljjohnson2_, Understood :)16:54
openstackgerritMoshe Levi proposed openstack/ironic-python-agent: tox: Update flake8 to ignore imagebuild folder  https://review.openstack.org/33224716: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.... hurray16:54
TheJuliasambetts: exactly what you were saying why I'm saying it would always need to be operator defined16: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.616:56
jlvillaljjohnson2_, :(16:56
openstackgerritMoshe Levi proposed openstack/ironic-python-agent: Replace assertRaisesRegexp with assertRaisesRegex  https://review.openstack.org/33225016:57
*** amotoki has quit IRC16: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 boost16:57
jjohnson2_I love it when we have a freshly deployed pre-production system, some leeway to measure  and optimize16:58
openstackgerritJarrod Johnson proposed openstack/pyghmi: Implement Python3 compatibility  https://review.openstack.org/31301217:00
dtantsurlucasagomes, jroll, the horizon fix was approved for liberty17:02
lucasagomesw00t!17:02
*** e0ne has quit IRC17:02
jrolldtantsur: horizon fix?17:02
dtantsurjroll, "can't bind port"17:03
jrolldtantsur: oh, turns out it was a zuul problem, not horizon :)17:03
dtantsurjroll, aka https://review.openstack.org/#/c/330846/17:03
*** penick has joined #openstack-ironic17:03
dtantsurwell, ok :) I'm still referring to it like that17:03
jrolldtantsur: they had also switched zuul's port so it was fixed ahead of time, but glad that landed17:03
JayFdtantsur: mat128: http://specs.openstack.org/openstack/ironic-specs/specs/backlog/exposing-hardware-capabilities.html17:04
lintan_One more question, do we need for raid API anymore?17:05
* JayF brb17:05
jrolllintan_: I think so, that allows operators to configure a complex raid ahead of time17:05
lintan_but the raid config is passing from nova to instance_info in current solution17:07
lintan_this is kind of duplicate17:07
jrollkind of, yes17:07
*** dprince has quit IRC17:07
rloolintan_: yes, for RAID to do predeployment17:08
rloowe aren't getting rid of configuring RAID in manual cleaning, right?17:08
jrollright17:08
rloosome people might still want to do RAID before making them available.17:08
lucasagomesrloo, right, I think we can not17:08
lintan_but node already have raid field17:10
rloolintan_: yes, true. i would like whoever is putting together the spec, to take these in consideration.17:11
rlooi don't think we can write the spec and get it right, in 60 min :)17:11
rloobreak for 17 min.17:13
sambettsmat128: 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
mat128nova boot --metadata key=value17:15
mat128we just read a magic key17:16
mat128in theory17:16
dtantsurjroll, did they change the zuul port? cause our cve just failed with the same failure to restart horizon...17:16
mat128metadata is something for the tenant to use on their own17:16
dtantsurs/horizon/apache/ ofc17:16
JayFdtantsur: what branch?17:16
mat128but other than a scheduler hint, we didnt see any other way of feeding data to nova17:16
dtantsurJayF, liberty17:16
JayFdtantsur: I think our fix only very recently landed there, if it has at all17:16
JayFdtantsur: it's possible the old value is cached, can you link the logs/17:17
JayFor the patch17:17
dtantsurJayF, our was just approved, but jroll says they changed it on zuul side..17:17
mat128sambetts: not sure if this is clear enough :(17:17
mat128I'll check if the doc is public yet17:17
dtantsurok, taking a break for real anyway17:17
JayFthe zuul side change, I was told, will take week+ to take effect17:17
JayFthat's why we changed it on our side too17:17
*** piet has quit IRC17:17
dtantsurah, got it17:17
*** jvgrant has quit IRC17:17
JayFso until like, friday, just assume infra is still breaking us unless that change is in place17:17
sambettsmat128: ah ok I see it now, http://docs.openstack.org/cli-reference/nova.html the nova boot cli ref lists it as --meta key=v17:17
sambettsmat128: have you been following the spec I have up for the networking stuff?17:18
jrolldtantsur: wat O_O17:18
jrolloh right17:18
jrollwhomp whomp17:18
mat128sambetts: the neutron integration stuff?17:18
sambettsmat128: yeah, the spec for VLAN Aware BMs17:19
mat128oh, attach/detach?%17:19
sambettsand that17:19
mat128yeah17:19
jrollman grenade is loooooong17:19
mat128following it all17:19
mat128but we already do all of this17:19
sambettsmat128: cool, hopefully it'll help you carry less code downstream :)17:19
mat128sambetts: yeah :P17:19
mat128and less ugly17:20
mat128right now attach/detach I think is driver_passthru17:20
sambettsmat128: I did exactly the same http://git.openstack.org/cgit/openstack/cisco-ironic-contrib/tree/cisco_ironic_contrib/nova/driver.py#n3317:20
sambettsmat128: I created an add_vnic and delete_vnic vendor passthru17:21
*** PollyZ has quit IRC17:21
mat128ours a bit longer than this17:22
mat128hmm17:22
mat128ah I see17:22
mat128we have len(vif) != len(pif)17:22
mat128I wanted to ask yesterday17:22
mat128deriving lacp from vif > pif doesnt sound like a good idea17:22
mat128I am offering my customers bonding even if they only have 1 vif17:22
sambettsmat128: that is also supported17:23
sambettswe don't derive LACP from number of vifs we only detrive trunks17:23
mat128ah17:23
mat128interesting17:23
*** electrofelix has quit IRC17:23
mat128but pif/2 when you're doing LACP?17:23
sambettsbonds are all defined in portgroups on the ironic side17:23
mat128oic17:24
mat128so they are "one port"17:24
sambettsyup17:24
*** sdake_ has joined #openstack-ironic17:24
mat128interesting17:24
sambettsso 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-ironic17:25
mat128yeah, got it17:25
*** rbradfor has quit IRC17:25
jrollsambetts: what happens when you want to add a vif later, everything gets reshuffled or?17:26
mat128oh man17:26
sambettsjroll: currently round robin I guess17:26
mat128jroll: good point17:26
jrollsambetts: I mean17:26
jrollif it isn't trunked17:26
mat128attaching makes it a trunk17:26
jrolland then you add a vif, and it suddenly needs to be trunked17:26
mat128you lose connectivity17:27
*** sdake has quit IRC17:27
sambettswe'd need an agent to configure the networking inside the instance17:27
mat128we do trunking during all active-backup and lacp scenarios17:27
jrollbesides the configuration17:27
mat128sambetts: ah17:27
jrollwhat do we do control plane side, shuffle things around so that it becomes a trunk?17:27
sambettsI guess we'd have too, although I like the idea of making all ports a trunk, and just setting the native to the first vif17:28
*** trown|lunch is now known as trown17:28
mat128sambetts: what happens when you detach that vif?17:28
mat128in all trunk scenarios we have no native17:28
*** moshele has quit IRC17:28
sambettsmat128: native becomes free, and then the next vif to attach becomes native17:28
jrollsambetts: yeah, I like the everything is trunk model too17:29
*** sacharya has joined #openstack-ironic17:29
mat128interesting17:29
sambettsits down to the neutron drivers to handle all of this though17:29
*** ohamada has quit IRC17:29
sambettsits down to us to make sure the instance is configured right17:29
mat128sambetts: thats hard, if neutron decides but nova has to generate the right thing17:30
*** PollyZ has quit IRC17:30
openstackgerritAndre Aranha proposed openstack/python-oneviewclient: Refactored to_oneview_dict and init methods  https://review.openstack.org/33167317:30
sambettsmat128: thats why we don't support on the fly right now because we can't change things in the instance right now17:30
sambettsmat128: we'd need an agent in the instance that ironic can talk to to change the networking17:31
mat128sambetts: we do support it, but the customer has to change their own config17:31
mat128sambetts: we're agent-less17:31
sambettsmat128: kinda, right now nova attach-port is a no-op17:31
rloomidcycle starting17:31
sambettsmat128: needs digging into :D17:31
rlooline 221 https://etherpad.openstack.org/p/ironic-newton-midcycle17:31
rloosoftware RAID support in IPA17:32
*** dprince has joined #openstack-ironic17:32
JayFdtantsur: we use the crazy hybrid stuff (Intel RSTe raid)... main use case is you can setup raid using linux/md tooling that works on windows17:32
jrollmat128: yeah, that's how we do volume-attach right now, put iscsi details in metadata and they need to configure it17:32
*** ayoung has quit IRC17:33
mat128jroll: totally acceptable for an MVP / desire to run agent-less17:33
jrollmat128: yeah, it's a support nightmare though :(17:33
mat128oh17:34
sambettsmat128: the difficulty with networking though is that a tenant might not be able to reach the node to configure it17:34
*** baoli has joined #openstack-ironic17:34
sambettsmat128: unless OOB serial console17:34
*** Sukhdev_ has quit IRC17:34
mat128sambetts: if you lock yourself out: https://review.openstack.org/#/c/306074/17:34
mat128;)_17:34
jrollmat128: 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 gross17:34
jlvillalmat128, Regarding: https://review.openstack.org/#/c/331936/  I left a comment.17:38
mat128jlvillal: I agree :)17:38
openstackgerritMerged openstack/ironic-inspector: devstack/plugin.sh: use screen_stop  https://review.openstack.org/33216417:38
openstackgerritAparna proposed openstack/proliantutils: Sort values of raid_config for 'share_physical_disks'  https://review.openstack.org/32631917:39
*** pcaruana has joined #openstack-ironic17:40
*** sacharya has quit IRC17:41
JayFhttps://gist.github.com/jayofdoom/247f50bdff458945fb26b1f857aec7cb17:41
mrdathanks JayF17:42
rlooJayF: if you wouldn't mind, would you please lower your volume?17:42
JayFmy outgoing volume?17:42
jrollrloo: have you met jay? :)17:42
rlooJayF: yup17:42
JayFif I have a slider for it I will17:43
rloojroll: YES, so he is EXTRA LOUD! :)17:43
openstackgerritAndre Aranha proposed openstack/python-oneviewclient: Refactor of models class to_oneview_dict and init methods  https://review.openstack.org/33167317:43
JayFpersonal volume controls have been broken snice a young age :P17:43
jrollJayF: yes, that's better17:43
*** rcernin has quit IRC17:43
rlooJayF: much better thx!17:43
jrollty17:43
rloonow JayF seems almost normal ;)17:43
JayFyou sure you've actaully met me? :P17:44
mrdaI thought the volume control was binary for JayF? :)17:44
*** piet has joined #openstack-ironic17:45
*** jaybeale has quit IRC17:45
JayFmrda: luckily for you I'm transitioning through technology that can slow it down :)17:45
JayFs/slow/quiet/17:46
openstackgerritMerged openstack/ironic: Mask password on agent lookup according to policy  https://review.openstack.org/33219617:48
*** livelace has quit IRC17:49
*** PollyZ has joined #openstack-ironic17:52
JayFin windows it's usually called 'dynamic disks' not raid17:55
JayFand I don't think you can boot off of them at all17:55
mat128iirc you could17:55
*** spandhe has joined #openstack-ironic17:55
mat128hmm17:55
JayFMy windows knowledge is covered in dust17:55
mat128but I think it left a tiny boot part in the front17:55
mat128:P17:55
JayFso if that's newerish I'm not sure17:55
sambettsSpanned volume? http://www.howtogeek.com/howto/36504/how-to-create-a-software-raid-array-in-windows-7/ :/17:55
mat128you had to configure it after booting off the first sidk17:55
dtantsurfor UEFI we'll also need "a tiny boot part" fwiw17:56
JayFsambetts: most consumer windows now don't support 'dynamic disks' which are the 'real' server raid support for windows17:56
mat128what I remember about dynamic disks, if you convert, you can't go back :D17:56
vdrokmat128: this happened to me too :)17:57
openstackgerritOpenStack Proposal Bot proposed openstack/bifrost: Updated from global requirements  https://review.openstack.org/33227517:57
*** hshiina has quit IRC17:57
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/33229517:59
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/33229617:59
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/33229717:59
*** ijw has joined #openstack-ironic18:00
*** Nisha1 has quit IRC18:01
JayFFWIW you can trigger evaluate_hardware_support() off items in node as well18:02
JayFso you could have a tag that says "always_deploy_as_software" in the node object somewhere, find it in hardware support and react accordingly18:03
dtantsurhttps://github.com/openstack/ironic/blob/69d4fd93533ed473bbbfb55a6d0433d3718b6474/ironic/drivers/modules/agent.py#L56118:03
JayFdespite the name evaluate_hardware_support does not have to strictly make decisions based on hardware18:03
mat128http://docs.openstack.org/developer/ironic/deploy/raid.html18:04
mat128JayF: I like this idea, putting it in the node18:04
mat128not sure how it fits with driver composition tho18:04
JayFyeah, I mean, you can do all sorts of custom logic in hardware managers because the node gets passed in18:05
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironic-inspector-client: Updated from global requirements  https://review.openstack.org/33236718:05
*** ayoung has joined #openstack-ironic18:05
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/33236818:05
JayFyou could probably implement a RaidInterface that just calls out to hardware managers :)18:05
openstackgerritOpenStack Proposal Bot proposed openstack/python-oneviewclient: Updated from global requirements  https://review.openstack.org/33237418:05
sambettsI would see it as a operator choice e.g. node.raid_interface = AgentRAID or CIMCRAID or ILORAID18:06
mat128but AgentRAID is "inband raid" more than agent18:06
mat128CIMC is oob, right?18:06
jrollAgentRAID exists today, it calls the agent18:06
*** edand has quit IRC18:06
jrollbut upstream agent doesn't have a clean step to do this18:06
JayFhttps://github.com/jayofdoom/ipa-example-hardware-managers18:06
mat128ooh18:07
mat128if we were to land mdraid, LSI and Adaptec support, where would they fit?18:07
* dtantsur has no clue about these18:08
JayFall could go upstream18:08
mat128yes, but where18:08
mat128in the same hw manager?18:08
JayFno, new managers for each18:08
mat128and "determine support" would just bail out18:08
mat128hmm18:08
mat128and if I have all of these in my pool18:09
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: devstack/plugin.sh: use screen_stop  https://review.openstack.org/33240818:09
mat128I write a manager that uses all of them?18:09
JayFno18:09
JayFthe managers automatically load/unload based on available hardware18:09
mat128ah18:09
mat128problem solved then18:09
JayFif you haven't read the example managers I linked, it answers all of this18:09
mat128mdraid module has lower priority18:09
JayFthe example_device.py being the relevant example here18:09
JayFexactly18:09
openstackgerritJohn L. Villalovos proposed openstack/ironic: WIP: For as_dict() functions return a copy.deepcopy()  https://review.openstack.org/33241018:11
*** Nisha has joined #openstack-ironic18:12
JayF#agreed raid is hard18:13
JayFlol18:13
*** Nisha_away has joined #openstack-ironic18:14
mat128example 1 http://docs.openstack.org/developer/ironic/deploy/raid.html18:15
*** Nisha has quit IRC18:17
*** Nisha_away has quit IRC18:19
*** e0ne has joined #openstack-ironic18:19
*** pcaruana has quit IRC18:22
rloohttps://review.openstack.org/200496 ? volume connection info spec18:22
TheJuliarloo: jroll is talking about driver composition at the moment18:23
*** sukhdev has quit IRC18:23
rlooTheJulia: thx. took me a few to figure that out!18:23
rloobreak for 20 min. Talk to you soon.18:25
*** ifarkas has quit IRC18:26
lintan_Good night, guys18:26
openstackgerritVladyslav Drok proposed openstack/ironic: Add test to verify ironic multitenancy  https://review.openstack.org/26915718:26
*** lintan_ has quit IRC18:26
openstackgerritVladyslav Drok proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643218:27
mrdaI'm going to have to give up, can't stay awake.  See you tomorrow.18:27
dtantsursee you mrda, thanks for coming!18:29
dtantsurgood night mrda and lintan18:29
TheJuliagoodnight!18:29
dtantsurrloo, 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-ironic18:35
jlvillalSo am I being foolish on: https://review.openstack.org/#/c/332195/2/ironic/drivers/modules/agent_base_vendor.py18:35
*** links has quit IRC18:36
rloodtantsur: yeah, type is better. something other than type would even be better, but i can't think of anything now :)18:37
rloodtantsur: will look through his comments today.18:37
*** milan has quit IRC18:37
rloonight mrda and lintan. thx for attending!18:37
dtantsuryes please18:37
* dtantsur is tired of this spec and probably said it several times already :)18:37
rloojlvillal: it might not be 'ipmi_password', could be other '*password*' I think that .mask_password looks for several strings.18:39
rloojlvillal: but i guess a comment would help :)18:39
*** ChubYann has joined #openstack-ironic18:39
rloojlvillal: i believe we/ironic already does a similar thing in code elsewhere.18:39
jrolljlvillal: it returns a string '{"foo": "bar"}' is why, and yes we have the same code elsewhere18:40
dtantsuryeah, the problem is that we also can have ilo_password, etc18:40
jrolland also covers many things besides ipmi_password18:40
jlvillalYeah, 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't18:40
*** PollyZ has quit IRC18:40
jrolloslo now has differently. Really mask_dict_password should do what we want, but doesn't18:40
jroller18:40
jroll...18:40
jrollmask_dict_password is what we really want. it didn't exist at the time18:40
jrollI've been waiting for this code to drop to change it in a few places18: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
jlvillaljroll, Because it checks to see if 'ipmi_password' is in the list of strings18:41
jlvillaljroll, While mask_password() checks to see if each string in the list is in the field name.18:41
*** sacharya has joined #openstack-ironic18:41
jlvillalSo maks_password does subset checking. mask_dict_password() does exact matching18:42
jrollaha18:42
jrollmaybe I won't change it then :)18:42
jlvillaljroll, https://bugs.launchpad.net/oslo.utils/+bug/159493518:42
openstackLaunchpad bug 1594935 in oslo.utils "mask_dict_password() is inconsistent with mask_password()" [Undecided,New]18:42
jrollfwiw, I tried my damndest to make an RCE out of this literal_eval and couldn't18:42
jrollit passes through some amount of json parsing before getting here, which is safe18:43
jrollso, still tech debt but not a huge deal imo18:43
mat128jroll: RCE?18:43
jrollremote code execution18:43
* jlvillal thanks mat128 for asking that :)18:43
*** piet has quit IRC18:43
mat128ahh18:43
mat128Google wasn't helpful18:44
*** PollyZ has joined #openstack-ironic18:44
mat128even wikipedia; https://en.wikipedia.org/wiki/RCE18:44
mat128;)18:44
mat128oy I think18:45
mat128hoy18:45
jrolloi!18:45
jrollOI OI OI18:45
* jroll https://en.wikipedia.org/wiki/Oi!18:46
jrollthat ! is part of the link btw18:46
jlvillaldtantsur, Howdy Y'all18:46
dtantsurah!18:46
mat128jroll: I see we're probably on the same IRC client ;)18:46
dtantsuror rather: ой ;)18:46
TheJuliaY'all ready to start discussing this spec?18:46
dtantsurjlvillal, ^^18:46
mat128https://review.openstack.org/#/c/200496/39/specs/approved/volume-connection-information.rst18:47
*** sacharya has quit IRC18:47
*** jvgrant has joined #openstack-ironic18:47
*** htruta` is now known as htruta18:48
*** catintheroof has quit IRC18:51
*** catintheroof has joined #openstack-ironic18:51
sambettsbrb18:52
openstackgerritMerged openstack/ironic: Centralize config options - [conductor]  https://review.openstack.org/30139918:52
*** ayoung has quit IRC18:54
*** PollyZ_ has joined #openstack-ironic18:55
openstackgerritJim Rollenhagen proposed openstack/ironic-specs: Volume connection information for Ironic nodes  https://review.openstack.org/20049618:56
*** sdake has joined #openstack-ironic18:56
*** piet has joined #openstack-ironic18:56
*** PollyZ has quit IRC18:56
TheJuliajroll: line 779 rdb_root18:57
jrollTheJulia: /me shakes fist at rbd vs rdb18:58
TheJuliaheh18:58
mat128boot from volume spec, all we had to correct are hyphens18:58
openstackgerritJim Rollenhagen proposed openstack/ironic-specs: Volume connection information for Ironic nodes  https://review.openstack.org/20049618:58
mat128:)18:58
*** ayoung has joined #openstack-ironic18:59
*** sdake_ has quit IRC18:59
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [iboot]  https://review.openstack.org/30185619:00
*** ayoung has quit IRC19:00
jlvillalsheep vote :D19:00
mat128https://review.openstack.org/#/c/294995/19:01
rloocan you hear me?19:02
rlooi guess not.19:02
mat128rloo: no19:02
rlooi got muted hmm...19:02
jrollHRM19:02
rlooi'll dial in again.19:02
rlooconnector_id19:03
mat128From 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
mat128jlvillal: ^19:03
mat128;)19:03
jlvillalmat128, heh19:03
rlooline 8019:03
TheJuliarloo: pondering19:03
TheJuliakind of in the no opinion boat19:04
openstackgerritBharath kumar proposed openstack/ironic: adding back the inspect_ports as an optional parameter  https://review.openstack.org/32523019:04
jrollmat128: I'm a huge fan of TLAs19:05
mat128got that one :P19:06
mat128https://en.wikipedia.org/wiki/Three-letter_acronym19:06
mat128jroll: ever worked in military systems?19:06
mat1283-4 letter acronyms are the norm :P19:06
jrollmat128: general motors close enough?19:06
mat128probably19:06
jroll:D19:06
jrollsorry, GM*19:06
dtantsurlol19:07
pas-hahas the sound become better?19:07
jrollpas-ha: yes19:07
jlvillalIf you are on the call, please mute!19:07
pas-hain fact I was muted19:07
pas-hasmth really wrong with my setup then ^(19:07
jroll:/19:07
jlvillalI think it got quieter :)19:08
rlooDONE19:08
TheJulia\o/19:08
mat128yeah19:08
rloonext..... TheJulia's spec!19:08
TheJuliayay!19:08
mat128https://review.openstack.org/#/c/294995/19:08
jlvillalUnrelated note: Proposed patch to fix make_dict_password() https://review.openstack.org/#/c/332438/19:09
* pas-ha sighs19:09
pas-haok then, 10 PM here, will leave you guys19:09
pas-hawill try tomorrow from other machine19:10
jlvillalpas-ha, Paka paka! Thanks :)19:10
rloobye pas-ha19:10
JayFTheJulia: was multitasking, heard my name, and missed context19:10
pas-hajlvillal: lol :)19:10
JayFTheJulia: like being called on by the teacher while sleeping19:10
*** bharath has quit IRC19:11
openstackgerritMerged openstack/ironic: Centralize config options - [database]  https://review.openstack.org/30181519:12
*** Sukhdev has joined #openstack-ironic19:13
* TheJulia does not want anyone to travel back in time 7 days19:16
mat128why is this so? :P19:16
jlvillalToo late!19:16
TheJuliadoh!19:16
mat128why 7 days?19:16
*** fragatina has joined #openstack-ironic19:16
* jlvillal bought 1000 shares of LinkedIn :)19:16
jrolljlvillal: just now? O_o19:18
TheJuliamat128: https://en.wikipedia.org/wiki/Seven_Days_(TV_series)19:20
jlvillaljroll, 7 days ago. TheJulia said don't go back in time 7 days and I said I did ;)19:20
mat128ahh19:20
jrolljlvillal: 7 days wouldn't get you much, 11 days looks better :P19:21
* vdrok is not able to review anything anymore19:21
vdrokgood night everyone!19:21
TheJuliagoodnight vdrok19:21
jroll\o vdrok19:21
* jroll brb19:21
jlvillaljroll, Oh darn. I thought Microsoft had announced the purchase about six days ago.19:21
*** edand has joined #openstack-ironic19:22
*** jvgrant has quit IRC19:24
rloojlvillal: don't give up your day job19:25
jlvillalrloo, :P19:26
lucasagomesjlvillal, you serious or kidding about the shares?19:27
*** kromanenko_ has joined #openstack-ironic19:27
*** kromanenko__ has joined #openstack-ironic19:27
*** Romanenko_K has joined #openstack-ironic19:27
* lucasagomes is intrigued 19:27
jlvillallucasagomes, joking!19:27
lucasagomesah right heh19:27
TheJuliabefore 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 sunday19:27
*** jcoufal_ has joined #openstack-ironic19:27
*** e0ne_ has joined #openstack-ironic19:28
*** e0ne_ has quit IRC19:28
*** piet has quit IRC19:29
*** e0ne has quit IRC19:30
*** david-lyle has quit IRC19:30
*** jcoufal has quit IRC19:30
*** vsaienko has quit IRC19:30
*** cinerama has quit IRC19:30
*** caiobo-afk has quit IRC19:30
*** aarefiev has quit IRC19:30
*** gabriel-bezerra has quit IRC19:30
*** hoonetorg has quit IRC19:31
*** tonyb has quit IRC19:31
* TheJulia suspects there has been a fiber cut someplace in the world19:31
*** sukhdev_ has joined #openstack-ironic19:31
JayFhttp://internetpulse.keynote.com/19:32
*** tonyb has joined #openstack-ironic19:32
JayFL3 to cogent and verizon is hurting19:32
*** MattMan has quit IRC19:33
*** MattMan has joined #openstack-ironic19:33
*** cinerama has joined #openstack-ironic19:34
jlvillalOff 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 IRC19:35
jlvillalNever mind19:35
jlvillalIt already is. Duh19:35
mat128was about to suggest that they should19:35
* jlvillal can't read a Zuul dashbord.19:35
mat128otherwise you're never running it against latest code19:35
*** ayoung has joined #openstack-ironic19:40
openstackgerritMerged openstack/ironic: Centralize config options - [dhcp]  https://review.openstack.org/30183019:40
openstackgerritMerged openstack/ironic: Mask password on agent lookup according to policy  https://review.openstack.org/33219519:40
*** david-lyle has joined #openstack-ironic19:41
TheJuliaJayF: yeouch, nearly 8% packet loss on one of the paths that is measuring19:41
*** caiobo-afk has joined #openstack-ironic19:42
dtantsurjlvillal, mat128, we are not allowed to have voting check jobs that are not in gate19:42
dtantsur(allowed by infra)19:42
mat128excellent19:43
jlvillaldtantsur, thanks19:43
mat128from a consistency/integrity standpoint :)19:43
*** gabriel-bezerra has joined #openstack-ironic19:43
*** aarefiev has joined #openstack-ironic19:43
*** hoonetorg has joined #openstack-ironic19:43
TheJuliaJayF: or take migraine meds19:43
*** vsaienko has joined #openstack-ironic19:44
rlooi would just cancel tonight's, but I am not going to be there. You could do bug triage :)19:45
openstackgerritVladyslav Drok proposed openstack/ironic: Add test to verify ironic multitenancy  https://review.openstack.org/26915719:45
openstackgerritVladyslav Drok proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643219:46
mat128for whoever reads scrollback, I'm going to be there at 0000 UTC for the next session19:47
dtantsuraaaaaaaaaaaaand I'm calling it a day :)19:48
dtantsurplease, please, please review the driver composition spec and all the comments to it19:49
lucasagomes++19:49
lucasagomeshave a great evening all19:49
dtantsurI really hope to land it tomorrow19:49
dtantsurg'night everyone, g'night lucasagomes19:49
rloobye dtantsur, lucasagomes. a demain.19:49
JayFYou know, a thought I had after how effective the mid-cycle is at unsticking specs19:49
*** edand has quit IRC19:49
JayFwould anyone be interested in doing a regular, over audio/video, review jam for specs?19:49
* rloo worries about JayF's thought19:49
dtantsurtrue19:49
JayFlike weekly or semi-weekly19:49
jrollyes, was going to suggest that19:50
dtantsurJayF++, probably 1-2 times a month19:50
mat128JayF: sure19:50
rlooJayF, jroll: i actually was thinking that it would be worthwhile to do these meetings sort of *whenever* there was something to discuss.19:50
rlooinstead of waiting for a midcycle19:50
JayFwhenever is kinda like eventually19:50
mat128just send a doodle to the list ;)19:50
JayFit's hard to ever come around19:50
TheJuliaJayF: ++19:50
JayFsomethign that's scheduled and cancelled if unneeded, the momentum helps19:51
rlooJayF: well, whenever would be when someone suggests it.19:51
*** dtantsur is now known as dtantsur|afk19:51
rlooJayF: 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
rlooJayF: how did that work for you?19:51
mat128rloo: we were 4 and JayF wrote the spec after our discussion19:51
*** PollyZ_ has quit IRC19:51
dtantsur|afkit's easier to dedicate time if it's regular19:51
rloook, so it was productive, right?19:52
JayFwell I didn't really write much of the spec19:52
JayFyet actaully19:52
dtantsur|afkit's also easier to communicate downstream that you're busy every Nth of the month, for example19:52
openstackgerritAndre Aranha proposed openstack/python-oneviewclient: Extend python-oneviewclient to Network  https://review.openstack.org/29455919:52
* dtantsur|afk is really afk now19:52
mat128rloo: IMO, yes19:52
rloothe problem with picking a regular time/date, is that you will be ostracizing? certain folks.19:52
JayFIt was mostly productive. I probably would try to have more of a framework in place before doing it again19:52
mat128rloo: just like the weekly meeting19:52
JayFi.e. not going in pre-spec full-on19:52
jrollJayF: 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/33245519:52
JayFI'd have a spec with todos or whatever in it19:52
JayFjroll: eek, I can cherry pick it back19:53
rloomat128: right, the meeting time isn't conducive to some people attending it.19:53
jrollJayF: cool, thanks19:53
rloomat128: 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
mat128rloo: but you risk never "calling out" those sessions19:53
mat128rloo: I say let's try it19:53
mat128the worst that can happen is that we learn from it19:54
mat128best case, we get stuff done<19:54
rloomat128: try me. I can call it out if I wanted to. And I would think that other cores would too. JayF did!19:54
JayFI am going to activate on doing an informal review jam, with a preferred focus on stuck specs. Participation optional.19:54
mat128+1 to this19:55
rloomy concern is that if we pick a time when 'most' folks can attend, folks in asia/australia/etc will be more inconvenienced.19:55
JayFI mean, IMO, they can feel free to have one at a convienient time for them too?19:55
mat128rloo: so more points for doing it ad-hoc, but Dimtry's point was "harder to block schedule time for it"19:55
JayFThere'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
mat128JayF: I guess19:55
rlooJayF: well, it seems like if there aren't enough cores/regulars/vocals, it isn't that useful.19:55
JayFanyone can do it :)19:55
mat128jlvillal, sambetts, myself and maybe TheJulia are going to be discussing multinode grenade thursday 0800 PST19:56
mat128there, done :)19:56
rloomat128: maybe send out email then. Would reach a wider audience i hope.19:57
mat128John was leading this tho, but I'm sure he will read it from the scrollback19:57
rloojlvillal: ^^19:57
mat128I kinda just jumped in the topic19:57
*** PollyZ has joined #openstack-ironic19:58
mat128alright, away until right before 0000 UTC19:59
openstackgerritMerged openstack/ironic-specs: Volume connection information for Ironic nodes  https://review.openstack.org/20049620:01
TheJuliaalso away for the next few hours20:01
jlvillalrloo, mat128 Yep. We are meeting on Thursday 8AM Pacific time20:01
jlvillalTo discuss multi-node grenade20:01
*** PollyZ has quit IRC20:02
jlvillalrloo, I'm not sure about wanting a giant group of people. Is that wrong???20:02
rloojlvillal: 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 IRC20:03
rloojlvillal: i wonder how inclusive we have been. but whatever works to get the work done.20:03
*** jjohnson2 has joined #openstack-ironic20:07
*** PollyZ has joined #openstack-ironic20:11
*** Sukhdev has quit IRC20:13
*** sukhdev_ is now known as sukhdev20:13
*** rcernin has joined #openstack-ironic20:13
*** ijw has quit IRC20:14
*** moshele has joined #openstack-ironic20:15
*** PollyZ has quit IRC20:16
openstackgerritAndre Aranha proposed openstack/python-oneviewclient: Extend python-oneviewclient to UplinkSet  https://review.openstack.org/29598120:21
*** PollyZ has joined #openstack-ironic20:25
*** ijw has joined #openstack-ironic20:26
*** PollyZ has quit IRC20:29
*** moshele has quit IRC20:33
*** moshele has joined #openstack-ironic20:34
*** moshele has quit IRC20:35
*** moshele has joined #openstack-ironic20:36
*** ayoung has quit IRC20:38
*** PollyZ has joined #openstack-ironic20:38
openstackgerritMerged openstack/ironic: Mask password on agent lookup according to policy  https://review.openstack.org/33219720:39
*** PollyZ has quit IRC20:43
*** jjohnson2 has quit IRC20:43
JayFjroll: https://review.openstack.org/#/c/332471/20:52
JayFalso, apparently stable/mitaka commits for ipa aren't coming in here20:52
jrollyeah, probably something missed in irc bot config20:52
rlooi thought jlvillal had a patch to fix that. maybe it hasn't merged yet.20:53
*** dprince has quit IRC20:53
jrollrloo: seems to be fixed for ironic20:53
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Fix unwanted unmounting of /usr/share/oem  https://review.openstack.org/33247220:54
JayFbut stable/liberty works20:54
JayF*shrug*20:54
JayFjroll: ^ that actually needs a review with brain involved20:55
jrolllol20:55
JayFjroll: it didn't cleanly cherry pick and more changes got pulled in20:55
JayFbut the changes are good stuff that should've been backported too, I think20:55
JayFso... yeah20:55
jrollJayF: cherry-pick pulled all of those in?20:55
JayFcherry-pick + I think a bad rebase by me20:55
JayFbut that's the end result I'd prefer over just adding the one mask20:56
jlvillalJayF, rloo, I did update it: https://review.openstack.org/#/c/330867/  For the bot announcements20:56
jlvillalIt got merged.20:56
JayFso IDK if technical correctness or git correctness is better20:56
*** Sukhdev_ has joined #openstack-ironic20:56
rloojlvillal: thx. I have no idea how I knew about it but I did :)20:57
jrollJayF: commented, liberty is security support right now :/20:58
jrolland you'll need devananda or dtantsur|afk for the +A on the mitaka20:59
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Fix unwanted unmounting of /usr/share/oem  https://review.openstack.org/33247220:59
JayFjroll: ^20:59
openstackgerritMoshe Levi proposed openstack/ironic-python-agent: Replace assertRaisesRegexp with assertRaisesRegex  https://review.openstack.org/33225020:59
jlvillaljroll, Speaking of security. When I searched for the CVE for the masked password. It said, no information yet.20:59
*** rbudden has joined #openstack-ironic20:59
jlvillalAs in, the information was still to be released.20:59
jrollJayF: done20:59
jrolljlvillal: hrm, not sure who is responsible for that, got a quick link?21:00
JayFdevananda: 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 IPA21:00
rloowhy announce it before the fix actually lands?21:00
JayFrloo: IDK if that was pointed at me or john, but for the IPA one, we unembargoed it after research showed it was non-exploitable21:01
jrollrloo: because that's the process21:01
jrollrloo: everything (bug, emails, patches proposed) happens at the same time https://security.openstack.org/vmt-process.html21:01
jrollpatches are included in the announcement, so people can apply without them merging21:01
rloojroll: Oh. Interesting. I guess if the patches are included, it is fine.21:02
jrollyep21:02
jlvillaljroll, http://www.cve.mitre.org/cgi-bin/cvename.cgi?name=2016-498521:02
jlvillaljroll, https://review.openstack.org/#/c/332197/3/releasenotes/notes/fix-cve-2016-4985-b62abae577025365.yaml21:02
*** moshele has quit IRC21:02
jrolljlvillal: thanks21:03
*** moshele has joined #openstack-ironic21:03
*** moshele has quit IRC21:03
*** raildo is now known as raildo-afk21:04
*** PollyZ has joined #openstack-ironic21:06
*** yolanda has joined #openstack-ironic21:06
*** trown is now known as trown|outtypewww21:07
*** baoli has quit IRC21:11
*** baoli has joined #openstack-ironic21:12
*** ijw has quit IRC21:23
*** ccamacho has quit IRC21:24
*** jcoufal_ has quit IRC21:30
*** baoli has quit IRC21:30
*** fragatina has quit IRC21:33
*** ayoung has joined #openstack-ironic21:41
*** rcernin has quit IRC21:41
JayFI just -1'd https://review.openstack.org/#/c/320584 because it removes two fields from the NetworkInterface class on IPA hardware module21:44
*** sacharya has joined #openstack-ironic21:44
JayFI'm afraid it could break backwards compat but could be convinced it doesn't matter ... but I'm not sure.21:44
rlooJayF: thx. I wondered about that (and asked). Thought it was sufficient to document it cuz it apparently wasn't used.21:45
JayFI mean, someone downstream could be using it21:45
sambettsJayF: 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
JayFand completely removing it could break code ... if it wasn't properly using .get() and if it populated themselves21:45
JayFsambetts: I don't know, I'm asking the question here because I feel like it oculd be a stretch21:45
sambettsJayF: do we support people using those fields?21:46
JayFWe support pluggable hardware managers. and list_hardware_info is overrideable.21:46
JayFWe can't assume that just because we don't populate it that nobody does.21:46
sambettsJayF: is NetworkInterface specifc to the generic hardware manager?21:46
JayFBut I agree that it's a bit of a giant stretch21:46
JayFno21:46
JayFit's in the hardware module21:46
JayFand populated by list_hardware_info, which can be overridden by a hardware manager21: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 manager21:47
* JayF not sure the proper terms to use for that21:47
* sambetts hates people adding stuff like this :'( I would have totally -1'd a patch adding fields for no reason 21:47
JayFI mean, this dates way, way, way back21:47
JayFwas probably in the code when it went from teeth-agent -> IPA21:47
jroll# TODO(russellhaering): Pull these from LLDP21:48
JayFI'm OK with us saying "we're OK breaking this"21:48
jrolllol21:48
JayFI just want us to make that decision explicitly21:48
jrollI mean, deprecate and remove next cycle?21:48
JayFjroll: how do you even detect that's being read to log it for deprecation?21:48
jrollJayF: override __getattr__ and __setattr__21:48
JayFjroll: I'm sorta on the side of just dump it, but wanted to make sure we knew what we were doing21:48
*** chlong|rhce_trng has quit IRC21:48
*** sacharya has quit IRC21:49
sambettswe've added a release note for it if thats any condolence21:49
JayFI mean, if the consensus is drop it, I'll land the patch now21:49
JayFI'm like, 90% sure I have the only downstream hwm21:49
JayFand we don't use it21:49
sambettsheh21:49
JayFbut that assumption only gets worse and worse as time goes by21:50
jrollI mean, I don't agree we should drop it21:50
jrollwhen there's a clear path to deprecating it21:50
*** catintheroof has quit IRC21:51
jroll(hacky, but clear)21:51
* sambetts can put up a fresh patch when I'm more awake deprecating it 21:51
sambettsI'll leave it as None though, I don't plan to populate it with anything, if thats ok?21:51
JayFoh absolutely21:52
rloooh, I think we should populate it now that we are grabbing that info.21:52
* rloo is joking21:52
* sambetts nearly started crying 21:52
jrollsambetts: yeah, that's fine21:52
rloono worries sambetts. guess I should have +A'd before JayF saw it :)21:53
JayFI mean, I don't have a strong opinion that we should keep it21:53
JayFI do have a strong opinion that we should provide a stable interface for downstream hwms21:53
* JayF feels bad man for -1'ing over something like that21:54
*** ametts has quit IRC21:54
*** mbound has joined #openstack-ironic21:55
jrollJayF: I would've done it if I saw it first, don't feel bad21:55
sambettsit 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" senario21:56
openstackgerritRuby Loo proposed openstack/python-ironicclient: Move methods to utils.py  https://review.openstack.org/33248921:56
JayFI think that's maybe the last case of something like that in IPA21:57
jrollhope so :P21:57
JayFI might breeze through hardware module and see if there's anything else we might want to "proatively deprecate" from the beforetime21:58
sambettsI hope so :-p /me has got his ban hammer out21:58
sambettsanyways its like 23:00 and my beer is getting warm over there so I'll see everyone tomorrow o/22:01
sambettsjlvillal: https://review.openstack.org/#/c/332490/122:02
jrollwhy'd you leave your beer over there? :)22:02
jroll\o night sam22:02
sambettsheh good point :-P22:02
jlvillalsambetts, You should be going to bed! :)22:02
*** sambetts is now known as sambetts|afk22:02
jlvillalsambetts|afk, You and mat128 are doing the same stuff :)22:02
sambetts|afkXD22:02
sambetts|afkwe'll work it out on Thursday I guess22:03
jlvillalsambetts|afk, Needs to be experimental though22:03
sambetts|afkjlvillal: yeah good point22:03
sambetts|afkjlvillal: I'll fix in the morning22:03
sambetts|afk:D22:03
sambetts|afknight all o/22:03
jlvillalsambetts|afk, Ciao22: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 UK22:04
*** jrist has quit IRC22:06
*** thrash is now known as thrash|g0ne22:07
*** Sukhdev_ has quit IRC22:16
*** Sukhdev_ has joined #openstack-ironic22:20
tonybJayF: np, I'll look at them after the school run22:22
JayFthanks!22:22
tonybJayF: actually they're so simple ..... mostly done22:24
tonybJayF: As I said in my comment.  I'll +W the liberty version once mitaka merges22:25
JayFtonyb: curious why the wait? Not complaining just wanna understand22:25
*** ijw has joined #openstack-ironic22:27
*** rbudden has quit IRC22:27
jrollJayF: I'll release those in the morning btw22:28
JayFthanks22:28
tonybJayF: 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 that22:28
JayFtonyb: gotcha, that's the rule I wasn't aware of, makes perfect sense though22:28
jrolldtantsur|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
jrolljlvillal: fyi, I emailed the oss-security list with the announcement, they will release the CVE at some point after they see that22:29
* jroll finds beer and goes outside, night everyone22:29
tonybJayF: I have a poll running on the merge event so as long as that happens today I'll do the liberty version22:29
tonybjroll: good plan!22:29
JayFyeah, as long as it's landed for jroll to release in the morning22:29
JayFI seriously was just curious about why you were waiting, always want to know why so I can understand :)22:30
tonybJayF: cool.  Always better to ask than sit and wonder.22:31
*** rcarrillocruz has quit IRC22:39
*** rcarrillocruz has joined #openstack-ironic22:41
*** rpioso has quit IRC22:48
*** sukhdev has quit IRC22:53
*** chlong has joined #openstack-ironic23:02
*** praneshp has quit IRC23:02
openstackgerritMerged openstack/ironic-python-agent: Fix unwanted unmounting of /usr/share/oem  https://review.openstack.org/33247123:09
*** Nisha has joined #openstack-ironic23:09
*** jrist has joined #openstack-ironic23:12
openstackgerritMerged openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/33229723:17
*** snecklifter has quit IRC23:24
openstackgerritMoshe Levi proposed openstack/ironic-python-agent: Add vendor id and vendor name to interface information  https://review.openstack.org/27994023:26
*** snecklifter has joined #openstack-ironic23:33
*** PollyZ has quit IRC23:37
mrdaMorning all23:44
*** sacharya has joined #openstack-ironic23:45
*** sacharya has quit IRC23:49
mat128hi mrda23:55
mat128pstn line is busy :(23:57
mrdaI'm not going to join23:57
mrdaI'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!