Skip to content
This repository has been archived by the owner on Jul 24, 2023. It is now read-only.

Archive this module #123

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Archive this module #123

wants to merge 2 commits into from

Conversation

RothAndrew
Copy link
Member

@RothAndrew RothAndrew commented Jul 12, 2023

See #122

@RothAndrew
Copy link
Member Author

I requested reviews from the people that I believe are stakeholders in this module, either from a maintenance side or a consumption side. Please let me know if you think there's anyone else who should weigh in here.

I'm thinking this won't happen unless it is unanimously approved by all stakeholders, so that's what I'll be seeking.

@RothAndrew RothAndrew self-assigned this Jul 12, 2023
@ntwkninja
Copy link
Member

agree that Cloud Posse has a very robust / tested suite of terraform modules and they are free to use.

If we're only looking to switch only one module it will carry a degree of CloudPosse opinionation that will see strange in comparison to others.

Recommend we look into a partnership and/or using the modules more broadly if we want to make the switch.

The CloudPosse tool chain is OSS) and could serve as a way to accelerate end-to-end delivery (less for us to maintain).

Cloud Posse info:

Copy link
Member

@ntwkninja ntwkninja left a comment

Choose a reason for hiding this comment

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

the initial concerns I had with the upstream module when we evaluated previously have been eliminated

left some comments for consideration in the PR but no issues with this change

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants