Earlier  
Posted Nick Remark
#openstack-nova - 2019-11-22
21:19:47 melwitt yeah so I rewind from what I said earlier and I'd do the extra step if --deleted is passed. grab the server id and then pass marker as id under the covers
21:21:07 openstackgerrit Merged openstack/nova master: Remove 'os-consoles' API https://review.opendev.org/687907
21:26:34 mriedem heh, this is a bit old https://docs.openstack.org/nova/latest/reference/vm-states.html#create-instance-states
21:26:42 mriedem api -> compute done!
21:37:05 openstackgerrit Matt Riedemann proposed openstack/nova master: api-guide: flesh out BUILD and ACTIVE server create transitions https://review.opendev.org/695771
21:43:15 openstackgerrit Matt Riedemann proposed openstack/nova master: api-guide: fix the file injection considerations drift https://review.opendev.org/695773
21:48:42 openstackgerrit Merged openstack/nova master: Avoid raise InstanceNotRunning exception https://review.opendev.org/541152
21:48:48 openstackgerrit Matt Riedemann proposed openstack/nova master: api-guide: remove empty sections about inter-service interactions https://review.opendev.org/695774
21:48:51 openstackgerrit Merged openstack/nova master: Start README.rst with a better title https://review.opendev.org/695025
22:16:53 openstackgerrit Matt Riedemann proposed openstack/nova master: api-guide: flesh out flavor extra specs and image properties https://review.opendev.org/695776
22:33:37 openstackgerrit Matt Riedemann proposed openstack/nova master: api-guide: flesh out networking concepts https://review.opendev.org/695777
22:33:44 melwitt efried: question on test cleanup https://review.opendev.org/685399
22:33:52 efried ...
22:35:01 efried I think you're probably right. Lemme check one thing quick...
22:36:05 melwitt ok. I wasn't sure how this is supposed to be
22:37:10 efried Yup, I think that was just a miss melwitt, good spot, will remove.
22:38:24 openstackgerrit Eric Fried proposed openstack/nova master: test cleanup: Make base TestCase subclass oslotest https://review.opendev.org/685399
22:38:30 efried melwitt: done, thanks.
22:38:46 melwitt kewl
22:39:30 efried thanks for those reviews melwitt
22:39:54 efried a day when my dashboard shrinks is a good day :)
22:40:32 efried and johnthetubaguy ^
22:41:14 melwitt np, nice cleanups
22:43:01 openstackgerrit Merged openstack/nova master: Remove 'nova-console' service, 'console' RPC API https://review.opendev.org/687908
22:44:10 openstackgerrit Merged openstack/nova master: nova-net: Remove 'networks' quota https://review.opendev.org/686812
22:47:12 openstackgerrit Matt Riedemann proposed openstack/nova master: api-guide: flesh out todos in user doc https://review.opendev.org/695778
22:59:04 openstackgerrit Matt Riedemann proposed openstack/nova master: doc: remove admin/manage-users https://review.opendev.org/695779
23:13:26 efried dansmith: you said something about hard reboot being admin-only by default, but I can't find how that's the case in the code, where it looks like soft/hard are under the same umbrella. Did I misunderstand?
23:17:24 openstackgerrit Merged openstack/nova master: test cleanup: Use oslotest's Timeout fixture https://review.opendev.org/685389
23:17:30 openstackgerrit Merged openstack/nova master: test cleanup: Use oslotest's CaptureOutput fixture https://review.opendev.org/685392
23:37:09 dansmith efried: that wasn't me
23:38:14 dansmith at least, not the hard reboot nova action
23:38:21 dansmith I talked a lot about *host* reboot...
23:38:33 efried dammit, that's probably where I got confused.
23:38:41 efried thanks.
#openstack-nova - 2019-11-23
00:08:59 openstackgerrit Dustin Cowles proposed openstack/nova master: Provider Config File: Function to further validate and retrieve configs https://review.opendev.org/676029
00:09:00 openstackgerrit Dustin Cowles proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree https://review.opendev.org/676522
00:09:00 openstackgerrit Dustin Cowles proposed openstack/nova master: WIP: Provider Config File: Enable loading and merging of provider configs https://review.opendev.org/693460
02:40:20 openstackgerrit Stephen Finucane proposed openstack/nova master: WIP https://review.opendev.org/686818
05:31:46 openstackgerrit Merged openstack/nova master: test cleanup: Make base TestCase subclass oslotest https://review.opendev.org/685399
06:28:19 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/694717
13:03:49 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Rewrite quotas documentation https://review.opendev.org/667165
13:08:13 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove nova-manage network, floating commands https://review.opendev.org/686813
13:08:14 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove associate, disassociate network APIs https://review.opendev.org/686814
13:08:14 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove 'nova-dhcpbridge' binary https://review.opendev.org/686815
13:08:15 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove 'nova-network' binary https://review.opendev.org/686816
13:08:15 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Blast most references to nova-network https://review.opendev.org/686817
13:08:16 openstackgerrit Stephen Finucane proposed openstack/nova master: WIP https://review.opendev.org/686818
14:05:32 openstackgerrit Merged openstack/nova master: docs: Rewrite quotas documentation https://review.opendev.org/667165
15:55:28 openstackgerrit Merged openstack/nova master: libvirt: check job status for VIR_DOMAIN_EVENT_SUSPENDED_MIGRATED event https://review.opendev.org/594527
#openstack-nova - 2019-11-24
00:31:50 openstackgerrit Merged openstack/nova master: Remove nova-manage network, floating commands https://review.opendev.org/686813
06:46:00 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/694717
#openstack-nova - 2019-11-25
00:45:33 openstackgerrit Eric Xie proposed openstack/nova master: Fix incorrect command examples https://review.opendev.org/695832
05:12:16 openstackgerrit Merged openstack/nova master: api-guide: flesh out BUILD and ACTIVE server create transitions https://review.opendev.org/695771
05:13:28 openstackgerrit Merged openstack/nova master: api-guide: fix the file injection considerations drift https://review.opendev.org/695773
06:30:41 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/694717
08:19:04 bauzas good morning Nova
08:44:35 openstackgerrit Lee Yarwood proposed openstack/nova master: Remove 'nova-xvpvncproxy' https://review.opendev.org/687909
08:45:03 lyarwood stephenfin: ^ hope you don't mind, thought I'd sort out the last few failures as the removal came up in RDO over the weekend.
08:53:29 stephenfin lyarwood: fine by me, but I think it needs to be rebased again
08:53:35 stephenfin "This change depends on a change that failed to merge."
08:53:55 lyarwood stephenfin: ah it's the tempest change?
08:54:22 lyarwood stephenfin: hmm no that's also up to date
08:54:23 lyarwood weird
09:07:07 lyarwood stephenfin: oh that was my bad, I pushed another PS to the tempest change after pushing the nova change, apologies!
09:07:16 lyarwood recheked
09:07:22 lyarwood rechecked*
09:38:43 openstackgerrit Luyao Zhong proposed openstack/nova-specs master: support live migration with virtual persistent memory https://review.opendev.org/695863
09:51:14 tssurya dansmith: whenever you are awake and have some time, review appreciated on this https://review.opendev.org/#/c/689422/ thanks in advance
09:52:48 openstackgerrit Luyao Zhong proposed openstack/nova-specs master: spec update: virtual persistent memory https://review.opendev.org/695868
10:15:06 openstackgerrit Elod Illes proposed openstack/nova stable/stein: Use admin neutron client to see if instance has qos ports https://review.opendev.org/694668
10:19:04 stephenfin dansmith: Could I get some input on RPC versioning issues here, when you're online? https://review.opendev.org/#/c/686814/11
10:59:35 stephenfin johnthetubaguy: Could you weigh in on https://review.opendev.org/#/c/686814/11/nova/network/rpcapi.py@a211, assuming you know RPC better than I do?
11:11:53 kashyap stephenfin: Hi, when you get a moment, want to ACK/NACK these? -- https://review.opendev.org/#/c/694821/3
11:12:28 kashyap stephenfin: I'd prefer to keep them seperate, although I know they _could_ be squashed. But if that's the "bone of contention" and you insist, can squash :-)
12:32:03 sean-k-mooney stephenfin: can you take a look at https://review.opendev.org/#/c/674072/ today if you get a chance
12:51:23 gibi sean-k-mooney, mriedem: hi! do we have live migration functional tests where the server has at least on neutron port? did we added such functional test when the bp neutron-new-port-binding-api has been implemented?
12:51:35 gibi s/at least on/at least one/
12:55:22 kashyap gibi: Hi, any chance you can have a look at this low-hanging-fruit (that lets us clean up dead code): https://review.opendev.org/#/q/topic:Bump_min_libvirt_and_QEMU_for_Ussuri+(status:open+OR+status:merged)
12:56:25 sean-k-mooney gibi: i would have to check but i dont think the neutron fixture implements the multiple port bindigns workflow
12:56:48 sean-k-mooney gibi: so i dont think we ahve live migration test that succeed.
12:57:02 sean-k-mooney we have 2 negitive tests i belive
12:57:10 gibi kashyap: added to my list but no promises for today
12:57:36 kashyap gibi: No worries, this week is fine; it's also advertized on the mailing list, etc. (FWIW.)
12:58:18 gibi sean-k-mooney: cool, then I understand why my test with qos + live migration fails with the neutron fixture with vif plug timeout
12:58:56 gibi sean-k-mooney: thanks. I think I will try to enhance the NeutronFixture to cover live migration with qos
12:59:20 tobias-urdin do i understand it correctly that nova refreshes the rbd in the libvirt XML by *adding any new* monitors to the config based on the image backend/connection_info from cinder but doesn't purge old entries? And that this patch tries to make sure it completely refreshes the entries by clearing it and adding the existing ones back in? https://review.opendev.org/#/c/579004/
13:01:52 sean-k-mooney gibi: right i dont think you wll get the network-vif-plugged event
13:03:13 gibi sean-k-mooney: yeah, now I was able to make an existing test fail with the same problem by simply adding networks="auto" to the boot request
13:04:00 sean-k-mooney that would normally be set by the ovs agent once that tap is create by libvirt. so you need to mock out the activate binding api endpoint in the fixture to send the vif plugged event to neutron
13:04:22 gibi sean-k-mooney: ack
13:04:23 gibi thanks
13:05:01 sean-k-mooney *form neutorn to nova
13:05:26 sean-k-mooney it would be useful to have that by they way
13:05:48 sean-k-mooney so thanks for taking a look at adding it
13:06:30 sean-k-mooney gibi: on a related note did any of the neutron folks respond on that tread regarding backport the agent changes for finding the RP
13:07:03 gibi sean-k-mooney: not yet. I have a face 2 face discussion with rubasov this morning. I hope he will reply at some point
13:07:11 gibi s/have/had/
13:08:01 sean-k-mooney ok i can poke some of the redhat folks too to comment on it
13:33:39 gibi sean-k-mooney: I opened a neutron bug https://bugs.launchpad.net/neutron/+bug/1853840

Earlier   Later