airflow icon indicating copy to clipboard operation
airflow copied to clipboard

Add options to Docker Operator

Open cardosofede opened this issue 2 years ago • 8 comments

Description

I'm trying to add options like log-opt max-size 5 and I can't.

Use case/motivation

I'm working in Hummingbot and I would like to offer the community a system to manage multiple bots, rebalance portfolio, etc. Our system needs a terminal to execute commands so currently I'm not able to use airflow to accomplish this task.

Related issues

No response

Are you willing to submit a PR?

  • [ ] Yes I am willing to submit a PR!

Code of Conduct

cardosofede avatar Jul 08 '22 12:07 cardosofede

Thanks for opening your first issue here! Be sure to follow the issue template!

boring-cyborg[bot] avatar Jul 08 '22 12:07 boring-cyborg[bot]

I marked it as good first issue and if someone would like to pick it they might. Hower just so you be aware how open-source work - if no-one will pick it up, it won't be implemented, so the most sure way of getting this feature if you need it, is that you or your company will invest time and effort in implementing it and make contribution - PR implementing what you think should be added.

potiuk avatar Jul 08 '22 13:07 potiuk

Thank you!

cardosofede avatar Jul 08 '22 17:07 cardosofede

I'm brand new to the Airflow contributors community and wanted to submit a pull request, can I take this issue?

yingw787 avatar Jul 12 '22 02:07 yingw787

Feel free @yingw787 . Assigned you :)

potiuk avatar Jul 12 '22 06:07 potiuk

wanted to give an update, no progress yet due to work having a tight deadline, but will see how much I can get done in the next week; I did have trouble getting started when I tried to run breeze, not sure whether I should create an issue or if it's something simpler: https://apache-airflow.slack.com/archives/CCQ7EGB1P/p1657592430809019

yingw787 avatar Jul 15 '22 01:07 yingw787

yo...honestly not sure if I have the bandwidth to take this on, pls feel free to reassign

yingw787 avatar Aug 06 '22 01:08 yingw787

@potiuk I can take this up.

nshetty15 avatar Aug 06 '22 05:08 nshetty15