Skip to content
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

Small fixes on Network Dashboard cloud function code #1218

Merged
merged 6 commits into from
Mar 12, 2023

Conversation

simonebruzzechesse
Copy link
Collaborator

@simonebruzzechesse simonebruzzechesse commented Mar 7, 2023

decreased severity of log in discover cai from INFO to DEBUG
added continue in case quota is missing for asset, otherwise runtime exception will raise for splitting self_link (None) value on line 49

decresed severity of log in discover cai from INFO to DEBUG
@ludoo ludoo marked this pull request as ready for review March 8, 2023 17:07
Copy link
Collaborator

@ludoo ludoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!

ludoo

This comment was marked as duplicate.

@ludoo ludoo enabled auto-merge (squash) March 12, 2023 09:33
@ludoo ludoo merged commit 6aa0fde into master Mar 12, 2023
@ludoo ludoo deleted the bruzz/network-dashboard-fixes branch March 12, 2023 09:53
lcaggio pushed a commit that referenced this pull request May 5, 2023
* small fix on discovery compute quota file
decresed severity of log in discover cai from INFO to DEBUG

* remove else statement in condition

* add flag for debug logging

---------

Co-authored-by: Ludo <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants