-
-
Notifications
You must be signed in to change notification settings - Fork 81
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
chore(deps): update terraform cloudposse/label/null to v0.24.1 #38
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
cloudposse-mergebot
approved these changes
Feb 9, 2021
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We've automatically approved this PR because the checks from the automated Pull Request have passed.
LawrenceWarren
added a commit
to humn-ai/tf-mod-aws-emr-cluster
that referenced
this pull request
Aug 18, 2022
* Initial commit * Initial implementation (#1) * 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 * Fix security groups (#2) * 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 * Add `configurations_json`. Fix `bootstrap_action` (#5) * Added application-autoscaling.amazonaws.com as Trusted entity for e… (cloudposse#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 (cloudposse#12) * 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 (cloudposse#10) * 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> * Prevent EMR cluster recreation when javax.jdo.option.ConnectionPassword 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 * Add `var.create_vpc_endpoint_s3` to control S3 endpoint creation (cloudposse#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 (cloudposse#16) * Added kerberos_attributes configuration * Readme updated * Updates to ChatOps - Automated commit (cloudposse#17) ## 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 * [AUTOMATED] Update terraform-null-label versions to support Terraform 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> * [AUTOMATED] Update Version Pinning for Terraform to support 0.13 (cloudposse#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. * Support terraform 0.13 (cloudposse#20) * fix: unquote references in ignore_changes (cloudposse#21) * fix: unquote references in ignore_changes * Updated README.md Co-authored-by: actions-bot <58130806+actions-bot@users.noreply.github.com> * Add support for step_concurrency_level (cloudposse#25) * 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> * Expose ec2 role (cloudposse#27) * fix: add ingress rule from managed master to service (cloudposse#22) * 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> * Update to `context.tf`. Use the latest EMR version in the example. Update 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 * Include Steps option in aws_emr_cluster (cloudposse#30) * 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 (cloudposse#33) * update emr module to use existing security groups * fix formatting & variable decription based on the review * Terraform 0.14 upgrade (cloudposse#34) * Update README.md and docs (cloudposse#37) Co-authored-by: osterman <osterman@users.noreply.github.com> * context.tf updated to v0.24.1, minimum required Terraform version bumped to 0.13.0 when needed, readme updated (cloudposse#36) * chore(deps): update terraform cloudposse/route53-cluster-hostname/aws to v0.12.0 (cloudposse#39) Co-authored-by: Renovate Bot <bot@renovateapp.com> * chore(deps): update terraform cloudposse/label/null to v0.24.1 (cloudposse#38) Co-authored-by: Renovate Bot <bot@renovateapp.com> * Add resource tags to created IAM roles (cloudposse#32) * 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> * Update module to be terraform v0.15 compatible (cloudposse#40) Co-authored-by: cloudpossebot <11232728+cloudpossebot@users.noreply.github.com> * Update context.tf from origin source (cloudposse#43) Co-authored-by: cloudpossebot <11232728+cloudpossebot@users.noreply.github.com> * Feature: Allow users to pass their own IAM roles (cloudposse#44) * Feature: Allow users to pass their own IAM roles * Rename variables following best practices * update wording for variables & remove unused variable * Add permissions boundaries and GovCloud support (cloudposse#42) * 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> * chore(deps): update terraform cloudposse/label/null to v0.25.0 (cloudposse#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 (cloudposse#45) * 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> * chore(deps): update terraform cloudposse/route53-cluster-hostname/aws 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> * Add missing tags to the aws_iam_instance_profile resource. (cloudposse#54) * Add missing tags to the aws_iam_instance_profile resource. * Auto Format Co-authored-by: cloudpossebot <11232728+cloudpossebot@users.noreply.github.com> * Add support for auto termination policy, resolves cloudposse#53 (cloudposse#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 (cloudposse#50) * Clarify descriptions to indicate secgroup id is required * Auto Format Co-authored-by: cloudpossebot <11232728+cloudpossebot@users.noreply.github.com> * git.io->cloudposse.tools update * git-xargs programmatic commit * Auto Format Co-authored-by: cloudpossebot <11232728+cloudpossebot@users.noreply.github.com> * Update all modules, examples and tests to the latest versions and patterns (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 * fix(TF): Delete dead files, update README.md * fix(TF): Update docs * fix(TF): Update config * fix(TF): Delete Makefile * fix(TF): Add Makefile * fix(TF): Cleanup docs * fix(TF): Cleanup docs * fix(TF): Cleanup docs * fix(TF): Add support for `certificate_arn` Co-authored-by: Andriy Knysh <aknysh@users.noreply.github.com> Co-authored-by: Navdeep <navdeep710@gmail.com> Co-authored-by: Maxim Mironenko <maxim@cloudposse.com> Co-authored-by: Erik Osterman <erik@cloudposse.com> Co-authored-by: Yujun Zhang <zhangyujun@gmail.com> Co-authored-by: Maxim Mironenko <simixido@gmail.com> Co-authored-by: actions-bot <58130806+actions-bot@users.noreply.github.com> Co-authored-by: marcin <marcin@cloudposse.com> Co-authored-by: Scott Cabrinha <scott@internaught.io> Co-authored-by: Matt Gowie <gowie.matt@gmail.com> Co-authored-by: Stepan Rakitin <svrakitin@yandex.ru> Co-authored-by: calvin-barker <54291801+calvin-barker@users.noreply.github.com> Co-authored-by: Igor D'Astolfo <i.dastolfo@gmail.com> Co-authored-by: Hannah <48397717+hannahkamundson@users.noreply.github.com> Co-authored-by: Hannah Amundson <amundson.hannah@heb.com> Co-authored-by: Tirumerla <57160285+tirumerla@users.noreply.github.com> Co-authored-by: Cloud Posse Bot (CI/CD) <bot@cloudposse.com> Co-authored-by: osterman <osterman@users.noreply.github.com> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Renovate Bot <bot@renovateapp.com> Co-authored-by: Adam Kunicki <kunickiaj@gmail.com> 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: Shabih <81912045+shabih-paystack@users.noreply.github.com> Co-authored-by: Matt Barclay <mbarclay@gmail.com> Co-authored-by: nitrocode <nitrocode@users.noreply.github.com> Co-authored-by: Paulo Edgar Castro <pauloedgarcastro@gmail.com> Co-authored-by: Szymon Matejczyk <smatejczyk@gmail.com> Co-authored-by: Philipp Erbelding <philipp@copythat.de> Co-authored-by: dylanbannon <dylanbannon5@gmail.com>
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.
This PR contains the following updates:
0.19.2
->0.24.1
Release Notes
cloudposse/terraform-null-label
v0.24.1
Compare Source
Allow control of letter case of outputs @SweetOps (#107)
You now have control over the letter case of generated tag names and supplied labels, which means you also have control over the letter case of the ultimate
id
.Labels are the elements you can include in
label_order
, namelynamespace
,environment
,stage
,name
, andattributes
. For every non-empty label, a corresponding tag name is generated. Fornamespace
,environment
,stage
, the output is the formatted, normalized input. (By "normalized" we mean that it goes throughregex_replace_chars
.), Forattributes
, which is a list, each element is normalized, duplicates are removed, and the resulting list is converted to a string by joining the elements with thedelimiter
(defaults to hyphen). Forname
, which is special, the output is the same asid
, which is the joining of the labels in the order specified bylabel_order
and separated bydelimiter
.label_key_case
to one ofupper
,lower
, ortitle
, which will result in generatedtag
names in the corresponding case:NAME
,name
, orName
. For backwards compatibility,title
is the defaultlabel_value_case
to one ofupper
,lower
,title
, ornone
, which will result in output label values in the corresponding case (withnone
meaning no case conversion of any kind will be done, though the labels will still be subject toregex_replace_chars
). The case converted labels will show up not just in the module output of the labels themselves, but also in thetag
values and in theid
string.You can look at the test cases in
examples/complete
and the expected results intest/src/examples_complete_test.go
to see examples of how this is supposed to work.One interesting example is that you can create
id
s in Pascal case by settinglabel_value_case = "title"
anddelimiter = ""
.Include updates to exports/context.tf @Nuru (#122 and #123)
##### what - Include updates to `exports/context.tf` - Update README with features and compatibilty - Add validation for `id_length_limit` ##### why - The `exports/context.tf` is what gets distributed and needs to be in sync - Replace outdated information - Was not validated earlier because validators are not supported in TF 0.12 but now we are dropping support for TF 0.12 and so we can add validatorsRestore backward compatibility with v0.22.1 and earlier @Nuru (#121)
##### what - Restore backward compatibility with v0.22.1 and earlier - Allow setting of `label_key_case` and `label_value_case` by vars, not just by context attributes. ##### why - Allow interoperability of old and new modules - Normally, root modules make settings via individual variables, not by setting an entire context block.Incorporates and closes #120
v0.24.0
Compare Source
Restore backward compatibility with v0.22.1 and earlier @Nuru (#121)
##### what - Restore backward compatibility with v0.22.1 and earlier - Allow setting of `label_key_case` and `label_value_case` by vars, not just by context attributes. ##### why - Allow interoperability of old and new modules - Normally, root modules make settings via individual variables, not by setting an entire context block.Incorporates and closes #120
Allow control of letter case of outputs @SweetOps (#107)
You now have control over the letter case of generated tag names and supplied labels, which means you also have control over the letter case of the ultimate
id
.Labels are the elements you can include in
label_order
, namelynamespace
,environment
,stage
,name
, andattributes
. For every non-empty label, a corresponding tag name is generated. Fornamespace
,environment
,stage
, the output is the formatted, normalized input. (By "normalized" we mean that it goes throughregex_replace_chars
.), Forattributes
, which is a list, each element is normalized, duplicates are removed, and the resulting list is converted to a string by joining the elements with thedelimiter
(defaults to hyphen). Forname
, which is special, the output is the same asid
, which is the joining of the labels in the order specified bylabel_order
and separated bydelimiter
.label_key_case
to one ofupper
,lower
, ortitle
, which will result in generatedtag
names in the corresponding case:NAME
,name
, orName
. For backwards compatibility,title
is the defaultlabel_value_case
to one ofupper
,lower
,title
, ornone
, which will result in output label values in the corresponding case (withnone
meaning no case conversion of any kind will be done, though the labels will still be subject toregex_replace_chars
). The case converted labels will show up not just in the module output of the labels themselves, but also in thetag
values and in theid
string.You can look at the test cases in
examples/complete
and the expected results intest/src/examples_complete_test.go
to see examples of how this is supposed to work.One interesting example is that you can create
id
s in Pascal case by settinglabel_value_case = "title"
anddelimiter = ""
.v0.23.0
Compare Source
Known issues
null-label
. The canonicalcontext = module.this.context
fails ifmodule.this.context
is an older versioncontext.tf
does not incorporatevar.label_key_case
andvar.label_value_case
into themodule.this
object, preventing those variables from taking effect in the root module'smodule.this
.feat: add support for setting letter case of context tags @SweetOps (#107)
With this release, you gain control over the letter case of generated tag names and supplied labels, which means you also have control over the letter case of the ultimate
id
.Labels are the elements you can include in
label_order
, namelynamespace
,environment
,stage
,name
, andattributes
. For every non-empty label, a corresponding tag name is generated. Fornamespace
,environment
,stage
, the output is the formatted, normalized input. (By "normalized" we mean that it goes throughregex_replace_chars
.), Forattributes
, which is a list, each element is normalized, duplicates are removed, and the resulting list is converted to a string by joining the elements with thedelimiter
(defaults to hyphen). Forname
, which is special, the output is the same asid
, which is the joining of the labels in the order specified bylabel_order
and separated bydelimiter
.label_key_case
to one ofupper
,lower
, ortitle
, which will result in generatedtag
names in the corresponding case:NAME
,name
, orName
. For backwards compatibility,title
is the defaultlabel_value_case
to one ofupper
,lower
,title
, ornone
, which will result in output label values in the corresponding case (withnone
meaning no case conversion of any kind will be done, though the labels will still be subject toregex_replace_chars
). The case converted labels will show up not just in the module output of the labels themselves, but also in thetag
values and in theid
string.You can look at the test cases in
examples/complete
and the expected results intest/src/examples_complete_test.go
to see examples of how this is supposed to work.One interesting example is that you can create
id
s in Pascal case by settinglabel_value_case = "title"
anddelimiter = ""
.v0.22.1
Compare Source
Add var.attributes to end of context.attributes, not vice versa @Nuru (#114)
##### what - Add `var.attributes` to end of `context.attributes`, not vice versa - Update to current workflows (with some exceptions) ##### why - Modules should append to attributes passed in, not insert themselves ahead of others - New features, like auto-format (but holding back some, because this is a special module) ##### references - closes #113 - closes #108v0.22.0
Compare Source
Breaking change: This release updates minimum Terraform version requirement to 0.12.26
Convert context.tf to registry reference @Nuru (#110)
#### what - Convert `context.tf` to use registry reference - Update version requirement to `>= 0.12.26` - Switch auto-publish to draft mode #### why - New standard way to reference modules - Minimum version that supports registry references - Ensure that version numbers are set manually and match what is in `context.tf`v0.21.0
Compare Source
Update `null-label` version used for exporting into other modules @aknysh (#106)
#### what * Update `null-label` version used for exporting into other modules #### why * New release will be made from this PR - use this release in `exports/context.tf` to be exported into other modulesv0.20.0
Compare Source
Update versions.tf to support tf .14 @kiddom-kq (#105)
Adds support for Terraform
0..14
(currently inbeta2
)I've currently got it pinned to
< 0.14.1
to be conservative. Can update to< 0.15
or similar.what
versions.tf
to support terraform0.14
why
0.13
. The proposed workaround for that unrelated issue does not work for me at this time.0.14
which is in beta, now.references
Fixes #104
Renovate configuration
📅 Schedule: At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻️ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR has been generated by WhiteSource Renovate. View repository job log here.