Skip to content

Commit bca1016

Browse files
authored
adjusting default s3 path to cid-data- prefix (#717)
1 parent c4e2a73 commit bca1016

File tree

4 files changed

+12
-12
lines changed

4 files changed

+12
-12
lines changed

cfn-templates/cid-cfn.tests.bats

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ setup_file() {
6969
QuickSightUser="$quicksight_user"\
7070
CURBucketPath="s3://$cur_bucket/$cur_prefix/$cur_name/$cur_name"\
7171
CURIsManagedByCloudFormation="yes"\
72-
CostOptimizationDataCollectionBucketPath="s3://costoptimizationdata{account_id}"\
72+
CostOptimizationDataCollectionBucketPath="s3://cid-data-{account_id}"\
7373
DeployCUDOSDashboard="yes"\
7474
DeployComputeOptimizerDashboard="no"\
7575
DeployCostIntelligenceDashboard="yes"\

cid/builtin/core/data/resources.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -528,7 +528,7 @@ views:
528528
File: tao/glue_table.json
529529
parameters:
530530
s3FolderPath:
531-
default: 's3://costoptimizationdata{account_id}/trusted-advisor/trusted-advisor-data'
531+
default: 's3://cid-data-{account_id}/trusted-advisor/trusted-advisor-data'
532532
description: Trusted Advisor Organization View report S3 path
533533
ta_org_view:
534534
File: tao/ta_org_view.sql

cid/test/bats/10-deploy-update-delete/compute-optimizer-dashboard.bats

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,10 @@ account_id=$(aws sts get-caller-identity --query "Account" --output text )
77
--dashboard-id compute-optimizer-dashboard \
88
--share-with-account \
99
--athena-database 'optimization_data' \
10-
--view-compute-optimizer-lambda-lines-s3FolderPath 's3://costoptimizationdata{account_id}/Compute_Optimizer/Compute_Optimizer_ec2_lambda' \
11-
--view-compute-optimizer-ebs-volume-lines-s3FolderPath 's3://costoptimizationdata{account_id}/Compute_Optimizer/Compute_Optimizer_ebs_volume' \
12-
--view-compute-optimizer-auto-scale-lines-s3FolderPath 's3://costoptimizationdata{account_id}/Compute_Optimizer/Compute_Optimizer_auto_scale' \
13-
--view-compute-optimizer-ec2-instance-lines-s3FolderPath 's3://costoptimizationdata{account_id}/Compute_Optimizer/Compute_Optimizer_ec2_instance'
10+
--view-compute-optimizer-lambda-lines-s3FolderPath 's3://cid-data-{account_id}/Compute_Optimizer/Compute_Optimizer_ec2_lambda' \
11+
--view-compute-optimizer-ebs-volume-lines-s3FolderPath 's3://cid-data-{account_id}/Compute_Optimizer/Compute_Optimizer_ebs_volume' \
12+
--view-compute-optimizer-auto-scale-lines-s3FolderPath 's3://cid-data-{account_id}/Compute_Optimizer/Compute_Optimizer_auto_scale' \
13+
--view-compute-optimizer-ec2-instance-lines-s3FolderPath 's3://cid-data-{account_id}/Compute_Optimizer/Compute_Optimizer_ec2_instance'
1414

1515
[ "$status" -eq 0 ]
1616
}
@@ -43,10 +43,10 @@ account_id=$(aws sts get-caller-identity --query "Account" --output text )
4343
@test "Update works" {
4444
run cid-cmd -vv --yes update --force --recursive \
4545
--dashboard-id compute-optimizer-dashboard \
46-
--view-compute-optimizer-lambda-lines-s3FolderPath 's3://costoptimizationdata{account_id}/Compute_Optimizer/Compute_Optimizer_ec2_lambda' \
47-
--view-compute-optimizer-ebs-volume-lines-s3FolderPath 's3://costoptimizationdata{account_id}/Compute_Optimizer/Compute_Optimizer_ebs_volume' \
48-
--view-compute-optimizer-auto-scale-lines-s3FolderPath 's3://costoptimizationdata{account_id}/Compute_Optimizer/Compute_Optimizer_auto_scale' \
49-
--view-compute-optimizer-ec2-instance-lines-s3FolderPath 's3://costoptimizationdata{account_id}/Compute_Optimizer/Compute_Optimizer_ec2_instance'
46+
--view-compute-optimizer-lambda-lines-s3FolderPath 's3://cid-data-{account_id}/Compute_Optimizer/Compute_Optimizer_ec2_lambda' \
47+
--view-compute-optimizer-ebs-volume-lines-s3FolderPath 's3://cid-data-{account_id}/Compute_Optimizer/Compute_Optimizer_ebs_volume' \
48+
--view-compute-optimizer-auto-scale-lines-s3FolderPath 's3://cid-data-{account_id}/Compute_Optimizer/Compute_Optimizer_auto_scale' \
49+
--view-compute-optimizer-ec2-instance-lines-s3FolderPath 's3://cid-data-{account_id}/Compute_Optimizer/Compute_Optimizer_ec2_instance'
5050

5151
[ "$status" -eq 0 ]
5252
echo "$output" | grep 'Update completed'

cid/test/bats/10-deploy-update-delete/ta-organizational-view.bats

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ account_id=$(aws sts get-caller-identity --query "Account" --output text )
88
--athena-database 'optimization_data' \
99
--share-with-account \
1010

11-
--view-ta-organizational-view-reports-s3FolderPath "s3://costoptimizationdata$account_id/optics-data-collector/ta-data'"
11+
--view-ta-organizational-view-reports-s3FolderPath "s3://cid-data-$account_id/optics-data-collector/ta-data'"
1212

1313
[ "$status" -eq 0 ]
1414
}
@@ -41,7 +41,7 @@ account_id=$(aws sts get-caller-identity --query "Account" --output text )
4141
@test "Update works" {
4242
run cid-cmd -vv --yes update --force --recursive \
4343
--dashboard-id ta-organizational-view \
44-
--view-ta-organizational-view-reports-s3FolderPath "s3://costoptimizationdata$account_id)/optics-data-collector/ta-data"
44+
--view-ta-organizational-view-reports-s3FolderPath "s3://cid-data-$account_id)/optics-data-collector/ta-data"
4545

4646
[ "$status" -eq 0 ]
4747
echo "$output" | grep 'Update completed'

0 commit comments

Comments
 (0)