Skip to content
This repository was archived by the owner on May 21, 2026. It is now read-only.

Fix missing osdo-results artifact causing workflow failures#2

Merged
Spectertj merged 3 commits into
feature/refactor-actionsfrom
copilot/sub-pr-1
Nov 28, 2025
Merged

Fix missing osdo-results artifact causing workflow failures#2
Spectertj merged 3 commits into
feature/refactor-actionsfrom
copilot/sub-pr-1

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Nov 28, 2025

The workflow downloads osdo-results artifact in three jobs (test-quality, security-scan, build-security) but never uploads it, causing "No artifact found" failures.

Changes

  • Added artifact upload in setup job: Uploads the results directory created by setup-osdo action as osdo-results artifact with if-no-files-found: warn to handle empty directories
  • Made downloads resilient: Added continue-on-error: true to all download steps to prevent workflow abortion when artifact is missing or empty
# In setup job
- name: Upload Results Directory
  uses: actions/upload-artifact@v4
  with:
    name: osdo-results
    path: ${{ steps.osdo-setup.outputs.results-dir }}
    if-no-files-found: warn

# In consuming jobs
- name: Download Results Directory
  uses: actions/download-artifact@v4
  continue-on-error: true
  with:
    name: osdo-results
    path: ${{ needs.setup.outputs.results-dir }}

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits November 28, 2025 11:17
Co-authored-by: Spectertj <1765371+Spectertj@users.noreply.github.com>
Co-authored-by: Spectertj <1765371+Spectertj@users.noreply.github.com>
Copilot AI changed the title [WIP] Update to address feedback on feature/refactor-actions PR Fix missing osdo-results artifact causing workflow failures Nov 28, 2025
Copilot AI requested a review from Spectertj November 28, 2025 11:22
@Spectertj Spectertj marked this pull request as ready for review November 28, 2025 11:36
@Spectertj Spectertj merged commit e413712 into feature/refactor-actions Nov 28, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants