Skip to content
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

fence_zvmip: Update longdesc to document all required functions #428

Merged
merged 1 commit into from
Sep 19, 2023

Conversation

nrwahl2
Copy link
Collaborator

@nrwahl2 nrwahl2 commented Aug 17, 2021

In RHBZ#1935641, IBM explained that the requesting user needs authorization
for more functions than what is currently documented.

They said:
"""
What we found is that you need rights from three different NICKS:
SYSTEM_MANAGEMENT, IMAGE_CHARACTERISTICS and IMAGE_OPERATIONS.
You won't be able to give a user all three NICKS.
Therefore, you have to create a new NICK with all capabilities from all
three NICKS together and then assign the new NICK to the USER
"ZCLUSTER".
Even better is to just use the needed Subset with a new NICK.
We found five commands which are used in the fencing code and on the
z/VM Log which should be enough for fencing to work.

We suggest creating following files:

File VSMWORK1 NAMELIST:

:nick.ZVM_FENCE
:list.
IMAGE_ACTIVATE
IMAGE_DEACTIVATE
IMAGE_STATUS_QUERY
CHECK_AUTHENTICATION
IMAGE_NAME_QUERY_DM

File VSMWORK1 AUTHLIST:

ZCLUSTER                            ALL                              ZVM_FENCE

For details, we suggest adding a link to the current z/VM docu:

Resolves: RHBZ1935641

Signed-off-by: Reid Wahl nrwahl@protonmail.com

agents/zvm/fence_zvmip.py Outdated Show resolved Hide resolved
@nrwahl2 nrwahl2 force-pushed the nrwahl2-rhbz1935641 branch 2 times, most recently from c6788f6 to f464d9a Compare August 17, 2021 08:57
@knet-jenkins
Copy link

knet-jenkins bot commented Jun 12, 2023

Can one of the admins check and authorise this run please: https://ci.kronosnet.org/job/fence-agents-pipeline/job/PR-428/1/input

@fabbione
Copy link
Member

retest this please

In RHBZ#1935641, IBM explained that the requesting user needs
authorization for more functions than what is currently documented.

They said:
"""
What we found is that you need rights from three different NICKS:
SERVER_MANAGEMENT, IMAGE_CHARACTERISTICS and IMAGE_OPERATIONS.
You won't be able to give a user all three NICKS.
Therefore, you have to create a new NICK with all capabilities from all
three NICKS together and then assign the new NICK to the USER
"ZCLUSTER".
Even better is to just use the needed Subset with a new NICK.
We found five commands which are used in the fencing code and on the
z/VM Log which should be enough for fencing to work.

We suggest creating following files:

File VSMWORK1 NAMELIST:
```
:nick.ZVM_FENCE
:list.
IMAGE_ACTIVATE
IMAGE_DEACTIVATE
IMAGE_STATUS_QUERY
CHECK_AUTHENTICATION
IMAGE_NAME_QUERY_DM
```

File VSMWORK1 AUTHLIST:
```
ZCLUSTER                            ALL                              ZVM_FENCE
```

For details, we suggest adding a link to the current z/VM docu:
 - NAMELIST: https://www.ibm.com/support/knowledgecenter/de/SSB27U_7.2.0/com.ibm.zvm.v720.dmse6/namelst.htm
 - AUTHLIST: https://www.ibm.com/support/knowledgecenter/de/SSB27U_7.2.0/com.ibm.zvm.v720.dmse6/auf.htm
"""

Resolves: RHBZ1935641

Signed-off-by: Reid Wahl <nrwahl@protonmail.com>
@knet-jenkins
Copy link

knet-jenkins bot commented Sep 12, 2023

Can one of the admins check and authorise this run please: https://ci.kronosnet.org/job/fence-agents/job/fence-agents-pipeline/job/PR-428/4/input

@nrwahl2
Copy link
Collaborator Author

nrwahl2 commented Sep 12, 2023

retest this please

1 similar comment
@fabbione
Copy link
Member

retest this please

@oalbrigt oalbrigt merged commit 72a1dc4 into ClusterLabs:main Sep 19, 2023
1 check passed
@oalbrigt
Copy link
Collaborator

Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants