Allow for dashes in clusterstack names.

[garloff]

We do this because the mere existence of a
openstack-hosted-control-plane-1-34-v0-sha-0ewkztd tag in the registry confused csctl and prevented us from releasing a new cluster stack.

So, we tolerate dashes in clusterstack names now by being more robust in parsing, So, if we don't have '-v' at the right position in the string spliited by '-', we look for a '-v' further to the right. We then assume that the extra dashes belong to the name and process accordingly.

The string parsing code is not very beautiful, but has been tested successfully.

This fixes #220.

[janiskemper]

you would have to change this in the CSO... The vendor directory imports only the package from the CSO.

[garloff]

you would have to change this in the CSO... The vendor directory imports only the package from the CSO.

Sure, that's where the fix belongs. But is it wanted? Then I'm happy to submit it to the CSO ...

[janiskemper]

The libraries from CSO are a dependency of csctl and the vendor directory is auto-generated. Your changes would be silently dropped when there is the next dependency upgrade.

So there are essentially two options:

• We can add another library based on the CSO's with your changes to this repo
• We add the change to the CSO directly.

From my point of view, it would be not too bad to do something, because the error message should at least be really helpful to understand the problem.