Skip to content

Commit 34e6179

Browse files
committed
address merge conflicts
2 parents 01d5da1 + ff7acd5 commit 34e6179

File tree

304 files changed

+8832
-3078
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

304 files changed

+8832
-3078
lines changed

.github/actions/pull_images/action.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ runs:
3939

4040
- name: clean-up
4141
shell: bash
42-
if: inputs.library == 'php' # PHP use heavy images
42+
if: inputs.library == 'php' || inputs.library == 'ruby' || inputs.library == 'golang' # PHP/ruby/golang use heavy images
4343
run: sudo rm -rf /usr/local/lib/android
4444

4545
#retry

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,6 @@ jobs:
6666
strategy:
6767
matrix:
6868
include: ${{ fromJson(needs.compute_libraries_and_scenarios.outputs.library_matrix) }}
69-
7069
fail-fast: false
7170
uses: ./.github/workflows/system-tests.yml
7271
permissions:
@@ -87,6 +86,7 @@ jobs:
8786
library: ${{ matrix.library }}
8887
scenarios: ${{ needs.compute_libraries_and_scenarios.outputs.scenarios }}
8988
scenarios_groups: ${{ needs.compute_libraries_and_scenarios.outputs.scenarios_groups }}
89+
excluded_scenarios: OTEL_COLLECTOR_E2E # rely on real backend
9090
parametric_job_count: ${{ matrix.version == 'dev' && 2 || 1 }} # test both use cases
9191
skip_empty_scenarios: true
9292
display_summary: true

.github/workflows/debug-harness.yml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,13 +23,18 @@ on:
2323
description: How many jobs
2424
default: "10"
2525
type: string
26+
dev_or_prod:
27+
description: Run scenario against last release (prod) or last commit on main (dev)?
28+
default: "prod"
29+
type: string
2630

2731
env:
2832
LIBRARY: ${{ (github.event_name == 'workflow_dispatch' && inputs.library) || 'php' }}
2933
WEBLOG: ${{ (github.event_name == 'workflow_dispatch' && inputs.weblog) || 'apache-mod-8.1' }}
3034
SCENARIO: ${{ (github.event_name == 'workflow_dispatch' && inputs.scenario) || 'DEBUGGER_EXPRESSION_LANGUAGE' }}
3135
JOB_COUNT: ${{ (github.event_name == 'workflow_dispatch' && inputs.job_count) || '10' }}
3236
AGENT_IMAGE: ${{ (github.event_name == 'workflow_dispatch' && inputs.agent_image) || 'datadog/agent:latest' }}
37+
DEV_OR_PROD: ${{ (github.event_name == 'workflow_dispatch' && inputs.dev_or_prod) || 'prod' }}
3338

3439
jobs:
3540
generate-matrix:
@@ -51,6 +56,9 @@ jobs:
5156
run: echo "${{ secrets.GITHUB_TOKEN }}" > "$RUNNER_TEMP/github_token.txt"
5257
- name: Select agent image
5358
run: echo ${{ env.AGENT_IMAGE }} > binaries/agent-image
59+
- name: Get dev artifact
60+
if: env.DEV_OR_PROD == 'dev'
61+
run: ./utils/scripts/load-binary.sh ${{ env.LIBRARY }}
5462
- name: Build weblog
5563
id: build
5664
run: SYSTEM_TEST_BUILD_ATTEMPTS=3 ./build.sh ${{ env.LIBRARY }} -i weblog -w ${{ env.WEBLOG }} -s --github-token-file "$RUNNER_TEMP/github_token.txt"

.github/workflows/lint.yml

Lines changed: 5 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,11 @@ jobs:
1313
uses: ./.github/actions/install_runner
1414
- run: source venv/bin/activate
1515
shell: bash
16-
- name: Black, pylint, tailing whitespaces, and yaml checks
16+
- name: Install node
17+
uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0
18+
with:
19+
node-version: 20
20+
- name: Black, pylint, tailing whitespaces, yaml, shellcheck, and language-specific checks
1721
shell: bash
1822
run: ./format.sh --check
1923
- if: ${{ failure() }}
@@ -24,25 +28,3 @@ jobs:
2428
exit 1
2529
2630
'
27-
- name: 'Install shellcheck'
28-
shell: bash
29-
run: sudo apt-get install -y shellcheck
30-
- name: 'Run shellcheck'
31-
shell: bash
32-
run: ./utils/scripts/shellcheck.sh
33-
- name: Install node
34-
uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0
35-
with:
36-
node-version: 20
37-
- name: 'Run nodejs express lint'
38-
shell: bash
39-
working-directory: ./utils/build/docker/nodejs/express
40-
run: |
41-
npm install
42-
npm run lint
43-
- name: 'Run nodejs fastify lint'
44-
shell: bash
45-
working-directory: ./utils/build/docker/nodejs/fastify
46-
run: |
47-
npm install
48-
npm run lint

.github/workflows/run-end-to-end.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -472,6 +472,11 @@ jobs:
472472
- name: Run OTEL_COLLECTOR scenario
473473
if: always() && steps.build.outcome == 'success' && contains(inputs.scenarios, '"OTEL_COLLECTOR"')
474474
run: ./run.sh OTEL_COLLECTOR
475+
- name: Run OTEL_COLLECTOR_E2E scenario
476+
if: always() && steps.build.outcome == 'success' && contains(inputs.scenarios, '"OTEL_COLLECTOR_E2E"')
477+
run: ./run.sh OTEL_COLLECTOR_E2E
478+
env:
479+
DD_API_KEY: ${{ secrets.DD_API_KEY }}
475480

476481
- name: Run all scenarios in replay mode
477482
if: success() && steps.build.outcome == 'success' && inputs._enable_replay_scenarios

.github/workflows/system-tests.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -187,7 +187,7 @@ jobs:
187187
name: Build end-to-end (${{ matrix.weblog.name }})
188188
needs:
189189
- compute_parameters
190-
if: ${{ needs.compute_parameters.outputs.endtoend_defs_parallel_enable == 'true' }}
190+
if: ${{ needs.compute_parameters.outputs.endtoend_defs_parallel_enable == 'true' && needs.compute_parameters.outputs.endtoend_defs_parallel_weblogs != '[]'}}
191191
runs-on: ubuntu-latest
192192
strategy:
193193
matrix:
@@ -249,7 +249,7 @@ jobs:
249249
needs:
250250
- compute_parameters
251251
- build_end_to_end
252-
if: ${{ needs.compute_parameters.outputs.endtoend_defs_parallel_enable == 'true' }}
252+
if: ${{ always() && needs.build_end_to_end.result != 'failure' && needs.compute_parameters.result != 'failure' && needs.compute_parameters.outputs.endtoend_defs_parallel_enable == 'true' }}
253253
strategy:
254254
matrix:
255255
job: ${{ fromJson(needs.compute_parameters.outputs.endtoend_defs_parallel_jobs) }}

.promptfoo/local_cursor_provider.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ class ResponseParsingError(Exception):
5757
def _raise_invalid_format_error() -> None:
5858
"""Helper function to raise invalid format error."""
5959
raise InvalidYAMLFormatError(
60-
"Invalid YAML format - expected a list of prompt-output pairs " "or a 'responses' key containing the list"
60+
"Invalid YAML format - expected a list of prompt-output pairs or a 'responses' key containing the list"
6161
)
6262

6363

.shellcheck

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,4 +33,5 @@ TODO=(
3333
utils/interfaces/schemas/serve.sh
3434
build.sh
3535
format.sh
36+
*node_modules*
3637
)

docs/edit/scenarios.md

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -13,19 +13,10 @@ class CustomScenario(Scenario):
1313
"""
1414
If needed, configure the context => mainly, only get infos from config
1515
At this point, logger.stdout is unavailable, so this function should not fail, unless
16-
there is some config error from the user
17-
"""
16+
there is some config error from the user.
1817
19-
def get_warmups(self):
20-
"""
21-
Use this function to start anything needed to your scenario (build, run targets)
22-
This function returns a list of callable that will be called sequentially
18+
If any warmup must be executed at startup, they must be added here using self.warmups.add()
2319
"""
24-
warmups = super().get_warmups()
25-
26-
warmups.append(self.start_target)
27-
28-
return warmups
2920

3021
def post_setup(self, session):
3122
""" called after setup functions, and before test functions """

docs/execute/binaries.md

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -138,15 +138,9 @@ Then run the tests from the repo root folder:
138138

139139
## Python library
140140

141-
1. Add a file `binaries/python-load-from-pip`, the content will be installed by pip. Content example:
142-
* `ddtrace @ git+https://github.com/DataDog/dd-trace-py.git`
143-
2. Add a `.tar.gz` or a `.whl` file in `binaries`, pip will install it
144-
3. Clone the dd-trace-py repo inside `binaries`
141+
1. Add a `.tar.gz` or a `.whl` file in `binaries`, pip will install it
142+
2. Clone the dd-trace-py repo inside `binaries`
145143

146-
You can also run:
147-
```bash
148-
echo “ddtrace @ git+https://github.com/DataDog/dd-trace-py.git@<name-of-your-branch>> binaries/python-load-from-pip
149-
```
150144

151145
## Ruby library
152146

0 commit comments

Comments
 (0)