[kafka] specify mimimal working resource presets - #1040

Reviewing files that changed from the base of the PR and between c467ed7 and ba97a45.

• packages/apps/kafka/Chart.yaml (1 hunks)
• packages/apps/kafka/README.md (1 hunks)
• packages/apps/kafka/values.schema.json (2 hunks)
• packages/apps/kafka/values.yaml (2 hunks)
• packages/apps/versions_map (1 hunks)

• GitHub Check: Build
• GitHub Check: pre-commit

packages/apps/kafka/Chart.yaml (1)

19-19: Approve chart version bump.

The update from 0.6.0 to 0.6.1 follows semantic versioning and aligns with the resource preset changes in the chart.

packages/apps/versions_map (1)

42-43:

Details

✅ Verification successful

Verify Kafka version mapping.

You’ve pinned kafka 0.6.0 to commit 6130f43d and introduced 0.6.1 at HEAD. Ensure that:

1. 6130f43d accurately corresponds to the last commit where Chart.yaml was at 0.6.0.
2. No intermediate Kafka versions are missing in the map.

---

🏁 Script executed:

#!/bin/bash
# Verify that the commit hash for kafka 0.6.0 matches the Chart.yaml version
if git show 6130f43d:packages/apps/kafka/Chart.yaml | grep -q 'version: 0.6.0'; then
  echo "✅ Hash matches Chart version 0.6.0"
else
  echo "❌ Hash does not match Chart version 0.6.0"
  exit 1
fi

Length of output: 156

---

🏁 Script executed:

#!/bin/bash
# List all historical versions in packages/apps/kafka/Chart.yaml
git log --pretty=format:%H -- packages/apps/kafka/Chart.yaml | while read commit; do
  git show "$commit:packages/apps/kafka/Chart.yaml" 2>/dev/null \
    | grep '^version:' \
    | awk '{print $2}'
done | sort -u

Length of output: 287

---

🏁 Script executed:

#!/bin/bash
# Correctly list all unique versions from the history of packages/apps/kafka/Chart.yaml
git log --pretty=format:%H -- packages/apps/kafka/Chart.yaml | while read commit; do
  git show "${commit}:packages/apps/kafka/Chart.yaml" 2>/dev/null \
    | sed -n 's/^version:[[:space:]]*//p'
done | sort -u

Length of output: 287

---

🏁 Script executed:

#!/bin/bash
# Locate all Chart.yaml files in the repository (case-insensitive)
find . -type f -iname chart.yaml

Length of output: 7035

---

🏁 Script executed:

#!/bin/bash
# Accurately list all unique Kafka versions from the history of packages/apps/kafka/Chart.yaml
git rev-list HEAD -- packages/apps/kafka/Chart.yaml | while IFS= read -r commit; do
  git show "${commit}:packages/apps/kafka/Chart.yaml" 2>/dev/null \
    | sed -n 's/^version:[[:space:]]*//p'
done | sort -u

Length of output: 2340

---

Kafka version mapping verified

Commit 6130f43 correctly pins Chart.yaml to version 0.6.0, and the version history (0.1.0 → … → 0.6.0 → 0.6.1) shows no missing intermediate Kafka releases. All looks good.

packages/apps/kafka/values.yaml (2)

28-28: Approve Kafka resourcesPreset default update.

Changing kafka.resourcesPreset from "nano" to "small" raises the baseline resources to better suit production workloads.

---

45-45: Approve ZooKeeper resourcesPreset default update.

Updating zookeeper.resourcesPreset from "nano" to "micro" provides a more resilient default configuration.

packages/apps/kafka/README.md (2)

17-17: Approve Kafka docs default update.

Documentation now correctly reflects kafka.resourcesPreset: small, matching the Helm chart defaults.

---

19-19: Approve ZooKeeper docs default update.

The zookeeper.resourcesPreset default is updated to micro, in sync with values.yaml.

packages/apps/kafka/values.schema.json (2)

36-36: Approve Kafka schema default update.

The JSON schema default for kafka.resourcesPreset is now "small", consistent with the Helm values.

---

66-66: Approve ZooKeeper schema default update.

The JSON schema default for zookeeper.resourcesPreset is updated to "micro", aligning with the chart defaults.