deep-learning-containers icon indicating copy to clipboard operation
deep-learning-containers copied to clipboard

[feature-request] Add basic input validation to manual scripts used for EKS cluster setup

Open kace opened this issue 3 years ago • 0 comments

Checklist

  • [x] I've prepended issue tag with type of change: [feature]
  • [ ] (If applicable) I've documented below the DLC image/dockerfile this relates to
  • [ ] (If applicable) I've documented the tests I've run on the DLC image
  • [ ] I'm using an existing DLC image listed here: https://docs.aws.amazon.com/deep-learning-containers/latest/devguide/deep-learning-containers-images.html
  • [ ] I've built my own container based off DLC (and I've attached the code used to build my own image)

Concise Description: We could provide some basic input validation in scripts like install_cluster_components.sh. In my case, a simple mistake in specifying the autoscalar version prevented the autoscalar pod from starting and broke all EKS tests.

kace avatar Sep 01 '22 09:09 kace