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 compute4punch compute_backend #780

Merged
merged 1 commit into from
Jul 5, 2024

Conversation

giffels
Copy link
Contributor

@giffels giffels commented Feb 21, 2024

This pull request adds the new compute4punch compute backend to the list of available compute backends. The compute4punch compute backend is currently implemented in reanahub/reana-job-controller#430.

@giffels giffels force-pushed the add/compute4punch-support branch from df74aad to 89bce51 Compare May 10, 2024 13:45
@giffels giffels changed the base branch from master to maint-0.9 May 10, 2024 13:46
Copy link

codecov bot commented May 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 18.15%. Comparing base (822c386) to head (89bce51).
Report is 1 commits behind head on maint-0.9.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           maint-0.9     #780   +/-   ##
==========================================
  Coverage      18.15%   18.15%           
==========================================
  Files             26       26           
  Lines           2478     2478           
==========================================
  Hits             450      450           
  Misses          2028     2028           
Files Coverage Δ
reana/config.py 100.00% <100.00%> (ø)
reana/reana_dev/cluster.py 0.00% <ø> (ø)

@arm2arm
Copy link

arm2arm commented May 31, 2024

Hi @tiborsimko , what is the situation with this integration, I have a branch with slurm integration as well, where to start merging them ?

@tiborsimko
Copy link
Member

Hi @tiborsimko , what is the situation with this integration, I have a branch with slurm integration as well, where to start merging them ?

Hi @arm2arm The integration was delayed due to some illnesses and absences on our end... We are getting to it this week and the next.

BTW some of the remarks made by @mdonadoni in @giffels's pull requests were still not fully closed and may be important, such as a possible code injection? Would you be able to look into them before the integration?

mdonadoni pushed a commit to giffels/reana that referenced this pull request Jul 4, 2024
@mdonadoni mdonadoni force-pushed the add/compute4punch-support branch from 89bce51 to 41792fe Compare July 4, 2024 12:54
mdonadoni pushed a commit to giffels/reana that referenced this pull request Jul 4, 2024
@mdonadoni mdonadoni force-pushed the add/compute4punch-support branch from 41792fe to 776ec76 Compare July 4, 2024 12:55
@mdonadoni mdonadoni force-pushed the add/compute4punch-support branch from 776ec76 to c2f490b Compare July 4, 2024 15:34
@mdonadoni mdonadoni merged commit c2f490b into reanahub:maint-0.9 Jul 5, 2024
13 checks passed
tiborsimko added a commit to tiborsimko/reana that referenced this pull request Aug 29, 2024
build(helm): add support for Kubernetes 1.31 (reanahub#822)
ci(python): pin setuptools 70 (reanahub#822)
feat(config): add Compute4PUNCH backend (reanahub#780)

Note: The merge commit removes the changes related to pinning
`setuptools` to version 70, because this was only necessary for the
`maint-0.9` branches.
tiborsimko added a commit to tiborsimko/reana that referenced this pull request Aug 29, 2024
build(helm): add support for Kubernetes 1.31 (reanahub#822)
ci(python): pin setuptools 70 (reanahub#822)
feat(config): add Compute4PUNCH backend (reanahub#780)

Note: The merge commit removes the changes related to pinning
`setuptools` to version 70, because this was only necessary for the
`maint-0.9` branches.
jlemesh pushed a commit to jlemesh/reana that referenced this pull request Sep 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants