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

feat(examples): Vertex Machine Learning Pipeline #66

Merged
Merged
Show file tree
Hide file tree
Changes from 75 commits
Commits
Show all changes
77 commits
Select commit Hold shift + click to select a range
ec29d99
refacto ml-vertexpipeline
renato-rudnicki Jun 12, 2024
2de509f
fix empty spaces
renato-rudnicki Jun 13, 2024
1d87258
sync with main
renato-rudnicki Jun 13, 2024
7900b96
fix conflict
renato-rudnicki Jun 13, 2024
40aae11
fix lint
renato-rudnicki Jun 13, 2024
e1250c2
Add aditional instructions before run Notebooks
renato-rudnicki Jun 14, 2024
fbee309
Add aditional instructions before run Notebooks
renato-rudnicki Jun 14, 2024
fdf2895
change data to use variable as input
renato-rudnicki Jun 14, 2024
9d7e411
add new variables values for data filters
renato-rudnicki Jun 20, 2024
e29bbca
fixes for envs and iam roles
renato-rudnicki Jun 20, 2024
8f7fb1c
add missing variables
renato-rudnicki Jun 20, 2024
f834958
Merge branch 'main' into vertex-pipeline-example
renato-rudnicki Jun 21, 2024
ce9d823
fix conflicts
renato-rudnicki Jun 25, 2024
45a199c
README update
renato-rudnicki Jun 27, 2024
76f4ea0
update with tests
caetano-colin Jun 27, 2024
ec52448
Merge pull request #2 from caetano-colin/test-renato-example
renato-rudnicki Jun 27, 2024
a78b5d7
update tests
caetano-colin Jun 28, 2024
0496599
kfp==2.7.0
caetano-colin Jun 28, 2024
010c5f1
add vertex_model_sa as prod_sa
caetano-colin Jun 28, 2024
f6bf1be
updating PR
caetano-colin Jul 1, 2024
e2c90db
add vpc-sc rules
caetano-colin Jul 1, 2024
980d59a
update
caetano-colin Jul 1, 2024
431c1b9
update readme
caetano-colin Jul 1, 2024
506bc5b
update vpc-sc rules
caetano-colin Jul 2, 2024
0259623
Merge pull request #3 from caetano-colin/test-renato
renato-rudnicki Jul 2, 2024
f614399
small fixes for lint and documentation
renato-rudnicki Jul 2, 2024
a91ec14
update README
renato-rudnicki Jul 2, 2024
5c953f7
update README
renato-rudnicki Jul 3, 2024
d31e7c0
Update README for Github App ID and more details about Develop, Non-P…
renato-rudnicki Jul 8, 2024
cceb460
Add missing logging project at vpc-sc directional rule
renato-rudnicki Jul 8, 2024
b68671a
Set github_app_installation_id and github_remote_uri value as empty
renato-rudnicki Jul 8, 2024
393756f
fix github_app_installation_id format
renato-rudnicki Jul 11, 2024
1a46078
fix for_each for artifact_registry_iam_member
renato-rudnicki Jul 11, 2024
c00478c
fix lint
renato-rudnicki Jul 11, 2024
3aefff2
add terraform init for 1-org
renato-rudnicki Jul 16, 2024
3ef8bc6
fix for_each for google_storage_bucket_iam_member
renato-rudnicki Jul 16, 2024
ba92368
Fixes for machine-learning-pipeline/README.md
renato-rudnicki Jul 17, 2024
7763ece
fix for Github_app_id
renato-rudnicki Jul 18, 2024
1a1f1a7
READMEs update
renato-rudnicki Jul 19, 2024
bb36619
Update README
renato-rudnicki Jul 19, 2024
3fd53b9
add changes
caetano-colin Jul 19, 2024
65c9c5a
Merge pull request #4 from caetano-colin/adjust-ml-pipe-example
renato-rudnicki Jul 19, 2024
6892500
bump project-factory version
renato-rudnicki Jul 19, 2024
be90d7b
add note about bash terminal
renato-rudnicki Jul 19, 2024
69b343f
bump project-factory version
renato-rudnicki Jul 19, 2024
13ebe00
Note about inconsistent final plan
renato-rudnicki Jul 19, 2024
736c521
fix project-factory bump version
renato-rudnicki Jul 22, 2024
67418a5
remove hardcode data
renato-rudnicki Jul 22, 2024
ec4d4f8
Fix and improvements for Machine Learning Example
renato-rudnicki Jul 26, 2024
d8aee2a
Automated replacement of placeholders
renato-rudnicki Jul 29, 2024
269508a
add discalimers
caetano-colin Aug 6, 2024
fe49961
rewriting
caetano-colin Aug 6, 2024
58fd65c
rewrite
caetano-colin Aug 6, 2024
0e229d6
Merge pull request #5 from caetano-colin/add-disclaimer
renato-rudnicki Aug 7, 2024
e3123b6
Path fix
renato-rudnicki Aug 13, 2024
8cb74df
Update for deploy with terraform local and cloudbuild sections
renato-rudnicki Aug 13, 2024
bdeff1c
Fix indentation.
renato-rudnicki Aug 13, 2024
f7f8d1e
update steps to add SA in the service perimeter
renato-rudnicki Aug 15, 2024
87a9753
perma-diff in provider causes Cloud Functions in 1-org to always fail
renato-rudnicki Aug 16, 2024
ea02006
fix command path
renato-rudnicki Aug 22, 2024
dbffa63
Revert "fix command path"
renato-rudnicki Aug 22, 2024
324536c
add step to unset billing/quota_project
renato-rudnicki Aug 22, 2024
ef146c0
fix path for BQ commands
renato-rudnicki Aug 22, 2024
03ab845
fix placeholders for census_pipeline.ipynb
renato-rudnicki Aug 22, 2024
79fb92b
fix path for terraform local deploy
renato-rudnicki Aug 27, 2024
2e27be8
update notebook dependencies
renato-rudnicki Aug 27, 2024
f6e1c47
update placeholders
renato-rudnicki Aug 28, 2024
b898d52
fix typo
renato-rudnicki Aug 28, 2024
f6c0684
fix placeholders for compile_pipeline
renato-rudnicki Aug 28, 2024
fc7e708
add detail about https in the clone repo step for Vertex
renato-rudnicki Aug 29, 2024
279cdec
fix conflicts
renato-rudnicki Sep 27, 2024
8297213
update docker image
renato-rudnicki Sep 27, 2024
b1d7d00
update juniper notebooks
renato-rudnicki Sep 27, 2024
ccd511e
fix conflict
renato-rudnicki Sep 30, 2024
cf4eeea
Update steps from machine-learning-pipeline example
renato-rudnicki Sep 30, 2024
da49015
add README
renato-rudnicki Oct 1, 2024
437704f
update README for machine learning example
renato-rudnicki Oct 4, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion 0-bootstrap/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,10 @@ To run the commands described in this document, install the following:
- [Terraform](https://www.terraform.io/downloads.html) version 1.5.7
- [jq](https://jqlang.github.io/jq/download/) version 1.6.0 or later

**Note:** Make sure that you use version 1.5.7 of Terraform throughout this series. Otherwise, you might experience Terraform state snapshot lock errors.
**Notes:**

- Make sure that you use version 1.5.7 of Terraform throughout this series. Otherwise, you might experience Terraform state snapshot lock errors.
- It is recommended to use Bash terminal de deploy the code from this repository. Using other terminals might cause unexpected behaviours.

Also make sure that you've done the following:

Expand Down
3 changes: 2 additions & 1 deletion 1-org/envs/shared/ml_key_rings.tf
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,8 @@ module "kms_keyring" {
keyring_admins = [
"serviceAccount:${local.projects_step_terraform_service_account_email}"
]
project_id = module.common_kms.project_id

project_id = module.org_kms.project_id
keyring_regions = var.keyring_regions
keyring_name = var.keyring_name
}
18 changes: 9 additions & 9 deletions 1-org/envs/shared/projects.tf
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ locals {

module "org_audit_logs" {
source = "terraform-google-modules/project-factory/google"
version = "~> 14.0"
version = "~> 15.0"

random_project_id = true
random_project_id_length = 4
Expand Down Expand Up @@ -66,7 +66,7 @@ module "org_audit_logs" {

module "org_billing_logs" {
source = "terraform-google-modules/project-factory/google"
version = "~> 14.0"
version = "~> 15.0"

random_project_id = true
random_project_id_length = 4
Expand Down Expand Up @@ -98,7 +98,7 @@ module "org_billing_logs" {

module "org_kms" {
source = "terraform-google-modules/project-factory/google"
version = "~> 14.0"
version = "~> 15.0"

random_project_id = true
random_project_id_length = 4
Expand Down Expand Up @@ -131,7 +131,7 @@ module "org_kms" {

module "org_secrets" {
source = "terraform-google-modules/project-factory/google"
version = "~> 14.0"
version = "~> 15.0"

random_project_id = true
random_project_id_length = 4
Expand Down Expand Up @@ -163,7 +163,7 @@ module "org_secrets" {

module "interconnect" {
source = "terraform-google-modules/project-factory/google"
version = "~> 14.0"
version = "~> 15.0"

random_project_id = true
random_project_id_length = 4
Expand Down Expand Up @@ -195,7 +195,7 @@ module "interconnect" {

module "scc_notifications" {
source = "terraform-google-modules/project-factory/google"
version = "~> 14.0"
version = "~> 15.0"

random_project_id = true
random_project_id_length = 4
Expand Down Expand Up @@ -227,7 +227,7 @@ module "scc_notifications" {

module "dns_hub" {
source = "terraform-google-modules/project-factory/google"
version = "~> 14.0"
version = "~> 15.0"

random_project_id = true
random_project_id_length = 4
Expand Down Expand Up @@ -267,7 +267,7 @@ module "dns_hub" {

module "base_network_hub" {
source = "terraform-google-modules/project-factory/google"
version = "~> 14.0"
version = "~> 15.0"
count = var.enable_hub_and_spoke ? 1 : 0

random_project_id = true
Expand Down Expand Up @@ -316,7 +316,7 @@ resource "google_project_iam_member" "network_sa_base" {

module "restricted_network_hub" {
source = "terraform-google-modules/project-factory/google"
version = "~> 14.0"
version = "~> 15.0"
count = var.enable_hub_and_spoke ? 1 : 0

random_project_id = true
Expand Down
6 changes: 3 additions & 3 deletions 1-org/modules/cai-monitoring/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ data "archive_file" "function_source_zip" {

module "cloudfunction_source_bucket" {
source = "terraform-google-modules/cloud-storage/google//modules/simple_bucket"
version = "~>3.4"
version = "~>5.0"

project_id = var.project_id
name = "bkt-cai-monitoring-${random_id.suffix.hex}-sources-${data.google_project.project.number}-${var.location}"
Expand Down Expand Up @@ -121,7 +121,7 @@ resource "google_cloud_asset_organization_feed" "organization_feed" {

module "pubsub_cai_feed" {
source = "terraform-google-modules/pubsub/google"
version = "~> 5.0"
version = "~> 6.0"

topic = "top-cai-monitoring-${random_id.suffix.hex}-event"
project_id = var.project_id
Expand All @@ -142,7 +142,7 @@ resource "google_scc_source" "cai_monitoring" {
// Cloud Function
module "cloud_function" {
source = "GoogleCloudPlatform/cloud-functions/google"
version = "0.4.1"
version = "0.5"

function_name = "caiMonitoring"
description = "Check on the Organization for members (users, groups and service accounts) that contains the IAM roles listed."
Expand Down
4 changes: 2 additions & 2 deletions 1-org/modules/cai-monitoring/versions.tf
Original file line number Diff line number Diff line change
Expand Up @@ -19,11 +19,11 @@ terraform {
required_providers {
google = {
source = "hashicorp/google"
version = ">= 3.77"
version = ">= 3.77, <=5.37"
}
google-beta = {
source = "hashicorp/google-beta"
version = ">= 3.77"
version = ">= 3.77, <=5.37"
}
random = {
source = "hashicorp/random"
Expand Down
4 changes: 2 additions & 2 deletions 1-org/modules/network/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@

module "base_shared_vpc_host_project" {
source = "terraform-google-modules/project-factory/google"
version = "~> 14.0"
version = "~> 15.0"

random_project_id = true
random_project_id_length = 4
Expand Down Expand Up @@ -56,7 +56,7 @@ module "base_shared_vpc_host_project" {

module "restricted_shared_vpc_host_project" {
source = "terraform-google-modules/project-factory/google"
version = "~> 14.0"
version = "~> 15.0"

random_project_id = true
random_project_id_length = 4
Expand Down
Loading