| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-06 | |||
| 11:11:50 | bhagyashri_s | johnthetubaguy: yeah. | |
| 11:17:08 | gibi | johnthetubaguy: hi! as we agreed on the PTG there is a spec proposed to show the scheduler-hints in the response of the GET /servers/ https://review.openstack.org/#/c/440580/ . If you have time could you check it please? | |
| 11:18:04 | johnthetubaguy | gibi: yeah, I hope to get to that | |
| 12:24:31 | alex_xu | johnthetubaguy: haha, that is sweet doc, happy to help that | |
| 12:25:45 | johnthetubaguy | alex_xu: awesome, thanks. The SWG is trying to help share best practices around all parts of the community, and look for new ideas to improve things too | |
| 12:28:04 | alex_xu | johnthetubaguy: got it, i will check if I have some idea can put in that doc | |
| 12:33:04 | johnthetubaguy | alex_xu: attempting to write up things in a central place here: https://review.openstack.org/441923 | |
| 12:33:15 | johnthetubaguy | alex_xu: would love your ideas around all that | |
| 12:34:34 | alex_xu | johnthetubaguy: cool, go tit, I will try to summary some | |
| 12:34:40 | alex_xu | s/tit/it/ | |
| 12:38:41 | satyar | Hi Everyone | |
| 12:39:12 | satyar | my quota usages are full because of the bug https://bugs.launchpad.net/nova/+bug/1668267 | |
| 12:39:12 | openstack | Launchpad 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] - Assigned to Pushkar Umaranikar (pushkar-umaranikar) | |
| 12:39:20 | satyar | any way to free it up? | |
| 12:39:38 | johnthetubaguy | satyar: did you try the nova-manage command to reset the quotas? | |
| 12:39:46 | satyar | yes | |
| 12:39:50 | satyar | didnt worked out | |
| 12:40:25 | johnthetubaguy | thats interesting in its self I guess | |
| 12:41:28 | johnthetubaguy | going for lunch right now, just say that pop up, will be back in 30 mins or so | |
| 12:41:46 | satyar | sure | |
| 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 | 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: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: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: objects: Add attachment_id to BlockDeviceMapping https://review.openstack.org/437665 | |