Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-03-29
22:46:42 TheJulia yeah
22:47:22 TheJulia kind of at least :(
22:53:04 Spaz-Home Morning
22:53:28 TheJulia Good morning
23:01:29 efried TheJulia: So yeah, it looks like on a per-request basis you can pass in a `headers` dict.
23:01:45 efried ...with the X-OpenStack-Version-Whatever thingy.
23:02:08 efried All the layers between there and the wire appear to use setdefault correctly, so it *should* make it through.
23:02:47 efried bbiab...
23:02:53 TheJulia hmmm
23:04:22 TheJulia yeah, same problem though
23:04:28 TheJulia we bypass all of our microversion logic then
23:04:33 TheJulia at least client side
23:11:26 TheJulia Well, looks like we would also re-negotiate with what was provided upon the client being called originally
23:15:40 TheJulia and I think we would have to add code to handle that this it would be a user defined override
23:17:17 jroll not to mention, then you're leaking implementation details out to the caller
23:17:48 TheJulia yeah....
23:18:20 openstackgerrit Merged openstack/nova master: [placement] Fix bad management of _TRAITS_SYNCED flag https://review.openstack.org/557722
23:18:51 jroll TheJulia: like dmitry, I'd also commit to reviewing client changes to accept that per method, but I don't have any problems with keeping a couple copies of the client
23:19:02 jroll especially if we plan to fix that later
23:24:35 efried Let me work something up right quick. Maybe you'll fall in love with the idea when you see it in action :)
23:26:45 efried melwitt: Oh, another thought I had was putting core reviewer names next to runway slots. Volunteers & SMEs first; then...
23:31:11 efried TheJulia, jroll: Okay, never mind; the (bizarrely resolved through _multi_getattr) target methods don't even take kwargs.
23:31:49 TheJulia heh
23:32:19 TheJulia I think I looked at that a couple of weeks ago pondering something similar
23:32:34 TheJulia of of similar use I should say
23:36:51 efried TheJulia: I removed my -1. Thanks for bearing with me.
23:37:36 TheJulia efried: no, thank you for looking and thinking
23:50:24 openstackgerrit Tony Breeds proposed openstack/nova stable/ocata: Only attempt a rebuild claim for an evacuation to a new host https://review.openstack.org/550560
#openstack-nova - 2018-03-30
00:19:44 openstackgerrit Merged openstack/nova master: Use ksa session for cinder microversion check https://review.openstack.org/557508
00:19:50 openstackgerrit Merged openstack/nova stable/pike: docs: Disable smartquotes https://review.openstack.org/554189
01:08:36 Spaz-Home Whew new shoes.. short workout.. let's do some nova
02:34:25 openstackgerrit Merged openstack/nova stable/pike: add check before adding cpus to cpuset_reserved https://review.openstack.org/557622
02:34:33 openstackgerrit Merged openstack/nova stable/queens: Use ksa session for cinder microversion check https://review.openstack.org/557837
03:04:18 openstackgerrit Jackie Truong proposed openstack/nova master: Add trusted_certs to instance_extra https://review.openstack.org/537897
03:04:19 openstackgerrit Jackie Truong proposed openstack/nova master: Add trusted_certs object https://review.openstack.org/489408
03:04:20 openstackgerrit Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949
03:04:23 openstackgerrit Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
04:33:36 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387
04:33:37 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240
04:33:37 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658
04:33:38 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344
04:33:38 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340
04:34:02 Spaz-Home How you doing this evening novaers
04:58:23 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in tests/unit/test_utils.py https://review.openstack.org/557883
05:39:03 openstackgerrit Leopard Ma proposed openstack/nova master: fix paused instance not support shelve https://review.openstack.org/541660
05:51:31 openstackgerrit jichenjc proposed openstack/nova master: WIP: remove ec2 in service and cmd https://review.openstack.org/556778
05:51:32 openstackgerrit jichenjc proposed openstack/nova master: WIP: remove db ec2 https://review.openstack.org/557572
05:51:32 openstackgerrit jichenjc proposed openstack/nova master: WIP: remove Ec2 object https://review.openstack.org/557150
05:58:51 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475
05:58:51 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: deprecate the config for image handler class path https://review.openstack.org/497201
06:43:57 openstackgerrit jichenjc proposed openstack/nova master: Remove a outdated warning https://review.openstack.org/557898
07:00:08 openstackgerrit Danfly proposed openstack/nova master: update availablity_zone in the instances table after live-migration https://review.openstack.org/491226
07:05:14 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/548772
07:29:15 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove unnecessary code encoding specification https://review.openstack.org/557903
07:49:14 openstackgerrit sahid proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658
07:49:14 openstackgerrit sahid proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387
07:51:57 openstackgerrit sahid proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658
07:51:57 openstackgerrit sahid proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387
08:10:05 openstackgerrit jichenjc proposed openstack/nova master: WIP: remove Ec2 object https://review.openstack.org/557150
08:10:05 openstackgerrit jichenjc proposed openstack/nova master: WIP: remove ec2 in service and cmd https://review.openstack.org/556778
08:10:06 openstackgerrit jichenjc proposed openstack/nova master: WIP: remove db ec2 https://review.openstack.org/557572
08:21:38 openstackgerrit Arvind Nadendla proposed openstack/nova master: Update ImageMetaProp object to expose traits https://review.openstack.org/557795
08:32:06 jianghuaw_ bauzas, hi
08:32:41 bauzas hi
08:32:46 jianghuaw_ Could you help to have a look at this patch to define traits for maximum display heads?
08:32:52 jianghuaw_ https://review.openstack.org/#/c/553277/
08:33:54 jianghuaw_ I will start to work on populating traits to resource providers for Xen.
08:36:42 jichen hi sahid: thanks for the review and upload , regarding this comment , actually it's used , so the patch https://review.openstack.org/#/c/523387 should be ok now (I think your -1 is on this comment)
08:37:35 Kevin_Zheng gibi, Hi, is there still a BP to add more versioned notifications?like for instance update etc.
08:38:25 sahid jichen: are you refering get_conn()?
08:38:46 jichen sahid: yes, that's I am talking about
08:39:23 sahid i see, but no no we should not use it
08:39:38 jichen sahid: ok, I can modify it
08:39:50 jichen that's the only reason for -1, right?
08:39:53 bauzas oh shit, forgot sahid's spec review :(
08:40:07 bauzas sahid: I'll look at your spec today :)
08:40:12 bauzas jianghuaw_: sure, I'll try
08:40:27 sahid yes, basically have self._reqh = ConnetorRequestHanlder(..)
08:40:28 jianghuaw_ bauzas, Thanks.
08:40:34 sahid jichen: ^ makes sense?
08:40:53 sahid also please check the UT, I did not have updated them
08:41:16 jichen sahid: I will run UT locally and upload another patch, thanks a lot!
08:41:22 openstackgerrit Merged openstack/nova master: remove a comment about ec2 https://review.openstack.org/555615
08:41:54 sahid jichen: baiscally i don't have strong opinion, you can passan instane of ConnectorRequestHandler to Hypervisor
08:42:01 sahid or make Hypeervisor to create it
08:42:18 jichen yes, I can do that, should not be a big issue :)
08:42:45 sahid jichen: did you see the comments on the next patch?
08:43:14 sahid bauzas: ok no worries
08:43:16 jichen yes, I will figure it out next week as have some private things to handle and have to leave in 30 min , really appreciate your help~
08:43:30 bauzas sahid: the spec is still under review, right?
08:43:56 sahid jichen: no problem, thanks for your patience :)
08:44:08 sahid bauzas: yes, jaypipes +2ed it
08:44:17 jichen sahid: thank you and have a nice weekend
08:44:28 sahid jichen: you too
08:56:48 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658
08:56:48 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387
08:56:49 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340
08:56:49 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240
08:56:50 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344
08:59:52 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313

Earlier   Later