fix(olm): don't annotate namespaces on operator creation

[ecordell]

instead let the standard apimachinary take care of calling syncNamespace

There's no need for it, and it fixes OLM startup issues we were seeing on OpenShift 4

[ecordell]

/approve

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ecordell

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [ecordell]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[wking]

There's no need for it, and it fixes OLM startup issues we were seeing on OpenShift 4

Is this fixing:

time="2018-10-16T21:43:30Z" level=info msg="Using in-cluster kube client config"
time="2018-10-16T21:43:40Z" level=fatal msg="error configuring operator: namespaces \"openshift-image-registry\" is forbidden: caches not synchronized"

for the olm-operator pod and:

Error: failed to create listener: failed to listen on 0.0.0.0:443: listen tcp 0.0.0.0:443: bind: permission denied

for the package-server pod?

[njhale]

@wking This looks like a fix for olm-operator. This other PR looks like the fix for package-server.

[njhale]

/lgtm

[wking]

Looks like it needs a rebase.

[njhale]

/lgtm