Add support for env variables in executor config about dagu HOT 7 CLOSED

@yohamta Hi! I will try to work on this !

from dagu.

@yohamta What do you think? I think this is a good idea.

1. Duplicate configurations can be managed by reconfiguring based on environment variables
2. This can prevent the direct exposure of sensitive information

from dagu.

Thank you for submitting this issue and suggestions. I completely agree that this would be a valuable feature.

To implement this, we will need to make the following changes:

1. We'll add support for environment variable expansion in the executor config for all executor types available the SSH executor.
2. We'll ensure that the expanded values are not written back to the Node data, as these values contain sensitive information that shouldn't be stored in status files.

If you have any thoughts on this implementation plan or additional considerations, please feel free to share them.

from dagu.

Thank you @halalala222, please feel free to ask if there's any questions!

from dagu.

@yohamta add support for environment variable expansion in the executor config for all executor types available Is it necessary to add this environment variable to executors other than the SSH executor?

from dagu.

Hi @halalala222, I made a mistake earlier - we actually only need support for environment variable expansion in the SSH executor right now. Sorry for the confusion.

from dagu.

@yohamta Hi!!! Thank you ! I completed this issue in this PR

from dagu.