Skip to content

Commit 857bdc7

Browse files
authored
docs: update s3 bucket physical names to use amazon owned s3 buckets (#31576)
### Issue # (if applicable) Closes #<issue number here>. ### Reason for this change By using Amazon owned s3 bucket physical names, we remove the possibility of documenting customer owned buckets in our examples. ### Description of changes Change physical S3 bucket names such as `my-bucket` to Amazon owned ones such as `amzn-s3-demo-bucket`. ### Description of how you validated changes ### Checklist - [X] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md) ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
1 parent a8b2f01 commit 857bdc7

File tree

31 files changed

+135
-117
lines changed

31 files changed

+135
-117
lines changed

CHANGELOG.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7484,10 +7484,10 @@ const bucket = new s3.Bucket(this, 'L2Bucket');
74847484
const bucketResource = bucket.findChild('Resource') as s3.cloudformation.BucketResource;
74857485

74867486
// strongly-typed overrides
7487-
bucketResource.propertyOverrides.bucketName = 'NewBucketName';
7487+
bucketResource.propertyOverrides.bucketName = 'amzn-s3-demo-bucket';
74887488

74897489
// weakly-typed overrides
7490-
bucketResource.addPropertyOverride('BucketName', 'NewerBucketName');
7490+
bucketResource.addPropertyOverride('BucketName', 'amzn-s3-demo-bucket1');
74917491
```
74927492

74937493
### Bug Fixes
@@ -8289,7 +8289,7 @@ new Pipeline(this, 'MyCoolPipeline', { artifactsBucket: bucket });
82898289
You can also import a bucket by just specifying its name:
82908290

82918291
```typescript
8292-
const bucket = Bucket.import({ bucketName: new BucketName('my-bucket') });
8292+
const bucket = Bucket.import({ bucketName: new BucketName('amzn-s3-demo-bucket') });
82938293
new Pipeline(this, 'MyCoolPipeline', { artifactsBucket: bucket });
82948294
```
82958295

packages/@aws-cdk-testing/framework-integ/test/aws-servicediscovery/test/integ.service-with-http-namespace.lit.js.snapshot/aws-servicediscovery-integ.assets.json

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/@aws-cdk-testing/framework-integ/test/aws-servicediscovery/test/integ.service-with-http-namespace.lit.js.snapshot/aws-servicediscovery-integ.template.json

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -22,15 +22,15 @@
2222
"Type": "AWS::ServiceDiscovery::Instance",
2323
"Properties": {
2424
"InstanceAttributes": {
25-
"arn": "arn:aws:s3:::mybucket"
25+
"arn": "arn:aws:s3:::amzn-s3-demo-bucket"
2626
},
27+
"InstanceId": "ervicediscoveryintegMyNamespaceNonIpServiceNonIpInstance45389A2A",
2728
"ServiceId": {
2829
"Fn::GetAtt": [
2930
"MyNamespaceNonIpService3B425009",
3031
"Id"
3132
]
32-
},
33-
"InstanceId": "ervicediscoveryintegMyNamespaceNonIpServiceNonIpInstance45389A2A"
33+
}
3434
}
3535
},
3636
"MyNamespaceIpService220F547F": {
@@ -57,13 +57,13 @@
5757
"AWS_INSTANCE_IPV4": "54.239.25.192",
5858
"AWS_INSTANCE_PORT": "80"
5959
},
60+
"InstanceId": "awsservicediscoveryintegMyNamespaceIpServiceIpInstance56070746",
6061
"ServiceId": {
6162
"Fn::GetAtt": [
6263
"MyNamespaceIpService220F547F",
6364
"Id"
6465
]
65-
},
66-
"InstanceId": "awsservicediscoveryintegMyNamespaceIpServiceIpInstance56070746"
66+
}
6767
}
6868
}
6969
},

packages/@aws-cdk-testing/framework-integ/test/aws-servicediscovery/test/integ.service-with-http-namespace.lit.js.snapshot/cdk.out

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/@aws-cdk-testing/framework-integ/test/aws-servicediscovery/test/integ.service-with-http-namespace.lit.js.snapshot/integ.json

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/@aws-cdk-testing/framework-integ/test/aws-servicediscovery/test/integ.service-with-http-namespace.lit.js.snapshot/manifest.json

Lines changed: 10 additions & 8 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/@aws-cdk-testing/framework-integ/test/aws-servicediscovery/test/integ.service-with-http-namespace.lit.js.snapshot/tree.json

Lines changed: 42 additions & 26 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/@aws-cdk-testing/framework-integ/test/aws-servicediscovery/test/integ.service-with-http-namespace.lit.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ const service1 = namespace.createService('NonIpService', {
1313
});
1414

1515
service1.registerNonIpInstance('NonIpInstance', {
16-
customAttributes: { arn: 'arn:aws:s3:::mybucket' },
16+
customAttributes: { arn: 'arn:aws:s3:::amzn-s3-demo-bucket' },
1717
});
1818

1919
const service2 = namespace.createService('IpService', {

packages/@aws-cdk/app-staging-synthesizer-alpha/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -336,7 +336,7 @@ class CustomStagingStack extends Stack implements IStagingResources {
336336

337337
public addFile(asset: FileAssetSource): FileStagingLocation {
338338
return {
339-
bucketName: 'myBucket',
339+
bucketName: 'amzn-s3-demo-bucket',
340340
assumeRoleArn: 'myArn',
341341
dependencyStack: this,
342342
};

packages/@aws-cdk/aws-codestar-alpha/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ new codestar.GitHubRepository(this, 'GitHubRepo', {
2929
accessToken: SecretValue.secretsManager('my-github-token', {
3030
jsonField: 'token',
3131
}),
32-
contentsBucket: s3.Bucket.fromBucketName(this, 'Bucket', 'bucket-name'),
32+
contentsBucket: s3.Bucket.fromBucketName(this, 'Bucket', 'amzn-s3-demo-bucket'),
3333
contentsKey: 'import.zip',
3434
});
3535
```

0 commit comments

Comments
 (0)