Skip to content

Added support for tape library#2092

Open
vontedduchaithra wants to merge 1 commit intomasterfrom
virtualtape
Open

Added support for tape library#2092
vontedduchaithra wants to merge 1 commit intomasterfrom
virtualtape

Conversation

@vontedduchaithra
Copy link
Contributor

No description provided.

@coveralls
Copy link
Collaborator

coveralls commented Feb 4, 2026

Coverage Status

coverage: 77.339% (+0.1%) from 77.2%
when pulling 76d6c17 on virtualtape
into 0e5c969 on master.

Signed-off-by: Vonteddu Chaithra <Vonteddu.Chaithra1@ibm.com>
primary-hostname-ipaddr: 10.11.12.13
search-distinguished-name: "test{0}"
replication-overwrite-possible: false
description: "LSD 1"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you remove the description?

primary-hostname-ipaddr: 10.11.12.13
search-distinguished-name: "test{0}"
replication-overwrite-possible: false
description: "LSD 1"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you remove the description?

@@ -0,0 +1,117 @@
# Copyright 2025 IBM Corp. All Rights Reserved.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a new file created in 2026, so please change the copyright to 2026.

@@ -0,0 +1,324 @@
# Copyright 2018,2021 IBM Corp. All Rights Reserved.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a new file created in 2026, so please change the copyright to 2026.

@@ -0,0 +1,448 @@
# Copyright 2018,2021 IBM Corp. All Rights Reserved.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a new file created in 2026, so please change the copyright to 2026.

self._base_uri + "/operations/request-tape-library-zoning",
body=body
)
print("RESULT", result)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove this print.

Authorization requirements:

* Object-access permission to this tape library.
* Task permission to the "Configure Storage - System Programmer" task.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Task permissions are: "Configure Storage – System Programmer" or "Configure Storage – Storage Administrator"

)

@logged_api_call
def request_tape_library_zoning(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suggest to change the method name to "request_zoning", since the method is already invoked on a tape library manager, so the tape library context is already clear.

return result

@logged_api_call
def discover_tape_libraries(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suggest to change the method name to "discover", since the method is already invoked on a tape library manager, so the tape library context is already clear.

self._cpc = None

@logged_api_call
def undefine_tape_library(self):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suggest to change the method name to "undefine", since the method is already invoked on a tape library resource, so the tape library context is already clear.

@andy-maier andy-maier added this to the 1.26.0 milestone Feb 5, 2026
@andy-maier andy-maier linked an issue Feb 5, 2026 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support for Tape Library resource

3 participants