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 node profile for cloud #174

Merged
merged 13 commits into from
Aug 9, 2022
Merged

add node profile for cloud #174

merged 13 commits into from
Aug 9, 2022

Conversation

hoelzer
Copy link
Contributor

@hoelzer hoelzer commented Apr 1, 2022

For computes in the cloud, we need a profile that just provides the container and nodes configuration

@hoelzer
Copy link
Contributor Author

hoelzer commented Apr 1, 2022

There are also processes that don't define a container which will fail in the cloud #135

I will push changes to this branch

@hoelzer
Copy link
Contributor Author

hoelzer commented Apr 5, 2022

srry for the uninformative commit messages. Basically, the dual label policy to add a smallTask label for HPC execution like on the FSU Jena Ara causes problems in the cloud. Basically, the smallTask label overwrites the more important label that defines the container. And for Cloud, each process needs a container. I think just switching the smallTask label definition before the other label might work but I decided now to just check of the node profile is used (for cloud) and if so skip the smallTask label definition.

@hoelzer hoelzer merged commit f3b8889 into master Aug 9, 2022
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.

1 participant