Skip to content

[bitnami/mlflow] Adds resources to init containers #33910

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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

Johannes-
Copy link
Contributor

Description of the change

Added resources to all init containers.

Benefits

  • Allows to set the resources for initContainers
  • Sometimes cluster policies could require to have resources at any deployment

Possible drawbacks

  • Used the resources of tracking and run deployment. Not possible to individual adjust the resources for each initContainer individually

Additional information

Checklist

  • Chart version bumped in Chart.yaml according to semver. This is not necessary when the changes only affect README.md files.
  • Variables are documented in the values.yaml and added to the README.md using readme-generator-for-helm
  • Title of the pull request follows this pattern [bitnami/<name_of_the_chart>] Descriptive title
  • All commits signed off and in agreement of Developer Certificate of Origin (DCO)

@github-actions github-actions bot added mlflow triage Triage is needed labels May 27, 2025
@github-actions github-actions bot requested a review from carrodher May 27, 2025 15:11
@carrodher
Copy link
Member

Thank you for initiating this pull request. We appreciate your effort. This is just a friendly reminder that signing your commits is important. Your signature certifies that you either authored the patch or have the necessary rights to contribute to the changes. You can find detailed information on how to do this in the “Sign your work” section of our contributing guidelines.

Feel free to reach out if you have any questions or need assistance with the signing process.

@Johannes- Johannes- force-pushed the feat/mlflow-add-resources-to-init-containers branch from cfb14de to d19321c Compare May 28, 2025 07:08
Copy link

This Pull Request has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thank you for your contribution.

@github-actions github-actions bot added the stale 15 days without activity label Jun 20, 2025
@Johannes-
Copy link
Contributor Author

Do you need anything further from my side?

@github-actions github-actions bot removed the stale 15 days without activity label Jun 21, 2025
@carrodher carrodher added verify Execute verification workflow for these changes in-progress labels Jun 21, 2025
@github-actions github-actions bot removed the triage Triage is needed label Jun 21, 2025
@github-actions github-actions bot removed the request for review from carrodher June 21, 2025 22:27
@github-actions github-actions bot requested a review from alvneiayu June 21, 2025 22:27
Signed-off-by: Bitnami Bot <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in-progress mlflow verify Execute verification workflow for these changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants