*** penick has quit IRC | 00:03 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/146692 | 00:03 |
---|---|---|
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/146693 | 00:03 |
* NobodyCam feels like to many context switches has caused his brain to segfault ... maybe that it is just monday | 00:07 | |
*** Marga_ has joined #openstack-ironic | 00:22 | |
*** david-lyle has joined #openstack-ironic | 00:24 | |
naohirot | good morning ironic | 00:28 |
*** penick has joined #openstack-ironic | 00:31 | |
*** chlong has quit IRC | 00:35 | |
*** Marga_ has quit IRC | 00:38 | |
*** Marga_ has joined #openstack-ironic | 00:38 | |
*** spandhe has quit IRC | 00:42 | |
NobodyCam | morning naohirot :) | 00:43 |
*** eghobo has quit IRC | 00:43 | |
naohirot | hi good evening | 00:44 |
*** Masahiro has joined #openstack-ironic | 00:45 | |
naohirot | NobodyCam: 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 | |
naohirot | NobodyCam: s/look/looked/ | 00:46 |
*** david-lyle has quit IRC | 00:46 | |
*** david-lyle has joined #openstack-ironic | 00:51 | |
NobodyCam | naohirot: 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 reasons | 00:51 |
naohirot | NobodyCam: Yeah, I know everybody is busy :) | 00:54 |
NobodyCam | :) | 00:55 |
naohirot | NobodyCam: just one thing. what is PTo? | 00:56 |
NobodyCam | personal time off | 00:56 |
naohirot | NobodyCam: Aha, I got it :) | 00:57 |
NobodyCam | :) | 00:58 |
*** penick has quit IRC | 00:59 | |
*** Marga_ has quit IRC | 01:00 | |
*** nosnos has joined #openstack-ironic | 01:04 | |
*** david-lyle has quit IRC | 01:04 | |
*** nosnos has quit IRC | 01:04 | |
*** nosnos has joined #openstack-ironic | 01:05 | |
*** nosnos has quit IRC | 01:09 | |
*** jay-s-b has quit IRC | 01:17 | |
openstackgerrit | Joshua Harlow proposed openstack/ironic: Add state machine to documentation https://review.openstack.org/146302 | 01:22 |
*** chlong has joined #openstack-ironic | 01:25 | |
*** nosnos has joined #openstack-ironic | 01:26 | |
*** nosnos has quit IRC | 01:34 | |
*** ryanpetrello has quit IRC | 01:35 | |
*** achanda has joined #openstack-ironic | 01:35 | |
*** david-lyle has joined #openstack-ironic | 01:37 | |
Haomeng | morning naohirot, NobodyCam | 01:40 |
naohirot | Haomeng: good morning:) | 01:40 |
Haomeng | naohirot: :) | 01:40 |
naohirot | Haomeng: I have a question regarding get_sensors_data in Management driver I/F | 01:41 |
Haomeng | naohirot: sure, go ahead:) | 01:41 |
*** chenglch has joined #openstack-ironic | 01:41 | |
NobodyCam | morning Haomeng I saw the your ping only this morning sorry :( | 01:42 |
Haomeng | naohirot: yes, that interface is added by me | 01:42 |
naohirot | Haomeng: currently there are two drivers which implement that are ipmitool and ipminative, right? | 01:42 |
Haomeng | NobodyCam: np, dont worry | 01:42 |
NobodyCam | :) | 01:42 |
Haomeng | NobodyCam: just want to ask if we have vlan configuration guide for ironic, or our ironic does not support vlan now? | 01:42 |
Haomeng | naohirot: and iLO driver I think | 01:43 |
naohirot | Haomeng: Yes, but iLO just inherits ipmitool. | 01:43 |
Haomeng | naohirot: yes | 01:43 |
naohirot | Haomeng: I'm implementing get_sensors_data in iRMC Management Driver right now | 01:44 |
Haomeng | naohirot: that is great:) | 01:44 |
NobodyCam | Haomeng: I only know of the raqspace ml2 plugin but not sure thats what your looking for | 01:45 |
Haomeng | NobodyCam: I just found our installation guide which support flat | 01:45 |
naohirot | Haomeng: as far as I know, ipminative's implementation seems quite general. | 01:45 |
NobodyCam | Haomeng: https://github.com/rackerlabs/ironic-neutron-plugin | 01:45 |
*** jerryz has joined #openstack-ironic | 01:46 | |
Haomeng | NobodyCam: yes, this one https://github.com/rackerlabs/ironic-neutron-plugin ? | 01:46 |
NobodyCam | yep | 01:46 |
Haomeng | same one:) | 01:46 |
naohirot | Haomeng: The format of ipminative conforms the minimum requirements which are Sensor Type and Sensor ID. | 01:46 |
Haomeng | NobodyCam: so without such plug, we can not support vlan, right? | 01:46 |
Haomeng | naohirot: yes | 01:47 |
Haomeng | naohirot: these data are consumed by Ceilomter, only one consumer now, that is Ceilometer | 01:47 |
*** achanda has quit IRC | 01:47 | |
Haomeng | NobodyCam: I know that is networking issue, not our Ironic issue, but we have the networking requirement for neutron | 01:48 |
naohirot | Haomeng: my question is , what happen if ipmitool's report and ipminative's report are mixed in ceilemeter? | 01:48 |
Haomeng | NobodyCam: this neutron spec - https://blueprints.launchpad.net/neutron/+spec/ml2-mechanism-snmp-vlan | 01:48 |
Haomeng | naohirot: 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 think | 01:49 |
Haomeng | naohirot: so ipminative just support the min set sensor fields | 01:49 |
naohirot | Haomeng: In case of iRMC implementation, iRMC returns sensor data in XML format | 01:50 |
*** achanda has joined #openstack-ironic | 01:50 | |
Haomeng | naohirot: 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 IRC | 01:51 | |
NobodyCam | Haomeng: that spec is Abandoned | 01:52 |
NobodyCam | https://review.openstack.org/#/c/90422 | 01:52 |
*** smoriya has joined #openstack-ironic | 01:52 | |
*** david-lyle has quit IRC | 01:52 | |
Haomeng | NobodyCam: yes, it is out of kilo plan | 01:52 |
naohirot | Haomeng: I just follows the way ipminative does which means just conform the Sensor Type and Sensor ID requirement of python dictionary | 01:52 |
naohirot | Haomeng: and then put sensor data in dictionary after converting xml to python dictionary. | 01:53 |
naohirot | Haomeng: Is this Okay? | 01:53 |
Haomeng | NobodyCam: 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 networking | 01:53 |
Haomeng | naohirot: yes, let me check the min required fields set | 01:54 |
naohirot | Haomeng: I'm going to submit iRMC Management source code today into gerrit, so please comment on the source if you have time. | 01:55 |
Haomeng | naohirot: sure, will do, my pleasure:) | 01:55 |
naohirot | Haomeng: thanks! | 01:55 |
Haomeng | naohirot: yw | 01:56 |
Haomeng | NobodyCam: and we have this one - https://review.openstack.org/#/c/143480/2/specs/kilo/network-provider.rst | 01:57 |
*** killer_prince is now known as lazy_prince | 01:58 | |
lazy_prince | jroll: Hi.. there..? | 01:58 |
NobodyCam | he's prob out for the night I will ping him tomorrow morning to see if he has plans on new rev | 02:01 |
NobodyCam | for that spec | 02:01 |
NobodyCam | speaking of I've got to run out. | 02:01 |
NobodyCam | have a good night folks | 02:02 |
Haomeng | NobodyCam: ok, good night:) | 02:02 |
Haomeng | NobodyCam: nice dream:) | 02:02 |
Haomeng | naohirot: 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#L1158 | 02:03 |
Haomeng | naohirot: so for the new driver, dont care these message header data, it is handled by common layer already | 02:04 |
Haomeng | naohirot: 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#L566 | 02:05 |
Haomeng | naohirot: to note the data format and 'Single quotation marks' are required | 02:06 |
naohirot | Haomeng: 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 |
Haomeng | naohirot: yes, but it is option, we can return NotImplement as exception if we dont neet it | 02:07 |
*** ryanpetrello has joined #openstack-ironic | 02:07 | |
Haomeng | naohirot: 'raise NotImplementedError() ' | 02:08 |
naohirot | Haomeng: 'Sensor Reading': 'current value', <--- is 'Sensor Reading' key just a sample or a part of requirement? | 02:08 |
Haomeng | naohirot: yes it is used for Ceilometer sample | 02:09 |
Haomeng | naohirot: will be populated the 'current value' as the ceilometer sample value | 02:09 |
Haomeng | naohirot: this is required, else ceilometer will get none for sample value | 02:09 |
naohirot | Haomeng: In case of iRMC, the default implementation is ipmitool's get_sensors_data(). Because iRMC Management Driver inherits ipmitool Management driver. | 02:10 |
Haomeng | naohirot: make sense | 02:10 |
Haomeng | naohirot: so if it works in iRMC platform, no new code is required, I think | 02:11 |
naohirot | Haomeng: So I don't have to use 'raise NotImplementedError() ' | 02:11 |
Haomeng | naohirot: dont worry, I just want to say it is an option if we dont care the get_sensor_data for iRMC | 02:11 |
naohirot | Haomeng: My implementation overwrite the default behavior if deployer specifies how to retrieve sensor data in config file. | 02:12 |
Haomeng | naohirot: I am confusing here, if iRMC support ipmitool commands, why it is in xml format | 02:13 |
Haomeng | naohirot: so I understand the ipmitool returns xml in iRMC, right, so we need to convert xml to json message format | 02:14 |
naohirot | Haomeng: iRMC has an I/F to retrieve sensor data in proprietary protocol, which returns data in XML. | 02:15 |
Haomeng | naohirot: ok, got:) | 02:15 |
naohirot | Haomeng: do you know what ceilometer does using sensor data? I mean, does ceilometer draw graph or something? | 02:17 |
lazy_prince | I 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 |
jroll | lazy_prince: kinda here now, I've been in and out today, what's up | 02:18 |
lazy_prince | do i need to do any manual VLAN tagging on the brbm bridge.. | 02:19 |
Haomeng | naohirot: ceilometer just store the sample data into ceilometer database, and have the api for retrieving from other consumers which need ceilometer data as the source | 02:19 |
Haomeng | naohirot: and ceilometer has simple UI in horizon can show some sample report | 02:19 |
naohirot | Haomeng: how frequently does ceilomenter poll sensor data? | 02:20 |
lazy_prince | jroll: wanted to check if using ironic-neutron-plugin, can we have neutron and ironic running on two different VMs....? | 02:20 |
Haomeng | naohirot: default is 10 mints, it is configuration option, can be changed on configuration time | 02:20 |
Haomeng | lazy_prince: so we have same default interval in ironic option | 02:20 |
lazy_prince | if so, how does the VM traffic from VLAN will be forwarded to the ironic node and vice versa.. | 02:20 |
Haomeng | naohirot: send_sensor_data_interval=600 | 02:21 |
naohirot | Haomeng: Okay, ceilometer polls every 10 min and deployer can see a report in horizon. | 02:21 |
jroll | lazy_prince: we run them on two different VMs. not sure I understand your question about forwarding to ironic node | 02:21 |
naohirot | Haomeng: I got the overall picture, thanks! | 02:22 |
jroll | lazy_prince: however we run a separate dhcp server that handles dhcp and tftp | 02:22 |
lazy_prince | jroll: as i plan to pxe_ipmitool as the ironic driver for node... | 02:22 |
Haomeng | naohirot: but horizon will not show all ceilometer samples | 02:22 |
Haomeng | naohirot: just part of them, let me double check | 02:22 |
lazy_prince | jroll: i thought you use config drive approach to get out of pxe and dhcp.. | 02:22 |
jroll | lazy_prince: if everything is on the same vlan it should be fine | 02:23 |
Haomeng | naohirot: and ceilometer support two method to collect data, one is polling, another one is event based our message | 02:23 |
jroll | lazy_prince: we still pxe boot the ramdisk in the same vlan as ironic/neutron/etc but use static dhcp configs | 02:23 |
naohirot | Haomeng: My concern is whether or not ceilomenter does machine processing using sensor data. | 02:23 |
lazy_prince | we plan to provide tenant isolation for the tenant and so tenant can create a network and provision an instance using that network.. | 02:23 |
Haomeng | naohirot: no | 02:24 |
Haomeng | naohirot: ceilometer has no logic to consume the data, just store it in db and show in gui and can be retrieved by api | 02:24 |
naohirot | Haomeng: If they do, I cannot change the format by myself, after releasing the iRMC driver. | 02:24 |
*** Marga_ has joined #openstack-ironic | 02:24 | |
jroll | lazy_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 network | 02:25 |
*** Marga_ has quit IRC | 02:25 | |
Haomeng | naohirot: so far only ceilometer receive our sensor data | 02:25 |
jroll | lazy_prince: I think we'll need to require local boot for that or do something weird | 02:25 |
*** Marga_ has joined #openstack-ironic | 02:25 | |
lazy_prince | jroll: aha.. would using IPA resolve this issue..? | 02:26 |
naohirot | Haomeng: I understood. | 02:26 |
Haomeng | jroll: one question, if we can support vlan with the spec https://review.openstack.org/#/c/143480, I have interested with VLAN networking for our ironic | 02:26 |
Haomeng | naohirot: :) | 02:26 |
jroll | lazy_prince: yes, or lucas is working on local boot for the pxe driver which would also resolve it | 02:26 |
naohirot | Haomeng: :) | 02:26 |
jroll | Haomeng: yes, in that spec when I say "network" I mean "VLAN" | 02:26 |
Haomeng | jroll: that is great, so I understand without such new spec, we can not support vlan for neutron+ironic, just supports flat, right? | 02:27 |
jroll | Haomeng: I think so, but maybe if everything is in the same vlan it will also work. "flat vlan"? | 02:28 |
Haomeng | jroll: if we have 3 vlans, ipmi vlan, tftp/pxe vlan, and tenant vlan, can we support? | 02:28 |
lazy_prince | jroll: 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 |
Haomeng | jroll: 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 flat | 02:29 |
jroll | Haomeng: no, not without that spec, how do you get to the tenant vlan if you need to pxe boot on the pxe vlan | 02:30 |
jroll | lazy_prince: yeah, I think that would be a good thing | 02:30 |
lazy_prince | jroll: but does it fall in line with ironic philosophy..? do do you see any issues there.. | 02:31 |
Haomeng | jroll: yes, so we need the spec to support *true* vlan:) | 02:31 |
jroll | lazy_prince: I think it's a good idea, yes :) probably only needs a bug, not a spec | 02:32 |
jroll | Haomeng: s/true/useful/ :) | 02:32 |
Haomeng | jroll: yes | 02:32 |
Haomeng | jroll: :) | 02:32 |
lazy_prince | okay.. let me put a patch for this.. | 02:33 |
lazy_prince | another thing. i am trying to test ironic with neutron (vlan)... | 02:33 |
Haomeng | lazy_prince: ++ | 02:33 |
lazy_prince | with KVM VM as baremetal node.. | 02:33 |
jroll | lazy_prince: it'll be a tough patch, though, as the tftp server needs the instance kernel/ramdisk | 02:33 |
lazy_prince | is it supposed to work..? | 02:33 |
Haomeng | lazy_prince: I have a reference from intel, it should be usefur for us to understand ironic vlan env | 02:34 |
jroll | lazy_prince: define "ironic with neutron (vlan)" everything on the same vlan or? | 02:34 |
Haomeng | https://software.intel.com/en-us/articles/physical-server-provisioning-with-openstack | 02:34 |
lazy_prince | not on same vlan.. they are on different VLAN... | 02:35 |
Haomeng | looks 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 |
jroll | lazy_prince: I have no idea if that should work, but I kind of doubt it | 02:35 |
lazy_prince | say a tenant creates a network with VLAN tag XX | 02:35 |
lazy_prince | and then provisions an instance using that network... | 02:35 |
Haomeng | lazy_prince: yes, we allow tenant to create vlan with tag id | 02:36 |
lazy_prince | I 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 |
jroll | lazy_prince: I have no idea if it's supported, sorry | 02:37 |
*** rwsu has quit IRC | 02:37 | |
lazy_prince | Haomeng: 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 |
Haomeng | lazy_prince: yes that should be a normal use case I think, how do you think? | 02:38 |
Haomeng | lazy_prince: but these vlan are created by neutron | 02:39 |
Haomeng | lazy_prince: not sure if neutron vlan support Ironic requirments, such as our agent will call back to conductor etc | 02:39 |
Haomeng | this is good reference, share to you - https://software.intel.com/en-us/articles/physical-server-provisioning-with-openstack | 02:40 |
Haomeng | this article talked about flat vs vlan in ironic | 02:40 |
*** ramineni has joined #openstack-ironic | 02:41 | |
Haomeng | so if we run with multi-vlan, the router is required to route traffic between conductor subnet and tenant subnet, right? | 02:42 |
lazy_prince | well.. 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_prince | Haomeng: 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 IRC | 02:54 | |
*** david-lyle has joined #openstack-ironic | 02:55 | |
Haomeng | lazy_prince: yes, it is about physical server, just a reference for us I think | 02:55 |
*** rloo has quit IRC | 02:56 | |
lazy_prince | Haomeng: 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_prince | Haomeng: my understanding was that the conductor subnet whould not be visible to the tenants.. | 03:04 |
lazy_prince | s/whould/would/ | 03:04 |
openstackgerrit | Anusha Ramineni proposed openstack/ironic: Add tests for ilo_deploy driver https://review.openstack.org/142985 | 03:13 |
*** Masahiro has quit IRC | 03:15 | |
Haomeng | lazy_prince: but if user create its own subnet, that will be another different subnet with our conductor, right? | 03:26 |
Haomeng | lazy_prince: so for such network env, the router is required | 03:26 |
*** penick has joined #openstack-ironic | 03:26 | |
*** Marga_ has quit IRC | 03:28 | |
*** harlowja is now known as harlowja_away | 03:31 | |
*** lazy_prince is now known as killer_prince | 03:33 | |
*** harlowja_away is now known as harlowja | 03:39 | |
*** dlaube has quit IRC | 03:46 | |
*** Nisha has joined #openstack-ironic | 03:56 | |
*** pcrews has quit IRC | 04:03 | |
*** Masahiro has joined #openstack-ironic | 04:06 | |
*** pcrews has joined #openstack-ironic | 04:08 | |
*** penick has quit IRC | 04:12 | |
*** david-lyle has quit IRC | 04:20 | |
*** ryanpetrello has quit IRC | 04:22 | |
*** pcrews has quit IRC | 04:30 | |
*** achanda has joined #openstack-ironic | 04:32 | |
*** rameshg87 has joined #openstack-ironic | 04:36 | |
*** saripurigopi has joined #openstack-ironic | 04:36 | |
*** pradipta_away is now known as pradipta | 04:36 | |
saripurigopi | hi 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-ironic | 04:38 | |
*** eghobo has quit IRC | 04:41 | |
Haomeng | saripurigopi: sure, thanks for your bp | 05:00 |
saripurigopi | Thank you Haomeng. | 05:01 |
Haomeng | saripurigopi: yw:) | 05:01 |
*** achanda has quit IRC | 05:20 | |
*** killer_prince is now known as lazy_prince | 05:24 | |
*** achanda has joined #openstack-ironic | 05:35 | |
*** pensu has joined #openstack-ironic | 05:36 | |
*** achanda has quit IRC | 05:37 | |
*** harlowja is now known as harlowja_away | 05:42 | |
*** teju has joined #openstack-ironic | 05:47 | |
*** jerryz has quit IRC | 05:48 | |
*** Marga_ has joined #openstack-ironic | 05:49 | |
*** jerryz has joined #openstack-ironic | 06:00 | |
*** lazy_prince is now known as killer_prince | 06:02 | |
*** pcrews has joined #openstack-ironic | 06:02 | |
*** killer_prince is now known as lazy_prince | 06:02 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Add iRMC Driver and its iRMC Power module https://review.openstack.org/144901 | 06:06 |
*** Marga_ has quit IRC | 06:11 | |
*** pcrews has quit IRC | 06:18 | |
*** vinbs has joined #openstack-ironic | 06:24 | |
*** Marga_ has joined #openstack-ironic | 06:31 | |
*** Haomeng has quit IRC | 06:37 | |
*** Haomeng has joined #openstack-ironic | 06:39 | |
vinbs | morning ironic! | 06:47 |
vinbs | Hello Haomeng | 06:47 |
*** Haomeng has quit IRC | 06:48 | |
*** Haomeng has joined #openstack-ironic | 06:50 | |
*** ryanpetrello has joined #openstack-ironic | 06:52 | |
*** lazy_prince is now known as killer_prince | 06:58 | |
*** killer_prince is now known as lazy_prince | 07:15 | |
*** ryanpetrello has quit IRC | 07:17 | |
*** Marga_ has quit IRC | 07:17 | |
*** Haomeng has quit IRC | 07:41 | |
*** Haomeng has joined #openstack-ironic | 07:42 | |
*** pcrews has joined #openstack-ironic | 07:44 | |
*** Haomeng has quit IRC | 07:47 | |
*** Haomeng has joined #openstack-ironic | 07:49 | |
*** dlpartain has joined #openstack-ironic | 08:03 | |
*** dlpartain has quit IRC | 08:03 | |
*** dlpartain has joined #openstack-ironic | 08:03 | |
*** dlpartain has left #openstack-ironic | 08:03 | |
*** pcrews has quit IRC | 08:07 | |
*** chlong has quit IRC | 08:09 | |
*** Nisha has quit IRC | 08:12 | |
*** k4n0 has joined #openstack-ironic | 08:18 | |
*** ifarkas has joined #openstack-ironic | 08:22 | |
*** datajerk has quit IRC | 08:25 | |
*** datajerk has joined #openstack-ironic | 08:25 | |
*** Haomeng has quit IRC | 08:27 | |
*** Haomeng has joined #openstack-ironic | 08:28 | |
*** saripurigopi has quit IRC | 08:31 | |
*** jcoufal has joined #openstack-ironic | 08:32 | |
*** Haomeng has quit IRC | 08:35 | |
*** Haomeng has joined #openstack-ironic | 08:37 | |
*** Haomeng has quit IRC | 08:43 | |
*** Haomeng has joined #openstack-ironic | 08:49 | |
*** rakesh_hs has joined #openstack-ironic | 08:52 | |
*** pradipta is now known as pradipta_away | 08:57 | |
*** Marga_ has joined #openstack-ironic | 08:59 | |
*** pradipta_away is now known as pradipta | 09:00 | |
*** romcheg has joined #openstack-ironic | 09:01 | |
*** dtantsur|afk is now known as dtantsur | 09:03 | |
dtantsur | Morning! | 09:03 |
*** pradipta is now known as pradipta_away | 09:04 | |
*** athomas has joined #openstack-ironic | 09:11 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Add iRMC Management module for iRMC Driver https://review.openstack.org/146803 | 09:15 |
*** derekh has joined #openstack-ironic | 09:23 | |
*** jistr has joined #openstack-ironic | 09:23 | |
*** MattMan has quit IRC | 09:25 | |
*** MattMan has joined #openstack-ironic | 09:26 | |
*** mkerrin has quit IRC | 09:40 | |
openstackgerrit | Merged stackforge/ironic-discoverd: Refactor stable API to be /v1/introspection/<UUID> https://review.openstack.org/146475 | 09:41 |
*** gsamfira_ has left #openstack-ironic | 09:42 | |
*** igordcard has joined #openstack-ironic | 09:48 | |
*** andreykurilin has joined #openstack-ironic | 09:58 | |
*** kbs1 has quit IRC | 10:08 | |
openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Add options table and convenience methods to work with it https://review.openstack.org/146809 | 10:14 |
openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Add options table and convenience methods to work with it https://review.openstack.org/146809 | 10:17 |
*** vinbs has quit IRC | 10:29 | |
openstackgerrit | Alexis Lee proposed openstack/ironic: Avoid querying the power state twice https://review.openstack.org/146554 | 10:33 |
*** ryanpetrello has joined #openstack-ironic | 10:41 | |
*** subscope has quit IRC | 10:43 | |
*** ryanpetrello has quit IRC | 10:45 | |
*** andreykurilin has quit IRC | 10:48 | |
*** andreykurilin has joined #openstack-ironic | 10:48 | |
*** chenglch has quit IRC | 10:50 | |
*** subscope has joined #openstack-ironic | 10:56 | |
*** smoriya has quit IRC | 10:56 | |
*** Masahiro has quit IRC | 11:00 | |
*** ramineni has quit IRC | 11:01 | |
openstackgerrit | Alexis Lee proposed openstack/ironic: Distinguish between prepare + deploy errors https://review.openstack.org/146135 | 11:02 |
*** rameshg87 has quit IRC | 11:06 | |
*** lucasagomes has joined #openstack-ironic | 11:09 | |
Haomeng | dtantsur: good morning | 11:09 |
Haomeng | dtantsur: do you know if our ironic support neutron vlan network? | 11:09 |
dtantsur | Haomeng, hi, sorry, I have no idea | 11:12 |
Haomeng | dtantsur: np:) | 11:13 |
Haomeng | nice day:) | 11:13 |
*** teju has left #openstack-ironic | 11:16 | |
openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Add introspect client call and switch functest to it https://review.openstack.org/146479 | 11:19 |
openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Add options table and convenience methods to work with it https://review.openstack.org/146809 | 11:19 |
*** rakesh_hs has quit IRC | 11:20 | |
dtantsur | ifarkas, please have one more look ^^^ once you have time | 11:28 |
ifarkas | dtantsur, ok | 11:28 |
*** pelix has joined #openstack-ironic | 11:33 | |
pensu | Hi all, could you share your views on this: http://lists.openstack.org/pipermail/openstack-dev/2015-January/054313.html | 11:33 |
*** chlong has joined #openstack-ironic | 11:33 | |
dtantsur | pensu, it's better to ask J*, bit I guess the problem there is assumption that Ironic does anything about DHCP, when provide is none | 11:37 |
*** kbyrne has quit IRC | 11:38 | |
*** kbyrne has joined #openstack-ironic | 11:41 | |
pensu | dtantsur: 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 |
dtantsur | pensu, 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 it | 11:44 |
pensu | dtantsur: 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 |
dtantsur | pelix, the question is whether deploy even starts for you. What driver do you use? | 11:51 |
pelix | dtantsur: mistype ;) | 11:57 |
dtantsur | pelix, oh sorry :) | 11:58 |
dtantsur | pensu, ^^^ | 11:58 |
openstackgerrit | Merged stackforge/ironic-discoverd: Add introspect client call and switch functest to it https://review.openstack.org/146479 | 11:59 |
openstackgerrit | Matthew Gilliard proposed openstack/ironic: Check glance for new kernel and ramdisk on prepare https://review.openstack.org/142782 | 11:59 |
pensu | dtantsur: yeah, deployment is happening, I am using pxe_ipmitool diver. | 12:00 |
*** Masahiro has joined #openstack-ironic | 12:01 | |
dtantsur | pensu, how do you get ramdisk on a node? static dnsmasq configuration? | 12:01 |
pensu | dtantsur: 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 |
openstackgerrit | Syed Ismail Faizan Barmawer proposed openstack/ironic-specs: Local boot support with partition images https://review.openstack.org/143636 | 12:03 |
dtantsur | pensu, so it serves the PXE ramdisk to everyone, right? | 12:03 |
pensu | dtantsur: yup... | 12:03 |
dtantsur | pensu, 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 IRC | 12:05 | |
pensu | dtantsur: 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 |
dtantsur | pensu, Ironic can't do, as you told it not to manage your DHCP | 12:11 |
dtantsur | (maybe I'm not the best source of information on this) | 12:11 |
pensu | dtantsur: 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 |
openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Add options table and convenience methods to work with it https://review.openstack.org/146809 | 12:12 |
dtantsur | pensu, np. you may try to ask jroll and JayF, they do use this dhcp=none mode | 12:13 |
* lucasagomes answered that email thread | 12:14 | |
* dtantsur brb | 12:14 | |
lucasagomes | pensu, I think it has to do with the extra dhcp configuration that Ironic passses to the DHCP server | 12:14 |
lucasagomes | https://github.com/openstack/ironic/blob/master/ironic/common/pxe_utils.py#L262-L266 | 12:14 |
lucasagomes | it may be an authentication problem, just as a test, try to set the auth_strategy to "noauth" in Ironic and reset the service | 12:15 |
lucasagomes | then try to deploy again | 12:15 |
*** alexpilotti has joined #openstack-ironic | 12:15 | |
*** pensu has quit IRC | 12:25 | |
*** ryanpetrello has joined #openstack-ironic | 12:27 | |
*** andreykurilin has quit IRC | 12:27 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Add iRMC Driver and its iRMC Power module https://review.openstack.org/144901 | 12:31 |
*** ryanpetrello has quit IRC | 12:41 | |
*** kbs has joined #openstack-ironic | 12:52 | |
*** Masahiro has joined #openstack-ironic | 13:02 | |
*** athomas has quit IRC | 13:03 | |
*** ndipanov has quit IRC | 13:03 | |
*** dprince has joined #openstack-ironic | 13:03 | |
*** ndipanov has joined #openstack-ironic | 13:05 | |
*** Masahiro has quit IRC | 13:06 | |
*** Marga_ has quit IRC | 13:07 | |
*** ryanpetrello has joined #openstack-ironic | 13:08 | |
*** Marga_ has joined #openstack-ironic | 13:13 | |
*** athomas has joined #openstack-ironic | 13:13 | |
*** ndipanov has quit IRC | 13:23 | |
*** Marga_ has quit IRC | 13:28 | |
*** lintan has joined #openstack-ironic | 13:30 | |
*** chlong has quit IRC | 13:30 | |
*** igordcard has quit IRC | 13:33 | |
*** jjohnson2 has joined #openstack-ironic | 13:36 | |
*** ndipanov has joined #openstack-ironic | 13:36 | |
*** aignatov has quit IRC | 13:37 | |
*** lsmola has quit IRC | 13:38 | |
*** k4n0 has quit IRC | 13:41 | |
*** rloo has joined #openstack-ironic | 13:44 | |
*** aignatov has joined #openstack-ironic | 13:45 | |
*** achanda has joined #openstack-ironic | 13:46 | |
*** lsmola has joined #openstack-ironic | 13:51 | |
openstackgerrit | Alexis Lee proposed openstack/ironic: Distinguish between prepare + deploy errors https://review.openstack.org/146135 | 13:53 |
*** lazy_prince is now known as killer_prince | 13:55 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Add iRMC Management module for iRMC Driver https://review.openstack.org/146803 | 13:59 |
NobodyCam | Good morning Ironic, From the man maken coffee | 14:00 |
jjohnson2 | NobodyCam, morning | 14:01 |
NobodyCam | morning jjohnson2 | 14:01 |
rloo | hey, morning NobodyCam, jjohnson2 | 14:03 |
NobodyCam | morning rloo :) | 14:07 |
openstackgerrit | Frederic Lepied proposed stackforge/ironic-discoverd: added an eDeploy plugin https://review.openstack.org/146599 | 14:09 |
*** k4n0 has joined #openstack-ironic | 14:10 | |
*** k4n0 has quit IRC | 14:10 | |
lucasagomes | NobodyCam, rloo jjohnson2 morning :) | 14:12 |
NobodyCam | morning lucasagomes :) | 14:12 |
rloo | afternoon lucasagomes | 14:14 |
dtantsur | NobodyCam, rloo, jjohnson2, morning | 14:14 |
rloo | hi dtantsur! | 14:14 |
NobodyCam | morning dtantsur | 14:19 |
NobodyCam | :) | 14:19 |
openstackgerrit | Imre Farkas proposed openstack/ironic: DracClient to handle ClientOptions creation https://review.openstack.org/146879 | 14:22 |
*** openstackgerrit has quit IRC | 14:22 | |
*** openstackgerrit has joined #openstack-ironic | 14:23 | |
openstackgerrit | Merged openstack/ironic: Avoid querying the power state twice https://review.openstack.org/146554 | 14:25 |
*** jerryz has quit IRC | 14:32 | |
NobodyCam | any 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 see | 14:35 |
*** Marga_ has joined #openstack-ironic | 14:41 | |
NobodyCam | brb | 14:41 |
*** r-daneel has joined #openstack-ironic | 14:46 | |
*** Masahiro has joined #openstack-ironic | 14:50 | |
*** pcaruana|off| is now known as pcaruana | 14:51 | |
*** killer_prince has quit IRC | 14:52 | |
*** Masahiro has quit IRC | 14:55 | |
*** killer_prince has joined #openstack-ironic | 14:55 | |
*** killer_prince is now known as lazy_prince | 14:56 | |
*** achanda has quit IRC | 14:56 | |
BadCub_ | morning Ironic | 15:04 |
NobodyCam | morning BadCub_ | 15:04 |
rloo | NobodyCam: are you planning (or devananda) on updating the fsm patches? I think they just require minor changes. https://review.openstack.org/#/c/140868 | 15:07 |
NobodyCam | rloo: I didn't get a chance to take a look yesterday | 15:08 |
rloo | NobodyCam: ok. I'd make the changes but want to hold on to my +2 powers ;) | 15:09 |
rloo | NobodyCam: on the other hand, if you haven't yet touched them. let me see. | 15:10 |
*** Lotus907efi has joined #openstack-ironic | 15:10 | |
rloo | NobodyCam: yeah, you've updated 140868 already. | 15:10 |
*** ndipanov_ has joined #openstack-ironic | 15:11 | |
NobodyCam | yea I fiddled with that one | 15:11 |
NobodyCam | looks like move the err_handler | 15:11 |
NobodyCam | and some test changes | 15:11 |
Lotus907efi | NobodyCam: With help from Deva I got iommu turned on for our overcloud systems | 15:12 |
NobodyCam | awecome Lotus907efi :) | 15:12 |
Lotus907efi | should this "trick" be written up somewhere? | 15:12 |
NobodyCam | did you by chance document what you had to do? | 15:12 |
Lotus907efi | that is what I am intending to do | 15:12 |
NobodyCam | yes I was just abiut to ask you | 15:12 |
*** ndipanov has quit IRC | 15:13 | |
NobodyCam | maybe a etherpad would work, as its a workaround at this point | 15:13 |
Lotus907efi | etherpad? | 15:13 |
jroll | what is iommu? | 15:13 |
jroll | also, morning ironic :) | 15:13 |
Shrews | rloo: ah, good memory re: the ascii-flow use | 15:13 |
Lotus907efi | input/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 memory | 15:14 |
rloo | Shrews: sometimes it isn't good to have a good memory. That diagram will be useful. | 15:14 |
rloo | Shrews: something to ask devananda I think | 15:15 |
Shrews | rloo++ | 15:15 |
jroll | interesting | 15:15 |
rloo | morning jroll | 15:15 |
jroll | \o rloo | 15:15 |
NobodyCam | Lotus907efi: maybe here: https://etherpad.openstack.org/p/AddKernelOptionsToIronicDeploys | 15:16 |
NobodyCam | morning jroll | 15:16 |
NobodyCam | morning Shrews | 15:16 |
Shrews | morning NobodyCam, et. al. | 15:17 |
jroll | oh NobodyCam, you mentioned last night that you would ask me to push a new version of https://review.openstack.org/#/c/143480/ | 15:19 |
jroll | but I don't see any reason to push a new version? | 15:19 |
openstackgerrit | Naohiro Tamura proposed openstack/ironic-specs: iRMC Management Driver for Ironic https://review.openstack.org/136020 | 15:20 |
NobodyCam | jroll: I thought there was a question about vlan on it | 15:24 |
NobodyCam | let me take another look | 15:24 |
jroll | NobodyCam: I need to answer that on the review | 15:24 |
jroll | but I did answer Haomeng in channel | 15:24 |
NobodyCam | ack.. thats what I was going to poke you about | 15:24 |
*** lucasagomes is now known as lucas-hungry | 15:24 | |
jroll | tl;dr when I say 'network' there I generally mean VLAN | 15:24 |
* lucas-hungry lunch time | 15:24 | |
NobodyCam | enjoy lucas-hungry | 15:24 |
jroll | yeah, let me respond on gerrit | 15:24 |
*** coolsvap is now known as coolsvap|afk | 15:25 | |
NobodyCam | awesome TY jroll :) | 15:25 |
Lotus907efi | NobodyCam: I just save a revision of that etherpad doc | 15:26 |
Lotus907efi | take a look and see if that is clear | 15:26 |
openstackgerrit | Matthew Gilliard proposed openstack/ironic: Adds get_glance_image_properties https://review.openstack.org/146099 | 15:26 |
NobodyCam | will do Lotus907efi :) | 15:27 |
*** jcoufal_ has joined #openstack-ironic | 15:29 | |
*** jcoufal has quit IRC | 15:31 | |
*** ndipanov_ has quit IRC | 15:32 | |
NobodyCam | Lotus907efi: Thank you :) | 15:35 |
gilliard | Is that check-tripleo-ironic-undercloud-precise-nonha (phew!) job voting? | 15:35 |
gilliard | it usually says NON-VOTING on such jobs, but it isn't actually voting on my patches. | 15:36 |
*** pcrews has joined #openstack-ironic | 15:36 | |
NobodyCam | I think that got switched | 15:38 |
jroll | gilliard: I don't believe so | 15:38 |
jroll | orly | 15:38 |
NobodyCam | I thought so ... Icbw | 15:39 |
NobodyCam | gilliard: 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_065 | 15:39 |
gilliard | OK. See how the 2 jobs above have (non-voting) against them though https://review.openstack.org/#/c/142782/8 | 15:40 |
gilliard | Of *course* there isn't a problem with *my* code :) | 15:41 |
NobodyCam | lol :) | 15:41 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic-specs: Add pluggable network provider spec https://review.openstack.org/143480 | 15:42 |
jroll | NobodyCam: added a note about VLAN vs network | 15:42 |
NobodyCam | jroll: :) will take a look. soon-ish | 15:43 |
jroll | yeah, might push another one quick | 15:43 |
NobodyCam | ugh I have a 10:00 conf call so I will have to leave our bug day earily | 15:44 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic-specs: Add pluggable network provider spec https://review.openstack.org/143480 | 15:44 |
*** ndipanov_ has joined #openstack-ironic | 15:45 | |
*** r-daneel has quit IRC | 15:49 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic-specs: iRMC Virtual Media Deploy Driver for Ironic https://review.openstack.org/134865 | 15:52 |
dtantsur | jroll, 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 |
dtantsur | jroll, and g'morning :) | 15:54 |
jroll | morning dtantsur :) | 15:54 |
jroll | dtantsur: this started as a PoC that ended up in production :P | 15:55 |
dtantsur | jroll, oh I know that feeling :D | 15:56 |
jroll | so 1) no reason for or against, just didn't do it | 15:56 |
jroll | 2) no idea | 15:56 |
jroll | however | 15:56 |
jroll | we're re-working this so that it's a drop in extension for neutron | 15:56 |
jroll | and AIUI won't need to copy anything from neutron | 15:56 |
dtantsur | ah good | 15:56 |
jroll | dtantsur: morgabra wrote most of that, he can maybe speak to it better than I can | 15:57 |
jroll | he'll also be at the SF meetup if you can make it to that | 15:57 |
dtantsur | jroll, no, SF is definitely unrealistic for me :( | 15:57 |
naohirot | good 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 |
jroll | dtantsur: I figured :( | 15:58 |
*** naohirot has quit IRC | 15:58 | |
NobodyCam | brb | 16:03 |
*** rameshg87 has joined #openstack-ironic | 16:04 | |
rameshg87 | victor_lowther, can you please take a look at raid configuration spec: https://review.openstack.org/#/c/135899/ | 16:07 |
vdrok | hi everyone! | 16:10 |
rameshg87 | hello vdrok | 16:12 |
vdrok | rameshg87, hi | 16:12 |
vdrok | rameshg87, I have a small question about https://review.openstack.org/#/c/145254/ | 16:12 |
jroll | morning vdrok rameshg87 :) | 16:12 |
rameshg87 | morning jroll :) | 16:12 |
rameshg87 | vdrok, yes | 16:12 |
vdrok | morning jroll | 16:12 |
vdrok | rameshg87, looking at ilo drivers | 16:12 |
vdrok | rameshg87, seems that doing clean_up on first deploying a node would break deployment? | 16:13 |
vdrok | rameshg87, as it tries to delete floppy image from swift | 16:13 |
vdrok | rameshg87, and unattach FLOPPY and CDROM? | 16:13 |
vdrok | rameshg87, I'm talking about this addition https://review.openstack.org/#/c/145254/3/ironic/conductor/manager.py | 16:14 |
rameshg87 | vdrok, i think it's perfectly fine because everything required for deploy is redone in in prepare() and deploy() which follows it | 16:15 |
rameshg87 | vdrok, i can give a test with your changes on ilo driver and inform you | 16:16 |
* rameshg87 remembers how imporant 3rd party ci is :( | 16:16 | |
vdrok | rameshg87, I'm concerned about this one - https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ilo/common.py#L394-L424 | 16:16 |
rameshg87 | vdrok, we are working on our ci and would have something in a week's time | 16:16 |
vdrok | rameshg87, i was unclear a bit, it wont break, but will give errors to log | 16:17 |
vdrok | rameshg87, when doing deploy, not rebuild | 16:17 |
rameshg87 | vdrok, oh i missed that part | 16:18 |
rameshg87 | vdrok, i hadn't seen patchset 3 infact | 16:18 |
rameshg87 | vdrok, i remember you were calling clean_up() only during rebuild in previous patchset | 16:18 |
vdrok | rameshg87, agent and pxe seem to work fine with these changes | 16:18 |
rameshg87 | vdrok, why don't we do it only on rebuild ? | 16:19 |
jroll | once this fix lands, we should change our tempest jobs to switch images when rebuilding | 16:19 |
vdrok | rameshg87, right, if doing it during rebuild will require to add some flag to do_node_deploy - whether it's rebuild or not | 16:19 |
jroll | e.g. reproduce this in tempest, fix it, update tempest to use it | 16:19 |
rameshg87 | vdrok, doesn't that make more sense given that only rebuild requires clean_up | 16:19 |
rameshg87 | vdrok, yeah that's easy right ? you could pass rebuild=True to _do_node_deploy() | 16:20 |
rameshg87 | vdrok, just like do_node_deploy() gets that argument | 16:20 |
vdrok | rameshg87, yup, just was trying to make it less code :) | 16:20 |
rameshg87 | vdrok, won't actually end up doing more work in the conductor ? :) | 16:21 |
vdrok | rameshg87, now i'm in doubt :) | 16:21 |
rameshg87 | vdrok, clean_up() needn't be called for each deploy, but we will end up calling more infact | 16:21 |
rameshg87 | vdrok, in ilo driver, it ends up talking to the bmc (to eject the virtual media devices) | 16:22 |
* dtantsur brb | 16:22 | |
vdrok | rameshg87, so you think it's more appropriate to fix it in image_cache? | 16:22 |
rameshg87 | vdrok, i feel we should do it only for rebuild just like your previous patch | 16:22 |
rameshg87 | vdrok, nope, i feel this is much better and clear, just that it needs to be done only for rebuild :) | 16:23 |
rameshg87 | jroll, have some thoughts ?? ^^ | 16:23 |
vdrok | rameshg87, also it might be the case that some old images would be deleted that are used for current node | 16:23 |
vdrok | rameshg87, and then should be downloaded again | 16:23 |
vdrok | if doing clean_up | 16:23 |
jroll | rameshg87: why doesn't just checking if the cached images are different, if so remove them, work? | 16:24 |
rameshg87 | jroll, work on won't work ? | 16:24 |
rameshg87 | jroll, work or won't work ? | 16:24 |
vdrok | jroll, it will, just it seemed easier to fix it in conductor at first glance | 16:24 |
jroll | vdrok: rameshg87: https://review.openstack.org/#/c/145254/1/ironic/drivers/modules/image_cache.py | 16:25 |
jroll | why doesn't that method work (other than the actual code doesn't work) | 16:25 |
jroll | seems like calling clean_up() might be overkill, idk | 16:26 |
*** ryanpetrello has quit IRC | 16:26 | |
vdrok | jroll, after looking deeper in code i tend to agree | 16:26 |
vdrok | rameshg87, jroll, i think i'll switch to fix in image_cache | 16:27 |
jroll | vdrok: cool, poke me when that's up and I'll review :) | 16:27 |
rameshg87 | vdrok, okay | 16:27 |
NobodyCam | rloo: something like this: http://paste.openstack.org/show/jVjnrPLMkmX0QHxG644A | 16:27 |
* rloo looks | 16:27 | |
rloo | NobodyCam: does that code work? you have _state_save and _save_state. but yeah, that's the general idea ;) | 16:29 |
NobodyCam | rloo: ya just quick mock up in paste screen | 16:30 |
rloo | NobodyCam: could just use 'prev_prov' and 'prev_target' variables. or 'prev_states'. this is local so I don't think you need _ | 16:30 |
NobodyCam | just wanted to see if it was what you have in mind | 16:30 |
*** ChuckC_ is now known as ChuckC | 16:30 | |
NobodyCam | rloo: ack.. will do for real now | 16:31 |
rloo | NobodyCam: I can't see the rest of the code, but that comment may be somewhat odd there. not sure. | 16:31 |
rloo | NobodyCam: the comment about 'stash current states...' | 16:31 |
rloo | NobodyCam: 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 |
NobodyCam | rloo: ya I'll update to say save nodes states | 16:32 |
NobodyCam | ok | 16:32 |
NobodyCam | i'll brb | 16:32 |
rloo | NobodyCam: 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-ironic | 16:34 | |
NobodyCam | ahh and not move the code :) | 16:35 |
rloo | NobodyCam: yeah. | 16:36 |
rameshg87 | rloo, do you mean s/if err_handler/if callback and err_handler/ on the above ? | 16:36 |
rloo | rameshg87: yeah, if i understand your question | 16:37 |
rloo | rameshg87: does that make sense? | 16:37 |
rameshg87 | rloo, won't that mean that callback cannot be provided without an error handler ? | 16:37 |
rameshg87 | rloo, error handler is not a *must* for callback right ? | 16:37 |
rloo | rameshg87: we only need to deail with err_handler if callback is true, right? | 16:38 |
rloo | rameshg87: I mean make that change with the original code, not NobodyCam's example | 16:38 |
rameshg87 | rloo, ah okay :) | 16:38 |
rloo | rameshg87: I added a comment about it in the patch. | 16:38 |
rloo | rameshg87: good :) | 16:39 |
rameshg87 | rloo, i missed that point, makes sense | 16:39 |
*** Masahiro has joined #openstack-ironic | 16:39 | |
*** jcoufal_ has quit IRC | 16:39 | |
*** Marga_ has quit IRC | 16:41 | |
*** Masahiro has quit IRC | 16:43 | |
*** ChuckC has quit IRC | 16:44 | |
*** zz_jgrimm is now known as jgrimm | 16:48 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Ensure that image link points to the correct image https://review.openstack.org/145254 | 16:49 |
dtantsur | If I'm not confusing anything, we have bug clean up party in 5 minutes, right? | 16:54 |
JayF | Ah I forgot about that | 16:55 |
openstackgerrit | Ruby Loo proposed openstack/ironic-python-agent: Consistent way to set details for Error instances https://review.openstack.org/146924 | 16:55 |
JayF | oooh | 16:57 |
JayF | that's an interesting review | 16:57 |
*** jcoufal has joined #openstack-ironic | 16:58 | |
*** igordcard has joined #openstack-ironic | 16:58 | |
victor_lowther | rameshg87: Sure, will take a look at the raid spec. | 16:59 |
rameshg87 | victor_lowther, thanks | 16:59 |
*** derekh has quit IRC | 16:59 | |
*** rwsu has joined #openstack-ironic | 16:59 | |
dtantsur | What 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 |
dtantsur | some are around for quite a while already. who is participating? | 17:00 |
JayF | I'm here and will participate | 17:01 |
* dtantsur is looking at https://bugs.launchpad.net/ironic/+bug/1286229 reported by devananda | 17:01 | |
JayF | jroll: ^ you in? | 17:01 |
rloo | dtantsur: I'm willing to help ;) | 17:02 |
jroll | JayF: yeah, going to finish a few other things real quick | 17:02 |
rloo | (after I finish writing up a bug) | 17:02 |
dtantsur | awesome :) | 17:02 |
JayF | I'm looking at https://www.youtube.com/watch?v=STH9ZpeFH2o | 17:03 |
JayF | erm, wrong link. LOL | 17:03 |
jroll | bahaha | 17:03 |
dtantsur | LOL | 17:03 |
JayF | although that's a hilarious video :P | 17:03 |
JayF | I'm looking at -> https://bugs.launchpad.net/ironic/+bug/1358820 | 17:03 |
dtantsur | so 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#L126 | 17:04 |
*** igordcard has quit IRC | 17:04 | |
rloo | JayF: I just added 1410372 -- one of you IPA guys should know if it is a bug or not. I wasn't sure. | 17:05 |
dtantsur | JayF, yeah, I don't understand how to reproduce 1358820 | 17:05 |
JayF | I think we apparently must have some tests | 17:06 |
JayF | which run against a real mysql db if it exists | 17:06 |
JayF | or something like that | 17:06 |
*** ChuckC has joined #openstack-ironic | 17:06 | |
JayF | I was going to spin up a container configured like he was talking about and try to repro | 17:06 |
dtantsur | JayF, will be awesome | 17:06 |
* dtantsur is looking at https://bugs.launchpad.net/ironic/+bug/1386978 | 17:07 | |
devananda | morning, all - sorry for being late | 17:07 |
JayF | rloo: 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 RESTError | 17:07 |
devananda | got pulled into the office today, so I'm onthe road right now | 17:07 |
NobodyCam | morning devananda | 17:07 |
JayF | rloo: but in case of things like UnknownNodeError; it failed lookup, so can't funnel a response back to Ironic | 17:07 |
NobodyCam | devananda: fyi: I'm updatingthe FSM patches atm | 17:07 |
dtantsur | devananda, o/ | 17:08 |
devananda | NobodyCam: thanks | 17:08 |
JayF | rloo: so I think for those, the answer for one may be yes, one may be no, and one may be "sometimes" | 17:08 |
JayF | rloo: 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 |
dtantsur | so, what we can fix about 1386978 is to make sure we do report the actual SSH failure, even if logging level is WARNING | 17:10 |
rloo | JayF: so how can that file be 'fixed' so it is clear what it should be :-) Maybe you can triage the bug. | 17:10 |
dtantsur | currently it's DEBUG level: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ssh.py#L268 | 17:11 |
JayF | rloo: 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 IRC | 17:11 | |
rloo | JayF: ah, ok. Probably some documentation could be added to that file then, and someone review those three to see which if any should be RESTErrors | 17:12 |
JayF | jroll: ^ does my description of RESTError vs non-RESTError in Ironic seem sane? | 17:12 |
JayF | if so I'll toss that in the bug report | 17:13 |
JayF | and modify it to say "document this in the file, too" | 17:13 |
rloo | dtantsur: that makes sense. | 17:13 |
dtantsur | right, I've updated the bug | 17:14 |
jroll | JayF: I agree, though everything could be a resterror and it would still work as expected | 17:14 |
rloo | dtantsur: is the idea for all of us to work on one bug at a time? | 17:14 |
JayF | If that's true; then we should probably make everything a RESTError | 17:14 |
dtantsur | rloo, depending on bug complexity | 17:14 |
dtantsur | rloo, e.g. now I'd like all to have a look at https://bugs.launchpad.net/ironic/+bug/1397438 | 17:15 |
*** MattMan has quit IRC | 17:15 | |
* lucas-hungry looks at the bugs assigned to himself | 17:15 | |
*** Hefeweizen has joined #openstack-ironic | 17:15 | |
dtantsur | JayF, jroll, have you seen something like https://bugs.launchpad.net/ironic/+bug/1397438 ? | 17:16 |
*** penick has joined #openstack-ironic | 17:16 | |
*** lucas-hungry is now known as lucasagomes | 17:16 | |
* dtantsur is also looking at https://bugs.launchpad.net/ironic/+bug/1398582 | 17:16 | |
JayF | dtantsur: not likely something we'd see as we run with dhcp_provider=none and agent driver | 17:17 |
jroll | dtantsur: no, haven't seen that but yeah what jay said | 17:17 |
JayF | which means more or less any conductor can do anything in our stetup | 17:17 |
dtantsur | ah right | 17:17 |
JayF | which lets us completely avoid those kind of bugs | 17:17 |
mjturek | dtantsur, fyi I just grabbed the bug you updated. I'll patch that today | 17:17 |
rloo | dtantsur: so wrt 1397438, seems like someone would need to try to reproduce | 17:17 |
jroll | however if the node went to active, dunno why another conductor would take it over | 17:17 |
jroll | also seems like that person may have flappy conductors | 17:18 |
rloo | and when would a conductor be flappy/flacky? | 17:18 |
jroll | I would guess that something would have to be wrong with rabbit in that case | 17:19 |
rloo | with the hashring stuff, the node should be handled by the same conductor. | 17:19 |
jroll | rloo: right, but if a conductor isn't heartbeating well, it might drop and the hash ring would change | 17:19 |
vdrok | jroll, rameshg87, gilliard: just tested all 3 cases from unit tests https://review.openstack.org/#/c/145254/ - seems to work | 17:20 |
JayF | rloo: make sure to tag agent bugs in the future with 'agent'. I use that search term often :D | 17:20 |
jroll | dtantsur: 1398582 seems valid | 17:21 |
rloo | JayF: ahh, forgot about that. Sorry. | 17:21 |
dtantsur | jroll, yeah, I've updated it. | 17:21 |
JayF | rloo: nbd, as I was going to add low-hanging-fruit anyway :) | 17:21 |
JayF | So I marked https://bugs.launchpad.net/ironic/+bug/1410372 as confirmed, low | 17:21 |
rloo | thx JayF | 17:22 |
dtantsur | JayF, mark "Triaged", if way to fix the bug is clear | 17:22 |
dtantsur | IIRC that's what we agreed on some time ago | 17:22 |
JayF | dtantsur: done | 17:22 |
dtantsur | thnx | 17:22 |
JayF | dtantsur: I think I have an idea on https://bugs.launchpad.net/ironic/+bug/1358820 | 17:22 |
JayF | dtantsur: "mysql" in fedora 20 is mariadb | 17:22 |
dtantsur | JayF, should it make any difference? | 17:24 |
dtantsur | I though this bug is about time out waiting for a partition... | 17:24 |
JayF | it shouldn't, but it does in nova | 17:24 |
JayF | let me find the link to the ML | 17:24 |
*** ryanpetrello has joined #openstack-ironic | 17:24 | |
JayF | dtantsur: http://lists.openstack.org/pipermail/openstack-dev/2015-January/053872.html | 17:25 |
* dtantsur clicks | 17:25 | |
victor_lowther | rameshg87: there ya go. | 17:25 |
JayF | dtantsur: I'm tempted to kick that bug back, and ask the person if it was MariaDB or MySQL | 17:25 |
rameshg87 | victor_lowther, thanks | 17:26 |
rameshg87 | victor_lowther, will take a look | 17:26 |
dtantsur | JayF, still, how do you think parititoning timeout is related to M{ySQL,ariadb}? Or am I missing something? | 17:26 |
rameshg87 | vdrok, okay :) | 17:26 |
jroll | JayF: 1358820 seems more like an selinux thing | 17:27 |
jroll | dtantsur: ^ | 17:27 |
JayF | oooh | 17:27 |
* JayF is just pondering on it while trying to get a test environment up | 17:27 | |
jroll | I don't think mysql/maria is related at all, it's related in nova because that's an actual db thing failing | 17:27 |
dtantsur | well, maybe selinux | 17:27 |
JayF | well I'll know soon eough | 17:28 |
JayF | almsot have maria up on this vm | 17:28 |
JayF | does anyone know how to setup the "openstack_citest" this is talking about? https://bugs.launchpad.net/ironic/+bug/1358820 | 17:29 |
dtantsur | no idea | 17:29 |
*** Marga_ has joined #openstack-ironic | 17:29 | |
jroll | JayF: I think that's just saying, that's the name of the database | 17:30 |
jroll | but idk | 17:30 |
*** Marga_ has quit IRC | 17:30 | |
JayF | that's a special user name | 17:30 |
JayF | used by CI | 17:30 |
JayF | but I can't find docs; only evidence that it's used in automation | 17:31 |
* dtantsur is looking at https://bugs.launchpad.net/ironic/+bug/1397792 in the meanwhile | 17:31 | |
lucasagomes | JayF, I don't know too, but may worth asking for instructions at #openstack-infra | 17:32 |
JayF | I already have :) | 17:32 |
*** Marga_ has joined #openstack-ironic | 17:32 | |
lucasagomes | oh, right :D | 17:32 |
jroll | dtantsur: https://bugs.launchpad.net/ironic/+bug/1397792 sounds like 1) a pbr thing, 2) do we care? | 17:34 |
*** Marga_ has quit IRC | 17:34 | |
*** Marga_ has joined #openstack-ironic | 17:34 | |
jroll | why would you use setup.py install unless you're cloning from git? | 17:34 |
*** Marga_ has quit IRC | 17:35 | |
dtantsur | jroll, 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 |
jroll | dtantsur: will that fix it? | 17:35 |
openstackgerrit | Chris Krelle proposed openstack/ironic: Enable async callbacks from task.process_event() https://review.openstack.org/140868 | 17:35 |
rloo | dtantsur: so the person that reported it, has assigned it to themself. Do we need to verify that it is a problem? | 17:35 |
NobodyCam | rloo: ^^^ let me know ehat you think | 17:36 |
*** Marga_ has joined #openstack-ironic | 17:36 | |
dtantsur | jroll, I think so. | 17:36 |
jroll | huh, ok | 17:36 |
rloo | NobodyCam: looking (don't tell dtantsur) | 17:36 |
dtantsur | lol | 17:36 |
NobodyCam | hehehe | 17:36 |
*** Marga_ has quit IRC | 17:36 | |
dtantsur | rloo, that's part of triaging :) we don't want him/her to waste time fixing what we don't want to fix | 17:36 |
*** ryanpetrello_ has joined #openstack-ironic | 17:37 | |
rloo | dtantsur: ok, so you actually go through all the bugs even if status is not 'new'? | 17:37 |
NobodyCam | quick brb then the rebasing | 17:37 |
dtantsur | jroll, rloo, I'm going to all bugs w/o assigned priority. but NEW bugs are the most important | 17:37 |
*** ryanpetrello has quit IRC | 17:37 | |
*** ryanpetrello_ is now known as ryanpetrello | 17:37 | |
dtantsur | (just wanting to get rid of easy once first) | 17:37 |
dtantsur | interesting, alembic.ini and pxe_config.template are installed | 17:40 |
dtantsur | elilo_efi_pxe_config.template is not. wtf? | 17:40 |
jroll | huh | 17:40 |
jroll | fresh clone? | 17:40 |
*** jistr has quit IRC | 17:40 | |
dtantsur | jroll, honestly it's not fresh, maybe that's the cause | 17:41 |
jroll | yeah, maybe was there already | 17:41 |
* jroll looks for next bug | 17:41 | |
jroll | I'll go from the bottom of new | 17:42 |
jroll | JoshNang: did you end up finding a fix for this? https://bugs.launchpad.net/ironic/+bug/1409169 | 17:42 |
*** vdrok is now known as vdrok_afk | 17:42 | |
jroll | slash root cause | 17:42 |
dtantsur | btw J's, please assign priorities to IPA bugs you report :) | 17:42 |
jroll | yeah :P | 17:42 |
JoshNang | jroll: yup, i had a fix running on that agent that was going crazy | 17:42 |
JoshNang | i need to make it an actual patch though | 17:43 |
jroll | JoshNang: can I assign this to you with medium priority then? | 17:43 |
JoshNang | jroll: sure! | 17:43 |
openstackgerrit | Merged openstack/ironic-python-agent: Error classes invoke their parent's __init__() https://review.openstack.org/146643 | 17:43 |
jroll | k | 17:43 |
jroll | you should push that patch :P | 17:43 |
*** Marga_ has joined #openstack-ironic | 17:45 | |
JayF | jroll: running tox on ironic in the cloud instead of locally = super fast | 17:47 |
jroll | JayF: well aware :P | 17:47 |
*** ryanpetrello has quit IRC | 17:47 | |
jroll | adam_g: I don't see many fails for this, is this bug valid? https://bugs.launchpad.net/ironic/+bug/1408067 | 17:48 |
*** spandhe has joined #openstack-ironic | 17:49 | |
adam_g | jroll, 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 track | 17:50 |
jroll | adam_g: ok, thanks | 17:50 |
adam_g | ill link on the bug | 17:50 |
jroll | I'm going to mark that medium priority, I think | 17:50 |
adam_g | k | 17:51 |
dtantsur | adam_g, hi! could you assign yourself to https://bugs.launchpad.net/ironic/+bug/1408808 ? I guess, you're working on it | 17:52 |
adam_g | dtantsur, done | 17:52 |
dtantsur | thx | 17:52 |
jroll | going to reproduce https://bugs.launchpad.net/ironic/+bug/1406270 | 17:52 |
adam_g | wonder why the bot doesn't do that sometimes | 17:52 |
*** EmilienM is now known as EmilienM|afk | 17:53 | |
dtantsur | adam_g, ditto about https://bugs.launchpad.net/ironic/+bug/1401298 | 17:53 |
dtantsur | and yeah, bot is not doing a good job nowadays | 17:54 |
JayF | Yeah, this fedora bug is valid | 17:55 |
JayF | we have tests escaping | 17:55 |
dtantsur | JayF, ouch! please update the bug | 17:56 |
JayF | ironic/tests/drivers/test_deploy_utils.py for both of them | 17:56 |
dtantsur | lucasagomes, is https://bugs.launchpad.net/ironic/+bug/1357878 fixed? | 17:56 |
*** ChuckC_ has joined #openstack-ironic | 17:57 | |
* lucasagomes looks | 17:57 | |
lucasagomes | yes! | 17:57 |
* lucasagomes mark as committed | 17:57 | |
NobodyCam | check-tripleo-ironic-undercloud-precise-nonha broken? | 17:57 |
*** Marga_ has quit IRC | 17:58 | |
*** Marga_ has joined #openstack-ironic | 17:58 | |
*** ChuckC has quit IRC | 18:00 | |
*** Marga_ has quit IRC | 18:01 | |
dtantsur | NobodyCam, your opinion on FSM-related bug https://bugs.launchpad.net/ironic/+bug/1405420 ? | 18:02 |
rloo | NobodyCam: just gave feedback on https://review.openstack.org/#/c/140868/ | 18:02 |
rloo | dtantsur: wrt 1405420, that is a problem. | 18:02 |
NobodyCam | what I'm getting pep8 errors on files i havent touched | 18:02 |
*** Marga_ has joined #openstack-ironic | 18:02 | |
dtantsur | rloo, could you triage the bug please? | 18:03 |
JayF | https://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 |
JayF | I updated it with my comments, but curious what severity we'd mark that as | 18:03 |
rloo | dtantsur: i can change the status to triage. but i'm not sure what the solution is. | 18:03 |
JayF | Medium or high? | 18:03 |
dtantsur | rloo, then just Confirmed | 18:03 |
dtantsur | JayF, ehhh.. I'm pretty sure I often run tests on F20 | 18:04 |
dtantsur | (and my often I mean always) | 18:04 |
dtantsur | my = by | 18:04 |
rloo | dtantsur: oh. looks like the diff between 'triaged' and 'confirmed' is that confirmed is verified by the 'bug supervisor' | 18:04 |
dtantsur | JayF, any specific conditions to see the failure? | 18:04 |
JayF | dtantsur: I did this: 1) Boot a fedora 20 cloud machine. 2) Install requirements as noted in our docs. 3) Run tox | 18:04 |
dtantsur | rloo, IIRC OpenStack has slightly different idea. lemme find a link | 18:04 |
dtantsur | JayF, then it's medium, I guess | 18:05 |
JayF | dtantsur: selinux = permissive locally, perhaps? | 18:05 |
dtantsur | $ getenforce >> Enforcing | 18:05 |
NobodyCam | dtantsur: on conf call now :( | 18:05 |
JayF | dtantsur: and not as root? | 18:06 |
rloo | dtantsur: that's what I saw in that popup when I clicked the status thingy in the bug | 18:06 |
JayF | this screams undocumented dependency then :( | 18:06 |
dtantsur | JayF, no, my regular user. lemme try with fresh clone, maybe I didn't run all tests since it was broken? | 18:06 |
jroll | JayF: don't be sad, that's way easier to fix :P | 18:06 |
JayF | jroll: well, the dependency would be on a file on disk it's trying to execute | 18:07 |
JayF | jroll: and I'm not sure we should be actually execing to the system in unit tests | 18:07 |
dtantsur | rloo, https://wiki.openstack.org/wiki/Bugs#Status | 18:07 |
jroll | JayF: I agree | 18:07 |
rloo | thx 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 break | 18:09 | |
rloo | devananda: when you have a second, your opinion is desired on https://bugs.launchpad.net/ironic/+bug/1405420. | 18:10 |
*** rameshg87 has quit IRC | 18:10 | |
rloo | devananda: if something is in *FAIL provision state, what operations (or not) can be done on it afterwards. | 18:11 |
jroll | ouch | 18:12 |
jroll | it's because target_provision_state is still set | 18:12 |
jroll | node-update should be able to be done any time a node is in a steady state | 18:12 |
jroll | (e.g. target state is null) | 18:12 |
JayF | how do I ever get it out of deployfail otherwise | 18:13 |
JayF | without massaging the db | 18:13 |
* JayF is going to end up fixing 1358820 because it's too annoying to not know what broke it | 18:13 | |
openstackgerrit | Josh Gachnang proposed openstack/ironic: Improve IPA client library https://review.openstack.org/111118 | 18:14 |
JayF | How do you run a single test in tesr? | 18:16 |
JayF | If I wanted to isolate the test run so I could strace it | 18:16 |
JayF | clif_h: morgabra: What's the Oakland hotel you guys stay at sometimes when coming into the office? | 18:17 |
clif_h | JayF: I've never stayed in Oakland | 18:17 |
JayF | okay, that must've just been Brad then | 18:17 |
morgabra | umm, courtyard right by city center I think | 18:17 |
morgabra | it was fine | 18:17 |
clif_h | and I haven't booked my feb sf travel yet | 18:17 |
clif_h | doing that today | 18:17 |
morgabra | JayF: ^ | 18:17 |
JayF | Shrews: ^ so apparently the courtyard by oakland city center gets a good review from brad | 18:18 |
morgabra | oh god | 18:18 |
clif_h | haha | 18:18 |
JayF | apparently the hotel by the office is like $500/night | 18:18 |
JayF | for the mid-cycle timeframe | 18:18 |
morgabra | as in, I traded money for a place to keep clothes and sometimes sleep | 18:18 |
morgabra | and it satisfied those conditions | 18:18 |
clif_h | is it a conflict of interest to airbnb at a coworkers place? >:) | 18:19 |
JayF | well, I was more concerned about neighborhood | 18:19 |
*** spandhe has quit IRC | 18:20 | |
jroll | JayF: tox -e py27 ironic.tests.drivers.test_agent.TestAgentDeploy.test_validate for example | 18:20 |
jroll | basically the python path to the test | 18:20 |
JayF | jroll: ty | 18:21 |
jroll | np | 18:21 |
JayF | jroll: literally a minute before I was about to strace the whole run | 18:21 |
JayF | lol | 18:21 |
jroll | lol | 18:21 |
jroll | that would be fun | 18:21 |
JayF | eh -e open to limit it to file opens | 18:21 |
dtantsur | JayF, tox succeeded on a fresh clone... | 18:21 |
JayF | and look for things in bin/ | 18:21 |
* dtantsur goes back to break | 18:21 | |
JayF | dtantsur: I think there's an undeclared dependency | 18:22 |
jroll | this hotel discussion reminds me I meant to send an email about the meetup | 18:22 |
JayF | dtantsur: that's not documented to install | 18:22 |
*** spandhe has joined #openstack-ironic | 18:22 | |
JayF | dtantsur: so you have it installed; cloud images don't | 18:22 |
JayF | jroll: I just ran tox with my py27 venv activated :((((( | 18:23 |
jroll | haaaaaa | 18:23 |
jroll | I don't feel for you, I'm running devstack on a new machine | 18:24 |
JayF | Well I had copies of the venv in root/ and ~/ so luckily I'm covered, lol | 18:24 |
*** EmilienM|afk is now known as EmilienM | 18:24 | |
lucasagomes | lol | 18:28 |
lucasagomes | JayF, if u have it activated you can also do a "testr run ironic.tests..." | 18:28 |
jroll | ok, midcycle email sent | 18:28 |
*** Masahiro has joined #openstack-ironic | 18:28 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic-specs: Discover node properties for iLO drivers https://review.openstack.org/103007 | 18:30 |
jroll | morgabra: can you add that hotel here? https://wiki.openstack.org/wiki/Sprints/IronicKiloSprint#San_Francisco_Sprint | 18:31 |
* jroll wonders if there's cheaper hotels closer to the office | 18:31 | |
*** Masahiro has quit IRC | 18:32 | |
* dtantsur is back | 18:33 | |
rloo | mjturek: I see you are working on https://bugs.launchpad.net/bugs/1410372 | 18:35 |
dtantsur | re https://bugs.launchpad.net/ironic/+bug/1400882 by adam_g: I believe new state machine might fix it. Ideas? | 18:35 |
mjturek | rloo yep! | 18:35 |
mjturek | rloo, just need to change the base class for those three correct? | 18:36 |
*** afaranha has joined #openstack-ironic | 18:36 | |
rloo | mjturek: I added some unit tests in a patch so you might want to leverage that. sec. | 18:36 |
JayF | dtantsur: 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 created | 18:36 |
mjturek | rloo cool cool, thanks | 18:36 |
dtantsur | JayF, wow. I wonder why... | 18:37 |
JayF | some missing depedency :) | 18:37 |
JayF | that I am not clever enough to ahve found yet | 18:37 |
dtantsur | JayF, if you try Python's mkdtemp, will it work at all? | 18:37 |
rloo | mjturek: https://review.openstack.org/#/c/146924/ | 18:37 |
afaranha | Hi, I'm trying to deploy a server using ironic but I got an error when deploying it, can anyone help? | 18:38 |
afaranha | I got this error: "Node 6461f204-39a0-4bcf-b88c-875471c05ece is locked by host localhost, please retry after the current operation is completed." | 18:38 |
JayF | dtantsur: yep | 18:38 |
NobodyCam | afaranha: what driver are you useing? | 18:39 |
mjturek | rloo thanks! I'll take a look | 18:39 |
NobodyCam | using even | 18:39 |
jroll | afaranha: 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 harlowja | 18:39 | |
afaranha | ipmitool | 18:39 |
afaranha | I had the same problem as described here | 18:39 |
afaranha | https://software.intel.com/en-us/articles/physical-server-provisioning-with-openstack | 18:39 |
adam_g | dtantsur, sorry, was OTP. | 18:39 |
rloo | afaranha: could be the periodic task to sync power | 18:39 |
afaranha | the system loads the ramdisk and the kernel, and then it says it's ready | 18:39 |
afaranha | and stuck there | 18:39 |
dtantsur | aren't we retrying? | 18:40 |
afaranha | jroll, I tried this but it's not retrying | 18:40 |
rloo | don't we log the retries? | 18:40 |
afaranha | ironic 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 |
afaranha | ops | 18:40 |
afaranha | UUID | Instance UUID | Power State | Provisioning State | Maintenance | 18:40 |
JayF | dtantsur: AHA! | 18:41 |
JayF | dtantsur: we require `fuser` but it's not installed on cloud images or documented | 18:41 |
JayF | woo | 18:41 |
dtantsur | adam_g, NobodyCam, I suspect https://bugs.launchpad.net/ironic/+bug/1400882 will be fixed/made obsolete by new state machine. wdyt? | 18:41 |
dtantsur | JayF, nice :) | 18:41 |
adam_g | dtantsur, 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 should | 18:42 |
jroll | afaranha: right, you need to try deploying again | 18:42 |
afaranha | jroll, just delete the instance and retrying booting it? | 18:42 |
jroll | afaranha: yes | 18:43 |
jroll | after clearing the lock like I mentioned | 18:43 |
dtantsur | adam_g, I'm thinking of closing this as WONTFIX and seeing what we'll get with new state machine. wdyt? | 18:43 |
afaranha | jroll, thats another problem I'm having, the instance is deleted only when I delete the node | 18:43 |
adam_g | dtantsur, 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 |
jroll | afaranha: if the node is locked, nova can't delete the instance | 18:44 |
afaranha | but I already set in the database the reservation to null | 18:44 |
openstackgerrit | Jay Faulkner proposed openstack/ironic: Document dependency on `fuser` for unit tests https://review.openstack.org/146948 | 18:45 |
JayF | dtantsur: others, ^ fix for 1358820 | 18:45 |
dtantsur | thx! | 18:45 |
afaranha | jroll, reservation is null, and if I set the instance_uuid also to null, I still unable to delete the instance :/ | 18:45 |
jroll | afaranha: what error does nova give you | 18:46 |
dtantsur | JayF, don't we need to mention it in the installation guide as well? | 18:46 |
JayF | dtantsur: this is a test-only depedency afaict | 18:46 |
JayF | dtantsur: nope, you're right | 18:47 |
jroll | no it's not | 18:47 |
JayF | dtantsur: will update that | 18:47 |
jroll | cool | 18:47 |
JayF | if that's using packages, the ironic-api package should just depend on psmisc | 18:48 |
JayF | and it not need it there | 18:48 |
jroll | question is, does it depend on psmisc | 18:48 |
JayF | http://packages.ubuntu.com/trusty/net/ironic-api and it doesn't | 18:48 |
* dtantsur is checking Fedora | 18:48 | |
dtantsur | JayF, why -api? | 18:49 |
JayF | they both have the same set of deps | 18:49 |
JayF | and api is less characters | 18:49 |
JayF | lol | 18:49 |
dtantsur | :D | 18:49 |
JayF | dtantsur: if it's yum installable from fedora/rhel, that's not documented either | 18:49 |
afaranha | jroll, 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_instance | 18:50 |
jroll | how do I make launchpad show me closed bugs? | 18:50 |
JayF | hmm. I also guess psmisc is only required for pxe driver. | 18:50 |
openstackgerrit | Jay Faulkner proposed openstack/ironic: Document dependency on `fuser` for pxe driver https://review.openstack.org/146948 | 18:50 |
dtantsur | JayF, anything using disk_partitioner.py is potentially affected | 18:50 |
JayF | which is only pxe deploy driver | 18:50 |
afaranha | jroll, another error: | 18:50 |
afaranha | Error 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_function | 18:51 |
JayF | agent deploy driver does not | 18:51 |
jroll | afaranha: odd, I'm not sure, you'll have to go to the logs | 18:51 |
dtantsur | JayF, still, I suggest you add downstream package to a bug and make them update it | 18:51 |
*** romcheg has quit IRC | 18:51 | |
JayF | dtantsur: IDK how to do that | 18:51 |
*** romcheg has joined #openstack-ironic | 18:51 | |
afaranha | jroll, I'm gonna set the instance_uuid to null and try again | 18:52 |
dtantsur | JayF, "also affects distribution" button | 18:52 |
jroll | ah, found it | 18:52 |
*** romcheg has quit IRC | 18:52 | |
afaranha | jroll, great, the instance was removed, now I'll boot again and check if it works now :) | 18:53 |
*** romcheg has joined #openstack-ironic | 18:53 | |
JayF | dtantsur: done, good call | 18:53 |
jroll | afaranha: great :) | 18:53 |
openstackgerrit | Jay Faulkner proposed openstack/ironic: Document dependency on `fuser` for pxe driver https://review.openstack.org/146948 | 18:55 |
afaranha | jroll, this is really strange, now the ironic node has power state as NULL and Provision state as error | 18:55 |
JayF | dtantsur: ^ I agreed with your comment, so dropped it | 18:55 |
* dtantsur is looking at his own https://bugs.launchpad.net/ironic/+bug/1403106 | 18:55 | |
afaranha | jroll, 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 |
dtantsur | JayF, btw, where do we publish a list of non-pypi dependencies? | 18:56 |
jroll | afaranha: 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 reached | 18:56 |
JayF | dtantsur: we don't, afaict | 18:56 |
dtantsur | oh | 18:56 |
JayF | dtantsur: we do in IPA via the dockerfile, IDK how Ironic does (I suspect not at all) | 18:56 |
jroll | afaranha: 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 out | 18:57 |
* JayF brb | 18:57 | |
*** penick has quit IRC | 18:58 | |
afaranha | jroll, Is there anything I can do to solve that? | 18:58 |
jroll | afaranha: fix your bmc or your network? idk. | 18:59 |
jroll | afaranha: you might try a cold reset on your bmc | 18:59 |
*** penick has joined #openstack-ironic | 19:00 | |
openstackgerrit | Chris Krelle proposed openstack/ironic: Enable async callbacks from task.process_event() https://review.openstack.org/140868 | 19:00 |
afaranha | jroll, 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 |
jroll | afaranha: correct | 19:01 |
*** alexpilotti has quit IRC | 19:01 | |
afaranha | jroll, 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 point | 19:02 |
jroll | afaranha: what kind of hardware are you using? | 19:02 |
jroll | there's a config where you can change time between bmc calls | 19:03 |
jroll | as too many calls too quickly can kill a bmc | 19:03 |
*** ChuckC_ has quit IRC | 19:04 | |
jroll | ipmi.min_command_interval | 19:04 |
jroll | brb | 19:04 |
afaranha | jroll, I'm using an Dell R410 | 19:04 |
afaranha | jroll gonna check that, thanks :) | 19:04 |
dtantsur | folks, 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-ironic | 19:05 | |
NobodyCam | grrrrrr http://logs.openstack.org/68/140868/15/check-tripleo/check-tripleo-ironic-undercloud-precise-nonha/9fbd6d3/console.html#_2015-01-13_19_05_38_843 | 19:07 |
NobodyCam | worth updaing the white board with ^^^^^? | 19:08 |
rloo | NobodyCam: is this a one-off? I suppose it doesn't help to update the white board | 19:09 |
NobodyCam | I've seen it at least twice today | 19:09 |
rloo | then yeah, worth updating. | 19:09 |
rloo | wonder if there is a bug for it | 19:09 |
dtantsur | so 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 |
rloo | thx dtantsur! | 19:11 |
dtantsur | would be nice if someone have a look at the remaining 5 new ones, if you have time | 19:12 |
dtantsur | now g'night and see you tomorrow | 19:12 |
*** erwan_taf has joined #openstack-ironic | 19:13 | |
rloo | night dtantsur | 19:13 |
*** dtantsur is now known as dtantsur|afk | 19:13 | |
JayF | jroll: rfr? I added this: https://wiki.openstack.org/wiki/Sprints/IronicKiloSprint#Accommodation_and_Transportation | 19:14 |
jroll | JayF: lgtm | 19:15 |
*** ChuckC has joined #openstack-ironic | 19:19 | |
* devananda returns from meeting-land | 19:21 | |
*** Nisha has joined #openstack-ironic | 19:24 | |
*** achanda has joined #openstack-ironic | 19:25 | |
*** penick has quit IRC | 19:27 | |
NobodyCam | wb devananda | 19:27 |
*** pensu has left #openstack-ironic | 19:29 | |
devananda | dtantsur|afk: updated https://bugs.launchpad.net/ironic/+bug/1403106 | 19:29 |
*** penick has joined #openstack-ironic | 19:30 | |
lucasagomes | devananda, that's an odd bug, I was trying to reproduce it http://paste.openstack.org/show/157270/ | 19:39 |
jroll | seems like it must be a race | 19:40 |
*** zer0c00l has joined #openstack-ironic | 19:42 | |
lucasagomes | yeah | 19:42 |
lucasagomes | jroll, btw, on the configdrive thing... You want me to check the label after dd'ing the image to the partition? | 19:42 |
JayF | I don't think we should | 19:43 |
lucasagomes | I don't think it's necessary cause maybe we want to use a different label for a different case than the nova configdrive | 19:43 |
JayF | because if configdrive spec changes | 19:43 |
lucasagomes | yeah | 19:43 |
jroll | lucasagomes: nah, I trust it anyway | 19:43 |
JayF | you shouldn't have to change the ironic code that just writes it to disk | 19:43 |
jroll | I just forgot it was in the image | 19:43 |
lucasagomes | jroll, right, you kept the -1 there. Mind taking a look again? | 19:43 |
lucasagomes | JayF, +1 | 19:43 |
jroll | oh, I never updated, sorry | 19:44 |
* jroll looks | 19:44 | |
lucasagomes | thanks :) | 19:44 |
jroll | lucasagomes: so about the requests thing... | 19:47 |
jroll | we already depend on requests though it isn't documented https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent_client.py#L17 | 19:47 |
jroll | I'm going to add that to requirements.txt now | 19:47 |
lucasagomes | jroll, ohh right | 19:48 |
lucasagomes | ok I can change that then, I thought we didn't depended on it (I looked at the requirements.txt) | 19:48 |
jroll | idk, I'm fine with it as-is, I just prefer requests | 19:48 |
jroll | yeah, that's our fault | 19:48 |
NobodyCam | lucasagomes: still around? | 19:49 |
jroll | launchpad is so slow | 19:49 |
lucasagomes | NobodyCam, yup | 19:50 |
NobodyCam | hey lucasagomes just a question on https://review.openstack.org/#/c/142838/ | 19:50 |
lucasagomes | jroll, right... well it's a very simple GET request that's why I didn't mind much either | 19:50 |
jroll | lucasagomes: yeah, I'll +2 | 19:50 |
lucasagomes | jroll, ta much! if it happen to me to send a new patch-set | 19:51 |
lucasagomes | I will update it | 19:51 |
jroll | cool | 19:51 |
JayF | jroll: you patching the requirements.txt? if not I will | 19:51 |
jroll | I got it | 19:51 |
JayF | k | 19:52 |
jroll | took this long for launchpad to give me a bug number | 19:52 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add requests to requirements.txt https://review.openstack.org/146971 | 19:52 |
jroll | there she is | 19:52 |
lucasagomes | NobodyCam, sure, whats up? | 19:52 |
NobodyCam | lucasagomes: just wanted to see if you have a time line for un-wipping it | 19:52 |
*** zer0c00l has quit IRC | 19:52 | |
*** achanda has quit IRC | 19:52 | |
*** achanda has joined #openstack-ironic | 19:55 | |
lucasagomes | NobodyCam, not exactly sure what you mean | 19:55 |
lucasagomes | in case the partitions does appear | 19:55 |
lucasagomes | should I wipe it? | 19:55 |
NobodyCam | lol... gah | 19:56 |
lucasagomes | doesn't* | 19:56 |
NobodyCam | no no the review. is wip just checking if your going to unwip it. I want to point someone to it | 19:56 |
lucasagomes | ohhhhh | 19:56 |
NobodyCam | lol | 19:56 |
lucasagomes | so, this bug was found by our internel qe | 19:57 |
lucasagomes | so I did that fix and posted it so him can test | 19:57 |
NobodyCam | yep same with us | 19:57 |
lucasagomes | but he didn't reply on the ticket yet | 19:57 |
NobodyCam | is it working for your guys | 19:57 |
NobodyCam | ya | 19:57 |
lucasagomes | oh right, I will check with him if he got some time to test it | 19:57 |
lucasagomes | if you guys can give it a go as well and comment on the patch whether it's fixed or not | 19:58 |
NobodyCam | sweet I'm also going to point a few folk to see if they can test it | 19:58 |
lucasagomes | that would be good | 19:58 |
lucasagomes | awesome! thanks | 19:58 |
lucasagomes | the patch is completed I just WIP'ed so we don't merge it until we guarantee it's fixed | 19:58 |
*** pensu has joined #openstack-ironic | 19:58 | |
lucasagomes | I tested it locally it seems fine, cause I know check with devfs to see whether the partition appeared to the OS or not | 19:59 |
*** dprince has quit IRC | 20:00 | |
*** pensu1 has joined #openstack-ironic | 20:02 | |
*** pensu has quit IRC | 20:03 | |
lucasagomes | I now* | 20:04 |
NobodyCam | :) we'll see if we can test it | 20:05 |
*** athomas has quit IRC | 20:06 | |
*** pelix has quit IRC | 20:07 | |
JayF | rloo: 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 |
rloo | JayF: I didn't notice it. Will look... | 20:08 |
JayF | ty | 20:09 |
rloo | JayF: just want to make sure. Can method be None? | 20:12 |
lucasagomes | NobodyCam, cool cheers | 20:12 |
rloo | JayF: if someone codes something incorrectly. | 20:12 |
lucasagomes | I will call it a day as well | 20:12 |
JayF | No; in fact, allowing it to be none is exactly what this is intended to fix | 20:12 |
jroll | rloo: method can be whatever a programmer passes to it | 20:12 |
lucasagomes | a bit late here, have a good night everyone! | 20:12 |
NobodyCam | have a good night lucasagomes | 20:13 |
rloo | jroll: yeah. just wondering how bad a programmer can be. | 20:13 |
jroll | rloo: with this new patch, they would get an exception "only one argument passed, requires 2" or whatever | 20:14 |
jroll | and 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#L376 | 20:14 |
rloo | jroll: yeah, this patch is fine. just wondering about rebasing. | 20:14 |
jroll | so unless a programmer does "raise errors.HardwareManagerMethodNotFound", it's fine | 20:15 |
jroll | rebasing what where | 20:15 |
rloo | JayF, jroll: I have a patch with unit tests, so depending on which lands first, the other will need to be rebased. | 20:15 |
jroll | oh | 20:15 |
JayF | I mean, land yours first? | 20:15 |
JayF | mine is easy to redo | 20:15 |
JayF | you already have a +2 from me on those fixes, btw | 20:16 |
rloo | JayF: either is fine. although of course, i'd rather not spend any more time on ipa :-) | 20:16 |
*** david-lyle has joined #openstack-ironic | 20:16 | |
JayF | but the gate is wedged right now so it's not going to go anywhere | 20:16 |
rloo | JayF: 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 |
JayF | that's perfectly fine :) | 20:16 |
*** achanda has quit IRC | 20:16 | |
JayF | I'm glad you wrote tests | 20:16 |
rloo | JayF: anyway, not a big deal either way. | 20:16 |
*** Masahiro has joined #openstack-ironic | 20:17 | |
*** zer0c00l has joined #openstack-ironic | 20:18 | |
rloo | JayF: I just realized, I don't regularly monitor patches to ipa, that's why I didn't notice your patch. | 20:18 |
JayF | that's fine, not many folks aside from us at Rackspace do | 20:18 |
JayF | which is not ideal, but usually OK | 20:18 |
JayF | just annoying when we have things we want non-Rackspace eyes on (like the HW Manager) | 20:18 |
rloo | JayF: just ping us then ;) | 20:19 |
JayF | that doesn't work as well as you might imagine :) | 20:19 |
JayF | we merged standalone mode which was pasted into channel probably 4-5 times with nobody else looking at it | 20:19 |
JayF | and 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, heh | 20:19 |
*** mikedillion has joined #openstack-ironic | 20:20 | |
rloo | JayF: 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 |
JayF | we just need to make IPA work as the ramdisk for pxe deploy driver too | 20:20 |
JayF | then you'll all be forced to reivew our code! bwahahahaha! | 20:20 |
JayF | jroll: ^ we should seriously do that for Friday's hackday | 20:21 |
jroll | I've said that for the last two hackdays | 20:21 |
jroll | and got distracted | 20:21 |
*** Masahiro has quit IRC | 20:21 | |
rloo | JayF 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 |
jroll | I hear that | 20:22 |
JayF | I'm trying to do the same except the opposite | 20:22 |
JayF | trying to review more Ironic code | 20:22 |
JayF | instead of just looking mostly at specs/IPA | 20:23 |
rloo | thx JayF! | 20:23 |
rloo | I'm worried that we don't have enough people reviewing (cores or no cores). just not-enough-people. | 20:23 |
JayF | have you ever worked on a codebase this size where that hasn't felt true? | 20:26 |
rloo | JayF: this is my first time working with open source. it is a different experience from working in house. | 20:27 |
JayF | I mean, I've not ever worked on a team that exercised the review-then-merge model in any form | 20:27 |
JayF | where "not enough reviews are happening" wasn't a constant problem | 20:27 |
rloo | JayF: 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-ironic | 20:28 | |
JayF | rloo: I completely agree, and honestly, de-prioritize reviewing those specs over those from people who participate fully in the process | 20:28 |
JayF | if 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 mask | 20:28 |
rloo | JayF: 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-ironic | 20:29 | |
JayF | I try to get IPA reviews merged within a week | 20:30 |
*** achanda has joined #openstack-ironic | 20:30 | |
jroll | JayF: keep in mind not everyone has priorities set in a fashion where they have sufficient time to review | 20:30 |
JayF | but that's a luxury of the size | 20:30 |
jroll | (is one of my guesses why some people don't review) | 20:30 |
JayF | jroll: In that case, their company or manager is wearing the mask | 20:30 |
JayF | jroll: not the person; but the result is the same | 20:30 |
jroll | sure | 20:30 |
*** pensu1 has quit IRC | 20:30 | |
jroll | but you can't de-prioritize their work because their manager isn't as awesome as yours | 20:31 |
rloo | so 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 |
jroll | yeah ^ | 20:31 |
Shrews | jroll: I am very surprised that requests isn't pulled in as part of one of the other requirements | 20:34 |
jroll | Shrews: it is, that's why we haven't noticed it | 20:34 |
jroll | that code has been in since july | 20:34 |
Shrews | especially the clients | 20:34 |
jroll | it's not a bug in that ironic doesn't work | 20:34 |
Shrews | oh, you just want to make it explicit then | 20:35 |
jroll | it's a bug in that we don't explicitly declare it | 20:35 |
jroll | yeah | 20:35 |
Shrews | jroll: coolio | 20:35 |
*** mikedillion has quit IRC | 20:36 | |
*** martini__ has joined #openstack-ironic | 20:47 | |
Shrews | jroll: rloo: If I update the commit message on 140869, will you guys +2? | 20:48 |
Shrews | seems that was the only issue you guys had with it | 20:49 |
rloo | for a beer? Nah, not worth a beer. I'll +2 if you update. | 20:50 |
jroll | Shrews: "other than that I'm +2 on this" | 20:50 |
jroll | :) | 20:50 |
NobodyCam | Shrews: 140869 will prob also need a rebase | 20:50 |
Shrews | NobodyCam: yay rebase buttons | 20:51 |
NobodyCam | :) brb | 20:51 |
openstackgerrit | David Shrewsbury proposed openstack/ironic: Convert check_deploy_timeout to use process_event https://review.openstack.org/140869 | 20:52 |
Shrews | jroll: rloo: cool. thanks. ^^^ | 20:52 |
rloo | Shrews: did you rebase it? | 20:52 |
Shrews | oh der | 20:53 |
Shrews | 1 min | 20:53 |
Shrews | no button... so more work | 20:53 |
rloo | There's sometimes a rebase button but I don't see it. | 20:53 |
jroll | which also means there will be junk in the diff between patchsets, lame | 20:54 |
*** erwan_taf has quit IRC | 20:54 | |
*** martini__ has quit IRC | 20:55 | |
*** penick has quit IRC | 20:57 | |
*** achanda has quit IRC | 21:03 | |
*** dprince has quit IRC | 21:03 | |
*** david-ly_ has joined #openstack-ironic | 21:07 | |
*** achanda has joined #openstack-ironic | 21:08 | |
*** david-lyle has quit IRC | 21:09 | |
*** lucasagomes has quit IRC | 21:09 | |
openstackgerrit | John Trowbridge proposed stackforge/ironic-discoverd: Changes utils.get_keystone(token) to utils.check_is_admin(token) https://review.openstack.org/145657 | 21:11 |
*** achanda has quit IRC | 21:12 | |
*** david-ly_ has quit IRC | 21:12 | |
Shrews | i have successfully managed to thoroughly confuse myself | 21:27 |
rloo | Shrews: congratulations! | 21:28 |
rloo | Shrews: it is wrt the rebasing? | 21:29 |
openstackgerrit | Frederic Lepied proposed stackforge/ironic-discoverd: added an eDeploy plugin https://review.openstack.org/146599 | 21:29 |
openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/146692 | 21:30 |
Shrews | rloo: yes :( | 21:30 |
*** Helimanic has quit IRC | 21:32 | |
*** jcoufal has quit IRC | 21:32 | |
*** ndipanov_ has quit IRC | 21:32 | |
*** lazy_prince has quit IRC | 21:32 | |
*** aignatov has quit IRC | 21:32 | |
*** lintan has quit IRC | 21:32 | |
*** harlowja has quit IRC | 21:32 | |
*** jrist has quit IRC | 21:32 | |
*** tteggel has quit IRC | 21:32 | |
*** sirushti has quit IRC | 21:32 | |
*** rodrigods has quit IRC | 21:32 | |
*** lifeless has quit IRC | 21:32 | |
*** annegentle_ has quit IRC | 21:32 | |
*** romcheg has quit IRC | 21:32 | |
*** pcrews has quit IRC | 21:32 | |
*** datajerk has quit IRC | 21:32 | |
*** morgabra has quit IRC | 21:32 | |
*** trown has quit IRC | 21:32 | |
*** yuanying has quit IRC | 21:32 | |
*** slagle has quit IRC | 21:32 | |
*** TheJulia has quit IRC | 21:32 | |
*** NobodyCam has quit IRC | 21:32 | |
*** lynxman has quit IRC | 21:32 | |
*** kragniz has quit IRC | 21:32 | |
*** davidlenwell has quit IRC | 21:32 | |
*** pleia2 has quit IRC | 21:32 | |
*** SpamapS has quit IRC | 21:32 | |
*** Lotus907efi has left #openstack-ironic | 21:33 | |
openstackgerrit | Frederic Lepied proposed stackforge/ironic-discoverd: added an eDeploy plugin https://review.openstack.org/146599 | 21:33 |
*** ChuckC has quit IRC | 21:33 | |
*** Marga_ has quit IRC | 21:33 | |
*** lsmola has quit IRC | 21:33 | |
*** jjohnson2 has quit IRC | 21:33 | |
*** subscope has quit IRC | 21:33 | |
*** bigjools has quit IRC | 21:33 | |
*** vdrok_afk has quit IRC | 21:33 | |
*** yuriyz has quit IRC | 21:33 | |
*** alexm__ has quit IRC | 21:33 | |
openstackgerrit | Michael Turek proposed openstack/ironic: Raise SSH failure messages to the error level. https://review.openstack.org/146985 | 21:34 |
*** SpamapS has joined #openstack-ironic | 21:34 | |
*** annegentle_ has joined #openstack-ironic | 21:34 | |
*** lifeless has joined #openstack-ironic | 21:34 | |
*** rodrigods has joined #openstack-ironic | 21:34 | |
*** sirushti has joined #openstack-ironic | 21:34 | |
*** pleia2 has joined #openstack-ironic | 21:34 | |
*** tteggel has joined #openstack-ironic | 21:34 | |
*** davidlenwell has joined #openstack-ironic | 21:34 | |
*** kragniz has joined #openstack-ironic | 21:34 | |
*** NobodyCam has joined #openstack-ironic | 21:34 | |
*** lynxman has joined #openstack-ironic | 21:34 | |
*** yuanying has joined #openstack-ironic | 21:34 | |
*** trown has joined #openstack-ironic | 21:34 | |
*** TheJulia has joined #openstack-ironic | 21:34 | |
*** jrist has joined #openstack-ironic | 21:34 | |
*** slagle has joined #openstack-ironic | 21:34 | |
*** harlowja has joined #openstack-ironic | 21:34 | |
*** morgabra has joined #openstack-ironic | 21:34 | |
*** datajerk has joined #openstack-ironic | 21:34 | |
*** lintan has joined #openstack-ironic | 21:34 | |
*** aignatov has joined #openstack-ironic | 21:34 | |
*** lazy_prince has joined #openstack-ironic | 21:34 | |
*** pcrews has joined #openstack-ironic | 21:34 | |
*** ndipanov_ has joined #openstack-ironic | 21:34 | |
*** jcoufal has joined #openstack-ironic | 21:34 | |
*** romcheg has joined #openstack-ironic | 21:34 | |
*** ChuckC has joined #openstack-ironic | 21:34 | |
*** Marga_ has joined #openstack-ironic | 21:34 | |
*** lsmola has joined #openstack-ironic | 21:34 | |
*** jjohnson2 has joined #openstack-ironic | 21:34 | |
*** subscope has joined #openstack-ironic | 21:34 | |
*** alexm__ has joined #openstack-ironic | 21:34 | |
*** bigjools has joined #openstack-ironic | 21:34 | |
*** vdrok_afk has joined #openstack-ironic | 21:34 | |
*** yuriyz has joined #openstack-ironic | 21:34 | |
*** aweeks has quit IRC | 21:35 | |
*** zsmithnyc has quit IRC | 21:35 | |
*** xxj has quit IRC | 21:35 | |
*** HenryG has quit IRC | 21:35 | |
*** xxj has joined #openstack-ironic | 21:35 | |
*** spandhe has quit IRC | 21:36 | |
*** yjiang5 has quit IRC | 21:36 | |
*** hypnosb has quit IRC | 21:36 | |
*** spandhe has joined #openstack-ironic | 21:37 | |
*** yjiang5 has joined #openstack-ironic | 21:37 | |
*** hypnosb has joined #openstack-ironic | 21:37 | |
*** zsmithnyc has joined #openstack-ironic | 21:37 | |
*** HenryG has joined #openstack-ironic | 21:37 | |
*** afaranha has quit IRC | 21:38 | |
*** victor_lowther has quit IRC | 21:38 | |
*** agordeev has quit IRC | 21:38 | |
*** bnemec has quit IRC | 21:38 | |
*** med_ has quit IRC | 21:38 | |
*** kylestev has quit IRC | 21:38 | |
*** ekarlso- has quit IRC | 21:38 | |
*** marios has quit IRC | 21:38 | |
openstackgerrit | David Shrewsbury proposed openstack/ironic: Convert check_deploy_timeout to use process_event https://review.openstack.org/140869 | 21:39 |
jroll | Shrews: you did it! | 21:40 |
*** afaranha has joined #openstack-ironic | 21:40 | |
*** victor_lowther has joined #openstack-ironic | 21:40 | |
*** agordeev has joined #openstack-ironic | 21:40 | |
*** bnemec has joined #openstack-ironic | 21:40 | |
*** med_ has joined #openstack-ironic | 21:40 | |
*** kylestev has joined #openstack-ironic | 21:40 | |
*** ekarlso- has joined #openstack-ironic | 21:40 | |
*** marios has joined #openstack-ironic | 21:40 | |
*** achanda has joined #openstack-ironic | 21:41 | |
Shrews | jroll: amazing, ain't it? | 21:41 |
*** david-lyle has joined #openstack-ironic | 21:41 | |
jroll | yeah :P | 21:41 |
jroll | +2'd | 21:41 |
Shrews | though i still don't know the "proper" procedure people use when they submit multiple rebases at once | 21:42 |
*** arif-ali has quit IRC | 21:42 | |
NobodyCam | deva does that. I find it to be black magic | 21:42 |
devananda | muahahaha | 21:42 |
*** arif-ali has joined #openstack-ironic | 21:42 | |
devananda | wait, huh? | 21:42 |
NobodyCam | lol | 21:42 |
Shrews | devananda: multiple 'git review -x' attempts? | 21:42 |
NobodyCam | multi patch rebasing | 21:42 |
devananda | Shrews: that's one way | 21:43 |
devananda | either | 21:43 |
*** rwsu has quit IRC | 21:43 | |
*** mjturek has quit IRC | 21:43 | |
*** jgrimm has quit IRC | 21:43 | |
*** enikanorov__ has quit IRC | 21:43 | |
*** cinerama has quit IRC | 21:43 | |
*** rwsu has joined #openstack-ironic | 21:43 | |
*** mjturek has joined #openstack-ironic | 21:43 | |
*** jgrimm has joined #openstack-ironic | 21:43 | |
*** enikanorov__ has joined #openstack-ironic | 21:43 | |
*** cinerama has joined #openstack-ironic | 21:43 | |
jroll | yeah, I use a lot of -x for that | 21:43 |
*** Nisha has quit IRC | 21:43 | |
devananda | pick a base. repeat(git review -x) for each patch. git review | 21:43 |
jroll | also -R helps not re-submit patches you didn't change | 21:43 |
devananda | or if I know the sha's already, I'll just git rebase -i BASE_SHA | 21:43 |
devananda | then edit there | 21:44 |
devananda | jroll: sometimes | 21:44 |
*** achanda has quit IRC | 21:44 | |
*** Hefeweizen has quit IRC | 21:44 | |
*** openstackgerrit has quit IRC | 21:44 | |
*** kbs has quit IRC | 21:44 | |
*** pradipta_away has quit IRC | 21:44 | |
*** alex_xu has quit IRC | 21:44 | |
*** cppforlife has quit IRC | 21:44 | |
*** dividehex has quit IRC | 21:44 | |
*** dguerri_ has quit IRC | 21:44 | |
*** kbyrne has quit IRC | 21:44 | |
*** PaulCzar has quit IRC | 21:44 | |
*** coolsvap|afk has quit IRC | 21:44 | |
*** vipul has quit IRC | 21:44 | |
*** yarker has quit IRC | 21:44 | |
*** greghaynes has quit IRC | 21:44 | |
*** lxsli has quit IRC | 21:44 | |
*** toabctl has quit IRC | 21:44 | |
*** mrda has quit IRC | 21:44 | |
*** tonyb has quit IRC | 21:44 | |
*** rwsu has quit IRC | 21:44 | |
*** mjturek has quit IRC | 21:44 | |
*** jgrimm has quit IRC | 21:44 | |
*** enikanorov__ has quit IRC | 21:44 | |
*** cinerama has quit IRC | 21:44 | |
*** arif-ali has quit IRC | 21:44 | |
*** afaranha has quit IRC | 21:44 | |
*** victor_lowther has quit IRC | 21:44 | |
*** agordeev has quit IRC | 21:44 | |
*** bnemec has quit IRC | 21:45 | |
*** med_ has quit IRC | 21:45 | |
*** kylestev has quit IRC | 21:45 | |
*** ekarlso- has quit IRC | 21:45 | |
*** marios has quit IRC | 21:45 | |
*** spandhe has quit IRC | 21:45 | |
*** yjiang5 has quit IRC | 21:45 | |
*** hypnosb has quit IRC | 21:45 | |
*** xxj has quit IRC | 21:45 | |
*** david-lyle has quit IRC | 21:45 | |
*** jcoufal has quit IRC | 21:45 | |
*** ndipanov_ has quit IRC | 21:45 | |
*** lazy_prince has quit IRC | 21:45 | |
*** aignatov has quit IRC | 21:45 | |
*** lintan has quit IRC | 21:45 | |
*** harlowja has quit IRC | 21:45 | |
*** jrist has quit IRC | 21:45 | |
*** tteggel has quit IRC | 21:45 | |
*** sirushti has quit IRC | 21:45 | |
*** rodrigods has quit IRC | 21:45 | |
*** lifeless has quit IRC | 21:45 | |
*** annegentle_ has quit IRC | 21:45 | |
*** romcheg has quit IRC | 21:45 | |
*** pcrews has quit IRC | 21:45 | |
*** datajerk has quit IRC | 21:45 | |
*** morgabra has quit IRC | 21:45 | |
*** trown has quit IRC | 21:45 | |
*** yuanying has quit IRC | 21:45 | |
*** HenryG has quit IRC | 21:45 | |
*** slagle has quit IRC | 21:45 | |
*** TheJulia has quit IRC | 21:45 | |
*** NobodyCam has quit IRC | 21:45 | |
*** lynxman has quit IRC | 21:45 | |
*** kragniz has quit IRC | 21:45 | |
*** davidlenwell has quit IRC | 21:45 | |
*** pleia2 has quit IRC | 21:45 | |
*** SpamapS has quit IRC | 21:45 | |
*** ChuckC has quit IRC | 21:45 | |
*** Marga_ has quit IRC | 21:45 | |
*** lsmola has quit IRC | 21:45 | |
*** jjohnson2 has quit IRC | 21:45 | |
*** subscope has quit IRC | 21:45 | |
*** bigjools has quit IRC | 21:45 | |
*** vdrok_afk has quit IRC | 21:45 | |
*** yuriyz has quit IRC | 21:45 | |
*** alexm__ has quit IRC | 21:45 | |
*** dlaube has quit IRC | 21:45 | |
*** rloo has quit IRC | 21:46 | |
*** Ng has quit IRC | 21:46 | |
*** krtaylor has quit IRC | 21:46 | |
*** russell_h has quit IRC | 21:46 | |
*** mgagne has quit IRC | 21:46 | |
*** dhellmann has quit IRC | 21:46 | |
*** pcaruana has quit IRC | 21:46 | |
*** dtantsur|afk has quit IRC | 21:46 | |
*** GheRivero has quit IRC | 21:46 | |
*** soren has quit IRC | 21:46 | |
*** Madasi has quit IRC | 21:46 | |
*** anteaya has quit IRC | 21:46 | |
*** devananda has quit IRC | 21:46 | |
*** tchaypo has quit IRC | 21:46 | |
*** JayF has quit IRC | 21:46 | |
*** kevinbenton has quit IRC | 21:46 | |
*** JoshNang has quit IRC | 21:46 | |
*** clif_h has quit IRC | 21:46 | |
*** jeblair has quit IRC | 21:46 | |
*** Isotopp has quit IRC | 21:46 | |
*** aszeszo has quit IRC | 21:46 | |
*** adam_g has quit IRC | 21:46 | |
*** d0ugal has quit IRC | 21:46 | |
*** mordred has quit IRC | 21:46 | |
*** Haomeng has quit IRC | 21:46 | |
*** wendar has quit IRC | 21:46 | |
*** sbadia has quit IRC | 21:46 | |
*** Shrews has quit IRC | 21:46 | |
*** antonym has quit IRC | 21:46 | |
*** dvorak has quit IRC | 21:46 | |
*** mikal has quit IRC | 21:46 | |
*** keekz has quit IRC | 21:46 | |
*** openstack has joined #openstack-ironic | 21:48 | |
*** aweeks has joined #openstack-ironic | 21:49 | |
*** david-lyle has joined #openstack-ironic | 21:51 | |
*** cinerama has joined #openstack-ironic | 21:51 | |
*** enikanorov__ has joined #openstack-ironic | 21:51 | |
*** jgrimm has joined #openstack-ironic | 21:51 | |
*** mjturek has joined #openstack-ironic | 21:51 | |
*** rwsu has joined #openstack-ironic | 21:51 | |
*** arif-ali has joined #openstack-ironic | 21:51 | |
*** marios has joined #openstack-ironic | 21:51 | |
*** ekarlso- has joined #openstack-ironic | 21:51 | |
*** kylestev has joined #openstack-ironic | 21:51 | |
*** med_ has joined #openstack-ironic | 21:51 | |
*** bnemec has joined #openstack-ironic | 21:51 | |
*** agordeev has joined #openstack-ironic | 21:51 | |
*** victor_lowther has joined #openstack-ironic | 21:51 | |
*** afaranha has joined #openstack-ironic | 21:51 | |
*** HenryG has joined #openstack-ironic | 21:51 | |
*** hypnosb has joined #openstack-ironic | 21:51 | |
*** yjiang5 has joined #openstack-ironic | 21:51 | |
*** spandhe has joined #openstack-ironic | 21:51 | |
*** xxj has joined #openstack-ironic | 21:51 | |
*** yuriyz has joined #openstack-ironic | 21:51 | |
*** vdrok_afk has joined #openstack-ironic | 21:51 | |
*** bigjools has joined #openstack-ironic | 21:51 | |
*** alexm__ has joined #openstack-ironic | 21:51 | |
*** subscope has joined #openstack-ironic | 21:51 | |
*** jjohnson2 has joined #openstack-ironic | 21:51 | |
*** lsmola has joined #openstack-ironic | 21:51 | |
*** Marga_ has joined #openstack-ironic | 21:51 | |
*** ChuckC has joined #openstack-ironic | 21:51 | |
*** romcheg has joined #openstack-ironic | 21:51 | |
*** jcoufal has joined #openstack-ironic | 21:51 | |
*** ndipanov_ has joined #openstack-ironic | 21:51 | |
*** pcrews has joined #openstack-ironic | 21:51 | |
*** lazy_prince has joined #openstack-ironic | 21:51 | |
*** aignatov has joined #openstack-ironic | 21:51 | |
*** lintan has joined #openstack-ironic | 21:51 | |
*** datajerk has joined #openstack-ironic | 21:51 | |
*** morgabra has joined #openstack-ironic | 21:51 | |
*** harlowja has joined #openstack-ironic | 21:51 | |
*** slagle has joined #openstack-ironic | 21:51 | |
*** jrist has joined #openstack-ironic | 21:51 | |
*** TheJulia has joined #openstack-ironic | 21:51 | |
*** trown has joined #openstack-ironic | 21:51 | |
*** yuanying has joined #openstack-ironic | 21:51 | |
*** lynxman has joined #openstack-ironic | 21:51 | |
*** NobodyCam has joined #openstack-ironic | 21:51 | |
*** kragniz has joined #openstack-ironic | 21:51 | |
*** davidlenwell has joined #openstack-ironic | 21:51 | |
*** tteggel has joined #openstack-ironic | 21:51 | |
*** pleia2 has joined #openstack-ironic | 21:51 | |
*** sirushti has joined #openstack-ironic | 21:51 | |
*** rodrigods has joined #openstack-ironic | 21:51 | |
*** lifeless has joined #openstack-ironic | 21:51 | |
*** annegentle_ has joined #openstack-ironic | 21:51 | |
*** SpamapS has joined #openstack-ironic | 21:51 | |
*** dlaube has joined #openstack-ironic | 21:51 | |
*** rloo has joined #openstack-ironic | 21:51 | |
*** kbyrne has joined #openstack-ironic | 21:51 | |
*** Haomeng has joined #openstack-ironic | 21:51 | |
*** Ng has joined #openstack-ironic | 21:51 | |
*** krtaylor has joined #openstack-ironic | 21:51 | |
*** russell_h has joined #openstack-ironic | 21:51 | |
*** mgagne has joined #openstack-ironic | 21:51 | |
*** PaulCzar has joined #openstack-ironic | 21:51 | |
*** wendar has joined #openstack-ironic | 21:51 | |
*** coolsvap|afk has joined #openstack-ironic | 21:51 | |
*** dhellmann has joined #openstack-ironic | 21:51 | |
*** pcaruana has joined #openstack-ironic | 21:51 | |
*** dtantsur|afk has joined #openstack-ironic | 21:51 | |
*** lxsli has joined #openstack-ironic | 21:51 | |
*** GheRivero has joined #openstack-ironic | 21:51 | |
*** sbadia has joined #openstack-ironic | 21:51 | |
*** soren has joined #openstack-ironic | 21:51 | |
*** vipul has joined #openstack-ironic | 21:51 | |
*** Madasi has joined #openstack-ironic | 21:51 | |
*** yarker has joined #openstack-ironic | 21:51 | |
*** Shrews has joined #openstack-ironic | 21:51 | |
*** greghaynes has joined #openstack-ironic | 21:51 | |
*** antonym has joined #openstack-ironic | 21:51 | |
*** tchaypo has joined #openstack-ironic | 21:51 | |
*** anteaya has joined #openstack-ironic | 21:51 | |
*** dvorak has joined #openstack-ironic | 21:51 | |
*** JayF has joined #openstack-ironic | 21:51 | |
*** kevinbenton has joined #openstack-ironic | 21:51 | |
*** mikal has joined #openstack-ironic | 21:51 | |
*** JoshNang has joined #openstack-ironic | 21:51 | |
*** toabctl has joined #openstack-ironic | 21:51 | |
*** keekz has joined #openstack-ironic | 21:51 | |
*** devananda has joined #openstack-ironic | 21:51 | |
*** clif_h has joined #openstack-ironic | 21:51 | |
*** jeblair has joined #openstack-ironic | 21:51 | |
*** Isotopp has joined #openstack-ironic | 21:51 | |
*** BadCub_ has joined #openstack-ironic | 21:51 | |
*** aszeszo has joined #openstack-ironic | 21:51 | |
*** adam_g has joined #openstack-ironic | 21:51 | |
*** d0ugal has joined #openstack-ironic | 21:51 | |
*** mordred has joined #openstack-ironic | 21:51 | |
*** tonyb has joined #openstack-ironic | 21:51 | |
*** mrda has joined #openstack-ironic | 21:51 | |
*** docaedo has joined #openstack-ironic | 21:51 | |
*** early has joined #openstack-ironic | 21:51 | |
*** zz_naotok has joined #openstack-ironic | 21:51 | |
*** david-lyle has quit IRC | 21:51 | |
*** david-lyle has joined #openstack-ironic | 21:51 | |
* devananda wanders off to find food | 21:51 | |
*** zsmithnyc has quit IRC | 21:51 | |
*** zsmithnyc has joined #openstack-ironic | 21:51 | |
*** aweeks has quit IRC | 21:52 | |
*** aweeks has joined #openstack-ironic | 21:52 | |
*** dlaube has quit IRC | 21:53 | |
*** rloo has quit IRC | 21:53 | |
*** Ng has quit IRC | 21:53 | |
*** krtaylor has quit IRC | 21:53 | |
*** russell_h has quit IRC | 21:53 | |
*** mgagne has quit IRC | 21:53 | |
*** dhellmann has quit IRC | 21:53 | |
*** pcaruana has quit IRC | 21:53 | |
*** dtantsur|afk has quit IRC | 21:53 | |
*** GheRivero has quit IRC | 21:53 | |
*** soren has quit IRC | 21:53 | |
*** Madasi has quit IRC | 21:53 | |
*** anteaya has quit IRC | 21:53 | |
*** devananda has quit IRC | 21:53 | |
*** dlaube has joined #openstack-ironic | 21:53 | |
*** rloo has joined #openstack-ironic | 21:53 | |
*** Ng has joined #openstack-ironic | 21:53 | |
*** krtaylor has joined #openstack-ironic | 21:53 | |
*** russell_h has joined #openstack-ironic | 21:53 | |
*** mgagne has joined #openstack-ironic | 21:53 | |
*** dhellmann has joined #openstack-ironic | 21:53 | |
*** pcaruana has joined #openstack-ironic | 21:53 | |
*** dtantsur|afk has joined #openstack-ironic | 21:53 | |
*** GheRivero has joined #openstack-ironic | 21:53 | |
*** soren has joined #openstack-ironic | 21:53 | |
*** Madasi has joined #openstack-ironic | 21:53 | |
*** anteaya has joined #openstack-ironic | 21:53 | |
*** devananda has joined #openstack-ironic | 21:53 | |
*** tchaypo has quit IRC | 21:54 | |
*** JayF has quit IRC | 21:54 | |
*** kevinbenton has quit IRC | 21:54 | |
*** JoshNang has quit IRC | 21:54 | |
*** clif_h has quit IRC | 21:54 | |
*** jeblair has quit IRC | 21:54 | |
*** Isotopp has quit IRC | 21:54 | |
*** aszeszo has quit IRC | 21:54 | |
*** adam_g has quit IRC | 21:54 | |
*** d0ugal has quit IRC | 21:54 | |
*** mordred has quit IRC | 21:54 | |
*** achanda has joined #openstack-ironic | 21:54 | |
*** Hefeweizen has joined #openstack-ironic | 21:54 | |
*** openstackgerrit has joined #openstack-ironic | 21:54 | |
*** kbs has joined #openstack-ironic | 21:54 | |
*** pradipta_away has joined #openstack-ironic | 21:54 | |
*** alex_xu has joined #openstack-ironic | 21:54 | |
*** cppforlife has joined #openstack-ironic | 21:54 | |
*** dividehex has joined #openstack-ironic | 21:54 | |
*** dguerri_ has joined #openstack-ironic | 21:54 | |
*** Hefeweiz1n has joined #openstack-ironic | 21:54 | |
*** tchaypo has joined #openstack-ironic | 21:54 | |
*** JayF has joined #openstack-ironic | 21:54 | |
*** kevinbenton has joined #openstack-ironic | 21:54 | |
*** JoshNang has joined #openstack-ironic | 21:54 | |
*** clif_h has joined #openstack-ironic | 21:54 | |
*** jeblair has joined #openstack-ironic | 21:54 | |
*** Isotopp has joined #openstack-ironic | 21:54 | |
*** aszeszo has joined #openstack-ironic | 21:54 | |
*** adam_g has joined #openstack-ironic | 21:54 | |
*** d0ugal has joined #openstack-ironic | 21:54 | |
*** mordred has joined #openstack-ironic | 21:54 | |
*** achanda_ has joined #openstack-ironic | 21:54 | |
*** Haomeng has quit IRC | 21:54 | |
*** wendar has quit IRC | 21:54 | |
*** sbadia has quit IRC | 21:55 | |
*** Shrews has quit IRC | 21:55 | |
*** antonym has quit IRC | 21:55 | |
*** dvorak has quit IRC | 21:55 | |
*** mikal has quit IRC | 21:55 | |
*** keekz has quit IRC | 21:55 | |
*** BadCub_ has quit IRC | 21:55 | |
*** docaedo has quit IRC | 21:55 | |
*** early has quit IRC | 21:55 | |
*** zz_naotok has quit IRC | 21:55 | |
*** cppforlife has quit IRC | 21:55 | |
*** dguerri_ has quit IRC | 21:55 | |
*** achanda has quit IRC | 21:55 | |
*** Hefeweizen has quit IRC | 21:55 | |
*** dguerri has joined #openstack-ironic | 21:55 | |
*** cppforlife has joined #openstack-ironic | 21:59 | |
*** Haomeng has joined #openstack-ironic | 21:59 | |
*** wendar has joined #openstack-ironic | 21:59 | |
*** sbadia has joined #openstack-ironic | 21:59 | |
*** Shrews has joined #openstack-ironic | 21:59 | |
*** antonym has joined #openstack-ironic | 21:59 | |
*** dvorak has joined #openstack-ironic | 21:59 | |
*** mikal has joined #openstack-ironic | 21:59 | |
*** keekz has joined #openstack-ironic | 21:59 | |
*** BadCub_ has joined #openstack-ironic | 21:59 | |
*** docaedo has joined #openstack-ironic | 21:59 | |
*** early has joined #openstack-ironic | 21:59 | |
*** zz_naotok has joined #openstack-ironic | 21:59 | |
*** cppforlife has quit IRC | 22:00 | |
*** cppforlife has joined #openstack-ironic | 22:00 | |
*** lazy_prince has quit IRC | 22:00 | |
Shrews | wheeeeeeee | 22:01 |
*** david-lyle has quit IRC | 22:02 | |
*** kbyrne has quit IRC | 22:02 | |
*** PaulCzar has quit IRC | 22:02 | |
*** coolsvap|afk has quit IRC | 22:02 | |
*** vipul has quit IRC | 22:02 | |
*** yarker has quit IRC | 22:02 | |
*** greghaynes has quit IRC | 22:02 | |
*** lxsli has quit IRC | 22:02 | |
*** toabctl has quit IRC | 22:02 | |
*** mrda has quit IRC | 22:02 | |
*** tonyb has quit IRC | 22:02 | |
*** cppforlife has quit IRC | 22:02 | |
*** Haomeng has quit IRC | 22:02 | |
*** wendar has quit IRC | 22:02 | |
*** sbadia has quit IRC | 22:02 | |
*** Shrews has quit IRC | 22:02 | |
*** antonym has quit IRC | 22:02 | |
*** dvorak has quit IRC | 22:02 | |
*** mikal has quit IRC | 22:02 | |
*** keekz has quit IRC | 22:02 | |
*** BadCub_ has quit IRC | 22:02 | |
*** docaedo has quit IRC | 22:02 | |
*** early has quit IRC | 22:02 | |
*** zz_naotok has quit IRC | 22:02 | |
*** tchaypo has quit IRC | 22:02 | |
*** JayF has quit IRC | 22:02 | |
*** kevinbenton has quit IRC | 22:02 | |
*** JoshNang has quit IRC | 22:02 | |
*** clif_h has quit IRC | 22:02 | |
*** jeblair has quit IRC | 22:03 | |
*** Isotopp has quit IRC | 22:03 | |
*** aszeszo has quit IRC | 22:03 | |
*** adam_g has quit IRC | 22:03 | |
*** d0ugal has quit IRC | 22:03 | |
*** mordred has quit IRC | 22:03 | |
*** dlaube has quit IRC | 22:03 | |
*** rloo has quit IRC | 22:03 | |
*** Ng has quit IRC | 22:03 | |
*** krtaylor has quit IRC | 22:03 | |
*** russell_h has quit IRC | 22:03 | |
*** mgagne has quit IRC | 22:03 | |
*** dhellmann has quit IRC | 22:03 | |
*** pcaruana has quit IRC | 22:03 | |
*** dtantsur|afk has quit IRC | 22:03 | |
*** GheRivero has quit IRC | 22:03 | |
*** soren has quit IRC | 22:03 | |
*** Madasi has quit IRC | 22:03 | |
*** anteaya has quit IRC | 22:03 | |
*** devananda has quit IRC | 22:03 | |
*** dguerri has quit IRC | 22:03 | |
*** openstackgerrit has quit IRC | 22:03 | |
*** kbs has quit IRC | 22:03 | |
*** pradipta_away has quit IRC | 22:03 | |
*** alex_xu has quit IRC | 22:03 | |
*** dividehex has quit IRC | 22:03 | |
*** rwsu has quit IRC | 22:03 | |
*** mjturek has quit IRC | 22:03 | |
*** jgrimm has quit IRC | 22:03 | |
*** enikanorov__ has quit IRC | 22:03 | |
*** cinerama has quit IRC | 22:03 | |
*** arif-ali has quit IRC | 22:03 | |
*** achanda_ has quit IRC | 22:04 | |
*** afaranha has quit IRC | 22:04 | |
*** victor_lowther has quit IRC | 22:04 | |
*** agordeev has quit IRC | 22:04 | |
*** bnemec has quit IRC | 22:04 | |
*** med_ has quit IRC | 22:04 | |
*** kylestev has quit IRC | 22:04 | |
*** ekarlso- has quit IRC | 22:04 | |
*** marios has quit IRC | 22:04 | |
*** spandhe has quit IRC | 22:04 | |
*** yjiang5 has quit IRC | 22:04 | |
*** hypnosb has quit IRC | 22:04 | |
*** xxj has quit IRC | 22:04 | |
*** Hefeweiz1n has quit IRC | 22:04 | |
*** jcoufal has quit IRC | 22:04 | |
*** ndipanov_ has quit IRC | 22:04 | |
*** aignatov has quit IRC | 22:04 | |
*** lintan has quit IRC | 22:04 | |
*** harlowja has quit IRC | 22:04 | |
*** jrist has quit IRC | 22:04 | |
*** tteggel has quit IRC | 22:04 | |
*** sirushti has quit IRC | 22:04 | |
*** rodrigods has quit IRC | 22:04 | |
*** lifeless has quit IRC | 22:04 | |
*** annegentle_ has quit IRC | 22:04 | |
*** romcheg has quit IRC | 22:04 | |
*** pcrews has quit IRC | 22:04 | |
*** datajerk has quit IRC | 22:04 | |
*** morgabra has quit IRC | 22:04 | |
*** trown has quit IRC | 22:04 | |
*** yuanying has quit IRC | 22:04 | |
*** HenryG has quit IRC | 22:05 | |
*** slagle has quit IRC | 22:05 | |
*** TheJulia has quit IRC | 22:05 | |
*** NobodyCam has quit IRC | 22:05 | |
*** lynxman has quit IRC | 22:05 | |
*** kragniz has quit IRC | 22:05 | |
*** davidlenwell has quit IRC | 22:05 | |
*** pleia2 has quit IRC | 22:05 | |
*** SpamapS has quit IRC | 22:05 | |
*** ChuckC has quit IRC | 22:05 | |
*** Marga_ has quit IRC | 22:05 | |
*** lsmola has quit IRC | 22:05 | |
*** jjohnson2 has quit IRC | 22:05 | |
*** subscope has quit IRC | 22:05 | |
*** bigjools has quit IRC | 22:05 | |
*** vdrok_afk has quit IRC | 22:05 | |
*** yuriyz has quit IRC | 22:05 | |
*** alexm__ has quit IRC | 22:05 | |
*** Masahiro has joined #openstack-ironic | 22:06 | |
*** zz_naotok has joined #openstack-ironic | 22:06 | |
*** early has joined #openstack-ironic | 22:06 | |
*** docaedo has joined #openstack-ironic | 22:06 | |
*** BadCub_ has joined #openstack-ironic | 22:06 | |
*** keekz has joined #openstack-ironic | 22:06 | |
*** mikal has joined #openstack-ironic | 22:06 | |
*** dvorak has joined #openstack-ironic | 22:06 | |
*** antonym has joined #openstack-ironic | 22:06 | |
*** Shrews has joined #openstack-ironic | 22:06 | |
*** sbadia has joined #openstack-ironic | 22:06 | |
*** wendar has joined #openstack-ironic | 22:06 | |
*** Haomeng has joined #openstack-ironic | 22:06 | |
*** cppforlife has joined #openstack-ironic | 22:06 | |
*** dguerri has joined #openstack-ironic | 22:06 | |
*** achanda_ has joined #openstack-ironic | 22:06 | |
*** mordred has joined #openstack-ironic | 22:06 | |
*** d0ugal has joined #openstack-ironic | 22:06 | |
*** adam_g has joined #openstack-ironic | 22:06 | |
*** aszeszo has joined #openstack-ironic | 22:06 | |
*** Isotopp has joined #openstack-ironic | 22:06 | |
*** jeblair has joined #openstack-ironic | 22:06 | |
*** clif_h has joined #openstack-ironic | 22:06 | |
*** JoshNang has joined #openstack-ironic | 22:06 | |
*** kevinbenton has joined #openstack-ironic | 22:06 | |
*** JayF has joined #openstack-ironic | 22:06 | |
*** tchaypo has joined #openstack-ironic | 22:06 | |
*** Hefeweiz1n has joined #openstack-ironic | 22:06 | |
*** dividehex has joined #openstack-ironic | 22:06 | |
*** alex_xu has joined #openstack-ironic | 22:06 | |
*** pradipta_away has joined #openstack-ironic | 22:06 | |
*** kbs has joined #openstack-ironic | 22:06 | |
*** openstackgerrit has joined #openstack-ironic | 22:06 | |
*** devananda has joined #openstack-ironic | 22:06 | |
*** anteaya has joined #openstack-ironic | 22:06 | |
*** Madasi has joined #openstack-ironic | 22:06 | |
*** soren has joined #openstack-ironic | 22:06 | |
*** GheRivero has joined #openstack-ironic | 22:06 | |
*** dtantsur|afk has joined #openstack-ironic | 22:06 | |
*** pcaruana has joined #openstack-ironic | 22:06 | |
*** dhellmann has joined #openstack-ironic | 22:06 | |
*** mgagne has joined #openstack-ironic | 22:06 | |
*** russell_h has joined #openstack-ironic | 22:06 | |
*** krtaylor has joined #openstack-ironic | 22:06 | |
*** Ng has joined #openstack-ironic | 22:06 | |
*** rloo has joined #openstack-ironic | 22:06 | |
*** dlaube has joined #openstack-ironic | 22:06 | |
*** david-lyle has joined #openstack-ironic | 22:06 | |
*** cinerama has joined #openstack-ironic | 22:06 | |
*** enikanorov__ has joined #openstack-ironic | 22:06 | |
*** jgrimm has joined #openstack-ironic | 22:06 | |
*** mjturek has joined #openstack-ironic | 22:06 | |
*** rwsu has joined #openstack-ironic | 22:06 | |
*** arif-ali has joined #openstack-ironic | 22:06 | |
*** marios has joined #openstack-ironic | 22:06 | |
*** ekarlso- has joined #openstack-ironic | 22:06 | |
*** kylestev has joined #openstack-ironic | 22:06 | |
*** med_ has joined #openstack-ironic | 22:06 | |
*** bnemec has joined #openstack-ironic | 22:06 | |
*** agordeev has joined #openstack-ironic | 22:06 | |
*** victor_lowther has joined #openstack-ironic | 22:06 | |
*** afaranha has joined #openstack-ironic | 22:06 | |
*** HenryG has joined #openstack-ironic | 22:06 | |
*** hypnosb has joined #openstack-ironic | 22:06 | |
*** yjiang5 has joined #openstack-ironic | 22:06 | |
*** spandhe has joined #openstack-ironic | 22:06 | |
*** xxj has joined #openstack-ironic | 22:06 | |
*** yuriyz has joined #openstack-ironic | 22:06 | |
*** vdrok_afk has joined #openstack-ironic | 22:06 | |
*** bigjools has joined #openstack-ironic | 22:06 | |
*** alexm__ has joined #openstack-ironic | 22:06 | |
*** subscope has joined #openstack-ironic | 22:06 | |
*** lsmola has joined #openstack-ironic | 22:06 | |
*** Marga_ has joined #openstack-ironic | 22:06 | |
*** ChuckC has joined #openstack-ironic | 22:06 | |
*** romcheg has joined #openstack-ironic | 22:06 | |
*** jcoufal has joined #openstack-ironic | 22:06 | |
*** ndipanov_ has joined #openstack-ironic | 22:06 | |
*** pcrews has joined #openstack-ironic | 22:06 | |
*** aignatov has joined #openstack-ironic | 22:06 | |
*** lintan has joined #openstack-ironic | 22:06 | |
*** datajerk has joined #openstack-ironic | 22:06 | |
*** morgabra has joined #openstack-ironic | 22:06 | |
*** harlowja has joined #openstack-ironic | 22:06 | |
*** slagle has joined #openstack-ironic | 22:06 | |
*** jrist has joined #openstack-ironic | 22:06 | |
*** TheJulia has joined #openstack-ironic | 22:06 | |
*** trown has joined #openstack-ironic | 22:06 | |
*** yuanying has joined #openstack-ironic | 22:06 | |
*** lynxman has joined #openstack-ironic | 22:06 | |
*** NobodyCam has joined #openstack-ironic | 22:06 | |
*** kragniz has joined #openstack-ironic | 22:06 | |
*** davidlenwell has joined #openstack-ironic | 22:06 | |
*** tteggel has joined #openstack-ironic | 22:06 | |
*** pleia2 has joined #openstack-ironic | 22:06 | |
*** sirushti has joined #openstack-ironic | 22:06 | |
*** rodrigods has joined #openstack-ironic | 22:06 | |
*** lifeless has joined #openstack-ironic | 22:06 | |
*** annegentle_ has joined #openstack-ironic | 22:06 | |
*** SpamapS has joined #openstack-ironic | 22:06 | |
*** kbyrne has joined #openstack-ironic | 22:06 | |
*** PaulCzar has joined #openstack-ironic | 22:06 | |
*** coolsvap|afk has joined #openstack-ironic | 22:06 | |
*** lxsli has joined #openstack-ironic | 22:06 | |
*** vipul has joined #openstack-ironic | 22:06 | |
*** yarker has joined #openstack-ironic | 22:06 | |
*** greghaynes has joined #openstack-ironic | 22:06 | |
*** toabctl has joined #openstack-ironic | 22:06 | |
*** tonyb has joined #openstack-ironic | 22:06 | |
*** mrda has joined #openstack-ironic | 22:06 | |
*** zsmithnyc has quit IRC | 22:07 | |
*** med_ has quit IRC | 22:07 | |
*** med_ has joined #openstack-ironic | 22:08 | |
*** med_ has quit IRC | 22:08 | |
*** med_ has joined #openstack-ironic | 22:08 | |
*** Masahiro has quit IRC | 22:10 | |
*** zsmithnyc has joined #openstack-ironic | 22:10 | |
openstackgerrit | John Trowbridge proposed stackforge/ironic-discoverd: Changes utils.get_keystone(token) to utils.check_is_admin(token) https://review.openstack.org/145657 | 22:11 |
*** penick has joined #openstack-ironic | 22:14 | |
*** ryanpetrello has joined #openstack-ironic | 22:21 | |
*** iron_ has joined #openstack-ironic | 22:22 | |
*** Marga_ has quit IRC | 22:22 | |
*** iron_ has left #openstack-ironic | 22:22 | |
*** Marga_ has joined #openstack-ironic | 22:23 | |
*** ironman has joined #openstack-ironic | 22:23 | |
*** killer_prince has joined #openstack-ironic | 22:24 | |
*** killer_prince is now known as lazy_prince | 22:24 | |
*** david-lyle has quit IRC | 22:27 | |
NobodyCam | wht up with IRC today | 22:27 |
jroll | freenode was moving things around, there was a global notice | 22:28 |
jroll | "rehubbing the network" | 22:28 |
NobodyCam | ahh | 22:28 |
jroll | now "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 |
jroll | which is nickserv/chanserv/etc iiuc | 22:28 |
NobodyCam | yep.. guess I should look at the status window now and then | 22:28 |
jroll | lol | 22:29 |
NobodyCam | :-p | 22:29 |
NobodyCam | Ty jroll | 22:29 |
jroll | np | 22:29 |
NobodyCam | :) | 22:29 |
*** penick has quit IRC | 22:29 | |
*** david-lyle has joined #openstack-ironic | 22:30 | |
*** jcoufal has quit IRC | 22:30 | |
*** penick has joined #openstack-ironic | 22:31 | |
*** ironman has quit IRC | 22:32 | |
*** Marga_ has quit IRC | 22:35 | |
*** Marga_ has joined #openstack-ironic | 22:36 | |
*** igordcard has joined #openstack-ironic | 22:37 | |
*** Marga_ has quit IRC | 22:38 | |
*** Marga_ has joined #openstack-ironic | 22:39 | |
*** Marga_ has quit IRC | 22:43 | |
*** Marga_ has joined #openstack-ironic | 22:43 | |
openstackgerrit | Adam Gandelman proposed openstack/ironic: Allow configuration of neutronclient retries https://review.openstack.org/147002 | 22:45 |
*** ijw__ has joined #openstack-ironic | 22:45 | |
*** ijw___ has joined #openstack-ironic | 22:47 | |
*** sambetts has joined #openstack-ironic | 22:48 | |
*** ijw_ has joined #openstack-ironic | 22:49 | |
*** ijw____ has joined #openstack-ironic | 22:51 | |
*** achanda_ has quit IRC | 22:51 | |
*** ijw__ has quit IRC | 22:51 | |
*** ijw___ has quit IRC | 22:52 | |
*** ijw_ has quit IRC | 22:54 | |
NobodyCam | wow your quick rloo :) | 22:56 |
NobodyCam | hehehe | 22:56 |
rloo | NobodyCam: ? What'd I do? | 22:56 |
NobodyCam | -1 on adam_g's patch | 22:56 |
NobodyCam | :) | 22:56 |
NobodyCam | its a good thing | 22:57 |
NobodyCam | :) | 22:57 |
*** ryanpetrello has quit IRC | 22:57 | |
rloo | NobodyCam: oh. Yeah, in a meeting, not much I can review that doesn't require too much brain power. | 22:57 |
NobodyCam | hehehe | 22:57 |
openstackgerrit | Adam Gandelman proposed openstack/ironic: Allow configuration of neutronclient retries https://review.openstack.org/147002 | 22:57 |
adam_g | :) | 22:58 |
rloo | I think adam_g is faster! | 22:58 |
NobodyCam | :-p | 22:59 |
*** achanda has joined #openstack-ironic | 22:59 | |
*** Marga_ has quit IRC | 23:00 | |
*** Marga_ has joined #openstack-ironic | 23:01 | |
*** Marga_ has quit IRC | 23:01 | |
*** Marga_ has joined #openstack-ironic | 23:02 | |
Haomeng | devananda: hi | 23:16 |
Haomeng | devananda: hi, I want to test neutron vlan+ironic env, not sure if our ironic work with neutron vlan dhcp, any comments? | 23:16 |
jroll | Haomeng: didn't we just talk about this last night? | 23:18 |
jroll | what more do you want to know? | 23:18 |
Haomeng | jroll: yes, not sure if HP production is runing with vlan mode, and need other physical switch control | 23:19 |
openstackgerrit | Adam Gandelman proposed openstack/ironic: Provided backward compat for enforcing admin policy https://review.openstack.org/145984 | 23:19 |
jroll | huh, didn't know HP ran ironic in production | 23:19 |
Haomeng | jroll: yes, I think for production env, vlan is required, so the physical vlan control is required | 23:20 |
openstackgerrit | Adam Gandelman proposed openstack/ironic: Increase openstack.common logging (DO NOT MERGE) https://review.openstack.org/147015 | 23:20 |
Haomeng | jroll: I will try to setup vlan env to play with vlan to identfy the gap | 23:20 |
jroll | Haomeng: right, do you need something more than my spec? | 23:20 |
Haomeng | jroll: yes | 23:21 |
jroll | ok | 23:21 |
Haomeng | jroll: if we run pyhsical switch to configure the vlan by maunally | 23:21 |
jroll | not sure I understand | 23:21 |
Haomeng | jroll: not sure if it will work or not | 23:21 |
jroll | Haomeng: my spec is about controlling switches with automation | 23:21 |
Haomeng | jroll: 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 assignements | 23:22 |
Haomeng | jroll: yes, great | 23:22 |
*** igordcard has quit IRC | 23:22 | |
Haomeng | jroll: run snmp to contrlo switch, right? | 23:23 |
*** Marga_ has quit IRC | 23:23 | |
Haomeng | jroll: I understand the case is that - user create vlan from neutron side first | 23:23 |
jroll | Haomeng: depends on the switch | 23:24 |
*** Marga_ has joined #openstack-ironic | 23:24 | |
Haomeng | jroll: so if our spec will be called from neutron | 23:24 |
Haomeng | jroll: the 2nd step from user is that they will nova boot in the vlan created by step 1 | 23:24 |
Haomeng | jroll: I just guess this is the normal user case, right? | 23:24 |
jroll | Haomeng: sounds correct | 23:25 |
jroll | Haomeng: it should work the same way it works already in virtual cloud | 23:25 |
Haomeng | jroll: yes | 23:25 |
jroll | Haomeng: the spec is about what ironic needs to do to achieve this | 23:25 |
Haomeng | jroll: so not sure if your new spec code will run from conductor side | 23:25 |
jroll | Haomeng: yes, from the conductor during deploy and teardown | 23:26 |
Haomeng | jroll: that mean we will control switch when our ironic node creation? | 23:26 |
Haomeng | jroll: ok, that is cool | 23:26 |
jroll | yep | 23:26 |
Haomeng | jroll: how about if we can create different vlan for deploy time and tenant os runing time | 23:27 |
*** david-lyle has quit IRC | 23:27 | |
*** ryanpetrello has joined #openstack-ironic | 23:27 | |
Haomeng | jroll: to make sure the tenant os is security, can be isolated with other vlans | 23:27 |
jroll | Haomeng: 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 deploy | 23:28 |
*** penick has quit IRC | 23:28 | |
jroll | Haomeng: POC code is here, maybe that will help make sense https://review.openstack.org/#/c/139687/ | 23:28 |
jroll | Haomeng: "POC code" meaning "the code running at rackspace" | 23:29 |
Haomeng | jroll: 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 |
jroll | yes | 23:29 |
Haomeng | jroll: this is great | 23:29 |
*** Marga_ has quit IRC | 23:29 | |
Haomeng | jroll: any steps I need, I am interested with this featur, want to play with it:) | 23:29 |
jroll | Haomeng: thanks :) we worked hard on it | 23:29 |
*** Marga_ has joined #openstack-ironic | 23:30 | |
Haomeng | jroll: ok, want to help to test this patch | 23:30 |
jroll | Haomeng: so, apply that ironic patch (hopefully it works as is, it's slightly different than what we've been running) | 23:30 |
jroll | and also run neutron with ironic-neutron-plugin from github | 23:30 |
Haomeng | jroll: ok | 23:30 |
Haomeng | cool | 23:30 |
Haomeng | jroll: I will have try today and give your feedback:) | 23:30 |
jroll | that plugin only has code for certain cisco switches right now | 23:31 |
Haomeng | jroll: to make this better:) | 23:31 |
jroll | so you may need to write code to talk to your switches | 23:31 |
Haomeng | jroll: you mean for different snmp mibs? | 23:31 |
Haomeng | jroll: yes, make sense, different switch support differetn snmp mibs | 23:31 |
jroll | Haomeng: the cisco code we have doesn't use snmp, it uses an API that cisco provides (xml over ssh) | 23:31 |
Haomeng | jroll: let me see code, not sure if contrloed via snmp | 23:31 |
Haomeng | jroll: that is fine, but if we have protocol adaptor layer, that is better:) | 23:32 |
Haomeng | jroll: to support different switch driver | 23:32 |
jroll | Haomeng: the switch control is pluggable already :) | 23:32 |
Haomeng | jroll: let me check my switch management protol | 23:32 |
jroll | Haomeng: https://github.com/rackerlabs/ironic-neutron-plugin/tree/master/ironic_neutron_plugin/drivers | 23:32 |
Haomeng | jroll: yes, I konw this plugin, that is cool | 23:33 |
Haomeng | jroll: so I should apply ironic-neutron-plugin and then apply your patch, right? | 23:33 |
*** cppforlife has quit IRC | 23:33 | |
*** Ng has quit IRC | 23:33 | |
*** aweeks has quit IRC | 23:33 | |
JayF | that plugin is not going to work on your average switch | 23:34 |
JayF | it only works on the exact model we have | 23:34 |
jroll | Haomeng: yes, but you will need to write a driver for your switch in ironic-neutron-plugin | 23:34 |
JayF | and TBH perhaps only with firmware specifically for us | 23:34 |
Haomeng | jroll: ok, let me check my switch snmp mib | 23:34 |
jroll | JayF: I think those commands all work in mainline firmware at this point | 23:34 |
Haomeng | jroll: thank you | 23:34 |
Haomeng | does ironic-neutron-plugin support snmp? | 23:35 |
jroll | Haomeng: not today, but it could with a driver | 23:35 |
jroll | Haomeng: patches welcome :) | 23:35 |
jroll | simple driver, even | 23:35 |
Haomeng | jroll: i know a snmp contrl switch patch in neutron - https://review.openstack.org/#/c/90422/2/specs/juno/ml2-mechanism-snmp-vlan.rst | 23:35 |
jroll | Haomeng: great, and when that code is in neutron, we should add a network provider to ironic that works with it | 23:36 |
Haomeng | jroll: 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 |
jroll | Haomeng: indeed, thanks :) | 23:37 |
Haomeng | jroll: how do you think the switch management protocol we should supports? | 23:38 |
Haomeng | will be back for a while:) | 23:38 |
jroll | Haomeng: I would love to support all switches, but I think it will be whatever folks contribute | 23:38 |
*** victor_lowther has quit IRC | 23:40 | |
*** shakamunyi has joined #openstack-ironic | 23:46 | |
jroll | jenkins passed here if someone wants to +A https://review.openstack.org/#/c/146971/ | 23:50 |
jroll | Shrews: ^ | 23:50 |
NobodyCam | gilliard: still around? | 23:51 |
*** Masahiro has joined #openstack-ironic | 23:54 | |
*** smoriya has joined #openstack-ironic | 23:56 | |
*** ryanpetrello has quit IRC | 23:59 | |
*** Masahiro has quit IRC | 23:59 | |
openstackgerrit | Merged openstack/ironic: Document dependency on `fuser` for pxe driver https://review.openstack.org/146948 | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!