Fixup NOTES.txt, facilitate NodePort

Jason Plumrequested to merge
fixup-20250922 into main

What does this MR do?

  • Update the NOTES.txt to function appropriately when ingress.enabled: false
  • When serviceActive.type: NodePort, expose the spec.nodePort from config.apiPort

Author's Checklist

For anything in this list which will not be completed, please provide a reason in the MR discussion.

Required

  • Ensure a release milestone is set.
  • MR title and description are up to date, accurate, and descriptive.
  • MR targeting the appropriate branch.
  • MR has a green pipeline on GitLab.com.
  • When ready for review, MR is labeled workflowready for review per the MR workflow.

Expected

  • Test plan indicating conditions for success has been posted and passes.
  • Documentation is created or updated.
  • Tests are added.

Related issues

#22 (closed)

Testing

helm install --dry-run ob . --set serviceActive.type=NodePort --set ingress.enabled=false

Before:

Error: INSTALLATION FAILED: template: openbao/templates/NOTES.txt:8:39: executing "openbao/templates/NOTES.txt" at <.Values.service.type>: nil pointer evaluating interface {}.type

After:

NOTES:
1. Get the application URL by running these commands:
  export NODE_PORT=$(kubectl get --namespace default -o jsonpath="{.spec.ports[0].nodePort}" services ob-openbao-active)
  export NODE_IP=$(kubectl get nodes --namespace default -o jsonpath="{.items[0].status.addresses[0].address}")
  echo http://$NODE_IP:$NODE_PORT

Merge request reports