Fixed issues with incorrect work directory and missing environment variables. #17

rlyle · 2020-11-25T20:07:14Z

I tried to use your plugin and hit a few issues.

Added the WORKDIR command when building the docker, so the correct CWD is set when the function runs. Without this, our code had trouble loading the configs for our nodejs application.
Added missing environment variables, like AWS_LAMBDA_FUNCTION_NAME and _X_AMZN_TRACE_ID so XRAY can work correctly for the batch process.

Obviously, you can ignore the package.json changes, as I needed those to be able to pull the plugin into our project from npmjs.

…s files." This reverts commit 2a81f0c.

…R command.

…e batch job.

…tly, but that could be due to the AWS outage.

…ch..

…rmissions to call CreateNetworkInterface on EC2 (Service: AWSLambdaInternal; Status Code: 400; Error Code: InvalidParameterValueException; Request ID: 723a9f5f-cf45-4363-9aa3-8b597ff06cc5; Proxy: null)

…all the JobDefinitions at once. Adding the DependsOn to link each one to the previous, so they will get created one at a time instead.

…e & job definition are pairs, and each one needs to be created at the same time. [skip test][skip subs]

…ecution role. [skip test][skip subs]

… if provided, otherwise use the default from our private ECR.

…stead of defaulting to nodejs16.x.

…stead.

… our batch logs can also get forwarded to the function.

…n each env.

rlyle added 30 commits November 23, 2020 04:11

Unzip the file into the v/ar/task/ folder, so it can find it's files.

2a81f0c

1.0.8

b6f0899

Updating package name.

009e532

Revert "Unzip the file into the v/ar/task/ folder, so it can find it'…

89d7502

…s files." This reverts commit 2a81f0c.

Don't prefix the addtional commands with RUN, so I can use the WORKDI…

c8d7f84

…R command.

1.0.9

e274468

Automatically set the work directory to where the package is extracted.

ae1edbf

1.0.10

474d618

Pass in the function name and XRAY trace ID into the batch job.

8f984bb

1.0.11

1ff01b6

Adding addtional enviroment variables to pass from the lambda into th…

fd0aee1

…e batch job.

1.0.12

9653d0f

Restoring the RUN command, was not needed after all.

2d0694e

Removing the log stream enviornment variables, not seeing logs curren…

02d4d1d

…tly, but that could be due to the AWS outage.

1.0.13

43f69e1

Removing other enironment variables except for the ones we really need.

1bf62b6

1.0.14

72d25aa

Allow the lambda schedule function to use XRAY.

4688297

1.0.15

80a4ea4

Fixed typo.

cebcb26

1.0.16

ec508ba

Adding support for XRAY.

9de1645

1.0.17

e439919

Removing the _X_AMAZ_TRACE_ID, to see if XRAY work inside the AWS bat…

8d97fdb

…ch..

1.0.18

ea321db

Restoring the _X_AMZN_TRACE_ID environment variable.

a535146

1.0.19

53c90a4

Adding AWS_REQUEST_ID to the environment variables to see if this works.

9a7df20

1.0.20

4905942

Trying to fix the error: The provided execution role does not have pe…

988bbaf

…rmissions to call CreateNetworkInterface on EC2 (Service: AWSLambdaInternal; Status Code: 400; Error Code: InvalidParameterValueException; Request ID: 723a9f5f-cf45-4363-9aa3-8b597ff06cc5; Proxy: null)

rlyle and others added 30 commits July 22, 2023 02:29

1.1.5

fee974a

IOA-23313 | Updating to pull from our private respository.

97f7503

1.1.6

4def15d

HOTFIX: Removing the 90 day Lifecycle policy.

c4613ed

1.1.7

a034a54

Ran into a problem with cloud formation failing when it's generating …

feee2d0

…all the JobDefinitions at once. Adding the DependsOn to link each one to the previous, so they will get created one at a time instead.

1.1.8

6f480b0

IOA-23781 | Add the DependsOn to the IAM execution row, since the rol…

13569d5

…e & job definition are pairs, and each one needs to be created at the same time. [skip test][skip subs]

1.1.9

466be19

IOA-23781 | Need to apply the DependsOn in the right place for the ex…

7121f10

…ecution role. [skip test][skip subs]

1.1.10

342e13a

1.1.11

c566bac

Allow the build image name to be pulled from the serverless.yaml file…

ca61097

… if provided, otherwise use the default from our private ECR.

Updating version to 1.1.10

7079385

Merged other changes made on my other machine.

f888930

Updating version to 1.1.12

428ede5

HOTFIX: Added batchImage support as well.

9c0c5e0

Updating to version 1.1.13

8a584b0

Use the runtime from the provider section of the config by default in…

dd4c13d

…stead of defaulting to nodejs16.x.

1.1.14

9feb32f

Default to node v18, use a variable from the custom config section in…

03f546e

…stead.

1.1.15

8f617e0

Updated to use AWS sdk v3.

2dbe4b6

1.1.16

0bb0e4f

MRF-162 | Adding support for the serverless-log-forwarding plugin, so…

3808ae8

… our batch logs can also get forwarded to the function.

1.1.17

685d748

MRF-162 | We are going to need a different log group for batch jobs i…

633686b

…n each env.

1.1.18

d0566dd

MRF-162 | Fixed typo.

93d5ee9

1.1.19

a644b41

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fixed issues with incorrect work directory and missing environment variables. #17

Fixed issues with incorrect work directory and missing environment variables. #17

rlyle commented Nov 25, 2020 •

edited

Loading

Fixed issues with incorrect work directory and missing environment variables. #17

Are you sure you want to change the base?

Fixed issues with incorrect work directory and missing environment variables. #17

Conversation

rlyle commented Nov 25, 2020 • edited Loading

rlyle commented Nov 25, 2020 •

edited

Loading