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.

Sign up for GitHub

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

Jump to bottom

Update deployment environments for "main" workflow #1267

Closed
5 tasks
TangoYankee opened this issue Sep 25, 2024 · 1 comment · Fixed by #1293
Closed
5 tasks

Update deployment environments for "main" workflow #1267

TangoYankee opened this issue Sep 25, 2024 · 1 comment · Fixed by #1293
Assignees
@TangoYankee
Milestone
Self-Service: Cre...

Comments

@TangoYankee
Copy link
Member

TangoYankee commented Sep 25, 2024 •
edited
Loading

Description

Because qa--applicant-portal.netlify.app was already used for a branch deployment, it cannot be used for a cli deployment. However, there is a training available. We will need to create a new environment and update the ngnix proxy redirect the NYCID login to the that environment.

Also, the master alias was already used for a branch deploy; it would be fruitless to try deploying to it. We should remove that job from the GitHub action.

Acceptance criteria

  • The qa deployment alias is changed to qa2
  • The deploy-client-master job is removed
  • The deploy-client-prod is renamed to deploy-client
  • The nginx proxy redirects both production domains to the production netlify environment
  • The nginx proxy redirects the qa to domain to the qa2 netlify environment
@TangoYankee TangoYankee self-assigned this Sep 25, 2024
TangoYankee added a commit that referenced this issue Sep 25, 2024
@TangoYankee
Remove branch deploy aliases
dd2c330 
Replace qa alias with training alias
Remove master alias

closes #1267
@TangoYankee TangoYankee mentioned this issue Sep 25, 2024
Closed
TangoYankee added a commit that referenced this issue Sep 27, 2024
@TangoYankee
Remove branch deploy aliases
922e4c7 
Replace qa alias with training alias
Remove master alias

closes #1267
@TangoYankee
Copy link
Member Author

TangoYankee commented Oct 7, 2024 •
edited
Loading

This might be changed to a new qa url

Edit: updated ticket to reflect new qa url

@TangoYankee TangoYankee changed the title Place QA deployment in "training" environment Update deployment environments for "main" workflow Oct 21, 2024
TangoYankee added a commit that referenced this issue Oct 21, 2024
@TangoYankee
Update branch deploy aliases
84d3773 
Refactor qa alias to qa2
Remove master alias

closes #1267
@TangoYankee TangoYankee mentioned this issue Oct 21, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@TangoYankee TangoYankee

Labels
None yet
Projects
None yet
Milestone
Self-Service: Create project with use...
1 participant
@TangoYankee