search

litmuschaos / litmus-go

Apache License 2.0
69 stars 122 forks source link

UBI migration of Images - go-runner #707

Closed dusdjhyeon closed 2 months ago

dusdjhyeon commented 3 months ago

What this PR does / why we need it: Migrate the base images of the g-runner container images of LitmusChaos from experiment-alpine to ubi minimal. Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #4634 (https://github.com/litmuschaos/litmus/issues/4634)

Reference: https://github.com/litmuschaos/litmus/wiki/Chaos-Experiments-Go%E2%80%90Runner-Base-Image-Migration-to-RedHat-UBI-Image

Special notes for your reviewer:

Checklist:

Jonsy13 commented 2 months ago

One more change, we will have to do is to update in all github action workflows - 

        with:
          go-version: 1.18

to 

        with:
          go-version: 1.20

to complete our 1.20 migration.

Jonsy13 commented 2 months ago

One change missing from https://github.com/litmuschaos/litmus/wiki/Chaos-Experiments-Go%E2%80%90Runner-Base-Image-Migration-to-RedHat-UBI-Image

Experiment Code using ps will have to be changed as ps aux works a little different in case of UBI - sudo kill -9 $(ps aux | grep [t]oxiproxy | awk 'FNR==1{print $1}') has to be changed to sudo kill -9 $(ps aux | grep [t]oxiproxy | awk 'FNR==2{print $2}')

You can try searching for ps aux & update those instances.