jeff-cook commented on issue #62592:
URL: https://github.com/apache/airflow/issues/62592#issuecomment-4024991440

   > > I tool a quick look at the PR and didn't see ports in the container spec 
yet.
   > 
   > It will not have one, as it is not required for probes to work, but the 
logic with HTTP_BIND, together with tests, will be there. The current content 
of the PR is the final one. Feel free to take a look at the HTTP_BIND-related 
part
   
   Ok, thanks. I can configure the bind and metrics via environment variables 
with the helm chart. 
   
   ```
   dags:
     gitSync:
         env:
           - name: GITSYNC_HTTP_BIND
             value: ":1234"
           - name: GITSYNC_HTTP_METRICS
             value: "true"
   ```
   
   But what I can't do is expose the port so prometheus can scape the metrics. 


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to