Thursday, 2015-10-01

*** alex_klimov has quit IRC00:02
*** aloga has quit IRC00:02
*** bswartz has joined #openstack-nova00:02
*** aloga has joined #openstack-nova00:03
*** Daisy_ has quit IRC00:04
*** nic has quit IRC00:04
*** tjones1 has joined #openstack-nova00:05
*** signed8bit has quit IRC00:08
*** tjones1 has quit IRC00:09
*** aloga has quit IRC00:11
*** su_zhang_ has joined #openstack-nova00:12
*** su_zhan__ has joined #openstack-nova00:13
*** su_zhang_ has quit IRC00:13
*** jwcroppe has joined #openstack-nova00:14
*** su_zhang has quit IRC00:15
*** amotoki has joined #openstack-nova00:17
*** karimb has joined #openstack-nova00:17
*** asettle has joined #openstack-nova00:18
*** mriedem1 has joined #openstack-nova00:18
*** aloga has joined #openstack-nova00:19
openstackgerritMichael Still proposed openstack/nova-specs: Locking servers by service token  https://review.openstack.org/20388000:20
*** mriedem has quit IRC00:21
*** amotoki has quit IRC00:22
*** tsekiyama has quit IRC00:26
*** Daisy has joined #openstack-nova00:27
*** mtanino has quit IRC00:29
*** mudassirlatif has joined #openstack-nova00:29
*** Daisy has quit IRC00:32
*** su_zhan__ is now known as su_zhang00:39
*** garyk has quit IRC00:42
*** tjones has quit IRC00:44
*** mriedem has joined #openstack-nova00:51
*** mriedem1 has quit IRC00:53
*** ssurana has quit IRC01:06
mriedemwow, never seen this in a tempest failure before01:06
mriedemhttp://logs.openstack.org/39/229639/2/check/gate-tempest-dsvm-full-juno/a0bb0c0/console.html#_2015-09-30_22_41_17_90001:06
mriedemthat's a juno job at least01:06
mriedemdansmith: ^ more ebtables weirdness...maybe something did change in the ubuntu kernel recently01:07
*** pm90_ has quit IRC01:10
mriedemonly failing in juno jobs though, so shrug?01:10
*** amotoki has joined #openstack-nova01:11
*** jwcroppe has quit IRC01:11
*** pm90_ has joined #openstack-nova01:12
*** thumpba has joined #openstack-nova01:14
*** amotoki has quit IRC01:15
*** pixelbeat has quit IRC01:19
*** vilobhmm111 has quit IRC01:22
*** otter768 has joined #openstack-nova01:24
*** su_zhang has quit IRC01:25
*** mwagner_ has joined #openstack-nova01:25
*** emagana has quit IRC01:27
*** karimb has quit IRC01:28
*** armax has quit IRC01:29
*** dustins has joined #openstack-nova01:34
*** thumpba has quit IRC01:38
*** mudassirlatif has quit IRC01:40
*** mudassirlatif has joined #openstack-nova01:41
*** mudassirlatif has quit IRC01:41
mtreinishmriedem: I'm not sure this is related: http://logs.openstack.org/39/229639/2/check/gate-tempest-dsvm-full-juno/a0bb0c0/logs/kern_log.txt.gz#_Sep_30_22_21_27 but the kernel log is saying to report a bug about the ebtables usage01:42
openstackgerritMichael Still proposed openstack/nova-specs: Locking servers by service token  https://review.openstack.org/20388001:42
mtreinishwell I'm assuming by author there they're refering to the application author and not the ebtables author01:42
mtreinishalthough I guess I really don't know01:42
*** unicell has quit IRC01:43
*** ijw has quit IRC01:46
*** angdraug has quit IRC01:46
*** pm90_ has quit IRC01:49
*** thumpba has joined #openstack-nova01:51
*** zzzeek has quit IRC01:58
*** EinstCrazy has joined #openstack-nova01:59
*** EinstCrazy has quit IRC02:04
*** edmondsw has quit IRC02:05
*** amotoki has joined #openstack-nova02:05
*** tpeoples has quit IRC02:06
*** rfolco has quit IRC02:06
*** abhishekk_ has quit IRC02:08
*** abhishekk_ has joined #openstack-nova02:09
*** tjones has joined #openstack-nova02:09
*** jerrygb has joined #openstack-nova02:09
*** amotoki has quit IRC02:09
*** tjones has quit IRC02:10
*** sambetts has quit IRC02:11
*** thorst has joined #openstack-nova02:13
*** markvoelker has quit IRC02:15
*** yuntongjin has joined #openstack-nova02:15
*** thorst has quit IRC02:16
*** sambetts has joined #openstack-nova02:17
*** vladikr has quit IRC02:21
*** yamahata has quit IRC02:26
*** amotoki has joined #openstack-nova02:28
*** dims__ has quit IRC02:29
*** dims_ has joined #openstack-nova02:29
*** asettle has quit IRC02:35
*** yuntongjin has quit IRC02:35
*** apoorvad has joined #openstack-nova02:45
*** tjones has joined #openstack-nova02:46
*** apoorvad_ has quit IRC02:48
*** amotoki has quit IRC02:49
*** haomaiwang has joined #openstack-nova02:50
*** apoorvad has quit IRC02:50
*** haomaiwang has quit IRC02:50
*** haomaiwa_ has joined #openstack-nova02:51
*** mriedem has quit IRC02:51
*** haomaiwa_ has quit IRC03:01
*** haomaiwang has joined #openstack-nova03:01
*** amotoki has joined #openstack-nova03:02
*** amotoki_ has joined #openstack-nova03:04
*** diogogmt has quit IRC03:05
*** dustins has quit IRC03:07
*** willsama has joined #openstack-nova03:07
*** dsanders has joined #openstack-nova03:08
*** amotoki has quit IRC03:08
*** dsanders1 has quit IRC03:10
*** tpeoples has joined #openstack-nova03:10
*** amotoki_ has quit IRC03:10
*** dave-mccowan has quit IRC03:10
*** otter768 has quit IRC03:11
*** otter768 has joined #openstack-nova03:12
*** ociuhandu has joined #openstack-nova03:12
*** EinstCrazy has joined #openstack-nova03:14
*** dims__ has joined #openstack-nova03:15
*** amotoki has joined #openstack-nova03:15
*** markvoelker has joined #openstack-nova03:15
*** dims___ has joined #openstack-nova03:16
*** tjones has quit IRC03:18
*** dims____ has joined #openstack-nova03:18
*** dims_ has quit IRC03:18
*** sajeesh has quit IRC03:19
*** markvoelker has quit IRC03:20
*** dims__ has quit IRC03:20
*** dims___ has quit IRC03:21
*** otter768 has quit IRC03:21
*** deepthi has joined #openstack-nova03:23
*** r-daneel has quit IRC03:26
*** dsanders has quit IRC03:32
*** armax has joined #openstack-nova03:36
*** dims____ has quit IRC03:37
*** ijw has joined #openstack-nova03:39
*** amotoki has quit IRC03:40
*** salv-orl_ has joined #openstack-nova03:43
*** jwcroppe has joined #openstack-nova03:43
*** su_zhang has joined #openstack-nova03:44
*** salv-orlando has quit IRC03:46
*** tpeoples has quit IRC03:46
*** links has joined #openstack-nova03:55
*** jerrygb has quit IRC03:57
*** jerrygb has joined #openstack-nova03:58
*** boris-42 has quit IRC03:59
*** haomaiwang has quit IRC04:01
*** haomaiwang has joined #openstack-nova04:01
*** vilobhmm11 has joined #openstack-nova04:06
*** apoorvad has joined #openstack-nova04:12
*** markvoelker has joined #openstack-nova04:16
*** dsanders has joined #openstack-nova04:16
*** markvoelker has quit IRC04:20
openstackgerritijw-ubuntu proposed openstack/nova-specs: Add a binding type negotiation between Nova and Neutron  https://review.openstack.org/19091704:21
*** willsama has quit IRC04:23
*** VW has joined #openstack-nova04:26
*** stevemar has joined #openstack-nova04:30
*** StevenK has quit IRC04:35
*** StevenK has joined #openstack-nova04:36
*** irenab has joined #openstack-nova04:36
*** su_zhang has quit IRC04:42
*** su_zhang has joined #openstack-nova04:43
*** VW has quit IRC04:46
*** VW has joined #openstack-nova04:46
*** unicell has joined #openstack-nova04:46
*** VikasC has joined #openstack-nova04:46
*** Marga_ has quit IRC04:47
*** signed8bit has joined #openstack-nova04:48
*** asettle has joined #openstack-nova04:50
*** asettle has quit IRC04:50
*** Sree has joined #openstack-nova04:52
*** signed8b_ has joined #openstack-nova04:54
*** asettle has joined #openstack-nova04:55
*** signed8bit has quit IRC04:57
*** rotbeard has joined #openstack-nova04:58
openstackgerritWen Zhi Yu proposed openstack/nova-specs: Spec for nova API blueprint correct-volume-attachment-id-in-nova-api-volumes  https://review.openstack.org/21402105:00
*** haomaiwang has quit IRC05:01
*** haomaiwang has joined #openstack-nova05:01
*** Sree has quit IRC05:03
*** Sree has joined #openstack-nova05:03
*** Sree has quit IRC05:07
*** vilobhmm11 has quit IRC05:14
*** otter768 has joined #openstack-nova05:22
*** signed8b_ has quit IRC05:25
*** signed8bit has joined #openstack-nova05:25
*** otter768 has quit IRC05:28
*** oomichi has quit IRC05:35
*** signed8bit is now known as signed8bit_ZZZzz05:41
*** vilobhmm11 has joined #openstack-nova05:43
*** su_zhang has quit IRC05:45
*** yuntongjin has joined #openstack-nova05:51
*** haomaiwang has quit IRC05:54
*** haomaiwang has joined #openstack-nova05:54
*** kiran-r has joined #openstack-nova05:54
*** VW has quit IRC05:55
*** VW has joined #openstack-nova05:56
*** abhishekk has joined #openstack-nova05:58
*** bfic has quit IRC05:59
*** oomichi has joined #openstack-nova05:59
*** abhishekk_ has quit IRC06:00
*** ijw has quit IRC06:00
*** VW has quit IRC06:00
*** haomaiwang has quit IRC06:01
*** haomaiwang has joined #openstack-nova06:01
*** VW has joined #openstack-nova06:02
*** apoorvad has quit IRC06:02
*** oomichi has quit IRC06:03
*** yuntongjin1 has joined #openstack-nova06:05
*** EinstCrazy has quit IRC06:05
*** lsmola has joined #openstack-nova06:06
*** yuntongjin has quit IRC06:07
*** VW has quit IRC06:10
*** VW has joined #openstack-nova06:10
*** nkrinner has joined #openstack-nova06:13
*** irenab has quit IRC06:15
*** VW has quit IRC06:15
*** markvoelker has joined #openstack-nova06:17
*** sahid has joined #openstack-nova06:18
*** scheuran has joined #openstack-nova06:18
*** garyk has joined #openstack-nova06:19
*** markvoelker has quit IRC06:21
*** xgsa has joined #openstack-nova06:35
*** vilobhmm11 has quit IRC06:38
*** rajesht_ has quit IRC06:38
*** armax has quit IRC06:39
*** kiran-r has quit IRC06:39
*** Marga_ has joined #openstack-nova06:39
*** yuntongjin has joined #openstack-nova06:45
*** Marga_ has quit IRC06:47
*** yuntongjin1 has quit IRC06:47
*** irenab has joined #openstack-nova06:49
*** xgsa has quit IRC06:53
*** aarefiev has joined #openstack-nova06:54
*** andreykurilin has joined #openstack-nova06:57
*** claudiub has joined #openstack-nova06:59
*** otter768 has joined #openstack-nova07:00
*** haomaiwang has quit IRC07:01
*** haomaiwang has joined #openstack-nova07:01
*** rubasov has joined #openstack-nova07:03
*** lpetrut has joined #openstack-nova07:03
*** kiran-r has joined #openstack-nova07:04
*** kiran-r has quit IRC07:05
*** kiranr has joined #openstack-nova07:05
*** kiranr is now known as kiran-r07:06
*** karimb has joined #openstack-nova07:06
*** boris-42 has joined #openstack-nova07:07
openstackgerritHans Lindgren proposed openstack/nova: Remove and deprecate conductor provider_fw_rule_get_all()  https://review.openstack.org/21624107:08
openstackgerritHans Lindgren proposed openstack/nova: Remove unused provider firewall rules functionality in nova  https://review.openstack.org/18402707:08
openstackgerritHans Lindgren proposed openstack/nova: Remove provider_fw_rules tables and associated db api methods  https://review.openstack.org/21624207:08
*** wolsen has quit IRC07:08
*** rdopiera has joined #openstack-nova07:10
*** jichen has joined #openstack-nova07:15
*** penick has joined #openstack-nova07:17
*** nsun has joined #openstack-nova07:18
*** lpetrut has quit IRC07:20
*** thumpba has quit IRC07:21
*** rotbeard has quit IRC07:21
*** jsuchome has joined #openstack-nova07:23
*** matrohon has joined #openstack-nova07:24
*** penick has quit IRC07:24
*** thumpba has joined #openstack-nova07:25
*** dsanders has quit IRC07:26
*** asettle has quit IRC07:31
*** alex_klimov has joined #openstack-nova07:31
*** claudiub has quit IRC07:32
*** thumpba has quit IRC07:33
*** jichen has quit IRC07:33
*** jichen has joined #openstack-nova07:34
*** jichen has joined #openstack-nova07:35
*** cfriesen has quit IRC07:35
*** jichen has quit IRC07:35
*** jichen has joined #openstack-nova07:36
*** jichen has quit IRC07:38
*** jlanoux has joined #openstack-nova07:42
*** markus_z has joined #openstack-nova07:43
*** andrearosa has quit IRC07:43
*** Marga_ has joined #openstack-nova07:44
*** openstackgerrit has quit IRC07:46
*** openstackgerrit has joined #openstack-nova07:47
*** e0ne has joined #openstack-nova07:47
*** Marga_ has quit IRC07:48
*** e0ne has quit IRC07:49
*** alex_klimov has quit IRC07:52
*** e0ne has joined #openstack-nova07:54
*** jichen has joined #openstack-nova07:55
*** e0ne has quit IRC07:56
openstackgerritTimofey Durakov proposed openstack/nova: Wrap setup_networks_on_host with try/except  https://review.openstack.org/22789707:59
*** tdurakov has joined #openstack-nova07:59
*** haomaiwang has quit IRC08:01
*** kairo has quit IRC08:01
*** haomaiwang has joined #openstack-nova08:01
*** haomaiwang has quit IRC08:03
*** alex_klimov has joined #openstack-nova08:05
*** Sree has joined #openstack-nova08:08
openstackgerritRadoslav Gerganov proposed openstack/nova-specs: ConsoleAuthTokens should work for all console types  https://review.openstack.org/22877808:08
*** derekh has joined #openstack-nova08:08
*** nsun has quit IRC08:09
*** nsun has joined #openstack-nova08:09
*** Sree has quit IRC08:11
*** Sree has joined #openstack-nova08:12
*** xgsa has joined #openstack-nova08:12
*** romainh has joined #openstack-nova08:13
*** Sree has quit IRC08:16
*** alexpilotti has joined #openstack-nova08:17
*** dguitarbite has quit IRC08:18
*** markvoelker has joined #openstack-nova08:18
*** jistr has joined #openstack-nova08:18
*** xgsa1 has joined #openstack-nova08:19
*** xgsa has quit IRC08:21
*** markvoelker has quit IRC08:22
bauzasgood morning Nova08:24
*** jlanoux has quit IRC08:24
*** pixelbeat has joined #openstack-nova08:25
*** e0ne has joined #openstack-nova08:26
*** pmurray has joined #openstack-nova08:27
*** pmurray is now known as Guest351508:27
*** tdurakov has quit IRC08:28
*** PaulMurray has quit IRC08:28
nsunHello Nova08:29
*** thumpba has joined #openstack-nova08:30
*** xgsa1 has quit IRC08:30
nsunIn Juno, nova compute did not have to go through conductor to reach the db, and that started with kilo, right?08:31
johnthetubaguynsun: unsure, exactly when, but that did change, yes08:32
*** xgsa has joined #openstack-nova08:32
johnthetubaguynsun: what problems are you seening?08:32
*** dguitarbite has joined #openstack-nova08:32
*** ndipanov has joined #openstack-nova08:34
*** claudiub has joined #openstack-nova08:34
*** thumpba has quit IRC08:34
bauzasjohnthetubaguy: meh, do you remember if we have a bug report for http://logs.openstack.org/87/188487/35/check/gate-tempest-dsvm-full/b837ff0/logs/screen-n-cpu.txt.gz?level=TRACE#_2015-09-30_13_11_53_22908:34
nsun@johnthetubaguy: It is not a probem per se. I am developing a filter for the scheduler, and nova compute needs the info from the filter. Unfortunately, our OpenStack release is still juno-based.08:34
bauzasjohnthetubaguy: I remember sdague and dansmith discussing about that08:34
bauzasjohnthetubaguy: because e-r seems not having the bug08:35
bauzasnsun: to answer your question, it depends08:35
*** e0ne has quit IRC08:36
bauzasnsun: because I remember some objectification was needed in Juno, but I'm unsure if all the methods were accessed thru the conductor on the compute manager codebase08:36
johnthetubaguybauzas: hmm, yes, actually I saw someone asking about this bug on #openstack the other day, unsure08:36
*** haomaiwang has joined #openstack-nova08:37
bauzasjohnthetubaguy: lemme look, I perhaps missed it08:37
bauzasjohnthetubaguy: because that one is just -1'ing the Tempest regular job08:37
johnthetubaguynsun: we don't usually have much info from from the scheduler to the compute, apart from the node and host08:37
nsunbauzas:sorry, depends on what? We have Mirantis 6.1, which is Juno with some backports from Kilo.08:37
*** e0ne has joined #openstack-nova08:38
*** tdurakov has joined #openstack-nova08:38
bauzasjohnthetubaguy: in Juno, we were no longer calling from the scheduler IIRC, alaski merged that by Icehouse08:38
johnthetubaguyndipanov: did you see this log message a lot in devstack "Trying to apply a migration context that does not seem to be set for this instance"08:38
bauzasnsun: it depends on which method the compute manager is calling08:39
johnthetubaguyndipanov: I guess that could be that context manager on instance spawn?08:39
bauzasnsun: but all of that was passed thru the conductor anyway08:39
johnthetubaguyndipanov: looking at: http://logs.openstack.org/87/188487/35/check/gate-tempest-dsvm-full/b837ff0/logs/screen-n-cpu.txt.gz?level=TRACE#_2015-09-30_13_11_53_22908:39
johnthetubaguynsun: I am not sure we really understand what you are trying to do, so its hard to help, whats the rough idea here?08:40
ndipanovjohnthetubaguy, hmmm08:40
*** browne1 has quit IRC08:41
*** yuntongjin has quit IRC08:44
*** Marga_ has joined #openstack-nova08:45
bauzasjohnthetubaguy: I remember danpb discussing on that too08:45
bauzaswe should have a bug :)08:45
*** salv-orlando has joined #openstack-nova08:46
ndipanovso maybe I should clarify that message - this is a normal situation when we do a rebuild (not evacuate)08:46
ndipanovjohnthetubaguy, ^08:46
bauzasjohnthetubaguy: okay, I was blind http://status.openstack.org/elastic-recheck/#150136608:46
ndipanovI am guessing that tests is doing a rebuild not a migrate/resize?08:46
*** salv-or__ has joined #openstack-nova08:46
johnthetubaguyndipanov: looks like a build08:47
johnthetubaguyndipanov: maybe just turn it down to debug? although I guess in some cases we want to know the migration is missing?08:47
bauzasthis is for a build08:47
johnthetubaguybauzas: ah, good we have an er08:48
nsunjohnthetubaguy: Basically, we are developing a PCI device with some functions and some resources (like memory in the PCI card). The problem we are solving is to place VMs on hosts which have PCI cards with free amounts of requires resources. My sktch of the solution is to have the scheduler pick not just the host but also the specific device in the host (there can be many) that have08:48
nsunfree resources. The device identity and resource info needs to be available to nova compute when VM is placed on the host08:48
nsunSorry for the long message08:48
bauzashah08:48
ndipanovfor a build??08:48
ndipanovO_o08:48
bauzasndipanov: yup, _build_resources()08:49
*** salv-orl_ has quit IRC08:49
ndipanovare we talking about the same thing?08:49
johnthetubaguynsun: so the upstream support in a later version, I think does all that in the resource tracker, the scheduler just estimates what is free, and then the compute picks the correct one08:49
bauzasndipanov: https://bugs.launchpad.net/nova/+bug/150136608:49
openstackLaunchpad bug 1501366 in OpenStack Compute (nova) "libvirtError: Error while building firewall: Some rules could not be created for interface" [High,Confirmed]08:49
*** Marga_ has quit IRC08:50
*** oomichi has joined #openstack-nova08:50
johnthetubaguybauzas: yeah, we are talking about extra logs08:50
johnthetubaguyI crossed the streams a bit there... oops08:50
*** sudipto has joined #openstack-nova08:50
*** salv-orlando has quit IRC08:50
ndipanovso johnthetubaguy never cross the streams :)08:50
ndipanovso hmmm08:50
bauzasnever cross the streams unless it's for Zuul08:50
johnthetubaguyndipanov: ah, but its fun08:51
* johnthetubaguy make big boom noises, in his head08:51
*** jlanoux has joined #openstack-nova08:51
bauzasoh, Gozer, not Zuul, my bad08:51
nsunjohnthetubaguy: if the scheduler does not track the free resources per device, we could have races. E.g. VM 1 is placed on a host/device assuming the device has X amount of free resources. By the time VM 1 is placed and boots up, VM 2 placement may be assigned the same resource.08:53
nsunjohnthetubaguy: The scheduler cusom filter should note it in a db, and let nova compute pick info from the db (through the conductor, perhaps)08:54
nsunjohnthetubaguy:: s/cusom/custom08:54
ndipanovtinking - maybe it is worth demoting that to DEBUG08:54
johnthetubaguynsun: yes, it does race, but we have a retry to loop to limit the damage08:55
johnthetubaguynsun: there are some rough plans to eventually call over to the resource tracker to make the reservation, but not sure anyone has started work on that yet08:56
openstackgerritTimofey Durakov proposed openstack/nova: Wrap setup_networks_on_host with try/except  https://review.openstack.org/22789708:56
johnthetubaguynsun: technically, I think the idea is the reverse, move the claims into the scheduler, but similar outcome08:56
*** deepthi has quit IRC08:57
*** rajesht has joined #openstack-nova08:58
*** jlanoux has quit IRC08:58
*** sudipto has quit IRC08:58
*** lpetrut has joined #openstack-nova08:58
bauzasjohnthetubaguy: nsun: yup, the idea is that we strenghen how we claim for resources with some distributed schedulers08:59
nsunjohnthetubaguy: Moving the claims into the scheduler is what I am looking at too. Allowing races and re-trying is less than optimal :) Any thoughts on using a db table to prevent races? Say scaling is not important for now, it is a custom filter, and I am willing to play by the rules of later releases to not touch db directly but only through conductor08:59
*** jlanoux has joined #openstack-nova08:59
bauzasnsun: to be clear, retries are good and I'm all for having some09:00
*** exploreshaifali has joined #openstack-nova09:00
bauzasnsun: what's bad is when that retry ratio is increasing or above a certain metric09:00
ndipanovnsun, so I'd like to point out (as I always do when this comes up) - retries and lockless scheduler is a _design decision_09:00
*** haomaiwang has quit IRC09:01
bauzasndipanov: I think the consensus is there :)09:01
*** 6A4AALXHF has joined #openstack-nova09:01
ndipanovso deciding to change this likely requires looking into what you are getting and giving up09:02
bauzaswho's you ?09:02
nsunbauzas,ndipanov: please explain why re-tries are good. I understand you don't want to lock a *host* but resevrving specific resources in specific devices in a host must be far less restrictive?09:02
ndipanovyou = an eager developer09:03
bauzasnsun: okay, lemme clarify09:03
bauzasnsun: having a locking decision is different from having a distributed resource handler09:03
johnthetubaguythey both work, and its a trade off really09:04
bauzasnsun: if we lock, it means that we need a 2-phase commit for ensuring that the compute manager is able to handle the resource (that's what we call ResourceTracker 'claims' )09:04
*** aix has joined #openstack-nova09:04
bauzasjohnthetubaguy: yup, I agree09:04
ndipanovnsun, if you have enough capacity - not locking in the scheduler (in theory) means that the average request completes more quickly with possibility of odd ones failing and retrying09:04
bauzasyeah that09:05
bauzassince we have 2 accountabilities for resources (one in the scheduler, and one in the compute node), we need somehow to accept to make errors in the scheduler in order to not wait for getting a feedback from the compute09:06
ndipanovretrying a failed request has no consequences from Nova POV - "your VM failed, oh that's so sad, go try it again"09:06
bauzasif the opposite, it would be sub-optimal09:06
ndipanovand not locking - and tracking resources on the compute nodes makes some things much simpler09:07
ndipanovso there are benefits under the circumstances09:07
bauzasto be clear, I take the word 'lock' as the action to wait for the compute manager answer09:07
ndipanovbauzas, I take it to mean "lock on the global resource record"09:08
bauzasndipanov: but considering a distributed scheduler, there need to have some concurrency management09:09
ndipanovi.e. there is only one view of resources not 209:09
ndipanovor N09:09
ndipanovso ACID databases provide that if you use them properly right?09:09
ndipanov(in theory) :)09:10
bauzashah09:10
*** deepthi has joined #openstack-nova09:10
*** deepakcs has joined #openstack-nova09:11
bauzaswe haven't yet discussed on what could be the concurrency system for a distributed scheduler, so I defer the implementation to the convos that will eventually occur09:11
ndipanovnsun, is thae above making any sense so far09:11
* bauzas doesn't try to look evasive :p09:11
ndipanovbauzas, regardless of the implementation - centralizing resoruce tracking would bring other issues which is the point09:12
nsunbauzas,ndipanov: Let me clarify. The resources I deal with are not plentiful. E.g. memory in a custom PCI card, with may be 1-2 GB but the device is shared among many VMs, so the resource is scarce. Secondly, in my model, the info about free devices/resources is available to scheduler --- say compute nodes periodially update it. So, it is a local lookup and reservation -- sched does09:12
nsunnot query/update compute node at decision time.09:12
bauzasndipanov: comments welcome on https://review.openstack.org/#/c/191914/09:12
nsunbauzas,ndipanov: Rather, the sched looks at local db tables and updates them. Nova compute on selected host retreives sched info and uses it09:12
*** alexpilotti has quit IRC09:13
*** foexle has joined #openstack-nova09:14
bauzasnsun: no, the scheduler doesn't write on DB09:14
ndipanovnsun, there have been several proposals in the past09:15
nsunbauzas: a custom filter cannot access a db?09:15
bauzasnsun: it reads an in-memory state which is populated once from the DB, but it never sends updates to the DB09:15
bauzasnsun: of course, you could land an out-of-tree filter which would call the DB, but that's not acceptable in-tree09:16
*** sudipto has joined #openstack-nova09:17
nsunbauzas: is the restriction against db access due to perf/scaling considerations?09:18
bauzasnsun: yup09:18
*** xgsa1 has joined #openstack-nova09:18
*** x3k` has joined #openstack-nova09:18
bauzasnsun: and also because we have a clear pattern09:18
bauzasnsun: it's a design decision again09:19
nsunA cloud admin can anyway choose which filters he will deploy. And this filter need not be in the default list in the tree. So, a default deployment will have no impact, and interested cloud admins can choose to use this filter after investigating its perf/scaling characteristics.09:19
*** xgsa has quit IRC09:19
*** deepthi has quit IRC09:19
bauzasnsun: sure, but it would still be out-of-tree :)09:20
bauzasnsun: filters are treated equal whether they are default or not09:21
bauzasjust because any operator shouldn't have to take in consideration design concerns before enabling a filter09:21
nsunbauzas: I see :) Let me think through this some more. Thanks very much to you and ndipanov !09:21
bauzasnsun: NP09:21
bauzasoops09:22
bauzasnp09:22
*** danpb has joined #openstack-nova09:23
*** andrearosa has joined #openstack-nova09:23
*** signed8bit_ZZZzz has quit IRC09:24
*** mgoddard_ has joined #openstack-nova09:25
*** kumar has joined #openstack-nova09:26
*** salv-orlando has joined #openstack-nova09:26
*** mgoddard_ has left #openstack-nova09:27
*** mgoddard_ has joined #openstack-nova09:28
*** IanGovett has joined #openstack-nova09:28
*** salv-or__ has quit IRC09:30
*** andrearosa has quit IRC09:30
*** kiran-r has quit IRC09:30
openstackgerritNikola Dipanov proposed openstack/nova: Change warn to debug logs when migration context is missing  https://review.openstack.org/22979209:31
*** kiran-r has joined #openstack-nova09:32
*** marzif has joined #openstack-nova09:32
*** haypo has joined #openstack-nova09:33
*** x3k` is now known as x3k09:35
*** deepthi has joined #openstack-nova09:37
*** Sree has joined #openstack-nova09:38
ndipanovjohnthetubaguy, ^09:38
johnthetubaguyndipanov: yeah, just got that open, I think thats a good move while we work out what we need there09:39
openstackgerritMarian Horban proposed openstack/nova: Fix Status-Line in HTTP response  https://review.openstack.org/22323509:39
johnthetubaguyndipanov: did we get your change into liberty, thinking we might want to add a bug and backport it?09:39
*** mgoddard_ has quit IRC09:40
ndipanovok I can do that09:40
ndipanovwhich one?09:40
*** mgoddard_ has joined #openstack-nova09:40
ndipanovthe context manager one is proposed but -2ed by ttx09:40
ndipanovthis one needs to merge first on master I guess09:40
ndipanovjohnthetubaguy, ^09:40
johnthetubaguyndipanov: yeah, he will unblock that on tuesday09:40
johnthetubaguybasically waiting to open RC2 while folks test RC109:41
johnthetubaguyunsure about why we can't just merge stuff in the meantime, but I suspect there is some reason09:41
*** salv-orl_ has joined #openstack-nova09:42
*** Sree has quit IRC09:44
*** salv-orlando has quit IRC09:45
*** Marga_ has joined #openstack-nova09:46
*** digimonde has joined #openstack-nova09:47
*** boris-42 has quit IRC09:49
*** Marga_ has quit IRC09:51
*** 6A4AALXHF has quit IRC09:52
*** xgsa has joined #openstack-nova09:52
*** xgsa1 has quit IRC09:52
*** haomaiwang has joined #openstack-nova09:52
*** e0ne has quit IRC09:53
*** exploreshaifali has quit IRC09:54
*** dims has joined #openstack-nova09:55
*** e0ne has joined #openstack-nova09:56
*** alexpilotti has joined #openstack-nova09:58
*** haomaiwang has quit IRC10:01
*** haomaiwang has joined #openstack-nova10:01
*** stevemar has quit IRC10:08
*** stevemar has joined #openstack-nova10:09
*** amotoki_ has joined #openstack-nova10:11
*** ZZelle has quit IRC10:12
*** ZZelle has joined #openstack-nova10:13
*** stevemar has quit IRC10:13
*** setuid has quit IRC10:15
*** setuid has joined #openstack-nova10:15
*** romainh has quit IRC10:19
*** markvoelker has joined #openstack-nova10:19
*** amotoki_ has quit IRC10:19
*** kiran-r has quit IRC10:21
*** markvoelker has quit IRC10:23
*** tdurakov has quit IRC10:25
*** mwagner_ has quit IRC10:26
*** thumpba has joined #openstack-nova10:32
*** eglynn has joined #openstack-nova10:33
*** nsun has left #openstack-nova10:33
*** alexpilotti has quit IRC10:35
*** alexpilotti has joined #openstack-nova10:36
*** andrearosa has joined #openstack-nova10:37
*** thumpba has quit IRC10:37
sdaguejohnthetubaguy: so, melwitt WIPed what is the concensus patch to fix the novaclient boot from volume issue (because she wanted concensus to emerge). ttx would like to merge and release as soon as possible because of RC210:45
sdaguedo you want to update the commit message to clear the WIP bit so we can land it?10:45
johnthetubaguyI can do, certainly seems urgent10:46
*** setuid has quit IRC10:46
johnthetubaguyyou got the link again?10:46
sdagueyep10:46
sdaguehttps://review.openstack.org/#/c/229669/10:46
*** amotoki_ has joined #openstack-nova10:46
*** smatzek has joined #openstack-nova10:46
*** Marga_ has joined #openstack-nova10:46
*** agireud has quit IRC10:48
*** setuid has joined #openstack-nova10:48
johnthetubaguysdague: FWIW, I have a feeling we should probably do #1 and #210:49
bauzasjohnthetubaguy: btw. re: RC2 and possibly RC3, do you have more info on any windows ?10:49
bauzasjohnthetubaguy: since the GA is in 2 weeks IIRC10:49
johnthetubaguybauzas: yes, stalling RC2 and just going for RC2 next week10:49
johnthetubaguybecause non of the current pending bugs are serious enough to stop folks testing RC110:49
sdaguejohnthetubaguy: well, consensus so far was just #110:49
johnthetubaguyunless I missed something10:49
bauzasjohnthetubaguy: okay, it means if we need a RC3, it will only for 3 or 4 days ?10:49
johnthetubaguysdague: yeah10:50
openstackgerritTang Chen proposed openstack/nova: Replace 'dummy' status with MIGRATION_STATUS_DUMMY.  https://review.openstack.org/22817710:50
openstackgerritTang Chen proposed openstack/nova: Replace 'failed' status with MIGRATION_STATUS_FAILED.  https://review.openstack.org/22817610:50
openstackgerritTang Chen proposed openstack/nova: Replace 'post-migrating' status with MIGRATION_STATUS_POST_MIGRATING.  https://review.openstack.org/22816610:50
openstackgerritTang Chen proposed openstack/nova: Replace 'finished' statuses with MIGRATION_STATUS_FINISHED.  https://review.openstack.org/22816710:50
openstackgerritTang Chen proposed openstack/nova: Replace 'migrating' status with MIGRATION_STATUS_MIGRATING.  https://review.openstack.org/22816410:50
openstackgerritTang Chen proposed openstack/nova: Replace 'running' status with MIGRATION_STATUS_MIGRATING.  https://review.openstack.org/22816510:50
openstackgerritTang Chen proposed openstack/nova: Replace 'preparing' status with MIGRATION_STATUS_PREPARING.  https://review.openstack.org/22816210:50
openstackgerritTang Chen proposed openstack/nova: Replace 'pre-migrating' status with MIGRATION_STATUS_PRE_MIGRATING.  https://review.openstack.org/22816310:50
openstackgerritTang Chen proposed openstack/nova: Introduce constants to indicate live migration status.  https://review.openstack.org/22816110:50
openstackgerritTang Chen proposed openstack/nova: Replace 'reverted' status with MIGRATION_STATUS_REVERTED.  https://review.openstack.org/22817410:50
openstackgerritTang Chen proposed openstack/nova: Replace 'error' status with MIGRATION_STATUS_ERROR.  https://review.openstack.org/22817510:50
openstackgerritTang Chen proposed openstack/nova: Replace 'confirmed' status with MIGRATION_STATUS_CONFIRMED.  https://review.openstack.org/22817210:50
sdaguewe could always do #2 later if you really wanted10:50
openstackgerritTang Chen proposed openstack/nova: Replace 'reverting' status with MIGRATION_STATUS_REVERTING.  https://review.openstack.org/22817310:50
openstackgerritTang Chen proposed openstack/nova: Replace 'queued' status with MIGRATION_STATUS_QUEUED.  https://review.openstack.org/22817010:50
openstackgerritTang Chen proposed openstack/nova: Replace 'confirming' status with MIGRATION_STATUS_CONFIRMING.  https://review.openstack.org/22817110:50
openstackgerritTang Chen proposed openstack/nova: Replace 'completed' status with MIGRATION_STATUS_FINISHED.  https://review.openstack.org/22816810:50
openstackgerritTang Chen proposed openstack/nova: Replace 'done' status with MIGRATION_STATUS_FINISHED.  https://review.openstack.org/22816910:50
johnthetubaguybauzas: I don't plan on needing an RC3, at least I hope not10:50
johnthetubaguyoh my...10:50
johnthetubaguysdague: yeah10:50
bauzasjohnthetubaguy: okay, my point is that we need to review any new bug to see if we need a RC3 then10:50
johnthetubaguybauzas: agreed10:51
bauzasbecause the window is quite short10:51
markus_zbauzas got overtaken :D10:51
johnthetubaguywell, depends how you look at it, we have had more time for RC2, so noting much changes there10:51
*** Marga_ has quit IRC10:51
bauzasper https://wiki.openstack.org/wiki/Liberty_Release_Schedule the release is planned Oct12-1610:51
johnthetubaguywell, except RC2 specific bugs I guess10:51
johnthetubaguybauzas: agreed10:51
bauzaswhich leaves 2 weeks10:52
johnthetubaguybauzas: we need to wait 10 days from RC1 to include updated translations10:52
bauzasokay10:52
johnthetubaguyas previously agreed10:52
bauzasyeah that's understandable10:52
johnthetubaguywe have some bugs all lined up for RC2, but non of them are bad enough to do RC2 and RC310:52
johnthetubaguyunless I missed one10:53
bauzasokay, if we open the RC2 window by tuesday, it'll leave one week to see whether we need a RC310:53
bauzasjohnthetubaguy: yup, that's also MHO10:53
johnthetubaguynot really sure what you are meaning here10:53
johnthetubaguyI am quite unforgiving I guess, if folks want there stuff to work, they best get a CI testing it against master10:54
bauzassorry, I meant +1 for "but non of them are bad enough to do RC2 and RC3"10:54
johnthetubaguyah, gotcha10:54
sdaguebauzas: rc3 isn't a thing you plan for, it's a thing you do if something terrible exposes at the last minute10:55
sdaguejohnthetubaguy: anyway, the novaclient patch10:55
bauzassdague: yup, MHO is that by having RC2 open in the last week, it leaves not a lot of time for getting a RC210:55
bauzasoops10:55
openstackgerritJohn Garbutt proposed openstack/python-novaclient: Always send volume_id when booting with legacy bdm  https://review.openstack.org/22966910:55
bauzasfor getting a RC3*10:55
*** sudipto has quit IRC10:56
johnthetubaguysdague: yeah, I should press the button, does that work?10:56
sdaguejohnthetubaguy: you can use the edit button10:56
johnthetubaguysdague: yeah, that should all be done now10:56
sdaguebauzas: sure, but the only things that can't be found in RC1 are bugs introduced by patches that are in RC210:57
johnthetubaguyedit button has saved me quite a few hours this release10:57
bauzassdague: oh I see your point, agred10:57
johnthetubaguysdague: yeah, thats what I was attempting to say!10:57
bauzas:)10:57
sdague:)10:58
sdagueok, so I approved the novaclient patch10:58
*** gszasz has joined #openstack-nova10:58
johnthetubaguysweet, thank you10:58
sdagueas soon as that's landed we can circle with ttx to get that released. He's lunching right now10:58
sdaguethen I'll see if we expose any new fun in trying to get upgrades from liberty working10:59
mgoddard_Hi all, SecurityFun23 has found an issue with my fix for https://bugs.launchpad.net/nova/+bug/1467581. First question - create a new bug or keep the work attached to the existing one?10:59
openstackLaunchpad bug 1467581 in OpenStack Compute (nova) "Concurrent interface attachment corrupts info cache" [High,Fix released] - Assigned to Mark Goddard (mgoddard)10:59
*** haomaiwang has quit IRC11:01
*** haomaiwang has joined #openstack-nova11:01
rajeshtgaryk: could you please review stable/kilo patch https://review.openstack.org/#/c/229261/11:04
rajeshtgaryk: thanks :)11:04
*** marzif has quit IRC11:04
*** marzif has joined #openstack-nova11:05
dimssdague: johnthetubaguy: the boot from volume review needs to be cherry picked to stable/liberty i think. which means this needs to be blessed first https://review.openstack.org/#/c/226032/11:06
sdaguedims: well, that just makes life easier11:07
sdaguebut sure11:07
sdaguedims: I think that's stable maint powers there11:07
sdagueso can you +2 ?11:07
*** bmotz has joined #openstack-nova11:08
johnthetubaguyyeah, I don't have +2 in stable11:08
dimschecking11:08
dimsonly +1 karma11:09
dimsgaryk: around? https://review.openstack.org/#/c/226032/ please11:10
dimsjohnthetubaguy: you can cherry pick and sdague can +2 the boot from volume review?11:11
flaper87dims: +A11:12
* flaper87 just happened to have this buffer opened11:12
dimsflaper87: thanks!. one more on the way11:12
johnthetubaguydims: sdague: ah, you mean the one we just approved?11:12
dimsjohnthetubaguy: yes, need to cut the python-novaclient from stable/liberty11:12
dimshttps://review.openstack.org/#/c/229669/11:13
sdaguedims: doing it11:13
sdaguedims: looks like it will be slightly more manual, one sec11:13
*** marzif has quit IRC11:14
dimssdague: ack. after those 2 merge on stable/liberty, we can throw in a review for openstack/releases repo to request a release11:14
*** marzif has joined #openstack-nova11:15
*** marzif has quit IRC11:16
mgoddard_bauzas: hi, any thoughts - ' SecurityFun23 has found an issue with my fix for https://bugs.launchpad.net/nova/+bug/1467581. First question - create a new bug or keep the work attached to the existing one?'11:16
openstackLaunchpad bug 1467581 in OpenStack Compute (nova) "Concurrent interface attachment corrupts info cache" [High,Fix released] - Assigned to Mark Goddard (mgoddard)11:16
*** marzif has joined #openstack-nova11:16
johnthetubaguysdague: dims: I just gave it a while, but there was a clash in the test, which I just blindly ignored: https://review.openstack.org/#/c/229826/11:17
bauzasmgoddard_: if the bug is fix released, then open a new one11:17
sdagueoh, hey, I just raced with johnthetubaguy on it :)11:17
dimsjohnthetubaguy: ack11:17
johnthetubaguysdague: so I just dropped the adding of the COMPUTE_APU_VERSION, but I didn't dig into why thats there11:17
sdagueit's there in master so I think we keep it11:18
mgoddard_bauzas: ok, will do. I have a fix that seems to work, needs a little more testing. Is Liberty fairly locked down now or might it be possible to go in?11:18
sdagueI ended up pushing over yours by accident because gerrit and racing11:18
sdaguebut I think it's fine11:18
bauzasmgoddard_: well, that depends on if it's a regression or a security breach or something else :)11:19
sdaguethe COMPUTE_API_VERSION is the bit that ensures that it hits the v2.1 api, which is important here, otherwise we don't trigger the schema validation issue11:19
*** Daisy has joined #openstack-nova11:19
bauzasmgoddard_: I'm not a vulnerability team member, so I guess you should double-check with those guys11:20
johnthetubaguysdague: ah, cool11:20
johnthetubaguysdague: that makes sense11:20
bauzasmgoddard_: oh sorry, I thought that SecuityFun23 was reporting a vulnerability11:21
mgoddard_bauzas: it shouldn't be a security issue but I can check. It may be a regression, but it was broken in a different way before so hard to verify11:21
bauzasmgoddard_: so he's just asking for a backport, right?11:21
mgoddard_bauzas: At first he wanted a backport, but then he found an issue with the fix. So now he wants a backport and a backported fix :)11:22
bauzas?11:22
sdaguejohnthetubaguy: you want to +2 https://review.openstack.org/#/c/229826/11:22
bauzasokay, I need more details11:22
*** exploreshaifali has joined #openstack-nova11:22
bauzasmgoddard_: so the fix of https://bugs.launchpad.net/nova/+bug/1467581 landed in master after we cut liberty?11:23
openstackLaunchpad bug 1467581 in OpenStack Compute (nova) "Concurrent interface attachment corrupts info cache" [High,Fix released] - Assigned to Mark Goddard (mgoddard)11:23
bauzasoh no11:23
bauzasit's in the liberty branch11:23
mgoddard_bauzas: That fix should be in Liberty, but the fix was found to highlight another bug11:25
bauzasmgoddard_: that's what I'm reading11:25
*** pbrooko has joined #openstack-nova11:25
bauzasmgoddard_: but I don't think it's a regression IMHO11:25
*** ildikov has joined #openstack-nova11:25
mgoddard_bauzas: whack-a-mole - fix one race condition, find another11:25
bauzasmgoddard_: I mean, you made a fix which was partial, so you still need to fix another stuff11:25
*** ansiwen has joined #openstack-nova11:26
*** marzif has quit IRC11:26
bauzasmgoddard_: so the fix of the fix should land in master, but I doubt it would be worth for liberty unless it's breaking something that I misunderstood11:26
bauzas(speaking of a fix of the fix backport to liberty of course)11:26
*** marzif has joined #openstack-nova11:26
sdaguemgoddard_: right, at this point unless it's a regression from behavior in kilo, it's probably not suitable for liberty11:27
mgoddard_bauzas: The new bug would have been present but less obvious the way that the code worked before, but can't say for sure whether it's now better or worse than before11:27
*** ansiwen has joined #openstack-nova11:27
mgoddard_I think better...11:27
bauzasmgoddard_: since I'm not fully confident with that, I suggest you to open a new bug explaining the current situation and what's missing11:27
*** aix has quit IRC11:27
bauzasmgoddard_: because the current bug is just super verbose also :)11:28
mgoddard_bauzas: will do. I'll submit a patch to master, and let others decide on its suitability for liberty11:28
mgoddard_bauzas: thanks for the help11:28
bauzasmgoddard_: yeah, ping again in the channel once the bug and the fix are landed so we could get better visibility11:29
bauzas(once the bug report*)11:29
mgoddard_bauzas: sure11:29
* bauzas goes grab some food11:30
* johnthetubaguy also goes for food11:30
*** baoli has joined #openstack-nova11:31
*** thumpba has joined #openstack-nova11:33
*** markvoelker has joined #openstack-nova11:34
*** marzif has quit IRC11:36
*** marzif has joined #openstack-nova11:37
*** thumpba has quit IRC11:37
*** markvoelker has quit IRC11:39
garykdims: i looked and it is goo - already approved :)11:39
dimsthanks!11:40
*** dave-mccowan has joined #openstack-nova11:41
*** pbrooko has quit IRC11:42
*** deepthi has quit IRC11:45
*** marzif has quit IRC11:46
*** Marga_ has joined #openstack-nova11:47
*** Daisy has quit IRC11:47
*** Daisy has joined #openstack-nova11:48
*** marzif has joined #openstack-nova11:49
*** baoli has quit IRC11:52
*** Marga_ has quit IRC11:52
*** amotoki_ is now known as amotoki11:52
*** Daisy has quit IRC11:52
*** apuimedo has quit IRC11:54
*** tdurakov has joined #openstack-nova11:55
*** abhishekk has quit IRC11:57
*** apuimedo has joined #openstack-nova11:57
*** haomaiwang has quit IRC12:01
*** haomaiwang has joined #openstack-nova12:01
*** markvoelker has joined #openstack-nova12:01
*** _cerberus_ has quit IRC12:03
*** MVenesio has joined #openstack-nova12:04
*** aarefiev has quit IRC12:04
*** _cerberus_ has joined #openstack-nova12:05
*** marcusvrn_ has joined #openstack-nova12:06
*** marzif has quit IRC12:06
*** marzif has joined #openstack-nova12:07
*** kiran-r has joined #openstack-nova12:09
*** brad[] has joined #openstack-nova12:09
*** romainh has joined #openstack-nova12:09
*** aarefiev has joined #openstack-nova12:12
*** rajesht has quit IRC12:15
*** pbrooko has joined #openstack-nova12:16
openstackgerritRadoslav Gerganov proposed openstack/nova: VMware: Use datastore copy when the image is already in vSphere  https://review.openstack.org/18671612:16
*** pbrooko has quit IRC12:17
*** marzif has quit IRC12:17
*** jerrygb has quit IRC12:17
*** marzif has joined #openstack-nova12:17
*** haomaiwang has quit IRC12:18
*** haomaiwang has joined #openstack-nova12:19
*** abhishekk has joined #openstack-nova12:21
*** kiran-r has quit IRC12:21
*** mwagner_ has joined #openstack-nova12:23
*** ociuhandu has quit IRC12:24
*** deepakcs has quit IRC12:25
*** annegentle has joined #openstack-nova12:25
*** VikasC has quit IRC12:27
*** alejandrito has joined #openstack-nova12:27
*** cristicalin has joined #openstack-nova12:28
*** raildo-afk is now known as raildo12:28
*** baoli has joined #openstack-nova12:28
*** Daisy has joined #openstack-nova12:31
*** alex_klimov has quit IRC12:33
*** thumpba has joined #openstack-nova12:34
openstackgerritjichenjc proposed openstack/nova: Add DiskFormat as Enum in fields  https://review.openstack.org/22985412:34
*** pixelbeat has quit IRC12:34
*** ctrath has joined #openstack-nova12:36
*** pixelbeat has joined #openstack-nova12:37
*** ctrath has quit IRC12:37
*** vladikr has joined #openstack-nova12:40
*** sakthi has quit IRC12:40
*** tdurakov has quit IRC12:41
*** stevemar has joined #openstack-nova12:41
*** salv-orl_ has quit IRC12:42
*** edmondsw has joined #openstack-nova12:42
*** tdurakov has joined #openstack-nova12:42
tdurakovjohnthetubaguy, hi12:44
tdurakovcould we discuss about https://review.openstack.org/#/c/227897/12:45
tdurakovso, imho we need to separate network and driver cleanup in 2 methods, this allows to use decorators.12:46
*** stevemar has quit IRC12:46
*** stevemar has joined #openstack-nova12:47
*** kumar has quit IRC12:47
*** Marga_ has joined #openstack-nova12:48
*** thumpba has quit IRC12:50
tdurakovwhat about wrap_instance_event? Should i remove it from network cleanup?12:50
*** stevemar has quit IRC12:52
*** tdurakov has quit IRC12:53
*** Marga_ has quit IRC12:53
*** tpeoples has joined #openstack-nova12:54
*** jaypipes has joined #openstack-nova12:58
*** thorst has joined #openstack-nova12:58
*** thorst has quit IRC12:59
*** haomaiwang has quit IRC13:01
*** pm90_ has joined #openstack-nova13:01
*** haomaiwang has joined #openstack-nova13:01
*** alex_klimov has joined #openstack-nova13:02
*** pm90__ has joined #openstack-nova13:02
*** rlrossit has joined #openstack-nova13:03
openstackgerritRaildo Mascena de Sousa Filho proposed openstack/nova-specs: Moving Nested Quota Driver to Mitaka  https://review.openstack.org/20996913:05
*** pm90__ has quit IRC13:05
*** pm90_ has quit IRC13:06
*** pm90_ has joined #openstack-nova13:11
*** mdrabe_ has joined #openstack-nova13:11
*** ccarmack has joined #openstack-nova13:13
*** rfolco has joined #openstack-nova13:16
*** pm90__ has joined #openstack-nova13:18
*** dustins has joined #openstack-nova13:18
*** _gryf has quit IRC13:19
*** _gryf has joined #openstack-nova13:19
*** ctrath has joined #openstack-nova13:19
*** marzif has quit IRC13:20
*** marzif has joined #openstack-nova13:20
*** pm90_ has quit IRC13:21
*** mriedem has joined #openstack-nova13:23
*** irenab has quit IRC13:23
*** zzzeek has joined #openstack-nova13:23
*** marzif has quit IRC13:25
*** signed8bit has joined #openstack-nova13:26
openstackgerritBartosz Fic proposed openstack/nova: Block migrate with attached volumes copies volumes to themselves  https://review.openstack.org/22727813:27
*** diogogmt has joined #openstack-nova13:30
openstackgerritBalazs Gibizer proposed openstack/nova-specs: Versioned notification API  https://review.openstack.org/22475513:32
openstackgerritjichenjc proposed openstack/nova: Add test cases for some classes in objects.fields  https://review.openstack.org/22988013:32
openstackgerritBalazs Gibizer proposed openstack/nova: [WIP] Add versioned notifications for keypair API  https://review.openstack.org/22988113:33
*** smatzek has quit IRC13:34
*** jecarey has joined #openstack-nova13:35
*** jichen has quit IRC13:37
*** tdurakov has joined #openstack-nova13:38
*** lpetrut has quit IRC13:39
*** lpetrut has joined #openstack-nova13:39
*** ijuwang has joined #openstack-nova13:40
bauzasndipanov: psssst pep8 import failure in https://review.openstack.org/#/c/229792/113:40
bauzas:)13:40
bauzasI stupidly asked for a recheck w/o seeing the pep8 failure :)13:41
openstackgerritBalazs Gibizer proposed openstack/nova-specs: Add notification for service status change  https://review.openstack.org/18235013:41
*** xyang1 has joined #openstack-nova13:41
*** Sree has joined #openstack-nova13:42
*** obedmr_ has joined #openstack-nova13:44
openstackgerritFlavio Percoco proposed openstack/nova-specs: Complete nova support for Glance's v2  https://review.openstack.org/22989113:46
*** Marga_ has joined #openstack-nova13:49
dansmithzzzeek: can you look at this? https://review.openstack.org/#/c/223320/4/nova/db/sqlalchemy/api.py,cm13:49
zzzeekdansmith: ok13:50
openstackgerritNikola Dipanov proposed openstack/nova: Change warn to debug logs when migration context is missing  https://review.openstack.org/22979213:51
*** nelsnelson has quit IRC13:52
*** armax has joined #openstack-nova13:52
zzzeekdansmith: if the query returns no rows prior to looking for “min”, then the function will return NULL which you’ll get back as None13:52
*** armax has quit IRC13:52
dansmithzzzeek: so does that mean my [0] will blow up?13:52
zzzeekdansmith: no13:52
zzzeekdansmith: um.   heh let me verify :)13:52
* alaski reads with interest13:52
dansmithheh, thanks13:53
dansmithso I hit this with one of my tests,13:53
dansmithwhich required me to tolerate getting back None from this function13:53
dansmithso I *think* it's okay13:53
dansmithbut there might be a case I'm not covering13:53
zzzeekdansmith: yes, you get None13:53
dansmithalaski: that's why I added None to the service object13:53
*** kylek3h has joined #openstack-nova13:53
dansmithzzzeek: None or [None]13:53
zzzeekhttp://paste.openstack.org/show/475074/13:53
*** Marga_ has quit IRC13:53
zzzeekfirst() will give you (None,) and then [0] will give you None.   pro-tip, call query.scalar() instead13:54
*** cristicalin has quit IRC13:54
dansmithah, cool.. so .scalar() instead of [0] ?13:54
*** stevemar has joined #openstack-nova13:54
zzzeekdansmith: yes13:54
dansmithcool13:54
zzzeekdansmith: no first() either13:54
dansmithorl13:55
zzzeekdansmith: query.sclalar()13:55
zzzeekscalar13:55
dansmithzzzeek: cool, thanks13:55
alaskidansmith: yeah, None makes sense.  just wasn't sure on the [0], though it seemed okay based on later patches13:55
*** armax has joined #openstack-nova13:55
dansmithalaski: yeah, but I'll change this anyway.. it'll make me look like I know what I'm doing13:55
alaskiheh13:56
bauzassdague: by looking at https://bugs.launchpad.net/nova/+bug/1500688 I saw that API v2.6 removed os-getVNCConsole resource13:56
openstackLaunchpad bug 1500688 in OpenStack Compute (nova) "VNC URL of instance unavailable in CLI" [Undecided,New]13:56
*** schwicke has joined #openstack-nova13:56
bauzassdague: so I suppose the error comes from the fact the client doesn't support microversions ?13:56
*** links has quit IRC13:57
bauzassdague: but if the client wasn't supporting microversions, we should serve v2.0 with os-getVNCConsole, nope13:57
bauzas?13:57
sdaguebauzas: the client supports microversions13:58
mriedembauzas: novaclient supports microversions13:58
bauzassdague: yup I know13:58
mriedembauzas: the cli defaults to the latest microversion on the server13:58
bauzassdague: I wonder then why it gets that 40413:58
*** stevemar has quit IRC13:59
bauzassdague: mriedem: https://github.com/openstack/nova/blob/master/nova/api/openstack/rest_api_version_history.rst#2613:59
sdagueso, I think the issue is that there is a gap in the implementations13:59
mriedembauzas: does novaclient handle that?13:59
mriedemsupport has to be added to novaclient to handle >=v2.614:00
sdagueno one implemented the post 2.6 return14:00
bauzasmriedem: IIRC, the client serves a client exception if the resource is not available on the server14:00
mriedemnova meeting14:00
johnthetubaguymikal tjones cburgess jgrimm adrian_otto funzo mjturek jcookekhugen irina_pov krtaylor danpb alexpilotti flip214 jaypipes garyk edleafe dims moshele anteaya Nisha sileht claudiub lxsli neiljerram markus_z swamireddy alevine tonyb andreykurilin ndipanov sc68cal akuriata artom jlvillal mnestratov kashyap aloga rgeragnov bauzas xyang tpatil med_ nic scottda nagyz dannywilson belliott dguitarbite sdague jroll diana_clarke _diana_ wznoinsk14:00
johnthetubaguy raildo its nova meeting time14:00
johnthetubaguythat list has got silly14:00
jrollohai14:00
sdagueiirc I saw 2.3 returns go in, then 2.11, but changes in the middle got skipped14:00
*** aix has joined #openstack-nova14:00
dimso/14:00
jaypipeslol14:00
*** haomaiwang has quit IRC14:01
*** thangp has joined #openstack-nova14:01
mriedemi added 2.4 support to the client14:01
dguitarbitehello14:01
mriedemit's kind of odd that the cli defaults to the latest microversion14:01
mriedemseems like a recipe for disaster (tm)14:01
*** haomaiwang has joined #openstack-nova14:01
raildojohnthetubaguy, thanks :)14:01
*** smatzek has joined #openstack-nova14:01
bauzasmriedem: not sure about defaulting to latest14:02
bauzasmriedem: IIRC, the client is exposing the max version he knows14:02
bauzasmriedem: so that the server knows if it's compatible14:02
bauzas(anyway, meeting time, holding off till we're freed)14:03
mriedembauzas: yeah, added to meeting open discussion14:03
*** eharney has joined #openstack-nova14:03
*** exploreshaifali has quit IRC14:03
*** rsritesh has joined #openstack-nova14:04
bauzasmriedem: cool thanks14:04
bauzasmriedem: I guess we'll need to poke john to ask him to refresh :)14:04
rsriteshHi,14:05
*** EinstCrazy has joined #openstack-nova14:05
*** VW has joined #openstack-nova14:05
*** willsama has joined #openstack-nova14:07
*** READ10 has joined #openstack-nova14:07
*** diogogmt has quit IRC14:07
*** mdrabe_ has quit IRC14:09
*** sajeesh has joined #openstack-nova14:10
*** dims has quit IRC14:10
*** david-lyle has quit IRC14:11
*** stevemar has joined #openstack-nova14:11
*** david-lyle has joined #openstack-nova14:13
*** signed8bit is now known as signed8bit_ZZZzz14:14
*** signed8bit_ZZZzz is now known as signed8bit14:17
*** electrocucaracha has joined #openstack-nova14:19
*** rsritesh has quit IRC14:19
*** rsritesh has joined #openstack-nova14:19
*** rsritesh has left #openstack-nova14:20
openstackgerritDan Smith proposed openstack/nova: Add get_minimum_version() to Service object and DB API  https://review.openstack.org/22332014:20
openstackgerritDan Smith proposed openstack/nova: Allow automatic determination of compute_rpc version by service  https://review.openstack.org/22332114:20
openstackgerritDan Smith proposed openstack/nova: Make service create/update fail if version is too old  https://review.openstack.org/22904914:20
*** erhudy has joined #openstack-nova14:20
*** nelsnelson has joined #openstack-nova14:23
openstackgerritRaildo Mascena de Sousa Filho proposed openstack/nova-specs: Moving Nested Quota Driver to Mitaka  https://review.openstack.org/20996914:23
*** EinstCrazy has quit IRC14:24
*** thumpba has joined #openstack-nova14:26
*** thumpba has quit IRC14:26
*** thumpba has joined #openstack-nova14:26
*** mtanino has joined #openstack-nova14:27
*** mdrabe has joined #openstack-nova14:29
openstackgerritPaul Carlton proposed openstack/nova-specs: Query and cancel ongoing live migrations  https://review.openstack.org/22882814:30
*** gszasz has quit IRC14:31
*** su_zhang_ has joined #openstack-nova14:33
*** ntpttr has joined #openstack-nova14:33
*** ntpttr has left #openstack-nova14:35
openstackgerritvenkatesh proposed openstack/nova: make the driver.Scheduler as abstract class  https://review.openstack.org/21201614:36
*** nelsnelson has quit IRC14:36
*** nelsnelson has joined #openstack-nova14:36
*** digimonde has quit IRC14:37
openstackgerritvenkatesh proposed openstack/nova: make the driver.Scheduler as abstract class  https://review.openstack.org/21201614:37
openstackgerritChuck Carmack proposed openstack/nova-specs: Allow user to set and retrieve the server Description  https://review.openstack.org/22890814:40
*** abhishekk has left #openstack-nova14:40
*** nelsnelson has quit IRC14:40
*** stevemar has quit IRC14:41
*** stevemar has joined #openstack-nova14:42
dulekHi, I'm trying to play with online schema migrations. Currently nova-manage db contract fails with "CRITICAL nova [-] DBError: expected string or buffer"14:42
dulekWhat's the current status of that feature?14:42
*** jsuchome has quit IRC14:43
openstackgerritChuck Carmack proposed openstack/nova-specs: Allow user to set and retrieve the server Description  https://review.openstack.org/22890814:43
*** Daisy has quit IRC14:44
*** Daisy has joined #openstack-nova14:45
dansmithdulek: extremely experimental at the moment, and specifically contract being known to destroy data14:45
dulekdansmith: Well, right now it doesn't do anything, as it fails to generate SQL.14:46
*** stevemar has quit IRC14:46
dansmithdulek: that's probably the best case scenario :)14:46
*** stevemar has joined #openstack-nova14:46
dulekdansmith: :D Is there any etherpad or other resource explaining the issues and way forward? I'm trying to mimic that approach into Cinder.14:47
dansmithdulek: at this point I don't think you should try to do that (seriously)14:47
*** tjones has joined #openstack-nova14:47
dansmithdulek: you can do online schema migrations without that specific bit of functionality14:48
dansmithdulek: we have been doing them in nova with normal sqlalchemy-migrate for a bit now14:48
dansmithdulek: it requires dilligence14:48
dulekdansmith: So what you propose is just to prevent contracting migrations at review level?14:49
dansmithdulek: yeah, and with some help in the tests to try to catch things14:49
openstackgerritSergey Nikitin proposed openstack/nova: Switch to oslo.cache lib  https://review.openstack.org/20304914:49
*** stevemar_ has joined #openstack-nova14:49
*** Marga_ has joined #openstack-nova14:50
dansmithdulek: I think we're going to at least have a session on this in the nova track in tokyo to discuss what has been successful and what needs more work14:50
*** yassine has joined #openstack-nova14:50
dansmithdulek: if you're going to be there, that would be worth attending I think14:50
dulekdansmith: I'll definitely attend.14:51
dulekdansmith: Any pointers to such tests in Nova's tree?14:51
*** stevemar has quit IRC14:51
openstackgerritSergey Nikitin proposed openstack/nova: Switch to oslo.cache lib  https://review.openstack.org/20304914:51
dansmithdulek: yeah, just a sec14:51
dansmithdulek: https://github.com/openstack/nova/blob/master/nova/tests/unit/db/test_migrations.py#L186-L22714:52
dansmithdulek: that can't catch everything, but actually seems to work pretty well14:53
*** Sree has quit IRC14:53
dansmithdulek: the fixture mocks out some operations like alter, which causes the test to explode14:53
*** Sree has joined #openstack-nova14:53
dulekdansmith: I'll take a look into it, thanks.14:54
dansmithdulek: cool14:54
*** Marga_ has quit IRC14:54
*** stevemar has joined #openstack-nova14:55
openstackgerritRaildo Mascena de Sousa Filho proposed openstack/nova-specs: Moving Nested Quota Driver to Mitaka  https://review.openstack.org/20996914:55
*** tonytan4ever has joined #openstack-nova14:56
*** takedakn has joined #openstack-nova14:57
*** Sree has quit IRC14:57
*** stevemar_ has quit IRC14:58
*** diogogmt has joined #openstack-nova14:58
*** tsekiyama has joined #openstack-nova14:58
*** signed8bit is now known as signed8bit_ZZZzz14:58
*** jlanoux has quit IRC14:58
*** haomaiwang has quit IRC14:59
*** jaypipes has quit IRC14:59
*** haomaiwang has joined #openstack-nova15:00
*** signed8bit_ZZZzz is now known as signed8bit15:00
*** sayali has quit IRC15:00
*** haomaiwang has quit IRC15:01
*** haomaiwa_ has joined #openstack-nova15:01
*** EinstCrazy has joined #openstack-nova15:01
*** sayali has joined #openstack-nova15:02
openstackgerritChuck Carmack proposed openstack/nova-specs: Allow user to set and retrieve the server Description  https://review.openstack.org/22890815:02
sdaguemriedem: ok, novaclient releases15:02
sdaguebecause liberty -> master upgrade testing is blocked on that right now15:03
mriedemsdague: yeah i'm finding the tag in stable/liberty now15:03
sdaguegreat15:03
mriedemupper-constraints is 2.3015:04
mriedemand i think that's what stable/liberty was cut from15:04
*** takedakn has quit IRC15:04
*** diogogmt_ has joined #openstack-nova15:04
*** diogogmt has quit IRC15:04
*** diogogmt_ is now known as diogogmt15:04
mriedemyup, so it's going to be 2.30.115:04
*** schwicke has quit IRC15:04
mriedemand we'll have to update upper-constraints15:04
mriedemupper-constraints should really be 2.31.0 in stable/liberty -gr15:04
*** haomaiwa_ has quit IRC15:04
mriedem*g-r15:04
mriedemso we have room to fix 2.30.x15:05
*** wolsen has joined #openstack-nova15:05
sdaguesure, you want to take this to release-mgr office and sort with ttx15:05
mriedemyeah15:05
*** schwicke has joined #openstack-nova15:06
openstackgerritPaul Carlton proposed openstack/nova-specs: Query and cancel ongoing live migrations  https://review.openstack.org/22882815:07
*** agireud has joined #openstack-nova15:08
openstackgerritChuck Carmack proposed openstack/nova-specs: Allow user to set and retrieve the server Description  https://review.openstack.org/22890815:09
*** dave-mccowan has quit IRC15:09
*** Alexus has joined #openstack-nova15:10
*** dsanders has joined #openstack-nova15:12
*** Marga_ has joined #openstack-nova15:14
*** dsanders has quit IRC15:14
garykjohnthetubaguy: mriedem: what do you guys suggest about the opaque networks? https://review.openstack.org/#/c/165750/. I posted as comment. the pain point for us is that liberty driver will not work with nova and everyone will need to patch for that - similarly to what people need to do with kilo and the NSxv driver.15:14
*** dsanders has joined #openstack-nova15:14
*** tonytan4ever has quit IRC15:14
johnthetubaguygaryk: I think a very small spec would really help everyone understand whats going on with that15:15
garykjohnthetubaguy: to post to the nova spec?15:15
johnthetubaguygaryk: even if its just noting the support matrix between different versions, like this only works with neutron plugin X after version Y, etc15:16
garykok, makes sense. i'll post15:16
johnthetubaguygaryk: its probably just two paragraphs of context, mostly because we are all a bit confused about it and understanding why its important15:17
johnthetubaguygaryk: thanks15:17
mriedemyeah i'd like the compat details spelled out a bit15:17
mriedemlike the upgrade impact15:17
openstackgerritMark Doffman proposed openstack/nova: libvirt: Don't attempt to convert initrd images.  https://review.openstack.org/22528915:18
garykmriedem: ok, will do15:18
johnthetubaguymriedem: yeah, I think thats the bit I am confused by right now15:18
mriedemjohnthetubaguy: here is the novaclient 2.30.1 release request for liberty https://review.openstack.org/#/c/229952/15:18
*** jerrygb has joined #openstack-nova15:19
*** dsanders1 has joined #openstack-nova15:19
*** jerrygb has quit IRC15:19
*** jerrygb has joined #openstack-nova15:19
johnthetubaguymriedem: thanks, that looks good15:20
johnthetubaguymriedem: should we release from master as well, I guess not, because that would just confuse everything?15:20
mriedemjohnthetubaguy: i'm doing it now15:20
mriedem2.3115:20
*** nkrinner has quit IRC15:21
*** jdurgin1 has joined #openstack-nova15:22
*** MVenesio has quit IRC15:22
*** dsanders has quit IRC15:22
*** MVenesio has joined #openstack-nova15:23
*** dave-mccowan has joined #openstack-nova15:23
*** danpb has quit IRC15:24
johnthetubaguymriedem: ah, cool15:24
*** danpb has joined #openstack-nova15:24
*** tonytan4ever has joined #openstack-nova15:27
*** scheuran has quit IRC15:27
*** wznoinsk has joined #openstack-nova15:27
*** cfriesen has joined #openstack-nova15:27
*** signed8bit is now known as signed8bit_ZZZzz15:30
*** jaypipes has joined #openstack-nova15:30
*** dims has joined #openstack-nova15:30
* wznoinsk pings jaypipes, johnthetubaguy15:30
johnthetubaguyhello15:31
bauzasmriedem: johnthetubaguy: I was wondering, how stable is managed for novaclient ?15:32
bauzasmriedem: johnthetubaguy: we're doing .z versions ?15:32
wznoinskhi johnthetubaguy , I was hoping to catch you and jaypipes on the nova meeting today but I had a clash and couldn't attent...15:33
mriedembauzas: yeah15:33
wznoinskin re to http://lists.openstack.org/pipermail/openstack-dev/2015-September/075626.html, I basically need Jay's and yours (or some other Core(s)) opinion(s) and green light eventually15:34
bauzasmriedem: okay I see15:34
bauzasmriedem: so g-r is still capping correctly15:34
openstackgerritDiana Clarke proposed openstack/nova: Replace N block_device_mapping queries with 1  https://review.openstack.org/22996415:34
mriedembauzas: yeah, upper-constraints15:35
bauzasmriedem: gotcha15:35
mriedemwhich is this https://review.openstack.org/#/c/229956/15:35
*** betherly has joined #openstack-nova15:35
*** jlanoux has joined #openstack-nova15:35
*** EinstCrazy has quit IRC15:36
*** artom has joined #openstack-nova15:36
_diana_dansmith: I'd like to take you up on that offer when you have some time. Re: versioned objects. I changed my first versioned object here (https://review.openstack.org/229964), but I'm not sure I did everything correctly.15:37
*** angdraug has joined #openstack-nova15:38
*** setuid has quit IRC15:38
dansmith_diana_: okay looking15:38
_diana_dansmith: For example, I didn't change obj_relationships in the object class I changed. The tests did pass though.15:38
_diana_dansmith: Thanks!15:38
dansmith_diana_: yep, this is going to be a fairly easy one I think15:39
*** signed8bit_ZZZzz is now known as signed8bit15:39
*** setuid has joined #openstack-nova15:39
*** schwicke has quit IRC15:41
bauzasmriedem: okay, thanks for the links, got it, thanks !15:41
*** Daisy has quit IRC15:42
*** Daisy has joined #openstack-nova15:42
dansmith_diana_: left a couple comments, but all the functional stuff looks fine15:42
dansmith_diana_: er, I mean, all the object functional stuff looks fine15:43
dansmithI didn't review the rest of the changes, but the version bump and hash change looks right15:43
dansmith_diana_: I would wait for the partial-ncpu job to run before fixing my nits and then if so, push up another15:43
dansmiththe partial job is important for object-related changes15:43
*** irenab has joined #openstack-nova15:44
_diana_dansmith: ah, thanks for the tip15:44
*** vilobhmm11 has joined #openstack-nova15:44
*** dsanders1 has quit IRC15:44
*** Marga_ has quit IRC15:44
*** Marga_ has joined #openstack-nova15:45
*** mgoddard__ has joined #openstack-nova15:45
*** sahid has quit IRC15:46
*** Daisy has quit IRC15:47
*** tonytan4ever has quit IRC15:47
*** mgoddard_ has quit IRC15:48
*** romainh has left #openstack-nova15:48
bauzas_diana_: dansmith: I need to go pick my kid, but I left a comment15:49
bauzasI was wondering whether the obj_relationship needed some amendment15:50
bauzasgiven dansmith's homework15:50
*** Daisy has joined #openstack-nova15:50
* bauzas afk for a bit15:50
dansmithI don't think anything else has a BlockDeviceList in it, so no relationship change15:50
*** sajeesh has quit IRC15:52
*** ociuhandu has joined #openstack-nova15:52
mriedemtime to tackle the api version mess in novaclient15:52
*** e0ne has quit IRC15:53
*** vilobhmm111 has joined #openstack-nova15:56
*** vilobhmm11 has quit IRC15:58
*** MVenesio has quit IRC15:58
*** alejandrito has quit IRC15:58
mriedemsdague: so the only microversions i see explicitly handled in novaclient today are 2.2 (keypair stuff), 2.4 (new attr in fixed-ip response), and 2.11 (forced-down on os-services)16:00
mriedem2.3 is just returning extra stuff in the server details, so not a breaking change16:00
mriedemhttps://github.com/openstack/nova/blob/master/nova/api/openstack/rest_api_version_history.rst16:00
sdagueright, most of them aren't breaking changes16:00
mriedem2.5 allows filtering on ipv616:01
mriedemwhich shouldn't affect the client i guess, it either works or it doesn't16:01
mriedemso 2.6 is the first thing that breaks the client it looks like16:01
sdaguethe client already passed it, that's why they wanted the change16:01
mriedemyeah16:01
sdagueyes, I think 2.6 is the only problem case16:02
*** mgoddard has joined #openstack-nova16:02
mriedemok, so i was going to make API_MAX_VERSION=2.616:02
sdaguethe other new attributes should be displayed if they come back16:02
mriedem*2.516:02
*** su_zhang_ has quit IRC16:02
sdaguebut that could be put in whenever16:02
mriedemyeah16:02
mriedemi can open a bug for it16:02
sdaguemax to 2.5 seems right16:03
*** mgoddard__ has quit IRC16:03
mriedemok, so sorry to anyone that needed 2.11 in the client16:03
sdaguethen we need tests for the vnc calls16:03
mriedemand i was going to change the default CLI to be the same as API_MAX_VERSION16:03
sdaguethat was the ha service group folks16:03
mriedemas in DEFAULT_OS_COMPUTE_API_VERSION16:03
mriedemDEFAULT_OS_COMPUTE_API_VERSION=API_MAX_VERSION16:04
mriedemyou update API_MAX_VERSION when you add support for it16:04
sdagueyeh, I think that's fine16:04
mriedemand those have to be incremental16:04
mriedemi also plan to add docs around this process in the client later once we get the initial change out and backported16:04
*** MVenesio has joined #openstack-nova16:05
*** signed8bit is now known as signed8bit_ZZZzz16:05
*** alejandrito has joined #openstack-nova16:05
*** lmtaylor has joined #openstack-nova16:05
_diana_dansmith: I'm going to push those nits after all, so that I can go get lunch. But I'll keep the tip in mind for next time.16:06
dansmithalright16:06
openstackgerritDiana Clarke proposed openstack/nova: Replace N block_device_mapping queries with 1  https://review.openstack.org/22996416:06
*** ildikov has quit IRC16:09
*** unicell has quit IRC16:09
*** markus_z has quit IRC16:09
*** tonytan4ever has joined #openstack-nova16:10
*** mudassirlatif has joined #openstack-nova16:10
*** alex_klimov has quit IRC16:11
*** Daisy has quit IRC16:11
*** Daisy has joined #openstack-nova16:12
*** jlanoux has quit IRC16:12
*** jlanoux has joined #openstack-nova16:13
*** r-daneel has joined #openstack-nova16:13
*** apoorvad has joined #openstack-nova16:13
*** apoorvad has quit IRC16:14
*** haypo has left #openstack-nova16:14
*** signed8bit_ZZZzz is now known as signed8bit16:14
*** breitz has quit IRC16:14
*** breitz has joined #openstack-nova16:14
*** Daisy has quit IRC16:16
jaypipeswznoinsk: I'll respond on the openstack-dev ML post, OK?16:17
*** lmtaylor has left #openstack-nova16:17
*** jdurgin1 has quit IRC16:17
*** salv-orlando has joined #openstack-nova16:17
*** jaypipes has quit IRC16:18
*** salv-orlando has quit IRC16:20
*** mriedem is now known as mriedem_lunch16:22
*** Daisy has joined #openstack-nova16:23
*** stevemar_ has joined #openstack-nova16:25
*** matrohon has quit IRC16:26
*** karimb has quit IRC16:26
johnthetubaguywznoinsk: I think its OK to start commenting, and see how it goes before we turn on voting, but I should reply on the ML16:26
*** vipul has quit IRC16:27
*** yamahata has joined #openstack-nova16:27
openstackgerritPaul Carlton proposed openstack/nova-specs: Query and cancel ongoing live migrations  https://review.openstack.org/22882816:27
*** stevemar has quit IRC16:27
*** yassine has quit IRC16:27
*** rdopiera has quit IRC16:27
wznoinskjaypipes, johnthetubaguy thanks guys, let's follow up on the ML then16:28
*** vipul has joined #openstack-nova16:28
johnthetubaguymriedem_lunch: sdague: if the max version is a specific version, and not latest, we probably need to negotiate the version for each call, I guess, but that seems OK16:28
*** jistr has quit IRC16:29
sdaguejohnthetubaguy: we already do that16:29
*** xgsa has quit IRC16:29
*** mriedem_lunch is now known as mriedem16:30
*** rotbeard has joined #openstack-nova16:30
*** ameade has quit IRC16:31
mriedemso on the way back from the cafeteria i was thinking, API_MAX_VERSION should probably be latest supported (implemented) version in the client, which is 2.11 - b/c you can opt into that16:31
mriedemDEFAULT_OS_COMPUTE_API_VERSION should be 2.516:31
mriedemsince we don't have 2.6 support, so we shouldn't default to >=2.616:31
mriedemam i thinking through that correctly?16:32
*** ameade has joined #openstack-nova16:33
*** ndipanov has quit IRC16:34
*** danpb has quit IRC16:34
*** jlanoux has quit IRC16:34
*** eglynn has quit IRC16:36
*** ZZelle_ has joined #openstack-nova16:37
*** erhudy has quit IRC16:39
openstackgerritBrian Elliott proposed openstack/nova: Remove unused read-only cell code  https://review.openstack.org/23001216:39
*** salv-orlando has joined #openstack-nova16:39
*** salv-orlando has quit IRC16:39
*** salv-orlando has joined #openstack-nova16:40
*** nic has joined #openstack-nova16:41
tdurakovjohnthetubaguy, short question about my patch for l-m rollback, what decorators do you think for separate methods?16:44
*** apoorvad has joined #openstack-nova16:45
*** apoorvad has quit IRC16:45
*** tonytan4ever has quit IRC16:45
*** apoorvad has joined #openstack-nova16:45
*** signed8bit is now known as signed8bit_ZZZzz16:49
*** browne has joined #openstack-nova16:51
*** signed8bit_ZZZzz is now known as signed8bit16:52
*** erlon has joined #openstack-nova16:53
*** e0ne has joined #openstack-nova16:54
*** e0ne has quit IRC16:55
*** markvoelker has quit IRC16:55
*** markvoelker has joined #openstack-nova16:56
*** dsanders has joined #openstack-nova16:56
mriedemccarmack: here is a thing if you're interested https://bugs.launchpad.net/python-novaclient/+bug/150183616:56
openstackLaunchpad bug 1501836 in python-novaclient "microversion 2.3 is not implemented in novaclient" [Low,Confirmed]16:56
*** unicell has joined #openstack-nova16:56
ccarmackmriedem: ok, I'll take a look16:57
*** baoli has quit IRC17:00
*** derekh has quit IRC17:01
cfriesenis there a way to check whether debug logs are enabled?    (If they're not, I want to skip a bunch of work to gather debug info.)17:01
*** baoli has joined #openstack-nova17:01
*** mriedem has quit IRC17:02
*** dims_ has joined #openstack-nova17:03
*** ctrath has quit IRC17:04
*** jerrygb has quit IRC17:04
*** dims has quit IRC17:04
*** Piet has quit IRC17:06
*** ildikov has joined #openstack-nova17:06
*** mriedem has joined #openstack-nova17:07
*** Marga_ has quit IRC17:07
*** Marga_ has joined #openstack-nova17:08
*** ctrath has joined #openstack-nova17:11
*** tsekiyama has quit IRC17:11
*** Daisy_ has joined #openstack-nova17:11
*** su_zhang_ has joined #openstack-nova17:12
*** Marga_ has quit IRC17:13
*** Daisy has quit IRC17:14
openstackgerritMatt Riedemann proposed openstack/python-novaclient: Set DEFAULT_OS_COMPUTE_API_VERSION to 2.5  https://review.openstack.org/23002417:17
mriedemsdague: johnthetubaguy: bauzas: melwitt: alaski: ^ WIP, still need to run functional tests17:17
*** Daisy_ has quit IRC17:17
*** stevemar_ has quit IRC17:18
*** lpetrut has quit IRC17:21
*** Alexus has quit IRC17:21
*** VW_ has joined #openstack-nova17:29
*** su_zhan__ has joined #openstack-nova17:31
*** VW has quit IRC17:33
*** su_zhan__ has quit IRC17:33
*** VW_ has quit IRC17:33
*** VW has joined #openstack-nova17:33
*** su_zhan__ has joined #openstack-nova17:33
*** su_zhang_ has quit IRC17:34
*** pm90__ has quit IRC17:35
*** signed8bit is now known as signed8bit_ZZZzz17:36
*** alejandrito has quit IRC17:36
*** ociuhandu has quit IRC17:36
*** Marga_ has joined #openstack-nova17:36
*** alejandrito has joined #openstack-nova17:38
*** signed8bit_ZZZzz is now known as signed8bit17:41
mriedemmelwitt: dansmith: do you know if the db archive stuff is currently busted? or just has known issues?17:42
mriedemwhen zzzeek posted about that in the ML i thought we reverted a change that broke it17:42
* dansmith shrugs17:42
mriedemok, b/c mfisch is asking me about the db purge command proposed for mitaka in the ops channel17:43
dansmithI thought it was just more that the archived copies were not always guaranteed to be clean17:43
dansmithlike the archive could be corrupt17:43
mriedemso you can't restore them?17:43
dansmithI dunno if we were ever expecting them to be restorable, but I was thinking more like: uniqueness and FKs aren't respected do you may archive an instance and it suddenly gains a bunch of resources through linkage17:44
dansmithbut I really don't know17:44
mriedemmfisch: btw, the purge command proposed is nothing like the archive stuff17:44
mriedemoh yeah, we don't have unique and fkey constraints in the shadow tables, i know that17:45
mfischso maybe it wont break forever then17:45
mfischthis is the first issue I found17:45
mfischhttp://lists.openstack.org/pipermail/openstack-dev/2014-December/17:45
mfischto which you responded17:45
*** alejandrito has quit IRC17:46
*** rlrossit has quit IRC17:46
*** alejandrito has joined #openstack-nova17:47
mriedem"mfisch: mriedem: I'd love to have delete tools for nova and cinder and glance too, I just dont think the projects will maintain them. We're not big enough to write our own tooling for this stuff"17:49
mriedemmfisch: if you're not big enough to write your own tooling, wouldn't you prefer that it's written in tree at least?17:49
mriedemthis is why we have backlog specs17:49
mriedemoperator says 'i'd really like to see x but i'm not going ot implement it'17:49
mfischyes I'd prefer it in tree for sure17:51
*** VW has quit IRC17:52
openstackgerritVilobh Meshram proposed openstack/nova-specs: Servicegroup foundational refactoring for Control Plane  https://review.openstack.org/22242317:52
*** VW has joined #openstack-nova17:52
openstackgerritVilobh Meshram proposed openstack/nova-specs: Servicegroup foundational refactoring for Control Plane  https://review.openstack.org/22242317:54
*** ccarmack has quit IRC17:55
vilobhmm111dansmith : ping17:55
*** tonytan4ever has joined #openstack-nova17:56
vilobhmm111dansmith : i have updated the spec as we discussed in nova weekly meeting…please let me know if that looks ok to you https://review.openstack.org/22242317:56
*** jecarey_ has joined #openstack-nova17:57
dansmithvilobhmm111: okay17:57
*** rlrossit has joined #openstack-nova17:58
vilobhmm111dansmith : thanks!17:58
*** ccarmack has joined #openstack-nova17:58
*** ociuhandu has joined #openstack-nova17:58
*** jecarey has quit IRC18:00
mriedemdoffm: 2.6 bug for client 150068818:00
mriedembug 150068818:00
openstackbug 1500688 in python-novaclient "VNC URL of instance unavailable in CLI" [High,In progress] https://launchpad.net/bugs/1500688 - Assigned to Matt Riedemann (mriedem)18:00
rlrossitctrath: http://stackalytics.com/report/contribution/nova/3018:05
*** Piet has joined #openstack-nova18:05
*** smatzek_ has joined #openstack-nova18:06
*** smatzek_ has quit IRC18:06
*** su_zhan__ is now known as su_zhang_18:08
*** nelsnelson has joined #openstack-nova18:08
*** READ10 has quit IRC18:09
*** smatzek has quit IRC18:09
*** browne has quit IRC18:10
*** tsekiyama has joined #openstack-nova18:10
*** amotoki has quit IRC18:10
*** zzzeek has quit IRC18:15
*** su_zhang_ has quit IRC18:15
*** zzzeek has joined #openstack-nova18:15
*** su_zhang_ has joined #openstack-nova18:16
*** tdurakov has quit IRC18:18
*** tdurakov has joined #openstack-nova18:20
*** su_zhang_ has quit IRC18:20
raildojohnthetubaguy, hey :) I fixed the nested quota spec with your comments. https://review.openstack.org/#/c/209969/  Is there anything more that I can do about it?18:20
*** su_zhang_ has joined #openstack-nova18:21
raildomelwitt, ping, if you have a couple minutes to see this too, I'll appreciate that ^ :)18:21
*** nelsnelson has quit IRC18:21
*** nelsnelson has joined #openstack-nova18:25
*** penick has joined #openstack-nova18:25
*** aix has quit IRC18:26
*** marzif has joined #openstack-nova18:27
ijuwangdoffm: Hi.. regarding to the bug I just submitted (https://bugs.launchpad.net/nova/+bug/1501831), I do have that fix.. Let me know if I misunderstand or there is something wrong with my setup18:28
openstackLaunchpad bug 1501831 in OpenStack Compute (nova) "Evacuate libvirt instance failed with error 'Cannot load 'disk_format' in the base class'" [Undecided,Fix committed]18:28
*** tsekiyama has quit IRC18:28
*** su_zhang_ has quit IRC18:30
*** su_zhang_ has joined #openstack-nova18:30
bauzasmriedem: reviewing https://review.openstack.org/#/c/230024/118:31
*** openstackgerrit has quit IRC18:31
bauzasmriedem: so IIUC, until we provide a compatible method for 2.6, that's why the client is only capping to 2.5 ?18:31
*** angdraug has quit IRC18:32
*** openstackgerrit has joined #openstack-nova18:32
bauzasmriedem: so another patch would then add the decorator for providing a 2.6 method and bump the client max version to 2.6 ?18:32
bauzasetc. for up 2.11 ?18:32
mriedembauzas: yup18:32
bauzasif so, that's oka18:32
bauzasokay18:32
mriedemi think doffm was going to work on adding 2.6 support to the client18:33
melwittmriedem: I thought db archive theoretically should have been fixed by https://review.openstack.org/#/c/109201/ and https://review.openstack.org/#/c/110875/ but I haven't heard of anyone trying it to confirm that18:33
bauzasokay, that's a low-hanging-fruit18:33
mriedemwhen he finishes rejecting all of ijuwang's bugs :P18:33
*** tjones has quit IRC18:33
ijuwang:-(18:33
mriedemjoking18:33
mriedemmelwitt: maybe it is18:33
ijuwangi know :-)18:34
mriedemmfisch: which version of nova are you running when you last tried archiving the db?18:34
*** dave-mccowan has quit IRC18:35
melwittmriedem: before those patches, it was definitely busted for the instance records -- they could not be hard-deleted because system_metadata was undeletable18:35
*** rotbeard has quit IRC18:35
bauzasdoffm: mriedem: okay, cool to see https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/servers.py#L664-L673 modified by adding a new deco18:36
doffmijuwang: Ok, sorry about that. I will take another look at the bug, see if I can repeat.18:38
ijuwangdoffm: sure..18:38
*** obedmr_ has quit IRC18:39
*** tdurakov has quit IRC18:39
*** tonytan4ever has quit IRC18:40
*** pixelbeat has quit IRC18:41
*** tdurakov has joined #openstack-nova18:42
*** tsekiyama has joined #openstack-nova18:45
*** marzif has quit IRC18:45
*** tjones has joined #openstack-nova18:46
openstackgerritTimofey Durakov proposed openstack/nova: Removed unused method from compute/rpcapi  https://review.openstack.org/23007218:48
openstackgerritDiana Clarke proposed openstack/nova: Replace N block_device_mapping queries with 1  https://review.openstack.org/22996418:49
*** tonytan4ever has joined #openstack-nova18:50
*** dave-mccowan has joined #openstack-nova18:51
*** su_zhang_ has quit IRC18:53
*** ZZelle_ has quit IRC18:53
*** angdraug has joined #openstack-nova18:53
*** signed8bit is now known as signed8bit_ZZZzz18:53
*** electrocucaracha has quit IRC18:54
*** electrocucaracha has joined #openstack-nova18:55
*** browne has joined #openstack-nova18:56
*** artom has quit IRC18:58
*** thumpba has quit IRC18:58
*** thumpba_ has joined #openstack-nova18:58
*** signed8bit_ZZZzz is now known as signed8bit18:59
*** electrocucaracha has quit IRC18:59
*** electrocucaracha has joined #openstack-nova19:04
*** tjones has quit IRC19:04
*** stevemar has joined #openstack-nova19:04
*** obedmr has joined #openstack-nova19:07
*** jichen has joined #openstack-nova19:07
*** ociuhandu has quit IRC19:09
*** ctrath has quit IRC19:11
*** ZZelle_ has joined #openstack-nova19:11
*** SourabhP has joined #openstack-nova19:14
bauzasmriedem: tbc, if I'm importing novaclient and use Client() it will support until 2.11, but if I'm using the shell CLI, it will only support 2.5, right .19:16
bauzas?19:16
*** ctrath has joined #openstack-nova19:16
mriedembauzas: the CLI will default to 2.5 but you can override that with the --os-compute-api-version option or whatever it's called19:17
mriedemthat's why max is still 2.1119:17
mriedemyou have to opt into anything over 2.519:17
*** tsekiyama has quit IRC19:18
*** david-lyle has quit IRC19:18
*** jwcroppe has quit IRC19:18
bauzasmriedem: okay, that's why I understood19:21
bauzaswhat*19:21
bauzas(still thinking on the coverage)19:21
*** mgoddard has quit IRC19:25
*** mgoddard has joined #openstack-nova19:25
*** david-lyle has joined #openstack-nova19:27
*** matrohon has joined #openstack-nova19:28
*** ssurana has joined #openstack-nova19:30
*** mgoddard has quit IRC19:32
*** vilobhmm111 has quit IRC19:35
*** vilobhmm11 has joined #openstack-nova19:35
*** mgoddard has joined #openstack-nova19:37
*** thumpba_ has quit IRC19:37
*** jwcroppe has joined #openstack-nova19:43
*** sambetts has quit IRC19:47
*** logan2 has quit IRC19:47
*** sambetts has joined #openstack-nova19:48
*** pbrooko has joined #openstack-nova19:49
openstackgerritMatt Riedemann proposed openstack/nova: Add checkpoint logging when building an instance in compute manager  https://review.openstack.org/22898019:49
openstackgerritMatt Riedemann proposed openstack/nova: Add checkpoint logging when terminating an instance  https://review.openstack.org/22949019:49
*** claudiub has quit IRC19:50
mriedemharlowja: am i using the StopWatch.restart stuff here correctly? https://review.openstack.org/#/c/229490/3/nova/compute/manager.py19:50
mriedemharlowja: as in i don't need to explicitly call stop() in here19:51
*** su_zhang_ has joined #openstack-nova19:51
harlowjarighty19:51
harlowjalooks good to me19:51
*** mgoddard has quit IRC19:52
alaskimriedem: you decided to go to the dark side?19:54
*** ociuhandu has joined #openstack-nova19:54
mriedemalaski: after i found out i could use it as a context manager19:55
mriedemand harlowja put up examples in the docs19:55
openstackgerritMatt Riedemann proposed openstack/nova: Add checkpoint logging when terminating an instance  https://review.openstack.org/22949019:55
*** pm90_ has joined #openstack-nova19:55
harlowjadark side being the common libraries that everyone uses :-/19:55
mriedemi consider it the shady side19:55
alaskicool.  I was going to comment that you should pull your implementation into a separate patch19:55
*** logan2 has joined #openstack-nova19:55
*** pbrooko has quit IRC19:56
*** pm90__ has joined #openstack-nova19:56
alaskiharlowja: just joking.  it didn't seem like the right fit at first based on the usage pattern we wanted19:56
harlowjahmmm, was that just cause it didn't have a visible __enter__ and __exit__ ? just wondering19:56
harlowjaguess i should make the docs show those magic methods19:56
*** mgoddard has joined #openstack-nova19:56
harlowjavs not show them19:56
*** pixelbeat has joined #openstack-nova19:58
alaskithey're just kind of hidden in the middle of other methods19:58
*** pbrooko has joined #openstack-nova19:58
alaskiin the code19:59
*** alex_klimov has joined #openstack-nova19:59
harlowjak19:59
harlowjaso guess that means docs need to expose them19:59
harlowjaunsure what other way to expose them19:59
*** pm90_ has quit IRC20:00
alaskiI missed them on a scan of methods.  all I could recommend would be to have them at the top/bottom of the methods, or called out in the class docstring20:00
*** tjones has joined #openstack-nova20:01
harlowjahttps://review.openstack.org/#/c/230113/ i guess20:01
harlowjasphinx seems to place them at the top of the generated docs (from a local build of that)20:01
*** mgoddard has quit IRC20:02
harlowjaso that solves that problem20:02
*** pixelbeat has quit IRC20:02
*** mgoddard has joined #openstack-nova20:02
alaskiI meant in the code.  I didn't look at the docs20:02
*** nelsnelson has quit IRC20:02
*** signed8bit has quit IRC20:03
*** pbrooko has quit IRC20:03
harlowjak20:03
harlowjawell ummm20:04
harlowjahmmm20:04
harlowjareadthedocs ;)20:04
*** ociuhandu has quit IRC20:04
*** tdurakov has quit IRC20:04
*** tdurakov has joined #openstack-nova20:05
alaskiI will admit that I have no idea where the docs are hosted20:05
mriedemharlowja: the docs are good now20:05
harlowjawoot20:05
mriedemyou have the examples with the context manager20:05
mriedemalaski: the docs showed up in the last 24 hours20:06
mriedemso don't feel bad20:06
harlowjahttp://docs.openstack.org/developer/openstack-projects.html20:06
harlowja'Using Libraries ' in ^20:06
*** ociuhandu has joined #openstack-nova20:06
openstackgerritCale Rath proposed openstack/nova: libvirt live-migration: Monitoring task tracks progress watermark incorrectly  https://review.openstack.org/22750120:06
mriedemharlowja: no, this http://docs.openstack.org/developer/oslo.utils/examples/timeutils.html20:06
harlowjaah20:06
harlowjak20:06
mriedemthat is new as of last 24 hours20:06
harlowjayes, thats new20:06
mriedemand is what i needed20:06
harlowjak20:06
harlowjacool20:06
*** pbrooko has joined #openstack-nova20:07
*** thumpba has joined #openstack-nova20:08
*** mgoddard has quit IRC20:08
*** pm90__ has quit IRC20:08
*** thumpba has quit IRC20:08
*** thumpba has joined #openstack-nova20:09
*** thumpba has quit IRC20:11
*** mgoddard has joined #openstack-nova20:11
*** pm90_ has joined #openstack-nova20:12
*** pbrooko has quit IRC20:12
mriedem_diana_: i knew i'd seen that same change before... https://review.openstack.org/#/c/211258/20:14
openstackgerritCale Rath proposed openstack/nova: libvirt live-migration: Monitoring task tracks progress watermark incorrectly  https://review.openstack.org/22750120:14
mriedemwe should probably just close this bug https://bugs.launchpad.net/nova/+bug/118095020:15
openstackLaunchpad bug 1180950 in OpenStack Compute (nova) "Openstack operators need to be able to move instances between tenants without modifying the database directly" [Wishlist,Confirmed] - Assigned to Chung Chih, Hung (lyanchih)20:15
mriedempenick: ^20:15
mriedemthere was actually a spec for that at one point20:15
mriedemi'll try to find it20:15
_diana_mriedem: fun! it's not often I get to read split-tested patches. I need to pick up my kid from school, but I'll address your comments (+ dansmith's) and read the other patch in about 30min.20:16
alaskimriedem: we have some interest in that as well.  but I think it needs to be a cross project spec/effort20:16
mriedemwell the other is abandoned but ok20:16
mriedemcern also wants it20:16
mriedemeveryone wants it20:16
*** pm90__ has joined #openstack-nova20:17
dansmithconverting N calls to 1 is always good, unless N is 020:17
*** mgoddard has quit IRC20:17
mriedemdansmith: no, everyone wants to be able to move instances between tenants20:17
penickmriedem: yeah, we definitely need that20:18
dansmithoh, heh, that20:18
dansmithI'm -2 on that20:18
dansmithmostly because penick wants it20:18
penickAw :(20:18
mriedemi can't find the spec20:18
* penick is crushed20:18
mriedemchaochin wrote the spec20:18
penickdan deleted it, to mess with me20:18
*** tsekiyama has joined #openstack-nova20:18
*** thumpba has joined #openstack-nova20:19
*** pm90_ has quit IRC20:19
*** thumpba has quit IRC20:20
penickand I used an unnecessary comma to upset his sense of neatness20:20
dansmithoh, dear.20:22
mriedemyar https://review.openstack.org/#/c/105367/20:22
alaskifound https://review.openstack.org/#/c/85806/ too20:23
*** tsekiyama has quit IRC20:23
alaskiunless there's a way to change tenant for glance/cinder/neutron as well it seems like a bad idea20:24
mriedemi'll duplicate that bp20:25
*** mgoddard has joined #openstack-nova20:25
*** thumpba has joined #openstack-nova20:29
*** tdurakov has quit IRC20:29
*** apuimedo has quit IRC20:31
*** thumpba has quit IRC20:32
*** vladikr has quit IRC20:33
*** apuimedo has joined #openstack-nova20:33
*** tpeoples has quit IRC20:34
*** tpeoples has joined #openstack-nova20:35
mriedemmtreinish: just saw that kernel ebtables thing on master20:36
mriedemhttp://logs.openstack.org/62/229962/1/check/gate-tempest-dsvm-full/28bb6b6/logs/screen-n-cpu.txt.gz?level=TRACE#_2015-10-01_17_39_59_18720:36
mriedemsuweet20:36
mtreinishmriedem: heh, I knew I'd seen it before20:36
mtreinishmriedem: did logstash show hits on master when you added taht query?20:36
mriedemit shows up in n-cpu on master, n-net in juno20:38
mriedemso i need to adjust the query20:38
mtreinishah ok20:39
*** breitz has quit IRC20:41
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [WIP] define liberty levels  https://review.openstack.org/23013220:41
dims_sdague: mriedem: the grenade failed later - looks like we need to define liberty levels in master? (WIP - https://review.openstack.org/230132)20:42
dims_sdague: mriedem: http://logs.openstack.org/01/230101/2/check/gate-grenade-dsvm-partial-ncpu/91f0f39/logs/new/screen-n-api.txt.gz?level=ERROR20:42
*** tdurakov has joined #openstack-nova20:44
mriedemdansmith: ^ is there already a change up for that?20:45
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Update rpc version aliases for liberty  https://review.openstack.org/23013220:45
*** markvoelker has quit IRC20:46
*** markvoelker has joined #openstack-nova20:46
*** derekh has joined #openstack-nova20:46
*** tsekiyama has joined #openstack-nova20:46
*** thumpba has joined #openstack-nova20:47
*** markvoelker has quit IRC20:47
*** markvoelker has joined #openstack-nova20:47
openstackgerritDiana Clarke proposed openstack/nova: Replace N block_device_mapping queries with 1  https://review.openstack.org/22996420:49
*** VW has quit IRC20:50
*** VW has joined #openstack-nova20:51
*** eharney has quit IRC20:53
*** ekarlso has quit IRC20:55
*** VW has quit IRC20:55
*** ijw has joined #openstack-nova20:56
*** ekarlso has joined #openstack-nova20:58
*** matrohon has quit IRC20:58
*** tsekiyama has quit IRC20:58
*** mgoddard has quit IRC20:59
*** derekh has quit IRC21:00
*** ctrath has quit IRC21:01
*** pm90_ has joined #openstack-nova21:02
*** pm90__ has quit IRC21:02
*** mudassirlatif_ has joined #openstack-nova21:04
*** nic has quit IRC21:04
*** mudassirlatif has quit IRC21:07
*** mudassirlatif_ is now known as mudassirlatif21:07
*** signed8bit has joined #openstack-nova21:07
*** mgoddard has joined #openstack-nova21:07
*** ccarmack has quit IRC21:08
*** raorn has quit IRC21:08
openstackgerritMark Doffman proposed openstack/python-novaclient: Support the server side remote-console API changes  https://review.openstack.org/23014921:08
*** signed8b_ has joined #openstack-nova21:09
*** raorn has joined #openstack-nova21:10
*** vilobhmm11 has quit IRC21:10
*** vilobhmm11 has joined #openstack-nova21:11
*** ccarmack has joined #openstack-nova21:12
*** mgoddard has quit IRC21:12
*** ctrath has joined #openstack-nova21:12
*** ijw has quit IRC21:13
*** signed8b_ is now known as signed8bit_ZZZzz21:13
*** signed8bit has quit IRC21:13
*** ccarmack has left #openstack-nova21:14
*** tsekiyama has joined #openstack-nova21:14
*** dustins has quit IRC21:15
*** ijw has joined #openstack-nova21:16
*** jichen has quit IRC21:16
*** mgoddard has joined #openstack-nova21:17
openstackgerritMatt Riedemann proposed openstack/python-novaclient: Set DEFAULT_OS_COMPUTE_API_VERSION to 2.5  https://review.openstack.org/23002421:18
*** signed8bit_ZZZzz has quit IRC21:18
*** tdurakov has quit IRC21:18
tjonesand idea why i can do this ./run_tests.sh compute.api  but i cannot do this ./run_tests.sh compute.api --debug?  in the 2nd case i get ImportError: No module named compute21:21
tjonesmust be some change in testtools cause this used to work21:22
*** raildo is now known as raildo-afk21:25
*** thangp has quit IRC21:26
*** jecarey_ has quit IRC21:28
*** Marga_ has quit IRC21:28
*** nic has joined #openstack-nova21:30
*** signed8bit has joined #openstack-nova21:30
*** karimb has joined #openstack-nova21:30
*** Marga_ has joined #openstack-nova21:30
dansmithmriedem: for rpc?21:32
*** mgoddard has quit IRC21:32
*** mgoddard has joined #openstack-nova21:33
mriedemdansmith: yeah21:33
dansmithmriedem: I was literally thinking about this in the shower this morning and then got distracted21:33
dansmithmriedem: I commented, but it's mostly right21:33
mriedemi must make a joke about that somehow21:33
*** e0ne has joined #openstack-nova21:34
*** rfolco has quit IRC21:35
*** annegentle has quit IRC21:35
*** rlrossit has left #openstack-nova21:38
*** john5223 is now known as zz_john522321:38
*** boris-42 has joined #openstack-nova21:38
*** MVenesio has quit IRC21:41
*** tjones has quit IRC21:41
*** salv-orl_ has joined #openstack-nova21:43
*** mriedem has quit IRC21:43
*** david-lyle has quit IRC21:43
*** salv-orlando has quit IRC21:46
*** Marga_ has quit IRC21:46
*** Marga_ has joined #openstack-nova21:47
*** apoorvad has quit IRC21:47
*** annegentle has joined #openstack-nova21:49
*** tjones has joined #openstack-nova21:51
*** ctrath has quit IRC21:52
*** mgoddard has quit IRC21:53
*** annegentle has quit IRC21:56
*** willsama has quit IRC21:56
*** david-lyle has joined #openstack-nova21:56
*** ctrath has joined #openstack-nova21:57
*** nelsnelson has joined #openstack-nova21:57
*** nelsnelson has quit IRC21:58
*** nelsnelson has joined #openstack-nova21:58
*** Marga_ has quit IRC21:59
*** Marga_ has joined #openstack-nova21:59
*** Marga_ has quit IRC22:00
*** Marga_ has joined #openstack-nova22:00
*** foexle has quit IRC22:01
*** mgoddard has joined #openstack-nova22:01
*** sambetts has quit IRC22:04
*** e0ne has quit IRC22:04
jrollis it expected that nova.servicegroup.drivers.db.DbDriver._report_state will stop running periodically if it fails once?22:06
jrollon a compute host, specifically22:06
*** tonytan4ever has quit IRC22:06
*** ijw has quit IRC22:06
*** sambetts has joined #openstack-nova22:06
*** mdrabe has quit IRC22:11
*** pixelbeat has joined #openstack-nova22:12
openstackgerritDan Smith proposed openstack/nova: Remove an unrelated line in test_extended_volumes  https://review.openstack.org/23017722:13
*** ociuhandu has quit IRC22:17
*** mgoddard has quit IRC22:18
_diana_dansmith: in addition to that one line of dead code, I also deleted an additional 30 lines of dead code here too: https://github.com/openstack/nova/blob/b0dd56d14ec23eb042f4f9dbef23ed22ffeaaede/nova/tests/functional/api_sample_tests/test_extended_volumes.py#L50-L7822:19
_diana_dansmith: do you want that removed from the patch too?22:19
*** nelsnelson has quit IRC22:19
_diana_dansmith: that is, it was dead before I got there22:19
dansmith_diana_: you're orphaning that code in your patch right?22:20
dansmithoh, is it?22:20
_diana_dansmith: I think it's really strange to not do this kind of gardening at the same time, but I'll do whatever I'm told.22:20
*** apoorvad has joined #openstack-nova22:21
dansmith_diana_: ah, I misread and thought you were removing that because of the changes below, but I guess those are unrelated as well22:21
dansmith_diana_: and I think it's strange to make changes unrelated to your actual functional change, so ...22:21
*** nelsnelson has joined #openstack-nova22:21
dansmith_diana_: I can add it to this, or you can if you prefer: https://review.openstack.org/#/c/230177/22:21
*** mgoddard has joined #openstack-nova22:22
*** mwagner_ has quit IRC22:23
*** achanda has joined #openstack-nova22:23
_diana_dansmith: yeah, but in this case that's a big chunk of dead test data that people will think is the actual test data (like you did)22:23
dansmith_diana_: yeah, all the more reason to remove it separately22:23
dansmithhere:22:25
openstackgerritDan Smith proposed openstack/nova: Remove orphaned code related to extended_volumes  https://review.openstack.org/23017722:25
dansmith_diana_: you can either git rebase on top of that or use the Depends-On linkage in your commit message22:26
_diana_dansmith: unused imports...22:26
dansmithheh22:26
melwitt_diana_: one reason is because if a patch needs to be reverted later, if gardening is in it too, we revert the unrelated cleanups. and in general when we go to trace history in git log, whether it's chasing down a bug or whatever, it's helpful for each commit to be a logical grouping of work when we are looking at the diffs22:27
dansmithyeah, that22:27
*** ZZelle_ has quit IRC22:28
dansmithI'm rebuilding my functional venv because I've got a test o.vo wedged in there, so it'll be a few22:28
_diana_dansmith, melwitt: Yup, I get all this. I just think there's a happy medium.22:28
openstackgerritAnne Gentle proposed openstack/nova: Rearranges to create new Compute API Guide  https://review.openstack.org/23018622:28
dansmithwhy are we using PIL in the functional venv? that's breaking now of course22:29
openstackgerritDan Smith proposed openstack/nova: Remove orphaned code related to extended_volumes  https://review.openstack.org/23017722:29
dansmithanyone else seeing this trying to build functional? http://paste.openstack.org/show/475127/22:31
jrolldansmith: needs moar libjpeg22:32
dansmithjroll: apparently my libjpeg level is too low, yeah22:32
jrollwhy does nova depend on PIL?22:33
dansmithjroll: my question above.. that's the silliest one yet22:33
*** xyang1 has quit IRC22:33
jrolldo we get pictures in API responses now?22:33
jrollthat would be sweet22:33
dansmithjroll: six degrees of separation between openstack and any python library I think22:33
jrollya22:33
dansmithjroll: I can think of some image API responses I'd like to return22:33
jrollheh22:33
jrollan architecture diagram of all your compute/storage/network would be cool22:34
jrollthough I imagine your images being much darker22:34
dansmithevery time nova returns novalidhost, we just return that epic openstack architecture diagram22:34
*** mgoddard has quit IRC22:34
jroll++22:34
openstackgerritDiana Clarke proposed openstack/nova: Replace N block_device_mapping queries with 1  https://review.openstack.org/22996422:35
*** signed8bit has quit IRC22:35
*** mgoddard has joined #openstack-nova22:38
*** thumpba has quit IRC22:38
*** ctrath has quit IRC22:40
dansmithapparently we have a new consistent failure22:40
dansmiththe pep8 job just failed because of that PIL dep as well22:40
dansmithso things are probably wedged22:40
dansmithand no mriedem when you need him22:40
dansmithlifeless: you around yet?22:41
*** thumpba has joined #openstack-nova22:41
dansmithlifeless: when you are: http://logs.openstack.org/77/230177/3/check/gate-nova-pep8/29a95c0/console.html#_2015-10-01_22_38_44_73922:42
*** thumpba has quit IRC22:42
*** mgoddard has quit IRC22:44
*** obedmr has quit IRC22:45
*** obedmr has joined #openstack-nova22:45
*** sambetts has quit IRC22:45
dansmithlifeless: ah, nevermind fix in the pipeline already.. continue sleeping.22:46
*** mgoddard has joined #openstack-nova22:48
*** sambetts has joined #openstack-nova22:48
*** DinaBelova has quit IRC22:49
*** MVenesio has joined #openstack-nova22:49
*** obedmr has quit IRC22:49
*** mriedem has joined #openstack-nova22:51
*** DinaBelova has joined #openstack-nova22:52
*** signed8bit has joined #openstack-nova22:52
*** alex_klimov has quit IRC22:53
*** mgoddard has quit IRC22:53
*** MVenesio has quit IRC22:53
*** mgoddard has joined #openstack-nova22:57
*** ijuwang has quit IRC22:58
*** signed8bit has quit IRC22:58
sdaguedansmith: so we branched liberty without the liberty const defined for rpc?22:59
sdaguedims_: also, it looks like oslo.service backports don't help22:59
sdaguedansmith: so, in looking at all the scenarios I pushed up to test.23:01
sdagueif we stop believing that oslo.service knows how to stop processes and double kill in devstack (I litterally cherry picked your kilo patch forward), the last remaining issue is there is no liberty rpc level23:02
*** stevemar has quit IRC23:02
sdagueso partial doesn't work23:02
*** alejandrito has quit IRC23:03
*** mgoddard has quit IRC23:03
dansmithsdague: we only need the liberty alias in mitaka23:03
sdagueright23:03
sdaguebut we need to segment stable/liberty23:03
dansmithsdague: and also, I'm hoping to make manual pins a thing of yesteryear23:03
*** tjones has quit IRC23:04
*** ctrath has joined #openstack-nova23:04
dansmithsdague: I don't follow.. you're talking about dims' patch right?23:04
sdagueno23:04
sdaguehttps://review.openstack.org/#/c/229363/23:04
sdaguewe can't land the branch selection in the gate23:04
sdaguebecause liberty -> master can't pass partial nova cpu23:05
*** IanGovett has quit IRC23:05
dansmithuntil master gets the alias23:05
sdagueright... until master gets it23:05
dansmithwhich dims has https://review.openstack.org/#/c/230132/23:05
sdaguesorry, was thinking through this wrong23:05
dansmiths/which/which is why/23:05
dansmithsdague: it's confusing, I know23:06
dansmithsdague: so I can fix up dims' patch if we want to try to slam that in now..23:06
sdaguedansmith: did enough of your service bits land that we don't need the pin?23:06
sdaguedansmith: that would be nice if we could23:06
dansmithsdague: still in review, but probably landing soon yeah23:06
dansmithsdague: okay I'll fix it up23:06
dansmithsdague: so we can use the alias now, but I'd like to set compute='auto' RSN23:07
*** mgoddard has joined #openstack-nova23:07
dansmithI look forward to that day23:07
*** asettle has joined #openstack-nova23:08
sdaguesure, it will default to auto right?23:08
sdagueso magic will just happen23:08
dansmithsdague: no, not yet.. but you could just set it to auto and leave it.. want to see it work before we default it on I think23:08
*** ijw has joined #openstack-nova23:09
sdagueok, so I've got the new giant stack of patches which maybe get us across the finish line23:09
dansmith\o/      |23:09
dansmith  \o/    |23:09
dansmith    \o/ |23:09
dansmith    \o/|23:10
dansmithdamn23:10
*** mriedem has quit IRC23:10
sdaguehttp://dl.dropbox.com/u/6514884/screenshot_265.png23:10
*** mriedem has joined #openstack-nova23:10
sdagueif that passes at the top, we have a path forward23:10
dansmithoof23:10
sdagueyeh, oof23:11
*** pixelbeat has quit IRC23:11
*** karimb has quit IRC23:11
*** mgoddard has quit IRC23:13
*** ijw has quit IRC23:13
dansmithsdague:23:14
openstackgerritDan Smith proposed openstack/nova: Update rpc version aliases for liberty  https://review.openstack.org/23013223:14
*** mriedem has quit IRC23:15
*** zzzeek has quit IRC23:16
*** mgoddard has joined #openstack-nova23:16
*** mwagner_ has joined #openstack-nova23:22
*** pm90_ has quit IRC23:22
*** mriedem has joined #openstack-nova23:25
*** sseverson has left #openstack-nova23:27
*** knikolla has joined #openstack-nova23:27
*** mgoddard has quit IRC23:28
*** markvoelker has quit IRC23:30
*** nic has quit IRC23:32
*** electrocucaracha has quit IRC23:34
*** mgoddard has joined #openstack-nova23:36
*** jwcroppe has quit IRC23:36
*** jwcroppe has joined #openstack-nova23:37
*** tjones has joined #openstack-nova23:38
*** mdenny has quit IRC23:39
*** baoli has quit IRC23:39
*** gyee has joined #openstack-nova23:39
*** jwcroppe has quit IRC23:41
*** mgoddard has quit IRC23:42
*** mgoddard has joined #openstack-nova23:43
*** hemna is now known as hemnafk23:44
*** achanda has quit IRC23:47
*** agireud has quit IRC23:47
*** mgoddard has quit IRC23:49
*** Daisy has joined #openstack-nova23:50
*** ijw has joined #openstack-nova23:52
*** mgoddard has joined #openstack-nova23:53
*** achanda has joined #openstack-nova23:54
*** dims has joined #openstack-nova23:56
*** ijw has quit IRC23:57
*** bnemec has joined #openstack-nova23:57
*** mgoddard has quit IRC23:58
*** dims has quit IRC23:58
*** dims_ has quit IRC23:58
*** dims has joined #openstack-nova23:59

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