asset_tracker_v2: Increase memfault event storage buffer size #8743
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
balaji-srin
requested review from
simensrostad,
jtguggedal,
jhn-nordic and
coderbyheart
as code owners
github-actions
bot
added
the
changelog-entry-required
balaji-srin
force-pushed
the
increase_evt_storage_size
branch
from
8ec37bb to
1e9bf6e
Compare
github-actions
bot
added
doc-required
Increase the memfault event storage buffer size so that all metrics generated by the application can be stored and forwarded to memfault cloud. The default value (1024) results in memfault event storage becoming full every 15 minutes. Tested by running the application for over 90 minutes and found that the error "mflt: Event storage full" is no longer printed. CIA-769 Signed-off-by: Balaji Srinivasan <balaji.srinivasan@nordicsemi.no> Signed-off-by: Simen S. Røstad <simen.rostad@nordicsemi.no>
balaji-srin
force-pushed
the
increase_evt_storage_size
branch
from
1e9bf6e to
13dc32d
Compare
jhn-nordic
approved these changes
Sep 14, 2022
divipillai
approved these changes
Sep 14, 2022
| @@ -206,6 +206,7 @@ nRF9160: Asset Tracker v2 | |||
|
|
|||
| * An issue that reports GNSS altitude, accuracy, and speed incorrectly when using LwM2M engine. | |||
| * An issue that caused modem FOTA jobs to be reported as not validated to nRF Cloud. | |||
| * An issue that caused the memfault event storage buffer getting full which in turn resulted in lost memfault events. | |||
There was a problem hiding this comment.
Suggested change
| * An issue that caused the memfault event storage buffer getting full which in turn resulted in lost memfault events. | |
| * An issue that caused the memfault event storage buffer to get full, which in turn resulted in lost memfault events. |
@greg-fer : FYI. A change log entry for 2.1.0 release.
There was a problem hiding this comment.
Fixed in the release notes and added there.
|
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. |
14 tasks
gregersrygg
approved these changes
Sep 14, 2022
Integration test specification
Detailed information of selected test modules Note: This message is automatically posted and updated by the CI |
maxd-nordic
approved these changes
Sep 14, 2022
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.
Increase the memfault event storage buffer size so that all metrics generated by the application can be stored and forwarded to memfault cloud. The default value (1024) results in memfault event storage becoming full every 15 minutes.
Tested by running the application for over 90 minutes and found that the error "mflt: Event storage full" is no longer printed.
CIA-769
Signed-off-by: Balaji Srinivasan balaji.srinivasan@nordicsemi.no
Signed-off-by: Simen S. Røstad simen.rostad@nordicsemi.no