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

Segfault when using namespaces #76

Open
norro opened this issue Apr 16, 2021 · 1 comment
Open

Segfault when using namespaces #76

norro opened this issue Apr 16, 2021 · 1 comment
Assignees
@norro
Labels
bug Something isn't working

Comments

@norro
Copy link
Collaborator

norro commented Apr 16, 2021

Mode manager raises a segfault when working with nodes that live in namespaces. The error happened when trying to prepend a namespace (e.g., in the example for one of the nodes) and prepending it to the node definition in the SMH file.
@norro norro added the bug Something isn't working label Apr 16, 2021
@norro norro self-assigned this Apr 16, 2021
@norro
Copy link
Collaborator Author

norro commented Apr 16, 2021

The problem is not directly related to namespaces. Namespaces not correctly declared for every instance of the particular part in the YAML file lead to a faulty vector access.
If namespaces are properly and consistently declared in the SMH yaml file, no segfault occurs.

norro added a commit that referenced this issue Apr 19, 2021
@norro
Test namespacing
25ab302 
Adapts the two_lifecycle_nodes launch test to have one node in a
separate namespace.

#76

Signed-off-by: Nordmann Arne (CR/ADT3) <[email protected]>
norro added a commit that referenced this issue Apr 21, 2021
@norro
Test namespacing
8b5a487 
Adapts the two_lifecycle_nodes launch test to have one node in a
separate namespace.

#76

Signed-off-by: Nordmann Arne (CR/ADT3) <[email protected]>
norro added a commit that referenced this issue May 10, 2021
@norro
Handle namespaces (#77)
241fcb4 
* Adapts the two_lifecycle_nodes launch test to have one node in a separate namespace.
* Adapts to new rolling declare_parameter API
* Minor fix for flake

#76

Signed-off-by: Nordmann Arne (CR/ADT3) <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@norro norro

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@norro