Make error handling consistent across all packages about cluster-api-provider-digitalocean HOT 8 CLOSED

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

from cluster-api-provider-digitalocean.

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.

from cluster-api-provider-digitalocean.

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 .

from cluster-api-provider-digitalocean.

@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.

from cluster-api-provider-digitalocean.

@solidnerd Sounds good, thanks! :)

from cluster-api-provider-digitalocean.

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

from cluster-api-provider-digitalocean.

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

/close

from cluster-api-provider-digitalocean.

@xmudrii: Closing this issue.

from cluster-api-provider-digitalocean.