Skip to content
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.

Sign up for GitHub

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

Jump to bottom

sys/sc_suit: add seq_no command #17941

Merged
merged 2 commits into from Apr 17, 2022
Merged

sys/sc_suit: add seq_no command #17941

merged 2 commits into from Apr 17, 2022

Conversation

fjmolinas
Copy link
Contributor

Contribution description

Adds a command to retrieve the highest seq_no of all storage units.

Shell interactions are also added.

Testing procedure

2022-04-14 08:45:25,013 # suit
2022-04-14 08:45:25,015 # Usage: suit fetch <manifest url>
2022-04-14 08:45:25,017 #        suit seq_no
2022-04-14 08:45:30,173 # suit seq_no
2022-04-14 08:45:30,180 # seq_no: 1649918660

@fjmolinas fjmolinas added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Apr 14, 2022
@fjmolinas fjmolinas requested a review from bergzand April 14, 2022 06:46
@github-actions github-actions bot added Area: sys Area: System Area: tests Area: tests and testing framework labels Apr 14, 2022
@fjmolinas fjmolinas mentioned this pull request Apr 14, 2022
Merged
@fjmolinas fjmolinas force-pushed the pr_suit_seq_no_cmd branch 2 times, most recently from 2bc93b7 to 6347bd0 Compare April 14, 2022 11:25
benpicco
benpicco approved these changes Apr 15, 2022
View reviewed changes
Copy link
Contributor

@benpicco benpicco left a comment

Choose a reason for hiding this comment

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

Nice addition!

@benpicco benpicco merged commit 0e5900d into RIOT-OS:master Apr 17, 2022
@fjmolinas fjmolinas deleted the pr_suit_seq_no_cmd branch April 19, 2022 06:47
@fjmolinas
Copy link
Contributor Author

Thanks!

This was referenced Apr 19, 2022
Merged
Merged
@chrysn chrysn added this to the Release 2022.07 milestone Aug 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@benpicco benpicco benpicco approved these changes

@bergzand bergzand Awaiting requested review from bergzand

Assignees
No one assigned
Labels
Area: sys Area: System Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR
Projects
None yet
Milestone
Release 2022.07
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@fjmolinas @benpicco @chrysn