-
Notifications
You must be signed in to change notification settings - Fork 54
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
Replace error log levels with warnings in tedge agent #2647
Replace error log levels with warnings in tedge agent #2647
Conversation
6bf4f22
to
daa3920
Compare
Robot Results
|
Codecov ReportAttention:
Additional details and impacted files
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@albinsuresh I wonder why the test "Update tedge version from base to current using Cumulocity" is not failing as it should since the version mismatch is not reported after an update.
Signed-off-by: Krzysztof Piotrowski <krzysztof.piotrowski@inetum.com>
daa3920
to
04f2fc6
Compare
@didier-wenzek Is this comment still valid as Krzystoff seem to have reverted the problematic change? |
Somehow, yes. This test was supposed to cover the feature broken by the problematic change, but failed to raise the issue. That said this is unrelated to this PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved
@didier-wenzek That's because his changes were not included in the I understand how this can be confusing to anyone. Upgrading from the current version to a dummy higher version would have been better and clearer. We resorted to this mechanism due to the complexity of always building that dummy higher version as part of the test setup. We can re-evaluate that choice post 1.0. |
It makes sense now.
To improve the test, one alternative could be to test both upgrade from dummy 0.0.1 to current and downgrade from current to dummy 0.0.1.
Sure. |
Proposed changes
This PR replaces error log levels with warnings and prevents exiting with error code in such cases.
Types of changes
Paste Link to the issue
Checklist
cargo fmt
as mentioned in CODING_GUIDELINEScargo clippy
as mentioned in CODING_GUIDELINESFurther comments