GitLab

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

- add doxygen style comment blocks to CoapServer methods
- rename getContext() method into get_context() to stay consistent with the rest of the code

Signed-off-by: Francesco Pham <francesco.pham@huawei.com> 

- if the assigned address is ipv6 then use the IPV6 coap multicast address otherwise use the IPV4 multicast address  
- use proper localhost ipv4/ipv6 address 
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

- add is_ipv6 method that checks if the assigned ip is an ipv6 address
- if the assigned ip is ipv6 then server listens to address "::" otherwise address "0.0.0.0"
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

add support to ipv4 by checking if CONFIG_NET_IPV4 is defined, 
and use AF_INET/sockaddr_in or AF_INET6/sockaddr_in6 
accordingly.
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

Using sockaddr_storage instead of sockaddr_in6 to add support to ipv4 addresses
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

Francesco Pham authored Sep 19, 2022 and Stefan Schmidt committed Sep 26, 2022

Add ci job in deploy stage to build Doxygen docs and upload html artifacts into docs.svc.ostc-eu.dev
The job runs only in protected branches

Artifacts are uploaded to a remote ssh server using 
ssh key stored in $SSH_PRIVATE_KEY variable 
and the passphrase if exists is stored in $SSH_PASSPHRASE 

Adding ssh and doxygen dependencies in Dockerfile
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

Francesco Pham authored Sep 07, 2022 and Stefan Schmidt committed Sep 26, 2022

- There are problems when running the server in a separate thread on arduino nano 33 ble, while it works on qemu_x86:
```
<err> os:   Stacking error (context area might be not valid)
<err> os:   Data Access Violation
```
Therefore for now we are running the server thread in the main thread in blocking mode.

- CoapSite::set_sock is now static and move set_sock call from start_server into CoapServer constructor
- Add check if no resource directory is found in the network then stop execution of the program.
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

- Add EDDIE structure section
- add build prerequisites section
- add reference to doxygen docs
Signed-off-by: Jacopo Zorzetto <jacopo.zorzetto@mail.polimi.it>
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>
Co-authored-by: Jacopo Zorzetto <jacopo.zorzetto@mail.polimi.it>

Fixing eddie-zephyr job not running in MR pipelines 
when .gitlab-ci.yml file is not changed.
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

adding cmake target to build Doxygen documentations.
Doxygen docs are built together with the application by default if Doxygen is installed in the system, and can be disabled by disabling BUILD_DOC option. 
The resulting html pages are generated in the build/html directory.
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

Francesco Pham authored Sep 06, 2022 and Stefan Schmidt committed Sep 07, 2022

struct message_t has a new src_port field so setting it when receiving a response.
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

Francesco Pham authored Sep 01, 2022 and Stefan Schmidt committed Sep 07, 2022

link-local addresses require interface name to be specified.
For example a correct url with link-local ip would be the following: `coap://[fe80::c63:43ed:a8ef:2ab3%wlx1cbfcedf85d7]:5683`
We are using getnameinfo to get the correct interface name from the address. 
As a consequence, url query must be percent-encoded so that the base address is correctly decoded
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

Francesco Pham authored Sep 01, 2022 and Stefan Schmidt committed Sep 07, 2022

- get_local_node_ip returns the first ip found instead of looping until the end
- do not strip the interface name from the host name because it is useful when the ip is a link-local address which needs the interface name to be specified

Signed-off-by: Francesco Pham <francesco.pham@huawei.com> 

- .ci: add eddie-zephyr job that builds eddie for a zephyr target 
- include `zephyr.yml` file from remote [repo](https://gitlab.com/zygoon/zephyr-ci-pipelines

) containing .zephyr job
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

- move resource directory object into EddieEndpoint, including methods that use the resource directory (ip/port getters, discover_rd, publish/unpublish)
- access the resource directory using only coap requests even if the resource directory is local, without having to use direct method calls when the rd is in the same node.
- remove remove_endpoint and add_endpoint from ResourceDirectory because not necessary anymore
- move endpoints_to_string_list_filtered and rd_resources_to_string_list_filtered into private in ResourceDirectory
- adapt tests, eddie-endpoint, VirtualizationReceiver  to the new changes
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

