Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dev lb maps #117

Closed
wants to merge 37 commits into from
Closed

Dev lb maps #117

wants to merge 37 commits into from

Conversation

Jakeasaurus
Copy link
Collaborator

Description

New LB module that will allow for the following:

  • ALB and NLB.
  • Target Group.
  • Listeners.
  • Listener rules.
  • Access logs.

NLB and ALB specific resources are created using logic that will use and allow the specific resources that apply to each.

Issue or Ticket

Fixes #110

Type of change

  • Bugfix
  • New feature
  • Version update

Breaking Changes

  • Yes
  • No

Breaking Changes Description

None, should be new functionality not currently in use.

TODOs

  • Validate your code matches the style of the project.
  • Update the docs.
  • Validate all tests run successfull, including pre-commit checks.
  • Include release notes and description. This should include both a summary of the changes and any necessary context.

zachreborn and others added 8 commits January 14, 2025 14:52
* feat: add aws amplify module

* chore: terraform fmt and terraform-docs

* ignore jscpd in the github actions test workflow

---------

Co-authored-by: zachreborn <zachreborn@users.noreply.github.com>
* removed the tfsec scan

* converting nighly scan from tfsec to checkov
* updated igw args to allow false == 0 count

* added new logic for igw

* added two new examples for disabling the igw

* fixed index of the pointers

* corrected variable to enable_internet_gateway

* added type to all vpc variables

* chore: terraform fmt and terraform-docs

---------

Co-authored-by: Jakeasaurus <Jakeasaurus@users.noreply.github.com>
Co-authored-by: Zachary Hill <zhill@octo.ventures>
@Jakeasaurus
Copy link
Collaborator Author

Closing. Please reference older PR that has more context. #112

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.

Module - AWS NLB
2 participants