-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WIP] Support for Partition Links #1685
base: master
Are you sure you want to change the base?
Conversation
e3498a8
to
3184b41
Compare
3184b41
to
f74db69
Compare
2a01040
to
15c7bf0
Compare
15c7bf0
to
eab61e3
Compare
zhmcclient/_partition_link.py
Outdated
self, partition, properties, wait_for_completion=True, | ||
operation_timeout=None): | ||
""" | ||
Attach this partition link to a partition, creating corresponding |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have the attach tape link/storage group operations on the Partition resource. I am wondering whether it makes sense to move the attach partition link operation there as well for consistency.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the idea, but it is not clear to me whether that can be done for CTC. Let's discuss.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have moved them and split them up between network and CTC.
bb1e363
to
4b70c5f
Compare
4b70c5f
to
5b61c6e
Compare
Signed-off-by: Andreas Maier <[email protected]>
5b61c6e
to
f775287
Compare
WIP TODOs: