| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-05-13 | |||
| 23:21:51 | gmann | I am going to release the new hacking version 4.0.0 which will pull new checks and then in 727589 patch we will bump hacking version to so that we can verify the fix | |
| 23:22:38 | gmann | I mean fix + new hacking version bump in a single patch. | |
| 23:31:57 | melwitt | gmann: yeah makes sense. +2 on the 3.0.1 patch though I wondered if it's needed to avoid gate failures, would have thought we'd have a gate-failure bug around it. that is, it wasn't clear to me if the patch is needed to fix gate failure | |
| 23:34:24 | gmann | melwitt: I did nova patch earlier before i described the situation in other patches cmt msg like this- https://review.opendev.org/#/c/727576/ | |
| 23:35:29 | gmann | melwitt: basically it will 1. fix local run where the latest fixed hacking 3.0.1 is not pulled automatically. in case of gate it is pulled as fresh installation. 2. it will protect if future flake8 3.9.0 version pull other new checks. | |
| 23:35:30 | melwitt | ah I see | |
| 23:35:55 | melwitt | oh ok so not a gate issue | |
| 23:36:04 | melwitt | cool thanks for explaining | |
| 23:36:12 | melwitt | I was confused before heh | |
| 23:37:05 | gmann | otherwise hacking before 3.0.1 is capping "flake8<4.0.0" so flake8 3.8.0 which ic current release and 3.9.0 future release both can be protected by bumping hacking to 3.0.1 | |
| 23:37:22 | melwitt | right | |
| 23:37:59 | melwitt | yeah, so needed for future | |
| 23:39:46 | gmann | it is actually confusing as all inbuild deps actually. hacking does protect cap for flake8 and flake8 does protect cap for pycodestyle. but if pycodestyle is pulled as latest by some flake8 plugins then again it fail like flake8-import-order then we need pycodestyle cap explicitly. like tempest case- https://review.opendev.org/#/c/727276/2/test-requirements.txt | |
| 23:40:13 | gmann | so our way of handling the flake8 with plugins is not so great. | |
| 23:40:22 | melwitt | wow. mind blown | |
| 23:41:24 | gmann | i added PR on flake8-import-order but not sure if they will accept or not with our hacky use case :) - https://github.com/PyCQA/flake8-import-order/pull/172 | |
| 23:41:56 | melwitt | ah, good luck :) | |
| #openstack-nova - 2020-05-14 | |||
| 00:46:44 | openstackgerrit | Wenping Song proposed openstack/nova master: delete sub resource provider when delete resource provider https://review.opendev.org/719163 | |
| 00:49:25 | openstackgerrit | melanie witt proposed openstack/nova master: Configure latex_engine = 'xelatex' for pdf docs build https://review.opendev.org/727898 | |
| 01:07:14 | openstackgerrit | melanie witt proposed openstack/nova master: Configure latex_engine = 'xelatex' for pdf docs build https://review.opendev.org/727898 | |
| 01:32:15 | openstackgerrit | Merged openstack/nova-specs master: Re-propose provider-config-file spec for Victoria https://review.opendev.org/725788 | |
| 01:36:51 | openstackgerrit | Merged openstack/nova-specs master: Re-proposes 'Proposal for a safer remote console with password authentication' https://review.opendev.org/719720 | |
| 02:48:28 | tony_su | gibi, sean-k-mooney : Thanks for reviewing and merging the spec. | |
| 02:50:43 | tony_su | gibi, sean-k-mooney : I will start to rebase related patches and send out for code review once they are ready. | |
| 03:03:55 | melwitt | fyi the openstack-tox-docs job is failing for nova since a recent upper-constraints bump, I have proposed a fix here https://review.opendev.org/727898 I have also sent a ML post | |
| 05:56:38 | openstackgerrit | melanie witt proposed openstack/nova master: Configure latex_engine = 'xelatex' for pdf docs build https://review.opendev.org/727898 | |
| 07:10:38 | openstackgerrit | Nalini Varshney proposed openstack/nova master: Add migration to make key field type VARBINARY in aggregate_metadata table, https://review.opendev.org/725522 | |
| 07:35:02 | brinzhang_ | sean-k-mooney: https://review.opendev.org/#/c/727589/2 do I need to update this patch? I have not concern stephenfin has already fixed these issue, but he abandoned. | |
| 07:49:09 | gibi | good morning Nova | |
| 07:49:29 | bauzas | whoops my manners | |
| 07:49:34 | bauzas | good morning Nova | |
| 07:50:26 | brinzhang_ | morning nova | |
| 07:50:59 | brinzhang_ | hi gibi, bauzas, I saw you are -1 for https://review.opendev.org/#/c/727140/1, the flake8 error fix patch | |
| 07:51:44 | gibi | brinzhang_: my -1 is about the fact that 3.8.1 added yet another check that needs to be fixed | |
| 07:51:48 | brinzhang_ | I submit a same one, before that I have not concern stephenfin's patch, do I need to continue ? | |
| 07:52:24 | brinzhang_ | gibi: I will see http://paste.openstack.org/show/793434/ | |
| 07:52:40 | bauzas | brinzhang_: my concerns was about making sure what we would need to change | |
| 07:52:49 | gibi | I guess stephenfin abandoned the patch as we went a different direction, capping the version | |
| 07:53:34 | brinzhang_ | yeah | |
| 07:53:55 | brinzhang_ | I mean, if there is no sense to fix, I think I would abandon too. | |
| 07:56:16 | brinzhang_ | https://review.opendev.org/#/c/727347/ gmann bumped hacking to 3.0.1, I am not sure whether this can avoid these pep8 error happen again | |
| 07:58:30 | brinzhang_ | gibi: stephenfin and I fixed the issue is reported by https://review.opendev.org/#/c/724260/6, and https://review.opendev.org/#/c/726564/2. | |
| 07:59:12 | brinzhang_ | I found your paste is not easy to fix, may need to change the function name, such as "./nova/tests/unit/api/openstack/compute/microversions.py:34:5: F811 redefinition of unused 'index' from line 29" | |
| 07:59:18 | brinzhang_ | :P | |
| 08:02:05 | gibi | brinzhang_: those redefinitions are intentional and not cause a python bug due to the decorator top of it will generate unique functions dynamically. So I think F811 needs to be ignored in tox.ini | |
| 08:03:01 | gibi | I think it is OK to fix the issues reported by the other new checks along with version bump | |
| 08:03:09 | gibi | but please consult stephenfin too | |
| 08:03:54 | brinzhang_ | gibi: agree | |
| 08:04:49 | brinzhang_ | gibi: yeah, I will try to update the patch to latest, and then consult | |
| 08:05:32 | gibi | OK, cool | |
| 08:20:58 | openstackgerrit | Brin Zhang proposed openstack/nova master: Fix flake8 error from hacking <3.0.1 https://review.opendev.org/727589 | |
| 08:45:27 | bauzas | brinzhang_: gibi: FWIW, +Wd https://review.opendev.org/#/c/727347/1 | |
| 08:45:57 | bauzas | brinzhang_: but then I don't understand why we should merge https://review.opendev.org/#/c/727589/ | |
| 08:46:04 | gibi | bauzas: thanks | |
| 08:50:36 | bauzas | gibi: any change I should be reviewing ? | |
| 08:51:23 | openstack | Launchpad bug 1878024 in OpenStack Compute (nova) "disk usage of the nova image cache is not counted as used disk space" [Undecided,Confirmed] - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 08:51:23 | gibi | bauzas: not from me at the moment. I still busy with https://bugs.launchpad.net/nova/+bug/1878024 | |
| 08:51:41 | bauzas | gibi: okay, just saying, you have https://review.opendev.org/#/q/topic:bug/1869050+(status:open+OR+status:merged) | |
| 08:51:51 | bauzas | should I look at it ? | |
| 08:52:13 | gibi | bauzas: yes please. I already forgot about that | |
| 08:52:23 | bauzas | heh, no worries | |
| 09:00:45 | brinzhang_ | bauzas: https://review.opendev.org/#/c/727589/ this patch just only fixed some unreasonable issue, IMO, if we don’t fix it, we don’t know when it will *explode* | |
| 09:01:08 | brinzhang_ | after all, it has reported in https://review.opendev.org/#/c/724260/6, and https://review.opendev.org/#/c/726564/2. | |
| 09:11:09 | bauzas | brinzhang_: that's the exact reason why we have minimum versions | |
| 09:11:41 | bauzas | once we bump the minimum, we no longer need to care about what was bad for older versions | |
| 09:12:23 | brinzhang_ | bauzas: I know you mean, but these code realy not have a good code style, right? | |
| 09:13:41 | brinzhang_ | I can say with certainty that merged this change will only make Nova's code more readable. | |
| 09:14:37 | bauzas | then that's not a bugfix | |
| 09:14:42 | brinzhang_ | especially https://review.opendev.org/#/c/727589/3/nova/tests/unit/compute/test_compute_mgr.py this patch | |
| 09:14:49 | brinzhang_ | s/patch/file/ | |
| 09:14:57 | bauzas | and just a bikeshed | |
| 09:15:23 | brinzhang_ | if there is no this blank line, it.. | |
| 09:16:09 | brinzhang_ | Yes, this bug was submitted before https://review.opendev.org/#/c/727347/1 | |
| 09:16:58 | brinzhang_ | because it caused pep8 error in https://review.opendev.org/#/c/724260/6, and https://review.opendev.org/#/c/726564/2. | |
| 09:17:26 | brinzhang_ | that time I think it is a bug, that why report | |
| 09:19:18 | sean-k-mooney | bauzas: we will eventually have to fix the pep8 issues by the way when we update hacking | |
| 09:19:36 | sean-k-mooney | and therefor get the newer flake8 | |
| 09:20:02 | bauzas | again, we have minimum versions FTW | |
| 09:20:16 | sean-k-mooney | yep wont help | |
| 09:20:43 | bauzas | the point is, if we want to support new flake8 version, then yes indeed we need to verify this works | |
| 09:20:46 | sean-k-mooney | the new min version of hacking is so we get a version that caps flake8 and we have capped the hacking version | |
| 09:20:50 | bauzas | and then updating what we need | |
| 09:21:07 | sean-k-mooney | so when we update the hacking version we will get a new flake8 and have to fix the issue then | |
| 09:21:09 | bauzas | but that's another concern | |
| 09:21:22 | bauzas | correct, that's another change | |
| 09:21:39 | bauzas | anyway, /me needs to get my kid from the school since it opened again today :( | |
| 09:22:00 | bauzas | (until next kids vacations) | |
| 09:22:12 | sean-k-mooney | i would have assumed your productivig increased with them going back to school | |
| 09:22:31 | sean-k-mooney | so in net it should still be better | |
| 09:23:03 | sean-k-mooney | even if its about to go down now that you are going to pick them up | |
| 09:25:17 | brinzhang_ | Agree, a good foundation can make a difference. | |
| 09:42:32 | bauzas | sean-k-mooney: hell, unfortunately no | |
| 09:42:48 | bauzas | sean-k-mooney: I was working between 9am-1pm and then 2pm-7.30pm | |
| 09:42:55 | bauzas | even with the kids | |
| 09:43:07 | bauzas | they were asking me for teaching only for 1 hour | |
| 09:43:30 | bauzas | now, it's 9am-11.30am and then 1.30pm-4.30pm | |
| 09:43:39 | bauzas | :( | |
| 09:53:01 | openstackgerrit | sean mooney proposed openstack/nova master: cyborg evacuate support https://review.opendev.org/715326 | |
| 09:55:27 | sean-k-mooney | gibi: brinzhang_ filed https://blueprints.launchpad.net/nova/+spec/cyborg-rebuild-and-evacuate | |
| 09:56:09 | sean-k-mooney | ill add it to the meeting adgenda but im hoping we are all fine with proceeding with that as a specless blueprint since i have already implemented it | |