| dims_ | mikal: sure. i was also thinking about the callback vs having the kill functionality in the method itself | 00:00 |
|---|---|---|
| dims_ | we can debate that on the review | 00:00 |
| mikal | Sure, works for me | 00:01 |
| dims_ | hope you feel better soon mikal | 00:01 |
| *** alexpilotti has quit IRC | 00:03 | |
| *** tjones1 has quit IRC | 00:04 | |
| *** alex_klimov has quit IRC | 00:04 | |
| *** otter768 has joined #openstack-nova | 00:04 | |
| *** alexpilotti has joined #openstack-nova | 00:05 | |
| *** alexpilotti has quit IRC | 00:08 | |
| *** otter768 has quit IRC | 00:09 | |
| *** hemna is now known as hemnafkj | 00:09 | |
| *** tpatil has quit IRC | 00:15 | |
| *** jaypipes has quit IRC | 00:27 | |
| *** iamjarvo has joined #openstack-nova | 00:28 | |
| *** iamjarvo has quit IRC | 00:28 | |
| *** iamjarvo has joined #openstack-nova | 00:29 | |
| *** iamjarvo has quit IRC | 00:29 | |
| *** iamjarvo has joined #openstack-nova | 00:29 | |
| *** iamjarvo has quit IRC | 00:30 | |
| *** iamjarvo has joined #openstack-nova | 00:31 | |
| *** iamjarvo has quit IRC | 00:31 | |
| *** iamjarvo has joined #openstack-nova | 00:32 | |
| *** vilobhmm1 has quit IRC | 00:40 | |
| *** ssurana has quit IRC | 00:49 | |
| *** ssurana has joined #openstack-nova | 00:49 | |
| *** xiaohui has joined #openstack-nova | 00:51 | |
| *** promulo has joined #openstack-nova | 00:51 | |
| *** takedakn has joined #openstack-nova | 00:53 | |
| *** ssurana has quit IRC | 00:53 | |
| *** takedakn has quit IRC | 00:54 | |
| *** takedakn has joined #openstack-nova | 00:54 | |
| *** promulo__ has quit IRC | 00:56 | |
| *** Sukhdev has quit IRC | 00:59 | |
| *** mwagner_afk has joined #openstack-nova | 01:00 | |
| *** armax has quit IRC | 01:00 | |
| *** zzzeek has quit IRC | 01:04 | |
| *** takedakn has quit IRC | 01:06 | |
| *** takedakn has joined #openstack-nova | 01:06 | |
| mriedem | dims_: turns out those _quota_reserve tests are all busted up | 01:11 |
| mriedem | after fixing them with the .hard_limit thing | 01:11 |
| mriedem | still working on it | 01:11 |
| mriedem | probably won't be ready until tomorrow though | 01:11 |
| dims_ | mriedem: ha! | 01:11 |
| dims_ | mriedem: not in a rush, glad we found it | 01:11 |
| *** pixelbeat_ has joined #openstack-nova | 01:14 | |
| *** openstack has joined #openstack-nova | 01:22 | |
| *** dan_ is now known as Guest32428 | 01:22 | |
| *** Kennan has joined #openstack-nova | 01:22 | |
| *** openstack has joined #openstack-nova | 01:37 | |
| *** iamjarvo has quit IRC | 01:42 | |
| *** mohitsharma has quit IRC | 01:42 | |
| *** yamahata has quit IRC | 01:46 | |
| *** bzhao has quit IRC | 01:46 | |
| lan | hi ndipanov | 01:47 |
| *** boris-42 has quit IRC | 01:48 | |
| *** openstack has quit IRC | 01:52 | |
| *** openstack has joined #openstack-nova | 01:54 | |
| *** sorrison_laptop has joined #openstack-nova | 01:54 | |
| *** ianbrown has quit IRC | 01:57 | |
| *** ian_brown has joined #openstack-nova | 01:57 | |
| *** iamjarvo has joined #openstack-nova | 02:00 | |
| *** iamjarvo has quit IRC | 02:00 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Fix _quota_reserve test setup for incompatible type checking https://review.openstack.org/188231 | 02:00 |
| mriedem_away | dims_: ^ got it | 02:00 |
| *** iamjarvo has joined #openstack-nova | 02:00 | |
| *** iamjarvo has quit IRC | 02:00 | |
| *** packet has quit IRC | 02:01 | |
| *** iamjarvo has joined #openstack-nova | 02:01 | |
| *** mohitsharma has joined #openstack-nova | 02:04 | |
| *** packet has joined #openstack-nova | 02:05 | |
| *** otter768 has joined #openstack-nova | 02:05 | |
| *** otter768 has quit IRC | 02:10 | |
| openstackgerrit | Eli Qiao proposed openstack/nova: API: Add test case to verify policy.json https://review.openstack.org/188235 | 02:13 |
| *** bzhao has joined #openstack-nova | 02:18 | |
| openstackgerrit | Lan Qi Song proposed openstack/nova: Remove useless volume when boot from volume failed https://review.openstack.org/177084 | 02:19 |
| *** nelsnelson has quit IRC | 02:21 | |
| *** unicell has quit IRC | 02:23 | |
| *** patrickeast has quit IRC | 02:25 | |
| *** nelsnelson has joined #openstack-nova | 02:35 | |
| *** gcb has quit IRC | 02:35 | |
| *** jyuso1 has joined #openstack-nova | 02:36 | |
| *** iamjarvo_ has joined #openstack-nova | 02:37 | |
| *** iamjarvo_ has quit IRC | 02:37 | |
| *** iamjarvo_ has joined #openstack-nova | 02:37 | |
| *** mriedem_away has quit IRC | 02:39 | |
| *** iamjarvo has quit IRC | 02:40 | |
| *** iamjarvo_ has quit IRC | 02:46 | |
| *** nelsnelson has quit IRC | 02:48 | |
| *** adrian_otto has joined #openstack-nova | 02:57 | |
| *** adrian_otto has quit IRC | 02:58 | |
| *** yamahata has joined #openstack-nova | 03:00 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Adding more py34 test https://review.openstack.org/188243 | 03:01 |
| openstackgerrit | Josh Durgin proposed openstack/nova-specs: RBD instance snapshots https://review.openstack.org/188244 | 03:02 |
| *** Jeffrey4l has quit IRC | 03:03 | |
| *** alanf-mc has quit IRC | 03:04 | |
| *** dims_ has quit IRC | 03:07 | |
| *** pixelbeat_ has quit IRC | 03:09 | |
| *** edleafe has quit IRC | 03:10 | |
| *** edleafe has joined #openstack-nova | 03:12 | |
| *** ssurana has joined #openstack-nova | 03:16 | |
| *** ssurana has quit IRC | 03:20 | |
| *** Jeffrey4l has joined #openstack-nova | 03:22 | |
| *** markvoelker has quit IRC | 03:23 | |
| *** takedakn has quit IRC | 03:28 | |
| *** rfolco has quit IRC | 03:30 | |
| *** alanf-mc has joined #openstack-nova | 03:34 | |
| *** Nastooh_ has quit IRC | 03:37 | |
| *** garyk has quit IRC | 03:38 | |
| *** coolsvap|afk is now known as coolsvap | 03:39 | |
| *** hemna has joined #openstack-nova | 03:40 | |
| openstackgerrit | Sean McCully proposed openstack/nova: Creates disk.fallocate_test https://review.openstack.org/188249 | 03:43 |
| *** gcb has joined #openstack-nova | 03:49 | |
| *** tobe has joined #openstack-nova | 03:51 | |
| *** EmilienM|off has quit IRC | 03:55 | |
| *** dave-mccowan has quit IRC | 03:57 | |
| *** EmilienM has joined #openstack-nova | 04:00 | |
| *** hemna has quit IRC | 04:02 | |
| *** Jeffrey4l has quit IRC | 04:02 | |
| *** adrian_otto has joined #openstack-nova | 04:03 | |
| *** pradipta has joined #openstack-nova | 04:04 | |
| *** deepthi has joined #openstack-nova | 04:05 | |
| *** sdake has joined #openstack-nova | 04:05 | |
| *** anthonyper has quit IRC | 04:05 | |
| *** anthonyper has joined #openstack-nova | 04:05 | |
| *** otter768 has joined #openstack-nova | 04:06 | |
| *** dims_ has joined #openstack-nova | 04:07 | |
| *** unicell has joined #openstack-nova | 04:10 | |
| *** garyk has joined #openstack-nova | 04:11 | |
| *** otter768 has quit IRC | 04:11 | |
| *** dims_ has quit IRC | 04:13 | |
| *** Longgeek has joined #openstack-nova | 04:14 | |
| *** Jeffrey4l has joined #openstack-nova | 04:15 | |
| openstackgerrit | Tomoki Sekiyama proposed openstack/nova: libvirt: Make behavior of os_require_quiesce consistent https://review.openstack.org/167819 | 04:20 |
| *** mordred has quit IRC | 04:21 | |
| *** gilliard has quit IRC | 04:22 | |
| *** timello_ has quit IRC | 04:22 | |
| *** adrian_otto has quit IRC | 04:22 | |
| *** markvoelker has joined #openstack-nova | 04:23 | |
| *** mordred has joined #openstack-nova | 04:25 | |
| *** timello has joined #openstack-nova | 04:27 | |
| *** markvoelker has quit IRC | 04:28 | |
| *** sorrison_laptop has quit IRC | 04:29 | |
| *** unicell has quit IRC | 04:30 | |
| *** irenab has quit IRC | 04:31 | |
| *** gabor-t has joined #openstack-nova | 04:31 | |
| *** unicell has joined #openstack-nova | 04:32 | |
| *** gilliard has joined #openstack-nova | 04:34 | |
| *** sdake has quit IRC | 04:34 | |
| *** anthonyper has quit IRC | 04:34 | |
| *** anthonyper has joined #openstack-nova | 04:34 | |
| *** Longgeek has quit IRC | 04:39 | |
| *** rushiagr_away is now known as rushiagr | 04:42 | |
| *** bzhao has quit IRC | 04:43 | |
| *** bzhao has joined #openstack-nova | 04:43 | |
| *** sorrison_laptop has joined #openstack-nova | 04:50 | |
| *** Longgeek has joined #openstack-nova | 04:53 | |
| *** hshiina has joined #openstack-nova | 04:54 | |
| *** adrian_otto has joined #openstack-nova | 04:57 | |
| *** Longgeek_ has joined #openstack-nova | 04:57 | |
| *** kiran-r has joined #openstack-nova | 05:00 | |
| *** Longgeek has quit IRC | 05:00 | |
| gabor-t | Hi alex_xu, I saw you work on an improved API Microversion bump guideline (https://review.openstack.org//#/c/177778). In patchset 9 API consistency improvements was highlighted as something that should not be done with microversioning. In the light of that, I would like to know what do you think about https://bugs.launchpad.net/nova/+bug/1217026? My idea was to implement it using microversions, but now I am not sure. | 05:01 |
| openstack | Launchpad bug 1217026 in OpenStack Compute (nova) "Nova os-services:update should be requested by service id to respect REST principles" [Wishlist,In progress] - Assigned to Gábor (gabor-t) | 05:01 |
| openstackgerrit | ZhaoBo proposed openstack/nova: Fixed raise SecurityGroupCannotBeApplied as network port_security off https://review.openstack.org/188260 | 05:08 |
| *** sudipto has joined #openstack-nova | 05:08 | |
| *** tsekiyama has quit IRC | 05:10 | |
| *** alanf-mc has quit IRC | 05:19 | |
| *** alanf-mc has joined #openstack-nova | 05:20 | |
| jogo | mikal: want to review https://review.openstack.org/#/c/184360/2 ? | 05:21 |
| * jogo wants brick | 05:21 | |
| *** tobe has quit IRC | 05:22 | |
| openstackgerrit | melanie witt proposed openstack/nova: Raise InstanceNotFound when save FK constraint fails https://review.openstack.org/188261 | 05:24 |
| *** sudipto has quit IRC | 05:29 | |
| *** gabor-t has quit IRC | 05:30 | |
| *** irenab has joined #openstack-nova | 05:35 | |
| *** veena has joined #openstack-nova | 05:35 | |
| *** armax has joined #openstack-nova | 05:36 | |
| mikal | jogo: sure | 05:37 |
| jogo | mikal: oh also https://review.openstack.org/#/c/184645/4 because that one makes me happy too | 05:37 |
| mikal | Heh | 05:37 |
| jogo | although I am afraid of operators yelling at us for that | 05:38 |
| jogo | so maybe need to advertise it a lot | 05:38 |
| *** ildikov has quit IRC | 05:39 | |
| mikal | That brick thing does sound nice | 05:39 |
| mikal | jogo: what do you think about adding a new config drive format? | 05:40 |
| mikal | jogo: i.e. in addition to vfat and iso9660 | 05:40 |
| mikal | jogo: I worry about it being confusing for operators and resulting in other hypervisors having to eventually implement it | 05:40 |
| mikal | jogo: i.e. we'd have to have damn good reasons to do such a thing | 05:40 |
| *** tobe has joined #openstack-nova | 05:40 | |
| jogo | mikal: agreed, we need a really good reason | 05:42 |
| mikal | jogo: good, as long as I'm not just being grumpy in my old age | 05:42 |
| jogo | we can be grumpy together | 05:42 |
| jogo | mikal: link? | 05:42 |
| * mikal finds it again | 05:44 | |
| mikal | https://review.openstack.org/#/c/149253/ | 05:44 |
| *** bzhao has quit IRC | 05:46 | |
| *** ajayaa has joined #openstack-nova | 05:47 | |
| openstackgerrit | IWAMOTO Toshihiro proposed openstack/nova: Omit RescheduledException in instance_fault.message https://review.openstack.org/165751 | 05:51 |
| *** alanf-mc has quit IRC | 05:51 | |
| *** yingjun has joined #openstack-nova | 05:54 | |
| *** belmoreira has joined #openstack-nova | 05:56 | |
| *** josecastroleon has joined #openstack-nova | 05:57 | |
| *** lpetrut has joined #openstack-nova | 06:03 | |
| *** pradipta has quit IRC | 06:04 | |
| *** otter768 has joined #openstack-nova | 06:07 | |
| *** ildikov has joined #openstack-nova | 06:08 | |
| openstackgerrit | melanie witt proposed openstack/nova: Send Instance object to cells instance_update_at_top https://review.openstack.org/176518 | 06:08 |
| *** otter768 has quit IRC | 06:12 | |
| *** markvoelker has joined #openstack-nova | 06:13 | |
| *** bzhao has joined #openstack-nova | 06:14 | |
| *** RuiChen has quit IRC | 06:17 | |
| *** RuiChen has joined #openstack-nova | 06:18 | |
| *** markvoelker has quit IRC | 06:18 | |
| *** tobe has quit IRC | 06:18 | |
| *** radek_ has joined #openstack-nova | 06:23 | |
| *** paul-carlton has joined #openstack-nova | 06:23 | |
| *** ajayaa has quit IRC | 06:28 | |
| *** gszasz has joined #openstack-nova | 06:29 | |
| *** destiny has joined #openstack-nova | 06:29 | |
| *** sudipto has joined #openstack-nova | 06:32 | |
| *** flaper87 has quit IRC | 06:33 | |
| *** flaper87 has joined #openstack-nova | 06:33 | |
| openstackgerrit | ZhaoBo proposed openstack/nova: Fixed raise SecurityGroupCannotBeApplied as network port_security off https://review.openstack.org/188260 | 06:34 |
| *** ajayaa has joined #openstack-nova | 06:34 | |
| *** yingjun has quit IRC | 06:34 | |
| *** claudiub has joined #openstack-nova | 06:37 | |
| *** oro_ has joined #openstack-nova | 06:40 | |
| *** exploreshaifali has joined #openstack-nova | 06:42 | |
| *** cfriesen has quit IRC | 06:50 | |
| *** amotoki has joined #openstack-nova | 06:52 | |
| *** hshiina has quit IRC | 06:53 | |
| *** ndipanov has quit IRC | 06:53 | |
| *** Jeffrey4l has quit IRC | 06:54 | |
| *** Jeffrey4l has joined #openstack-nova | 06:55 | |
| *** sahid has joined #openstack-nova | 06:56 | |
| *** tobe has joined #openstack-nova | 06:57 | |
| openstackgerrit | Sean McCully proposed openstack/nova: Creates disk.fallocate_test https://review.openstack.org/188249 | 06:58 |
| *** pradipta has joined #openstack-nova | 06:59 | |
| *** amotoki has quit IRC | 07:03 | |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Remove and deprecate conductor task_log methods https://review.openstack.org/174247 | 07:04 |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Remove unused compute utils methods https://review.openstack.org/174246 | 07:04 |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Make instance usage audit use the brand new TaskLog object https://review.openstack.org/174245 | 07:04 |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Add a TaskLog object https://review.openstack.org/174244 | 07:04 |
| *** abhishekk has joined #openstack-nova | 07:05 | |
| *** apuimedo has joined #openstack-nova | 07:06 | |
| *** adrian_otto has quit IRC | 07:06 | |
| *** irenab has quit IRC | 07:06 | |
| *** oro_ has quit IRC | 07:10 | |
| *** irenab has joined #openstack-nova | 07:12 | |
| *** prashant-upadhya has joined #openstack-nova | 07:12 | |
| prashant-upadhya | hi guys, i was looking into nova code. and i couldn't find entry point for hook create_instance | 07:13 |
| prashant-upadhya | can somebody help me to find it ? | 07:13 |
| *** markvoelker has joined #openstack-nova | 07:13 | |
| *** paul-carlton has quit IRC | 07:14 | |
| *** matrohon has joined #openstack-nova | 07:16 | |
| *** Kevin_Zheng has joined #openstack-nova | 07:17 | |
| *** Administrator_ has joined #openstack-nova | 07:17 | |
| *** markvoelker has quit IRC | 07:18 | |
| RuiChen | prashant-upadhya: this doc may be helpful for you, http://docs.openstack.org/developer/nova/devref/hooks.html | 07:20 |
| *** yingjun has joined #openstack-nova | 07:21 | |
| *** veena has quit IRC | 07:21 | |
| prashant-upadhya | RuiChen i went through that doc. | 07:23 |
| prashant-upadhya | i m trying to find entrypoint for this create_instance . but nowhere i could find it. | 07:24 |
| prashant-upadhya | is it there or left for future usage ? | 07:24 |
| prashant-upadhya | or i m missing something here ? | 07:24 |
| jogo | prashant-upadhya: did you add a nova,hooks section to setup.py as documented http://docs.openstack.org/developer/nova/devref/hooks.html#adding-hook-object-code | 07:26 |
| RuiChen | prashant-upadhya: create_instance hook is custom code, is not include in nova, nova just provide a custom point. | 07:26 |
| prashant-upadhya | no i haven't | 07:26 |
| prashant-upadhya | oh. | 07:26 |
| prashant-upadhya | so in master there is no implementation for create_instance | 07:27 |
| prashant-upadhya | right?? | 07:27 |
| *** amotoki has joined #openstack-nova | 07:27 | |
| RuiChen | prashant-upadhya: yeah, it's not in master, i find a simple in github, https://github.com/larsks/demo_nova_hooks | 07:29 |
| prashant-upadhya | yeah i have looked at it as well | 07:30 |
| prashant-upadhya | thanks man | 07:30 |
| prashant-upadhya | :) | 07:30 |
| prashant-upadhya | that solves my query | 07:30 |
| *** paul-carlton has joined #openstack-nova | 07:30 | |
| *** exploreshaifali has quit IRC | 07:31 | |
| openstackgerrit | Ritesh proposed openstack/nova: Users will not be allowed to delete instance during snapshot https://review.openstack.org/185942 | 07:32 |
| openstackgerrit | ZhaoBo proposed openstack/nova: Fixed raise SecurityGroupCannotBeApplied as network port_security off https://review.openstack.org/188260 | 07:34 |
| *** jlanoux has joined #openstack-nova | 07:37 | |
| *** claudiub|2 has joined #openstack-nova | 07:38 | |
| RuiChen | prashant-upadhya: enjoy nova :) | 07:39 |
| *** claudiub has quit IRC | 07:39 | |
| openstackgerrit | Andrey Pavlov proposed openstack/nova: Extract helper method to get image metadata from volume https://review.openstack.org/187461 | 07:42 |
| *** pixelbeat_ has joined #openstack-nova | 07:43 | |
| *** lpetrut has quit IRC | 07:44 | |
| *** dims_ has joined #openstack-nova | 07:44 | |
| *** matrohon has quit IRC | 07:46 | |
| *** SourabhP has joined #openstack-nova | 07:48 | |
| openstackgerrit | Joe Gordon proposed openstack/nova: Remove unused exceptions https://review.openstack.org/188290 | 07:48 |
| *** dims_ has quit IRC | 07:49 | |
| *** pbandzi has joined #openstack-nova | 07:50 | |
| *** jistr has joined #openstack-nova | 07:50 | |
| RuiChen | bauzas: can you have a look https://review.openstack.org/#/c/182165/ again? I had replied your question. | 07:52 |
| *** alex_klimov has joined #openstack-nova | 07:52 | |
| *** jistr is now known as jistr|mt | 07:54 | |
| *** jistr|mt is now known as jistr|mtg | 07:54 | |
| *** paul-carlton has left #openstack-nova | 07:55 | |
| *** SourabhP has quit IRC | 07:56 | |
| *** romainh has joined #openstack-nova | 07:57 | |
| *** sahid has quit IRC | 07:57 | |
| *** afazekas has joined #openstack-nova | 08:01 | |
| *** sahid has joined #openstack-nova | 08:03 | |
| mdbooth | Remind me, is the global-requirements -> nova flow automatic (at least by process if not necessarily untouched by human hands), or do I have to manually pull in updates from it? | 08:03 |
| *** veena has joined #openstack-nova | 08:03 | |
| *** aix has joined #openstack-nova | 08:03 | |
| gilliard | AIUI, Jobs will fail if nova's requirements.txt contradicts the g-r one. If you want to be more specific than g-r is, in nova, you have to update requirements.txt | 08:04 |
| gilliard | so, it depends :) | 08:04 |
| mdbooth | gilliard: But it's acceptable for me to create a commit to update a nova/requirements.txt entry to correspond to g-r? | 08:05 |
| mdbooth | i.e. I won't be treading on some procedure or automated process | 08:05 |
| gilliard | this happens automatically: https://review.openstack.org/#/c/188099/ | 08:06 |
| *** ndipanov has joined #openstack-nova | 08:07 | |
| ndipanov | lan, still here? | 08:07 |
| *** otter768 has joined #openstack-nova | 08:08 | |
| *** lucasagomes has joined #openstack-nova | 08:09 | |
| *** sahid has quit IRC | 08:10 | |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: scheduler: weight nodes based on their cached images https://review.openstack.org/173304 | 08:10 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: Add new image cache monitor in compute nodes https://review.openstack.org/173303 | 08:10 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: vmware: add image_cache_manager property https://review.openstack.org/187602 | 08:10 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: compute monitors: set update delta from base class https://review.openstack.org/173987 | 08:10 |
| mdbooth | gilliard: I thought there might have been. Thanks! | 08:11 |
| *** armax has quit IRC | 08:11 | |
| openstackgerrit | Joe Gordon proposed openstack/python-novaclient: Add docs tox env https://review.openstack.org/188296 | 08:12 |
| openstackgerrit | Joe Gordon proposed openstack/python-novaclient: Fix all doc warnings and gate on warnings https://review.openstack.org/188297 | 08:12 |
| *** sahid has joined #openstack-nova | 08:13 | |
| *** otter768 has quit IRC | 08:13 | |
| lan | ndipanov, yes, could you please take a look at my comment in this patch: https://review.openstack.org/#/c/177084/ | 08:14 |
| openstackgerrit | Matthew Booth proposed openstack/nova: Implement compare-and-swap for instance update https://review.openstack.org/141115 | 08:14 |
| *** yamahata has quit IRC | 08:14 | |
| lan | ndipanov, I think this patch won't change the behavior . | 08:14 |
| *** Administrator_ has joined #openstack-nova | 08:15 | |
| *** yassine has joined #openstack-nova | 08:16 | |
| *** yingjun has quit IRC | 08:17 | |
| *** Kevin_Zheng has quit IRC | 08:18 | |
| johnthetubaguy | jogo: hey, hows things? are you busy traveling, or just having issues sleeping? | 08:19 |
| *** yingjun has joined #openstack-nova | 08:19 | |
| *** matrohon has joined #openstack-nova | 08:20 | |
| *** amotoki has quit IRC | 08:20 | |
| ndipanov | lan, yeah this looks better | 08:20 |
| *** amotoki has joined #openstack-nova | 08:20 | |
| *** mpaolino has joined #openstack-nova | 08:22 | |
| *** mpaolino has quit IRC | 08:24 | |
| lan | ndipanov, thanks for your vote : ) | 08:26 |
| belmoreira | ndipanov: can you have a look on https://bugs.launchpad.net/nova/+bug/1461777 | 08:28 |
| openstack | Launchpad bug 1461777 in OpenStack Compute (nova) "Random NUMA cell selection can leave NUMA cells unused" [Undecided,New] | 08:28 |
| *** ZZelle has quit IRC | 08:28 | |
| belmoreira | ndipanov: maybe I’m doing something really wrong | 08:28 |
| *** ZZelle has joined #openstack-nova | 08:29 | |
| jogo | johnthetubaguy: both | 08:29 |
| jogo | johnthetubaguy: in Tokyo at the moment | 08:29 |
| ndipanov | belmoreira, no that seems correct - I chatted breifly to danpb about it yesterday | 08:32 |
| *** nihilifer has joined #openstack-nova | 08:32 | |
| *** Jeffrey4l has quit IRC | 08:32 | |
| ndipanov | belmoreira, the best option imho is to remove that "feature" and just let the kernel handle it | 08:35 |
| belmoreira | ndipanov: I will patch our deployment for that. | 08:35 |
| belmoreira | ndipanov: I agree with you | 08:35 |
| ndipanov | I will comment on the bug and remove it upstream hopefully today | 08:36 |
| belmoreira | ndipanov: I can remove that if you want | 08:37 |
| *** danpb has joined #openstack-nova | 08:37 | |
| *** tobe has quit IRC | 08:39 | |
| *** tobe has joined #openstack-nova | 08:39 | |
| *** gcb has quit IRC | 08:41 | |
| *** sahid has quit IRC | 08:42 | |
| *** sahid has joined #openstack-nova | 08:43 | |
| *** Jeffrey4l has joined #openstack-nova | 08:44 | |
| *** oro_ has joined #openstack-nova | 08:45 | |
| *** lpetrut has joined #openstack-nova | 08:46 | |
| garyk | danpb: following the patches yesterday I posted https://review.openstack.org/188122 | 08:49 |
| openstackgerrit | Petrut Lucian proposed openstack/nova: HyperV: Add IOUtils class for asynchronous IO operations https://review.openstack.org/184035 | 08:52 |
| openstackgerrit | Petrut Lucian proposed openstack/nova: HyperV: Add IOQueue class used for named pipe IO operations https://review.openstack.org/184036 | 08:52 |
| openstackgerrit | Petrut Lucian proposed openstack/nova: HyperV: Add named pipe handler https://review.openstack.org/184037 | 08:52 |
| openstackgerrit | Petrut Lucian proposed openstack/nova: HyperV: Add serial console proxy https://review.openstack.org/184038 | 08:53 |
| openstackgerrit | Petrut Lucian proposed openstack/nova: HyperV: Add serial console handler class https://review.openstack.org/184039 | 08:53 |
| openstackgerrit | Petrut Lucian proposed openstack/nova: HyperV: Add SerialConsoleOps class https://review.openstack.org/184040 | 08:53 |
| *** matrohon has quit IRC | 08:55 | |
| *** matrohon has joined #openstack-nova | 09:00 | |
| *** markvoelker has joined #openstack-nova | 09:02 | |
| *** ociuhandu has quit IRC | 09:02 | |
| *** jyuso1 has quit IRC | 09:04 | |
| *** markvoelker has quit IRC | 09:07 | |
| *** salv-orlando has quit IRC | 09:08 | |
| *** amotoki has quit IRC | 09:08 | |
| *** yuntong has quit IRC | 09:11 | |
| openstackgerrit | Petrut Lucian proposed openstack/nova: HyperV: Nova serial console access support https://review.openstack.org/145004 | 09:12 |
| *** matrohon has quit IRC | 09:13 | |
| *** matrohon has joined #openstack-nova | 09:19 | |
| *** jistr|mtg is now known as jistr | 09:19 | |
| *** ociuhandu has joined #openstack-nova | 09:20 | |
| *** matrohon has quit IRC | 09:24 | |
| tobasco | this is silly how would anybody ever find out why the vif is binding_failed | 09:25 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: Improve compute swap_volume logging https://review.openstack.org/188320 | 09:30 |
| openstackgerrit | garyk proposed openstack/nova: Objects: add limits, reservations and shares to ImageMetaProps object https://review.openstack.org/188321 | 09:32 |
| *** tobe has quit IRC | 09:33 | |
| lxsli | mikal: johnthetubaguy: do you also prefer the cron solution re: https://review.openstack.org/#/c/184637/6/specs/liberty/approved/purge-soft-deleted-rows.rst ? | 09:37 |
| *** amotoki has joined #openstack-nova | 09:38 | |
| *** ajayaa has quit IRC | 09:40 | |
| *** yingjun has quit IRC | 09:41 | |
| *** amotoki has quit IRC | 09:42 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: DO NOT MERGE: Ajouter un log moche pour le scheduler https://review.openstack.org/188178 | 09:44 |
| *** afazekas is now known as afazekas_mtg | 09:44 | |
| *** sorrison_laptop has quit IRC | 09:45 | |
| *** dims_ has joined #openstack-nova | 09:45 | |
| *** ociuhandu has quit IRC | 09:46 | |
| *** yassine has quit IRC | 09:49 | |
| *** nihilifer has quit IRC | 09:49 | |
| *** dims_ has quit IRC | 09:51 | |
| *** romainh has quit IRC | 09:51 | |
| lxsli | any sqlalchemy experts around please? | 09:51 |
| *** matrohon has joined #openstack-nova | 09:52 | |
| lxsli | I'm wondering if it's possible to have an optional column, IE neither created nor deleted automatically | 09:52 |
| *** tobe has joined #openstack-nova | 09:53 | |
| claudiub|2 | what do you mean? | 09:54 |
| lxsli | I want to remove the deleted and deleted_at columns from most tables | 09:54 |
| lxsli | however this could be quite time-consuming so mikal suggested making it something operators could choose the timing of | 09:55 |
| claudiub|2 | you should have a blueprint for that | 09:55 |
| lxsli | I do :) | 09:55 |
| claudiub|2 | ah, cool | 09:55 |
| lxsli | https://review.openstack.org/#/c/184645/ | 09:55 |
| claudiub|2 | it is an interesting idea, but I'm thinking there might be some other implications | 09:55 |
| lxsli | oh? | 09:56 |
| claudiub|2 | for example, I think there are some parts that actually look for those values | 09:56 |
| claudiub|2 | the delete or deleted_al values | 09:56 |
| claudiub|2 | at* | 09:56 |
| lxsli | not for most tables - full details in the spec | 09:56 |
| claudiub|2 | k, looking | 09:56 |
| lxsli | thanks :) | 09:56 |
| *** ajayaa has joined #openstack-nova | 09:57 | |
| lxsli | I can't see anything in the sqlalchemy docs about Column which would suggest mikal's suggestion is possible | 09:58 |
| lxsli | but I'm no expert, I could easily be looking in the wrong place | 09:58 |
| lxsli | http://sqlalchemy.readthedocs.org/en/rel_1_0/core/metadata.html#column-table-metadata-api | 09:59 |
| mikal | lxsli: so its late here, but sqlalchemy doesn't mind there being extra columns | 09:59 |
| mikal | lxsli: we've had models not match tables in the past | 09:59 |
| lxsli | mikal: hey! that's sweet, but wouldn't a contract try to delete them all at once? | 09:59 |
| mikal | lxsli: if we force the removal of those columns, we have to delete the rows containing deleted data, which forces a table re-write of potentially large tables | 09:59 |
| lxsli | mikal: which would be bad. | 10:00 |
| mikal | lxsli: so, I was hoping for some way for operators to defer that if they wanted | 10:00 |
| *** danpb has quit IRC | 10:00 | |
| lxsli | Yep totally agreed, just exploring how to make that possible | 10:00 |
| mikal | lxsli: i.e. delete the rows, remove the columsn from the model, perhaps let operators delete the columns at their leisure | 10:00 |
| mikal | lxsli: we could add something in say M to enforce column removal if it hadn't been manually done before then | 10:01 |
| mikal | lxsli: I'd recommend talking to johannes or johnthetubaguy about expand / contract | 10:01 |
| johnthetubaguy | mikal: lxsli: the online schema migration does change that... | 10:01 |
| mikal | lxsli: it tries to solve some of these problems for us | 10:01 |
| mikal | lxsli: and you might be able to steal some of that | 10:01 |
| lxsli | mikal: OK excellent, thank you | 10:01 |
| mikal | lxsli: sorry, leverage | 10:01 |
| mikal | lxsli: np | 10:01 |
| johnthetubaguy | we need to get this written up, I think dansmith was going to, how we do the data migrations | 10:01 |
| mikal | johnthetubaguy: this is in the context of lxsli's soft delete mixin cleanup spec | 10:02 |
| openstackgerrit | Lan Qi Song proposed openstack/nova: Remove useless volume when boot from volume failed https://review.openstack.org/177084 | 10:02 |
| mikal | johnthetubaguy: it occurred to me today it would involve some quite large tabel re-writes, which worried me | 10:02 |
| johnthetubaguy | mikal: ah | 10:02 |
| johnthetubaguy | mikal: dropping columns we don't use should be fine, that can be online if you have new enough mysql | 10:02 |
| johnthetubaguy | lxsli: ^ | 10:03 |
| lxsli | o/ | 10:03 |
| *** dims_ has joined #openstack-nova | 10:03 | |
| mikal | johnthetubaguy: oh, that's news to me | 10:03 |
| lxsli | johnthetubaguy: and it won't be punitively slow? | 10:03 |
| mikal | johnthetubaguy: cool | 10:03 |
| mikal | And I would think deleting rows should be relatively cheap too | 10:03 |
| mikal | So we might be ok-ish here for modern mysqls | 10:03 |
| lxsli | providing there aren't 1E1000 of them, which we can clearly document as pre-release preparation for deployers | 10:04 |
| mikal | Well, the metadata tables might have millions of rows, right? | 10:04 |
| mikal | (Although I am not sure if they're in the set you intend to immediately clean up) | 10:05 |
| lxsli | I won't touch InstanceSystemMetadata in this spec | 10:05 |
| johnthetubaguy | the delete would take longer I think, thats a data migration, as I see it | 10:05 |
| lxsli | oops I mean InstanceTypes | 10:06 |
| johnthetubaguy | lxsli: so we can't force people till the next release, as far as I understand it | 10:06 |
| johnthetubaguy | lxsli: I would do them all really, if thats possible | 10:06 |
| johnthetubaguy | so... I should read the spec I guess | 10:06 |
| lxsli | thanks :) | 10:07 |
| johnthetubaguy | currently snowed under with non-spec stuff post summit though, so it might be next week before that happens I am afraid | 10:07 |
| mikal | Yeah, I think you should do them all too (well, all the ones where soft delete isn't used by the code) | 10:08 |
| lxsli | Excellent - I'll push a revision | 10:08 |
| *** ildikov has quit IRC | 10:08 | |
| lxsli | Thanks chaps | 10:08 |
| mikal | My comment about adding the mixin everywhere and then removing them one by one is more about making the change more obvious and testable in chunks | 10:08 |
| mikal | i.e. in case you've missed a case where we do need soft delete, it would be more obvious in a change which is just one table with tests to verify that we didn't break anything | 10:08 |
| lxsli | Yes I understood that to be a review process comment | 10:08 |
| mikal | Cool | 10:08 |
| lxsli | Would say 5 at a time be OK? | 10:08 |
| mikal | Also, it will make your commit count _awesome_ | 10:09 |
| mikal | :P | 10:09 |
| lxsli | \o/ | 10:09 |
| *** salv-orlando has joined #openstack-nova | 10:09 | |
| mikal | Sure, the tables per change could be more than one | 10:09 |
| mikal | But I'd say less than 10 | 10:09 |
| *** otter768 has joined #openstack-nova | 10:09 | |
| mikal | Maybe it depends, smaller at first while we learn the gotchas, and then growing after that? | 10:09 |
| mikal | Dunno, I haven't meditated on that bit much | 10:09 |
| lxsli | OK, I'll do that | 10:09 |
| johnthetubaguy | yeah, try a few, see how it goes | 10:10 |
| *** moshele has joined #openstack-nova | 10:10 | |
| johnthetubaguy | but the spec should mention any of that detail | 10:10 |
| mikal | Anyway, time for me to put the kids to bed | 10:10 |
| mikal | Laters | 10:10 |
| lxsli | johnthetubaguy: should / shouldn't? | 10:10 |
| lxsli | mikal: g'night! | 10:10 |
| johnthetubaguy | lxsli: shouldn't | 10:10 |
| lxsli | ok thanks | 10:10 |
| johnthetubaguy | lxsli: stuff thats best discussed in code review, is best staying out of the spec, as a rough guideline | 10:11 |
| johnthetubaguy | lxsli: just need to cover the impact to deployers, etc, the more bigger stroke things | 10:11 |
| *** wwriverrat has quit IRC | 10:11 | |
| *** ociuhandu has joined #openstack-nova | 10:11 | |
| *** bzhao has quit IRC | 10:12 | |
| *** mohitsharma has quit IRC | 10:12 | |
| *** danpb has joined #openstack-nova | 10:13 | |
| *** otter768 has quit IRC | 10:13 | |
| *** mohitsharma has joined #openstack-nova | 10:14 | |
| *** sorrison_laptop has joined #openstack-nova | 10:15 | |
| *** salv-orlando has quit IRC | 10:17 | |
| *** markvoelker has joined #openstack-nova | 10:18 | |
| gilliard | Nova has dropped python 2.6 support, correct? | 10:18 |
| *** mohitsharma has quit IRC | 10:19 | |
| *** Kennan2 has joined #openstack-nova | 10:20 | |
| *** Kennan has quit IRC | 10:20 | |
| lxsli | yes - gate jobs are gone | 10:20 |
| gilliard | answering my own q: https://wiki.openstack.org/wiki/Python3#Python_2:_Python_2.6_support_dropped.2C_Python_2.7_only | 10:20 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: libvirt: Disable NUMA for broken libvirt https://review.openstack.org/178188 | 10:20 |
| *** salv-orlando has joined #openstack-nova | 10:20 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: libvirt: Disable NUMA for broken libvirt https://review.openstack.org/178188 | 10:20 |
| *** salv-orlando has quit IRC | 10:21 | |
| *** Kennan2 is now known as Kennan | 10:21 | |
| *** Administrator__ has joined #openstack-nova | 10:22 | |
| *** markvoelker has quit IRC | 10:22 | |
| *** salv-orlando has joined #openstack-nova | 10:23 | |
| claudiub|2 | question: if you do nova live-migration, can you choose the same instace's host? | 10:24 |
| claudiub|2 | e.g.: if instance foo is on host bar, can you do nova live-migration foo bar? | 10:24 |
| *** Administrator_ has quit IRC | 10:25 | |
| bauzas | claudiub|2: IIRC, there is a check preventing this | 10:27 |
| claudiub|2 | cool, ty. :) | 10:27 |
| *** wwriverrat has joined #openstack-nova | 10:27 | |
| bauzas | claudiub|2: yeah https://github.com/openstack/nova/blob/master/nova/conductor/tasks/live_migrate.py#L105-L108 | 10:28 |
| claudiub|2 | bauzas: cool, good to know. :D | 10:29 |
| bauzas | np | 10:29 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: libvirt: make live_migration_uri flag dependant on virt_type https://review.openstack.org/175780 | 10:30 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/objects/test*.py https://review.openstack.org/187785 | 10:30 |
| *** coolsvap is now known as coolsvap|afk | 10:30 | |
| *** lpetrut1 has joined #openstack-nova | 10:33 | |
| *** lpetrut has quit IRC | 10:33 | |
| *** boris-42 has joined #openstack-nova | 10:33 | |
| *** lpetrut1 has left #openstack-nova | 10:34 | |
| *** lpetrut1 has joined #openstack-nova | 10:34 | |
| *** ociuhandu has quit IRC | 10:39 | |
| *** sorrison_laptop has quit IRC | 10:40 | |
| *** ildikov has joined #openstack-nova | 10:41 | |
| *** s0mik_ has quit IRC | 10:42 | |
| *** sorriso__ has joined #openstack-nova | 10:43 | |
| *** sorriso__ has quit IRC | 10:44 | |
| *** ociuhandu has joined #openstack-nova | 10:44 | |
| *** mwagner_afk has quit IRC | 10:45 | |
| *** markvoelker has joined #openstack-nova | 10:46 | |
| *** marzif_ has joined #openstack-nova | 10:47 | |
| openstackgerrit | garyk proposed openstack/nova: Objects: add limits, reservations and shares to ImageMetaProps object https://review.openstack.org/188321 | 10:49 |
| *** marzif_ has quit IRC | 10:50 | |
| *** pradipta has quit IRC | 10:50 | |
| *** marzif_ has joined #openstack-nova | 10:50 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Adding more py34 test https://review.openstack.org/188243 | 10:52 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/scheduler/test*.py https://review.openstack.org/187968 | 10:52 |
| *** Longgeek_ has quit IRC | 10:53 | |
| *** coolsvap|afk is now known as coolsvap | 10:54 | |
| *** Longgeek has joined #openstack-nova | 10:54 | |
| *** romainh has joined #openstack-nova | 10:57 | |
| *** mwagner_afk has joined #openstack-nova | 10:58 | |
| *** vladikr has quit IRC | 10:58 | |
| *** yassine_ has joined #openstack-nova | 11:09 | |
| *** bzhao has joined #openstack-nova | 11:11 | |
| *** Vek has quit IRC | 11:12 | |
| *** exploreshaifali has joined #openstack-nova | 11:15 | |
| *** coolsvap is now known as coolsvap|afk | 11:16 | |
| *** flwang1 has joined #openstack-nova | 11:17 | |
| *** mwagner_afk has quit IRC | 11:19 | |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: Improve compute swap_volume logging https://review.openstack.org/188320 | 11:20 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: make _create_domain return a Guest object https://review.openstack.org/182185 | 11:20 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce method poweroff to Guest object https://review.openstack.org/182194 | 11:20 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce method resume to Guest object https://review.openstack.org/182195 | 11:20 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce a Guest to wrap around virDomain https://review.openstack.org/174313 | 11:20 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce method get_guest to Host object https://review.openstack.org/181969 | 11:20 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce method to get vcpus info https://review.openstack.org/188346 | 11:20 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: introduce method to erase instance config https://review.openstack.org/188347 | 11:20 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: libvirt: make live_migration_uri flag dependent on virt_type https://review.openstack.org/175780 | 11:21 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Adding more py34 test https://review.openstack.org/188243 | 11:21 |
| *** danpb1 has joined #openstack-nova | 11:22 | |
| *** danpb has quit IRC | 11:22 | |
| bauzas | is that me, or is it look possible to migrate an instance which is paused ? | 11:23 |
| bauzas | does it look | 11:23 |
| * bauzas wishes a state machine for Christmas | 11:24 | |
| *** danpb1 has quit IRC | 11:26 | |
| *** lucasagomes is now known as lucas-hungry | 11:27 | |
| *** danpb has joined #openstack-nova | 11:27 | |
| *** aix has quit IRC | 11:30 | |
| johnthetubaguy | bauzas: oddly, you may be correct, but yeah, it can take some detective work to find out. | 11:32 |
| *** deepthi has quit IRC | 11:32 | |
| johnthetubaguy | I always hoped tasks would fix that, but thats mostly a silly dream | 11:32 |
| *** veena has quit IRC | 11:33 | |
| bauzas | johnthetubaguy: yeah, I'm just trying to figure out all the corner cases due to https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L296-L301 | 11:33 |
| bauzas | johnthetubaguy: since RequestSpec would need to get the vm_state... | 11:33 |
| bauzas | johnthetubaguy: but alaski and I are not liking that | 11:34 |
| bauzas | johnthetubaguy: anyway, as I said to alaski, I would try to hop to Tasks API by Lib-2 if I'm done with ReqSpec | 11:34 |
| andreykurilin | sdague: yesturday I talk about this ironic spec http://specs.openstack.org/openstack/ironic-specs/specs/kilo/api-microversions.html . It looks like only use cases 3b and 8 are not covered by my patches | 11:34 |
| johnthetubaguy | bauzas: doesn't sound like something the scheduler should be bothering with | 11:35 |
| andreykurilin | *yesterday I told | 11:35 |
| bauzas | johnthetubaguy: yeah, that's why I would just remove the conditional | 11:35 |
| bauzas | johnthetubaguy: ie. a sched call is just an IO op, that's it | 11:35 |
| johnthetubaguy | bauzas: ah, thats different | 11:35 |
| johnthetubaguy | bauzas: an IO op is something thats doing lots of IO load on the hypervisor thats not related to user load | 11:36 |
| johnthetubaguy | bauzas: its missing live-migrate, which is a shame | 11:36 |
| bauzas | johnthetubaguy: agreed, but when looking at the items when the scheduler is called for a dest, that means the hypervisor will have to work right ? :) | 11:37 |
| bauzas | johnthetubaguy: since it's in consume_from_instance(), that just means the VM will go there, so it's an io op anyway | 11:37 |
| sdague | andreykurilin: ok, great. I didn't realize the client was doing latest already from the cli. | 11:38 |
| sdague | I think that was the concern | 11:38 |
| andreykurilin | sdague: do we need a nova-specific spec for this? | 11:40 |
| sdague | andreykurilin: mirroring the spec from ironic would be useful for documentation | 11:43 |
| sdague | it would be a quick merge | 11:43 |
| andreykurilin | sdague: ok, I'll copy and edit a bit. | 11:43 |
| *** baoli has joined #openstack-nova | 11:44 | |
| sdague | thanks | 11:44 |
| *** tellesnobrega has joined #openstack-nova | 11:47 | |
| *** Administrator__ is now known as Kevin_Zheng | 11:47 | |
| *** kiran-r has quit IRC | 11:49 | |
| *** baoli has quit IRC | 11:50 | |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: scheduler: weight nodes based on their cached images https://review.openstack.org/173304 | 11:51 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: Add new image cache monitor in compute nodes https://review.openstack.org/173303 | 11:51 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: vmware: add image_cache_manager property https://review.openstack.org/187602 | 11:51 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: compute monitors: set update delta from base class https://review.openstack.org/173987 | 11:51 |
| *** baoli has joined #openstack-nova | 11:51 | |
| *** afazekas_mtg has quit IRC | 11:53 | |
| *** baoli has quit IRC | 11:53 | |
| johnthetubaguy | bauzas: the scheuduler is not involved in all the things where the hypervisor has an IO op, in the general case, snapshot could be counted as one, I guess, although its maybe not counted | 11:58 |
| *** sorrison_laptop has joined #openstack-nova | 11:59 | |
| *** ferest has joined #openstack-nova | 12:01 | |
| bauzas | johnthetubaguy: yup, that's why I think io_ops should be a metric done by the RT | 12:01 |
| bauzas | johnthetubaguy: because for example, migrations with a dest host provided just bypass the scheduler | 12:02 |
| *** dprince has joined #openstack-nova | 12:04 | |
| bauzas | johnthetubaguy: well actually it is | 12:04 |
| bauzas | johnthetubaguy: the consume_from_instance() stuff just adds one | 12:05 |
| bauzas | anyway | 12:05 |
| *** tobe has quit IRC | 12:06 | |
| *** lucas-hungry is now known as lucasagomes | 12:07 | |
| *** ferest has quit IRC | 12:07 | |
| *** otter768 has joined #openstack-nova | 12:10 | |
| *** ferest has joined #openstack-nova | 12:12 | |
| *** ivasev has joined #openstack-nova | 12:13 | |
| *** otter768 has quit IRC | 12:15 | |
| openstackgerrit | wangxiyuan proposed openstack/nova: Add size check when boot with blank in volume https://review.openstack.org/188367 | 12:15 |
| *** mtanino has joined #openstack-nova | 12:16 | |
| *** irenab has quit IRC | 12:17 | |
| *** irenab has joined #openstack-nova | 12:17 | |
| openstackgerrit | Rui Chen proposed openstack/nova: Fix resource leaking when consume_from_instance raise exception https://review.openstack.org/182165 | 12:17 |
| *** mwagner_afk has joined #openstack-nova | 12:22 | |
| *** liusheng has quit IRC | 12:26 | |
| *** flwang1 has quit IRC | 12:26 | |
| *** ajayaa has quit IRC | 12:29 | |
| *** prashant-upadhya has quit IRC | 12:30 | |
| openstackgerrit | Joe Gordon proposed openstack/nova: Remove unused instance_group_policy db calls https://review.openstack.org/188373 | 12:30 |
| *** moshele has quit IRC | 12:30 | |
| *** aix has joined #openstack-nova | 12:30 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova-specs: Adds Hyper-V Cluster spec https://review.openstack.org/188374 | 12:31 |
| claudiub|2 | hello. If you have a bit of time to take a look at the hyper-v nova rescue, we would greatly appreciate it. https://review.openstack.org/#/c/127159/ It has been reviewed quite a lot, Jenkins likes it, Hyper-V CI likes it, we like it, we hope you do too. :D | 12:32 |
| *** rushiagr is now known as rushiagr_away | 12:36 | |
| *** rwsu has joined #openstack-nova | 12:36 | |
| alaski | johnthetubaguy bauzas the scheduler is not involved in everything that should increment io_iops, but my assertion is that anything going through the scheduler would increment it, so keying off of vm/task_state in that case is unnecessary | 12:39 |
| *** xyang1 has joined #openstack-nova | 12:41 | |
| *** zzzeek has joined #openstack-nova | 12:42 | |
| *** sudipto has quit IRC | 12:42 | |
| *** heyongli has joined #openstack-nova | 12:44 | |
| *** moshele has joined #openstack-nova | 12:44 | |
| *** marzif_ has quit IRC | 12:44 | |
| *** afazekas_mtg has joined #openstack-nova | 12:45 | |
| *** sorrison_laptop has quit IRC | 12:47 | |
| johnthetubaguy | alaski: ah, got you, true | 12:52 |
| *** IanGovett has joined #openstack-nova | 12:53 | |
| heyongli | johnthetubaguy, hi | 12:53 |
| *** kylek3h has joined #openstack-nova | 12:55 | |
| *** kylek3h has quit IRC | 12:56 | |
| *** baoli has joined #openstack-nova | 12:58 | |
| *** kiran-r has joined #openstack-nova | 12:58 | |
| *** zhenguo has quit IRC | 12:58 | |
| *** dane-fichter has joined #openstack-nova | 12:58 | |
| johnthetubaguy | heyongli: hi | 12:59 |
| *** artom has joined #openstack-nova | 13:00 | |
| dane-fichter | hello | 13:00 |
| heyongli | johnthetubaguy, i want to synce the CI status, we are writing some scripts to monitoring the Ci machines, in all following event sent mail to maintain team: | 13:00 |
| heyongli | 1. any case failed | 13:00 |
| heyongli | 2. any machine down, | 13:01 |
| johnthetubaguy | heyongli: its possible its broken code, so you might get false negatives there, but its possible | 13:02 |
| heyongli | then the PCI CI/NGFW/ etc CI, will got quick response for anylize why testcase fail, or bring it up again | 13:02 |
| heyongli | johnthetubaguy, sure, we try make it valueable , ie, pull the upstream status also, if both failed most like mean the patch is buggy or CI all ci need to be updated | 13:03 |
| heyongli | anyway, this will make sure the CI running smooth and catch any possible broken | 13:03 |
| *** afazekas_mtg has quit IRC | 13:04 | |
| openstackgerrit | Daniel Berrange proposed openstack/nova: libvirt: convert imagebackend to support nova.virt.image.model classes https://review.openstack.org/132024 | 13:04 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: virt: convert VFS API to use nova.virt.image.model https://review.openstack.org/132022 | 13:04 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: virt: convert disk API over to use nova.virt.image.model https://review.openstack.org/132023 | 13:04 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: virt: introduce model for describing local image metadata https://review.openstack.org/132020 | 13:04 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: virt: convert disk mount API to use nova.virt.image.model https://review.openstack.org/132021 | 13:04 |
| heyongli | case fail monitoring almost done, it will help, anyway. | 13:04 |
| johnthetubaguy | heyongli: sounds like a good step forward, certainly the #2 is really important | 13:05 |
| johnthetubaguy | heyongli: have you see the work infra have done on elasticrecheck? http://docs.openstack.org/infra/elastic-recheck/ | 13:05 |
| *** mtanino has quit IRC | 13:05 | |
| heyongli | yeah, johnthetubaguy , we will setting up a machine ping each machine to find posiible down. | 13:06 |
| johnthetubaguy | heyongli: sounds like its time to install nagios for your CI system | 13:06 |
| heyongli | johnthetubaguy, ok, i got this task, will get it deploy soon. | 13:07 |
| *** iamjarvo has joined #openstack-nova | 13:08 | |
| johnthetubaguy | heyongli: cool, thanks for the good work on that, it takes a lot of effort to get CI into a stable state, appreciate you pushing on that | 13:08 |
| heyongli | for the step2, it started, a ping check is easy to deploy, if any one have good ideas for checking machine done, tell me. | 13:09 |
| *** Vek has joined #openstack-nova | 13:09 | |
| heyongli | johnthetubaguy, i really want it stable, most important factor is been matained continuely way. | 13:09 |
| *** vladikr has joined #openstack-nova | 13:10 | |
| *** busterswt has joined #openstack-nova | 13:11 | |
| heyongli | i will, and going to update the CI state or progress when anything move further. it help -:) johnthetubaguy | 13:12 |
| *** oro_ has quit IRC | 13:13 | |
| *** adrian_otto has joined #openstack-nova | 13:14 | |
| *** mohitsharma has joined #openstack-nova | 13:14 | |
| *** nelsnelson has joined #openstack-nova | 13:15 | |
| *** mriedem has joined #openstack-nova | 13:15 | |
| *** bzhao has quit IRC | 13:16 | |
| *** irenab has quit IRC | 13:18 | |
| *** prashant-upadhya has joined #openstack-nova | 13:19 | |
| johnthetubaguy | mdbooth: hey, is my memory correct in saying you were looking at the olso.db facade, or did I get that mixed up in my head? | 13:19 |
| mdbooth | johnthetubaguy: I was | 13:19 |
| johnthetubaguy | mdbooth: did you have a spec for that? | 13:20 |
| mdbooth | I even have a half-rewritten spec | 13:20 |
| johnthetubaguy | ah sweet | 13:20 |
| *** eharney has quit IRC | 13:20 | |
| johnthetubaguy | mdbooth: I mentioned that in the summit session (https://etherpad.openstack.org/p/YVR-nova-database-internals) just wanted to make sure we were still thinking about that | 13:20 |
| johnthetubaguy | mdbooth: I will give you the action to finish the spec, if thats OK? | 13:21 |
| *** adrian_otto has quit IRC | 13:21 | |
| *** heyongli has quit IRC | 13:21 | |
| mdbooth | Yes, it remains a good idea | 13:21 |
| mdbooth | johnthetubaguy: Sure. There's a fly in the ointment now that there wasn't before, though | 13:21 |
| mdbooth | Which is that the current oslo.db functionality doesn't cover the new cells case of connecting to an alternate db | 13:21 |
| johnthetubaguy | mdbooth: ah, OK, so I will try and do a forward looking DB doc soon, and we could add our rough intentions in there, while we get over that hump | 13:22 |
| mdbooth | It's not a big deal to add, but it hasn't been added and there's no design afaik | 13:22 |
| johnthetubaguy | mdbooth: ah, gotcha, could we just use the new method for the old DB, for the moment? | 13:22 |
| johnthetubaguy | mdbooth: oh wait, thats a really bad idea, ignore me | 13:22 |
| johnthetubaguy | dims_: just wondering if you knew about our oslo.db facade issues, just a quick heads up ^ | 13:23 |
| dims_ | johnthetubaguy: zzzeek and rpodolyaka were talking about it yesterday on oslo channel. please hop there and see if they are around | 13:24 |
| johnthetubaguy | dims_: oh thats, cool, if they all know about that, mdbooth does that ^ help your effort? | 13:25 |
| zzzeek | johnthetubaguy: rpodolyaka was going to look into some enginefacade / nova work. the new enginefacade does support as many separate configs as needed | 13:25 |
| mdbooth | zzzeek: Cool. Hadn't caught up with that development. | 13:26 |
| zzzeek | mdbooth: nothing has changed :) | 13:26 |
| *** iamjarvo has quit IRC | 13:27 | |
| johnthetubaguy | zzzeek: mdbooth: thanks both, just going through things from the summit and found a note to follow up on this, I was totally missing context on that effort | 13:27 |
| dims_ | mriedem: with my 3 py34 reviews, we cover abour 20% of the tests that py27 covers | 13:28 |
| zzzeek | johnthetubaguy: enginefacade has been sitting in oslo.db queue with the chicken/egg problem of people not +2ing it b.c. nobody was using it, and nobody using it because it is not committed | 13:28 |
| *** rushiagr_away is now known as rushiagr | 13:28 | |
| johnthetubaguy | zzzeek: ah, oops | 13:29 |
| dims_ | zzzeek: last rebase was 3 days ago. will spend some time on it today | 13:29 |
| zzzeek | dims_: cool | 13:30 |
| johnthetubaguy | zzzeek: if we can get the two DB case done, and the nova spec from mdbooth approved, would that help that effort? | 13:30 |
| johnthetubaguy | its not quite used, but the intent to use it I guess | 13:30 |
| zzzeek | johnthetubaguy: that woudl be fine, i think there’s renewed interest in oslo on getting it +2’ed also | 13:31 |
| dims_ | zzzeek: if i remember right, there was an effort to try running nova with Depends-On against that review | 13:31 |
| mriedem | dims_: cool | 13:31 |
| zzzeek | dims_: *that*s what i was looking for the other day! | 13:31 |
| dims_ | or some other project, just to be sure it works since it's a large patch | 13:32 |
| zzzeek | dims_: where does “depends-on” go ? | 13:32 |
| *** iamjarvo has joined #openstack-nova | 13:32 | |
| dims_ | in your nova review, you can add 'Depends-On: xyz' in the commit message | 13:32 |
| *** VW has joined #openstack-nova | 13:33 | |
| *** gholler__ has joined #openstack-nova | 13:33 | |
| dims_ | zzzeek: to be fair its a bit complicated than that since nova picks up oslo.db released version | 13:34 |
| dims_ | zzzeek: let me setup some reviews for you to play with | 13:34 |
| dims_ | let's hop to oslo channel | 13:34 |
| zzzeek | johnthetubaguy / mdbooth : an example of running a subset of calls under a totally different database URL is in this test: https://review.openstack.org/gitweb?p=openstack/oslo.db.git;a=blob;f=oslo_db/tests/sqlalchemy/test_enginefacade.py;h=1a4c992593003c36ca5ec21dcfaf2635ff58c627;hb=5e0d0b98f771ec711e9b5e6a738c5edd8b2e39a5#l1454 | 13:34 |
| *** marun_ has joined #openstack-nova | 13:34 | |
| *** thangp has joined #openstack-nova | 13:36 | |
| openstackgerrit | garyk proposed openstack/nova: Compute: be consistent with logs about NotImplemented methods https://review.openstack.org/188392 | 13:37 |
| garyk | + | 13:37 |
| *** ajayaa has joined #openstack-nova | 13:38 | |
| *** ferest has quit IRC | 13:40 | |
| *** kiran-r has quit IRC | 13:41 | |
| *** garyk has quit IRC | 13:42 | |
| *** nihilifer has joined #openstack-nova | 13:43 | |
| *** adrian_otto has joined #openstack-nova | 13:44 | |
| *** thiagop has joined #openstack-nova | 13:45 | |
| *** nihilifer has quit IRC | 13:46 | |
| alex_xu | johnthetubaguy: hi, can I ask what is our expectation for line 39 in https://review.openstack.org/#/c/187272/3/priorities/liberty-priorities.rst briefly? | 13:46 |
| *** nihilifer has joined #openstack-nova | 13:46 | |
| johnthetubaguy | alex_xu: its from the summit session basically | 13:47 |
| *** eharney has joined #openstack-nova | 13:47 | |
| johnthetubaguy | on flavors | 13:47 |
| *** abhishekk has quit IRC | 13:47 | |
| *** marun has joined #openstack-nova | 13:49 | |
| *** marun_ has quit IRC | 13:49 | |
| *** pbandzi has quit IRC | 13:49 | |
| *** sahid has quit IRC | 13:50 | |
| alex_xu | johnthetubaguy: expect flavors extra specs and host aggregate properites like imagemeta object https://github.com/openstack/nova/blob/master/nova/objects/image_meta.py#L37 ? | 13:51 |
| *** adrian_otto has quit IRC | 13:54 | |
| johnthetubaguy | alex_xu: yeah, but we need to do more | 13:54 |
| openstackgerrit | Dan Smith proposed openstack/nova: Handle FlavorNotFound when augmenting migrated flavors https://review.openstack.org/187227 | 13:54 |
| alex_xu | johnthetubaguy: ok, thanks, let me dig into etherpad more | 13:55 |
| *** oro_ has joined #openstack-nova | 13:58 | |
| mdbooth | zzzeek: I think I remember what the problem was... | 13:59 |
| mdbooth | zzzeek: Sorry, oslo would be more appropraite | 13:59 |
| mriedem | johnthetubaguy: let's abandon this https://review.openstack.org/#/c/131597/ | 14:04 |
| mriedem | no activity since december | 14:04 |
| openstackgerrit | Petrut Lucian proposed openstack/nova: Hyper-V: Retry destroying instances in case of exceptions https://review.openstack.org/188408 | 14:05 |
| *** samueldmq has joined #openstack-nova | 14:06 | |
| *** iamjarvo has quit IRC | 14:08 | |
| *** sdake has joined #openstack-nova | 14:09 | |
| johnthetubaguy | mriedem: ah, yeah, +1, good spot | 14:10 |
| openstackgerrit | Andrey Kurilin proposed openstack/nova-specs: Nova API Microversions support in NovaClient https://review.openstack.org/188410 | 14:10 |
| mriedem | i might actually do some code reviews today | 14:11 |
| *** sdake_ has joined #openstack-nova | 14:11 | |
| *** otter768 has joined #openstack-nova | 14:11 | |
| mriedem | feeling a bit randy | 14:11 |
| johnthetubaguy | alex_xu: so I have an action to write all that up into devref if that helps | 14:11 |
| johnthetubaguy | mriedem: heh | 14:11 |
| *** adrian_otto has joined #openstack-nova | 14:11 | |
| *** mohitsharma has quit IRC | 14:11 | |
| alex_xu | johnthetubaguy: cool, waiting for that :) | 14:11 |
| mriedem | lxsli: you added this to the trivial bug fix list but you're not +1 on it https://review.openstack.org/#/c/177084/ | 14:12 |
| *** iamjarvo has joined #openstack-nova | 14:13 | |
| andreykurilin | sdague: I just send to review a spec for microversions https://review.openstack.org/#/c/188410 . Please take a look, when you will have a bit of free time :) | 14:13 |
| *** nickchase has joined #openstack-nova | 14:13 | |
| andreykurilin | alex_xu: ^ | 14:14 |
| *** IanGovett has quit IRC | 14:14 | |
| *** sdake has quit IRC | 14:15 | |
| alex_xu | andreykurilin: got it, will help on review | 14:15 |
| andreykurilin | alex_xu: thanks | 14:15 |
| johnthetubaguy | alex_xu andreykurilin have you folks seen the ironic spec around clients? | 14:15 |
| andreykurilin | johnthetubaguy: this spec is based on ironic one:) | 14:15 |
| *** otter768 has quit IRC | 14:16 | |
| johnthetubaguy | andreykurilin: sweet, good answer :D | 14:16 |
| *** mtanino has joined #openstack-nova | 14:16 | |
| johnthetubaguy | andreykurilin: heh, yeah, I see the familiar pattern now, cools | 14:16 |
| alex_xu | johnthetubaguy: yea, and the only different between nova and ironic is return min/max version header when requested header didn't supported. | 14:16 |
| andreykurilin | most of changes look like "s/Ironic/Nova/d", "s/ironicclient/novaclient/d" and so on:) | 14:17 |
| johnthetubaguy | alex_xu: yep thats cool, it would be interesting to see if we loose any abilities because of that | 14:17 |
| *** marun has quit IRC | 14:18 | |
| *** marun has joined #openstack-nova | 14:18 | |
| *** dulek has joined #openstack-nova | 14:19 | |
| *** cfriesen has joined #openstack-nova | 14:20 | |
| *** iamjarvo has quit IRC | 14:20 | |
| *** mtanino_ has joined #openstack-nova | 14:21 | |
| *** jecarey has joined #openstack-nova | 14:21 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Fix an objects layering violation in compute/api https://review.openstack.org/188110 | 14:22 |
| openstackgerrit | Dan Smith proposed openstack/nova: Replace metaclass registry with explicit opt-in registry from oslo https://review.openstack.org/188111 | 14:22 |
| openstackgerrit | Dan Smith proposed openstack/nova: Remove assertRemotes() from objects tests https://review.openstack.org/188109 | 14:22 |
| openstackgerrit | Dan Smith proposed openstack/nova: Convert test objects to new field formats https://review.openstack.org/187719 | 14:22 |
| openstackgerrit | Dan Smith proposed openstack/nova: Begin the transition to an explicit object registry https://review.openstack.org/187718 | 14:22 |
| openstackgerrit | Dan Smith proposed openstack/nova: Use fields from oslo.versionedobjects https://review.openstack.org/187720 | 14:22 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make readonly field tests use exception from o.vo https://review.openstack.org/188112 | 14:22 |
| *** mtanino has quit IRC | 14:23 | |
| *** moshele has quit IRC | 14:23 | |
| *** eharney_ has joined #openstack-nova | 14:25 | |
| *** sdake has joined #openstack-nova | 14:25 | |
| *** tonytan4ever has joined #openstack-nova | 14:25 | |
| *** matrohon has quit IRC | 14:25 | |
| *** eharney has quit IRC | 14:26 | |
| *** eharney_ is now known as eharney | 14:26 | |
| *** matrohon has joined #openstack-nova | 14:27 | |
| *** sdake_ has quit IRC | 14:28 | |
| *** dboik has joined #openstack-nova | 14:28 | |
| *** unicell1 has joined #openstack-nova | 14:29 | |
| *** unicell has quit IRC | 14:29 | |
| johnthetubaguy | zzzeek: i don't want to open old wounds... but did you have any thoughts about on moving to DB thread pools talking to C libs vs moving to pymysql? | 14:30 |
| zzzeek | johnthetubaguy: certainly going to pymysql is more expedient | 14:30 |
| johnthetubaguy | zzzeek: yeah, I think we probably have to go there as step 1, given the python 3.4 push | 14:31 |
| *** armax has joined #openstack-nova | 14:31 | |
| zzzeek | johnthetubaguy: id probably stick with that unless there’s really motivation to change the architecture of the db api | 14:31 |
| johnthetubaguy | zzzeek: I guess we see how that performs and if we need something better, we look at the thread pool an C lib again? | 14:31 |
| zzzeek | johnthetubaguy: sure. but pymysql should perform fine | 14:32 |
| *** dulek has quit IRC | 14:32 | |
| *** dulek_ has joined #openstack-nova | 14:32 | |
| *** dulek_ has quit IRC | 14:32 | |
| johnthetubaguy | zzzeek: OK, I keep hearing conflicting things there, I suspect is improved somewhat since it was last tested, or something like that | 14:32 |
| *** dulek_ has joined #openstack-nova | 14:32 | |
| *** eharney has quit IRC | 14:32 | |
| johnthetubaguy | zzzeek: good to hear it should be OK though, I got confused in where all the ML threads had landed in that respect | 14:33 |
| *** garyk has joined #openstack-nova | 14:33 | |
| openstackgerrit | Adrien Vergé proposed openstack/nova: Fix API misimplementation for security groups https://review.openstack.org/183909 | 14:34 |
| zzzeek | johnthetubaguy: pymysql is a slower driver than the C based ones but the scale we are talking about is an order of magnitude less than what would be apparent in nova | 14:34 |
| *** eharney has joined #openstack-nova | 14:34 | |
| sdague | johnthetubaguy, and any other nova folks interested. This is what I've got - https://dague.net/?p=4461&shareadraft=baba4461_557061c963266 for communication about our API changes. It probably needs editing (it always does), but comments are welcome from a missing concept, or tone perspective. I'll come back to it this afternoon for typo and grammar fixes. | 14:35 |
| *** afazekas_mtg has joined #openstack-nova | 14:35 | |
| *** electrocucaracha has joined #openstack-nova | 14:36 | |
| * gilliard looks | 14:36 | |
| gilliard | Thanks sdague | 14:36 |
| *** mtanino_ has quit IRC | 14:39 | |
| johnthetubaguy | sdague: so the validation relax stuff makes v2.1 min version != v2.0 API (thinking last paragraph of nova v2.1) | 14:40 |
| openstackgerrit | Matthew Gilliard proposed openstack/nova-specs: Query and cancel ongoing live migrations https://review.openstack.org/179149 | 14:40 |
| *** iamjarvo has joined #openstack-nova | 14:40 | |
| sdague | johnthetubaguy: right, that's true. Do you think the detail is important for this? | 14:41 |
| *** exploreshaifali has quit IRC | 14:41 | |
| sdague | I guess I should probably have some pointers to the spec towards the end for people that want to dig deeper | 14:41 |
| johnthetubaguy | sdague: probably not, just thinking of addding "(but with some extra validation)" after its like v2.0 API, maybe? although its just me being a pedant really | 14:42 |
| johnthetubaguy | sdague: looking good by the way | 14:42 |
| *** _gryf is now known as _gryf|afk | 14:42 | |
| *** mtanino has joined #openstack-nova | 14:43 | |
| *** afazekas_mtg has quit IRC | 14:43 | |
| *** gszasz has quit IRC | 14:44 | |
| gilliard | does "spanner" mean anything to americans? To me it both a "wrench", and an "idiot". So "Sophie the Cloud Spanner" is a funny name :) | 14:44 |
| gilliard | hypenate? | 14:44 |
| dansmith | gilliard: no idiot connotation here, AFAIK | 14:45 |
| *** claudiub|2 has quit IRC | 14:45 | |
| johnthetubaguy | sdague: so, in summary, thats awesome, I didn't spot any typos and things, but I am a dyslexic mole when it comes to those kinds of things! | 14:46 |
| *** apuimedo has quit IRC | 14:47 | |
| dansmith | johnthetubaguy: you and sdague not being able to find each others' typos seems completely unsurprising to me :P | 14:47 |
| johnthetubaguy | dansmith: heh | 14:47 |
| ericksonsantos | Does someone know how to use keystoneclient from Nova? I want to call client.projects.get(), but I don't know the right way to instantiate keystoneclient | 14:48 |
| * alex_xu decide to take a shower, then reading Sean's blog on the bed... | 14:48 | |
| johnthetubaguy | dansmith: its like using those letter things, thats good right? | 14:48 |
| dansmith | johnthetubaguy: yeah, just string em together until it sounds right | 14:48 |
| johnthetubaguy | dansmith: :) | 14:49 |
| sdague | gilliard: if you have a better idea for a name, let me know. It was the best my brain came up with yesterday. | 14:49 |
| openstackgerrit | Adrien Vergé proposed openstack/nova: Fix API misimplementation for security groups https://review.openstack.org/183909 | 14:49 |
| johnthetubaguy | sdague: spanner -> wrangler ? | 14:49 |
| sdague | maybe just "Multi Cloud User" ? | 14:49 |
| sdague | or I guess Cloud Herder or something. Trying to have a quick quip that she works with applications that use multiple clouds. | 14:51 |
| sdague | which is described right after | 14:51 |
| mriedem | scruffy nerve herder? | 14:51 |
| sdague | mriedem: heh | 14:51 |
| gilliard | Works for me. | 14:51 |
| sdague | gilliard: which one works for you :) | 14:51 |
| mriedem | i guess it is 'nerf' herder | 14:51 |
| sdague | there were a bunch out there | 14:51 |
| mriedem | i always heard that but thought nerf was wrong | 14:51 |
| gilliard | oh, I thought multi cloud user was nice :) | 14:52 |
| *** sudipto has joined #openstack-nova | 14:53 | |
| sdague | gilliard: ok, I'll go with that | 14:53 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: Disable NUMA for broken libvirt https://review.openstack.org/178188 | 14:53 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: log when BAD_LIBVIRT_NUMA_VERSIONS detected https://review.openstack.org/188427 | 14:53 |
| *** eglynn has joined #openstack-nova | 14:53 | |
| *** nelsnels_ has joined #openstack-nova | 14:54 | |
| gilliard | sdague: overall I think it's great. We've been missing background and definitive statements like this for a while. | 14:55 |
| gilliard | Would we welcome patches which remove if/else switching on extensions being enabled now? | 14:55 |
| mriedem | gilliard: in v2 api? | 14:56 |
| *** nelsnelson has quit IRC | 14:56 | |
| *** claudiub has joined #openstack-nova | 14:57 | |
| gilliard | Well I guess so. If v2 == v2.1-w/out-validation | 14:57 |
| mriedem | but v2 is not v2.1 technically in the tree, right | 14:57 |
| mriedem | so someone talking to the v2 api with extensions loaded or not is still using the v2 code path | 14:57 |
| mriedem | not the v3 path | 14:57 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: libvirt: make live_migration_uri flag dependent on virt_type https://review.openstack.org/175780 | 14:58 |
| mriedem | v2.1 is the v3 code path and v2.1 specific is meant to be compatible with v2 | 14:58 |
| mriedem | at least that's my understanding | 14:58 |
| *** VW has quit IRC | 14:58 | |
| gilliard | So there's a hole in my understanding here I think. | 14:58 |
| *** nihilifer has quit IRC | 14:59 | |
| *** VW has joined #openstack-nova | 14:59 | |
| *** VW has quit IRC | 14:59 | |
| gilliard | I guess I only mean from the v2.1 code paths, in that case. | 15:00 |
| *** VW has joined #openstack-nova | 15:00 | |
| johnthetubaguy | gilliard: yeah, I think we can remove those bits | 15:00 |
| mriedem | gilliard: do those exist in the v2.1 code paths? | 15:00 |
| johnthetubaguy | gilliard: sounds stupid, but might need a spec | 15:00 |
| johnthetubaguy | mriedem: I was just looking I think there are some still... | 15:00 |
| gilliard | I'd have to look. It was mentioned in the summit a couple of times. | 15:00 |
| mriedem | ok, weird | 15:00 |
| *** sdake has quit IRC | 15:00 | |
| mriedem | do we say in v2.1 that all extensions are loaded? | 15:01 |
| mriedem | sorry, haven't read the manifesto yet | 15:01 |
| gilliard | ^^ That's the gap in my understanding, actually. | 15:01 |
| mriedem | i'm all for removing the if/else ext loading logic in v2.1 code paths | 15:01 |
| mriedem | i do'nt know why you wouldn't have extensions loaded, since you can control access through the policy file | 15:01 |
| johnthetubaguy | mriedem: its yet to be officially agreed in the specs repo, as I understand it | 15:01 |
| johnthetubaguy | mriedem: gilliard: its these bits right? https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/plugins/v3/servers.py#L962 | 15:02 |
| johnthetubaguy | I think thats what replaced this bit: https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L981 | 15:02 |
| *** primechuck has joined #openstack-nova | 15:02 | |
| primechuck | Is there an admin API to migrate a VM off a host for maintenance? Or otherwise issue an evacuation for all VMs on a host? | 15:03 |
| mriedem | johnthetubaguy: i'm not aware of that i guess, i don't see an in-tree server_rebuild extension, so is this something someone else can plug in? | 15:03 |
| johnthetubaguy | primechuck: there are several APIs, and python-novaclient has a helper method to do that for all VMs on a host, if I remember correctly | 15:03 |
| johnthetubaguy | primechuck: depends on exactly the situation you are in, we probably need some better docs around that, migrate vs live-migrate vs evacuate, basically | 15:04 |
| mriedem | oh here it is https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/plugins/v3/preserve_ephemeral_rebuild.py | 15:04 |
| *** agireud has joined #openstack-nova | 15:04 | |
| sdague | johnthetubaguy: I thought we were all agreed there on the extensions all loaded path, we just needed to start enforcing it for real | 15:04 |
| johnthetubaguy | sdague: we are, just not on paper | 15:04 |
| mriedem | virtual paper | 15:04 |
| mriedem | i agree with requiring all extensions are loaded though | 15:05 |
| mriedem | remove the gorp | 15:05 |
| gilliard | Well I think that's worth adding to the blog then sdague | 15:05 |
| johnthetubaguy | -paper +git | 15:05 |
| *** belmoreira has quit IRC | 15:05 | |
| gilliard | primechuck: "r/nova/api/openstack/compute/servers.py#L981 | 15:05 |
| primechuck | I found the migrate and live-migrate processes, but it looks like there is no to move suspended VMs. | 15:05 |
| gilliard | ummm. I mean: primechuck: nova help host-evacuate | 15:05 |
| primechuck | Thanks, I'll take a look | 15:06 |
| johnthetubaguy | primechuck: what do you mean by suspended VMs though? | 15:06 |
| *** hemnafkj is now known as hemna | 15:06 | |
| edleafe | sdague: excellent post. If you want it proofread, I'd be happy to nitpick away :) | 15:06 |
| sdague | johnthetubaguy: ok, so who has the todo to write up that part of the policy to make the no extensions part not a lie :) | 15:07 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: libvirt: call host.get_capabilities after checking for bad numa versions https://review.openstack.org/188431 | 15:07 |
| johnthetubaguy | sdague: consults etherpad of doom | 15:07 |
| johnthetubaguy | sdague: ah, I haven't done the API bit yet | 15:07 |
| sdague | edleafe: sure, if you want to throw up an etherpad somewhere to capture them that would be great. I've been writing this for a couple of days, so I need to walk away from the computer for a bit and just my head clear. | 15:08 |
| *** prashant-upadhya has quit IRC | 15:08 | |
| dansmith | alaski: johnthetubaguy: So, now that flavor data is out of system_metadata, is there a desire to go back to joining that and metadata for more atomic queries and/or updates? | 15:08 |
| * johnthetubaguy goes back to etherpad of doom | 15:08 | |
| sdague | I'll do a typo fix bit towards the end of my day | 15:08 |
| johnthetubaguy | dansmith: its worth a try | 15:08 |
| dansmith | johnthetubaguy: well, I'd hate to do it if it turns out to be bad (again) and then just have to revert | 15:09 |
| primechuck | The use case is, a nova node is being removed from cloud. It might have VMs on it in various states with various tennants. There doesn't seem to be a single way to move all VMs, including VMs that are in a suspended state. | 15:09 |
| johnthetubaguy | dansmith: yeah, I wonder if there is a way we could try that out easily, I guess there should be | 15:09 |
| dansmith | johnthetubaguy: like, if you guys could do some perf guessing or something | 15:09 |
| dansmith | primechuck: this is really a usage question, so please see topic | 15:09 |
| edleafe | sdague: ok, will do. I'll get to it this morning. | 15:10 |
| sdague | edleafe: thanks! | 15:10 |
| *** romainh has left #openstack-nova | 15:10 | |
| primechuck | Evacuating a nova node is usage? | 15:10 |
| primechuck | Oh, sorry, dev change | 15:10 |
| dansmith | primechuck: absolutely | 15:10 |
| *** primechuck has left #openstack-nova | 15:11 | |
| johnthetubaguy | dansmith: so it was the pure bandwidth before, I guess we need this almost everywhere, so it makes no difference skipping it... I think before it was system_meta being joined to an instance that caused the issue with the bandwidth, or something like that | 15:11 |
| *** alexpilotti has joined #openstack-nova | 15:11 | |
| johnthetubaguy | dansmith: having typed that I realised how little I remember about the issue now | 15:12 |
| dansmith | johnthetubaguy: right, the row inflation of (sizeof(instance) * rows(system_metadata)) that was the problem | 15:12 |
| *** ssurana has joined #openstack-nova | 15:12 | |
| *** claudiub has quit IRC | 15:12 | |
| dansmith | johnthetubaguy: i.e. instance is big, and system_metadata was many | 15:12 |
| dansmith | johnthetubaguy: however, once we saw that we realized that user-controlled metadata could cause a similar inflation | 15:12 |
| dansmith | johnthetubaguy: it's quota'd so not unbounded, but still, you can add tons of metadata to all your instances and put stress on the db->nova relationship | 15:13 |
| *** annegentle has joined #openstack-nova | 15:13 | |
| *** eharney has quit IRC | 15:13 | |
| mriedem | quota is 255 i think | 15:13 |
| dansmith | johnthetubaguy: I'm not arguing for going back to joining for either, but was reading a db-related change this morning and it made me think of how we've removed the major row explosion in sysmeta now | 15:13 |
| dansmith | so I thought I'd ask | 15:13 |
| dansmith | mriedem: if that's the default, then yeah, joining with that could be, like, bad and stuff :) | 15:14 |
| *** claudiub has joined #openstack-nova | 15:14 | |
| mriedem | for totally | 15:14 |
| *** gmann has quit IRC | 15:14 | |
| mriedem | oh i guess it's 128 but still | 15:14 |
| dansmith | I also think some providers stuff naughty bits into sysmeta, so maybe joining just because the upstream case is okay is just asking for trouble | 15:14 |
| dansmith | yeah, 30 was what made the pain so bad | 15:14 |
| dansmith | so 128 is infinity bad :) | 15:14 |
| mriedem | i know people that stuff stuff into sysmeta | 15:15 |
| dansmith | I bet you do :) | 15:15 |
| dansmith | johnthetubaguy: anyway, maybe something to keep in the back of our minds | 15:17 |
| johnthetubaguy | dansmith: right it was the multiplication thing | 15:17 |
| *** marun has quit IRC | 15:17 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/nova-specs: Use os-brick library https://review.openstack.org/184360 | 15:18 |
| johnthetubaguy | dansmith: yeah, I guess while we work fine without it, that seems cool | 15:18 |
| *** marun has joined #openstack-nova | 15:18 | |
| *** gmann has joined #openstack-nova | 15:19 | |
| johnthetubaguy | dansmith: FWIW, seen IAD, SYD and HKG take that code now, and it seems the sky is not falling down, so thats cool | 15:19 |
| dansmith | johnthetubaguy: that's britiish for "everything is working" right? | 15:19 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova-specs: WIP: Propose to support spot instances https://review.openstack.org/104883 | 15:19 |
| *** nelsnelson has joined #openstack-nova | 15:20 | |
| dansmith | johnthetubaguy: I think we've got all those changes applied or proposed to stable/kilo, so we should be good I think | 15:20 |
| dansmith | johnthetubaguy: thanks for finding that stuff, sorry it didn't get caught in pre-prod though :) | 15:20 |
| alaski | just got out of a meeting. But I would wait a bit before going back to default joining | 15:20 |
| alaski | I was just looking at a graph that shows our db load is up with the flavor migration code in place, well up since our last deploy | 15:21 |
| johnthetubaguy | dansmith: yeah, not done the force migration bit yet though | 15:21 |
| johnthetubaguy | alaski: ah, maybe I spoke too soon | 15:21 |
| alaski | the sky isn't falling, so you're right | 15:22 |
| *** nelsnels_ has quit IRC | 15:22 | |
| dansmith | alaski: well, that's not surprising right? | 15:22 |
| alaski | I think we'll handle it fine, but I don't want to add more speculatively | 15:22 |
| alaski | dansmith: right. I'm not overly concerned, just don't want to add more yet if not necessary | 15:22 |
| dansmith | alaski: this is live migrations happening and will subside, right? | 15:22 |
| johnthetubaguy | alaski: but it might in when the F5 gets in the way, but yeah, will find out soon | 15:22 |
| johnthetubaguy | dansmith: that was totally my hope | 15:22 |
| johnthetubaguy | I never dug enough to find out really | 15:23 |
| alaski | dansmith: we went from a steady-ish line to a waveform, and the amplitude is decreasing over time | 15:23 |
| alaski | but the average is still up | 15:23 |
| dansmith | alaski: decreasing waveform makes sense for migrations, but yeah | 15:23 |
| johnthetubaguy | alaski: how was the bandwidth rather than the query count? | 15:24 |
| dansmith | alaski: oh, also, we're doing another db thing that we weren't | 15:24 |
| openstackgerrit | Neil Jerram proposed openstack/nova: Add new VIF type VIF_TYPE_TAP https://review.openstack.org/146914 | 15:24 |
| dansmith | alaski: so another query at least for save() | 15:24 |
| alaski | I suspect the overall increase is the code looking for whether or not to migrate, and we'll do better when that's gone. | 15:24 |
| *** mwagner_afk has quit IRC | 15:24 | |
| alaski | dansmith: right | 15:24 |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: rename parallels driver to virtuozzo https://review.openstack.org/184311 | 15:25 |
| alaski | johnthetubaguy: not sure if I have that available | 15:25 |
| *** eharney has joined #openstack-nova | 15:25 | |
| dansmith | alaski: so that, plus removing the migration code will be better I expect | 15:25 |
| alaski | dansmith: agreed | 15:25 |
| johnthetubaguy | alaski: ah, its in graphite, I dug it out for HKG if we need that | 15:25 |
| Jeffrey4l | any guy have time to review my PS https://review.openstack.org/#/c/187657/。 It fixed the all_tenants param in security group when using neutron backend. | 15:25 |
| dansmith | alaski: speaking of reducing bandwidth: https://github.com/kk7ds/openstack-gerrit-dashboard/commit/5f9fed52d88693242ed41aee0b441dd4b94a0b3a | 15:26 |
| *** nelsnels_ has joined #openstack-nova | 15:26 | |
| mriedem | huh, liberty conductor supports icehouse compute? | 15:26 |
| alaski | johnthetubaguy: I had never looked at graphite until about 10 minutes ago. But just found it | 15:26 |
| johnthetubaguy | alaski: there are some gems in there | 15:26 |
| alaski | johnthetubaguy: bandwidth is slightly increased, not worringly so IMO | 15:27 |
| *** nelsnelson has quit IRC | 15:27 | |
| alaski | dansmith: ahh, nice | 15:27 |
| johnthetubaguy | alaski: thats was I saw too in the first region we did, sweet | 15:27 |
| dansmith | johnthetubaguy: any chance you guys could share a snapshot of that graph? | 15:27 |
| *** prashant-upadhya has joined #openstack-nova | 15:27 | |
| johnthetubaguy | alaski: I think CPU io wait and bandwidth are usually the critical bits, but I could be wrong | 15:27 |
| dansmith | even if you snip the units (if you have to) | 15:27 |
| *** sudipto has quit IRC | 15:28 | |
| johnthetubaguy | dansmith: not been done before, but we should find a way to make that happen | 15:29 |
| dansmith | johnthetubaguy: not a huge deal, just would like to see it spanning this change | 15:29 |
| mriedem | Jeffrey4l: looks like you borrowed heavily from the previous (now abandoned) change https://review.openstack.org/#/c/81237/ | 15:30 |
| johnthetubaguy | dansmith: gotcha | 15:30 |
| mriedem | Jeffrey4l: you should probably give some co-author credit in your change | 15:30 |
| Jeffrey4l | mriedem, ok u r right. I will add that. | 15:30 |
| openstackgerrit | Andrey Kurilin proposed openstack/nova-specs: Nova API Microversions support in NovaClient https://review.openstack.org/188410 | 15:30 |
| mriedem | Jeffrey4l: alternatively we could restore the orginal change and you could take that over | 15:32 |
| mriedem | up to you | 15:32 |
| *** dulek_ has quit IRC | 15:32 | |
| *** nelsnelson has joined #openstack-nova | 15:33 | |
| *** nelsnels_ has quit IRC | 15:33 | |
| *** marun has quit IRC | 15:34 | |
| *** marun has joined #openstack-nova | 15:34 | |
| *** nelsnels_ has joined #openstack-nova | 15:35 | |
| *** angdraug has joined #openstack-nova | 15:35 | |
| *** nelsnels_ has quit IRC | 15:35 | |
| *** nelsnelson has quit IRC | 15:38 | |
| *** nelsnelson has joined #openstack-nova | 15:38 | |
| *** artom has quit IRC | 15:39 | |
| *** artom has joined #openstack-nova | 15:39 | |
| *** dims_ has quit IRC | 15:43 | |
| *** nelsnelson has quit IRC | 15:44 | |
| mriedem | dansmith: either this is wrong or i totally don't understand it https://review.openstack.org/#/c/173723/ | 15:45 |
| *** nelsnelson has joined #openstack-nova | 15:47 | |
| dansmith | mriedem: I actually tried to figure that out on this patch a bit ago and ... didn't (and then got distracted) | 15:48 |
| dansmith | so lets see what he says | 15:48 |
| *** lpetrut1 has left #openstack-nova | 15:48 | |
| mriedem | dansmith: it's on the trivial bug list and has a slew of +1s but i don't understand it for the life of me | 15:48 |
| *** lpetrut1 has joined #openstack-nova | 15:48 | |
| mriedem | i suspect no one else does either | 15:48 |
| *** nelsnelson has quit IRC | 15:48 | |
| mriedem | but i'm not afraid of looking dumb and asking obvious questions | 15:48 |
| *** nelsnelson has joined #openstack-nova | 15:49 | |
| dansmith | mriedem: it probably needs to come off the trivial list, tbh | 15:49 |
| *** lpetrut1 has quit IRC | 15:49 | |
| mriedem | doing that now | 15:49 |
| dansmith | trivial does not mean "few lines" | 15:49 |
| dansmith | IMHO | 15:49 |
| *** alex_klimov has quit IRC | 15:50 | |
| mriedem | agreed | 15:50 |
| mriedem | 'allow non-admin to stop services for bug xyz, it's only 2 lines' | 15:51 |
| *** tjones1 has joined #openstack-nova | 15:52 | |
| openstackgerrit | Jeffrey Zhang proposed openstack/nova: Support all_tenants search_opts for neutron https://review.openstack.org/187657 | 15:52 |
| cfriesen | for upgrades do we have a defined procedure? like do we always update nova-scheduler/nova-conductor before the nova-computes? I'm wondering if I can extend the interpretation of an existing text field in the DB (written to by nova-compute, read by nova-scheduler). | 15:52 |
| mriedem | cfriesen: there is a devref on upgrades | 15:53 |
| mriedem | http://docs.openstack.org/developer/nova/devref/upgrade.html | 15:53 |
| mriedem | there is probably also a thing in docs.o.o somewhere | 15:53 |
| mriedem | http://docs.openstack.org/openstack-ops/content/ops_upgrades-general-steps.html | 15:53 |
| mriedem | assuming you could follow this http://docs.openstack.org/openstack-ops/content/upgrade-icehouse-juno.html and treat it the same for juno->kilo | 15:55 |
| cfriesen | yeah, that would work. basically I want to take the existing cpu_info dict in the compute-nodes table and extend it to store a few more things. looks like that should be possible without changing the actual schema | 15:57 |
| *** baoli has quit IRC | 15:59 | |
| mriedem | fork at your own risk, sure | 16:00 |
| *** baoli has joined #openstack-nova | 16:00 | |
| *** VW has quit IRC | 16:01 | |
| *** jistr has quit IRC | 16:01 | |
| *** sdake has joined #openstack-nova | 16:02 | |
| *** ssurana has quit IRC | 16:04 | |
| *** iamjarvo has quit IRC | 16:05 | |
| *** cknight has joined #openstack-nova | 16:06 | |
| *** jlanoux has quit IRC | 16:07 | |
| *** unicell1 has quit IRC | 16:07 | |
| *** VW has joined #openstack-nova | 16:08 | |
| *** annegentle has quit IRC | 16:08 | |
| cfriesen | mriedem: no, this would be for something to push upstream | 16:08 |
| *** jaypipes has joined #openstack-nova | 16:08 | |
| dansmith | johnthetubaguy: I didn't think we "agreed" to rename evacuate | 16:09 |
| dansmith | it was discussed, but it seemed far from consensus to me | 16:10 |
| johnthetubaguy | dansmith: so I guess we didn't | 16:10 |
| cfriesen | dansmith: I thought we had consensus | 16:10 |
| dansmith | I agree that evacuate is the wrong name, but completely renaming an api is a pretty big deal too | 16:10 |
| johnthetubaguy | dansmith: I was thinking an alias available in a new micro-version, and in python-novaclient, but it would be a first | 16:11 |
| *** dane-fichter has quit IRC | 16:11 | |
| cfriesen | dansmith: add a new alias, leave the old name in for backwards compatibility? | 16:11 |
| johnthetubaguy | dansmith: lets split that effort | 16:11 |
| *** otter768 has joined #openstack-nova | 16:11 | |
| dansmith | an alias doesn't make it better, IMHO | 16:12 |
| dansmith | with microversions we can just make the change >= a given version | 16:12 |
| dansmith | the alias is just confusing IMHO | 16:12 |
| dansmith | that said, I don't think it needs to be part of my robustify spec | 16:12 |
| cfriesen | dansmith: I thought we some people had issues with removing stuff completely with just a microversion | 16:12 |
| johnthetubaguy | its certainly clean to change it | 16:12 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: libvirt: Don't try to confine a non-NUMA instance https://review.openstack.org/188468 | 16:12 |
| johnthetubaguy | dansmith: yeah, lets un hook those too efforts, I combined them in my head for some reason | 16:13 |
| dansmith | cfriesen: there are certainly issues with it, which is why I don't think it's worth it | 16:13 |
| dansmith | but an alias confuses the issue because it says: | 16:13 |
| dansmith | < 2.5 evacuate is evacuate | 16:13 |
| *** baoli has quit IRC | 16:13 | |
| dansmith | 2.5 < x < 2.10 evacuate and resurrect are the same | 16:13 |
| dansmith | > 2.10 evacuate is gone and resurrect is the only thing | 16:13 |
| dansmith | does nothing for us because we have the expressive nature of the version | 16:14 |
| *** baoli has joined #openstack-nova | 16:14 | |
| dansmith | johnthetubaguy: I do need to add the thing alaski wants, which is require (if configured) the host-down confirmation before allowing evacuate | 16:14 |
| dansmith | that makes sense to be in my spec I think | 16:15 |
| *** danpb1 has joined #openstack-nova | 16:15 | |
| johnthetubaguy | dansmith: yeah, changing the acton items to match that | 16:15 |
| johnthetubaguy | cfriesen: did you fancy driving the rename in a spec, or at least starting the discussion? | 16:15 |
| *** danpb has quit IRC | 16:16 | |
| *** claudiub has quit IRC | 16:16 | |
| *** otter768 has quit IRC | 16:17 | |
| johnthetubaguy | dansmith: cfriesen: so I was thinking a rename would be dumb, but my logic was bogus... I was thinking you rename in v2.5 but someone wants a feature in v2.6, so they need to rework their old code, but thats rubbish, the old code can still request the older version | 16:17 |
| dansmith | johnthetubaguy: right | 16:17 |
| *** SourabhP has joined #openstack-nova | 16:17 | |
| dansmith | I wonder what jaypipes thinks about the worthwhileness of a rename | 16:18 |
| cfriesen | johnthetubaguy: not sure I can commit to that, I still owe you a spec for removing support for "min_count" and "max_count" when booting an instance. | 16:18 |
| bauzas | cfriesen: need help ? | 16:18 |
| johnthetubaguy | cfriesen: no worries | 16:19 |
| cfriesen | bauzas: for which one? | 16:19 |
| *** claudiub has joined #openstack-nova | 16:19 | |
| bauzas | cfriesen: for the spec | 16:19 |
| *** danpb1 has quit IRC | 16:19 | |
| cfriesen | bauzas: the min/max one? | 16:19 |
| bauzas | cfriesen: yup | 16:19 |
| cfriesen | bauzas: if you want to run with that, then go for it. :) I'm working on internal issues at the moment. | 16:19 |
| bauzas | cfriesen: uh, my question was about giving you some mentoring for the spec | 16:21 |
| cfriesen | bauzas: lol. | 16:21 |
| bauzas | cfriesen: I mean, I'm trying to help people contributing and that one seems quite cool to help | 16:22 |
| *** SourabhP has quit IRC | 16:22 | |
| cfriesen | bauzas: sure, that'd be great. I'll run it by you once I get a first draft written up | 16:22 |
| bauzas | cfriesen: np | 16:22 |
| bauzas | cfriesen: I'm trying to figure out which kind of features can be done easily, so it would help some contributors to ramp up, not only for bugfixign | 16:23 |
| cfriesen | bauzas: if we can get agreement on actually doing it, the implementation should be straightforward | 16:23 |
| bauzas | cfriesen: yup | 16:23 |
| bauzas | cfriesen: but the spec deserves some thinking :) | 16:23 |
| bauzas | oh, is openstackgerrit on PTO ? | 16:24 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements https://review.openstack.org/188500 | 16:24 |
| bauzas | because I just uploaded a huge series... | 16:24 |
| *** moshele has joined #openstack-nova | 16:25 | |
| jaypipes | dansmith: rename of...? | 16:26 |
| dansmith | jaypipes: evacuate to resurrect | 16:26 |
| dansmith | jaypipes: I contend that resurrect is a better name, but not worth the confusion and pain to users to actually change it | 16:27 |
| bauzas | alaski: following our convo https://review.openstack.org/#/c/188486 | 16:27 |
| dansmith | jaypipes: we do have the machinery in microversions to do it, I just don't think it's worth it | 16:27 |
| bauzas | alaski: maybe something really about discussing | 16:27 |
| *** yassine_ has quit IRC | 16:28 | |
| *** busterswt has quit IRC | 16:28 | |
| jaypipes | dansmith: ah, I see... yes, resurrect is a better name for it. Honestly, I think there *is* value in a rename, if not for anything but the documentation of it. | 16:28 |
| jaypipes | dansmith: but it's not high priority of course. | 16:28 |
| dansmith | jaypipes: there's value for sure, I just don't think it's enough value. But, okay | 16:28 |
| *** whenry has joined #openstack-nova | 16:29 | |
| jaypipes | dansmith: I can't count how many folks I've talked to just since *I* learned about evacuate's real purpose, that have been shocked it's not what they thought it was (live-migrate all instances from a host) | 16:29 |
| VirtualPCGuy | dansmith: "Evacuate" is confusing for new users | 16:29 |
| dansmith | VirtualPCGuy: I know it is | 16:29 |
| jaypipes | VirtualPCGuy: yep, dansmith agrees with you :) | 16:29 |
| jaypipes | VirtualPCGuy: he's just saying it's not super-important to change. | 16:29 |
| *** ildikov has quit IRC | 16:30 | |
| VirtualPCGuy | jaypipes: I spend a lot of my time teaching people how to use this stuff, so I have a different perspective on importance :) | 16:30 |
| dansmith | well, not just not super important, but not worth it | 16:31 |
| dansmith | like, I think if we made the rename in the cli portion of the client, | 16:31 |
| *** jcru has joined #openstack-nova | 16:31 | |
| *** iamjarvo has joined #openstack-nova | 16:31 | |
| ndipanov | jaypipes, yeah - naming tings is kind of important if you deal with other humans | 16:31 |
| dansmith | I think that would get 90% of the uses | 16:31 |
| *** iamjarvo has quit IRC | 16:31 | |
| cfriesen | dansmith: and in the GUI, presumably | 16:32 |
| dansmith | cfriesen: horizon, yeah | 16:32 |
| dansmith | (if it's even in horizon) | 16:32 |
| mriedem | can we just call it compute operation 23b | 16:32 |
| jaypipes | heh | 16:32 |
| VirtualPCGuy | IMHO - while renaming in CLI, GUI is safer now, it will increase pressure to rename the API in the future. | 16:32 |
| dansmith | mriedem: that would be less confusing :) | 16:32 |
| *** sajeesh has joined #openstack-nova | 16:33 | |
| *** aix has quit IRC | 16:33 | |
| VirtualPCGuy | Mriedem: I have always said all APIs should be GUIDs with a big lookup table. Then we could change the description at will :P | 16:33 |
| dansmith | VirtualPCGuy: we have lots of things that are inconsistent between the cli and the api, and have been for some time | 16:33 |
| *** matrohon has quit IRC | 16:33 | |
| VirtualPCGuy | dansmith: aware of that :) | 16:33 |
| *** yamahata has joined #openstack-nova | 16:36 | |
| *** marzif_ has joined #openstack-nova | 16:37 | |
| *** ssurana has joined #openstack-nova | 16:38 | |
| *** lpetrut has joined #openstack-nova | 16:38 | |
| *** dims_ has joined #openstack-nova | 16:39 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs: api: relax validation for unversioned requests https://review.openstack.org/173243 | 16:39 |
| mriedem | jaypipes: btw, news on your favorite dog | 16:39 |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: rename parallels driver to virtuozzo https://review.openstack.org/184311 | 16:39 |
| *** Jeffrey4l has quit IRC | 16:39 | |
| *** amotoki has joined #openstack-nova | 16:39 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs: api: relax validation for unversioned requests https://review.openstack.org/173243 | 16:39 |
| mriedem | jaypipes: laura had him out w/o a leash on and ding dong mosied over to the neighbor's house to say high to their dogs - he came back with a punctured left shoulder which is now infected and oozing blood and puss | 16:40 |
| mriedem | *hi | 16:40 |
| *** iamjarvo has joined #openstack-nova | 16:41 | |
| jaypipes | mriedem: :( that's awful! | 16:41 |
| *** iamjarvo has quit IRC | 16:41 | |
| mriedem | he was super doped up on pain meds last night, i've never seen a dog like that, pretty funny. he wouldn't even eat the food i put right in front of him. | 16:41 |
| mriedem | stoned out of his gorde | 16:41 |
| jaypipes | mriedem: yeah, doesn't take much for them to be out cold. | 16:41 |
| *** iamjarvo has joined #openstack-nova | 16:41 | |
| dims_ | mriedem: ouch! :( | 16:41 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Adds VolumeOps unit tests https://review.openstack.org/139797 | 16:42 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Adds MigrationOps unit tests https://review.openstack.org/139796 | 16:42 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: removes test_hypervapi tests https://review.openstack.org/139798 | 16:42 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Adds HyperVDriver unit tests https://review.openstack.org/148980 | 16:42 |
| mriedem | we didn't notice the puncture wound until last night, but he was limping all day, i thought he just slept on his leg wrong - so i literally told him he was a pussy and needed to walk it off, repeatedly | 16:42 |
| *** unicell has joined #openstack-nova | 16:43 | |
| ndipanov | mriedem, gah :( how did it happen ? | 16:44 |
| mriedem | he walked over, the other dog bit him | 16:44 |
| mriedem | the end | 16:44 |
| ndipanov | oh so it was a dog that did it | 16:44 |
| mriedem | ha, yeah | 16:44 |
| ndipanov | :( | 16:44 |
| alaski | bauzas: is consume_from_instance used outside of the scheduler? | 16:45 |
| *** amotoki has quit IRC | 16:45 | |
| mriedem | i've been saying those neighbor dogs were bad news since we moved in, but no one would believe me! well who's laughing now?!?!?! | 16:45 |
| mriedem | hahahaha | 16:45 |
| ndipanov | what kind of dog was it? | 16:45 |
| mriedem | one is a husky and one is a great dane | 16:45 |
| mriedem | i believe the husky is the biter | 16:45 |
| mriedem | she has that 'i want to rip your heart out' kind of look | 16:46 |
| *** nihilifer has joined #openstack-nova | 16:46 | |
| alaski | bauzas: commented on the review | 16:47 |
| *** sajeesh has quit IRC | 16:48 | |
| *** sdake_ has joined #openstack-nova | 16:48 | |
| *** iamjarvo has quit IRC | 16:49 | |
| *** prashant-upadhya has quit IRC | 16:49 | |
| *** nelsnelson has quit IRC | 16:50 | |
| *** eglynn has quit IRC | 16:51 | |
| *** sdake has quit IRC | 16:52 | |
| *** nic has joined #openstack-nova | 16:52 | |
| *** tonytan4ever has quit IRC | 16:52 | |
| *** moshele has quit IRC | 16:53 | |
| openstackgerrit | Baodong (Robert) Li proposed openstack/nova: Handle port delete initiated by neutron https://review.openstack.org/179390 | 16:54 |
| *** prashant-upadhya has joined #openstack-nova | 16:55 | |
| bauzas | alaski: sorry, family time :) | 16:58 |
| bauzas | alaski: to answer your question, there are 2 consume_from_instance() methods, one for the vanilla HostManager, and one for the Ironic one | 16:59 |
| *** alanf-mc has joined #openstack-nova | 16:59 | |
| bauzas | alaski: since IronicHostManager is inherited from the vanilla one, there is only one caller, from FilterScheduler | 17:00 |
| *** VW has quit IRC | 17:00 | |
| *** Nisha has joined #openstack-nova | 17:00 | |
| bauzas | alaski: back in the batcave now | 17:00 |
| *** marzif_ has quit IRC | 17:00 | |
| *** nikhil_k is now known as nikhil-afk | 17:01 | |
| *** VW has joined #openstack-nova | 17:03 | |
| *** vilobhmm has joined #openstack-nova | 17:03 | |
| *** lucasagomes has quit IRC | 17:10 | |
| *** alanf-mc has quit IRC | 17:10 | |
| *** claudiub has quit IRC | 17:11 | |
| *** marzif_ has joined #openstack-nova | 17:11 | |
| *** alanf-mc has joined #openstack-nova | 17:12 | |
| *** marun has quit IRC | 17:12 | |
| *** salv-orlando has quit IRC | 17:12 | |
| *** marun has joined #openstack-nova | 17:13 | |
| johnthetubaguy | I have mostly got the actions from the summit, I guess I probably missed some though https://etherpad.openstack.org/p/YVR-nova-liberty-summit-action-items | 17:16 |
| johnthetubaguy | what a bit list | 17:17 |
| * johnthetubaguy runs away for dinner | 17:17 | |
| *** nikhil-afk is now known as nikhil_k | 17:17 | |
| *** pradipta has joined #openstack-nova | 17:17 | |
| *** Kennan2 has joined #openstack-nova | 17:18 | |
| *** bradjones is now known as bradjones_away | 17:19 | |
| *** haomaiwang has joined #openstack-nova | 17:19 | |
| *** Kennan has quit IRC | 17:20 | |
| *** haomaiw__ has quit IRC | 17:20 | |
| *** Nisha has quit IRC | 17:20 | |
| *** oro_ has quit IRC | 17:21 | |
| *** adrian_otto has quit IRC | 17:22 | |
| openstackgerrit | Chris Friesen proposed openstack/nova-specs: Allow specifying cpu model/features in flavor extra-specs or image metadata https://review.openstack.org/168982 | 17:23 |
| *** dan has quit IRC | 17:23 | |
| *** dan| is now known as dan | 17:23 | |
| *** Nisha has joined #openstack-nova | 17:23 | |
| *** sudipto has joined #openstack-nova | 17:23 | |
| cfriesen | ^ this is a rework of the previous spec....Alex Xu and I have joined forces to try and come up with something simple but useful. Will write up a blurb on the mailing list. | 17:24 |
| *** mwagner_afk has joined #openstack-nova | 17:24 | |
| *** Viswanath has joined #openstack-nova | 17:25 | |
| *** josecastroleon has quit IRC | 17:28 | |
| *** tonytan4ever has joined #openstack-nova | 17:29 | |
| *** Viswanath has quit IRC | 17:30 | |
| *** ndipanov has quit IRC | 17:30 | |
| *** ildikov has joined #openstack-nova | 17:35 | |
| *** baoli has quit IRC | 17:38 | |
| *** haomaiwa_ has joined #openstack-nova | 17:39 | |
| *** marun has quit IRC | 17:39 | |
| *** claudiub has joined #openstack-nova | 17:40 | |
| *** salv-orlando has joined #openstack-nova | 17:40 | |
| *** ildikov_ has joined #openstack-nova | 17:40 | |
| *** dims_ has quit IRC | 17:40 | |
| *** haomaiwang has quit IRC | 17:41 | |
| *** VW_ has joined #openstack-nova | 17:46 | |
| *** VW has quit IRC | 17:47 | |
| *** dontalton has joined #openstack-nova | 17:47 | |
| *** Nisha has quit IRC | 17:53 | |
| *** patrickeast has joined #openstack-nova | 17:53 | |
| *** Nisha has joined #openstack-nova | 17:54 | |
| *** thangp has quit IRC | 17:55 | |
| *** yamahata has quit IRC | 17:55 | |
| *** yamahata has joined #openstack-nova | 17:55 | |
| *** artom has quit IRC | 17:55 | |
| *** igordcard_ has joined #openstack-nova | 17:55 | |
| *** thangp has joined #openstack-nova | 17:55 | |
| *** marzif_ has quit IRC | 17:55 | |
| *** igordcard_ has quit IRC | 17:56 | |
| *** marzif_ has joined #openstack-nova | 17:56 | |
| *** igordcard_ has joined #openstack-nova | 17:56 | |
| *** Nisha has quit IRC | 17:57 | |
| *** adrian_otto has joined #openstack-nova | 17:57 | |
| *** agireud has quit IRC | 17:57 | |
| *** rushiagr is now known as rushiagr_away | 17:57 | |
| *** Nisha has joined #openstack-nova | 17:58 | |
| ildikov_ | jaypipes: hi | 17:58 |
| jaypipes | ildikov: on phone... | 17:59 |
| ildikov_ | jaypipes: k, ping me when you're available, tnx | 18:00 |
| *** ijw has joined #openstack-nova | 18:00 | |
| *** annegentle has joined #openstack-nova | 18:01 | |
| *** Viswanath has joined #openstack-nova | 18:01 | |
| *** barra204 has joined #openstack-nova | 18:01 | |
| *** mmedvede has quit IRC | 18:03 | |
| *** Viswanath has quit IRC | 18:04 | |
| *** adrian_otto has quit IRC | 18:05 | |
| *** gabor-t has joined #openstack-nova | 18:06 | |
| *** tellesnobrega_ has joined #openstack-nova | 18:06 | |
| *** barra204 has quit IRC | 18:07 | |
| *** artom has joined #openstack-nova | 18:08 | |
| *** adrian_otto has joined #openstack-nova | 18:08 | |
| *** dims_ has joined #openstack-nova | 18:08 | |
| *** mmedvede has joined #openstack-nova | 18:09 | |
| *** haomai___ has joined #openstack-nova | 18:09 | |
| *** haomaiwa_ has quit IRC | 18:10 | |
| *** adrian_otto has quit IRC | 18:11 | |
| *** burt has joined #openstack-nova | 18:11 | |
| *** otter768 has joined #openstack-nova | 18:12 | |
| *** dprince has quit IRC | 18:13 | |
| *** adrian_otto has joined #openstack-nova | 18:14 | |
| *** packet has quit IRC | 18:15 | |
| *** iamjarvo has joined #openstack-nova | 18:15 | |
| *** pradipta has quit IRC | 18:17 | |
| jaypipes | cfriesen: hey, I'm grabbing a bite to eat, but perhaps we could chat about the ChunkedFileIterator thing in about 40 minutes? | 18:18 |
| *** otter768 has quit IRC | 18:18 | |
| jaypipes | cfriesen: can I ping you later? | 18:18 |
| cfriesen | sure | 18:18 |
| jaypipes | cfriesen: cheers. | 18:18 |
| cfriesen | later | 18:18 |
| *** adrian_otto has quit IRC | 18:21 | |
| *** iamjarvo has quit IRC | 18:21 | |
| *** annegentle has quit IRC | 18:21 | |
| *** claudiub has quit IRC | 18:22 | |
| *** annegentle has joined #openstack-nova | 18:22 | |
| *** Sukhdev has joined #openstack-nova | 18:22 | |
| *** MVenesio has joined #openstack-nova | 18:23 | |
| *** iamjarvo has joined #openstack-nova | 18:25 | |
| *** prashant-upadhya has quit IRC | 18:25 | |
| *** nickchase has quit IRC | 18:27 | |
| *** nickchase has joined #openstack-nova | 18:27 | |
| *** amotoki has joined #openstack-nova | 18:28 | |
| *** bradjones_away is now known as bradjones | 18:29 | |
| *** tellesnobrega_ has quit IRC | 18:29 | |
| *** nickchase has quit IRC | 18:29 | |
| *** tellesnobrega_ has joined #openstack-nova | 18:29 | |
| *** tellesnobrega_ has quit IRC | 18:29 | |
| *** nickchase has joined #openstack-nova | 18:29 | |
| *** jcru has quit IRC | 18:30 | |
| *** tellesnobrega_ has joined #openstack-nova | 18:32 | |
| *** tellesnobrega_ has quit IRC | 18:32 | |
| *** iamjarvo has quit IRC | 18:32 | |
| *** amotoki has quit IRC | 18:33 | |
| *** openstackstatus has joined #openstack-nova | 18:40 | |
| *** ChanServ sets mode: +v openstackstatus | 18:40 | |
| *** bradjones is now known as bradjones_away | 18:40 | |
| *** tellesnobrega_ has joined #openstack-nova | 18:41 | |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: rename parallels driver to virtuozzo https://review.openstack.org/184311 | 18:41 |
| *** agireud has joined #openstack-nova | 18:41 | |
| -openstackstatus- NOTICE: Gerrit has been restarted to clear an issue with its event stream. Any change events between 17:25 and 18:38 UTC should be rechecked or have their approvals reapplied to initiate testing. | 18:42 | |
| *** tellesnobrega_ has quit IRC | 18:43 | |
| *** VW_ has quit IRC | 18:44 | |
| *** sudipto has quit IRC | 18:44 | |
| *** ajayaa has quit IRC | 18:45 | |
| *** tellesnobrega_ has joined #openstack-nova | 18:45 | |
| *** marzif_ has quit IRC | 18:47 | |
| *** ajo has quit IRC | 18:47 | |
| *** VW has joined #openstack-nova | 18:49 | |
| *** dprince has joined #openstack-nova | 18:49 | |
| *** packet has joined #openstack-nova | 18:50 | |
| *** packet has quit IRC | 18:51 | |
| *** tsbot_ has quit IRC | 18:51 | |
| *** flwang1 has joined #openstack-nova | 18:53 | |
| mriedem | flwang: hey, so https://review.openstack.org/#/c/164494/ - i dropped the -2 hammer on that | 18:58 |
| mriedem | flwang: with the backup rotation stuff creeping in there, i think this is ballooning in complexity to the point that we need a blueprint and a spec to think it over | 18:58 |
| mriedem | flwang: i think you're on the right track, but it's definitely a feature add, not a bug fix, | 18:59 |
| mriedem | and as part of that i'd like to see a tempest change being worked on concurrently that depends on the nova change and tests it / shows that it works - which would show it works in the cells case too | 18:59 |
| *** krtaylor has quit IRC | 18:59 | |
| *** vishwana_ has joined #openstack-nova | 19:04 | |
| *** patrickeast_ has joined #openstack-nova | 19:04 | |
| *** marzif_ has joined #openstack-nova | 19:04 | |
| *** vishwanathj has quit IRC | 19:06 | |
| *** patrickeast has quit IRC | 19:06 | |
| *** patrickeast_ is now known as patrickeast | 19:06 | |
| mriedem | sdague: question about ^ as well wrt microversions | 19:07 |
| *** annegentle has quit IRC | 19:07 | |
| mriedem | sdague: as in, once we have backup support for volume-backed instances, is that something that needs a microversion b/c if cloud A has the change and cloud B doesn't, do i as sophie cloud pants user need to tell which i can talk to for that capability? | 19:07 |
| mnestratov | Guys could please take a look at these bluprints that are all about Virtuozzo (former Parallels Cloud Server) enhancements: | 19:09 |
| mnestratov | https://blueprints.launchpad.net/nova/+spec/rename-pcs-to-virtuozzo | 19:09 |
| mnestratov | https://blueprints.launchpad.net/nova/+spec/virtuozzo-container-boot-from-volume | 19:09 |
| mnestratov | https://blueprints.launchpad.net/nova/+spec/virtuozzo-instance-resize-support | 19:09 |
| mnestratov | https://blueprints.launchpad.net/nova/+spec/ploop-snapshot-support | 19:09 |
| *** Longgeek has quit IRC | 19:12 | |
| *** Nisha has quit IRC | 19:12 | |
| *** exploreshaifali has joined #openstack-nova | 19:14 | |
| *** salv-orlando has quit IRC | 19:15 | |
| *** Viswanath has joined #openstack-nova | 19:18 | |
| sdague | mriedem: so I guess it all comes down to is it just a bug where it was supposed to work and doesn't? I'm going to be inclined to say it's a bug because it doesn't change the API. Which also would make it candidate for backport. | 19:19 |
| mriedem | sdague: we said we weren't going to backport it in kilo b/c the rpc api version changes | 19:19 |
| sdague | ok | 19:19 |
| mriedem | so in kilo we merged a fail fast change and backported that | 19:19 |
| dansmith | sdague: it's really not a bug that was supposed to work | 19:20 |
| mriedem | yeah this is definitely a new feature | 19:20 |
| dansmith | and it includes some different behavior for the volume instances | 19:20 |
| mriedem | this is like saying, nova doesn't support containers, it's a bug :) | 19:20 |
| dansmith | right | 19:20 |
| sdague | ok, then I guess yes | 19:20 |
| *** Viswanath has quit IRC | 19:21 | |
| mriedem | yes to microversion? | 19:21 |
| flwang1 | mriedem: ping | 19:22 |
| mriedem | i'm not sure a microversion makes sense anyway since this isn't in the extension layer, it's in the compute api, so v2 and v2.1 would both get it | 19:22 |
| mriedem | flwang1: we're talking about your change ^ | 19:22 |
| dansmith | mriedem: not if you just exclude the feature when v2.x isn't requested | 19:22 |
| bauzas | does anyone could help me triaging https://bugs.launchpad.net/nova/+bug/1439870 ? | 19:22 |
| openstack | Launchpad bug 1439870 in OpenStack Compute (nova) "Fixed IPs not being recorded in database" [Undecided,New] - Assigned to venkata anil (anil-venkata) | 19:22 |
| dansmith | mriedem: being in the servers api doesn't mean anything different | 19:22 |
| flwang1 | cool, reading | 19:22 |
| *** krtaylor has joined #openstack-nova | 19:22 | |
| bauzas | I'm missing some network context here | 19:22 |
| mriedem | dansmith: does anything else do that today? | 19:23 |
| dansmith | mriedem: all the code that checks the version, yeah | 19:23 |
| dansmith | mriedem: where it is in the code doesn't matter | 19:23 |
| *** salv-orlando has joined #openstack-nova | 19:23 | |
| mriedem | ah ok | 19:23 |
| dansmith | mriedem: https://review.openstack.org/#/c/70485/9/nova/api/openstack/compute/plugins/v3/keypairs.py,cm | 19:23 |
| dansmith | actually, the better example is my evacuate one: | 19:24 |
| *** alanf-mc has quit IRC | 19:24 | |
| mriedem | actually the better example is my fixed_ips reserved one https://review.openstack.org/#/c/168966/8/nova/api/openstack/compute/plugins/v3/fixed_ips.py | 19:24 |
| mriedem | :) | 19:24 |
| dansmith | https://review.openstack.org/#/c/183200/5/nova/api/openstack/compute/plugins/v3/migrations.py,cm | 19:24 |
| dansmith | line 59 | 19:24 |
| dansmith | no | 19:24 |
| dansmith | my example is more useful for this case I think, | 19:24 |
| dansmith | where I just check the version in the middle of the code, no decorator/alternate call | 19:24 |
| mriedem | oh i thought we were just showing off | 19:24 |
| dansmith | heh | 19:25 |
| mriedem | do we have the request down in the compute api module? | 19:25 |
| *** gabor-t has quit IRC | 19:26 | |
| dansmith | sure hope so :) | 19:26 |
| dansmith | oh, you mean nova/compute/api ? | 19:27 |
| mriedem | yes | 19:27 |
| flwang1 | the conclusion is the patch is really adding a new feature instead of fixing bug? | 19:27 |
| dansmith | no, we don't | 19:27 |
| dansmith | you'd need to catch this earlier | 19:27 |
| mriedem | yeah | 19:27 |
| *** cknight has quit IRC | 19:27 | |
| mriedem | which means checking it in the server extension | 19:27 |
| dansmith | flwang1: it's always been a feature add, hence the no-backport decision :) | 19:27 |
| dansmith | mriedem: sorry, I assumed we were clear that I was talking about where in nova/api/openstack/* we were.. my bad | 19:28 |
| mriedem | what's not clear about nova/api/openstack/compute and nova/compute/api? | 19:28 |
| *** cknight has joined #openstack-nova | 19:28 | |
| mriedem | o-) | 19:28 |
| dansmith | yeah | 19:28 |
| flwang1 | i just want to figure out why it's requested to provide a spec now :) | 19:29 |
| mriedem | flwang1: the rotation stuff makes this more complicated | 19:29 |
| dansmith | when you said "compute api" above I read it as "servers api" | 19:29 |
| flwang1 | mriedem: you mean, delete the snapshot volumes? | 19:29 |
| mriedem | flwang1: also i think this probably means a microversion so people can discover that this works in v2.1 but not v2 | 19:29 |
| mriedem | flwang1: yeah - i also added a bunch of comments before -2 today | 19:29 |
| flwang1 | mriedem: i saw your comments, but most of them won't impact the code :) | 19:30 |
| mriedem | flwang1: well, i'm pretty sure the rotation logic is busted since you're not snapshotting an image so the query to the image_api will return 0 and it won't rotate/delete old volume snapshots, so that's kind of busted | 19:30 |
| mriedem | flwang1: and i think we have to figure out if a microversion is needed for this so users can 'discover' if the capability is there or not | 19:31 |
| flwang1 | no, actually, we're stilling snapshoting image | 19:31 |
| mriedem | i guess if it's just fixed in nova/compute/api.py then it's in v2 and v2.1 and if you know you're talking to a liberty cloud you assume it's there, | 19:31 |
| mriedem | but i don't know if users can tell which version of nova-api they are talking to | 19:32 |
| dansmith | they cannot | 19:32 |
| flwang1 | so we should still use image_api to get the image list | 19:32 |
| *** ildikov_ has quit IRC | 19:32 | |
| flwang1 | because backup is actually a snapshot, the difference is backup do it many times | 19:32 |
| mriedem | oh i see that nova.compute.api.snapshot_volume_backed calls self.image_api.create(context, image_meta) | 19:33 |
| flwang1 | yes | 19:33 |
| flwang1 | i'm ok for creating a new bp/spec to track this, but could I assume it can be approved it quickly ? :) | 19:35 |
| mriedem | well | 19:35 |
| mriedem | i don't think it's all that controversial | 19:35 |
| mriedem | i think we need to document the decisions made though, like if a microversion is required or not (i'm thinking it's not required, but not sure) | 19:36 |
| mriedem | and i'd really want to see a tempest change testing this at the same time it's being developed so we see it works in cells and non-cells jobs | 19:36 |
| mriedem | the spec would call that out in the testing section | 19:36 |
| *** iamjarvo has joined #openstack-nova | 19:37 | |
| flwang1 | mriedem: good point | 19:37 |
| flwang1 | but should the tempest patch block this one? | 19:37 |
| mriedem | flwang1: i think so | 19:37 |
| mriedem | with the depends-on tag it makes it easy | 19:37 |
| dansmith | mriedem: so, tempest will try to check the behavior, which will fail on, say, kilo | 19:38 |
| flwang1 | mriedem: ok | 19:38 |
| mriedem | dansmith: tempest can be configured | 19:38 |
| mriedem | we do that quite a bit actually | 19:38 |
| mriedem | dansmith: so default it's not supported, but devstack on master will enable it for testing | 19:38 |
| dansmith | sure, but doesn't that indicate to you that it needs to be user-detectable? | 19:38 |
| dansmith | i.e. a microversion | 19:38 |
| mriedem | good point | 19:39 |
| dansmith | it fails now with some 40x error, not 50x, right? | 19:39 |
| mriedem | just saying, things have been in before | 19:39 |
| flwang1 | would you guys mind explaining why the microversion matters here? | 19:39 |
| dansmith | flwang1: see the last 30 seconds | 19:39 |
| flwang1 | dansmith: yep | 19:39 |
| mriedem | like live migration of a paused instance, and deleting an instance doesn't delete neutron ports if they originated from neutron and not nova boot | 19:39 |
| dansmith | the former is very user-visible, the latter is rather obscure | 19:40 |
| mriedem | yeah, and live migration a paused instance that doesn't support it fails fast with a 400 | 19:40 |
| mriedem | the latter is a bit in between, anyway, off topic a bit | 19:40 |
| dansmith | backup fails fast with 40x too, yes? | 19:41 |
| mriedem | it does now | 19:41 |
| dansmith | I'm not completely convinced we need one for this either, | 19:41 |
| dansmith | but we're going to have more and more cases like this, | 19:41 |
| dansmith | so we kinda need to decide what we do here, IMHO | 19:41 |
| mriedem | i agree with agreeing on things being agreeable | 19:42 |
| dansmith | agreed. | 19:42 |
| *** matrohon has joined #openstack-nova | 19:42 | |
| mriedem | so...we agree a spec for sure | 19:42 |
| dansmith | yeah | 19:42 |
| dansmith | flwang1: if you're going to not do a microversion, | 19:42 |
| mriedem | flwang1: if you want to write the spec as not needing a microversion and explaining the case why not, then it can be debated in gerrit for others | 19:42 |
| dansmith | please document all the above in the REST impact section | 19:42 |
| *** beagles is now known as beagles_brb | 19:43 | |
| dansmith | so we can argue about it there, or at least so we're clearly voting for not doing one for $explanation | 19:43 |
| mriedem | exactly | 19:43 |
| mriedem | and htf tempest should be dealing with this stuff | 19:43 |
| mriedem | b/c up until now it's just be tempest.conf toggles | 19:43 |
| mriedem | *been | 19:43 |
| *** ildikov_ has joined #openstack-nova | 19:46 | |
| *** VW has quit IRC | 19:49 | |
| *** VW has joined #openstack-nova | 19:49 | |
| *** ijw_ has joined #openstack-nova | 19:50 | |
| flwang1 | ok, thank you guys | 19:50 |
| *** exploreshaifali has quit IRC | 19:50 | |
| flwang1 | though i may need some times to summarize why we don't go for the microversion | 19:51 |
| *** ijw has quit IRC | 19:52 | |
| jaypipes | cfriesen: OK, I'm pushing our chat to tomorrow, sorry :( I need to complete about 5 specs today and need to go into seclusion :) | 19:53 |
| *** artom has quit IRC | 19:53 | |
| mriedem | flwang1: the main point for not having a microversion i think is b/c it's not a change to the API extensions, it's a change in the nova.compute.api module which would make it work for v2 and v2.1 | 19:53 |
| mriedem | flwang1: the question is about discoverability | 19:54 |
| *** samueldmq has quit IRC | 19:54 | |
| *** VW has quit IRC | 19:54 | |
| *** Viswanath has joined #openstack-nova | 19:55 | |
| *** dprince has quit IRC | 19:56 | |
| *** baoli has joined #openstack-nova | 19:57 | |
| *** iamjarvo has quit IRC | 19:59 | |
| *** nelsnelson has joined #openstack-nova | 19:59 | |
| *** VW has joined #openstack-nova | 20:00 | |
| *** marun has joined #openstack-nova | 20:00 | |
| *** Viswanath has quit IRC | 20:00 | |
| *** marzif_ has quit IRC | 20:00 | |
| *** iamjarvo has joined #openstack-nova | 20:04 | |
| *** iamjarvo has quit IRC | 20:04 | |
| *** iamjarvo has joined #openstack-nova | 20:05 | |
| *** iamjarvo has quit IRC | 20:05 | |
| *** iamjarvo has joined #openstack-nova | 20:05 | |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: Add config drive support for PCS containers https://review.openstack.org/149253 | 20:06 |
| *** Viswanath has joined #openstack-nova | 20:07 | |
| *** nelsnelson has quit IRC | 20:07 | |
| openstackgerrit | Auktavian Garrett proposed openstack/nova: Use unique values for hypervisor_hostnames https://review.openstack.org/187844 | 20:08 |
| *** ijw_ has quit IRC | 20:08 | |
| *** baoli has quit IRC | 20:08 | |
| bauzas | melwitt: if you don't disagree, putting your change to https://etherpad.openstack.org/p/liberty-nova-priorities-tracking | 20:08 |
| bauzas | melwitt: L32 | 20:09 |
| *** ijw has joined #openstack-nova | 20:09 | |
| bauzas | melwitt: as I said, a little hairy and a question, but I think it's mergeable | 20:09 |
| *** Viswanath has quit IRC | 20:10 | |
| *** flwang1 has quit IRC | 20:12 | |
| *** otter768 has joined #openstack-nova | 20:13 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements https://review.openstack.org/188500 | 20:14 |
| melwitt | bauzas: cool, thanks. what do you mean by hairy? hard to follow? | 20:16 |
| *** Viswanath has joined #openstack-nova | 20:16 | |
| *** baoli has joined #openstack-nova | 20:16 | |
| dansmith | melwitt: big and complicated, I imagine | 20:17 |
| *** alanf-mc has joined #openstack-nova | 20:17 | |
| *** amotoki has joined #openstack-nova | 20:17 | |
| dansmith | but it's probably as small as it can be | 20:17 |
| bauzas | melwitt: the main change was about objectifying the instance, but you also added the RPC API bump, the context manager for race conditions and you made a few modifications (like the one I commented) | 20:17 |
| bauzas | melwitt: dansmith: yeah that's my point, I made a +1 because it went into the right direction and perhaps it was atomic | 20:18 |
| *** baoli has quit IRC | 20:18 | |
| bauzas | melwitt: dansmith: but it took me a while to make sure it was okay :) | 20:18 |
| *** baoli has joined #openstack-nova | 20:18 | |
| *** otter768 has quit IRC | 20:19 | |
| dansmith | I'll take a look once I get some food in me | 20:19 |
| *** sinval_ has quit IRC | 20:19 | |
| *** Viswanath has quit IRC | 20:20 | |
| *** VW has quit IRC | 20:20 | |
| *** artom has joined #openstack-nova | 20:21 | |
| melwitt | bauzas: ah, yeah. I understand. the version bump is required because the heart of the change is sending an object to cells rpcapi, and only a newer version could handle that. the "skip sync" thing is also needed because changing to objects introduces an instance.save() forever loop otherwise. it is complicated, unfortunately | 20:21 |
| *** amotoki has quit IRC | 20:22 | |
| bauzas | melwitt: yup, I understood it :) | 20:22 |
| *** Sukhdev has quit IRC | 20:22 | |
| bauzas | melwitt: was just saying it was hairy :) | 20:22 |
| melwitt | haha okay | 20:22 |
| bauzas | melwitt: but most of that was atomic | 20:22 |
| bauzas | melwitt: I still have an open question on https://review.openstack.org/#/c/176518/13/nova/cells/messaging.py,cm | 20:23 |
| bauzas | melwitt: so, you're now calling instance.save() which itself manages the instance actions ? | 20:23 |
| melwitt | bauzas: yes, didn't reply yet. instance.save() will info_cache.save() the info_cache object if it's part of the update. what do you mean by instance actions? is that something separate? | 20:24 |
| bauzas | info_cache indeed, my bad | 20:24 |
| bauzas | okay, that's what I thought, hence my +1 | 20:25 |
| openstackgerrit | Chris Friesen proposed openstack/nova-specs: Drop support for booting multiple instances https://review.openstack.org/188575 | 20:25 |
| cfriesen | johnthetubaguy: stole some time...there's your spec for opening discussion. ^ | 20:26 |
| *** bradjones_away has quit IRC | 20:26 | |
| melwitt | bauzas: okay, cool | 20:26 |
| cfriesen | bauzas: ^ | 20:26 |
| bauzas | cfriesen: CCing it | 20:26 |
| *** ijw_ has joined #openstack-nova | 20:29 | |
| *** ivasev has quit IRC | 20:32 | |
| *** bradjones has joined #openstack-nova | 20:32 | |
| *** ijw has quit IRC | 20:32 | |
| *** kylek3h has joined #openstack-nova | 20:32 | |
| *** ociuhandu has quit IRC | 20:32 | |
| *** busterswt has joined #openstack-nova | 20:35 | |
| *** vladikr has quit IRC | 20:36 | |
| *** busterswt has quit IRC | 20:36 | |
| *** artom has quit IRC | 20:37 | |
| *** adrian_otto has joined #openstack-nova | 20:38 | |
| *** tonytan4ever has quit IRC | 20:38 | |
| *** tonytan4ever has joined #openstack-nova | 20:39 | |
| *** lpetrut has quit IRC | 20:41 | |
| *** markvoelker has quit IRC | 20:41 | |
| *** tsekiyama has joined #openstack-nova | 20:43 | |
| *** tonytan4ever has quit IRC | 20:44 | |
| *** tsbot_ has joined #openstack-nova | 20:44 | |
| *** Viswanath has joined #openstack-nova | 20:44 | |
| *** igordcard_ has quit IRC | 20:45 | |
| *** Viswanath has quit IRC | 20:47 | |
| dansmith | melwitt: why do you think you need to catch infocachenotfound? | 20:54 |
| melwitt | dansmith: besides the fact it was doing it before, during instance.save() info_cache.save() will go before the instance_update, so it's possible that can be raised and if we don't catch it separately, it'll fall under the existing NotFound and do an instance.create() | 20:56 |
| dansmith | melwitt: yeah, but if it's on the instance, then it should have already been created anyway, right? | 20:56 |
| dansmith | you're not getting it from somewhere else | 20:56 |
| melwitt | dansmith: one would think, but remember there's that cells rpcapi update_instance_info_cache_at_top which receives only an info cache and an instance object is created just to hold it, when it's passed to instance_update_at_top | 20:57 |
| dansmith | I guess if it's deleted underneath us | 20:57 |
| *** cknight has quit IRC | 20:57 | |
| dansmith | hmm, not sure I understand that, but... I probably don't want to know :) | 20:58 |
| bauzas | nova meeting in 2' ? | 20:58 |
| mikal | Nova meeting in a couple of minutes! | 20:58 |
| *** ijw_ has quit IRC | 21:00 | |
| melwitt | dansmith: in here L262 https://review.openstack.org/#/c/176518/13/nova/cells/rpcapi.py,cm | 21:00 |
| *** ijw has joined #openstack-nova | 21:00 | |
| *** thangp has quit IRC | 21:00 | |
| *** beagles_brb is now known as beagles | 21:00 | |
| *** electrocucaracha has quit IRC | 21:00 | |
| *** iamjarvo has quit IRC | 21:01 | |
| *** raildo has quit IRC | 21:01 | |
| *** operator99 is now known as gyee | 21:02 | |
| *** electrocucaracha has joined #openstack-nova | 21:02 | |
| *** tonytan4ever has joined #openstack-nova | 21:05 | |
| alaski | I was looking for good places to split that patch, but outside of one small cleanup spot it really is about as minimal as it can get | 21:06 |
| openstackgerrit | Chris Friesen proposed openstack/nova-specs: Drop support for booting multiple instances https://review.openstack.org/188575 | 21:06 |
| *** ijw has quit IRC | 21:10 | |
| *** ijw has joined #openstack-nova | 21:10 | |
| cfriesen | bauzas: as long as you're interested in looking at specs...could you take a look at https://review.openstack.org/#/c/168982/ too? | 21:11 |
| bauzas | cfriesen: nova meeting but sure :) | 21:11 |
| cfriesen | bauzas: no rush | 21:11 |
| *** SourabhP has joined #openstack-nova | 21:17 | |
| openstackgerrit | Earle F. Philhower, III proposed openstack/nova: Add support for HGST Solutions volumes to libvirt https://review.openstack.org/186594 | 21:17 |
| *** Kennan has joined #openstack-nova | 21:18 | |
| *** Kennan2 has quit IRC | 21:19 | |
| *** eharney has quit IRC | 21:19 | |
| *** ijw_ has joined #openstack-nova | 21:20 | |
| SourabhP | jogo: Hello! | 21:21 |
| *** tellesnobrega_ has quit IRC | 21:21 | |
| SourabhP | jogo: Can you provide some guidance how to proceed for https://review.openstack.org/#/c/157616/ | 21:22 |
| *** ijw has quit IRC | 21:23 | |
| *** adrian_otto has quit IRC | 21:27 | |
| *** Raildo has joined #openstack-nova | 21:27 | |
| *** Raildo_ has joined #openstack-nova | 21:29 | |
| *** Raildo__ has joined #openstack-nova | 21:29 | |
| *** Raildo__ has quit IRC | 21:30 | |
| *** adrian_otto has joined #openstack-nova | 21:30 | |
| *** angdraug has quit IRC | 21:30 | |
| *** Raildo has quit IRC | 21:33 | |
| *** Raildo_ has quit IRC | 21:34 | |
| *** baoli has quit IRC | 21:34 | |
| *** adrian_otto has quit IRC | 21:34 | |
| *** baoli has joined #openstack-nova | 21:35 | |
| *** salv-orlando has quit IRC | 21:35 | |
| mriedem | SourabhP: restore the change | 21:35 |
| mriedem | SourabhP: you probably want to find danpb | 21:35 |
| mriedem | he's in the UK so not around right now | 21:35 |
| SourabhP | mriedem: yes, I restored the change. Will contact danpb, thanks for the info! | 21:37 |
| mriedem | mestery: i saw a server create failure with kilo code where the binding:vif_type on a port came back to nova as 'binding_failed' - seems like any requests to neutron should have failed in that case no? | 21:37 |
| mriedem | unless it's on the vif plug where nova is waiting for the event back from neutron | 21:38 |
| *** electrocucaracha has quit IRC | 21:38 | |
| mriedem | mestery: but looking at our neutron api it looks like we are putting this into our network_model in the db and could be detecting it and failing in much more obvious ways | 21:38 |
| mriedem | beagles: sc68cal: ^ if you have ever heard of this | 21:38 |
| *** electrocucaracha has joined #openstack-nova | 21:38 | |
| sc68cal | looking | 21:39 |
| *** MVenesio has quit IRC | 21:39 | |
| mestery | mriedem: Could be, I'm in the middle of something now, I'll defer to sc68cal at the moment :) | 21:40 |
| *** mwagner_afk has quit IRC | 21:40 | |
| mriedem | sc68cal: this is the paste http://paste.openstack.org/show/263976/ | 21:40 |
| sc68cal | I thought there was a spec somewhere to make the errors for binding_failed more clear, but it's been a while | 21:40 |
| mriedem | i've never seen this | 21:40 |
| mriedem | i see neutron has a constant that it checks for specifically 'binding_failed' | 21:40 |
| *** salv-orlando has joined #openstack-nova | 21:41 | |
| mriedem | i told the guy to check neutron logs but he hasn't come back with anything from that yet | 21:41 |
| mriedem | i'm guessing this is misconfiguration, but seems we have a better error response to give somewhere in nova | 21:41 |
| *** markvoelker has joined #openstack-nova | 21:41 | |
| sc68cal | mriedem: yeah he may need to go find the log for the specific host where the instance was scheduled and check the neutron agent log for the failure | 21:42 |
| mriedem | ok, will dig in later | 21:44 |
| *** agireud has quit IRC | 21:44 | |
| *** markvoelker has quit IRC | 21:46 | |
| *** claudiub has joined #openstack-nova | 21:47 | |
| *** matrohon has quit IRC | 21:49 | |
| *** mriedem has quit IRC | 21:51 | |
| *** angdraug has joined #openstack-nova | 21:53 | |
| *** krtaylor has quit IRC | 21:56 | |
| *** whenry has quit IRC | 21:57 | |
| flwang | jaypipes: ping | 21:57 |
| jaypipes | flwang: pong (though I'm really in seclusion working on specs right now...) | 21:57 |
| *** iamjarvo has joined #openstack-nova | 21:58 | |
| *** iamjarvo has quit IRC | 21:58 | |
| *** iamjarvo has joined #openstack-nova | 21:58 | |
| *** iamjarvo has quit IRC | 21:58 | |
| *** apuimedo has joined #openstack-nova | 21:58 | |
| flwang | jaypipes: ok, if so, i can ping you later when you're not in a shadow corner | 21:58 |
| jaypipes | flwang: :) how about tomorrow morning. either that, or shoot me an email? | 21:59 |
| *** iamjarvo has joined #openstack-nova | 21:59 | |
| flwang | ah, just wanna have a catch up about the glance v2 in nova | 21:59 |
| *** electrocucaracha has quit IRC | 21:59 | |
| flwang | ok, i can ping you on Monday, since tomorrow is my Sat, I'm based in NZ | 22:00 |
| jaypipes | flwang: sounds good. I'll try to review flaper87's glance-related Nova specs tomorrow. | 22:01 |
| *** adrian_otto has joined #openstack-nova | 22:02 | |
| flwang | jaypipes: cool, I'm trying to rebase his patch and working on something from the glance side | 22:02 |
| ildikov_ | jaypipes: could you please add one more spec to your review list: https://review.openstack.org/#/c/179104/ ? :) | 22:04 |
| jaypipes | ildikov: yes, I will. | 22:04 |
| jaypipes | thx for reminding me of that one. | 22:04 |
| *** electrocucaracha has joined #openstack-nova | 22:05 | |
| *** burt has quit IRC | 22:05 | |
| *** dboik has quit IRC | 22:05 | |
| ildikov_ | jaypipes: me thanks | 22:05 |
| *** amotoki has joined #openstack-nova | 22:06 | |
| *** electrocucaracha has quit IRC | 22:08 | |
| *** ijw_ has quit IRC | 22:09 | |
| *** jecarey has quit IRC | 22:09 | |
| openstackgerrit | melanie witt proposed openstack/nova: Raise InstanceNotFound when save FK constraint fails https://review.openstack.org/188261 | 22:09 |
| *** flip214 has quit IRC | 22:09 | |
| *** flip214 has joined #openstack-nova | 22:10 | |
| *** flip214 has quit IRC | 22:10 | |
| *** flip214 has joined #openstack-nova | 22:10 | |
| melwitt | dansmith: this is my attempt at the instance.save() FK constraint failure handling we talked about the other day. I broadened it to any ObjectField instead of only flavor https://review.openstack.org/#/c/188261/ | 22:10 |
| *** amotoki has quit IRC | 22:11 | |
| dansmith | melwitt: cool | 22:11 |
| *** electroc_ has joined #openstack-nova | 22:11 | |
| dansmith | melwitt: we should let some of the dbish people look at it first, but I think that's the right thing to do | 22:12 |
| *** ijw has joined #openstack-nova | 22:12 | |
| *** claudiub has quit IRC | 22:13 | |
| *** otter768 has joined #openstack-nova | 22:14 | |
| melwitt | dansmith: okay, makes sense | 22:15 |
| *** slong has joined #openstack-nova | 22:18 | |
| *** iamjarvo has quit IRC | 22:18 | |
| openstackgerrit | Josh Durgin proposed openstack/nova-specs: RBD instance snapshots https://review.openstack.org/188244 | 22:19 |
| *** otter768 has quit IRC | 22:19 | |
| dansmith | melwitt: made a teensy nit comment | 22:19 |
| melwitt | dansmith: I wasn't sure about that one because tags is an object field TagList but there isn't a _save_tags | 22:21 |
| *** mnestratov|3 has joined #openstack-nova | 22:21 | |
| dansmith | melwitt: my complaint is the naming of the variable | 22:21 |
| dansmith | it says "fields with save methods" but that's not right | 22:21 |
| dansmith | it's actually everything minus tags | 22:22 |
| melwitt | dansmith: I thought it was being that tags doesn't have a _save_* method | 22:22 |
| openstackgerrit | Josh Durgin proposed openstack/nova-specs: RBD instance snapshots https://review.openstack.org/188244 | 22:22 |
| *** tonytan4ever has quit IRC | 22:23 | |
| melwitt | that is, the isinstance call would still pick up "tags" | 22:23 |
| dansmith | melwitt: http://pastebin.com/NSDfG3xY | 22:23 |
| *** mnestratov has quit IRC | 22:23 | |
| *** tonytan4ever has joined #openstack-nova | 22:24 | |
| melwitt | dansmith: oh, sorry. :( | 22:24 |
| dansmith | heh | 22:25 |
| dansmith | figured it was easier to show it in code :) | 22:25 |
| melwitt | yeah, it's very clear now. haha | 22:25 |
| *** krtaylor has joined #openstack-nova | 22:26 | |
| *** sorrison_laptop has joined #openstack-nova | 22:28 | |
| *** kylek3h has quit IRC | 22:28 | |
| *** tonytan4ever has quit IRC | 22:28 | |
| *** Sukhdev has joined #openstack-nova | 22:29 | |
| *** adrian_otto has left #openstack-nova | 22:30 | |
| *** nic has quit IRC | 22:30 | |
| *** rwsu has quit IRC | 22:30 | |
| *** nic has joined #openstack-nova | 22:31 | |
| *** winston-d has quit IRC | 22:31 | |
| *** winston-d has joined #openstack-nova | 22:31 | |
| *** ijw has quit IRC | 22:32 | |
| *** ijw has joined #openstack-nova | 22:32 | |
| *** nic has quit IRC | 22:33 | |
| *** dontalton has quit IRC | 22:35 | |
| *** annegentle has joined #openstack-nova | 22:35 | |
| *** baoli has quit IRC | 22:36 | |
| *** iamjarvo has joined #openstack-nova | 22:37 | |
| *** iamjarvo has quit IRC | 22:38 | |
| *** iamjarvo has joined #openstack-nova | 22:38 | |
| *** mrodden has quit IRC | 22:39 | |
| *** mrodden_ has joined #openstack-nova | 22:39 | |
| *** mrodden_ is now known as mrodden | 22:40 | |
| *** annegentle has quit IRC | 22:40 | |
| *** sorrison_laptop has quit IRC | 22:41 | |
| *** annegentle has joined #openstack-nova | 22:42 | |
| openstackgerrit | melanie witt proposed openstack/nova: Raise InstanceNotFound when save FK constraint fails https://review.openstack.org/188261 | 22:42 |
| *** ijw has quit IRC | 22:42 | |
| *** markvoelker has joined #openstack-nova | 22:42 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/objects/test*.py https://review.openstack.org/187785 | 22:43 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Adding more py34 test https://review.openstack.org/188243 | 22:43 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Enable python34 tests for nova/tests/unit/scheduler/test*.py https://review.openstack.org/187968 | 22:43 |
| *** ijw has joined #openstack-nova | 22:43 | |
| *** electroc_ has quit IRC | 22:44 | |
| *** iamjarvo has quit IRC | 22:47 | |
| *** markvoelker has quit IRC | 22:47 | |
| *** iamjarvo has joined #openstack-nova | 22:47 | |
| *** zzzeek has quit IRC | 22:50 | |
| openstackgerrit | Jesse Keating proposed openstack/python-novaclient: Handle retry-after headers in 403 errors https://review.openstack.org/188620 | 22:53 |
| *** ijw_ has joined #openstack-nova | 22:53 | |
| *** ijw has quit IRC | 22:56 | |
| *** packet has joined #openstack-nova | 23:02 | |
| *** annegentle has quit IRC | 23:08 | |
| *** radek_ has quit IRC | 23:09 | |
| *** sorrison_laptop has joined #openstack-nova | 23:16 | |
| wwriverrat | I hereby respectfully request a core dev +2 review this mindlessly simple patch: https://review.openstack.org/#/c/179978/ | 23:17 |
| wwriverrat | Makes multiple runs of libvirt driver's cleanup method tolerant of serial ports missing (typically when it needs to be run a 2nd time). | 23:17 |
| *** ian_brown has quit IRC | 23:19 | |
| *** packet has quit IRC | 23:19 | |
| *** ian_brown has joined #openstack-nova | 23:19 | |
| *** tjones1 has quit IRC | 23:25 | |
| *** zhenguo has joined #openstack-nova | 23:30 | |
| *** bradjones has quit IRC | 23:36 | |
| *** vilobhmm has quit IRC | 23:38 | |
| *** bradjones has joined #openstack-nova | 23:39 | |
| *** hemna is now known as hemnafk | 23:43 | |
| *** sdake_ is now known as sdake | 23:44 | |
| openstackgerrit | Sam Morrison proposed openstack/nova: Cells: Support attach/detach interface and external events https://review.openstack.org/184155 | 23:50 |
| *** VW has joined #openstack-nova | 23:51 | |
| *** salv-orlando has quit IRC | 23:53 | |
| *** destiny has quit IRC | 23:54 | |
| *** sorrison has joined #openstack-nova | 23:57 | |
| *** sorrison_laptop has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!