| *** hemna is now known as hemnafk | 00:09 | |
| *** sandeepr has quit IRC | 00:12 | |
| *** sandeepr has joined #openstack-ironic | 00:12 | |
| *** ndipanov_gone has quit IRC | 00:31 | |
| *** matsuhashi has joined #openstack-ironic | 00:32 | |
| *** lexx has quit IRC | 00:32 | |
| *** nosnos has joined #openstack-ironic | 01:09 | |
| *** rongze has joined #openstack-ironic | 01:21 | |
| *** rongze has quit IRC | 01:26 | |
| *** crandquist has quit IRC | 01:27 | |
| *** crandquist has joined #openstack-ironic | 01:27 | |
| *** sjing has joined #openstack-ironic | 01:28 | |
| *** rongze has joined #openstack-ironic | 01:39 | |
| *** rongze has quit IRC | 01:46 | |
| *** crandquist has quit IRC | 01:51 | |
| Haomeng | good morning Ironic:) | 01:56 |
|---|---|---|
| *** rongze has joined #openstack-ironic | 02:01 | |
| *** nosnos_ has joined #openstack-ironic | 02:02 | |
| *** nosnos has quit IRC | 02:05 | |
| devananda | good evening, Haomeng :) | 02:12 |
| Haomeng | devananda: :) | 02:13 |
| Haomeng | devananda: for the IPMI ceilometer integration, I think we will discuss with Ceilometer team to work out the final interface and data model, then we can start to code, how do you think? | 02:14 |
| devananda | Haomeng: ++ | 02:15 |
| Haomeng | devananda: if we can not send the valid/signed-message for Ceilometer Collector, maybe have to all Ceilometer API sample post method to inform Ceilometer we have IPMI data should be handled | 02:15 |
| devananda | Haomeng: if you want to start drafting some of the ironic.drivers changes now, i think that would be OK. | 02:15 |
| *** arata has joined #openstack-ironic | 02:16 | |
| Haomeng | devananda: thank you, but I think we have to have a CLEAR interface with Ceilometer to avoid more confusing and more issue during coding | 02:16 |
| Haomeng | devananda: I think we have time to implement these bp | 02:16 |
| devananda | Haomeng: this is not required for Icehouse | 02:17 |
| Haomeng | devananda: there is an Intel dev LianHao from Ceilometer team, I will try to discuss with him | 02:17 |
| Haomeng | devananda: ok, so we have time to do this bp:) | 02:17 |
| devananda | Haomeng: yep :) | 02:17 |
| Haomeng | devananda: one more question | 02:18 |
| Haomeng | devananda: as we know, IPMI data contains the 'status' field, how do you think if we can leverage this field do send some alarm to ceilometer? | 02:19 |
| Haomeng | devananda: let me show you the instance data | 02:19 |
| devananda | Haomeng: can you give an example? | 02:19 |
| Haomeng | devananda: sure | 02:19 |
| Haomeng | devananda: for your reference - http://paste.openstack.org/show/53953/ | 02:21 |
| devananda | Haomeng: would you mind creating an etherpad with your various notes/plans on this, and adding the ceilomter<->ironic interface specification, etc, to it? | 02:22 |
| Haomeng | devananda: we have three fields returned from IPMI, the first one is the data name, then the value, the last field is the status | 02:22 |
| Haomeng | devananda: sure, thanks for your hint:) I use etherpad first:) | 02:22 |
| Haomeng | devananda: can I modify it now? | 02:22 |
| devananda | Haomeng: this should then be linked from the BP | 02:22 |
| *** epim has joined #openstack-ironic | 02:22 | |
| devananda | Haomeng: what is your launchpad id? | 02:22 |
| Haomeng | devananda: whaom | 02:23 |
| devananda | Haomeng: i will assign the BP to you, then you can update description // set URL | 02:23 |
| Haomeng | devananda: launchpad.net/~whaom | 02:23 |
| Haomeng | devananda: thank you | 02:23 |
| devananda | Haomeng: thank you :) | 02:23 |
| Haomeng | devananda: just want to discuss with you, if we should support 'status' field, for some hardware failer case, maybe that is hardware event/alram to be sent to ceilometer as Ceilometer Alarm, I think this is another feature | 02:24 |
| Haomeng | devananda: any time | 02:24 |
| devananda | Haomeng: hm. so yes, i see "chassis intru" and "PS Status" | 02:25 |
| Haomeng | devananda: I think these two data are not supported as well for all hardware server | 02:25 |
| devananda | Haomeng: you will find a VERY large amount of variation in different hardware support for IPMI | 02:26 |
| devananda | Haomeng: i think it is best if we start with a simple set of data | 02:26 |
| Haomeng | devananda: yes | 02:26 |
| Haomeng | devananda: agree with you, we support with a min set | 02:26 |
| Haomeng | devananda: that is good begaining for us, then we can support more types including alarm/events | 02:26 |
| devananda | right | 02:27 |
| devananda | Haomeng: I have dinner plans and need to head out. thanks again for taking this blueprint on :) | 02:27 |
| Haomeng | devananda: I just notice the 'status' field, that should be 'alarm/event' status value we can use, but not in current bp | 02:27 |
| Haomeng | devananda: enjoy your dinner, and good evening:) | 02:27 |
| Haomeng | devananda: will take care for our bp:) | 02:28 |
| Haomeng | devananda: thanks for your trust:) | 02:29 |
| Haomeng | Haomeng: :) | 02:29 |
| devananda | Haomeng: oh, I just remembered -- 57854 does not change all cases of uuidutils | 02:32 |
| devananda | Haomeng: are you going to fix the rest in another patch? | 02:32 |
| Haomeng | devananda: ok, will focus on 57854 today, to fix all uuid issue | 02:33 |
| Haomeng | devananda: thanks for your reminder | 02:33 |
| Haomeng | devananda: because uuidutils has more method we called | 02:34 |
| Haomeng | devananda: not only generate_uuid method | 02:34 |
| devananda | Haomeng: exactly | 02:34 |
| Haomeng | devananda: so will check oslo guys to see how to plan for more method in uuidutils | 02:34 |
| devananda | Haomeng: ok, review posted. Thanks again! | 02:37 |
| * devananda runs out the door to dinner | 02:37 | |
| Haomeng | devananda: thank you | 02:37 |
| Haomeng | devananda: will take it care:) | 02:37 |
| Haomeng | devananda: how do you think, can I remove our oslo uuid.is_uuid_like call with new patch? | 02:44 |
| *** ndipanov has joined #openstack-ironic | 02:50 | |
| *** matsuhashi has quit IRC | 03:11 | |
| openstackgerrit | Haomeng,Wang proposed a change to openstack/ironic: Remove oslo uuidutils.generate_uuid() call https://review.openstack.org/57854 | 03:25 |
| *** epim has quit IRC | 03:25 | |
| *** epim has joined #openstack-ironic | 03:26 | |
| *** vkozhukalov has joined #openstack-ironic | 03:26 | |
| openstackgerrit | Haomeng,Wang proposed a change to openstack/ironic: Remove oslo uuidutils.is_uuid_like call https://review.openstack.org/58430 | 03:36 |
| *** matsuhashi has joined #openstack-ironic | 04:19 | |
| *** urulama has joined #openstack-ironic | 04:24 | |
| *** nosnos_ has quit IRC | 04:32 | |
| *** nosnos has joined #openstack-ironic | 04:32 | |
| *** rongze has quit IRC | 04:48 | |
| *** zigo_ has quit IRC | 05:10 | |
| *** zigo has joined #openstack-ironic | 05:11 | |
| *** epim has quit IRC | 05:14 | |
| *** rongze has joined #openstack-ironic | 05:19 | |
| *** rongze has quit IRC | 05:27 | |
| *** matsuhashi has quit IRC | 05:50 | |
| *** matsuhashi has joined #openstack-ironic | 05:51 | |
| *** matsuhas_ has joined #openstack-ironic | 05:53 | |
| *** matsuhashi has quit IRC | 05:53 | |
| openstackgerrit | Jenkins proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/58438 | 06:00 |
| *** vkozhukalov has quit IRC | 06:15 | |
| *** michchap has quit IRC | 06:16 | |
| *** michchap has joined #openstack-ironic | 06:16 | |
| *** matsuhas_ has quit IRC | 06:20 | |
| *** rongze has joined #openstack-ironic | 06:24 | |
| *** rongze has quit IRC | 06:28 | |
| *** matsuhashi has joined #openstack-ironic | 06:29 | |
| *** rongze has joined #openstack-ironic | 06:38 | |
| *** matsuhashi has quit IRC | 06:53 | |
| *** matsuhashi has joined #openstack-ironic | 06:54 | |
| *** urulama has quit IRC | 06:55 | |
| *** matsuhashi has quit IRC | 06:58 | |
| *** matsuhashi has joined #openstack-ironic | 06:59 | |
| *** sjing has quit IRC | 07:04 | |
| *** sjing has joined #openstack-ironic | 07:06 | |
| *** matsuhashi has quit IRC | 07:26 | |
| *** matsuhashi has joined #openstack-ironic | 07:27 | |
| *** urulama has joined #openstack-ironic | 07:32 | |
| GheRivero | morning all | 07:34 |
| *** matsuhas_ has joined #openstack-ironic | 07:39 | |
| Haomeng | GheRivero: morning:) | 07:42 |
| *** matsuhashi has quit IRC | 07:42 | |
| *** lexx has joined #openstack-ironic | 08:00 | |
| *** matsuhas_ has quit IRC | 08:05 | |
| *** matsuhashi has joined #openstack-ironic | 08:05 | |
| *** romcheg has joined #openstack-ironic | 08:18 | |
| *** romcheg1 has joined #openstack-ironic | 08:20 | |
| *** romcheg has quit IRC | 08:20 | |
| *** vkozhukalov has joined #openstack-ironic | 08:23 | |
| *** romcheg1 has quit IRC | 08:26 | |
| *** lexx has quit IRC | 08:27 | |
| *** ndipanov has quit IRC | 08:29 | |
| *** ndipanov has joined #openstack-ironic | 08:30 | |
| Haomeng | GheRivero: hi | 08:32 |
| Haomeng | GheRivero: for this one - https://review.openstack.org/#/c/58430/, thanks for your coments, but not sure how to remove the requirement from openstack-common.conf | 08:32 |
| GheRivero | just remove the line module=uuidutils | 08:34 |
| GheRivero | that is the list of modules that are imported from oslo.incubator | 08:34 |
| *** vkozhukalov has quit IRC | 08:34 | |
| *** vkozhukalov has joined #openstack-ironic | 08:41 | |
| Haomeng | GheRivero: got it thank you | 08:43 |
| *** sjing has quit IRC | 08:53 | |
| *** derekh has joined #openstack-ironic | 08:54 | |
| *** romcheg has joined #openstack-ironic | 08:54 | |
| Haomeng | GheRivero: one question, do you know if our ironic.conf.sample is auto-generated or not? | 08:56 |
| GheRivero | ./tools/conf/generate_sample.sh is manually launched | 09:00 |
| *** jistr has joined #openstack-ironic | 09:06 | |
| *** _coolsvap_ has joined #openstack-ironic | 09:06 | |
| Haomeng | GheRivero: cool, thank you:) | 09:07 |
| *** lucasagomes has joined #openstack-ironic | 09:24 | |
| *** max_lobur_afk is now known as max_lobur | 09:27 | |
| openstackgerrit | Yuriy Zveryanskyy proposed a change to openstack/ironic: Add port address unique constraint for sqlite https://review.openstack.org/57722 | 09:32 |
| *** _coolsvap_ has quit IRC | 09:37 | |
| openstackgerrit | Yuriy Zveryanskyy proposed a change to openstack/ironic: Add port address unique constraint for sqlite https://review.openstack.org/57722 | 09:44 |
| *** matsuhas_ has joined #openstack-ironic | 09:58 | |
| *** matsuhas_ has quit IRC | 09:58 | |
| *** matsuhashi has quit IRC | 10:02 | |
| zigo | GheRivero: Are you around? | 10:09 |
| zigo | GheRivero: I finally got a (free) account on the HP cloud to maintain Debian images, though just like you at the summit, I'm stuck with knowing how to do the export OS_* env vars... | 10:09 |
| zigo | GheRivero: What should I put as username and password? | 10:10 |
| zigo | Did you figured that out? | 10:10 |
| GheRivero | zigo: give me a minute... | 10:13 |
| zigo | ok | 10:13 |
| GheRivero | zigo: see private | 10:15 |
| *** rongze has quit IRC | 10:29 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Sync common.context changes from olso https://review.openstack.org/58490 | 10:32 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Remove openstack.common.uuidutils https://review.openstack.org/58492 | 10:36 |
| *** martyntaylor has joined #openstack-ironic | 10:50 | |
| *** arata has left #openstack-ironic | 11:23 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Use oslo's execute() and ssh_execute() methods https://review.openstack.org/58346 | 11:45 |
| *** michchap has quit IRC | 11:46 | |
| *** michchap has joined #openstack-ironic | 11:48 | |
| *** martyntaylor1 has joined #openstack-ironic | 11:49 | |
| *** martyntaylor has quit IRC | 11:51 | |
| *** romcheg has quit IRC | 11:53 | |
| *** nosnos has quit IRC | 12:11 | |
| *** michchap has quit IRC | 12:13 | |
| *** max_lobur has quit IRC | 12:13 | |
| *** rongze has joined #openstack-ironic | 12:15 | |
| *** tatyana has joined #openstack-ironic | 12:18 | |
| *** mihgen has joined #openstack-ironic | 12:34 | |
| *** max_lobur has joined #openstack-ironic | 12:36 | |
| *** lucasagomes is now known as lucas-hungry | 12:41 | |
| *** michchap has joined #openstack-ironic | 12:43 | |
| *** romcheg has joined #openstack-ironic | 12:50 | |
| *** michchap has quit IRC | 12:51 | |
| *** michchap has joined #openstack-ironic | 12:57 | |
| *** michchap has quit IRC | 13:01 | |
| *** rongze has quit IRC | 13:17 | |
| *** jbjohnso has joined #openstack-ironic | 13:20 | |
| *** michchap has joined #openstack-ironic | 13:28 | |
| *** rongze has joined #openstack-ironic | 13:29 | |
| *** michchap has quit IRC | 13:37 | |
| *** lucas-hungry is now known as lucasagomes | 13:44 | |
| *** rloo has joined #openstack-ironic | 13:45 | |
| *** arata has joined #openstack-ironic | 13:46 | |
| *** michchap has joined #openstack-ironic | 13:58 | |
| max_lobur | morning Ironic | 14:00 |
| lucasagomes | morning max_lobur :) | 14:00 |
| max_lobur | morning lucasagomes =) | 14:01 |
| max_lobur | can someone help me with this: | 14:01 |
| max_lobur | when I do | 14:01 |
| max_lobur | ironic node-show 6cbec892-f328-4bcc-ad7c-429f0ad3303a | 14:01 |
| max_lobur | i'm getting | 14:01 |
| max_lobur | You must provide a username via either --os-username or via env[OS_USERNAME] | 14:01 |
| max_lobur | but | 14:01 |
| max_lobur | curl -i -X GET -H 'Content-Type: application/json' -H 'Accept: application/json' -H 'User-Agent: python-ironicclient' http://192.168.122.20:6385/v1/nodes/6cbec892-f328-4bcc-ad7c-429f0ad3303a | 14:01 |
| max_lobur | HTTP/1.0 200 OK | 14:02 |
| max_lobur | Date: Tue, 26 Nov 2013 14:02:03 GMT | 14:02 |
| max_lobur | Server: WSGIServer/0.1 Python/2.7.3 | 14:02 |
| max_lobur | Content-Length: 703 | 14:02 |
| max_lobur | Content-Type: application/json; charset=UTF-8 | 14:02 |
| max_lobur | {"instance_uuid": null, "target_power_state": null, "driver_info": {}, "links": [{"href": "http://192.168.122.20:6385/v1/nodes/6cbec892-f328-4bcc-ad7c-429f0ad3303a", "rel": "self"}, {"href": "http://192.168.122.20:6385/nodes/6cbec892-f328-4bcc-ad7c-429f0ad3303a", "rel": "bookmark"}], "extra": {}, "target_provision_state": null, "driver": "d3", "chassis_id": null, "properties": {}, "power_state": null, "provision_state": null, "ports": [{"href": " | 14:02 |
| max_lobur | http://192.168.122.20:6385/v1/nodes/6cbec892-f328-4bcc-ad7c-429f0ad3303a/ports", "rel": "self"}, {"href": "http://192.168.122.20:6385/nodes/6cbec892-f328-4bcc-ad7c-429f0ad3303a/ports", "rel": "bookmark"}], "uuid": "6cbec892-f328-4bcc-ad7c-429f0ad3303a"} | 14:02 |
| max_lobur | what am I missing? Is there some option in our configs e.g. "do not check auth token" | 14:02 |
| *** michchap has quit IRC | 14:03 | |
| openstackgerrit | Yuriy Zveryanskyy proposed a change to openstack/ironic: Allow clean reservation at update node https://review.openstack.org/55549 | 14:04 |
| lucasagomes | max_lobur, I usually export this variables here: http://paste.openstack.org/show/53987/ | 14:08 |
| max_lobur | true, they're empty | 14:10 |
| max_lobur | but I wanted to get 'not authorized error' via curl | 14:10 |
| max_lobur | but I can't | 14:10 |
| max_lobur | I mean it works without X-Auth-Token header, is this a bug? | 14:12 |
| lucasagomes | hmm idk either | 14:13 |
| lucasagomes | gotta take a look | 14:13 |
| max_lobur | I will be very thankful if you try to reproduce it | 14:14 |
| lucasagomes | right, will take a look. but yea looks weird to not require the token authentication | 14:21 |
| lucasagomes | need to finish something before tho | 14:21 |
| *** jdob has joined #openstack-ironic | 14:21 | |
| max_lobur | ok, thanks | 14:22 |
| *** prekarat has joined #openstack-ironic | 14:22 | |
| *** lexx has joined #openstack-ironic | 14:37 | |
| GheRivero | hi all | 14:44 |
| *** saranya31 has joined #openstack-ironic | 14:49 | |
| *** jdob has quit IRC | 14:51 | |
| *** saranya31 has left #openstack-ironic | 14:55 | |
| *** michchap has joined #openstack-ironic | 14:59 | |
| NobodyCam | good morning Ironic | 15:02 |
| *** michchap has quit IRC | 15:03 | |
| GheRivero | morning NobodyCam | 15:07 |
| NobodyCam | good mornig GheRivero :) | 15:08 |
| rloo | morning NobodyCam, GheRivero! | 15:08 |
| NobodyCam | morning rloo :) | 15:09 |
| *** vkozhukalov has quit IRC | 15:09 | |
| *** jdob has joined #openstack-ironic | 15:10 | |
| dkehn | NobodyCam: GheRivero I'm going to be calling devananda this morning concerning the neutron integration (yesterdays's meeting topic) would you all like to be involved | 15:12 |
| dkehn | g+ I think will be the medium | 15:12 |
| GheRivero | at what time? but yeah, will be lovely | 15:14 |
| NobodyCam | dkehn: sure ... when? | 15:15 |
| dkehn | NobodyCam: not sure, gotta get ahold of devananda 1st | 15:15 |
| NobodyCam | lol | 15:15 |
| dkehn | so kindof an adhoc meeting | 15:15 |
| NobodyCam | dkehn: ++ just ping me :) | 15:15 |
| dkehn | but planned sorta | 15:15 |
| dkehn | k | 15:16 |
| NobodyCam | :) | 15:16 |
| dkehn | GheRivero: ? | 15:16 |
| GheRivero | yeah, sure, ping me back when settled | 15:16 |
| dkehn | GheRivero: I'm sure the PXE is going to come up | 15:16 |
| dkehn | or I'm pretty sure | 15:16 |
| GheRivero | :) | 15:16 |
| *** arata has left #openstack-ironic | 15:17 | |
| dkehn | GheRivero: is that a yes? | 15:17 |
| GheRivero | a big YES | 15:17 |
| dkehn | ok will ping you all | 15:17 |
| dkehn | might be skype too | 15:18 |
| dkehn | whatever the consensis is | 15:18 |
| * NobodyCam can support both | 15:19 | |
| GheRivero | :/ i hate this weather! 23-43F | 15:19 |
| NobodyCam | GheRivero: Ouch | 15:19 |
| GheRivero | whatever is ok | 15:19 |
| openstackgerrit | Ruby Loo proposed a change to openstack/ironic: Adds doc strings to API FunctionalTest class https://review.openstack.org/57344 | 15:19 |
| NobodyCam | http://www.weather.com/weather/tenday/USCA0512 | 15:20 |
| NobodyCam | :-p | 15:20 |
| GheRivero | http://www.weather.com/weather/tenday/SPXX0196 | 15:22 |
| GheRivero | NobodyCam: can we interchange locations? | 15:22 |
| NobodyCam | lol :) GheRivero you wanta live in a RV? | 15:23 |
| NobodyCam | brb | 15:24 |
| GheRivero | I could live anyplace I wouldnt' need the Ironic hoodie inside home | 15:24 |
| *** urulama has quit IRC | 15:25 | |
| dkehn | GheRivero: is that cold for your area | 15:26 |
| GheRivero | it will get worst | 15:26 |
| GheRivero | but without snow | 15:26 |
| dkehn | GheRivero: bummer, we are getting out of a snow, but here at this altitude, it has melted | 15:27 |
| dkehn | GheRivero: I would love to live in a warmer climate during the winter | 15:27 |
| GheRivero | dkehn: ++ | 15:28 |
| dkehn | GheRivero: when my kids are gone, I'm thinking tropics in the winter | 15:28 |
| GheRivero | jaja... I will have to deploy kids first :) | 15:29 |
| dkehn | I think this is the part were words of wisdow would be departed, but alass there is none | 15:30 |
| NobodyCam | dkehn: lol | 15:32 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: API ports update for WSME 0.5b6 compliance https://review.openstack.org/56682 | 15:39 |
| devananda | morning, all | 15:45 |
| devananda | dkehn: pong | 15:46 |
| dkehn | devananda: ok, let me see about setting up a G+ group with NobodyCam & GheRivero, you need time? | 15:47 |
| devananda | dkehn: yea. gimme 15 or so | 15:48 |
| dkehn | devananda: you gotit, ping me, and in the meantime I'll try and figure out g+ | 15:48 |
| NobodyCam | morning devananda :) | 15:49 |
| *** briancline has joined #openstack-ironic | 15:50 | |
| openstackgerrit | Chris Krelle proposed a change to openstack/ironic: Allow FakePower to return node objects power_state https://review.openstack.org/58333 | 15:51 |
| openstackgerrit | Max Lobur proposed a change to openstack/ironic: Do not allow API to expose error stacktrace https://review.openstack.org/58558 | 15:51 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Rework patch validation on Ports https://review.openstack.org/56984 | 15:54 |
| *** tatyana has quit IRC | 15:56 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Reworks Node validations https://review.openstack.org/57495 | 15:59 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Reworks Chassis validations https://review.openstack.org/57686 | 15:59 |
| *** michchap has joined #openstack-ironic | 15:59 | |
| max_lobur | hi devananda | 16:03 |
| max_lobur | TY for so fast review of https://review.openstack.org/#/c/58558/ | 16:03 |
| max_lobur | pls take a look again, I left a few thoughts from my side | 16:04 |
| max_lobur | maybe you'll find them not correct | 16:04 |
| *** michchap has quit IRC | 16:04 | |
| max_lobur | lucasagomes, https://review.openstack.org/#/c/58558/ please take a look too when you have time | 16:04 |
| lucasagomes | sure thing :D thanks | 16:05 |
| lucasagomes | I added myself to the reviewers list | 16:05 |
| max_lobur | also, devananda I'd like you to look at the https://bugs.launchpad.net/ironic/+bug/1254002 | 16:05 |
| max_lobur | I proposed a solution there | 16:05 |
| max_lobur | but not sure if it's OF | 16:05 |
| max_lobur | *OK | 16:05 |
| max_lobur | lucasagomes, thanks =) | 16:06 |
| devananda | lucasagomes: on 56682, you removed test_types.py ? | 16:06 |
| max_lobur | leaving for today. have a nice day! | 16:06 |
| lucasagomes | devananda, test_types on that review was only testing the mactype and uuidtype | 16:07 |
| *** max_lobur is now known as max_lobur_afk | 16:07 | |
| devananda | lucasagomes: i thought my suggestion was to test the code paths by mocking is_valid_mac / is_uuid_like | 16:07 |
| * lucasagomes re-read, I think might have get confused then | 16:08 | |
| lucasagomes | devananda, ahh | 16:08 |
| lucasagomes | urgh, haha I thought it was duplicated and won't need to be tested | 16:09 |
| lucasagomes | lemme readd and modify the tests | 16:09 |
| devananda | lucasagomes: I reread my comment. it wasn't the most well worded comment i've ever left :) | 16:09 |
| openstackgerrit | A change was merged to openstack/ironic: Remove oslo uuidutils.generate_uuid() call https://review.openstack.org/57854 | 16:09 |
| lucasagomes | devananda, hehe it's grand | 16:09 |
| devananda | :) | 16:09 |
| devananda | lucasagomes: oh, also, this will conflict with a patch that Haomeng is working on | 16:10 |
| lucasagomes | devananda, yes, the uuidutils one? | 16:10 |
| devananda | lucasagomes: yes. which just landed. and the next noe will move is_uuid_like into ironic.common, too | 16:10 |
| lucasagomes | devananda, I will wait it get merged and then I can rebase | 16:11 |
| devananda | :) | 16:11 |
| lucasagomes | lemme mark as wip | 16:11 |
| devananda | lucasagomes: actually, lemme just approve Haomeng 's patch | 16:14 |
| lucasagomes | cool if you think it's ready to go | 16:15 |
| lucasagomes | if not I can also put a dependency on it | 16:15 |
| lucasagomes | I +2 that patch already, lgtm | 16:15 |
| lucasagomes | ok got approved :) | 16:16 |
| * NobodyCam makes more coffee | 16:16 | |
| dkehn | devananda: NobodyCam GheRivero we ready? | 16:17 |
| devananda | dkehn: few more min. gotta step afk for a moment | 16:18 |
| dkehn | devananda: k | 16:18 |
| openstackgerrit | A change was merged to openstack/ironic: Remove oslo uuidutils.is_uuid_like call https://review.openstack.org/58430 | 16:19 |
| *** mihgen has quit IRC | 16:22 | |
| devananda | dkehn: right. call when ever | 16:26 |
| dkehn | devananda: one minute | 16:27 |
| dkehn | devananda: GheRivero NobodyCam all ready | 16:33 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Import uuidutils unit tests from oslo https://review.openstack.org/58572 | 16:34 |
| devananda | Haomeng, lucasagomes: last patch for the uuid series ^ | 16:34 |
| devananda | dkehn: https://blueprints.launchpad.net/ironic/+spec/instance-mapping-by-consistent-hash | 16:35 |
| lucasagomes | devananda, nice | 16:36 |
| lucasagomes | added myself to the list of reviewers | 16:36 |
| devananda | dkehn: https://docs.google.com/drawings/d/16c3g8Rx4y24mTDA76A0gw0ZxrHugeCN1r0zSzohTVKw/edit?usp=sharing | 16:36 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: API ports update for WSME 0.5b6 compliance https://review.openstack.org/56682 | 16:40 |
| *** kobier has joined #openstack-ironic | 16:51 | |
| *** kobier has quit IRC | 16:51 | |
| *** lexx has quit IRC | 16:59 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Rework patch validation on Ports https://review.openstack.org/56984 | 16:59 |
| *** michchap has joined #openstack-ironic | 17:00 | |
| *** michchap has quit IRC | 17:05 | |
| NobodyCam | bbt.. brb | 17:05 |
| *** martyntaylor1 has quit IRC | 17:06 | |
| *** romcheg has quit IRC | 17:08 | |
| *** lexx has joined #openstack-ironic | 17:09 | |
| *** vkozhukalov has joined #openstack-ironic | 17:23 | |
| *** urulama has joined #openstack-ironic | 17:26 | |
| *** urulama has quit IRC | 17:28 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add driver interfaces validation https://review.openstack.org/54593 | 17:36 |
| *** jistr has quit IRC | 17:39 | |
| *** anniec has joined #openstack-ironic | 17:40 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add driver interfaces validation https://review.openstack.org/54593 | 17:41 |
| lucasagomes | urgh >.< | 17:43 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add driver interfaces validation https://review.openstack.org/54593 | 17:44 |
| *** rongze has quit IRC | 17:44 | |
| *** romcheg has joined #openstack-ironic | 17:48 | |
| NobodyCam | brb | 17:49 |
| *** anniec has quit IRC | 17:51 | |
| *** derekh has quit IRC | 17:54 | |
| *** hemnafk is now known as hemna | 17:56 | |
| *** anniec has joined #openstack-ironic | 17:56 | |
| *** michchap has joined #openstack-ironic | 18:01 | |
| *** michchap has quit IRC | 18:05 | |
| *** anniec has quit IRC | 18:06 | |
| lucasagomes | night everyone :) | 18:08 |
| *** lucasagomes has quit IRC | 18:08 | |
| devananda | g'night lucas | 18:10 |
| NobodyCam | he's quick like that | 18:10 |
| devananda | NobodyCam: consistent hash -- it seems like we're actually mapping nodes <-> conductors, not instances <-> conductors | 18:11 |
| devananda | NobodyCam: because nova-scheduler is doing the job of assigning instance -> node | 18:11 |
| NobodyCam | true | 18:12 |
| *** anniec has joined #openstack-ironic | 18:16 | |
| devananda | NobodyCam: we also need to track separate rings for each driver | 18:20 |
| devananda | NobodyCam: eg, if a conductor servicing the Foo driver goes offline, we only need to rebalance nodes that use the Foo driver | 18:21 |
| NobodyCam | yes | 18:21 |
| NobodyCam | i think... if we lose a conductor several rings will have to re balance | 18:22 |
| NobodyCam | different conductors will back up deferent rings ... | 18:24 |
| NobodyCam | quick walkies ... brb | 18:25 |
| *** prekarat has quit IRC | 18:34 | |
| *** rongze has joined #openstack-ironic | 18:44 | |
| NobodyCam | back | 18:50 |
| *** rongze has quit IRC | 18:53 | |
| *** jistr has joined #openstack-ironic | 18:57 | |
| *** lexx has quit IRC | 18:58 | |
| *** michchap has joined #openstack-ironic | 19:02 | |
| *** michchap has quit IRC | 19:06 | |
| *** epim has joined #openstack-ironic | 19:12 | |
| *** lexx has joined #openstack-ironic | 19:38 | |
| *** rongze has joined #openstack-ironic | 19:49 | |
| *** rongze has quit IRC | 19:54 | |
| *** michchap has joined #openstack-ironic | 20:03 | |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Implement consistent hashing common methods https://review.openstack.org/58607 | 20:05 |
| *** derekh has joined #openstack-ironic | 20:05 | |
| *** michchap has quit IRC | 20:07 | |
| *** jdob has quit IRC | 20:16 | |
| *** jdob has joined #openstack-ironic | 20:16 | |
| *** anniec has quit IRC | 20:25 | |
| *** anniec has joined #openstack-ironic | 20:28 | |
| *** jdob has quit IRC | 20:38 | |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Implement consistent hashing common methods https://review.openstack.org/58607 | 20:38 |
| *** epim has quit IRC | 20:41 | |
| *** jistr has quit IRC | 20:49 | |
| *** rongze has joined #openstack-ironic | 20:50 | |
| *** rongze has quit IRC | 20:56 | |
| NobodyCam | humm brb... | 20:58 |
| *** ndipanov has quit IRC | 21:01 | |
| *** michchap has joined #openstack-ironic | 21:03 | |
| *** michchap has quit IRC | 21:08 | |
| *** urulama has joined #openstack-ironic | 21:28 | |
| hemna | ping | 21:29 |
| *** urulama has quit IRC | 21:33 | |
| *** anniec has quit IRC | 21:35 | |
| openstackgerrit | A change was merged to openstack/ironic: Adds doc strings to API FunctionalTest class https://review.openstack.org/57344 | 21:40 |
| openstackgerrit | A change was merged to openstack/ironic: Add troubleshoot option to PXE template https://review.openstack.org/58262 | 21:43 |
| devananda | hemna: pong | 21:45 |
| hemna | so I had a question about the iscsi code in ironic | 21:45 |
| hemna | where does that iscsi lun come from currently ? | 21:46 |
| hemna | from what I've gathered, ironic attaches an iscsi lun, partitions it, formats it, then unattaches it. | 21:47 |
| hemna | dealing with the volume attach/detach seems like a good candidate for cinder orchestration | 21:51 |
| hemna | cinder can present the volume to the ironic host, then ironic can continue to do the partitioning and formatting as it does now, then tell cinder to detach | 21:52 |
| *** vkozhukalov has quit IRC | 21:52 | |
| hemna | I presume at some point there is a way to tell the bios on the BM box to boot from san | 21:52 |
| *** rongze has joined #openstack-ironic | 21:52 | |
| devananda | hemna: what code are you referring to? | 21:55 |
| devananda | hemna: there is currently no cinder or boot-from-volume code in ironic | 21:55 |
| hemna | devananda, correct | 21:56 |
| hemna | I'm looking at adding cinder integration :) | 21:56 |
| devananda | hemna: ok. so when you say, "ironic attaches .. aprtitions .. formats .. " | 21:57 |
| hemna | so the current iscsi code in ironic lives in drivers/modules/deploy_utils.py | 21:57 |
| devananda | that is specific to the PXE driver -- drivers/modules/pxe.py | 21:57 |
| devananda | the PXE deployment ramdisk exposes the physical machine's local disk (/dev/sda) over iSCSI | 21:58 |
| devananda | POSTs the LUN back to Ironic | 21:58 |
| hemna | ah ok, so that's the BM's local disk. | 21:58 |
| devananda | this is passed down to drivers.modules.pxe:VendorPassthru | 21:58 |
| hemna | so if we wanted boot from san via cinder volume... | 21:58 |
| devananda | and then pxe driver formats it and copies the user's image to the disk with dd | 21:58 |
| devananda | boot from san - that's _totaly_ seaprate | 21:58 |
| hemna | ok | 21:59 |
| *** rongze has quit IRC | 21:59 | |
| devananda | hemna: you would create a new driver | 21:59 |
| hemna | (still climbing the ironic learning curve) | 21:59 |
| devananda | hemna: and implement the driver.deploy interface | 21:59 |
| devananda | :) | 21:59 |
| hemna | so, I presume it could go something like this then... | 22:00 |
| hemna | write a cinder.py driver | 22:00 |
| devananda | hemna: http://git.openstack.org/cgit/openstack/ironic/tree/ironic/drivers/base.py#n75 | 22:00 |
| hemna | in that it would ask cinder to attach the volume to the ironic host, partition, format it, dd the image to it, detach | 22:00 |
| hemna | then ask cinder to attach the volume to the BM, and then reboot the BM ? | 22:00 |
| devananda | hm, ideally, no | 22:01 |
| hemna | the only thing I'm not sure about is how to ask the bios to boot from san remotely | 22:01 |
| devananda | well, question | 22:02 |
| hemna | k | 22:02 |
| devananda | when doing boot-from-volume, doesnt the volume already exist? | 22:02 |
| devananda | or do you need to make a new volume & copy the image to it? | 22:02 |
| hemna | I think boot from volume, the volume already exists | 22:03 |
| devananda | that's waht i thought | 22:03 |
| devananda | so you dont need to attach the volume to the ironic host at all | 22:03 |
| hemna | could give it the uuid of the volume you want to boot from | 22:03 |
| hemna | ironic would just have to tell cinder to attach the volume to the BM | 22:03 |
| devananda | i think the driver.deploy would look something like this | 22:03 |
| *** michchap has joined #openstack-ironic | 22:04 | |
| *** derekh has quit IRC | 22:04 | |
| devananda | - do some check that volume exists, node status is OK, etc | 22:04 |
| devananda | - tell hw node's BIOS to mount the iSCSI target // SAN volume as a local disk, and to boot from it | 22:04 |
| devananda | - cycle power of the machine | 22:05 |
| devananda | end | 22:05 |
| hemna | that seems "easy" enough | 22:05 |
| devananda | well. the hard part is the middle step :) | 22:05 |
| hemna | cinder supports Fibre Channel arrays as well | 22:05 |
| hemna | talking to the BIOS is a mystery to me at this point. | 22:06 |
| devananda | right | 22:06 |
| hemna | does IPMI support something like that? | 22:06 |
| devananda | hmmm, not really, but kinda | 22:06 |
| devananda | AFAIK, it's not part of the IPMI spec | 22:06 |
| devananda | so each vendor has done it differently | 22:06 |
| hemna | I'm sure talking to the BIOS is going to be vendor specific | 22:06 |
| hemna | so maybe a "driver" layer is needed | 22:07 |
| hemna | to allow supporting different vendors | 22:07 |
| devananda | that's already in place :) | 22:07 |
| hemna | ok great. | 22:07 |
| devananda | we don't have any vendor drivers yet | 22:07 |
| hemna | :P | 22:07 |
| devananda | but the separation would be like this... | 22:07 |
| hemna | where is the current code for it ? | 22:08 |
| devananda | same file | 22:08 |
| hemna | drivers/base.py ? | 22:08 |
| devananda | the BaseDriver class defines the interfaces | 22:08 |
| hemna | ok | 22:08 |
| *** michchap has quit IRC | 22:09 | |
| devananda | we dont have an interface defined for "mount this volume" -- but there are interfaces defined for power mgmt and for allowing vendors to extend the itnerfaces | 22:09 |
| hemna | I see the VendorInterface | 22:09 |
| hemna | so would it make sense to add a mount_volume, unmount_volume API to the VendorInterface ? | 22:10 |
| devananda | the VendorInterface is there to allow vendors to do things that aren't part of the API yet | 22:11 |
| devananda | so eg, you might implement a CinderWithILO driver | 22:12 |
| *** michchap has joined #openstack-ironic | 22:12 | |
| devananda | and a CinderWithDRAC driver | 22:12 |
| devananda | both would share the same driver.deploy == CinderDeploy | 22:12 |
| devananda | one would have driver.vendor == IloVendor. the other would have driver.vendor == DracVendor | 22:13 |
| devananda | or something. you get the idea, i hope | 22:13 |
| hemna | ok yah I think that makes sense | 22:13 |
| hemna | so don't add a mount_volume to the VendorInterface then ? | 22:13 |
| devananda | no | 22:13 |
| devananda | another possibility, but i am not sold on it yet, would be to add a BaseDriver.volume interface | 22:14 |
| hemna | I presume then the CinderWithILO will make the ILO specific calls via the VendorInterface::vendor_passthru then | 22:14 |
| devananda | ya | 22:14 |
| hemna | ok I think I get ya :P | 22:14 |
| hemna | so we could still use an ILOVendor and DRACVendor | 22:15 |
| devananda | hemna: slight tangent | 22:15 |
| devananda | hemna: are you aiming specifically for deployment? or a general mount-this-cinder-volume? | 22:16 |
| devananda | i would prefer the latter. deploy is just a special case where the volume is also marked bootable | 22:16 |
| hemna | well, my initial task was to see how to integrate with cinder to use it's volumes. | 22:16 |
| hemna | so we can have diskless servers provisioned by ironic using cinder backends (3PAR/LeftHand) | 22:17 |
| hemna | and others :P | 22:17 |
| devananda | gotcha | 22:17 |
| devananda | so then, yea, I'd prefer to add a volume interface to BaseDriver | 22:18 |
| hemna | and that volume could either be a cinder volume or a local disk on the BM ? | 22:19 |
| devananda | hmmm | 22:20 |
| hemna | I guess the default would be the local disk | 22:20 |
| *** anniec has joined #openstack-ironic | 22:21 | |
| *** epim has joined #openstack-ironic | 22:24 | |
| *** anniec has quit IRC | 22:29 | |
| devananda | hemna: are you working with any server hardware vendors? (hp, dell, etc) | 22:33 |
| hemna | I work at HP | 22:33 |
| hemna | I presume there is a way to do this on proliant machines | 22:34 |
| hemna | I work in MSDU (Storage) supporting 3PAR and LeftHand, but am also core in Cinder. | 22:35 |
| *** lexx has quit IRC | 22:37 | |
| devananda | hemna: ah! great | 22:38 |
| hemna | devananda thanks a bunch for your help. | 22:47 |
| hemna | I'll throw together a BP once I'm back from the holiday break | 22:47 |
| devananda | hemna: great, thanks | 22:54 |
| devananda | hemna: this isn't part of our plan for nova-baremetal equivalency, so i will probably target teh BP to the J release at this point | 22:55 |
| *** rongze has joined #openstack-ironic | 22:55 | |
| devananda | hemna: but i'm very happy to discuss and see the work start | 22:55 |
| *** romcheg has quit IRC | 22:56 | |
| hemna | ok I'd like to see what we can do for Icehouse, if anything. | 22:56 |
| *** romcheg has joined #openstack-ironic | 22:57 | |
| *** romcheg has quit IRC | 22:58 | |
| dkehn | devananda: are you using the generate_sample.sh from nova to generate your etc/ironic/ironic.conf.sample, just curious becuase I don't see a tools dir, or are you just adding by hand? | 22:58 |
| devananda | dkehn: ironic/tools/conf/generate_sample.sh | 22:59 |
| *** romcheg has joined #openstack-ironic | 22:59 | |
| dkehn | devananda: sorry missed that | 22:59 |
| *** romcheg has left #openstack-ironic | 22:59 | |
| *** rongze has quit IRC | 22:59 | |
| *** jcooley_ has joined #openstack-ironic | 23:00 | |
| devananda | dkehn: np | 23:00 |
| devananda | speaking of which, i need to regenerate it | 23:00 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Update ironic.conf.sample https://review.openstack.org/58642 | 23:01 |
| hemna | have a good break folks...I'm out. | 23:03 |
| *** hemna is now known as hemnafk | 23:04 | |
| * devananda steps afk for a bit, too | 23:04 | |
| NobodyCam | :) | 23:04 |
| * NobodyCam pulls some hair out | 23:05 | |
| NobodyCam | lol | 23:05 |
| *** epim has quit IRC | 23:22 | |
| *** epim has joined #openstack-ironic | 23:26 | |
| *** urulama has joined #openstack-ironic | 23:30 | |
| *** urulama has quit IRC | 23:34 | |
| *** epim has quit IRC | 23:47 | |
| *** rongze has joined #openstack-ironic | 23:56 | |
| *** epim has joined #openstack-ironic | 23:56 | |
| *** jcooley_ has quit IRC | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!