- add virtualization_tests.cpp containing virtualization tests. For now there is just one test copied from eddie-virt-client 
- add targets in linux/tests/CMakeLists.txt
- fixture now starts `eddie-virt-server` instead of `resource-directory`
- fixture starts `eddie-virt-server` with 3 mock example resources therefore adapting the communication tests
- fixture-start.sh sleep time increased to 5 because timeout time is 5 seconds at initialization when the node searches for a non existent resource directory
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

dbus is required for the virtualization layer
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

- add --exampleres flag to add mock resources to the node
- remove unused node parameter from VirtualizationReceiver::run
- move mock_comm_setup from VirtualizationReceiver to virt_server
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

- VirtualizationSender: fix return of destroyed string by returning std::string instead of char*
- VirtualizationReceiver: remove check if preferred_resource_type exists
- parse program argument and forward ip and port parameter to EddieEndpoint
- remove unused stuff
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

every time a new coap session is created the token generated are restarted from 0 causing the same token to be reused in different coap request, this gives rise to problem in matching responses with the correct requests. FIxing by initializing session token with the last generated token.
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

- replacing the eddie blueprint repo with the correct url on EF.
- eddie-server and eddie-client executable are no longer available
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

- Adding doxyfile that can be used to generate Doxygen documentation running `doxygen Doxyfile` inside docs/ directory.
- .gitignore: ignore generated doxygen pages
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

- Add comment blocks to each public method in each class using doxygen style documentation, describing each method, their parameters and return values.
- CoapServer: move get_quit and set_quit into private
- CoapSite: return resources in link format as a string instead of a pointer into a buffer
- CoapClient: rename max_timeout to timeout to keep in sync with the linux code
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

- Add comment blocks to each public method in each class using doxygen style documentation, describing each method, their parameters and return values.
- CoapServer: move get_quit and set_quit into private
- ResourceDirectory: move get_quit and set_quit into private
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

remove multicast_messages and treat multicast messages same as unicast messages by accepting the first response to the multicast request. For now this is enough for our purposes.
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

- documenting the code using doxygen comment block
- send_message receive request_t parameter instead of the whole list of parameter
- moving some methods into private because not used publicly
- remove unused get_context method
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

This reverts commit ef15ba9d.

This has been changed to the IPv4 localhost address because  IPv6 was
not enabled on our docker daemon running the Gitlab CI runners. This has
been fixed now
(#14

), which
means we can switch back to IPv6.
Signed-off-by: Stefan Schmidt <stefan.schmidt@huawei.com>

This Dockerfile is used to build the image used to run the EDDIE CI.
Keeping it here as a reference and to extend it over time when updating
the image on the registry.
Signed-off-by: Stefan Schmidt <stefan.schmidt@huawei.com>

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

- because the resource directory infer the port number from the resource publish request, we use the same socket used by the server in CoapClient.  
- adding a CoapClient constructor in case we want CoapClient to initialize a new socket instead of using a preinitialized one (i.e. the previous case)
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

- move server thread into CoapServer
- Zephyr: run server thread in CoapServer instead of main
- add start_server and stop_server method in CoapServer
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

- start server in a thread and allow non blocking server thread spawn, stop the thread using stop_server method.
- CoapServer quit is now bool instead of int
- CoapClient: returns GATEWAY_TIMEOUT in case of request timeout
- add test that send post request to discovered resource and then a get request to check resource status
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

- set default values to request_t
- fixture starts resource-directory executable instead of eddie-endpoint now: this allows to skip waiting 5 seconds for eddie-endpoint to set up
- add tests for client request send and receive response
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>

- run server on a separate thread so that it does not block and also this allows the node to answer its own request
Signed-off-by: Francesco Pham <francesco.pham@huawei.com>