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

[artifactory-oss] remove unused values #1932

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

Conversation

gulecroc
Copy link

PR Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • Chart Version bumped
  • CHANGELOG.md updated
  • Variables and other changes are documented in the README.md
  • Title of the PR starts with chart name (e.g. [artifactory])

What this PR does / why we need it:

artifactory-oss chart only override actifactory chart values under artifactory.x

This PR is to remove unused values

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #

Special notes for your reviewer:

Copy link

github-actions bot commented Oct 24, 2024

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@gulecroc
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

@gulecroc
Copy link
Author

recheck

@chukka chukka added bug Something isn't working enhancement New feature or request and removed bug Something isn't working labels Dec 5, 2024
@chukka
Copy link
Collaborator

chukka commented Dec 5, 2024

@gulecroc Thanks for the PR , we would like this to be part of next minor release , mostly in jan SH release . This happens for jcr and cpp-ce charts as well . Fixed internally . waiting for public release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants