feat: Zookeeper 3.8.4 as LTS

[NickLarsenNZ]

Description

Add Zookeeper 3.8.4 as LTS.

Definition of Done Checklist

• Not all of these items are applicable to all PRs, the author should update this template to only leave the boxes in that are relevant
• Please make sure all these things are done and tick the boxes

Author

Edit tasklist title

Beta Give feedback Tasklist Author, more options

• Rename
• Copy markdown
• Delete tasklist

Delete tasklist

Delete tasklist block?

Are you sure? All relationships in this tasklist will be removed.

Cancel Delete

1. Changes are OpenShift compatible

   Changes are OpenShift compatible
   Options

   • Convert to issue
   • Toggle completion
   • Rename
   • Remove
2. Helm chart can be installed and deployed operator works

   Helm chart can be installed and deployed operator works
   Options

   • Convert to issue
   • Toggle completion
   • Rename
   • Remove
3. Integration tests passed (for non trivial changes)

   Integration tests passed (for non trivial changes)
   Options

   • Convert to issue
   • Toggle completion
   • Rename
   • Remove

Add item to Author

Reviewer

Edit tasklist title

Beta Give feedback Tasklist Reviewer, more options

• Rename
• Copy markdown
• Delete tasklist

Delete tasklist

Delete tasklist block?

Are you sure? All relationships in this tasklist will be removed.

Cancel Delete

1. Code contains useful comments

   Code contains useful comments
   Options

   • Convert to issue
   • Toggle completion
   • Rename
   • Remove
2. Code contains useful logging statements

   Code contains useful logging statements
   Options

   • Convert to issue
   • Toggle completion
   • Rename
   • Remove
3. (Integration-)Test cases added

   (Integration-)Test cases added
   Options

   • Convert to issue
   • Toggle completion
   • Rename
   • Remove
4. Documentation added or updated. Follows the [style guide](https://docs.stackable.tech/home/nightly/contributor/docs-style-guide).

   Documentation added or updated. Follows the style guide.
   Options

   • Convert to issue
   • Toggle completion
   • Rename
   • Remove
5. Changelog updated

   Changelog updated
   Options

   • Convert to issue
   • Toggle completion
   • Rename
   • Remove
6. Cargo.toml only contains references to git tags (not specific commits or branches)

   Cargo.toml only contains references to git tags (not specific commits or branches)
   Options

   • Convert to issue
   • Toggle completion
   • Rename
   • Remove

Add item to Reviewer

Acceptance

Edit tasklist title

Beta Give feedback Tasklist Acceptance, more options

• Rename
• Copy markdown
• Delete tasklist

Delete tasklist

Delete tasklist block?

Are you sure? All relationships in this tasklist will be removed.

Cancel Delete

1. Feature Tracker has been updated

   Feature Tracker has been updated
   Options

   • Convert to issue
   • Toggle completion
   • Rename
   • Remove
2. Proper release label has been added

   Proper release label has been added
   Options

   • Convert to issue
   • Toggle completion
   • Rename
   • Remove
3. [Roadmap](https://github.com/orgs/stackabletech/projects/25/views/1) has been updated

   Roadmap has been updated
   Options

   • Convert to issue
   • Toggle completion
   • Rename
   • Remove

Add item to Acceptance

[sbernauer]

Well that was easy... Ran ack '3\.8\.3'

[NickLarsenNZ]

Just holding off merging this until docker.stackable.tech/stackable/zookeeper:3.8.4-stackable0.0.0-dev is biult, then I'll run the test suite.

It is currently failing to build: https://github.com/stackabletech/docker-images/actions/runs/8233448524/job/22512946842

[NickLarsenNZ]

Tests are passing now

=== NAME  kuttl
    harness.go:405: run tests finished
    harness.go:513: cleaning up
    harness.go:570: removing temp folder: ""
--- PASS: kuttl (857.70s)
    --- PASS: kuttl/harness (0.00s)
        --- PASS: kuttl/harness/znode_zookeeper-latest-3.9.1 (30.64s)
        --- PASS: kuttl/harness/smoke_zookeeper-3.8.3_use-server-tls-false_use-client-auth-tls-false (109.78s)
        --- PASS: kuttl/harness/smoke_zookeeper-3.9.1_use-server-tls-true_use-client-auth-tls-false (103.81s)
        --- PASS: kuttl/harness/cluster-operation_zookeeper-latest-3.9.1 (34.31s)
        --- PASS: kuttl/harness/delete-rolegroup_zookeeper-3.9.1 (83.16s)
        --- PASS: kuttl/harness/delete-rolegroup_zookeeper-3.8.4 (87.13s)
        --- PASS: kuttl/harness/delete-rolegroup_zookeeper-3.8.3 (94.18s)
        --- PASS: kuttl/harness/logging_zookeeper-3.9.1 (81.01s)
        --- PASS: kuttl/harness/logging_zookeeper-3.8.4 (66.87s)
        --- PASS: kuttl/harness/logging_zookeeper-3.8.3 (68.55s)
        --- PASS: kuttl/harness/smoke_zookeeper-3.9.1_use-server-tls-true_use-client-auth-tls-true (99.36s)
        --- PASS: kuttl/harness/smoke_zookeeper-3.8.4_use-server-tls-false_use-client-auth-tls-true (110.46s)
        --- PASS: kuttl/harness/smoke_zookeeper-3.9.1_use-server-tls-false_use-client-auth-tls-true (89.33s)
        --- PASS: kuttl/harness/smoke_zookeeper-3.9.1_use-server-tls-false_use-client-auth-tls-false (100.22s)
        --- PASS: kuttl/harness/smoke_zookeeper-3.8.4_use-server-tls-true_use-client-auth-tls-true (89.52s)
        --- PASS: kuttl/harness/smoke_zookeeper-3.8.4_use-server-tls-true_use-client-auth-tls-false (87.96s)
        --- PASS: kuttl/harness/smoke_zookeeper-3.8.3_use-server-tls-true_use-client-auth-tls-true (89.74s)
        --- PASS: kuttl/harness/smoke_zookeeper-3.8.4_use-server-tls-false_use-client-auth-tls-false (88.12s)
        --- PASS: kuttl/harness/smoke_zookeeper-3.8.3_use-server-tls-true_use-client-auth-tls-false (87.51s)
        --- PASS: kuttl/harness/smoke_zookeeper-3.8.3_use-server-tls-false_use-client-auth-tls-true (89.83s)
PASS
~/dev/github.com/stackabletech/zookeeper-operator