| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-01 | |||
| 03:45:14 | Spaz-Work | Hrm.. True enough, I guess I was seeing it from our angle, we call --block-migration no matter what because we don't use aggregates | |
| 03:45:44 | Spaz-Work | the flag has an interesting side effect of skipping the host aggregate lookup heh | |
| 03:52:02 | jianghuaw | Spaz-Work, thanks for the sharing. Wish you'll have a happy vacation. | |
| 03:52:35 | jianghuaw | I'm happy to leave the work on "get the code-block out of --block-migrate specifics" and wait for you return from the vacation:-) | |
| 03:53:01 | Spaz-Work | Sounds good. Would you like to see the exception changed at that time since it will open it up outside of the block-migration ? | |
| 03:54:59 | jianghuaw | sure. | |
| 04:07:01 | openstackgerrit | lei zhang proposed openstack/nova master: Zuul: Remove project name https://review.openstack.org/539801 | |
| 04:14:09 | Spaz-Work | . | |
| 05:23:08 | ankkumar | Hello | |
| 05:23:43 | ankkumar | we created vm with nova and we are putting our physical interface to virtual bridge | |
| 05:24:19 | ankkumar | Also inside the vm we installed openvswitch and assigning ip to bridge but it is not able to ping or it is not reachable. | |
| 05:24:42 | ankkumar | Is there any way we can make ip pingable? | |
| 05:25:30 | ankkumar | Or what is the way to add the virtual port through nova? | |
| 05:25:51 | Spaz-Work | Hi ankkumar, this channel is generally about Nova Development. Someone may have the answer for you here but you may have better luck in #openstack | |
| 05:26:43 | ankkumar | Spaz-Work, ok | |
| 05:32:28 | Spaz-Work | can see another failure condition this block migration if statement opened up.. | |
| 05:33:01 | Spaz-Work | It only checks to ensure aggregation in the destination check if it's NOT a block migration but then pulls by aggregate in the block migration if statement.. heh | |
| 05:33:49 | Spaz-Work | Not completely sure why these XenAPI drivers pull by aggregate instead of just pulling the resident | |
| 05:43:16 | Spaz-Work | n/m actaully read that wrong hehe.. am dum | |
| 06:13:03 | openstackgerrit | Tuan Luong-Anh proposed openstack/nova master: Fix some reST field lists in docstrings https://review.openstack.org/538809 | |
| 06:41:43 | openstackgerrit | Merged openstack/nova master: [placement] Separate API schemas (resource_provider) https://review.openstack.org/528629 | |
| 06:42:03 | openstackgerrit | Merged openstack/nova stable/pike: libvirt: Re-initialise volumes, encryptors, and vifs on hard reboot https://review.openstack.org/531407 | |
| 06:42:22 | openstackgerrit | Merged openstack/nova master: Implement get_traits() for the ironic virt driver https://review.openstack.org/532288 | |
| 06:42:32 | openstackgerrit | Merged openstack/nova master: Add functional tests for virt driver get_traits() method https://review.openstack.org/538047 | |
| 06:42:44 | openstackgerrit | Merged openstack/nova master: placement: enable required traits from the flavor extra specs https://review.openstack.org/536085 | |
| 06:44:13 | openstackgerrit | Merged openstack/nova stable/pike: tests: Use correct response type in tests https://review.openstack.org/538938 | |
| 06:45:22 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Encode libvirt domain XML in UTF-8 https://review.openstack.org/522161 | |
| 07:20:40 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/539841 | |
| 07:26:20 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/533918 | |
| 07:42:46 | ameeda | Hello. I try to alter table in sqlite as following http://paste.openstack.org/show/658280/ , but I got functional test failure | |
| 07:42:53 | ameeda | I got this error 'instances' not in {'instance_system_metadata': 11, 'instance_actions_events': 2, 'block_device_mapping': 1, 'instance_info_caches': 1, 'instance_actions': 2, 'instance_extra': 1, 'security_group_instance_association': 1} | |
| 07:47:46 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Improve performance when list instances with IP filter https://review.openstack.org/539469 | |
| 07:49:34 | maciejjozefczyk_ | bauzas: pong :) | |
| 07:58:40 | openstackgerrit | Merged openstack/nova stable/pike: Handle images with no data https://review.openstack.org/538939 | |
| 08:02:54 | hrw | mriedem: asked and left. no one replied anyway | |
| 08:25:12 | hrw | die i440fx, die | |
| 08:33:03 | openstackgerrit | sahid proposed openstack/nova-specs master: virt: allow instances to be booted with trusted VFs https://review.openstack.org/485522 | |
| 08:36:59 | openstackgerrit | sahid proposed openstack/nova-specs master: update: introducing isolate emulthreads on host https://review.openstack.org/511188 | |
| 08:48:35 | openstackgerrit | Brooks Kaminski proposed openstack/nova master: XenAPI: XCP2.1+ Swallow VDI_NOT_IN_MAP Exception https://review.openstack.org/538415 | |
| 08:49:18 | Spaz-Work | jianghuaw: I did go with a swallow here.. I updated the commit message for my reasoning. | |
| 08:49:25 | Spaz-Work | Will get a post on the bug report here in a few too | |
| 08:50:08 | jianghuaw | good. Will review it a little later. | |
| 08:50:16 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: return non-None value for cpuset_reserved https://review.openstack.org/539865 | |
| 08:51:09 | Spaz-Work | No rush, just wanted to let you know :) | |
| 08:52:50 | bauzas | maciejjozefczyk_: hola, was afk | |
| 08:52:58 | bauzas | good morning folks, btw. | |
| 08:53:04 | Spaz-Work | Morning sir | |
| 08:53:29 | tetsuro | morning | |
| 08:55:37 | jianghuaw | bauzas, good morning:-) | |
| 08:57:51 | maciejjozefczyk_ | bauzas: hey | |
| 08:58:00 | maciejjozefczyk_ | bauzas: reading your comments | |
| 08:58:29 | openstackgerrit | Merged openstack/nova master: Fix bug case by none token context https://review.openstack.org/522112 | |
| 08:58:39 | openstackgerrit | Merged openstack/nova master: report client: get_provider_tree_and_ensure_root https://review.openstack.org/521098 | |
| 08:58:52 | openstackgerrit | Merged openstack/nova master: ComputeDriver.update_provider_tree() https://review.openstack.org/521187 | |
| 08:59:00 | bauzas | oh, btw. for folks not in European hours, "good morning" means "good [afternoon,night]" also :p | |
| 08:59:15 | bauzas | maciejjozefczyk_: do you understand the possible problem ? | |
| 08:59:26 | openstackgerrit | Merged openstack/nova master: reno for notification-transformation-queens https://review.openstack.org/518018 | |
| 08:59:33 | bauzas | it will work for Pike computes if the operator didn't modified the nova.conf | |
| 09:00:17 | bauzas | but if the operator modified the compute nova.conf for Pike, then we would possibly change the ratio into the conf opt value of any other service calling it | |
| 09:01:46 | maciejjozefczyk_ | bauzas: yes I see your point | |
| 09:02:21 | maciejjozefczyk_ | Maybe we should do distinguish between serviceand compute calling that method? | |
| 09:02:57 | ameeda | bauzas: I right sqlite migration script to alter column type as following http://paste.openstack.org/show/658286/, but when I run functional test I got this errors http://logs.openstack.org/00/526900/27/check/nova-tox-functional/f6ce415/testr_results.html.gz | |
| 09:02:58 | bauzas | maciejjozefczyk_: my thoughts is maybe we could just look at the ComputeNode object version | |
| 09:03:12 | ameeda | sorry "I write*" | |
| 09:04:02 | ameeda | bauzas: can you help me with that ? | |
| 09:04:11 | maciejjozefczyk_ | hmm bauzas could be good | |
| 09:04:32 | maciejjozefczyk_ | leave the logic for pike, move new one if object is newest one | |
| 09:09:35 | bauzas | maciejjozefczyk: sec, looking at code | |
| 09:09:43 | bauzas | ie. where we are accessing ratios | |
| 09:09:59 | bauzas | AFAIR, we only get or set ratios either in compute service or within the scheduler | |
| 09:10:17 | bauzas | so, the big problem is only for the scheduler | |
| 09:10:19 | maciejjozefczyk | bauzas: I think too thats its only in compute-node or scheduler | |
| 09:10:22 | maciejjozefczyk | bauzas: right | |
| 09:10:44 | bauzas | of course, let's find a solution that would work for any service | |
| 09:11:09 | bauzas | because if we just fix that only for the scheduler, if after some time, someone lookups ratios in the nova-api worker for example, it couldn't owrk | |
| 09:11:28 | maciejjozefczyk | bauzas: yes, it should be complex | |
| 09:11:50 | bauzas | okay, so it's confirmed | |
| 09:12:01 | jianghuaw | Spaz-Work, thanks for updating the patch with well defined commit messages. It looks very reasonable. I totally agreed with you:-) | |
| 09:12:13 | bauzas | we load ComputeNode objects in the scheduler HostManager to correctly set the HostStates | |
| 09:12:45 | Spaz-Work | jianghuaw: Thanks sir. Definately nerve-wracking commit since it's nasty, wanted to make sure I was clear ;) | |
| 09:14:46 | jianghuaw | The commit message did help me to understand why the fix goes with that way:-) | |
| 09:15:19 | bauzas | maciejjozefczyk: so, after looking at the object code, I think checking the object version won't work | |
| 09:16:50 | bauzas | because if we call the object from the scheduler side, it will get the latest version, then calling the remotable method for getting the primitives, then transforming the primitives if necessary and then hydrating the object from the primitive | |
| 09:17:15 | bauzas | but when we hydrate the primitive, we're already having a computenode object version that is the latest | |
| 09:18:23 | maciejjozefczyk | bauzas: the stupid solution is duplicating the fields, right?:) | |
| 09:18:54 | bauzas | what do you mean ? | |
| 09:20:25 | maciejjozefczyk | like adding ram_allocation_ratio_v2 for Rocky and do magic in filtering primitives? If I understand it correctly | |
| 09:21:08 | bauzas | oh please no | |
| 09:21:10 | maciejjozefczyk | but yes, its not a solution, its workaround | |
| 09:21:12 | maciejjozefczyk | :D | |
| 09:21:22 | maciejjozefczyk | bauzas: yes, just kidding | |
| 09:25:21 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: Make sure that we have USB Host controller for USB tablet https://review.openstack.org/538003 | |
| 09:27:31 | kashyap | mriedem: About the question you asked me in the scroll, I asked the libvirt upstream folks, and this was the response: | |
| 09:28:06 | maciejjozefczyk | bauzas: So maybe some if logic from scheduler side? If the object is in version X, and we know that its from Rocky, then we'll not do thing from https://review.openstack.org/#/c/532924/6/nova/objects/compute_node.py L185, | |
| 09:28:11 | kashyap | mriedem: That "note" you pointed out from the OpenStack docs was never the case, so that statement wasn't ever true | |
| 09:28:30 | maciejjozefczyk | bauzas: I'm looking for better solution but I havent found any | |
| 09:28:48 | kashyap | mriedem: One can use `virsh nodedev-dumpxml` on the parent device and it will return a dump of the 'sysfs' attributes. | |
| 09:29:39 | bauzas | maciejjozefczyk: one possible outcome could be to transform in obj_make_compatible the primitive we got from the compute | |
| 09:30:26 | bauzas | maciejjozefczyk: ie. say that if the primitive is having a target version older than me, and if ratio is 0.0, set the value to what we want | |
| 09:32:20 | maciejjozefczyk | bauzas: Yes, obj_make_compatible is right place for that | |