Skip to content

Add route table to airlock processor subnet#2414

Merged
tamirkamara merged 6 commits into
mainfrom
tamirkamara/2097-airlock-proc-subnet-route-table
Aug 8, 2022
Merged

Add route table to airlock processor subnet#2414
tamirkamara merged 6 commits into
mainfrom
tamirkamara/2097-airlock-proc-subnet-route-table

Conversation

@tamirkamara

@tamirkamara tamirkamara commented Aug 7, 2022

Copy link
Copy Markdown
Collaborator

Resolves #2097 Resolves #2222

What is being addressed

Airlock processor subnet was missing a route table.

How is this addressed

  • Add the route table to airlock processor subnet
  • Add the default nsg to the airlock notification subnet
  • Create subnets and assign nsg sequentially as parallel operations are not allowed and sometimes cause a failure.

@github-actions

github-actions Bot commented Aug 7, 2022

Copy link
Copy Markdown

Unit Test Results

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

Results for commit 6539a95.

♻️ This comment has been updated with latest results.

@tamirkamara tamirkamara changed the title Tamirkamara/2097 airlock proc subnet route table Add route table to airlock processor subnet Aug 7, 2022
@tamirkamara

Copy link
Copy Markdown
Collaborator Author

/test-extended

@github-actions

github-actions Bot commented Aug 7, 2022

Copy link
Copy Markdown

🤖 pr-bot 🤖

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

(in response to this comment from @tamirkamara)

@tamirkamara

Copy link
Copy Markdown
Collaborator Author

/test-destroy-env

@github-actions

github-actions Bot commented Aug 7, 2022

Copy link
Copy Markdown

Destroying branch test environment (RG: rg-tre1a06a6cd)... (run: https://github.com/microsoft/AzureTRE/actions/runs/2812246474)

@github-actions

github-actions Bot commented Aug 7, 2022

Copy link
Copy Markdown

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

@github-actions

github-actions Bot commented Aug 7, 2022

Copy link
Copy Markdown

Branch test environment destroy complete (RG: rg-tre1a06a6cd)

@github-actions

github-actions Bot commented Aug 7, 2022

Copy link
Copy Markdown

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

@tamirkamara

Copy link
Copy Markdown
Collaborator Author

/test-destroy-env

@github-actions

github-actions Bot commented Aug 7, 2022

Copy link
Copy Markdown

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

@github-actions

github-actions Bot commented Aug 7, 2022

Copy link
Copy Markdown

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

@github-actions

github-actions Bot commented Aug 7, 2022

Copy link
Copy Markdown

Destroying branch test environment (RG: rg-tre1a06a6cd)... (run: https://github.com/microsoft/AzureTRE/actions/runs/2812306282)

@github-actions

github-actions Bot commented Aug 7, 2022

Copy link
Copy Markdown

Branch test environment destroy complete (RG: rg-tre1a06a6cd)

@tamirkamara

Copy link
Copy Markdown
Collaborator Author

/test-extended

@github-actions

github-actions Bot commented Aug 7, 2022

Copy link
Copy Markdown

🤖 pr-bot 🤖

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

(in response to this comment from @tamirkamara)

@tamirkamara tamirkamara marked this pull request as ready for review August 7, 2022 15:15
@tamirkamara

Copy link
Copy Markdown
Collaborator Author

/test-extended

@github-actions

github-actions Bot commented Aug 7, 2022

Copy link
Copy Markdown

🤖 pr-bot 🤖

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

(in response to this comment from @tamirkamara)

1 similar comment
@github-actions

github-actions Bot commented Aug 7, 2022

Copy link
Copy Markdown

🤖 pr-bot 🤖

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

(in response to this comment from @tamirkamara)

@tamirkamara

Copy link
Copy Markdown
Collaborator Author

/test-extended

@github-actions

github-actions Bot commented Aug 8, 2022

Copy link
Copy Markdown

🤖 pr-bot 🤖

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

(in response to this comment from @tamirkamara)

@tamirkamara tamirkamara merged commit f61d9bb into main Aug 8, 2022
@tamirkamara tamirkamara deleted the tamirkamara/2097-airlock-proc-subnet-route-table branch August 8, 2022 14:02
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.

AnotherOperationInProgress in Airlock Terraform Add network route to the airlock processor subnet

2 participants