Skip to content

fix: ensure when there's a dupe function name during a perf profile run that program exits #96

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 20 commits into from
Jan 10, 2022
Merged

fix: ensure when there's a dupe function name during a perf profile run that program exits #96

Changes from all commits
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
2808a6d
feat: initial commit
Nov 22, 2021
da9a783
feat: in progress
Nov 23, 2021
c2c701a
feat: in progress
Nov 23, 2021
0da557b
fix: remove
Nov 23, 2021
422501f
fix: remove
Nov 24, 2021
3d17504
fix: code from wrong branch
Nov 30, 2021
4eca311
Merge branch 'main' into astorm/cli-profile
Dec 6, 2021
94e8275
removing
Dec 6, 2021
fbaf423
feat: generating zip file from local source
Dec 6, 2021
528d8e9
feat: working loop, removing cruft
Dec 7, 2021
05691dc
feat: remove actual config
Dec 7, 2021
41687ec
feat: add elasticagent scenario
Dec 7, 2021
0952a81
Merge branch 'main' into astorm/cli-profile
Dec 7, 2021
9f3a8ce
feat: lint
Dec 7, 2021
72e4af5
feat: finished profiler and first docs pass
Dec 23, 2021
af84c2f
Merge branch 'main' into astorm/cli-profile
Dec 23, 2021
49ac4b3
fix: lint and extra file
Dec 23, 2021
5066a37
fix: typos
Dec 23, 2021
45c7f9c
fix: control flow problems when not building extension
Jan 6, 2022
e108152
Merge branch 'main' into astorm/control-flow-profile
Jan 10, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions apm-lambda-extension/cli/profile.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -141,11 +141,12 @@ async function runScenario (scenario, config) {
}
)
})
}).catch(function (e) {
}).catch(async function (e) {
console.log('Error creating function')
if (e.statusCode === 409 && e.code === 'ResourceConflictException') {
console.log('Function already exists, deleting. Rerun profiler.')
cleanup(functionName)
await cleanup(functionName)
process.exit(1)
} else {
console.log(e)
}
Expand Down Expand Up @@ -176,6 +177,7 @@ function buildAndDeployArn (config) {

if (arns.indexOf(FLAG_LATEST) === -1) {
resolve()
return
}

// build latest arn, modify config to include it
Expand Down