Skip to content

Conversation

omidtavakoli
Copy link
Contributor

This is the first try to add Kubernetes deployment readiness to the Sauron

@omidtavakoli omidtavakoli self-assigned this Aug 28, 2025
@omidtavakoli omidtavakoli requested a review from ocristian August 28, 2025 11:56
@omidtavakoli omidtavakoli marked this pull request as draft August 28, 2025 12:00
Copy link
Contributor

@ocristian ocristian left a comment

Choose a reason for hiding this comment

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

As we discussed, here some minor changes we can apply. 👍🏻

@omidtavakoli omidtavakoli marked this pull request as ready for review September 1, 2025 10:17
Copy link

@loiro loiro left a comment

Choose a reason for hiding this comment

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

I provided some feedback

Copy link
Member

@mohammedalics mohammedalics left a comment

Choose a reason for hiding this comment

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

What about the StatefulSets and DaemonSets? do they also covered?

@omidtavakoli
Copy link
Contributor Author

What about the StatefulSets and DaemonSets? do they also covered?

They are mostly used by SRE team services which are not the target of this ticket but we can talk about it to be covered.

@omidtavakoli omidtavakoli merged commit 05d44d6 into main Sep 4, 2025
1 check passed
@omidtavakoli omidtavakoli deleted the production-readiness-check-defined branch September 4, 2025 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants