| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-08 | |||
| 13:56:26 | bauzas | so the index is required anyway | |
| 13:56:34 | mriedem | the index is for the filter, | |
| 13:56:37 | mriedem | we order by created_at | |
| 13:56:39 | mriedem | not updated_at | |
| 13:56:41 | bauzas | but I'm still curious | |
| 13:57:51 | bauzas | meeting time in 3 mins, right? | |
| 13:58:01 | gibi | right | |
| 13:58:28 | bauzas | cool | |
| 14:07:32 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Workaround glanceclient bug when CONF.glance.api_servers not set https://review.openstack.org/541008 | |
| 14:07:38 | mordred | mriedem: yes, I would like it if GET /flavors/detail included the extra specs - but it's not *super* important so I certainly wouldn't prioritize it just for me :) | |
| 14:09:05 | mriedem | mordred: just getting user feedback; our product team wants it because they have a time limit on loading flavors in the UI, and they have to first get all flavors and then for each one, get their extra specs, and that can timeout the UI panel | |
| 14:09:34 | mriedem | i actually found a blueprint from 4 years ago for the same thing when we were doing nova v3 api | |
| 14:12:10 | mordred | mriedem: yah - I have the same issue in shade (minus the timelimit) because I was dumb and made the default behavior of list_flavors be to add the extra-specs by default - so doing a list_flavors on a cloud with a lot of flavors takes *forever* | |
| 14:13:12 | mordred | mriedem: I swapped the default behavior when we merged the shade code into the sdk repo, but I do think it would be nice to just get it all at once | |
| 14:13:30 | mriedem | we actually load it all up from the db too, so we have it in the api, we just never returned it | |
| 14:16:16 | mriedem | ameeda: should be an easy bug to fix https://bugs.launchpad.net/nova/+bug/1746386 | |
| 14:16:18 | openstack | Launchpad bug 1746386 in OpenStack Compute (nova) "Bindep does not catch missing libpcre3-dev on Ubuntu" [Low,Confirmed] | |
| 14:16:39 | ameeda | mriedem: thanks a lot | |
| 14:27:19 | ingy | mordred: o/ | |
| 14:35:35 | jianghuaw | bauzas, mriedem: I guess we should pick up this small patch which only corrects the example for the vGPU config option: https://review.openstack.org/#/c/539183/ | |
| 14:37:02 | bauzas | jianghuaw: right, I catched it but forgot to update it | |
| 14:37:16 | bauzas | jianghuaw: FWIW, we don't need brackets | |
| 14:37:34 | jianghuaw | bauzas, yes. | |
| 14:37:45 | mordred | ingy: heya! | |
| 14:38:00 | bauzas | jianghuaw: I mean, we don't need quotes | |
| 14:38:29 | jianghuaw | it should work regardless there is quotes. | |
| 14:38:37 | ingy | mordred: hi. msged you yesterday but not sure you got it. | |
| 14:39:17 | ingy | cdent told me to look for you here... | |
| 14:40:06 | jianghuaw | bauzas, I mean oslo will parse it correctly. | |
| 14:40:08 | mriedem | jianghuaw: another question inline | |
| 14:41:47 | openstackgerrit | Adrien Cunin proposed openstack/nova master: Fixed auto-convergence option name in doc https://review.openstack.org/542237 | |
| 14:42:26 | jianghuaw | mriedem, yes. I will update it. thanks. | |
| 14:43:16 | cdent | mriedem: I'm continuing my explorations with slamming placement etc. and am now wondering: What are the reasons (if any) that we don't allow nova-scheduler to have worker processes? | |
| 14:43:52 | bauzas | cdent: IIRC, that was because we were having racinesses | |
| 14:44:02 | bauzas | I can find old threads | |
| 14:44:10 | cdent | bauzas: thanks | |
| 14:44:26 | mriedem | here you go https://review.openstack.org/#/c/159382/ | |
| 14:44:29 | mriedem | cdent: bauzas: ^ | |
| 14:44:40 | mriedem | that's a huawei patch so i've got'er starred :) | |
| 14:44:44 | bauzas | ideally, I just feel we should pursue the idea to drop the whole n-sch service and do the thing in the conductor | |
| 14:45:04 | stephenfin | dansmith: Highlighted the RPC 5.0-ish things I'm aware of here https://review.openstack.org/#/c/541005/ | |
| 14:45:13 | mriedem | cdent: fwiw, i've had a todo sitting here to update nova-next in devstack to run with 2 nova-scheduler processes | |
| 14:45:21 | dansmith | stephenfin: thanks will look in a sec | |
| 14:45:33 | cdent | forcing some workers makes the rpc timeout problems go away and the instances to land, but I've got all filters turned off, so I suspect raciness won't rear its head if it is still there | |
| 14:45:37 | bauzas | cdent: in particular if you split placement, having multiple n-sch besides conductors is a PITA for operators | |
| 14:45:41 | stephenfin | Not sure if that can be done or not, so I'm relying on you to educate me. Can tackle it myself though, if so (and if it would help) | |
| 14:46:17 | bauzas | cdent: mriedem: the problem was tied to the HostManager.consume_from_instance() lock, IIRC | |
| 14:46:19 | cdent | bauzas: you say that but I find it hard to believe in a world of automation that running another service is a big deal, especially in the face of so many services... | |
| 14:46:33 | bauzas | call it microservices :) | |
| 14:46:52 | openstackgerrit | Jianghua Wang proposed openstack/nova master: VGPU: Modify the example of vgpu white_list set https://review.openstack.org/539183 | |
| 14:47:06 | mriedem | most places are running multiple schedulers, they have to for the load, they just get reschedules if they are packing | |
| 14:47:19 | jianghuaw | bauzas, mriedem: ^ | |
| 14:47:21 | ameeda | mriedem: when I try to do "tox bindep" I got those errors http://paste.openstack.org/show/666615/ , I use master branch "queens" , any idea ? | |
| 14:47:29 | bauzas | multiple schedulers != multiple workers, and I don't exactly remember why | |
| 14:47:51 | bauzas | that's old plain story I don't remind | |
| 14:48:57 | stephenfin | ameeda: What host OS are you using? | |
| 14:49:12 | ameeda | Centos , should I do it on ubuntu ? | |
| 14:49:24 | cdent | mriedem, bauzas : do you recall if Nikola ever produced the email message or blog post that was claimed? | |
| 14:49:26 | bauzas | mriedem: oh thanks for the patch, yeah I got the problem now | |
| 14:50:07 | stephenfin | Not yet. Try adding 'basepython = python2.7' above 'deps' in the '[testenv:bindep]' section of tox.ini | |
| 14:50:10 | bauzas | cdent: what do you mean by "claimed" ? | |
| 14:50:12 | stephenfin | ameeda: ^ | |
| 14:50:25 | bauzas | cdent: I can point you some ndipanov's changes about fixing the locking mechanism | |
| 14:50:32 | stephenfin | ameeda: That'll force tox to use Python 2.7 | |
| 14:50:35 | bauzas | that's related | |
| 14:50:39 | cdent | bauzas: he said in the comments that he would right a more detail comment in an email message. did he? | |
| 14:50:49 | stephenfin | Actually, nvm me | |
| 14:50:57 | stephenfin | ameeda: It's 'tox -e bindep' | |
| 14:50:58 | bauzas | cdent: don't blame me, I'm 37 but I have poor memory :) | |
| 14:51:24 | ameeda | stephenfin: thanks !! | |
| 14:51:49 | bauzas | but I know we merged some concurrency patches around consume_from_instance, which later became consume_from_request() | |
| 14:52:40 | openstackgerrit | Jianghua Wang proposed openstack/nova master: VGPU: Modify the example of vgpu white_list set https://review.openstack.org/539183 | |
| 14:52:48 | dansmith | stephenfin: the instance object stuff is definitely clear, and I think host as well, but if you could track down the host bit that would be appreciated | |
| 14:53:02 | stephenfin | dansmith: Sure thing | |
| 14:53:13 | ameeda | stephenfin: Missing packages: postgresql-server python34-devel sqlite-devel ERROR: InvocationError: '/opt/stack/nova/.tox/bindep/bin/bindep test' | |
| 14:53:21 | dansmith | even so, I'm not sure those compat blocks are fully legit, as we've had obj_compat decorators to hide that stuff for many year | |
| 14:53:32 | cdent | Well, for now I will continue my experiments with multiple scheduler workers and see what happens. | |
| 14:54:12 | stephenfin | ameeda: Right, so bindep isn't tracking the correct packages for CentOS, That's another bug | |
| 14:54:39 | stephenfin | ameeda: But given the issue was reported against Ubuntu, you probably _should_ use that to fix _that_ particular bug :) | |
| 14:54:46 | bauzas | cdent: I think I found the ML thread | |
| 14:54:50 | ameeda | stephenfin: I will test it on ubuntu | |
| 14:55:10 | ameeda | stephenfin: do you know the link of current centos bug :$ | |
| 14:55:11 | jianghuaw | bauzas, have removed the quotes:-) https://review.openstack.org/#/c/539183 | |
| 14:55:19 | cdent | bauzas: awesome | |
| 14:55:23 | bauzas | cdent: http://thread.gmane.org/gmane.comp.cloud.openstack.devel/47882 | |
| 14:55:30 | stephenfin | ameeda: Nope, there probably isn't one. You should report it (and maybe try to fix it) | |
| 14:55:52 | ameeda | stephenfin: great, Thanks for help :) | |
| 14:55:53 | stephenfin | Assuming we support CentOS for that. I'm not sure if we do, given that we don't gate on it | |
| 14:55:56 | stephenfin | np | |
| 14:56:09 | mriedem | just add the rpm and deb packages to bindep in the same patch | |
| 14:57:16 | mriedem | libpcre3-dev [platform:dpkg] | |
| 14:57:19 | mriedem | i think that's all you need | |
| 14:58:03 | mriedem | looks like it's pcre-devel on fedora/centos | |
| 14:58:06 | mriedem | https://rpmfind.net/linux/rpm2html/search.php?query=pkgconfig(libpcre) | |
| 14:58:13 | ameeda | mriedem: you are talking about current opened bug of ubuntu . what do you think about the Centos one ? | |
| 14:58:30 | mriedem | ameeda: it's a nova bug that the entries are missing from nova's bindep, | |
| 14:58:38 | mriedem | just add the entries for deb and rpm | |
| 14:58:43 | mriedem | which i've given above | |
| 14:59:01 | ameeda | to this file bindep.txt right ? | |
| 14:59:07 | mriedem | yes | |