Skip to content
This repository has been archived by the owner on Feb 14, 2024. It is now read-only.

feat: GWLB integration #227

Merged
merged 7 commits into from
Sep 6, 2023
Merged

feat: GWLB integration #227

merged 7 commits into from
Sep 6, 2023

Conversation

michalbil
Copy link
Contributor

@michalbil michalbil commented Jan 16, 2023

Description

New module for GWLB deployment, support for service chaining in existing loadbalancer, new example with usage demonstration.

Motivation and Context

Resolves #218 .

How Has This Been Tested?

Apply, configure VM-series, verify traffic flow.

Types of changes

  • New feature (non-breaking change which adds functionality)

Checklist

  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes if appropriate.
  • All new and existing tests passed.

@michalbil michalbil requested a review from a team as a code owner January 16, 2023 15:01
Copy link
Contributor

@FoSix FoSix left a comment

Choose a reason for hiding this comment

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

👍🏻

modules/gwlb/versions.tf Outdated Show resolved Hide resolved
modules/gwlb/versions.tf Outdated Show resolved Hide resolved
modules/gwlb/versions.tf Outdated Show resolved Hide resolved
examples/gwlb/example.tfvars Outdated Show resolved Hide resolved
examples/gwlb/variables.tf Outdated Show resolved Hide resolved
modules/gwlb/variables.tf Outdated Show resolved Hide resolved
modules/gwlb/variables.tf Outdated Show resolved Hide resolved
modules/gwlb/variables.tf Show resolved Hide resolved
examples/gwlb/outputs.tf Outdated Show resolved Hide resolved
examples/gwlb/outputs.tf Outdated Show resolved Hide resolved
@michalbil michalbil force-pushed the gwlb-support branch 2 times, most recently from 99f389b to 861f163 Compare January 20, 2023 00:46
@michalbil michalbil requested a review from FoSix January 20, 2023 00:48
Copy link
Contributor

@pimielowski pimielowski left a comment

Choose a reason for hiding this comment

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

Nice work!

examples/gwlb/main.tf Outdated Show resolved Hide resolved
examples/gwlb/main.tf Outdated Show resolved Hide resolved
examples/gwlb/main.tf Outdated Show resolved Hide resolved
@migara
Copy link
Member

migara commented Apr 12, 2023

Waiting to be reviewed by @FoSix that this example is compliant with the structure in #251

@michalbil michalbil force-pushed the gwlb-support branch 2 times, most recently from 07b0db0 to 2d5c31c Compare June 19, 2023 11:18
.github/actions/plan_apply/action.yml Outdated Show resolved Hide resolved
examples/gwlb/example.tfvars Outdated Show resolved Hide resolved
examples/gwlb/example.tfvars Outdated Show resolved Hide resolved
examples/gwlb/example.tfvars Outdated Show resolved Hide resolved
examples/gwlb/outputs.tf Outdated Show resolved Hide resolved
examples/gwlb/variables.tf Outdated Show resolved Hide resolved
@michalbil michalbil requested a review from FoSix July 4, 2023 13:55
Copy link
Member

@migara migara left a comment

Choose a reason for hiding this comment

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

Let's change the name of the example to gwlb_with_vmseries before merging

Copy link

@prisma-cloud-devsecops prisma-cloud-devsecops bot left a comment

Choose a reason for hiding this comment

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

Prisma Cloud has found errors in this PR ⬇️

@@ -64,6 +64,16 @@ resource "azurerm_virtual_machine" "this" {
version = var.custom_image_id == null ? var.img_version : null
}

dynamic "plan" {

Choose a reason for hiding this comment

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

HIGH  Azure instance does not authenticate using SSH keys
    Resource: module.appvm.azurerm_virtual_machine.this | Policy ID: 806079733791502336_AZR_1680031706399 | Checkov ID: CKV_AZURE_1

Description

Refer the documentation for more details, https://docs.bridgecrew.io/docs/bc_azr_networking_1

Copy link
Contributor

Choose a reason for hiding this comment

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

@migara is there a way to skip some checks for this check? we do ignore them already in our checkov config

@michalbil michalbil merged commit 2ef8771 into main Sep 6, 2023
@michalbil michalbil deleted the gwlb-support branch September 6, 2023 12:40
sebastianczech pushed a commit that referenced this pull request Oct 18, 2023
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.

GWLB Integration
4 participants