Friday, 2014-04-18

« Thursday, 2014-04-17 Index Saturday, 2014-04-19 »
*** zdiN0bot has joined #openstack-ironic00:04
*** zdiN0bot has quit IRC00:09
*** eghobo has joined #openstack-ironic00:10
*** rloo has quit IRC00:11
*** rloo has joined #openstack-ironic00:11
*** ilives has joined #openstack-ironic00:13
*** eghobo has quit IRC00:16
*** eghobo has joined #openstack-ironic00:16
*** ilives has quit IRC00:17
*** ilives has joined #openstack-ironic00:17
*** rloo has quit IRC00:19
devanandaShrews: 86993 needs a rebase00:19
*** rloo has joined #openstack-ironic00:19
*** matsuhashi has joined #openstack-ironic00:23
*** rloo has quit IRC00:24
devanandaok, i'm off for the night! see ya'll tmw00:28
*** derekh has quit IRC00:31
*** epim has quit IRC00:33
openstackgerritA change was merged to openstack/ironic: Use DiskPartitioner  https://review.openstack.org/8339900:44
openstackgerritAdam Gandelman proposed a change to openstack/ironic: Decouple state inspection and availability check  https://review.openstack.org/8847600:58
adam_gdevananda, ^ i think thats a better fix for at least one of those bugs (the nova-compute startup issue)00:59
*** zdiN0bot has joined #openstack-ironic01:20
*** nosnos has joined #openstack-ironic01:24
*** zdiN0bot has quit IRC01:24
*** zdin0bot has joined #openstack-ironic01:32
*** foexle_ has joined #openstack-ironic01:47
*** foexle has quit IRC01:50
*** rwsu has quit IRC02:22
*** harlowja is now known as harlowja_away02:49
*** coolsvap|afk is now known as coolsvap02:58
*** eghobo has quit IRC03:06
*** matsuhashi has quit IRC03:09
*** matsuhashi has joined #openstack-ironic03:09
*** matsuhashi has quit IRC03:09
*** nosnos has quit IRC03:19
*** matsuhashi has joined #openstack-ironic03:25
*** matsuhashi has quit IRC03:27
*** matsuhashi has joined #openstack-ironic03:28
*** matsuhashi has quit IRC03:31
*** matsuhashi has joined #openstack-ironic03:31
*** matsuhas_ has joined #openstack-ironic03:34
*** matsuhashi has quit IRC03:35
*** lnxnut has quit IRC03:37
*** zdin0bot has quit IRC03:37
*** lnxnut has joined #openstack-ironic03:39
*** matsuhas_ has quit IRC03:54
*** matsuhashi has joined #openstack-ironic03:57
*** eghobo has joined #openstack-ironic03:58
*** nosnos has joined #openstack-ironic04:10
*** lnxnut has quit IRC04:22
*** zdin0bot has joined #openstack-ironic04:32
*** zdin0bot has quit IRC04:36
*** coolsvap is now known as coolsvap|afk04:39
*** blamar has joined #openstack-ironic04:43
*** zdin0bot has joined #openstack-ironic04:48
*** Mikhail_D_ltp has joined #openstack-ironic04:49
*** jgrimm has joined #openstack-ironic04:50
*** vkozhukalov has joined #openstack-ironic04:53
*** coolsvap|afk is now known as coolsvap04:58
*** coolsvap is now known as coolsvap|afk05:05
*** zdin0bot has quit IRC05:24
*** zdin0bot has joined #openstack-ironic05:26
*** romcheg1 has joined #openstack-ironic05:28
*** Mikhail_D_ltp has quit IRC05:35
*** pradipta_away is now known as pradipta06:00
*** max_lobur has joined #openstack-ironic06:06
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ironic: Imported Translations from Transifex  https://review.openstack.org/8850806:07
*** pradipta is now known as pradipta_away06:13
*** vkozhukalov has quit IRC06:14
*** coolsvap|afk is now known as coolsvap06:18
*** pradipta_away is now known as pradipta06:21
*** zdin0bot has quit IRC06:43
*** ifarkas has joined #openstack-ironic06:45
*** viktors has joined #openstack-ironic06:54
*** eghobo has quit IRC06:58
*** matsuhashi has quit IRC07:06
*** matsuhashi has joined #openstack-ironic07:08
openstackgerritYuriy Zveryanskyy proposed a change to openstack/ironic: Add worker threads limit to _check_deploy_timeouts task  https://review.openstack.org/7865107:11
*** jistr has joined #openstack-ironic07:16
*** vkozhukalov has joined #openstack-ironic07:40
*** lazy_prince has joined #openstack-ironic07:42
*** lazy_prince has quit IRC07:46
*** zdin0bot has joined #openstack-ironic07:48
*** max_lobur has quit IRC07:53
*** jistr has quit IRC08:01
*** jistr has joined #openstack-ironic08:25
*** lucasagomes has joined #openstack-ironic08:27
*** zdin0bot has quit IRC08:33
*** lsmola has joined #openstack-ironic08:42
*** max_lobur has joined #openstack-ironic08:50
*** ndipanov has joined #openstack-ironic08:55
*** foexle_ has quit IRC09:02
*** martyntaylor has joined #openstack-ironic09:29
*** pradipta is now known as pradipta_away09:47
*** foexle has joined #openstack-ironic10:06
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Get rid of the swap partition  https://review.openstack.org/8372610:11
*** dkehn__ has joined #openstack-ironic10:11
*** lsmola has quit IRC10:14
*** dkehn_ has quit IRC10:14
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Use GB instead of MB for swap  https://review.openstack.org/8378810:15
*** blamar has quit IRC10:21
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Add ManagementInterface  https://review.openstack.org/8606310:25
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: IPMITool to use the new ManagementInterface  https://review.openstack.org/8609210:25
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: SeaMicro to use the new ManagementInterface  https://review.openstack.org/8632810:25
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: IPMINative to use the new ManagementInterface  https://review.openstack.org/8658810:25
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: IPMINative set_boot_device persistent  https://review.openstack.org/8574210:46
*** Mikhail_D_wk has quit IRC10:48
*** Mikhail_D_wk has joined #openstack-ironic10:50
*** coolsvap is now known as coolsvap|afk10:50
dtantsurGood afternoon, everyone :)10:51
*** nosnos has quit IRC11:01
*** matsuhashi has quit IRC11:12
*** matsuhashi has joined #openstack-ironic11:14
romcheg1Hi dtantsur!11:18
*** romcheg1 has quit IRC11:19
*** nosnos has joined #openstack-ironic11:26
*** nosnos has quit IRC11:29
*** lsmola has joined #openstack-ironic11:30
*** ifarkas has quit IRC11:31
*** matsuhashi has quit IRC11:32
*** matsuhashi has joined #openstack-ironic11:32
*** matsuhashi has quit IRC11:38
*** romcheg has quit IRC11:40
*** romcheg has joined #openstack-ironic11:40
*** lucasagomes is now known as lucas-hungry11:46
lucas-hungryafternoon romcheg1 dtantsur :)11:46
lucas-hungrybrb lunch time11:46
openstackgerritA change was merged to openstack/python-ironicclient: Adds documentation for ironicclient API  https://review.openstack.org/8510711:50
dtantsurlucas-hungry, oh hi! Could you (after lunch) have a short look at https://review.openstack.org/#/c/85387 ? it is by no means finished, but I'd like to collect early feedback on the direction I am going, thanks :)11:51
*** romcheg1 has joined #openstack-ironic11:56
*** romcheg has quit IRC11:58
openstackgerritJarrod Johnson proposed a change to stackforge/pyghmi: Add sensor information from IPMI 2.0 rev 1.1  https://review.openstack.org/8855811:59
*** foexle has quit IRC12:01
*** jbjohnso has joined #openstack-ironic12:06
*** lnxnut has joined #openstack-ironic12:08
*** martyntaylor has left #openstack-ironic12:08
*** andreykurilin has joined #openstack-ironic12:13
andreykurilinhi all! can someone review my patch? https://review.openstack.org/#/c/71500/12:14
openstackgerritDmitry Tantsur proposed a change to openstack/ironic: Implement caching for master images  https://review.openstack.org/8538712:21
dtantsurandreykurilin, will see (though I'm not a core)12:22
andreykurilindtantsur: thanks12:25
openstackgerritDavid Shrewsbury proposed a change to openstack/ironic: Complete wrapping ironic client calls  https://review.openstack.org/8699312:28
openstackgerritDavid Shrewsbury proposed a change to openstack/ironic: Complete wrapping ironic client calls  https://review.openstack.org/8699312:34
openstackgerritDavid Shrewsbury proposed a change to openstack/ironic: Complete wrapping ironic client calls  https://review.openstack.org/8699312:37
Shrews*sigh*12:37
Shrewsmuch too early and too little caffeine for difficult rebases12:38
dtantsurShrews, it sometimes seems to me that rebasing is one of the hardest parts of our job :)12:40
*** foexle has joined #openstack-ironic12:44
*** dkehn__ is now known as dkehnx12:46
*** lucas-hungry is now known as lucasagomes12:51
lucasagomesdtantsur, sure12:51
lucasagomesdtantsur, +1 about rebasing heh12:51
NobodyCamGood Morning and TGIF Ironic13:12
lucasagomesmorning NobodyCam :) yay13:18
ShrewsNobodyCam: morning13:18
NobodyCammorning lucasagomes & Shrews :)13:20
*** linggao has joined #openstack-ironic13:21
*** foexle has quit IRC13:33
*** Mikhail_D_wk has quit IRC13:42
*** Mikhail_D_wk has joined #openstack-ironic13:44
*** foexle has joined #openstack-ironic13:45
*** blamar has joined #openstack-ironic14:02
NobodyCamanyone up to help me look at a neutron error?14:05
*** ndipanov has quit IRC14:08
openstackgerritAndrey Kurilin proposed a change to openstack/python-ironicclient: Sync latest code and reuse exceptions from oslo  https://review.openstack.org/7150014:09
openstackgerritAndrey Kurilin proposed a change to openstack/python-ironicclient: Reuse module `cliutils` from common code  https://review.openstack.org/7241814:23
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: N-cpu to not die if Ironic is not running  https://review.openstack.org/8859014:26
lucasagomesNobodyCam, what's up?14:26
*** zdin0bot has joined #openstack-ironic14:29
NobodyCamoh let me get the error14:30
NobodyCamxuhaiwei ]14:30
NobodyCamhttp://paste.openstack.org/show/16VqycxIn7eilvdwdGTF/14:30
*** zdin0bot has quit IRC14:32
Shrewslucasagomes: why not just catch NovaException instead of re-introducing the MaximumRetriesReached exception in 88590?14:33
*** foexle has quit IRC14:37
lucasagomesShrews, hey hmm I thought about it, but sounds too generic14:37
lucasagomesI sometimes prefer to have a very specific exception for cases like that14:38
lucasagomesif the maximum number of retries reached then caputure that exception... as we add more conditions to the clientwrapper I think it's get easier14:38
Shrewslucasagomes: the only other exceptions thrown will be ironicclient ones14:38
NobodyCambrb14:39
lucasagomesShrews, that's true indeed14:39
lucasagomeshmm, will put some though on it14:39
Shrewslucasagomes: just a suggestion. something about it is itching my brain, but i don't have the brainpower to concentrate on it and my other task right now14:39
lucasagomesI may remove that exception then14:39
lucasagomesShrews, heh, yeah I was feeling the same on raising novaexception there14:40
lucasagomesbut I may be wrong as well14:40
*** ilives has quit IRC14:40
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: N-cpu to not die if Ironic is not running  https://review.openstack.org/8859014:47
*** max_lobur has quit IRC15:00
*** viktors has quit IRC15:14
*** killer_prince has quit IRC15:14
openstackgerritVladimir Kozhukalov proposed a change to openstack/ironic-python-agent: Added list and report disk utils  https://review.openstack.org/8860215:32
openstackgerritVladimir Kozhukalov proposed a change to openstack/ironic-python-agent: Added list and report disk utils  https://review.openstack.org/8860215:36
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Overwrite instance_exists in Nova Ironic Driver  https://review.openstack.org/8861115:48
NobodyCamoh nice lucasagomes :)15:50
NobodyCamnice patch15:50
lucasagomesNobodyCam, :D15:50
lucasagomescheers15:51
Shrewshrm...  ERROR (BadRequest): Ephemeral disks requested are larger than the instance type allows. (HTTP 400)15:51
lucasagomesNobodyCam, btw off the top of my head, I dunno what is that neutron error15:51
lucasagomesShrews, nova?15:51
NobodyCamlucasagomes: ack :) ty for looking15:51
Shrewslucasagomes: yes15:51
Shrewsnova boot --image 3accaf6c-6fee-47a2-a35a-b297a51afbfb --flavor baremetal --key-name default --ephemeral size=1 shrews115:52
NobodyCamShrews: is falvor root disk == actual disk size?15:52
NobodyCam*flavor even15:53
*** vkozhukalov has quit IRC15:53
ShrewsNobodyCam: hrm, i think so... if i'm interpreting this output correctly. let me play with that15:55
*** eghobo has joined #openstack-ironic16:01
*** lazy_prince has joined #openstack-ironic16:02
*** lazy_prince is now known as killer_prince16:02
NobodyCamGRRRRR why does it work localy : and not in the gate jobs... http://paste.openstack.org/show/PVvLRzQKXThG8ggRujt416:08
*** hemna_ has quit IRC16:08
ShrewsNobodyCam: because the gate hates you?16:11
NobodyCamheheheh :) /me thinks Shrews may be on to somehting16:12
Shrewshrm, i wonder if i need to set BM_FLAVOR_EPHEMERAL_DISK in devstack to get this to work16:13
NobodyCam*something (could be my poor typing today too)16:13
NobodyCam:-p16:13
ShrewsNobodyCam: oh, i know why. it works on your elitebook b/c you have to run it upside down. ask -infra to flip their vm's over16:16
Shrewsyay friday solutions!16:17
*** shakamunyi has joined #openstack-ironic16:18
ShrewsSo, if anyone knows how to make devstack work with ironic to build vms with ephemeral partitions, please pass that along. In the meantime, lunch...16:26
openstackgerritChris Krelle proposed a change to openstack/ironic: Fix for tripleO undercloud gate tests DO NOT MERGE  https://review.openstack.org/8552916:26
openstackgerritChris Krelle proposed a change to openstack/ironic: Workaround race with Neutron port updates  https://review.openstack.org/8436116:26
NobodyCamShrews: lol16:27
NobodyCamhumm16:27
NobodyCamokay just cast my TC vote16:35
*** comstud is now known as bearhands16:39
*** jistr has quit IRC16:47
*** BadCub01 has joined #openstack-ironic16:51
*** lsmola has quit IRC16:54
*** linggao has quit IRC16:55
*** harlowja_away is now known as harlowja16:58
*** newell has joined #openstack-ironic17:07
devanandag'morning all17:08
NobodyCamgood morning devananda :) TGIF!!!!17:08
* devananda isin the office today17:08
NobodyCamahh free lunch day..lol17:09
* NobodyCam is still on a farm in texas17:09
* devananda starts reviewing all the things17:12
lucasagomesmorning devananda17:16
*** linggao has joined #openstack-ironic17:22
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Overwrite instance_exists in Nova Ironic Driver  https://review.openstack.org/8861117:31
openstackgerritDmitry Tantsur proposed a change to openstack/ironic: Implement caching for master images  https://review.openstack.org/8538717:34
dtantsurlucasagomes, any chances you have a look at this ^^^ (still unfinished though)?17:35
lucasagomesack I'm done for the day... have a good night devananda NobodyCam Shrews dtantsur17:35
lucasagomeshappy easter!17:35
lucasagomesdtantsur, oh, now or can be later on?17:35
dtantsuroh, sorry then, happy easter :)17:35
NobodyCamhappy easter lucasagomes17:35
Shrewslucasagomes: have a great weekend!17:35
NobodyCamlucasagomes: are you in on monday17:35
dtantsurno, not now than, maybe someone else agrees to have a look)17:35
lucasagomesdtantsur, ack, I will take a look later on if that's fine :)17:36
*** romcheg has joined #openstack-ironic17:36
lucasagomesNobodyCam, it's holiday here17:36
dtantsursure, good night :)17:36
lucasagomesNobodyCam, but I will attend the meeting17:36
lucasagomesNobodyCam, unless something happens :)17:37
NobodyCamw00t :)17:37
NobodyCamlol17:37
NobodyCamack17:37
lucasagomes:D17:37
NobodyCamhave a great weekend17:37
lucasagomesenjoy the weekend!17:37
lucasagomesu too17:37
*** lucasagomes is now known as lucas-dinner17:37
dtantsurdevananda, morning! I understand that your review queue is huge already, but I would be thankful if you find time to have a look at https://review.openstack.org/#/c/85387 . It's still WIP, but I want to make sure I am not moving in a completely wrong direction :)17:38
devanandadtantsur: hi! have you gotten feedback from GheRivero on the patch yet?17:40
devanandadtantsur: i dont see any reviews from him, and he's the original author of that code, so i'd like to see what he thinks17:41
*** romcheg has quit IRC17:41
dtantsurdevananda, added Ghe as I reviewer, thank you. Would be great if you can add your overall opinion as well :)17:42
*** vkozhukalov has joined #openstack-ironic17:44
devanandadtantsur: doc strings dont list any :param, :return, or :raises17:44
devanandadtantsur: so, we can't delete the kernel&ramdisk pair for an active instance17:45
devanandadtantsur: no matter how old it is17:45
devanandadtantsur: i dont see any protection in this code against that17:45
devanandathat'll result in any instance older than TTL being un-restart-able17:46
dtantsurdevananda, won't active kernel&ramdisk have link count > 1?17:46
dtantsurdocstings - thank you, will fix17:46
devanandaImages are deleted using the following algorythm:17:46
devananda- All images older than configurable TTL are always deleted17:46
devanandathe commit message indicates link count doesn' tmatter for old images, and the code appears to confirm that17:47
dtantsurdevananda, actually the commit message is wrong: see listing definition on line 11217:48
devanandadtantsur: ah, i see. that's well hidden17:49
dtantsuryep, will add a comment17:49
Shrewsdevananda: have you tested 87396?17:50
*** EmilienM has quit IRC17:50
*** EmilienM has joined #openstack-ironic17:50
*** Hefeweizen has joined #openstack-ironic17:50
devanandaShrews: not in a few days17:51
Shrewsdevananda: i'm seeing this in the ir-api logs: 2014-04-18 17:43:51.425 23132 WARNING wsme.api [-] Client-side error: 'add' and 'replace' operations needs value17:51
*** Mikhail_D_ltp has joined #openstack-ironic17:52
openstackgerritDevananda van der Veen proposed a change to openstack/ironic: nova.virt.ironic passes ephemeral_gb to ironic  https://review.openstack.org/8739617:53
devanandalets see what jenkins says now17:53
Shrewsdriver gets the error in _add_driver_fields() during the client api call. just stabbing around for possible causes17:53
Shrewsdevananda: it doesn't look like devstack is creating vms with ephemeral partitions. once i changed that, i saw this error17:54
dtantsurThanks for feedback, devananda, and good night and happy Easter to everyone :)17:54
* Shrews keeps digging17:55
devanandadtantsur: why is this using both lockutils.synchronized AND a separate lock file?17:55
dtantsurdevananda, well, I was also interesting, but for now just blindly copy-pasted17:55
dtantsurwill think/ask GheRivero about it17:55
*** eghobo has quit IRC17:55
*** foexle has joined #openstack-ironic17:56
*** eghobo has joined #openstack-ironic17:56
dtantsurdevananda, lockutils also work via file, right?17:56
devanandadtantsur: it can, but that's horridly slow and racy17:56
devanandadtantsur: and only needed for inter-process locking17:56
devanandawhich, considering right now everything is greenthreaded in one process, we dont need17:56
dtantsurdevananda, maybe point here is to only lock downloading this file? As you see, fetch_image is not protected by lockutils17:57
dtantsuryeah, no I understand again, I think :) We avoid locking for the whole duration of downloading and lock with lockutils only during file move/link17:58
dtantsurwe use lock file to avoid redownloading in the meanwhile17:58
*** foexle has quit IRC18:01
Shrewsdevananda: ah, found it18:02
Shrewsdevananda: left a comment on the review18:03
devanandaShrews: thanks18:03
Shrewsdevananda: curious as to how you'll resolve that. the config is in ironic, but needed in nova  :/18:13
*** foexle has joined #openstack-ironic18:13
adam_ggah! sooo close http://logs.openstack.org/11/88611/2/check/check-tempest-dsvm-virtual-ironic/201e76e/logs/testr_results.html.gz18:15
* Shrews takes a walk. bbiab18:18
*** foexle has quit IRC18:23
*** romcheg has joined #openstack-ironic18:44
*** tatyana_ has joined #openstack-ironic18:44
*** romcheg has quit IRC18:49
openstackgerritA change was merged to stackforge/pyghmi: Add sensor information from IPMI 2.0 rev 1.1  https://review.openstack.org/8855819:02
*** tatyana_ has quit IRC19:08
openstackgerritlinggao proposed a change to openstack/ironic: Handle API error for XML client  https://review.openstack.org/8844819:10
*** Mikhail_D_ltp has quit IRC19:18
devanandajroll: fyi, i commented (a while back) on http://summit.openstack.org/cfp/details/26 and am awaiting your feedback there19:22
* devananda finishes reviewing all 14 session proposals19:22
NobodyCamand we have 4 slots19:22
devanandayep19:22
devanandaAND those 4 slots overlap with all the cross-project sessions19:23
devanandaso a few of us will need to miss the ironic sessions to go sit in on eg. an oslo session19:23
devanandaor such19:23
NobodyCamyep19:23
NobodyCamand some of will be having to go to TC meetings LOL :)19:24
NobodyCam..some of *US* will..19:24
*** foexle has joined #openstack-ironic19:26
*** foexle has quit IRC19:26
*** shakamunyi has quit IRC19:27
*** shakayumi has joined #openstack-ironic19:27
*** BadCub01 has quit IRC19:32
devanandajroll: huh, just realized the worker pool issue is also addressed here: https://review.openstack.org/#/c/88307/4/ironic/conductor/manager.py19:34
devanandathat patch is huge :(19:34
devanandalucas-dinner: that patch is huge and making LOTS of changes19:35
devanandalucas-dinner: the commit header is so underwhelmingly misleading - it's doing way more than just "port to oslo.messaging" :(19:35
*** tatyana_ has joined #openstack-ironic19:36
lucas-dinnerdevananda, :( yeah... I didn't know hw to break them into diff patches19:45
lucas-dinnerdevananda, I can give it a go, see if I can split it into more, but I don't think it will be many19:45
lucas-dinnerdevananda, nova was the same, cinder as well (https://review.openstack.org/#/c/39929/78, https://review.openstack.org/#/c/71873/)19:46
lucas-dinnerdevananda, I will try to break into smaller parts, can you leave a comment there?19:46
devanandadamn19:46
devanandalucas-dinner: no.. it's probably fine19:46
lucas-dinnerdevananda, heh nova was had #78 patch sets :P19:46
devanandajust going to take me a bit to digest the changes19:46
lucas-dinnerdevananda, ack, yeah I can figure out19:47
devanandahah19:47
lucas-dinnernova one*19:47
devanandalucas-dinner: so in rpcapi, you're no longer passing the topic in to call19:48
devanandait's being set on client.prepare19:48
lucas-dinneryeah19:48
devanandado we know that eventlet _cant_ switch between those?19:48
openstackgerritlinggao proposed a change to openstack/ironic: Modify the get console API  https://review.openstack.org/8776019:48
lucas-dinnerdevananda, hmm not sure19:49
devanandalucas-dinner: also i'm not clear on the reason for the new ironic.common.manager class, which is only used by ironic.conductor.manager19:51
lucas-dinnerdevananda, yeah, I thought about just adding it in the conductor manager, but then I saw that other projects they do have a base class for that19:52
lucas-dinnerso I thought about having one as well, in case we add more managers19:52
devananda"in case we..."19:52
lucas-dinnerbut I could get rid of it, and just have conductor19:52
devanandapremature optimization :)19:52
lucas-dinnerheh19:52
lucas-dinneryeah19:52
NobodyCamlol that sounds dirty19:53
*** lucas-dinner is now known as lucasagomes19:53
devanandaadding comments19:53
lucasagomesdevananda, ack cheers19:53
lucasagomesI knew it wouldn't be a straight forward patch19:53
lucasagomesbut I was a bit lost when implementing it19:53
lucasagomesa lot of try and error19:53
devanandayea... it's really nice work, though19:53
lucasagomesso I really appriciate the feedback, I will polish it more as it goes19:53
devanandajust a lot to digest19:53
lucasagomesack, I'll take a look at the comments, and the patch in general to see if I can simplify some parts19:54
lucasagomesbrb19:57
*** lucasagomes is now known as lucas-afk19:57
openstackgerritlinggao proposed a change to openstack/python-ironicclient: node-get-console incorporate the changes in API  https://review.openstack.org/8776920:04
devanandaNobodyCam: i just replied to http://summit.openstack.org/cfp/details/121 -- can you see if you still feel that topic is worth a session?20:05
* devananda breaks for lunch20:05
* NobodyCam looks20:05
NobodyCamdevananda: thats good for me.. maybe needed if we get a flood of drivers.20:06
*** linggao has quit IRC20:08
adam_ghmm. what are peoples thoughts about tracking instance names as well as uuids somewhere on the ironic side?20:18
adam_gcontext: https://bugs.launchpad.net/ironic/+bug/130971920:18
NobodyCamlike db field or something added to extra field?20:20
adam_gNobodyCam, probably something in the extra field, or driver info?20:27
NobodyCam:)20:28
*** eguz has joined #openstack-ironic20:32
*** eghobo has quit IRC20:35
*** harlowja is now known as harlowja_away20:50
openstackgerritAdam Gandelman proposed a change to openstack/ironic: Implement list_instance_uuids() in Nova driver  https://review.openstack.org/8868420:51
devanandaadam_g: there was a proposal to store human-readable node names in ironic, as a top-level property20:51
devanandaadam_g: but i dont think that addresses that bug at all20:51
devanandaadam_g: which is based on normal hypervisor function wherein each VM has a user-defined name (often just the nova instance name)20:52
devanandaShrews: hi!20:55
*** romcheg has joined #openstack-ironic21:00
*** tatyana_ has quit IRC21:03
openstackgerritChris Krelle proposed a change to openstack/ironic: Fix for tripleO undercloud gate tests DO NOT MERGE  https://review.openstack.org/8552921:11
*** Mikhail_D_ltp has joined #openstack-ironic21:17
Shrewsdevananda: hey21:18
devanandaShrews: replied to your comment on the ephemeral patch21:19
devanandaand rerunning my tests now21:19
Shrewsdevananda: so... yeah, we're saying the same thing  :)21:20
devanandaexcept i'm saying, the patch does that already ;)21:20
devanandaShrews: also it seems to work21:21
Shrewsdevananda: mmm, no. you're having nova tell ironic what it's default is21:21
Shrewsdevananda: oh, wait a sec...21:22
devanandahm, ignore my last msg. it *ran* but something's not right21:22
Shrewsdevananda: ah, i didn't realize that default_ephemeral_format is already defined21:23
devanandaShrews: Nova has a setting. Ironic has a setting. I'm telling nova to send nova's default to ironic. and ironic to use it's default ONLY if it didn't get one from nova21:23
Shrewsdevananda: right, i'm on the same page now21:23
Shrewsbut it definitely didn't work for me21:23
devanandaso it definitely worked for me at one point21:23
devanandabut the DiskPartitioner patch just landed today... maybe theyre not playing well21:23
devanandaoh21:23
devanandanvm21:24
adam_gdevananda, so is the fact that list_intances() returns nodes and not instances intentional?21:24
devanandai forgot to restart nova-compute21:24
Shrewsdevananda: ok, so the problem, then, is that nova's default doesn't have a default21:24
Shrewsso you send None in the patcher21:25
Shrewswhich it doesn't like21:25
Shrews(assuming the config value isn't set by the user)21:25
devanandaadam_g: maybe? or maybe a mistake?21:25
Shrewsdevananda: do you have that option set in your nova.conf?21:27
*** jbjohnso has quit IRC21:30
devanandanope21:30
devanandanow i'm getting that error21:30
Shrewsk21:30
openstackgerritAdam Gandelman proposed a change to openstack/ironic: Decouple state inspection and availability check  https://review.openstack.org/8847621:32
devanandaShrews: i see the problem21:32
devanandatesting fix21:32
Shrewsawesome21:33
Shrewsdevananda: also, how are you testing this?21:34
devanandawith cirros image, login, run sfdisk -l21:35
devananda* edit flavor * nova boot * then login and run sfdisk -l21:35
Shrewsah, ok. you manually edit the flavor21:35
Shrewsi patched devstack to do: nova flavor-create --ephemeral 1 ...21:36
devanandathat's going to break21:37
devanandayou need to lower the root_gb too21:37
Shrewsyeah, wasn't sure about the interaction of all of those values21:37
devanandaby default, devstack is preparing the VM with 10GB virtual disk21:37
devanandaand a 10GB root part + 1 GB ephemeral ....21:37
devanandawell. it'll break21:37
Shrewsi assumed that would be my next error, but i didn't get that far due to this other issue  :)21:38
devanandamysql -uroot nova -e 'update instance_types set root_gb=8, ephemeral_gb=2 where name="baremetal"'21:38
*** romcheg has quit IRC21:39
Shrewsjust need to decrease IRONIC_VM_SPECS_DISK by the ephemeral amount then (when it's done properly)21:39
Shrewsor brute force it like you21:39
NobodyCambbiafm21:39
*** harlowja_away is now known as harlowja21:41
openstackgerritDevananda van der Veen proposed a change to openstack/ironic: nova.virt.ironic passes ephemeral_gb to ironic  https://review.openstack.org/8739621:44
devanandaShrews: thanks for catching that :)21:45
Shrewsdevananda: np. thx for setting me straight on the config values  :)21:46
*** vkozhukalov has quit IRC21:46
*** vkozhukalov has joined #openstack-ironic21:46
*** vkozhukalov has quit IRC21:48
*** romcheg has joined #openstack-ironic21:49
Shrewsdinner (and beer) time. have a good weekend everyone21:51
devanandaShrews: ciao! enjoy!21:51
*** eghobo has joined #openstack-ironic21:59
*** eghobo has quit IRC21:59
*** romcheg has quit IRC22:03
*** eghobo has joined #openstack-ironic22:04
*** eghobo has quit IRC22:05
*** eghobo has joined #openstack-ironic22:05
devanandabearhands: nova instance['name'] -- i'm probably being obtuse. where is this set? it doesn't seem to be a property of the InstanceObject22:31
openstackgerritA change was merged to openstack/ironic: Complete wrapping ironic client calls  https://review.openstack.org/8699323:27
openstackgerritDevananda van der Veen proposed a change to openstack/ironic: Remove 'fake' driver from default enabled drivers  https://review.openstack.org/8871123:34
*** eghobo has quit IRC23:39
« Thursday, 2014-04-17 Index Saturday, 2014-04-19 »

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