yunchipang commented on PR #50292:
URL: https://github.com/apache/airflow/pull/50292#issuecomment-2860913206

   @bugraoz93 Thanks for the review!
   
   Just wanted to clarify—I was using the endpoint below to get all configs 
while implementing the `list` command, and it seemed to be working as expected:
   
https://github.com/apache/airflow/blob/393668361b5fcb974df34a113d5b471ad1c7fa91/airflow-core/src/airflow/api_fastapi/core_api/routes/public/config.py#L97-L98
   
   This is the what the current `list` command gets me.
   ```
   root@24ae150ff6a0:/opt/airflow# airflowctl config list
   Please enter password for encrypted keyring:
   Config(
       sections=[
           ConfigSection(
               name='atlas',
               options=[
                   ConfigOption(key='sasl_enabled', value='False'),
                   ConfigOption(key='host', value=''),
                   ConfigOption(key='port', value='21000'),
                   ConfigOption(key='username', value=''),
                   ConfigOption(key='password', value='')
               ]
           ),
           ConfigSection(
               name='hive',
               options=[ConfigOption(key='default_hive_mapred_queue', value='')]
           ),
           ...
   ```
   If the goal is to format it more like this:
   ```
   [core]
   dags_folder = /opt/airflow/dags
   base_log_folder = /opt/airflow/logs
   ...
   ```
   Would it be correct to assume this is just a matter of formatting? Or am I 
missing something deeper in the implementation? Happy to adjust if needed—just 
want to make sure I fully understand your feedback. Appreciate your insights!


-- 
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