d/aws_batch_job_definition #34663

danquack · 2023-11-30T18:59:47Z

Description

Creates a data source for batch_job_definition.

Relations

References

Output from Acceptance Testing

% make testacc TESTS="TestAccBatchJobDefinitionDataSource" PKG=batch                        
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./internal/service/batch/... -v -count 1 -parallel 20 -run='TestAccBatchJobDefinitionDataSource'  -timeout 360m
=== RUN   TestAccBatchJobDefinitionDataSource_basicName
=== PAUSE TestAccBatchJobDefinitionDataSource_basicName
=== RUN   TestAccBatchJobDefinitionDataSource_basicARN
=== PAUSE TestAccBatchJobDefinitionDataSource_basicARN
=== RUN   TestAccBatchJobDefinitionDataSource_basicARN_NodeProperties
=== PAUSE TestAccBatchJobDefinitionDataSource_basicARN_NodeProperties
=== RUN   TestAccBatchJobDefinitionDataSource_basicARN_EKSProperties
=== PAUSE TestAccBatchJobDefinitionDataSource_basicARN_EKSProperties
=== CONT  TestAccBatchJobDefinitionDataSource_basicName
=== CONT  TestAccBatchJobDefinitionDataSource_basicARN_NodeProperties
=== CONT  TestAccBatchJobDefinitionDataSource_basicARN
=== CONT  TestAccBatchJobDefinitionDataSource_basicARN_EKSProperties
--- PASS: TestAccBatchJobDefinitionDataSource_basicARN_EKSProperties (16.03s)
--- PASS: TestAccBatchJobDefinitionDataSource_basicARN_NodeProperties (17.46s)
--- PASS: TestAccBatchJobDefinitionDataSource_basicName (26.61s)
--- PASS: TestAccBatchJobDefinitionDataSource_basicARN (26.78s)
PASS
ok  	github.com/hashicorp/terraform-provider-aws/internal/service/batch	28.724s

github-actions · 2023-11-30T19:00:07Z

Community Note

Voting for Prioritization

Please vote on this pull request by adding a 👍 reaction to the original post to help the community and maintainers prioritize this pull request.
Please see our prioritization guide for information on how we prioritize.
Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request.

For Submitters

Review the contribution guide relating to the type of change you are making to ensure all of the necessary steps have been taken.
For new resources and data sources, use skaff to generate scaffolding with comments detailing common expectations.
Whether or not the branch has been rebased will not impact prioritization, but doing so is always a welcome surprise.

github-actions · 2023-11-30T19:00:36Z

Thank you for your contribution! 🚀

Please note that typically Go dependency changes are handled in this repository by dependabot or the maintainers. This is to prevent pull request merge conflicts and further delay reviews of contributions. Remove any changes to the go.mod or go.sum files and commit them into this pull request.

Additional details:

Check open pull requests with the dependencies label to view other dependency updates.
If this pull request includes an update the AWS Go SDK (or any other dependency) version, only updates submitted via dependabot will be merged. This pull request will need to remove these changes and will need to be rebased after the existing dependency update via dependabot has been merged for this pull request to be reviewed.
If this pull request is for supporting a new AWS service:
- Ensure the new AWS service changes are following the Contributing Guide section on new services, in particular that the dependency addition and initial provider support are in a separate pull request from other changes (e.g. new resources). Contributions not following this item will not be reviewed until the changes are split.
- If this pull request is already a separate pull request from the above item, you can ignore this message.

drewmullen

1 requested change

internal/service/batch/job_definition_data_source.go

drewmullen

LGTM!

make testacc TESTS="TestAccBatchJobDefinitionDataSource" PKG=batch  
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./internal/service/batch/... -v -count 1 -parallel 20 -run='TestAccBatchJobDefinitionDataSource'  -timeout 360m
=== RUN   TestAccBatchJobDefinitionDataSource_basicName
=== PAUSE TestAccBatchJobDefinitionDataSource_basicName
=== RUN   TestAccBatchJobDefinitionDataSource_basicARN
=== PAUSE TestAccBatchJobDefinitionDataSource_basicARN
=== RUN   TestAccBatchJobDefinitionDataSource_basicARN_NodeProperties
=== PAUSE TestAccBatchJobDefinitionDataSource_basicARN_NodeProperties
=== RUN   TestAccBatchJobDefinitionDataSource_basicARN_EKSProperties
=== PAUSE TestAccBatchJobDefinitionDataSource_basicARN_EKSProperties
=== CONT  TestAccBatchJobDefinitionDataSource_basicName
=== CONT  TestAccBatchJobDefinitionDataSource_basicARN_NodeProperties
=== CONT  TestAccBatchJobDefinitionDataSource_basicARN
=== CONT  TestAccBatchJobDefinitionDataSource_basicARN_EKSProperties
--- PASS: TestAccBatchJobDefinitionDataSource_basicARN_EKSProperties (41.02s)
--- PASS: TestAccBatchJobDefinitionDataSource_basicARN_NodeProperties (42.70s)
--- PASS: TestAccBatchJobDefinitionDataSource_basicName (57.54s)
--- PASS: TestAccBatchJobDefinitionDataSource_basicARN (67.12s)
PASS
ok      github.com/hashicorp/terraform-provider-aws/internal/service/batch      69.494s

Co-authored-by: Drew Mullen <[email protected]>

…mod tidy'.

ewbankkit

LGTM 🚀.

% make testacc TESTARGS='-run=TestAccBatchJobDefinitionDataSource_' PKG=batch ACCTEST_PARALLELISM=2
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./internal/service/batch/... -v -count 1 -parallel 2  -run=TestAccBatchJobDefinitionDataSource_ -timeout 360m
=== RUN   TestAccBatchJobDefinitionDataSource_basicName
=== PAUSE TestAccBatchJobDefinitionDataSource_basicName
=== RUN   TestAccBatchJobDefinitionDataSource_basicARN
=== PAUSE TestAccBatchJobDefinitionDataSource_basicARN
=== RUN   TestAccBatchJobDefinitionDataSource_basicARN_NodeProperties
=== PAUSE TestAccBatchJobDefinitionDataSource_basicARN_NodeProperties
=== RUN   TestAccBatchJobDefinitionDataSource_basicARN_EKSProperties
=== PAUSE TestAccBatchJobDefinitionDataSource_basicARN_EKSProperties
=== CONT  TestAccBatchJobDefinitionDataSource_basicName
=== CONT  TestAccBatchJobDefinitionDataSource_basicARN_EKSProperties
--- PASS: TestAccBatchJobDefinitionDataSource_basicARN_EKSProperties (17.97s)
=== CONT  TestAccBatchJobDefinitionDataSource_basicARN_NodeProperties
--- PASS: TestAccBatchJobDefinitionDataSource_basicName (30.52s)
=== CONT  TestAccBatchJobDefinitionDataSource_basicARN
--- PASS: TestAccBatchJobDefinitionDataSource_basicARN_NodeProperties (16.76s)
--- PASS: TestAccBatchJobDefinitionDataSource_basicARN (26.84s)
PASS
ok  	github.com/hashicorp/terraform-provider-aws/internal/service/batch	68.476s

# Conflicts: # internal/service/batch/service_endpoints_gen_test.go

ewbankkit · 2024-02-19T20:15:51Z

@danquack Thanks for the contribution 🎉 👏.

danquack · 2024-02-19T21:07:30Z

