Skip to content

feat: add OpenVox View dashboard alongside Puppetboard#20

Merged
anthonysomerset merged 1 commit into
OpenVoxProject:mainfrom
Obmondo:feature/openvoxview-alongside-puppetboard
May 16, 2026
Merged

feat: add OpenVox View dashboard alongside Puppetboard#20
anthonysomerset merged 1 commit into
OpenVoxProject:mainfrom
Obmondo:feature/openvoxview-alongside-puppetboard

Conversation

@sratslla
Copy link
Copy Markdown
Contributor

Fixes #19

Summary

  • Adds OpenVox View support to openvox-helm-chart as an additional optional dashboard (Option B), without removing existing Puppetboard support.
  • Introduces openvoxview.* values (image/tag/port/resources/env/ingress/securityContext) to enable/disable OpenVox View independently.
  • Deploys OpenVox View as a sidecar container in the PuppetDB pod, mounts PuppetDB certs, and exposes it via the PuppetDB service on port 5000.
  • Adds an optional openvoxview ingress template and helper labels.
  • Updates README with configuration and usage examples.

PR sponsored by Obmondo.com


Note: Since this is a company-paid contribution, we fork to the company account repo on GitHub. GitHub does not support "allow maintainer to edit in repo" for organization accounts.

  • Maintainers can either provide feedback for required changes before merging, OR
  • Pull our branch, make changes in their repo, and create a new PR including our commits.

@sratslla
Copy link
Copy Markdown
Contributor Author

Just saw this PR missed it when I created the issue. We have implemented it in slightly different approach. Lemme know how you wanna take this forward.

@anthonysomerset
Copy link
Copy Markdown
Collaborator

I have to say i prefer this implementation method as it allows users the choice and allows us a transition path (e.g. leave puppetboard default, then switch to openvoxview as default, then later remove puppetboard)

@JGodin-C2C wanting to draw your attention to this for any input you may wish to add

I haven't reviewed yet but please ensure that default remains puppetboard at this time

Copy link
Copy Markdown
Collaborator

@anthonysomerset anthonysomerset left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am happy with the change, please update the unit tests and bump the version number like other merged changes and push to this PR

Comment thread templates/puppetdb-deployment.yaml Outdated
@JGodin-C2C
Copy link
Copy Markdown
Contributor

Yep, overall the MR seems solid ( i left a comment, but that kinda splinting hairs ) .
For next improvement (in another MR ).

So, kudos for the new code.

@sratslla
Copy link
Copy Markdown
Contributor Author

I have made the suggested changes, please have a look.
Thanks!

@sratslla
Copy link
Copy Markdown
Contributor Author

sratslla commented Apr 6, 2026

Hi @anthonysomerset , checking in on this. Thanks!

@anthonysomerset
Copy link
Copy Markdown
Collaborator

I'm somewhat blocked by #22 to be cleared as this will resolve the testing fails here - have re-requested a review as i can't review my own PR and merge it :)

@anthonysomerset
Copy link
Copy Markdown
Collaborator

anthonysomerset commented May 13, 2026

@sratslla

approving the PR but merging is blocked as Commits need to have verified signatures

fastest solution that works for me is to rebase all your commits down to one once you've verified commit signing is setup and enabled in your environment, then force push back to the branch - that should also trigger CI again which should pass

@sratslla sratslla force-pushed the feature/openvoxview-alongside-puppetboard branch from 7c44bd1 to 55107dd Compare May 15, 2026 12:49
@sratslla sratslla requested a review from a team as a code owner May 15, 2026 12:49
@sratslla sratslla marked this pull request as draft May 15, 2026 12:50
@sratslla sratslla force-pushed the feature/openvoxview-alongside-puppetboard branch from 55107dd to 71d1ee7 Compare May 15, 2026 13:01
@sratslla sratslla marked this pull request as ready for review May 15, 2026 13:01
@sratslla sratslla force-pushed the feature/openvoxview-alongside-puppetboard branch from 71d1ee7 to d44375c Compare May 15, 2026 13:03
@sratslla sratslla marked this pull request as draft May 15, 2026 13:04
Add optional OpenVox View sidecar on PuppetDB with service and ingress
support, keep Puppetboard as default, add helm unit tests, and bump chart
version to 10.0.3.
Signed-off-by: Shubh Gaur <sratslla@gmail.com>
@sratslla sratslla force-pushed the feature/openvoxview-alongside-puppetboard branch from d44375c to b05b92a Compare May 15, 2026 13:06
@sratslla sratslla marked this pull request as ready for review May 15, 2026 13:06
@anthonysomerset anthonysomerset merged commit 617c5dc into OpenVoxProject:main May 16, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature request]: Add support for OpenVox View dashboard to Helm chart

3 participants