*** zdiN0bot has joined #openstack-ironic | 00:04 | |
*** zdiN0bot has quit IRC | 00:09 | |
*** eghobo has joined #openstack-ironic | 00:10 | |
*** rloo has quit IRC | 00:11 | |
*** rloo has joined #openstack-ironic | 00:11 | |
*** ilives has joined #openstack-ironic | 00:13 | |
*** eghobo has quit IRC | 00:16 | |
*** eghobo has joined #openstack-ironic | 00:16 | |
*** ilives has quit IRC | 00:17 | |
*** ilives has joined #openstack-ironic | 00:17 | |
*** rloo has quit IRC | 00:19 | |
devananda | Shrews: 86993 needs a rebase | 00:19 |
---|---|---|
*** rloo has joined #openstack-ironic | 00:19 | |
*** matsuhashi has joined #openstack-ironic | 00:23 | |
*** rloo has quit IRC | 00:24 | |
devananda | ok, i'm off for the night! see ya'll tmw | 00:28 |
*** derekh has quit IRC | 00:31 | |
*** epim has quit IRC | 00:33 | |
openstackgerrit | A change was merged to openstack/ironic: Use DiskPartitioner https://review.openstack.org/83399 | 00:44 |
openstackgerrit | Adam Gandelman proposed a change to openstack/ironic: Decouple state inspection and availability check https://review.openstack.org/88476 | 00:58 |
adam_g | devananda, ^ i think thats a better fix for at least one of those bugs (the nova-compute startup issue) | 00:59 |
*** zdiN0bot has joined #openstack-ironic | 01:20 | |
*** nosnos has joined #openstack-ironic | 01:24 | |
*** zdiN0bot has quit IRC | 01:24 | |
*** zdin0bot has joined #openstack-ironic | 01:32 | |
*** foexle_ has joined #openstack-ironic | 01:47 | |
*** foexle has quit IRC | 01:50 | |
*** rwsu has quit IRC | 02:22 | |
*** harlowja is now known as harlowja_away | 02:49 | |
*** coolsvap|afk is now known as coolsvap | 02:58 | |
*** eghobo has quit IRC | 03:06 | |
*** matsuhashi has quit IRC | 03:09 | |
*** matsuhashi has joined #openstack-ironic | 03:09 | |
*** matsuhashi has quit IRC | 03:09 | |
*** nosnos has quit IRC | 03:19 | |
*** matsuhashi has joined #openstack-ironic | 03:25 | |
*** matsuhashi has quit IRC | 03:27 | |
*** matsuhashi has joined #openstack-ironic | 03:28 | |
*** matsuhashi has quit IRC | 03:31 | |
*** matsuhashi has joined #openstack-ironic | 03:31 | |
*** matsuhas_ has joined #openstack-ironic | 03:34 | |
*** matsuhashi has quit IRC | 03:35 | |
*** lnxnut has quit IRC | 03:37 | |
*** zdin0bot has quit IRC | 03:37 | |
*** lnxnut has joined #openstack-ironic | 03:39 | |
*** matsuhas_ has quit IRC | 03:54 | |
*** matsuhashi has joined #openstack-ironic | 03:57 | |
*** eghobo has joined #openstack-ironic | 03:58 | |
*** nosnos has joined #openstack-ironic | 04:10 | |
*** lnxnut has quit IRC | 04:22 | |
*** zdin0bot has joined #openstack-ironic | 04:32 | |
*** zdin0bot has quit IRC | 04:36 | |
*** coolsvap is now known as coolsvap|afk | 04:39 | |
*** blamar has joined #openstack-ironic | 04:43 | |
*** zdin0bot has joined #openstack-ironic | 04:48 | |
*** Mikhail_D_ltp has joined #openstack-ironic | 04:49 | |
*** jgrimm has joined #openstack-ironic | 04:50 | |
*** vkozhukalov has joined #openstack-ironic | 04:53 | |
*** coolsvap|afk is now known as coolsvap | 04:58 | |
*** coolsvap is now known as coolsvap|afk | 05:05 | |
*** zdin0bot has quit IRC | 05:24 | |
*** zdin0bot has joined #openstack-ironic | 05:26 | |
*** romcheg1 has joined #openstack-ironic | 05:28 | |
*** Mikhail_D_ltp has quit IRC | 05:35 | |
*** pradipta_away is now known as pradipta | 06:00 | |
*** max_lobur has joined #openstack-ironic | 06:06 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/88508 | 06:07 |
*** pradipta is now known as pradipta_away | 06:13 | |
*** vkozhukalov has quit IRC | 06:14 | |
*** coolsvap|afk is now known as coolsvap | 06:18 | |
*** pradipta_away is now known as pradipta | 06:21 | |
*** zdin0bot has quit IRC | 06:43 | |
*** ifarkas has joined #openstack-ironic | 06:45 | |
*** viktors has joined #openstack-ironic | 06:54 | |
*** eghobo has quit IRC | 06:58 | |
*** matsuhashi has quit IRC | 07:06 | |
*** matsuhashi has joined #openstack-ironic | 07:08 | |
openstackgerrit | Yuriy Zveryanskyy proposed a change to openstack/ironic: Add worker threads limit to _check_deploy_timeouts task https://review.openstack.org/78651 | 07:11 |
*** jistr has joined #openstack-ironic | 07:16 | |
*** vkozhukalov has joined #openstack-ironic | 07:40 | |
*** lazy_prince has joined #openstack-ironic | 07:42 | |
*** lazy_prince has quit IRC | 07:46 | |
*** zdin0bot has joined #openstack-ironic | 07:48 | |
*** max_lobur has quit IRC | 07:53 | |
*** jistr has quit IRC | 08:01 | |
*** jistr has joined #openstack-ironic | 08:25 | |
*** lucasagomes has joined #openstack-ironic | 08:27 | |
*** zdin0bot has quit IRC | 08:33 | |
*** lsmola has joined #openstack-ironic | 08:42 | |
*** max_lobur has joined #openstack-ironic | 08:50 | |
*** ndipanov has joined #openstack-ironic | 08:55 | |
*** foexle_ has quit IRC | 09:02 | |
*** martyntaylor has joined #openstack-ironic | 09:29 | |
*** pradipta is now known as pradipta_away | 09:47 | |
*** foexle has joined #openstack-ironic | 10:06 | |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Get rid of the swap partition https://review.openstack.org/83726 | 10:11 |
*** dkehn__ has joined #openstack-ironic | 10:11 | |
*** lsmola has quit IRC | 10:14 | |
*** dkehn_ has quit IRC | 10:14 | |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Use GB instead of MB for swap https://review.openstack.org/83788 | 10:15 |
*** blamar has quit IRC | 10:21 | |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add ManagementInterface https://review.openstack.org/86063 | 10:25 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: IPMITool to use the new ManagementInterface https://review.openstack.org/86092 | 10:25 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: SeaMicro to use the new ManagementInterface https://review.openstack.org/86328 | 10:25 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: IPMINative to use the new ManagementInterface https://review.openstack.org/86588 | 10:25 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: IPMINative set_boot_device persistent https://review.openstack.org/85742 | 10:46 |
*** Mikhail_D_wk has quit IRC | 10:48 | |
*** Mikhail_D_wk has joined #openstack-ironic | 10:50 | |
*** coolsvap is now known as coolsvap|afk | 10:50 | |
dtantsur | Good afternoon, everyone :) | 10:51 |
*** nosnos has quit IRC | 11:01 | |
*** matsuhashi has quit IRC | 11:12 | |
*** matsuhashi has joined #openstack-ironic | 11:14 | |
romcheg1 | Hi dtantsur! | 11:18 |
*** romcheg1 has quit IRC | 11:19 | |
*** nosnos has joined #openstack-ironic | 11:26 | |
*** nosnos has quit IRC | 11:29 | |
*** lsmola has joined #openstack-ironic | 11:30 | |
*** ifarkas has quit IRC | 11:31 | |
*** matsuhashi has quit IRC | 11:32 | |
*** matsuhashi has joined #openstack-ironic | 11:32 | |
*** matsuhashi has quit IRC | 11:38 | |
*** romcheg has quit IRC | 11:40 | |
*** romcheg has joined #openstack-ironic | 11:40 | |
*** lucasagomes is now known as lucas-hungry | 11:46 | |
lucas-hungry | afternoon romcheg1 dtantsur :) | 11:46 |
lucas-hungry | brb lunch time | 11:46 |
openstackgerrit | A change was merged to openstack/python-ironicclient: Adds documentation for ironicclient API https://review.openstack.org/85107 | 11:50 |
dtantsur | lucas-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-ironic | 11:56 | |
*** romcheg has quit IRC | 11:58 | |
openstackgerrit | Jarrod Johnson proposed a change to stackforge/pyghmi: Add sensor information from IPMI 2.0 rev 1.1 https://review.openstack.org/88558 | 11:59 |
*** foexle has quit IRC | 12:01 | |
*** jbjohnso has joined #openstack-ironic | 12:06 | |
*** lnxnut has joined #openstack-ironic | 12:08 | |
*** martyntaylor has left #openstack-ironic | 12:08 | |
*** andreykurilin has joined #openstack-ironic | 12:13 | |
andreykurilin | hi all! can someone review my patch? https://review.openstack.org/#/c/71500/ | 12:14 |
openstackgerrit | Dmitry Tantsur proposed a change to openstack/ironic: Implement caching for master images https://review.openstack.org/85387 | 12:21 |
dtantsur | andreykurilin, will see (though I'm not a core) | 12:22 |
andreykurilin | dtantsur: thanks | 12:25 |
openstackgerrit | David Shrewsbury proposed a change to openstack/ironic: Complete wrapping ironic client calls https://review.openstack.org/86993 | 12:28 |
openstackgerrit | David Shrewsbury proposed a change to openstack/ironic: Complete wrapping ironic client calls https://review.openstack.org/86993 | 12:34 |
openstackgerrit | David Shrewsbury proposed a change to openstack/ironic: Complete wrapping ironic client calls https://review.openstack.org/86993 | 12:37 |
Shrews | *sigh* | 12:37 |
Shrews | much too early and too little caffeine for difficult rebases | 12:38 |
dtantsur | Shrews, it sometimes seems to me that rebasing is one of the hardest parts of our job :) | 12:40 |
*** foexle has joined #openstack-ironic | 12:44 | |
*** dkehn__ is now known as dkehnx | 12:46 | |
*** lucas-hungry is now known as lucasagomes | 12:51 | |
lucasagomes | dtantsur, sure | 12:51 |
lucasagomes | dtantsur, +1 about rebasing heh | 12:51 |
NobodyCam | Good Morning and TGIF Ironic | 13:12 |
lucasagomes | morning NobodyCam :) yay | 13:18 |
Shrews | NobodyCam: morning | 13:18 |
NobodyCam | morning lucasagomes & Shrews :) | 13:20 |
*** linggao has joined #openstack-ironic | 13:21 | |
*** foexle has quit IRC | 13:33 | |
*** Mikhail_D_wk has quit IRC | 13:42 | |
*** Mikhail_D_wk has joined #openstack-ironic | 13:44 | |
*** foexle has joined #openstack-ironic | 13:45 | |
*** blamar has joined #openstack-ironic | 14:02 | |
NobodyCam | anyone up to help me look at a neutron error? | 14:05 |
*** ndipanov has quit IRC | 14:08 | |
openstackgerrit | Andrey Kurilin proposed a change to openstack/python-ironicclient: Sync latest code and reuse exceptions from oslo https://review.openstack.org/71500 | 14:09 |
openstackgerrit | Andrey Kurilin proposed a change to openstack/python-ironicclient: Reuse module `cliutils` from common code https://review.openstack.org/72418 | 14:23 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: N-cpu to not die if Ironic is not running https://review.openstack.org/88590 | 14:26 |
lucasagomes | NobodyCam, what's up? | 14:26 |
*** zdin0bot has joined #openstack-ironic | 14:29 | |
NobodyCam | oh let me get the error | 14:30 |
NobodyCam | xuhaiwei ] | 14:30 |
NobodyCam | http://paste.openstack.org/show/16VqycxIn7eilvdwdGTF/ | 14:30 |
*** zdin0bot has quit IRC | 14:32 | |
Shrews | lucasagomes: why not just catch NovaException instead of re-introducing the MaximumRetriesReached exception in 88590? | 14:33 |
*** foexle has quit IRC | 14:37 | |
lucasagomes | Shrews, hey hmm I thought about it, but sounds too generic | 14:37 |
lucasagomes | I sometimes prefer to have a very specific exception for cases like that | 14:38 |
lucasagomes | if the maximum number of retries reached then caputure that exception... as we add more conditions to the clientwrapper I think it's get easier | 14:38 |
Shrews | lucasagomes: the only other exceptions thrown will be ironicclient ones | 14:38 |
NobodyCam | brb | 14:39 |
lucasagomes | Shrews, that's true indeed | 14:39 |
lucasagomes | hmm, will put some though on it | 14:39 |
Shrews | lucasagomes: 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 now | 14:39 |
lucasagomes | I may remove that exception then | 14:39 |
lucasagomes | Shrews, heh, yeah I was feeling the same on raising novaexception there | 14:40 |
lucasagomes | but I may be wrong as well | 14:40 |
*** ilives has quit IRC | 14:40 | |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: N-cpu to not die if Ironic is not running https://review.openstack.org/88590 | 14:47 |
*** max_lobur has quit IRC | 15:00 | |
*** viktors has quit IRC | 15:14 | |
*** killer_prince has quit IRC | 15:14 | |
openstackgerrit | Vladimir Kozhukalov proposed a change to openstack/ironic-python-agent: Added list and report disk utils https://review.openstack.org/88602 | 15:32 |
openstackgerrit | Vladimir Kozhukalov proposed a change to openstack/ironic-python-agent: Added list and report disk utils https://review.openstack.org/88602 | 15:36 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Overwrite instance_exists in Nova Ironic Driver https://review.openstack.org/88611 | 15:48 |
NobodyCam | oh nice lucasagomes :) | 15:50 |
NobodyCam | nice patch | 15:50 |
lucasagomes | NobodyCam, :D | 15:50 |
lucasagomes | cheers | 15:51 |
Shrews | hrm... ERROR (BadRequest): Ephemeral disks requested are larger than the instance type allows. (HTTP 400) | 15:51 |
lucasagomes | NobodyCam, btw off the top of my head, I dunno what is that neutron error | 15:51 |
lucasagomes | Shrews, nova? | 15:51 |
NobodyCam | lucasagomes: ack :) ty for looking | 15:51 |
Shrews | lucasagomes: yes | 15:51 |
Shrews | nova boot --image 3accaf6c-6fee-47a2-a35a-b297a51afbfb --flavor baremetal --key-name default --ephemeral size=1 shrews1 | 15:52 |
NobodyCam | Shrews: is falvor root disk == actual disk size? | 15:52 |
NobodyCam | *flavor even | 15:53 |
*** vkozhukalov has quit IRC | 15:53 | |
Shrews | NobodyCam: hrm, i think so... if i'm interpreting this output correctly. let me play with that | 15:55 |
*** eghobo has joined #openstack-ironic | 16:01 | |
*** lazy_prince has joined #openstack-ironic | 16:02 | |
*** lazy_prince is now known as killer_prince | 16:02 | |
NobodyCam | GRRRRR why does it work localy : and not in the gate jobs... http://paste.openstack.org/show/PVvLRzQKXThG8ggRujt4 | 16:08 |
*** hemna_ has quit IRC | 16:08 | |
Shrews | NobodyCam: because the gate hates you? | 16:11 |
NobodyCam | heheheh :) /me thinks Shrews may be on to somehting | 16:12 |
Shrews | hrm, i wonder if i need to set BM_FLAVOR_EPHEMERAL_DISK in devstack to get this to work | 16:13 |
NobodyCam | *something (could be my poor typing today too) | 16:13 |
NobodyCam | :-p | 16:13 |
Shrews | NobodyCam: 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 over | 16:16 |
Shrews | yay friday solutions! | 16:17 |
*** shakamunyi has joined #openstack-ironic | 16:18 | |
Shrews | So, 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 |
openstackgerrit | Chris Krelle proposed a change to openstack/ironic: Fix for tripleO undercloud gate tests DO NOT MERGE https://review.openstack.org/85529 | 16:26 |
openstackgerrit | Chris Krelle proposed a change to openstack/ironic: Workaround race with Neutron port updates https://review.openstack.org/84361 | 16:26 |
NobodyCam | Shrews: lol | 16:27 |
NobodyCam | humm | 16:27 |
NobodyCam | okay just cast my TC vote | 16:35 |
*** comstud is now known as bearhands | 16:39 | |
*** jistr has quit IRC | 16:47 | |
*** BadCub01 has joined #openstack-ironic | 16:51 | |
*** lsmola has quit IRC | 16:54 | |
*** linggao has quit IRC | 16:55 | |
*** harlowja_away is now known as harlowja | 16:58 | |
*** newell has joined #openstack-ironic | 17:07 | |
devananda | g'morning all | 17:08 |
NobodyCam | good morning devananda :) TGIF!!!! | 17:08 |
* devananda isin the office today | 17:08 | |
NobodyCam | ahh free lunch day..lol | 17:09 |
* NobodyCam is still on a farm in texas | 17:09 | |
* devananda starts reviewing all the things | 17:12 | |
lucasagomes | morning devananda | 17:16 |
*** linggao has joined #openstack-ironic | 17:22 | |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Overwrite instance_exists in Nova Ironic Driver https://review.openstack.org/88611 | 17:31 |
openstackgerrit | Dmitry Tantsur proposed a change to openstack/ironic: Implement caching for master images https://review.openstack.org/85387 | 17:34 |
dtantsur | lucasagomes, any chances you have a look at this ^^^ (still unfinished though)? | 17:35 |
lucasagomes | ack I'm done for the day... have a good night devananda NobodyCam Shrews dtantsur | 17:35 |
lucasagomes | happy easter! | 17:35 |
lucasagomes | dtantsur, oh, now or can be later on? | 17:35 |
dtantsur | oh, sorry then, happy easter :) | 17:35 |
NobodyCam | happy easter lucasagomes | 17:35 |
Shrews | lucasagomes: have a great weekend! | 17:35 |
NobodyCam | lucasagomes: are you in on monday | 17:35 |
dtantsur | no, not now than, maybe someone else agrees to have a look) | 17:35 |
lucasagomes | dtantsur, ack, I will take a look later on if that's fine :) | 17:36 |
*** romcheg has joined #openstack-ironic | 17:36 | |
lucasagomes | NobodyCam, it's holiday here | 17:36 |
dtantsur | sure, good night :) | 17:36 |
lucasagomes | NobodyCam, but I will attend the meeting | 17:36 |
lucasagomes | NobodyCam, unless something happens :) | 17:37 |
NobodyCam | w00t :) | 17:37 |
NobodyCam | lol | 17:37 |
NobodyCam | ack | 17:37 |
lucasagomes | :D | 17:37 |
NobodyCam | have a great weekend | 17:37 |
lucasagomes | enjoy the weekend! | 17:37 |
lucasagomes | u too | 17:37 |
*** lucasagomes is now known as lucas-dinner | 17:37 | |
dtantsur | devananda, 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 |
devananda | dtantsur: hi! have you gotten feedback from GheRivero on the patch yet? | 17:40 |
devananda | dtantsur: i dont see any reviews from him, and he's the original author of that code, so i'd like to see what he thinks | 17:41 |
*** romcheg has quit IRC | 17:41 | |
dtantsur | devananda, 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-ironic | 17:44 | |
devananda | dtantsur: doc strings dont list any :param, :return, or :raises | 17:44 |
devananda | dtantsur: so, we can't delete the kernel&ramdisk pair for an active instance | 17:45 |
devananda | dtantsur: no matter how old it is | 17:45 |
devananda | dtantsur: i dont see any protection in this code against that | 17:45 |
devananda | that'll result in any instance older than TTL being un-restart-able | 17:46 |
dtantsur | devananda, won't active kernel&ramdisk have link count > 1? | 17:46 |
dtantsur | docstings - thank you, will fix | 17:46 |
devananda | Images are deleted using the following algorythm: | 17:46 |
devananda | - All images older than configurable TTL are always deleted | 17:46 |
devananda | the commit message indicates link count doesn' tmatter for old images, and the code appears to confirm that | 17:47 |
dtantsur | devananda, actually the commit message is wrong: see listing definition on line 112 | 17:48 |
devananda | dtantsur: ah, i see. that's well hidden | 17:49 |
dtantsur | yep, will add a comment | 17:49 |
Shrews | devananda: have you tested 87396? | 17:50 |
*** EmilienM has quit IRC | 17:50 | |
*** EmilienM has joined #openstack-ironic | 17:50 | |
*** Hefeweizen has joined #openstack-ironic | 17:50 | |
devananda | Shrews: not in a few days | 17:51 |
Shrews | devananda: 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 value | 17:51 |
*** Mikhail_D_ltp has joined #openstack-ironic | 17:52 | |
openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: nova.virt.ironic passes ephemeral_gb to ironic https://review.openstack.org/87396 | 17:53 |
devananda | lets see what jenkins says now | 17:53 |
Shrews | driver gets the error in _add_driver_fields() during the client api call. just stabbing around for possible causes | 17:53 |
Shrews | devananda: it doesn't look like devstack is creating vms with ephemeral partitions. once i changed that, i saw this error | 17:54 |
dtantsur | Thanks for feedback, devananda, and good night and happy Easter to everyone :) | 17:54 |
* Shrews keeps digging | 17:55 | |
devananda | dtantsur: why is this using both lockutils.synchronized AND a separate lock file? | 17:55 |
dtantsur | devananda, well, I was also interesting, but for now just blindly copy-pasted | 17:55 |
dtantsur | will think/ask GheRivero about it | 17:55 |
*** eghobo has quit IRC | 17:55 | |
*** foexle has joined #openstack-ironic | 17:56 | |
*** eghobo has joined #openstack-ironic | 17:56 | |
dtantsur | devananda, lockutils also work via file, right? | 17:56 |
devananda | dtantsur: it can, but that's horridly slow and racy | 17:56 |
devananda | dtantsur: and only needed for inter-process locking | 17:56 |
devananda | which, considering right now everything is greenthreaded in one process, we dont need | 17:56 |
dtantsur | devananda, maybe point here is to only lock downloading this file? As you see, fetch_image is not protected by lockutils | 17:57 |
dtantsur | yeah, no I understand again, I think :) We avoid locking for the whole duration of downloading and lock with lockutils only during file move/link | 17:58 |
dtantsur | we use lock file to avoid redownloading in the meanwhile | 17:58 |
*** foexle has quit IRC | 18:01 | |
Shrews | devananda: ah, found it | 18:02 |
Shrews | devananda: left a comment on the review | 18:03 |
devananda | Shrews: thanks | 18:03 |
Shrews | devananda: curious as to how you'll resolve that. the config is in ironic, but needed in nova :/ | 18:13 |
*** foexle has joined #openstack-ironic | 18:13 | |
adam_g | gah! sooo close http://logs.openstack.org/11/88611/2/check/check-tempest-dsvm-virtual-ironic/201e76e/logs/testr_results.html.gz | 18:15 |
* Shrews takes a walk. bbiab | 18:18 | |
*** foexle has quit IRC | 18:23 | |
*** romcheg has joined #openstack-ironic | 18:44 | |
*** tatyana_ has joined #openstack-ironic | 18:44 | |
*** romcheg has quit IRC | 18:49 | |
openstackgerrit | A change was merged to stackforge/pyghmi: Add sensor information from IPMI 2.0 rev 1.1 https://review.openstack.org/88558 | 19:02 |
*** tatyana_ has quit IRC | 19:08 | |
openstackgerrit | linggao proposed a change to openstack/ironic: Handle API error for XML client https://review.openstack.org/88448 | 19:10 |
*** Mikhail_D_ltp has quit IRC | 19:18 | |
devananda | jroll: fyi, i commented (a while back) on http://summit.openstack.org/cfp/details/26 and am awaiting your feedback there | 19:22 |
* devananda finishes reviewing all 14 session proposals | 19:22 | |
NobodyCam | and we have 4 slots | 19:22 |
devananda | yep | 19:22 |
devananda | AND those 4 slots overlap with all the cross-project sessions | 19:23 |
devananda | so a few of us will need to miss the ironic sessions to go sit in on eg. an oslo session | 19:23 |
devananda | or such | 19:23 |
NobodyCam | yep | 19:23 |
NobodyCam | and some of will be having to go to TC meetings LOL :) | 19:24 |
NobodyCam | ..some of *US* will.. | 19:24 |
*** foexle has joined #openstack-ironic | 19:26 | |
*** foexle has quit IRC | 19:26 | |
*** shakamunyi has quit IRC | 19:27 | |
*** shakayumi has joined #openstack-ironic | 19:27 | |
*** BadCub01 has quit IRC | 19:32 | |
devananda | jroll: huh, just realized the worker pool issue is also addressed here: https://review.openstack.org/#/c/88307/4/ironic/conductor/manager.py | 19:34 |
devananda | that patch is huge :( | 19:34 |
devananda | lucas-dinner: that patch is huge and making LOTS of changes | 19:35 |
devananda | lucas-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-ironic | 19:36 | |
lucas-dinner | devananda, :( yeah... I didn't know hw to break them into diff patches | 19:45 |
lucas-dinner | devananda, I can give it a go, see if I can split it into more, but I don't think it will be many | 19:45 |
lucas-dinner | devananda, nova was the same, cinder as well (https://review.openstack.org/#/c/39929/78, https://review.openstack.org/#/c/71873/) | 19:46 |
lucas-dinner | devananda, I will try to break into smaller parts, can you leave a comment there? | 19:46 |
devananda | damn | 19:46 |
devananda | lucas-dinner: no.. it's probably fine | 19:46 |
lucas-dinner | devananda, heh nova was had #78 patch sets :P | 19:46 |
devananda | just going to take me a bit to digest the changes | 19:46 |
lucas-dinner | devananda, ack, yeah I can figure out | 19:47 |
devananda | hah | 19:47 |
lucas-dinner | nova one* | 19:47 |
devananda | lucas-dinner: so in rpcapi, you're no longer passing the topic in to call | 19:48 |
devananda | it's being set on client.prepare | 19:48 |
lucas-dinner | yeah | 19:48 |
devananda | do we know that eventlet _cant_ switch between those? | 19:48 |
openstackgerrit | linggao proposed a change to openstack/ironic: Modify the get console API https://review.openstack.org/87760 | 19:48 |
lucas-dinner | devananda, hmm not sure | 19:49 |
devananda | lucas-dinner: also i'm not clear on the reason for the new ironic.common.manager class, which is only used by ironic.conductor.manager | 19:51 |
lucas-dinner | devananda, 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 that | 19:52 |
lucas-dinner | so I thought about having one as well, in case we add more managers | 19:52 |
devananda | "in case we..." | 19:52 |
lucas-dinner | but I could get rid of it, and just have conductor | 19:52 |
devananda | premature optimization :) | 19:52 |
lucas-dinner | heh | 19:52 |
lucas-dinner | yeah | 19:52 |
NobodyCam | lol that sounds dirty | 19:53 |
*** lucas-dinner is now known as lucasagomes | 19:53 | |
devananda | adding comments | 19:53 |
lucasagomes | devananda, ack cheers | 19:53 |
lucasagomes | I knew it wouldn't be a straight forward patch | 19:53 |
lucasagomes | but I was a bit lost when implementing it | 19:53 |
lucasagomes | a lot of try and error | 19:53 |
devananda | yea... it's really nice work, though | 19:53 |
lucasagomes | so I really appriciate the feedback, I will polish it more as it goes | 19:53 |
devananda | just a lot to digest | 19:53 |
lucasagomes | ack, I'll take a look at the comments, and the patch in general to see if I can simplify some parts | 19:54 |
lucasagomes | brb | 19:57 |
*** lucasagomes is now known as lucas-afk | 19:57 | |
openstackgerrit | linggao proposed a change to openstack/python-ironicclient: node-get-console incorporate the changes in API https://review.openstack.org/87769 | 20:04 |
devananda | NobodyCam: 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 lunch | 20:05 | |
* NobodyCam looks | 20:05 | |
NobodyCam | devananda: thats good for me.. maybe needed if we get a flood of drivers. | 20:06 |
*** linggao has quit IRC | 20:08 | |
adam_g | hmm. what are peoples thoughts about tracking instance names as well as uuids somewhere on the ironic side? | 20:18 |
adam_g | context: https://bugs.launchpad.net/ironic/+bug/1309719 | 20:18 |
NobodyCam | like db field or something added to extra field? | 20:20 |
adam_g | NobodyCam, probably something in the extra field, or driver info? | 20:27 |
NobodyCam | :) | 20:28 |
*** eguz has joined #openstack-ironic | 20:32 | |
*** eghobo has quit IRC | 20:35 | |
*** harlowja is now known as harlowja_away | 20:50 | |
openstackgerrit | Adam Gandelman proposed a change to openstack/ironic: Implement list_instance_uuids() in Nova driver https://review.openstack.org/88684 | 20:51 |
devananda | adam_g: there was a proposal to store human-readable node names in ironic, as a top-level property | 20:51 |
devananda | adam_g: but i dont think that addresses that bug at all | 20:51 |
devananda | adam_g: which is based on normal hypervisor function wherein each VM has a user-defined name (often just the nova instance name) | 20:52 |
devananda | Shrews: hi! | 20:55 |
*** romcheg has joined #openstack-ironic | 21:00 | |
*** tatyana_ has quit IRC | 21:03 | |
openstackgerrit | Chris Krelle proposed a change to openstack/ironic: Fix for tripleO undercloud gate tests DO NOT MERGE https://review.openstack.org/85529 | 21:11 |
*** Mikhail_D_ltp has joined #openstack-ironic | 21:17 | |
Shrews | devananda: hey | 21:18 |
devananda | Shrews: replied to your comment on the ephemeral patch | 21:19 |
devananda | and rerunning my tests now | 21:19 |
Shrews | devananda: so... yeah, we're saying the same thing :) | 21:20 |
devananda | except i'm saying, the patch does that already ;) | 21:20 |
devananda | Shrews: also it seems to work | 21:21 |
Shrews | devananda: mmm, no. you're having nova tell ironic what it's default is | 21:21 |
Shrews | devananda: oh, wait a sec... | 21:22 |
devananda | hm, ignore my last msg. it *ran* but something's not right | 21:22 |
Shrews | devananda: ah, i didn't realize that default_ephemeral_format is already defined | 21:23 |
devananda | Shrews: 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 nova | 21:23 |
Shrews | devananda: right, i'm on the same page now | 21:23 |
Shrews | but it definitely didn't work for me | 21:23 |
devananda | so it definitely worked for me at one point | 21:23 |
devananda | but the DiskPartitioner patch just landed today... maybe theyre not playing well | 21:23 |
devananda | oh | 21:23 |
devananda | nvm | 21:24 |
adam_g | devananda, so is the fact that list_intances() returns nodes and not instances intentional? | 21:24 |
devananda | i forgot to restart nova-compute | 21:24 |
Shrews | devananda: ok, so the problem, then, is that nova's default doesn't have a default | 21:24 |
Shrews | so you send None in the patcher | 21:25 |
Shrews | which it doesn't like | 21:25 |
Shrews | (assuming the config value isn't set by the user) | 21:25 |
devananda | adam_g: maybe? or maybe a mistake? | 21:25 |
Shrews | devananda: do you have that option set in your nova.conf? | 21:27 |
*** jbjohnso has quit IRC | 21:30 | |
devananda | nope | 21:30 |
devananda | now i'm getting that error | 21:30 |
Shrews | k | 21:30 |
openstackgerrit | Adam Gandelman proposed a change to openstack/ironic: Decouple state inspection and availability check https://review.openstack.org/88476 | 21:32 |
devananda | Shrews: i see the problem | 21:32 |
devananda | testing fix | 21:32 |
Shrews | awesome | 21:33 |
Shrews | devananda: also, how are you testing this? | 21:34 |
devananda | with cirros image, login, run sfdisk -l | 21:35 |
devananda | * edit flavor * nova boot * then login and run sfdisk -l | 21:35 |
Shrews | ah, ok. you manually edit the flavor | 21:35 |
Shrews | i patched devstack to do: nova flavor-create --ephemeral 1 ... | 21:36 |
devananda | that's going to break | 21:37 |
devananda | you need to lower the root_gb too | 21:37 |
Shrews | yeah, wasn't sure about the interaction of all of those values | 21:37 |
devananda | by default, devstack is preparing the VM with 10GB virtual disk | 21:37 |
devananda | and a 10GB root part + 1 GB ephemeral .... | 21:37 |
devananda | well. it'll break | 21:37 |
Shrews | i assumed that would be my next error, but i didn't get that far due to this other issue :) | 21:38 |
devananda | mysql -uroot nova -e 'update instance_types set root_gb=8, ephemeral_gb=2 where name="baremetal"' | 21:38 |
*** romcheg has quit IRC | 21:39 | |
Shrews | just need to decrease IRONIC_VM_SPECS_DISK by the ephemeral amount then (when it's done properly) | 21:39 |
Shrews | or brute force it like you | 21:39 |
NobodyCam | bbiafm | 21:39 |
*** harlowja_away is now known as harlowja | 21:41 | |
openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: nova.virt.ironic passes ephemeral_gb to ironic https://review.openstack.org/87396 | 21:44 |
devananda | Shrews: thanks for catching that :) | 21:45 |
Shrews | devananda: np. thx for setting me straight on the config values :) | 21:46 |
*** vkozhukalov has quit IRC | 21:46 | |
*** vkozhukalov has joined #openstack-ironic | 21:46 | |
*** vkozhukalov has quit IRC | 21:48 | |
*** romcheg has joined #openstack-ironic | 21:49 | |
Shrews | dinner (and beer) time. have a good weekend everyone | 21:51 |
devananda | Shrews: ciao! enjoy! | 21:51 |
*** eghobo has joined #openstack-ironic | 21:59 | |
*** eghobo has quit IRC | 21:59 | |
*** romcheg has quit IRC | 22:03 | |
*** eghobo has joined #openstack-ironic | 22:04 | |
*** eghobo has quit IRC | 22:05 | |
*** eghobo has joined #openstack-ironic | 22:05 | |
devananda | bearhands: nova instance['name'] -- i'm probably being obtuse. where is this set? it doesn't seem to be a property of the InstanceObject | 22:31 |
openstackgerrit | A change was merged to openstack/ironic: Complete wrapping ironic client calls https://review.openstack.org/86993 | 23:27 |
openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Remove 'fake' driver from default enabled drivers https://review.openstack.org/88711 | 23:34 |
*** eghobo has quit IRC | 23:39 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!