| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-06 | |||
| 10:36:08 | openstackgerrit | sahid proposed openstack/nova master: numa: take into account cpus reserved https://review.openstack.org/400876 | |
| 10:36:48 | openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Port binding based on events during live migration https://review.openstack.org/434870 | |
| 10:49:53 | johnthetubaguy | alex_xu: been meaning to reach out to you about things a bit related to what is written down here: https://review.openstack.org/#/c/440917/9/doc/contributor-guide/source/non-native-english-speakers.rst | |
| 10:50:26 | johnthetubaguy | alex_xu: looking at creating some place where folks can share good ideas on how to work with upstream when you are a non-native-english-speaker | |
| 10:50:41 | johnthetubaguy | alex_xu: would love your help with that, if you are keen? | |
| 10:58:59 | openstack | Launchpad bug 1670319 in OpenStack Compute (nova) "Reschedule of failed instance doesn't happening when scheduler placed two instances to the same ironic node" [Undecided,New] | |
| 10:58:59 | vsaienk01 | jroll, dtantsur, mriedem fyi found issue when reschedule won't happening for failed instance https://bugs.launchpad.net/nova/+bug/1670319 it leads to sporadic ironic-multinode job failures | |
| 11:03:03 | bhagyashri_s | johnthetubaguy: Hi, I have put my thoughts on patch: https://review.openstack.org/#/c/409644/ could you please take some insight and suggest your opinion about it? | |
| 11:04:21 | johnthetubaguy | bhagyashri_s: I don't think we should do the delete in the API, we try to keep the API doing as little work as possible, and dispatch any long running work to either a conductor worker or a compute worker | |
| 11:05:52 | bhagyashri_s | johnthetubaguy: ok. | |
| 11:08:47 | bhagyashri_s | johnthetubaguy: thank you. | |
| 11:10:26 | johnthetubaguy | bhagyashri_s: no problem, it was an idea certainly worth thinking about, but yeah it would chance how we scale the ystem | |
| 11:11:41 | openstackgerrit | Roman Podoliaka proposed openstack/nova master: nova-status: require placement >= 1.4 https://review.openstack.org/441881 | |
| 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 | 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:12 | satyar | my quota usages are full because of the bug https://bugs.launchpad.net/nova/+bug/1668267 | |
| 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 | 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. | |