targetcli-fb-devel
January 2015
targetcli-fb-devel@lists.fedorahosted.org
- 2 participants
- 2 discussions
Re: Fwd: initiator group support in rtslib
by Andy Grover
On 11/26/2014 01:17 AM, Gris Ge wrote:
> On Tue, Nov 25, 2014 at 07:54:50AM -0800, Andy Grover wrote:
>> Hi Gris, saw your irc msg.
>>
>> Fwding an email I sent to targetcli-fb and targetd lists, not sure
>> if you're on those. But yes, I was not going to merge the rtslib
>> changes until we knew that they were what we needed for targetd. It
>> sounds like you're signed up for the targetd work, so please let me
>> know of any bugs you find or API changes to rtslib that would make
>> your work easier.
>>
>> And then once we're happy with both, we can commit to both repos.
>>
>> Thanks! -- Andy
>>
> Hi Andy,
Hi Gris. I'm CCing the lists on my response, just for the sake of having
good technical discussions like this in public.
> These might make targetd life easier:
>
> 1. Change NodeACLGroup._node_acls as a public property.
> # It is require to get a list of group member NodeACL
I did agree, but then changed my mind, based on the below.
> 2. How about let NodeACLGroup.remove_acl() remove mapping status
> also?
> # If not, targetd have to do the mapping removal when remove a
> # initiator(ACL) from a initiator group(NodeACLGroup).
nodeacl.delete() deletes the nodeacl as well as any mappedluns within it.
> # It means targetd will use both initiator mapping and group
> # mapping.
> #
> # I thought the intention of adding NodeACLGroup and MappedLUNGroup is
> # to hide/replace the NodeACL and MappedLUN. Is it?
I wasn't sure, to be honest. How would this work? No methods in
NodeACLGroup or MappedLUNGroup would take or return NodeACLs or
MappedLUNs. This would mean #1 suggestion above should not be done.
NodeACL and MappedLUN are still used internally, but the API can be
*Group classes and wwn strings entirely I think. It could be good.
> #
> # My understanding of initiator group is:
> # * Volume mapping status is maintained at group level.
> # * Initiator in this group just inherit that mapping status.
> # * Initiator add or remove means grant or revoke that inheritance.
>
> 3. It seems we have no quick way to query masked NodeACLGroup for
> given LUN.
> # We have the revert one:
> # nag = NodeACLGroup(tpg, nag_name)
> # nag.mapped_luns()
> #
> # I am expecting something like this:
> # lun = LUN(tpg, lun=<target_lun_id>, storage_object=<st_obj>)
> # lun.mapped_node_acl_groups()
That's not the reverse, nag.mapped_luns gives MappedLUNGroups. The
reverse is MappedLUNGroup.parent_nodeaclgroup. (Should we change
nag.mapped_luns property name to nag.mapped_lun_groups perhaps?)
> 4. NodeACLGroup.add_acl() didn't override old iSCSI CHAP setting.
It doesn't? That's what the "if model.has_feature("auth")" section was
supposed to do. Is that not working right? Or are you referring to
something else that was missed?
OK I'm going to update the dev-groups branch to change the usage model
to be more that users just use Group'd class versions exclusively. I
think just add_acl and remove_acl have to change.
Not tested, sorry. BTW USA workers have Thursday and Friday off so I
won't be around, but if this change is an improvement then we can sync
up on Monday, and if it's not, then we can also sync up on Monday :)
Thanks! -- Regards -- Andy
9 years, 3 months
- 2
- 12
New releases: targetcli.fb39, rtslib.fb52, and configshell.fb17
by Andy Grover
Hi all,
The big change in this release is that rtslib-fb and configshell-fb have
started transitioning to different package names. Instead of "import
rtslib" or "import configshell", users should import rtslib_fb and
configshell_fb. This is better because it avoids clashes with the
original library import names, which we no longer match 100%.
These releases export "rtslib" and "rtslib_fb", with "rtslib" imports
emitting a warning. A future version will only support "rtslib_fb". The
same goes for configshell and configshell_fb.
Also, there are some nice documentation improvements for parameters and
attributes by Christophe in targetcli, and rtslib exports a NodeACLGroup
class, that makes it easier to configure NodeACLs as if LIO supported
initiator groups.
Github:
https://github.com/agrover/targetcli-fb
https://github.com/agrover/rtslib-fb
https://github.com/agrover/configshell-fb
tarballs:
https://fedorahosted.org/released/targetcli-fb/
targetcli-fb 2.1.fb39:
Andy Grover <agrover(a)redhat.com> (5):
Change targetcli to use new package import names
Autocomplete for restoreconfig and clearconfig
Catch exception if an invalid tag is given when creating a tpg
Add a global pref for adding default portal
update version to 2.1.fb39
Christophe Vu-Brugier <cvubrugier(a)fastmail.fm> (2):
Add UI parameter type helper for "Yes" and "No" boolean values
Document target parameters and attributes
rtslib-fb 2.1.fb52:
Andy Grover <agrover(a)redhat.com> (8):
Change to exporting under 'rtslib' and 'rtslib_fb" pkg names
Create symlink from rtslib_fb to rtslib
Mention default save/restore file in docstrings
Alias cleanup
Improve repr for MappedLUN
Revert "Increase MAX_LUN to 16383"
Add support for NodeACLGroup
update version to 2.1.fb52
configshell-fb 1.1.fb17:
Andy Grover <agrover(a)redhat.com> (3):
Package as both 'configshell' and 'configshell_fb'
Add symlink from configshell_fb to configshell/
update version to 1.1.fb17
Regards -- Andy
9 years, 3 months
- 1
- 0
- ← Newer
- 1
- Older →