maroshmka commented on pull request #11793:
URL: https://github.com/apache/airflow/pull/11793#issuecomment-717088709
thank you very much for such an thorough explanation, it clears a lot stuff
in my head!
> Each DAG example should allow bucket to be changed using an environment
variable. In this particular case, it did not happen, and I think it is worth
adding this possibility.
amended with possibility to take it from env vars, this was missing and
thats why I got stuck to most :/
> We don't want random names generated every time we run the test as this
will make debugging bugs difficult. The bucket name should only be generated
once on a given computer to make behavior more predictable.
agree. my idea was to append some random suffix when starting breeze and set
the env vars, e.g. `bucket_test_{str(uuid.uuid4())[:6]}`. as you are already
working on it, its cool I guess :)
that'd be all from my side, so you can test 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.
For queries about this service, please contact Infrastructure at:
[email protected]
