Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-10-13
22:42:21 fried_rice noyce
23:19:49 openstackgerrit Merged openstack/nova master: Split get_allocations_for_instance() into useful bits https://review.openstack.org/511306
23:20:34 openstackgerrit Merged openstack/nova master: Revert allocations by migration uuid https://review.openstack.org/498949
23:21:36 openstackgerrit Merged openstack/nova master: Refactor resource tracker to account for migration allocations https://review.openstack.org/506419
23:44:30 superdan holy crap, patches of mine merged
23:44:31 superdan amazing
#openstack-nova - 2017-10-14
03:20:38 openstackgerrit Merged openstack/nova master: rp: rework AllocList.get_all_by_consumer_id() https://review.openstack.org/509035
09:33:29 openstackgerrit garyk proposed openstack/nova master: Add debug information to metadata requests https://review.openstack.org/511895
12:09:30 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif stable/newton: Updated from global requirements https://review.openstack.org/373293
14:39:26 Guest47 hi
14:40:35 Guest47 Till Ocata release 'multipath' functionality was working fine.. have placed 'volume_use_multipath=True' in nova.con
14:40:49 Guest47 nova.conf* file
14:43:26 Guest47 but from pike onwards, even if i placed 'volume_use_multipath=True'.. it failed to create multiple vluns after attaching volume to instance
14:44:53 Guest47 After debugging, I have found that nova is passing 'multipath=False' instead of 'True'
14:46:13 Guest47 connector['multipath']=False is passed to cinder's driver 'initialize_connection'
14:47:16 Guest47 further analysing on this, i found that nova is not picking up the 'volume_use_multipath=True' value from nova.conf
14:47:40 Guest47 which is in 'libvirt' section in nova.conf
16:08:27 openstackgerrit LIU Yulong proposed openstack/nova master: Enable reset keypair while rebuilding instance https://review.openstack.org/379128
#openstack-nova - 2017-10-15
19:01:51 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Network bandwidth resource provider https://review.openstack.org/502306
22:01:15 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Intel Fortville Dynamic Device Personalization (DDP) https://review.openstack.org/503001
22:28:08 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955
22:28:23 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
22:28:45 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608
22:28:57 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483
22:30:42 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732
22:31:08 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix test_get_volume_config method https://review.openstack.org/489467
#openstack-nova - 2017-10-16
01:02:07 openstackgerrit Yikun Jiang proposed openstack/nova master: Update Instance action's updated_at when action event updated. https://review.openstack.org/507473
03:20:34 openstackgerrit Zhenyu Zheng proposed openstack/nova master: block_device_mapping_v2.bus_type is missing from api-ref https://review.openstack.org/499488
03:23:27 openstackgerrit Jianghua Wang proposed openstack/nova master: rt: Implement XenAPI get_inventory() method https://review.openstack.org/339204
04:44:24 openstackgerrit Hironori Shiina proposed openstack/nova-specs master: Ironic: Resize and cold migration support https://review.openstack.org/449155
06:16:20 vivsoni Hi, when 'volume_use_multipath' is set to 'True' in nova.conf 'libvirt' section, the get_connector_properties() is not picking up this value..
06:16:50 vivsoni it always takes default hard-coded value from libvirt.py
07:03:26 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach interface https://review.openstack.org/512190
07:03:59 priya_ hi , Could somebody review the commit - https://review.openstack.org/#/c/508533/
07:14:40 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732
07:22:12 priya_ hi , Could somebody review the commit - https://review.openstack.org/#/c/508533/
08:16:29 gibi stephenfin: hi! Did you intentionally leave out the idamshift manpage from the latest version of https://review.openstack.org/#/c/502105 ?
08:17:03 stephenfin gibi: I did, yes. mikal has removed that in another patch and I didn't want to introduce a conflict
08:17:54 belmoreira I'm trying to change the default nova log level to ERROR. After some googling I can't find a solution... errrr... How can I do this? any pointer is appreciated
08:19:33 gibi stephenfin: ahh OK, then I +2 on the manpage patch
08:20:24 openstackgerrit Merged openstack/nova master: Move test_uuid_sentinels to NoDBTestCase https://review.openstack.org/507253
08:34:00 priya_ belmoreira - the default log level can be set in nova.conf
08:34:54 priya_ hi , Could somebody review the commit - https://review.openstack.org/#/c/508533/
08:46:55 openstackgerrit jichenjc proposed openstack/nova master: Add create inventories doc for placement https://review.openstack.org/512215
08:48:42 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach interface https://review.openstack.org/512190
09:21:32 openstackgerrit Viktor Bass proposed openstack/nova master: doc: Fix command output in scheduler document https://review.openstack.org/511839
09:22:54 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Update "The nova Shell Utility" in the user guide https://review.openstack.org/512224
09:24:27 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Update "The nova Shell Utility" in the user guide https://review.openstack.org/512224
09:38:51 openstackgerrit Yikun Jiang proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406
09:50:58 openstackgerrit Yikun Jiang proposed openstack/nova master: [WIP]Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406
10:04:53 openstackgerrit jichenjc proposed openstack/nova master: check query param for server groups function https://review.openstack.org/500347
10:05:09 openstackgerrit jichenjc proposed openstack/nova master: remove CONF.vendordata_driver https://review.openstack.org/501510
11:01:25 efried Good morning nova
11:19:05 efried alex_xu yt?
11:26:50 openstackgerrit Balazs Gibizer proposed openstack/nova master: fix cleaning up evacuated instances https://review.openstack.org/510938
11:41:37 openstackgerrit Yikun Jiang proposed openstack/nova master: [WIP]Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406
11:50:24 openstackgerrit Yikun Jiang proposed openstack/nova master: [WIP]Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406
11:58:47 openstackgerrit Merged openstack/nova master: TrivialFix: Fix the incorrect test case https://review.openstack.org/509758
12:24:14 priya_ hi , Could somebody review the commit - https://review.openstack.org/#/c/508533/
12:25:54 openstackgerrit Merged openstack/nova master: Remove system_metadata loading in Instance._load_flavor https://review.openstack.org/508357
12:25:59 openstackgerrit Merged openstack/nova master: Don't use mock.patch.stopall https://review.openstack.org/507527
12:26:40 openstackgerrit Merged openstack/nova master: nova-manage map_instances is not using the cells info from the API database https://review.openstack.org/510844
12:26:46 openstackgerrit Merged openstack/nova master: Filter out stale migrations in resource audit https://review.openstack.org/339715
12:26:54 openstackgerrit Merged openstack/nova master: Refactor init_instance:resume_guests_state https://review.openstack.org/349582
12:27:29 efried stephenfin FYI, I'm spinning up another rev of that spec, in case you wanted to, I don't know, make fun of my hair or something.
12:30:07 alex_xu efried: hi, i'm here
12:30:30 efried alex_xu Hi, I was going to ask you something about aggregates wrt the granular RP syntax spec...
12:30:50 alex_xu efried: yea
12:31:48 efried alex_xu It was about dansmith's comment here: https://review.openstack.org/#/c/510244/4/specs/queens/approved/granular-resource-requests.rst@45
12:32:50 efried IIUC, the way you're implementing it, if I ask for a particular trait, you'll succeed even if the trait isn't on the compute node?
12:33:04 efried alex_xu ...as long as it's on an aggregate?
12:33:57 alex_xu yes, but the compute node and shared storage pool in the same aggregate
12:34:29 efried alex_xu This actually brings up an interesting set of test cases to cover. If I ask for storage-related traits X and Y; and my compute node has storage with trait X but not Y; and my shared storage has trait Y but not X - what happens?
12:35:01 efried I should *not* get that host/storage as a candidate.
12:35:30 alex_xu efried: first, we don't have the case the comput enode have local storage and shared storage at same time
12:35:32 efried But I suspect with the logic you're currently implementing, it might erroneously return that host.
12:35:44 efried alex_xu What do you mean?
12:36:02 efried You mean nobody has done it yet? Or we don't support it at all?
12:36:04 alex_xu efried: yea, that's true, my implementation doesn't work for that
12:36:16 alex_xu efried: we don't support it at all
12:36:28 efried Okay.
12:36:55 efried I mean, I get the impression that we're not trying to get shared RPs "working" in Queens, but I don't really know what that means in terms of what code we're actually going to merge around aggregates and such.
12:37:30 efried Because ultimately it's not just about shared storage. There could be other kinds of shared RPs, and we would probably want that scenario to work properly.
12:37:45 alex_xu efried: yea, but the current implementation of 'GET /allocation_candidates' support a buggy version shared RPs...
12:37:55 efried right, I saw that too :(
12:38:35 alex_xu efried: yea, I'm also think about, but I'm still trying to find a similar case in the real world for that
12:38:58 alex_xu efried: thanks for point that out, I will keep that in mind
12:39:05 efried Sure thing.
12:39:14 efried alex_xu Thanks for the review, as well. I'm addressing your comments now.
12:39:22 alex_xu efried: np
12:43:14 alex_xu efried: for the case you pointed out, I think the code should generate two candidates, 1. node and local storage, 2. node and shared strorage. The current implementation without traits doesn't work for this case also.
12:44:43 alex_xu I guess Jay may say we can't have duplicated resource class in the same aggregate
12:45:26 efried alex_xu I would be very surprised if he said that
12:45:56 efried mm, maybe not.
12:46:01 efried I need to noodle that through.
12:46:38 efried But for the case I pointed out, I would expect *no* candidates. Because neither the local nor the shared storage has *both* X and Y.
12:47:11 efried Unless you're saying that the code as currently written would generate two candidates. I agree. And I think that's not correct.
12:48:38 alex_xu efried: I mean the currently written with the same request but without required traits, there should be two candidates
12:48:56 efried Oh, yeah, without traits involved, sure.
13:12:29 openstackgerrit sean mooney proposed openstack/nova-specs master: Use neutron's new port binding API https://review.openstack.org/375580

Earlier   Later