Stefan Prodan
Stefan Prodan
> The docs - https://fluxcd.io/flagger/usage/webhooks/#load-testing - should probably make this clear :-) Yes good point, Flagger docs were written with a focus on service mesh, latter on Flagger got integrated...
If the initial health check fails, then rollback is triggered automatically after `spec.progressDeadlineSeconds` has passed.
@pjbgf I would argue that we show the correct stats by looking at Go stats (go_memstats_alloc_bytes) instead of the OS.
If `go_memstats_heap_sys_bytes` captures the CGO usage, Ok then let's change it to that.
@Santosh1176 Yes, please note that the `go_memstats_alloc_bytes_total` is a counter while the `go_memstats_heap_sys_bytes` is a gauge.
@Santosh1176 can you post a print screen with the two graphs, how it looked before and after? Thanks
> this wasn't working when I tried it locally It doesn’t work because the query is wrong, remove the `()[1m]`
Looks good now! Thank you, can you please squash the commits into a single one, rebase with upstream and force push.
@Santosh1176 please rebase instead of merging, The [0540722](https://github.com/fluxcd/flux2/pull/3098/commits/0540722ba5954f6d50921b506336c85e566fc7ea) commit has nothing to do with this PR.
Please squash the commits into a single one and it's ready to go. Thanks!