Skip to content

Profile operations mk202602#1227

Draft
mkurzman wants to merge 2 commits intospdx:profile-operationsfrom
boschglobal:profile_operations_mk202602
Draft

Profile operations mk202602#1227
mkurzman wants to merge 2 commits intospdx:profile-operationsfrom
boschglobal:profile_operations_mk202602

Conversation

@mkurzman
Copy link
Contributor

This branch was created from develop as of today.
The draw.io file was extended by the OpenSourceProject Class and merely the md-file for OpenSourceProject was added in the model section as base for next review round.
This is merely a PR as base for common review in the operations profile group.
The missing md-files for properties etc. still need to be added in the next iteration after the high-level discussion on Class-level have been done.

Based on the model reviewed in the Operations Profile Meeting and changed the classname from "UpstreamProject" to "OpenSourceProject" to have a clear reference and avoid semantics-confusion (Doubts from the review: Upstream relative to what downstream?).

Signed-off-by: Marcel Kurzmann <marcel.kurzmann@bosch.com>
Naming adapted from UpstreamProject to OpenSourceProject after review in the business operations profile meeting.

Signed-off-by: Marcel Kurzmann <marcel.kurzmann@bosch.com>
@bact
Copy link
Collaborator

bact commented Feb 27, 2026

I would suggest to leave draw.io diagram unchanged in this PR.

And once everything is settled, update the diagram in another PR.

Or there will be a difficult to manage conflicts over the diagram file.

We now have one PR pending for diagrams changes
#1220

@bact
Copy link
Collaborator

bact commented Feb 27, 2026

Looking forward, we may like to put one diagram per one file in the future, so it will be easier to manage.

Or rely on "text based" descriptive diagram (like PlantUML and Mermaid), which is more friendly to text version control system.

@zvr
Copy link
Member

zvr commented Feb 27, 2026

+1 for having different .drawio files, one per profile. That's how they are being used, anyway: each one in a separate chapter in annex A.

FYI, spec-parser already produces a complete "text-based" diagram, in PlantUML format (command line switches -P or --output-plantuml, IIRC). But manually creating/manipulating in diagrams.net results in a more aesthetically appealing diagram.

@bact bact added Profile:Operations Operations profile and related matters diagram Diagrams for SPDX model and others labels Feb 27, 2026
@mkurzman mkurzman marked this pull request as draft February 27, 2026 15:47
@umm0 umm0 changed the base branch from profile-operations to develop February 27, 2026 16:04
@umm0 umm0 changed the base branch from develop to profile-operations February 27, 2026 16:04
@umm0
Copy link
Collaborator

umm0 commented Feb 27, 2026

Just FYI, I needed to switch the base branch back and forth to update the file diff view to show the correct state after #1228 .
Github apparently does not do this automatically.

@bact
Copy link
Collaborator

bact commented Feb 27, 2026

FYI #1220 (that modifies diagram) has just merged (thanks all the reviewers). So it should be good now to edit the diagram file -- please merge from develop again one more time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

diagram Diagrams for SPDX model and others Profile:Operations Operations profile and related matters

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants