Skip to content

Conversation

Geogouz
Copy link
Contributor

@Geogouz Geogouz commented Jul 17, 2025

This commit basically cleans up a bit the project's Docker container definitions.

  • It removes several packages that are no longer needed, such as , gcc, libnsl, and libaio (similar to dependencies: Remove unused libnsl and libaio from dev test image; Fix #7861 rucio#7867), which were likely required by cx_oracle only.
  • It standardizes the Python package installations to use the python3 versions, for example, by replacing python-pip with python3-pip.
  • Additionally, a minor typo in a file copy command was corrected, and license headers were added to some files.

Although I tested locally that the images are built successfully, it is important to test them while being used because in practice things might break during runtime too.

Couple extra points:

  • It would be nice if we had a way to track which package requires what dependencies.
  • It would be nice to have some CI/testing workflow where the pushed changes are evaluated in practice.

@Geogouz Geogouz requested review from maany and rdimaio July 17, 2025 12:51
@Geogouz Geogouz self-assigned this Jul 17, 2025
@Geogouz Geogouz added enhancement New feature or request invalid This doesn't seem right dependencies Pull requests that update a dependency file labels Jul 17, 2025
@Geogouz Geogouz changed the title Update dependencies; Fix #420 Update dependencies; Fix #425 Jul 17, 2025
@Geogouz Geogouz marked this pull request as draft October 6, 2025 05:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file enhancement New feature or request invalid This doesn't seem right

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clean up and streamline Docker container definitions

2 participants