-
Notifications
You must be signed in to change notification settings - Fork 240
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
Add unit test for SequentialReader when metadata file does not exist #254
Add unit test for SequentialReader when metadata file does not exist #254
Conversation
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
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.
This is great. Thanks for adding this extra test which hopefully should allow us to avoid these kind of bugs more easily in the future.
Please run CI for this together with the rosbag2_bag_v2
repo.
@ros2/aws-oncall - please run this CI job |
Edit: |
I updated the gist (see above) to include the @ros2/aws-oncall - please run this CI job |
I made a mistake, CI needs to run with packages-up-to |
You'd need to start a packaging job to test the |
e2dc4bb
to
9a84b85
Compare
@thomas-moulard do we have the ability to do packaging job launches? |
…ile doesn't exist Signed-off-by: Zachary Michaels <zmichaels11@gmail.com>
Signed-off-by: Zachary Michaels <zmichaels11@gmail.com>
Signed-off-by: Zachary Michaels <zmichaels11@gmail.com>
9a84b85
to
ab11443
Compare
The changes to @ros2/aws-oncall - please run this CI job |
Changes
PassResolved in [compression] Move compression artifacts from rosbag2_cpp to rosbag2_compression #257storage_options.storage_id
toopen_storage
when metadata file doesn't existAdd line break inResolved in Fix uncrustify warnings #256compression_options
since linters were complaining about line length.rosbag_cpp
checking ifstorage_id
is resolved whenmetadata_file
does not exist.Issues
Signed-off-by: Zachary Michaels zmichaels11@gmail.com