| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-03-12 | |||
| 19:00:23 | openstackgerrit | Merged openstack/nova master: [Trivial] Fix code comment of admin password tests https://review.opendev.org/711734 | |
| 19:06:12 | openstack | melwitt: Error: Error getting bugzilla.redhat.com bug #1744079: NotPermitted | |
| 19:06:12 | melwitt | stephenfin: here's the bug ticket https://bugzilla.redhat.com/show_bug.cgi?id=1744079 says it's fixed, so need to look at what versions this is showing up in? maybe they just need to update or maybe a backport is missing like you said? | |
| 19:07:02 | openstack | melwitt: Error: Error getting bugzilla.redhat.com bug #1741782: NotPermitted | |
| 19:07:02 | melwitt | sorry that was a clone https://bugzilla.redhat.com/show_bug.cgi?id=1741782 | |
| 19:09:57 | kplant | if i have a instance on say "compute-01" and virsh shows it there, but nova thinks it's on "compute-02" how can i fix that? | |
| 19:12:29 | stephenfin | melwitt: Yup, looks like this is the commit we need https://github.com/libvirt/libvirt/commit/c8007fdc5d2ce43fec2753cda60fb4963f55abd5 | |
| 19:12:50 | stephenfin | $ git tag --contains c8007fdc5d2ce43fec2753cda60fb4963f55abd5 | head -1 | |
| 19:12:52 | stephenfin | v5.10.0 | |
| 19:15:20 | melwitt | stephenfin: hm that looks like it. but v5.10.0 doesn't match the fixed in versions on the rhbzs? I don't understand that | |
| 19:17:29 | stephenfin | backport, I'd say | |
| 19:17:41 | melwitt | cause according to that there should be a fix in the 4.5.x series too | |
| 19:17:47 | melwitt | yeah.. | |
| 19:18:15 | stephenfin | okay, I definitely don't think we need to do anything here in nova so | |
| 19:18:51 | stephenfin | The bug's there. People can find it via their favourite search engine and annoy their vendors to backport the fix if needed | |
| 19:20:10 | sean-k-mooney | we cant rely on backport in downstream version upstream | |
| 19:20:21 | melwitt | yeah, agreed nothing to do in nova. I was trying to find some links to leave in the lp bug but I realize I don't know how upstream libvirt works, how to tell them what version etc | |
| 19:20:38 | sean-k-mooney | so until we can raise the min libvirt version to 5.10 we cant assume that commit is there | |
| 19:21:18 | melwitt | well, shouldn't upstream libvirt be backporting like we do for stable? or do they not do that | |
| 19:21:33 | sean-k-mooney | melwitt: upstream libvirt does not do backports | |
| 19:21:48 | stephenfin | I left some words. Hopefully they help | |
| 19:22:15 | sean-k-mooney | melwitt: which is why each os vendor needs to do it downstream which is annoying | |
| 19:22:23 | melwitt | dang | |
| 19:35:47 | melwitt | dansmith: I added you to this review bc it's updating code that has a NOTE with yo name on it https://review.opendev.org/705764 | |
| 21:58:21 | openstack | bug 1844929 in OpenStack Compute (nova) "grenade jobs failing due to "Timed out waiting for response from cell" in scheduler" [High,Confirmed] https://launchpad.net/bugs/1844929 | |
| 21:58:21 | openstackgerrit | melanie witt proposed openstack/nova master: DNM: try to get some debug info for bug 1844929 https://review.opendev.org/701478 | |
| #openstack-nova - 2020-03-13 | |||
| 00:32:07 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: ksa auth conf and client for Cyborg access https://review.opendev.org/631242 | |
| 00:32:08 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Create and bind Cyborg ARQs. https://review.opendev.org/631244 | |
| 00:32:08 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Add Cyborg device profile groups to request spec. https://review.opendev.org/631243 | |
| 00:32:09 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Compose accelerator PCI devices into domain XML in libvirt driver. https://review.opendev.org/631245 | |
| 00:32:09 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Pass accelerator requests to each virt driver from compute manager. https://review.opendev.org/698581 | |
| 00:32:10 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Enable hard/soft reboot with accelerators. https://review.opendev.org/697940 | |
| 00:32:10 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Delete ARQs for an instance when the instance is deleted. https://review.opendev.org/673735 | |
| 00:32:11 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Enable and use COMPUTE_ACCELERATORS trait. https://review.opendev.org/699554 | |
| 00:32:11 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Enable start/stop of instances with accelerators. https://review.opendev.org/699553 | |
| 00:32:12 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Block unsupported instance operations with accelerators. https://review.opendev.org/674726 | |
| 00:32:12 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Bump compute rpcapi version and reduce Cyborg calls. https://review.opendev.org/704227 | |
| 00:32:13 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Add cyborg tempest job. https://review.opendev.org/670999 | |
| 00:53:44 | openstackgerrit | Merged openstack/nova stable/stein: Fix os-keypairs pagination links https://review.opendev.org/711896 | |
| 01:25:46 | brinzhang | gmann:hi | |
| 01:27:28 | brinzhang | gmann: I combined the os-instance-actions policy patch, and that has previous issue in cmd/test_policy https://review.opendev.org/#/c/706470/ | |
| 01:27:49 | brinzhang | gmann: can you have a fast check ? | |
| 04:09:27 | openstackgerrit | Merged openstack/nova master: Catch exception when use invalid architecture of image https://review.opendev.org/711363 | |
| 04:52:21 | openstackgerrit | Brin Zhang proposed openstack/python-novaclient master: Microversion 2.83: Add volume-patch CLI https://review.opendev.org/712651 | |
| 05:33:55 | openstackgerrit | Brin Zhang proposed openstack/nova master: Add new default roles in os-instance-actions policies https://review.opendev.org/706470 | |
| 08:11:55 | openstackgerrit | Brin Zhang proposed openstack/nova master: Add SYSTEM_READER role to servers actions API https://review.opendev.org/706179 | |
| 08:58:19 | kashyap | melwitt: stephenfin: Just caught up with that "lift the SCSI unit restriction" libivirt issue: I'm pretty damn sure it's fixed upstream _and_ for CentOS / RHEL | |
| 08:59:40 | kashyap | Okay, I see that stephenfin has pointed out the RHBZ | |
| 09:00:06 | kashyap | melwitt: stephenfin: It looks like Ubunutu Virt folks need to backport that patch series | |
| 12:48:27 | mnaser | has anyne seen this in nova's ci? | |
| 12:48:30 | mnaser | libvirt.libvirtError: Requested operation is not valid: format of backing image '/var/lib/nova/instances/_base/791111176e5cb97db82b0a71a670431f65838a05' of image '/var/lib/nova/instances/ff811290-bbb0-4d30-b1e7-ee9aec4913ee/disk' was not specified in the image metadata (See https://libvirt.org/kbase/backing_chains.html for troubleshooting) | |
| 12:48:51 | mnaser | OSA's jobs are failing with that and i wonder if its because the libvirt version with bionic is different... maybe | |
| 12:50:21 | openstack | bugzilla.redhat.com bug 1798148 in libvirt "Regression: Requested operation is not valid: format of backing image ... was not specified in the image metadata" [Unspecified,On_qa] - Assigned to pkrempa | |
| 12:50:21 | mnaser | i fonud this - https://bugzilla.redhat.com/show_bug.cgi?id=1798148 | |
| 12:50:51 | openstack | Launchpad bug 1864020 in OpenStack Compute (nova) "libvirt.libvirtError: Requested operation is not valid: format of backing image %s of image %s was not specified in the image metadata (See https://libvirt.org/kbase/backing_chains.html for troubleshooting)" [Undecided,In progress] - Assigned to Lee Yarwood (lyarwood) | |
| 12:50:51 | mnaser | ok i see https://bugs.launchpad.net/nova/+bug/1864020 | |
| 13:03:19 | lyarwood | mnaser: yeah that should be resolved now | |
| 13:05:13 | mnaser | lyarwood: cool, we are bumping our versions which should help with this in OSA. thanks for looking into it! | |
| 13:06:58 | lyarwood | mnaser: np, which jobs hit this in OSA btw? You need a very recent version of libvirt to hit this. | |
| 13:10:10 | sean-k-mooney | gibi: efried_gone https://review.opendev.org/#/c/676522/44/nova/compute/resource_tracker.py@1738 i think that adresses the duplicate resouce provider definitons in different yaml files | |
| 13:10:47 | sean-k-mooney | also is efried_gone gone permently now or will erric be back before he finishes his current role | |
| 13:17:12 | gibi | sean-k-mooney: i will look at the provider series next week. thanks for taking it over | |
| 13:17:48 | sean-k-mooney | gibi: no rush bar m3, i still need to adress the final patch and then look at the testing | |
| 13:18:11 | sean-k-mooney | i think i covered the main funcitonal changes already however | |
| 13:18:48 | gibi | sean-k-mooney: ack | |
| 13:19:18 | gibi | m3 is in 4 weeks | |
| 13:19:39 | sean-k-mooney | yep so no rush | |
| 13:19:49 | gibi | sh*t i have to fix up the qos series too til | |
| 13:20:18 | gibi | anyhow I expect a rush of reviews at m3 as usual | |
| 13:22:18 | efried_gone | sean-k-mooney: I'm gone gone. Spending maybe a few minutes a day paying attention to OpenStack stuff. | |
| 13:22:56 | sean-k-mooney | cool good to know. ill miss having you around | |
| 13:48:20 | kashyap | efried_gone: Was good knowing you; see you on the Other Side(tm). | |
| 13:48:49 | kashyap | And thanks for all the outstanding (as in, excellent, not "remaining") work! ;-) | |
| 13:49:11 | kashyap | mnaser: Hey, yes... | |
| 13:49:42 | kashyap | mnaser: That issue known due to a libvirt regression, as lyarwood got a patch merged to solve it in Nova -- which is the right thing to do _anyway) | |
| 13:50:05 | kashyap | mnaser: See this one: | |
| 13:50:32 | stephenfin | lyarwood, smcginnis: Think you folks could hit this backport from elod today? It fixes issues with failing tests seen in other backports :) https://review.opendev.org/#/c/712751/ | |
| 13:50:36 | kashyap | mnaser: https://review.opendev.org/#/c/708745/ ("libvirt: Provide the backing file format when creating qcow2 disks") | |
| 13:50:44 | kashyap | mnaser: It's merged in master, Train backport in-progress. | |
| 13:50:45 | stephenfin | (like this one https://review.opendev.org/#/c/711670/) | |
| 13:51:02 | kashyap | Oh, lyarwood already answered it; silly me. I should read the scrollback in full before spamming the channel. | |
| 14:02:59 | lyarwood | kashyap: looking | |
| 14:03:04 | lyarwood | kashyap: ah this one | |
| 14:03:07 | brinzhang | stephenfin: https://review.opendev.org/#/c/712651/ could please check this patch of novaclient? I add a PATCH ``volume-patch`` CLI, debug it, and not found where is wrong | |
| 14:03:23 | kashyap | lyarwood: Yeah, sorry for the noise. | |
| 14:05:45 | lyarwood | stephenfin: ack I'll take a swing later today | |
| 14:10:28 | stephenfin | brinzhang: You need to add a 'patch_servers_1234_os_volume_attachments_Work' method to 'FakeSessionClient' in 'novaclient/tests/unit/v2/fakes.py' | |
| 14:11:04 | stephenfin | with a response mocking what you'd see from nova-api | |
| 14:13:17 | brinzhang | stephenfin: cools, thanks, yep, I will try, I missing that response in 'FakeSessionClient' | |
| 14:17:01 | sean-k-mooney | stephenfin: i cant cross link to source code form nova docs right | |
| 14:17:02 | brinzhang | stephenfin: thanks, I will complete its unit tests, and then update ^^ | |
| 14:17:36 | stephenfin | sean-k-mooney: Only if the code is autodoc'd somewhere, and I think only the notifier stuff falls in that bracket | |
| 14:18:02 | stephenfin | I _think_ you should still be able to use e.g. :py:method:`nova.foo.bar` but it won't resolve to anything | |
| 14:18:05 | sean-k-mooney | stephenfin: ok in that case i think ill have to link to opendev/github | |
| 14:18:47 | sean-k-mooney | stephenfin: context is https://review.opendev.org/#/c/693460/18/doc/source/admin/managing-resource-providers.rst,unified@66 | |
| 14:19:07 | sean-k-mooney | just trying to figure out how i would go ablout liking to the schem file for the provider.yaml validation | |
| 14:19:47 | sean-k-mooney | i think an external link is the only way to do it so ill figure out what the opendev path would be | |
| 14:19:49 | stephenfin | you could just '.. include' it | |
| 14:20:15 | sean-k-mooney | oh i didnt know you could do that am ill try | |
| 14:24:37 | brinzhang | johnthetubaguy: I resolved the os-instance-actions policy issue, and granular the GET API, pls review again. https://review.opendev.org/#/c/706470/ | |