Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-08-03
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 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: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:29:02 openstack Launchpad bug 1700583 in OpenStack Compute (nova) "No volume Block Device Mapping in assisted snapshot with Quobyte" [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: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 mdbooth My concern would exclusively be around potential for weirdness if an instance reboots and gets very slightly different domain xml.
10:46:50 lyarwood running instances wouldn't be touched until they reboot and the domain is recreated
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
12:06:34 sdague stephenfin: if you want I can
12:06:49 sdague some of them are big enough that one per file seemed fine
12:06:58 sdague and less likely to merge conflict the world and redo them all
12:07:25 sdague the don't use much test resource
12:08:17 sdague stephenfin: the space you point out here doesn't matter for html rendering - https://review.openstack.org/#/c/490431
12:09:54 openstackgerrit Matthew Booth proposed openstack/nova master: Fix scope of errors_out_migration in resize_instance https://review.openstack.org/487495
12:09:55 openstackgerrit Matthew Booth proposed openstack/nova master: Automatically revert resize which fails on destination https://review.openstack.org/462521
12:09:55 openstackgerrit Matthew Booth proposed openstack/nova master: Split Compute.errors_out_migration into a separate contextmanager https://review.openstack.org/485734
12:09:56 openstackgerrit Matthew Booth proposed openstack/nova master: fake_notifier: Refactor wait_for_versioned_notification https://review.openstack.org/489637
12:09:56 openstackgerrit Matthew Booth proposed openstack/nova master: Functional test for auto revert resize https://review.openstack.org/489654
12:09:57 openstackgerrit Matthew Booth proposed openstack/nova master: Ensure errors_out_migration errors out migration https://review.openstack.org/479802
12:09:57 openstackgerrit Matthew Booth proposed openstack/nova master: Fix scope of errors_out_migration in finish_resize https://review.openstack.org/487515
12:10:44 openstackgerrit Sean Dague proposed openstack/nova master: fix list rendering in policy-enforcement https://review.openstack.org/490451
12:12:39 sdague stephenfin: that looks like all the top level ones at least
12:17:37 openstackgerrit sahid proposed openstack/nova master: libvirt: update the active migrations DS to support migration result https://review.openstack.org/409260
12:17:39 openstackgerrit sahid proposed openstack/nova master: libvirt: ensure live-migration success https://review.openstack.org/409261
12:18:11 openstackgerrit Sean Dague proposed openstack/nova master: reflow rpc doc to 80 columns https://review.openstack.org/490455
12:18:58 openstackgerrit Sean Dague proposed openstack/nova master: reflow rpc doc to 80 columns https://review.openstack.org/490455
12:21:52 moshele sahid: hi can you review https://review.openstack.org/#/c/398265/ ?
12:29:50 stephenfin sdague, asettle: Oh, did we change the docs job to only build docs now?
12:30:11 asettle Uhh I don't think so, no
12:30:15 asettle I asked about it
12:30:27 asettle You can adjust it, but there was a recommendation not to keep it
12:31:03 stephenfin asettle: I ask because I note that https://review.openstack.org/#/c/490448/ has returned Jenkins results for only two jobs
12:31:06 stephenfin so something changed
12:31:27 asettle Oh, weird... I didn't touch that (I know, I know, I didn't stick my nose in this one :P)
12:31:38 asettle stephenfin: maybe check in infra
12:32:14 asettle It is only a white space edit... hmmm
12:39:02 sambetts jaypipes, edleafe: I think we might need to do this to fix the ironic resource class logic: https://review.openstack.org/#/c/490459/
12:41:22 openstackgerrit Balazs Gibizer proposed openstack/nova master: WIP: resize with custom resource https://review.openstack.org/490461

Earlier   Later