Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(mirrorbits-lite) allow defining ingress and PV(C)s from parent chart #817

Conversation

dduportal
Copy link
Contributor

@dduportal dduportal commented Oct 2, 2023

Related to jenkins-infra/helpdesk#2649

This PR is an extract of #813 scoped to only mirrorbits-lite

It introduces the following changes:

  • Increased the unit test coverage to PV and PVC mount points / volume name
  • Changed the binary volume name to data (kept the same mountpoint)
  • Allow defining ingress through a parent chart with a global value global.ingress.enabled
  • Allow defining an existing PVC for the data volume through a parent chart with a global value global.storage.enabled
    • In this case, the parent chart must implement a "mirrorbits-parent.pvc-name" template function to retrieve the claimName
    • A "mock" template has been added to allow unit testing

…obal value from parent chart

chore(mirrorbits-lite) increase unit test coverage to PV and PVC with their volumeMounts

Signed-off-by: Damien Duportal <[email protected]>
@dduportal dduportal added enhancement New feature or request mirrorbits-lite labels Oct 2, 2023
@dduportal dduportal marked this pull request as ready for review October 2, 2023 15:36
Signed-off-by: Damien Duportal <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request mirrorbits-lite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants