Skip to content

Commit

Permalink
fix(cli): wrong order for cli arn role
Browse files Browse the repository at this point in the history
  • Loading branch information
tchiotludo committed Feb 14, 2025
1 parent 57e768a commit 6fe0f7c
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/main/java/io/kestra/plugin/aws/cli/AwsCLI.java
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,7 @@ public class AwsCLI extends AbstractConnection implements RunnableTask<ScriptOut

@Override
public ScriptOutput run(RunContext runContext) throws Exception {
List<String> allCommands = new ArrayList<>(this.commands);
List<String> allCommands = new ArrayList<>();

// hack for missing env vars supports: https://github.com/aws/aws-cli/issues/5639
if (this.stsRoleArn != null) {
Expand All @@ -169,6 +169,8 @@ public ScriptOutput run(RunContext runContext) throws Exception {
allCommands.add("aws configure set credential_source " + this.stsCredentialSource.value);
}

allCommands.addAll(this.commands);

var renderedOutputFiles = runContext.render(outputFiles).asList(String.class);

CommandsWrapper commands = new CommandsWrapper(runContext)
Expand Down

0 comments on commit 6fe0f7c

Please sign in to comment.