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

RP executor #6

Merged
merged 3 commits into from
Mar 11, 2024
Merged

RP executor #6

merged 3 commits into from
Mar 11, 2024

Conversation

andre-merzky
Copy link
Collaborator

This PR implements a minimal RP executor. It accepts task requests and proxies them to the pilot instance created upon startup. The tasks are the run by the pilot and the executor gets a state notification upon completion.

Missing: the state notification is not yet communicated back to airflow, the dag thus stalls.

@andre-merzky andre-merzky added the enhancement New feature or request label Feb 26, 2024
@andre-merzky andre-merzky self-assigned this Feb 26, 2024
@andre-merzky andre-merzky marked this pull request as draft February 26, 2024 11:35
@andre-merzky
Copy link
Collaborator Author

andre-merzky commented Feb 26, 2024

Result collection now works.

Note that I had to execute the following steps in order to get the examples running:

$ pip install MDAnalysis
$ pip install alchemlyb
$ pip uninstall typing

The last triggered some module incompatibility within RP, I'l try to track this down (see radical-cybertools/radical.pilot/issues/3139).

I also needed to change the following values in airflow.cfg from

enable_xcom_deserialize_support = False
allowed_deserialization_classes = airflow\.*

to

enable_xcom_deserialize_support = True
allowed_deserialization_classes = airflow.*

For the latter, see apache/airflow#34093 - the fix was merged but is not yet released.

In order to use the executor, please create a file like this:

$ cat ~/.radical/pilot/configs/resource_local.json
{
    "localhost": {
        "rp_version"    : "installed",
        "virtenv_mode"  : "local"
    }
}

@ejjordan ejjordan merged commit 94b9b6c into main Mar 11, 2024
@andre-merzky andre-merzky deleted the feature/rp_executor branch September 4, 2024 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants