-
Notifications
You must be signed in to change notification settings - Fork 42
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
Policy package support for Rucio server #223
Conversation
Amended version works correctly. Used these values:
First deployment installs both policy packages as expected, second deployment does not install anything as they already installed. |
31625d0
to
360a0c7
Compare
9faa675
to
63c6a6a
Compare
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.
Looks fine. I took the liberty of making a few additional style-related comments that are undoubtedly a matter of personal opinion. They apply in general and not exclusively to the lines and/or file they refer to. Feel free to ignore them.
Initial PR to add policy package support to the Rucio server.
Current questions:
|
'd command work when using multiple packages? (edit: it does)