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

Fixes #RHINENG-14257 - Add retry and wait to performance profile creation #289

Merged

Conversation

upadhyeammit
Copy link
Contributor

PR Title 💥

Please title this PR with a summary of the change, along with the JIRA card number.

Suggested formats:

  1. Fixes/Refs #RHIROS-XXX - Title
  2. RHIROS-XXX Title
  3. RHINENG-XXX Title

Feel free to remove this section from PR description once done.

Why do we need this change? 💭

Please include the context of this change here. If the change depends on Kruize add details about that as well!

Documentation update? 📝

  • Yes
  • No

Security Checklist 🔒

Upon raising this PR please go through RedHatInsights/secure-coding-checklist

💂‍♂️ Checklist 🎯

  • Does this change depend on specific version of Kruize
    • If yes what is the version no:
    • Is that image available in production or needs deployment?
  • Bugfix
  • New Feature
  • Refactor
  • Unittests Added
  • DRY code
  • Dependency Added
  • DB Migration Added

Additional 📣

Feel free to add any other relevant details such as links, notes, screenshots, here.

@upadhyeammit upadhyeammit force-pushed the retry-logic-for-perf-profile branch 8 times, most recently from b3b05de to de5c521 Compare December 20, 2024 14:13
@upadhyeammit upadhyeammit force-pushed the retry-logic-for-perf-profile branch from de5c521 to d576c60 Compare December 20, 2024 14:29
@upadhyeammit upadhyeammit changed the title Add retry and wait to performance profile creation Fixes #RHINENG-14257 - Add retry and wait to performance profile creation Dec 20, 2024
Copy link
Contributor

@saltgen saltgen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @upadhyeammit!

@upadhyeammit upadhyeammit merged commit 7a74ec6 into RedHatInsights:main Dec 20, 2024
9 checks passed
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