[GitHub] [airflow] aru-trackunit commented on pull request #28852: Fixing get_conn_uri method - adding ``@`` when login, password, host, port is not given

[GitBox <gi...@apache.org>]

aru-trackunit commented on PR #28852:
URL: https://github.com/apache/airflow/pull/28852#issuecomment-1378590272

   @Taragolis ok, I will revert my changes and update only docs.
   
   As a side note: I am not a python developer and I find contributing pretty difficult. Especially setting up the environment and running tests.
   When setting up a local environment I went into issues like missing ``pygraphviz``, then after executing tests using breeze it was missing some packages. Then `zshrc` script couldn't find `compdef`. Stackoverflow again to the rescue. Finally when I got them to work then some tests failed even when I haven't done any code changes.
   
   When running `breeze testing tests` it took my local aws account and tried to execute something which obviously failed
   ```
   botocore.exceptions.ClientError: An error occurred (AccessDeniedException) when calling the GetParameter operation: User: arn:aws:sts::----:assumed-role/---- is not authorized to perform: ssm:GetParameter on resource: arn:aws:ssm:us-east-1: :parameter/example_step_functions because no identity-based policy allows the ssm:GetParameter action
   ```
   
   It's hard to contribute when you are not very familiar with python. I have a feeling that there is tons of documentation to read, just to make a few line changes. Probably I know to little about the ecosystem to propose solution, but the problems I have faced discourage me from contributing.


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