HT-8112: Merge upstream for EMR cluster #2
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Initial implementation * Initial implementation * Initial implementation * Initial implementation * Initial implementation * Initial implementation * Initial implementation * Initial implementation * Initial implementation * Initial implementation * Initial implementation * Initial implementation * Initial implementation * Initial implementation * Initial implementation * Initial implementation * Add `subnet_type` var * Add `aws_vpc_endpoint` for private subnets * Add `aws_vpc_endpoint` for private subnets * Add `aws_vpc_endpoint` for private subnets * Add `aws_vpc_endpoint` for private subnets * Find `aws_route_table` by `subnet_id` * Find `aws_route_table` by `subnet_id` * Add `aws_route_table` * Fix route table
* Use EMR default security groups for all managed security groups * Use EMR default security groups for all managed security groups * Use EMR default security groups for all managed security groups
…loudposse#11) * Added application-autoscaling.amazonaws.com as Trusted entities for emr auto scaling role Earlier autoscaling policy is failing to attach with the error Failed to provision the AutoScaling policy: Unable to assume IAM role: arn:aws:iam::216727*****:role/emr-stage-dataorc-emr-ec2-autoscaling this is due to absence of application-autoscaling.amazonaws.com as Trusted entity * Ran terraform fmt * corrected role for autosclaing policy, which requires 'elasticmapreduce.amazonaws.com', 'application-autoscaling.amazonaws.com' as trust entities * New README.md generated from Package terraform-docs already installed Generated README.md from /Users/navdeepagarwal/projects/open-projects/terraform-aws-emr-cluster/build-harness/templates/README.md using data from /Users/navdeepagarwal/projects/open-projects/terraform-aws-emr-cluster/build-harness/templates/README.yaml
* Add `slash-command-dispatch` GitHub Actions workflow * random suffix added to name to avoid s3 bucket problems with existing buckets * seed for rand applied * test fixed * test fixed * Update examples/complete/fixtures.us-east-2.tfvars Co-Authored-By: Erik Osterman <erik@cloudposse.com> Co-authored-by: Erik Osterman <erik@cloudposse.com>
* Fix `log_uri` in example Closes: cloudposse#8 * Updated README.md Co-authored-by: Maxim Mironenko <simixido@gmail.com> Co-authored-by: actions-bot <58130806+actions-bot@users.noreply.github.com>
…rd is used in configuration_json (cloudposse#13) * Add dummy bootstrap action that use md5 of configuration_json as an argument to prevent force recreation * Terratest fixed, empty string as new default for configurations_json * More descriptive comment for changes around bootstrap_action
…udposse#14) * Add `var.create_vpc_endpoint_s3` to control S3 endpoint creation * variable description * Updated README.md Co-authored-by: actions-bot <58130806+actions-bot@users.noreply.github.com>
* Added kerberos_attributes configuration * Readme updated
## What * Adds chatops commands - '/test all' - '/test bats' - '/test readme' - '/test terratest' * Drops codefresh * Drops slash-command-dispatch * Removes codefresh badge * Rebuilds README ## Why * Change over from codefresh to GH Actions * Facilitate testing of PRs from forks
… 0.13 (cloudposse#19) * [AUTOMATED] Update terraform-null-label versions to support Terraform 0.13 * Updated README.md Co-authored-by: actions-bot <58130806+actions-bot@users.noreply.github.com>
…udposse#18) ## What 1. Update Version Pinning for Terraform to support 0.13 ## Why 1. This is a relatively minor update that the CloudPosse module already likely supports. 1. This allows module consumers to not individually update our Terraform module to support Terraform 0.13.
* fix: unquote references in ignore_changes * Updated README.md Co-authored-by: actions-bot <58130806+actions-bot@users.noreply.github.com>
* Adding step concurrency level support * Dropping rogue " * dropping potentially confusing 'default is 1' * Updated README.md Co-authored-by: actions-bot <58130806+actions-bot@users.noreply.github.com>
* fix: add ingress rule from managed master to service * Make ingress rule consistent with other rules * Join security groups ids, instead of using count Co-authored-by: Andriy Knysh <aknysh@users.noreply.github.com> * Join security groups ids, instead of using count Co-authored-by: Andriy Knysh <aknysh@users.noreply.github.com> * Updated README.md Co-authored-by: Andriy Knysh <aknysh@users.noreply.github.com> Co-authored-by: actions-bot <58130806+actions-bot@users.noreply.github.com>
…date GitHub Actions (cloudposse#28) * Update to `context.tf`. Use the latest EMR version in the example. Update GitHub Actions * Update to `context.tf`. Use the latest EMR version in the example. Update GitHub Actions
* Updating for issue number 7 so that we can include steps in the emr cluster * Executed 'terraform fmt' * Updated README.md Co-authored-by: Hannah Amundson <amundson.hannah@heb.com> Co-authored-by: actions-bot <58130806+actions-bot@users.noreply.github.com>
* update emr module to use existing security groups * fix formatting & variable decription based on the review
Co-authored-by: osterman <osterman@users.noreply.github.com>
…ped to 0.13.0 when needed, readme updated (cloudposse#36)
… to v0.12.0 (cloudposse#39) Co-authored-by: Renovate Bot <bot@renovateapp.com>
…posse#38) Co-authored-by: Renovate Bot <bot@renovateapp.com>
* Add resource tags to created IAM roles * Auto Format Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> Co-authored-by: cloudpossebot <11232728+cloudpossebot@users.noreply.github.com>
Co-authored-by: cloudpossebot <11232728+cloudpossebot@users.noreply.github.com>
Co-authored-by: cloudpossebot <11232728+cloudpossebot@users.noreply.github.com>
* Feature: Allow users to pass their own IAM roles * Rename variables following best practices * update wording for variables & remove unused variable
* feat: Added permissions boundary variables for iam roles * feat: Add variable to adjust policy ARNs for use in AWS GovCloud. * fix: type of govcloud set to bool instead of boolean * Auto Format * Auto Format * partition * Auto Format * govcloud partition * Update variables.tf * Auto Format Co-authored-by: nitrocode <nitrocode@users.noreply.github.com> Co-authored-by: cloudpossebot <11232728+cloudpossebot@users.noreply.github.com>
…posse#48) * chore(deps): update terraform cloudposse/label/null to v0.25.0 * Auto Format Co-authored-by: Renovate Bot <bot@renovateapp.com> Co-authored-by: cloudpossebot <11232728+cloudpossebot@users.noreply.github.com>
* Upgrade aws provider to 3.0 * Auto Format * Update versions.tf * Update main.tf * Update versions.tf * Auto Format Co-authored-by: cloudpossebot <11232728+cloudpossebot@users.noreply.github.com>
… to v0.12.2 (cloudposse#47) * chore(deps): update terraform cloudposse/route53-cluster-hostname/aws to v0.12.2 * Auto Format Co-authored-by: Renovate Bot <bot@renovateapp.com> Co-authored-by: cloudpossebot <11232728+cloudpossebot@users.noreply.github.com>
…e#54) * Add missing tags to the aws_iam_instance_profile resource. * Auto Format Co-authored-by: cloudpossebot <11232728+cloudpossebot@users.noreply.github.com>
…dposse#55) * Add support for auto termination policy * Auto Format * Fix null check Co-authored-by: cloudpossebot <11232728+cloudpossebot@users.noreply.github.com>
* Clarify descriptions to indicate secgroup id is required * Auto Format Co-authored-by: cloudpossebot <11232728+cloudpossebot@users.noreply.github.com>
* git-xargs programmatic commit * Auto Format Co-authored-by: cloudpossebot <11232728+cloudpossebot@users.noreply.github.com>
…terns (cloudposse#57) * Update all modules, examples and tests to the latest versions and patterns * Update all modules, examples and tests to the latest versions and patterns * Update all modules, examples and tests to the latest versions and patterns * Update all modules, examples and tests to the latest versions and patterns * Update all modules, examples and tests to the latest versions and patterns * Update all modules, examples and tests to the latest versions and patterns * Update all modules, examples and tests to the latest versions and patterns * Update all modules, examples and tests to the latest versions and patterns * Update all modules, examples and tests to the latest versions and patterns * Update all modules, examples and tests to the latest versions and patterns * Update all modules, examples and tests to the latest versions and patterns * Update all modules, examples and tests to the latest versions and patterns * Update all modules, examples and tests to the latest versions and patterns * Update all modules, examples and tests to the latest versions and patterns * Update all modules, examples and tests to the latest versions and patterns
DesmondH0
reviewed
Aug 18, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes