镜像站点 · 本页由第三方 GitHub 只读镜像提供,非 GitHub 官方站点,不接受任何登录或凭据输入。前往 github.com
Skip to content

Workspace azmonitor depends on airlock - #3194

Merged
Tamir Kamara (tamirkamara) merged 1 commit into
mainfrom
tamirkamara/3193-workspace-delete
Feb 5, 2023
Merged

Tamir Kamara (tamirkamara) merged 1 commit into
mainfrom
tamirkamara/3193-workspace-delete

Conversation

@tamirkamara

Copy link
Copy Markdown
Collaborator

What is being addressed

Sometimes a conflict will occur when a workspace is being deleted. The source of this is networking changed done inside the airlock and az-monitor modules of the bundle.
This should be handled by terraform without manual intervention, but since it seem to not happen like that...

How is this addressed

  • Add a dependency so that az-monitor and airlock will not run together
  • [small] add a retry in the e2e to handle transient api connection errors

@tamirkamara
Tamir Kamara (tamirkamara) marked this pull request as ready for review February 5, 2023 09:10
@github-actions

github-actions Bot commented Feb 5, 2023

Copy link
Copy Markdown

Unit Test Results

0 tests   0 ✔️  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit 08ba884.

@tamirkamara

Copy link
Copy Markdown
Collaborator Author

/test-extended

@github-actions

github-actions Bot commented Feb 5, 2023

Copy link
Copy Markdown

🤖 pr-bot 🤖

🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/4095907898 (with refid f3c150f5)

(in response to this comment from Tamir Kamara (@tamirkamara))

1 similar comment
@github-actions

github-actions Bot commented Feb 5, 2023

Copy link
Copy Markdown

🤖 pr-bot 🤖

🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/4095907898 (with refid f3c150f5)

(in response to this comment from Tamir Kamara (@tamirkamara))

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.

Workspace deletion fails due to AnotherOperationInProgress pe-ampls-*

2 participants