Monday, 2013-11-25

*** matsuhashi has joined #openstack-ironic01:03
lifelessdevananda: what are the terms for find-machines vs find-machine-innards?01:05
lifelessdevananda: (w.r.t. https://etherpad.openstack.org/p/icehouse-deployment-hardware-autodiscovery)01:05
*** matsuhashi has quit IRC01:11
*** matsuhashi has joined #openstack-ironic01:11
*** matsuhas_ has joined #openstack-ironic01:15
*** matsuhashi has quit IRC01:16
*** nosnos has joined #openstack-ironic01:18
*** rongze has joined #openstack-ironic01:19
*** rongze has quit IRC01:24
*** sjing has joined #openstack-ironic01:35
*** sjing has quit IRC01:52
*** sjing has joined #openstack-ironic01:53
*** rongze has joined #openstack-ironic01:56
*** rongze has quit IRC01:57
*** rongze has joined #openstack-ironic01:57
*** rongze_ has joined #openstack-ironic01:59
*** rongze has quit IRC02:02
openstackgerritHaomeng,Wang proposed a change to openstack/python-ironicclient: Enable created_at/updated_at for port-show/chassis-show  https://review.openstack.org/5782902:32
*** Haomeng has joined #openstack-ironic02:32
*** sjing has quit IRC03:01
*** sjing has joined #openstack-ironic03:03
*** rongze_ has quit IRC03:09
*** matsuhas_ has quit IRC03:17
*** matsuhashi has joined #openstack-ironic03:19
*** rongze has joined #openstack-ironic03:23
openstackgerritSun Jing proposed a change to openstack/ironic: Add tftp_server pattern in ironic.conf  https://review.openstack.org/5489103:27
*** jbjohnso has joined #openstack-ironic03:38
*** matsuhashi has quit IRC03:40
*** jbjohnso has quit IRC03:47
*** prekarat has joined #openstack-ironic04:21
openstackgerritHaomeng,Wang proposed a change to openstack/python-ironicclient: Enable created_at/updated_at for port-show/chassis-show  https://review.openstack.org/5782904:25
openstackgerritHaomeng,Wang proposed a change to openstack/ironic: Replace generate_uuid() with str(uuid.uuid4())  https://review.openstack.org/5785404:37
openstackgerritSun Jing proposed a change to openstack/ironic: Add tftp_server pattern in ironic.conf  https://review.openstack.org/5489104:43
*** rongze has quit IRC04:50
*** matsuhashi has joined #openstack-ironic04:51
*** rongze has joined #openstack-ironic04:53
*** sjing has quit IRC05:05
*** sjing has joined #openstack-ironic05:22
openstackgerritHaomeng,Wang proposed a change to openstack/ironic: Replace generate_uuid() with str(uuid.uuid4())  https://review.openstack.org/5785405:47
*** nosnos_ has joined #openstack-ironic05:51
*** nosnos has quit IRC05:54
*** matsuhashi has quit IRC05:55
*** nosnos has joined #openstack-ironic05:57
*** nosnos_ has quit IRC05:57
*** matsuhashi has joined #openstack-ironic05:59
openstackgerritJenkins proposed a change to openstack/ironic: Imported Translations from Transifex  https://review.openstack.org/5808306:00
openstackgerritHaomeng,Wang proposed a change to openstack/ironic: Replace generate_uuid() with str(uuid.uuid4())  https://review.openstack.org/5785406:00
*** matsuhashi has quit IRC06:04
*** sjing has quit IRC06:07
*** matsuhashi has joined #openstack-ironic06:19
*** matsuhashi has quit IRC06:47
*** matsuhashi has joined #openstack-ironic06:58
*** matsuhashi has quit IRC06:59
*** arata has joined #openstack-ironic07:01
*** matsuhashi has joined #openstack-ironic07:02
*** nosnos has quit IRC07:05
*** nosnos_ has joined #openstack-ironic07:05
*** urulama has joined #openstack-ironic07:12
*** matsuhashi has quit IRC07:23
*** matsuhashi has joined #openstack-ironic07:23
*** matsuhashi has quit IRC07:28
*** matsuhashi has joined #openstack-ironic07:29
*** matsuhashi has quit IRC07:29
*** matsuhashi has joined #openstack-ironic07:29
*** sjing has joined #openstack-ironic07:34
*** michchap has quit IRC07:53
*** michchap has joined #openstack-ironic07:53
*** matsuhashi has quit IRC08:09
*** mattymo has quit IRC08:09
*** matsuhashi has joined #openstack-ironic08:10
*** nosnos_ has quit IRC08:14
*** sjing has quit IRC08:15
*** nosnos has joined #openstack-ironic08:15
*** sjing has joined #openstack-ironic08:16
*** romcheg has joined #openstack-ironic08:39
*** ndipanov has joined #openstack-ironic08:40
*** sjing has quit IRC08:43
*** derekh has joined #openstack-ironic08:58
*** lexx has joined #openstack-ironic09:04
*** prekarat has quit IRC09:06
*** prekarat has joined #openstack-ironic09:08
*** jistr has joined #openstack-ironic09:09
*** matsuhashi has quit IRC09:12
*** matsuhashi has joined #openstack-ironic09:14
*** martyntaylor has joined #openstack-ironic09:39
*** max_lobur has joined #openstack-ironic09:46
openstackgerritlifeless proposed a change to openstack/ironic: Ignore H803, it's not needed.  https://review.openstack.org/5823609:47
openstackgerritlifeless proposed a change to openstack/ironic: PEP8-ify imports in test_ipmitool.  https://review.openstack.org/5755009:47
*** martyntaylor has quit IRC09:49
lexxHi guys! I was working on IPMI inspector for monitoring physical devices on Ceilometer09:52
lexxIs somebody starting working on that?09:53
*** martyntaylor has joined #openstack-ironic09:55
lexxto expose IPMI sensor data via Ironic09:56
*** rongze has quit IRC10:13
*** arata has left #openstack-ironic10:19
Haomenglexx: morning lexx10:19
Haomenglexx: are you from Ceilometer project?10:20
*** tatyana has joined #openstack-ironic10:20
Haomenglexx: I understand, Ironic will call Ceilometer sample api to post the IPMI data, right?10:21
*** prekarat has quit IRC10:27
Haomenglexx: and Ceilometer will translate the Ironic message to Ceilometer counter object10:27
Haomenglexx: and translate to meter to save to Ceilometer database10:27
*** martyntaylor has quit IRC10:28
openstackgerritA change was merged to openstack/ironic: Imported Translations from Transifex  https://review.openstack.org/5808310:34
*** martyntaylor has joined #openstack-ironic10:35
*** romcheg has left #openstack-ironic10:40
lexxHaomeng, Yes, I'm from Ceilometer project ...  So how I understand Ceilometer agent and Ironic counter will run simultaneously. Ironic will expose data and Ceilometer will post it... In Ceilometer project they already start to work on that, my question is: "What I can do for Ironic to IPMI inspector?"10:45
*** romcheg has joined #openstack-ironic10:45
Haomenglexx: yes, we need to discuss and define the interface between our Ceiloemter and Ironic10:46
Haomenglexx: we have blueprint in our Ironic, now it is owned by devananda10:46
lexxHaomeng, this one https://blueprints.launchpad.net/ironic/+spec/expose-ipmi-sensor-data ?10:47
Haomenglexx: I think we have two way, the one is Ironic send the IPMI notification message to Ceilometer, Ceilometer get and parse, and map to sample object and  save it to database10:48
Haomenglexx: yes10:48
Haomenglexx: another one way is Ironic call Ceilometer Sample Post API to post a IPMI hardware sensor data10:49
Haomenglexx: let me find out our Ironic blueprint link to you10:49
Haomenglexx: I think should be this one - https://blueprints.launchpad.net/ironic/+spec/send-data-to-ceilometer10:50
lexxHaomeng, I think API the easiest way10:50
Haomenglexx: but not sure how to maping the data model between Ironic and Ceilometer10:50
Haomenglexx: As we know, Ceilometer has data model, such as resource->meter->sample10:51
lexxHaomeng, yes, it is10:51
Haomenglexx: but from Ironic, we just have IPMI sensor data, have to fill more releationship data to match Ceilometer data model I think10:52
Haomenglexx: so we have to interface to be defined10:52
Haomenglexx: the first one is the intergation interface, such as API call, or MQ message interface10:52
Haomenglexx: 2nd one is the data model defination in our interface10:53
Haomenglexx: from your Ceilometer, how do you think these?10:53
Haomenglexx: for MQ message interface, I think Ceilometer only accept the SIGNED message, right?10:53
Haomenglexx: not sure if the message sent from Ironic can be accepted by Ceilometer, as ceilometer central/computer agent publisher role10:54
lexxHaomeng, I'm not sure also... I'm working only at this IPMI inspector... I think we need to discuss it with Lianhao - he is working on this hardware agent10:57
*** matsuhashi has quit IRC10:58
Haomenglexx: no problem, we can send mail to the maillist to discuss the interface10:58
Haomenglexx: I know Lianhao, he is from Intel, and he is Chinese:)10:59
Haomenglexx: I am Chinese also:) lexx, where are you from?10:59
Haomenglexx: one more question, do you want to create a new inspector for Ironic?11:00
Haomenglexx: I know there is some inspector in Ceilometer, like cpu inspector, or other inspectors run from Ceilometer Compute Agent11:01
lexxHaomeng, I'm from Ukraine, but in this moment I'm intership in ZHAW Switzerland, and this is my student project11:01
Haomenglexx: great, welcome:)11:01
Haomenglexx: you mean, I can discuss with Lianhao, he will focus on IPMI data collection for Ceilometer, right?11:02
lexxHaomeng, it is independ how we will to transfer data to Ceilometer... But anyway, I think we need to create Ironic inspector11:04
Haomenglexx: we have another mail thread, not sure if you read, let me find11:05
lexxHaomeng, yes, it will be greate =) in this moment how I  know he is working on merge the hardware agent with central agent and using Ironic for ipmi11:05
Haomenglexx: http://lists.openstack.org/pipermail/openstack-dev/2013-November/thread.html - this is nov mail list, we can find these11:06
*** nosnos has quit IRC11:07
Haomenglexx: search "A question about getting IPMI data"11:07
Haomenglexx: http://lists.openstack.org/pipermail/openstack-dev/2013-November/019804.html11:07
Haomenglexx: http://lists.openstack.org/pipermail/openstack-dev/2013-November/019926.html11:07
Haomenglexx: http://lists.openstack.org/pipermail/openstack-dev/2013-November/020272.html11:07
Haomenglexx: just for your reference11:09
Haomenglexx: you can subscribe the openstack-dev maillist to see these mail discusstions11:09
lexxHaomeng, thx =)11:10
Haomenglexx: welcome11:11
lexxHaomeng, but there are discussing only about pyghmi ... we should discuss about interface and functionality with Lianhao. Can you send a mail to mailling list to discuss it?11:13
lexxHaomeng, please11:13
Haomenglexx: sure11:13
*** rongze has joined #openstack-ironic11:13
Haomenglexx: I just prepare the proposed solution these days, try to send out to discuss11:14
Haomenglexx: will do:)11:14
lexxHaomeng, I will be very thankful =)11:14
Haomenglexx: :)11:14
Haomenglexx: I try to understand IPMI and Ceilometer in deep these days, for a better solution11:15
Haomenglexx: I just think the points are - interface and datamodel11:15
lexxHaomeng, okey, I will try11:16
Haomenglexx: :)11:18
*** rongze has quit IRC11:18
Haomenglexx: do you know which is IPMI bp in Ceilometer?11:19
Haomenglexx: let me try to find11:19
lexxhttps://blueprints.launchpad.net/ceilometer/+spec/ipmi-inspector-for-monitoring-physical-devices11:19
Haomenglexx: got, thank you11:20
Haomenglexx: this bp owner is not LianHao,Lu, current owner is Oleksii Serhiienko11:21
Haomenglexx: and our Ironic team member lifeless and devananda has comments in this bp11:23
lexxHaomeng, This is I Oleksii Serhiienko11:23
Haomenglexx: so not sure if LianHao will focus on this work11:24
Haomenglexx: yes Oleksii Serhiienko proposed some code to impl this from Ceilometer, that is good reference for our Ironic I think11:24
lexxHaomeng, he is working here https://review.openstack.org/#/c/43074/ on agent11:25
Haomenglexx: and this great patch - https://review.openstack.org/#/c/51828/111:25
Haomenglexx: I think https://review.openstack.org/#/c/43074/ is pending on the dependency with Ironic IPMI data11:26
lexxHaomeng, I'm Oleksii Serhiienko =)11:26
Haomenglexx: sorry:)11:27
Haomenglexx: can not find your name from IRC:)11:27
Haomenglexx: thanks for your great job:)11:27
lexxHaomeng, no problem =)11:27
Haomenglexx: I think your code is working but we need to modify the interface and integrate with our Ironic for IPMI11:28
lexxHaomeng, yes, I agree with you11:28
lexxHaomeng, but we also need to modify Ceilometer agent https://blueprints.launchpad.net/ceilometer/+spec/monitoring-physical-devices11:29
lexxHaomeng, and Lianhao was working on that11:30
Haomenglexx: looks like it is using SNMP to get data11:30
Haomenglexx: so confusing here, what the releationship between this Ceilometer Agent bp and our IPMI protocol based hardware information collection?11:31
lexxHaomeng, yes, at first it was monitoring physical devices,  and Toni Zehnder was created this bp... after that, they finish to work on this project and Lainhao modify it... I come to ZHAW after Toni Zehnder and my task was create a IPMI inspector for monitoring physical devices11:33
lexxHaomeng, to create IPMI inspector on the same way as SNMP inspector11:34
Haomenglexx: yes, same framework, just different protocol11:35
*** lexx_ has joined #openstack-ironic11:35
lexx_Haomeng, yes11:35
Haomenglexx: one more question, did you test your code with real hadware server which support IPMI? I want to enable IPMI from my Linux OS to do some testing freely11:37
lexx_Haomeng, yes11:37
Haomenglexx_: great, sounds I have to find some hardware server:)11:38
lexx_Haomeng, one problem is about login and password... you should put it on pipeline.yaml11:39
lexx_Haomeng, I didn't find better solution11:39
Haomenglexx_: yes, you mean the IPMI credential have to be configured into Ceilometer pipeline.yaml, right?11:40
*** romcheg1 has joined #openstack-ironic11:40
*** romcheg has quit IRC11:40
Haomenglexx_: I think that should be fine, but not sure if we support the different credential for different Server, maybe a list is required, from our Ironic, we have such information11:40
lexx_Haomeng, yes11:40
Haomenglexx_: stored in our node object11:41
Haomenglexx_: for each node11:41
Haomenglexx_: so that is the point why we should do this IPMI from Ironic, because Ironic maintain IPMI credential I think11:42
lexx_Haomeng, It was the main point11:42
Haomenglexx_: let me study your code and I think there should be some good reference can be reused, so thanks for your hardwork11:45
Haomenglexx_: I see you created such file to handle volt - https://review.openstack.org/#/c/51828/1/ceilometer/hardware/pollsters/volt.py11:46
*** arata has joined #openstack-ironic11:47
Haomenglexx_: not sure where such special requirement we get, because as we know we have a lot of hardware sensor data can be retrieved via IPMI, such as the cpu temp, fan rpm, etc11:47
Haomenglexx_: found this rpm Pollster - ceilometer/hardware/pollsters/rpm.py11:50
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Add troubleshoot option to PXE template  https://review.openstack.org/5826211:51
Haomenglexx_: these are HardwarePollster extensions, cool11:51
lexx_Haomeng, actually it is not a lot af data retrive vai IPMI only temperature, voltage, and fan speed11:52
lexx_Haomeng, and syslogs11:52
Haomenglexx_: eys11:53
Haomenglexx_: yes11:53
lexx_Haomeng, I was collect all data that I found11:53
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Add troubleshoot option to PXE template  https://review.openstack.org/5826211:54
Haomenglexx_: so we have to define the scope, which data we support , such as you have implemented temperature, voltage, and fan speed11:54
Haomenglexx_: that is great11:54
Haomenglexx_: I think this is enough for common use11:55
Haomenglexx_: we can extend if required11:55
Haomenglexx_: so do you think we have to define these temperature, voltage, and fan speed as Ceilometer Meters?11:55
Haomenglexx_: I think this is data model11:55
lexx_ Haomeng, maybe you right=) when I was code it, Lianhao told me to collect data in the same way as it is on SNMP inspector11:58
lexx_Haomeng, and there all data as Ceilometer meters11:59
Haomenglexx_: yes11:59
Haomenglexx_: so for these new hardware meters, we have to defined as well, you have implemented it, so your code is very good reference12:00
Haomenglexx_: and can be reused in our new solution with new interface12:00
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Log IPMI power on/off timeouts  https://review.openstack.org/5826612:01
lexx_Haomeng, so you recomend create new meters not like saparete pollsters, but like data model12:02
lexx_?12:03
Haomenglexx_: no, have to discuss with LianHao and you Ceilometer dev team:)12:03
lexx_Haomeng, okey =)12:03
*** lucasagomes has joined #openstack-ironic12:04
Haomenglexx_: I just want to find a more common way to support more IPMI hardware sensor data12:04
Haomenglexx_: your implemention is cool, we have several poster for different IPMI data, such as rpm pollster and volt pollster12:06
lexx_Haomeng, I think it be better if Ironic will send Ceilometer just data model, and after this Ceilometer will parse it, put in samples, and publish12:07
Haomenglexx_: yes12:07
Haomenglexx_: common data model is great12:07
Haomenglexx_: but need to be defined well, and discuss with Ceilometer core team and Ironic core team12:08
Haomenglexx_: IPMI can support a lot of data, and we can design to a common framework to collect these, and configure these meters freely from Ceilometer side and Ironic side12:09
Haomenglexx_: but not sure if this can be implemented from both Ceilometer and Ironic side12:09
Haomenglexx_: I guess your local time is around 9am, correct me if I am wrong:) my time is 8pm now:)12:11
lexx_13 : 0012:11
Haomenglexx_: :)12:12
Haomenglexx_: so my zone is +8, and your is +1 right:)12:13
*** arata has left #openstack-ironic12:14
lexx_Haomengm, yes12:14
*** rongze has joined #openstack-ironic12:14
*** michchap has quit IRC12:18
*** rongze has quit IRC12:18
*** michchap has joined #openstack-ironic12:20
*** rongze has joined #openstack-ironic12:24
*** lucasagomes is now known as lucas-hungry12:29
*** vkozhukalov has joined #openstack-ironic12:39
Haomenglexx_: FYI - this is another mail thread to disscuss our IPMI with Ceilometer and Ironic team members - http://lists.openstack.org/pipermail/openstack-dev/2013-November/019517.html12:46
*** lexx_ has quit IRC12:56
*** lexx has quit IRC12:56
*** martyntaylor has quit IRC12:59
*** martyntaylor has joined #openstack-ironic13:08
*** rongze has quit IRC13:25
romcheg1Morning everyone13:32
*** romcheg1 is now known as romcheg13:33
*** jdob has joined #openstack-ironic13:33
*** lucas-hungry is now known as lucasagomes13:34
Haomengromcheg1: morning13:35
*** urulama has quit IRC13:38
*** dmojoryder has left #openstack-ironic13:52
*** arata has joined #openstack-ironic13:54
*** arata has left #openstack-ironic13:54
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Log IPMI power on/off timeouts  https://review.openstack.org/5826613:55
*** rongze has joined #openstack-ironic14:04
*** linggao has joined #openstack-ironic14:07
*** jdob has quit IRC14:16
*** jdob has joined #openstack-ironic14:16
*** jbjohnso has joined #openstack-ironic14:21
*** rongze has quit IRC14:26
*** jdob_ has joined #openstack-ironic14:30
*** jdob has quit IRC14:31
*** rongze_ has joined #openstack-ironic14:31
GheRiveromorning all14:36
HaomengGheRivero: morning14:41
*** lexx has joined #openstack-ironic14:41
*** rustlebee is now known as russellb14:49
*** lexx has quit IRC14:52
*** zigo_ has joined #openstack-ironic14:52
NobodyCamgood morning Ironic14:52
EmilienMGheRivero: hello ! i was wondering if you planned to add a release number/name. We want to make packaging of Ironic in Debian14:53
*** martyntaylor1 has joined #openstack-ironic14:54
*** martyntaylor has quit IRC14:55
zigo_GheRivero: Hi there! :)14:59
GheRiverowow... hello all, French Army :)15:00
romchegMorning NobodyCam15:01
NobodyCamMorning romcheg15:01
GheRiverozigo_ EmilienM: ironic is going to be released in Icehouse, tso the first release/number name should be...15:02
zigo_GheRivero: The problem is that I'd need tags to do the packaging, though of course, I can use 0_0+20131125+git+e94e918589 ...15:03
zigo_(latest commit right now)15:03
EmilienMGheRivero: i'm going to prepare a puppet module also, are you aware about existing work ? or need i start from scratch ?15:04
GheRiverozigo_: i guess first one will be December 5th, with the Icehouse-115:05
zigo_Ok.15:05
zigo_GheRivero: Will you review my Debian package when it's done? :)15:05
GheRiveroEmilienM: no puppet module, but there is a TripleO element you can take a look15:05
GheRiverozigo_: yes, sure :)15:05
EmilienMGheRivero: ack, thx!15:06
*** vkozhukalov has quit IRC15:13
*** romcheg1 has joined #openstack-ironic15:14
*** romcheg has quit IRC15:14
zigo_GheRivero: Can you confirm that I will only need to produce: ironic-api & ironic-conductor?15:19
zigo_GheRivero: Or is there more debian binary packages that should be there? (of course, I'm not even talking about python-ironic and ironic-common ...)15:20
GheRiverozigo_: just api and conductor15:22
zigo_Ok, cool.15:22
GheRiverobesides the client, and the new ipmi python module15:22
GheRiverowhich makes me wonder... all: any comments about general openstack-client vs ironic-client?15:23
NobodyCamGheRivero: I'm thinking we need to get the ironic client (and ironic in general) working before looking at the openstacl-client15:34
lucasagomesmorning all :)15:38
max_loburmorning Ironic!15:39
NobodyCamgood morning lucasagomes and max_lobur15:41
openstackgerritLucas Alvares Gomes proposed a change to openstack/python-ironicclient: Replace chassis_id with chassis_uuid on Nodes  https://review.openstack.org/5831415:47
*** romcheg1 has quit IRC15:49
*** jdob_ has quit IRC15:53
*** romcheg has joined #openstack-ironic15:54
openstackgerritLucas Alvares Gomes proposed a change to openstack/python-ironicclient: Fix several i18n  https://review.openstack.org/5831816:03
openstackgerritLucas Alvares Gomes proposed a change to openstack/python-ironicclient: Fix several i18n  https://review.openstack.org/5831816:04
NobodyCamlucasagomes: what do you think of something like this for the fake driver? http://paste.openstack.org/show/p41bqbeFjtG1CH9JsQdV/16:17
* lucasagomes clicks16:17
NobodyCamreally on ly FakePower16:20
lucasagomesNobodyCam, so instead of mocking the get_power_state it would return the power_state that is in the node object?16:20
NobodyCamyep16:20
NobodyCamthat way I can do a deploy with fake power16:21
lucasagomesNobodyCam, I see, but wouldn't be easier to just "return task.node.get('power_state')" then?16:22
NobodyCamalso set_power_state should work16:22
lucasagomesNobodyCam, return task.node.get('power_state', state.NOSTATE) better16:23
NobodyCami modeled it on https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ipminative.py16:23
NobodyCambut ya16:24
lucasagomesI don't have anything against it :) will make tests easier as well since we don't have to mock the get_power_state to return what we want16:24
NobodyCam:)16:25
*** romcheg has quit IRC16:25
NobodyCamlucasagomes: revove the _parse_info or keep just to match other driver layouts?16:27
NobodyCamremove even16:27
* NobodyCam gits more coffee16:27
lucasagomesthe _parse_info on the fake driver looks kinda useless16:30
NobodyCam:)16:30
NobodyCamwill clean up16:31
lucasagomescool16:31
openstackgerritChris Krelle proposed a change to openstack/ironic: Allow FakePower to return node objects power_state  https://review.openstack.org/5833316:38
*** romcheg has joined #openstack-ironic16:38
*** max_lobur has quit IRC16:40
*** max_lobur has joined #openstack-ironic16:41
openstackgerritChris Krelle proposed a change to openstack/ironic: Allow FakePower to return node objects power_state  https://review.openstack.org/5833316:45
*** blamar has joined #openstack-ironic16:49
*** martyntaylor1 has quit IRC16:54
NobodyCamlucasagomes: https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L141 vs. https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ipminative.py#L21017:05
*** jimjiang has quit IRC17:05
NobodyCamnode vs node_id ???17:06
NobodyCambbt brb17:06
*** jimjiang has joined #openstack-ironic17:08
lucasagomesNobodyCam, a-ha looking at the other drivers https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ipmitool.py#L19317:08
NobodyCamback17:08
lucasagomesseems it expecting a node object there17:08
lucasagomesso manager.py should pass the node_obj instead17:09
NobodyCamthats what I thought17:10
NobodyCamI will patch with my power patch17:10
lucasagomesyea these parts are kinda untouched so I bet u going to find a couple of mistakes like that :D17:12
NobodyCamhehehhehe17:12
*** vkozhukalov has joined #openstack-ironic17:14
openstackgerritChris Krelle proposed a change to openstack/ironic: Allow FakePower to return node objects power_state  https://review.openstack.org/5833317:19
NobodyCampost bbt walkies17:20
devanandag'morning, all17:21
*** lexx has joined #openstack-ironic17:29
NobodyCamgood morning devananda17:35
openstackgerritChris Krelle proposed a change to openstack/ironic: Allow FakePower to return node objects power_state  https://review.openstack.org/5833317:38
NobodyCamGheRivero: are you around.. or better question will you be around for the meeting this morning]17:39
GheRiveroNobodyCam: yes and yes17:39
*** jistr has quit IRC17:41
NobodyCam:)17:43
devanandaromcheg: have you seen https://review.openstack.org/#/c/56439/1 ?17:48
romchegdevananda: hmmmm17:49
romcheg0_o17:50
romchegThat won't work17:51
romchegIronic does not even get installed in those tests...17:52
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Use oslo's execute() and ssh_execute() methods  https://review.openstack.org/5834617:52
openstackgerritChris Krelle proposed a change to openstack/ironic: Allow FakePower to return node objects power_state  https://review.openstack.org/5833317:54
*** derekh has quit IRC17:54
devanandaromcheg: i think that patch was proposed before some of your work17:57
romchegNov 14 on that patch vs Oct 25 on my :)17:58
romchegdevananda: What I mean is that mordred's patch won't do any tests for Ironic because devstack-gate does not install Ironic to the nodes for those jobs17:59
mordreduhoh. what did I do?17:59
romchegmordred: Hi, we're just talking about your patch https://review.openstack.org/#/c/56439/1 18:01
*** lucasagomes is now known as lucas-afk18:01
romchegWhich apparently does nothing in the end :)18:01
mordredromcheg: I'm sure everything I did was wrong18:01
mordred:)18:01
NobodyCamdevananda: if I push to the nova driver can you click WIP?18:02
devanandaya18:03
romchegmordred: your patch adds some jobs to post and prerelease, which I think is useful to have18:05
NobodyCamdevananda: please click WIP18:06
romchegSo maybe you can rebase it ower this one https://review.openstack.org/#/c/53917/18:06
romcheg?18:06
devanandarussellb: you beat me to the WIP button :P18:06
romchegmordred: That patch introduces new jobs that configure devstack-gate to install Ironic18:07
NobodyCamlol18:07
NobodyCamhow can he click and I can not lol18:07
NobodyCamrussellb: THANK YOU18:07
devanandaNobodyCam: he's the PTL?18:07
NobodyCam:)18:07
devanandamagical powers18:07
NobodyCamdevananda: you see https://review.openstack.org/#/c/58333/ :-p18:08
russellb:)18:08
russellbi clicked it before i even saw this conversation18:08
NobodyCamlol :)18:08
*** lexx has quit IRC18:11
devanandaNobodyCam: within the drivers, are we standardinzing on using task.node ?18:14
*** lexx has joined #openstack-ironic18:15
NobodyCamno . I did that becuase I fixed conductor.manager after the fact.. I should update to use node18:16
NobodyCam:-p18:17
*** hemnafk is now known as hemna18:21
lifelessGheRivero: still around?18:34
lifelessGheRivero: ah, you saw - nevermind!18:35
GheRiverolifeless: :)18:37
*** max_lobur is now known as max_lobur_afk18:39
*** rongze_ has quit IRC18:40
NobodyCamlast chance to get fresh coffee18:55
*** lucas-afk is now known as lucasagomes18:55
vkozhukalovHey, guys. I am wondering if ironic is supposed to be something like generic provisioning service. I mean we can try to integrate it with custom deployment tool to use it as a provisioner. At the moment we use Cobbler.18:56
vkozhukalovHey, guys. I am wondering if ironic is supposed to be something like generic provisioning service. I mean we want to integrate it with custom deployment tool to use it as a provisioner. At the moment we use Cobbler.18:57
vkozhukalovAnd what we want to do is to contribute in Ironic but we want it to be able to work without glance neutron and keystone.18:58
vkozhukalovis it possible according to it's architecture?18:59
devanandameeting time19:01
*** mugu has joined #openstack-ironic19:09
*** rongze has joined #openstack-ironic19:11
*** tatyana has quit IRC19:13
*** rongze has quit IRC19:17
*** hemna has quit IRC19:23
*** epim has joined #openstack-ironic19:25
*** bigjools has quit IRC19:28
*** bigjools has joined #openstack-ironic19:28
*** rongze has joined #openstack-ironic19:44
*** hemna has joined #openstack-ironic19:46
*** rongze has quit IRC19:49
EmilienMhi people, just to let you know i've started a puppet module for ironic which is *really* a WIP : https://github.com/enovance/puppet-ironic - once the module will work & 100% unit test coverage done, i'll move it into stackforge20:07
devanandaEmilienM: awesome! thanks!20:10
devanandavkozhukalov: at this time, it's not possible for ironic to work without keystone. the deploy driver we're writing will need neutron and glance, but in principle, ironic itself doesn't require either of those20:11
devanandavkozhukalov: keystone is currently the only tight coupling for ironic's API service, and the conductor service requires a database (eg MysQL) and a message broker (eg Rabbit)20:12
NobodyCamgreat meeting :)20:12
lucasagomes:) yup20:12
romcheg+120:12
NobodyCamromcheg: how long do think to switch to alembic20:13
vkozhukalovdevananda: thanks20:13
lucasagomesright, it's late here! night everyone :)20:13
devanandag'night lucasagomes !20:13
NobodyCamnight lucasagomes TY for staying up20:14
romchegNobodyCam: Rewriting the migrations is easy and can be done in several days.20:14
*** lucasagomes has quit IRC20:14
romchegNobodyCam: the hard thing is changing the way we create a new DB20:14
romchegi.e. alembic does not perform upgrades for sqlalchemy so we need generate the DB using the data model for it20:15
romchegI have an intern here (or maybe even more than one:) ) whom I need to give a task to20:16
romchegSo I can ask him to do some work just as a proof of concept20:16
NobodyCamromcheg: that could work20:17
NobodyCamdevananda: ^^^20:17
devanandalet me see what other projects are doing first20:18
NobodyCam:) ok :)20:19
romchegI spoke to Roman Podoliaka who is our DB guy (and one of the guys who worked on oslo.db) and he said that it should be easier to switch to alembic than fix sqlalchemy-migrate20:22
NobodyCamlol20:23
devanandaromcheg: talk to boris-42?20:30
romchegI talk to him every day :)20:31
romchegWill ask him about his opinion tomorrow20:31
devanandak20:31
hemnaworking on setting up my own PXE/tftpd setup so I can start playing finally20:43
*** rongze has joined #openstack-ironic20:46
devanandaromcheg: http://www.stillhq.com/openstack/icehouse/000001.html20:46
devanandaromcheg: an interesting description of how neutron is using alembic and some issues mikal ran into when trying to create a new migration20:47
romchegdevananda: thanks, will take a look20:48
*** rongze has quit IRC20:50
*** ndipanov is now known as ndipanov_gone20:57
devanandaromcheg: fwiw, it looks like only neutron and ceilometer are using alembic, everyone else -- even trove -- is using sqlamigrate, and Nova is not in any rush to move off of it21:00
* devananda goes afk to make lunch21:00
NobodyCamlol /me is fixing his tox on the mac :-p21:01
*** tatyana has joined #openstack-ironic21:02
*** kui has joined #openstack-ironic21:02
devanandaactualy21:08
* devananda goes out to get lunch21:08
devanandabbiah21:08
*** vkozhukalov has quit IRC21:09
NobodyCamyou got it21:10
*** linggao has quit IRC21:34
* NobodyCam brb21:36
*** rongze has joined #openstack-ironic21:46
*** rongze has quit IRC21:54
lifelessI find it amusing that This makes use of the mandatory option of WSME, to remove some of the21:56
lifelesscustom validation code. The patch also includes a new attribute on21:56
lifelessPorts API object called nodes_uuid to store the UUID of the node that21:56
lifelessbah21:56
lifelessamusing that  https://review.openstack.org/#/c/56682/ which is reducing code actually increases it :P21:57
*** tatyana has quit IRC21:58
*** jbjohnso has quit IRC22:08
*** tatyana has joined #openstack-ironic22:08
openstackgerritChris Krelle proposed a change to openstack/ironic: Allow FakePower to return node objects power_state  https://review.openstack.org/5833322:24
*** tatyana has quit IRC22:31
*** romcheg has quit IRC22:33
devanandaback22:37
NobodyCamwb22:39
*** rongze has joined #openstack-ironic22:50
*** rongze has quit IRC22:55
*** kui has quit IRC22:55
openstackgerritChris Krelle proposed a change to openstack/ironic: Allow FakePower to return node objects power_state  https://review.openstack.org/5833323:00
NobodyCamgah pep823:00
NobodyCamreboot s modem brb23:24
NobodyCamthat seems a bit better23:28
NobodyCamnot really23:28
*** epim has quit IRC23:28
NobodyCamreboots router brb23:28
NobodyCamnope still tons of lag23:35
*** crandquist has joined #openstack-ironic23:42
*** rongze has joined #openstack-ironic23:51
*** rongze has quit IRC23:55

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