| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-02-02 | |||
| 02:29:45 | mriedem | i don't have a great solution to not be redundant but at the same time, have everything anyone would ever want or need to know in a single section and not make that so overly complicate with if/else conditions that no one can understand it | |
| 02:29:48 | mriedem | can't win really | |
| 02:30:07 | mriedem | i've always failed at process/guide docs like that | |
| 02:30:25 | mriedem | the balance between too much and too little info, because no matter how you write it, someone won't be able to follow it | |
| 02:30:34 | mriedem | lived with that for years in my last project before openstack | |
| 02:31:22 | melwitt | yeah. what's there already is a thorough explanation of all the things, but for the sort of roll up or quick reference, we don't have anything | |
| 02:31:41 | melwitt | maybe that page isn't the best place for it but I wasn't sure where else we would put it | |
| 02:33:19 | openstackgerrit | Hidekazu Nakamura proposed openstack/nova master: Do not attempt volume swap when guest is stopped https://review.openstack.org/389798 | |
| 02:41:59 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle NotImplementedError in _process_instance_vif_deleted_event https://review.openstack.org/427974 | |
| 02:51:21 | mriedem | melwitt: so i thought about this on the way home for some reason https://review.openstack.org/#/c/355091/ | |
| 02:51:34 | mriedem | i don't really want to open a can of worms, | |
| 02:51:56 | mriedem | but i'm not sure if what we have for placement right now resolves that with reporting 0 DISK_GB for boot from volume or not | |
| 02:52:03 | mriedem | i think it does | |
| 02:53:33 | melwitt | mriedem: yeah, I skimmed some ML threads on that and it wasn't clear to me if the boot from volume disk reporting problem is solved yet with placement | |
| 02:53:41 | mriedem | "We're already reporting proper allocation information for instances with no disk (volume-backed and others) and as soon as we look to the placement API in the scheduler we'll stop suffering from this problem." | |
| 02:53:58 | mriedem | ^ as long as the scheduler is using placement, we're filtering on DISK_GB now | |
| 02:54:56 | melwitt | mriedem: is it now? I've been way out of the loop on what's merged | |
| 02:56:01 | mriedem | melwitt: yeah https://review.openstack.org/#/c/417961/ | |
| 02:56:17 | mriedem | if the minimum nova-compute service version in the deployment is new enough, the filter scheduler will use placement now, | |
| 02:56:21 | mriedem | including filtering on DISK_GB | |
| 02:56:40 | mriedem | https://review.openstack.org/#/c/417961/41/nova/scheduler/filter_scheduler.py@139 | |
| 02:57:52 | melwitt | okay, cool | |
| 02:58:19 | melwitt | well, that's a win, glad to see that | |
| 02:58:52 | mriedem | https://review.openstack.org/#/c/427367/ adds some upgrade notes to the placement devref | |
| 02:58:55 | mriedem | might help you catch up a bit | |
| 02:59:17 | melwitt | thanks | |
| 02:59:22 | mriedem | i missed pretty much all of the moving instance create to conductor, but after the last 24 hours of bugs i feel like i've caught up :) | |
| 03:00:10 | melwitt | I wonder how it makes sure volume backed instance doesn't request the flavor's amount of disk when rebuilding etc | |
| 03:00:33 | melwitt | haha, yeah really | |
| 03:11:05 | mriedem | got some other cells v2 related docs changes here, small ones https://review.openstack.org/#/q/topic:man+project:openstack/nova+status:open | |
| 03:54:26 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix typo in help for discover_hosts_in_cells_interval https://review.openstack.org/427997 | |
| 04:21:23 | mriedem | gibi: https://review.openstack.org/#/c/427647/ apparently the cinder people have figured out how to short circuit sending notifications if they aren't enabled | |
| 04:22:01 | mriedem | https://review.openstack.org/#/c/426547 | |
| 05:03:30 | archies | @team, I am facing an interesting issue while attaching a volume. In libvirt the error is "operation failed: open disk image file failed" | |
| 05:03:33 | archies | Any idea? | |
| 05:42:57 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add release note for filter/sort whitelist https://review.openstack.org/427978 | |
| 05:53:07 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add release note for filter/sort whitelist https://review.openstack.org/427978 | |
| 06:01:20 | trinaths | Hi- While enabling VM with huge pages, I get the following error. Insufficient compute resources: Requested instance NUMA topology cannot fit the given host NUMA topology. I’m using devstack (stable/newton) and nova (6.0.0). I have configured hugepages and flavors following the guide at http://docs.openstack.org/admin-guide/compute-huge-pages.html . Here is the configuration of the host. http://paste.openstack.org/sh | |
| 06:11:05 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add release note for filter/sort whitelist https://review.openstack.org/427978 | |
| 07:16:58 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add release note for filter/sort whitelist https://review.openstack.org/427978 | |
| 07:30:55 | markus_z | mikal: Would you maybe take a look at this newton backport? https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/newton+topic:%22bug+1455252%22 | |
| 07:31:25 | markus_z | mikal: ^ lyarwood already checked it. | |
| 07:43:48 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | |
| 08:35:29 | gibi | Q~. | |
| 08:35:43 | gibi | sorry ^^ | |
| 08:35:54 | gibi | mriedem: thanks for the link, I will check the cinder solution | |
| 08:49:33 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | |
| 08:59:39 | gema | can someone point me in the right direction regarding how to update this page: http://docs.openstack.org/developer/nova/feature_classification.html | |
| 08:59:54 | gema | are there any requirements in terms of testing that need to be met or is this manually maintained? | |
| 09:05:02 | BobBall | johnthetubaguy: OK - so the XenServer CI problem is that https://review.openstack.org/#/c/396269/ merged breaking support for the Ubuntu we're using. | |
| 09:15:07 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Remove redundant bdm serial mangling and saving during swap_volume https://review.openstack.org/427364 | |
| 09:16:48 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926 | |
| 09:16:52 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926 | |
| 09:19:48 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | |
| 09:19:54 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | |
| 09:21:56 | johnthetubaguy | BobBall: really :S | |
| 09:22:06 | BobBall | Indeed. | |
| 09:23:00 | johnthetubaguy | BobBall: how did that break the uploads from inside the XenAPI plugin? | |
| 09:23:22 | BobBall | It broke Cinder which meant that the glance upload couldn't create the volume it was going to upload to | |
| 09:23:49 | johnthetubaguy | but I didn't think the xenapi plugin is used for any cinder volumes? | |
| 09:24:19 | johnthetubaguy | oh, it breaks all glance uploads in the destination sense? | |
| 09:24:38 | BobBall | uhhhh ok now you've got me confused... | |
| 09:29:48 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | |
| 10:04:39 | mdbooth | lyarwood: Thinking about https://review.openstack.org/#/c/427364/ | |
| 10:05:36 | mdbooth | Specifically about the chunk in libvirt driver which you delete that we discussed yesterday | |
| 10:05:59 | mdbooth | I think the fundamental issue there is that libvirt volume drivers mutate connection_info | |
| 10:06:50 | mdbooth | The example which is specifically called out is LibvirtISCSIVolumeDriver. This adds the local device path that created to connect to the remote iscsi volume to connection_info. | |
| 10:07:21 | lyarwood | mdbooth: that's still added after my change | |
| 10:07:27 | mdbooth | But this isn't strictly related to the volume connection. | |
| 10:07:37 | lyarwood | mdbooth: we pass the modified connection_info back to the compute layer and update the BDM with it | |
| 10:07:41 | mdbooth | lyarwood: Yeah, I'm being expansive here :) | |
| 10:07:44 | lyarwood | ah | |
| 10:07:46 | lyarwood | :D | |
| 10:08:23 | mdbooth | So, anyway, I think the bug here is in LibvirtISCSIVolumeDriver and all other places which mutate connection_info | |
| 10:08:23 | openstackgerrit | Tomi Juvonen proposed openstack/nova-specs master: Add Host Details https://review.openstack.org/428070 | |
| 10:08:30 | mdbooth | They need to stash that metadata somewhere else. | |
| 10:08:52 | mdbooth | I also disapprove of even looking inside connection_info, tbh | |
| 10:09:05 | lyarwood | mdbooth: yeah that's all going to change with cinder v3 anyway | |
| 10:09:15 | mdbooth | If we could have cinder pass it to us as an opaque packed C struct, that would be awesome | |
| 10:09:28 | mdbooth | The only thing you can do with this data is give it back to cinder | |
| 10:09:29 | lyarwood | mdbooth: but I can't see how that relates to this issue tbh | |
| 10:09:59 | mdbooth | It relates because it means we *can't* refetch connection info from cinder after cinder also mutates it | |
| 10:10:29 | mdbooth | Because Nova futzes with it, we're now stuck trying to second guess one or the other. | |
| 10:11:05 | lyarwood | mdbooth: ah right you're talking about the cinder retype situation | |
| 10:11:20 | mdbooth | cinder volume migration | |
| 10:11:31 | lyarwood | mdbooth: yeah same thing | |
| 10:11:49 | mdbooth | volume migration is meaningful to me :) | |
| 10:12:03 | lyarwood | mdbooth: I need to drop now, can we pick this up when I get back? | |
| 10:12:08 | mdbooth | lyarwood: Sure thing. | |
| 10:12:23 | lyarwood | mdbooth: cool | |
| 10:13:21 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix access_ip_v4/6 filters params for servers filter https://review.openstack.org/428071 | |
| 10:13:57 | gmann | johnthetubaguy: mriedem alex_xu seems like we missed another bit in filter/sort thing - https://bugs.launchpad.net/nova/+bug/1661195 | |
| 10:13:57 | openstack | Launchpad bug 1661195 in OpenStack Compute (nova) "Servers filter by access_ip_v4 does not filter servers" [Undecided,In progress] - Assigned to Ghanshyam Mann (ghanshyammann) | |
| 10:14:18 | gmann | johnthetubaguy: alex_xu mriedem this is fix - https://review.openstack.org/428071 | |
| 10:14:45 | johnthetubaguy | gmann: oh, you mean we should allow them, because we used to allow them | |
| 10:15:02 | gmann | but different filed name in filter and sort than what API represent is odd to me | |
| 10:15:16 | gmann | johnthetubaguy: i think so, or no ? | |
| 10:15:32 | johnthetubaguy | gmann: don't they now match what the API has? | |
| 10:15:34 | gmann | i may be missed the discussion on that if any | |
| 10:16:15 | gmann | johnthetubaguy: oh so you mean new list will match the what API has either it used to work before or not | |