Skip to content

Let's try to slim the image (public: 1.1GB -> 734MB)#109

Open
RaphaelIT7 wants to merge 2 commits intoCFC-Servers:mainfrom
RaphaelIT7:patch-20
Open

Let's try to slim the image (public: 1.1GB -> 734MB)#109
RaphaelIT7 wants to merge 2 commits intoCFC-Servers:mainfrom
RaphaelIT7:patch-20

Conversation

@RaphaelIT7
Copy link
Copy Markdown
Collaborator

@RaphaelIT7 RaphaelIT7 commented Mar 6, 2026

Did funnies, all tests seem to pass (GLuaTest & gmod_tests) / no new issues seem to appear
Could probably try to push it further by unpacking the vpk files and merging all into one content directory to avoid duplicate files (iirc VPK files love to have duplicate content)

Depends on #108 as without the build still fails due to steamcmd

@sarahsturgeon
Copy link
Copy Markdown
Member

I worry about removing content packs - makes everything in them unable to be used for tests

@RaphaelIT7
Copy link
Copy Markdown
Collaborator Author

Docker slim already removes a lot xd
all content_hl2_*, content_cstrike_*, hl2_textures_*hl2_sound_vo_english_*hl2_sound_misc_000hl2_sound_misc_002 are all already being nuked soo just finishing that a bit for an benefit doens't seem like it would hurt much at that point

Most content either seems to be in hl2_misc_00x.vpk or garrysmod_00x.vpk
The content pack content_hl2_005hl2_sound_misc_001 don't seem to contain any content that is used by any tests / I saw no errors related to missing things

Also removing all the _dir.vpk's of dead packs like content_cstrike_dir.vpk and such solves all the NULL FileHandle errors, as it won't try to read imaginary vpk files anymore

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.

2 participants