-
Notifications
You must be signed in to change notification settings - Fork 172
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
[v6r21] Add DN list from lhcb_pilot group to json file #4008
[v6r21] Add DN list from lhcb_pilot group to json file #4008
Conversation
WorkloadManagementSystem/Utilities/PilotCStoJSONSynchronizer.py
Outdated
Show resolved
Hide resolved
WorkloadManagementSystem/Utilities/PilotCStoJSONSynchronizer.py
Outdated
Show resolved
Hide resolved
This is an LHCb specific fix. To make it generic, one should go through the Operations section and look up for all the Pilot sections and GenericPilotDN option inside them |
Replace by generic option.
Ok. I change it according to the suggestion. I hope it is fine now. |
WorkloadManagementSystem/Utilities/PilotCStoJSONSynchronizer.py
Outdated
Show resolved
Hide resolved
@@ -171,6 +170,20 @@ def _getCSDict(self): | |||
|
|||
return pilotDict | |||
|
|||
def _getDNs(self, certGroupName): |
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.
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.
If not, please anyway update that module, do not add generic functions to this one. And possibly port the test you did here, to there.
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.
Yes, seems the same. The whole function is not needed anyway since it is not used anymore, so I removed it.
just fix the format to make pycodestyle happy. |
Damm sorry. Let's see if it is fine now. |
BEGINRELEASENOTES
WorkloadManagementSystem
CHANGE: Modification of utility function PilotCStoJSONSynchronizer. The modification allows to add information to created json file about the DNs fields of users belonging to 'lhcb_pilot' group. This information is needed for the second level authorization used in the Pilot Logger architecture.
Also, some basic unit tests are added.
ENDRELEASENOTES