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

Implement flux for testing #3207

Closed
wants to merge 2 commits into from
Closed

Implement flux for testing #3207

wants to merge 2 commits into from

Conversation

christailu
Copy link
Contributor

@christailu christailu commented Mar 7, 2024

Description

In this PR, we are enhancing the existing workflow for testing Parsl with Flux integration. The goal is to ensure that Parsl continues to work seamlessly with Flux and that any changes do not impact the functionality negatively..

Changed Behaviour

The main change in behavior is the addition of testing for Parsl with Flux integration.
The workflow now includes steps to install the correct version of Parsl based on the pull request, run local thread tests to validate the installation and perform Flux-specific tests to verify Parsl's functionality with Flux.
We aim to provide comprehensive testing coverage for Parsl with Flux to identify and address any potential issues early in the development process.

Fixes

Fixes # (#2713 )

Type of change

Choose which options apply, and delete the ones which do not apply.

  • New feature

@christailu christailu marked this pull request as draft March 7, 2024 20:36
@christailu christailu mentioned this pull request Mar 7, 2024
@christailu christailu changed the title Implent flux for testing Implement flux for testing Mar 7, 2024
@benclifford
Copy link
Collaborator

you might be interested in following along on PR #3159 which is a little bit further along that your PR

@benclifford
Copy link
Collaborator

do you still want to work on this PR? @christailu

@benclifford
Copy link
Collaborator

PR #3159 provides flux testing, so I'll close this PR now. Thanks for participating in Outreachy!

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