DRAFT AGENDA
...
Type @ and your name to indicate your attendance
Community: Jose Luis Urien Pinedo Ludovic Robert Cetin Alpaycetin Joachim Dahlgren Rafal Artych Fernando Prado Cabrillo Javier Carro
Agenda
- Approve previous meeting minutes
- Approved ??
- Open issues and PRs
Minutes
...
- Issue: Consistency problem in Location retrieval regarding lastLocationTime cardinality
- Review comments
- Decision: Fix the yaml → lastLocationTime is mandatory and documentation must be updated
- Ludovic Robert will do the PR
- Issue: Provide Test Defintion for Device Retrieval API
- We already had a PR for Add Test Definition for location Retrieval
- Done for admin purpose
- PR will be done soon by Ludovic Robert
- We already had a PR for Add Test Definition for location Retrieval
Ongoing
- Issue: [Geofencing] Support for polygon area type
- PR: Polygon area type added
- From previous meeting:
- Makes sense from a dev perspective but makes implementation more complicated
- Possibly we may need this as well for Location Verification, but not necessarily
- Align schemas with Location retrieval and Population Density in case we accept PR. In the future it would make sense to move it to Common artifacts.
- Discussion:
- Review latest comments
- Related to discussion in Define guidelines for geofencing implementation
- Decision about adding support or not, making this support mandatory or not ?
- Does the following statement could be acceptable for first meta release: "We add the polygon. Circle management is mandatory while polygon is nice to have. " ?
- We have still some time to discuss this.
- Does the following statement could be acceptable for first meta release: "We add the polygon. Circle management is mandatory while polygon is nice to have. " ?
- How to managed not supported polygon?
- We can have a 422 for synchronous response for lack of support for a given area + add a terminationEvent reason for lack of support when the area assessment is performed asynchronously.
- Issue: Align security scope with guideline
- Along with previous issue: Based on the new explicit subscriptions guideline in commonalities, change name of geofencing API to include the keyword subscriptions
- PR: update base-path & security-scope for geofencing-subscriptions → fix both PRs, the old one changing the API name and the new one adapting also event name.
- Ready for merge?
- Yes !
- Issue: [Location-Retrieval]: add
POINT
as possible Location response- Some comments on the issue. Minimum right now is a CIRCLE with radius=1m, Is it not enough?
- From previous meeting:
- Having the radius as 1 does not seem like a practical problem since even GPS often have a few meters of error margin
- From a documentation perspective clients might think that using a point the precision will be high, which is not the case
- We don't have use cases for precision lower than 1 meter
- Decision: Keep the discussion open. Using circle with 1m radius as points seems enough
- Related with location-verification:
- Javier asks if the WG is open to lower the 2km minumum of radius for CIRCLE location-verification
- There were issues open regarding this topic in the past where several problems and discussions raised
- Javier will open a new issue to continue the discussion
- Discussion:
- Discard point and rely on Circle?
- Agreement from Jose Luis Urien Pinedo Joachim Dahlgren and Ludovic Robert to consider that 1m radius will be good enough (and no need for Point)
- Should we lower minimum radius for location-verification, Javier Carro ?
- Javier Carro has still some discussion with legal team about privacy issue
- Keep this point open.
- Discard point and rely on Circle?
Test plans
- Commonalities PR: Enhance API-Testing-Guidelines.md
- Once new guidelines and consensuated, all test plans will need to be updated
- For this meta-release it is still not mandatory to have the completed test plans, but the more we have advanced, the better
...
- Subscription related issues → Dedicated workshop took place
Geofencing API - Add Subscription type 'area-left-or-entered' to subscribe to both event in one time
- Geofencing API - Defining a standard behavior for first event
- Issue in Commonalities Issue 140
- Proposal to tackle open subscription issues Common proposal to tackle subscription-based open issues. · Issue #185 · camaraproject/Commonalities (github.com) - mains points:
- Align our subscription model with CloudsEvents subscriptions one
- Introduce a
status
attribute in the template as we have fair UCs that can leverage this (Retrieve expired subscriptions for monitoring, deactivate a subscription if an user revoked her/his consent, etc...). API subproject could decide to use it or not. - Improve the model to allow consumers to subscribe to more than one event types with a single subscription but at least for the first meta-release we enforce to have only event type per subscription. After this first meta release decision to handle several event types in one subscription request should be discussed at API sub projet level
- Add
filters
and it's up to API Project to use it. We recommend to be very cautious as it add complexity so it should be keep for very relevant UC. - Add
initialEvent
inconfig
as well to manage request to get event when current situation of a device corresponds to the subscriptions type. .
- A template yaml is available for review here: Create subscription-notification-template.yaml by bigludo7 · Pull Request #189 · camaraproject/Commonalities (github.com)
...