Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-03-14
22:17:23 mriedem bmace: let's chat here,
22:17:40 mriedem bmace: there is a weekly nova/cinder meeting http://eavesdrop.openstack.org/#Cinder,_Nova
22:17:55 mriedem bmace: for anyone interested in the multiattach stuff, that's a good meeting to be in for latest status and what needs attention or help
22:18:19 bmace mriedem: thanks much
22:18:46 mriedem bmace: we're tracking review priorities per subteam in here https://etherpad.openstack.org/p/pike-nova-priorities-tracking
22:19:06 mriedem cinder stuff is at L75
22:19:21 mriedem i'm trying to keep as much of it contained to per-week increments as possible,
22:19:47 mriedem so right now the priorities i believe are getting a cinderclient release done so nova can determine the min/max versions from the cinder api,
22:20:03 mriedem getting some data model changes into the nova block_device_mappings table (lyarwood's patches),
22:20:16 mriedem and reviewing johnthetubaguy's spec and making sure it looks ok
22:20:49 mriedem so eyes on the spec would be good - the more people looking the better. i don't know how familiar your people are with the bdm attach/detach flows in nova, or the compute manager and block_device code
22:20:54 mriedem it gets a bit hairy
22:22:56 bmace mriedem: i have heard some stories from the folks looking at it about the hair ;)
22:25:41 smcginnis mriedem: cinderclient is released. I think we need to raise the minimum version though.
22:26:00 smcginnis mriedem: But new deployments are likely getting the latest and goodest.
22:27:45 openstackgerrit Ed Leafe proposed openstack/nova master: WIP - Combine the two auth.py files https://review.openstack.org/444521
22:27:45 openstackgerrit Ed Leafe proposed openstack/nova master: WIP - move urlmap.py https://review.openstack.org/444524
22:27:46 openstackgerrit Ed Leafe proposed openstack/nova master: WIP - Move wsgi.py https://review.openstack.org/444523
22:27:46 openstackgerrit Ed Leafe proposed openstack/nova master: WIP - Move __init__.py https://review.openstack.org/444522
22:27:47 openstackgerrit Ed Leafe proposed openstack/nova master: WIP - move api_version_request https://review.openstack.org/445669
22:27:47 openstackgerrit Ed Leafe proposed openstack/nova master: WIP - Move common.py https://review.openstack.org/445668
22:27:50 openstackgerrit Ed Leafe proposed openstack/nova master: WIP - move compute directory https://review.openstack.org/445670
22:27:51 openstackgerrit Ed Leafe proposed openstack/nova master: WIP - Move the test fakes https://review.openstack.org/445671
22:32:14 mriedem smcginnis: ok yeah we need the g-r bump then for ildikov's patch to check the max available version in nova
22:45:29 openstackgerrit Anusha Unnam proposed openstack/nova master: POC: add descriptions to server tags https://review.openstack.org/434842
22:46:03 EmilienM mriedem: FYI I abandon https://review.openstack.org/#/c/428424/ - you win
22:57:15 openstackgerrit Samantha Blanco proposed openstack/nova master: [WIP] Make flavor-rxtx policy more granular https://review.openstack.org/444036
#openstack-nova - 2017-03-15
00:15:08 openstackgerrit christopher.uhler proposed openstack/nova master: Libvirt volume driver for Veritas HyperScale https://review.openstack.org/443951
00:40:52 mriedem edleafe: dansmith: on https://review.openstack.org/#/c/441544/7/nova/objects/fields.py did we decide to not check before adding the CUSTOM_ prefix if it were already in the norm_name?
00:41:02 mriedem i believe that's come up before
00:42:08 dansmith mriedem: we're depending on the drivers to return legit classes, yes
00:52:08 mriedem so, we could still create CUSTOM_CUSTOM_BAREMETAL_GOLD then
00:52:17 openstackgerrit Dan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info https://review.openstack.org/439891
00:52:17 openstackgerrit Dan Smith proposed openstack/nova master: Teach HostAPI about cells https://review.openstack.org/442162
00:52:18 openstackgerrit Dan Smith proposed openstack/nova master: Sort CellMappingList.get_all() for safety https://review.openstack.org/443174
00:52:19 openstackgerrit Dan Smith proposed openstack/nova master: Clean up ClientRouter debt https://review.openstack.org/444487
00:52:19 openstackgerrit Dan Smith proposed openstack/nova master: Add get_by_instance_uuids() to InstanceMappingList https://review.openstack.org/443292
00:52:20 openstackgerrit Dan Smith proposed openstack/nova master: Remove Mitaka-era service version check https://review.openstack.org/442861
00:52:20 openstackgerrit Dan Smith proposed openstack/nova master: Make server_groups determine deleted-ness from InstanceMappingList https://review.openstack.org/443293
00:52:21 openstackgerrit Dan Smith proposed openstack/nova master: Add workaround to disable group policy check upcall https://review.openstack.org/442736
00:52:21 openstackgerrit Dan Smith proposed openstack/nova master: Make conductor ask scheduler to limit migrates to same cell https://review.openstack.org/438025
00:52:22 openstackgerrit Dan Smith proposed openstack/nova master: Avoid lazy-loading projects during flavor notification https://review.openstack.org/445697
00:52:31 mriedem and it wouldn't be the drivers fault, it could be the operator set the node.resource_class value to be 'CUSTOM_BAREMETAL_GOLD'
00:53:07 dansmith right, but that's what we should do, IMHO
00:53:17 dansmith otherwise there's this odd ambiguity,
00:53:44 dansmith and the node resource class is an ironic construct, not specifically "this is (or may be) the exact resource class in placement"
00:54:40 mriedem so ironic's API should then return a 400 if i try to set node.resource_class to anything with the CUSTOM_ prefix?
00:55:01 dansmith that's up to them, but I'd expect not
00:55:59 dansmith and since they haven't placed those restrictions thus far (any of the ones required to be a resource class in placement),
00:56:23 dansmith we should just treat that as a thing, prefix with CUSTOM_ and normalize as we need)
00:56:41 dansmith like, maybe they let the resource class include spaces, they clearly allow lowercase, maybe punctuation, etc
00:56:46 dansmith and that's cool
01:00:10 mriedem ok so CUSTOM_CUSTOM_BAREMETAL_GOLD just might be a thing you end up with, and if so, oops, but it doesn't break
01:00:37 dansmith right
01:00:58 mriedem not sure i want to start thinking about what it would take to unwind that if you wanted to fix it,
01:01:04 mriedem across both services
01:01:13 dansmith I don't think it's a big deal actually, but,
01:01:23 dansmith when they start reporting into placement, they can generate and handle their classes however they want
01:03:12 mriedem ok
01:03:21 mriedem i've left comments in the code parts, will do tests later
01:57:11 openstackgerrit Danfly proposed openstack/nova master: Completely remove mox from unit/network/test_linux_net.py https://review.openstack.org/445504
02:19:44 openstackgerrit Zhenyu Zheng proposed openstack/nova-specs master: Send out notifications when instance tags changed https://review.openstack.org/429569
02:47:36 openstackgerrit Wei Cao proposed openstack/nova master: vmware: Change the VM Create spec https://review.openstack.org/444119
03:09:03 alex_xu gmann: johnthetubaguy some test https://review.openstack.org/445361
03:46:51 openstackgerrit Zhenyu Zheng proposed openstack/nova-specs master: Repropose tag-instance-when-boot https://review.openstack.org/415315
04:28:24 openstackgerrit Hieu LE proposed openstack/nova master: Completely remove mox from objects/test_instance.py https://review.openstack.org/343465
05:08:44 openstackgerrit Wei Cao proposed openstack/nova master: vmware: Change the VM Create spec https://review.openstack.org/444119
05:16:19 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/443357
05:25:42 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add api-ref for filter/sort whitelist https://review.openstack.org/421760
05:42:12 openstackgerrit Sivasathurappan Radhakrishnan proposed openstack/nova master: Permit Live Migration of Rescued Instances https://review.openstack.org/308198
06:55:20 openstackgerrit Sergey Nikitin proposed openstack/nova master: Standardization of VM diagnostics info API. https://review.openstack.org/355540
07:08:33 openstackgerrit Dinesh Bhor proposed openstack/nova master: Use HostAddressOpt for opts that accept IP and hostnames https://review.openstack.org/445782
07:11:32 openstackgerrit Sivasathurappan Radhakrishnan proposed openstack/nova-specs master: Live Migration of Rescued Instances https://review.openstack.org/347161
08:10:02 jichen alex_xu, hi have time for a quick talk?
08:10:29 alex_xu jichen: yea, sure, please
08:10:47 openstack Launchpad bug 1670978 in OpenStack Compute (nova) "most of extended server attributes returned in 2.3 api versions should not require admin role" [Undecided,New] - Assigned to jichenjc (jichenjc)
08:10:47 jichen https://bugs.launchpad.net/nova/+bug/1670978, this need a spec ,right? if any changes might need to be done
08:11:12 alex_xu jichen: 不需要
08:11:16 alex_xu sorry
08:11:19 alex_xu jichen: it needn't
08:11:26 jichen why ?
08:11:46 jichen API changes no spec needed any more ?
08:11:58 alex_xu jichen: it is only about change a default value of a policy rule from admin-only to owner_or_admin
08:12:54 jichen um... part of the return value affected ,I thought if we affect output format , this is API change?
08:13:33 alex_xu jichen: yea, that always confuse us, but actually, that can't be resolved by microversion, it should be resolved by capabilities API
08:14:09 alex_xu jichen: here is https://review.openstack.org/#/c/386555/
08:14:12 jichen alex_xu: so this bug might need the API-WG discussion on cap talk settle down then continue?
08:14:24 alex_xu jichen: i think no...
08:14:29 jichen alex_xu: yeah, I knew this discussion
08:14:41 jichen ok, I will give another thought and see how to proceed this
08:14:44 alex_xu johnthetubaguy: ^ I think we needn't waiting for cap API, right?
08:15:19 alex_xu jichen: thanks, if johnthetubaguy also ensure this needn't waiting for cap API, then ping me when the patch ready
08:15:32 jichen ok
08:15:42 jichen another question
08:15:54 alex_xu jichen: yea, please
08:16:41 jichen https://review.openstack.org/307675, it's abandoned but this seems discussed before and a potential item? any discussion on PTG about it ?
08:17:15 jichen Change subject: Make API errors conform to API-WG schema
08:17:16 alex_xu jichen: there is no discussion about this in the PTG
08:17:40 jichen ok... I went through the ethpad and didn't find anything....
08:17:58 alex_xu jichen: yea, I didn't remember we bring that idea into the api meeting before PTG also

Earlier   Later