New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Extract 'aws-infra' types into their own package independent of cloud-aws #472
Conversation
// TODO[pulumi/pulumi#377] Workaround for issue with long names not generating per-deplioyment randomness, leading | ||
// to collisions. For now, limit the size of hashes to ensure we generate shorter/ resource names. | ||
return shasum.digest("hex").substring(0, 8); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this code was copied. but it's so simple i don't mind that.
@@ -82,23 +81,23 @@ export function getComputeIAMRolePolicies(): aws.ARN[] { | |||
return computePolicies; | |||
} | |||
|
|||
let network: Network; | |||
let network: awsinfra.Network; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i preferred making it explicit tht these are from another module.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
aws/examples/cluster/package.json
Outdated
@@ -13,6 +13,7 @@ | |||
"peerDependencies": { | |||
"@pulumi/pulumi": "^0.11.2", | |||
"@pulumi/cloud": "latest", | |||
"@pulumi/cloud-aws": "latest" | |||
"@pulumi/cloud-aws": "latest", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we need cloud-aws
here anymore.
Note: further work will be done her.e For example, this change does not include changes to make cluster/network into components. I wanted to do the minimal amount to just the code into a separate package.