Skip to content

Dovetail Analytics v2 beta - #841

Merged
cavis merged 4 commits into
mainfrom
feat/analytics_v2_beta
Oct 8, 2025
Merged

cavis merged 4 commits into
mainfrom
feat/analytics_v2_beta

Conversation

@cavis

@cavis cavis commented Oct 7, 2025

Copy link
Copy Markdown
Member

For PRX/analytics-ingest-lambda#115

Deploys the new lambdas to staging only. Before merging:

  • Alter than analytics-ingest branch buildspec to push to /prx/${EnvironmentTypeAbbreviation}/Spire/Dovetail-Analytics/v2-beta/pkg/s3-object-key
  • Set the /prx/${EnvironmentTypeAbbreviation}/dovetail-analytics/BIGQUERY_CREDENTIALS with a json blob cred file

DovetailVerifiedMetricsKinesisStreamName: { Type: String }
DovetailCountedKinesisStreamArn: { Type: String }
DovetailCountedKinesisStreamName: { Type: String }
BigqueryCredentials: { Type: AWS::SSM::Parameter::Value<String> }

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Realized for the 5th time, Value<SecureString> doesn't exist. Bah!

@cavis
cavis requested a review from farski October 8, 2025 15:19
@cavis cavis self-assigned this Oct 8, 2025

@farski farski left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This all appears to be valid CloudFormation

@cavis
cavis merged commit fcf9234 into main Oct 8, 2025
4 checks passed
@cavis
cavis deleted the feat/analytics_v2_beta branch October 8, 2025 21:53
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.

2 participants