Bug #44393
closedpybind/mgr/volumes: add `mypy` support
0%
Description
Adds mypy checks to the mgr/volumes modules.
Updated by Patrick Donnelly about 4 years ago
- Project changed from mgr to CephFS
- Subject changed from mgr/volumes: add `mypy` support to pybind/mgr/volumes: add `mypy` support
- Category deleted (
ceph-mgr) - Status changed from New to Fix Under Review
- Assignee set to Michael Fritch
- Priority changed from Normal to High
- Backport set to nautilus
- Component(FS) mgr/volumes added
Updated by Patrick Donnelly about 4 years ago
- Status changed from Fix Under Review to Pending Backport
Updated by Nathan Cutler about 4 years ago
- Copied to Backport #44473: nautilus: pybind/mgr/volumes: add `mypy` support added
Updated by Nathan Cutler about 4 years ago
(venv) smithfarm@wilbur:~/src/ceph/smithfarm/ceph> git branch --contains e20ad96dbfbdc9396e852a40a8d878f0cfb9a584 master * octopus
No "octopus" backport needed.
Updated by Michael Fritch about 4 years ago
a backport for this is more involved that it would appear due to lack of
mypysupport in nautilus and the need for the py3
typingmodule.
should we consider closing this issue and only support
mypyfrom octopus forward?
Updated by Ramana Raja about 4 years ago
- Status changed from Pending Backport to Resolved
Updated by Ramana Raja about 4 years ago
Michael Fritch wrote:
a backport for this is more involved that it would appear due to lack of [...] support in nautilus and the need for the py3 [...] module.
should we consider closing this issue and only support [...] from octopus forward?
Agree.
Updated by Nathan Cutler about 4 years ago
Just to clarify, there is no src/pybind/mgr/tox.ini in nautilus and fef6450834b665b8b05703a8cbb690a04c4ba217 which added mgr/volumes to the mypy checks in that file was not backported, but a substantial part of https://github.com/ceph/ceph/pull/33674 (i.e. commits fixing various bugs found by running the mypy checks on the code in master) was backported to nautilus via this tracker issue.