Bug #40014
closed
mgr/volumes: Name 'sub_name' is not defined
Added by Sebastian Wagner almost 5 years ago.
Updated almost 5 years ago.
Component(FS):
mgr/volumes
Description
I'm getting a new mypy error in master:
pybind/mgr/volumes/module.py: note: In member "_cmd_fs_subvolumegroup_snapshot_rm" of class "Module":
pybind/mgr/volumes/module.py:552: error: Name 'sub_name' is not defined
Regression of https://github.com/ceph/ceph/pull/27594
- Project changed from mgr to CephFS
- Component(FS) mgr/volumes added
- Status changed from New to Fix Under Review
- Pull request ID set to 28407
Ramana, I think we should just mention that this issue will be fixed w/ subvolume refactor and mark as resolved once that's merged?
It seems a bit odd (to me) mentioning this tracker in one of the commits in the refactor pr (as I do not want to have a separate commit just for this).
Venky Shankar wrote:
Ramana, I think we should just mention that this issue will be fixed w/ subvolume refactor and mark as resolved once that's merged?
It seems a bit odd (to me) mentioning this tracker in one of the commits in the refactor pr (as I do not want to have a separate commit just for this).
I didn't expect that you create a separate commit, but just add 'Fixes: <tracker link>' to the commit message of the commit that fixes the issue. But if you think this is odd, I guess it's OK as long as the issue gets fixed in master and nautilus.
- Target version set to v15.0.0
- Start date deleted (
05/23/2019)
- Backport set to nautilus
Ramana Raja wrote:
Venky Shankar wrote:
Ramana, I think we should just mention that this issue will be fixed w/ subvolume refactor and mark as resolved once that's merged?
It seems a bit odd (to me) mentioning this tracker in one of the commits in the refactor pr (as I do not want to have a separate commit just for this).
I didn't expect that you create a separate commit, but just add 'Fixes: <tracker link>' to the commit message of the commit that fixes the issue. But if you think this is odd, I guess it's OK as long as the issue gets fixed in master and nautilus.
yeh -- don't want to just add `Fixes` the commit does much more than that...
- Status changed from Fix Under Review to Pending Backport
- Copied to Backport #40338: nautilus: mgr/volumes: Name 'sub_name' is not defined added
- Status changed from Pending Backport to Resolved
Also available in: Atom
PDF