| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-05-14 | |||
| 18:16:33 | melwitt | but it sounds like you have a situation where nova is trying to create the resource provider but it already exists | |
| 18:18:55 | NobodyCam | yea | |
| 18:19:40 | NobodyCam | I can delete the record from resource_providers | |
| 18:20:32 | melwitt | this is likely be related to a WIP patch https://review.opendev.org/678100 (just fyi) but to recover, you will need to delete the resource provider. but before you do it, please check if it has any allocations on it by using the placement cli | |
| 18:21:02 | melwitt | openstack resource provider show --allocations <uuid> | |
| 18:21:34 | melwitt | and if there are allocations, save those (copy paste somewhere) because you'll want to restore them after you delete the resource provider | |
| 18:22:22 | melwitt | after you save the allocations, delete the resource provider 'openstack resource provider delete <uuid>' | |
| 18:22:34 | melwitt | restart nova-compute, that should create the new resource provider | |
| 18:23:13 | melwitt | then if you had allocations from earlier, set them on the new resource provider https://docs.openstack.org/osc-placement/rocky/cli/index.html#resource-provider-allocation-set | |
| 18:23:35 | NobodyCam | Awesome :) thank you melwitt Working on getting placement plugin going | |
| 18:25:21 | melwitt | ok cool. yeah after you install osc-placement (either from pip or distro package) then the commands will magically be part of the 'openstack' command | |
| 19:35:57 | openstackgerrit | Andreas Jaeger proposed openstack/nova master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728263 | |
| 19:37:51 | NobodyCam | melwitt: Again massive thank you, we are back up and working | |
| 19:38:39 | NobodyCam | no allocations so just a delete and restart fixed us right up | |
| 19:39:00 | melwitt | NobodyCam: np, glad to hear. ah good, I was hoping it would be the easy case :) | |
| 19:39:19 | NobodyCam | :) | |
| 19:46:38 | openstackgerrit | Andreas Jaeger proposed openstack/python-novaclient master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728270 | |
| 19:46:58 | openstackgerrit | Andreas Jaeger proposed openstack/python-novaclient master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/728270 | |
| 19:53:12 | openstackgerrit | melanie witt proposed openstack/nova master: Add packages required for pdf docs build to bindep.txt https://review.opendev.org/727898 | |
| 20:11:46 | gmann | sean-k-mooney: change lgtm and not changing the API but i am not sure about mentioned use case of this API to refresh the cache. server external event API is for openstack services mainly. - https://review.opendev.org/#/c/702394/3 | |
| 22:55:22 | openstackgerrit | melanie witt proposed openstack/nova master: DNM Try out running sphinx-build in parallel for releasenotes https://review.opendev.org/727429 | |
| #openstack-nova - 2020-05-15 | |||
| 00:05:40 | openstackgerrit | melanie witt proposed openstack/nova master: Configure latex_engine = 'xelatex' for pdf docs build https://review.opendev.org/727898 | |
| 00:09:08 | openstackgerrit | melanie witt proposed openstack/nova master: Fix pep8 errors uncovered by flake8 3.8.1 https://review.opendev.org/728312 | |
| 00:37:37 | openstackgerrit | Brin Zhang proposed openstack/nova master: Fix flake8 error from hacking <3.0.1 https://review.opendev.org/727589 | |
| 00:44:12 | gmann | melwitt: we need this to be in to have right cap for hacking and flake8 - https://review.opendev.org/#/c/727347/1 | |
| 00:44:19 | gmann | i thought it was merged | |
| 00:44:44 | gmann | but is doc job still failing ? | |
| 00:45:34 | melwitt | gmann: rechecking not gonna help, docs job still broken | |
| 00:46:11 | melwitt | it did merge but didn't stop the failure, I replied on the ML post | |
| 00:46:42 | gmann | i thought openstackdocstheme 2.1.0 fixed that on gate | |
| 00:47:06 | melwitt | 2.1.1 was intended to fix it but it did not work | |
| 00:47:35 | melwitt | my nova side fix is still proposed but now we have a chicken and egg with the pep8 job | |
| 00:47:56 | melwitt | I sent a ML post about that too a little while ago | |
| 00:48:29 | gmann | ah just saw, i think we can squash it with 727898 | |
| 00:49:39 | brinzhang | gmann: the hacking verion should change to 3.1.0? | |
| 00:49:42 | brinzhang | right? | |
| 00:50:31 | gmann | brinzhang: yeah in your patch yes as it will fix the code together | |
| 00:50:57 | brinzhang | gmann: ok, update soon | |
| 00:51:53 | melwitt | I wonder if alex_xu is around | |
| 00:56:17 | gmann | melwitt: this should avoid doc failure - https://review.opendev.org/#/c/727943/4 | |
| 00:56:44 | melwitt | gmann: it doesn't because 2.1.1 also fails | |
| 00:57:50 | melwitt | if you look in the docs job build logs, 2.1.1 is installed now | |
| 00:58:05 | openstackgerrit | Brin Zhang proposed openstack/nova master: Bump hacking min version to 3.1.0 https://review.opendev.org/727589 | |
| 00:58:57 | gmann | melwitt: not sure why not complete revert that instead if backlist. PS1 was right revert - https://review.opendev.org/#/c/727943/1/upper-constraints.txt | |
| 01:00:11 | brinzhang | gmann: updated, I just only keep the hacking min version to 3.1.0, i.e. hacking>=3.1.0, please check | |
| 01:18:38 | openstackgerrit | Brin Zhang proposed openstack/nova master: Bump hacking min version to 3.1.0 https://review.opendev.org/727589 | |
| 01:22:11 | brinzhang | gmann: do we need to continue merged hacking 3.0.1 https://review.opendev.org/#/c/727347/? | |
| 01:48:11 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Bump hacking min version to 3.0.1 https://review.opendev.org/727347 | |
| 01:51:08 | melwitt | gmann: lol, what a day huh! | |
| 01:51:31 | melwitt | (gate stuff) | |
| 01:54:04 | gmann | :). I think we made little hurry to release new hacking. | |
| 01:55:49 | gmann | I vote is to move back openstackdocstheme to 2.0.2 as nova, cinder are hitting doc + pep8 and may be more projects but not checked | |
| 01:55:55 | gmann | *my vote | |
| 01:56:40 | gmann | asked in requirement channel, let's see if we can get any other fast way to unblock | |
| 01:56:51 | melwitt | yeah, I lean that way too | |
| 01:57:49 | openstackgerrit | Brin Zhang proposed openstack/nova master: Bump hacking min version to 3.1.0 https://review.opendev.org/727589 | |
| 01:58:23 | gmann | because hacking is blacklist req so it cannot be controlled on single place (g-r side). | |
| 01:58:48 | melwitt | I see | |
| 02:08:37 | openstackgerrit | Brin Zhang proposed openstack/nova master: Bump hacking min version to 3.1.0 https://review.opendev.org/727589 | |
| 02:35:23 | melwitt | gmann: do you plan to depends-on the roll back patch or did you want to try it out some other way? | |
| 02:36:07 | gmann | i can do that. | |
| 02:36:46 | melwitt | or could do a dummy patch | |
| 02:37:01 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Bump hacking min version to 3.0.1 https://review.opendev.org/727347 | |
| 02:37:29 | gmann | melwitt: as it needs to re-run anyways. done ^^ | |
| 02:37:43 | melwitt | yeah, I saw it had already failed the docs job | |
| 02:37:54 | melwitt | cool I'll keep an eye on it | |
| 03:13:50 | openstackgerrit | melanie witt proposed openstack/nova master: Configure latex_engine = 'xelatex' for pdf docs build https://review.opendev.org/727898 | |
| 03:15:02 | melwitt | gmann: oh no, depends-on doesn't work for requirements changes?? | |
| 03:15:05 | melwitt | still failing | |
| 03:15:10 | melwitt | and pulling in 2.1.1 | |
| 03:19:19 | gmann | melwitt: yeah, we need to get requirement revert merge first | |
| 03:19:49 | melwitt | gmann: oh really? depends-on known not to work in this case? I didn't know that, guh | |
| 03:20:17 | melwitt | I thought we were going to be able to test if job passes with the requirement revert | |
| 03:43:18 | gmann | melwitt: left note on this to recheck after req patch merge - https://review.opendev.org/#/c/727347/3 | |
| 03:43:45 | melwitt | gmann: cool, I've re-approved it so it's ready to go | |
| 03:44:16 | gmann | bauzas: stephenfin gibi ^^ whoever wake up early. this is req patch to merge first - https://review.opendev.org/#/c/728335/ | |
| 03:44:21 | gmann | melwitt: thanks. | |
| 03:45:10 | gmann | going to bed... | |
| 03:45:47 | melwitt | k seeya | |
| 06:57:10 | openstackgerrit | Andreas Jaeger proposed openstack/nova master: DNM: Test openstackdocstheme master https://review.opendev.org/728372 | |
| 07:01:18 | gibi | good morning nova | |
| 07:03:13 | gibi | gmann: I see melwitt rechecked it | |
| 07:03:14 | melwitt | gibi: yeah rechecked it just a little while ago, once that lands, our gate should be unblocked. fingers crossed nothing goes wrong | |
| 07:03:20 | gibi | gmann: I'm approving https://blueprints.launchpad.net/nova/+spec/policy-defaults-refresh-deprecated-apis as we agreed about it on yesterday's team meeting | |
| 07:18:51 | bauzas | good Friday Nova | |
| 07:24:33 | bauwser | there it is | |
| 08:05:33 | kashyap | bauwser: /me waves | |
| 08:56:11 | openstackgerrit | Andreas Jaeger proposed openstack/nova master: DNM: Test openstackdocstheme master https://review.opendev.org/728372 | |
| 09:17:13 | lyarwood | stephenfin: https://review.opendev.org/#/c/676208/ - I'm onboard with using mypy just not adding it to the pep8 while it's still optional and not used outside of the libvirt driver. | |
| 09:17:29 | lyarwood | pep8 env* | |
| 09:19:24 | stephenfin | lyarwood: I originally had a separate mypy zuul job and tox target but when I proposed said zuul job against openstack-zuul-jobs, mordred noted that zuul simply integrated it into their pep8 target (or linters, I think they call it) | |
| 09:19:27 | stephenfin | hence why I did it that way | |
| 09:19:34 | stephenfin | where would be better? | |
| 09:25:35 | lyarwood | stephenfin: okay it might not be worth splitting hairs over this but given it's not widely used in openstack/nova yet it seems like overkill to run it everytime we run pep8 | |
| 09:26:00 | lyarwood | stephenfin: I guess zuul is using it as standard across their codebase and that's why they've merged things | |
| 09:27:52 | lyarwood | stephenfin: if gibi and others are cool with it being in the pep8 env then I'll shut up and let it merge :) | |
| 09:29:04 | gibi | stephenfin, lyarwood: is it something that makes tox -e fast8 target significantly slower? | |
| 09:29:35 | stephenfin | it doesn't affect fast8, no | |
| 09:29:46 | stephenfin | and even for pep8, it takes seconds to run | |
| 09:29:55 | lyarwood | with a single file at the moment ;) | |