fides icon indicating copy to clipboard operation
fides copied to clipboard

Add a CLI command to tell users when they haven't annotated something

Open ThomasLaPiana opened this issue 3 years ago • 3 comments

Is your feature request related to a specific problem?

I can not add data categories or qualifiers to a dataset and nothing yells at me/warns me

Describe the solution you'd like

We should add stricter checking possibly by default or via flag to one of our commands...potentially fides parse --strict ?

Describe alternatives you've considered, if any

Adding an entirely separate command feels like too much, maybe we would only add it to the evaluation flow?

Additional context

This would make annotating a huge dataset file so much easier...being able to pinpoint exactly what needs annotation and not letting potentially dangerous use-cases slip through

ThomasLaPiana avatar Oct 03 '22 02:10 ThomasLaPiana

@mfbrown @NevilleS any opinions here from a product standpoint? On multiple occasions we've noticed that the lack of strict checking, specifically at parse/push time has come to bite us, and will therefore surely bite our customers

I'm thinking making fides parse do this by default is also a viable alternative to fides parse --strict, and instead give users the escape hatch of fides parse --no-strict or something

ThomasLaPiana avatar Oct 03 '22 02:10 ThomasLaPiana

Would this fit better as a Q4FidesCls Enhancements @mfbrown ?

rsilvery avatar Oct 11 '22 17:10 rsilvery

I don't think this is fidescls related specifically, it affects fides as well

ThomasLaPiana avatar Oct 12 '22 00:10 ThomasLaPiana

@ThomasLaPiana , @seanpreston can you help me understand this function? How does this compare to our UI implementation?

rsilvery avatar Nov 28 '22 14:11 rsilvery

@ThomasLaPiana we're rethinking this whole flow right now. We're going to bench this ticket in the meanwhile unless you think it's urgent right now.

rsilvery avatar Nov 28 '22 19:11 rsilvery

@rsilvery this is related to the CLI-flow, not the UI flow (if that was in question)

What was happening here is that I was working on annotating a dataset, and it would parse without issue, but then it would false the fides scan command because I had left out data categories. I wish I could have known without running the scan command that I was missing certain fields, hence the potential for some kind of --strict command which we use elsewhere sometimes

ThomasLaPiana avatar Nov 29 '22 09:11 ThomasLaPiana