feat(influxdb): support for influxdbv2

[xBlaz3kx]

Feature

What does this PR do?

I added a WithV2Env option to the InfluxDB module to setup the V2 instance correctly according to their confguration guidelines. Added tests for changes and updated docs/examples for setting up a V2 instance.

Why is it important?

Lack of documentation on how to setup V2 instances and prettified the workaround with a neater solution (custom options with configuration for V2).

Related issues

• Closes [Feature]: Support for InfluxDB v2 #3066

	ctx := context.Background()

	username := "username"
	password := "password"
	org := "org"
	bucket := "bucket"
	authEnabled := true
	token := "influxdbv2token"

	influxdbContainer, err := influxdb.Run(ctx,
		"influxdb:2.7.11",
		influxdb.WithV2Env(influxdb.InfluxDBV2Config{
			Username:    &username,
			Password:    &password,
			Org:         org,
			Bucket:      bucket,
			Token:       &token,
			AuthEnabled: &authEnabled,
		}),
	)
	defer func() {
		if err := testcontainers.TerminateContainer(influxdbContainer); err != nil {
			log.Printf("failed to terminate container: %s", err)
		}
	}()
	if err != nil {
		log.Printf("failed to start container: %s", err)
		return
	}
	// }

	state, err := influxdbContainer.State(ctx)
	if err != nil {
		log.Printf("failed to get container state: %s", err)
		return
	}

	fmt.Println(state.Running)
	// Output:
	// true

	// Query the InfluxDB API to verify the setup
	url, err := influxdbContainer.ConnectionUrl(ctx)
	if err != nil {
		log.Printf("failed to get host: %s", err)
		return
	}

	// Initialize a new InfluxDB client
	client := influxdb2.NewClientWithOptions(url, token, influxdb2.DefaultOptions())
	defer client.Close()

	// Get the bucket
	influxBucket, err := client.BucketsAPI().FindBucketByName(ctx, bucket)
	if err != nil {
		log.Printf("failed to get bucket: %s", err)
		return
	}

	log.Printf("Got bucket: %s", influxBucket.Name)

[netlify]

✅ Deploy Preview for testcontainers-go ready!

Name	Link
🔨 Latest commit	568c0d5
🔍 Latest deploy log	https://app.netlify.com/sites/testcontainers-go/deploys/680a25f50e43ef000881dc8a
😎 Deploy Preview	https://deploy-preview-3072--testcontainers-go.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[stevenh]

Thanks for the PR, a few suggestions and clarifying question

[mdelapenya]

Left a few comments. I do not have a strong opinion on the Nil Vs zero values discussion, as I see the value in separating required Vs optional values using pointers.

We are usually setting zero values, in general, without complains for our users, so I'd say we are fine not using pointers, checking the zero value was not set (booleans raising hands atm 😄 )

[mdelapenya]

suggestion: for consistency with the v1 import

Suggested change

	influxdb2 "github.com/influxdata/influxdb-client-go/v2"
	influxclient2 "github.com/influxdata/influxdb-client-go/v2"

[stevenh]

Updated with clarification of previous asks, hopefully this makes sense.

Having thought about this more, the big config struct is an antipattern to functional options, sorry for not spotting this earlier, see comments for details.

[mdelapenya]

@xBlaz3kx I added a commit on top of yours, if you approve, I'm merging this PR, thanks!

[xBlaz3kx]

Sounds good!

[mdelapenya]

@stevenh all good with this one?

[stevenh]

LGTM