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 orchestration state to calculations without changing existing flow #2273

Merged
merged 29 commits into from
May 22, 2024

Conversation

ebbeknudsen
Copy link
Contributor

@ebbeknudsen ebbeknudsen commented May 17, 2024

Description

Add (and use) CalculationOrchestrationState for Calculation

  • Add OrchestrationState column to Calculation table
  • Update calculation orchestration state when execution state is updated
  • Change (some) usages of execution state to orchestration state
  • Use CalculationOrchestrationState in calculation orchestration
  • Set CalculationOrchestrationState to MessagesEnqueuing when completed calculation is published
  • Update tests

Task: https://app.zenhub.com/workspaces/mosaic-60a6105157304f00119be86e/issues/gh/energinet-datahub/team-mosaic/203

Pull-request quality

  • The title adheres to this guide
  • Tests are written and executed locally
  • Subsystem tests have been tested (by manually deploying to dev_002)

Copy link

github-actions bot commented May 17, 2024

Test results for \source\dotnet\wholesale-api\Calculations\Calculations.IntegrationTests\bin\Release\net8.0\Energinet.DataHub.Wholesale.Calculations.IntegrationTests.dll

31 tests   31 ✅  2s ⏱️
 1 suites   0 💤
 1 files     0 ❌

Results for commit 31e70a7.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented May 17, 2024

Test results for \source\dotnet\wholesale-api\WebApi.IntegrationTests\bin\Release\net8.0\Energinet.DataHub.Wholesale.WebApi.IntegrationTests.dll

32 tests   29 ✅  15s ⏱️
 1 suites   3 💤
 1 files     0 ❌

Results for commit 31e70a7.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented May 17, 2024

Test results for \source\dotnet\wholesale-api\Orchestrations.IntegrationTests\bin\Release\net8.0\Energinet.DataHub.Wholesale.Orchestrations.IntegrationTests.dll

14 tests   14 ✅  1m 6s ⏱️
 1 suites   0 💤
 1 files     0 ❌

Results for commit d8657bf.

♻️ This comment has been updated with latest results.

lasrinnil
lasrinnil previously approved these changes May 22, 2024
Copy link
Contributor

@dstenroejl dstenroejl left a comment

Choose a reason for hiding this comment

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

LGTM

@ebbeknudsen ebbeknudsen enabled auto-merge (squash) May 22, 2024 11:30
@ebbeknudsen ebbeknudsen merged commit d224d02 into main May 22, 2024
34 checks passed
@ebbeknudsen ebbeknudsen deleted the xedkn/add-orchestration-state-to-calculation branch May 22, 2024 11:38
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.

None yet

3 participants