| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-12 | |||
| 12:24:17 | openstackgerrit | Zhihai Song proposed openstack/nova: Update py35 functional tests blacklist https://review.openstack.org/415149 | |
| 12:31:42 | dims | mriedem : bauzas : the review for the py35 functional test blacklist is ready - https://review.openstack.org/#/c/419250/ | |
| 12:42:30 | openstackgerrit | Chris Dent proposed openstack/nova: Small improvements to placement.rst https://review.openstack.org/403811 | |
| 12:46:42 | cdent | mriedem, bauzas : can we get that merged or otherwise reviewed? Or if not, let me know so I can abandon it. | |
| 12:50:50 | openstackgerrit | Béla Vancsics proposed openstack/nova: Reduced the complexity of the _init_instance method https://review.openstack.org/374661 | |
| 12:53:42 | openstackgerrit | John Garbutt proposed openstack/nova: Pass objects into _create_ports_for_instance https://review.openstack.org/418512 | |
| 13:02:26 | bauzas | cdent: looking | |
| 13:02:38 | cdent | thanks | |
| 13:03:01 | bauzas | cdent: +2d | |
| 13:03:08 | cdent | more thanks! | |
| 13:03:12 | bauzas | cdent: I'll abandon my doc patch changing the port number | |
| 13:13:42 | gibi | hi nova! | |
| 13:13:50 | gibi | I have a question about lazy-loaded fields | |
| 13:13:55 | cdent | hi gibi ! | |
| 13:14:05 | cdent | (someone has had their coffee today) | |
| 13:14:14 | gibi | cdent: :) | |
| 13:14:36 | gibi | Do I see correctly that instance.flavor.projects access is a double lazy-load? | |
| 13:14:45 | gibi | and that it will not work | |
| 13:14:54 | gibi | as when the flavor is loaded into the instance | |
| 13:15:00 | gibi | then the Flavor object is orphaned | |
| 13:15:05 | gibi | https://github.com/openstack/nova/blob/master/nova/objects/instance.py#L315 | |
| 13:16:29 | gibi | so I have an instance object | |
| 13:16:40 | gibi | and I access instance.flavor | |
| 13:16:48 | gibi | which will lazy load the Falvor object | |
| 13:17:10 | gibi | and then I try to access the projects field of that Flavor object that is again a lazy load | |
| 13:17:18 | gibi | then that is fails as the Falvor object is orphaned | |
| 13:18:30 | gibi | I started here https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L5031 | |
| 13:18:36 | gibi | with an instance | |
| 13:19:07 | gibi | then that notification sending first access instance.flavor and that is OK as the instance is not orphaned | |
| 13:19:23 | gibi | but after that the code needs instance.flavor.projects and that fails | |
| 13:19:28 | cdent | gibi: this is the sort of question I'd love to see on the mailing list so that the response is preserved somewhere more readable by more people than IRC. Also I don't know the answer :) | |
| 13:19:37 | gibi | OrphanedObjectError: Cannot call _load_projects on orphaned Flavor object | |
| 13:19:52 | gibi | cdent: OK, I post to ML | |
| 13:21:17 | cdent | it looks like those flavor objects being created never get a context? | |
| 13:22:42 | openstackgerrit | Zhenyu Zheng proposed openstack/nova: [poc] add some code to reject known bad filters. https://review.openstack.org/419256 | |
| 13:22:49 | gibi | yes, it seems to me the context is missingf | |
| 13:23:56 | cdent | which presumably means someone wanted to make sure you can't do what you're trying to do? :) | |
| 13:24:41 | gibi | I'm not sure about that | |
| 13:25:25 | gibi | I think it is more likely that we forgot the context there | |
| 13:25:50 | Dinesh_Bhor | cdent: Hi, do you have some time? I am not much clear about 'Content-Type' in response so thought to ask you few doubts. | |
| 13:26:24 | cdent | Dinesh_Bhor: I can try. What's up? | |
| 13:26:39 | Dinesh_Bhor | cdent: Is it compulsory that all openstack services should return 'Content-Type' in response header? | |
| 13:26:49 | Dinesh_Bhor | cdent: I have seen most of the openstack services returning 'Content-Type' in response header. | |
| 13:28:17 | cdent | Dinesh_Bhor: that's a requirement of HTTP, not just openstack | |
| 13:28:38 | cdent | if there is a body in the response there needs to be a content-type header to identify what media type it is | |
| 13:29:47 | cdent | Dinesh_Bhor: there's some extensive information in the RFC: https://tools.ietf.org/html/rfc7231#section-3.1 | |
| 13:31:12 | cdent | Dinesh_Bhor: does that help? | |
| 13:31:34 | Dinesh_Bhor | cdent: yes, thank you so much | |
| 13:32:01 | cdent | the same thing is true of a request: if it has a body it needs a content-type header too | |
| 13:35:45 | openstackgerrit | Béla Vancsics proposed openstack/nova: Reduced the complexity of the _init_instance method (2) https://review.openstack.org/419463 | |
| 13:36:32 | dims | bauzas : you may have missed my ping :) https://review.openstack.org/#/c/419250/ | |
| 13:36:48 | bauzas | dims: heh, oops :) | |
| 13:36:58 | dims | no worries :) | |
| 13:37:22 | openstackgerrit | Julian proposed openstack/nova: Create script that prints JSON Schema of Notification Payload Classes https://review.openstack.org/411836 | |
| 13:41:02 | Dinesh_Bhor | cdent: sry to disturb you again. agree that If the request contains body then there must have content-type but if service returns an http exception then in response will that content-type be available? | |
| 13:41:33 | efried | mriedem: Can we add a PowerVM section in https://etherpad.openstack.org/p/ocata-nova-priorities-tracking ? | |
| 13:42:15 | cdent | Dinesh_Bhor: the content-type header should always say what the media type of the body is, even when returning an exception. If the exception has been formatted to JSON then the content-type should be application/json. If it is html, then text/html. If it is straight text then text/plain | |
| 13:43:32 | Dinesh_Bhor | cdent: ok, explains a lot | |
| 13:43:56 | cdent | Dinesh_Bhor: just out of curiosity what are you working on that raised these questions? | |
| 13:45:11 | Dinesh_Bhor | cdent: https://github.com/openstack/python-glanceclient/blob/master/glanceclient/exc.py#L156 this one | |
| 13:45:26 | Dinesh_Bhor | cdent: here all logic is based on content-type | |
| 13:47:15 | Dinesh_Bhor | cdent: and I am trying to solve bug in openstacksdk by refering that logic | |
| 13:51:58 | openstackgerrit | Gábor Antal proposed openstack/nova: Transform instance.volume_detach notification https://review.openstack.org/408676 | |
| 13:54:19 | mriedem | efried: yea | |
| 13:55:09 | dims | mriedem : please bless this https://review.openstack.org/#/c/419250/ | |
| 13:56:18 | mriedem | dims: done | |
| 13:56:32 | dims | thanks mriedem | |
| 13:57:07 | mriedem | lyarwood: fyi the placement job on stable/newton should be fixed now | |
| 13:57:09 | mriedem | bauzas: ^ | |
| 13:57:22 | bauzas | cool | |
| 13:58:02 | mriedem | jroll: now that https://review.openstack.org/#/c/364420/ is merged are you good to do an ironicclient release? | |
| 13:59:09 | dims | mriedem : yep, added the failing ones, then sort and uniq, that's it | |
| 13:59:45 | efried | mriedem: done (etherpad update) | |
| 14:05:33 | bauzas | mriedem: nova meeting ? :) | |
| 14:10:38 | openstackgerrit | sahid proposed openstack/nova: libvirt: update the active migrations DS to support migration result https://review.openstack.org/409260 | |
| 14:10:39 | openstackgerrit | sahid proposed openstack/nova: libvirt: ensure live-migration success https://review.openstack.org/409261 | |
| 14:15:41 | openstackgerrit | Tanvir Talukder proposed openstack/nova-specs: Support sub-admin roles in Nova https://review.openstack.org/419474 | |
| 14:16:44 | openstackgerrit | Tanvir Talukder proposed openstack/nova-specs: Support sub-admin roles in Nova https://review.openstack.org/419474 | |
| 14:17:16 | openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP][py35] Trying to get cinder+rally working https://review.openstack.org/419476 | |
| 14:19:05 | openstackgerrit | Stephen Finucane proposed openstack/nova: scheduler: Don't modify RequestSpec.numa_topology https://review.openstack.org/419479 | |
| 14:23:06 | jroll | mriedem: yep, will be doing that today | |
| 14:26:19 | openstackgerrit | Tanvir Talukder proposed openstack/nova-specs: Support sub-admin roles in Nova https://review.openstack.org/419474 | |
| 14:32:01 | openstackgerrit | Claudiu Belu proposed openstack/nova: hyper-v: Adds support for VMs with nested virtualization https://review.openstack.org/398509 | |
| 14:36:17 | openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP][py35] Trying to get cinder+rally working https://review.openstack.org/419476 | |
| 14:41:04 | artom | mdbooth, around? Feel like weighing in on a debate between vladikr and I? We're discussing what's the best way to match block devices between the BDM object (in the DB) and the instance XML in libvirt. It used to be done with device name, and I'm changing it to serial (aka volume uuid). Here's the review: https://review.openstack.org/#/c/418100/4 | |
| 14:41:31 | mdbooth | artom: Looking | |
| 14:50:50 | openstackgerrit | Maciej Szankin proposed openstack/nova: config options: remove libvirt imagecache https://review.openstack.org/378497 | |
| 14:57:13 | openstackgerrit | Istvan Imre proposed openstack/python-novaclient: Pass relevant parameters to Token based authentication https://review.openstack.org/419441 | |
| 15:02:09 | openstackgerrit | Radoslav Gerganov proposed openstack/nova: Update all instances when their AZ is renamed https://review.openstack.org/419502 | |
| 15:02:35 | mriedem | bauzas: thanks for running the meeting - i had to get my daughter ready for school so the 8am meeting always throws me off | |
| 15:02:50 | bauzas | mriedem: I have the exact same problem for the scheduler meeting | |
| 15:02:59 | bauzas | mriedem: so I understand you | |
| 15:03:37 | bauzas | my proposal to stop scholarizing my daughter unfortunately got a NACK from my wife :/ | |
| 15:03:46 | bauzas | don't understand why... | |
| 15:03:47 | mriedem | heh | |
| 15:04:10 | mriedem | just send her out during the day and let her learn the lessons of the wild | |
| 15:04:30 | bauzas | at 6, yeah | |
| 15:05:17 | edleafe | mriedem: drop her off early. That'll toughen her up in Minnesota winters! | |
| 15:05:30 | bauzas | I should have a dog by april, I'll teach my daughter how to dog sled | |
| 15:05:50 | mriedem | i had to walk to school at her age in the winter... | |