Auto-provisioned Subscriptions do not support upper-case characters in channel names

[cdjohnson]

Bug Report

What did you do?

1. Create an operator with default channel: V1.0
2. Create a second operator that depends on the first operator's API in the CSV
3. Install the second operator via a subscription.

What did you expect to see?
Both the first and second operator should be installed successfully, where the first operator is installed using the default channel.

What did you see instead? Under which circumstances?
When OLM attempts to create the second subscription in the installplan, it auto-generates a subscription name in the format: <operatorname>-<channel>-<catalogname>-<catalognamespace>

The problem is that the Channel name is NOT lower-cased, resulting in an error in the status section:
error creating subscription: myoperator-V1.0-mycatalog-openshift-marketplace: Subscription.operators.coreos.com "myoperator-V1.0-mycatalog-openshift-marketplace" is invalid: metadata.namee: Invalid value: "myoperator-V1.0-mycatalog-openshift-marketplace" a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')

Environment

• operator-lifecycle-manager version:
  0.14.2 (OpenShift 4.4.3)

• Kubernetes version information:
  Server Version: 4.4.3
  Kubernetes Version: v1.17.1

• Kubernetes cluster kind:

Possible Solution
use lower-case channels

Additional context

[awgreene]

Hello @cdjohnson, it looks like you created a Subscription where the metadata.name field, myoperator-V1.0-mycatalog-openshift-marketplace, includes an Upper Case letter, Kubernetes does not support Upper Case letters in names, example:

$ kubectl create ns Test
The Namespace "Test" is invalid: metadata.name: Invalid value: "Test": a DNS-1123 label must consist of lower case alphanumeric characters or '-', and must start and end with an alphanumeric character (e.g. 'my-name',  or '123-abc', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?')

[awgreene]

Sorry - just saw that OLM is generating the subscription that is failing. This is valid.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[jeyaramashok]

@awgreene I am interested to work on this. Can you please assign this issue to me ? , if it's not already worked on

I noticed you have created PR: #1560 previously to address this but it wasn't merged