Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-06-07
19:42:24 efried but only if it's a tuple
19:42:24 sean-k-mooney but it wont to a substring match on each of the values of the tuple
19:42:28 efried right
19:42:51 efried we are in violent agreement
19:42:59 sean-k-mooney yep :)
19:43:07 efried And I think we're not broken here, because there's no arch const that's a substring of another arch const (is there?)
19:43:25 efried but it's by luck. It's still wrong and should be fixed.
19:43:48 sean-k-mooney i dont know if we have x86 and x86_64 but i think if we did we would threat them as an alias
19:44:03 openstackgerrit Eric Fried proposed openstack/nova master: Fix some bogus `in (not_a_tuple)` uses https://review.opendev.org/664041
19:44:12 efried Enhanced commit message which (hopefully) makes it clearer ^
19:44:24 efried (also fixed the other two spots in the nova codebase that do the same)
19:44:40 sean-k-mooney this is the set we support https://github.com/openstack/nova/blob/master/nova/objects/fields.py#L112-L153
19:45:01 sean-k-mooney technically tere a few
19:45:12 sean-k-mooney ppc64 and ppc64le
19:45:22 efried mm, s390/s390x
19:45:37 efried ppc is a substring of a bunch
19:45:42 sean-k-mooney ya look like only ibm was broken
19:45:46 efried sh4
19:45:56 efried xtensa
19:45:59 efried mips
19:46:06 efried arm7
19:46:10 sean-k-mooney ok
19:46:20 sean-k-mooney look like only x86 is not broken :P
19:46:29 efried the one in question was specifically saying in (AARCH64)
19:46:43 efried which looks like it's probably okay.
19:46:58 sean-k-mooney i dont think we use most of these constants anyway
19:49:18 sean-k-mooney if only we had static types...
20:14:55 openstackgerrit Edward Hope-Morley proposed openstack/nova master: Fix python3 compatibility of rbd get_fsid https://review.opendev.org/663607
20:48:16 openstackgerrit Matt Riedemann proposed openstack/nova master: Add SchedulerReportClient.get_provider_by_name https://review.opendev.org/664062
21:06:46 mriedem well zuul just got restarted so i'm out of here for the weekend
21:06:52 mriedem sniff you jerks later
21:09:40 openstackgerrit Eric Fried proposed openstack/nova master: Hacking N363: `in (not_a_tuple)` https://review.opendev.org/664041
23:15:21 redkrieg Is there any way to run a virt-sparsify on a snapshot during creation?
#openstack-nova - 2019-06-08
02:32:36 openstackgerrit Merged openstack/nova master: Remove an unused method https://review.opendev.org/663502
04:51:45 openstackgerrit Merged openstack/nova master: Remove 'ComputeManager._reschedule' https://review.opendev.org/660774
07:00:24 openstackgerrit Merged openstack/nova master: Avoid unnecessary joins in InstanceGroup.get_hosts https://review.opendev.org/661032
11:40:23 openstackgerrit Merged openstack/nova master: Delete unused get_all_host_states method https://review.opendev.org/663377
12:55:12 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove cells v1 parameter from 'ComputeTaskAPI.resize_instance' https://review.opendev.org/651311
12:55:13 openstackgerrit Stephen Finucane proposed openstack/nova master: Stop passing 'kwargs' to 'rebuild_instance' https://review.opendev.org/651312
12:55:13 openstackgerrit Stephen Finucane proposed openstack/nova master: Stop passing 'delete_type' to 'terminate_instance' https://review.opendev.org/651313
12:55:14 openstackgerrit Stephen Finucane proposed openstack/nova master: filters: Stop handling cells v1 https://review.opendev.org/651314
12:55:14 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove nova.compute.*API() shims https://review.opendev.org/660527
12:55:15 openstackgerrit Stephen Finucane proposed openstack/nova master: Add reno for removed cells v1 policies https://review.opendev.org/662031
12:55:15 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'InstanceUnknownCell' exception https://review.opendev.org/662411
12:55:16 openstackgerrit Stephen Finucane proposed openstack/nova master: Ensure controllers all call super https://review.opendev.org/660950
15:36:13 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Use SATA bus for cdrom devices when using Q35 machine type https://review.opendev.org/663011
15:36:13 openstackgerrit Lee Yarwood proposed openstack/nova master: DNM: Run tempest-full-py3 with q35 machine type https://review.opendev.org/662887
15:41:00 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Use SATA bus for cdrom devices when using Q35 machine type https://review.opendev.org/663011
15:41:01 openstackgerrit Lee Yarwood proposed openstack/nova master: DNM: Run tempest-full-py3 with q35 machine type https://review.opendev.org/662887
15:48:52 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove cells v1 parameter from 'ComputeTaskAPI.resize_instance' https://review.opendev.org/651311
15:48:53 openstackgerrit Stephen Finucane proposed openstack/nova master: Stop passing 'kwargs' to 'rebuild_instance' https://review.opendev.org/651312
15:48:53 openstackgerrit Stephen Finucane proposed openstack/nova master: Stop passing 'delete_type' to 'terminate_instance' https://review.opendev.org/651313
15:48:54 openstackgerrit Stephen Finucane proposed openstack/nova master: filters: Stop handling cells v1 https://review.opendev.org/651314
15:48:54 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove nova.compute.*API() shims https://review.opendev.org/660527
15:48:55 openstackgerrit Stephen Finucane proposed openstack/nova master: Add reno for removed cells v1 policies https://review.opendev.org/662031
15:48:55 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'InstanceUnknownCell' exception https://review.opendev.org/662411
15:48:56 openstackgerrit Stephen Finucane proposed openstack/nova master: Ensure controllers all call super https://review.opendev.org/660950
15:52:31 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Use SATA bus for cdrom devices when using Q35 machine type https://review.opendev.org/663011
15:52:31 openstackgerrit Lee Yarwood proposed openstack/nova master: DNM: Run tempest-full-py3 with q35 machine type https://review.opendev.org/662887
16:09:41 openstackgerrit Lee Yarwood proposed openstack/nova master: fup: Merge machine_type_mappings into get_default_machine_type https://review.opendev.org/664036
20:34:13 Nick_A Trying to snapshot lxd containers - is there a way to force the snapshot to occur "online" like kvm?
#openstack-nova - 2019-06-09
13:57:52 openstackgerrit Lee Yarwood proposed openstack/nova master: DNM: Run tempest-full-py3 with q35 machine type https://review.opendev.org/662887
22:56:10 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.opendev.org/566153
#openstack-nova - 2019-06-10
00:22:27 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Enable omission of the value of '--config-drive' https://review.opendev.org/653683
02:23:54 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix no propagation of nova context request_id https://review.opendev.org/662715
04:59:08 openstackgerrit Takashi NATSUME proposed openstack/nova master: Replace joinedload_all with joinedload https://review.opendev.org/664193
06:01:23 openstackgerrit Merged openstack/nova-specs master: Specifying az when restore shelved server https://review.opendev.org/624689
06:29:20 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Clarify the error response in proposed change https://review.opendev.org/664198
06:31:38 openstackgerrit Boxiang Zhu proposed openstack/nova master: Validate requested host/node during servers create https://review.opendev.org/661237
08:05:58 sapd1 Hi everyone, In this line, https://github.com/openstack/nova/blob/master/nova/api/metadata/base.py#L362 I think I can get devices which are being attached to the instance, But When I tried, It returns None.
08:07:57 sapd1 Can any one tell me why?
08:14:35 openstackgerrit Takashi NATSUME proposed openstack/nova master: Replace 'is comprised of' with 'comprises' https://review.opendev.org/663175
08:14:49 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove unnecessary setUp methods https://review.opendev.org/663179
09:45:54 openstackgerrit Edward Hope-Morley proposed openstack/nova master: Fix python3 compatibility of rbd get_fsid https://review.opendev.org/663607
09:52:54 openstackgerrit Brin Zhang proposed openstack/nova master: Specify availability_zone to unshelve https://review.opendev.org/663851
10:50:52 openstackgerrit sean mooney proposed openstack/nova master: libvirt: delegate ovs plug to os-vif https://review.opendev.org/602432
11:04:24 openstackgerrit Stephen Finucane proposed openstack/nova master: Recalculate 'RequestSpec.numa_topology' on resize https://review.opendev.org/662522
11:04:24 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Cleanup of '_test_resize' helper test https://review.opendev.org/664245
13:28:24 stephenfin efried: I'd like a python-novaclient release with https://review.opendev.org/#/c/650363/ in it. Have proposed https://review.opendev.org/#/c/664259/, assuming that's okay?
13:32:04 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Add '_setup_compute_services' helper https://review.opendev.org/663102
13:47:06 efried stephenfin: looking
13:51:06 efried stephenfin: gross. Are you going to blacklist all 8 versions the original got included in, for py27?
13:51:19 efried I guess if this only affects the CLI, no need for that.
13:51:23 stephenfin I don't think so
13:51:45 stephenfin It's a lot of effort and I figure distros will pick this up pretty quickly
13:51:46 dansmith is gibi out?
13:53:12 efried stephenfin: +1
14:20:03 stephenfin efried: Want to take a look at this too and unblock the rest of the series https://review.opendev.org/#/c/651311/
14:21:44 efried stephenfin: Will do. I was just looking at the "all call super" patch in that series...
14:21:49 stephenfin Also got https://review.opendev.org/#/c/660950/ working, though that's lower priority (turns out it's 'foo(x=None, *args, **kwargs)' and not 'foo(*args, x=None, **kwargs)', but flake8 doesn't catch that)
14:21:52 stephenfin Heh, jinx
14:29:05 efried stephenfin: bottom one is +A.
14:29:17 efried imo the super one shouldn't be adding *args or **kwargs anywhere they weren't already.
14:36:16 openstackgerrit Matt Riedemann proposed openstack/nova master: Make get_provider_by_name public and remove safe_connect https://review.opendev.org/664062
14:38:21 openstackgerrit Stephen Finucane proposed openstack/nova master: Ensure controllers all call super https://review.opendev.org/660950
14:38:30 stephenfin efried: Cool. Fixed the super one by removing all the *args and **kwargs (none of them were necessary, fwict) ^
14:53:18 efried stephenfin: Cool man, +2.
14:53:27 stephenfin mriedem, sean-k-mooney: What am I missing here? I see only one callers to the revert_resize function and that's from the API https://review.opendev.org/#/c/662522/3/nova/compute/api.py@3420
14:53:52 stephenfin We wouldn't call "openstack resize --revert" on a failed migration, would we?
14:54:38 sean-k-mooney stephenfin: im pretty sure we use teh revert_resize api endpoint to revert a cold-migration

Earlier   Later