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

Move echo to S3 🎉 #39

Merged
merged 6 commits into from
Oct 20, 2020
Merged

Move echo to S3 🎉 #39

merged 6 commits into from
Oct 20, 2020

Conversation

pvinis
Copy link
Contributor

@pvinis pvinis commented Oct 9, 2020

Follow up:

  • make a production branch from master.
  • fixup https://releases.artsy.net/admin/projects/56 to have master instead of s3 and production instead of s3-production for all stages and deploy strategies.
  • Do we need a way to force deploy, even if there are no json changes? 🤔
    No. Even if we do, it's easy enough to add a dummy flag.

scripts/deploy.js Outdated Show resolved Hide resolved
@pvinis pvinis marked this pull request as ready for review October 20, 2020 16:17
@pvinis pvinis requested a review from ashfurrow October 20, 2020 16:17
@pvinis
Copy link
Contributor Author

pvinis commented Oct 20, 2020

This is ready to go. I'll do the horizon changes now, and once this is merged I'll make the production branch.

Copy link
Contributor

@ashfurrow ashfurrow left a comment

Choose a reason for hiding this comment

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

🎉 Looks great! Thanks again.

* __Github:__ [https://github.com/artsy/echo/](https://github.com/artsy/echo/)
* __CI:__ [Semaphore](https://semaphoreci.com/artsy-it/echo/); merged PRs to artsy/echo#master are automatically deployed to staging; production is manually deployed from Semaphore
* __Point People:__ [@ashfurrow](https://github.com/ashfurrow)
* __Point People:__ [@ashfurrow](https://github.com/ashfurrow), [@pvinis](https://github.com/pvinis)
Copy link
Contributor

Choose a reason for hiding this comment

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

🌟

@ashfurrow ashfurrow merged commit 505cfcc into master Oct 20, 2020
@ashfurrow ashfurrow deleted the s3 branch October 20, 2020 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants