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

Jump to bottom

Move the methods calls in check_rse_attributes from the API to the core #519

Closed
cserf opened this issue Jan 24, 2018 · 0 comments
Closed

Move the methods calls in check_rse_attributes from the API to the core #519

cserf opened this issue Jan 24, 2018 · 0 comments
Assignees
@cserf
Labels
enhancement patch Probes & Alarms
Milestone
1.14.8

Comments

@cserf
Copy link
Contributor

cserf commented Jan 24, 2018

Motivation

Using the API methods in check_rse_attributes induces a lot of commits since we open a new session each time.

Modification

Move the methods calls in check_rse_attributes from the API to the core and reuse the DB session.
cserf added a commit to cserf/rucio that referenced this issue Jan 24, 2018
@cserf
Probes : Move the methods calls in check_rse_attributes from the API …
dc29e7e 
…to the core rucio#519
This was referenced Jan 24, 2018
Merged
Merged
@bari12 bari12 added this to the 1.14.8 milestone Jan 24, 2018
bari12 added a commit that referenced this issue Jan 25, 2018
@bari12
Merge pull request #520 from cserf/patch-519-Move_the_methods_calls_i…
cfc258a 
…n_check_rse_attributes_from_the_API_to_the_core__519

Probes : Move the methods calls in check_rse_attributes from the API to the core #519
bari12 added a commit that referenced this issue Jan 25, 2018
@bari12
Merge pull request #521 from cserf/patch-519-Move_the_methods_calls_i…
3141d24 
…n_check_rse_attributes_from_the_API_to_the_core__519

Probes : Move the methods calls in check_rse_attributes from the API to the core #519
@bari12 bari12 closed this as completed Jan 25, 2018
yuyiguo pushed a commit to yuyiguo/rucio that referenced this issue Sep 20, 2023
@ericvaandering
Merge pull request rucio#519 from dynamic-entropy/use-rucio-policy-pa…
ec2a45b 
…ckage

Updating dockefiles with relative paths from repo root
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cserf cserf

Labels
enhancement patch Probes & Alarms
Projects
None yet
Milestone
1.14.8
Development

No branches or pull requests
2 participants
@bari12 @cserf