bottlerocket icon indicating copy to clipboard operation
bottlerocket copied to clipboard

Update `BUILDING.md` and `PUBLISHING-AWS.md` to mention need for aws creds

Open jpmcb opened this issue 2 years ago • 3 comments

Issue number:

Closes #2327

Description of changes:

Update BUILDING.md and PUBLISHING-AWS.md to mention need for aws creds

Several commands referenced in the developer docs interact with AWS
services; AMI, EKS, etc. In order to successfully publish AMIs via the
quickstart, AWS creds must be setup. This patch updates both BUILDING.md
and PUBLISHING-AWS.md to reflect the need to have this setup.

Signed-off-by: John McBride <[email protected]>

Testing done:

N/a

Terms of contribution:

By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.

jpmcb avatar Aug 11 '22 21:08 jpmcb

Fore-pushed to addressed @stmcginnis comments around grammar and formatting

jpmcb avatar Aug 12 '22 14:08 jpmcb

Force-pushed to resolve @zmrow's comment on removing "usually" - great suggestion! 👏🏼

jpmcb avatar Aug 12 '22 17:08 jpmcb

Force-pushed to address Ben's comment on wording in BUILDING.md

jpmcb avatar Aug 16 '22 15:08 jpmcb