search

apache / accumulo-testing

Apache Accumulo Testing
https://accumulo.apache.org
Apache License 2.0
15 stars 40 forks source link

Upgrade Maven, ZooKeeper, and Hadoop versions #222

Closed dlmarion closed 2 years ago

DomGarguilo commented 2 years ago

I noticed there is a newer version of the aws provider that might be able to be updated. I don't know if this makes much of a difference or will cause any issues but I thought I would point it out just in case it would be worthwhile to include into this PR.

According to https://registry.terraform.io/providers/hashicorp/aws/latest it looks like its up to version 4.31

https://github.com/apache/accumulo-testing/blob/e8e69a054f46979cec499724e6931bfb77947f23/contrib/terraform-testing-infrastructure/shared_state/aws/main.tf#L22

https://github.com/apache/accumulo-testing/blob/e9cbb856a0fac3335fff13fc175d7439c0940cca/contrib/terraform-testing-infrastructure/aws/main.tf#L62

dlmarion commented 2 years ago

@DomGarguilo - yeah, good thought. I will update that in this branch on Monday and test it out.

ctubbsii commented 2 years ago

Hadoop has 3.3.4 now, but this is good with or without that update.

dlmarion commented 2 years ago

Pushing a commit here in a minute with changes from testing. The upgrade of the aws provider from 3.68 to 4.31 deprecated some of the S3 configuration with a new way to specify it. I removed the deprecated items and replaced them with the new way of doing things in c872d9e. However, in testing, the new S3 ACL resource was returning a 403. Backing out just that change allowed things to work. Also note that if you had used version 3.68 in the past, then this will require you to perform an update. There's a nice error message with instructions on the command you need to run.