Use Ark repofiles for additional repos #832
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
There was a problem hiding this comment.
The cernvm repos used for EESSI were added to Ark in stackhpc/stackhpc-release-train#427 (rather than stackhpc/stackhpc-release-train#435 which is what the rest of this is handling). But I think they are currently unused.
If otherwise this is basically good-to-go I'm not averse to punting that fix into a separate PR.
There was a problem hiding this comment.
Missing cernvm, see PLATFORM-941.
| @@ -1,12 +1,5 @@ | |||
| --- | |||
| # Should be run on compute nodes you want to run the graphical desktop on | |||
| - name: Enable TurboVNC repo | |||
There was a problem hiding this comment.
Could we just add a comment to say repofile provided by dnf_repos?
Similarly if its OK to remove this task, I think we can remove the "install epel" task right? dnf_repos will always /will always have to provide it
| ood_auth_openidc: "{{ openondemand_auth_defaults.oidc.ood_auth_openidc if (openondemand_auth | lower) == 'oidc' else none }}" | ||
| httpd_auth: "{{ openondemand_auth_defaults[openondemand_auth | lower].httpd_auth }}" | ||
|
|
||
| # Use Ark pulp repos by default |
There was a problem hiding this comment.
| # Use Ark pulp repos by default | |
| # Use repo file provided by dnf_repos by default |
b/c it could be local Pulp, not Ark directly
| httpd_auth: "{{ openondemand_auth_defaults[openondemand_auth | lower].httpd_auth }}" | ||
|
|
||
| # Use Ark pulp repos by default | ||
| ood_use_existing_repo_file: true |
There was a problem hiding this comment.
Indentation is wrong here, so its part of the ssl: var.
2 participants
No description provided.