I noticed you switched the data source from using Single Object -> List to work with autoflex. By switching to lists, should these tests also work without an index? One of the reasons I did object originally was for the purity of not having to do .0, but that may be something we want to do with the provider (#35813).

terraform-provider-aws/internal/service/batch/job_definition_data_source_test.go

Lines 132 to 133 in 4a89bec

    
           resource.TestCheckResourceAttr(dataSourceName, "eks_properties.pod_properties.containers.#", "1"), 
        
           resource.TestCheckResourceAttr(dataSourceName, "eks_properties.pod_properties.containers.0.image", "public.ecr.aws/amazonlinux/amazonlinux:1"),

Also, when I run the test command I am seeing some panics in the auto flexing. I see yours passed, but maybe it is something in my local development setup, just borked?

make testacc TESTARGS='-run=TestAccBatchJobDefinitionDataSource_' PKG=batch ACCTEST_PARALLELISM=2
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./internal/service/batch/... -v -count 1 -parallel 2  -run=TestAccBatchJobDefinitionDataSource_ -timeout 360m
=== RUN   TestAccBatchJobDefinitionDataSource_basicName
=== PAUSE TestAccBatchJobDefinitionDataSource_basicName
=== RUN   TestAccBatchJobDefinitionDataSource_basicARN
=== PAUSE TestAccBatchJobDefinitionDataSource_basicARN
=== RUN   TestAccBatchJobDefinitionDataSource_basicARN_NodeProperties
=== PAUSE TestAccBatchJobDefinitionDataSource_basicARN_NodeProperties
=== RUN   TestAccBatchJobDefinitionDataSource_basicARN_EKSProperties
=== PAUSE TestAccBatchJobDefinitionDataSource_basicARN_EKSProperties
=== CONT  TestAccBatchJobDefinitionDataSource_basicName
=== CONT  TestAccBatchJobDefinitionDataSource_basicARN_NodeProperties
=== NAME  TestAccBatchJobDefinitionDataSource_basicName
    job_definition_data_source_test.go:23: Step 1/2 error: Check failed: Check 3/3 error: data.aws_batch_job_definition.test: Attribute 'retry_strategy.attempts' not found
=== NAME  TestAccBatchJobDefinitionDataSource_basicARN_NodeProperties
    job_definition_data_source_test.go:91: Step 1/1 error: Check failed: Check 1/3 error: data.aws_batch_job_definition.test: Attribute 'node_properties.main_node' not found
--- FAIL: TestAccBatchJobDefinitionDataSource_basicName (10.53s)
=== CONT  TestAccBatchJobDefinitionDataSource_basicARN_EKSProperties
--- FAIL: TestAccBatchJobDefinitionDataSource_basicARN_NodeProperties (10.59s)
=== CONT  TestAccBatchJobDefinitionDataSource_basicARN
panic: Value Conversion Error

An unexpected error was encountered trying to convert from struct value. This is always an error in the provider. Please report the following to the provider developer:

error retrieving field names from struct tags: [0].limits: can't use field name for both Limits and Requests
[0]

goroutine 10330 [running]:
github.com/hashicorp/terraform-provider-aws/internal/errs.Must[...](...)
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/errs/must.go:13
github.com/hashicorp/terraform-provider-aws/internal/errs/fwdiag.Must[...](...)
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/errs/fwdiag/must.go:17
github.com/hashicorp/terraform-provider-aws/internal/framework/types.newListNestedObjectValueOf[...]({0x113262c30, 0x14004633860}, {0x110558960, 0x14002d68de0})
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/framework/types/list_nested_objectof.go:236 +0x138
github.com/hashicorp/terraform-provider-aws/internal/framework/types.NewListNestedObjectValueOfSlice[...](...)
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/framework/types/list_nested_objectof.go:228
github.com/hashicorp/terraform-provider-aws/internal/framework/types.NewListNestedObjectValueOfPtr[...](...)
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/framework/types/list_nested_objectof.go:224
github.com/hashicorp/terraform-provider-aws/internal/framework/types.listNestedObjectTypeOf[...].ValueFromObjectPtr(0x1132e1280?, {0x113262c30?, 0x14004633860}, {0x110510600, 0x14002d4b180})
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/framework/types/list_nested_objectof.go:115 +0xd4
github.com/hashicorp/terraform-provider-aws/internal/framework/flex.autoFlattener.structToNestedObject({}, {0x113262c30, 0x14004633860}, {0x11234ea20?, 0x14002d68b10?, 0x140036247e8?}, 0xc4?, {0x1295abcd8, 0x14004f47930}, {0x112d84860?, ...})
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/framework/flex/auto_flatten.go:721 +0x22c
github.com/hashicorp/terraform-provider-aws/internal/framework/flex.autoFlattener.struct_({}, {0x113262c30, 0x14004633860}, {0x11234ea20?, 0x14002d68b10?, 0x14004f47920?}, 0x68?, {0x113278ad8?, 0x14004f47930?}, {0x112d84860?, ...})
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/framework/flex/auto_flatten.go:304 +0x20c
github.com/hashicorp/terraform-provider-aws/internal/framework/flex.autoFlattener.ptr({}, {0x113262c30, 0x14004633860}, {0x1111436e0?, 0x1400b3b5ad0?, 0x1132d85e8?}, {0x113278ad8?, 0x14004f47930?}, {0x112d84860?, 0x1400b5f9bd8?, ...})
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/framework/flex/auto_flatten.go:284 +0x3d4
github.com/hashicorp/terraform-provider-aws/internal/framework/flex.autoFlattener.convert({}, {0x113262c30, 0x14004633860}, {0x1111436e0?, 0x1400b3b5ad0?, 0x1400acae780?}, {0x112d84860?, 0x1400b5f9bd8?, 0x102ee53a0?})
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/framework/flex/auto_flatten.go:76 +0x250
github.com/hashicorp/terraform-provider-aws/internal/framework/flex.autoFlexConvertStruct({0x113262c30, 0x14004633860}, {0x112d8f700?, 0x1400b3b5a70?}, {0x1105106c0?, 0x1400b5f9b00?}, {0x1130dea00, 0x11aff1940})
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/framework/flex/autoflex.go:110 +0x2c4
github.com/hashicorp/terraform-provider-aws/internal/framework/flex.autoFlattener.sliceOfStructNestedObjectCollection({}, {0x113262c30, 0x14004633860}, {0x1105614e0?, 0x1400d950f00?, 0x1121355c0?}, {0x1295abba8, 0x14004f478c0}, {0x112d84c80?, 0x14002cf4480?, ...})
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/framework/flex/auto_flatten.go:762 +0x42c
github.com/hashicorp/terraform-provider-aws/internal/framework/flex.autoFlattener.slice({}, {0x113262c30, 0x14004633860}, {0x1105614e0?, 0x1400d950f00?, 0x1132d85e8?}, {0x113278778?, 0x14004f478c0}, {0x112d84c80?, 0x14002cf4480?, ...})
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/framework/flex/auto_flatten.go:489 +0x450
github.com/hashicorp/terraform-provider-aws/internal/framework/flex.autoFlattener.convert({}, {0x113262c30, 0x14004633860}, {0x1105614e0?, 0x1400d950f00?, 0x1400acae648?}, {0x112d84c80?, 0x14002cf4480?, 0x102ee53a0?})
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/framework/flex/auto_flatten.go:80 +0x2a4
github.com/hashicorp/terraform-provider-aws/internal/framework/flex.autoFlexConvertStruct({0x113262c30, 0x14004633860}, {0x112bd4820?, 0x1400d950f00?}, {0x110510840?, 0x14002cf4480?}, {0x1130dea00, 0x11aff1940})
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/framework/flex/autoflex.go:110 +0x2c4
github.com/hashicorp/terraform-provider-aws/internal/framework/flex.autoFlattener.structToNestedObject({}, {0x113262c30, 0x14004633860}, {0x112bd4820?, 0x1400d950cc0?, 0x111b68800?}, 0x0?, {0x1295abb48, 0x14004f47830}, {0x112d854c0?, ...})
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/framework/flex/auto_flatten.go:715 +0x190
github.com/hashicorp/terraform-provider-aws/internal/framework/flex.autoFlattener.struct_({}, {0x113262c30, 0x14004633860}, {0x112bd4820?, 0x1400d950cc0?, 0x112d85360?}, 0xa9?, {0x1132780b8?, 0x14004f47830?}, {0x112d854c0?, ...})
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/framework/flex/auto_flatten.go:304 +0x20c
github.com/hashicorp/terraform-provider-aws/internal/framework/flex.autoFlattener.ptr({}, {0x113262c30, 0x14004633860}, {0x111143b00?, 0x14004f47730?, 0x1132d85e8?}, {0x1132780b8?, 0x14004f47830?}, {0x112d854c0?, 0x14006ee6de0?, ...})
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/framework/flex/auto_flatten.go:284 +0x3d4
github.com/hashicorp/terraform-provider-aws/internal/framework/flex.autoFlattener.convert({}, {0x113262c30, 0x14004633860}, {0x111143b00?, 0x14004f47730?, 0x1400acae510?}, {0x112d854c0?, 0x14006ee6de0?, 0x102ee53a0?})
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/framework/flex/auto_flatten.go:76 +0x250
github.com/hashicorp/terraform-provider-aws/internal/framework/flex.autoFlexConvertStruct({0x113262c30, 0x14004633860}, {0x11206bf20?, 0x14004f47730?}, {0x110510880?, 0x14006ee6de0?}, {0x1130dea00, 0x11aff1940})
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/framework/flex/autoflex.go:110 +0x2c4
github.com/hashicorp/terraform-provider-aws/internal/framework/flex.autoFlattener.structToNestedObject({}, {0x113262c30, 0x14004633860}, {0x11206bf20?, 0x14004f47570?, 0x14004f47701?}, 0x28?, {0x129479e90, 0x14004f47720}, {0x112d85620?, ...})
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/framework/flex/auto_flatten.go:715 +0x190
github.com/hashicorp/terraform-provider-aws/internal/framework/flex.autoFlattener.struct_({}, {0x113262c30, 0x14004633860}, {0x11206bf20?, 0x14004f47570?, 0x111b68800?}, 0x40?, {0x113277f98?, 0x14004f47720?}, {0x112d85620?, ...})
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/framework/flex/auto_flatten.go:304 +0x20c
github.com/hashicorp/terraform-provider-aws/internal/framework/flex.autoFlattener.ptr({}, {0x113262c30, 0x14004633860}, {0x111143c20?, 0x140072b2df8?, 0x0?}, {0x113277f98?, 0x14004f47720?}, {0x112d85620?, 0x14007644830?, ...})
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/framework/flex/auto_flatten.go:284 +0x3d4
github.com/hashicorp/terraform-provider-aws/internal/framework/flex.autoFlattener.convert({}, {0x113262c30, 0x14004633860}, {0x111143c20?, 0x140072b2df8?, 0x1400acae3c8?}, {0x112d85620?, 0x14007644830?, 0x102ee5300?})
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/framework/flex/auto_flatten.go:76 +0x250
github.com/hashicorp/terraform-provider-aws/internal/framework/flex.autoFlexConvertStruct({0x113262c30, 0x14004633860}, {0x111144160?, 0x140072b2dc0?}, {0x1105108c0?, 0x14007644800?}, {0x1130dea20, 0x11aff1940})
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/framework/flex/autoflex.go:110 +0x2c4
github.com/hashicorp/terraform-provider-aws/internal/framework/flex.autoFlexConvert({0x113262c30, 0x14004633860}, {0x111144160, 0x140072b2dc0}, {0x1105108c0, 0x14007644800}, {0x1130dea20?, 0x11aff1940?})
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/framework/flex/autoflex.go:48 +0x1c8
github.com/hashicorp/terraform-provider-aws/internal/framework/flex.Flatten({0x113262c30, 0x14004633860}, {0x111144160, 0x140072b2dc0}, {0x1105108c0, 0x14007644800}, {0x0, 0x0, 0x1105108c0?})
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/framework/flex/auto_flatten.go:36 +0xb0
github.com/hashicorp/terraform-provider-aws/internal/service/batch.(*jobDefinitionDataSource).Read(0x14006d9fc30, {0x113262c30, 0x14004633860}, {{{{0x113292ea0, 0x14006fbb740}, {0x11138c440, 0x140041ff6b0}}, {0x1132a21d8, 0x14005a372c0}}, {{{0x0, ...}, ...}, ...}}, ...)
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/service/batch/job_definition_data_source.go:193 +0x5cc
github.com/hashicorp/terraform-provider-aws/internal/provider/fwprovider.(*wrappedDataSource).Read(0x14002d4a800, {0x113262c30?, 0x1400508b800?}, {{{{0x113292ea0, 0x14006fbb740}, {0x11138c440, 0x140041ff6b0}}, {0x1132a21d8, 0x14005a372c0}}, {{{0x0, ...}, ...}, ...}}, ...)
	/Users/danielquackenbush/code/aws/terraform-provider-aws/internal/provider/fwprovider/intercept.go:173 +0x100
github.com/hashicorp/terraform-plugin-framework/internal/fwserver.(*Server).ReadDataSource(0x14005190680, {0x113262c30, 0x1400508b800}, 0x1400508b830, 0x14003627608)
	/Users/danielquackenbush/go/pkg/mod/github.com/hashicorp/[email protected]/internal/fwserver/server_readdatasource.go:79 +0x394
github.com/hashicorp/terraform-plugin-framework/internal/proto5server.(*Server).ReadDataSource(0x14005190680, {0x113262c30?, 0x1400508b560?}, 0x140084d9240)
	/Users/danielquackenbush/go/pkg/mod/github.com/hashicorp/[email protected]/internal/proto5server/server_readdatasource.go:55 +0x31c
github.com/hashicorp/terraform-plugin-mux/tf5muxserver.(*muxServer).ReadDataSource(0x113262c68?, {0x113262c30?, 0x1400508b290?}, 0x140084d9240)
	/Users/danielquackenbush/go/pkg/mod/github.com/hashicorp/[email protected]/tf5muxserver/mux_server_ReadDataSource.go:36 +0x18c
github.com/hashicorp/terraform-plugin-go/tfprotov5/tf5server.(*server).ReadDataSource(0x1400b7d41e0, {0x113262c30?, 0x1400508aab0?}, 0x14001ff41e0)
	/Users/danielquackenbush/go/pkg/mod/github.com/hashicorp/[email protected]/tfprotov5/tf5server/server.go:685 +0x344
github.com/hashicorp/terraform-plugin-go/tfprotov5/internal/tfplugin5._Provider_ReadDataSource_Handler({0x112e280e0?, 0x1400b7d41e0}, {0x113262c30, 0x1400508aab0}, 0x1400538f300, 0x0)
	/Users/danielquackenbush/go/pkg/mod/github.com/hashicorp/[email protected]/tfprotov5/internal/tfplugin5/tfplugin5_grpc.pb.go:572 +0x164
google.golang.org/grpc.(*Server).processUnaryRPC(0x1400652c000, {0x113262c30, 0x1400508aa20}, {0x113295740, 0x1400269b380}, 0x14002cf9680, 0x1400731ddd0, 0x11af99660, 0x0)
	/Users/danielquackenbush/go/pkg/mod/google.golang.org/[email protected]/server.go:1385 +0xb8c
google.golang.org/grpc.(*Server).handleStream(0x1400652c000, {0x113295740, 0x1400269b380}, 0x14002cf9680)
	/Users/danielquackenbush/go/pkg/mod/google.golang.org/[email protected]/server.go:1796 +0xc44
google.golang.org/grpc.(*Server).serveStreams.func2.1()
	/Users/danielquackenbush/go/pkg/mod/google.golang.org/[email protected]/server.go:1029 +0x8c
created by google.golang.org/grpc.(*Server).serveStreams.func2 in goroutine 9254
	/Users/danielquackenbush/go/pkg/mod/google.golang.org/[email protected]/server.go:1040 +0x150
FAIL	github.com/hashicorp/terraform-provider-aws/internal/service/batch	20.710s
FAIL
make: *** [testacc] Error 1

ewbankkit · 2024-02-19T21:14:01Z

@danquack We want to keep to stick to "list of objects" rather than just "objects" to maintain compatibility with data source written using Plugin SDK v2, plus we can share the "model" structures used for AutoFlEx with the corresponding resource when we (eventually) migrate it.
The tests should all pass if you the latest commits.

ewbankkit · 2024-02-19T21:15:19Z

Actually, something seems to have gone awry with that last merge. They were working...

ewbankkit · 2024-02-19T21:17:26Z

374458a fixes those final problems. Thanks for spotting that 👏.

% make testacc TESTARGS='-run=TestAccBatchJobDefinitionDataSource_' PKG=batch ACCTEST_PARALLELISM=2 
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./internal/service/batch/... -v -count 1 -parallel 2  -run=TestAccBatchJobDefinitionDataSource_ -timeout 360m
=== RUN   TestAccBatchJobDefinitionDataSource_basicName
=== PAUSE TestAccBatchJobDefinitionDataSource_basicName
=== RUN   TestAccBatchJobDefinitionDataSource_basicARN
=== PAUSE TestAccBatchJobDefinitionDataSource_basicARN
=== RUN   TestAccBatchJobDefinitionDataSource_basicARN_NodeProperties
=== PAUSE TestAccBatchJobDefinitionDataSource_basicARN_NodeProperties
=== RUN   TestAccBatchJobDefinitionDataSource_basicARN_EKSProperties
=== PAUSE TestAccBatchJobDefinitionDataSource_basicARN_EKSProperties
=== CONT  TestAccBatchJobDefinitionDataSource_basicName
=== CONT  TestAccBatchJobDefinitionDataSource_basicARN_NodeProperties
--- PASS: TestAccBatchJobDefinitionDataSource_basicARN_NodeProperties (19.57s)
=== CONT  TestAccBatchJobDefinitionDataSource_basicARN
--- PASS: TestAccBatchJobDefinitionDataSource_basicName (34.06s)
=== CONT  TestAccBatchJobDefinitionDataSource_basicARN_EKSProperties
--- PASS: TestAccBatchJobDefinitionDataSource_basicARN (34.60s)
--- PASS: TestAccBatchJobDefinitionDataSource_basicARN_EKSProperties (20.60s)
PASS
ok  	github.com/hashicorp/terraform-provider-aws/internal/service/batch	68.058s

github-actions · 2024-02-23T12:43:31Z

This functionality has been released in v5.38.0 of the Terraform AWS Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you!

github-actions · 2024-03-25T02:06:02Z

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

terraform-aws-provider bot added the needs-triage Waiting for first response or review from a maintainer. label Nov 30, 2023

danquack force-pushed the 28665 branch 2 times, most recently from 64cee84 to 9543780 Compare December 5, 2023 18:37

justinretzolk added new-data-source Introduces a new data source. and removed needs-triage Waiting for first response or review from a maintainer. labels Dec 6, 2023

This was referenced Dec 12, 2023

Add ability to set EKS properties on batch #34892

Closed

Add ability to set EKS properties on batch #34931

Merged

danquack force-pushed the 28665 branch 2 times, most recently from ea2c1c8 to ad586a2 Compare December 20, 2023 00:10

danquack marked this pull request as ready for review December 20, 2023 00:12

danquack force-pushed the 28665 branch 2 times, most recently from 90e2860 to fdfd8b0 Compare December 20, 2023 16:02

drewmullen suggested changes Jan 3, 2024

View reviewed changes

internal/service/batch/job_definition_data_source.go Outdated Show resolved Hide resolved

drewmullen approved these changes Jan 4, 2024

View reviewed changes

drewmullen mentioned this pull request Jan 4, 2024

batch job definition remove force new and increment revisions #35149

Merged

danquack force-pushed the 28665 branch from 7ae6abd to c3c5539 Compare January 18, 2024 13:50

danquack and others added 6 commits February 15, 2024 09:14

Batch datasource

c15347c

Co-authored-by: Drew Mullen <[email protected]>

CHANGELOG

62e943a

CHANGELOG

cf2e274

Add missing annotation for framework data source

32a563a

🧹 linting

1b09676

semgrep ignore

d6dbd98

danquack added 2 commits February 15, 2024 09:14

more linting

0d685d9

arn_prefix

4d38da8

danquack force-pushed the 28665 branch from c3c5539 to 4d38da8 Compare February 15, 2024 14:14

go generate call

2cc52d9

ewbankkit self-assigned this Feb 15, 2024

terraform-aws-provider bot added the prioritized Part of the maintainer teams immediate focus. To be addressed within the current quarter. label Feb 15, 2024

ewbankkit added 7 commits February 15, 2024 11:43

Merge branch 'main' into HEAD

e000e3f

Run 'go get github.com/aws/aws-sdk-go-v2/service/[email protected] && go …

f205f98

…mod tidy'.

Tweak CHANGELOG entry.

8923b62

d/aws_batch_job_definition: Add structures in preparation for AutoFlEx.

8b8a444

d/aws_batch_job_definition: Another step to AutoFlEx.

102ba10

d/aws_batch_job_definition: AutoFlEx.

a9c1061

d/aws_batch_job_definition: Set 'arn_prefix' and 'tags'.

1c301ba

ewbankkit approved these changes Feb 19, 2024

View reviewed changes

ewbankkit added 2 commits February 19, 2024 15:09

d/aws_batch_job_definition: Get acceptance tests passing.

2505938

Merge branch 'main' into HEAD

a82360f

# Conflicts: # internal/service/batch/service_endpoints_gen_test.go

Fix golangci-lint 'revive/receiver-naming'.

4a89bec

Merge commit '2505938303' into HEAD

374458a

ewbankkit merged commit 7ea9619 into hashicorp:main Feb 19, 2024
50 checks passed

github-actions bot added this to the v5.38.0 milestone Feb 19, 2024

github-actions bot removed the prioritized Part of the maintainer teams immediate focus. To be addressed within the current quarter. label Feb 23, 2024

harto mentioned this pull request Mar 7, 2024

[Enhancement]: Add container properties attribute to aws_batch_job_definition data source #36263

Open

github-actions bot locked as resolved and limited conversation to collaborators Mar 25, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

d/aws_batch_job_definition #34663

d/aws_batch_job_definition #34663

danquack commented Nov 30, 2023 •

edited

Loading

github-actions bot commented Nov 30, 2023

github-actions bot commented Nov 30, 2023

drewmullen left a comment

drewmullen left a comment

ewbankkit left a comment

ewbankkit commented Feb 19, 2024

danquack commented Feb 19, 2024

ewbankkit commented Feb 19, 2024 •

edited

Loading

ewbankkit commented Feb 19, 2024

ewbankkit commented Feb 19, 2024 •

edited

Loading

github-actions bot commented Feb 23, 2024

github-actions bot commented Mar 25, 2024

d/aws_batch_job_definition #34663

d/aws_batch_job_definition #34663

Conversation

danquack commented Nov 30, 2023 • edited Loading

Description

Relations

References

Output from Acceptance Testing

github-actions bot commented Nov 30, 2023

Community Note

github-actions bot commented Nov 30, 2023

drewmullen left a comment

Choose a reason for hiding this comment

drewmullen left a comment

Choose a reason for hiding this comment

ewbankkit left a comment

Choose a reason for hiding this comment

ewbankkit commented Feb 19, 2024

danquack commented Feb 19, 2024

ewbankkit commented Feb 19, 2024 • edited Loading

ewbankkit commented Feb 19, 2024

ewbankkit commented Feb 19, 2024 • edited Loading

github-actions bot commented Feb 23, 2024

github-actions bot commented Mar 25, 2024

danquack commented Nov 30, 2023 •

edited

Loading

ewbankkit commented Feb 19, 2024 •

edited

Loading

ewbankkit commented Feb 19, 2024 •

edited

Loading