Deepthi Sigireddi

Results 173 comments of Deepthi Sigireddi

@h4l0gen @Ayush9026 The best thing I can recommend for you is to spend a little time exploring the Vitess [website](https://vitess.io) and the community slack (invite linked in issue description). We...

Mentorship is complete.

I removed the website docs label. We don't actually have a page that lists all available metrics, so there's no good place to add docs for new metrics.

@shlomi-noach please feel free to dismiss my review if @timvaillancourt is able complete the requested changes before code freeze.

> > OK, now I understand the intent of this metric, it won't work as implemented. Counters should only be used for monotonically increasing numbers, prometheus will not let you...

Some minor wording changes to release notes are required to avoid user confusion. Rest LGTM. @shlomi-noach you may dismiss my review once these changes are done.

> To clarify, the use of `Version` is not causing a functional issue in Vitess, right? This change is only to be consistent? > > Since we are watching a...

This particular issue is already being worked on. Please note that It is not part of the scope of LFX term 3, it's just related. PRs are still welcome for...

Flag docs are auto-generated, so we don't need any manual changes on the website for this. @frouioui can you confirm that I'm getting this right?

> After talking with @mattlord, I realized that actually the automation does not work. We should update the website docs manually at the same time as this PR. @frouioui can...