Skip to content

Is logging sufficiently integration tested? #128

Description

@NebulousPsychology

In a container environment roughly that of /tests/Dockerfile_jazzy, observed a test duplicate of the logging example node failing to publish to /rosout topic.

'roughly' referring to the addition of jazzy tutorial packages, along with rust and (for rqt) desktop-lite being added as vscode devcontainer features:

RUN apt-get update && apt-get install -y
    ros-jazzy-demo-nodes-cpp \
    ros-jazzy-test-msgs \
    ros-jazzy-example-interfaces \
    ros-jazzy-turtlesim \
    ros-jazzy-rqt \
    ros-jazzy-rqt-common-plugins

Is the rcutils_log() sufficiently bound and integration tested?
Are instructions missing for a correct demonstration of the Logging example?
Do other domains require additional integration tests to validate their function?

Reproduction:

  1. r2r 0.9.5 in a ros2 Jazzy docker image
  2. run the r2r logging example in a container, equipped with rqt and similar.
    • With a copy-paste of the logging example as main.rs: cargo build; cargo run -- --ros-args --log-level DEBUG
    • Or, from a clone of r2r repo: cargo run --example logging -- --ros-args --log-level DEBUG
    • For comparison: ros2 run demo_nodes_cpp talker
  3. In a separate, attached shell session inspect the /rosout topic:
    • ros2 topic echo /rosout
    • or rqt's console ros2 run rqt_console rqt_console

Expect: messages appear in rqt and ros2 topic echo /rosout just like ros2 run demo_nodes_cpp talker
Actual: builds, runs, and appears in nodes, but messages appear in output only; rqt and topic echo do not observe traffic from "logger_node".

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions