Make error handling consistent across all packages

[xmudrii]

User story

In #87 the error handling got improved for some packages. We should check other packages and make sure the error checking and handling is consistent across all packages.

We could research is it possible to overall improve logging and error handling. For ideas, we can see how other providers handle this.

Acceptance criteria

• Make error handling consistent across all packages
• Potentially, find a better way for error handling and logging

[nikhita]

Created #109 to make the error handling consistent across the project.

Re logging: we use glog currently but there has been discussion about how glog might not be a good choice for logging - kubernetes/kubernetes#61006.

Fwiw, with respects to the project moving upstream, Kubernetes itself uses glog right now and it doesn't look like it is going to move away from it soon.

[solidnerd]

Are there any updates regarding the logging topic ?

It seems that a lot of upstream projects use klog right now. This is a kubernetes fork of glog .

[xmudrii]

@solidnerd Moving to klog would be very useful and my plan is to do it once #126 is merged.

If you'd like to take on it as well, let us know. I would just recommend waiting for #126 or working based on that branch, as there are some packages that are added and removed.

[solidnerd]

@xmudrii I can take it and work from #126 as base. After the merge we can rebase it.

[xmudrii]

@solidnerd Sounds good, thanks! :)

[solidnerd]

@xmudrii what is missing here to bring this issue over the line ?

[xmudrii]

@solidnerd I think it's okay to close this issue. Thanks for the help!

/close

[k8s-ci-robot]

@xmudrii: Closing this issue.

In response to this:

@solidnerd I think it's okay to close this issue. Thanks for the help!

/close

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.