| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-11-21 | |||
| 01:11:14 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add confirm_snapshot_based_resize_at_source compute method https://review.opendev.org/637058 | |
| 01:11:15 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Confirm cross-cell resize from the API https://review.opendev.org/637316 | |
| 01:11:15 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add confirm_snapshot_based_resize conductor RPC method https://review.opendev.org/637075 | |
| 01:11:16 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Deal with cross-cell resize in _remove_deleted_instances_allocations https://review.opendev.org/639453 | |
| 01:11:16 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add revert_snapshot_based_resize_at_dest compute method https://review.opendev.org/637630 | |
| 01:11:17 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add RevertResizeTask https://review.opendev.org/638046 | |
| 01:11:17 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add finish_revert_snapshot_based_resize_at_source compute method https://review.opendev.org/637647 | |
| 01:11:18 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Revert cross-cell resize from the API https://review.opendev.org/638048 | |
| 01:11:18 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add revert_snapshot_based_resize conductor RPC method https://review.opendev.org/638047 | |
| 01:11:19 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add archive_deleted_rows wrinkle to cross-cell functional test https://review.opendev.org/651650 | |
| 01:11:19 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Confirm cross-cell resize while deleting a server https://review.opendev.org/638268 | |
| 01:11:20 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add CrossCellWeigher https://review.opendev.org/614353 | |
| 01:16:35 | gmann | efried: any idea or have you seen 'openstack:' as resource provider name in https://zuul.opendev.org/t/openstack/build/b26bd7fc82d94d4cac97b5481623b629/log/logs/grenade.sh.txt.gz#30397 | |
| 01:16:55 | gmann | this is grenade job on octavia which is failing due to that when moving to py3. | |
| 05:23:13 | openstackgerrit | Merged openstack/nova stable/pike: Only nil az during shelve offload https://review.opendev.org/693839 | |
| 06:36:10 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/694717 | |
| 08:10:09 | bauzas | good morning Nova | |
| 08:16:32 | gibi | bauzas: good morning | |
| 08:16:51 | bauzas | :) | |
| 08:50:26 | openstackgerrit | ya.wang proposed openstack/nova master: Handle instance crash event in libvirt driver https://review.opendev.org/695369 | |
| 09:25:39 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Rewrite quotas documentation https://review.opendev.org/667165 | |
| 09:26:36 | stephenfin | bauzas: Morning. Any chance you could look at one or two of the "die, nova-network, die" patches I have up today? Like, say, this really easy one https://review.opendev.org/#/c/684345/ ? | |
| 09:26:47 | bauzas | lol ok | |
| 09:28:32 | openstackgerrit | Tushar Patil proposed openstack/nova-specs master: Allow compute nodes to use DISK_GB from shared storage RP https://review.opendev.org/650188 | |
| 09:34:15 | openstackgerrit | Stephen Finucane proposed openstack/nova master: zuul: Remove unnecessary 'USE_PYTHON3' https://review.opendev.org/695380 | |
| 09:45:22 | openstackgerrit | Sylvain Bauza proposed openstack/nova stable/stein: Don't delete compute node, when deleting service other than nova-compute https://review.opendev.org/695381 | |
| 09:49:08 | openstackgerrit | Sylvain Bauza proposed openstack/nova stable/rocky: Don't delete compute node, when deleting service other than nova-compute https://review.opendev.org/695382 | |
| 09:51:44 | openstackgerrit | Sylvain Bauza proposed openstack/nova stable/queens: Don't delete compute node, when deleting service other than nova-compute https://review.opendev.org/695383 | |
| 10:32:34 | yaawang | mdbooth: Hello, can you review the spec again, I'd updated it :) https://review.opendev.org/#/c/693655/ | |
| 10:52:05 | kashyap | yaawang: Hi, on the "no_performance_impact" name -- I still find it too broad and sweeping | |
| 10:52:15 | kashyap | yaawang: Thanks for addressing my feedback, though! | |
| 10:53:14 | kashyap | I agree, we don't want to expose "hypervisor"-specific features. (BTW, we are abusing the term "hyerpvisor" here to include QEMU; the term normally includes KVM/Kernel area only.) | |
| 10:53:22 | kashyap | Maybe "no_migration_perf_impact" | |
| 10:57:01 | kashyap | But just plain "no_performance_impact" is just _awful_ name (Cc: mdbooth.) At least make it: "no_migration_perf_impact" | |
| 11:10:15 | yaawang | kashyap: Agree, "no_performance_impact" sounds include live-migrate and migrate. How about "no_perf_impact_live_migration"? | |
| 11:12:09 | kashyap | yaawang: Yeah, I thought of including "live" as well; almost good - but reverse it: "no_live_emigration_perf_impact" | |
| 11:12:55 | kashyap | Typo: s/emigration/migration/ | |
| 11:14:22 | kashyap | yaawang: Commented on the change. I'd say, let's go with the above ("no_live_migration_perf_impact"). | |
| 11:17:24 | yaawang | kashyap: Thanks, need mdbooth to post his comment. | |
| 12:03:41 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/pike: Explicitly fail if trying to attach SR-IOV port https://review.opendev.org/695408 | |
| 12:16:39 | gibi | bauzas: you can send this to the age too https://review.opendev.org/#/c/686808 | |
| 12:34:50 | openstackgerrit | Wei Hui proposed openstack/nova master: bugfix device_type=type-PCI passthrough failed https://review.opendev.org/695416 | |
| 12:39:19 | gibi | mriedem, stephenfin: thanks for the laugh https://review.opendev.org/#/c/489267/2/nova/network/neutronv2/api.py@181 | |
| 12:41:09 | gibi | stephenfin: btw you have now a lot of +2s on your nova-net removal series :) | |
| 12:42:29 | efried | gmann: looking... | |
| 12:49:22 | mriedem | gibi: heh, anytime | |
| 12:51:07 | efried | gmann: that log file isn't loading up for me :( | |
| 12:51:45 | efried | It would be weird for a provider managed by nova to have 'openstack:' in its name. If it was made by some other service, maybe, but I've never heard of it. | |
| 12:53:28 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Include removal of ephemeral backing files in the image cache manager https://review.opendev.org/689422 | |
| 12:58:15 | johnthetubaguy | stephenfin: I just had a look at the network API removal... are we not adding a microversion to signal when we removed the API, I think that would be more consistent with our rules? | |
| 12:58:45 | johnthetubaguy | I am guessing I missed about a year of conversation, so probably missed the reasoning | |
| 13:00:05 | openstack | Launchpad bug 1853453 in tempest "test_shelve_volume_backed_instance intermittently fails guest ssh with dhcp lease fail" [Undecided,New] | |
| 13:00:05 | mriedem | fun gate bug i finally wrote up after rechecking several times https://bugs.launchpad.net/tempest/+bug/1853453 | |
| 13:00:11 | mriedem | looks like this only hits in multinode jobs, | |
| 13:00:21 | mriedem | and looking at a recent failure, we shelve from the primary node and unshelve on the subnode | |
| 13:00:45 | mriedem | i wonder if the snapshot is no good in some cases and unshelving on another node causes some kind of issue | |
| 13:06:33 | mriedem | oh nvm there is no snapshot, it's volume-backed | |
| 13:06:34 | mriedem | derp | |
| 13:12:08 | gibi | efried: do you know if provider_tree in the report client will contain RPs associated to the compute RP via aggregate? this comment states it does https://github.com/openstack/nova/blob/1cd5563f2dd2b218db2422397c8aab394d484626/nova/scheduler/client/report.py#L593 | |
| 13:12:36 | gibi | efried: but I did not find in the impl where we query for those RPs from placement | |
| 13:12:51 | efried | gibi: it should, yes; the code has been written that way for a while, but we've never had a real scenario that uses it. | |
| 13:13:07 | efried | hold on, let me find you the code (It's somewhere under _refresh_associations...) | |
| 13:13:52 | efried | ah, it looks like we specifically target only sharing providers (we specifically filter on MISC_SHARES) | |
| 13:13:55 | efried | which makes sense | |
| 13:14:04 | efried | we wouldn't want to just arbitrarily pick up anything in an aggregate. | |
| 13:14:37 | gibi | efried: cool, MISC_SHARES... is what I'm looking for. Do you have a link? | |
| 13:15:09 | efried | gibi: https://opendev.org/openstack/nova/src/branch/master/nova/scheduler/client/report.py#L787-L806 ish | |
| 13:15:22 | gibi | efried: thanks a lot! | |
| 13:15:42 | gibi | shilpasd: ^^ | |
| 13:15:44 | bauzas | johnthetubaguy: we deprecated the APIs so AFAIK we don't need a microversion | |
| 13:15:49 | efried | gibi: that comment you pointed out should really say *sharing* providers. | |
| 13:16:20 | gibi | I can fix that quickly.. | |
| 13:16:24 | efried | bauzas: I see johnthetubaguy's point, though. If you come into one cloud and try to use nova-net at microversion 2.58 and it works, and then you come into a different cloud and it fails at the same microversion... | |
| 13:16:54 | bauzas | efried: sure, it's an interop issue | |
| 13:16:55 | bauzas | but | |
| 13:17:09 | johnthetubaguy | efried: I am more thinking, the SDK and CLI should be able to know when its missing, like with all our other APIs | |
| 13:17:16 | bauzas | if you use nova-network since 2.58, you are a bit having problems | |
| 13:17:32 | efried | Either way, I think that ship has sailed, cause we've already ripped a bunch of stuff out. Pretty sure the point of no return was in train, too. | |
| 13:17:32 | johnthetubaguy | to be clear, I think we should remove it, and it should return 210 | |
| 13:17:37 | bauzas | also, if you use OSC, you won't see the APIs be deleted | |
| 13:17:54 | johnthetubaguy | I just think we should have a microversion that tells a user, if that is here, you know those APIs are gone | |
| 13:18:12 | johnthetubaguy | we can add it as the last patch in the series, and I think that kinda works, its just a handy hit | |
| 13:18:34 | johnthetubaguy | it makes the docs easier though, if microversion X is available, you know this API will always return HTTP gone | |
| 13:19:05 | zigo | PCI alias nvidia-p1000 is not defined (HTTP 400) (Request-ID: req-b0514752-9e50-4e9b-a085-6ef9169b1d59) | |
| 13:19:05 | zigo | $ openstack server create --image bionic-server-cloudimg-amd64_20190726_GPU --nic net-id=bdb-blue-int01 --key-name yubikey-zigo --flavor cpu4-ram12-disk20-gpu-nvidia-p1000 --availability-zone=AZ2 zigo-gpu | |
| 13:19:05 | zigo | bauzas: Hi there! I'm trying to start an instance with a GPU, and I get this: | |
| 13:19:23 | zigo | bauzas: Though I do have this in nova.conf: | |
| 13:19:24 | zigo | alias={"vendor_id":"10de","product_id":"1cb1","name":"nvidia-p1000","device_type":"type-PCI"} | |
| 13:19:27 | openstackgerrit | Elod Illes proposed openstack/nova stable/pike: Explicitly fail if trying to attach SR-IOV port https://review.opendev.org/695408 | |
| 13:19:27 | zigo | What am I missing? | |
| 13:19:54 | bauzas | johnthetubaguy: well, we could indeed | |
| 13:20:17 | bauzas | once all the APIs are giving HTTP410 | |
| 13:20:22 | bauzas | stephenfin: ^ | |
| 13:20:22 | johnthetubaguy | bauzas: it is a nice to have, really its the docs that worried me | |
| 13:20:44 | johnthetubaguy | zigo: is that in the API and Compute nova.conf? | |
| 13:20:52 | zigo | johnthetubaguy: Yeah... | |
| 13:21:12 | zigo | alias=<something> | |
| 13:21:12 | zigo | [pci] | |
| 13:21:12 | zigo | johnthetubaguy: It's ok to have multiple times: | |
| 13:21:13 | zigo | right? | |