imsudiproy / auditScript

This script is create to reduce the human efforts in verifying a script on multiple distros
2 stars 0 forks source link

Update : Add Port Mapping to the docker command #10

Open SonalDeshmukh1 opened 5 months ago

SonalDeshmukh1 commented 5 months ago

Add Port Mapping to docker command when creating the container, as some packages might require it for UI Verification . Thanks

sudip-ibm commented 5 months ago

Hi @SonalDeshmukh1 , in that case we won't be deleting the containers. And if we don't delete it, we won't be able to create new container exposing same ports. @Prashant-Jagtap any suggestions how we can handle this?

Prashant-Jagtap commented 5 months ago

If we are not deleting the containers, same port number cannot be used. Maybe we can try keeping 2 new variables in the config :

port=true/false
port_no=<list of ports which can be used>     # similar approach like handled for images.

Note : Also make sure to use a default port (container port) on which the service is starting inside the container which will be same for all.