| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-20 | |||
| 18:22:49 | trinaths1 | sean-k-mooney: oh ok. but as you have said above. i require libvirt 1.3.5 and qemu 2.4 | |
| 18:23:52 | openstackgerrit | Merged openstack/nova master: Make key_pairs use check_deltas() for quota https://review.openstack.org/477699 | |
| 18:24:14 | sean-k-mooney | trinaths1: do you need exactly those version or are they the minium. fedroas rpms should actully be newer then that but if you are trying to reporduce an issue that wont help you. | |
| 18:25:31 | trinaths1 | sean-k-mooney: agree. but libvirt 1.3.5 is mandatory. | |
| 18:25:48 | trinaths1 | sean-k-mooney: but qemu can be something different above 2.2 | |
| 18:27:53 | sean-k-mooney | trinaths1: it might be worth installing qemu from dnf then instead and see if it resolves the issue. | |
| 18:27:58 | sean-k-mooney | actully | |
| 18:28:14 | sean-k-mooney | trinaths1: do you have selinux enabled | |
| 18:28:30 | sean-k-mooney | its a long shot but that could be blocking the request | |
| 18:29:29 | sean-k-mooney | trinaths1: your main error though is internal error: No JSON parser implementation is available | |
| 18:29:43 | trinaths1 | sean-k-mooney: SELinux status: disabled | |
| 18:29:48 | sean-k-mooney | trinaths1: that is why i taught it might be related to how you compiled libvirt | |
| 18:30:11 | trinaths1 | sean-k-mooney: yes. True said. but I find no help from google on this. | |
| 18:30:26 | trinaths1 | sean-k-mooney: so finally i have come here. | |
| 18:30:33 | sean-k-mooney | trinaths1: if you still have it i would check the configure output ans see if there is is any reference to json | |
| 18:32:49 | trinaths1 | sean-k-mooney: sure. I'm currently rebuilding libvirt fresh with qemu 2.4 refering the devstack plugin link you have provided. | |
| 18:33:57 | sean-k-mooney | trinaths1: in our devstack pluging we do "yum-builddep -y libvirt" and "yum-builddep -y qemu" and then install libnuma-devel gcc make gettext-devel pkgconfig. my guess would be "yum-builddep -y libvirt" is installing a json lib and enableing json support when we run configure and you are missing the lib | |
| 18:36:20 | trinaths1 | sean-k-mooney: yes true said. and its giving a parsing error | |
| 18:37:06 | trinaths1 | sean-k-mooney: here is libvirt configure summary http://paste.openstack.org/show/616072/ | |
| 18:39:00 | melwitt | dansmith: hm, I don't remember. maybe to make them match create_limit and update_limit, which I don't think were called from compute or non-db-having services in the first place either. but I need to think about it more | |
| 18:40:03 | dansmith | melwitt: well, if you can make them not remotable, then they don't need a version bump and we don't have to have an awkward conversation about the naughtyness that lies within :) | |
| 18:40:28 | sean-k-mooney | trinaths1: i think you are missing yajl | |
| 18:40:44 | sean-k-mooney | trinaths1: https://github.com/libvirt/libvirt/blob/e64f2fab920fe8a08e97b790cb5d959f3566aea3/m4/virt-yajl.m4#L24-L45 | |
| 18:41:51 | trinaths1 | sean-k-mooney: I need that lib installed before libvirt configure ? | |
| 18:41:57 | sean-k-mooney | trinaths1: i would try installing yajl-devel | |
| 18:42:05 | sean-k-mooney | trinaths1: yes | |
| 18:43:05 | trinaths1 | sean-k-mooney: ok. doing that now. | |
| 18:43:16 | sean-k-mooney | trinaths1: just so you are aware without also installing libnuma-devel you will not be able to use hugepages or cpu pinning | |
| 18:43:58 | trinaths1 | sean-k-mooney: agree. | |
| 18:45:06 | melwitt | dansmith: heh. I shall sanity check that. I expect they don't need to be remotable | |
| 18:45:11 | trinaths1 | sean-k-mooney: its numactl-devel for fedora 23 | |
| 18:46:11 | dansmith | melwitt: cool | |
| 18:46:49 | sean-k-mooney | trinaths1: ah i just have centos and ubunut vms to hand currently so i assumed it would be the same as centos | |
| 18:47:53 | trinaths1 | sean-k-mooney: oh okay. | |
| 18:48:12 | trinaths1 | sean-k-mooney: even yum is moved to dnf | |
| 18:49:13 | sean-k-mooney | trinaths1: yes i think you can use dnf on centos too. they have a compaitblity shim on fedora to allow yum to still work but dnf is faster at installing things | |
| 18:50:07 | trinaths1 | sean-k-mooney: agree | |
| 18:54:33 | sean-k-mooney | trinaths1: anyway, that is my best quess of how to fix the issue. if you reconfig and recomiple then restart libvirtd then it might solve your issue | |
| 18:55:33 | trinaths1 | sean-k-mooney: ok. re-configuring and re-compiling libvirt now. | |
| 18:55:57 | trinaths1 | sean-k-mooney: i will share the configure summary in a while. | |
| 18:56:32 | trinaths1 | sean-k-mooney: http://paste.openstack.org/show/616077/ | |
| 18:57:25 | sean-k-mooney | trinaths1: well that looks a little better numa and yajl are now enabled. | |
| 18:57:53 | trinaths1 | sean-k-mooney: yes. any more options/packages i need to enable to compile ? | |
| 18:59:30 | sean-k-mooney | trinaths1: from what i can see you should have enough libvires installed for it to work with openstack. | |
| 19:00:07 | trinaths1 | sean-k-mooney: did i miss other libraries ? | |
| 19:00:10 | sean-k-mooney | trinaths1: if you want to use glsterfs or ceph you will need to install extra packages | |
| 19:01:21 | sean-k-mooney | trinaths1: but for a normal install i dont think you need any other packages | |
| 19:01:29 | mriedem_afk | ildikov: so i think we can remove handling VolumeUnattached from the rest api controller for delete | |
| 19:01:34 | mriedem_afk | (detach_volume) | |
| 19:04:25 | trinaths1 | sean-k-mooney: we did not more to ceph end. i have started re-compile of libvirt. I will update when its completed. | |
| 19:08:45 | mriedem | ildikov: ok comments inline | |
| 19:08:50 | mriedem | which are hopefully clear | |
| 19:13:53 | openstackgerrit | Jay Pipes proposed openstack/nova master: style-only: s/context/ctx/ https://review.openstack.org/485791 | |
| 19:13:54 | openstackgerrit | Jay Pipes proposed openstack/nova master: use os_traits.MISC_SHARES_VIA_AGGREGATE https://review.openstack.org/485792 | |
| 19:17:18 | ildikov | mriedem: ah ok, I didn't get to the point of fully removing it from the previous comments as it wasn't introduced in check_detach | |
| 19:17:28 | ildikov | mriedem: thanks for the comments, I'll update the patch | |
| 19:17:45 | ildikov | mriedem: cleaning up is always good, so yeah, let's do it properly | |
| 19:17:46 | mriedem | jaypipes: i for one love trailing parans | |
| 19:17:50 | mriedem | *parens | |
| 19:18:02 | mriedem | 'know': { | |
| 19:18:09 | mriedem | 'what': 'i mean?' | |
| 19:18:09 | mriedem | } | |
| 19:19:54 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/485634 | |
| 19:19:59 | jangutter | mriedem: fun fact, when I signed my current employment contract, there was an unmatched parenthesis. I still am uncertain whether I am actually employed. | |
| 19:20:25 | mriedem | ha | |
| 19:21:10 | openstackgerrit | Sean Dague proposed openstack/nova master: DNM: WIP adding tests for requestlog https://review.openstack.org/485793 | |
| 19:49:58 | jaypipes | mriedem: :) | |
| 20:09:01 | openstackgerrit | Robert Ellis proposed openstack/nova master: Changed code or added clarifying documentation where nodename is used in ironic driver. https://review.openstack.org/485803 | |
| 20:13:23 | openstackgerrit | Robert Ellis proposed openstack/nova master: Clarifying node_uuid usage in ironic driver. https://review.openstack.org/485803 | |
| 20:52:00 | openstackgerrit | melanie witt proposed openstack/nova master: Detach volume after deleting instance with no host https://review.openstack.org/340614 | |
| 20:52:22 | mriedem | nova team meeting in 8 minutes | |
| 20:53:20 | jaypipes | supernovas... UNITE! | |
| 20:53:53 | openstackgerrit | Sean Dague proposed openstack/nova master: WIP: request_log addition for running under uwsgi https://review.openstack.org/485602 | |
| 20:56:17 | jaypipes | dansmith: Voltron. :) | |
| 20:56:26 | jangutter | hey, I just got that. | |
| 20:56:35 | jangutter | coffee levels at 10% | |
| 20:56:40 | dansmith | I' | |
| 20:56:48 | dansmith | jaypipes: I'm proud to have no idea what that is | |
| 20:56:52 | jaypipes | heh | |
| 20:58:02 | melwitt | I always thought power rangers were a rip off voltron | |
| 20:58:16 | mriedem | they are all a rip off of some thing that started in japan | |
| 20:58:19 | mriedem | and are all terrible | |
| 20:58:38 | jangutter | mriedem: "Voltron - The End" is amazing though. | |
| 20:59:14 | melwitt | nova meeting in one minute | |
| 20:59:28 | mriedem | is the end of voltron better then the end of seinfeld? | |
| 20:59:30 | mriedem | because that's not hard | |
| 20:59:41 | jangutter | mriedem: it's a 4 minute youtube video. | |
| 20:59:59 | mriedem | no time | |
| 21:00:07 | mriedem | that's 3:45 too long | |
| 21:02:09 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Cleanup unnecessary logic in os-volume_attachments controller code https://review.openstack.org/485823 | |
| 21:03:27 | openstackgerrit | melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945 | |
| 21:04:24 | openstackgerrit | melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946 | |
| 21:22:28 | cfriesen | If anyone wants a short break from what they're doing, I'd welcome some additional reviewers for https://review.openstack.org/#/c/468203 and https://review.openstack.org/#/c/461456 . Sahid doesn't really like the proposed tweaks to cpu_realtime_mask but sfinucan is fine with it...would be useful to get a few more opinions. | |
| 21:45:20 | melwitt | dansmith: I removed the remotables from the Quotas object patch. those methods are only called in the quota classes API | |
| 21:45:30 | melwitt | thanks for pointing that out | |
| 21:45:39 | dansmith | melwitt: oh sweet | |
| 21:45:58 | melwitt | I must have had remotable fever when I first added them | |
| 21:46:12 | dansmith | a common affliction | |
| 21:46:34 | melwitt | haha, yup!! | |
| 21:47:03 | mriedem | cfriesen: so you mean, take a break from reviewing code to review some code? | |