lf-edge / eve-api

Repository for eve-api code
Apache License 2.0
0 stars 13 forks source link

Implement acknowledgement for the compound config #27

Closed rouming closed 1 year ago

rouming commented 1 year ago

This PR adds 'timestamp' and 'last_cmd_timestamp' fields. New fields have the following meaning (copied from the local_profile.proto):

     A list of commands requested to be executed for locally running application instances.
     A new request created for the same application should overwrite the previous entry
     with the 'timestamp' field updated. In other words, the list should contain at most
     one entry for each application instance.
     It is not required for the Local profile server to persist command requests.
     Also, it is not required for the Local profile server to stop submitting command
     requests that have been already processed by EVE. Using the `timestamp` field,
     EVE is able to determine if a given command request has been already handled or not.
     To check if the last requested command has completed, compare its timestamp with
     'last_cmd_timestamp' from `LocalAppInfo` message, submitted by EVE in the request

For the compound config message I change the order of all fields to have the sequence field to be the first. This is safe, because the structure is not in use anywhere.

rouming commented 1 year ago

Difference to the previous version:

rouming commented 1 year ago

Difference to the previous version:

rouming commented 1 year ago

Difference to the previous version: