Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-09-10
19:44:56 openstackgerrit Doug Hellmann proposed openstack/nova stable/queens: import zuul job settings from project-config https://review.openstack.org/601396
19:45:03 openstackgerrit Doug Hellmann proposed openstack/osc-placement stable/queens: import zuul job settings from project-config https://review.openstack.org/601397
19:45:08 openstackgerrit Doug Hellmann proposed openstack/os-traits stable/queens: import zuul job settings from project-config https://review.openstack.org/601398
19:45:13 openstackgerrit Doug Hellmann proposed openstack/os-vif stable/queens: import zuul job settings from project-config https://review.openstack.org/601399
19:45:18 openstackgerrit Doug Hellmann proposed openstack/python-novaclient stable/queens: import zuul job settings from project-config https://review.openstack.org/601400
19:45:27 openstackgerrit Doug Hellmann proposed openstack/nova stable/rocky: import zuul job settings from project-config https://review.openstack.org/601401
19:45:31 openstackgerrit Doug Hellmann proposed openstack/osc-placement stable/rocky: import zuul job settings from project-config https://review.openstack.org/601402
19:45:37 openstackgerrit Doug Hellmann proposed openstack/os-traits stable/rocky: import zuul job settings from project-config https://review.openstack.org/601403
19:45:45 openstackgerrit Doug Hellmann proposed openstack/os-vif stable/rocky: import zuul job settings from project-config https://review.openstack.org/601404
19:45:50 openstackgerrit Doug Hellmann proposed openstack/python-novaclient stable/rocky: import zuul job settings from project-config https://review.openstack.org/601405
19:57:25 bauzas stephenfin: you could be interested in https://review.openstack.org/#/c/601371/
19:59:02 bauzas FYI, https://review.openstack.org/#/q/topic:python3-first+(status:open+OR+status:merged)+project:openstack/nova for all py3 gate jobs changes
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 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

Earlier   Later