| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-01 | |||
| 03:04:04 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: trivial: Fix few policy doc https://review.openstack.org/539780 | |
| 03:25:36 | jianghuaw | Spazmotic, see my comments in https://review.openstack.org/#/c/538415 | |
| 03:36:06 | Spaz-Work | jianghuaw: thanks for the update and the extra info on that bug report | |
| 03:36:10 | Spaz-Work | I'll be adjusting the checking tonight sir | |
| 03:36:37 | Spaz-Work | I'm curious the best way to go about that since we're guarunteed a micro version here.. I am debating between converting to a tuple and comparing or importing the oslo versioning method to handle the comparison | |
| 03:37:16 | Spaz-Work | Riedman also showed me an openstack method I need to look into that may work for it more elegantly.. but yeah.. just havn't been at a computer. Will commit it within a few hours :) | |
| 03:37:53 | jianghuaw | Spaz-Work, looks good. thanks. | |
| 03:38:20 | Spaz-Work | jianghuaw: I would like to changei t from a full skip to just swallowing the VDI_NOT_IN_MAP exception IF the platform version >= 2.1.0 | |
| 03:38:26 | Spaz-Work | That way we can avoid the skip for lower versions | |
| 03:38:59 | Spaz-Work | And then very likely an immediate follow-up commit when I get home from Korea to get the code-block out of --block-migrate specifics, unless you guys want to handle it while i'm away | |
| 03:40:41 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Encode libvirt domain XML in UTF-8 https://review.openstack.org/522161 | |
| 03:42:27 | jianghuaw | Spaz-Work, Actually I prefer to skip the whole check and leave the work spreading the check out of block migration block. But I'd +1 also if you go with swallowing the exception. | |
| 03:42:54 | jianghuaw | Spaz-Work, May I know your vacation plan? | |
| 03:43:11 | Spaz-Work | Any reason why you'd prefer a full skip? Just to avoid the extra code when we need to redesign anyway? | |
| 03:43:32 | Spaz-Work | Sure sir. Flying into Incheon on the 6th your time zone and will spend a Week in Seoul, and then a week in Daegu | |
| 03:43:45 | Spaz-Work | And some smaller cities along the way in the mountains and such. | |
| 03:43:59 | Spaz-Work | Spending 2 weeks with my finace while we wait for Visa | |
| 03:44:40 | jianghuaw | the reason is: at the moment it's only invoked for block migration. And the exception is of block. | |
| 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 | |