opendatahub-io/data-science-pipelines-operator

[Feature Request]: Improve handling of db/ObjStore Avaialable Status+Error in dspipeline_controller.go

Closed this issue · 1 comments

gmfrasca commented

Feature description

The handling/passing of the status bool, error message, etc for dbAvailable and objStoreAvailable are rather messy. Implement some way to clean them up, be it via error, an wrapper class, etc.

Rather than pass around strings for errs that already exist, I feel like we should update these to take in _, dbAvailableErr, and then have GenerateStatus to just take these errors in as parameters rather than a bool AND string that communicate the same thing.

Describe alternatives you've considered

No response

Anything else?