Add logs for OLM resources

[tommaso-borgato]

This PR contains modifications to add logs for OLM resources;

NOTE: the PR is in Draft because we are waiting for fabric8io/kubernetes-client#2918 to be merged;
PR fabric8io/kubernetes-client#2918 removes the following error:

com.fasterxml.jackson.databind.exc.MismatchedInputException: Cannot construct instance of 
`io.fabric8.openshift.api.model.operatorhub.v1alpha1.OperatorVersion` (although at least one Creator exists): no String- 
argument constructor/factory method to deserialize from String value ('2.1.1')
 at [Source: (BufferedInputStream); line: 1, column: 11260] (through reference chain: 
io.fabric8.openshift.api.model.operatorhub.v1alpha1.ClusterServiceVersionList["items"]->java.util.ArrayList[0]- 
io.fabric8.openshift.api.model.operatorhub.v1alpha1.ClusterServiceVersion["spec"]- 
io.fabric8.openshift.api.model.operatorhub.v1alpha1.ClusterServiceVersionSpec["version"])

Please make sure your PR meets the following requirements:

• Pull Request contains a description of the changes
• Pull Request does not include fixes for multiple issues/topics
• Code is formatted, imports ordered, code compiles and tests are passing
• Code is self-descriptive and/or documented

[simkam]

fabric8 issue: fabric8io/kubernetes-client#2784

[istraka]

A few minor issues. Otherwise as a draft looks good. I raised one point about CustomResourceHelper though.

Moreover, can you create an XTF issue for this?

[tommaso-borgato]

Moreover, can you create an XTF issue for this?

#420

@istraka ^^^

[istraka]

@tommaso-borgato can you please list requirements for the PR? What blocks us and so on. It helps us to track it efficiently.

[tommaso-borgato]

@istraka I wrote it in the description: shall I white it somewhere else?

[istraka]

@tommaso-borgato Oh, I must have missed it, I am sorry. It is okay, thanks.

[fabiobrz]

Hi @tommaso-borgato (CC @simkam and @istraka), FYI I see that fabric8io/kubernetes-client#2918 has been merged already.

Anyway I'd raise another issue here, related to this one, which is about the fact that in we noticed an IOException being thrown in case the logs have been previously downloaded. I suspect the changes here could be affected as well since they were first developed externally and then ported right here.

Again, this is just an heads-up for investigation which should be verified before merging this.