Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-03-28
22:29:44 openstackgerrit Eric Fried proposed openstack/nova master: PowerVM Driver: Network interface attach/detach https://review.openstack.org/546813
22:29:46 efried mriedem: ight. ^
22:29:57 mriedem thanks
22:29:59 efried My daughter cooked me a steak. I'm out.
22:30:09 esberglu efried: tx
22:30:26 melwitt mgagne: I interpret it the same way. so, the source material for this API was intended to be able to change things between versions and break compat. it will be interesting to see how cloud-init is written and whether it would be insulated to such an API change. (and that's aside from all of the other things people may have written to consume the metadata API)
22:30:55 mikal cloud-init has heavy contributions from amazon, so we're probably ok then
22:31:14 mikal Although the openstack implementation in cloud-init probably doesn't get much love from amazon
22:31:25 melwitt yeah, that's my concern
22:31:28 mikal I do think asking someone to check cloud-init as part of the spec review is reasonable
22:31:39 mgagne melwitt: supported versions are hardcoded: https://git.launchpad.net/cloud-init/tree/cloudinit/sources/helpers/openstack.py?h=ubuntu/xenial#n44
22:32:09 mikal LOL, they're quite far behind.
22:32:35 mikal Unless xenial is shipping an old version?
22:32:36 mgagne that's for xenial, should check for latest
22:32:55 mgagne but it supports network_data.json so I'm happy =)
22:33:41 mgagne same for latest version it seems: https://git.launchpad.net/cloud-init/tree/cloudinit/sources/helpers/openstack.py?h=ubuntu/devel#n44
22:33:42 mikal https://github.com/cloud-init/cloud-init/blob/master/cloudinit/sources/helpers/openstack.py
22:33:46 mikal Trunk is just as far behind
22:34:08 mikal I guess maybe they expect us to be keeping that stuff up to date
22:34:18 mikal That means no vendordata v2 for anyone in cloud-init land for example
22:34:23 mgagne up to date => support older versions ?
22:34:34 mgagne if nobody asks ¯\_(ツ)_/¯
22:34:56 mgagne had to ask for network_data.json and contribute, wasn't easy
22:37:54 dansmith mikal: melwitt mgagne: we've not done non-additive changes in the past, and I don't really see any reason to make that change here.. we can totally do it additive-ly
22:38:20 dansmith we could re-audit all that stuff and make those checks more explicit about version spans (i.e start doing a thing in X, stop doing it in Y) but I don't see the point
22:38:46 dansmith yes, anything consuming latest is playing with fire, but.. shell scripts
22:39:05 mgagne dansmith: sure, just want to make sure we don't force ourselves to abide ourselves to some non-written rules which happens to have no reasons to exist
22:39:54 mgagne I updated spec to keep backward compat
22:43:25 melwitt yeah, ip_address and ip_addresses would be redundant but maybe that's not that big of a deal
22:43:45 melwitt or, ip_address would be redundant
22:52:30 openstackgerrit Michael Still proposed openstack/nova master: Start untangling network utilities. https://review.openstack.org/557542
22:52:31 openstackgerrit Michael Still proposed openstack/nova master: Move two more generic network utilities to a move obvious place. https://review.openstack.org/557543
22:52:42 mikal melwitt: ^--- that's the sort of untangling I was talking about
22:53:54 melwitt ah, okay. thanks for getting that started
22:54:36 mikal melwitt: some of that makes it more apparently that there is some privsep to do in nova.network, event if nova-net is nuked from orbit
22:54:44 mikal melwitt: which wasn't as obvious before
22:55:22 melwitt mikal: so that means, some privsep work is needed to support the utils that are not really part of nova-net?
22:55:51 mikal melwitt: yes, as we split out the bits of nova-net that we need post nova-net, we will find things that need to be privsep'ed
22:56:04 mikal melwitt: none of them look hard to do, but the previous assumption was that all of nova-net was just getting deleted
22:56:07 mikal melwitt: which is not true
22:56:15 melwitt mikal: understood
22:56:43 mikal melwitt: so, these ones got the remove-nova-network bp tag, but could have had the privsep one if we'd wanted to, they're all tangled up together
22:57:04 mikal I don't think you can tag two blueprints. Maybe you can?
22:57:44 melwitt I think you can, but from what I can tell, they make more sense under nova-net removal being that they're decoupling things that need to remain after removing nova-net
22:57:52 mikal Yeah
22:58:04 mikal I also don't care how they're tagged as long as they get reviewed one day
22:58:12 melwitt that work would have to be done anyway to remove nova-net, even without privsep
22:58:16 mikal I'mma just going to keep rebasing them and smashing the gate until someone gives in and reviews them
22:58:18 melwitt yeah, same
22:58:46 mikal Given we're up to 13 interrelated patches so far, and that list will grow
22:58:57 openstackgerrit Dan Smith proposed openstack/nova master: Make get_allocation_candidates() honor aggregate restrictions https://review.openstack.org/547990
22:58:57 openstackgerrit Dan Smith proposed openstack/nova master: Add an index on aggregate_metadata.value https://review.openstack.org/555851
22:58:58 openstackgerrit Dan Smith proposed openstack/nova master: Add AggregateList.get_by_metadata() query method https://review.openstack.org/544728
22:58:58 openstackgerrit Dan Smith proposed openstack/nova master: Add require_tenant_aggregate request filter https://review.openstack.org/545002
22:58:59 openstackgerrit Dan Smith proposed openstack/nova master: WIP: Documentation for tenant isolation with placement https://review.openstack.org/557490
22:58:59 openstackgerrit Dan Smith proposed openstack/nova master: WIP: Honor availability_zone hint via placement https://review.openstack.org/546282
#openstack-nova - 2018-03-29
00:40:36 openstackgerrit Michael Still proposed openstack/nova master: Move IVS VIF plugging into the new utilities module. https://review.openstack.org/557552
01:21:54 Spaz-Home Annnnd weekend over
01:29:57 openstackgerrit Jake Yip proposed openstack/nova master: Add --before option to db archive_deleted_rows https://review.openstack.org/556751
01:37:10 openstackgerrit Zhenyu Zheng proposed openstack/nova-specs master: Allow abort live migrations in queued status https://review.openstack.org/536722
01:37:11 openstackgerrit Merged openstack/nova stable/queens: Change compute mgr placement check to region_name https://review.openstack.org/555901
01:47:42 openstackgerrit jichenjc proposed openstack/nova master: WIP: remove Ec2 object https://review.openstack.org/557150
02:14:58 openstackgerrit jichenjc proposed openstack/nova master: WIP: remove ec2 in service and cmd https://review.openstack.org/556778
02:14:59 openstackgerrit jichenjc proposed openstack/nova master: WIP: remove Ec2 object https://review.openstack.org/557150
02:14:59 openstackgerrit jichenjc proposed openstack/nova master: WIP: remove db ec2 https://review.openstack.org/557572
02:56:23 openstackgerrit jichenjc proposed openstack/nova master: Move update_task_state out of try/except https://review.openstack.org/557152
03:00:50 openstackgerrit Jianle He proposed openstack/nova-specs master: Back up volume-backed-instance https://review.openstack.org/530214
03:01:22 openstackgerrit Tetsuro Nakamura proposed openstack/nova-specs master: Fix endpoint URI /allocation_requests https://review.openstack.org/557580
03:07:32 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add host/hostId to instance action events API https://review.openstack.org/557145
03:07:59 openstackgerrit Merged openstack/nova stable/queens: Allow force-delete even if task_state is not None https://review.openstack.org/543343
03:12:22 openstackgerrit Merged openstack/nova stable/queens: Ensure the JSON-Schema covers the legacy v2 API https://review.openstack.org/543490
03:28:52 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add host/hostId to instance action events API https://review.openstack.org/557145
03:31:19 openstackgerrit Merged openstack/nova stable/queens: [placement] Add sending global request ID in get https://review.openstack.org/543116
03:31:31 openstackgerrit Merged openstack/nova stable/queens: docs: Disable smartquotes https://review.openstack.org/554188
03:31:37 openstackgerrit Merged openstack/nova stable/queens: Return 400 when compute host is not found https://review.openstack.org/550661
03:47:16 melwitt tonyb: thanks for all the stable reviews
03:48:53 tonyb melwitt: np sorry I'm so sporadic
03:49:03 melwitt no worries
03:49:52 openstackgerrit Merged openstack/nova stable/pike: Remove osprofiler tests https://review.openstack.org/548886
03:57:53 openstackgerrit Merged openstack/nova stable/pike: Handle spawning error on unshelving https://review.openstack.org/548526
03:58:33 openstackgerrit Merged openstack/nova stable/pike: libvirt: mask InjectionInfo.admin_pass https://review.openstack.org/548312
04:12:12 openstackgerrit Tetsuro Nakamura proposed openstack/nova-specs master: Fix endpoint URI /allocation_requests https://review.openstack.org/557580
04:34:09 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers.inc https://review.openstack.org/529520
05:48:45 openstackgerrit Bhagyashri Shewale proposed openstack/nova-specs master: Disallow rotation parameter 0 for 'createBackup' API https://review.openstack.org/511825
05:53:16 takashin
06:16:31 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/548772
06:22:41 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313
06:26:33 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313
06:48:04 openstackgerrit Tetsuro Nakamura proposed openstack/nova stable/queens: add check before adding cpus to cpuset_reserved https://review.openstack.org/557621
06:50:18 openstackgerrit Tetsuro Nakamura proposed openstack/nova stable/pike: add check before adding cpus to cpuset_reserved https://review.openstack.org/557622
06:57:04 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387
06:57:04 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658
06:57:05 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240
06:57:05 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340
06:57:06 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344
07:22:39 openstackgerrit jichenjc proposed openstack/nova master: Move update_task_state out of try/except https://review.openstack.org/557152
07:25:11 openstackgerrit jichenjc proposed openstack/nova master: Fix issue for pep8 on py3 https://review.openstack.org/557633
07:36:45 openstackgerrit do3meli proposed openstack/nova-specs master: Adds resize on shared storage without ssh keys https://review.openstack.org/551927
07:45:49 bauzas good morning stackers
07:46:33 openstackgerrit licanwei proposed openstack/nova master: Make nova-manage capable of syncing all cell databases https://review.openstack.org/519275

Earlier   Later