Re: [PR] changing dag_processing.processes from UpDownCounter to guage [airflow]

2024-05-21 Thread via GitHub
ferruzzi commented on PR #38400: URL: https://github.com/apache/airflow/pull/38400#issuecomment-2122995292 > @ferruzzi do I have any update on this part? How would that work? A gauge is an arbitrary reading, you just tell it what the new/current value is. Unless you see some way

Re: [PR] changing dag_processing.processes from UpDownCounter to guage [airflow]

2024-05-19 Thread via GitHub
Bowrna commented on PR #38400: URL: https://github.com/apache/airflow/pull/38400#issuecomment-2119179128 > > when setting the `dag_processing.processes` as gauge instead of the counter, we may need to initialize the value. > > ``` > > Gauge works in the following way: > >

Re: [PR] changing dag_processing.processes from UpDownCounter to guage [airflow]

2024-05-17 Thread via GitHub
github-actions[bot] commented on PR #38400: URL: https://github.com/apache/airflow/pull/38400#issuecomment-2118514080 This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for

Re: [PR] changing dag_processing.processes from UpDownCounter to guage [airflow]

2024-03-27 Thread via GitHub
Bowrna commented on PR #38400: URL: https://github.com/apache/airflow/pull/38400#issuecomment-2024398539 > when setting the `dag_processing.processes` as gauge instead of the counter, we may need to initialize the value. > > ``` > Gauge works in the following way: >

Re: [PR] changing dag_processing.processes from UpDownCounter to guage [airflow]

2024-03-27 Thread via GitHub
ferruzzi commented on code in PR #38400: URL: https://github.com/apache/airflow/pull/38400#discussion_r1541522673 ## airflow/dag_processing/manager.py: ## @@ -1207,6 +1209,7 @@ def _kill_timed_out_processors(self): processor.start_time.isoformat(),

Re: [PR] changing dag_processing.processes from UpDownCounter to guage [airflow]

2024-03-27 Thread via GitHub
ferruzzi commented on code in PR #38400: URL: https://github.com/apache/airflow/pull/38400#discussion_r1541520266 ## airflow/dag_processing/manager.py: ## @@ -1207,6 +1209,7 @@ def _kill_timed_out_processors(self): processor.start_time.isoformat(),

Re: [PR] changing dag_processing.processes from UpDownCounter to guage [airflow]

2024-03-26 Thread via GitHub
Bowrna commented on code in PR #38400: URL: https://github.com/apache/airflow/pull/38400#discussion_r1540465952 ## airflow/dag_processing/manager.py: ## @@ -1207,6 +1209,7 @@ def _kill_timed_out_processors(self): processor.start_time.isoformat(),

Re: [PR] changing dag_processing.processes from UpDownCounter to guage [airflow]

2024-03-26 Thread via GitHub
ferruzzi commented on code in PR #38400: URL: https://github.com/apache/airflow/pull/38400#discussion_r1539753482 ## airflow/dag_processing/manager.py: ## @@ -1207,6 +1209,7 @@ def _kill_timed_out_processors(self): processor.start_time.isoformat(),

Re: [PR] changing dag_processing.processes from UpDownCounter to guage [airflow]

2024-03-25 Thread via GitHub
Bowrna commented on code in PR #38400: URL: https://github.com/apache/airflow/pull/38400#discussion_r1538589431 ## airflow/dag_processing/manager.py: ## @@ -1207,6 +1209,7 @@ def _kill_timed_out_processors(self): processor.start_time.isoformat(),

Re: [PR] changing dag_processing.processes from UpDownCounter to guage [airflow]

2024-03-25 Thread via GitHub
ferruzzi commented on code in PR #38400: URL: https://github.com/apache/airflow/pull/38400#discussion_r1538280611 ## airflow/dag_processing/manager.py: ## @@ -1207,6 +1209,7 @@ def _kill_timed_out_processors(self): processor.start_time.isoformat(),

Re: [PR] changing dag_processing.processes from UpDownCounter to guage [airflow]

2024-03-25 Thread via GitHub
ferruzzi commented on code in PR #38400: URL: https://github.com/apache/airflow/pull/38400#discussion_r1538280611 ## airflow/dag_processing/manager.py: ## @@ -1207,6 +1209,7 @@ def _kill_timed_out_processors(self): processor.start_time.isoformat(),

Re: [PR] changing dag_processing.processes from UpDownCounter to guage [airflow]

2024-03-25 Thread via GitHub
Bowrna commented on code in PR #38400: URL: https://github.com/apache/airflow/pull/38400#discussion_r1537109916 ## docs/apache-airflow/administration-and-deployment/logging-monitoring/metrics.rst: ## @@ -219,6 +219,7 @@ Name

Re: [PR] changing dag_processing.processes from UpDownCounter to guage [airflow]

2024-03-24 Thread via GitHub
Bowrna commented on PR #38400: URL: https://github.com/apache/airflow/pull/38400#issuecomment-2017270573 when setting the `dag_processing.processes` as gauge instead of the counter, we may need to initialize the value. Gauge works in the following way: gaugor:333|g the above

Re: [PR] changing dag_processing.processes from UpDownCounter to guage [airflow]

2024-03-24 Thread via GitHub
Bowrna closed pull request #38400: changing dag_processing.processes from UpDownCounter to guage URL: https://github.com/apache/airflow/pull/38400 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

Re: [PR] changing dag_processing.processes from UpDownCounter to guage [airflow]

2024-03-23 Thread via GitHub
Bowrna commented on PR #38400: URL: https://github.com/apache/airflow/pull/38400#issuecomment-2016455324 > Add deprecation details around the old metric. Can you update the metrics documentation? yes @dirrao Let me do it -- This is an automated message from the Apache Git Service.

Re: [PR] changing dag_processing.processes from UpDownCounter to guage [airflow]

2024-03-22 Thread via GitHub
Bowrna commented on code in PR #38400: URL: https://github.com/apache/airflow/pull/38400#discussion_r1535234940 ## airflow/dag_processing/manager.py: ## @@ -1207,6 +1209,7 @@ def _kill_timed_out_processors(self): processor.start_time.isoformat(),

[PR] changing dag_processing.processes from UpDownCounter to guage [airflow]

2024-03-22 Thread via GitHub
Bowrna opened a new pull request, #38400: URL: https://github.com/apache/airflow/pull/38400 issue: #36188 related: #36188 --- **^ Add meaningful description above** Read the **[Pull Request