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

remove the need for perl in the engine environment #550

Closed
k-rister opened this issue Sep 11, 2024 · 5 comments · Fixed by #552
Closed

remove the need for perl in the engine environment #550

k-rister opened this issue Sep 11, 2024 · 5 comments · Fixed by #552
Assignees

Comments

@k-rister
Copy link
Contributor

No description provided.

@k-rister k-rister converted this from a draft issue Sep 11, 2024
@k-rister k-rister self-assigned this Sep 11, 2024
@k-rister
Copy link
Contributor Author

We are having some build issues with the toolbox perl dependency issues in some experimental userenvs which brings focus on this. Ultimately, we don't intend to require perl in the engine environment but it has snuck in because we are dependent on https://github.com/perftool-incubator/toolbox/blob/main/bin/get-json-settings.pl. We have decided to adjust this because it will fix the current problems being faced as well as reduce the size of the engine container images.

@k-rister
Copy link
Contributor Author

The first step towards resolving this is creating a Python implementation of get-json-settings.pl (Python is ok since we already depend on that for things like Roadblock). That is being addressed by perftool-incubator/toolbox#84.

@k-rister k-rister moved this from Queued to In Progress in Crucible Tracking Sep 11, 2024
@k-rister
Copy link
Contributor Author

perftool-incubator/toolbox#84 has been merged.

The second step is to change the engine code to use the new Python implementation of get-json-settings.
This is being addressed by #551.

@k-rister
Copy link
Contributor Author

#551 has been merged.

The third step is to remove the perl module requirements from the toolbox workshop.json. By removing the requirements but leaving the workshop.json intact and included by rickshaw-run it allows for the future addition of other toolbox dependencies should the need arise (simply by adding them to the workshop.json without having to do anything else). This is being addressed by perftool-incubator/toolbox#85.

@k-rister k-rister linked a pull request Sep 13, 2024 that will close this issue
@k-rister
Copy link
Contributor Author

perftool-incubator/toolbox#85 has been merged

The fourth and final (?) step is to remove all the perl packages that are "natively" included in the userenvs by the userenv embedded requirements. This is being address by #552.

@github-project-automation github-project-automation bot moved this from In Progress to Done in Crucible Tracking Sep 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant