| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-06-09 | |||
| 13:23:32 | kashyap | stephenfin: Saw your review on that display change. I'll respond in a bit | |
| 13:23:36 | kashyap | Thank you | |
| 13:23:38 | stephenfin | wonderful \o/ | |
| 13:23:46 | gibi | https://mypy-lang.blogspot.com/2021/05/the-upcoming-switch-to-modular-typeshed.html | |
| 13:24:03 | gibi | we are lucky that only one thing was missing so far :) | |
| 13:24:32 | sean-k-mooney | :( non nova type definition we can steal | |
| 13:26:10 | sean-k-mooney | i wonder is there a way we can tell it to ignore third party libs if the hints are not present | |
| 13:26:49 | sean-k-mooney | i.e. can we supress the error and maybe make ti a warning | |
| 13:29:19 | sean-k-mooney | "You can also now run mypy --install-types to install all missing stub packages in the mypy run (or the previous run, if you don’t pass any files to check): " | |
| 13:29:59 | sean-k-mooney | we could look at that or perhaps using --ignore-missing-imports | |
| 13:30:17 | stephenfin | We already have that configured via '[mypy] ignore_missing_imports = true' in setup.cfg | |
| 13:30:26 | stephenfin | apparently it doesn't work for things in typeshed | |
| 13:30:36 | sean-k-mooney | ya it is ignored for thing that were previously bundeled | |
| 13:30:41 | sean-k-mooney | https://mypy-lang.blogspot.com/2021/06/ | |
| 13:30:48 | sean-k-mooney | https://mypy-lang.blogspot.com/2021/06/mypy-0900-released.html | |
| 13:31:26 | sean-k-mooney | oh TypeGuard we can now do user defined narrowing | |
| 13:31:39 | amodi | lyarwood: https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/786979/20/whitebox_tempest_plugin/config.py#138 | |
| 13:34:32 | sean-k-mooney | stephenfin: do we want to consider adding mypy --install-types to the command | |
| 13:34:46 | sean-k-mooney | before we run mypy | |
| 13:34:51 | stephenfin | hmm, that would be a good short-term fix | |
| 13:35:04 | stephenfin | long-term though, I suspect these things might want to be tracked? | |
| 13:35:15 | sean-k-mooney | im not sure | |
| 13:35:20 | sean-k-mooney | they might want to be yes | |
| 13:35:43 | sean-k-mooney | but since its just for type checking im not sure | |
| 13:35:56 | stephenfin | fair point | |
| 13:35:59 | stephenfin | lemme try that | |
| 13:40:48 | sean-k-mooney | stephenfin: by the way we could use https://github.com/pre-commit/mirrors-mypy | |
| 13:41:09 | opendevreview | Rodrigo Barbieri proposed openstack/nova stable/victoria: Error anti-affinity violation on migrations https://review.opendev.org/c/openstack/nova/+/795542 | |
| 13:42:49 | stephenfin | 'mypy --install-types' is interactive | |
| 13:42:50 | stephenfin | :( | |
| 13:43:04 | stephenfin | so I have to pipe 'yes' into it | |
| 13:43:05 | stephenfin | lovely | |
| 13:43:35 | stephenfin | sean-k-mooney: https://github.com/python/mypy/issues/10600#issuecomment-857351152 | |
| 13:43:38 | stephenfin | see next comment | |
| 13:43:45 | stephenfin | I wouldn't recommend running --install-types as it currently works in CI, since in the worst case it can almost double the mypy runtime. It also produces noisy output, as mentioned above. | |
| 13:43:51 | stephenfin | </quote> | |
| 13:44:15 | stephenfin | so we can do it to unblock the gate, but it definitely shouldn't stay | |
| 13:44:26 | sean-k-mooney | stephenfin: ok i guess we can just run it once do a freeze before and after and see what it adds | |
| 13:44:31 | sean-k-mooney | if this happens again | |
| 13:44:37 | stephenfin | Successfully installed types-cryptography-0.1.1 types-enum34-0.1.5 types-ipaddress-0.1.2 types-paramiko-0.1.3 | |
| 13:44:58 | stephenfin | I guess not all of those are actually needed since we don't typecheck everything | |
| 13:45:21 | sean-k-mooney | not right now at least | |
| 13:45:35 | sean-k-mooney | but if we expand the checking they proably will be | |
| 13:46:13 | opendevreview | Stephen Finucane proposed openstack/nova master: requirements: Add types-paramiko https://review.opendev.org/c/openstack/nova/+/795533 | |
| 13:46:14 | opendevreview | Stephen Finucane proposed openstack/nova master: tox: Install mypy type stubs https://review.opendev.org/c/openstack/nova/+/795544 | |
| 13:46:24 | stephenfin | lyarwood, gibi: Alternate approach ^ | |
| 14:12:05 | spatel | sean-k-mooney morning! i am trying to delete one of vm which doesn't existing on compute machine (i can't see it on compute host). i am not able to delete it, any good way to clean up ? | |
| 14:13:14 | gibi | stephenfin: https://review.opendev.org/c/openstack/nova/+/795544 doesnt seem to work | |
| 14:17:41 | stephenfin | oh, well then | |
| 14:17:58 | stephenfin | so I guess you need to run it, let it build a cache, then run it again to install the missing deps | |
| 14:18:04 | stephenfin | so that won't work | |
| 14:18:10 | stephenfin | pity :-( | |
| 14:19:19 | opendevreview | Stephen Finucane proposed openstack/nova master: requirements: Add types-paramiko https://review.opendev.org/c/openstack/nova/+/795533 | |
| 14:19:46 | stephenfin | back to v2 we go | |
| 14:24:35 | sean-k-mooney | spatel: i think we have force delete option | |
| 14:24:56 | spatel | hmm --force ? | |
| 14:25:04 | sean-k-mooney | spatel: if the vm no longere exits on the compute node but is still in the nova db that is what i woudl try next | |
| 14:25:21 | spatel | perfect! let me try | |
| 14:25:40 | sean-k-mooney | well i was thinking of "nova force-delete <server uuid>" | |
| 14:25:45 | sean-k-mooney | let me check if its in osc | |
| 14:26:06 | sean-k-mooney | yes there is a --force | |
| 14:26:19 | sean-k-mooney | os openstack server delete --force <uuid> | |
| 14:27:00 | spatel | penstack server delete: error: unrecognized arguments: --force | |
| 14:27:12 | spatel | I am running victoria | |
| 14:28:27 | spatel | nova force-delete works! | |
| 14:29:44 | spatel | sean-k-mooney openstack server delete doesn't have --force option | |
| 14:30:00 | sean-k-mooney | it does in the latest release | |
| 14:30:34 | sean-k-mooney | spatel: stephenfin added it along with closing most of the other gaps recently | |
| 14:31:24 | spatel | may be available in current release but not in victoria | |
| 14:31:56 | sean-k-mooney | when i said recent i ment in the last month or so | |
| 14:32:08 | sean-k-mooney | nova force-delete does exactly the same thing | |
| 14:33:05 | opendevreview | sean mooney proposed openstack/osc-placement master: default to max version when no session https://review.opendev.org/c/openstack/osc-placement/+/794276 | |
| 14:33:24 | sean-k-mooney | gibi: filed https://storyboard.openstack.org/#!/story/2008958 and updated ^ | |
| 14:33:39 | sean-k-mooney | gibi: let me know if there is anything else i need to do | |
| 14:34:11 | sean-k-mooney | storyboard is still kind of a pain to naveigate but its got much faster then i rememeber | |
| 14:39:30 | opendevreview | Balazs Gibizer proposed openstack/osc-placement master: default to max version when no session https://review.opendev.org/c/openstack/osc-placement/+/794276 | |
| 14:40:11 | gibi | sean-k-mooney: thanks. just added the task number to the commit message too ^^ | |
| 14:40:25 | gibi | this will connect the review to the task in storyboard | |
| 14:40:28 | sean-k-mooney | oh ok i have seen both but i tought task was optional | |
| 14:40:30 | gibi | and update its state | |
| 14:40:33 | sean-k-mooney | oh ok | |
| 14:40:37 | sean-k-mooney | cool | |
| 14:40:56 | sean-k-mooney | am general question | |
| 14:41:13 | sean-k-mooney | if placment is going to remain a deliverable of compute | |
| 14:41:31 | sean-k-mooney | do we want to continue to maintain both storyboard and lauchpad | |
| 14:41:54 | sean-k-mooney | i also just realised the placement repariting spec should have been proposed in the placement repo | |
| 14:42:06 | sean-k-mooney | since that is where placment specs are ment to be tracked | |
| 14:42:35 | gibi | sean-k-mooney: I checked the placement repo but it does not have specs folder | |
| 14:42:35 | sean-k-mooney | https://docs.openstack.org/placement/latest/specs/index.html | |
| 14:42:51 | sean-k-mooney | its in the docs folder | |
| 14:42:55 | sean-k-mooney | so that it renders there | |
| 14:42:56 | gibi | /o\ | |
| 14:43:12 | sean-k-mooney | https://github.com/openstack/placement/tree/master/doc/source/specs | |
| 14:43:49 | sean-k-mooney | i kind of forgot this was there until i rememebred a few days ago | |
| 14:44:52 | gibi | OK, I will move the re-prenting spec to the placement repo :/ | |
| 14:45:04 | sean-k-mooney | well i was not going to ask you to do that | |
| 14:45:18 | sean-k-mooney | just going forward do we want to keep the split | |
| 14:45:39 | sean-k-mooney | e.g. nova has nova-specs and launchpad placement has in repo specs and sotry borad | |
| 14:46:03 | sean-k-mooney | this is more a wider team question or mailing list topic | |
| 14:46:17 | gibi | I don't feel the power to do a migration from storyboard | |