Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-03-06
12:50:26 openstackgerrit Hironori Shiina proposed openstack/nova master: Ironic: Support boot from Cinder volume https://review.openstack.org/215385
13:08:32 satyar johnthetubaguy: nova-manage account quota refresh workout for me if you know the file which does the update of quota after deletion i can look at the code to fix it
13:08:45 satyar i would say try to fix it ;)
13:10:05 cdent sdague: thanks for making that post clarifying the log translation question. I think _not_ translating would be good (it breaks googling).
13:14:31 sdague cdent: I've been on all sides of this one, and am fine with letting the i18n team set the policy
13:14:43 sdague but it looks like the policy changed, and it wasn't fully communicated
13:14:47 cdent yes
13:32:37 johnthetubaguy satyar: this is the code: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1776
13:33:28 johnthetubaguy satyar: this path is quite new, it might be worth some extra logging to see what is happening: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1730
13:36:45 satyar Thanks johnthetubaguy
13:37:52 johnthetubaguy satyar: its worth looking in your DB at the reservations table, its possible there are stale ones in there (we should probably get that nova-manage cmd to clear those out too)
13:38:29 satyar yeah i ran the nova-manage to clean those out
13:39:42 satyar those which are deleted but not gone out from quota of CPU/RAM/instances after running the nova-manage are freed up
13:39:51 satyar not quota is showing the right number
13:43:01 satyar not -> now
13:43:31 jsheeren hi all; quick question about the spice-proxy
13:43:57 jsheeren when i setup a windows image with virt-manager and install all drivers as needed, the gui is shown as you would expect
13:44:07 jsheeren but when i launch an instance from that image on my openstack env, the spice console shows the gui upside down :s
13:44:17 jsheeren like so: http://imgur.com/a/LqGai
13:44:40 jsheeren has anyone seen something like this?
13:44:41 johnthetubaguy satyar: yeah, sounds a lot like we are leaking reservations somewhere
13:44:50 satyar yeah
13:45:30 satyar its also reproduced by mszankin
13:46:14 openstackgerrit Rafael Folco proposed openstack/nova-specs master: DELETE all inventories for a resource provider https://review.openstack.org/415885
13:46:23 johnthetubaguy satyar: yeah, I think that is macsz on IRC
13:46:33 satyar :)
13:49:56 johnthetubaguy satyar: thinking about this, I suspect it might be because of this logic here not commiting quota: https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L869
13:50:41 edleafe Nova scheduler subteam meeting in 10 minutes in #openstack-meeting-alt
13:55:49 satyar yes seems thats the one johnthetubaguy
13:56:02 satyar logic seems to be incomplete
14:00:24 arne__r Hi channel, I'm trying to get a patch through newton: https://review.openstack.org/#/c/436868/3
14:01:10 arne__r without it, nova on s390 is broken
14:02:34 johnthetubaguy arne__r: have we got that patch merged in master yet?
14:03:25 johnthetubaguy arne__r: hmm, I see links to lots of reviews I guess
14:03:56 arne__r johnthetubaguy: A different one got merged in master, and the cherry pick to newton got abandoned
14:04:28 johnthetubaguy arne__r: afraid I don't have any context to help with that one
14:05:45 arne__r johnthetubaguy: i planned to bug matt about it, since he made the original patch
14:07:56 arne__r Any help from other people would also be apreciated of course =)
14:16:57 gibi jaypipes, melwitt, dansmith: hi! we shortly talked about instance.flavor.projects lazy load on the PTG. I think I found why sometimes lazy load works through instance.flavor. Could you check my fix https://review.openstack.org/#/c/440347/ ?
14:28:20 dansmith gibi: that's probably okay but let me think a bit
14:35:47 openstack bug 1668267 in OpenStack Compute (nova) "Nova fails to correctly update the quota when deleting lots of VMs, some of which previously failed due to a quota error" [High,In progress] https://launchpad.net/bugs/1668267 - Assigned to Pushkar Umaranikar (pushkar-umaranikar)
14:35:47 johnthetubaguy satyar: this from gibi feels very related https://review.openstack.org/#/c/437222 I wonder if dansmith or melwitt could take a quick look at your bug 1668267
14:36:37 dansmith johnthetubaguy: speaking of that,
14:36:59 dansmith johnthetubaguy: I've pounded on melwitt's counting quotas thing quite a bit and it fixes some issues we currently have when using multiple cells
14:37:15 johnthetubaguy dansmith: it probably also fixes the above bug right?
14:37:39 johnthetubaguy looks like a reservations leak, or something like that
14:37:43 dansmith johnthetubaguy: probably categorically yeah
14:38:04 dansmith johnthetubaguy: I think she said she had some feedback from you still to resolve yet, but otherwise I think we oughta get those in
14:39:13 johnthetubaguy dansmith: I seem to be stuck in a pile of post PTG admin at the moment, I feel like quitting that and doing some reviewing, that looks like a good place to start
14:39:43 dansmith johnthetubaguy: heh, well, I think she still has to address some feedback from you, or so she said on friday
14:40:38 johnthetubaguy dansmith: ah, OK, at a first glance its looking much closer now
14:41:11 openstackgerrit Alex Xu proposed openstack/nova-specs master: The ResourceProviderTraits API - Manage traits with ResourceProviders https://review.openstack.org/345138
14:42:05 dansmith johnthetubaguy: cool, I've not gone through it in detail, but was just testing it underneath my cells one as it resolved several problems for me
14:42:20 johnthetubaguy cool
14:50:33 openstackgerrit John L. Villalovos proposed openstack/nova master: Use flake8-import-order https://review.openstack.org/435575
14:58:23 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Check if domain is persistent before detaching volume https://review.openstack.org/441204
15:06:40 jaypipes_ cdent: morning. got in after midnight last night. just getting started this morning. can you recap the scheduler meeting for me please? anything urgent for me to address other than fixing up the few things mriedem and you pointed out on the Ironic get_inventory() patch?
15:07:40 edleafe jaypipes_: does http://eavesdrop.openstack.org/meetings/nova_scheduler/2017/nova_scheduler.2017-03-06-14.00.log.html help? :)
15:07:51 openstackgerrit Alin Balutoiu proposed openstack/os-vif master: vif_plug_ovs: Skip setting MTU on Windows when plugging devices https://review.openstack.org/441855
15:07:56 jaypipes_ edleafe: don't have time to read the full thing, really..
15:08:17 edleafe The only thing was it wasn't clear why PUT/DELETE was considered so dangerous
15:08:33 artom dansmith, oh hey, you're up :) As main reviewer for device tagging last time around, can I ask you to start checking out https://review.openstack.org/#/c/376021/ ?
15:08:38 artom mriedem, same to you ^^
15:08:39 edleafe especially since they would both fail with 409 if any of the traits were in use
15:08:57 cdent jaypipes_: yeah, as edleafe points out the main discussion was around traits api, and we reached some sound conclusions
15:09:16 dansmith artom: yeah, I've got it in the queue
15:09:25 jaypipes_ cdent: k. guess I'll read back through the IRC log.
15:09:54 artom dansmith, wootness
15:10:15 cdent jaypipes_: nothing pressing, so I wouldn't stress it
15:11:11 jaypipes_ cdent: meh, I'd love to make some good forward progress on traits this week, possibly merging the spec and the first few in the code series (after more reviews of course)
15:11:57 edleafe jaypipes_: well, we agreed to go with the simpler approach, with the understanding that we could always add to it later if necessary
15:11:58 cdent jaypipes_: well, it's under control, I would guess we'll have a spec merge in the next 24-48 hours
15:12:15 jaypipes_ cool, that'
15:12:23 jaypipes_ s great news guys, thanks cdent and edleafe
15:12:36 jaypipes_ and sorry for missing the meeting.
15:21:07 arne_r mriedem: Hey Matt, could you take a look at this? https://review.openstack.org/#/c/436868/3 Without it, nova is still broken on s390 in newton
15:21:15 openstackgerrit Evgeny Antyshev proposed openstack/nova master: get_model method missing for Ploop image https://review.openstack.org/441989
15:31:08 openstackgerrit Balazs Gibizer proposed openstack/nova master: Short circuit notifiations when not enabled https://review.openstack.org/428260
15:55:40 openstackgerrit Lee Yarwood proposed openstack/nova master: compute: Move detach logic from manager into driver BDM https://review.openstack.org/439520
15:55:41 openstackgerrit Lee Yarwood proposed openstack/nova master: compute: Only destroy BDMs after successful detach call https://review.openstack.org/440693
15:55:41 openstackgerrit Lee Yarwood proposed openstack/nova master: objects: Add attachment_id to BlockDeviceMapping https://review.openstack.org/437665
15:55:42 openstackgerrit Lee Yarwood proposed openstack/nova master: db: Add attachment_id to block_device_mapping https://review.openstack.org/437597
15:55:43 openstackgerrit Lee Yarwood proposed openstack/nova master: Make BlockDeviceMapping object support uuid https://review.openstack.org/242603
15:55:44 openstackgerrit Lee Yarwood proposed openstack/nova master: Add uuid column to BlockDeviceMapping https://review.openstack.org/242602
16:04:04 cfriesen can someone explain to me the logic behind https://github.com/openstack/nova/blob/master/nova/virt/hardware.py#L1109 ? I thought that if the flavor policy was "prefer" then we would use the image policy, if one was specified.
16:04:55 cfriesen sfinucane: ^
16:05:24 johnthetubaguy sfinucan: ^
16:08:52 cfriesen sfinucan: I think Nikola's commit 24997343 changed the logic of this code block, but I don't think it was intentional based on the commit message
16:09:28 johnthetubaguy mriedem: could you remove the -2 on this one now please https://review.openstack.org/#/c/418025/
16:23:45 smcginnis I don't see any placement-api setup in the Ocata install guide.
16:23:54 smcginnis Is that missing, or am I overlooking it. https://docs.openstack.org/ocata/install-guide-ubuntu/nova-compute-install.html
16:30:25 mriedem smcginnis: https://review.openstack.org/#/c/438328/
16:30:49 mriedem smcginnis: and https://docs.openstack.org/developer/nova/placement.html
16:30:54 smcginnis mriedem: Thanks
16:35:48 mriedem dansmith: nova.conf on the controller doesn't need access to the main nova [database] right?
16:36:02 mriedem i'm looking at http://docs-draft.openstack.org/28/438328/3/check/gate-openstack-manuals-tox-doc-publish-checkbuild/319104b//publish-docs/draft/install-guide-ubuntu/nova-controller-install.html
16:36:12 dansmith mriedem: it does today
16:36:23 dansmith mriedem: going forward it will point at cell0
16:36:34 mriedem oh plus compute nodes
16:36:37 mriedem for the scheduler fallback

Earlier   Later