-
-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bump pydantic #646
base: master
Are you sure you want to change the base?
Bump pydantic #646
Conversation
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
@pritamghanghas since you originally worked on this(#404 #420 ), could you take a look? Do we need to bump the diag schema and the relevant python files to include all of the new information available in the diagnostics JSON? Or this is not necessary? |
I don't think anyone is using it. This was to upload the data to gcp bucket that triggers a function, that puts it in big query. I would say just retire it and save billing. |
@KevinWassermann94 would know more if anyone is using the big query data for analysis. We have to keep in mind if more freeloaders join our fleets this will be unnecessary cost bump. |
may be we should just change perform_hw_diagnostics(ship=True) to perform_hw_diagnostics(ship=False) in app.py and be done with it. |
@pritamghanghas Kevin says it's costing next to nothing and we do sometimes use it, albeit infrequently. Do I need to modify it to actually provide all the data that's in the json |
Closes: #644
Issue
How
Screenshots
References
Checklist