Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IDD 5.0 review discussion - Orchestration-pull #83

Open
AlexChiquito opened this issue Feb 19, 2024 · 3 comments
Open

IDD 5.0 review discussion - Orchestration-pull #83

AlexChiquito opened this issue Feb 19, 2024 · 3 comments
Labels
5.0 Core Specification The issue concerns fundamental Arrowhead specifications or documentation Core System: Orchestrator The issue concerns the Core Orchestrator system

Comments

@AlexChiquito
Copy link
Contributor

n this Issue we will collect the comments about the orchestration-pull-http-json interface definition.

@emanuelpalm emanuelpalm added 5.0 Core System: Orchestrator The issue concerns the Core Orchestrator system Core Specification The issue concerns fundamental Arrowhead specifications or documentation labels Feb 19, 2024
@emanuelpalm
Copy link
Contributor

Link to reviewed document: eu.arrowhead.orchestration-pull-http-json.yml.

@borditamas
Copy link
Member

AITIA review comments

  • File name should be 'service-orchestration'
  • In case of orchestration there is an inconsistency between the IDDs and the SysD. In the SysD there is one orchestration service with three operations. One for pull and two (subscribe, unsubscribe) for the push orchestration. In the the IDDs, the pull and push are different services. During the previous roadmap discussions it was decided to have only one orchestration service.
  • This IDD should state that only describes the store orchestration strategy.

"echo" operation should not be part of this service, but a different "monitor" service.


The phrase "recommendation" shouldn't be used in the operation. It is up to the developer of the consumer system that it is considering the orchestration results as a recomenndation or not.


  • We don't understand the persence of the auth-tokens in the orchestration request. This orchestration service should never use tokens (and if uses, it should be interface independent and should be placed in the header in case of HTTP).

  • The input payload is way too simple-store strategy specific. It should cover all the three strategies we have discussed about: simple-store, flexible-store, dynamic. Also, the output payload should cover the case when the consumer gets all the access info as well (not just service instance ids).

@borditamas
Copy link
Member

@AlexChiquito @emanuelpalm @PerOlofsson-Sinetiq
Could you please provide Sinetiq's feedback before the next RoadMap (05.02) in order to being able to discuss it there?
As you know, last time the 14th of May (before AIMS 5.0 GA) was agreed to target the specification being finalized, so we don't have so much time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5.0 Core Specification The issue concerns fundamental Arrowhead specifications or documentation Core System: Orchestrator The issue concerns the Core Orchestrator system
Projects
None yet
Development

No branches or pull requests

3 participants