Skip to content

Conversation

@ZePan110
Copy link
Collaborator

@ZePan110 ZePan110 commented Aug 13, 2025

Description

Remove libgl1-mesa-glx from Dockerfiles

Issues

List the issue or RFC link this PR is working on. If there is no such link, please mark it as n/a.

Type of change

List the type of change like below. Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds new functionality)
  • Breaking change (fix or feature that would break existing design and interface)
  • Others (enhancement, documentation, validation, etc.)

Dependencies

List the newly introduced 3rd party dependency if exists.

Tests

Describe the tests that you ran to verify your changes.

@github-actions
Copy link

ℹ️ INFO
File comps/guardrails/src/bias_detection/Dockerfile resulted in a change in the image size from 3018 -> 2855 MB (diff: -163 MB)
ℹ️ INFO
File comps/third_parties/llama-vision/src/Dockerfile resulted in a change in the image size from 5199 -> 5199 MB (diff: 0 MB)

@ZePan110
Copy link
Collaborator Author

ZePan110 commented Aug 13, 2025

Check Docker Image Size Change fail is not related to this PR modification.
It is due to base images update that the main branch build failed. So it can be directly merged

@letonghan
Copy link
Collaborator

Check Docker Image Size Change fail is not related to this PR modification. It is due to base images update that the main branch build failed. So it can be directly merged

Please check the failed microservice tests and make sure all of them passed.

@ZePan110
Copy link
Collaborator Author

ZePan110 commented Aug 13, 2025

Known issues:
dataprep_opensearch and retriever-opensearch is known issue #1869
text2cypher #1885
llms #1886

@hteeyeoh
Copy link
Collaborator

the purpose of removing this packages is because of build issues?

@ZePan110
Copy link
Collaborator Author

ZePan110 commented Aug 13, 2025

the purpose of removing this packages is because of build issues?

Yes, this PR includes that purpose.
Error link: https://github.com/opea-project/GenAIComps/actions/runs/16930366993/job/47974304068#step:6:314

@hteeyeoh
Copy link
Collaborator

the purpose of removing this packages is because of build issues?

Yes, this PR includes that purpose. Error link: https://github.com/opea-project/GenAIComps/actions/runs/16930366993/job/47974304068#step:6:314

Ya alright. There was issue when compiling with ubuntu 23.10 or newer version.

@ZePan110
Copy link
Collaborator Author

Check Docker Image Size Change fail is not related to this PR modification. It is due to base images update that the main branch build failed. So it can be directly merged

Please check the failed microservice tests and make sure all of them passed.

All failed UTs are known issues #1881 (comment) and are unrelated to this PR.

@chensuyue chensuyue merged commit b1ae19c into main Aug 13, 2025
142 of 175 checks passed
@chensuyue chensuyue deleted the ze-fix/1781 branch August 13, 2025 08:45
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.

5 participants