Skip to content

Conversation

@C8Y-Mario
Copy link
Collaborator

Set the default log level to INFO for all logger instances
Add a new "loglevel" setting in base config that will be respected by the logger instances

@C8Y-Mario C8Y-Mario temporarily deployed to Test Pull Request April 16, 2025 14:03 — with GitHub Actions Inactive
@reubenmiller reubenmiller changed the title Feature 22 large log files feat: set default log level to INFO May 12, 2025
@C8Y-Mario C8Y-Mario temporarily deployed to Test Pull Request May 12, 2025 19:27 — with GitHub Actions Inactive
@reubenmiller reubenmiller force-pushed the feature-22-large-log-files branch from 3c3bb0f to c6c2dfd Compare May 12, 2025 19:51
@reubenmiller reubenmiller temporarily deployed to Test Pull Request May 12, 2025 19:51 — with GitHub Actions Inactive
@github-actions
Copy link

Robot Results

✅ Passed ❌ Failed ⏭️ Skipped Total Pass % ⏱️ Duration
15 0 0 15 100 0s

Passed Tests

Name ⏱️ Duration Suite
Device should support the operation c8y_ModbusConfiguration 0.000 s Device
Set values via c8y_ModbusConfiguration Operation 0.000 s Operation
Poll rate and transmit rate should be updated for the Device 0.000 s Operation
Poll rate and transmit rate should be updated on the Device 0.000 s Operation
Device should support the operation c8y_ModbusDevice 0.000 s Device
Device should have installed software tedge-modbus-plugin 0.000 s Debian
Service should be active 0.000 s Debian
ReInstall Modbus Plugin 0.000 s Debian
Device should have the fragment c8y_ModbusConfiguration 0.000 s Device
ChildDevice TestCase1 should be created 0.000 s Device
ChildDevice TestCase1 should have the fragment c8y_ModbusDevice 0.000 s Device
Service tedge-modbus-plugin should be enabled 0.000 s Device
ChildDevice TestCase1 should have a Test.Int16 Measurement 0.000 s Telemetry
ChildDevice TestCase1 should have a Test.Float32 Measurement 0.000 s Telemetry
ChildDevice TestCase1 should have Alarms of type TestAlarm on Coil Value 1 0.000 s Telemetry

Copy link
Contributor

@reubenmiller reubenmiller left a comment

Choose a reason for hiding this comment

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

Approved. I pushed the required formatting changes and had to rebase the PR after pushing some PRs to update the github Runner image used to run the system tests.

@reubenmiller reubenmiller added this pull request to the merge queue May 12, 2025
Merged via the queue into thin-edge:main with commit 955d0dd May 12, 2025
5 checks passed
@reubenmiller reubenmiller linked an issue May 12, 2025 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Large logs caused by hardcoded debug logging

2 participants