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

feat(MAJOR): Refactor module vpc, remove module subnet_set #63

Closed
wants to merge 27 commits into from

Conversation

sebastianczech
Copy link
Contributor

@sebastianczech sebastianczech commented Jul 23, 2024

Description

Resolves #62 by:

  • remove subnet_set module
  • extend vpc module to create subnets and route tables
  • simplify subnets creation (not using complex locals from subnet_set module)
  • adjust the variables in variables.tf and outputs inoutputs.tf in order that makes sense
  • refactor variables e.g. create new variable internet_gateway, which defines object with attributes for IGW creation/usage
  • adjust variables description to standard from Major refactor for modules and examples #53
  • add a comment with a link to Terraform Registry before a resource block
  • adjust all examples to new approach with vpc module
  • adjust CloudNGFW examples to changes delivered for other examples in fix(examples): Refactor examples for reference architectures #49
  • change approach for CloudWatch logs group in CloudNGFW (as the same name PaloAltoCloudNGFW has to be used for both examples, groups was created on account used in workflows and in TFVARS we are not creating new log group, we are reusing existing one)

Motivation and Context

#53

How Has This Been Tested?

Code was tested locally and via ChatOps.

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.

@sebastianczech
Copy link
Contributor Author

sebastianczech commented Jul 31, 2024

/idempotence paths="examples/cloudngfw_combined_design examples/cloudngfw_isolated_design"

Testing job ID: 10177600450
Job result: FAILURE

@sebastianczech
Copy link
Contributor Author

sebastianczech commented Jul 31, 2024

/idempotence paths="examples/centralized_design examples/centralized_design_autoscale examples/combined_design examples/combined_design_autoscale examples/isolated_design examples/isolated_design_autoscale examples/panorama_standalone examples/vmseries_standalone"

Testing job ID: 10178193597
Job result: FAILURE

@sebastianczech
Copy link
Contributor Author

sebastianczech commented Jul 31, 2024

/idempotence paths="examples/centralized_design examples/centralized_design_autoscale examples/combined_design examples/combined_design_autoscale examples/isolated_design examples/isolated_design_autoscale examples/panorama_standalone examples/vmseries_standalone"

Testing job ID: 10180773803
Job result: FAILURE
Job result: FAILURE
Job result: FAILURE
Job result: SUCCESS

@sebastianczech
Copy link
Contributor Author

sebastianczech commented Jul 31, 2024

/idempotence paths="examples/cloudngfw_combined_design examples/cloudngfw_isolated_design"

Testing job ID: 10181481847
Job result: FAILURE

@sebastianczech
Copy link
Contributor Author

sebastianczech commented Aug 1, 2024

/idempotence paths="examples/centralized_design examples/centralized_design_autoscale examples/combined_design examples/combined_design_autoscale examples/isolated_design examples/isolated_design_autoscale examples/panorama_standalone examples/vmseries_standalone examples/cloudngfw_combined_design examples/cloudngfw_isolated_design"

Testing job ID: 10193841826
Job result: FAILURE

@sebastianczech
Copy link
Contributor Author

sebastianczech commented Aug 1, 2024

/idempotence paths="examples/centralized_design examples/centralized_design_autoscale examples/combined_design examples/combined_design_autoscale examples/isolated_design examples/isolated_design_autoscale"

Testing job ID: 10194992834
Job result: SUCCESS

@sebastianczech sebastianczech marked this pull request as ready for review August 1, 2024 08:43
@sebastianczech sebastianczech requested a review from a team as a code owner August 1, 2024 08:43
@sebastianczech
Copy link
Contributor Author

sebastianczech commented Aug 1, 2024

/idempotence paths="examples/panorama_standalone examples/vmseries_standalone"

Testing job ID: 10195370346
Job result: SUCCESS

@sebastianczech sebastianczech changed the title feat(module/vpc): Refactor module vpc, remove module subnet_set feat(MAJOR): Refactor module vpc, remove module subnet_set Aug 1, 2024
@sebastianczech
Copy link
Contributor Author

sebastianczech commented Aug 1, 2024

/idempotence paths="examples/cloudngfw_combined_design examples/cloudngfw_isolated_design"

Testing job ID: 10196534585
Job result: SUCCESS

@sebastianczech sebastianczech changed the base branch from main to develop August 1, 2024 10:46
@sebastianczech
Copy link
Contributor Author

As we agreed, I changed destination branch from main to develop , from which we can create release candidates.

@sebastianczech
Copy link
Contributor Author

sebastianczech commented Aug 1, 2024

/plan paths="examples/centralized_design examples/centralized_design_autoscale examples/combined_design examples/combined_design_autoscale examples/isolated_design examples/isolated_design_autoscale examples/panorama_standalone examples/vmseries_standalone examples/cloudngfw_combined_design examples/cloudngfw_isolated_design"

Testing job ID: 10199525688
Job result: SUCCESS

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.

Refactor subnet_set and vpc modules
1 participant