addPropertyOverride() does not seem to work

[spg]

Describe the bug
addPropertyOverride does not seem to work

To Reproduce
Synthetize the following stack:

import cdk = require('@aws-cdk/cdk');
import s3 = require('@aws-cdk/aws-s3');


export class TestStack extends cdk.Stack {
  constructor(scope: cdk.App, id: string, props?: cdk.StackProps) {
    super(scope, id, props);

    const bucket = new s3.Bucket(this, 'MyBucket');
    const bucketResource = bucket.node.findChild('Resource') as s3.CfnBucket;
    bucketResource.addPropertyOverride('AccessControl', 'bucket-owner-read');
  }
}

yields the following:

npx cdk synth test-stack
Resources:
  MyBucketF68F3FF0:
    Type: AWS::S3::Bucket
    DeletionPolicy: Retain
    Metadata:
      aws:cdk:path: test-stack/MyBucket/Resource
    Properties: {}
  CDKMetadata:
    Type: AWS::CDK::Metadata
    Properties:
      Modules: aws-cdk=0.32.0,@aws-cdk/aws-events=0.32.0,@aws-cdk/aws-iam=0.32.0,@aws-cdk/aws-kms=0.32.0,@aws-cdk/aws-s3=0.32.0,@aws-cdk/aws-s3-notifications=0.32.0,@aws-cdk/cdk=0.32.0,@aws-cdk/cx-api=0.32.0,@aws-cdk/region-info=0.32.0,jsii-runtime=node.js/v11.6.0

Note that AccessControl is absent from the synth output.

Expected behavior
AccessControl: "bucket-owner-read" should be seen in the synth output

Version:

• OS: OS X 10.14.5
• Programming Language: Typescript
• CDK Version: 0.32.0

[eladb]

Try to use camel case: addPropertyOverride('accessControl', 'bucket-owner-read').

This is actually not the intended behavior and we should fix, but in the meantime, I hope this will allow you to workaround the issue.

[machender1]

I am facing a similar issue when trying to override Path property on an instance profile. I do not see path getting attached to my resource generated on cfn.

`if (node instanceof iam.CfnInstanceProfile ) {

      //node.addPropertyOverride('Path', this.pathArn);
      const instanceProfileResource = node.node.findChild('Resource') as iam.CfnInstanceProfile;
      instanceProfileResource.addPropertyOverride('path', this.pathArn);
  }`

Produces ...
ASGInstanceProfile0A2834D7: Type: AWS::IAM::InstanceProfile Properties: Roles: - Ref: ConfluanceTestRoleEB6CE1EB Metadata: aws:cdk:path: Ec2AppCdkStack/ASG/InstanceProfile

Could you please advise on what am i missing here to add Path to instance profile?

Thanks