Skip to content

fix: use correct cliOptions key for scenarioName in Fargate workers#3669

Merged
hassy merged 1 commit intoartilleryio:mainfrom
barth12:fix/scenario-name-fargate
Jan 21, 2026
Merged

fix: use correct cliOptions key for scenarioName in Fargate workers#3669
hassy merged 1 commit intoartilleryio:mainfrom
barth12:fix/scenario-name-fargate

Conversation

@barth12
Copy link
Contributor

@barth12 barth12 commented Dec 18, 2025

Summary

The --scenario-name flag was not being passed correctly to Fargate workers, causing them to receive null instead of the actual scenario name.

Root Cause

In generateTaskOverrides(), the condition checked cliOptions['scenarioName'] (camelCase) but the value was read from cliOptions['scenario-name'] (kebab-case):

// Before (bug):
context.cliOptions['scenarioName']
  ? ['--scenario-name', context.cliOptions['scenario-name']]  // reads undefined → "null"
  : [],

Fix

Use the correct camelCase key consistently:

// After (fix):
context.cliOptions['scenarioName']
  ? ['--scenario-name', context.cliOptions['scenarioName']]
  : [],

Related

Fixes #3589 - this appears to be a regression or incomplete fix from #3651

The condition checked `cliOptions['scenarioName']` (camelCase) but
the value was read from `cliOptions['scenario-name']` (kebab-case),
resulting in `null` being passed to workers.

Fixes artilleryio#3589
@CLAassistant
Copy link

CLAassistant commented Dec 18, 2025

CLA assistant check
All committers have signed the CLA.

@hassy
Copy link
Member

hassy commented Jan 21, 2026

thanks @barth12!

@hassy hassy merged commit 4c4f409 into artilleryio:main Jan 21, 2026
5 of 6 checks passed
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.

--scenario-name option does not work for artillery run-fargate CLI command

3 participants