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

Add the possibility of to save the data with hostpath. #605

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

jbazp
Copy link

@jbazp jbazp commented Dec 11, 2024

Sonarqube will not have more than one instance at a time, I want it to always run on a node and within that node save the information with hostpath, I do not want PVC or PV, I want to directly access the host disk to save it and it will go very quickly , easy and without intermediaries (Storage Provider).

Please ensure your pull request adheres to the following guidelines:

  • Document your Changes in the CHANGELOG.md file of the respected chart as well as the Chart.yaml

@davividal
Copy link
Collaborator

davividal commented Dec 23, 2024

Hi @jbazp.

Thank you for your contribution. It looks like a cool improvement.

Could you please rebase your branch against the current master and provide a testing values.yaml under tests/unit-compatibility-test/sonarqube?

EDIT: when adding your testing values.yaml, make sure that you have pre-commit configured or run .cirrus/generate_helm_fixtures.sh manually.

Can you also replicate this to the sonarqube-dce chart?

@davividal davividal self-requested a review December 23, 2024 10:30
@davividal davividal self-assigned this Dec 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants