search

estuary / flow

🌊 Continuously synchronize the systems where your data lives, to the systems where you _want_ it to live, with Estuary Flow. 🌊
https://estuary.dev
Other
530 stars 45 forks source link

crates/flowctl: improve handling for unauthenticated clients (for CI) #1495

Closed jgraettinger closed 1 week ago

jgraettinger commented 1 week ago

Ensure we're surfacing an error from LiveCatalog resolution. Remove LiveCatalog as a result from local_specs::validate() because no callers actually care about it.

Extend Client to track whether it's authenticated or not, and then update the flowctl Resolver to no-op resolutions for unauthenticated clients instead of returning an error.

Workflow steps:

(How does one use this feature, and how has it changed)

Documentation links affected:

(list any documentation links that you created, or existing ones that you've identified as needing updates, along with a brief description)

Notes for reviewers:

(anything that might help someone review this PR)

This change is Reviewable

jgraettinger commented 1 week ago

I'm not aware of any. I've confirmed catalog tests continue to pass with this change when authenticated.