2024-06-04 Device Location - Meeting Minutes
Date
Jun 4, 2024
Attendees & Representation
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
New
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
Discussion:ย
Done for admin purpose
PR will be done soon by @Ludovic Robertย
Ongoing
Issue:ย [Geofencing] Support for polygon area type
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.
How to managed not supported polygon?
Proposal discussed:
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.
Action: have a dedicated issue for this.
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.
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
We discussed about the use of this Test Definition by GSMA for certification which will be one of the main 'consumer' of this work.
PR: Testing plan for location verification
Used as example for Commonalities Issueย Enhancement of the Testing Guidelines, but has to be updated
Discuss licensing header
To be aligned in Commonalities as not sure we have ruling for this.
(Meeting Action) Akos will check internally โย Feedback?
PR:ย Add Test Definition for location Retrieval #119 (see New issue above)
APIs error alignment
@Rafal Artych has prepared a confluence page with Http Errors in different APIs and a comparaison with the CAMARA_common.yaml to check differences and misalignments so we can adapt the guidelines: Error response guidelines
Newย PR in Commonalities
Action Point: Review errors in our APIs after Commonalities guidelines are approved ?
Issues pending onย Commonalities outcome, review status and cleanup
Are we happy with security scope for geofencing
Is it covered by Align security scope with guidelineย discussed above? โ Probably, let's double check with @Ludovic Robertย
Last meeting:
Scope name should reflect the data provided by the notification (notย
geofencing:subscriptions:write
ย but more precise likeยgeofencing:subscriptions:read-location
ย )Discussion on this topic in progress in Commonalities (camaraproject/Commonalities#163)
Open pull request in Commonalities update-design-doc-with-explicit-sub-scope-changes by shilpa-padgaonkar ยท Pull Request #177 ยท camaraproject/Commonalities (github.com)
Proposal is to add scope per even-type additionally to scope managing the right to create a subscription.
Action: @Ludovic Robert This issue could be closed
Geofencing - Adding a value in Termination Reason value enum
Discussion in progress in Commonalities:ย camaraproject/Commonalities#153
Related issues inย DeviceStatusย and QoD
Still open, PR ongoing
Action: Once PR mergedย @Ludovic Robert will close this issue
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
ย inยconfig
ย 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)
Action: Once PR mergedย @Ludovic Robert will close this issue
Geofencing API - Add "format: uri" to notificationUrl
Issue inย commonalities
No progress in commonalities
On hold discussions
Implementation
Define guidelines for geofencing implementation (mentioned above)
New comments
Connected withย Issue #85. A document with implementation guideline should cover this also.
As requested by Joachim, issue #85 will remain open until TEF uploads the document with more detail or the information is added to an API_documentation file to keep in the repo for future references.
Joachim: Should be also aligned/synchronized with Geofencing
On issue #133 we probably need to have a base agreement for Geofencing guidelines
Akos & Josรฉ proposed some suggestion - in particular to improve the subscription to allow consumer to indicate the 'reliability' expected.
Ludovic also raised the point above UC where we are not able to send any notification.... and we need to inform the consumer.
Discussion/Action: DT team can move forward with a PR with the proposal about adding triggerSensibility
ย
Administrative Code Area
Relate also with discussion about polygons for Geofencing, as it has similar implications.
Issue #83, with formal requirements from GSMA Product track
Document uploaded by TEFย with a proposal.
Review priority and next steps โ Clarify this:
More generally probably we need to have feedback loop with GSMA team about API work (and review the priority as focus could change)
Done during last TSC - see here:ย https://lf-camaraproject.atlassian.net/wiki/display/CAM/2024-03-21+TSC+Minutes
ย
AOB