MAISTRA-2302: Bump openshift dependency

[jwendell]

This is needed because maistra/api Status object depends
on Operator's version package. This needs to be fixed as
part of https://issues.redhat.com/browse/MAISTRA-2331.

Without this we get this go mod error:

github.com/openshift/api@v3.9.1-0.20190924102528-32369d4db2ad+incompatible: invalid pseudo-version: preceding tag (v3.9.0) not found

[maistra-bot]

@jwendell: The following test failed, say /retest to rerun all failed tests:

Test name	Commit	Details	Rerun command
istio-operator_gen-check	c947864	link	/test istio-operator_gen-check

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[rcernich]

Shouldn't we do this when we migrate to using maistra/api? I don't understand the need for this at this time. I don't think maistra/api should depend on maistra/istio-operator at all, as it will create a circular dependency between api and operator.

[jwendell]

I don't think maistra/api should depend on maistra/istio-operator at all, as it will create a circular dependency between api and operator.

Yeah, me too. This is described in MAISTRA-2331. I believe this should be fixed as part of that ticket.

[jwendell]

Ok, I've removed that circular dependency for now, we will need to move it from status API to operator code base. It's kind of a static method that's only relevant to the operator, should stay in operator codebase:

istio-operator/pkg/apis/maistra/status/status.go

Line 190 in 4ee3870

return ComposeReconciledVersion(version.Info.Version, generation)

So, we are safe to close this PR. maistra/api doesn't depend on operator anymore.