You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "Jarvl (via GitHub)" <gi...@apache.org> on 2025/03/05 20:28:57 UTC

[PR] Upgrade pgbouncer_exporter version to 0.18.0 [airflow]

Jarvl opened a new pull request, #47416:
URL: https://github.com/apache/airflow/pull/47416

   <!--
    Licensed to the Apache Software Foundation (ASF) under one
    or more contributor license agreements.  See the NOTICE file
    distributed with this work for additional information
    regarding copyright ownership.  The ASF licenses this file
    to you under the Apache License, Version 2.0 (the
    "License"); you may not use this file except in compliance
    with the License.  You may obtain a copy of the License at
   
      http://www.apache.org/licenses/LICENSE-2.0
   
    Unless required by applicable law or agreed to in writing,
    software distributed under the License is distributed on an
    "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
    KIND, either express or implied.  See the License for the
    specific language governing permissions and limitations
    under the License.
    -->
   
   <!--
   Thank you for contributing! Please make sure that your code changes
   are covered with tests. And in case of new features or big changes
   remember to adjust the documentation.
   
   Feel free to ping committers for the review!
   
   In case of an existing issue, reference it using one of the following:
   
   closes: #ISSUE
   related: #ISSUE
   
   How to write a good git commit message:
   http://chris.beams.io/posts/git-commit/
   -->
   
   This PR bumps `pgbouncer_exporter` to version `0.18.0`.
   
   Version [0.18.0](https://github.com/jbub/pgbouncer_exporter/releases/tag/v0.18.0) of `pgbouncer_exporter` has been released which provides support for `pgbouncer` [1.24.0](https://github.com/pgbouncer/pgbouncer/releases/tag/pgbouncer_1_24_0). The intent is to publish an image that is compatible with the most recent `pgbouncer` image [airflow-pgbouncer-2025.01.10-1.24.0](https://hub.docker.com/layers/apache/airflow/airflow-pgbouncer-2025.01.10-1.24.0/images/sha256-e8fd120604e8113082e9ad070e638b715cf512c279299782e76cc5ad431a25ad) which is currently incompatible with the most recent `pgbouncer_exporter` image [airflow-pgbouncer-exporter-2024.06.18-0.17.0](https://hub.docker.com/layers/apache/airflow/airflow-pgbouncer-exporter-2024.06.18-0.17.0/images/sha256-48f8442cfe19451303e718896edc45373c15157014f1327c2aed4dabc972229d).
   
   This a preliminary change to address incompatibilities between the two images mentioned above in the Airflow helm chart, which is outlined in https://github.com/apache/airflow/issues/47295
   - This does not directly close this issue, as it calls out that the current helm chart defaults are incompatible. Changes to these defaults can be addresses in a follow-up PR once a new image is published from this PR.
   
   Version 0.18.0 of `pgbouncer_exporter` also increments the go version used to 1.23, (see: https://github.com/jbub/pgbouncer_exporter/commit/479710be0d06a12c08a05694f1bb4ae8e4a1062c). The expected go version is bumped here to 1.23.7 here to support this change.
   
   <!-- Please keep an empty line above the dashes. -->
   ---
   


-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade helm chart pgbouncer_exporter version to 0.18.0 [airflow]

Posted by "Jarvl (via GitHub)" <gi...@apache.org>.
Jarvl commented on code in PR #47416:
URL: https://github.com/apache/airflow/pull/47416#discussion_r1983901647


##########
chart/newsfragments/47416.misc.rst:
##########
@@ -0,0 +1 @@
+The chart uses newer version of pgbouncer exporter - ``airflow-pgbouncer-exporter-2025.03.05-0.18.0``

Review Comment:
   One step ahead of you 😄 https://github.com/apache/airflow/pull/47416/commits/642759085d914f75ad02a4e8313e338a0eb8be41
   
   Edit: not sure why that commit diff isn't showing the newline. Oh well



-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade helm chart pgbouncer_exporter version to 0.18.0 [airflow]

Posted by "Jarvl (via GitHub)" <gi...@apache.org>.
Jarvl commented on code in PR #47416:
URL: https://github.com/apache/airflow/pull/47416#discussion_r1983901647


##########
chart/newsfragments/47416.misc.rst:
##########
@@ -0,0 +1 @@
+The chart uses newer version of pgbouncer exporter - ``airflow-pgbouncer-exporter-2025.03.05-0.18.0``

Review Comment:
   One step ahead of you 😄 https://github.com/apache/airflow/pull/47416/commits/642759085d914f75ad02a4e8313e338a0eb8be41



-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade helm chart pgbouncer_exporter version to 0.18.0 [airflow]

Posted by "Jarvl (via GitHub)" <gi...@apache.org>.
Jarvl commented on PR #47416:
URL: https://github.com/apache/airflow/pull/47416#issuecomment-2704789029

   > Just tried the new image alongside pgbouncer 1.24, still getting the same error.
   > 
   > It looks like v0.18.0 of pgbouncer_exporter only provides support up to pgbouncer 1.23, so what's in my PR description currently is inaccurate. This will not resolve the errors noted in #47295.
   > 
   > I created an issue in the upstream library to address: [jbub/pgbouncer_exporter#34](https://github.com/jbub/pgbouncer_exporter/issues/34)
   > 
   > How do we want to proceed? It's not going to be as simple as performing this upgrade to provide support for pgbouncer 1.24 since the upstream needs updated with support.
   > 
   > One option could be to proceed with this PR, then decrement the pgbouncer version in the helm chart to 1.23 so there are no longer incompatibilities. Or at least build an image for pgbouncer 1.23. Then once pgbouncer_exporter has adequate support we can bump that to a newer version in the helm chart.
   > 
   > Edit: To clarify, there is not an image built for pgbouncer 1.23 (it was incremented from 1.22 -> 1.24 in #45542).
   
   I updated my PR description to reflect what I noted here.


-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade helm chart pgbouncer_exporter version to 0.18.0 [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #47416:
URL: https://github.com/apache/airflow/pull/47416#issuecomment-2704695861

   added missing EOL


-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade helm chart pgbouncer_exporter version to 0.18.0 [airflow]

Posted by "Jarvl (via GitHub)" <gi...@apache.org>.
Jarvl commented on PR #47416:
URL: https://github.com/apache/airflow/pull/47416#issuecomment-2704745494

   Just tried the new image alongside pgbouncer 1.24, still getting the same error.
   
   It looks like v0.18.0 of pgbouncer_exporter only provides support up to pgbouncer 1.23, so what's in my PR description currently is inaccurate. This will not resolve the errors noted in #47295.
   
   I created an issue in the upstream library to address: https://github.com/jbub/pgbouncer_exporter/issues/34
   
   How do we want to proceed? It's not going to be as simple as performing this upgrade to provide support for pgbouncer 1.24 since the upstream needs updated with support.
   
   One option could be to proceed with this PR, then decrement the pgbouncer version in the helm chart to 1.23 so there are no longer incompatibilities. Or at least build an image for pgbouncer 1.23. Then once pgbouncer_exporter has adequate support we can bump that to a newer version here.


-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade pgbouncer_exporter version to 0.18.0 [airflow]

Posted by "boring-cyborg[bot] (via GitHub)" <gi...@apache.org>.
boring-cyborg[bot] commented on PR #47416:
URL: https://github.com/apache/airflow/pull/47416#issuecomment-2701994198

   Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contributors' Guide (https://github.com/apache/airflow/blob/main/contributing-docs/README.rst)
   Here are some useful points:
   - Pay attention to the quality of your code (ruff, mypy and type annotations). Our [pre-commits]( https://github.com/apache/airflow/blob/main/contributing-docs/08_static_code_checks.rst#prerequisites-for-pre-commit-hooks) will help you with that.
   - In case of a new feature add useful documentation (in docstrings or in `docs/` directory). Adding a new operator? Check this short [guide](https://github.com/apache/airflow/blob/main/docs/apache-airflow/howto/custom-operator.rst) Consider adding an example DAG that shows how users should use it.
   - Consider using [Breeze environment](https://github.com/apache/airflow/blob/main/dev/breeze/doc/README.rst) for testing locally, it's a heavy docker but it ships with a working Airflow and a lot of integrations.
   - Be patient and persistent. It might take some time to get a review or get the final approval from Committers.
   - Please follow [ASF Code of Conduct](https://www.apache.org/foundation/policies/conduct) for all communication including (but not limited to) comments on Pull Requests, Mailing list and Slack.
   - Be sure to read the [Airflow Coding style]( https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#coding-style-and-best-practices).
   - Always keep your Pull Requests rebased, otherwise your build might fail due to changes not related to your commits.
   Apache Airflow is a community-driven project and together we are making it better 🚀.
   In case of doubts contact the developers at:
   Mailing List: dev@airflow.apache.org
   Slack: https://s.apache.org/airflow-slack
   


-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade helm chart pgbouncer_exporter version to 0.18.0 [airflow]

Posted by "Jarvl (via GitHub)" <gi...@apache.org>.
Jarvl commented on PR #47416:
URL: https://github.com/apache/airflow/pull/47416#issuecomment-2704837686

   > I think we never released chart with 1.24
   
   Ah good point, I should have checked the release notes!


-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade helm chart pgbouncer_exporter version to 0.18.0 [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #47416:
URL: https://github.com/apache/airflow/pull/47416#issuecomment-2704826367

   I think we never released chart with 1.24


-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade pgbouncer_exporter version to 0.18.0 [airflow]

Posted by "Jarvl (via GitHub)" <gi...@apache.org>.
Jarvl commented on PR #47416:
URL: https://github.com/apache/airflow/pull/47416#issuecomment-2704601218

   > ok. Pushed the image. `apache/airflow:airflow-pgbouncer-exporter-2025.03.05-0.18.0` - can you please modfy helm-chart and change the defaults?
   
   To clarify - do you mean as part of this PR or a separate PR?


-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Update pgbouncer to 1.23.1 and pgbouncer_exporter to 0.18.0 in helm chart [airflow]

Posted by "Jarvl (via GitHub)" <gi...@apache.org>.
Jarvl commented on PR #47416:
URL: https://github.com/apache/airflow/pull/47416#issuecomment-2707167926

   > Me :)
   
   Thank you so much for your help and attention!


-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade helm chart pgbouncer_exporter version to 0.18.0 [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #47416:
URL: https://github.com/apache/airflow/pull/47416#issuecomment-2704792631

   Let's decrease it to 1.23 in this PR


-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade pgbouncer_exporter version to 0.18.0 [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #47416:
URL: https://github.com/apache/airflow/pull/47416#issuecomment-2704602798

   As part of this PR is best :)


-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Update pgbouncer to 1.23.1 and pgbouncer_exporter to 0.18.0 in helm chart [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk merged PR #47416:
URL: https://github.com/apache/airflow/pull/47416


-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Update pgbouncer to 1.23.1 and pgbouncer_exporter to 0.18.0 in helm chart [airflow]

Posted by "Jarvl (via GitHub)" <gi...@apache.org>.
Jarvl commented on PR #47416:
URL: https://github.com/apache/airflow/pull/47416#issuecomment-2704959977

   > Note that the pgbouncer Dockerfile needed changes to support the tag name used upstream so those changes are included in [7457aa6](https://github.com/apache/airflow/commit/7457aa65886a05a79e2dd1cc3469b0f48a964469).
   > 
   > For example: https://github.com/pgbouncer/pgbouncer/releases/tag/pgbouncer_1_23_1-fixed
   > 
   > Didn't test that the build works locally though, still working on getting my dev environment set up.
   
   I validated locally that the `build_and_push.sh` script works as intended 👍 


-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Update pgbouncer to 1.23.1 and pgbouncer_exporter to 0.18.0 in helm chart [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #47416:
URL: https://github.com/apache/airflow/pull/47416#issuecomment-2705005354

   Built and pushed, approved workflows


-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade helm chart pgbouncer_exporter version to 0.18.0 [airflow]

Posted by "Jarvl (via GitHub)" <gi...@apache.org>.
Jarvl commented on PR #47416:
URL: https://github.com/apache/airflow/pull/47416#issuecomment-2704811059

   > Let's decrease it to 1.23 in this PR
   
   How comfortable are we with doing this and potentially introducing a breaking change into the helm chart? pgbouncer [1.24](https://github.com/pgbouncer/pgbouncer/releases/tag/pgbouncer_1_24_0) introduces new features so decrementing the version in the helm chart could be an unexpected breaking change for people using the defaults.


-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade pgbouncer_exporter version to 0.18.0 [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #47416:
URL: https://github.com/apache/airflow/pull/47416#issuecomment-2704593887

   ok. Pushed the image. `apache/airflow:airflow-pgbouncer-exporter-2025.03.05-0.18.0`  - can you please modfy helm-chart and change the defaults?


-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade helm chart pgbouncer_exporter version to 0.18.0 [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on code in PR #47416:
URL: https://github.com/apache/airflow/pull/47416#discussion_r1983901326


##########
chart/newsfragments/47416.misc.rst:
##########
@@ -0,0 +1 @@
+The chart uses newer version of pgbouncer exporter - ``airflow-pgbouncer-exporter-2025.03.05-0.18.0``

Review Comment:
   ```suggestion
   The chart uses newer version of pgbouncer exporter - ``airflow-pgbouncer-exporter-2025.03.05-0.18.0``
   
   ```



-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Update pgbouncer to 1.23.1 and pgbouncer_exporter to 0.18.0 in helm chart [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #47416:
URL: https://github.com/apache/airflow/pull/47416#issuecomment-2707023006

   Me :) 


-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Update pgbouncer to 1.23.1 and pgbouncer_exporter to 0.18.0 in helm chart [airflow]

Posted by "boring-cyborg[bot] (via GitHub)" <gi...@apache.org>.
boring-cyborg[bot] commented on PR #47416:
URL: https://github.com/apache/airflow/pull/47416#issuecomment-2707023256

   Awesome work, congrats on your first merged pull request! You are invited to check our [Issue Tracker](https://github.com/apache/airflow/issues) for additional contributions.
   


-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade helm chart pgbouncer_exporter version to 0.18.0 [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #47416:
URL: https://github.com/apache/airflow/pull/47416#issuecomment-2704649467

   Let's see if it passes the tests :)


-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade pgbouncer_exporter version to 0.18.0 [airflow]

Posted by "Jarvl (via GitHub)" <gi...@apache.org>.
Jarvl commented on PR #47416:
URL: https://github.com/apache/airflow/pull/47416#issuecomment-2704580428

   @potiuk tagging you since you pushed the image for the last version bump to pgbouncer_exporter in https://github.com/apache/airflow/pull/40303


-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade pgbouncer_exporter version to 0.18.0 [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #47416:
URL: https://github.com/apache/airflow/pull/47416#issuecomment-2704581643

   Yep. I am just on it :)
   
   
   


-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade helm chart pgbouncer_exporter version to 0.18.0 [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #47416:
URL: https://github.com/apache/airflow/pull/47416#issuecomment-2704795002

   With some comments left and link to the issue you opened -> then I can  build and push the image - initially it will fail when you make the change but then I can push the image and restart failing jobs to pick up the new image


-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade helm chart pgbouncer_exporter version to 0.18.0 [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #47416:
URL: https://github.com/apache/airflow/pull/47416#issuecomment-2704827700

   We have not released chart for a looooong time.


-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Upgrade helm chart pgbouncer_exporter version to 0.18.0 [airflow]

Posted by "Jarvl (via GitHub)" <gi...@apache.org>.
Jarvl commented on PR #47416:
URL: https://github.com/apache/airflow/pull/47416#issuecomment-2704880507

   Note that the pgbouncer Dockerfile needed changes to support the tag name used upstream. Didn't test that the build works locally though, still working on getting my dev environment set up.


-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Update pgbouncer to 1.23.1 and pgbouncer_exporter to 0.18.0 in helm chart [airflow]

Posted by "Jarvl (via GitHub)" <gi...@apache.org>.
Jarvl commented on PR #47416:
URL: https://github.com/apache/airflow/pull/47416#issuecomment-2706951895

   @potiuk Who would be the best person to ping for a review on this PR?


-- 
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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org