Faulty naming convention for Experiments with collision handling

[khhirani]

We currently use dot-convention to handle collisions in event that there are multiple Experiments with same name and Spec. This is an issue because the name is not DNS compatible and causes problems in the AWS console.

Here is the problematic code:

https://github.com/argoproj/argo-rollouts/blob/master/rollout/experiment.go#L208-#L236

[harikrongali]

@jessesuen can we name with - (hyphen) instead of . (dot) ? for collision extension?
As service uses experiment name and kubernetes doesn't allow dots in name

[jessesuen]

Yes that makes sense.