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

release changes to test environment #748

Merged
merged 140 commits into from
Jun 27, 2022
Merged

release changes to test environment #748

merged 140 commits into from
Jun 27, 2022

Conversation

afeld
Copy link
Contributor

@afeld afeld commented Jun 27, 2022

The last deploy to test was done in #695 (12 days ago). Let's get the latest changes into the test environment to match its recent fixture update.

machikoyasuda and others added 30 commits June 14, 2022 21:23
https://docs.microsoft.com/en-us/azure/app-service/deploy-ftp

We aren't using this, so for better security, turn it off.
in anticipation of dynamic settings per-instance
always send the 'openid' scope, allow for additional scopes via param

remove hardcoded setting now handled here
chore: reflect the sticky settings in Terraform
for anyone landing on the repo
afeld added 18 commits June 23, 2022 19:21
Set up test instance of Azure Insights
ANALYTICS_KEY was added manually, the DJANGO_OAUTH_* ones are no longer needed.
document the deployment slot settings
Removes a bunch that were auto-populated but aren't needed, and adds a couple that should be slot-specific.
update list of deployment slot settings
These seem to be necessary for connecting the App Service instance to Application Insights.
The path filter was potentially making deploys not happen when we would expect them; this will avoid those cases by always deploying.
chore: re-add auto-instrumented variables to deployment slot settings
chore: add debugging messages around Azure configuration
This way we can avoid having to copy changes across them.
This will help with confirming changes have been deployed.
allow the SHA of the deployed image to be viewed
For safety, in case a fixture configuration change breaks something.
@afeld afeld requested a review from a team as a code owner June 27, 2022 17:27
@github-actions github-actions bot added actions Related to GitHub Actions workflows front-end HTML/CSS/JavaScript and Django templates back-end Django views, sessions, middleware, models, migrations etc. i18n Copy: Language files or Django i18n framework tests Related to automated testing (unit, UI, integration, etc.) labels Jun 27, 2022
chore: enable versioning on the storage account
@afeld afeld temporarily deployed to dev June 27, 2022 17:28 Inactive
@angela-tran angela-tran added this to the Hosting migration milestone Jun 27, 2022
@afeld afeld merged commit f3316cd into test Jun 27, 2022
@github-actions github-actions bot added the deployment [auto] Pull requests that deploy to test or prod label Jun 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
actions Related to GitHub Actions workflows back-end Django views, sessions, middleware, models, migrations etc. deployment [auto] Pull requests that deploy to test or prod front-end HTML/CSS/JavaScript and Django templates i18n Copy: Language files or Django i18n framework tests Related to automated testing (unit, UI, integration, etc.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants