| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-06-09 | |||
| 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 | |
| 14:46:21 | sean-k-mooney | but that bug just reminded me of it | |
| 14:46:59 | gibi | we have very few incoming things in placemnet | |
| 14:47:42 | sean-k-mooney | true | |
| 14:47:55 | sean-k-mooney | do we want to add it to the meeting adgenda | |
| 14:48:08 | sean-k-mooney | we have the bug review for nova in lanuch pad | |
| 14:48:17 | sean-k-mooney | do we have somethign that will do the same for story borad | |
| 14:48:37 | sean-k-mooney | e.g. a linke to the open stories | |
| 14:48:58 | sean-k-mooney | its pretty stable and well tested so placment bugs are rare | |
| 14:50:55 | gibi | sean-k-mooney: please put it up to the agenda. Honestly I don't think it worth the effort to do extra things with placement but it make sense to discuss it | |
| 14:52:12 | sean-k-mooney | oh i was suggesting adding a "check for open placment bugs" item | |
| 14:52:19 | sean-k-mooney | like when we do the nova bug review | |
| 14:52:38 | sean-k-mooney | more just to prompt use to look for placment bugs if tehy areever reported | |
| 14:53:00 | sean-k-mooney | most get brought up here anyway since nova typicly is what finds them if they exist | |