Closed OrangeDog closed 3 years ago
Partial thread in Corp Slack for Salt Project: Pedro Algarvio It's related to the fact that of too many changes files are passed, it blows up (edited)
Daniel Wozniak Maybe we should shove those changed files into an environment variable
Daniel Wozniak or a file
Pedro Algarvio the test suite is prepared for a file, as long as we pass it as a full path
Pedro Algarvio but that might require changes in either the CI pipeline jobs/libs or salt kitchen
Daniel Wozniak Right, in any case, it's something we should address
Pedro Algarvio yes
Ok. Now, it's fixed.
It looks like the test runner builds a lost of tests to run based on changed files in a PR, but this can exceed the max command line limit for cmd.exe.
It needs to construct/pass this list some other way to avoid limits.
https://github.com/saltstack/salt/pull/59034/checks?check_run_id=2003456215