Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-06-12
15:07:14 openstackgerrit Stephen Finucane proposed openstack/nova master: Use plain routes list for os-consoles, os-console-auth-tokens endpoint instead of stevedore https://review.openstack.org/462175
15:07:14 openstackgerrit Stephen Finucane proposed openstack/nova master: Use plain routes list for user_data instead of stevedore https://review.openstack.org/462183
15:07:15 openstackgerrit Stephen Finucane proposed openstack/nova master: Use plain routes list for block_device_mapping instead of stevedore https://review.openstack.org/462180
15:07:42 sean-k-mooney that is useful for tings like dpdk that require sse3+ to function but perform best with avx.
15:08:44 sfinucan Kevin_Zheng: Rebased above patches to put them in series ^^^
15:08:58 sfinucan If we don't do that, we get endless merge conflicts :(
15:09:36 jaypipes sean-k-mooney: right.
15:09:58 alex_xu sean-k-mooney: not sure how people bill that, people pay a price of avx, but may not get avx. or people pay a price for sse3, but maybe they get some luck
15:11:03 sean-k-mooney alex_xu: i would guest the get billed by the minium gurantee but it would be up to the operator
15:12:01 sean-k-mooney alex_xu: even without the perfer if you got allocated to a sse3 node it may have avx
15:12:30 sean-k-mooney i would see the --require as an input to the filters and --prefer as an input to the weighers
15:13:00 alex_xu sean-k-mooney: emm...why not just required the avx
15:13:35 sean-k-mooney alex_xu: you could but i may not need that level of performance so might want to use a cheaper flavor
15:15:37 alex_xu sean-k-mooney: so, just two flavors one cheap and another one expensive should works. btw you may interesting this spec https://review.openstack.org/468797
15:16:24 alex_xu sean-k-mooney: we explore the prefer_traits in the spec, but removed, as I'm not very sure the use-case. but welcome the review and comments
15:16:46 sean-k-mooney alex_xu: yep it would but if we want to move away form usign the flavor to ecode the request in the futer you may want to be more dynamic
15:18:10 sean-k-mooney alex_xu: well for me the real usecase for for prefer vs require come fi you have a mano system or some level of intelagent that understands that expected kpi/level of performance that this instance will need to achive and can tailor the request accordingly
15:19:23 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add memoize function using oslo.cache https://review.openstack.org/472773
15:19:30 sean-k-mooney alex_xu: it could simply chose between 2 fixed flavors or dynamicl change if the trait is expressed in prefered or require depentind on if it knows it is requrieed to meet the sla or not
15:20:26 alex_xu sean-k-mooney: oh, you want to set the minmum performance for a flavor
15:20:57 sean-k-mooney alex_xu: ill be glad to take a look that the spec though. yes require set minium performece
15:21:46 alex_xu sean-k-mooney: thanks, I will keep that in mind also
15:22:59 Kevin_Zheng sfinucan: will do thanks
15:23:11 sfinucan Kevin_Zheng: No - I have it done already :)
15:23:26 sfinucan so please don't undo it, heh
15:23:33 Kevin_Zheng sure
15:28:26 openstackgerrit Baodong (Robert) Li proposed openstack/nova-specs master: Expose vlan trunking in metadata/configdrive https://review.openstack.org/471815
15:33:17 openstackgerrit Baodong (Robert) Li proposed openstack/nova-specs master: Expose vlan trunking in metadata/configdrive https://review.openstack.org/471815
15:36:37 openstackgerrit Matt Wisch proposed openstack/nova master: conf: Corrected compute_monitors help text https://review.openstack.org/468538
15:47:06 openstackgerrit Matt Wisch proposed openstack/nova master: Clarify conf/compute.py help text for ListOpts https://review.openstack.org/468538
15:53:11 efried sfinucan I'm a little confused by https://review.openstack.org/#/c/412634/15/nova/conf/glance.py@101 followed by +2+W. Did you mean to put up another patch set?
15:53:34 sfinucan efried: Nope - I just wanted to explain why it was done that way
15:53:43 sfinucan My phrasing wasn't great, admittedly
15:54:06 efried "Dodgy test removed." <== what is this referring to?
15:54:26 efried Cause the dodgy test you point out in PS15 doesn't appear to have been removed.
15:54:37 sfinucan efried: mriedem's comments on Jan 27 03:42
15:55:09 sfinucan efried: The 'test_glanceclient_session_cached' test is removed, right?
15:55:30 sfinucan that's the test I referred to (it's attempted mocking of a global is apparently not do'able)
15:57:34 efried sfinucan [Still digging]
15:59:59 efried sfinucan Phew, that took me longer than it should. Okay, I'm happy. I got confused because those two comments looked like they were related to each other.
16:00:06 efried sfinucan Thanks!
16:00:50 sfinucan efried: Cool cool. Sorry for the poor phrasing https://imgur.com/Lffqx89
16:16:53 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add BDM to InstancePayload https://review.openstack.org/448779
16:16:53 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add key_name field to InstancePayload https://review.openstack.org/463002
16:16:54 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add keypairs field to InstanceCreatePayload https://review.openstack.org/419730
16:16:54 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add separate instance.create payload type https://review.openstack.org/463001
16:19:18 mriedem oomichi: sdague: checking my understanding on something, which i think is a bug - the forced_down field in the PUT /os-services/force-down API uses the boolean parameter type in the schema https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/schemas/services.py#L42
16:19:38 mriedem which can be something like 'yes' https://github.com/openstack/nova/blob/master/nova/api/validation/parameter_types.py#L190
16:20:02 sdague mriedem: ok
16:20:04 mriedem which isn't converted to an actual boolean https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/services.py#L136
16:20:28 mriedem which methinks will explode https://github.com/openstack/nova/blob/master/nova/objects/service.py#L151
16:20:32 sdague mriedem: yeh, that should be turned into a real boolean
16:20:45 sdague mriedem: agreed
16:20:55 mriedem alright, writing up a bug
16:24:35 openstackgerrit Maciej Kucia proposed openstack/nova master: [doc] Updated sqlalchemy URL in migrate readme https://review.openstack.org/472549
16:24:39 openstackgerrit Maciej Kucia proposed openstack/nova master: [doc] Updated sqlalchemy URL in migrate readme https://review.openstack.org/472549
16:25:28 mriedem https://bugs.launchpad.net/nova/+bug/1697497
16:25:28 openstack Launchpad bug 1697497 in OpenStack Compute (nova) "PUT /os-services/force-down won't work with non-boolean forced_down field" [Medium,Triaged]
16:26:46 openstackgerrit Gábor Antal proposed openstack/nova master: Transform aggregate.update_metadata notification https://review.openstack.org/460625
16:38:27 openstackgerrit Merged openstack/nova master: [placement] Add api-ref for aggregates https://review.openstack.org/470932
16:50:49 openstackgerrit Dan Smith proposed openstack/nova master: Remove old service version check for mitaka https://review.openstack.org/472395
17:03:08 mriedem dtp: fyi that i've got the PUT /os-services/{service_id} change for this https://review.openstack.org/#/c/464280/ done locally, just working on tests and docs changes for it
17:05:22 openstackgerrit Gábor Antal proposed openstack/nova master: Add json style checking for sample notifications https://review.openstack.org/443677
17:11:11 dansmith cdent: fwiw, I also think those need not be OVOs
17:11:14 dansmith and maybe not even Os
17:11:53 cdent I guess people like the O aspect because of enforcement?
17:13:56 dansmith the field type enforcement?
17:14:27 stvnoyes mriedem: johnthetubaguy: The live migrate rv is ready. I was able to successfully live migrate (with an attached volume) back and forth using the new cinder v3 api. Please take a look when you get a chance. thanks. https://review.openstack.org/#/c/463987/
17:14:32 dansmith perhaps, but we're pulling stuff out of a pretty rigid DB schema and will filter it through jsonschema validation on the sending side at least, so I'm not sure I really see the point
17:14:52 dansmith 's just me tho
17:15:53 cdent dansmith: well your thoughts on the matter are certainly much more well formed than mine, so ... congratulations!
17:26:09 dtp mriedem coolio
17:51:23 openstackgerrit Alex Szarka proposed openstack/nova master: Transform instance.exists notification https://review.openstack.org/403660
18:09:10 openstackgerrit Merged openstack/nova master: Remove translation of log messages https://review.openstack.org/466621
18:09:42 openstackgerrit Merged openstack/nova master: Convert additional disassociate tests to mock https://review.openstack.org/471408
18:16:16 openstackgerrit Jay Pipes proposed openstack/nova master: placement: produce set of alternative allocations https://review.openstack.org/473377
18:19:39 oomichi mriedem: re: forced_down: A nice catch, I agree that is a bug
18:20:59 efried Is that a football analogy?
18:21:18 mriedem turned over on downs?
18:21:38 oomichi now 20 parameters are using parameter_types.boolean, it would be nice to dig them from the same pov
18:21:53 mriedem oomichi: https://bugs.launchpad.net/nova/+bug/1697497 if you want to fix it
18:21:53 openstack Launchpad bug 1697497 in OpenStack Compute (nova) "PUT /os-services/force-down won't work with non-boolean forced_down field" [Medium,Triaged]
18:22:31 oomichi mriedem: if you don't mind, i will
18:23:34 oomichi mriedem: sdague: I am wondering the bug-fix requires microversion bump. strictly, maybe that requires it
18:23:59 mriedem oomichi: no, it's a 500 otherwise
18:24:03 sdague when did that get added in?
18:24:04 mriedem oomichi: start with the test and you'll see
18:24:10 mriedem sdague: 2.11 is liberty
18:24:11 sdague yeh, this feels like a backport
18:24:25 oomichi mriedem: cool, ok we can fix without bumping
18:24:26 mriedem o.vo will give you a ValueError
18:24:29 mriedem which would be a 500
18:53:09 openstackgerrit Merged openstack/nova master: XenAPI: Remove bittorrent.py which is already deprecated https://review.openstack.org/468273
18:53:43 openstackgerrit Merged openstack/nova master: nfs fix for xenial images https://review.openstack.org/329466
18:54:14 openstackgerrit Merged openstack/nova master: raise exception if create Virtuozzo container with swap disk https://review.openstack.org/397829
18:54:44 openstackgerrit Merged openstack/nova master: xen: pass Xen console in cmdline https://review.openstack.org/469826
18:55:13 openstackgerrit Merged openstack/nova master: Adopts keystoneauth with glance client. https://review.openstack.org/412634
19:17:50 openstackgerrit Matt Riedemann proposed openstack/nova master: DNM: Test service token in Glance https://review.openstack.org/419666
19:22:03 openstackgerrit Merged openstack/nova master: Use plain routes list for os-consoles, os-console-auth-tokens endpoint instead of stevedore https://review.openstack.org/462175
19:38:22 openstackgerrit Ken'ichi Ohmichi proposed openstack/nova master: Add bool_from_string for force-down action https://review.openstack.org/473564

Earlier   Later