Monday, 2016-09-12

*** causten has quit IRC00:07
*** rbudden has joined #openstack-ironic00:08
*** mbound has joined #openstack-ironic00:16
*** mbound has quit IRC00:21
*** fragatina has joined #openstack-ironic00:39
*** hoangcx has joined #openstack-ironic00:39
*** aabashkin_ has joined #openstack-ironic00:47
*** AlexeyAbashkin has quit IRC00:48
*** causten has joined #openstack-ironic00:56
*** baoli has joined #openstack-ironic01:01
*** baoli has quit IRC01:06
*** causten has quit IRC01:12
*** chlong_ has joined #openstack-ironic01:12
*** chlong has joined #openstack-ironic01:12
*** bfournie has joined #openstack-ironic01:15
*** hoangcx has quit IRC01:35
*** hoangcx has joined #openstack-ironic01:36
*** hoangcx2 has joined #openstack-ironic01:38
*** hoangcx has quit IRC01:40
*** hoangcx has joined #openstack-ironic01:42
*** hoangcx2 has quit IRC01:45
*** hoangcx2 has joined #openstack-ironic01:49
*** hoangcx has quit IRC01:51
*** vishwanathj has joined #openstack-ironic01:56
*** vishwanathj has quit IRC01:57
*** rbrud has joined #openstack-ironic02:04
*** hoangcx has joined #openstack-ironic02:17
*** hoangcx2 has quit IRC02:18
*** lifeless_ has quit IRC02:19
*** lifeless has joined #openstack-ironic02:34
*** hoangcx has quit IRC02:42
*** hoangcx has joined #openstack-ironic02:42
*** rbudden has quit IRC02:46
*** yuanying has quit IRC02:51
*** hoangcx2 has joined #openstack-ironic03:00
*** hoangcx has quit IRC03:00
*** hoangcx has joined #openstack-ironic03:03
*** hoangcx2 has quit IRC03:04
*** baoli has joined #openstack-ironic03:26
*** rbudden has joined #openstack-ironic03:26
*** baoli has quit IRC03:34
*** baoli has joined #openstack-ironic03:34
*** baoli has quit IRC03:35
*** baoli has joined #openstack-ironic03:36
*** watanabe_isao has joined #openstack-ironic03:40
*** baoli has quit IRC03:40
*** tuanla_ has joined #openstack-ironic03:46
*** tuanla_ has quit IRC03:47
*** tuanla_ has joined #openstack-ironic03:48
*** yuanying has joined #openstack-ironic03:50
*** hshiina has joined #openstack-ironic03:57
*** rbudden has quit IRC03:58
*** hshiina has quit IRC04:03
*** amotoki has joined #openstack-ironic04:05
*** baoli has joined #openstack-ironic04:27
*** Sukhdev has joined #openstack-ironic04:29
*** baoli has quit IRC04:34
*** _vsaienko has joined #openstack-ironic04:34
*** rbrud has quit IRC04:38
openstackgerritNguyen Hung Phuong proposed openstack/ironic-specs: Update code to use Newton, Mitaka as the code name  https://review.openstack.org/36854504:51
*** jaosorior has joined #openstack-ironic05:02
*** chlong_ has quit IRC05:03
*** chlong has quit IRC05:03
*** chlong_ has joined #openstack-ironic05:17
*** chlong has joined #openstack-ironic05:21
*** chlong has quit IRC05:25
*** _vsaienko has quit IRC05:29
*** rcernin has joined #openstack-ironic05:33
*** rbartal has joined #openstack-ironic05:36
*** ChubYann has quit IRC05:55
*** chlong_ has quit IRC06:05
*** jtomasek has joined #openstack-ironic06:20
*** mjura has joined #openstack-ironic06:21
*** mjura has quit IRC06:26
*** hoangcx2 has joined #openstack-ironic06:27
*** mjura has joined #openstack-ironic06:27
*** hoangcx has quit IRC06:27
*** pcaruana has joined #openstack-ironic06:49
*** davidlenwell has quit IRC06:55
*** tesseract- has joined #openstack-ironic07:00
pas-hamorning Ironic07:03
*** Sukhdev has quit IRC07:04
*** davidlenwell has joined #openstack-ironic07:08
*** mgoddard has joined #openstack-ironic07:20
*** hoangcx2 has quit IRC07:25
*** hoangcx has joined #openstack-ironic07:27
*** jpich has joined #openstack-ironic07:28
*** ifarkas_afk is now known as ifarkas07:28
*** moshele has joined #openstack-ironic07:37
openstackgerritVasyl Saienko proposed openstack/ironic: Neutron notifications  https://review.openstack.org/34596307:37
openstackgerritVasyl Saienko proposed openstack/ironic: Test patch that verify external events  https://review.openstack.org/35799507:37
*** hoangcx2 has joined #openstack-ironic07:38
*** hoangcx has quit IRC07:39
*** fxpester has joined #openstack-ironic07:53
*** ccamacho has joined #openstack-ironic07:54
openstackgerritMerged openstack/proliantutils: Fix Inspection failure in Gen7  https://review.openstack.org/35713407:56
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Refactoring code of generating temp_url  https://review.openstack.org/35228907:57
*** ohamada has joined #openstack-ironic07:58
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:00
*** baoli has joined #openstack-ironic08:05
*** hoangcx2 has quit IRC08:06
*** hoangcx2 has joined #openstack-ironic08:06
openstackgerritYuriy Zveryanskyy proposed openstack/ironic-specs: Add CRUD and node states notifications to ironic  https://review.openstack.org/34724208:07
*** mbound has joined #openstack-ironic08:08
*** hoangcx has joined #openstack-ironic08:10
*** hoangcx2 has quit IRC08:10
*** baoli has quit IRC08:10
*** Romanenko_K has joined #openstack-ironic08:14
*** lucas-afk is now known as lucasagomes08:17
lucasagomesmorning all08:17
*** aabashkin_ is now known as AlexeyAbashkin08:20
*** AlexeyAbashkin has quit IRC08:22
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Add node provision state change notification  https://review.openstack.org/34843708:25
openstackgerritGalyna Zholtkevych proposed openstack/ironic: IPMI command should depend on console type  https://review.openstack.org/36364708:28
*** mgoddard has quit IRC08:32
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Add CRUD notifications for ironic resources  https://review.openstack.org/35654108:39
*** AlexeyAbashkin has joined #openstack-ironic08:42
ifarkasmorning all08:46
*** mgoddard has joined #openstack-ironic08:47
*** jaosorior is now known as jaosorior_brb08:51
openstackgerritVasyl Saienko proposed openstack/ironic: Fix setting custom IRONIC_VM_NETWORK_BRIDGE  https://review.openstack.org/36511608:51
openstackgerritVasyl Saienko proposed openstack/ironic: Configure clean network to provision network  https://review.openstack.org/35663208:51
openstackgerritVasyl Saienko proposed openstack/ironic: Source openrc on subnode in multinode topology  https://review.openstack.org/35330208:51
openstackgerritVasyl Saienko proposed openstack/ironic: Skip db configuration on subnodes  https://review.openstack.org/35330308:51
openstackgerritVasyl Saienko proposed openstack/ironic: Ignore required_services for multinode topology  https://review.openstack.org/35279308:51
openstackgerritVasyl Saienko proposed openstack/ironic: Update devstack provision net config for multihost  https://review.openstack.org/36864408:51
openstackgerritVasyl Saienko proposed openstack/ironic: Update ironic node names for multinode case  https://review.openstack.org/36864508:51
openstackgerritVasyl Saienko proposed openstack/ironic: Skip some steps for multinode case  https://review.openstack.org/36864608:51
openstackgerritVasyl Saienko proposed openstack/ironic: Add devstack setup_vxlan_network()  https://review.openstack.org/36864708:51
openstackgerritVasyl Saienko proposed openstack/ironic: Update iptables rules and services IPs for multinode  https://review.openstack.org/36864808:51
openstackgerritVasyl Saienko proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/29643208:53
openstackgerritVasyl Saienko proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817308:54
*** vsaienk0 has joined #openstack-ironic08:55
*** _vsaienko has joined #openstack-ironic08:55
*** watanabe_isao has quit IRC08:57
*** electrofelix has joined #openstack-ironic08:59
tangchenmorning all :)09:02
*** openstackgerrit has quit IRC09:04
*** openstackgerrit has joined #openstack-ironic09:05
*** hoonetorg has joined #openstack-ironic09:05
*** derekh has joined #openstack-ironic09:05
*** dtantsur|afk is now known as dtantsur09:07
dtantsurMorning Ironic09:07
*** hoangcx has quit IRC09:08
*** hoangcx has joined #openstack-ironic09:09
openstackgerritMerged openstack/python-ironic-inspector-client: Add oslotest to test-requirements  https://review.openstack.org/36720609:11
*** _vsaienko has quit IRC09:12
*** zhugaoxiao has quit IRC09:16
*** zhugaoxiao has joined #openstack-ironic09:17
vsaienk0morning dtantsur, lucasagomes, sambetts, pas-ha, vdrok, tangchen, and all Ironic'ers!09:18
dtantsuro/09:18
lucasagomeshi there :D09:19
openstackgerritShivanand Tendulker proposed openstack/ironic: Timing metrics: iLO drivers  https://review.openstack.org/35274509:24
*** _vsaienko has joined #openstack-ironic09:24
sambetts|afkMorning morning!09:26
*** sambetts|afk is now known as sambetts09:26
lucasagomessambetts, sup morning09:30
sambettso/ lucasagomes, hows it going?09:30
lucasagomessambetts, all is well over here! Going to brazil for a month next weekend, I gotta attend a weeding and also check my parents/friends etc09:31
lucasagomessambetts, you?09:31
dtantsurhey sambetts, lucasagomes09:31
dtantsurlucasagomes, cool! are you going to WFB (work from Brasil) or is it a vacation?09:31
lucasagomesdtantsur, I will take some days off, but I will mostly work from there09:32
lucasagomessince most of my friends will be working in the morning/afternoon as well09:32
dtantsurstill pretty cool09:32
lucasagomesbut yeah, looking forward to it!09:32
openstackgerritTuan Luong-Anh proposed openstack/ironic: Replace six iteration methods with standard ones  https://review.openstack.org/36867609:36
*** vnogin has joined #openstack-ironic09:41
tuanla_good afternoon,09:42
sambettslucasagomes: Oh nice! I'm finally taking some holiday next week too, taking my new touring motorcycle and driving up to the north of england09:43
lucasagomessambetts, that sounds amazing!09:44
*** vnogin has left #openstack-ironic09:45
sambettslucasagomes: yeah, I'm really looking forward to it, I've not taken a proper vacation for >6 months09:46
sambettsif not longer09:46
*** _vsaienko has quit IRC09:47
lucasagomesright, well deserved then09:47
tuanla_Hi sambetts, lucasagomes, dtantsur: Have you ever came to VietNam09:47
dtantsurhi tuanla_, no, never09:48
tuanla_Hello, Have many places for relax and also VietOpenStack teams will help you if you guide have plan to come to VietNam09:49
lucasagomestuanla_, I have not, would love to tho :-)09:50
tuanla_yes, Hope you guide will come to VietNam09:51
*** baoli has joined #openstack-ironic09:54
dtantsurI would love as well.. it's quite far from us though :)09:54
sambettsI've wanted to go since I saw the TopGear episode where they rode motorcycles, it looking stunning09:56
openstackgerritTuan Luong-Anh proposed openstack/ironic: Use assertEqual() instead of assertDictEqual()  https://review.openstack.org/36870009:57
tuanla_Yes, dtantsur Vietnam is near Russian than USA.09:57
*** hoangcx has quit IRC09:58
*** baoli has quit IRC09:58
tuanla_sambetts: what type of motorcycles you like?09:58
openstackgerritMerged openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/36810509:59
sambettstuanla_: I have a sports bike and I've just got an Adventure bike for doing some touring09:59
tuanla_sound like spoiled to me, I also want to have sports bike for go many province in VietNam with friends10:02
*** mgould has joined #openstack-ironic10:05
mgouldmorning Ironic!10:06
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Add simple Sphinx documentation  https://review.openstack.org/36870510:06
*** jaosorior_brb is now known as jaosorior10:06
sambettso/ mgould10:06
mgouldmorning sambetts10:09
*** narasimha_SV has joined #openstack-ironic10:12
*** fragatin_ has joined #openstack-ironic10:15
*** fragatina has quit IRC10:19
lucasagomesmgould, hi there morning10:22
mgouldmorning lucasagomes!10:23
*** jvgrant_ has quit IRC10:26
*** jvgrant has joined #openstack-ironic10:26
*** e0ne has joined #openstack-ironic10:27
*** jbcraig has quit IRC10:30
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Migrate to swift API for generating temp_url  https://review.openstack.org/35228910:35
openstackgerritTuan Luong-Anh proposed openstack/ironic: Replace six iteration methods with standard ones  https://review.openstack.org/36867610:46
*** baoli has joined #openstack-ironic10:54
*** baoli has quit IRC10:59
*** hoonetorg has quit IRC11:03
*** tuanla_ has quit IRC11:03
*** _vsaienko has joined #openstack-ironic11:12
sambettslucasagomes: do you know much about our tempest plugin??11:14
dtantsurit's magic and ponies :)11:14
sambettshahaha11:14
sambettsdtantsur: I'm trying to test a new feature, but the tempest plugin seems to be using a API version thats too old, do you know what version its supposed to be using? Looking in the plugin itself I see BAREMETAL_MICROVERSION = None but I don't know if its overridden by some magic and ponies somewhere11:16
lucasagomessambetts, heh not really11:16
dtantsursambetts, see inspector tempest plugin, we do mess with versions11:16
*** daemontool has joined #openstack-ironic11:19
sambettsI can't remember if we changed it recently, whats the default version if your don't specify an API version?11:20
sambettslucasagomes, dtantsur: I think I found some magic and ponies ;) there is a api_microversion_fixture in the tempest plugin that manages that global variable11:22
lucasagomessambetts, the default is 1.6 I think11:24
lucasagomessambetts, a-ha there you go11:24
lucasagomesI will keep that in mind next time I have to do something with tempest11:24
sambettshmm, so this is interesting, so the API tests use the fixture to set/manage the API version, but it doesn't seem like the scenario tests are using it :/11:25
*** lucasagomes is now known as lucas-hungry11:25
openstackgerritMerged openstack/ironic-inspector: Set node to the error if reapply fails  https://review.openstack.org/36566211:26
openstackgerritVasyl Saienko proposed openstack/ironic: Update iptables rules and services IPs for multinode  https://review.openstack.org/36864811:27
openstackgerritVasyl Saienko proposed openstack/ironic: Fix setting custom IRONIC_VM_NETWORK_BRIDGE  https://review.openstack.org/36511611:27
openstackgerritVasyl Saienko proposed openstack/ironic: Add devstack setup_vxlan_network()  https://review.openstack.org/36864711:27
openstackgerritVasyl Saienko proposed openstack/ironic: Skip some steps for multinode case  https://review.openstack.org/36864611:27
openstackgerritVasyl Saienko proposed openstack/ironic: Update ironic node names for multinode case  https://review.openstack.org/36864511:27
openstackgerritVasyl Saienko proposed openstack/ironic: Update devstack provision net config for multihost  https://review.openstack.org/36864411:27
openstackgerritVasyl Saienko proposed openstack/ironic: Skip db configuration on subnodes  https://review.openstack.org/35330311:27
openstackgerritVasyl Saienko proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817311:27
*** baoli has joined #openstack-ironic11:29
vsaienk0I'm sorry for the spam :(11:30
sambetts;)11:31
*** milan has joined #openstack-ironic11:32
milanmorning Ironic! :)11:32
*** thrash|g0ne is now known as thrash11:32
sambettso/ vsaienk0, milan11:32
milanhey sambetts! :)11:32
mgouldmorning vsaienk011:32
*** baoli has quit IRC11:33
*** bfournie has quit IRC11:35
mat128morning vsaienk0, sambetts, milan and mgould11:37
milanmorning mat128! :)11:37
vsaienk0morning sambetts, mat128, milan, mgould11:38
milanmorning vsaienk0 :)11:38
*** baoli has joined #openstack-ironic11:39
*** _vsaienko has quit IRC11:39
TheJuliaGood morning!11:49
jrollmorning y'all11:50
jrollTIL sambetts is into motorcycles11:50
jrollmgagne: you asked about logging stdout, agree, it can't hurt to call strip() on it and get rid of that extra new line11:50
*** baoli_ has joined #openstack-ironic11:50
*** baoli has quit IRC11:53
dtantsurmorning TheJulia, jroll, mat128, vsaienk0, milan11:54
milanmorning dtantsur jroll TheJulia! :)11:54
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Migrate to swift API for generating temp_url  https://review.openstack.org/35228911:54
sambettsjroll: QQ about the tempest testing, in the scenario testing should we always test with the minimum API version the API supports or can we bump api version for exposing new fields11:55
*** daemontool has quit IRC11:56
jrollsambetts: there's a way to have a single test with a given api version11:57
jrollor do you mean the actual 'build a thing' test11:57
sambettsthe actual build a thing test11:57
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Add new config options to config  https://review.openstack.org/36878212:02
jrollsambetts: hm, I guess we've never talked about it12:04
jrollsorry, bouncing all over irc reading scrollback12:04
sambettsjroll: I'm working on the attach/detach API code, and I've hidden/locked the vif_port_id info in the port.internal_info field if you use the attach/detach API, but the scenario tests try to verify the ports have been updated by nova, by looking for the vif_port_id data, but the API version is to low to expose the internal_info field12:04
*** daemontool has joined #openstack-ironic12:05
jrolldo they really? :/12:05
jrollthat seems silly12:05
*** _vsaienko has joined #openstack-ironic12:05
sambettsjroll: https://github.com/openstack/ironic/blob/master/ironic_tempest_plugin/tests/scenario/test_baremetal_basic_ops.py#L9812:06
jrolldo we not verify that by making sure the build is successful and has connectivity? :)12:06
jrollseems like internal details12:06
* jroll clicks12:06
jrollI'd be +1 to removing the vif_port_id check12:07
jrollsambetts: this is a fun back-compat thing too12:07
jrollwe'll definitely have to have nova remove port.extra[vif_port_id] for (a while || ever?)12:07
jrollfor instances built before upgrade12:07
jroll(or have ironic clean it up)12:08
*** hshiina has joined #openstack-ironic12:08
mat128JayF: first victory http://docs.openstack.org/project-install-guide/baremetal/draft/12:08
jrollwill also need to look in both places in ironic for a cycle, for instances that are building during the upgrade, I think12:08
sambettsjroll: I've made my attach/detach stuff handle both so old instances will be cleaned up even if nova calls ironic.detach_interface12:08
*** trown|outtypewww is now known as trown12:09
jrollawesome12:09
sambettsjroll: the only part that I wasn't sure on is if I want the attach API to populate both the internal_info and extra fields with vif_port_id12:09
*** bfournie has joined #openstack-ironic12:10
sambettsjroll: then I have to handle if some one adds it via the new API but deletes it via the old one12:10
jrollsambetts: idk if there's a reason to keep populating extra12:10
sambettsjroll: this test ;)12:10
jrollbooooo hissssss12:10
sambettshahaha12:10
* jroll votes remove it12:10
openstackgerritzhangcf1 proposed openstack/ironic-specs: power rsd servers using rsc  https://review.openstack.org/36478312:12
openstackgerritMathieu Mitchell proposed openstack/ironic: Import install and configure into new install guide  https://review.openstack.org/36798812:14
openstackgerritMathieu Mitchell proposed openstack/ironic: Import Bare Metal service overview into install-guide  https://review.openstack.org/36632912:14
mat128lucas-hungry: ^ fixed that 404 link since we started publishing12:15
openstackgerritSam Betts proposed openstack/ironic: DO NOT MERGE Testing Attach/Detach changes  https://review.openstack.org/36794812:18
openstackgerritSam Betts proposed openstack/ironic: Add Network Interface Attach and Detach APIs  https://review.openstack.org/32704612:18
*** fragatin_ has quit IRC12:23
*** narasimha_SV has quit IRC12:26
openstackgerritzhangcf1 proposed openstack/ironic-specs: power rsd servers using rsc  https://review.openstack.org/36478312:27
openstackgerritVasyl Saienko proposed openstack/ironic: Update iptables rules and services IPs for multinode  https://review.openstack.org/36864812:27
openstackgerritVasyl Saienko proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817312:27
*** daemontool has quit IRC12:27
*** daemontool has joined #openstack-ironic12:28
*** lucas-hungry is now known as lucasagomes12:29
lucasagomesmat128, thanks, I will take a look soon-ish :D12:29
*** pester has joined #openstack-ironic12:30
*** fxpester has quit IRC12:32
*** leitan has joined #openstack-ironic12:34
lucasagomesfolks, can someone confirm that changing the values of the intervals passed to the periodic tasks is not working ?12:37
lucasagomeslike, changing sync_power_state_interval from 60 (default) to 180 seems to have no effect, it still runs every 60 seconds12:37
lucasagomesI think it's because the options are being evaluated after the functions are decorated with the periodic task12:38
sambetts:/12:40
jrolllucasagomes: I haven't seen that but I have seen similar issues. ouch :(12:41
*** mmedvede has quit IRC12:42
lucasagomes:-/ I will take a look12:42
* lucasagomes adds to his todo12:42
jrollalso heads up for ops folks... http://legalhackers.com/advisories/MySQL-Exploit-Remote-Root-Code-Execution-Privesc-CVE-2016-6662.html12:42
jrolland another coming? http://seclists.org/oss-sec/2016/q3/48312:42
lucasagomesI really which we could pass a function to the periodic task interval instead of an int12:42
lucasagomesthat would allow us to also use SIGHUP to update such values12:43
lucasagomeslike, periodic(spacing_func=<func>)12:43
jrollthat would be nice, I wonder if they'd take that (don't see why not)12:43
* lucasagomes fails to find josh online12:43
lucasagomesjroll, yeah i think that would be a great addition. Maybe I should submit a patch to futurist12:44
jroll+112:44
lucasagomesit should be fairly easy to do12:44
ricardoasmorning!12:50
openstackgerritMathieu Mitchell proposed openstack/ironic: Import install and configure into new install guide  https://review.openstack.org/36798812:50
openstackgerritMathieu Mitchell proposed openstack/ironic: Import Bare Metal service overview into install-guide  https://review.openstack.org/36632912:50
lucasagomesricardoas, morning12:53
mat128jroll: those MySQL CVEs12:53
mat128ouch12:53
sambettsmuch ouch12:53
*** jaosorior has quit IRC12:56
*** jaosorior has joined #openstack-ironic12:57
mgouldouch indeed12:58
xavierrgood morning Ironicers12:59
* mgould considers going back to the glacier until we all learn to write critical computing infrastructure securely...12:59
xavierrmorning mat128 lucasagomes sambetts jroll ricardoas mgould13:00
*** ppiela has joined #openstack-ironic13:00
mgouldmorning xavierr13:00
*** hshiina has quit IRC13:01
*** Goneri has joined #openstack-ironic13:04
lucasagomesxavierr, hi there13:06
mat128o/ xavierr13:09
openstackgerritVasyl Saienko proposed openstack/ironic: Update iptables rules and services IPs for multinode  https://review.openstack.org/36864813:11
openstackgerritVasyl Saienko proposed openstack/ironic: Testing multinode stuff  https://review.openstack.org/36817313:11
*** rloo has joined #openstack-ironic13:13
*** rbudden has joined #openstack-ironic13:16
*** mmedvede has joined #openstack-ironic13:19
*** openstackgerrit has quit IRC13:19
*** mmedvede has quit IRC13:19
*** openstackgerrit has joined #openstack-ironic13:19
*** mmedvede has joined #openstack-ironic13:20
*** Goneri has quit IRC13:23
openstackgerritPeter Piela proposed openstack/ironic-ui: Updated to ironic API v1.11 and added manageable state  https://review.openstack.org/36109813:25
openstackgerritPeter Piela proposed openstack/ironic-ui: Updated to ironic API v1.11 and added manageable state  https://review.openstack.org/36109813:26
*** moshele has quit IRC13:28
*** gabor_antal_ has joined #openstack-ironic13:34
*** gabor_antal has quit IRC13:34
*** hoangcx has joined #openstack-ironic13:35
*** hshiina has joined #openstack-ironic13:35
*** Goneri has joined #openstack-ironic13:38
*** vishwanathj has joined #openstack-ironic13:39
*** vishwanathj has quit IRC13:40
*** vishwanathj has joined #openstack-ironic13:41
*** lennyb has joined #openstack-ironic13:44
*** Goneri has quit IRC13:48
*** hoangcx has quit IRC13:51
openstackgerritLucas Alvares Gomes proposed openstack/virtualbmc: Bump pyghmi version to 1.0.9  https://review.openstack.org/36885813:52
*** hoangcx has joined #openstack-ironic13:52
lucasagomes^ if someone has few seconds, pyghmi version 1.0.9 fixes that IPMI session timeout problems and is already bumped in the g-r13:53
*** Goneri has joined #openstack-ironic13:53
jrolllucasagomes: virtualbmc doesn't use the bot thing?13:53
jrollor just going sooner13:53
jrolleither way +2, just curious13:53
lucasagomesjroll, yeah I was expecting it to use, but apparently it is not13:54
*** ametts has joined #openstack-ironic13:54
lucasagomesI should investigate why, any idea ?13:54
jrolllucasagomes: dunno, it's a list in g-r repo, I think13:54
lucasagomesjroll, ha, it might be it13:54
* lucasagomes checks13:54
*** hoangcx2 has joined #openstack-ironic13:57
lucasagomesjroll, yeah it wasn't there https://review.openstack.org/#/c/368860/13:58
patchbotpatch 368860 - requirements - Add openstack/virtualbmc to projects.txt13:58
jrollah ok13:58
jrollsomeone want to +A 368858?13:58
*** hoangcx has quit IRC13:59
sambettsjroll: don14:00
sambettsdone*14:00
jrollthaaaanks14:00
*** anush__ has joined #openstack-ironic14:00
*** rbrud has joined #openstack-ironic14:01
*** syed_ has joined #openstack-ironic14:02
jlvillalGood morning Ironic :)14:05
mgouldmorning jlvillal!14:05
jlvillalAs an FYI, I have a dentist appointment today. So I will miss today's meeting :(14:06
lucasagomessambetts, cheers14:07
lucasagomesjlvillal, morning :-)14:07
jrolldtantsur: thinking more, not sure that $project-release applies to us, since we don't do RCs. but anyway I've updated it to match stable, look okay to you? https://review.openstack.org/#/admin/groups/308,members14:07
jrolldtantsur: you might want to look at inspector though https://review.openstack.org/#/admin/groups/978,members14:07
jlvillalmgould, lucasagomes Hi!14:07
dtantsurjroll, lgtm (both, inspector includes ironic)14:08
dtantsurmorning jlvillal14:08
jrolldtantsur: oh, missed that :)14:08
jrollthanks14:08
*** causten has joined #openstack-ironic14:08
jrollthat reminds me, I have a meeting I can't miss during our meeting - I'll be around but not really paying attention to the ironic meeting. dtantsur or devananda, mind running our meeting today?14:09
jlvillalHi dtantsur.14:10
dtantsurjroll, I can, unless devananda also wants to :)14:10
jrolldtantsur: awesome, thank you14:10
*** _vsaienko has quit IRC14:16
vsaienk0morning jroll14:18
jrollhi vsaienk0 :)14:19
vsaienk0jroll, I have good news, multinode job is stable enough, I've did experiemental rechecks and got 8 success on 8 recheckes: please have look at rechecks for 2-nd patchset https://review.openstack.org/#/c/368173/214:19
patchbotpatch 368173 - ironic - Testing multinode stuff14:19
sambettsvsaienk0: Awesome!!14:19
jrollvsaienk0: amazing!14:19
jrolla couple questions14:19
vsaienk0we need to review near 18 bash patches :)14:20
jroll1) does this make vms run on both nodes or only one? 2) is this doing grenade or just running tests?14:20
jrolloh wow14:20
vsaienk0ironic-conductor is on both nodes, n-cpu also, vms are located on both nodes also14:20
vsaienk0conductor from primary node can spawn vms on subnode and vice verse14:21
jrollnice \o/14:21
sambettswe shouldn't have 2 n-cpus right?14:21
jrollcan't hurt :)14:21
vsaienk0sambetts: right, n-cpu is enabled on both primary and subnode14:21
sambettsjroll: I thought we didn't support that yet, won't we get a double count of the resources?14:22
jrollsambetts: we do support it!14:22
jrollin nova master14:22
vsaienk0sambetts: it is experimental feature14:22
sambettsOh nice! that stuff got merged?!14:22
dtantsuryep14:22
*** mkam has joined #openstack-ironic14:22
sambettssweeeeeeet! I missed that completely14:22
openstackgerritMerged openstack/virtualbmc: Bump pyghmi version to 1.0.9  https://review.openstack.org/36885814:22
jrollhttps://github.com/openstack/nova/commit/6047d790a32ef5a65d4d6b029f673ce53c3d414114:22
jroll:D14:22
sambettsawesome!!!!!14:23
vsaienk0so bash experts please have look on the patches: https://etherpad.openstack.org/p/ironic-newton-grenade-whiteboard14:23
jrollthere's some small caveats but it does work14:23
*** mkam has quit IRC14:23
sambettsjroll: btw I've had a successful test run of my attach/detach stuff https://review.openstack.org/#/c/367948/414:23
patchbotpatch 367948 - ironic - DO NOT MERGE Testing Attach/Detach changes14:23
jrollsambetts: todos on that: 1) move hash_ring.py to oslo for both ironic and nova, 2) auto-relocate instances if a compute goes away14:24
jrollnice!14:24
dtantsurif somebody needs statistics, I did give it a try on HA (3 controllers) TripleO environment14:24
jrollvsaienk0: do you mind putting a common topic for these?14:24
dtantsurworked for me, though I only tried a few instances14:24
*** mkam has joined #openstack-ironic14:24
vsaienk0jroll: sure, do we have a bug to track multinode stuff?14:25
jrollvsaienk0: not sure, but even just "ironic-multinode" or something is fine14:25
dtantsurand I'm afraid we're going to put it to production pretty soon :D14:25
jrollvsaienk0: or make a multinode rfe14:25
jrolldtantsur: I'm hoping we do the same - we have some work to do like moving old instances to the new computes though14:25
*** igordcard has quit IRC14:26
lucasagomes++14:26
sambettsman thats freakin awesome!14:26
*** igordcard has joined #openstack-ironic14:26
jrollthis cycle rules :)14:27
sambetts+++14:27
*** anush___ has joined #openstack-ironic14:28
*** anush__ has quit IRC14:28
sambettsjroll: one cool thing about the attach/detach API stuff, is that I can use it to implement the vlan aware bm stuff as its own network interface if we wanted to make it less controversial14:28
jrollsambetts: don't get what you mean?14:29
*** moshele has joined #openstack-ironic14:29
*** mkam has quit IRC14:29
sambettsjroll: because it abstracts the vif to pif mapping logic, I don't have to break/change the exist implementation in order to implement the vif_port_ids list thing, because i can do it as a separate driver from the existing one14:31
sambettsif we wanted to14:31
jrollhm, interesting, not sure if it's worth forcing people to change the network driver14:31
jrollbut good to know14:31
sambettsjroll: only worth doing if people are worried about the implementation changing14:32
jrollright14:32
jrollI think we can make it work14:32
sambettsyup :) should be pretty straight forward I hope, and once the attach/detach stuff merges we shouldn't have to worry about mergeing anything to do it into nova either14:33
*** michauds has joined #openstack-ironic14:33
jrollexcept for configdrive :)14:34
sambettsyup except that :/14:34
* sambetts kind of wishes we could add stuff to the config drive as it goes through Ironic 14:35
jrollyeah, we should consider re-thinking that, maybe14:35
*** hoonetorg has joined #openstack-ironic14:38
dtantsurjroll, could you please take a look at https://review.openstack.org/368722 ? infra is pretty keen on us cleaning up jobs we generate...14:39
patchbotpatch 368722 - openstack-infra/project-config - Clean up generated jobs for ironic-inspector-specs14:39
dtantsuroops, wrong link14:40
dtantsurjroll, this one: https://review.openstack.org/36874014:40
patchbotpatch 368740 - openstack-infra/project-config - Clean up Ironic dsvm jobs definition14:40
jrolldtantsur: "pxe_* drivers do not work with whole disks yet" - um? not true?14:42
dtantsurjroll, maybe? I assumed that's why we don't have any jobs for them.14:42
jrolldtantsur: I think they do work, surprised we don't test that14:43
*** mkam has joined #openstack-ironic14:43
jrollor thought they do work, now I'm not sure14:43
*** beekneemech is now known as bnemec14:43
jrolldtantsur: also don't understand the changes here: https://review.openstack.org/#/c/368740/3/zuul/layout.yaml14:43
patchbotpatch 368740 - openstack-infra/project-config - Clean up Ironic dsvm jobs definition14:43
dtantsurjroll, I'm just trying to reduce the number of jobs we generate. I think it does not quite matter which of our "flavors" we run on external projects, so I'd prefer it to be one job14:44
jrolldtantsur: oh! missed which project it was :)14:44
jrollsorry, +114:44
dtantsurjroll, so, should I add nonvoting pxe_ssh+wholedisk jobs to ironic?14:45
jrolldtantsur: I guess so, yeah14:45
dtantsurjroll, ok, please leave the comment, so that I don't forget14:45
jrollI'm really surprised it isn't there14:45
jrollI left a comment14:46
* lucasagomes can confirm the intervals are not being honored but dunno why, prepare_service() should do it >.<14:51
openstackgerritRicardo Araújo Santos proposed openstack/ironic: Fixes parameters validation in SSH power manager.  https://review.openstack.org/36615014:52
jrolllucasagomes: decorators are evaluated at import time, so that's before conf is loaded14:54
sambettsdtantsur: isn't partition images + pxe_ssh -nv on nova?14:55
lucasagomesjroll, right, but I also don't see ironic.conductor.manager being imported before, in fact, ironic/cmd/conductor appears to lazy evaluate it14:55
openstackgerritMichael Turek proposed openstack/python-ironicclient: [WIP] Add --wait to OSC provisioning commands  https://review.openstack.org/35480614:55
jrollorly14:55
*** dprince has joined #openstack-ironic14:57
jrolllucasagomes: I wonder if we need to be using ironic.conf.CONF instead of oslo.config.cfg14:57
jrollin ironic.cmd.conductor14:57
lucasagomesjroll, yeah lemme try changing it14:58
lucasagomesjroll, I found a workaround to reload ironic.conductor.manager prior to invoking launch()14:58
lucasagomesthat seems to work as well14:58
jrollI bet the conf import is the problem then14:58
lucasagomeslemme try14:58
* lucasagomes have a meeting in 2 min T.T14:58
lucasagomesjroll, anyway, I will open a bug for it and keep looking, probably will put a fix today or tomorrow tops14:59
jrolllucasagomes: sounds good, thanks14:59
*** mgoddard_ has joined #openstack-ironic15:00
*** lucasagomes is now known as lucas-afk15:02
* lucas-afk meeting15:02
*** dprince has quit IRC15:02
*** mjura has quit IRC15:03
* jroll also bbiab15:03
*** mgoddard has quit IRC15:04
*** daemontool_ has joined #openstack-ironic15:07
*** daemontool has quit IRC15:10
*** hshiina has quit IRC15:11
*** thrash is now known as thrash|biab15:13
*** rpioso has joined #openstack-ironic15:16
*** moshele has quit IRC15:17
*** rbudden has quit IRC15:19
*** moshele has joined #openstack-ironic15:24
JayFmat128: why under /draft/?15:25
*** cdearborn has joined #openstack-ironic15:25
*** cdearborn has quit IRC15:26
*** harlowja_at_home has joined #openstack-ironic15:27
*** jheroux has joined #openstack-ironic15:27
*** harlowja_still_a has joined #openstack-ironic15:29
*** harlowja_still_a has quit IRC15:30
*** cdearborn has joined #openstack-ironic15:30
*** pester has quit IRC15:34
*** anush___ has quit IRC15:34
*** sahil has joined #openstack-ironic15:34
sahilHello15:34
sahilI am trying to setup ironic to boot baremetal nodes as a part of devstack running in a Vm15:35
sahilSo far with help from great guys here, I have been able to setup devstack with ironic such that the baremetal is able to get the ip address  from the dhcp run by neutron15:36
sahilit is stuck after that, Cannot get any tftp server15:36
sahilMy question is15:36
sahilwill I have to set it up myself15:36
sahilor where to look for the default cirros based set up15:37
sahilI could not find tftp server running or any files under the tftp folder15:37
*** rbudden has joined #openstack-ironic15:37
sambettssahil: tftp should all be setup by devstack/ironic, the tftp folder for devstack is in /opt/stack/data/15:40
sambettsunder ironic15:40
*** e0ne has quit IRC15:41
sahilthanks will look ther15:41
sahilfound it15:43
sahilbut it is empty15:43
sambettsif the node is failing to find the TFTP server, you might need to check the IP address being configured on the node, or another thing to investigate is if your devstack VM is receiving the packets from the node, by doing a tcpdump on the interface15:43
sambettsthat your baremetal talks to the VM on15:44
sahilfolder /opt/stack/data/ironic/httpboot/pxelinux.cfg has the mac address file15:45
sahilbut /opt/stack/data/ironic/tftpboot/pxelinux.cfg15:45
sahilis empty15:45
sambettsI belive that might have something to do with how we chain together TFTP PXE to iPXE for HTTP15:46
sambettsJayF: do you know more about this ^15:46
sahilHi Sambetts15:47
JayFsambetts: not really :/15:47
sahilSo here is where I am stuck15:47
sahilwhen my node boots and gets the ip addres from the range I specified15:47
sahilfrom devstack15:47
sahilthen gets an pxe11 error15:47
sahilon doing tcpdump on the server running devstack15:48
*** MattMan has quit IRC15:48
sahilI saw errors15:48
*** MattMan has joined #openstack-ironic15:48
sahil"Looking for <devstack hostname> tell <baremetal node ip address>15:48
patchbotError: No closing quotation15:48
sahiland it failed at that15:48
*** Sukhdev has joined #openstack-ironic15:49
sahilso my node definetely is able to listen and get ip address from the devstack VM15:49
sahilbut nothing after thtat15:49
sahilthat*15:49
*** moshele has quit IRC15:50
*** joprovost has joined #openstack-ironic15:50
sambettssahil: that is an arp timeout, so your devstack VM isn't responding to ARPs15:51
sahilyes yes15:51
sambettssahil: does your devstack VM have 2 NICs in the same subnet15:51
sambetts?15:52
sahil2 nics15:52
*** causten has quit IRC15:52
*** hoangcx2 has quit IRC15:52
sambettsare they in the same network or different networks? I've seen cases where having 2 nics in the same network prevents a machine responding to arps15:53
lucas-afksahil, if you are using iPXE the configuration files will be under /opt/stack/data/ironic/httpboot/pxelinux.cfg15:53
*** causten has joined #openstack-ironic15:53
sahilnope15:53
openstackgerritMerged openstack/ironic: __ne__() unit tests & have special methods use (self, other)  https://review.openstack.org/36764315:53
sahilI dont think so15:53
*** rcernin has quit IRC15:53
sahilbut my vm has multiple nics15:53
lucas-afkthere's two files to look at when chainloading /opt/stack/data/ironic/httpboot/boot.ipxe which will try to find a file that matches the mac address being booted under /opt/stack/data/ironic/httpboot/pxelinux.cfg/<mac>15:54
sahilone for internet, one for baremetal vlan, one for ipmi15:54
lucas-afksahil, it's only created when ironic is deploying the node15:54
* lucas-afk goes15:54
openstackgerritMerged openstack/ironic: Replace six iteration methods with standard ones  https://review.openstack.org/36867615:55
*** hshiina has joined #openstack-ironic15:55
sambettssahil: as long as your NICs are all in different subnets that shouldn't be causing a problem then15:55
*** pcaruana has quit IRC15:56
*** rbartal has quit IRC16:02
sahilsambetts16:03
sahilsorry*16:03
sahilsambetts: will it be a problem if the bridge on the host also has an ip address from the same subnet16:04
*** lucas-afk is now known as lucasagomes16:04
* lucasagomes is back16:04
sahilI am using the ip address designated to the bridge on the host as the gateway16:04
*** ChubYann has joined #openstack-ironic16:08
*** daemontool_ has quit IRC16:09
*** daemontool has joined #openstack-ironic16:10
sambettssahil: a server shouldn't normally have 2 nics in the same subnet, and if you do it can cause problems with things like ARP16:10
sahilsambetts: let me clarify16:11
sahilI connect the devstack VM to a bridge on the host16:11
sahilthis bridge has a ip address x.x.x.11 which also serves as the gateway16:11
sahilbetween the node and devstack16:12
sahilso is that a wrong setup16:12
sahilbefore this the node would not even get a IP address from the devstack dhcp16:12
*** ifarkas is now known as ifarkas_afk16:12
sambettsthe host to VM config should be ok if the VM is receiving the ARP requests, the problem is the VM not responding which can happen if linux sees that it has 2 network interfaces on the same subnet16:14
*** joprovost1 has joined #openstack-ironic16:17
*** joprovost has quit IRC16:19
*** derekh has quit IRC16:21
*** joprovost1 has quit IRC16:22
*** milan has quit IRC16:23
*** joprovost has joined #openstack-ironic16:23
*** thrash|biab is now known as thrash16:26
*** sahil has quit IRC16:26
*** aNupoisc has joined #openstack-ironic16:26
*** aNupoisc has quit IRC16:27
*** joprovost has quit IRC16:27
*** causten has quit IRC16:29
*** anush__ has joined #openstack-ironic16:29
lucasagomesjroll, think i found it, the import is not the problem... Apparently ironic/cmd/conductor.py -> ironic/common/service.py -> ironic/objects/base.py -> ironic/conductor/rpcapi.py -> ironic/conductor/manager.py16:29
lucasagomesI mean the CONF is not the problem*16:30
jrolloh jeez16:30
lucasagomesso manager.py gets imported before we run that prepare service, at that point the periodic tasks are created already16:30
lucasagomesball of wax16:30
*** sdake_ has joined #openstack-ironic16:30
*** jpich has quit IRC16:32
*** krtaylor has quit IRC16:32
*** anush__ has quit IRC16:34
NobodyCamGood Morning Ironic'ers :)16:34
*** harlowja_at_home has quit IRC16:35
*** rama_y has joined #openstack-ironic16:35
sambettslucasagomes: \o/ we can make some candles!16:35
lucasagomesNobodyCam, morning16:36
lucasagomessambetts, hahah16:36
NobodyCammorning lucasagomes, sambetts, jroll :)16:36
mgouldmorning NobodyCam16:36
NobodyCammorning mgould :)16:37
openstackgerritRicardo Araújo Santos proposed openstack/ironic: Fixes parameters validation in SSH power manager.  https://review.openstack.org/36615016:39
rloodtantsur: no one approved this? https://bugs.launchpad.net/python-ironicclient/+bug/159075216:39
openstackLaunchpad bug 1590752 in python-ironicclient "[RFE] --wait flag for node-set-provision-state" [Wishlist,In progress] - Assigned to Michael Turek (mjturek)16:39
rloohi and good morning ironic'ers!16:40
mgouldmorning rloo16:40
rloodtantsur: don't worry about it; i'm going to approve it.16:40
NobodyCammorning rloo :)16:41
rloomorning NobodyCam, mgould!16:41
lucasagomesjroll, sambetts false alarm. It's coming from "from ironic.api import app", moving the import to inside the class it's called gets the conductor to load the configurations correctly (which in my case I know because it will fail to start since because I put a bogus value at the sync_power_state_interval)16:44
lucasagomesproblem is how to test it16:44
*** ohamada has quit IRC16:44
*** ametts has quit IRC16:45
sambettslucasagomes: thats good :) but yeah :/ hard to test16:45
sambettsthis isn't the first time we've had problems with configurations being used on module load... we should really stop doing that :/16:46
lucasagomessambetts, yeah, I'm wondering whether having a reload() before starting the ConductorManager is actually more beautiful then making sure the imports are done in the correctly order16:47
lucasagomessambetts, or, futurist should really accept a function callback for it's interval16:47
lucasagomesI've openend a RFE about it16:47
sambettsusing functions like that makes things much nicer, thats basically how I solved it for the configurable defaults in the node object16:48
lucasagomeshttps://bugs.launchpad.net/futurist/+bug/162261216:48
openstackLaunchpad bug 1622612 in futurist "[RFE] Periodic tasks: Allow passing a function callback to the spacing parameter" [Undecided,New]16:48
lucasagomessambetts, indeed16:48
lucasagomessambetts, also is SIGHUP friendly16:48
sambettsyup yup16:49
lucasagomeswhich right now is impossible16:49
* mgould -> home; good night everyone!16:50
sambettscya mgould16:51
mgouldI'd stay for the meeting, but I'm still not quite feeling 100%16:51
*** mgould is now known as mgould|afk16:51
NobodyCamhave a good night mgould|afk16:51
mgould|afkNobodyCam: thanks :_)16:51
*** milan has joined #openstack-ironic16:51
lucasagomesmgould|afk, see ya, get better soon16:52
dtantsurmorning rloo, thanks!16:53
dtantsurmorning NobodyCam as well :)16:53
rloodtantsur: :)16:53
NobodyCammorning dtantsur :)16:54
*** tesseract- has quit IRC16:56
dtantsurnearly meeting time. anybody typing faster than me wants to chair it (jroll is out)16:58
dtantsur?16:58
*** jaosorior has quit IRC16:59
lucasagomesdevananda, ^ maybe ?!17:00
*** lindycoder has joined #openstack-ironic17:00
devanandaohhai!17:00
lucasagomesdevananda, heh morning17:01
devanandaI didn't realize jroll was out - wasn't prepared to chair but happy to help17:01
jrolldevananda: sorry, kinda last minute17:01
*** fragatina has joined #openstack-ironic17:01
devanandano worries. I just would have prepped a bit :)17:01
*** chihhsin_ has quit IRC17:02
*** krtaylor has joined #openstack-ironic17:06
*** sdake has joined #openstack-ironic17:06
dtantsurhave you seen https://wiki.openstack.org/wiki/Rsc folks?17:07
*** wajdi has joined #openstack-ironic17:08
*** wajdi has quit IRC17:08
*** wajdi has joined #openstack-ironic17:09
*** sdake_ has quit IRC17:09
*** chihhsin has joined #openstack-ironic17:09
*** rbrud has quit IRC17:11
lucasagomesdtantsur, cool I will take a look17:16
*** hshiina has quit IRC17:17
*** harlowja has joined #openstack-ironic17:19
sambettsvsaienk0: how does the multitenant CI deal with the fact the default sec-groups block the required communication for provisioning?17:20
*** joprovost has joined #openstack-ironic17:20
*** rbudden_ has joined #openstack-ironic17:22
*** rbudden has quit IRC17:24
*** rbudden_ is now known as rbudden17:24
*** electrofelix has quit IRC17:27
openstackgerritMerged openstack/ironic: Updated from global requirements  https://review.openstack.org/36810417:33
*** trown is now known as trown|lunch17:36
*** e0ne has joined #openstack-ironic17:38
*** fragatina has quit IRC17:42
*** harlowja has quit IRC17:43
*** Sukhdev has quit IRC17:44
openstackgerritMerged openstack/bifrost: Fix unbound variable error in scripts/collect-test-info.sh  https://review.openstack.org/36683517:44
*** harlowja has joined #openstack-ironic17:45
openstackgerritDevananda van der Veen proposed openstack/ironic-specs: [WIP] API Evolution  https://review.openstack.org/36758317:56
* sambetts -> home17:59
* dtantsur sends a email18:00
sambettssee everyone tomorrow o/18:00
*** sambetts is now known as sambetts|afk18:00
gabriel-bezerrasee ya18:01
dtantsuro/18:01
lucasagomessee ya dtantsur sambetts|afk18:02
*** dtantsur is now known as dtantsur|afk18:05
NobodyCamnight sambetts|afk dtantsur|afk lucasagomes18:05
gabriel-bezerrasambetts|afk: in our case, we have security groups removed by the port security extension set to false in the network.18:10
gabriel-bezerrasambetts|afk: not sure if it would work for your case18:10
*** milan has quit IRC18:10
*** anush__ has joined #openstack-ironic18:11
*** fragatina has joined #openstack-ironic18:13
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Ironic conductor to honor the interval values for periodic tasks  https://review.openstack.org/36900018:15
lucasagomesrloo, jroll ^18:15
lucasagomescalling it a day now :-)18:16
rloothx lucasagomes, will look later today. bye!18:16
lucasagomeshave a great evening all, talk to you tomorrow18:16
jrollcool, thanks lucasagomes, see you tomorrow18:16
lucasagomessee ya18:16
*** lucasagomes is now known as lucas-dinner18:16
*** _vsaienko has joined #openstack-ironic18:22
*** jheroux has quit IRC18:22
*** fragatina has quit IRC18:27
*** fragatina has joined #openstack-ironic18:27
rlooJayF: the software metrics is basically done, right? I want to remove it from the subteam report list in our etherpad18:32
JayFsure, go for it18:32
rlooJayF: done18:33
*** Romanenko_K has quit IRC18:34
*** causten has joined #openstack-ironic18:35
*** manjeets- is now known as msbhatia18:37
*** msbhatia is now known as manjeets-18:38
*** mrhillsman has joined #openstack-ironic18:39
*** causten has quit IRC18:40
*** rbrud has joined #openstack-ironic18:40
*** mrhillsman has left #openstack-ironic18:42
*** trown|lunch is now known as trown18:56
*** mgoddard_ has quit IRC18:59
*** causten has joined #openstack-ironic19:02
*** pcaruana has joined #openstack-ironic19:04
*** causten has quit IRC19:06
*** e0ne has quit IRC19:08
openstackgerritXavier proposed openstack/ironic: Add inspection feature for the OneView drivers  https://review.openstack.org/36706519:13
*** Sukhdev has joined #openstack-ironic19:14
*** moshele has joined #openstack-ironic19:24
*** _vsaienko has quit IRC19:26
*** _vsaienko has joined #openstack-ironic19:27
*** _vsaienko has quit IRC19:28
*** _vsaienko has joined #openstack-ironic19:34
*** joprovost has quit IRC19:36
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit  https://review.openstack.org/36902119:45
openstackgerritXavier proposed openstack/ironic: Add inspection feature for the OneView drivers  https://review.openstack.org/36706519:51
openstackgerritJarrod Johnson proposed openstack/pyghmi: Fix multiple console object behavior  https://review.openstack.org/36902619:52
*** daemontool has quit IRC19:59
*** bfournie has quit IRC20:11
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit  https://review.openstack.org/36902120:14
openstackgerritJarrod Johnson proposed openstack/pyghmi: Fix multiple console object behavior  https://review.openstack.org/36902620:17
*** Goneri has quit IRC20:17
*** vishwanathj has quit IRC20:20
*** vishwana_ has joined #openstack-ironic20:20
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit  https://review.openstack.org/36902120:21
*** sdake_ has joined #openstack-ironic20:25
*** sdake has quit IRC20:28
*** d0ugal has quit IRC20:28
*** Sukhdev has quit IRC20:31
*** Sukhdev has joined #openstack-ironic20:32
*** rbudden_ has joined #openstack-ironic20:36
*** rbudden has quit IRC20:38
*** rbudden_ is now known as rbudden20:38
*** d0ugal has joined #openstack-ironic20:44
xavierrironicers, I have a question. whats the difference between 'api_version' and 'os_ironic_api_version' here http://docs.openstack.org/developer/python-ironicclient/api/ironicclient.client.html#ironicclient.client.get_client ?20:44
*** trown is now known as trown|outtypewww20:45
*** pcaruana has quit IRC20:46
jlvillalxavierr: So api_version is the API version of the Ironic API20:48
jlvillalxavierr: os_api_version, I'm guessing is the OpenStack (os) API version. Maybe that is for keystone?20:48
* jlvillal thinks it is for keystone20:49
jlvillalxavierr: I'm pretty sure the os_api_version is for Keystone. For doing the authentication.20:49
TheJuliaI _think_ api_version is major version, which we only have version 120:49
TheJuliaif memory serves, os_ironic_api_version is for passing a specific microversion down to the client to use20:50
jlvillalTheJulia: I don't believe so. You can but in: api_version 1.1920:50
*** Sukhdev has quit IRC20:50
jlvillalOh, I missed that it was os_IRONIC_api_version20:50
TheJuliaIt has likely changed since I asked the same question then :)20:50
jlvillalTheJulia: I don't see os_IRONIC_api_version20:50
jlvillalTheJulia: Now I see20:51
jlvillalCTRL-F is my friend20:51
*** Sukhdev has joined #openstack-ironic20:51
jlvillalrloo: Do you know ^^^^  Since you have been a lot of client work recently20:51
* TheJulia goes back to packing20:52
rloojlvillal: scrolling back...20:52
rlooapi_version and os_ironic_api_version are the same thing xavierr, jlvillal20:52
jlvillalrloo: Weird20:53
rloojlvillal, xavierr: oh, sorry20:53
rlooapi_version is 120:54
rlooos_ironic_api_version is 1.x, the microversion20:54
jlvillalrloo: Like:  1.10   ?20:54
jlvillalThe entire "1.10"20:54
rloojlvillal: yeah, for os_ironic_api_version, 1.10 or 1.22 i think is the highest right now20:55
rloojlvillal: makes me wonder if we can get rid of api_version...20:55
jlvillalrloo: I wonder if it is even being used?20:55
rloojlvillal: they are both being used20:56
rloojlvillal: i just looked. but we can get api-version from os_ironic-api-version.20:56
rloojlvillal: the client needs more attention20:56
jlvillalrloo: Yeah seems confusing to have both20:56
*** bfournie has joined #openstack-ironic20:57
rloojlvillal: yup. we must have missed that. worth looking into, maybe deprecate api_version.20:58
TheJuliaI seem to remember there was an intentional reason20:59
TheJuliabut we've gone back and forth regarding versioning quite a bit20:59
*** mmedvede has quit IRC21:00
rlooTheJulia: and we are bad at recording any intentional reasons :-(21:00
*** moshele has quit IRC21:01
TheJuliaI think this is more something that would have to shake out with v2 api discussions because the case could be made that I might want the latest of v2, or a specific v1 api via different means, or "i as a user don't care", and really all the logic is in the api so the client really doesn't care anyway...21:03
xavierroh, ok! tks jlvillal TheJulia rloo ;)21:03
rlooTheJulia: i was wondering about that, a mix of 1 and 2, but i don't think that makes sense.21:05
rlooTheJulia: a V2 would not support any v1.x21:05
xavierrother thing that makes me confused is, creating client instances for ironic and nova, for example. Nova uses something like 'my_client_variable.Client(parameters) ironic does something like 'my_client_variable.get_client(parameters)'21:08
xavierrthere is not a unique way of doing that :/21:08
TheJuliarloo: but an intermediate library that is focused on providing consistency might need to mix things up a little21:09
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit  https://review.openstack.org/36902121:18
rlooTheJulia: possibly although i would hope not. i mean, if you want v1, you specify v1, if you want v2, you specify v2.21:22
*** rpioso has quit IRC21:22
rlooTheJulia: whether we have an intermediate library or not, should be hidden from the user.21:22
*** penick has joined #openstack-ironic21:27
openstackgerritHugo Nicodemos proposed openstack/ironic: Reusing oneview_client when possible  https://review.openstack.org/35804121:32
*** rpioso has joined #openstack-ironic21:36
*** mmedvede has joined #openstack-ironic21:37
*** Goneri has joined #openstack-ironic21:38
*** sdake_ has quit IRC21:38
*** sdake has joined #openstack-ironic21:39
TheJuliarloo: agreed, although at that level, I don't think it is really being exposed to the user, more so the developer, because the main version is not really exposed, if I am remembering correctly.  But it has been a while since I dug through the client.21:41
xavierrTheJulia please, could you add this patch in you review list? :)21:45
xavierrhttps://review.openstack.org/#/c/367065/21:45
patchbotpatch 367065 - ironic - Add inspection feature for the OneView drivers21:45
rlooTheJulia: OH. You're right. I thought that the api_version and os_ironic_api_version were args that the user could specify. it isn't, at least not at the CLI level.21:45
TheJuliaxavierr: if you can add me, that would be awesome.  If not I'll try to remember.  On vacation this week.21:46
xavierrI'll add you, thank you!21:47
*** sdake_ has joined #openstack-ironic21:47
xavierrTheJulia: so, are you working on your vacation? what?21:48
TheJuliaxavierr: thank you!21:48
*** wajdi_ has joined #openstack-ironic21:48
TheJuliaxavierr: I'm an IRC addict :)21:49
*** sdake has quit IRC21:49
*** lindycoder has quit IRC21:50
*** sdake has joined #openstack-ironic21:50
xavierrTheJulia: oh, I see.. ;)21:51
xavierrTheJulia: Will you be in barcelona summit?21:51
*** wajdi has quit IRC21:51
*** sdake_ has quit IRC21:52
jrollTheJulia: "I swear I'm only online to talk to my friends!"21:52
* jroll knows that feeling21:52
*** causten has joined #openstack-ironic21:52
TheJuliaxavierr: I don't know yet21:52
TheJuliajroll: exactly!21:53
*** wajdi has joined #openstack-ironic21:53
xavierrhahaha21:53
jrollxavierr: re: that patch, the create_if_enabled should stop it from initializing the inspector module and blowing up, no?21:54
jrolland log.info21:55
jrollhttps://github.com/openstack/ironic/blob/master/ironic/drivers/modules/inspector.py#L6321:55
*** Goneri has quit IRC21:56
*** wajdi_ has quit IRC21:56
xavierrjroll: I think a better approach would be load the driver and keep the inspect interface unavailable (false) when no inspector enabled21:57
xavierrin this case, we could keep only 2 drivers for OneView21:57
*** wajdi has quit IRC21:57
xavierrjroll: wyt?21:57
*** rbrud has quit IRC21:58
*** amotoki has quit IRC21:58
jrollxavierr: why is that better? why not do what every other driver does?21:59
jrollcreate_if_enabled() only logs and then doesn't load the inspector driver if it's disabled, it's the same end result21:59
*** ccamacho has quit IRC22:00
jrolland keeps it unavailable, and marked false in node-validate22:00
jrollsame end result, really22:00
xavierrjroll: ok, but how oneview drivers users will use that drivers without a inspector enabled? I was trying to load the driver doing like you are saing and it crashes the conductor :/22:01
jrollO_o it shouldn't22:02
xavierryes, it shouldn't, but it does :(22:02
jrollyou had CONF.inspector.enabled=False?22:02
jrollwell we should fix that problem, then :)22:02
xavierryeap22:02
jrollhowever that's how the gate runs, afaik, and it doesn't do that22:02
xavierrawesome!22:02
mrdaMorning Ironic22:03
jrollhey mrda :)22:04
xavierrif it could run without crash the conductor will be the perfect case, jroll!! :D22:04
xavierrmrda: hi there22:04
jrollxavierr: well, yeah, that's what it should do :) maybe chat with dtantsur|afk tomorrow or something22:04
jrollI'd try to help now but it's 6pm and I need to mow the lawn22:04
mrdao/22:05
mrda(get off my lawn? :)22:05
*** cdearborn has quit IRC22:05
jrollheh22:05
jrollsee y'all tomorrow22:06
xavierrjroll: I'll talk to you tomorrow about the inspect interface. night and thank you!22:06
xavierro/22:07
*** zigo has quit IRC22:08
*** michauds has quit IRC22:10
*** zigo has joined #openstack-ironic22:11
*** zigo is now known as Guest4138822:11
NobodyCamnight jroll :)22:12
*** mkam has quit IRC22:14
*** mkam has joined #openstack-ironic22:14
*** mkam has quit IRC22:15
xavierrhey NobodyCam o/22:16
xavierrNobodyCam: long time no see you22:17
NobodyCamhey hey xavierr :) happy monday! :p22:17
xavierrhahaha22:17
*** Guest41388 is now known as zigo_22:17
NobodyCam:p22:17
notmorganooh i see people here like NobodyCam, TheJulia, devananda, and jroll. all names i recognize :P /me goes back under my rock.22:17
xavierrNobodyCam: are you in US??22:17
NobodyCamyep .. been up in San Jose for the last several months :)22:18
NobodyCamhey hey notmorgan :)22:18
xavierrNobodyCam: awesome!22:19
xavierrNobodyCam: take care. will you be in barcelona?22:19
NobodyCamI don't think I'll be making it this time :(22:20
xavierroh, thats a bad news... :(22:21
NobodyCamit is what it is :p ... hehehe22:22
*** leitan has quit IRC22:23
xavierrNobodyCam: I gotta go. Nice to see you again! night!!22:25
xavierr::)22:25
xavierroh, too many eyes...22:25
xavierr:)22:26
xavierrbetter22:26
NobodyCamlol ... night xavierr :)22:26
*** xavierr is now known as xavierr_afk22:26
*** rbudden has quit IRC22:40
*** rbudden has joined #openstack-ironic22:42
*** krtaylor has quit IRC22:45
*** sdake has quit IRC22:48
*** openstackgerrit has quit IRC22:48
*** openstackgerrit has joined #openstack-ironic22:49
TheJulianotmorgan: visit from under the rock often :)22:59
* notmorgan goes back to trying to eliminate a terrible bit of code from keystone.23:00
*** rbudden has quit IRC23:03
*** rama_y has quit IRC23:03
*** penick has quit IRC23:03
*** anush__ has quit IRC23:23
*** rpioso has quit IRC23:26
*** vinsh has joined #openstack-ironic23:34
stevemarnotmorgan: i appreciate all attempts of eliminating terrible keystone code23:38
notmorganstevemar: hehe23:47
* notmorgan starts saying keystone in randome channels to get stevemar to look there.23:48

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!