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.

Sign up for GitHub

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

Jump to bottom

added play to install texlive on all nodes #245

Closed
wants to merge 3 commits into from
Closed

added play to install texlive on all nodes #245

wants to merge 3 commits into from

Conversation

andriy-safe-ai
Copy link
Contributor

Tested on dev cluster.

@andriy-safe-ai andriy-safe-ai requested review from a user and steven-safeai December 4, 2023 21:37
@andriy-safe-ai andriy-safe-ai self-assigned this Dec 4, 2023
@andriy-safe-ai andriy-safe-ai linked an issue Dec 4, 2023 that may be closed by this pull request
Install the texlive package #244
Open
@steven-safeai
Copy link
Contributor

Did someone ask or request this?

@andriy-safe-ai
Copy link
Contributor Author

@steven-basart

Yes. Check the help desk.

@andriy-safe-ai andriy-safe-ai changed the title added play to install texlive on compute nodes added play to install texlive on all nodes Dec 4, 2023
@andriy-safe-ai
Copy link
Contributor Author

Changed to install texlive on all nodes.

@steven-safeai
Copy link
Contributor

This is borderline a no from me. I don't want people running lots of python stuff to compile plots on the login node. Compute node is also kind of meh where we want to make sure they don't request a GPU node to make plots.

steven-safeai
steven-safeai reviewed Dec 4, 2023
View reviewed changes
playbooks/roles/cais-all/tasks/ol-7.yml Outdated
@@ -72,6 +72,17 @@
name: safe_yum
ignore_errors: true

# Install texlive
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not on all nodes. Only on compute and we want to make sure they request a CPU only for this.

@andriy-safe-ai
Copy link
Contributor Author

andriy-safe-ai commented Dec 4, 2023 •
edited
Loading

@steven-basart

I'm not sure how we can enforce cpu only for this other than emphasizing that they should only request cpus.

I've pushed a change to only install texlive on our compute nodes. I've also added dvipng which is also needed by the user.

@steven-safeai
Copy link
Contributor

dvipng this one can be installed by pip or conda I think?

@andriy-safe-ai andriy-safe-ai deleted the 244-install-the-texlive-package branch December 5, 2023 22:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@steven-safeai steven-safeai Awaiting requested review from steven-safeai

Assignees

@andriy-safe-ai andriy-safe-ai

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Install the texlive package
2 participants
@andriy-safe-ai @steven-safeai