Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-03-28
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
07:47:11 openstackgerrit Rajesh Tailor proposed openstack/nova stable/pike: Allow force-delete even if task_state is not None https://review.openstack.org/543345
07:47:47 Spazmotic Morning bauzas
08:21:01 gibi morning folks
08:22:35 jichen hi sahid: regarding https://review.openstack.org/#/c/523387/, for singleton, if we want to keep
08:22:37 jichen it, we need take something like 'http://127.0.0.1' as array key, do you think that's reasonable ?
08:54:00 sahid jichen: oh i see what you mean, that's because of the parameters
08:54:30 jichen sahid; yes, so that's why I want to avoid the singleton , you ok with that?

Earlier   Later