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

Add docker-staging.yml for deployment to staging #334

Merged
merged 1 commit into from
Mar 17, 2023

Conversation

humphd
Copy link
Contributor

@humphd humphd commented Mar 14, 2023

Fixes #43

This defines a Docker Swarm Stack for staging. Some notes on what's included:

  • Redis configured to only run on the manager node (not worker(s)), and to use a named volume
  • Starchart with secrets and staging env vars configured, 2 instances will be run in parallel

I'm going to manually test this on staging and report back with fixes and info on how it goes.

@humphd humphd added category: deployment Related to building our local code into a working unit category: containers A tool that helps us make our solutions into separate microservice containers with docker labels Mar 14, 2023
@humphd humphd added this to the Milestone 0.6 milestone Mar 14, 2023
@humphd humphd self-assigned this Mar 14, 2023
sfrunza13
sfrunza13 previously approved these changes Mar 14, 2023
@humphd
Copy link
Contributor Author

humphd commented Mar 14, 2023

Rebased on the other fixes we did this morning. If this passes tests, I think it's ready to land.

@humphd humphd requested review from sfrunza13, a user, Genne23v and SerpentBytes March 14, 2023 16:05
@humphd
Copy link
Contributor Author

humphd commented Mar 14, 2023

OK, this is all green, ready for review!

sfrunza13
sfrunza13 previously approved these changes Mar 14, 2023
DEPLOY.md Outdated Show resolved Hide resolved
app/lib/saml.server.ts Outdated Show resolved Hide resolved
sfrunza13
sfrunza13 previously approved these changes Mar 14, 2023
@humphd humphd changed the title Add docker-staging.yml for deploymen to staging Add docker-staging.yml for deployment to staging Mar 17, 2023
SAML fixes based on testing with staging

Use HOSTNAME=http://host.docker.internal:8080 so tests work
@humphd humphd requested a review from sfrunza13 March 17, 2023 13:57
Copy link
Contributor

@Genne23v Genne23v left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks good to me!

- HOSTNAME=https://mycustomdomain-dev.senecacollege.ca
- AWS_ROUTE53_HOSTED_ZONE_ID=Z0228625ICAL609E0BBT
- [email protected]
- LETS_ENCRYPT_DIRECTORY_URL=https://acme-staging-v02.api.letsencrypt.org/directory
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to confirm this with @dadolhay.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm going to land, but if this is needing to get changed, let me know if a follow up.

@humphd humphd merged commit 4a23021 into DevelopingSpace:main Mar 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: containers A tool that helps us make our solutions into separate microservice containers with docker category: deployment Related to building our local code into a working unit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deploy the app to staging
4 participants