elixir-cloud-aai / drs-filer

Lightweight, flexible Flask/Gunicorn-based GA4GH DRS implementation
Apache License 2.0
5 stars 8 forks source link

Remove clusterType #62

Closed lvarin closed 1 month ago

lvarin commented 1 month ago

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Type of change

Please delete options that are not relevant.

Checklist:

Summary by Sourcery

This pull request removes the 'clusterType' variable from the Helm chart deployment configuration and updates the host_name. It also enhances the Helm chart description and increments its version.

sourcery-ai[bot] commented 1 month ago

Reviewer's Guide by Sourcery

This pull request removes the 'clusterType' variable from the Helm chart configuration files and updates the related logic to use the OpenShift route API version check. Additionally, it updates the chart description and increments the chart version.

File-Level Changes

Files Changes
deployment/Chart.yaml
deployment/values.yaml
deployment/templates/drs-filer-route.yaml
Removed 'clusterType' variable and updated related configurations to use OpenShift route API version check.

Tips - Trigger a new Sourcery review by commenting `@sourcery-ai review` on the pull request. - Continue your discussion with Sourcery by replying directly to review comments. - You can change your review settings at any time by accessing your [dashboard](https://app.sourcery.ai): - Enable or disable the Sourcery-generated pull request summary or reviewer's guide; - Change the review language; - You can always [contact us](mailto:support@sourcery.ai) if you have any questions or feedback.