-
Notifications
You must be signed in to change notification settings - Fork 9.1k
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
provider: Converts conns.AWSClient
IgnoreTagsConfig
to function and adds transparent tagging to several data sources
#39761
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
Community NoteVoting for Prioritization
For Submitters
|
github-actions
bot
added
documentation
Introduces or discusses updates to documentation.
tests
PRs: expanded test coverage. Issues: expanded coverage, enhancements to test infrastructure.
service/iam
Issues and PRs that pertain to the iam service.
service/ec2
Issues and PRs that pertain to the ec2 service.
service/route53
Issues and PRs that pertain to the route53 service.
service/elbv2
Issues and PRs that pertain to the elbv2 service.
service/sns
Issues and PRs that pertain to the sns service.
service/directconnect
Issues and PRs that pertain to the directconnect service.
service/codebuild
Issues and PRs that pertain to the codebuild service.
service/acm
Issues and PRs that pertain to the acm service.
service/elb
Issues and PRs that pertain to the elb service.
service/inspector
Issues and PRs that pertain to the inspector service.
service/medialive
Issues and PRs that pertain to the medialive service.
service/autoscaling
Issues and PRs that pertain to the autoscaling service.
service/mq
Issues and PRs that pertain to the mq service.
service/dynamodb
Issues and PRs that pertain to the dynamodb service.
service/efs
Issues and PRs that pertain to the efs service.
service/servicediscovery
Issues and PRs that pertain to the servicediscovery service.
service/servicecatalog
Issues and PRs that pertain to the servicecatalog service.
service/budgets
Issues and PRs that pertain to the budgets service.
service/elasticache
Issues and PRs that pertain to the elasticache service.
service/glue
Issues and PRs that pertain to the glue service.
service/kinesis
Issues and PRs that pertain to the kinesis service.
service/elasticsearch
Issues and PRs that pertain to the elasticsearch service.
service/batch
Issues and PRs that pertain to the batch service.
provider
Pertains to the provider itself, rather than any interaction with AWS.
service/route53domains
Issues and PRs that pertain to the route53domains service.
service/eks
Issues and PRs that pertain to the eks service.
labels
Oct 17, 2024
github-actions
bot
added
service/oam
Issues and PRs that pertain to the oam service.
service/vpclattice
Issues and PRs that pertain to the vpclattice service.
service/meta
Issues and PRs that pertain to the meta service.
prioritized
Part of the maintainer teams immediate focus. To be addressed within the current quarter.
labels
Oct 17, 2024
gdavison
force-pushed
the
td-ignore-tags-config-function
branch
from
October 17, 2024 00:18
bed2f40
to
3d507ff
Compare
ewbankkit
approved these changes
Oct 17, 2024
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.
LGTM 🚀.
% make testacc TESTARGS='-run=TestAccBudgetsBudgetDataSource_' PKG=budgets
make: Verifying source code with gofmt...
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go1.23.2 test ./internal/service/budgets/... -v -count 1 -parallel 20 -run=TestAccBudgetsBudgetDataSource_ -timeout 360m
2024/10/17 08:16:01 Initializing Terraform AWS Provider...
=== RUN TestAccBudgetsBudgetDataSource_tags
=== PAUSE TestAccBudgetsBudgetDataSource_tags
=== RUN TestAccBudgetsBudgetDataSource_tags_NullMap
=== PAUSE TestAccBudgetsBudgetDataSource_tags_NullMap
=== RUN TestAccBudgetsBudgetDataSource_tags_EmptyMap
=== PAUSE TestAccBudgetsBudgetDataSource_tags_EmptyMap
=== RUN TestAccBudgetsBudgetDataSource_tags_DefaultTags_nonOverlapping
=== PAUSE TestAccBudgetsBudgetDataSource_tags_DefaultTags_nonOverlapping
=== RUN TestAccBudgetsBudgetDataSource_tags_IgnoreTags_Overlap_DefaultTag
=== PAUSE TestAccBudgetsBudgetDataSource_tags_IgnoreTags_Overlap_DefaultTag
=== RUN TestAccBudgetsBudgetDataSource_tags_IgnoreTags_Overlap_ResourceTag
=== PAUSE TestAccBudgetsBudgetDataSource_tags_IgnoreTags_Overlap_ResourceTag
=== RUN TestAccBudgetsBudgetDataSource_basic
=== PAUSE TestAccBudgetsBudgetDataSource_basic
=== CONT TestAccBudgetsBudgetDataSource_tags
=== CONT TestAccBudgetsBudgetDataSource_tags_EmptyMap
=== CONT TestAccBudgetsBudgetDataSource_tags_IgnoreTags_Overlap_DefaultTag
=== CONT TestAccBudgetsBudgetDataSource_tags_DefaultTags_nonOverlapping
=== CONT TestAccBudgetsBudgetDataSource_basic
=== CONT TestAccBudgetsBudgetDataSource_tags_NullMap
=== CONT TestAccBudgetsBudgetDataSource_tags_IgnoreTags_Overlap_ResourceTag
--- PASS: TestAccBudgetsBudgetDataSource_tags_NullMap (22.06s)
--- PASS: TestAccBudgetsBudgetDataSource_tags_EmptyMap (22.21s)
--- PASS: TestAccBudgetsBudgetDataSource_tags_DefaultTags_nonOverlapping (22.22s)
--- PASS: TestAccBudgetsBudgetDataSource_tags (22.25s)
--- PASS: TestAccBudgetsBudgetDataSource_tags_IgnoreTags_Overlap_DefaultTag (22.34s)
--- PASS: TestAccBudgetsBudgetDataSource_tags_IgnoreTags_Overlap_ResourceTag (22.98s)
--- PASS: TestAccBudgetsBudgetDataSource_basic (37.36s)
PASS
ok github.com/hashicorp/terraform-provider-aws/internal/service/budgets 42.691s
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
client-connections
Pertains to the AWS Client and service connections.
documentation
Introduces or discusses updates to documentation.
generators
Relates to code generators.
linter
Pertains to changes to or issues with the various linters.
prioritized
Part of the maintainer teams immediate focus. To be addressed within the current quarter.
provider
Pertains to the provider itself, rather than any interaction with AWS.
service/acm
Issues and PRs that pertain to the acm service.
service/apigatewayv2
Issues and PRs that pertain to the apigatewayv2 service.
service/auditmanager
Issues and PRs that pertain to the auditmanager service.
service/autoscaling
Issues and PRs that pertain to the autoscaling service.
service/backup
Issues and PRs that pertain to the backup service.
service/batch
Issues and PRs that pertain to the batch service.
service/budgets
Issues and PRs that pertain to the budgets service.
service/ce
Issues and PRs that pertain to the ce service.
service/codebuild
Issues and PRs that pertain to the codebuild service.
service/codestarconnections
Issues and PRs that pertain to the codestarconnections service.
service/cognitoidp
Issues and PRs that pertain to the cognitoidp service.
service/directconnect
Issues and PRs that pertain to the directconnect service.
service/ds
Issues and PRs that pertain to the ds service.
service/dynamodb
Issues and PRs that pertain to the dynamodb service.
service/ec2
Issues and PRs that pertain to the ec2 service.
service/efs
Issues and PRs that pertain to the efs service.
service/eks
Issues and PRs that pertain to the eks service.
service/elasticache
Issues and PRs that pertain to the elasticache service.
service/elasticsearch
Issues and PRs that pertain to the elasticsearch service.
service/elb
Issues and PRs that pertain to the elb service.
service/elbv2
Issues and PRs that pertain to the elbv2 service.
service/fsx
Issues and PRs that pertain to the fsx service.
service/globalaccelerator
Issues and PRs that pertain to the globalaccelerator service.
service/glue
Issues and PRs that pertain to the glue service.
service/iam
Issues and PRs that pertain to the iam service.
service/imagebuilder
Issues and PRs that pertain to the imagebuilder service.
service/inspector
Issues and PRs that pertain to the inspector service.
service/ipam
Issues and PRs that pertain to the ipam service.
service/ivs
Issues and PRs that pertain to the ivs service.
service/kafka
Issues and PRs that pertain to the kafka service.
service/kendra
Issues and PRs that pertain to the kendra service.
service/kinesis
Issues and PRs that pertain to the kinesis service.
service/location
Issues and PRs that pertain to the location service.
service/medialive
Issues and PRs that pertain to the medialive service.
service/memorydb
Issues and PRs that pertain to the memorydb service.
service/meta
Issues and PRs that pertain to the meta service.
service/mq
Issues and PRs that pertain to the mq service.
service/networkmanager
Issues and PRs that pertain to the networkmanager service.
service/oam
Issues and PRs that pertain to the oam service.
service/opensearch
Issues and PRs that pertain to the opensearch service.
service/opensearchserverless
Issues and PRs that pertain to the opensearchserverless service.
service/outposts
Issues and PRs that pertain to the outposts service.
service/qldb
Issues and PRs that pertain to the qldb service.
service/quicksight
Issues and PRs that pertain to the quicksight service.
service/route53domains
Issues and PRs that pertain to the route53domains service.
service/route53resolver
Issues and PRs that pertain to the route53resolver service.
service/route53
Issues and PRs that pertain to the route53 service.
service/servicecatalog
Issues and PRs that pertain to the servicecatalog service.
service/servicediscovery
Issues and PRs that pertain to the servicediscovery service.
service/signer
Issues and PRs that pertain to the signer service.
service/sns
Issues and PRs that pertain to the sns service.
service/ssmcontacts
Issues and PRs that pertain to the ssmcontacts service.
service/ssmincidents
Issues and PRs that pertain to the ssmincidents service.
service/ssoadmin
Issues and PRs that pertain to the ssoadmin service.
service/transfer
Issues and PRs that pertain to the transfer service.
service/vpc
Issues and PRs that pertain to the vpc service.
service/vpclattice
Issues and PRs that pertain to the vpclattice service.
skaff
Issues and pull requested related to the skaff tool
tests
PRs: expanded test coverage. Issues: expanded coverage, enhancements to test infrastructure.
verify
Pertains to the verify package (i.e., provider-level validating, diff suppression, etc.)
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.
Description
Converts
conns.AWSClient
IgnoreTagsConfig
to function and adds transparent tagging to several data sources. Adds tagging tests as needed