| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-24 | |||
| 19:58:43 | mriedem | where after 2 weeks your vm was shutdown | |
| 19:59:09 | mriedem | some of that was tied to licenses on the os i think | |
| 19:59:21 | mriedem | if you were using a vm for longer than 2 weeks we had to register it for internal compliance audits and such | |
| 19:59:22 | nkorabli | I guess there is some shelve offload time option? Like it will be automatically destroyed after some period of time, or not? | |
| 19:59:35 | mriedem | nkorabli: yes, by default it offloads immediately | |
| 19:59:44 | mriedem | offload == delete the guest from the hypervisor | |
| 20:00:01 | mriedem | so when you unshelve, we have to build the guest again on another compute | |
| 20:00:57 | nkorabli | Ohh... I see! I wasn't sure what the difference between 'shelved' and 'shelved_offloaded'. | |
| 20:01:00 | nkorabli | Cool | |
| 20:01:21 | melwitt | yeah, but shelved instances still take up quota. so deleting them while they're offloaded would free up quota for others in the tenant | |
| 20:03:05 | melwitt | which reminds me, I need to keep an eye on that for how it works with counting | |
| 20:03:42 | nkorabli | mriedem: Thanks a lot for your attention! :) Will send an e-mail today or tomorrow morning. | |
| 20:04:46 | mriedem | nkorabli: np, thanks for persisting on this one | |
| 20:13:10 | jroll | mriedem: sorry, I was being half serious about the forensics thing - i.e. that behavior being a feature for an admin taking a snapshot for purposes of criminal investigation or whatever | |
| 20:18:57 | mriedem | jroll: ack | |
| 20:19:15 | mriedem | sdague: on a server update, i see we call the server view builder show() method and return that in the response body, | |
| 20:19:31 | mriedem | what i'm unclear of, is does that trigger the server show extensions to add their special bits to the response? | |
| 20:22:25 | mriedem | for example, when i look at https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/extended_volumes.py | |
| 20:22:33 | mriedem | that extends the show and detail methods for the servers resource | |
| 20:22:49 | mriedem | but that will only be called on a GET for servers, right? | |
| 20:23:01 | mriedem | it would have nothing to do with the response for a POST or PUT | |
| 20:23:08 | mriedem | the server view builder just does it's own thing | |
| 20:25:05 | mriedem | reason i ask is we currently list os-extended-volumes:volumes_attached in the response for server create and update, and i think those are wrong | |
| 20:29:33 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: fix os-extended-volumes:volumes_attached in servers responses https://review.openstack.org/449809 | |
| 20:37:52 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/delete #1: no-ops https://review.openstack.org/438119 | |
| 20:39:56 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #2: functional https://review.openstack.org/438598 | |
| 20:40:14 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: power_on/off and reboot https://review.openstack.org/427380 | |
| 20:40:40 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #3: TaskFlow https://review.openstack.org/438729 | |
| 20:40:57 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #4: full flavor https://review.openstack.org/391288 | |
| 20:41:16 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: console https://review.openstack.org/409402 | |
| 20:42:00 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: SSP emphemeral disk support https://review.openstack.org/443189 | |
| 20:42:08 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: config drive https://review.openstack.org/409404 | |
| 20:43:20 | efried | adreznec esberglu mriedem ^^ Trivial rebases all the way up, because of a change to exception.py (In case you were in the middle of reviewing previous patch set.) | |
| 20:43:48 | mriedem | i was not | |
| 20:44:01 | efried | But, you know, just in case. | |
| 20:44:04 | mriedem | you seem unable to sit still on that bottom change | |
| 20:44:18 | mriedem | are you addressing feedback from within the powervm team or other? | |
| 20:44:32 | efried | No. It's only rebases. The tip of the master branch is a moving target. | |
| 20:45:43 | efried | Only real change since 3/16 (PS6) has been requirements update for new pypowervm release. | |
| 20:47:21 | efried | ...and flake8-import-order, which seems to have come in waves of increasing strictness. | |
| 20:49:24 | efried | Bottom two are the only ones that touch common files so far; so after those are in, we can expect a lot less no-op churn. | |
| 20:57:49 | mriedem | has this kind of spec come up before besides the spec around kilo/liberty to allow adding more servers to an existing group? https://review.openstack.org/#/c/389216/2 | |
| 20:58:13 | mriedem | ^ is basically, add config that reserves 'slots' in a server group for additional later servers in the same group for affinity to the same host | |
| 21:10:59 | mriedem | jroll: is this still an issue? https://review.openstack.org/#/c/321014/ | |
| 21:11:03 | mriedem | because that patch is really old | |
| 21:13:07 | openstackgerrit | Maxim Nestratov proposed openstack/nova master: libvirt: Virtuozzo containers config drive support https://review.openstack.org/449818 | |
| 21:22:06 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Empty machine_id file causes get_sysinfo_serial_auto to fail. https://review.openstack.org/343943 | |
| 21:25:03 | mriedem | tjones: someone from the vmware subteam might want to pick this up if it's still valid https://review.openstack.org/#/c/402256/ | |
| 21:30:23 | tjones | sure. looking | |
| 21:31:11 | tjones | yes we will. i'll ping yang to pick it back up | |
| 21:52:14 | tjones | this one too https://review.openstack.org/#/c/270116/ | |
| 21:56:25 | efried | mdbooth Anyone reporting [[os.fsync(data.fileno()) OSError: [Errno 22] Invalid argument]] since https://review.openstack.org/#/c/443583/ ? | |
| 22:03:56 | efried | (Thing is, I don't know how that call can get EINVAL. Different errors occur if, e.g., the fileno is bogus, or the file is closed, or anything else I can come up with.) | |
| 22:16:38 | eandersson_ | Anyone experienced an issue with large images causing nova to try to fail to set up the network, and ends up creating two neutron ports (since it fails with a neutron timeout)? | |
| 22:17:12 | eandersson_ | > nova.compute.manager ConnectTimeout: Request to http://netutron:9696/v2.0/ports.json timed out | |
| 23:56:37 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Regression test for local delete with an attached volume https://review.openstack.org/449334 | |
| 23:56:38 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Set size/status during image create with FakeImageService https://review.openstack.org/449702 | |
| 23:58:53 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Regression test for local delete with an attached volume https://review.openstack.org/449334 | |
| 23:58:53 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Set size/status during image create with FakeImageService https://review.openstack.org/449702 | |
| #openstack-nova - 2017-03-25 | |||
| 00:33:32 | mriedem | superdan: your favorite question on a friday night at 5:30 - we don't ever delete InstanceMapping records do we? because if we did, you couldn't get details on a soft deleted instance from the cell - in the instances.deleted != 0 sense | |
| 00:33:45 | mriedem | even though i see there is an InstanceMapping.destroy() method, i don't see that ever used | |
| 00:36:44 | melwitt | I didn't think we delete InstanceMappings | |
| 00:36:54 | mriedem | i'm not seeing that we do anywhere | |
| 00:37:06 | mriedem | almost wonder if we should drop the InstanceMapping.destroy() method from the object... | |
| 00:37:33 | mriedem | or at least add a comment to not ever use it for real because it breaks the API | |
| 00:38:57 | melwitt | maybe yeah. those interfaces were written well before they were ever used, by months at least | |
| 00:49:24 | mriedem | that all came up in https://review.openstack.org/#/c/441692/ | |
| 00:49:41 | mriedem | and the pain that is what do we do about listing deleted instances if we're using searchlight which doesn't keep track of indexes for deleted instances | |
| 00:50:09 | mriedem | i think i'm going to submit yet another forum topic for using searchlight in nova-api since it's a fairy hairy change | |
| 00:50:14 | mriedem | *fairly | |
| 00:50:15 | mriedem | ha | |
| 01:04:37 | openstackgerrit | 龚肖 proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/449537 | |
| 01:55:26 | superdan | melwitt: we delete instance mappings | |
| 01:55:52 | superdan | we just went over this with bauwser I think | |
| 01:56:02 | superdan | we don't in the soft delete case or something | |
| 01:56:39 | superdan | wait, I think I'm thinking of reqspec | |
| 01:56:49 | superdan | I guess we have to have IMs until purge | |
| 01:57:13 | superdan | which is what we discussed with bauwser yeah, purging those at instance purge time since that's when we know we can delete them | |
| 01:57:17 | melwitt | superdan: oh, I didn't know. I did find one destroy() so far in conductor/manager that happens in the case of a HostMappingNotFound | |
| 01:57:31 | superdan | melwitt: yeah we have to delete it for certain race conditions | |
| 01:57:36 | superdan | but not normally I think | |
| 01:57:40 | superdan | because ^ | |
| 01:57:41 | melwitt | gotcha | |
| 01:57:54 | superdan | but we can delete reqspecs, but never do | |
| 01:58:08 | superdan | and that is on his list also | |
| 02:03:41 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add description to policies in pause_server.py https://review.openstack.org/449478 | |
| 02:04:55 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add description to policies in server_groups.py https://review.openstack.org/449486 | |
| 02:10:53 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add description to policies in lock_server.py https://review.openstack.org/448468 | |
| 02:23:38 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Don't delete neutron port when attach failed https://review.openstack.org/403568 | |
| 03:11:30 | openstackgerrit | 龚肖 proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/449537 | |
| 03:11:48 | openstackgerrit | Eric Fried proposed openstack/nova master: Don't delete neutron port when attach failed https://review.openstack.org/403568 | |
| 06:06:46 | openstackgerrit | Bing Li proposed openstack/python-novaclient master: Remove log translations https://review.openstack.org/448336 | |
| 06:50:55 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix api-ref for adminPass behavior https://review.openstack.org/449450 | |
| 08:45:55 | Kevin_Zheng | mriedm: semms neutron port also not cleaned https://bugs.launchpad.net/nova/+bug/1675570 | |
| 08:45:55 | openstack | Launchpad bug 1675570 in OpenStack Compute (nova) "Volumes are not detached when deleting shelved offloaded servers" [High,Triaged] | |
| 08:46:43 | Kevin_Zheng | mriedem: ^ | |
| 08:47:18 | Kevin_Zheng | mriedem: I added my console Log to the report | |
| 08:52:58 | openstackgerrit | zhichao zhu proposed openstack/nova master: Fix some reST field lists in docstrings https://review.openstack.org/449891 | |
| 10:21:17 | openstackgerrit | zhichao zhu proposed openstack/nova master: Fix some reST field lists in docstrings https://review.openstack.org/449891 | |
| 12:57:45 | openstackgerrit | Zhou Jianming proposed openstack/nova master: code comment redundant https://review.openstack.org/449934 | |