remove heroku concerns from deploy scripts #2010
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ballPointPenguin
added a commit
that referenced
this pull request
May 2, 2025
* Makefile Improvements (#1937) * optional DETACH=true arg for make commands using docker compose up * add rebuild-web shortcut to Makefile * update rebuild-web make action * Development Data Seeds via Cypress (#1938) * Helpful `npm run seed` script in e2e * improved seed logic * improved seed performance * include xids in seed votes * increment seed xid * default server logging to 'warn' (#1939) * typecheck on morepids * begin cdk setup * begin setting up arch * store notes in readme for now * configure db * remove comment * encorporate external branch changes * final config * use yum instead of dnf * pubsub * add customResource for asg versioning * correct custom resource dependencies * switch to docker compose * new db secret strategy * deployment config added * add scripts and appspec * remove cloudwatch for now * ecr prep * update db secrets * minor config fixes * updates for db connection * modify healthcheck url * test deploy workflow * add docker-compose * alt compose install * add env step * pass vars differently for docker pull * revert to latest tag * log config * update db instance * start nginx with server * add subdomain config * manual dns validation * more dns config * final domain config * env var config * secrets conf * update jq logic * add debugging lines * reorder user script * update bash script to add env vars at end * try appending entire env * update .env strategy * final env simplification strategy * va fixes * fix aws command * debug port 443 conflict * fix before_install * debug after install * stop before start * more after install fixes * temp disable redirect middleware * add use network host * add network host in more places * add network host * add again * add one more place * try new method * update * debug logging * last try * try brute force * more debug * more debug 2 * testing * change trust proxy * set to true temp * follow up on proxy * force build and recreate * flip dev mode back off * target redir * set trust proxy to responsible value * fix docker en * increase timeout duration for streaming responses * remove echo * attempt node image upgrade * different node upgrade strategy * explicitly install openssl and deps * upgrade translate api * docker pruin and size updates * fix instnace state * debug logging * try replace newline * try diff newline replacement * use base64 instead * add readme and refine alarms, notifications, and autoscaling * lower math worker alarm min cpu threshold * drop threshold even lower * more alarm adjustments * force pull * git config in ec2 setup * hardcode report cache and remove env var example * remove deployment in alarm * turn off autorollback * attempt promisify topics * rework async streaming * remove alarms * debug logging * more debug * change cache strategy * change job to manual run * error checking * switch branch to edge * upgrade math instance memory * change math to c5a image * allow all to read from ecr * cutoff for large-convo processing if > 5000 comments (#1961) * Fix and update the E2E build (#1963) * FIx e2e build * e2e updates * Add local DynamoDB support and update configuration * remove unused dev dependency * upgrade math image * Client-Admin updates and cleanup (#1966) * update eslint, babel, and prettier * more update eslint, babel, and prettier * WIP new webpack build * update minor versions * update dev packages and eslint; remove unused * remove socials * improved webpack with hmr * update Dockerfile * update nodes; enable custom apiUrl in dev * bump client-admin workflows to node 20 * install markdown-loader * remove bundlewatch * favicon; mdx-loader * prepare for deploy * xid report via uuid (#1964) * xid report via uuid * fix math.ts MPromise * escape quotes in react * e2e assertion fix * Update linting in `server` and `client-report` (#1969) * fix client-report lint & test * fix server lint; use logger; use config * github workflow to lint all lintable sub-projects * use node v22 for lint job * allow cdk resources to upload to s3 (#1972) * upsert on conflict for `translateAndStoreComment` (#1985) * add desired capacity * Added a space between 'vote' and 'on' on the participantHelpWelcomeText string in the en_us.js file (#1988) * fully remove facebook & twitter (#1974) * remove all facebook, twitter, and related features * remove facebook/twitter from top-level files * remove facebook and twitter from client-participation * remove demographics from client-report * add simple analytics (#1992) * Bump aws-cdk-lib from 2.178.2 to 2.189.1 in /cdk (#1990) Bumps [aws-cdk-lib](https://github.com/aws/aws-cdk/tree/HEAD/packages/aws-cdk-lib) from 2.178.2 to 2.189.1. - [Release notes](https://github.com/aws/aws-cdk/releases) - [Changelog](https://github.com/aws/aws-cdk/blob/v2.189.1/CHANGELOG.v2.alpha.md) - [Commits](https://github.com/aws/aws-cdk/commits/v2.189.1/packages/aws-cdk-lib) --- updated-dependencies: - dependency-name: aws-cdk-lib dependency-version: 2.189.1 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump http-proxy-middleware from 2.0.7 to 2.0.9 in /client-report (#1995) Bumps [http-proxy-middleware](https://github.com/chimurai/http-proxy-middleware) from 2.0.7 to 2.0.9. - [Release notes](https://github.com/chimurai/http-proxy-middleware/releases) - [Changelog](https://github.com/chimurai/http-proxy-middleware/blob/v2.0.9/CHANGELOG.md) - [Commits](chimurai/http-proxy-middleware@v2.0.7...v2.0.9) --- updated-dependencies: - dependency-name: http-proxy-middleware dependency-version: 2.0.9 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump @babel/helpers from 7.20.13 to 7.26.10 in /server (#1967) Bumps [@babel/helpers](https://github.com/babel/babel/tree/HEAD/packages/babel-helpers) from 7.20.13 to 7.26.10. - [Release notes](https://github.com/babel/babel/releases) - [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md) - [Commits](https://github.com/babel/babel/commits/v7.26.10/packages/babel-helpers) --- updated-dependencies: - dependency-name: "@babel/helpers" dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * fix embed.js function call (#1998) * fix embed.js function call * test spacing fix * add in missing embed service hostename var --------- Co-authored-by: tevko <timevko@gmail.com> * upgrade victorycharts make bad console message disappear (#2006) * remove heroku concerns from deploy scripts (#2010) * clean up gh actions a bit (#2011) --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: tevko <tim@devzero.io> Co-authored-by: Tim <timevko@gmail.com> Co-authored-by: Mark Simpkins <mark@maahee.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.