| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-03 | |||
| 02:28:36 | gmann | yes | |
| 02:28:39 | Kevin_Zheng | and delete is delete there | |
| 02:28:59 | Kevin_Zheng | horizon could not get the flavor from db using id | |
| 02:30:51 | gmann | yea but GET instance shows the flavor id (<2.47) | |
| 02:31:05 | Kevin_Zheng | yeah, | |
| 02:31:31 | gmann | so instance detail in horizon will have old id | |
| 02:31:41 | Kevin_Zheng | correct | |
| 02:31:56 | gmann | when horizon try to fetch the old flavor ? | |
| 02:32:51 | Kevin_Zheng | yes | |
| 02:33:14 | gmann | i mean why horizon try to get old flavor? | |
| 02:34:09 | Kevin_Zheng | because instance saved the old flavor id | |
| 02:34:23 | Kevin_Zheng | and return it to horizon | |
| 02:34:36 | Kevin_Zheng | horizon use it to show the flavor details | |
| 02:34:53 | gmann | show flavor details in Instance detail tab? | |
| 02:35:10 | Kevin_Zheng | yeah | |
| 02:35:21 | Kevin_Zheng | also the instances list tab | |
| 02:35:48 | gmann | humm | |
| 02:36:13 | gmann | so with >2.47 horizon do not need to do that as instance can give flavor detail | |
| 02:36:20 | Kevin_Zheng | maybe | |
| 02:36:36 | gmann | so 2.47 actually solved the problem ? | |
| 02:36:55 | Kevin_Zheng | yeah it could | |
| 02:37:05 | gmann | i mean just show the what instance return either it exit or deleted | |
| 02:37:25 | Kevin_Zheng | yeah | |
| 02:44:05 | gmann | yea but that will be old stale thing that people would not find such flavor in system as horizon modified it | |
| 02:44:26 | gmann | may be you are asking about disabling this feature is good idea | |
| 02:44:46 | Kevin_Zheng | Yeah, I can think of one usecase of this | |
| 02:45:06 | Kevin_Zheng | since flavor have alot of fields | |
| 02:45:20 | gmann | yea | |
| 02:45:21 | Kevin_Zheng | maybe people want to only change one of the field | |
| 02:45:42 | Kevin_Zheng | so they don't want to create a new one and do all the input | |
| 02:46:27 | Kevin_Zheng | I was thinking, maybe instead of delete-and-create, horizon can provide a copy-and-modify | |
| 02:46:37 | gmann | yea | |
| 02:46:39 | Kevin_Zheng | leave the deleting to user | |
| 02:47:00 | gmann | because flavor id changes and thats problem for other instance operation like resize etc | |
| 02:47:09 | Kevin_Zheng | yeah | |
| 02:47:27 | Kevin_Zheng | and people don't know that editing is actually deleting and creating | |
| 02:48:29 | gmann | true | |
| 02:49:02 | Kevin_Zheng | replies on my maillist post is welcomed :) | |
| 02:52:36 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_certs to instance_extra https://review.openstack.org/457711 | |
| 02:53:45 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_certs to Instance object https://review.openstack.org/489408 | |
| 02:58:37 | openstackgerrit | zhangdaolong proposed openstack/nova master: Add debug log for fail to get console https://review.openstack.org/490302 | |
| 03:10:11 | openstackgerrit | zhangdaolong proposed openstack/nova master: Keep the code consistent https://review.openstack.org/490304 | |
| 03:38:22 | openstackgerrit | Tony Breeds proposed openstack/nova master: Testing reno 2.5.0 https://review.openstack.org/490313 | |
| 04:01:36 | openstackgerrit | Tony Breeds proposed openstack/nova master: Testing reno 2.5.0 https://review.openstack.org/490321 | |
| 04:55:12 | openstackgerrit | Jonghan Park proposed openstack/nova master: Network should be distinguished by UUID instead of Name https://review.openstack.org/490332 | |
| 06:07:36 | trinaths | guys! I get this error, Instance failed to spawn: VolumeDeviceNotFound . Do I need cinder volumes mandatory for nova-compute to spawn VMS? | |
| 06:22:49 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix getting instance bdms in multiple cells https://review.openstack.org/490340 | |
| 06:40:36 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | |
| 06:55:44 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | |
| 07:03:27 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | |
| 08:17:13 | openstackgerrit | Jacek Tomasiak proposed openstack/nova master: ironic: Use internal API endpoint by default https://review.openstack.org/489537 | |
| 08:35:26 | openstackgerrit | Jacek Tomasiak proposed openstack/nova master: ironic: Use internal API endpoint by default https://review.openstack.org/489537 | |
| 09:13:28 | openstackgerrit | zhangdaolong proposed openstack/nova master: Add debug log for fail to get console https://review.openstack.org/490302 | |
| 09:48:46 | mamandle | alex_xu: bauzas: can you please take a look at https://review.openstack.org/#/c/489848/1 which is a backport for Ocata? Thanks. | |
| 10:24:43 | sambetts | edleafe: Hi, someone testing out Ironic+resource classes accidentally stubbled onto this error, we've put together a bug report for you: https://bugs.launchpad.net/nova/+bug/1708424 | |
| 10:24:43 | openstack | Launchpad bug 1708424 in OpenStack Compute (nova) "When a flavor has resource extra_specs disabling all standard fields, nova tries to make a request to the placements API with no resources" [Undecided,New] | |
| 10:29:02 | kaisers2 | mriedem: Hi! FYI regarding your email (http://lists.openstack.org/pipermail/openstack-dev/2017-July/120120.html) i found an issue (https://bugs.launchpad.net/devstack/+bug/1700583), took me quite a while to find the connection, so sorry for beeing this late | |
| 10:29:02 | openstack | Launchpad bug 1700583 in OpenStack Compute (nova) "No volume Block Device Mapping in assisted snapshot with Quobyte" [Undecided,New] | |
| 10:35:50 | mdbooth | lyarwood: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/ocata+topic:bug/1686116 | |
| 10:36:12 | mdbooth | The code isn't invasive | |
| 10:37:11 | mdbooth | Just putting my tinfoil hat on, trying to think what this would change to an existing instance that was rebooted, for eg | |
| 10:39:11 | mdbooth | disks would have a disk_unit where previously they didn't | |
| 10:43:02 | mdbooth | lyarwood: https://review.openstack.org/#/c/481137/ | |
| 10:43:19 | mdbooth | Disks will have a device address, but only if they've been previously parsed from the domain xml? | |
| 10:44:41 | mdbooth | lyarwood: Anyway, the question was whether it's fit for stable. | |
| 10:44:52 | mdbooth | lyarwood: Is there a specific driver to backport this to stable? | |
| 10:45:01 | robcresswell | sdague: Followed the email chain; my thought on the Horizon "feature" is to gate it behind a setting, disable by default and warning the hell out of it. | |
| 10:45:03 | lyarwood | mdbooth: yeah correct re the question | |
| 10:45:06 | mdbooth | I suspect that it won't break anything. | |
| 10:45:34 | mdbooth | However, if I was of a nervous disposition I wouldn't take it. | |
| 10:45:46 | mdbooth | The code's fine. | |
| 10:46:08 | lyarwood | right, I can't see anything in master that would handle a corner case for active instances | |
| 10:46:25 | lyarwood | so having it in stable should be the same | |
| 10:46:47 | robcresswell | sdague: Referring the edit flavor issue, if that wasnt apparent. I'm not a fan of UI's inventing features that APIs dont support, so I'm on board with getting rid of it. So I'll make the settings changes and deprecate it for Pike. | |
| 10:46:50 | lyarwood | running instances wouldn't be touched until they reboot and the domain is recreated | |
| 10:46:50 | mdbooth | My concern would exclusively be around potential for weirdness if an instance reboots and gets very slightly different domain xml. | |
| 10:47:30 | mdbooth | It's probably fine, though. | |
| 10:47:41 | sdague | robcresswell: works for me | |
| 10:48:05 | mdbooth | It's just a question about how conservative we are with stable, and user's reasonable expectations around such changes with minor updates within stable. | |
| 10:48:07 | robcresswell | sdague: Cool. Sorry for the trouble thats caused you. I wasn't really aware of the issue. | |
| 10:48:30 | sdague | robcresswell: or at least a popup warning. I think the way it was presented to users they didn't realize that it wasn't a native feature and that it would cause issues | |
| 10:48:48 | sdague | robcresswell: yeh, honestly, a lot of the nova team didn't realize it was a thing until atlanta ptg | |
| 10:55:36 | robcresswell | sdague: Yeah, fair enough. At least we can start solving it now. | |
| 10:55:56 | robcresswell | sdague: If anything else like this comes up, feel free to ping me about it. I do love to remove code. | |
| 10:56:55 | sdague | will do | |
| 11:08:00 | maestropandy | Please vote (+3) for our presentation submitted for openstack sydney 2017. Today last day for voting, requesting all to vote. OpenStack cloud storage - Advanced performance tuning & operational best practices with Ceph https://www.openstack.org/summit/sydney-2017/vote-for-speakers/#/19056 ##### Docker Networking over OpenStack Cloud – Challenges, Solutions & Road-map >> https://www.openstack.org/summit/sydney-2017/vote-for-speakers/#/1 | |
| 11:08:51 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Avoid error log on 405 response https://review.openstack.org/490021 | |
| 11:30:43 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091 | |
| 11:32:13 | openstackgerrit | Sean Dague proposed openstack/nova master: Fix lists in process doc https://review.openstack.org/490431 | |
| 11:33:42 | stephenfin | maestropandy: I would imagine this also applies to IRC http://lists.openstack.org/pipermail/openstack-dev/2017-July/120121.html Please don't do it | |
| 11:42:33 | openstackgerrit | Sean Dague proposed openstack/nova master: Fix whitespace in rest_api_version_history https://review.openstack.org/490435 | |
| 11:47:03 | openstackgerrit | Sean Dague proposed openstack/nova master: Fix list rendering in code-review.rst https://review.openstack.org/490437 | |
| 11:51:37 | openstackgerrit | Sean Dague proposed openstack/nova master: fix list rendering in rpc doc https://review.openstack.org/490442 | |
| 11:53:56 | sdague | stephenfin: so I'm just going through the toc list and looking for the blockquotes popping out and fixing | |
| 12:00:37 | stephenfin | sdague: Done | |
| 12:00:54 | openstackgerrit | Sean Dague proposed openstack/nova master: Fix list rendering in bdm doc. https://review.openstack.org/490446 | |
| 12:05:07 | openstackgerrit | Sean Dague proposed openstack/nova master: fix list rendering in aggregates https://review.openstack.org/490448 | |
| 12:05:52 | stephenfin | sdague: That's a lot of patches | |
| 12:06:10 | stephenfin | tbh, if they're all that simple, I think you could squash em | |
| 12:06:23 | openstackgerrit | Sean Dague proposed openstack/nova master: fix list rendering in cells https://review.openstack.org/490449 | |