pkg/cdi: add missing error checks in cache

[marquiz]

Mostly retain the existing behavior of ignoring errors but using _ to make golangci-lint happy.

[marquiz]

I was not completely confident I'm doing the right thing here. Thus draft :) As a general note I think the very common pattern here of omitting error checking is bad design. Either check it or then don't even return an error in the first place.

[elezar]

Syntactically, I think this is fine. I'll defer to @klihub for when we should actually return the errors when refreshing the cache.

[marquiz]

Syntactically, I think this is fine. I'll defer to @klihub

Yeah, no syntax error 😇 ping @klihub

[elezar]

@marquiz this can be closed now, correct?

[marquiz]

We agreed to not break the Cache API (return an error from Configure() even though it's always nil), so I'll close this one