Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-09-10
16:55:49 dansmith that just talks about the tag field, right?
16:55:52 dansmith where do you get boot?
16:56:28 mriedem it's not in that spec, but we could build it in to a general "cleanup the os-volume_attachments API" spec
16:56:34 dansmith okay
16:57:07 mriedem b/c we also don't expose vif tags anywhere, so we'd maybe do those in a single spec
16:57:12 dansmith yep
16:57:46 mriedem https://review.openstack.org/#/c/393930/
16:59:12 mriedem keeping up gets a bit messy...
16:59:18 mriedem specs aplenty
17:32:41 mriedem gibi: btw since i just remembered we're going to talk about exposing hints out of the servers API https://review.openstack.org/#/c/581410/
17:32:52 mriedem i hope i addressed your comments
17:41:07 openstackgerrit Jonte Watford proposed openstack/nova master: Modified version of 0027-Numa-object-string-representations.patch with some updates from the current numa files for nova: numa.py instance_numa_topology.py https://review.openstack.org/600269
17:41:52 mriedem speaking of operations that update the request spec (resize), why don't we update the request spec on rebuild where the image can change?
17:42:45 mriedem oh nvm we do
17:42:48 mriedem and i added that code...
18:14:15 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Add missing options in CLI reference https://review.openstack.org/601349
19:37:34 openstackgerrit Doug Hellmann proposed openstack/nova master: import zuul job settings from project-config https://review.openstack.org/601370
19:37:35 openstackgerrit Doug Hellmann proposed openstack/nova master: switch documentation job to new PTI https://review.openstack.org/601371
19:37:37 openstackgerrit Doug Hellmann proposed openstack/nova master: add python 3.6 unit test job https://review.openstack.org/601372
19:37:52 openstackgerrit Doug Hellmann proposed openstack/nova-specs master: import zuul job settings from project-config https://review.openstack.org/601373
19:38:06 openstackgerrit Doug Hellmann proposed openstack/osc-placement master: import zuul job settings from project-config https://review.openstack.org/601374
19:38:08 openstackgerrit Doug Hellmann proposed openstack/osc-placement master: switch documentation job to new PTI https://review.openstack.org/601375
19:38:09 openstackgerrit Doug Hellmann proposed openstack/osc-placement master: add python 3.6 unit test job https://review.openstack.org/601376
19:38:15 openstackgerrit Doug Hellmann proposed openstack/os-traits master: import zuul job settings from project-config https://review.openstack.org/601377
19:38:17 openstackgerrit Doug Hellmann proposed openstack/os-traits master: switch documentation job to new PTI https://review.openstack.org/601378
19:38:18 openstackgerrit Doug Hellmann proposed openstack/os-traits master: add python 3.6 unit test job https://review.openstack.org/601379
19:38:57 openstackgerrit Doug Hellmann proposed openstack/os-vif master: import zuul job settings from project-config https://review.openstack.org/601380
19:38:58 openstackgerrit Doug Hellmann proposed openstack/os-vif master: switch documentation job to new PTI https://review.openstack.org/601381
19:39:00 openstackgerrit Doug Hellmann proposed openstack/os-vif master: add python 3.6 unit test job https://review.openstack.org/601382
19:39:06 openstackgerrit Doug Hellmann proposed openstack/python-novaclient master: import zuul job settings from project-config https://review.openstack.org/601383
19:39:07 openstackgerrit Doug Hellmann proposed openstack/python-novaclient master: switch documentation job to new PTI https://review.openstack.org/601384
19:39:08 openstackgerrit Doug Hellmann proposed openstack/python-novaclient master: add python 3.6 unit test job https://review.openstack.org/601385
19:39:10 openstackgerrit Doug Hellmann proposed openstack/python-novaclient master: add lib-forward-testing-python3 test job https://review.openstack.org/601386
19:39:18 openstackgerrit Doug Hellmann proposed openstack/nova stable/ocata: import zuul job settings from project-config https://review.openstack.org/601387
19:40:07 openstackgerrit Doug Hellmann proposed openstack/os-vif stable/ocata: import zuul job settings from project-config https://review.openstack.org/601388
19:44:24 openstackgerrit Doug Hellmann proposed openstack/python-novaclient stable/ocata: import zuul job settings from project-config https://review.openstack.org/601391
19:44:31 openstackgerrit Doug Hellmann proposed openstack/nova stable/pike: import zuul job settings from project-config https://review.openstack.org/601392
19:44:35 openstackgerrit Doug Hellmann proposed openstack/os-traits stable/pike: import zuul job settings from project-config https://review.openstack.org/601393
19:44:42 openstackgerrit Doug Hellmann proposed openstack/os-vif stable/pike: import zuul job settings from project-config https://review.openstack.org/601394
19:44:48 openstackgerrit Doug Hellmann proposed openstack/python-novaclient stable/pike: import zuul job settings from project-config https://review.openstack.org/601395
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

Earlier   Later