Change default remote loglevel #1044
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
For Eden tests it's easier to check the logs sent to the controller than to check the logs on the EVE. This change sets the default remote log level to the same value as the default log level to make all logs produced on the device also available on the controller. Signed-off-by: Paul Gaiduk <[email protected]>
Since the test is looking for a log message that's generated at level info we need to make sure that we set the log levels of the producing agent to info as well. After the test the original settings are restored. Signed-off-by: Paul Gaiduk <[email protected]>
europaul
force-pushed
the
change-default-remote-loglevel
branch
from
2a6141b to
d4ca35e
Compare
|
I'll merge the thing and will release new eden in a bit, once I'll add GHA Workflow to run neoeden alongside with other tests (currently just sec test) Edit: as per comment here |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR is preparing Eden tests to work with the re-enabled remote log level config parameter that's coming in lf-edge/eve#4413.