Skip to content

Fix/user executor varcheck#871

Merged
yesnault merged 6 commits into
ovh:masterfrom
socotra:fix/user-executor-varcheck
Sep 22, 2025
Merged

Fix/user executor varcheck#871
yesnault merged 6 commits into
ovh:masterfrom
socotra:fix/user-executor-varcheck

Conversation

@ivan-velasco

@ivan-velasco ivan-velasco commented Jun 23, 2025

Copy link
Copy Markdown
Contributor

User executor setting null on input var was not storing properly the default input var on the executor

    - type: get_foo_executor
      var1: null

User executor definition

executor: get_foo_executor
input:
  var1: "2020-01-01T00:00:00Z"
  file: {}

ncharaf and others added 5 commits June 23, 2025 15:47
Signed-off-by: Nabil <26463615+ncharaf@users.noreply.github.com>
Signed-off-by: Ivan Velasco <ivan.velasco@socotra.com>
Signed-off-by: Ivan Velasco <ivan.velasco@socotra.com>
This reverts commit 3e3f43e, reversing
changes made to 044ac99.

Signed-off-by: Ivan Velasco <ivan.velasco@socotra.com>
Signed-off-by: Gabriel Augendre <gabriel.augendre@ovhcloud.com>
Signed-off-by: Ivan Velasco <ivan.velasco@socotra.com>
…nputs

Signed-off-by: Ivan Velasco <ivan.velasco@socotra.com>
@ivan-velasco ivan-velasco force-pushed the fix/user-executor-varcheck branch from 3f9f641 to ef2c8ad Compare June 23, 2025 21:48
@yesnault yesnault merged commit f257f5e into ovh:master Sep 22, 2025
2 checks passed
@yesnault

Copy link
Copy Markdown
Member

Thank you @ivan-velasco

@ivan-velasco ivan-velasco deleted the fix/user-executor-varcheck branch February 12, 2026 17:12
ivan-velasco added a commit to socotra/venom that referenced this pull request Mar 25, 2026
Signed-off-by: Ivan Velasco <ivan.velasco@socotra.com>
storagebits pushed a commit to storagebits/venom that referenced this pull request Apr 17, 2026
ivan-velasco added a commit to socotra/venom that referenced this pull request Jun 6, 2026
Signed-off-by: Ivan Velasco <ivan.velasco@socotra.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants