Tuesday, 2015-01-13

*** penick has quit IRC00:03
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/14669200:03
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/14669300:03
* NobodyCam feels like to many context switches has caused his brain to segfault ... maybe that it is just monday00:07
*** Marga_ has joined #openstack-ironic00:22
*** david-lyle has joined #openstack-ironic00:24
naohirotgood morning ironic00:28
*** penick has joined #openstack-ironic00:31
*** chlong has quit IRC00:35
*** Marga_ has quit IRC00:38
*** Marga_ has joined #openstack-ironic00:38
*** spandhe has quit IRC00:42
NobodyCammorning naohirot :)00:43
*** eghobo has quit IRC00:43
naohirothi good evening00:44
*** Masahiro has joined #openstack-ironic00:45
naohirotNobodyCam: yesterday was public holiday in Japan, when I look at the meeting page I thought that no meeting.00:45
* naohirot I'm reading the minute.00:45
naohirotNobodyCam: s/look/looked/00:46
*** david-lyle has quit IRC00:46
*** david-lyle has joined #openstack-ironic00:51
NobodyCamnaohirot: a lot was left there because the previous meeting was very lightly attened with holiday and folks on PTo so a bunch was left there for informational reasons00:51
naohirotNobodyCam: Yeah, I know everybody is busy :)00:54
NobodyCam:)00:55
naohirotNobodyCam: just one thing. what is PTo?00:56
NobodyCampersonal time off00:56
naohirotNobodyCam: Aha, I got it :)00:57
NobodyCam:)00:58
*** penick has quit IRC00:59
*** Marga_ has quit IRC01:00
*** nosnos has joined #openstack-ironic01:04
*** david-lyle has quit IRC01:04
*** nosnos has quit IRC01:04
*** nosnos has joined #openstack-ironic01:05
*** nosnos has quit IRC01:09
*** jay-s-b has quit IRC01:17
openstackgerritJoshua Harlow proposed openstack/ironic: Add state machine to documentation  https://review.openstack.org/14630201:22
*** chlong has joined #openstack-ironic01:25
*** nosnos has joined #openstack-ironic01:26
*** nosnos has quit IRC01:34
*** ryanpetrello has quit IRC01:35
*** achanda has joined #openstack-ironic01:35
*** david-lyle has joined #openstack-ironic01:37
Haomengmorning naohirot, NobodyCam01:40
naohirotHaomeng: good morning:)01:40
Haomengnaohirot: :)01:40
naohirotHaomeng: I have a question regarding get_sensors_data in Management driver I/F01:41
Haomengnaohirot: sure, go ahead:)01:41
*** chenglch has joined #openstack-ironic01:41
NobodyCammorning Haomeng I saw the your ping only this morning sorry :(01:42
Haomengnaohirot: yes, that interface is added by me01:42
naohirotHaomeng: currently there are two drivers which implement that are ipmitool and ipminative, right?01:42
HaomengNobodyCam: np, dont worry01:42
NobodyCam:)01:42
HaomengNobodyCam: just want to ask if we have vlan configuration guide for ironic, or our ironic does not support vlan now?01:42
Haomengnaohirot: and iLO driver I think01:43
naohirotHaomeng: Yes, but iLO just inherits ipmitool.01:43
Haomengnaohirot: yes01:43
naohirotHaomeng: I'm implementing get_sensors_data in iRMC Management Driver right now01:44
Haomengnaohirot: that is great:)01:44
NobodyCamHaomeng: I only know of the raqspace ml2 plugin but not sure thats what your looking for01:45
HaomengNobodyCam: I just found our installation guide which support flat01:45
naohirotHaomeng: as far as I know, ipminative's implementation seems quite general.01:45
NobodyCamHaomeng: https://github.com/rackerlabs/ironic-neutron-plugin01:45
*** jerryz has joined #openstack-ironic01:46
HaomengNobodyCam: yes, this one https://github.com/rackerlabs/ironic-neutron-plugin ?01:46
NobodyCamyep01:46
Haomengsame one:)01:46
naohirotHaomeng: The format of ipminative conforms the minimum requirements which are Sensor Type and Sensor ID.01:46
HaomengNobodyCam: so without such plug, we can not support vlan, right?01:46
Haomengnaohirot: yes01:47
Haomengnaohirot: these data are consumed by Ceilomter, only one consumer now, that is Ceilometer01:47
*** achanda has quit IRC01:47
HaomengNobodyCam: I know that is networking issue, not our Ironic issue, but we have the networking requirement for neutron01:48
naohirotHaomeng: my question is , what happen if ipmitool's report and ipminative's report are mixed in ceilemeter?01:48
HaomengNobodyCam: this neutron spec - https://blueprints.launchpad.net/neutron/+spec/ml2-mechanism-snmp-vlan01:48
Haomengnaohirot: depends on the node driver, if node driver is ipmitool, then we run with ipmitool and collect the sensor data, and for ipminative's sensor data, that is not rich then ipmitool I think01:49
Haomengnaohirot: so ipminative just support the min set sensor fields01:49
naohirotHaomeng: In case of iRMC implementation, iRMC returns sensor data in XML format01:50
*** achanda has joined #openstack-ironic01:50
Haomengnaohirot: for Ceilometer, only the queue message can be accepted, so can we convert xml to json format message and send via MQ?01:50
*** achanda has quit IRC01:51
NobodyCamHaomeng: that spec is Abandoned01:52
NobodyCamhttps://review.openstack.org/#/c/9042201:52
*** smoriya has joined #openstack-ironic01:52
*** david-lyle has quit IRC01:52
HaomengNobodyCam: yes, it is out of kilo plan01:52
naohirotHaomeng: I just follows the way ipminative does which means just conform the Sensor Type and Sensor ID requirement of python dictionary01:52
naohirotHaomeng: and then put sensor data in dictionary after converting xml to python dictionary.01:53
naohirotHaomeng: Is this Okay?01:53
HaomengNobodyCam: so do you know if we have no racespace's plugin, can we deploy ironic bm in the vlan env, and I want to do such testing, but not sure if neutron+ironic supports the bm provisioning with vlan networking01:53
Haomengnaohirot: yes, let me check the min required fields set01:54
naohirotHaomeng: I'm going to submit iRMC Management source code today into gerrit, so please comment on the source if you have time.01:55
Haomengnaohirot: sure, will do, my pleasure:)01:55
naohirotHaomeng: thanks!01:55
Haomengnaohirot: yw01:56
HaomengNobodyCam: and we have this one - https://review.openstack.org/#/c/143480/2/specs/kilo/network-provider.rst01:57
*** killer_prince is now known as lazy_prince01:58
lazy_princejroll: Hi.. there..?01:58
NobodyCamhe's prob out for the night I will ping him tomorrow morning to see if he has plans on new rev02:01
NobodyCamfor that spec02:01
NobodyCamspeaking of I've got to run out.02:01
NobodyCamhave a good night folks02:02
HaomengNobodyCam: ok, good night:)02:02
HaomengNobodyCam: nice dream:)02:02
Haomengnaohirot: the common data, message header, which sent to data is populated by these codes - https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L115802:03
Haomengnaohirot: so for the new driver, dont care these message header data, it is handled by common layer already02:04
Haomengnaohirot: we just need to implement the get_sensors_data with this format as return value - https://github.com/openstack/ironic/blob/master/ironic/drivers/base.py#L56602:05
Haomengnaohirot: to note the data format and 'Single quotation marks' are required02:06
naohirotHaomeng: Yes, converting dict to json is out of scope of Management driver, and what Management driver has to do is to return dict with Seonsor ID key in Sensor Type, right?02:06
Haomengnaohirot: yes, but it is option, we can return NotImplement as exception if we dont neet it02:07
*** ryanpetrello has joined #openstack-ironic02:07
Haomengnaohirot:  'raise NotImplementedError() '02:08
naohirotHaomeng: 'Sensor Reading': 'current value',  <--- is 'Sensor Reading' key just a sample or a part of requirement?02:08
Haomengnaohirot: yes it is used for Ceilometer sample02:09
Haomengnaohirot: will be populated the 'current value' as the ceilometer sample value02:09
Haomengnaohirot: this is required, else ceilometer will get none for sample value02:09
naohirotHaomeng: In case of iRMC, the default implementation is ipmitool's get_sensors_data(). Because iRMC Management Driver inherits ipmitool Management driver.02:10
Haomengnaohirot: make sense02:10
Haomengnaohirot: so if it works in iRMC platform, no new code is required, I think02:11
naohirotHaomeng: So I don't have to use 'raise NotImplementedError() '02:11
Haomengnaohirot: dont worry, I just want to say it is an option if we dont care the get_sensor_data for iRMC02:11
naohirotHaomeng: My implementation overwrite the default behavior if deployer specifies how to retrieve sensor data in config file.02:12
Haomengnaohirot: I am confusing here, if iRMC support ipmitool commands, why it is in xml format02:13
Haomengnaohirot: so I understand the ipmitool returns xml in iRMC, right, so we need to convert xml to json message format02:14
naohirotHaomeng: iRMC has an I/F to retrieve sensor data in proprietary protocol, which returns data in XML.02:15
Haomengnaohirot: ok, got:)02:15
naohirotHaomeng: do you know what ceilometer does using sensor data? I mean, does ceilometer draw graph or something?02:17
lazy_princeI have a question.. I have a setup where ironic and Neutron (VLAN) are on same server.. and I have a KVM VM registered in ironic using pxe_ssh driver.. When I provision an instance, its not pxe bootiong..02:18
jrolllazy_prince: kinda here now, I've been in and out today, what's up02:18
lazy_princedo i need to do any manual VLAN tagging on the brbm bridge..02:19
Haomengnaohirot: ceilometer just store the sample data into ceilometer database, and have the api for retrieving from other consumers which need ceilometer data as the source02:19
Haomengnaohirot: and ceilometer has simple UI in horizon can show some sample report02:19
naohirotHaomeng: how frequently does ceilomenter poll sensor data?02:20
lazy_princejroll: wanted to check if using ironic-neutron-plugin, can we have neutron and ironic running on two different VMs....?02:20
Haomengnaohirot: default is 10 mints, it is configuration option, can be changed on configuration time02:20
Haomenglazy_prince: so we have same default interval in ironic option02:20
lazy_princeif so, how does the VM traffic from VLAN will be forwarded to the ironic node and vice versa..02:20
Haomengnaohirot: send_sensor_data_interval=60002:21
naohirotHaomeng: Okay, ceilometer polls every 10 min and deployer can see a report in horizon.02:21
jrolllazy_prince: we run them on two different VMs. not sure I understand your question about forwarding to ironic node02:21
naohirotHaomeng: I got the overall picture, thanks!02:22
jrolllazy_prince: however we run a separate dhcp server that handles dhcp and tftp02:22
lazy_princejroll: as i plan to pxe_ipmitool as the ironic driver for node...02:22
Haomengnaohirot: but horizon will not show all ceilometer samples02:22
Haomengnaohirot: just part of them, let me double check02:22
lazy_princejroll: i thought you use config drive approach to get out of pxe and dhcp..02:22
jrolllazy_prince: if everything is on the same vlan it should be fine02:23
Haomengnaohirot: and ceilometer support two method to collect data, one is polling, another one is event based our message02:23
jrolllazy_prince: we still pxe boot the ramdisk in the same vlan as ironic/neutron/etc but use static dhcp configs02:23
naohirotHaomeng: My concern is whether or not ceilomenter does machine processing using sensor data.02:23
lazy_princewe plan to provide tenant isolation for the tenant and so tenant can create a network and provision an instance using that network..02:23
Haomengnaohirot: no02:24
Haomengnaohirot: ceilometer has no logic to consume the data, just store it in db and show in gui and can be retrieved by api02:24
naohirotHaomeng: If they do, I cannot change the format by myself, after releasing the iRMC driver.02:24
*** Marga_ has joined #openstack-ironic02:24
jrolllazy_prince: right, so that's what my spec outlines here https://review.openstack.org/#/c/143480 however this clearly won't work if the instance is pxe booted unless neutron/ironic is also on the tenant network02:25
*** Marga_ has quit IRC02:25
Haomengnaohirot: so far only ceilometer receive our sensor data02:25
jrolllazy_prince: I think we'll need to require local boot for that or do something weird02:25
*** Marga_ has joined #openstack-ironic02:25
lazy_princejroll: aha.. would using IPA resolve this issue..?02:26
naohirotHaomeng: I understood.02:26
Haomengjroll: one question, if we can support vlan with the spec  https://review.openstack.org/#/c/143480, I have interested with VLAN networking for our ironic02:26
Haomengnaohirot: :)02:26
jrolllazy_prince: yes, or lucas is working on local boot for the pxe driver which would also resolve it02:26
naohirotHaomeng: :)02:26
jrollHaomeng: yes, in that spec when I say "network" I mean "VLAN"02:26
Haomengjroll: that is great, so I understand without such new spec, we can not support vlan for neutron+ironic, just supports flat, right?02:27
jrollHaomeng: I think so, but maybe if everything is in the same vlan it will also work. "flat vlan"?02:28
Haomengjroll: if we have 3 vlans, ipmi vlan, tftp/pxe vlan, and tenant vlan, can we support?02:28
lazy_princejroll: do you think it would make sense to take out the tftp and make it user configurable so that i could configure remote/local tftp service.. this might help me resolve this for case where ironic and neutron are on different nodes..02:28
Haomengjroll: yes, if all in same vlan, it shoudl be same with flat, but th difference is from neutron dhcp port support difference between vlan and flat02:29
jrollHaomeng: no, not without that spec, how do you get to the tenant vlan if you need to pxe boot on the pxe vlan02:30
jrolllazy_prince: yeah, I think that would be a good thing02:30
lazy_princejroll: but does it fall in line with ironic philosophy..? do do you see any issues there..02:31
Haomengjroll: yes, so we need the spec to support *true* vlan:)02:31
jrolllazy_prince: I think it's a good idea, yes :) probably only needs a bug, not a spec02:32
jrollHaomeng: s/true/useful/ :)02:32
Haomengjroll: yes02:32
Haomengjroll: :)02:32
lazy_princeokay.. let me put a patch for this..02:33
lazy_princeanother thing. i am trying to test ironic with neutron (vlan)...02:33
Haomenglazy_prince: ++02:33
lazy_princewith KVM VM as baremetal node..02:33
jrolllazy_prince: it'll be a tough patch, though, as the tftp server needs the instance kernel/ramdisk02:33
lazy_princeis it supposed to work..?02:33
Haomenglazy_prince: I have a reference from intel, it should be usefur for us to understand ironic vlan env02:34
jrolllazy_prince: define "ironic with neutron (vlan)" everything on the same vlan or?02:34
Haomenghttps://software.intel.com/en-us/articles/physical-server-provisioning-with-openstack02:34
lazy_princenot on same vlan.. they are on different VLAN...02:35
Haomenglooks like the auther thnik neutron vlan mode dhcp does not support ironic as well, but not very sure, I am not neutron expert:)02:35
jrolllazy_prince: I have no idea if that should work, but I kind of doubt it02:35
lazy_princesay a tenant creates a network with VLAN tag XX02:35
lazy_princeand then provisions an instance using that network...02:35
Haomenglazy_prince: yes, we allow tenant to create vlan with tag id02:36
lazy_princeI kinda wanted to extend ironic neutron plugin to work with VMs too.. so wanted to check if we need to do this or if this is supported Out of Box..02:37
jrolllazy_prince: I have no idea if it's supported, sorry02:37
*** rwsu has quit IRC02:37
lazy_princeHaomeng: so when you say, we allow tenant to create vlan with tag id, do you mean ironic can create instances on that newly created network..?02:38
Haomenglazy_prince: yes that should be a normal use case I think, how do you think?02:38
Haomenglazy_prince: but these vlan are created by neutron02:39
Haomenglazy_prince: not sure if neutron vlan support Ironic requirments, such as our agent will call back to conductor etc02:39
Haomengthis is good reference, share to you - https://software.intel.com/en-us/articles/physical-server-provisioning-with-openstack02:40
Haomengthis article talked about flat vs vlan in ironic02:40
*** ramineni has joined #openstack-ironic02:41
Haomengso if we run with multi-vlan, the router is required to route traffic between conductor subnet and tenant subnet, right?02:42
lazy_princewell.. to me, it looks like it may not work as the VM is connected to brbm directly and taggig/untagging needs to be done by some one on the VM side too.. My thought was that brbm needs to do that for VM.. but who (if at all) is gonna configure that is what i wanted to find out..02:43
lazy_princeHaomeng: looked at the intel doc.. it talks about Physical server.. however, i am talking about VMs being treated as physical server.. not sure if it is supposed to work.. someone needs to confirm..02:47
*** Lotus907efi has quit IRC02:54
*** david-lyle has joined #openstack-ironic02:55
Haomenglazy_prince: yes, it is about physical server, just a reference for us I think02:55
*** rloo has quit IRC02:56
lazy_princeHaomeng: right and regarding the router, if its a tenant network created by tenant, do you also expect the tenant to create the routers for connectivity to conductor subnet..?03:03
lazy_princeHaomeng: my understanding was that the conductor subnet whould not be visible to the tenants..03:04
lazy_princes/whould/would/03:04
openstackgerritAnusha Ramineni proposed openstack/ironic: Add tests for ilo_deploy driver  https://review.openstack.org/14298503:13
*** Masahiro has quit IRC03:15
Haomenglazy_prince: but if user create its own subnet, that will be another different subnet with our conductor, right?03:26
Haomenglazy_prince: so for such network env, the router is required03:26
*** penick has joined #openstack-ironic03:26
*** Marga_ has quit IRC03:28
*** harlowja is now known as harlowja_away03:31
*** lazy_prince is now known as killer_prince03:33
*** harlowja_away is now known as harlowja03:39
*** dlaube has quit IRC03:46
*** Nisha has joined #openstack-ironic03:56
*** pcrews has quit IRC04:03
*** Masahiro has joined #openstack-ironic04:06
*** pcrews has joined #openstack-ironic04:08
*** penick has quit IRC04:12
*** david-lyle has quit IRC04:20
*** ryanpetrello has quit IRC04:22
*** pcrews has quit IRC04:30
*** achanda has joined #openstack-ironic04:32
*** rameshg87 has joined #openstack-ironic04:36
*** saripurigopi has joined #openstack-ironic04:36
*** pradipta_away is now known as pradipta04:36
saripurigopihi everyone, I've submitted new pxe_ucs driver BP for review @ https://review.openstack.org/#/c/139517/04:37
saripurigopi@ core team, could you take a look at it and provide feedback/approvals ?04:38
*** eghobo has joined #openstack-ironic04:38
*** eghobo has quit IRC04:41
Haomengsaripurigopi: sure, thanks for your bp05:00
saripurigopiThank you Haomeng.05:01
Haomengsaripurigopi: yw:)05:01
*** achanda has quit IRC05:20
*** killer_prince is now known as lazy_prince05:24
*** achanda has joined #openstack-ironic05:35
*** pensu has joined #openstack-ironic05:36
*** achanda has quit IRC05:37
*** harlowja is now known as harlowja_away05:42
*** teju has joined #openstack-ironic05:47
*** jerryz has quit IRC05:48
*** Marga_ has joined #openstack-ironic05:49
*** jerryz has joined #openstack-ironic06:00
*** lazy_prince is now known as killer_prince06:02
*** pcrews has joined #openstack-ironic06:02
*** killer_prince is now known as lazy_prince06:02
openstackgerritNaohiro Tamura proposed openstack/ironic: Add iRMC Driver and its iRMC Power module  https://review.openstack.org/14490106:06
*** Marga_ has quit IRC06:11
*** pcrews has quit IRC06:18
*** vinbs has joined #openstack-ironic06:24
*** Marga_ has joined #openstack-ironic06:31
*** Haomeng has quit IRC06:37
*** Haomeng has joined #openstack-ironic06:39
vinbsmorning ironic!06:47
vinbsHello Haomeng06:47
*** Haomeng has quit IRC06:48
*** Haomeng has joined #openstack-ironic06:50
*** ryanpetrello has joined #openstack-ironic06:52
*** lazy_prince is now known as killer_prince06:58
*** killer_prince is now known as lazy_prince07:15
*** ryanpetrello has quit IRC07:17
*** Marga_ has quit IRC07:17
*** Haomeng has quit IRC07:41
*** Haomeng has joined #openstack-ironic07:42
*** pcrews has joined #openstack-ironic07:44
*** Haomeng has quit IRC07:47
*** Haomeng has joined #openstack-ironic07:49
*** dlpartain has joined #openstack-ironic08:03
*** dlpartain has quit IRC08:03
*** dlpartain has joined #openstack-ironic08:03
*** dlpartain has left #openstack-ironic08:03
*** pcrews has quit IRC08:07
*** chlong has quit IRC08:09
*** Nisha has quit IRC08:12
*** k4n0 has joined #openstack-ironic08:18
*** ifarkas has joined #openstack-ironic08:22
*** datajerk has quit IRC08:25
*** datajerk has joined #openstack-ironic08:25
*** Haomeng has quit IRC08:27
*** Haomeng has joined #openstack-ironic08:28
*** saripurigopi has quit IRC08:31
*** jcoufal has joined #openstack-ironic08:32
*** Haomeng has quit IRC08:35
*** Haomeng has joined #openstack-ironic08:37
*** Haomeng has quit IRC08:43
*** Haomeng has joined #openstack-ironic08:49
*** rakesh_hs has joined #openstack-ironic08:52
*** pradipta is now known as pradipta_away08:57
*** Marga_ has joined #openstack-ironic08:59
*** pradipta_away is now known as pradipta09:00
*** romcheg has joined #openstack-ironic09:01
*** dtantsur|afk is now known as dtantsur09:03
dtantsurMorning!09:03
*** pradipta is now known as pradipta_away09:04
*** athomas has joined #openstack-ironic09:11
openstackgerritNaohiro Tamura proposed openstack/ironic: Add iRMC Management module for iRMC Driver  https://review.openstack.org/14680309:15
*** derekh has joined #openstack-ironic09:23
*** jistr has joined #openstack-ironic09:23
*** MattMan has quit IRC09:25
*** MattMan has joined #openstack-ironic09:26
*** mkerrin has quit IRC09:40
openstackgerritMerged stackforge/ironic-discoverd: Refactor stable API to be /v1/introspection/<UUID>  https://review.openstack.org/14647509:41
*** gsamfira_ has left #openstack-ironic09:42
*** igordcard has joined #openstack-ironic09:48
*** andreykurilin has joined #openstack-ironic09:58
*** kbs1 has quit IRC10:08
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Add options table and convenience methods to work with it  https://review.openstack.org/14680910:14
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Add options table and convenience methods to work with it  https://review.openstack.org/14680910:17
*** vinbs has quit IRC10:29
openstackgerritAlexis Lee proposed openstack/ironic: Avoid querying the power state twice  https://review.openstack.org/14655410:33
*** ryanpetrello has joined #openstack-ironic10:41
*** subscope has quit IRC10:43
*** ryanpetrello has quit IRC10:45
*** andreykurilin has quit IRC10:48
*** andreykurilin has joined #openstack-ironic10:48
*** chenglch has quit IRC10:50
*** subscope has joined #openstack-ironic10:56
*** smoriya has quit IRC10:56
*** Masahiro has quit IRC11:00
*** ramineni has quit IRC11:01
openstackgerritAlexis Lee proposed openstack/ironic: Distinguish between prepare + deploy errors  https://review.openstack.org/14613511:02
*** rameshg87 has quit IRC11:06
*** lucasagomes has joined #openstack-ironic11:09
Haomengdtantsur: good morning11:09
Haomengdtantsur: do you know if our ironic support neutron vlan network?11:09
dtantsurHaomeng, hi, sorry, I have no idea11:12
Haomengdtantsur: np:)11:13
Haomengnice day:)11:13
*** teju has left #openstack-ironic11:16
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Add introspect client call and switch functest to it  https://review.openstack.org/14647911:19
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Add options table and convenience methods to work with it  https://review.openstack.org/14680911:19
*** rakesh_hs has quit IRC11:20
dtantsurifarkas, please have one more look ^^^ once you have time11:28
ifarkasdtantsur, ok11:28
*** pelix has joined #openstack-ironic11:33
pensuHi all, could you share your views on this: http://lists.openstack.org/pipermail/openstack-dev/2015-January/054313.html11:33
*** chlong has joined #openstack-ironic11:33
dtantsurpensu, it's better to ask J*, bit I guess the problem there is assumption that Ironic does anything about DHCP, when provide is none11:37
*** kbyrne has quit IRC11:38
*** kbyrne has joined #openstack-ironic11:41
pensudtantsur: yeah, but in case of external DHCP, shouldn't there be a mechanism to let ironic/nova know that deployment is done, just wondering how Ironic will get to know that deployment is over?11:43
dtantsurpensu, how is it related to DHCP? It's up to a deployment driver to notify Ironic, e.g. PXE driver ramdisk has a vendor passthru for it11:44
pensudtantsur: okay, I didn't know that. The thing I am trying to understand is that earlier I was able to get the node state as active, now it doesn't get there and eventually times out, the only difference is I switched to external DHCP. May be, it has nothing to do with DHCP, but I haven't changed anything else.11:50
dtantsurpelix, the question is whether deploy even starts for you. What driver do you use?11:51
pelixdtantsur: mistype ;)11:57
dtantsurpelix, oh sorry :)11:58
dtantsurpensu, ^^^11:58
openstackgerritMerged stackforge/ironic-discoverd: Add introspect client call and switch functest to it  https://review.openstack.org/14647911:59
openstackgerritMatthew Gilliard proposed openstack/ironic: Check glance for new kernel and ramdisk on prepare  https://review.openstack.org/14278211:59
pensudtantsur: yeah, deployment is happening, I am using pxe_ipmitool diver.12:00
*** Masahiro has joined #openstack-ironic12:01
dtantsurpensu, how do you get ramdisk on a node? static dnsmasq configuration?12:01
pensudtantsur: No, I am not using dnsmasq, I have stopped neutron-dhcp-agent and I am running a dhcp server of my own.....that is taking care of all the requests...12:02
openstackgerritSyed Ismail Faizan Barmawer proposed openstack/ironic-specs: Local boot support with partition images  https://review.openstack.org/14363612:03
dtantsurpensu, so it serves the PXE ramdisk to everyone, right?12:03
pensudtantsur: yup...12:03
dtantsurpensu, so you did ensure that ramdisk gets there? also, how are you going to make deployed node boot instance, not deploy ramdisk again, after reboot?12:04
*** Masahiro has quit IRC12:05
pensudtantsur: I suppose that is being taken care by ironic itself, because as I deploy the cirros image, I am able to reach till login, but that is not being reflected in ironic server.....12:10
dtantsurpensu, Ironic can't do, as you told it not to manage your DHCP12:11
dtantsur(maybe I'm not the best source of information on this)12:11
pensudtantsur: okay, looks like I am missing something(or a lot!) here, let me have a fresh look, will get back to you on this...thanks....:)12:12
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Add options table and convenience methods to work with it  https://review.openstack.org/14680912:12
dtantsurpensu, np. you may try to ask jroll and JayF, they do use this dhcp=none mode12:13
* lucasagomes answered that email thread12:14
* dtantsur brb12:14
lucasagomespensu, I think it has to do with the extra dhcp configuration that Ironic passses to the DHCP server12:14
lucasagomeshttps://github.com/openstack/ironic/blob/master/ironic/common/pxe_utils.py#L262-L26612:14
lucasagomesit may be an authentication problem, just as a test, try to set the auth_strategy to "noauth" in Ironic and reset the service12:15
lucasagomesthen try to deploy again12:15
*** alexpilotti has joined #openstack-ironic12:15
*** pensu has quit IRC12:25
*** ryanpetrello has joined #openstack-ironic12:27
*** andreykurilin has quit IRC12:27
openstackgerritNaohiro Tamura proposed openstack/ironic: Add iRMC Driver and its iRMC Power module  https://review.openstack.org/14490112:31
*** ryanpetrello has quit IRC12:41
*** kbs has joined #openstack-ironic12:52
*** Masahiro has joined #openstack-ironic13:02
*** athomas has quit IRC13:03
*** ndipanov has quit IRC13:03
*** dprince has joined #openstack-ironic13:03
*** ndipanov has joined #openstack-ironic13:05
*** Masahiro has quit IRC13:06
*** Marga_ has quit IRC13:07
*** ryanpetrello has joined #openstack-ironic13:08
*** Marga_ has joined #openstack-ironic13:13
*** athomas has joined #openstack-ironic13:13
*** ndipanov has quit IRC13:23
*** Marga_ has quit IRC13:28
*** lintan has joined #openstack-ironic13:30
*** chlong has quit IRC13:30
*** igordcard has quit IRC13:33
*** jjohnson2 has joined #openstack-ironic13:36
*** ndipanov has joined #openstack-ironic13:36
*** aignatov has quit IRC13:37
*** lsmola has quit IRC13:38
*** k4n0 has quit IRC13:41
*** rloo has joined #openstack-ironic13:44
*** aignatov has joined #openstack-ironic13:45
*** achanda has joined #openstack-ironic13:46
*** lsmola has joined #openstack-ironic13:51
openstackgerritAlexis Lee proposed openstack/ironic: Distinguish between prepare + deploy errors  https://review.openstack.org/14613513:53
*** lazy_prince is now known as killer_prince13:55
openstackgerritNaohiro Tamura proposed openstack/ironic: Add iRMC Management module for iRMC Driver  https://review.openstack.org/14680313:59
NobodyCamGood morning Ironic, From the man maken coffee14:00
jjohnson2NobodyCam, morning14:01
NobodyCammorning jjohnson214:01
rloohey, morning NobodyCam, jjohnson214:03
NobodyCammorning rloo :)14:07
openstackgerritFrederic Lepied proposed stackforge/ironic-discoverd: added an eDeploy plugin  https://review.openstack.org/14659914:09
*** k4n0 has joined #openstack-ironic14:10
*** k4n0 has quit IRC14:10
lucasagomesNobodyCam, rloo jjohnson2 morning :)14:12
NobodyCammorning lucasagomes :)14:12
rlooafternoon lucasagomes14:14
dtantsurNobodyCam, rloo, jjohnson2, morning14:14
rloohi dtantsur!14:14
NobodyCammorning dtantsur14:19
NobodyCam:)14:19
openstackgerritImre Farkas proposed openstack/ironic: DracClient to handle ClientOptions creation  https://review.openstack.org/14687914:22
*** openstackgerrit has quit IRC14:22
*** openstackgerrit has joined #openstack-ironic14:23
openstackgerritMerged openstack/ironic: Avoid querying the power state twice  https://review.openstack.org/14655414:25
*** jerryz has quit IRC14:32
NobodyCamany one have a free second to take a quick look at a NOVA spec, maybe toss up a +1 if you agree, (https://review.openstack.org/#/c/133534) its to Enhance the ComputeCapabilities filter Nisha Has a ffe request out for it I see14:35
*** Marga_ has joined #openstack-ironic14:41
NobodyCambrb14:41
*** r-daneel has joined #openstack-ironic14:46
*** Masahiro has joined #openstack-ironic14:50
*** pcaruana|off| is now known as pcaruana14:51
*** killer_prince has quit IRC14:52
*** Masahiro has quit IRC14:55
*** killer_prince has joined #openstack-ironic14:55
*** killer_prince is now known as lazy_prince14:56
*** achanda has quit IRC14:56
BadCub_morning Ironic15:04
NobodyCammorning BadCub_15:04
rlooNobodyCam: are you planning (or devananda) on updating the fsm patches? I think they just require minor changes. https://review.openstack.org/#/c/14086815:07
NobodyCamrloo: I didn't get a chance to take a look yesterday15:08
rlooNobodyCam: ok. I'd make the changes but want to hold on to my +2 powers ;)15:09
rlooNobodyCam: on the other hand, if you haven't yet touched them. let me see.15:10
*** Lotus907efi has joined #openstack-ironic15:10
rlooNobodyCam: yeah, you've updated 140868 already.15:10
*** ndipanov_ has joined #openstack-ironic15:11
NobodyCamyea I fiddled with that one15:11
NobodyCamlooks like move the err_handler15:11
NobodyCamand some test changes15:11
Lotus907efiNobodyCam: With help from Deva I got iommu turned on for our overcloud systems15:12
NobodyCamawecome Lotus907efi :)15:12
Lotus907efishould this "trick" be written up somewhere?15:12
NobodyCamdid you by chance document what you had to do?15:12
Lotus907efithat is what I am intending to do15:12
NobodyCamyes I was just abiut to ask you15:12
*** ndipanov has quit IRC15:13
NobodyCammaybe a etherpad would work, as its a workaround at this point15:13
Lotus907efietherpad?15:13
jrollwhat is iommu?15:13
jrollalso, morning ironic :)15:13
Shrewsrloo: ah, good memory re: the ascii-flow use15:13
Lotus907efiinput/output memory management unit (IOMMU) is a memory management unit (MMU) that connects a direct memory access-capable (DMA-capable) I/O bus to the main memory15:14
rlooShrews: sometimes it isn't good to have a good memory. That diagram will be useful.15:14
rlooShrews: something to ask devananda I think15:15
Shrewsrloo++15:15
jrollinteresting15:15
rloomorning jroll15:15
jroll\o rloo15:15
NobodyCamLotus907efi: maybe here: https://etherpad.openstack.org/p/AddKernelOptionsToIronicDeploys15:16
NobodyCammorning jroll15:16
NobodyCammorning Shrews15:16
Shrewsmorning NobodyCam, et. al.15:17
jrolloh NobodyCam, you mentioned last night that you would ask me to push a new version of https://review.openstack.org/#/c/143480/15:19
jrollbut I don't see any reason to push a new version?15:19
openstackgerritNaohiro Tamura proposed openstack/ironic-specs: iRMC Management Driver for Ironic  https://review.openstack.org/13602015:20
NobodyCamjroll: I thought there was a question about vlan on it15:24
NobodyCamlet me take another look15:24
jrollNobodyCam: I need to answer that on the review15:24
jrollbut I did answer Haomeng in channel15:24
NobodyCamack.. thats what I was going to poke you about15:24
*** lucasagomes is now known as lucas-hungry15:24
jrolltl;dr when I say 'network' there I generally mean VLAN15:24
* lucas-hungry lunch time 15:24
NobodyCamenjoy lucas-hungry15:24
jrollyeah, let me respond on gerrit15:24
*** coolsvap is now known as coolsvap|afk15:25
NobodyCamawesome TY jroll :)15:25
Lotus907efiNobodyCam: I just save a revision of that etherpad doc15:26
Lotus907efitake a look and see if that is clear15:26
openstackgerritMatthew Gilliard proposed openstack/ironic: Adds get_glance_image_properties  https://review.openstack.org/14609915:26
NobodyCamwill do Lotus907efi :)15:27
*** jcoufal_ has joined #openstack-ironic15:29
*** jcoufal has quit IRC15:31
*** ndipanov_ has quit IRC15:32
NobodyCamLotus907efi: Thank you :)15:35
gilliardIs that check-tripleo-ironic-undercloud-precise-nonha (phew!) job voting?15:35
gilliardit usually says NON-VOTING on such jobs, but it isn't actually voting on my patches.15:36
*** pcrews has joined #openstack-ironic15:36
NobodyCamI think that got switched15:38
jrollgilliard: I don't believe so15:38
jrollorly15:38
NobodyCamI thought so ... Icbw15:39
NobodyCamgilliard: thats a real error (not yours) http://logs.openstack.org/99/146099/4/check-tripleo/check-tripleo-ironic-undercloud-precise-nonha/0d4f259/console.html#_2015-01-13_15_32_14_06515:39
gilliardOK. See how the 2 jobs above have (non-voting) against them though https://review.openstack.org/#/c/142782/815:40
gilliardOf *course* there isn't a problem with *my* code :)15:41
NobodyCamlol :)15:41
openstackgerritJim Rollenhagen proposed openstack/ironic-specs: Add pluggable network provider spec  https://review.openstack.org/14348015:42
jrollNobodyCam: added a note about VLAN vs network15:42
NobodyCamjroll: :) will take a look. soon-ish15:43
jrollyeah, might push another one quick15:43
NobodyCamugh I have a 10:00 conf call so I will have to leave our bug day earily15:44
openstackgerritJim Rollenhagen proposed openstack/ironic-specs: Add pluggable network provider spec  https://review.openstack.org/14348015:44
*** ndipanov_ has joined #openstack-ironic15:45
*** r-daneel has quit IRC15:49
openstackgerritNaohiro Tamura proposed openstack/ironic-specs: iRMC Virtual Media Deploy Driver for Ironic  https://review.openstack.org/13486515:52
dtantsurjroll, wrt https://github.com/rackerlabs/ironic-neutron-plugin 1. why not have it on stackforge? 2. why not use Neutron as git submodule instead of (IIUC) copying parts of it?15:54
dtantsurjroll, and g'morning :)15:54
jrollmorning dtantsur :)15:54
jrolldtantsur: this started as a PoC that ended up in production :P15:55
dtantsurjroll, oh I know that feeling :D15:56
jrollso 1) no reason for or against, just didn't do it15:56
jroll2) no idea15:56
jrollhowever15:56
jrollwe're re-working this so that it's a drop in extension for neutron15:56
jrolland AIUI won't need to copy anything from neutron15:56
dtantsurah good15:56
jrolldtantsur: morgabra wrote most of that, he can maybe speak to it better than I can15:57
jrollhe'll also be at the SF meetup if you can make it to that15:57
dtantsurjroll, no, SF is definitely unrealistic for me :(15:57
naohirotgood night all, I spent most of time for git rebase and merge thins, I think I got better understand by a lot of hands on.15:58
jrolldtantsur: I figured :(15:58
*** naohirot has quit IRC15:58
NobodyCambrb16:03
*** rameshg87 has joined #openstack-ironic16:04
rameshg87victor_lowther, can you please take a look at raid configuration spec: https://review.openstack.org/#/c/135899/16:07
vdrokhi everyone!16:10
rameshg87hello vdrok16:12
vdrokrameshg87, hi16:12
vdrokrameshg87, I have a small question about https://review.openstack.org/#/c/145254/16:12
jrollmorning vdrok rameshg87 :)16:12
rameshg87morning jroll :)16:12
rameshg87vdrok, yes16:12
vdrokmorning jroll16:12
vdrokrameshg87, looking at ilo drivers16:12
vdrokrameshg87, seems that doing clean_up on first deploying a node would break deployment?16:13
vdrokrameshg87, as it tries to delete floppy image from swift16:13
vdrokrameshg87, and unattach FLOPPY and CDROM?16:13
vdrokrameshg87, I'm talking about this addition https://review.openstack.org/#/c/145254/3/ironic/conductor/manager.py16:14
rameshg87vdrok, i think it's perfectly fine because everything required for deploy is redone in in prepare() and deploy() which follows it16:15
rameshg87vdrok, i can give a test with your changes on ilo driver and inform you16:16
* rameshg87 remembers how imporant 3rd party ci is :(16:16
vdrokrameshg87, I'm concerned about this one - https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ilo/common.py#L394-L42416:16
rameshg87vdrok, we are working on our ci and would have something in a week's time16:16
vdrokrameshg87, i was unclear a bit, it wont break, but will give errors to log16:17
vdrokrameshg87, when doing deploy, not rebuild16:17
rameshg87vdrok, oh i missed that part16:18
rameshg87vdrok, i hadn't seen patchset 3 infact16:18
rameshg87vdrok, i remember you were calling clean_up() only during rebuild in previous patchset16:18
vdrokrameshg87, agent and pxe seem to work fine with these changes16:18
rameshg87vdrok, why don't we do it only on rebuild ?16:19
jrollonce this fix lands, we should change our tempest jobs to switch images when rebuilding16:19
vdrokrameshg87, right, if doing it during rebuild will require to add some flag to do_node_deploy - whether it's rebuild or not16:19
jrolle.g. reproduce this in tempest, fix it, update tempest to use it16:19
rameshg87vdrok, doesn't that make more sense given that only rebuild requires clean_up16:19
rameshg87vdrok, yeah that's easy right ? you could pass rebuild=True to _do_node_deploy()16:20
rameshg87vdrok, just like do_node_deploy() gets that argument16:20
vdrokrameshg87, yup, just was trying to make it less code :)16:20
rameshg87vdrok, won't actually end up doing more work in the conductor ? :)16:21
vdrokrameshg87, now i'm in doubt :)16:21
rameshg87vdrok, clean_up() needn't be called for each deploy, but we will end up calling more infact16:21
rameshg87vdrok, in ilo driver, it ends up talking to the bmc (to eject the virtual media devices)16:22
* dtantsur brb16:22
vdrokrameshg87, so you think it's more appropriate to fix it in image_cache?16:22
rameshg87vdrok, i feel we should do it only for rebuild just like your previous patch16:22
rameshg87vdrok, nope, i feel this is much better and clear, just that it needs to be done only for rebuild :)16:23
rameshg87jroll, have some thoughts ?? ^^16:23
vdrokrameshg87, also it might be the case that some old images would be deleted that are used for current node16:23
vdrokrameshg87, and then should be downloaded again16:23
vdrokif doing clean_up16:23
jrollrameshg87: why doesn't just checking if the cached images are different, if so remove them, work?16:24
rameshg87jroll, work on won't work ?16:24
rameshg87jroll, work or won't work ?16:24
vdrokjroll, it will, just it seemed easier to fix it in conductor at first glance16:24
jrollvdrok: rameshg87: https://review.openstack.org/#/c/145254/1/ironic/drivers/modules/image_cache.py16:25
jrollwhy doesn't that method work (other than the actual code doesn't work)16:25
jrollseems like calling clean_up() might be overkill, idk16:26
*** ryanpetrello has quit IRC16:26
vdrokjroll, after looking deeper in code i tend to agree16:26
vdrokrameshg87, jroll, i think i'll switch to fix in image_cache16:27
jrollvdrok: cool, poke me when that's up and I'll review :)16:27
rameshg87vdrok, okay16:27
NobodyCamrloo: something like this: http://paste.openstack.org/show/jVjnrPLMkmX0QHxG644A16:27
* rloo looks16:27
rlooNobodyCam: does that code work? you have _state_save and _save_state. but yeah, that's the general idea ;)16:29
NobodyCamrloo: ya just quick mock up in paste screen16:30
rlooNobodyCam: could just use 'prev_prov' and 'prev_target' variables. or 'prev_states'. this is local so I don't think you need _16:30
NobodyCamjust wanted to see if it was what you have in mind16:30
*** ChuckC_ is now known as ChuckC16:30
NobodyCamrloo: ack.. will do for real now16:31
rlooNobodyCam: I can't see the rest of the code, but that comment may be somewhat odd there. not sure.16:31
rlooNobodyCam:  the comment about 'stash current states...'16:31
rlooNobodyCam: just thinking.  I suppose we oculd just add a 'if callback and err_handler' and not move the code. let me look at the code.16:32
NobodyCamrloo: ya I'll update to say save nodes states16:32
NobodyCamok16:32
NobodyCami'll brb16:32
rlooNobodyCam: your choice (I think I'm too familiar with the code), but I *think* it might be easier to understand if we just change it s/if err_handler/if callback and err_handler/16:34
*** dlaube has joined #openstack-ironic16:34
NobodyCamahh and not move the code :)16:35
rlooNobodyCam: yeah.16:36
rameshg87rloo, do you mean s/if err_handler/if callback and err_handler/ on the above ?16:36
rloorameshg87: yeah, if i understand your question16:37
rloorameshg87: does that make sense?16:37
rameshg87rloo, won't that mean that callback cannot be provided without an error handler ?16:37
rameshg87rloo, error handler is not a *must* for callback right ?16:37
rloorameshg87: we only need to deail with err_handler if callback is true, right?16:38
rloorameshg87: I mean make that change with the original code, not NobodyCam's example16:38
rameshg87rloo, ah okay :)16:38
rloorameshg87: I added a comment about it in the patch.16:38
rloorameshg87: good :)16:39
rameshg87rloo, i missed that point, makes sense16:39
*** Masahiro has joined #openstack-ironic16:39
*** jcoufal_ has quit IRC16:39
*** Marga_ has quit IRC16:41
*** Masahiro has quit IRC16:43
*** ChuckC has quit IRC16:44
*** zz_jgrimm is now known as jgrimm16:48
openstackgerritVladyslav Drok proposed openstack/ironic: Ensure that image link points to the correct image  https://review.openstack.org/14525416:49
dtantsurIf I'm not confusing anything, we have bug clean up party in 5 minutes, right?16:54
JayFAh I forgot about that16:55
openstackgerritRuby Loo proposed openstack/ironic-python-agent: Consistent way to set details for Error instances  https://review.openstack.org/14692416:55
JayFoooh16:57
JayFthat's an interesting review16:57
*** jcoufal has joined #openstack-ironic16:58
*** igordcard has joined #openstack-ironic16:58
victor_lowtherrameshg87: Sure, will take a look at the raid spec.16:59
rameshg87victor_lowther, thanks16:59
*** derekh has quit IRC16:59
*** rwsu has joined #openstack-ironic16:59
dtantsurWhat I'd like to do with you folks is to go throw bugs in "New" and "Undecided importance" sections of http://ironic-bugs.divius.net/17:00
dtantsursome are around for quite a while already. who is participating?17:00
JayFI'm here and will participate17:01
* dtantsur is looking at https://bugs.launchpad.net/ironic/+bug/1286229 reported by devananda17:01
JayFjroll: ^ you in?17:01
rloodtantsur: I'm willing to help ;)17:02
jrollJayF: yeah, going to finish a few other things real quick17:02
rloo(after I finish writing up a bug)17:02
dtantsurawesome :)17:02
JayFI'm looking at https://www.youtube.com/watch?v=STH9ZpeFH2o17:03
JayFerm, wrong link. LOL17:03
jrollbahaha17:03
dtantsurLOL17:03
JayFalthough that's  a hilarious video :P17:03
JayFI'm looking at -> https://bugs.launchpad.net/ironic/+bug/135882017:03
dtantsurso I believe https://bugs.launchpad.net/ironic/+bug/1286229 is no longer valid, seems like code was fixed: https://github.com/openstack/ironic/blob/master/ironic/conductor/utils.py#L12617:04
*** igordcard has quit IRC17:04
rlooJayF: I just added 1410372 -- one of you IPA guys should know if it is a bug or not. I wasn't sure.17:05
dtantsurJayF, yeah, I don't understand how to reproduce 135882017:05
JayFI think we apparently must have some tests17:06
JayFwhich run against a real mysql db if it exists17:06
JayFor something like that17:06
*** ChuckC has joined #openstack-ironic17:06
JayFI was going to spin up a container configured like he was talking about and try to repro17:06
dtantsurJayF, will be awesome17:06
* dtantsur is looking at https://bugs.launchpad.net/ironic/+bug/138697817:07
devanandamorning, all - sorry for being late17:07
JayFrloo: re: 1410372; I think it's basically if it's an exception we want to bubble up for Ironic to see, it should be a RESTError17:07
devanandagot pulled into the office today, so I'm onthe road right now17:07
NobodyCammorning devananda17:07
JayFrloo: but in case of things like UnknownNodeError; it failed lookup, so can't funnel a response back to Ironic17:07
NobodyCamdevananda: fyi: I'm updatingthe FSM patches atm17:07
dtantsurdevananda, o/17:08
devanandaNobodyCam: thanks17:08
JayFrloo: so I think for those, the answer for one may be yes, one may be no, and one may be "sometimes"17:08
JayFrloo: like probably we should catch HardwareManagerNotFound in the lookup_node() method and raise another exception like "LookupFailed", then HardwareManagerNotFound could be a RESTError (since it'd only bubble up *after* lookup)17:09
dtantsurso, what we can fix about 1386978 is to make sure we do report the actual SSH failure, even if logging level is WARNING17:10
rlooJayF: so how can that file be 'fixed'  so it is clear what it should be :-) Maybe you can triage the bug.17:10
dtantsurcurrently it's DEBUG level: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ssh.py#L26817:11
JayFrloo: I'm not sure either :) AIUI though, RESTErrors are ones that are intended to be serialized and shipped back to Ironic. That can only happen after lookup. So I sorta mentally categorized them as "pre-lookup (meaning we don't know what Ironic node we are)" and "post-lookup (meaning we know what Ironic node we are and are actively heartbeating)17:11
*** ifarkas has quit IRC17:11
rlooJayF: ah, ok. Probably some documentation could be added to that file then, and someone review those three to see which if any should be RESTErrors17:12
JayFjroll: ^ does my description of RESTError vs non-RESTError in Ironic seem sane?17:12
JayFif so I'll toss that in the bug report17:13
JayFand modify it to say "document this in the file, too"17:13
rloodtantsur: that makes sense.17:13
dtantsurright, I've updated the bug17:14
jrollJayF: I agree, though everything could be a resterror and it would still work as expected17:14
rloodtantsur: is the idea for all of us to work on one bug at a time?17:14
JayFIf that's true; then we should probably make everything a RESTError17:14
dtantsurrloo, depending on bug complexity17:14
dtantsurrloo, e.g. now I'd like all to have a look at https://bugs.launchpad.net/ironic/+bug/139743817:15
*** MattMan has quit IRC17:15
* lucas-hungry looks at the bugs assigned to himself17:15
*** Hefeweizen has joined #openstack-ironic17:15
dtantsurJayF, jroll, have you seen something like https://bugs.launchpad.net/ironic/+bug/1397438 ?17:16
*** penick has joined #openstack-ironic17:16
*** lucas-hungry is now known as lucasagomes17:16
* dtantsur is also looking at https://bugs.launchpad.net/ironic/+bug/139858217:16
JayFdtantsur: not likely something we'd see as we run with dhcp_provider=none and agent driver17:17
jrolldtantsur: no, haven't seen that but yeah what jay said17:17
JayFwhich means more or less any conductor can do anything in our stetup17:17
dtantsurah right17:17
JayFwhich lets us completely avoid those kind of bugs17:17
mjturekdtantsur, fyi I just grabbed the bug you updated. I'll patch that today17:17
rloodtantsur: so wrt 1397438, seems like someone would need to try to reproduce17:17
jrollhowever if the node went to active, dunno why another conductor would take it over17:17
jrollalso seems like that person may have flappy conductors17:18
rlooand when would a conductor be flappy/flacky?17:18
jrollI would guess that something would have to be wrong with rabbit in that case17:19
rloowith the hashring stuff, the node should be handled by the same conductor.17:19
jrollrloo: right, but if a conductor isn't heartbeating well, it might drop and the hash ring would change17:19
vdrokjroll, rameshg87, gilliard: just tested all 3 cases from unit tests https://review.openstack.org/#/c/145254/ - seems to work17:20
JayFrloo: make sure to tag agent bugs in the future with 'agent'. I use that search term often :D17:20
jrolldtantsur: 1398582 seems valid17:21
rlooJayF: ahh, forgot about that. Sorry.17:21
dtantsurjroll, yeah, I've updated it.17:21
JayFrloo: nbd, as I was going to add low-hanging-fruit anyway :)17:21
JayFSo I marked https://bugs.launchpad.net/ironic/+bug/1410372 as confirmed, low17:21
rloothx JayF17:22
dtantsurJayF, mark "Triaged", if way to fix the bug is clear17:22
dtantsurIIRC that's what we agreed on some time ago17:22
JayFdtantsur: done17:22
dtantsurthnx17:22
JayFdtantsur: I think I have an idea on https://bugs.launchpad.net/ironic/+bug/135882017:22
JayFdtantsur: "mysql" in fedora 20 is mariadb17:22
dtantsurJayF, should it make any difference?17:24
dtantsurI though this bug is about time out waiting for a partition...17:24
JayFit shouldn't, but it does in nova17:24
JayFlet me find the link to the ML17:24
*** ryanpetrello has joined #openstack-ironic17:24
JayFdtantsur: http://lists.openstack.org/pipermail/openstack-dev/2015-January/053872.html17:25
* dtantsur clicks17:25
victor_lowtherrameshg87: there ya go.17:25
JayFdtantsur: I'm tempted to kick that bug back, and ask the person if it was MariaDB or MySQL17:25
rameshg87victor_lowther, thanks17:26
rameshg87victor_lowther, will take a look17:26
dtantsurJayF, still, how do you think parititoning timeout is related to M{ySQL,ariadb}? Or am I missing something?17:26
rameshg87vdrok, okay :)17:26
jrollJayF: 1358820 seems more like an selinux thing17:27
jrolldtantsur: ^17:27
JayFoooh17:27
* JayF is just pondering on it while trying to get a test environment up17:27
jrollI don't think mysql/maria is related at all, it's related in nova because that's an actual db thing failing17:27
dtantsurwell, maybe selinux17:27
JayFwell I'll know soon eough17:28
JayFalmsot have maria up on this vm17:28
JayFdoes anyone know how to setup the "openstack_citest" this is talking about? https://bugs.launchpad.net/ironic/+bug/135882017:29
dtantsurno idea17:29
*** Marga_ has joined #openstack-ironic17:29
jrollJayF: I think that's just saying, that's the name of the database17:30
jrollbut idk17:30
*** Marga_ has quit IRC17:30
JayFthat's a special user name17:30
JayFused by CI17:30
JayFbut I can't find docs; only evidence that it's used in automation17:31
* dtantsur is looking at https://bugs.launchpad.net/ironic/+bug/1397792 in the meanwhile17:31
lucasagomesJayF, I don't know too, but may worth asking for instructions at #openstack-infra17:32
JayFI already have :)17:32
*** Marga_ has joined #openstack-ironic17:32
lucasagomesoh, right :D17:32
jrolldtantsur: https://bugs.launchpad.net/ironic/+bug/1397792 sounds like 1) a pbr thing, 2) do we care?17:34
*** Marga_ has quit IRC17:34
*** Marga_ has joined #openstack-ironic17:34
jrollwhy would you use setup.py install unless you're cloning from git?17:34
*** Marga_ has quit IRC17:35
dtantsurjroll, I tend to agree with you, but I'm reproducing the issue anyway. It won't harm to add these files to MANIFEST.in, will it?17:35
jrolldtantsur: will that fix it?17:35
openstackgerritChris Krelle proposed openstack/ironic: Enable async callbacks from task.process_event()  https://review.openstack.org/14086817:35
rloodtantsur: so the person that reported it, has assigned it to themself. Do we need to verify that it is a problem?17:35
NobodyCamrloo: ^^^ let me know ehat you think17:36
*** Marga_ has joined #openstack-ironic17:36
dtantsurjroll, I think so.17:36
jrollhuh, ok17:36
rlooNobodyCam: looking (don't tell dtantsur)17:36
dtantsurlol17:36
NobodyCamhehehe17:36
*** Marga_ has quit IRC17:36
dtantsurrloo, that's part of triaging :) we don't want him/her to waste time fixing what we don't want to fix17:36
*** ryanpetrello_ has joined #openstack-ironic17:37
rloodtantsur: ok, so you actually go through all the bugs even if status is not 'new'?17:37
NobodyCamquick brb then the rebasing17:37
dtantsurjroll, rloo, I'm going to all bugs w/o assigned priority. but NEW bugs are the most important17:37
*** ryanpetrello has quit IRC17:37
*** ryanpetrello_ is now known as ryanpetrello17:37
dtantsur(just wanting to get rid of easy once first)17:37
dtantsurinteresting, alembic.ini and pxe_config.template are installed17:40
dtantsurelilo_efi_pxe_config.template is not. wtf?17:40
jrollhuh17:40
jrollfresh clone?17:40
*** jistr has quit IRC17:40
dtantsurjroll, honestly it's not fresh, maybe that's the cause17:41
jrollyeah, maybe was there already17:41
* jroll looks for next bug17:41
jrollI'll go from the bottom of new17:42
jrollJoshNang: did you end up finding a fix for this? https://bugs.launchpad.net/ironic/+bug/140916917:42
*** vdrok is now known as vdrok_afk17:42
jrollslash root cause17:42
dtantsurbtw J's, please assign priorities to IPA bugs you report :)17:42
jrollyeah :P17:42
JoshNangjroll: yup, i had a fix running on that agent that was going crazy17:42
JoshNangi need to make it an actual patch though17:43
jrollJoshNang: can I assign this to you with medium priority then?17:43
JoshNangjroll: sure!17:43
openstackgerritMerged openstack/ironic-python-agent: Error classes invoke their parent's __init__()  https://review.openstack.org/14664317:43
jrollk17:43
jrollyou should push that patch :P17:43
*** Marga_ has joined #openstack-ironic17:45
JayFjroll: running tox on ironic in the cloud instead of locally = super fast17:47
jrollJayF: well aware :P17:47
*** ryanpetrello has quit IRC17:47
jrolladam_g: I don't see many fails for this, is this bug valid? https://bugs.launchpad.net/ironic/+bug/140806717:48
*** spandhe has joined #openstack-ironic17:49
adam_gjroll, i think its happening periodically, i have some devstack patches  up to make the failure result in an ironic error, so we have something to fingreprint and track17:50
jrolladam_g: ok, thanks17:50
adam_gill link on the bug17:50
jrollI'm going to mark that medium priority, I think17:50
adam_gk17:51
dtantsuradam_g, hi! could you assign yourself to https://bugs.launchpad.net/ironic/+bug/1408808 ? I guess, you're working on it17:52
adam_gdtantsur, done17:52
dtantsurthx17:52
jrollgoing to reproduce https://bugs.launchpad.net/ironic/+bug/140627017:52
adam_gwonder why the bot doesn't do that sometimes17:52
*** EmilienM is now known as EmilienM|afk17:53
dtantsuradam_g, ditto about https://bugs.launchpad.net/ironic/+bug/140129817:53
dtantsurand yeah, bot is not doing a good job nowadays17:54
JayFYeah, this fedora bug is valid17:55
JayFwe have tests escaping17:55
dtantsurJayF, ouch! please update the bug17:56
JayFironic/tests/drivers/test_deploy_utils.py for both of them17:56
dtantsurlucasagomes, is https://bugs.launchpad.net/ironic/+bug/1357878 fixed?17:56
*** ChuckC_ has joined #openstack-ironic17:57
* lucasagomes looks17:57
lucasagomesyes!17:57
* lucasagomes mark as committed 17:57
NobodyCamcheck-tripleo-ironic-undercloud-precise-nonha broken?17:57
*** Marga_ has quit IRC17:58
*** Marga_ has joined #openstack-ironic17:58
*** ChuckC has quit IRC18:00
*** Marga_ has quit IRC18:01
dtantsurNobodyCam, your opinion on FSM-related bug https://bugs.launchpad.net/ironic/+bug/1405420 ?18:02
rlooNobodyCam: just gave feedback on https://review.openstack.org/#/c/140868/18:02
rloodtantsur: wrt 1405420, that is a problem.18:02
NobodyCamwhat I'm getting pep8 errors on files i havent touched18:02
*** Marga_ has joined #openstack-ironic18:02
dtantsurrloo, could you triage the bug please?18:03
JayFhttps://bugs.launchpad.net/ironic/+bug/1358820 so for this, unit tests don't run on Fedora 20 as a normal user. At all. Ever.18:03
JayFI updated it with my comments, but curious what severity we'd mark that as18:03
rloodtantsur: i can change the status to triage. but i'm not sure what the solution is.18:03
JayFMedium or high?18:03
dtantsurrloo, then just Confirmed18:03
dtantsurJayF, ehhh.. I'm pretty sure I often run tests on F2018:04
dtantsur(and my often I mean always)18:04
dtantsurmy = by18:04
rloodtantsur: oh. looks like the diff between 'triaged' and 'confirmed' is that confirmed is verified by the 'bug supervisor'18:04
dtantsurJayF, any specific conditions to see the failure?18:04
JayFdtantsur: I did this: 1) Boot a fedora 20 cloud machine. 2) Install requirements as noted in our docs. 3) Run tox18:04
dtantsurrloo, IIRC OpenStack has slightly different idea. lemme find a link18:04
dtantsurJayF, then it's medium, I guess18:05
JayFdtantsur: selinux = permissive locally, perhaps?18:05
dtantsur$ getenforce   >> Enforcing18:05
NobodyCamdtantsur: on conf call now :(18:05
JayFdtantsur: and not as root?18:06
rloodtantsur: that's what I saw in that popup when I clicked the status thingy in the bug18:06
JayFthis screams undocumented dependency then :(18:06
dtantsurJayF, no, my regular user. lemme try with fresh clone, maybe I didn't run all tests since it was broken?18:06
jrollJayF: don't be sad, that's way easier to fix :P18:06
JayFjroll: well, the dependency would be on a file on disk it's trying to execute18:07
JayFjroll: and I'm not sure we should be actually execing to the system in unit tests18:07
dtantsurrloo, https://wiki.openstack.org/wiki/Bugs#Status18:07
jrollJayF: I agree18:07
rloothx dtantsur. I wonder if it is possible to link to that page from the bug. probably not.18:09
* dtantsur is taking a ~15 minute break18:09
rloodevananda: when you have a second, your opinion is desired on https://bugs.launchpad.net/ironic/+bug/1405420.18:10
*** rameshg87 has quit IRC18:10
rloodevananda: if something is in *FAIL provision state, what operations (or not) can be done on it afterwards.18:11
jrollouch18:12
jrollit's because target_provision_state is still set18:12
jrollnode-update should be able to be done any time a node is in a steady state18:12
jroll(e.g. target state is null)18:12
JayFhow do I ever get it out of deployfail otherwise18:13
JayFwithout massaging the db18:13
* JayF is going to end up fixing 1358820 because it's too annoying to not know what broke it18:13
openstackgerritJosh Gachnang proposed openstack/ironic: Improve IPA client library  https://review.openstack.org/11111818:14
JayFHow do you run a single test in tesr?18:16
JayFIf I wanted to isolate the test run so I could strace it18:16
JayFclif_h: morgabra: What's the Oakland hotel you guys stay at sometimes when coming into the office?18:17
clif_hJayF: I've never stayed in Oakland18:17
JayFokay, that must've just been Brad then18:17
morgabraumm, courtyard right by city center I think18:17
morgabrait was fine18:17
clif_hand I haven't booked my feb sf travel yet18:17
clif_hdoing that today18:17
morgabraJayF: ^18:17
JayFShrews: ^ so apparently the courtyard by oakland city center gets a good review from brad18:18
morgabraoh god18:18
clif_hhaha18:18
JayFapparently the hotel by the office is like $500/night18:18
JayFfor the mid-cycle timeframe18:18
morgabraas in, I traded money for a place to keep clothes and sometimes sleep18:18
morgabraand it satisfied those conditions18:18
clif_his it a conflict of interest to airbnb at a coworkers place? >:)18:19
JayFwell, I was more concerned about neighborhood18:19
*** spandhe has quit IRC18:20
jrollJayF: tox -e py27 ironic.tests.drivers.test_agent.TestAgentDeploy.test_validate for example18:20
jrollbasically the python path to the test18:20
JayFjroll: ty18:21
jrollnp18:21
JayFjroll: literally a minute before I was about to strace the whole run18:21
JayFlol18:21
jrolllol18:21
jrollthat would be fun18:21
JayFeh -e open to limit it to file opens18:21
dtantsurJayF, tox succeeded on a fresh clone...18:21
JayFand look for things in bin/18:21
* dtantsur goes back to break18:21
JayFdtantsur: I think there's an undeclared dependency18:22
jrollthis hotel discussion reminds me I meant to send an email about the meetup18:22
JayFdtantsur: that's not documented to install18:22
*** spandhe has joined #openstack-ironic18:22
JayFdtantsur: so you have it installed; cloud images don't18:22
JayFjroll: I just ran tox with my py27 venv activated :(((((18:23
jrollhaaaaaa18:23
jrollI don't feel for you, I'm running devstack on a new machine18:24
JayFWell I had copies of the venv in root/ and ~/ so luckily I'm covered, lol18:24
*** EmilienM|afk is now known as EmilienM18:24
lucasagomeslol18:28
lucasagomesJayF, if u have it activated you can also do a "testr run  ironic.tests..."18:28
jrollok, midcycle email sent18:28
*** Masahiro has joined #openstack-ironic18:28
openstackgerritNisha Agarwal proposed openstack/ironic-specs: Discover node properties for iLO drivers  https://review.openstack.org/10300718:30
jrollmorgabra: can you add that hotel here? https://wiki.openstack.org/wiki/Sprints/IronicKiloSprint#San_Francisco_Sprint18:31
* jroll wonders if there's cheaper hotels closer to the office18:31
*** Masahiro has quit IRC18:32
* dtantsur is back18:33
rloomjturek: I see you are working on https://bugs.launchpad.net/bugs/141037218:35
dtantsurre https://bugs.launchpad.net/ironic/+bug/1400882 by adam_g: I believe new state machine might fix it. Ideas?18:35
mjturekrloo yep!18:35
mjturekrloo, just need to change the base class for those three correct?18:36
*** afaranha has joined #openstack-ironic18:36
rloomjturek: I added some unit tests in a patch so you might want to leverage that. sec.18:36
JayFdtantsur: I found the bug; /tmp/tmp8u4Qa0/tmpSDcAqW (or a similarly named file, depending on run) is supposed to be created at some point; it appears to never have been created18:36
mjturekrloo cool cool, thanks18:36
dtantsurJayF, wow. I wonder why...18:37
JayFsome missing depedency :)18:37
JayFthat I am not clever enough to ahve found yet18:37
dtantsurJayF, if you try Python's mkdtemp, will it work at all?18:37
rloomjturek: https://review.openstack.org/#/c/146924/18:37
afaranhaHi, I'm trying to deploy a server using ironic but I got an error when deploying it, can anyone help?18:38
afaranhaI got this error: "Node 6461f204-39a0-4bcf-b88c-875471c05ece is locked by host localhost, please retry after the current operation is completed."18:38
JayFdtantsur: yep18:38
NobodyCamafaranha: what driver are you useing?18:39
mjturekrloo thanks! I'll take a look18:39
NobodyCamusing even18:39
jrollafaranha: sounds like a stuck lock, run in mysql: update nodes set reservation=null where uuid='6461f204-39a0-4bcf-b88c-875471c05ece';18:39
*** harlowja_away is now known as harlowja18:39
afaranhaipmitool18:39
afaranhaI had the same problem as described here18:39
afaranhahttps://software.intel.com/en-us/articles/physical-server-provisioning-with-openstack18:39
adam_gdtantsur, sorry, was OTP.18:39
rlooafaranha: could be the periodic task to sync power18:39
afaranhathe system loads the ramdisk and the kernel, and then it says it's ready18:39
afaranhaand stuck there18:39
dtantsuraren't we retrying?18:40
afaranhajroll, I tried this but it's not retrying18:40
rloodon't we log the retries?18:40
afaranhaironic node-list returns this:18:40
afaranha| 6461f204-39a0-4bcf-b88c-875471c05ece | 457cc26c-030c-4e6d-aabd-b5e0a61d5bea | power on    | error              | False       |18:40
afaranha| 6461f204-39a0-4bcf-b88c-875471c05ece | 457cc26c-030c-4e6d-aabd-b5e0a61d5bea | power on    | error              | False       |18:40
afaranhaops18:40
afaranhaUUID                                 | Instance UUID                        | Power State | Provisioning State | Maintenance18:40
JayFdtantsur: AHA!18:41
JayFdtantsur: we require `fuser` but it's not installed on cloud images or documented18:41
JayFwoo18:41
dtantsuradam_g, NobodyCam, I suspect https://bugs.launchpad.net/ironic/+bug/1400882 will be fixed/made obsolete by new state machine. wdyt?18:41
dtantsurJayF, nice :)18:41
adam_gdtantsur, well, i think the current bug against the current state machine is actually the expected behavior. im not 100% sure the new state machine will fix it without trying, but i think that it should18:42
jrollafaranha: right, you need to try deploying again18:42
afaranhajroll, just delete the instance and retrying booting it?18:42
jrollafaranha: yes18:43
jrollafter clearing the lock like I mentioned18:43
dtantsuradam_g, I'm thinking of closing this as WONTFIX and seeing what we'll get with new state machine. wdyt?18:43
afaranhajroll, thats another problem I'm having, the instance is deleted only when I delete the node18:43
adam_gdtantsur, that sounds good. the issue as it is can be worked around on the operator/devstack side (properly polling resources before doing anythign)_18:44
jrollafaranha: if the node is locked, nova can't delete the instance18:44
afaranhabut I already set in the database the reservation to null18:44
openstackgerritJay Faulkner proposed openstack/ironic: Document dependency on `fuser` for unit tests  https://review.openstack.org/14694818:45
JayFdtantsur: others, ^ fix for 135882018:45
dtantsurthx!18:45
afaranhajroll, reservation is null, and if I set the instance_uuid also to null, I still unable to delete the instance :/18:45
jrollafaranha: what error does nova give you18:46
dtantsurJayF, don't we need to mention it in the installation guide as well?18:46
JayFdtantsur: this is a test-only depedency afaict18:46
JayFdtantsur: nope, you're right18:47
jrollno it's not18:47
JayFdtantsur: will update that18:47
jrollcool18:47
JayFif that's using packages, the ironic-api package should just depend on psmisc18:48
JayFand it not need it there18:48
jrollquestion is, does it depend on psmisc18:48
JayFhttp://packages.ubuntu.com/trusty/net/ironic-api and it doesn't18:48
* dtantsur is checking Fedora18:48
dtantsurJayF, why -api?18:49
JayFthey both have the same set of deps18:49
JayFand api is less characters18:49
JayFlol18:49
dtantsur:D18:49
JayFdtantsur: if it's yum installable from fedora/rhel, that's not documented either18:49
afaranhajroll, it just don't delete and show the error that was raised before:18:50
afaranha"Build of instance 457cc26c-030c-4e6d-aabd-b5e0a61d5bea aborted: Could not clean up failed build, not rescheduling", "code": 500, "details": "  File \"/opt/stack/nova/nova/compute/manager.py\", line 2067, in _do_build_and_run_instance18:50
jrollhow do I make launchpad show me closed bugs?18:50
JayFhmm. I also guess psmisc is only required for pxe driver.18:50
openstackgerritJay Faulkner proposed openstack/ironic: Document dependency on `fuser` for pxe driver  https://review.openstack.org/14694818:50
dtantsurJayF, anything using disk_partitioner.py is potentially affected18:50
JayFwhich is only pxe deploy driver18:50
afaranhajroll, another error:18:50
afaranhaError contacting Ironic server for 'node.set_provision_state'. Attempt 60 of 60", "code": 500, "details": "  File \"/opt/stack/nova/nova/compute/manager.py\", line 311, in decorated_function18:51
JayFagent deploy driver does not18:51
jrollafaranha: odd, I'm not sure, you'll have to go to the logs18:51
dtantsurJayF, still, I suggest you add downstream package to a bug and make them update it18:51
*** romcheg has quit IRC18:51
JayFdtantsur: IDK how to do that18:51
*** romcheg has joined #openstack-ironic18:51
afaranhajroll, I'm gonna set the instance_uuid to null and try again18:52
dtantsurJayF, "also affects distribution" button18:52
jrollah, found it18:52
*** romcheg has quit IRC18:52
afaranhajroll, great, the instance was removed, now I'll boot again and check if it works now :)18:53
*** romcheg has joined #openstack-ironic18:53
JayFdtantsur: done, good call18:53
jrollafaranha: great :)18:53
openstackgerritJay Faulkner proposed openstack/ironic: Document dependency on `fuser` for pxe driver  https://review.openstack.org/14694818:55
afaranhajroll, this is really strange, now the ironic node has power state as NULL and Provision state as error18:55
JayFdtantsur: ^ I agreed with your comment, so dropped it18:55
* dtantsur is looking at his own https://bugs.launchpad.net/ironic/+bug/140310618:55
afaranhajroll, I have thge following error in the node "During sync_power_state, max retries exceeded for node 6461f204-39a0-4bcf-b88c-875471c05ece, node state None does not match expected state 'None'. Updating DB state to 'None' Switching node to maintenance mode."18:56
dtantsurJayF, btw, where do we publish a list of non-pypi dependencies?18:56
jrollafaranha: yeah, I think you may need to reset the ironic node's provision state. and if power state is null, that probably means the bmc can't be reached18:56
JayFdtantsur: we don't, afaict18:56
dtantsuroh18:56
JayFdtantsur: we do in IPA via the dockerfile, IDK how Ironic does (I suspect not at all)18:56
jrollafaranha: yep, the conductor can't reach the bmc. which is probably why it was locked so long, it was waiting for the bmc call to time out18:57
* JayF brb18:57
*** penick has quit IRC18:58
afaranhajroll, Is there anything I can do to solve that?18:58
jrollafaranha: fix your bmc or your network? idk.18:59
jrollafaranha: you might try a cold reset on your bmc18:59
*** penick has joined #openstack-ironic19:00
openstackgerritChris Krelle proposed openstack/ironic: Enable async callbacks from task.process_event()  https://review.openstack.org/14086819:00
afaranhajroll, so the the fact that the deployment isn't complete is probable because of the bmc is unreached? Does that mean that I won't be able to power it on or off?19:00
jrollafaranha: correct19:01
*** alexpilotti has quit IRC19:01
afaranhajroll, So, the bmc is reached until certain point, and then it just stop in the same place everytime, I don't have any idea why this happens at same point19:02
jrollafaranha: what kind of hardware are you using?19:02
jrollthere's a config where you can change time between bmc calls19:03
jrollas too many calls too quickly can kill a bmc19:03
*** ChuckC_ has quit IRC19:04
jrollipmi.min_command_interval19:04
jrollbrb19:04
afaranhajroll, I'm using an Dell R41019:04
afaranhajroll gonna check that, thanks :)19:04
dtantsurfolks, can anyone read https://bugs.launchpad.net/ironic/+bug/1403106 and tell me, if my conclusion there is right?19:05
*** pensu has joined #openstack-ironic19:05
NobodyCamgrrrrrr http://logs.openstack.org/68/140868/15/check-tripleo/check-tripleo-ironic-undercloud-precise-nonha/9fbd6d3/console.html#_2015-01-13_19_05_38_84319:07
NobodyCamworth updaing the white board with ^^^^^?19:08
rlooNobodyCam: is this a one-off? I suppose it doesn't help to update the white board19:09
NobodyCamI've seen it at least twice today19:09
rloothen yeah, worth updating.19:09
rloowonder if there is a bug for it19:09
dtantsurso folks, it's >8pm and I'm going to call it a day. For now we have (diff with Mon): Open: 129 (-4), 5 new (-9) and some more got a priority. Good job everyone!19:11
rloothx dtantsur!19:11
dtantsurwould be nice if someone have a look at the remaining 5 new ones, if you have time19:12
dtantsurnow g'night and see you tomorrow19:12
*** erwan_taf has joined #openstack-ironic19:13
rloonight dtantsur19:13
*** dtantsur is now known as dtantsur|afk19:13
JayFjroll: rfr? I added this: https://wiki.openstack.org/wiki/Sprints/IronicKiloSprint#Accommodation_and_Transportation19:14
jrollJayF: lgtm19:15
*** ChuckC has joined #openstack-ironic19:19
* devananda returns from meeting-land19:21
*** Nisha has joined #openstack-ironic19:24
*** achanda has joined #openstack-ironic19:25
*** penick has quit IRC19:27
NobodyCamwb devananda19:27
*** pensu has left #openstack-ironic19:29
devanandadtantsur|afk: updated https://bugs.launchpad.net/ironic/+bug/140310619:29
*** penick has joined #openstack-ironic19:30
lucasagomesdevananda, that's an odd bug, I was trying to reproduce it http://paste.openstack.org/show/157270/19:39
jrollseems like it must be a race19:40
*** zer0c00l has joined #openstack-ironic19:42
lucasagomesyeah19:42
lucasagomesjroll, btw, on the configdrive thing... You want me to check the label after dd'ing the image to the partition?19:42
JayFI don't think we should19:43
lucasagomesI don't think it's necessary cause maybe we want to use a different label for a different case than the nova configdrive19:43
JayFbecause if configdrive spec changes19:43
lucasagomesyeah19:43
jrolllucasagomes: nah, I trust it anyway19:43
JayFyou shouldn't have to change the ironic code that just writes it to disk19:43
jrollI just forgot it was in the image19:43
lucasagomesjroll, right, you kept the -1 there. Mind taking a look again?19:43
lucasagomesJayF, +119:43
jrolloh, I never updated, sorry19:44
* jroll looks19:44
lucasagomesthanks :)19:44
jrolllucasagomes: so about the requests thing...19:47
jrollwe already depend on requests though it isn't documented https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent_client.py#L1719:47
jrollI'm going to add that to requirements.txt now19:47
lucasagomesjroll, ohh right19:48
lucasagomesok I can change that then, I thought we didn't depended on it (I looked at the requirements.txt)19:48
jrollidk, I'm fine with it as-is, I just prefer requests19:48
jrollyeah, that's our fault19:48
NobodyCamlucasagomes: still around?19:49
jrolllaunchpad is so slow19:49
lucasagomesNobodyCam, yup19:50
NobodyCamhey lucasagomes just a question on https://review.openstack.org/#/c/142838/19:50
lucasagomesjroll, right... well it's a very simple GET request that's why I didn't mind much either19:50
jrolllucasagomes: yeah, I'll +219:50
lucasagomesjroll, ta much! if it happen to me to send a new patch-set19:51
lucasagomesI will update it19:51
jrollcool19:51
JayFjroll: you patching the requirements.txt? if not I will19:51
jrollI got it19:51
JayFk19:52
jrolltook this long for launchpad to give me a bug number19:52
openstackgerritJim Rollenhagen proposed openstack/ironic: Add requests to requirements.txt  https://review.openstack.org/14697119:52
jrollthere she is19:52
lucasagomesNobodyCam, sure, whats up?19:52
NobodyCamlucasagomes: just wanted to see if you have a time line for un-wipping it19:52
*** zer0c00l has quit IRC19:52
*** achanda has quit IRC19:52
*** achanda has joined #openstack-ironic19:55
lucasagomesNobodyCam, not exactly sure what you mean19:55
lucasagomesin case the partitions does appear19:55
lucasagomesshould I wipe it?19:55
NobodyCamlol... gah19:56
lucasagomesdoesn't*19:56
NobodyCamno no the review. is wip just checking if your going to unwip it. I want to point someone to it19:56
lucasagomesohhhhh19:56
NobodyCamlol19:56
lucasagomesso, this bug was found by our internel qe19:57
lucasagomesso I did that fix and posted it so him can test19:57
NobodyCamyep same with us19:57
lucasagomesbut he didn't reply on the ticket yet19:57
NobodyCamis it working for your guys19:57
NobodyCamya19:57
lucasagomesoh right, I will check with him if he got some time to test it19:57
lucasagomesif you guys can give it a go as well and comment on the patch whether it's fixed or not19:58
NobodyCamsweet I'm also going to point a few folk to see if they can test it19:58
lucasagomesthat would be good19:58
lucasagomesawesome! thanks19:58
lucasagomesthe patch is completed I just WIP'ed so we don't merge it until we guarantee it's fixed19:58
*** pensu has joined #openstack-ironic19:58
lucasagomesI tested it locally it seems fine, cause I know check with devfs to see whether the partition appeared to the OS or not19:59
*** dprince has quit IRC20:00
*** pensu1 has joined #openstack-ironic20:02
*** pensu has quit IRC20:03
lucasagomesI now*20:04
NobodyCam:) we'll see if we can test it20:05
*** athomas has quit IRC20:06
*** pelix has quit IRC20:07
JayFrloo: this was the follow up patch you requested to IPA when my HardwareManager one merged; it's been up a while with no votes -> https://review.openstack.org/#/c/146133/20:08
rlooJayF: I didn't notice it. Will look...20:08
JayFty20:09
rlooJayF: just want to make sure. Can method be None?20:12
lucasagomesNobodyCam, cool cheers20:12
rlooJayF: if someone codes something incorrectly.20:12
lucasagomesI will call it a day as well20:12
JayFNo; in fact, allowing it to be none is exactly what this is intended to fix20:12
jrollrloo: method can be whatever a programmer passes to it20:12
lucasagomesa bit late here, have a good night everyone!20:12
NobodyCamhave a good night lucasagomes20:13
rloojroll: yeah. just wondering how bad a programmer can be.20:13
jrollrloo: with this new patch, they would get an exception "only one argument passed, requires 2" or whatever20:14
jrolland the only thing that uses this today is this, which requires a method https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/hardware.py#L37620:14
rloojroll: yeah, this patch is fine. just wondering about rebasing.20:14
jrollso unless a programmer does "raise errors.HardwareManagerMethodNotFound", it's fine20:15
jrollrebasing what where20:15
rlooJayF, jroll: I have a patch with unit tests, so depending on which lands first, the other will need to be rebased.20:15
jrolloh20:15
JayFI mean, land yours first?20:15
JayFmine is easy to redo20:15
JayFyou already have a +2 from me on those fixes, btw20:16
rlooJayF: either is fine. although of course, i'd rather not spend any more time on ipa :-)20:16
*** david-lyle has joined #openstack-ironic20:16
JayFbut the gate is wedged right now so it's not going to go anywhere20:16
rlooJayF: but I did have a test for this error, so that test will have to change cuz it has a test for method=None.20:16
JayFthat's perfectly fine :)20:16
*** achanda has quit IRC20:16
JayFI'm glad you wrote tests20:16
rlooJayF: anyway, not a big deal either way.20:16
*** Masahiro has joined #openstack-ironic20:17
*** zer0c00l has joined #openstack-ironic20:18
rlooJayF: I just realized, I don't regularly monitor patches to ipa, that's why I didn't notice your patch.20:18
JayFthat's fine, not many folks aside from us at Rackspace do20:18
JayFwhich is not ideal, but usually OK20:18
JayFjust annoying when we have things we want non-Rackspace eyes on (like the HW Manager)20:18
rlooJayF: just ping us then ;)20:19
JayFthat doesn't work as well as you might imagine :)20:19
JayFwe merged standalone mode which was pasted into channel probably 4-5 times with nobody else looking at it20:19
JayFand we went ahead and approved it because if nobody cares enough to review it when we post it channel, hopefully they won't care that it was merged, heh20:19
*** mikedillion has joined #openstack-ironic20:20
rlooJayF: yeah, if I decide to ignore ipa, i'll ignore ipa requests from irc too. not sure why I didn't ignore your multihardware one. now i may be in trouble for future asks ;)20:20
JayFwe just need to make IPA work as the ramdisk for pxe deploy driver too20:20
JayFthen you'll all be forced to reivew our code! bwahahahaha!20:20
JayFjroll: ^ we should seriously do that for Friday's hackday20:21
jrollI've said that for the last two hackdays20:21
jrolland got distracted20:21
*** Masahiro has quit IRC20:21
rlooJayF and jroll: it's been on my list to look at the ipa code. i'm trying to avoid burning out from doing reviews though.20:22
jrollI hear that20:22
JayFI'm trying to do the same except the opposite20:22
JayFtrying to review more Ironic code20:22
JayFinstead of just looking mostly at specs/IPA20:23
rloothx JayF!20:23
rlooI'm worried that we don't have enough people reviewing (cores or no cores). just not-enough-people.20:23
JayFhave you ever worked on a codebase this size where that hasn't felt true?20:26
rlooJayF: this is my first time working with open source. it is a different experience from working in house.20:27
JayFI mean, I've not ever worked on a team that exercised the review-then-merge model in any form20:27
JayFwhere "not enough reviews are happening" wasn't a constant problem20:27
rlooJayF: having said that, I feel like there are quite a few people submitting patches/specs, but rare to see them reviewing anything. I don't know what a reasonable % might be, if it is as expected in ironic vs other projects or not.20:27
*** dprince has joined #openstack-ironic20:28
JayFrloo: I completely agree, and honestly, de-prioritize reviewing those specs over those from people who participate fully in the process20:28
JayFif all someone does is write specs and bug people to review them; that's not a participating in the process, that's taking from the process like a bandit with a mask20:28
rlooJayF: so I had this goal of trying to make sure that all reviews had some eyes on it w/i one week, but I think that won't be doable (by me)20:29
*** Helimanic has joined #openstack-ironic20:29
JayFI try to get IPA reviews merged within a week20:30
*** achanda has joined #openstack-ironic20:30
jrollJayF: keep in mind not everyone has priorities set in a fashion where they have sufficient time to review20:30
JayFbut that's a luxury of the size20:30
jroll(is one of my guesses why some people don't review)20:30
JayFjroll: In that case, their company or manager is wearing the mask20:30
JayFjroll: not the person; but the result is the same20:30
jrollsure20:30
*** pensu1 has quit IRC20:30
jrollbut you can't de-prioritize their work because their manager isn't as awesome as yours20:31
rlooso I didn't want people to be discouraged from contributing, if they contribute and then don't get feedback in a 'reasonable' amount of time.20:31
jrollyeah ^20:31
Shrewsjroll: I am very surprised that requests isn't pulled in as part of one of the other requirements20:34
jrollShrews: it is, that's why we haven't noticed it20:34
jrollthat code has been in since july20:34
Shrewsespecially the clients20:34
jrollit's not a bug in that ironic doesn't work20:34
Shrewsoh, you just want to make it explicit then20:35
jrollit's a bug in that we don't explicitly declare it20:35
jrollyeah20:35
Shrewsjroll: coolio20:35
*** mikedillion has quit IRC20:36
*** martini__ has joined #openstack-ironic20:47
Shrewsjroll: rloo: If I update the commit message on 140869, will you guys +2?20:48
Shrewsseems that was the only issue you guys had with it20:49
rloofor a beer? Nah, not worth a beer. I'll +2 if you update.20:50
jrollShrews: "other than that I'm +2 on this"20:50
jroll:)20:50
NobodyCamShrews: 140869 will prob also need a rebase20:50
ShrewsNobodyCam: yay rebase buttons20:51
NobodyCam:) brb20:51
openstackgerritDavid Shrewsbury proposed openstack/ironic: Convert check_deploy_timeout to use process_event  https://review.openstack.org/14086920:52
Shrewsjroll: rloo: cool. thanks. ^^^20:52
rlooShrews: did you rebase it?20:52
Shrewsoh der20:53
Shrews1 min20:53
Shrewsno button... so more work20:53
rlooThere's sometimes a rebase button but I don't see it.20:53
jrollwhich also means there will be junk in the diff between patchsets, lame20:54
*** erwan_taf has quit IRC20:54
*** martini__ has quit IRC20:55
*** penick has quit IRC20:57
*** achanda has quit IRC21:03
*** dprince has quit IRC21:03
*** david-ly_ has joined #openstack-ironic21:07
*** achanda has joined #openstack-ironic21:08
*** david-lyle has quit IRC21:09
*** lucasagomes has quit IRC21:09
openstackgerritJohn Trowbridge proposed stackforge/ironic-discoverd: Changes utils.get_keystone(token) to utils.check_is_admin(token)  https://review.openstack.org/14565721:11
*** achanda has quit IRC21:12
*** david-ly_ has quit IRC21:12
Shrewsi have successfully managed to thoroughly confuse myself21:27
rlooShrews: congratulations!21:28
rlooShrews: it is wrt the rebasing?21:29
openstackgerritFrederic Lepied proposed stackforge/ironic-discoverd: added an eDeploy plugin  https://review.openstack.org/14659921:29
openstackgerritMerged openstack/ironic: Updated from global requirements  https://review.openstack.org/14669221:30
Shrewsrloo: yes  :(21:30
*** Helimanic has quit IRC21:32
*** jcoufal has quit IRC21:32
*** ndipanov_ has quit IRC21:32
*** lazy_prince has quit IRC21:32
*** aignatov has quit IRC21:32
*** lintan has quit IRC21:32
*** harlowja has quit IRC21:32
*** jrist has quit IRC21:32
*** tteggel has quit IRC21:32
*** sirushti has quit IRC21:32
*** rodrigods has quit IRC21:32
*** lifeless has quit IRC21:32
*** annegentle_ has quit IRC21:32
*** romcheg has quit IRC21:32
*** pcrews has quit IRC21:32
*** datajerk has quit IRC21:32
*** morgabra has quit IRC21:32
*** trown has quit IRC21:32
*** yuanying has quit IRC21:32
*** slagle has quit IRC21:32
*** TheJulia has quit IRC21:32
*** NobodyCam has quit IRC21:32
*** lynxman has quit IRC21:32
*** kragniz has quit IRC21:32
*** davidlenwell has quit IRC21:32
*** pleia2 has quit IRC21:32
*** SpamapS has quit IRC21:32
*** Lotus907efi has left #openstack-ironic21:33
openstackgerritFrederic Lepied proposed stackforge/ironic-discoverd: added an eDeploy plugin  https://review.openstack.org/14659921:33
*** ChuckC has quit IRC21:33
*** Marga_ has quit IRC21:33
*** lsmola has quit IRC21:33
*** jjohnson2 has quit IRC21:33
*** subscope has quit IRC21:33
*** bigjools has quit IRC21:33
*** vdrok_afk has quit IRC21:33
*** yuriyz has quit IRC21:33
*** alexm__ has quit IRC21:33
openstackgerritMichael Turek proposed openstack/ironic: Raise SSH failure messages to the error level.  https://review.openstack.org/14698521:34
*** SpamapS has joined #openstack-ironic21:34
*** annegentle_ has joined #openstack-ironic21:34
*** lifeless has joined #openstack-ironic21:34
*** rodrigods has joined #openstack-ironic21:34
*** sirushti has joined #openstack-ironic21:34
*** pleia2 has joined #openstack-ironic21:34
*** tteggel has joined #openstack-ironic21:34
*** davidlenwell has joined #openstack-ironic21:34
*** kragniz has joined #openstack-ironic21:34
*** NobodyCam has joined #openstack-ironic21:34
*** lynxman has joined #openstack-ironic21:34
*** yuanying has joined #openstack-ironic21:34
*** trown has joined #openstack-ironic21:34
*** TheJulia has joined #openstack-ironic21:34
*** jrist has joined #openstack-ironic21:34
*** slagle has joined #openstack-ironic21:34
*** harlowja has joined #openstack-ironic21:34
*** morgabra has joined #openstack-ironic21:34
*** datajerk has joined #openstack-ironic21:34
*** lintan has joined #openstack-ironic21:34
*** aignatov has joined #openstack-ironic21:34
*** lazy_prince has joined #openstack-ironic21:34
*** pcrews has joined #openstack-ironic21:34
*** ndipanov_ has joined #openstack-ironic21:34
*** jcoufal has joined #openstack-ironic21:34
*** romcheg has joined #openstack-ironic21:34
*** ChuckC has joined #openstack-ironic21:34
*** Marga_ has joined #openstack-ironic21:34
*** lsmola has joined #openstack-ironic21:34
*** jjohnson2 has joined #openstack-ironic21:34
*** subscope has joined #openstack-ironic21:34
*** alexm__ has joined #openstack-ironic21:34
*** bigjools has joined #openstack-ironic21:34
*** vdrok_afk has joined #openstack-ironic21:34
*** yuriyz has joined #openstack-ironic21:34
*** aweeks has quit IRC21:35
*** zsmithnyc has quit IRC21:35
*** xxj has quit IRC21:35
*** HenryG has quit IRC21:35
*** xxj has joined #openstack-ironic21:35
*** spandhe has quit IRC21:36
*** yjiang5 has quit IRC21:36
*** hypnosb has quit IRC21:36
*** spandhe has joined #openstack-ironic21:37
*** yjiang5 has joined #openstack-ironic21:37
*** hypnosb has joined #openstack-ironic21:37
*** zsmithnyc has joined #openstack-ironic21:37
*** HenryG has joined #openstack-ironic21:37
*** afaranha has quit IRC21:38
*** victor_lowther has quit IRC21:38
*** agordeev has quit IRC21:38
*** bnemec has quit IRC21:38
*** med_ has quit IRC21:38
*** kylestev has quit IRC21:38
*** ekarlso- has quit IRC21:38
*** marios has quit IRC21:38
openstackgerritDavid Shrewsbury proposed openstack/ironic: Convert check_deploy_timeout to use process_event  https://review.openstack.org/14086921:39
jrollShrews: you did it!21:40
*** afaranha has joined #openstack-ironic21:40
*** victor_lowther has joined #openstack-ironic21:40
*** agordeev has joined #openstack-ironic21:40
*** bnemec has joined #openstack-ironic21:40
*** med_ has joined #openstack-ironic21:40
*** kylestev has joined #openstack-ironic21:40
*** ekarlso- has joined #openstack-ironic21:40
*** marios has joined #openstack-ironic21:40
*** achanda has joined #openstack-ironic21:41
Shrewsjroll: amazing, ain't it?21:41
*** david-lyle has joined #openstack-ironic21:41
jrollyeah :P21:41
jroll+2'd21:41
Shrewsthough i still don't know the "proper" procedure people use when they submit multiple rebases at once21:42
*** arif-ali has quit IRC21:42
NobodyCamdeva does that. I find it to be black magic21:42
devanandamuahahaha21:42
*** arif-ali has joined #openstack-ironic21:42
devanandawait, huh?21:42
NobodyCamlol21:42
Shrewsdevananda: multiple 'git review -x' attempts?21:42
NobodyCammulti patch rebasing21:42
devanandaShrews: that's one way21:43
devanandaeither21:43
*** rwsu has quit IRC21:43
*** mjturek has quit IRC21:43
*** jgrimm has quit IRC21:43
*** enikanorov__ has quit IRC21:43
*** cinerama has quit IRC21:43
*** rwsu has joined #openstack-ironic21:43
*** mjturek has joined #openstack-ironic21:43
*** jgrimm has joined #openstack-ironic21:43
*** enikanorov__ has joined #openstack-ironic21:43
*** cinerama has joined #openstack-ironic21:43
jrollyeah, I use a lot of -x for that21:43
*** Nisha has quit IRC21:43
devanandapick a base. repeat(git review -x) for each patch. git review21:43
jrollalso -R helps not re-submit patches you didn't change21:43
devanandaor if I know the sha's already, I'll just git rebase -i BASE_SHA21:43
devanandathen edit there21:44
devanandajroll: sometimes21:44
*** achanda has quit IRC21:44
*** Hefeweizen has quit IRC21:44
*** openstackgerrit has quit IRC21:44
*** kbs has quit IRC21:44
*** pradipta_away has quit IRC21:44
*** alex_xu has quit IRC21:44
*** cppforlife has quit IRC21:44
*** dividehex has quit IRC21:44
*** dguerri_ has quit IRC21:44
*** kbyrne has quit IRC21:44
*** PaulCzar has quit IRC21:44
*** coolsvap|afk has quit IRC21:44
*** vipul has quit IRC21:44
*** yarker has quit IRC21:44
*** greghaynes has quit IRC21:44
*** lxsli has quit IRC21:44
*** toabctl has quit IRC21:44
*** mrda has quit IRC21:44
*** tonyb has quit IRC21:44
*** rwsu has quit IRC21:44
*** mjturek has quit IRC21:44
*** jgrimm has quit IRC21:44
*** enikanorov__ has quit IRC21:44
*** cinerama has quit IRC21:44
*** arif-ali has quit IRC21:44
*** afaranha has quit IRC21:44
*** victor_lowther has quit IRC21:44
*** agordeev has quit IRC21:44
*** bnemec has quit IRC21:45
*** med_ has quit IRC21:45
*** kylestev has quit IRC21:45
*** ekarlso- has quit IRC21:45
*** marios has quit IRC21:45
*** spandhe has quit IRC21:45
*** yjiang5 has quit IRC21:45
*** hypnosb has quit IRC21:45
*** xxj has quit IRC21:45
*** david-lyle has quit IRC21:45
*** jcoufal has quit IRC21:45
*** ndipanov_ has quit IRC21:45
*** lazy_prince has quit IRC21:45
*** aignatov has quit IRC21:45
*** lintan has quit IRC21:45
*** harlowja has quit IRC21:45
*** jrist has quit IRC21:45
*** tteggel has quit IRC21:45
*** sirushti has quit IRC21:45
*** rodrigods has quit IRC21:45
*** lifeless has quit IRC21:45
*** annegentle_ has quit IRC21:45
*** romcheg has quit IRC21:45
*** pcrews has quit IRC21:45
*** datajerk has quit IRC21:45
*** morgabra has quit IRC21:45
*** trown has quit IRC21:45
*** yuanying has quit IRC21:45
*** HenryG has quit IRC21:45
*** slagle has quit IRC21:45
*** TheJulia has quit IRC21:45
*** NobodyCam has quit IRC21:45
*** lynxman has quit IRC21:45
*** kragniz has quit IRC21:45
*** davidlenwell has quit IRC21:45
*** pleia2 has quit IRC21:45
*** SpamapS has quit IRC21:45
*** ChuckC has quit IRC21:45
*** Marga_ has quit IRC21:45
*** lsmola has quit IRC21:45
*** jjohnson2 has quit IRC21:45
*** subscope has quit IRC21:45
*** bigjools has quit IRC21:45
*** vdrok_afk has quit IRC21:45
*** yuriyz has quit IRC21:45
*** alexm__ has quit IRC21:45
*** dlaube has quit IRC21:45
*** rloo has quit IRC21:46
*** Ng has quit IRC21:46
*** krtaylor has quit IRC21:46
*** russell_h has quit IRC21:46
*** mgagne has quit IRC21:46
*** dhellmann has quit IRC21:46
*** pcaruana has quit IRC21:46
*** dtantsur|afk has quit IRC21:46
*** GheRivero has quit IRC21:46
*** soren has quit IRC21:46
*** Madasi has quit IRC21:46
*** anteaya has quit IRC21:46
*** devananda has quit IRC21:46
*** tchaypo has quit IRC21:46
*** JayF has quit IRC21:46
*** kevinbenton has quit IRC21:46
*** JoshNang has quit IRC21:46
*** clif_h has quit IRC21:46
*** jeblair has quit IRC21:46
*** Isotopp has quit IRC21:46
*** aszeszo has quit IRC21:46
*** adam_g has quit IRC21:46
*** d0ugal has quit IRC21:46
*** mordred has quit IRC21:46
*** Haomeng has quit IRC21:46
*** wendar has quit IRC21:46
*** sbadia has quit IRC21:46
*** Shrews has quit IRC21:46
*** antonym has quit IRC21:46
*** dvorak has quit IRC21:46
*** mikal has quit IRC21:46
*** keekz has quit IRC21:46
*** openstack has joined #openstack-ironic21:48
*** aweeks has joined #openstack-ironic21:49
*** david-lyle has joined #openstack-ironic21:51
*** cinerama has joined #openstack-ironic21:51
*** enikanorov__ has joined #openstack-ironic21:51
*** jgrimm has joined #openstack-ironic21:51
*** mjturek has joined #openstack-ironic21:51
*** rwsu has joined #openstack-ironic21:51
*** arif-ali has joined #openstack-ironic21:51
*** marios has joined #openstack-ironic21:51
*** ekarlso- has joined #openstack-ironic21:51
*** kylestev has joined #openstack-ironic21:51
*** med_ has joined #openstack-ironic21:51
*** bnemec has joined #openstack-ironic21:51
*** agordeev has joined #openstack-ironic21:51
*** victor_lowther has joined #openstack-ironic21:51
*** afaranha has joined #openstack-ironic21:51
*** HenryG has joined #openstack-ironic21:51
*** hypnosb has joined #openstack-ironic21:51
*** yjiang5 has joined #openstack-ironic21:51
*** spandhe has joined #openstack-ironic21:51
*** xxj has joined #openstack-ironic21:51
*** yuriyz has joined #openstack-ironic21:51
*** vdrok_afk has joined #openstack-ironic21:51
*** bigjools has joined #openstack-ironic21:51
*** alexm__ has joined #openstack-ironic21:51
*** subscope has joined #openstack-ironic21:51
*** jjohnson2 has joined #openstack-ironic21:51
*** lsmola has joined #openstack-ironic21:51
*** Marga_ has joined #openstack-ironic21:51
*** ChuckC has joined #openstack-ironic21:51
*** romcheg has joined #openstack-ironic21:51
*** jcoufal has joined #openstack-ironic21:51
*** ndipanov_ has joined #openstack-ironic21:51
*** pcrews has joined #openstack-ironic21:51
*** lazy_prince has joined #openstack-ironic21:51
*** aignatov has joined #openstack-ironic21:51
*** lintan has joined #openstack-ironic21:51
*** datajerk has joined #openstack-ironic21:51
*** morgabra has joined #openstack-ironic21:51
*** harlowja has joined #openstack-ironic21:51
*** slagle has joined #openstack-ironic21:51
*** jrist has joined #openstack-ironic21:51
*** TheJulia has joined #openstack-ironic21:51
*** trown has joined #openstack-ironic21:51
*** yuanying has joined #openstack-ironic21:51
*** lynxman has joined #openstack-ironic21:51
*** NobodyCam has joined #openstack-ironic21:51
*** kragniz has joined #openstack-ironic21:51
*** davidlenwell has joined #openstack-ironic21:51
*** tteggel has joined #openstack-ironic21:51
*** pleia2 has joined #openstack-ironic21:51
*** sirushti has joined #openstack-ironic21:51
*** rodrigods has joined #openstack-ironic21:51
*** lifeless has joined #openstack-ironic21:51
*** annegentle_ has joined #openstack-ironic21:51
*** SpamapS has joined #openstack-ironic21:51
*** dlaube has joined #openstack-ironic21:51
*** rloo has joined #openstack-ironic21:51
*** kbyrne has joined #openstack-ironic21:51
*** Haomeng has joined #openstack-ironic21:51
*** Ng has joined #openstack-ironic21:51
*** krtaylor has joined #openstack-ironic21:51
*** russell_h has joined #openstack-ironic21:51
*** mgagne has joined #openstack-ironic21:51
*** PaulCzar has joined #openstack-ironic21:51
*** wendar has joined #openstack-ironic21:51
*** coolsvap|afk has joined #openstack-ironic21:51
*** dhellmann has joined #openstack-ironic21:51
*** pcaruana has joined #openstack-ironic21:51
*** dtantsur|afk has joined #openstack-ironic21:51
*** lxsli has joined #openstack-ironic21:51
*** GheRivero has joined #openstack-ironic21:51
*** sbadia has joined #openstack-ironic21:51
*** soren has joined #openstack-ironic21:51
*** vipul has joined #openstack-ironic21:51
*** Madasi has joined #openstack-ironic21:51
*** yarker has joined #openstack-ironic21:51
*** Shrews has joined #openstack-ironic21:51
*** greghaynes has joined #openstack-ironic21:51
*** antonym has joined #openstack-ironic21:51
*** tchaypo has joined #openstack-ironic21:51
*** anteaya has joined #openstack-ironic21:51
*** dvorak has joined #openstack-ironic21:51
*** JayF has joined #openstack-ironic21:51
*** kevinbenton has joined #openstack-ironic21:51
*** mikal has joined #openstack-ironic21:51
*** JoshNang has joined #openstack-ironic21:51
*** toabctl has joined #openstack-ironic21:51
*** keekz has joined #openstack-ironic21:51
*** devananda has joined #openstack-ironic21:51
*** clif_h has joined #openstack-ironic21:51
*** jeblair has joined #openstack-ironic21:51
*** Isotopp has joined #openstack-ironic21:51
*** BadCub_ has joined #openstack-ironic21:51
*** aszeszo has joined #openstack-ironic21:51
*** adam_g has joined #openstack-ironic21:51
*** d0ugal has joined #openstack-ironic21:51
*** mordred has joined #openstack-ironic21:51
*** tonyb has joined #openstack-ironic21:51
*** mrda has joined #openstack-ironic21:51
*** docaedo has joined #openstack-ironic21:51
*** early has joined #openstack-ironic21:51
*** zz_naotok has joined #openstack-ironic21:51
*** david-lyle has quit IRC21:51
*** david-lyle has joined #openstack-ironic21:51
* devananda wanders off to find food21:51
*** zsmithnyc has quit IRC21:51
*** zsmithnyc has joined #openstack-ironic21:51
*** aweeks has quit IRC21:52
*** aweeks has joined #openstack-ironic21:52
*** dlaube has quit IRC21:53
*** rloo has quit IRC21:53
*** Ng has quit IRC21:53
*** krtaylor has quit IRC21:53
*** russell_h has quit IRC21:53
*** mgagne has quit IRC21:53
*** dhellmann has quit IRC21:53
*** pcaruana has quit IRC21:53
*** dtantsur|afk has quit IRC21:53
*** GheRivero has quit IRC21:53
*** soren has quit IRC21:53
*** Madasi has quit IRC21:53
*** anteaya has quit IRC21:53
*** devananda has quit IRC21:53
*** dlaube has joined #openstack-ironic21:53
*** rloo has joined #openstack-ironic21:53
*** Ng has joined #openstack-ironic21:53
*** krtaylor has joined #openstack-ironic21:53
*** russell_h has joined #openstack-ironic21:53
*** mgagne has joined #openstack-ironic21:53
*** dhellmann has joined #openstack-ironic21:53
*** pcaruana has joined #openstack-ironic21:53
*** dtantsur|afk has joined #openstack-ironic21:53
*** GheRivero has joined #openstack-ironic21:53
*** soren has joined #openstack-ironic21:53
*** Madasi has joined #openstack-ironic21:53
*** anteaya has joined #openstack-ironic21:53
*** devananda has joined #openstack-ironic21:53
*** tchaypo has quit IRC21:54
*** JayF has quit IRC21:54
*** kevinbenton has quit IRC21:54
*** JoshNang has quit IRC21:54
*** clif_h has quit IRC21:54
*** jeblair has quit IRC21:54
*** Isotopp has quit IRC21:54
*** aszeszo has quit IRC21:54
*** adam_g has quit IRC21:54
*** d0ugal has quit IRC21:54
*** mordred has quit IRC21:54
*** achanda has joined #openstack-ironic21:54
*** Hefeweizen has joined #openstack-ironic21:54
*** openstackgerrit has joined #openstack-ironic21:54
*** kbs has joined #openstack-ironic21:54
*** pradipta_away has joined #openstack-ironic21:54
*** alex_xu has joined #openstack-ironic21:54
*** cppforlife has joined #openstack-ironic21:54
*** dividehex has joined #openstack-ironic21:54
*** dguerri_ has joined #openstack-ironic21:54
*** Hefeweiz1n has joined #openstack-ironic21:54
*** tchaypo has joined #openstack-ironic21:54
*** JayF has joined #openstack-ironic21:54
*** kevinbenton has joined #openstack-ironic21:54
*** JoshNang has joined #openstack-ironic21:54
*** clif_h has joined #openstack-ironic21:54
*** jeblair has joined #openstack-ironic21:54
*** Isotopp has joined #openstack-ironic21:54
*** aszeszo has joined #openstack-ironic21:54
*** adam_g has joined #openstack-ironic21:54
*** d0ugal has joined #openstack-ironic21:54
*** mordred has joined #openstack-ironic21:54
*** achanda_ has joined #openstack-ironic21:54
*** Haomeng has quit IRC21:54
*** wendar has quit IRC21:54
*** sbadia has quit IRC21:55
*** Shrews has quit IRC21:55
*** antonym has quit IRC21:55
*** dvorak has quit IRC21:55
*** mikal has quit IRC21:55
*** keekz has quit IRC21:55
*** BadCub_ has quit IRC21:55
*** docaedo has quit IRC21:55
*** early has quit IRC21:55
*** zz_naotok has quit IRC21:55
*** cppforlife has quit IRC21:55
*** dguerri_ has quit IRC21:55
*** achanda has quit IRC21:55
*** Hefeweizen has quit IRC21:55
*** dguerri has joined #openstack-ironic21:55
*** cppforlife has joined #openstack-ironic21:59
*** Haomeng has joined #openstack-ironic21:59
*** wendar has joined #openstack-ironic21:59
*** sbadia has joined #openstack-ironic21:59
*** Shrews has joined #openstack-ironic21:59
*** antonym has joined #openstack-ironic21:59
*** dvorak has joined #openstack-ironic21:59
*** mikal has joined #openstack-ironic21:59
*** keekz has joined #openstack-ironic21:59
*** BadCub_ has joined #openstack-ironic21:59
*** docaedo has joined #openstack-ironic21:59
*** early has joined #openstack-ironic21:59
*** zz_naotok has joined #openstack-ironic21:59
*** cppforlife has quit IRC22:00
*** cppforlife has joined #openstack-ironic22:00
*** lazy_prince has quit IRC22:00
Shrewswheeeeeeee22:01
*** david-lyle has quit IRC22:02
*** kbyrne has quit IRC22:02
*** PaulCzar has quit IRC22:02
*** coolsvap|afk has quit IRC22:02
*** vipul has quit IRC22:02
*** yarker has quit IRC22:02
*** greghaynes has quit IRC22:02
*** lxsli has quit IRC22:02
*** toabctl has quit IRC22:02
*** mrda has quit IRC22:02
*** tonyb has quit IRC22:02
*** cppforlife has quit IRC22:02
*** Haomeng has quit IRC22:02
*** wendar has quit IRC22:02
*** sbadia has quit IRC22:02
*** Shrews has quit IRC22:02
*** antonym has quit IRC22:02
*** dvorak has quit IRC22:02
*** mikal has quit IRC22:02
*** keekz has quit IRC22:02
*** BadCub_ has quit IRC22:02
*** docaedo has quit IRC22:02
*** early has quit IRC22:02
*** zz_naotok has quit IRC22:02
*** tchaypo has quit IRC22:02
*** JayF has quit IRC22:02
*** kevinbenton has quit IRC22:02
*** JoshNang has quit IRC22:02
*** clif_h has quit IRC22:02
*** jeblair has quit IRC22:03
*** Isotopp has quit IRC22:03
*** aszeszo has quit IRC22:03
*** adam_g has quit IRC22:03
*** d0ugal has quit IRC22:03
*** mordred has quit IRC22:03
*** dlaube has quit IRC22:03
*** rloo has quit IRC22:03
*** Ng has quit IRC22:03
*** krtaylor has quit IRC22:03
*** russell_h has quit IRC22:03
*** mgagne has quit IRC22:03
*** dhellmann has quit IRC22:03
*** pcaruana has quit IRC22:03
*** dtantsur|afk has quit IRC22:03
*** GheRivero has quit IRC22:03
*** soren has quit IRC22:03
*** Madasi has quit IRC22:03
*** anteaya has quit IRC22:03
*** devananda has quit IRC22:03
*** dguerri has quit IRC22:03
*** openstackgerrit has quit IRC22:03
*** kbs has quit IRC22:03
*** pradipta_away has quit IRC22:03
*** alex_xu has quit IRC22:03
*** dividehex has quit IRC22:03
*** rwsu has quit IRC22:03
*** mjturek has quit IRC22:03
*** jgrimm has quit IRC22:03
*** enikanorov__ has quit IRC22:03
*** cinerama has quit IRC22:03
*** arif-ali has quit IRC22:03
*** achanda_ has quit IRC22:04
*** afaranha has quit IRC22:04
*** victor_lowther has quit IRC22:04
*** agordeev has quit IRC22:04
*** bnemec has quit IRC22:04
*** med_ has quit IRC22:04
*** kylestev has quit IRC22:04
*** ekarlso- has quit IRC22:04
*** marios has quit IRC22:04
*** spandhe has quit IRC22:04
*** yjiang5 has quit IRC22:04
*** hypnosb has quit IRC22:04
*** xxj has quit IRC22:04
*** Hefeweiz1n has quit IRC22:04
*** jcoufal has quit IRC22:04
*** ndipanov_ has quit IRC22:04
*** aignatov has quit IRC22:04
*** lintan has quit IRC22:04
*** harlowja has quit IRC22:04
*** jrist has quit IRC22:04
*** tteggel has quit IRC22:04
*** sirushti has quit IRC22:04
*** rodrigods has quit IRC22:04
*** lifeless has quit IRC22:04
*** annegentle_ has quit IRC22:04
*** romcheg has quit IRC22:04
*** pcrews has quit IRC22:04
*** datajerk has quit IRC22:04
*** morgabra has quit IRC22:04
*** trown has quit IRC22:04
*** yuanying has quit IRC22:04
*** HenryG has quit IRC22:05
*** slagle has quit IRC22:05
*** TheJulia has quit IRC22:05
*** NobodyCam has quit IRC22:05
*** lynxman has quit IRC22:05
*** kragniz has quit IRC22:05
*** davidlenwell has quit IRC22:05
*** pleia2 has quit IRC22:05
*** SpamapS has quit IRC22:05
*** ChuckC has quit IRC22:05
*** Marga_ has quit IRC22:05
*** lsmola has quit IRC22:05
*** jjohnson2 has quit IRC22:05
*** subscope has quit IRC22:05
*** bigjools has quit IRC22:05
*** vdrok_afk has quit IRC22:05
*** yuriyz has quit IRC22:05
*** alexm__ has quit IRC22:05
*** Masahiro has joined #openstack-ironic22:06
*** zz_naotok has joined #openstack-ironic22:06
*** early has joined #openstack-ironic22:06
*** docaedo has joined #openstack-ironic22:06
*** BadCub_ has joined #openstack-ironic22:06
*** keekz has joined #openstack-ironic22:06
*** mikal has joined #openstack-ironic22:06
*** dvorak has joined #openstack-ironic22:06
*** antonym has joined #openstack-ironic22:06
*** Shrews has joined #openstack-ironic22:06
*** sbadia has joined #openstack-ironic22:06
*** wendar has joined #openstack-ironic22:06
*** Haomeng has joined #openstack-ironic22:06
*** cppforlife has joined #openstack-ironic22:06
*** dguerri has joined #openstack-ironic22:06
*** achanda_ has joined #openstack-ironic22:06
*** mordred has joined #openstack-ironic22:06
*** d0ugal has joined #openstack-ironic22:06
*** adam_g has joined #openstack-ironic22:06
*** aszeszo has joined #openstack-ironic22:06
*** Isotopp has joined #openstack-ironic22:06
*** jeblair has joined #openstack-ironic22:06
*** clif_h has joined #openstack-ironic22:06
*** JoshNang has joined #openstack-ironic22:06
*** kevinbenton has joined #openstack-ironic22:06
*** JayF has joined #openstack-ironic22:06
*** tchaypo has joined #openstack-ironic22:06
*** Hefeweiz1n has joined #openstack-ironic22:06
*** dividehex has joined #openstack-ironic22:06
*** alex_xu has joined #openstack-ironic22:06
*** pradipta_away has joined #openstack-ironic22:06
*** kbs has joined #openstack-ironic22:06
*** openstackgerrit has joined #openstack-ironic22:06
*** devananda has joined #openstack-ironic22:06
*** anteaya has joined #openstack-ironic22:06
*** Madasi has joined #openstack-ironic22:06
*** soren has joined #openstack-ironic22:06
*** GheRivero has joined #openstack-ironic22:06
*** dtantsur|afk has joined #openstack-ironic22:06
*** pcaruana has joined #openstack-ironic22:06
*** dhellmann has joined #openstack-ironic22:06
*** mgagne has joined #openstack-ironic22:06
*** russell_h has joined #openstack-ironic22:06
*** krtaylor has joined #openstack-ironic22:06
*** Ng has joined #openstack-ironic22:06
*** rloo has joined #openstack-ironic22:06
*** dlaube has joined #openstack-ironic22:06
*** david-lyle has joined #openstack-ironic22:06
*** cinerama has joined #openstack-ironic22:06
*** enikanorov__ has joined #openstack-ironic22:06
*** jgrimm has joined #openstack-ironic22:06
*** mjturek has joined #openstack-ironic22:06
*** rwsu has joined #openstack-ironic22:06
*** arif-ali has joined #openstack-ironic22:06
*** marios has joined #openstack-ironic22:06
*** ekarlso- has joined #openstack-ironic22:06
*** kylestev has joined #openstack-ironic22:06
*** med_ has joined #openstack-ironic22:06
*** bnemec has joined #openstack-ironic22:06
*** agordeev has joined #openstack-ironic22:06
*** victor_lowther has joined #openstack-ironic22:06
*** afaranha has joined #openstack-ironic22:06
*** HenryG has joined #openstack-ironic22:06
*** hypnosb has joined #openstack-ironic22:06
*** yjiang5 has joined #openstack-ironic22:06
*** spandhe has joined #openstack-ironic22:06
*** xxj has joined #openstack-ironic22:06
*** yuriyz has joined #openstack-ironic22:06
*** vdrok_afk has joined #openstack-ironic22:06
*** bigjools has joined #openstack-ironic22:06
*** alexm__ has joined #openstack-ironic22:06
*** subscope has joined #openstack-ironic22:06
*** lsmola has joined #openstack-ironic22:06
*** Marga_ has joined #openstack-ironic22:06
*** ChuckC has joined #openstack-ironic22:06
*** romcheg has joined #openstack-ironic22:06
*** jcoufal has joined #openstack-ironic22:06
*** ndipanov_ has joined #openstack-ironic22:06
*** pcrews has joined #openstack-ironic22:06
*** aignatov has joined #openstack-ironic22:06
*** lintan has joined #openstack-ironic22:06
*** datajerk has joined #openstack-ironic22:06
*** morgabra has joined #openstack-ironic22:06
*** harlowja has joined #openstack-ironic22:06
*** slagle has joined #openstack-ironic22:06
*** jrist has joined #openstack-ironic22:06
*** TheJulia has joined #openstack-ironic22:06
*** trown has joined #openstack-ironic22:06
*** yuanying has joined #openstack-ironic22:06
*** lynxman has joined #openstack-ironic22:06
*** NobodyCam has joined #openstack-ironic22:06
*** kragniz has joined #openstack-ironic22:06
*** davidlenwell has joined #openstack-ironic22:06
*** tteggel has joined #openstack-ironic22:06
*** pleia2 has joined #openstack-ironic22:06
*** sirushti has joined #openstack-ironic22:06
*** rodrigods has joined #openstack-ironic22:06
*** lifeless has joined #openstack-ironic22:06
*** annegentle_ has joined #openstack-ironic22:06
*** SpamapS has joined #openstack-ironic22:06
*** kbyrne has joined #openstack-ironic22:06
*** PaulCzar has joined #openstack-ironic22:06
*** coolsvap|afk has joined #openstack-ironic22:06
*** lxsli has joined #openstack-ironic22:06
*** vipul has joined #openstack-ironic22:06
*** yarker has joined #openstack-ironic22:06
*** greghaynes has joined #openstack-ironic22:06
*** toabctl has joined #openstack-ironic22:06
*** tonyb has joined #openstack-ironic22:06
*** mrda has joined #openstack-ironic22:06
*** zsmithnyc has quit IRC22:07
*** med_ has quit IRC22:07
*** med_ has joined #openstack-ironic22:08
*** med_ has quit IRC22:08
*** med_ has joined #openstack-ironic22:08
*** Masahiro has quit IRC22:10
*** zsmithnyc has joined #openstack-ironic22:10
openstackgerritJohn Trowbridge proposed stackforge/ironic-discoverd: Changes utils.get_keystone(token) to utils.check_is_admin(token)  https://review.openstack.org/14565722:11
*** penick has joined #openstack-ironic22:14
*** ryanpetrello has joined #openstack-ironic22:21
*** iron_ has joined #openstack-ironic22:22
*** Marga_ has quit IRC22:22
*** iron_ has left #openstack-ironic22:22
*** Marga_ has joined #openstack-ironic22:23
*** ironman has joined #openstack-ironic22:23
*** killer_prince has joined #openstack-ironic22:24
*** killer_prince is now known as lazy_prince22:24
*** david-lyle has quit IRC22:27
NobodyCamwht up with IRC today22:27
jrollfreenode was moving things around, there was a global notice22:28
jroll"rehubbing the network"22:28
NobodyCamahh22:28
jrollnow "There will be a services outage at around 0000 UTC (just under two hours from this notice) which is expected to last between 15 and 30 minutes."22:28
jrollwhich is nickserv/chanserv/etc iiuc22:28
NobodyCamyep.. guess I should look at the status window now and then22:28
jrolllol22:29
NobodyCam:-p22:29
NobodyCamTy jroll22:29
jrollnp22:29
NobodyCam:)22:29
*** penick has quit IRC22:29
*** david-lyle has joined #openstack-ironic22:30
*** jcoufal has quit IRC22:30
*** penick has joined #openstack-ironic22:31
*** ironman has quit IRC22:32
*** Marga_ has quit IRC22:35
*** Marga_ has joined #openstack-ironic22:36
*** igordcard has joined #openstack-ironic22:37
*** Marga_ has quit IRC22:38
*** Marga_ has joined #openstack-ironic22:39
*** Marga_ has quit IRC22:43
*** Marga_ has joined #openstack-ironic22:43
openstackgerritAdam Gandelman proposed openstack/ironic: Allow configuration of neutronclient retries  https://review.openstack.org/14700222:45
*** ijw__ has joined #openstack-ironic22:45
*** ijw___ has joined #openstack-ironic22:47
*** sambetts has joined #openstack-ironic22:48
*** ijw_ has joined #openstack-ironic22:49
*** ijw____ has joined #openstack-ironic22:51
*** achanda_ has quit IRC22:51
*** ijw__ has quit IRC22:51
*** ijw___ has quit IRC22:52
*** ijw_ has quit IRC22:54
NobodyCamwow your quick rloo :)22:56
NobodyCamhehehe22:56
rlooNobodyCam: ? What'd I do?22:56
NobodyCam-1 on adam_g's patch22:56
NobodyCam:)22:56
NobodyCamits a good thing22:57
NobodyCam:)22:57
*** ryanpetrello has quit IRC22:57
rlooNobodyCam: oh. Yeah, in a meeting, not much I can review that doesn't require too much brain power.22:57
NobodyCamhehehe22:57
openstackgerritAdam Gandelman proposed openstack/ironic: Allow configuration of neutronclient retries  https://review.openstack.org/14700222:57
adam_g:)22:58
rlooI think adam_g is faster!22:58
NobodyCam:-p22:59
*** achanda has joined #openstack-ironic22:59
*** Marga_ has quit IRC23:00
*** Marga_ has joined #openstack-ironic23:01
*** Marga_ has quit IRC23:01
*** Marga_ has joined #openstack-ironic23:02
Haomengdevananda: hi23:16
Haomengdevananda: hi, I want to test neutron vlan+ironic env, not sure if our ironic work with neutron vlan dhcp, any comments?23:16
jrollHaomeng: didn't we just talk about this last night?23:18
jrollwhat more do you want to know?23:18
Haomengjroll: yes, not sure if HP production is runing with vlan mode, and need other physical switch control23:19
openstackgerritAdam Gandelman proposed openstack/ironic: Provided backward compat for enforcing admin policy  https://review.openstack.org/14598423:19
jrollhuh, didn't know HP ran ironic in production23:19
Haomengjroll: yes, I think for production env, vlan is required, so the physical vlan control is required23:20
openstackgerritAdam Gandelman proposed openstack/ironic: Increase openstack.common logging (DO NOT MERGE)  https://review.openstack.org/14701523:20
Haomengjroll: I will try to setup vlan env to play with vlan to identfy the gap23:20
jrollHaomeng: right, do you need something more than my spec?23:20
Haomengjroll: yes23:21
jrollok23:21
Haomengjroll: if we run pyhsical switch to configure the vlan by maunally23:21
jrollnot sure I understand23:21
Haomengjroll: not sure if it will work or not23:21
jrollHaomeng: my spec is about controlling switches with automation23:21
Haomengjroll: if it is neutron dhcp in vlan mode does not support our ironic, then have to add new physical switch control plugin from neutron side to control vlan assignements23:22
Haomengjroll: yes, great23:22
*** igordcard has quit IRC23:22
Haomengjroll: run snmp to contrlo switch, right?23:23
*** Marga_ has quit IRC23:23
Haomengjroll: I understand the case is that - user create vlan from neutron side first23:23
jrollHaomeng: depends on the switch23:24
*** Marga_ has joined #openstack-ironic23:24
Haomengjroll: so if our spec will be called from neutron23:24
Haomengjroll: the 2nd step from user is that they will nova boot in the vlan created by step 123:24
Haomengjroll: I just guess this is the normal user case, right?23:24
jrollHaomeng: sounds correct23:25
jrollHaomeng: it should work the same way it works already in virtual cloud23:25
Haomengjroll: yes23:25
jrollHaomeng: the spec is about what ironic needs to do to achieve this23:25
Haomengjroll: so not sure if your new spec code will run from conductor side23:25
jrollHaomeng: yes, from the conductor during deploy and teardown23:26
Haomengjroll: that mean we will control switch when our ironic node creation?23:26
Haomengjroll: ok, that is cool23:26
jrollyep23:26
Haomengjroll: how about if we can create different vlan for deploy time and tenant os runing time23:27
*** david-lyle has quit IRC23:27
*** ryanpetrello has joined #openstack-ironic23:27
Haomengjroll: to make sure the tenant os is security, can be isolated with other vlans23:27
jrollHaomeng: yes, the idea is there will be a "provisioning" VLAN that nodes are in when not in use by a tenant, and switch to tenant VLAN after deploy23:28
*** penick has quit IRC23:28
jrollHaomeng: POC code is here, maybe that will help make sense https://review.openstack.org/#/c/139687/23:28
jrollHaomeng: "POC code" meaning "the code running at rackspace"23:29
Haomengjroll: yes, got it from the problem section - In order to secure the provisioning network, Ironic should be able to switch    29between these networks when spawning and destroying instances.23:29
jrollyes23:29
Haomengjroll: this is great23:29
*** Marga_ has quit IRC23:29
Haomengjroll: any steps I need, I am interested with this featur, want to play with it:)23:29
jrollHaomeng: thanks :) we worked hard on it23:29
*** Marga_ has joined #openstack-ironic23:30
Haomengjroll: ok, want to help to test this patch23:30
jrollHaomeng: so, apply that ironic patch (hopefully it works as is, it's slightly different than what we've been running)23:30
jrolland also run neutron with ironic-neutron-plugin from github23:30
Haomengjroll: ok23:30
Haomengcool23:30
Haomengjroll: I will have try today and give your feedback:)23:30
jrollthat plugin only has code for certain cisco switches right now23:31
Haomengjroll: to make this better:)23:31
jrollso you may need to write code to talk to your switches23:31
Haomengjroll: you mean for different snmp mibs?23:31
Haomengjroll: yes, make sense, different switch support differetn snmp mibs23:31
jrollHaomeng: the cisco code we have doesn't use snmp, it uses an API that cisco provides (xml over ssh)23:31
Haomengjroll: let me see code, not sure if contrloed via snmp23:31
Haomengjroll: that is fine, but if we have protocol adaptor layer, that is better:)23:32
Haomengjroll: to support different switch driver23:32
jrollHaomeng: the switch control is pluggable already :)23:32
Haomengjroll: let me check my switch management protol23:32
jrollHaomeng: https://github.com/rackerlabs/ironic-neutron-plugin/tree/master/ironic_neutron_plugin/drivers23:32
Haomengjroll: yes, I konw this plugin, that is cool23:33
Haomengjroll: so I should apply ironic-neutron-plugin and then apply your patch, right?23:33
*** cppforlife has quit IRC23:33
*** Ng has quit IRC23:33
*** aweeks has quit IRC23:33
JayFthat plugin is not going to work on your average switch23:34
JayFit only works on the exact model we have23:34
jrollHaomeng: yes, but you will need to write a driver for your switch in ironic-neutron-plugin23:34
JayFand TBH perhaps only with firmware specifically for us23:34
Haomengjroll: ok, let me check my switch snmp mib23:34
jrollJayF: I think those commands all work in mainline firmware at this point23:34
Haomengjroll: thank you23:34
Haomengdoes ironic-neutron-plugin support snmp?23:35
jrollHaomeng: not today, but it could with a driver23:35
jrollHaomeng: patches welcome :)23:35
jrollsimple driver, even23:35
Haomengjroll: i know a snmp contrl switch patch in neutron - https://review.openstack.org/#/c/90422/2/specs/juno/ml2-mechanism-snmp-vlan.rst23:35
jrollHaomeng: great, and when that code is in neutron, we should add a network provider to ironic that works with it23:36
Haomengjroll: ok, let me check the code first, then try to contribute, and maybe I have to find a physical switch and get control on it first:)23:37
jrollHaomeng: indeed, thanks :)23:37
Haomengjroll: how do you think the switch management protocol we should supports?23:38
Haomengwill be back for a while:)23:38
jrollHaomeng: I would love to support all switches, but I think it will be whatever folks contribute23:38
*** victor_lowther has quit IRC23:40
*** shakamunyi has joined #openstack-ironic23:46
jrolljenkins passed here if someone wants to +A https://review.openstack.org/#/c/146971/23:50
jrollShrews: ^23:50
NobodyCamgilliard: still around?23:51
*** Masahiro has joined #openstack-ironic23:54
*** smoriya has joined #openstack-ironic23:56
*** ryanpetrello has quit IRC23:59
*** Masahiro has quit IRC23:59
openstackgerritMerged openstack/ironic: Document dependency on `fuser` for pxe driver  https://review.openstack.org/14694823:59

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