Skip to content

add f5-bigip-ctlr-0.0.38 pack - #310

Merged
vishwanaths merged 3 commits into
mainfrom
f5-bigip-ctlr-0.0.38
Aug 25, 2026
Merged

add f5-bigip-ctlr-0.0.38 pack#310
vishwanaths merged 3 commits into
mainfrom
f5-bigip-ctlr-0.0.38

Conversation

@kylejep

@kylejep kylejep commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

adding f5 controller pack

@vishwanaths

Copy link
Copy Markdown
Collaborator

Pack review — f5-bigip-ctlr-0.0.38 (repo: spectrocloud/pack-central)

Packs Validation: FAILURE (run 32760199334) — individual errors listed under Blockers below
source annotation: community matches expected community (OK)
Pack metadata: name=f5-bigip-ctlr version=0.0.38 addonType=load balancer layer=addon displayName=f5-bigip-ctlr contributor=spectrocloud cloudTypes=[all]

New pack — no prior version on main.

Summary of contents

  • pack.json: 17 lines; declares one chart charts/f5-bigip-ctlr-0.0.38.tgz.
  • values.yaml (pack-level, 121 lines): pack.content.images is empty — this is the root cause of the validator failure. charts.f5-bigip-ctlr.image sets user: f5networks, repo: k8s-bigip-ctlr, version: latest (see nits).
  • charts/f5-bigip-ctlr-0.0.38.tgz: 15,816 bytes (new).
  • charts/f5-bigip-ctlr/ (unpacked chart, also committed alongside the tarball): Chart.yaml name=f5-bigip-ctlr, version=0.0.38 — matches pack name.
  • logo.png: 20,435 bytes (new).
  • README.md, RBAC-UPGRADE-GUIDE.md: docs.
  • CRDs, templates, helpers: 12 template files + one large CRD (1,589 lines).

Relevant excerpt from values.yaml

pack:
  spectrocloud.com/display-name: f5-bigip-ctlr
  releaseNameOverride:
      f5-bigip-ctlr: f5-bigip-ctlr
  namespace: kube-system
  content:
    images:          # <-- empty; validator requires at least one entry

Naming drift (informational)

  • None. Chart.yaml name: f5-bigip-ctlr matches pack.json.name: f5-bigip-ctlr, and no prior versions of this pack exist on main to compare against.

Blockers

  • Packs Validation: Image sources size is 0. Include the images used in the pack in packs/f5-bigip-ctlr-0.0.38/values.yaml
  • Packs Validation: Image content verification failed for packs/f5-bigip-ctlr-0.0.38. Make sure you have image: array defined under images: section of values.yaml
  • Packs Validation: Packs validation failed...

To unblock, populate the pack.content.images list with the concrete image references used by the chart, e.g.:

pack:
  content:
    images:
      - image: f5networks/k8s-bigip-ctlr:<pinned-tag>

Nits / observations

  • charts.f5-bigip-ctlr.image.version: latest — using the latest tag is not reproducible; consider pinning to a specific released tag (which should also match the image reference under pack.content.images).
  • Both the packaged tarball charts/f5-bigip-ctlr-0.0.38.tgz and the unpacked chart directory charts/f5-bigip-ctlr/ are committed. Most pack-central packs ship only the tarball; the unpacked tree may be redundant.
  • annotations.contributor is spectrocloud on a source: community pack — accurate if a Spectro engineer is submitting on behalf of the community, worth confirming.

Review generated by the pack-reviewer Claude Code subagent.

@vishwanaths
vishwanaths merged commit 987a75a into main Aug 25, 2026
4 of 5 checks passed
@vishwanaths
vishwanaths deleted the f5-bigip-ctlr-0.0.38 branch August 25, 2026 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants