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

Provide Airlock Import Review Workspace with its own DNS zone #3769

Merged
merged 4 commits into from
Nov 7, 2023

Conversation

marrobi
Copy link
Member

@marrobi marrobi commented Oct 27, 2023

Fixes #3767

To resolve the Airlock import issue described in (#3767) the new airlock import review tempalte in this PR will need registering using make workspace_bundle BUNDLE=airlock-import-review and any existing airlock import review workspaces will need upgrading. After upgrade run make deploy-core to reinstate and deleted DNS records.

How is this addressed

  • Remove DNS zone registration for review workspace private endpoint
  • Create new DNS zone specific to import inprogress storage account
  • Create root record to the storage account IP.
  • Add temporary and permenant vnet_id output of the network module. Temp fix can be removed after the next release.

I have tested the end to end import process.

@marrobi marrobi enabled auto-merge (squash) October 27, 2023 12:14
@marrobi
Copy link
Member Author

marrobi commented Oct 27, 2023

/test-extended

@github-actions
Copy link

🤖 pr-bot 🤖

🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/6667034468 (with refid 3624d7ea)

(in response to this comment from @marrobi)

@github-actions
Copy link

github-actions bot commented Oct 27, 2023

Unit Test Results

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

Results for commit 9255f2d.

♻️ This comment has been updated with latest results.

@SvenAelterman
Copy link
Collaborator

Successfully tested new workspace deployment. Troubleshooting workspace upgrade.

@marrobi
Copy link
Member Author

marrobi commented Nov 7, 2023

Successfully tested new workspace deployment. Troubleshooting workspace upgrade.

Verified upgrade works:

image

@marrobi
Copy link
Member Author

marrobi commented Nov 7, 2023

/test-force-approve

Passed: https://github.com/microsoft/AzureTRE/actions/runs/6667034468

Copy link

github-actions bot commented Nov 7, 2023

🤖 pr-bot 🤖

✅ Marking tests as complete (for commit 9255f2d)

(in response to this comment from @marrobi)

@marrobi marrobi merged commit 1c85eaa into microsoft:main Nov 7, 2023
10 checks passed
@marrobi marrobi deleted the marrobi/issue3767 branch November 7, 2023 11:52
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.

Airlock fails due to DNS timeout - returns "Request failed due to an unknown reason."
3 participants