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

Allow URLs in $MACHINEFEATURES and $JOBFEATURES #3

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

Conversation

Andrew-McNab-UK
Copy link

This patch allows one or both of the environment variables to contain URLs rather than local directories. If so, the remote directories are parsed with HTMLParser to find the key names and the dictionaries created as normal. It also has self._collect() in _run() which I needed to make the command line mjf work.

@roiser
Copy link
Owner

roiser commented May 5, 2014

Hi Andrew,

thanks for the patch, in principal this is also what I intended, just that the values should be retrieved as a json from couchdb instead of plain apache. The advantage would be that we can have a web interface to the information (couchdb provides this to us) and also in a later stage of bi-directional communication, the clients could write back simply via HTTP PUT into couchdb. It would only require marginal changes for what you have committed. Could you agree on such a change?

Also if you like to play with the current infrastructure there is one available at CERN, I can give you admin user/password so you can populate and retrieve information from there if you would like to play with it.

cheers
Stefan

@roiser
Copy link
Owner

roiser commented May 5, 2014

for the other fix _run() in _collect() that's all fine with me.

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.

2 participants