Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-09-10
22:04:03 mriedem but the resize itself can fail
22:04:11 mriedem and we've done borked up the request spec
22:04:21 melwitt ok, so this is the analog of what we're doing there
22:04:26 mriedem arguably the revert thing (my patch) is much less risky
22:04:42 mriedem because on revert, we delete from the dest and restore on the source
22:04:43 melwitt yeah, just wanted to make sure I understand
22:05:01 mriedem in general, if we shit the bed on most nova operations, we require operator intervention....
22:05:35 melwitt yeah, I know. just saying that occurred to me looking at the change
22:05:57 mriedem yar matey
22:06:25 cfriesen would it make sense to put a "try" block around the actual revert-resize call and restore the "new" flavor if we get an exception?
22:07:08 melwitt well, the call down to compute isn't synchronous, I assume. /me checks
22:07:22 melwitt yeah it's a cast
22:07:40 melwitt so we can't know whether it failed or not
22:07:58 mriedem right
22:08:08 mriedem and if we fail in the compute, updating the request spec would be an up-call to the api db
22:08:12 mriedem which is a non-starter
22:08:21 melwitt I think it's fine, it's better than what we have now, and is not a new bork mode in resize land
22:08:33 melwitt I just wanted to ask before I +2 it
22:08:41 mriedem mmmm new bork mode
22:08:41 mriedem 2.0
22:08:46 cfriesen yeah, it's a cast
23:05:26 dansmith mriedem: that seems really odd to provide a parameter to a status check like that
23:05:39 dansmith it's never going to be error, right? just a warning that you can ignore
23:12:19 mriedem yes warning at most severe
23:12:26 mriedem i also just sent it to the ML
23:12:35 mriedem it's not a parameter to the CLI
23:12:39 mriedem it's a config option for a specific check
23:12:47 dansmith config option in nova.conf?
23:12:52 mriedem correct
23:13:02 dansmith that's kinda weird no?
23:13:57 mriedem weird in that, why don't people just run a cron to do the purge themselves?
23:23:54 openstackgerrit Merged openstack/python-novaclient master: Add missing options in CLI reference https://review.openstack.org/601349
#openstack-nova - 2018-09-11
00:39:39 litao__ morning
00:40:12 litao__ Hello , This is my patch https://review.openstack.org/#/c/592252/, pls review it in your spare time.
03:01:04 openstackgerrit Balazs Gibizer proposed openstack/nova master: DNM: Make nova tests pass with externalized placement https://review.openstack.org/596291
03:01:05 openstackgerrit Balazs Gibizer proposed openstack/nova master: customization https://review.openstack.org/601483
03:01:45 openstackgerrit Balazs Gibizer proposed openstack/nova master: DNM: Make nova tests pass with externalized placement https://review.openstack.org/596291
03:12:16 openstackgerrit Balazs Gibizer proposed openstack/nova master: DNM: Make nova tests pass with externalized placement https://review.openstack.org/596291
04:47:29 openstackgerrit Andreas Jaeger proposed openstack/nova master: Cleanup zuul.yaml https://review.openstack.org/601410
06:27:25 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient stable/rocky: Add missing options in CLI reference https://review.openstack.org/601499
06:42:12 openstackgerrit Takashi NATSUME proposed openstack/nova master: Adds view builders for keypairs controller https://review.openstack.org/347289
06:42:31 openstackgerrit Takashi NATSUME proposed openstack/nova master: Transform libvirt.error notification https://review.openstack.org/484851
06:48:20 openstackgerrit Andreas Jaeger proposed openstack/os-vif master: switch documentation job to new PTI https://review.openstack.org/601381
06:48:21 openstackgerrit Andreas Jaeger proposed openstack/os-vif master: add python 3.6 unit test job https://review.openstack.org/601382
06:48:22 openstackgerrit Andreas Jaeger proposed openstack/os-vif master: Cleanup zuul config file https://review.openstack.org/601504
07:02:48 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/601047
07:45:02 litao__ hello
09:10:47 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313
09:17:55 openstackgerrit Brin Zhang proposed openstack/nova master: Resource retrieving: add changes-before filter https://review.openstack.org/599276
09:18:58 openstackgerrit Merged openstack/nova master: Add an example to add more pci devices in nova.conf https://review.openstack.org/592243
10:17:03 openstackgerrit Artom Lifshitz proposed openstack/nova master: Ensure attachment cleanup on failure in driver.pre_live_migration https://review.openstack.org/587439
10:20:32 openstackgerrit Artom Lifshitz proposed openstack/nova master: Ensure attachment cleanup on failure in driver.pre_live_migration https://review.openstack.org/587439
10:21:44 openstackgerrit Brin Zhang proposed openstack/nova master: Resource retrieving: add changes-before filter https://review.openstack.org/599276
10:24:52 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717
11:28:57 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P)(WIP): support compute node resource provider update https://review.openstack.org/521041
11:43:28 openstackgerrit Vladyslav Drok proposed openstack/nova master: Fix resource tracker updates during instance evacuation https://review.openstack.org/600425
11:47:07 openstackgerrit Vladyslav Drok proposed openstack/nova master: Fix resource tracker updates during instance evacuation https://review.openstack.org/600425
12:39:13 openstackgerrit Luiz Benedito Aidar Gavioli proposed openstack/nova master: [DNM] Test Netapp CI triggering. https://review.openstack.org/601588
13:06:15 openstackgerrit Rui Zang proposed openstack/nova-specs master: Virtual non-volatile memory (or non-volatile dual in-line memory module 'NVDIMM') or in other words persistent memory devices have already been enabled in both QEMU and libvirt. This spec seeks the support for such devices in OpenStack Nova. The terms -- https://review.openstack.org/601596
13:10:28 openstackgerrit Rui Zang proposed openstack/nova-specs master: Virtual non-volatile memory (or non-volatile dual in-line memory module 'NVDIMM') or in other words persistent memory devices have already been enabled in both QEMU and libvirt. This spec seeks the support for such devices in OpenStack Nova. The terms -- https://review.openstack.org/601596
13:30:07 openstackgerrit Rui Zang proposed openstack/nova-specs master: supporting virtual NVDIMM devices https://review.openstack.org/601596
13:52:20 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge security groups extension response into server view builder https://review.openstack.org/585475
13:57:52 openstackgerrit Artom Lifshitz proposed openstack/nova master: Ensure attachment cleanup on failure in driver.pre_live_migration https://review.openstack.org/587439
14:43:32 openstackgerrit Georg Hoesch proposed openstack/nova master: handle logfiles for tcp-based consoles https://review.openstack.org/575735
15:36:26 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Improve the description of optional arguments https://review.openstack.org/601620
15:55:50 openstackgerrit Balazs Gibizer proposed openstack/nova master: DNM: Make nova tests pass with externalized placement https://review.openstack.org/596291
16:00:29 mriedem gmann: do you have an etherpad from the api subteam for generic compute API cleanups we could do in a single microversion?
16:00:40 mriedem we could throw https://bugs.launchpad.net/openstack-publiccloud-wg/+bug/1790879 into that list of we have one
16:00:40 openstack Launchpad bug 1790879 in OpenStack Public Cloud WG "Different error codes from different projects on quota exceeded" [Undecided,New] - Assigned to Tobias Rydberg (tobberydberg)
16:01:06 gmann mriedem: i created one but need to find. i will add this item also
16:02:14 mriedem thanks
16:20:29 mriedem dansmith: melwitt: shelve on stop bp with an api idea https://blueprints.launchpad.net/nova/+spec/shelve-on-stop
16:22:58 melwitt mriedem: on the surface, that seems weird to effectively alias one of our APIs to another of our APIs, but maybe it would be worth it
16:23:17 melwitt since nobody understands shelve
16:28:14 dansmith it would be confusing to people expecting to do a stop and end up in shelved state I think
16:28:29 dansmith especially if they are using stop to do a quick boot to apply changes or swap something like VIP
16:28:37 dansmith because unshelve is way slower than start
16:28:39 dansmith so I dunno.
16:31:45 melwitt yeah, that's true. state won't say STOPPED if nova.conf is set to do the shelve thing. by default though, users would get the current stop behavior of no shelve
16:40:30 mriedem yeah i know, hence the defaults for backward compat and possible override for users that actually know the difference and care
16:40:37 mriedem plus it would be a microversion so you'd have to opt into it
16:40:43 mriedem anyway just an idea
16:40:49 mriedem i know you nova people like to say no to everything
16:43:15 openstackgerrit Balazs Gibizer proposed openstack/nova master: Consumer gen: remove_provider_from_instance_allocation https://review.openstack.org/591784
16:51:24 openstackgerrit Ken'ichi Ohmichi proposed openstack/nova master: Merge security groups extension response into server view builder https://review.openstack.org/585475
16:58:18 med_ here: some donuts outside of the atrium Ops room in PTG Denver
16:58:37 med_ Not up to Chet's voodoo standards but Krispy Kreme
17:00:55 stephenfin bauzas: Wanna review https://review.openstack.org/#/c/601415/ and predecessors when you have a chance?
17:03:44 openstackgerrit Balazs Gibizer proposed openstack/nova master: DNM: Make nova tests pass with externalized placement https://review.openstack.org/596291
17:03:45 openstackgerrit Balazs Gibizer proposed openstack/nova master: DNM: Make nova tests pass with externalized placement https://review.openstack.org/601641
17:21:53 mriedem dansmith: does this capture what we talked about wrt the problem in the tenant isolation filter? https://bugs.launchpad.net/openstack-publiccloud-wg/+bug/1771523/comments/7
17:21:53 openstack Launchpad bug 1771523 in OpenStack Public Cloud WG "Dedicated Host" [High,Confirmed] - Assigned to Tobias Rydberg (tobberydberg)
17:29:10 dansmith mriedem: aye
18:43:38 openstackgerrit Merged openstack/python-novaclient master: import zuul job settings from project-config https://review.openstack.org/601383
18:56:45 openstackgerrit Merged openstack/python-novaclient master: switch documentation job to new PTI https://review.openstack.org/601384
19:09:27 openstackgerrit Merged openstack/python-novaclient master: add python 3.6 unit test job https://review.openstack.org/601385
19:09:29 openstackgerrit Merged openstack/python-novaclient master: add lib-forward-testing-python3 test job https://review.openstack.org/601386
19:09:31 openstackgerrit Merged openstack/python-novaclient master: Cleanup zuul.yaml https://review.openstack.org/601415
20:09:30 openstackgerrit Sylvain Bauza proposed openstack/nova-specs master: Proposes NUMA topology with RPs https://review.openstack.org/552924
20:42:49 openstackgerrit Merged openstack/nova master: import zuul job settings from project-config https://review.openstack.org/601370

Earlier   Later