Disclaimer: I'm not familiar enouch with the ClearML codebase to vouch for the quality of this PR, although it is short which is typically good . The feature we're interested in is the ability to specify the subnet_id
.
they do not need to be automated. In general, I would expect testing with the default subnet (i.e. vanilla AWS configuration) when the subnet is not provided, and with a different subnet
Hi @<1541954607595393024:profile|BattyCrocodile47> the main issue is indeed testing, it will probably take us some time to verify this. If you can provide some more testing results in various scenarios it might be good enough to justify a merge 🙂
I may be able to prepare a PR that only allows specifying the subnet ID. Can you help me brainstorm scenarios you’d want to see tested? Also, do these need to be automated tests?