Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-09-30
22:39:52 efried mriedem: we used to have something in nova-specs that enforced conformance to the template, no??
22:41:55 efried ah, it's hidden in pep8
22:59:12 openstackgerrit Eric Fried proposed openstack/nova-specs master: Add 'Core Liaison' section to spec template https://review.opendev.org/685857
#openstack-nova - 2019-10-01
01:33:39 openstackgerrit Eric Fried proposed openstack/nova master: Allow versioned discovery unauthenticated https://review.opendev.org/685181
02:47:08 SonPham hi
02:47:19 SonPham had any project to create live-snapshot for openstack?
02:47:48 SonPham snapshot with save ram state
03:01:28 brinzhang_ Now, nova not support to do for that
03:04:55 openstackgerrit Seyeong Kim proposed openstack/nova master: block_device: Optionally recreate attachments when refreshing connection_info https://review.opendev.org/579004
03:07:43 brinzhang_ SonPham: sorry for above said, current, cannot do a live-snapshot for hte paused server of libvirt/qemu driver.
03:08:13 brinzhang_ paused and shutdown status server.
03:10:13 melwitt dansmith, mriedem: the doc reference got removed here https://github.com/openstack/nova/commit/009fd0f35bcb88acc80f12e69d5fb72c0ee5391f
03:26:16 SonPham brinzhang_ Can we save VM state (save Ram) in the snapshot?
03:31:27 brinzhang_ SonPham: I think you can, bug I am not sure, you can review that and try, with "disable_libvirt_livesnapshot" configuration in you env.
03:31:43 brinzhang_ SonPham: https://docs.openstack.org/nova/stein/configuration/config.html#workarounds.disable_libvirt_livesnapshot
03:54:41 SonPham brinzhang_ I'm using libvirt to create snapshot for instance
08:33:17 mgoddard mriedem: yeah, I meant multiple cells. We would be quite behind to not have any support for cells v2...
09:57:58 bauzas gibi: can I ask you something about how to create a new allocation ?
10:01:01 gibi bauzas: sure hit me
10:03:09 bauzas gibi: to explain it : http://paste.openstack.org/show/780546/
10:03:30 bauzas gibi: I want to recreate the migration allocation that was deleted when confirming the resize
10:03:47 bauzas gibi: so after that, after running my audit command, it would find it
10:04:24 bauzas but then, I need a consumer_generation, right?
10:04:42 bauzas or should I just update the allocations ?
10:05:07 bauzas ie. https://docs.openstack.org/api-ref/placement/?expanded=update-allocations-detail#id72
10:05:54 gibi bauzas: you want to recreate the migration allocation so I assume that there is no allocation with that consumer. In that case the consumer generation when you create the allocation should be 0
10:06:09 bauzas gibi: that's correct
10:06:22 bauzas I no longer have allocations for the migration UUID
10:06:51 bauzas if so, we don't need to provide a generation version
10:07:11 bauzas gibi: do we have any helper for putting a new allocation in the functional tests ?
10:07:25 gibi I mean: "The generation of the consumer. Should be set to null when indicating that the caller expects the consumer does not yet exist."
10:07:54 gibi bauzas: I think I never needed to manually create allocation in the functional test
10:08:23 bauzas ok
10:10:22 stephenfin gmann: Still about? I'm stuck with https://review.opendev.org/#/c/684335/5/doc/api_samples/servers/v2.69/servers-details-resp.json
10:11:25 stephenfin gibi: You might be able to advise on ^ too
10:20:09 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_server_tags' functional tests https://review.opendev.org/684335
10:20:10 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_servers' functional tests https://review.opendev.org/684336
10:20:10 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_hosts' functional tests https://review.opendev.org/684337
10:20:11 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_hypervisors' functional tests https://review.opendev.org/684340
10:20:11 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_attach_interfaces' functional tests https://review.opendev.org/684341
10:20:12 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_simple_tenant_usage' functional tests https://review.opendev.org/684342
10:20:13 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Neutron fixture cleanup https://review.opendev.org/684343
10:20:13 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_floating_ips' functional tests https://review.opendev.org/684344
10:20:14 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Add TODOs for remaining nova-network functional tests https://review.opendev.org/684345
10:20:14 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Correctly mock out security groups in NeutronFixture https://review.opendev.org/685927
10:21:51 openstackgerrit Matthew Booth proposed openstack/nova master: libvirt: Ignore DiskNotFound during update_available_resource https://review.opendev.org/685391
10:25:22 gibi stephenfin: do I understand the situation that the nova net base sample tests was not covered the sec group case and now that you add neutron based sample tests which has sec group you discovered a bug in the api behavior?
10:25:36 stephenfin that seems to be my understanding
10:25:43 stephenfin *that is my understanding :)
10:26:06 stephenfin gibi: Basically, I'm not sure if https://review.opendev.org/685927 is correct or not
10:26:23 stephenfin I mean, the addition of that field to two APIs
10:26:58 stephenfin I guess I will need to look at a real cloud with a down cell to see what actually comes back
10:27:07 stephenfin Or spend ages trying to trace stuff :(
10:28:59 gibi your fixture is coded with the assumption that neutron returns at least the default security group for every instance. If this assumption correct then what I think what you see in the test is the actual nova behavior
10:29:33 gibi if that leads to unexpecte API response then that is a bug
10:29:57 stephenfin Sort. I think it's more like I'm assuming an instance has at least one security group regardless of its state
10:30:41 gibi stephenfin: yeah your view is more precies you would need to validate that assumption first
10:30:58 gibi I think neutron behavior should not be cell dependent
10:31:06 stephenfin Yeah, neither do I
10:31:19 stephenfin I've no idea where to start on validating that assumption though so ima wait til sean-k-mooney or mriedem are about :)
10:31:25 gibi :)
10:34:09 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Remove 'adv-config', 'system-admin' subdocs https://review.opendev.org/684402
10:34:10 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Replacing underscores with dashes https://review.opendev.org/685929
10:34:57 gibi in devstack if I do nothing special I get the default sec group
10:35:15 stephenfin Yeah, that's what I was seeing too
10:36:33 stephenfin gibi: https://docs.openstack.org/nova/pike/admin/security-groups.html
10:36:41 stephenfin "All projects have a default security group which is applied to any instance that has no other defined security group. Unless you change the default, this security group denies all incoming traffic and allows only outgoing traffic to your instance."
10:37:33 gibi after I stop the n-cond-cell1 service in my devstack I still see the default sec group returned by nova
10:38:02 gibi stephenfin: based on that doc your assumption is valid
10:45:37 stephenfin gibi: Just discussed with slaweq on #openstack-neutron. While it's possible to create ports with no security, that's not the default and none of our tests are checking for that, so expecting the default security group seems sane
10:45:45 stephenfin Left comments on https://review.opendev.org/#/c/685927/ to that effect
10:46:26 stephenfin I think we still have a bug though, in as far as there are times where the security_groups key is omitted rather than being set to an empty list or null. I need to discuss with gmann or mriedem later
10:47:20 gibi OK, cool
10:51:27 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_servers' functional tests https://review.opendev.org/684336
10:51:27 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_hosts' functional tests https://review.opendev.org/684337
10:51:28 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_hypervisors' functional tests https://review.opendev.org/684340
10:51:28 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_attach_interfaces' functional tests https://review.opendev.org/684341
10:51:29 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_simple_tenant_usage' functional tests https://review.opendev.org/684342
10:51:29 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Neutron fixture cleanup https://review.opendev.org/684343
10:51:30 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_floating_ips' functional tests https://review.opendev.org/684344
10:51:30 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Add TODOs for remaining nova-network functional tests https://review.opendev.org/684345
10:56:23 openstackgerrit OpenStack Proposal Bot proposed openstack/nova stable/train: Imported Translations from Zanata https://review.opendev.org/685534
11:13:11 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Use nova-net explicitly in functional tests https://review.opendev.org/684326
11:13:12 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_availability_zone' functional tests https://review.opendev.org/684332
11:13:12 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_floating_ip_pools' functional tests https://review.opendev.org/684333
11:13:13 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_quota_sets' functional tests https://review.opendev.org/684334
11:13:13 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Correctly mock out security groups in NeutronFixture https://review.opendev.org/685927
11:13:14 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_server_tags' functional tests https://review.opendev.org/684335
11:13:14 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_servers' functional tests https://review.opendev.org/684336
11:13:15 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_hosts' functional tests https://review.opendev.org/684337
11:13:15 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_rescue' functional tests https://review.opendev.org/684339
11:13:16 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_hypervisors' functional tests https://review.opendev.org/684340
11:13:16 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_attach_interfaces' functional tests https://review.opendev.org/684341
11:13:17 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_simple_tenant_usage' functional tests https://review.opendev.org/684342
11:13:17 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Neutron fixture cleanup https://review.opendev.org/684343
11:13:18 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Migrate 'test_floating_ips' functional tests https://review.opendev.org/684344
11:13:18 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Add TODOs for remaining nova-network functional tests https://review.opendev.org/684345
11:13:19 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Remove explicit 'USE_NEUTRON = True' https://review.opendev.org/685937
11:25:54 lyarwood stephenfin: https://github.com/lyarwood/nova/blob/a51921fe195ba6d8ccf60941b12c07ed304496da/nova/tests/functional/libvirt/base.py#L102-L105 - I don't understand what you're suggesting here
11:26:24 lyarwood stephenfin: AssertionError: 16 not greater than 16 : Host.get_online_cpus is only accounting for 16 CPUs but you're requesting 16; change the mock or your test currently showing up in https://review.opendev.org/#/c/682621/2 as a result

Earlier   Later