DR-009-ACL-Concept: Generic Format for Access Control List for Crypto#3055
DR-009-ACL-Concept: Generic Format for Access Control List for Crypto#3055christianbe9 wants to merge 8 commits into
Conversation
christianbe9
requested review from
PandaeDo,
aschemmel-tech,
masc2023 and
pahmann
as code owners
github-project-automation
Bot
added this to
TLC - Technical Lead Circle and S-CORE Roadmap
|
The created documentation from the pull request is available at: docu-html |
There was a problem hiding this comment.
According to existing DR, this should be DR-003-Arch, compare folder
There was a problem hiding this comment.
fine for me - then we would also need to rename DR-008-int, as we currently have 2 different "counting" shemes:
for int we just in general incremented+1
for arch and infra we incremented only "arch internally" +1 and "infra internally" +1
There was a problem hiding this comment.
fine for me - then we would also need to rename DR-008-int, as we currently have 2 different "counting" shemes:
for int we just in general incremented+1 for arch and infra we incremented only "arch internally" +1 and "infra internally" +1
DR-xyz-<infra|arch|proc>, each should be in row, int may wrong, so
we have DR-001-arch, DR-002-arch -> DR-003-arch is with you
There was a problem hiding this comment.
I rebased everything so it also matches the branch name: #3058 (comment)
i just dont understand based on what the reviewers are chosen, as there are only 2 now.
There was a problem hiding this comment.
I rebased everything so it also matches the branch name: #3058 (comment)
i just dont understand based on what the reviewers are chosen, as there are only 2 now.
The system detects, if Architecture DR -> Architecture people has to review,
compare https://github.com/eclipse-score/score/blob/main/.github/CODEOWNERS
Row 31:
/docs/design_decisions/arch @qor-lb @arsibo
By the Way, your PR is closed, you need to reopen it
There was a problem hiding this comment.
Understood, thanks a lot!
So that every matches (commit names, branch name, filename) I updated everything, thats why this review got closed and was automatically generated here: #3058 (comment)
And I think due to the intial wrong naming by me, the reviewer list now also automatically changed, what I understood now why :)
christianbe9
force-pushed
the
dr_009_acl_concept
branch
2 times, most recently
from
8354965 to
07adae2
Compare
Introducing a common format for an Access Control List (Crypto, SOME/IP, IPC and further potential usecases), such that it is only required to use one common parser, one common format for reviews and also one common "ACL Manager" aka editor for ACLs and also compiler for the ACL into a binary format. Signed-off-by: Christian Becker <christian.becker.ext@qorix.ai>
Signed-off-by: Christian Becker <christian.becker.ext@qorix.ai>
Signed-off-by: Christian Becker <christian.becker.ext@qorix.ai>
Signed-off-by: Christian Becker <christian.becker.ext@qorix.ai>
Signed-off-by: Christian Becker <christian.becker.ext@qorix.ai>
Signed-off-by: Christian Becker <christian.becker.ext@qorix.ai>
Signed-off-by: Christian Becker <christian.becker.ext@qorix.ai>
Signed-off-by: Christian Becker <christian.becker.ext@qorix.ai>�
christianbe9
force-pushed
the
dr_009_acl_concept
branch
from
07adae2 to
266ab1c
Compare
2 participants
SOME/IP and IPC (and preparation for further use cases)
Introducing a common format for an Access Control List, such that it is
only required to use one common parser, one common format for reviews
and also one common "ACL Manager" aka editor for ACLs and also compiler
for the ACL into a binary format.