| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-08 | |||
| 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 | |
| 14:59:14 | mriedem | see the existing entries in there as an example | |
| 14:59:17 | ameeda | cool, thanks | |
| 14:59:59 | mriedem | same for opensuse https://software.opensuse.org/package/pcre-devel | |
| 15:00:21 | mriedem | looks like we don't have specific entries for suse | |
| 15:00:53 | sean-k-mooney | stephenfin: since the intel nfv ci has been offline we dont test int 300 times a day and i have only been using it on ubuntu lately but i fixed the centos support a few months ago when centos decided qemu-kvm should nolonger install qemu.. | |
| 15:01:01 | sean-k-mooney | stephenfin: if you have issue let me know | |
| 15:01:08 | Roamer` | mriedem, pcre-devel should cover all RPM-based distributions | |
| 15:01:09 | mriedem | ameeda: btw, you can see the same thing done in devstack https://github.com/openstack-dev/devstack/commits/master/files | |
| 15:01:22 | stephenfin | sean-k-mooney: I'm using CentOS, so I sure will | |
| 15:01:31 | mriedem | devstack changes aren't bindep, but that shows you which packages for which distro | |
| 15:01:39 | Roamer` | mriedem, there's a bit less divergence in RPM-based distributions than there is between Debian and Ubuntu sometimes, at least WRT libraries and development packages, not the system startup stuff :) | |
| 15:02:29 | ameeda | mriedem: now I am using Centos, and the bug for Ubuntu , so the fix needs to be on ubuntu or don't care ? | |
| 15:02:35 | Roamer` | (and I say this a bit sadly, with all my love for Debian and respect for Ubuntu and, well, reluctance when it comes to working on RPM-based distributions) | |
| 15:02:57 | mriedem | ameeda: i'm not sure how to answer that | |
| 15:03:11 | mriedem | ameeda: you can update nova and push the fix from a gentoo system, it doesn't matter, | |
| 15:03:18 | mriedem | just add the packages to nova's bindep file | |
| 15:03:22 | Roamer` | ameeda, try to figure out what the bug actually is, what libpcre3-dev is and why it needs to be added | |
| 15:03:36 | mriedem | Roamer`: it's a transitive dependency, | |
| 15:03:42 | Roamer` | mriedem, I know, I fixed it for our CI | |
| 15:03:43 | mriedem | because of the new dep on the whereto package | |
| 15:03:51 | mriedem | whereto requires pcre | |
| 15:03:59 | cdent | bauzas: thanks for that. I read through and I wonder to what extent claims in the scheduler are now making a difference to the concerns? | |
| 15:04:02 | ameeda | thanks all for help, I have to go now, I will back soon. | |
| 15:06:11 | bauzas | cdent: I feel we solved the main blocker | |
| 15:06:13 | gibi | mriedem: I think it is not a big deal but we have some inconsistency here https://review.openstack.org/#/c/541008/9/nova/tests/unit/image/test_glance.py@1629 | |
| 15:06:22 | bauzas | cdent: because I was wrong in the past | |
| 15:06:35 | bauzas | cdent: the fact that we have workers doesn't mean greenlets | |
| 15:06:53 | bauzas | hence separate processes, exactly like having multiple n-sch services | |
| 15:07:21 | bauzas | so, what we solved with scheduler claims is also a valid argument for saying we can have workers | |
| 15:10:57 | cdent | I've made reference to the code and the email thread in the notes I'm writing up, so we'll have that to refer back to later if needed | |
| 15:11:12 | mriedem | gibi: i think it's probably fine, the only place we append anything to that url is in "generate_image_url" and we append /images/{id} to it | |
| 15:11:23 | mriedem | for the createImage response location header | |
| 15:11:43 | mriedem | and some notification during rebuild | |
| 15:13:17 | edleafe | bauzas: I do remember that the argument was made for claiming in the scheduler that it would allow for multiple scheduler processes | |
| 15:13:33 | mriedem | the pike release notes also say you can now run multiple schedulers b/c we do claims in the scheduler | |
| 15:13:44 | mriedem | which is why i wanted to add that test wrinkle to the nova-next job | |
| 15:13:54 | mriedem | since we have had 409 issues since pike with claims in the scheduler | |
| 15:13:57 | stephenfin | sean-k-mooney: So etcd failed to start. Lovely :) | |
| 15:14:25 | bauzas | mriedem: sure, but we don't support yet multiple workers, hence the discussion | |
| 15:14:41 | bauzas | it's just a very simple patch | |
| 15:14:48 | bauzas | but someone has to write it | |
| 15:15:04 | mriedem | bauzas: you mean this patch? https://review.openstack.org/#/c/159382/ | |
| 15:15:28 | bauzas | and again, I think I wonder what is the opportunity of having a separate process for filtering | |
| 15:15:45 | bauzas | in particular now we have cells v2 and superconductor/conductors | |
| 15:16:41 | bauzas | mriedem: yup | |
| 15:17:00 | bauzas | anyway, something to merge in Rocky :) | |
| 15:17:14 | openstackgerrit | Merged openstack/nova master: Fix 500 in test_resize_server_negative_invalid_state https://review.openstack.org/531117 | |
| 15:21:14 | openstackgerrit | Jianghua Wang proposed openstack/nova master: VGPU: Modify the example of vgpu white_list set https://review.openstack.org/539183 | |
| 15:23:16 | openstackgerrit | Merged openstack/nova master: Add index(instance_uuid, updated_at) on instance_actions table https://review.openstack.org/530429 | |
| 15:23:41 | openstackgerrit | Matt Riedemann proposed openstack/nova master: VGPU: Modify the example of vgpu white_list set https://review.openstack.org/539183 | |
| 15:24:53 | gibi | mriedem: OK, then I'm +2 | |
| 15:26:25 | jianghuaw | mriedem, thanks. | |
| 15:28:47 | sean-k-mooney | stephenfin: that is unrelated i normally trun it off as it is not need at all for a default devstack install | |
| 15:29:27 | sean-k-mooney | stephenfin: i have found the etcd installation in devstack to be kindo of flaky | |
| 15:29:34 | efried | lajoskatona: I'm going to rebase the series upon which you have https://review.openstack.org/#/c/527728/9 stacked. I'll go ahead and pull yours into the rebase unless you have objections? | |
| 15:30:10 | stephenfin | sean-k-mooney: So I had HOST_IP configured to the IP of the IF I was binding DPDK to. Think I had that wrong, tbhg | |
| 15:30:40 | sean-k-mooney | stephenfin: haha ya that could cause issues allright | |
| 15:31:16 | dansmith | mriedem: in case it's not clear I think this patch from tssurya is closer to being ready than takashi's https://review.openstack.org/#/c/541246/ | |
| 15:33:26 | sean-k-mooney | stephenfin: are you deploying on a phyicla server or a vm | |
| 15:33:35 | stephenfin | sean-k-mooney: The latter | |
| 15:33:36 | mriedem | dansmith: i think it is too | |