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

Draft PR: policy package support for Rucio server #223

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

rdimaio
Copy link
Contributor

@rdimaio rdimaio commented Feb 4, 2025

Initial PR to add policy package support to the Rucio server.

Current questions:

  • Does the |'d command work when using multiple packages?

@rdimaio
Copy link
Contributor Author

rdimaio commented Feb 5, 2025

Amended version works correctly. Used these values:

policyPackages:
  enabled: true
  # Make sure the trailing slash is present
  mountPath: /opt/policy_packages/
  # Use underscores instead of hyphens for package names
  # (e.g. atlas_rucio_policy_package instead of atlas-rucio-policy-package)
  packages:
    - moduleName: atlas_rucio_policy_package
      requirement: atlas-rucio-policy-package==0.4.0
      version: 0.4.0
    - moduleName: belleii_rucio_policy_package
      requirement: git+https://github.com/rucio/[email protected]
      version: 0.1.0
  resources:
    requests:
      storage: 100Mi

First deployment installs both policy packages as expected, second deployment does not install anything as they already installed.

@rdimaio rdimaio force-pushed the 221-policy branch 2 times, most recently from c1abfed to 31625d0 Compare February 10, 2025 09:54
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.

1 participant