{"payload":{"feedbackUrl":"https://github.com/orgs/community/discussions/53140","repo":{"id":100982449,"defaultBranch":"main","name":"argo-workflows","ownerLogin":"argoproj","currentUserCanPush":false,"isFork":false,"isEmpty":false,"createdAt":"2017-08-21T18:50:44.000Z","ownerAvatar":"https://avatars.githubusercontent.com/u/30269780?v=4","public":true,"private":false,"isOrgOwned":true},"refInfo":{"name":"","listCacheKey":"v0:1720695427.0","currentOid":""},"activityList":{"items":[{"before":"f24afc8a453bd0c032f8b5b06b1217146a1f3fd4","after":"b959898599b630a8c8df1275124e62df11dc56a9","ref":"refs/heads/opentelemetry-templatecount","pushedAt":"2024-07-12T08:49:57.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"Joibel","name":"Alan Clucas","path":"/Joibel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1827156?s=80&v=4"},"commit":{"message":"feat: new template counting and phase counting metrics\n\nFrom #12589.\n\nNew metric `total_count` which is like the old `count` metric and the\nnew `gauge` metric, but a counter, not a gauge. The gauge shows a\nsnapshot of what is happening right now in the cluster, the counter\ncan answer questions like how many `Failed` workflows have there been\nin the last 24 hours.\n\nTwo further metrics for counting uses of WorkflowTemplates via\nworkflowTemplateRef only. These store the name of the WorkflowTemplate\nor ClusterWorkflowTemplate if the `cluster_scope` label is true, and\nthe namespace where it is used. `workflowtemplate_triggered_total`\ncounts the number of uses. `workflowtemplate_runtime` records how long\neach phase the workflow running the template spent in seconds.\n\nNote to reviewers: this is part of a stack of reviews for metrics\nchanges. Please don't merge until the rest of the stack is also ready.\n\nSigned-off-by: Alan Clucas ","shortMessageHtmlLink":"feat: new template counting and phase counting metrics"}},{"before":"18bb3a9c36d965eec429c3dc702f202fe1dece25","after":"9def3b4c63b94e9d87378b383b0d0e1c505ebf27","ref":"refs/heads/opentelemetry-crontrigger","pushedAt":"2024-07-12T08:49:50.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"Joibel","name":"Alan Clucas","path":"/Joibel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1827156?s=80&v=4"},"commit":{"message":"feat: new cron workflow trigger counter metric\n\nFrom #12589.\n\nA new metric which counts how many times each cron workflow has\ntriggered. A simple enough counter which can be checked against\nexpectations for the cron.\n\nNote to reviewers: this is part of a stack of reviews for metrics\nchanges. Please don't merge until the rest of the stack is also ready.\n\nSigned-off-by: Alan Clucas ","shortMessageHtmlLink":"feat: new cron workflow trigger counter metric"}},{"before":"c2aa5eb071fe0e6bbb5d3704cf4bd5f12f51ac1f","after":"a269114b29bf6cf06a97a7d0f2a345ff092c0070","ref":"refs/heads/opentelemetry-podpending","pushedAt":"2024-07-12T08:49:26.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"Joibel","name":"Alan Clucas","path":"/Joibel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1827156?s=80&v=4"},"commit":{"message":"feat: new pod pending counter metric\n\nThe workflow controller is a kubernetes controller creating\npods. Sometimes those pods do not start, and will remain in pending.\n\nThis metric counts the number of pods that may have been observed as\npending, by namespace and truncated reason. The reason is the first\npart of the kubernetes pod pending `Reason` up to the first `:` if\npresent.\n\nIt ignores all pods in the `PodInitializing` state as this I consider\nunremarkable and temporary.\n\nThis is intended for users to create alerts on particular `reasons` or\nif this metric is climbing unusually rapidly.\n\nNote to reviewers: this is part of a stack of reviews for metrics\nchanges. Please don't merge until the rest of the stack is also ready.\n\nSigned-off-by: Alan Clucas ","shortMessageHtmlLink":"feat: new pod pending counter metric"}},{"before":"9cf929176ec0447db3831b26d9f441be85276ab1","after":"55742dd4d6a6d4834fadf09a75a6a7645a98d7fd","ref":"refs/heads/opentelemetry-podphase","pushedAt":"2024-07-12T08:49:22.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"Joibel","name":"Alan Clucas","path":"/Joibel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1827156?s=80&v=4"},"commit":{"message":"feat: new pod phase counter metric\n\nFrom #12589\n\nThis is a new metric counting how many pods have gone into each pod\nphase as observed by the controller.\n\nThis is like pods_gauge, but as a counter rather than a gauge.\n\nThe gauge is useful at telling you what is happening right now in\nthe cluster, but is not useful for long term statistics such as \"How\nmany pods has workflows run\" because it may never report some pods at\nall. This counter can answer that question.\n\nNote to reviewers: this is part of a stack of reviews for metrics\nchanges. Please don't merge until the rest of the stack is also ready.\n\nSigned-off-by: Alan Clucas ","shortMessageHtmlLink":"feat: new pod phase counter metric"}},{"before":"fc9b88eeac5d334f6fb8f1da41f4a5865dfd350a","after":"06a658322023207182f2722c9b31895796689a41","ref":"refs/heads/opentelemetry-k8sduration","pushedAt":"2024-07-12T08:48:46.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"Joibel","name":"Alan Clucas","path":"/Joibel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1827156?s=80&v=4"},"commit":{"message":"feat: new k8s request duration histogram metric\n\nThis is a new metric which is a superset of the k8s_request_total\nmetric. The old metric is retained for backwards compatibility.\n\nA common issue with workflows is that it stresses the kubernetes\nAPI. Using this metric you can get insights into how long each type of\nrequest is taking which should enable optimizations to help alleviate\nproblems in this area.\n\nThe new metric has the same attributes/labels of `kind`, `verb` and\n`status_code` but now emits them with a time duration for each type of\nresponse.\n\nNote to reviewers: this is part of a stack of reviews for metrics\nchanges. Please don't merge until the rest of the stack is also ready.\n\nSigned-off-by: Alan Clucas ","shortMessageHtmlLink":"feat: new k8s request duration histogram metric"}},{"before":"3cd78efbf32b42d656023c124f216a26279c7e19","after":"8eb108e3108e76ad5847506bdf32ca4a202cf4d5","ref":"refs/heads/opentelemetry-leader","pushedAt":"2024-07-12T08:47:50.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"Joibel","name":"Alan Clucas","path":"/Joibel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1827156?s=80&v=4"},"commit":{"message":"missing bits\n\nSigned-off-by: Alan Clucas ","shortMessageHtmlLink":"missing bits"}},{"before":"bb7cc532f2235a337a22078def2ad32a99a1b69d","after":"464965c20c9cd310ea1f958db2a4ef620023bce6","ref":"refs/heads/release-3.4","pushedAt":"2024-07-11T16:46:20.000Z","pushType":"push","commitsCount":3,"pusher":{"login":"agilgur5","name":"Anton Gilgur","path":"/agilgur5","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/4970083?s=80&v=4"},"commit":{"message":"fix(cli): Ensure `--dry-run` and `--server-dry-run` flags do not create workflows. fixes #12944 (#13183)\n\nSigned-off-by: Miltiadis Alexis \nCo-authored-by: Alan Clucas \n(cherry picked from commit c59ff53a37a7e7ac41f73b23e621f601eab9d604)","shortMessageHtmlLink":"fix(cli): Ensure --dry-run and --server-dry-run flags do not crea…"}},{"before":"9f55687e38fc89176ad1ce99d43b70f8f47d584b","after":"16bfe2c24f9885006213010e0fce6d8ba91c5bd0","ref":"refs/heads/release-3.5","pushedAt":"2024-07-11T16:14:14.000Z","pushType":"push","commitsCount":5,"pusher":{"login":"agilgur5","name":"Anton Gilgur","path":"/agilgur5","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/4970083?s=80&v=4"},"commit":{"message":"fix: allow nodes without `taskResultCompletionStatus` (#13332)\n\nSigned-off-by: Alan Clucas \nCo-authored-by: Anton Gilgur <4970083+agilgur5@users.noreply.github.com>\n(cherry picked from commit c94e21e16ed9fc1d09a7e37f6b7810c7f215e91d)","shortMessageHtmlLink":"fix: allow nodes without taskResultCompletionStatus (#13332)"}},{"before":"c94e21e16ed9fc1d09a7e37f6b7810c7f215e91d","after":"5aac5a8f61f4e8273d04509dffe7d80123ff67f5","ref":"refs/heads/main","pushedAt":"2024-07-11T15:19:30.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"agilgur5","name":"Anton Gilgur","path":"/agilgur5","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/4970083?s=80&v=4"},"commit":{"message":"docs: full copy-edit of CronWorkflows page (#13292)\n\nSigned-off-by: Anton Gilgur ","shortMessageHtmlLink":"docs: full copy-edit of CronWorkflows page (#13292)"}},{"before":"c59ff53a37a7e7ac41f73b23e621f601eab9d604","after":"c94e21e16ed9fc1d09a7e37f6b7810c7f215e91d","ref":"refs/heads/main","pushedAt":"2024-07-11T14:29:14.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"agilgur5","name":"Anton Gilgur","path":"/agilgur5","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/4970083?s=80&v=4"},"commit":{"message":"fix: allow nodes without `taskResultCompletionStatus` (#13332)\n\nSigned-off-by: Alan Clucas \r\nCo-authored-by: Anton Gilgur <4970083+agilgur5@users.noreply.github.com>","shortMessageHtmlLink":"fix: allow nodes without taskResultCompletionStatus (#13332)"}},{"before":"80b222657ee161846fa930994d1bddf881df3d0e","after":"e66219991bf4d84f92acca6dcc94a2d3e46e7a74","ref":"refs/heads/priority-mutex-removal","pushedAt":"2024-07-11T11:05:51.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"Joibel","name":"Alan Clucas","path":"/Joibel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1827156?s=80&v=4"},"commit":{"message":"chore: remove PriorityMutex implementation\n\nThis is just a code refactor, removing unnecessary extra code.\n\nAs part of #11859 and #5022 I noticed that PriorityMutex is not useful\nand can just be replaced by PrioritySemaphore.\n\nThe only code path that is optimised vs PrioritySempahore is resize\nwhich will [never be\ncalled](https://github.com/argoproj/argo-workflows/blob/c59ff53a37a7e7ac41f73b23e621f601eab9d604/workflow/sync/sync_manager.go#L340).\n\nOtherwise we're performing double sync.Mutex locking for every call\nbecause all the implementation of PriorityMutex just calls the\nrelevant method from PrioritySemaphore with it's own lock held, which\nis wasteful.\n\nSigned-off-by: Alan Clucas ","shortMessageHtmlLink":"chore: remove PriorityMutex implementation"}},{"before":"29c92e5c816f1dcf0f1f92b743365659b989ca6b","after":"80b222657ee161846fa930994d1bddf881df3d0e","ref":"refs/heads/priority-mutex-removal","pushedAt":"2024-07-11T11:04:33.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"Joibel","name":"Alan Clucas","path":"/Joibel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1827156?s=80&v=4"},"commit":{"message":"chore: remove PriorityMutex implementation\n\nThis is just a code refactor, removing unnecessary extra code.\n\nAs part of #11859 and #5022 I noticed that PriorityMutex is not useful\nand can just be replaced by PrioritySemaphore.\n\nThe only code path that is optimised vs PrioritySempahore is resize\nwhich will [never be\ncalled](https://github.com/argoproj/argo-workflows/blob/c59ff53a37a7e7ac41f73b23e621f601eab9d604/workflow/sync/sync_manager.go#L340).\n\nOtherwise we're performing double sync.Mutex locking for every call\nbecause all the implementation of PriorityMutex just calls the\nrelevant method from PrioritySemaphore with it's own lock held, which\nis wasteful.\n\nSigned-off-by: Alan Clucas ","shortMessageHtmlLink":"chore: remove PriorityMutex implementation"}},{"before":"9e5d68d6151a3de1094d5b0b8b4cc1f39f550d43","after":"29c92e5c816f1dcf0f1f92b743365659b989ca6b","ref":"refs/heads/priority-mutex-removal","pushedAt":"2024-07-11T11:00:12.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"Joibel","name":"Alan Clucas","path":"/Joibel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1827156?s=80&v=4"},"commit":{"message":"chore: remove PriorityMutex implementation\n\nThis is just a code refactor, removing unnecessary extra code.\n\nAs part of #11859 and #5022 I noticed that PriorityMutex is not useful\nand can just be replaced by PrioritySemaphore.\n\nThe only code path that is optimised vs PrioritySempahore is resize\nwhich will [never be\ncalled](https://github.com/argoproj/argo-workflows/blob/c59ff53a37a7e7ac41f73b23e621f601eab9d604/workflow/sync/sync_manager.go#L340).\n\nOtherwise we're performing double sync.Mutex locking for every call\nbecause all the implementation of PriorityMutex just calls the\nrelevant method from PrioritySemaphore with it's own lock held, which\nis wasteful.\n\nSigned-off-by: Alan Clucas ","shortMessageHtmlLink":"chore: remove PriorityMutex implementation"}},{"before":null,"after":"9e5d68d6151a3de1094d5b0b8b4cc1f39f550d43","ref":"refs/heads/priority-mutex-removal","pushedAt":"2024-07-11T10:57:07.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"Joibel","name":"Alan Clucas","path":"/Joibel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1827156?s=80&v=4"},"commit":{"message":"chore: remove PriorityMutex implementation\n\nThis is just a code refactor, removing unnecessary extra code.\n\nAs part of #11859 and #5022 I noticed that PriorityMutex is not useful\nand can just be replaced by PrioritySemaphore.\n\nThe only code path that is optimised vs PrioritySempahore is resize\nwhich will [never be\ncalled](https://github.com/argoproj/argo-workflows/blob/c59ff53a37a7e7ac41f73b23e621f601eab9d604/workflow/sync/sync_manager.go#L340).\n\nOtherwise we're performing double sync.Mutex locking for every call\nbecause all the implementation of PriorityMutex just calls the\nrelevant method from PrioritySemaphore with it's own lock held, which\nis wasteful.\n\nSigned-off-by: Alan Clucas ","shortMessageHtmlLink":"chore: remove PriorityMutex implementation"}},{"before":"fc0f34ecba45744500aa78e01c43d42414141613","after":"c59ff53a37a7e7ac41f73b23e621f601eab9d604","ref":"refs/heads/main","pushedAt":"2024-07-11T08:35:22.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"Joibel","name":"Alan Clucas","path":"/Joibel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1827156?s=80&v=4"},"commit":{"message":"fix(cli): Ensure `--dry-run` and `--server-dry-run` flags do not create workflows. fixes #12944 (#13183)\n\nSigned-off-by: Miltiadis Alexis \r\nCo-authored-by: Alan Clucas ","shortMessageHtmlLink":"fix(cli): Ensure --dry-run and --server-dry-run flags do not crea…"}},{"before":"ab64c9ae0896ef0703e0e1714aaf8a80c6dd7915","after":"3cd78efbf32b42d656023c124f216a26279c7e19","ref":"refs/heads/opentelemetry-leader","pushedAt":"2024-07-11T08:09:57.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"Joibel","name":"Alan Clucas","path":"/Joibel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1827156?s=80&v=4"},"commit":{"message":"Update workflow/metrics/leader.go\n\nCo-authored-by: Anton Gilgur <4970083+agilgur5@users.noreply.github.com>\nSigned-off-by: Alan Clucas ","shortMessageHtmlLink":"Update workflow/metrics/leader.go"}},{"before":"559916e6a7b69ab63693461001e7a0229d5297b5","after":"fc0f34ecba45744500aa78e01c43d42414141613","ref":"refs/heads/main","pushedAt":"2024-07-11T08:05:14.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"Joibel","name":"Alan Clucas","path":"/Joibel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1827156?s=80&v=4"},"commit":{"message":"fix: Update modification timestamps on untar. Fixes #12885 (#13172)\n\nSigned-off-by: afenner \r\nSigned-off-by: andrew-est ","shortMessageHtmlLink":"fix: Update modification timestamps on untar. Fixes #12885 (#13172)"}},{"before":"e0220e84e1a3f293a7db4ac33dd78a6687e173bf","after":"559916e6a7b69ab63693461001e7a0229d5297b5","ref":"refs/heads/main","pushedAt":"2024-07-11T07:06:30.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"Joibel","name":"Alan Clucas","path":"/Joibel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1827156?s=80&v=4"},"commit":{"message":"chore(gh): fix grammar error in regression template (#13333)\n\nSigned-off-by: Anton Gilgur ","shortMessageHtmlLink":"chore(gh): fix grammar error in regression template (#13333)"}},{"before":"eadd9a785ec5b0126618a85df0ba2fd63c4da04b","after":"e0220e84e1a3f293a7db4ac33dd78a6687e173bf","ref":"refs/heads/main","pushedAt":"2024-07-11T02:30:59.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"terrytangyuan","name":"Yuan Tang","path":"/terrytangyuan","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/4269898?s=80&v=4"},"commit":{"message":"build(docs): exclude more commits in changelog grep (#13236)","shortMessageHtmlLink":"build(docs): exclude more commits in changelog grep (#13236)"}},{"before":"ebaebcd282518bedd79cbd93070b6ea33c6113b2","after":"9f55687e38fc89176ad1ce99d43b70f8f47d584b","ref":"refs/heads/release-3.5","pushedAt":"2024-07-10T16:42:32.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"agilgur5","name":"Anton Gilgur","path":"/agilgur5","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/4970083?s=80&v=4"},"commit":{"message":"docs: only display `release-3.5` status for Snyk badge (#13320)\n\n- modified backport to use `release-3.5` instead of `main`\n\nSigned-off-by: Anton Gilgur \n(cherry picked from commit 1a81dfb305a8576bea276a8cb2d0ad30c03c2cfb)","shortMessageHtmlLink":"docs: only display release-3.5 status for Snyk badge (#13320)"}},{"before":"12ce8fd062b8dd40ce70ca04e4633a123420ee53","after":"eadd9a785ec5b0126618a85df0ba2fd63c4da04b","ref":"refs/heads/main","pushedAt":"2024-07-10T00:18:46.000Z","pushType":"pr_merge","commitsCount":1,"pusher":{"login":"agilgur5","name":"Anton Gilgur","path":"/agilgur5","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/4970083?s=80&v=4"},"commit":{"message":"docs: fix typo in title example (#13328)\n\nSigned-off-by: Harrison Kim ","shortMessageHtmlLink":"docs: fix typo in title example (#13328)"}},{"before":"ec5cc911c66b15c05a3a6589c1a6b9127149070a","after":"f24afc8a453bd0c032f8b5b06b1217146a1f3fd4","ref":"refs/heads/opentelemetry-templatecount","pushedAt":"2024-07-09T14:28:01.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"Joibel","name":"Alan Clucas","path":"/Joibel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1827156?s=80&v=4"},"commit":{"message":"feat: new template counting and phase counting metrics\n\nFrom #12589.\n\nNew metric `total_count` which is like the old `count` metric and the\nnew `gauge` metric, but a counter, not a gauge. The gauge shows a\nsnapshot of what is happening right now in the cluster, the counter\ncan answer questions like how many `Failed` workflows have there been\nin the last 24 hours.\n\nTwo further metrics for counting uses of WorkflowTemplates via\nworkflowTemplateRef only. These store the name of the WorkflowTemplate\nor ClusterWorkflowTemplate if the `cluster_scope` label is true, and\nthe namespace where it is used. `workflowtemplate_triggered_total`\ncounts the number of uses. `workflowtemplate_runtime` records how long\neach phase the workflow running the template spent in seconds.\n\nNote to reviewers: this is part of a stack of reviews for metrics\nchanges. Please don't merge until the rest of the stack is also ready.\n\nSigned-off-by: Alan Clucas ","shortMessageHtmlLink":"feat: new template counting and phase counting metrics"}},{"before":"481c2366554b9841c8988fc9ab08de9a2a32a083","after":"18bb3a9c36d965eec429c3dc702f202fe1dece25","ref":"refs/heads/opentelemetry-crontrigger","pushedAt":"2024-07-09T14:27:35.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"Joibel","name":"Alan Clucas","path":"/Joibel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1827156?s=80&v=4"},"commit":{"message":"feat: new cron workflow trigger counter metric\n\nFrom #12589.\n\nA new metric which counts how many times each cron workflow has\ntriggered. A simple enough counter which can be checked against\nexpectations for the cron.\n\nNote to reviewers: this is part of a stack of reviews for metrics\nchanges. Please don't merge until the rest of the stack is also ready.\n\nSigned-off-by: Alan Clucas ","shortMessageHtmlLink":"feat: new cron workflow trigger counter metric"}},{"before":"34739c8aab1f7e73b2105eada1e38a61088a465f","after":"c2aa5eb071fe0e6bbb5d3704cf4bd5f12f51ac1f","ref":"refs/heads/opentelemetry-podpending","pushedAt":"2024-07-09T14:27:10.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"Joibel","name":"Alan Clucas","path":"/Joibel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1827156?s=80&v=4"},"commit":{"message":"feat: new pod pending counter metric\n\nThe workflow controller is a kubernetes controller creating\npods. Sometimes those pods do not start, and will remain in pending.\n\nThis metric counts the number of pods that may have been observed as\npending, by namespace and truncated reason. The reason is the first\npart of the kubernetes pod pending `Reason` up to the first `:` if\npresent.\n\nIt ignores all pods in the `PodInitializing` state as this I consider\nunremarkable and temporary.\n\nThis is intended for users to create alerts on particular `reasons` or\nif this metric is climbing unusually rapidly.\n\nNote to reviewers: this is part of a stack of reviews for metrics\nchanges. Please don't merge until the rest of the stack is also ready.\n\nSigned-off-by: Alan Clucas ","shortMessageHtmlLink":"feat: new pod pending counter metric"}},{"before":"14a762e1f579402c9c044601f044ae8b5af4993a","after":"9cf929176ec0447db3831b26d9f441be85276ab1","ref":"refs/heads/opentelemetry-podphase","pushedAt":"2024-07-09T14:27:06.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"Joibel","name":"Alan Clucas","path":"/Joibel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1827156?s=80&v=4"},"commit":{"message":"feat: new pod phase counter metric\n\nFrom #12589\n\nThis is a new metric counting how many pods have gone into each pod\nphase as observed by the controller.\n\nThis is like pods_gauge, but as a counter rather than a gauge.\n\nThe gauge is useful at telling you what is happening right now in\nthe cluster, but is not useful for long term statistics such as \"How\nmany pods has workflows run\" because it may never report some pods at\nall. This counter can answer that question.\n\nNote to reviewers: this is part of a stack of reviews for metrics\nchanges. Please don't merge until the rest of the stack is also ready.\n\nSigned-off-by: Alan Clucas ","shortMessageHtmlLink":"feat: new pod phase counter metric"}},{"before":"f8c31f70c66d3d7a45b35fcf4c7d70d313716704","after":"fc9b88eeac5d334f6fb8f1da41f4a5865dfd350a","ref":"refs/heads/opentelemetry-k8sduration","pushedAt":"2024-07-09T14:27:02.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"Joibel","name":"Alan Clucas","path":"/Joibel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1827156?s=80&v=4"},"commit":{"message":"feat: new k8s request duration histogram metric\n\nThis is a new metric which is a superset of the k8s_request_total\nmetric. The old metric is retained for backwards compatibility.\n\nA common issue with workflows is that it stresses the kubernetes\nAPI. Using this metric you can get insights into how long each type of\nrequest is taking which should enable optimizations to help alleviate\nproblems in this area.\n\nThe new metric has the same attributes/labels of `kind`, `verb` and\n`status_code` but now emits them with a time duration for each type of\nresponse.\n\nNote to reviewers: this is part of a stack of reviews for metrics\nchanges. Please don't merge until the rest of the stack is also ready.\n\nSigned-off-by: Alan Clucas ","shortMessageHtmlLink":"feat: new k8s request duration histogram metric"}},{"before":"2679f3051df385308133a18938ffda7bca6adf80","after":"ab64c9ae0896ef0703e0e1714aaf8a80c6dd7915","ref":"refs/heads/opentelemetry-leader","pushedAt":"2024-07-09T14:26:37.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"Joibel","name":"Alan Clucas","path":"/Joibel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1827156?s=80&v=4"},"commit":{"message":"Update workflow/metrics/test_exporter.go\n\nCo-authored-by: Anton Gilgur <4970083+agilgur5@users.noreply.github.com>\nSigned-off-by: Alan Clucas ","shortMessageHtmlLink":"Update workflow/metrics/test_exporter.go"}},{"before":"72b309b1b1ed4fcd4b53f5206673a233fc90ebf1","after":"32716e2f7584b8fee2302561c7ab687e3400aac0","ref":"refs/heads/opentelemetry-buildinfo","pushedAt":"2024-07-09T14:14:46.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"Joibel","name":"Alan Clucas","path":"/Joibel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1827156?s=80&v=4"},"commit":{"message":"docs: rename\n\nSigned-off-by: Alan Clucas ","shortMessageHtmlLink":"docs: rename"}},{"before":"c57efb2f85d2ecea6dea574e924527ce0a3ef96b","after":"72b309b1b1ed4fcd4b53f5206673a233fc90ebf1","ref":"refs/heads/opentelemetry-buildinfo","pushedAt":"2024-07-09T14:12:19.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"Joibel","name":"Alan Clucas","path":"/Joibel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1827156?s=80&v=4"},"commit":{"message":"Update workflow/metrics/labels.go\n\nCo-authored-by: Anton Gilgur <4970083+agilgur5@users.noreply.github.com>\nSigned-off-by: Alan Clucas ","shortMessageHtmlLink":"Update workflow/metrics/labels.go"}},{"before":"b5455d0d9e66a7bf0148e0ca917739f18d76a65b","after":"c57efb2f85d2ecea6dea574e924527ce0a3ef96b","ref":"refs/heads/opentelemetry-buildinfo","pushedAt":"2024-07-09T14:12:13.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"Joibel","name":"Alan Clucas","path":"/Joibel","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/1827156?s=80&v=4"},"commit":{"message":"Update docs/metrics.md\n\nCo-authored-by: Anton Gilgur <4970083+agilgur5@users.noreply.github.com>\nSigned-off-by: Alan Clucas ","shortMessageHtmlLink":"Update docs/metrics.md"}}],"hasNextPage":true,"hasPreviousPage":false,"activityType":"all","actor":null,"timePeriod":"all","sort":"DESC","perPage":30,"cursor":"djE6ks8AAAAEfaY0KQA","startCursor":null,"endCursor":null}},"title":"Activity · argoproj/argo-workflows"}