Skip to content

Commit

Permalink
Merge pull request #76 from uktrade/feat/support-for-second-mlflow-ar…
Browse files Browse the repository at this point in the history
…tefacts

feat: support for a second MLFlow to store artefacts in a bucket
  • Loading branch information
michalc authored May 22, 2024
2 parents 7520dc4 + f836880 commit 9cbd2ea
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions infra/ecs_notebooks_notebook.tf
Original file line number Diff line number Diff line change
Expand Up @@ -260,6 +260,7 @@ data "aws_iam_policy_document" "aws_vpc_endpoint_s3_notebooks" {
resources = [
"${aws_s3_bucket.notebooks.arn}",
"${aws_s3_bucket.mlflow[0].arn}",
"${aws_s3_bucket.mlflow[1].arn}",
]
}

Expand All @@ -278,6 +279,7 @@ data "aws_iam_policy_document" "aws_vpc_endpoint_s3_notebooks" {
resources = [
"${aws_s3_bucket.notebooks.arn}/*",
"${aws_s3_bucket.mlflow[0].arn}/*",
"${aws_s3_bucket.mlflow[1].arn}/*",
]
}

Expand All @@ -294,6 +296,7 @@ data "aws_iam_policy_document" "aws_vpc_endpoint_s3_notebooks" {

resources = [
"${aws_s3_bucket.mlflow[0].arn}",
"${aws_s3_bucket.mlflow[1].arn}",
]
}

Expand Down

0 comments on commit 9cbd2ea

Please sign in to comment.