Also update package.xmls.

Signed-off-by: Kyle Fazzari <kyle@canonical.com>

Create a new public `sros2.keystore` package exposing the following
public functions:

- create_keystore()
- is_valid_keystore()
- create_enclave()
- get_enclaves()
- create_permission()

Resolve #237

Signed-off-by: Kyle Fazzari <kyle@canonical.com>

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

Signed-off-by: Jose Luis Rivero <jrivero@osrfoundation.org>

NodeStrategy does not look for an argument named 'use_daemon'.
Instead it checks for an argument named 'no_daemon'.
https://github.com/ros2/ros2cli/blob/95bfb056df795b6eef8bb73b165db31fcf2d867a/ros2cli/ros2cli/node/strategy.py#L26



Signed-off-by: Jacob Perron <jacob@openrobotics.org>

* Remove the use of pkg_resources.

Importing pkg_resources is slow; really slow.  pkg_resources
is also somewhat deprecated.  For both of these reaons, we
would prefer to move away from pkg_resources and start using
importlib instead.

sros2 was using 'resource_filename' from pkg_resources.  The
equivalent in importlib is importlib.resources, but there is a catch.
As of Python 3.8, importlib.resources does not support getting
data from subdirectories.  The workaround is to make each
level of the directory structure a python module, so that the
module name can be looked up.  Python 3.9 will have support for
directories for resources, but that won't help on Ubuntu Focal.
More information is in https://gitlab.com/python-devs/importlib_resources/issues/58



Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>

* Make use of ros_testing to test policy generation.

Signed-off-by: Brian Ezequiel Marchi <brian.marchi65@gmail.com>
Co-authored-by: Michel Hidalgo <michel@ekumenlabs.com>

* Skip test_generate_policy.py tests on Windows.

Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>

* Revert "Skip test_generate_policy.py tests on Windows."

This reverts commit c68f0a0a68d51c0cb74cdec57f5dfc5dcc0295d3.

Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>

* Wait for expected ROS graph state.

Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>

* Boost discovery delay.

Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>

* Please linter.

Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>

* Skip tests using RTI Connext w/o a daemon.

Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>

* Bump generate_policy execution timeout on Windows.

Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>

* Bump generate_policy execution timeout on Windows again.

Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>

Co-authored-by: Brian Ezequiel Marchi <brian.marchi65@gmail.com>

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

* print nested keys with full name
* update tests accordingly
* use as_posix() to print fully qualified enclave names

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

* use nightly image including all 3 RMWs

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

* add comment about Connext security plugins not being included in docker images

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

* start valid date a day before to account for timezone mismatch

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

* add comment to justify the hack

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

* make load_cert a utility and reuse it everywhere

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

* use same validity period in permissions and cert

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

* update validity_dates in tests to match new defaults

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

Signed-off-by: ruffsl <roxfoxpox@gmail.com>
Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>
Co-authored-by: Mikael Arguedas <mikael.arguedas@gmail.com>

* update basic tutorials for foxy

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

* remove spurious line change

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

Signed-off-by: ruffsl <roxfoxpox@gmail.com>

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

Extract policy generation API from api's __init__.py and move into
verb as nothing else uses it.

Signed-off-by: Kyle Fazzari <kyle@canonical.com>

Proof that we're still completely missing test coverage of some verbs.
Add a basic `create_permissions` test as well.

Signed-off-by: Kyle Fazzari <kyle@canonical.com>

api: reorganize artifact generation API

Merge branch 'feature/api-clean-artifact-generation-move' into feature/api-clean-artifact-generation

Signed-off-by: Kyle Fazzari <kyle@canonical.com>

Signed-off-by: Kyle Fazzari <kyle@canonical.com>

Signed-off-by: Kyle Fazzari <kyle@canonical.com>

api: reorganize key API

Signed-off-by: Kyle Fazzari <kyle@canonical.com>

Signed-off-by: Kyle Fazzari <kyle@canonical.com>