| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-20 | |||
| 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 | } | |
| 19:18:09 | mriedem | 'what': 'i mean?' | |
| 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? | |
| 21:47:40 | cfriesen | mriedem: but it's different code! | |
| 21:47:46 | mriedem | OH! | |
| 21:48:01 | dansmith | less critical for the release code? | |
| 21:48:03 | dansmith | sign me up. | |
| 21:48:13 | cfriesen | less stressful, more laid-back | |
| 21:48:14 | mriedem | cfriesen: did you miss this while you were on vacation? | |
| 21:48:21 | mriedem | this abuse | |
| 21:48:43 | cfriesen | all the time. had to go search out random strangers | |