Skip to content

[DLStreamer] CI reorganization - aligning and merging DLS Pipeline Server to DLS#1088

Merged
dmichalo merged 72 commits intomainfrom
dls_dmichalo_IntegrateDlsPsCI
Oct 23, 2025
Merged

[DLStreamer] CI reorganization - aligning and merging DLS Pipeline Server to DLS#1088
dmichalo merged 72 commits intomainfrom
dls_dmichalo_IntegrateDlsPsCI

Conversation

@dmichalo
Copy link
Copy Markdown
Contributor

Description

Please include a summary of the changes and the related issue. List any dependencies that are required for this change.

Fixes # (issue)

Any Newly Introduced Dependencies

Please describe any newly introduced 3rd party dependencies in this change. List their name, license information and how they are used in the project.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Checklist:

  • I agree to use the APACHE-2.0 license for my code changes.
  • I have not introduced any 3rd party components incompatible with APACHE-2.0.
  • I have not included any company confidential information, trade secret, password or security token.
  • I have performed a self-review of my code.

Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
… report upload

Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Copy link
Copy Markdown
Contributor

@nszczygl9 nszczygl9 left a comment

Choose a reason for hiding this comment

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

In my opinion, the current workflow is complex and difficult to understand for team members who aren't familiar with DLS/DLSPS CI processes. Since we already build the DLStreamer image separately for DLSPS, I believe we could improve readability by splitting this into separate, parallel workflows. For example, we could have one PR workflow dedicated to DLS and another for DLSPS, both triggered simultaneously. Or we can separate scans to another workflow and build and test in main worfklow (?) Just a suggestion. @tbujewsk let us know what do you think

@tbujewsk
Copy link
Copy Markdown
Contributor

In my opinion, the current workflow is complex and difficult to understand for team members who aren't familiar with DLS/DLSPS CI processes. Since we already build the DLStreamer image separately for DLSPS, I believe we could improve readability by splitting this into separate, parallel workflows. For example, we could have one PR workflow dedicated to DLS and another for DLSPS, both triggered simultaneously. Or we can separate scans to another workflow and build and test in main worfklow (?) Just a suggestion. @tbujewsk let us know what do you think

I agree, let's split it into two separate workflows.

Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
@dmichalo
Copy link
Copy Markdown
Contributor Author

Separation implemented - both DLS and DLS PS have similar main PR action dls[ps]-pr-workflow.yamls which triggers next actions and workflows.

Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
nszczygl9
nszczygl9 previously approved these changes Oct 22, 2025
tbujewsk
tbujewsk previously approved these changes Oct 22, 2025
@dmichalo dmichalo dismissed stale reviews from tbujewsk and nszczygl9 via 5cf4422 October 23, 2025 07:43
@dmichalo dmichalo merged commit c83f982 into main Oct 23, 2025
37 of 46 checks passed
@dmichalo dmichalo deleted the dls_dmichalo_IntegrateDlsPsCI branch October 23, 2025 08:55
nszczygl9 pushed a commit that referenced this pull request Dec 16, 2025
…rver to DLS (#1088)

Signed-off-by: Michalowski, Dawid <dawid.michalowski@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants