Skip to content

Amend session ID to prevent parallel operations on a resource.#3807

Closed
marrobi wants to merge 7 commits intomicrosoft:mainfrom
marrobi:marrobi/issue3177
Closed

Amend session ID to prevent parallel operations on a resource.#3807
marrobi wants to merge 7 commits intomicrosoft:mainfrom
marrobi:marrobi/issue3177

Conversation

@marrobi
Copy link
Copy Markdown
Member

@marrobi marrobi commented Dec 6, 2023

Resolves #3177

What is being addressed

How is this addressed

  • Modify session_id of service bus messages to be resource ID or it's parent where risk of conflicting operations.
  • Add missing defaults to Databricks template to enable this to be tested using API
  • Update CHANGELOG.md if needed

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 6, 2023

Unit Test Results

596 tests   594 ✅  7s ⏱️
  1 suites    2 💤
  1 files      0 ❌

Results for commit 7a3188d.

♻️ This comment has been updated with latest results.

@marrobi marrobi requested a review from damoodamoo December 7, 2023 15:24
@marrobi
Copy link
Copy Markdown
Member Author

marrobi commented Dec 7, 2023

@damoodamoo before I finish this of, value your opinion on if this is a good "solution" to this challenge.

@marrobi
Copy link
Copy Markdown
Member Author

marrobi commented Dec 12, 2023

/test-destroy-env

@github-actions
Copy link
Copy Markdown

Destroying PR test environment (RG: rg-tre6c41c63c)... (run: https://github.com/microsoft/AzureTRE/actions/runs/7180372009)

@github-actions
Copy link
Copy Markdown

PR test environment destroy complete (RG: rg-tre6c41c63c)

@marrobi
Copy link
Copy Markdown
Member Author

marrobi commented Jan 5, 2024

Think it's worth trying to replicate with just terraform as the TF provider should wait for the VNet to be ready. If can reproduce, create issue upstream, and remove workaround when resolved.

@marrobi
Copy link
Copy Markdown
Member Author

marrobi commented May 20, 2024

Need to verify behaviour if try to start/deploy multiple VMs at once, does it happen in series?

@martinpeck martinpeck added triage Being triaged before any action taken. spike labels Mar 20, 2025
@martinpeck
Copy link
Copy Markdown
Member

Closing this PR as it has been open for too long.
Adding label spike as the code still has value.
Associated with issue #3177

@martinpeck martinpeck closed this Mar 20, 2025
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Azure TRE - Engineering Mar 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

spike triage Being triaged before any action taken.

Projects

Development

Successfully merging this pull request may close these issues.

Workspace Service Installs Fails with PutSubnetOperation or CanceledAndSupersededDueToAnotherOperation

2 participants