This feature adds basic ROS1 support, to enable work in the Kubernetes using ROS. This feature deploys the ros-core container in the Pod alongside the Network Application. The additional container is deployed only then the Instance is configured with the correct ROS version and distribution.
Fixes #172
Type of change
[x] New feature (non-breaking change which adds functionality)
What has been changed?
This section lists what has been changed with this pull request. Each entry lists the type of change introduced in the following order:
Feature: New ros-core container deployed with the NetApp that uses ROS1
Feature: NetApp is configured with ROS_MASTER_URI environment variable pointing to the newly deployed container
Feature: Added basic structure to enable ROS2 communication
Fix: Changed RosDistro Enum to Struct with multiple static values. Needed for name and value parsing
How Has This Been Tested?
[x] Unit tests are passing
Checklist:
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
Description
This feature adds basic ROS1 support, to enable work in the Kubernetes using ROS. This feature deploys the ros-core container in the Pod alongside the Network Application. The additional container is deployed only then the
Instance
is configured with the correct ROS version and distribution.Fixes #172
Type of change
What has been changed?
This section lists what has been changed with this pull request. Each entry lists the type of change introduced in the following order:
ROS_MASTER_URI
environment variable pointing to the newly deployed containerRosDistro
Enum to Struct with multiple static values. Needed for name and value parsingHow Has This Been Tested?
Checklist: