Skip to content

Conversation

@zackproser
Copy link
Contributor

@zackproser zackproser commented May 18, 2023

Implement RefArch access section. Reworked the sections slightly for simplicity and active voice, but feedback very welcome as always.

Currently implemented:

  1. Ref Arch Auth setup (root users, IAM users)
  2. VPN Authentication

@zackproser
Copy link
Contributor Author

One sec - ripping out the cruft I accidentally shipped in this one.

@zackproser
Copy link
Contributor Author

Should be good to review now 🙇

@zackproser zackproser force-pushed the feature/core-946-ref-arch-access branch from 74140b2 to 1bc2045 Compare May 18, 2023 20:11
Copy link
Contributor

@MoonMoon1919 MoonMoon1919 left a comment

Choose a reason for hiding this comment

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

Everything here looks great. A few nit but all small ones. Great work!

zackproser and others added 11 commits May 19, 2023 11:16
Copy link
Contributor

@MoonMoon1919 MoonMoon1919 left a comment

Choose a reason for hiding this comment

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

A few additional small nits but otherwise looks great!

MoonMoon1919

This comment was marked as duplicate.

@pete0emerson pete0emerson requested a review from MoonMoon1919 May 25, 2023 15:15
@MoonMoon1919 MoonMoon1919 merged commit e1077f0 into example/CORE-861/demonstrate-iac-ia May 26, 2023
@MoonMoon1919 MoonMoon1919 deleted the feature/core-946-ref-arch-access branch May 26, 2023 21:28
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.

7 participants