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

Feat: Add Record ID to Golden Record Tasks #991

Merged
merged 1 commit into from
Jul 12, 2024
Merged

Conversation

nicoprow
Copy link
Contributor

@nicoprow nicoprow commented Jul 8, 2024

Description

This pull request adds a gate record ID to the Orchestrator tasks. By this, tasks belonging to the same gate record can be identified by the cleaning services.

Implements eclipse-tractusx/sig-release#711

Pre-review checks

Please ensure to do as many of the following checks as possible, before asking for committer review:

@nicoprow nicoprow force-pushed the feat/orch-record-id branch 2 times, most recently from a7466bb to 45ad8e8 Compare July 10, 2024 08:20
@nicoprow nicoprow marked this pull request as ready for review July 11, 2024 02:57
@nicoprow nicoprow requested a review from SujitMBRDI July 11, 2024 02:57
…e records

- create gate record IDs on receiving task create request which do not already specify a record ID
- Gate now stores received record IDs to each sharing state, so that next time the record ID can be given to the orchestrator
- In the Orchestrator records are identified by both a private and a public record ID. The private record ID is for the Gates to use while the public record ID is sent to the cleaning services
@nicoprow nicoprow force-pushed the feat/orch-record-id branch from 45ad8e8 to 86c2581 Compare July 11, 2024 09:55
@SujitMBRDI SujitMBRDI assigned nicoprow and unassigned SujitMBRDI Jul 11, 2024
@nicoprow nicoprow merged commit 8af4933 into main Jul 12, 2024
16 checks passed
@nicoprow nicoprow deleted the feat/orch-record-id branch July 12, 2024 02:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants