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

🧹 only log warnings and higher for providers #3097

Merged
merged 1 commit into from
Jan 23, 2024

Conversation

imilchev
Copy link
Member

Make sure we don't show info messages for the providers since that is a bit too much

Copy link
Contributor

Test Results

742 tests  ±0   742 ✅ ±0   12s ⏱️ ±0s
 83 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 7343a1e. ± Comparison against base commit 4f720e8.

This pull request removes 6 and adds 6 tests. Note that renamed tests count towards both.
go.mondoo.com/cnquery/v10/llx ‑ TestRawData_JSON/0001-01-01_00:00:00_+0000_UTC
go.mondoo.com/cnquery/v10/llx ‑ TestRawData_JSON/292277026596-12-04_15:30:07_+0000_UTC
go.mondoo.com/cnquery/v10/llx ‑ TestRawData_JSON/292277026596-12-04_15:30:09_+0000_UTC
go.mondoo.com/cnquery/v10/llx ‑ TestSuccess/2024-01-22_09:09:14.937592195_+0000_UTC_m=+0.001043342
go.mondoo.com/cnquery/v10/llx ‑ TestTruthy/2024-01-22_09:09:14.937592195_+0000_UTC_m=+0.001043342
go.mondoo.com/cnquery/v10/llx ‑ TestTruthy/2024-01-22_09:09:14.937592195_+0000_UTC_m=+0.001043342#01
go.mondoo.com/cnquery/v10/llx ‑ TestRawData_JSON/0001-01-01_00:53:28_+0053_LMT
go.mondoo.com/cnquery/v10/llx ‑ TestRawData_JSON/292277026596-12-04_16:23:37_+0053_LMT
go.mondoo.com/cnquery/v10/llx ‑ TestRawData_JSON/292277026596-12-04_16:30:07_+0100_CET
go.mondoo.com/cnquery/v10/llx ‑ TestSuccess/2024-01-22_13:18:46.364798094_+0100_CET_m=+0.001130520
go.mondoo.com/cnquery/v10/llx ‑ TestTruthy/2024-01-22_13:18:46.364798094_+0100_CET_m=+0.001130520
go.mondoo.com/cnquery/v10/llx ‑ TestTruthy/2024-01-22_13:18:46.364798094_+0100_CET_m=+0.001130520#01

Copy link
Member

@chris-rock chris-rock left a comment

Choose a reason for hiding this comment

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

Thank you @imilchev

@chris-rock chris-rock merged commit 3ef716e into main Jan 23, 2024
12 checks passed
@chris-rock chris-rock deleted the ivan/provider-logging branch January 23, 2024 09:33
@github-actions github-actions bot locked and limited conversation to collaborators Jan 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants