| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-08 | |||
| 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 | |
| 15:33:38 | stephenfin | Sorry - former | |
| 15:33:44 | stephenfin | Got a machine sat here beside me | |
| 15:36:08 | sean-k-mooney | ok this let me see if i have the vm i was doing the centos fixes on still and i can see if i can grab the local.conf. that said if you have issue with the getting started guide it would be nice to fix them too | |
| 15:37:08 | stephenfin | sean-k-mooney: Well, let's see. It's proceeding quite nicely so far | |
| 15:37:12 | stephenfin | since I fixed the HOST_IP issue | |
| 15:43:02 | lajoskatona | efried: from my side it is ok for rebasing. I just tried to put my patch for tests on top of yours to see how it goes on latest nested patches | |
| 15:43:29 | efried | lajoskatona: Okay. Should be coming up in the next 10-15 minutes. | |
| 15:44:58 | mriedem | dansmith: one of the proxy methods in https://review.openstack.org/#/c/541005/ is broken | |
| 15:45:03 | mriedem | otherwise just some questions | |
| 15:45:50 | dansmith | mriedem: ack thanks /cc stephenfin | |
| 15:49:49 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Fix wrong link for "Manage Flavors" in CPU topologies doc https://review.openstack.org/542270 | |
| 15:50:19 | stephenfin | mriedem, dansmith: Alrighty then, dumb question time 🎉 How are we able to drop the v4 proxy in the above change so early? Surely that means all our clients would have to talk 5.0 as soon as we do that? | |
| 15:50:44 | dansmith | stephenfin: we merge the 5.0 server and client in queens, | |
| 15:50:53 | dansmith | then in rocky we can drop the 4.x server proxy | |
| 15:51:02 | dansmith | stephenfin: we send 5.0 by default after that second change | |
| 15:51:21 | dansmith | which means queens supports 5.0 client side, and since we only support one gap, we can drop 4.x in rocky | |