Skip to content

added few functions to make statefulset healthy and unhealthy.#3

Open
karthikkesarla wants to merge 1 commit intomainfrom
CPDEV-100741
Open

added few functions to make statefulset healthy and unhealthy.#3
karthikkesarla wants to merge 1 commit intomainfrom
CPDEV-100741

Conversation

@karthikkesarla
Copy link

@karthikkesarla karthikkesarla commented Feb 18, 2025

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Description

Acutally in Consul-service robot tests we have alert by name "ConsulIsDownAlarm" to make this alert activated this functions are useful.

What does this MR do?

Consul-service robot tests we have alert by name "ConsulIsDownAlarm" to make this alert activated we were deleting all the pods but it was not activating the alert. To make that alert active we need have used 2 approches 'changing memory to very low values' and 'changing the readiness probe unhealty'. In this tests we first capture the existing details and change to make pods unhealty and once alert is activated revert it back to orginal values.

Are there points in the code the reviewer needs to double check?

we had only one test to make this alert activated we have used two approches please suggest which one would suite this case.

Related Tickets & Documents

  • Related Issue #
  • Closes #

QA Instructions, Screenshots, Recordings

Related Information

Breaking Change checklist

If your PR includes any deployment or processing changes, please utilize this checklist:

  • Does it change any deployment parameters, logic of their working or rename them?
  • Did update from previous version tested with the same set of deployment parameters?

Added/updated tests?

  • Yes
  • No, and this is why: please replace this line with details on why tests
    have not been included
  • I need help with writing tests

[optional] Are there any things to highlight or double check?

[optional] What gif best describes this PR or how it makes you feel?

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