-
Notifications
You must be signed in to change notification settings - Fork 448
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
[xray] add defaultRetentionDaysForIndexedRepo support #1833
Conversation
CLA Assistant Lite bot: I have read the CLA Document and I hereby sign the CLA You can retrigger this bot by commenting recheck in this Pull Request |
I have read the CLA Document and I hereby sign the CLA |
Is there anyone to review ? |
@gregoryboue Thanks for the PR. We will include this change in one of the upcoming releases. |
Hi, do you have any information about the release date ? |
@rahulsadanandan , is this PR will be merged ? |
@rahulsadanandan any news ? |
We're exploring the concept of structured system YAML as part of another PR (#1842). This method enables us to modify individual system YAML parameters without the need to supply the entire system YAML block. |
@rahulsadanandan Thanks for the answer, it would be a very useful feature for us, i will waiting for it. The ETA is for Q1 2024, do you have any precision about the month ? |
@gregoryboue Recently have merged a feature (#1842) that allows overriding the system YAML with the |
PR Checklist
[artifactory]
)What this PR does / why we need it:
Currently the Xray indexing default retention period can be defined in Xray System yaml with server.repo.defaultRetentionDaysForIndexedRepo settings.
If we want to configure this settings, we have to override xray system yaml (see systemYamlOverride), so it would be great to be able to configure this setting via a value in the chart's values.yaml file.
Which issue this PR fixes : fixes #1832