Closed YoussB closed 4 years ago
The changes look good in overall :grin: happy to see this going forward!
One thing I couldn't figure out though:
If the latter, it might be worth getting rid of some of the extra customizations that we might not need right now (e.g., podSecurityContext, etc). Wdyt?
thanks!
@cirocosta It is currently for internal consumption, with no current intention for external use, but I didn't want to remove things that might come in handy at some point.
@cirocosta @taylorsilva I think I have everything covered now. Let me know if it looks good now or there are more comments to address?
This PR adds the first step towards making the website container ready and also k8s ready.
This PR:
docker build -t dutyfree . && docker run dutyfree -p 9090:9090
and there it will run.