Skip to content

Conversation

@zchase
Copy link
Contributor

@zchase zchase commented Feb 24, 2020

This PR is a continuation of #2376. This applies the suggested changes after 97th Floor did an initial optimization.

97thFloor and others added 3 commits January 28, 2020 18:16
@cnunciato optimized for KW: CloudFormation. I'm not sure how to create headers, but "What is Pulumi?", "What is CloudFormation?" and "The Pulumi Difference" were intended to be subheadings. If that's not possible they're also fine as they are.
@zchase zchase requested a review from cnunciato February 24, 2020 17:44
Copy link

@pulumi pulumi bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🍹 The Update (preview) for stack pulumi/www.pulumi.com/production was successful.

Resource Changes

    Name                         Type                                          Operation
>-  default-vpc                  aws:ec2/vpc:Vpc                               read
~   archive-bucket-callback      aws:lambda/function:Function                  update
>-  default-vpc-public-1         aws:ec2/subnet:Subnet                         read
>-  default-vpc-public-0         aws:ec2/subnet:Subnet                         read
*   archive-handler              aws:ecs/taskDefinition:TaskDefinition         replaced
~   archive-bucket-subscription  aws:s3/bucketNotification:BucketNotification  update

@github-actions
Copy link
Contributor

Your site preview for commit 44935bf is now available! 🍹

http://content-bucket-8bec55f.s3-website-us-east-1.amazonaws.com

Deploy previews powered by Pulumify

aliases: ["/docs/reference/vs/cloud_templates/"]
---

## What is CloudFormation?
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This heading doesn't work, as the content addresses ARM also. IMO we should just remove it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍


Pulumi's model shares a lot with these systems. Although Pulumi programs are written in imperative, familiar languages,
they are ultimately evaluated to produce a similar set of create, update, or delete operations for your cloud resources.
## The Pulumi Difference
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same with this one -- the phrase has an advertising-y ring to it and doesn't add much value, and since already seem to rank #1 for "difference between pulumi and cloudformation", presumably because of the "vs" in the title, so I'd suggest dropping this heading, too.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

## The Pulumi Difference

This is where Pulumi begins to diverge from these other solutions, however.
Pulumi offers a multi-cloud alternative to more restrictive infrastructure-as-code template solutions, such as AWS CloudFormation. Pulumi allows you to streamline processes through managing resources from different platforms all in one place, using real code, in your favorite language.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Pulumi offers a multi-cloud alternative to more restrictive infrastructure-as-code template solutions, such as AWS CloudFormation. Pulumi allows you to streamline processes through managing resources from different platforms all in one place, using real code, in your favorite language.
Pulumi offers a multi-cloud alternative to more restrictive infrastructure-as-code template solutions like AWS CloudFormation. Pulumi allows you to streamline processes by managing resources from different platforms all in one place, using real code, in your favorite language.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

This is where Pulumi begins to diverge from these other solutions, however.
Pulumi offers a multi-cloud alternative to more restrictive infrastructure-as-code template solutions, such as AWS CloudFormation. Pulumi allows you to streamline processes through managing resources from different platforms all in one place, using real code, in your favorite language.

Pulumi's model shares a lot with these systems. Although Pulumi programs are written in imperative, familiar languages,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This paragraph doesn't make sense wedged in where it is. It interrupts the flow between the second and fourth paragraphs, and is essentially saying the same thing as the fifth. We should reposition it, make it work, or remove it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since it is redundant I opted to remove it. No reason to have redundant content IMO.

@zchase zchase requested a review from cnunciato February 25, 2020 23:45
Copy link

@pulumi pulumi bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🍹 The Update (preview) for stack pulumi/www.pulumi.com/production was successful.

Resource Changes

    Name                         Type                                          Operation
>-  default-vpc-public-1         aws:ec2/subnet:Subnet                         read
>-  default-vpc-public-0         aws:ec2/subnet:Subnet                         read
~   archive-bucket-callback      aws:lambda/function:Function                  update
>-  default-vpc                  aws:ec2/vpc:Vpc                               read
~   archive-bucket-subscription  aws:s3/bucketNotification:BucketNotification  update
*   archive-handler              aws:ecs/taskDefinition:TaskDefinition         replaced

@github-actions
Copy link
Contributor

Your site preview for commit 4105442 is now available! 🍹

http://content-bucket-8bec55f.s3-website-us-east-1.amazonaws.com

Deploy previews powered by Pulumify

Copy link
Contributor

@cnunciato cnunciato left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The phrasing in the multi-cloud alternative paragraph still bugs me -- e.g., "streamline processes"? What processes, and streamline them how? But needn't block. Moves us forward, and we can always come back to improve it later.

@zchase zchase merged commit cd88610 into master Feb 27, 2020
@pulumi-bot pulumi-bot deleted the zchase/97th-floor-patch-1 branch February 27, 2020 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants