feat(core): use parameter store refs instead of exports

[corrjo]

---

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

[gitpod-io]

[corrjo]

Hey @skinny85, @rix0rrr I finally found some cycles to work on this (re: #13184) but hit a broken test in @aws-cdk/aws-eks-legacy I started working on a fix but also noticed

aws-cdk/packages/@aws-cdk/aws-eks-legacy/lib/cluster.ts

Line 343 in c6ca2ab

Annotations.of(this).addWarning('The @aws-cdk/aws-eks-legacy module will no longer be released as part of the AWS CDK starting March 1st, 2020. Please refer to https://github.com/aws/aws-cdk/issues/5544 for upgrade instructions');

is it worth spending time to fix this?

[rix0rrr]

Thanks for the intent, but this is really a fundamental design change that's going to need discussion. exports definitely have value which should be taken into account.

I recommend you open up an RFC first with a proposal and a motivation and thinking through some of the impacts.

Pull request has been modified.

[aws-cdk-automation]

AWS CodeBuild CI Report

• CodeBuild project: AutoBuildProject89A8053A-LhjRyN9kxr8o
• Commit ID: 01873bd
• Result: FAILED
• Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

[corrjo]

Thanks for the intent, but this is really a fundamental design change that's going to need discussion. exports definitely have value which should be taken into account.

I recommend you open up an RFC first with a proposal and a motivation and thinking through some of the impacts.

aws/aws-cdk-rfcs#311

[corrjo]

relates to #12778

[TheRealAmazonKendra]

This PR has been deemed to be abandoned, and will be closed. Please create a new PR for these changes if you think this decision has been made in error.