yoyama comments

Results 23 comments of


                                            yoyama

Reopen "get k8s config from requestConfig. and added test."

IIUC, this PR will allow to configure K8 in workflow definition ? If so, I would like to ask you to add server configuration to enable/disable of it and default...

Reopen "get k8s config from requestConfig. and added test."

@hnarimiya Could you rebase your PR to the latest master and fix the CI failures ?

/api/schedules does not support pagination

@kiyoto Same with #1106 . We will discuss about it.

Backfill sessions are not sequential after unpause

I would like to propose introducing **wait_until_last_schedule** option in schedule as follows. https://github.com/treasure-data/digdag/compare/master...yoyama:feature-wait_until_last_schedule If this option is true and there is active attempt, schedule will be delayed. As result, only...

Backfill sessions are not sequential after unpause

Thank you for your proposal on the name of option. I don't know which is better. As you mention, wait_on_overtime match with skip_on_overtime. But wait_until_last_schedule may be easy to understand....

Operations within "for_each>:" operator executed twice when retried

Thanks for the report. I also confirmed your procedure. I need more time to investigate it.

Incorrect description of the `last_id` query param on GET /api/attempts endpoint.

Hi, thank you for your report. I confirmed your report is right. We will fix it later.

Drop Python2

@chezou Thank you for your PR. I agree with drop of Python2. The failure on build of Docker image may be fixed in #1740 . Anyway we may need to...

Add language api class init examples

@KentFujii Thank you for your contribution. I think it might be better to describe what you want to explain in this example. I think adding some comments to examples is...

[Doc] Add missing server options.

IIUC, admin function is deprecated. So it might be better to add (deprecated).