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

add merged scripts #136

Merged
merged 7 commits into from Jun 6, 2016
Merged

add merged scripts #136

merged 7 commits into from Jun 6, 2016

Conversation

areinsvo
Copy link
Contributor

Added new script, assign.py, which merges the functionality of assignWorkflow.py and assignProdTaskChain.py.

print "source /data/srv/wmagent/current/apps/wmagent/etc/profile.d/init.sh"
sys.exit(1)

global_pool = 'vocms099.cern.ch'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to change this line because the HTCondor collector is going to be migrated (https://hypernews.cern.ch/HyperNews/CMS/get/comp-ops/2929.html). It is better if we do some tests.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I changed the line Paola mentioned above, so that global_pool points to the landb alias 'cmsgwms-collector-global.cern.ch' for monitoring systems. I've tested it and it works as expected.

@vlimant
Copy link
Contributor

vlimant commented Apr 22, 2016

Could you please go as far as using the siteInfo class in utils.py to get the list of sites in a more dynamic way ?
Can you please report below what are the tests that you have performed for this assign.

@vlimant
Copy link
Contributor

vlimant commented Jun 2, 2016

please make --lfn mandatory.
we'll find a way to fetch the value from a relevant place in due time : i.e after merging this

@vlimant vlimant merged commit 632eca4 into CMSCompOps:master Jun 6, 2016
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