Skip to content

[Multi-MDS] Added Inter MDS Relationship extension section to content module#494

Merged
JavierEspina merged 15 commits intomasterfrom
48-append-toi-mdibmds-modeling-for-device-aggregators
Dec 19, 2025
Merged

[Multi-MDS] Added Inter MDS Relationship extension section to content module#494
JavierEspina merged 15 commits intomasterfrom
48-append-toi-mdibmds-modeling-for-device-aggregators

Conversation

@PeterKranich
Copy link
Copy Markdown
Collaborator

@PeterKranich PeterKranich commented Nov 17, 2025

📑 Description

  • Multi-MDS Feature Part 1: Added Inter MDS Relationship extension section to content module (this PR)
  • Multi-MDS Feature Part 2: Define Mulit-MDS scenarios and requirement (tbd)

☑ Mandatory Tasks

The following aspects have been respected by the pull request assignee and at least one reviewer:

  • Changelog update (necessity checked and entry added or not added respectively)
    • Pull Request Assignee
    • Reviewer

@PeterKranich PeterKranich linked an issue Nov 17, 2025 that may be closed by this pull request
@PeterKranich PeterKranich self-assigned this Nov 17, 2025
@PeterKranich PeterKranich added Volume 3 Volume 3 contents Multi MDS Concerns devices that support multiple MDS containment labels Nov 17, 2025
@PeterKranich PeterKranich added this to the SDPi 2.3 Review milestone Nov 17, 2025
Copy link
Copy Markdown
Collaborator

@d-gregorczyk d-gregorczyk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I changed some of the texts. I will also modify your branch and add a proper extension schema plus adaption of the GetMdibResponse.

JavierEspina and others added 5 commits December 17, 2025 10:20
Co-authored-by: David Gregorczyk <54440227+d-gregorczyk@users.noreply.github.com>
…0-extension-inter-mds-relationship.adoc

Co-authored-by: David Gregorczyk <54440227+d-gregorczyk@users.noreply.github.com>
…0-extension-inter-mds-relationship.adoc

Co-authored-by: David Gregorczyk <54440227+d-gregorczyk@users.noreply.github.com>
…0-extension-inter-mds-relationship.adoc

Co-authored-by: David Gregorczyk <54440227+d-gregorczyk@users.noreply.github.com>
…0-extension-inter-mds-relationship.adoc

Co-authored-by: David Gregorczyk <54440227+d-gregorczyk@users.noreply.github.com>
Copy link
Copy Markdown
Collaborator

@JavierEspina JavierEspina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do not merge till Stefan feels confident with this PR that he has inherited from Peter.

@JavierEspina
Copy link
Copy Markdown
Collaborator

JavierEspina commented Dec 17, 2025

@PaulMartinsen , there are two requirements defined by this PR (R0613 and R0614). Do they look fine in terms of RI "tagging"?

With the many changes you introduced lately (mostly with PR·469) I am often unsure if we are missing to use any important part of the new RI "tagging" functionality...

@JavierEspina
Copy link
Copy Markdown
Collaborator

JavierEspina commented Dec 17, 2025

@d-gregorczyk , @stefandreher , I have noticed that the Relations extension defined here uses the AbstractMetricDescriptor/Relation defined in BICEPS (and slightly amended in the BICEPS Corrigendum that will be published VERY VERY soon). The usage scope in BICEPS is "represents a relationship between a METRIC and one or more other CONTAINMENT TREE ENTRIEs". However, here it is used to represent relationships between MDSs.

  1. Is that stretched usage compatible with BICEPS?
  2. And if not (clearly), would it make sense to file a ticket on BICEPS (for the Revision project) to expand the usage scope of AbstractMetricDescriptor/Relation?

@d-gregorczyk
Copy link
Copy Markdown
Collaborator

@d-gregorczyk , @stefandreher , I have noticed that the Relations extension defined here uses the AbstractMetricDescriptor/Relation defined in BICEPS (and slightly amended in the BICEPS Corrigendum that will be published VERY VERY soon). The usage scope in BICEPS is "represents a relationship between a METRIC and one or more other CONTAINMENT TREE ENTRIEs". However, here it is used to represent relationships between MDSs.

  1. Is that stretched usage compatible with BICEPS?
  2. And if not (clearly), would it make sense to file a ticket on BICEPS (for the Revision project) to expand the usage scope of AbstractMetricDescriptor/Relation?

Actually, I had to introduce a dedicated Relation type (sdpi:Relation) as the metric Relation type is anonymously specified and hence cannot be used anywhere except for AbstractMetricDescriptor. Conceptually, we're doing the same thing as we did with metrics. There has been quite some discussions and regrets as to why we did not add the Relation type to AbstractDescriptor - which would have spared us the SDPi version of a relation...

@PaulMartinsen
Copy link
Copy Markdown
Collaborator

@PaulMartinsen , there are two requirements defined by this PR (R0613 and R0614). Do they look fine in terms of RI "tagging"?

With the many changes you introduced lately (mostly with PR·469) I am often unsure if we are missing to use any important part of the new RI "tagging" functionality...

Yep. Looks okay to me.

@JavierEspina
Copy link
Copy Markdown
Collaborator

SDPi call - 19 Dec 2025: concluded that if the only blocker today is that Stefan did not have a chance to review in depth, we should approve and merge. If Stefan finds an issue later, we can address it then.

@JavierEspina JavierEspina merged commit 76ea35a into master Dec 19, 2025
2 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Gemini SDPi Releases Dec 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Multi MDS Concerns devices that support multiple MDS containment Volume 3 Volume 3 contents

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Append TOI: MDIB/MDS Modeling for Device Aggregators

5 participants