Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-01-12
11:57:46 openstackgerrit Zhenyu Zheng proposed openstack/nova: [poc] add a decorator to reject know bad filters. https://review.openstack.org/419256
11:59:48 openstackgerrit Zhihai Song proposed openstack/nova: Enable tests.functional.db.api.test_migrations for py35 https://review.openstack.org/419435
12:02:02 openstackgerrit Zhenyu Zheng proposed openstack/nova: [poc] add a decorator to reject know bad filters. https://review.openstack.org/419256
12:02:48 openstackgerrit Gábor Antal proposed openstack/nova: Transform instance.volume_attach notification https://review.openstack.org/401992
12:13:02 openstackgerrit Gábor Antal proposed openstack/nova: Transform instance.volume_detach notification https://review.openstack.org/408676
12:15:11 openstackgerrit Béla Vancsics proposed openstack/nova: Reduce code duplication https://review.openstack.org/270911
12:21:25 openstackgerrit Istvan Imre proposed openstack/python-novaclient: Pass relevant parameters to Token based authentication https://review.openstack.org/419441
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 :/

Earlier   Later