Skip to content

Disable local authentication between Airlock Processor and Azure Service Bus in Function Binding#4277

Merged
guybartal merged 17 commits into
mainfrom
guybartal/airlock-disable-local-auth-for-sb
Feb 8, 2025
Merged

Disable local authentication between Airlock Processor and Azure Service Bus in Function Binding#4277
guybartal merged 17 commits into
mainfrom
guybartal/airlock-disable-local-auth-for-sb

Conversation

@guybartal

Copy link
Copy Markdown
Collaborator

Resolves #4259

What is being addressed

Customers may want to disable local auth between azure functions and
service bus queues, this PR sets the requiered environment variables for Azure Function bindings in Airlock Processor module.

How is this addressed

@guybartal

Copy link
Copy Markdown
Collaborator Author

/help

@github-actions

Copy link
Copy Markdown

🤖 pr-bot 🤖

Hello!

You can use the following commands:
    /test - build, deploy and run smoke tests on a PR
    /test-extended - build, deploy and run smoke & extended tests on a PR
    /test-extended-aad - build, deploy and run smoke & extended AAD tests on a PR
    /test-shared-services - test the deployment of shared services on a PR build
    /test-force-approve - force approval of the PR tests (i.e. skip the deployment checks)
    /test-destroy-env - delete the validation environment for a PR (e.g. to enable testing a deployment from a clean start after previous tests)
    /help - show this help

(in response to this comment from @guybartal)

@github-actions

github-actions Bot commented Jan 19, 2025

Copy link
Copy Markdown

Unit Test Results

21 tests   21 ✅  0s ⏱️
 1 suites   0 💤
 1 files     0 ❌

Results for commit 8092c18.

♻️ This comment has been updated with latest results.

@guybartal guybartal changed the title Guybartal/airlock-disable-local-auth-for-sb Disable local authentication between Airlock Processor and Azure Service Bus in Function Binding Jan 19, 2025
@guybartal

Copy link
Copy Markdown
Collaborator Author

/test

@github-actions

Copy link
Copy Markdown

🤖 pr-bot 🤖

🏃 Running tests: https://github.com/microsoft/AzureTRE/actions/runs/12854924727 (with refid 07c0773e)

(in response to this comment from @guybartal)

Comment thread core/terraform/airlock/airlock_processor.tf Outdated
Comment thread airlock_processor/host.json Outdated
Comment thread core/terraform/airlock/variables.tf
@guybartal guybartal enabled auto-merge (squash) February 2, 2025 09:58
@guybartal

Copy link
Copy Markdown
Collaborator Author

/test-extended

@github-actions

github-actions Bot commented Feb 5, 2025

Copy link
Copy Markdown

🤖 pr-bot 🤖

⚠️ Cannot run tests as PR is not mergeable. Ensure that the PR is open and doesn't have any conflicts.

(in response to this comment from @guybartal)

@guybartal

Copy link
Copy Markdown
Collaborator Author

/test-extended

@github-actions

github-actions Bot commented Feb 5, 2025

Copy link
Copy Markdown

🤖 pr-bot 🤖

🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/13165376926 (with refid 07c0773e)

(in response to this comment from @guybartal)

@guybartal

Copy link
Copy Markdown
Collaborator Author

/test-extended

@github-actions

github-actions Bot commented Feb 5, 2025

Copy link
Copy Markdown

🤖 pr-bot 🤖

🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/13165500132 (with refid 07c0773e)

(in response to this comment from @guybartal)

@tamirkamara

Copy link
Copy Markdown
Collaborator

/test-extended

@github-actions

github-actions Bot commented Feb 6, 2025

Copy link
Copy Markdown

🤖 pr-bot 🤖

🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/13174494866 (with refid 07c0773e)

(in response to this comment from @tamirkamara)

@tamirkamara

Copy link
Copy Markdown
Collaborator

/test-destroy-env

@github-actions

github-actions Bot commented Feb 6, 2025

Copy link
Copy Markdown

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

@github-actions

github-actions Bot commented Feb 6, 2025

Copy link
Copy Markdown

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

@github-actions

github-actions Bot commented Feb 6, 2025

Copy link
Copy Markdown

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

@github-actions

github-actions Bot commented Feb 6, 2025

Copy link
Copy Markdown

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

@tamirkamara

Copy link
Copy Markdown
Collaborator

/test-extended

@github-actions

github-actions Bot commented Feb 6, 2025

Copy link
Copy Markdown

🤖 pr-bot 🤖

🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/13175197173 (with refid 07c0773e)

(in response to this comment from @tamirkamara)

@tamirkamara

Copy link
Copy Markdown
Collaborator

/test-extended

@github-actions

github-actions Bot commented Feb 6, 2025

Copy link
Copy Markdown

🤖 pr-bot 🤖

🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/13177381524 (with refid 07c0773e)

(in response to this comment from @tamirkamara)

@guybartal

Copy link
Copy Markdown
Collaborator Author

/test-extended

@github-actions

github-actions Bot commented Feb 8, 2025

Copy link
Copy Markdown

🤖 pr-bot 🤖

🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/13218797121 (with refid 07c0773e)

(in response to this comment from @guybartal)

@guybartal guybartal merged commit 133746a into main Feb 8, 2025
@guybartal guybartal deleted the guybartal/airlock-disable-local-auth-for-sb branch February 8, 2025 22:06
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.

Disable local authentication in ServiceBus

3 participants