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.

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

[Humble] move storage preset validation to sqlite3 plugin (backport #1135) #1184

Merged
merged 2 commits into from
Dec 3, 2022

Conversation

wep21
Copy link
Contributor

@wep21 wep21 commented Nov 27, 2022

Currently, it is impossible to record bag with mcap storage plugin by the follwing error at humble.

❯ ros2 bag record -a -s mcap
[INFO] [1669557755.363458962] [rosbag2_recorder]: Press SPACE for pausing/resuming
[ERROR] [1669557755.364893365] [rosbag2_storage]: Could not open 'rosbag2_2022_11_27-23_02_35/rosbag2_2022_11_27-23_02_35_0' with 'mcap'. Error: unknown MCAP storage preset profile (valid options are 'fastwrite', 'zstd_fast', 'zstd_small'): none
[ERROR] [1669557755.364913668] [rosbag2_storage]: Could not load/open plugin with storage id 'mcap'
No storage could be initialized. Abort

❯ ros2 bag record -a -s mcap --storage-preset-profile fastwrite
usage: ros2 bag record [-h] [-a] [-e REGEX] [-x EXCLUDE] [--include-unpublished-topics] [--include-hidden-topics] [-o OUTPUT]
                       [-s {mcap,my_test_plugin,sqlite3}] [-f {s,a}] [--no-discovery] [-p POLLING_INTERVAL] [-b MAX_BAG_SIZE]
                       [-d MAX_BAG_DURATION] [--max-cache-size MAX_CACHE_SIZE] [--compression-mode {none,file,message}]
                       [--compression-format {zstd,fake_comp}] [--compression-queue-size COMPRESSION_QUEUE_SIZE]
                       [--compression-threads COMPRESSION_THREADS] [--snapshot-mode] [--ignore-leaf-topics]
                       [--qos-profile-overrides-path QOS_PROFILE_OVERRIDES_PATH] [--storage-preset-profile {none,resilient}]
                       [--storage-config-file STORAGE_CONFIG_FILE] [--start-paused] [--use-sim-time]
                       [topics ...]
ros2 bag record: error: argument --storage-preset-profile: invalid choice: 'fastwrite' (choose from 'none', 'resilient')

rosbag2 version

❯ apt-cache policy ros-humble-rosbag2
ros-humble-rosbag2:
  Installed: 0.15.3-1jammy.20221109.192253
  Candidate: 0.15.3-1jammy.20221109.192253
  Version table:
 *** 0.15.3-1jammy.20221109.192253 500
        500 http://packages.ros.org/ros2/ubuntu jammy/main amd64 Packages
        500 http://packages.ros.org/ros2-testing/ubuntu jammy/main amd64 Packages
        100 /var/lib/dpkg/status

rosbag2_storage_mcap version

❯ apt-cache policy ros-humble-rosbag2-storage-mcap
ros-humble-rosbag2-storage-mcap:
  Installed: 0.5.0-1jammy.20221109.184113
  Candidate: 0.5.0-1jammy.20221109.184113
  Version table:
 *** 0.5.0-1jammy.20221109.184113 500
        500 http://packages.ros.org/ros2/ubuntu jammy/main amd64 Packages
        500 http://packages.ros.org/ros2-testing/ubuntu jammy/main amd64 Packages
        100 /var/lib/dpkg/status

@wep21 wep21 requested a review from a team as a code owner November 27, 2022 14:04
@wep21 wep21 requested review from emersonknapp and MichaelOrlov and removed request for a team November 27, 2022 14:04
@james-rms
Copy link
Contributor

@wep21 thanks for the PR! Just FYI, i've merged and bloomed ros-tooling/rosbag2_storage_mcap#86 which should also resolve the underlying issue.

Signed-off-by: James Smith <james@foxglove.dev>

Signed-off-by: James Smith <james@foxglove.dev>
Signed-off-by: Daisuke Nishimatsu <border_goldenmarket@yahoo.co.jp>
@MichaelOrlov
Copy link
Contributor

Gist: https://gist.githubusercontent.com/MichaelOrlov/19a7ca0de8eeb73481593e8d9390bee8/raw/2cc1a3297b982cd094559d007ca1a6fea2bf7331/ros2.repos
BUILD args: --packages-above-and-dependencies ros2bag rosbag2_storage_sqlite3 rosbag2_tests
TEST args: --packages-above ros2bag rosbag2_storage_sqlite3 rosbag2_tests
ROS Distro: humble
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/11186

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Windows Build Status

@MichaelOrlov MichaelOrlov changed the title [backport humble] ros2bag: move storage preset validation to sqlite3 plugin [Humble] move storage preset validation to sqlite3 plugin (backport #1135) Nov 29, 2022
@MichaelOrlov
Copy link
Contributor

It looks like some random unrelated error on CI.
Re-running CI jobs
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/11193

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Windows Build Status

@MichaelOrlov
Copy link
Contributor

It looks like some random unrelated error on CI.
Re-running CI jobs
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/11198

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Windows Build Status

@MichaelOrlov
Copy link
Contributor

Gist: https://gist.githubusercontent.com/MichaelOrlov/56384324a64241198a77f5e584fcba63/raw/2cc1a3297b982cd094559d007ca1a6fea2bf7331/ros2.repos
BUILD args: --packages-above-and-dependencies ros2bag rosbag2_storage_default_plugins rosbag2_tests
TEST args: --packages-above ros2bag rosbag2_storage_default_plugins rosbag2_tests
ROS Distro: humble
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/11206

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Windows Build Status

@MichaelOrlov MichaelOrlov merged commit e9c1015 into ros2:humble Dec 3, 2022
@wep21 wep21 deleted the humble-backport-1135 branch December 3, 2022 03:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants