-
Notifications
You must be signed in to change notification settings - Fork 10
[e2e] add e2e tests #98
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.
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need a "install as much as possible" use case as well?
Regarding the install_script url, is it possible to pass a custom one (in case we don't necessarily put the script in custom
?
test/e2e/agentplatform.go
Outdated
} | ||
|
||
const ( | ||
scriptURL = "https://dd-agent.s3.amazonaws.com/custom" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It means we might state somewhere we need to push the install_scripts to be tested in the custom
folder prior launching the tests, correct?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
True, will change it in a follow up PR to take a param or an env var
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually will move it in this PR to a flag
Meant to run locally, for now
3b8dc0e
to
3efb18f
Compare
I'm closing this PR and opening a new, smaller one. Will add up based on it |
Meant to run locally, for now
They implement in go https://github.com/DataDog/experimental/blob/main/teams/agent-platform/package-scripts-test/install_test.sh