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

[2041] dfe_analytics module #123

Merged
merged 4 commits into from
Oct 22, 2024

Conversation

saliceti
Copy link
Member

@saliceti saliceti commented Oct 11, 2024

Context

New module to configure resources for dfe-analytics in Google cloud, and provide the required variables on the Azure side.

Changes proposed in this pull request

The new module can either reuse the existing dataset, and simply provide the variables for the app on AKS, or create the dataset and events table.
It also provides a bach script to configure Google cloud IAM, so the terraform code can be run in a Github action workflow.
The README dscribes the usage.

Guidance to review

Before merging

Remove the TEMP commit

After merging

Update the module tag in the AFQTS PR

Checklist

  • I have performed a self-review of my code, including formatting and typos
  • I have cleaned the commit history
  • I have added the Devops label
  • I have attached the pull request to the trello card

@saliceti saliceti force-pushed the 2041-configure-bigquery-in-the-terraform-module branch 4 times, most recently from 6505cf7 to e7df23d Compare October 11, 2024 17:15
@saliceti saliceti force-pushed the 2041-configure-bigquery-in-the-terraform-module branch 2 times, most recently from 9eb5633 to b533d74 Compare October 20, 2024 12:44
@saliceti saliceti marked this pull request as ready for review October 20, 2024 12:51
@saliceti saliceti force-pushed the 2041-configure-bigquery-in-the-terraform-module branch 4 times, most recently from 86313df to 8e91b4c Compare October 20, 2024 13:24
Create resources in Google cloud Bigquery and IAM to receive events from
applications
@saliceti saliceti force-pushed the 2041-configure-bigquery-in-the-terraform-module branch 2 times, most recently from cf6f846 to 8732079 Compare October 20, 2024 13:29
@saliceti saliceti changed the title dfe_analytics module [2041] dfe_analytics module Oct 21, 2024
@saliceti
Copy link
Member Author

aks/dfe_analytics/README.md Outdated Show resolved Hide resolved
aks/dfe_analytics/README.md Outdated Show resolved Hide resolved
@saliceti saliceti force-pushed the 2041-configure-bigquery-in-the-terraform-module branch from faac60b to 8600499 Compare October 22, 2024 15:46
@saliceti saliceti force-pushed the 2041-configure-bigquery-in-the-terraform-module branch from 8600499 to 0fcc184 Compare October 22, 2024 16:22
@saliceti saliceti merged commit 8e53a3d into main Oct 22, 2024
3 checks passed
@saliceti saliceti deleted the 2041-configure-bigquery-in-the-terraform-module branch October 22, 2024 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants