| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-02-02 | |||
| 01:54:28 | melwitt | "Title level inconsistent" I'm trying to figure out what level is one step smaller than ~~~~~ for a section | |
| 01:59:15 | mriedem | ----- ? | |
| 01:59:56 | mriedem | melwitt: ^ | |
| 02:00:02 | mriedem | Cells V2 ======== Manifesto ~~~~~~~~~ Proposal -------- | |
| 02:00:13 | melwitt | yeah, I think you're right. thanks | |
| 02:03:17 | openstackgerrit | melanie witt proposed openstack/nova master: Add step-by-step examples for Cells V2 setup https://review.openstack.org/427981 | |
| 02:04:49 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add release note for filter/sort whitelist https://review.openstack.org/427978 | |
| 02:04:53 | melwitt | mriedem: ^ | |
| 02:05:22 | gmann | Kevin_Zheng: looks like you are back, updating doc patch as per mriedem comments ? | |
| 02:08:13 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Trivial Fix typo in schema https://review.openstack.org/427983 | |
| 02:08:44 | Kevin_Zheng | gmann: yeah I'm trying to | |
| 02:09:33 | gmann | Kevin_Zheng: thanks, please let me know any help. we should get those last bits in this week | |
| 02:10:32 | Kevin_Zheng | gmann: sure | |
| 02:10:35 | Kevin_Zheng | Thanks | |
| 02:15:15 | gmann | Kevin_Zheng: terminated_at is fixed in https://review.openstack.org/#/c/427964/ | |
| 02:15:55 | Kevin_Zheng | Yeah just saw it | |
| 02:16:23 | gmann | ok | |
| 02:20:42 | openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Enable cold migration with target host https://review.openstack.org/426636 | |
| 02:21:09 | openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732 | |
| 02:21:58 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Fix deprecated proxy API parameters https://review.openstack.org/416882 | |
| 02:22:03 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Use ListOfIntegersField in oslo.versionedobjects https://review.openstack.org/423912 | |
| 02:22:08 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameters and examples in aggregate API https://review.openstack.org/425512 | |
| 02:22:14 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameters in os-hosts.inc https://review.openstack.org/416444 | |
| 02:22:30 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix doc generation warnings https://review.openstack.org/425549 | |
| 02:22:47 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Refactor a test method including 3 test cases https://review.openstack.org/394738 | |
| 02:24:05 | mriedem | melwitt: comments inline | |
| 02:24:21 | mriedem | melwitt: i worry a little about duplicating some of the stuff that dan wrote above that in the first time setup section | |
| 02:26:03 | melwitt | thanks | |
| 02:28:44 | melwitt | mriedem: I thought about that a little too. I was thinking we should probably have short "do these steps" things for people to look at to help understand the different commands. well, based on the convos we had with operators in the channel about it | |
| 02:29:08 | mriedem | yeah, | |
| 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 | |