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

Update resource configuration in remote monitoring tests for openshift #1397

Merged
merged 1 commit into from
Nov 29, 2024

Conversation

shreyabiradar07
Copy link
Contributor

Description

With the latest updates by default Kruize uses 500Mi PVC storage, 0.7 cores - 350Mi for kruize and 0.5 cores - 100Mi for kruize-db.

This PR restores the previous default values i.e 1Gi of PVC storage, 2 cores of CPU and 2Gi of memory while deploying Kruize in remote monitoring tests for Openshift. To fix the database system is shut down issue faced during testing.

Fixes #1393

Type of change

  • Bug fix
  • New feature
  • Docs update
  • Breaking change (What changes might users need to make in their application due to this PR?)
  • Requires DB changes

How has this been tested?

Please describe the tests that were run to verify your changes and steps to reproduce. Please specify any test configuration required.

  • New Test X
  • Functional testsuite - Remote monitoring sanity tests [In progress]

Test Configuration

  • Kubernetes clusters tested on: Openshift, ResourceHub

Checklist 🎯

  • Followed coding guidelines
  • Comments added
  • Dependent changes merged
  • Documentation updated
  • [] Tests added or updated

Additional information

.

@shreyabiradar07 shreyabiradar07 self-assigned this Nov 27, 2024
@shreyabiradar07 shreyabiradar07 added the bug Something isn't working label Nov 27, 2024
@shreyabiradar07 shreyabiradar07 added this to the Kruize 0.2 Release milestone Nov 27, 2024
Copy link
Contributor

@chandrams chandrams left a comment

Choose a reason for hiding this comment

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

LGTM

@dinogun dinogun merged commit 7fbbfeb into kruize:master Nov 29, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Kruize remote monitoring functional tests failures due to performance_profile not found
3 participants