autogen icon indicating copy to clipboard operation
autogen copied to clipboard

removed alpine image

Open Yanni8 opened this issue 1 year ago • 2 comments

Autogen currently uses the alpine Python Docker image to execute code.

This can result in unexpected behaviors: https://pythonspeed.com/articles/alpine-docker-python/

I think that it would make sense to replace alpine with slim

Yanni8 avatar Jan 16 '24 08:01 Yanni8

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (adbfb20) 32.08% compared to head (b532019) 41.96%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1279      +/-   ##
==========================================
+ Coverage   32.08%   41.96%   +9.87%     
==========================================
  Files          32       32              
  Lines        4394     4394              
  Branches     1025     1080      +55     
==========================================
+ Hits         1410     1844     +434     
+ Misses       2867     2401     -466     
- Partials      117      149      +32     
Flag Coverage Δ
unittests 41.89% <ø> (+9.85%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov-commenter avatar Jan 16 '24 13:01 codecov-commenter

@sonichi any specific reason why we choose to put alpine at the very beginning? If not, I believe this suggested change is reasonable.

qingyun-wu avatar Jan 16 '24 13:01 qingyun-wu