-
Notifications
You must be signed in to change notification settings - Fork 146
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
Enable scm basic auth in airlock notifier #3509
Conversation
/test |
🤖 pr-bot 🤖 🏃 Running tests: https://github.com/microsoft/AzureTRE/actions/runs/5036255947 (with refid (in response to this comment from @tamirkamara) |
3138c13
to
3ca3963
Compare
/test-shared-services |
🤖 pr-bot 🤖 🏃 Running shared service tests: https://github.com/microsoft/AzureTRE/actions/runs/5036298731 (with refid (in response to this comment from @tamirkamara) |
/test-shared-services |
🤖 pr-bot 🤖 🏃 Running shared service tests: https://github.com/microsoft/AzureTRE/actions/runs/5036602152 (with refid (in response to this comment from @tamirkamara) |
/test-force-approve |
🤖 pr-bot 🤖 ✅ Marking tests as complete (for commit 7c96daa) (in response to this comment from @tamirkamara) |
/test |
🤖 pr-bot 🤖 🏃 Running tests: https://github.com/microsoft/AzureTRE/actions/runs/5037299300 (with refid (in response to this comment from @tamirkamara) |
Resolves #3508
What is being addressed
Airlock notifier uses the zip deploy method to install. This method requires SCM to support basic auth which seem to be now disabled by default.
How is this addressed