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

Ensure helm chart is consistent with registry operator #1260

Closed
8 tasks
michael-valdron opened this issue Sep 20, 2023 · 3 comments
Closed
8 tasks

Ensure helm chart is consistent with registry operator #1260

michael-valdron opened this issue Sep 20, 2023 · 3 comments
Labels
area/registry Devfile registry for stacks and infrastructure

Comments

@michael-valdron
Copy link
Member

michael-valdron commented Sep 20, 2023

Which area/kind this issue is related to?

/area registry

Issue Description

The setup for the devfile registry helm chart and devfile registry operator are currently inconsistent. We need to ensure that the registry helm chart setup is in sync with the setup of the registry operator in how they deploy the devfile registry before we can provide official releases #1009.

Related to #1193.

Acceptance Criteria

  • Ensure specifications under registry helm chart are consistent
    • Deployment specifications
    • Service specifications
    • Route specifications
    • Ingress specifications
    • Configmap specifications
    • Persistent Volume Claim (PVC) specifications
  • Ensure helm chart parameters are consistent with DevfileRegistry CR, can provide the same setup, and use the same defaults
  • Ensure Chart.yaml is consistent with the registry operator cluster service version (CSV) file
  • Ensure the helm chart supports the same requirements as the registry operator
  • Ensure hostname setup is consistent
  • Ensure app name setup is consistent
  • Update repository documentation to reflect the changes
  • Update PR template to include acceptance criteria where if there is any changes to these items there should be a tied PR to update the registry operator
@michael-valdron
Copy link
Member Author

Updated acceptance criteria to include additional PR acceptance criteria for the helm chart, setting for next refinement call.

@michael-valdron michael-valdron moved this from Backlog to Waiting in Devfile Project Sep 29, 2023
@michael-valdron
Copy link
Member Author

#1193 (comment)

@michael-valdron
Copy link
Member Author

#1193 (comment)

Due to the above change in direction this issue is no longer needed.

Ensure the helm chart supports the same requirements as the registry operator

This extra criteria has been included under #1193 to be part of the investigation.

@michael-valdron michael-valdron closed this as not planned Won't fix, can't repro, duplicate, stale Sep 29, 2023
@github-project-automation github-project-automation bot moved this from Waiting to Done ✅ in Devfile Project Sep 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/registry Devfile registry for stacks and infrastructure
Projects
Status: Done ✅
Development

No branches or pull requests

1 participant