Update deploy-fleet.md

[mikermcneil]

Remove settings that aren't minimally required to make it simpler to follow along

@rfairburn @edwardsb @lukeheath Y'all, please stop me if this is a bad idea.

[rfairburn]

@mikermcneil if we are using the osquery-carve module at all, these values must be set as they have to be universally unique across all of AWS and therefore cannot be optional.

If we don't want to use osquery-carve in our reference architecture, we can remove these sections:

• https://github.com/fleetdm/fleet-terraform/blob/main/example/main.tf#L132-L138
• https://github.com/fleetdm/fleet-terraform/blob/main/example/main.tf#L30-L34
• https://github.com/fleetdm/fleet-terraform/blob/main/example/main.tf#L82

Or we could just leave them commented out in the example so they can be optionally added and then remove those lines in the documentation.

Let me know, and I can update the example this is based on to make s3 carves off by default.

[lukeheath]

@rfairburn I like your idea of removing from the docs but leaving commented out in the example. Simplifies the docs without losing the example.

[rfairburn]

@lukeheath I will make the change to the example and link the PR here so we can make sure it's in place prior to changing the docs.

[mikermcneil]

@mikermcneil if we are using the osquery-carve module at all, these values must be set as they have to be universally unique across all of AWS and therefore cannot be optional.

If we don't want to use osquery-carve in our reference architecture, we can remove these sections:

• https://github.com/fleetdm/fleet-terraform/blob/main/example/main.tf#L132-L138
• https://github.com/fleetdm/fleet-terraform/blob/main/example/main.tf#L30-L34
• https://github.com/fleetdm/fleet-terraform/blob/main/example/main.tf#L82

Or we could just leave them commented out in the example so they can be optionally added and then remove those lines in the documentation.

Let me know, and I can update the example this is based on to make s3 carves off by default.

Got it. @rachaelshaw @noahtalerman Since carves are not required for every instance, it sounds like we can put this (and anything else like it) at the bottom, simplifying everyday deployment.

[rfairburn]

With fleetdm/fleet-terraform#15 in place, this should be approved.