-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
net: lib: aws_fota: Improve error handling for FOTA jobs #11995
Conversation
534908b
to
cee258e
Compare
Test specificationCI/Jenkins/NRF
CI/Jenkins/integration
Detailed information of selected test modules Note: This message is automatically posted and updated by the CI |
049c480
to
bd2739f
Compare
You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds. Note: This comment is automatically posted by the Documentation Publishing GitHub Action. |
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.
Overall looks good, few nits.
b03d3a6
to
47fa2a9
Compare
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.
Sorry about the late review, slipped through the cracks..
47fa2a9
to
3dc82f0
Compare
3dc82f0
to
8eea68c
Compare
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.
LGTM 🐶
doc/nrf/releases_and_maturity/releases/release-notes-changelog.rst
Outdated
Show resolved
Hide resolved
Improves error handling by providing more fine grained errors on job parsing. Jobs are now marked as failed if document is invalid but the job id could be parsed. A new error state is introduced to enable retries for marking jobs as failed. Additionally, the job update accepted/rejected topics are now longer subscribed to, as this is not necessary to receive the respective messages and thus adds unnecessary overhead. Signed-off-by: Fabian Nawratil <fabian.nawratil@nordicsemi.no>
8eea68c
to
e94d736
Compare
Improves error handling by providing more fine grained errors on job parsing.
Jobs are now marked as failed if document is invalid but the job id could be parsed.
A new error state is introduced to enableretries for marking jobs as failed.
Additionally, the job update accepted/rejected topics are no longer subscribed to, as this is not necessary to receive the respective messages and thus adds unnecessary overhead.