Skip to content

[TILES-V2-8]: add env vars #986

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

Open
wants to merge 1 commit into
base: tiles-v2/trunk
Choose a base branch
from

Conversation

pregnantboy
Copy link
Contributor

@pregnantboy pregnantboy commented May 10, 2025

TL;DR

Added Tiles Postgres database environment variables to ECS configuration.

What changed?

Added five new environment variables to the ECS environment configuration:

  • TILES_POSTGRES_HOST
  • TILES_POSTGRES_DATABASE
  • TILES_POSTGRES_USERNAME
  • TILES_POSTGRES_PASSWORD
  • TILES_POSTGRES_PORT

Each variable references corresponding parameter store values with the plumber-<ENVIRONMENT>-tiles-rds-* pattern.

@datadog-opengovsg
Copy link

datadog-opengovsg bot commented May 10, 2025

Datadog Report

Branch report: tiles-v2/add-env-vars
Commit report: fe6b4d2
Test service: plumber

✅ 0 Failed, 773 Passed, 0 Skipped, 2m 30.54s Total Time
⬆️ Test Sessions change in coverage: 1 increased (+0.89%)

@pregnantboy pregnantboy changed the title feat: add env vars [TILES-V2-8]: add env vars May 11, 2025
@pregnantboy pregnantboy force-pushed the tiles-v2/update-actions branch from 4a191cd to 6c74578 Compare May 23, 2025 08:11
@pregnantboy pregnantboy force-pushed the tiles-v2/add-env-vars branch from 81100c6 to 2655ab3 Compare May 23, 2025 08:11
@pregnantboy pregnantboy force-pushed the tiles-v2/add-env-vars branch from 2655ab3 to c9b84f1 Compare May 29, 2025 08:22
@pregnantboy pregnantboy force-pushed the tiles-v2/update-actions branch from 6c74578 to b1913e1 Compare May 29, 2025 08:22
@pregnantboy pregnantboy force-pushed the tiles-v2/add-env-vars branch from c9b84f1 to d0a2797 Compare May 29, 2025 09:01
@pregnantboy pregnantboy force-pushed the tiles-v2/update-actions branch from b1913e1 to 9ba6275 Compare May 29, 2025 09:01
@pregnantboy pregnantboy marked this pull request as ready for review June 3, 2025 09:41
@pregnantboy pregnantboy requested a review from a team as a code owner June 3, 2025 09:41
@pregnantboy pregnantboy force-pushed the tiles-v2/update-actions branch from 9ba6275 to 9bc7e39 Compare June 3, 2025 10:37
@pregnantboy pregnantboy force-pushed the tiles-v2/add-env-vars branch from d0a2797 to b47b2f9 Compare June 3, 2025 10:37
cursor-com[bot]

This comment was marked as outdated.

cursor-com[bot]

This comment was marked as outdated.

@pregnantboy pregnantboy force-pushed the tiles-v2/add-env-vars branch from b47b2f9 to ed375ee Compare June 3, 2025 11:02
@pregnantboy pregnantboy force-pushed the tiles-v2/update-actions branch from 9bc7e39 to 3c3cb76 Compare June 3, 2025 11:02
cursor-com[bot]

This comment was marked as outdated.

cursor-com[bot]

This comment was marked as outdated.

@pregnantboy pregnantboy force-pushed the tiles-v2/update-actions branch from 3c3cb76 to 40af4a4 Compare June 5, 2025 07:29
@pregnantboy pregnantboy force-pushed the tiles-v2/add-env-vars branch from ed375ee to 619e2e8 Compare June 5, 2025 07:29
cursor-com[bot]

This comment was marked as outdated.

@pregnantboy pregnantboy changed the base branch from tiles-v2/update-actions to graphite-base/986 June 6, 2025 07:17
@pregnantboy pregnantboy force-pushed the tiles-v2/add-env-vars branch from 619e2e8 to fce7994 Compare June 6, 2025 07:19
@graphite-app graphite-app bot changed the base branch from graphite-base/986 to tiles-v2/trunk June 6, 2025 07:20
@pregnantboy pregnantboy force-pushed the tiles-v2/add-env-vars branch from fce7994 to fe6b4d2 Compare June 6, 2025 07:20
Copy link

@cursor-com cursor-com bot left a comment

Choose a reason for hiding this comment

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

✅ BugBot reviewed your changes and found no bugs!


BugBot free trial expires on June 10, 2025
You have used $0.00 of your $1.00 spend limit so far. Manage your spend limit in the Cursor dashboard.

Was this report helpful? Give feedback by reacting with 👍 or 👎

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.

1 participant