Skip to content
This repository has been archived by the owner on Oct 9, 2023. It is now read-only.

fix region error. fix env vars not propagated to task if -s not provided #30

Merged
merged 3 commits into from
Mar 24, 2020

Conversation

Eli-Goldberg
Copy link
Contributor

  • AWS Region had to be provided by AWS_REGION env var. Now -r flag can be specified.
  • "-e" flag was not honored if "-s" flag was not provided. Now env vars provided with "-e" are propagated to the task as overrides, regardless of the "-s" flag.
  • Lint errors
  • added some common values to .gitignore

@Eli-Goldberg
Copy link
Contributor Author

ecs-run -r us-east-1
-e NODE_ENV=production
-c mycluster
-f input.json
-n myjob
-l /ecs/myjob

and got an error saying --region wasnt provided (aws expected an env var) - so I added that.

Then it ran but my task wasnt getting any of the "-e" flags (NODE_ENV above).

@Eli-Goldberg
Copy link
Contributor Author

Eli-Goldberg commented Feb 10, 2020

@lox
Also I havent updated the help output in the README file.
Will add in GMT+2 am.

@lox
Copy link
Contributor

lox commented Mar 24, 2020

Thanks! Fixed a conflict.

@lox lox merged commit 98e3058 into buildkite:master Mar 24, 2020
dsantibanezvera pushed a commit to dsantibanezvera/ecs-run-task that referenced this pull request Sep 28, 2020
fix region error. fix env vars not propagated to task if -s not provided
dsantibanezvera pushed a commit to dsantibanezvera/ecs-run-task that referenced this pull request Sep 28, 2020
fix region error. fix env vars not propagated to task if -s not provided
dsantibanezvera pushed a commit to dsantibanezvera/ecs-run-task that referenced this pull request Sep 28, 2020
fix region error. fix env vars not propagated to task if -s not provided
dsantibanezvera pushed a commit to dsantibanezvera/ecs-run-task that referenced this pull request Sep 28, 2020
fix region error. fix env vars not propagated to task if -s not provided
dsantibanezvera pushed a commit to dsantibanezvera/ecs-run-task that referenced this pull request Sep 28, 2020
fix region error. fix env vars not propagated to task if -s not provided
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants