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

Streamline naming crunchydb cluster #4175

Merged
merged 4 commits into from
Dec 12, 2024
Merged

Conversation

dgboss
Copy link
Collaborator

@dgboss dgboss commented Dec 12, 2024

  • parameterize and centralize the cluster name into envars (which the PGO also uses to construct the role name and pguser secret name)
  • change APP_NAME variable in crunchy.yaml to CRUNCHY_NAME to reflect its purpose. APP_NAME is used elsewhere to refer to the overall application name and the database cluster naming is separate

Test Links:

Landing Page
MoreCast
Percentile Calculator
C-Haines
FireBat
FireBat bookmark
Auto Spatial Advisory (ASA)
HFI Calculator
PSU Insights

Copy link

codecov bot commented Dec 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.73%. Comparing base (9a263be) to head (3e0beaf).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4175   +/-   ##
=======================================
  Coverage   80.73%   80.73%           
=======================================
  Files         314      314           
  Lines       12040    12040           
  Branches      540      540           
=======================================
  Hits         9721     9721           
  Misses       2131     2131           
  Partials      188      188           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@dgboss dgboss requested review from conbrad and brettedw December 12, 2024 16:47
@dgboss dgboss temporarily deployed to production December 12, 2024 17:32 Inactive
@dgboss dgboss merged commit 79d9274 into main Dec 12, 2024
24 checks passed
@dgboss dgboss deleted the task/streamline-crunchy-name branch December 12, 2024 17:58
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