Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-08-08
22:07:51 dansmith but it seems like the fix is worth getting in, nits on the test aside
22:07:54 melwitt I'd rather not have to convert them to unit tests at this point but if you want me to change it, I will
22:08:29 mriedem no i'm not talking about full on re-writing them as unit tests
22:08:39 mriedem i just meant with stuff like the AllServicesCurrent fixture and NoopConductorFixture
22:09:06 mriedem but we must not be hitting https://github.com/openstack/nova/blob/stable/ocata/nova/compute/api.py#L1729 because your change to the tests in the next patch are ok
22:09:25 melwitt yeah ... seems like it would return early because of the service version but the test does pass after the fix, which means it's getting to the quota calls somehow
22:09:45 dansmith because there are no services?
22:09:53 mriedem the api service is running
22:09:54 dansmith I don't see why we want noopconductor over just not starting it
22:10:29 dansmith but it's all just forced because this should be a unit test where it doesn't matter anyway :)
22:10:49 melwitt SORRY GUYS for my blasphemous functional test
22:10:50 mriedem i'm just surprised that fixture isn't needed is all i'm saying
22:11:03 mriedem like, why do we even need it in the first place in other func tests is my point
22:11:31 mriedem thou shalt be smoten with review nit-pickery
22:11:34 melwitt we have definitely needed it before. I'm not sure what's going on here
22:11:46 melwitt re: ServicesCurrent
22:12:11 dansmith if you have no services you get back the current I think
22:12:18 dansmith we used to return zero,
22:12:23 melwitt the api service doesn't count as a service?
22:12:39 melwitt os_apicompute
22:13:01 mriedem idk, might have something to do with wsgi-intercept usage in OSAPIFixture now
22:13:03 dansmith you're not starting an api service for real here
22:13:05 dansmith right
22:13:14 melwitt O.o
22:13:19 dansmith but anyway if this was A UNIT TEST
22:13:28 mriedem we used to
22:13:29 mriedem self.osapi = service.WSGIService("osapi_compute")
22:13:29 mriedem self.osapi.start()
22:13:29 mriedem self.addCleanup(self.osapi.stop)
22:13:32 melwitt :***(
22:13:37 mriedem which is probably why AllServicesCurrent was added before
22:15:12 mriedem +W
22:15:32 mriedem moving onto soul crusher #2
22:27:51 mriedem bazinga https://review.openstack.org/#/c/582413/8
22:28:04 melwitt argh
22:29:05 melwitt ok that's not so bad
22:29:28 mriedem do you want/need to hold the functional test patch below it out to add checks for core and ram usage?
22:29:57 mriedem or just slap those into this one
22:30:41 melwitt I'm fine with adding them to the second one
22:50:07 openstackgerrit Matt Riedemann proposed openstack/nova master: Docs: Add guide to migrate instance with snapshot https://review.openstack.org/584442
23:00:18 mriedem heh https://review.openstack.org/#/c/382853/
23:00:25 mriedem i knew i'd seen this recently
23:00:26 mriedem https://github.com/starlingx-staging/stx-nova/commit/71acfeae0d1c59fdc77704527d763bd85a276f9a#diff-77f9348ab09642ba46409b6828af4af0R3245
23:00:32 mriedem oct of 2016
23:04:48 openstackgerrit melanie witt proposed openstack/nova stable/ocata: [stable only] Handle quota usage during create/delete races https://review.openstack.org/582413
23:11:45 openstackgerrit Matt Riedemann proposed openstack/nova master: Update nova network info when doing rebuild for evacuate operation https://review.openstack.org/382853
23:12:57 openstackgerrit Matt Riedemann proposed openstack/nova master: Update nova network info when doing rebuild for evacuate operation https://review.openstack.org/382853
23:15:24 mriedem wow so that evacuate fix was approved in march of 2017,
23:15:27 mriedem but apparently needed a rebase
23:15:34 mriedem and then no one ever mentioned needing re-approval
23:17:00 melwitt yikes
23:18:01 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Fix instance evacuation with PCI devices https://review.openstack.org/590059
23:29:22 melwitt I'm going to propose stable releases tomorrow. I meant to do it last week but didn't get to it
23:33:38 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Update nova network info when doing rebuild for evacuate operation https://review.openstack.org/590062
23:34:41 openstackgerrit Merged openstack/nova master: get provider IDs once when building summaries https://review.openstack.org/589945
23:57:15 openstackgerrit Merged openstack/nova master: libvirt: Reduce calls to qemu-img during update_available_resource https://review.openstack.org/589513
23:57:22 openstackgerrit Merged openstack/nova master: Add tempest-slow job to run the tempest slow tests https://review.openstack.org/567697
23:57:32 openstackgerrit Merged openstack/nova master: Fix bad links for admin-guide https://review.openstack.org/580259
23:57:41 openstackgerrit Merged openstack/nova master: [placement] Avoid rp.get_by_uuid in allocation_candidates https://review.openstack.org/589941
23:57:49 openstackgerrit Merged openstack/nova master: api-ref: fix min_version for parent_provider_uuid in responses https://review.openstack.org/579577
23:57:57 openstackgerrit Merged openstack/nova master: doc: mark the max microversion for rocky https://review.openstack.org/589598
#openstack-nova - 2018-08-09
00:24:58 openstackgerrit Merged openstack/nova master: Add functional test for forced live migration rollback allocs https://review.openstack.org/586636
01:36:59 openstackgerrit Merged openstack/nova stable/ocata: [stable only] Add functional regression test for bug 1783613 https://review.openstack.org/588416
01:36:59 openstack bug 1783613 in OpenStack Compute (nova) ocata "[ocata only] quota usage not decremented during boot/delete race" [Undecided,In progress] https://launchpad.net/bugs/1783613 - Assigned to melanie witt (melwitt)
02:08:17 openstackgerrit Chen proposed openstack/nova stable/queens: Fix bad links for admin-guide https://review.openstack.org/590068
02:13:36 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Update nova network info when doing rebuild for evacuate operation https://review.openstack.org/590070
02:16:02 openstackgerrit Chen proposed openstack/nova stable/queens: Fix bad links for admin-guide https://review.openstack.org/590068
02:16:37 openstackgerrit Chen proposed openstack/nova stable/pike: Fix bad links for admin-guide https://review.openstack.org/590072
02:39:05 openstackgerrit Chen proposed openstack/nova master: Update ssh configuration doc https://review.openstack.org/589844
02:45:27 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Update the parameter explain when updating a volume attachment https://review.openstack.org/565181
02:56:50 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] api-ref: add description for 1.29 https://review.openstack.org/589407
03:12:36 openstackgerrit Vishakha Agarwal proposed openstack/nova master: Quota details for key_pair "in_use" is 0. https://review.openstack.org/590081
03:17:19 openstackgerrit Chen proposed openstack/nova master: Trivial fix on migration doc https://review.openstack.org/589028
04:13:29 openstackgerrit Merged openstack/nova stable/queens: Fix bad links for admin-guide https://review.openstack.org/590068
05:32:31 openstackgerrit Vishakha Agarwal proposed openstack/nova master: Quota details for key_pair "in_use" is 0. https://review.openstack.org/590081
06:05:26 openstackgerrit Vishakha Agarwal proposed openstack/nova master: Quota details for key_pair "in_use" is 0. https://review.openstack.org/590081
06:55:14 openstackgerrit Sergii Golovatiuk proposed openstack/nova master: libvirt: Always escape IPv6 addresses when used in migration URI https://review.openstack.org/589548
07:15:50 openstackgerrit liuyamin proposed openstack/python-novaclient master: Replace os-client-config to openstacksdk https://review.openstack.org/590141
07:23:14 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Adds a test for _get_provider_ids_matching() https://review.openstack.org/590150
08:04:48 gibi melwitt, mriedem: opened a versioned notification bp for stein https://blueprints.launchpad.net/nova/+spec/versioned-notification-transformation-stein
08:19:07 mdbooth lyarwood: Passing: https://review.openstack.org/#/c/587013/ !
08:21:47 mdbooth lyarwood: Also passed without the rebase workaround. I'll merge them and resubmit.
08:27:24 openstackgerrit Matthew Booth proposed openstack/nova master: fixtures: Track volume attachments within CinderFixtureNewAttachFlow https://review.openstack.org/587013
08:36:02 openstackgerrit Jose Castro Leon proposed openstack/nova master: Fix get_device_path from network mounted volume https://review.openstack.org/590188
08:49:46 openstackgerrit Matthew Booth proposed openstack/nova master: Add regression test for bug#1784353 https://review.openstack.org/587014
09:13:44 lyarwood mdbooth: cool, thanks again for working through this :)
09:14:27 mdbooth lyarwood: np.
09:14:46 mdbooth lyarwood: Started accidentally while reviewing and seemed silly to stop :)
09:21:19 lyarwood mdbooth: are you going to rebase https://review.openstack.org/#/c/587071/ ?
09:21:30 lyarwood mdbooth: np if not, I have time to work on it this morning finally
09:21:46 mdbooth lyarwood: I was going to leave that for you, that's the real bit :)
09:21:56 lyarwood mdbooth: tis cool, thanks again
09:22:12 mdbooth lyarwood: Incidentally, did you consider the 'fix it in compute' approach
09:22:26 mdbooth I know that's where the patch started, then you moved to conductor
09:22:55 mdbooth But in fixing the fixture I can across other cleanup in compute which already does exactly what I was talking about
09:23:10 lyarwood mdbooth: yeah the remove_volume_connections call
09:23:33 mdbooth Yeah
09:24:12 lyarwood mdbooth: yeah I'll take a look now, it's a shame to flip back again but meh
09:25:02 mdbooth lyarwood: I'm not saying do it, just asking if it's feasible/worth considering
09:25:12 mdbooth Or if you've already considered and rejected it, in fact

Earlier   Later