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

RHIROS-1359 Probes for Kruize #135

Merged
merged 3 commits into from
Apr 3, 2024

Conversation

saltgen
Copy link
Contributor

@saltgen saltgen commented Oct 11, 2023

No description provided.

kruize-clowdapp.yaml Outdated Show resolved Hide resolved
kruize-clowdapp.yaml Outdated Show resolved Hide resolved
@kgaikwad
Copy link
Collaborator

Hello @dgaikwad - could you please look into this and try to test these probes under EE?

@patilsuraj767
Copy link
Collaborator

/retest

kruize-clowdapp.yaml Outdated Show resolved Hide resolved
kruize-clowdapp.yaml Outdated Show resolved Hide resolved
kruize-clowdapp.yaml Outdated Show resolved Hide resolved
@saltgen
Copy link
Contributor Author

saltgen commented Oct 23, 2023

@patilsuraj767 All changes pushed

kruize-clowdapp.yaml Show resolved Hide resolved
@saltgen
Copy link
Contributor Author

saltgen commented Oct 26, 2023

/retest

@saltgen saltgen force-pushed the feature/probes-for-kruize branch from f687818 to 6ac7c74 Compare October 26, 2023 11:16
@saltgen
Copy link
Contributor Author

saltgen commented Oct 26, 2023

@patilsuraj767 I have also removed the following fields from create-kruize-partitions, could you review the same as well?

Screenshot from 2023-10-26 16-48-52

@saltgen
Copy link
Contributor Author

saltgen commented Oct 26, 2023

/retest

kruize-clowdapp.yaml Show resolved Hide resolved
@saltgen
Copy link
Contributor Author

saltgen commented Nov 17, 2023

/retest all

@yash2189
Copy link
Contributor

yash2189 commented Dec 5, 2023

/retest

@yash2189
Copy link
Contributor

yash2189 commented Dec 5, 2023

Changes look good and we have added observations on respective Jira ticket
cc: @kgaikwad @saltgen

@kgaikwad
Copy link
Collaborator

kgaikwad commented Dec 7, 2023

Thank you @yash2189 for the observation. As per offline discussion, @saltgen still needs to visit and update one of the setting.

@saltgen
Copy link
Contributor Author

saltgen commented Dec 15, 2023

/retest

@kgaikwad
Copy link
Collaborator

  • adding @msvinaykumar on this PR. Vinay, please request to review as it includes probes for Kruize resources.

@saltgen
Copy link
Contributor Author

saltgen commented Mar 21, 2024

/retest

@saltgen saltgen force-pushed the feature/probes-for-kruize branch from 07bf596 to 9148ec6 Compare March 21, 2024 08:27
@saltgen
Copy link
Contributor Author

saltgen commented Mar 21, 2024

Changes tested on EE

@saltgen
Copy link
Contributor Author

saltgen commented Mar 21, 2024

The PR check failure seems to be cause of performance profile creation failure, I'm able to see recomm on EE.

Copy link
Collaborator

@patilsuraj767 patilsuraj767 left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@saltgen
Copy link
Contributor Author

saltgen commented Apr 3, 2024

@kgaikwad I haven't updated the param values post @msvinaykumar's ack on internal IM, hence merging this PR

@saltgen saltgen dismissed kgaikwad’s stale review April 3, 2024 10:02

Query resolved on discussion thread

@saltgen saltgen merged commit 971c198 into RedHatInsights:main Apr 3, 2024
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants