Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-09-10
20:07:22 openstackgerrit Andreas Jaeger proposed openstack/nova master: import zuul job settings from project-config https://review.openstack.org/601370
20:07:23 openstackgerrit Andreas Jaeger proposed openstack/nova master: switch documentation job to new PTI https://review.openstack.org/601371
20:07:25 openstackgerrit Andreas Jaeger proposed openstack/nova master: add python 3.6 unit test job https://review.openstack.org/601372
20:07:26 openstackgerrit Andreas Jaeger proposed openstack/nova master: Cleanup zuul.yaml https://review.openstack.org/601410
20:17:17 openstackgerrit Chris Dent proposed openstack/nova master: DNM: Use external placement in nova-next https://review.openstack.org/601412
20:17:57 openstackgerrit Merged openstack/nova master: Remove redundant image GET call in _do_rebuild_instance https://review.openstack.org/600260
20:18:35 openstackgerrit Merged openstack/nova stable/queens: Fix cancel_all_events event name parsing https://review.openstack.org/592086
20:18:42 openstackgerrit Merged openstack/nova master: Document differences and similaries between extra specs and hints https://review.openstack.org/581410
20:25:36 openstackgerrit Andreas Jaeger proposed openstack/python-novaclient master: Cleanup zuul.yaml https://review.openstack.org/601415
20:31:17 mriedem let it be known: upgrade room said nova does the right thing with online data migrations and other projects don't
20:31:30 mriedem and operators want other projects to do online data migrations
20:33:39 mriedem tssurya: was here, she heard it
20:34:10 tssurya yea I am the evidence :)
20:34:34 melwitt \o/
20:34:43 melwitt we did something right
20:35:48 gibi :)
20:48:58 mriedem dansmith: this was a request in the upgrades room, you might have an opinion... https://bugs.launchpad.net/nova/+bug/1791824
20:48:59 openstack Launchpad bug 1791824 in OpenStack Compute (nova) "RFE: add configurable limit nova-status upgrade check for warning on archive/purge limit" [Wishlist,Triaged]
20:49:43 sorrison We're also working on https://review.openstack.org/#/c/556751/ just need to rebase it
21:15:29 openstackgerrit Ghanshyam Mann proposed openstack/nova stable/rocky: import zuul job settings from project-config https://review.openstack.org/601401
21:17:10 mriedem i wonder if i should have mentioned the vgpu reshaper upgrade for stein in this room...
21:19:03 openstackgerrit Ghanshyam Mann proposed openstack/nova stable/queens: import zuul job settings from project-config https://review.openstack.org/601396
21:19:41 melwitt yeah, could as a heads up for anyone who is or is going to use vGPUs in queens or rocky
21:20:40 openstackgerrit Ghanshyam Mann proposed openstack/nova stable/pike: import zuul job settings from project-config https://review.openstack.org/601392
21:24:39 openstackgerrit Ghanshyam Mann proposed openstack/nova stable/ocata: import zuul job settings from project-config https://review.openstack.org/601387
21:29:07 gmann bauzas: stephenfin updated the stable branch patches for job import series. compared those with list here and lgtm - https://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul.d/projects.yaml#n8623
21:54:04 cfriesen_ there's a month-old patch from mriedem that's been sitting with multiple +1s and no core reviews...any cores feel like reviewing? It's pretty tiny. https://review.openstack.org/#/c/588689
21:56:34 melwitt cfriesen_: looks simple, but the question that arises when I look at it is, can revert resize fail? is there a chance we could bork the request spec in that case?
21:57:47 mriedem it would be no riskier than borking the request spec if the resize itself fails,
21:57:57 mriedem because we change the requestspec to point at the new flavor in conductor at the start of resize,
21:58:04 mriedem this is to change the req spec back if we revert
21:58:56 cfriesen_ it's a valid point though...if the revert fails we should probably put the reqspec flavor back to the "new" flavor
22:01:33 melwitt mriedem: so you're saying right off the bat, we update the request spec to the new flavor before we've even attempted the resize?
22:01:46 melwitt ok I think that's what you just said
22:03:37 mriedem melwitt: yes this https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L352
22:03:49 mriedem "and only if nothing bad happened"
22:03:57 mriedem meaning, we successfully cast to the selected compute
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 2.0
22:08:41 mriedem mmmm new bork mode
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

Earlier   Later