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

Changed domain origin_hostname to aks #2991

Closed
wants to merge 3 commits into from

Conversation

temitope777
Copy link
Contributor

WHY: This is so that the website could be accessed via the aks endpoint
HOW: Replaced existing PAAS domain origin_hostname with the AKS one

Trello card

Context

changed domain host-name to aks host

Changes proposed in this pull request

replaced existing paas with aks domain name

Guidance to review

@temitope777 temitope777 temporarily deployed to review_aks October 10, 2023 12:17 — with GitHub Actions Inactive
@temitope777 temitope777 temporarily deployed to review_aks October 10, 2023 13:21 — with GitHub Actions Inactive
@temitope777 temitope777 temporarily deployed to Review October 10, 2023 13:21 — with GitHub Actions Inactive
WHY: This is so that the website could be accessed via the aks endpoint
HOW: Replaced existing PAAS domain origin_hostname with the AKS one
@temitope777 temitope777 force-pushed the feature/5757-change-host-name branch from 239fea1 to 848fb96 Compare October 10, 2023 13:35
@saliceti saliceti marked this pull request as draft October 10, 2023 13:37
@temitope777 temitope777 temporarily deployed to Review October 10, 2023 13:51 — with GitHub Actions Inactive
@temitope777 temitope777 temporarily deployed to review_aks October 10, 2023 13:51 — with GitHub Actions Inactive
Tope Emmanuel and others added 2 commits October 10, 2023 17:00
We need a minimum of 2 web replicas for high availability. The 2 workers
are for sidekiq performance.
We're not sure if they are used by the app or not but this caused errors
in the restore so we're enabling as well on Azure.
@temitope777 temitope777 temporarily deployed to review_aks October 10, 2023 16:05 — with GitHub Actions Inactive
@temitope777 temitope777 temporarily deployed to Review October 10, 2023 16:05 — with GitHub Actions Inactive
@saliceti saliceti force-pushed the feature/5757-change-host-name branch from a833a18 to c63039a Compare October 10, 2023 16:12
@saliceti saliceti marked this pull request as ready for review October 10, 2023 16:16
@saliceti saliceti temporarily deployed to Review October 10, 2023 16:27 — with GitHub Actions Inactive
@saliceti saliceti temporarily deployed to review_aks October 10, 2023 16:27 — with GitHub Actions Inactive
@saliceti
Copy link
Member

Closing in favour of #2990

@saliceti saliceti closed this Oct 10, 2023
@saliceti saliceti temporarily deployed to Review October 10, 2023 16:27 — with GitHub Actions Inactive
@saliceti saliceti temporarily deployed to review_aks October 10, 2023 16:27 — with GitHub Actions Inactive
@github-actions
Copy link

Review app deployed to https://get-school-experience-review-pr-2991.test.teacherservices.cloud
⚠️ DfE sign in route pool exhausted (close some open PRs!)

@github-actions
Copy link

Review app deployed to https://review-school-experience-2991.london.cloudapps.digital
⚠️ DfE sign in route pool exhausted (close some open PRs!)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants