Skip to content

Conversation

@JoshuaL3000
Copy link
Collaborator

@JoshuaL3000 JoshuaL3000 commented Nov 8, 2024

Description

Add example workflow serving API setup scripts under WorkflowExecAgent/tests/example_workflow for users to setup local API server as the SDK class URL.

Type of change

List the type of change like below. Please delete options that are not relevant.

  • Others (enhancement, documentation, validation, etc.)

Tests

Verified results with /WorkflowExecAgent/tests/test_compose_vllm_example_wf_xeon.sh. This test script will be run in the CICD.

JoshuaL3000 and others added 17 commits September 30, 2024 05:24
* Use single tool implementation
* Add CICD validation script
* Update example readme

Signed-off-by: JoshuaL3000 <[email protected]>
Signed-off-by: JoshuaL3000 <[email protected]>
* Add files for example workflow
* Add test scripts for example workflow CICD
* Update tool docstrings

Signed-off-by: JoshuaL3000 <[email protected]>
* Add files for example workflow
* Add test scripts for example workflow CICD
* Update tool docstrings
* Update readme and add launch workflow example guide

Signed-off-by: JoshuaL3000 <[email protected]>
@JoshuaL3000 JoshuaL3000 changed the title Workflow executor example Workflow executor example workflow API Nov 8, 2024
Signed-off-by: JoshuaL3000 <[email protected]>
@ftian1 ftian1 added the r1.2 OPEA 1.2 RELEASE TAG label Nov 11, 2024
@ftian1
Copy link
Collaborator

ftian1 commented Nov 11, 2024

@JoshuaL3000 I will temporarily put it under OPEA 1.2 release tag as OPEA 1.1 has been in feature freeze phase.

@github-actions
Copy link

github-actions bot commented Dec 23, 2024

Dependency Review

The following issues were found:
  • ❌ 1 vulnerable package(s)
  • ✅ 0 package(s) with incompatible licenses
  • ✅ 0 package(s) with invalid SPDX license definitions
  • ⚠️ 1 package(s) with unknown licenses.
See the Details below.

Vulnerabilities

WorkflowExecAgent/tests/example_workflow/requirements.txt

NameVersionVulnerabilitySeverity
requests2.28.1Unintended leak of Proxy-Authorization header in requestsmoderate
Requests `Session` object does not verify requests after making first request with verify=Falsemoderate
Requests vulnerable to .netrc credentials leak via malicious URLsmoderate

License Issues

WorkflowExecAgent/tests/example_workflow/requirements.txt

PackageVersionLicenseIssue Type
numpyNullUnknown License

Scanned Files

  • WorkflowExecAgent/tests/example_workflow/requirements.txt

@JoshuaL3000
Copy link
Collaborator Author

Hi @ftian1 and @lvliang-intel could I get your help to review this PR to merge for 1.2 release? chensuyue is updating the CI to skip the Dockerfile check under tests/ folder

Signed-off-by: JoshuaL3000 <[email protected]>
@JoshuaL3000 JoshuaL3000 force-pushed the workflow-executor-example branch from 22383a9 to b32ffe6 Compare June 4, 2025 07:04
@yinghu5
Copy link
Collaborator

yinghu5 commented Jun 13, 2025

@JoshuaL3000 thank you a lot for addressing the issues. Almost done, only 4 conficts, please help to fix them.

@yinghu5 yinghu5 added this to the v1.4 milestone Jun 13, 2025
@JoshuaL3000
Copy link
Collaborator Author

JoshuaL3000 commented Jun 13, 2025

Hi @yinghu5, conflicts are fixed and E2E test has passed. Could I get your help to review? Thanks

Copy link
Collaborator

@yinghu5 yinghu5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you!

@JoshuaL3000 JoshuaL3000 merged commit 5457289 into opea-project:main Jul 2, 2025
15 checks passed
zhihangdeng pushed a commit to zhihangdeng/GenAIExamples that referenced this pull request Jul 29, 2025
Signed-off-by: JoshuaL3000 <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: xiguiw <[email protected]>
Co-authored-by: chen, suyue <[email protected]>
Co-authored-by: ZePan110 <[email protected]>
alexsin368 pushed a commit to alexsin368/GenAIExamples that referenced this pull request Aug 13, 2025
Signed-off-by: JoshuaL3000 <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: xiguiw <[email protected]>
Co-authored-by: chen, suyue <[email protected]>
Co-authored-by: ZePan110 <[email protected]>
Signed-off-by: alexsin368 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

r1.2 OPEA 1.2 RELEASE TAG

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants