Skip to content

Conversation

AAraKKe
Copy link
Contributor

@AAraKKe AAraKKe commented Aug 14, 2025

What does this PR do?

This fixes the title of the jobs launched on test during the test-agent-target workflow.

Motivation

The other job name, while useful, is not valid to upload artifacts because the agent image has a colon which is a forbidden character. See this job

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
  • If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

@AAraKKe AAraKKe added the qa/skip-qa Automatically skip this PR for the next QA label Aug 14, 2025
@AAraKKe AAraKKe requested a review from a team as a code owner August 14, 2025 15:22
@AAraKKe AAraKKe force-pushed the aarakke/fix-test-agent-target branch from 55d7d06 to 91dc87d Compare August 14, 2025 15:22
steveny91
steveny91 previously approved these changes Aug 14, 2025
@AAraKKe AAraKKe force-pushed the aarakke/fix-test-agent-target branch from 91dc87d to 40a8076 Compare September 25, 2025 10:17
@temporal-github-worker-1 temporal-github-worker-1 bot dismissed steveny91’s stale review September 25, 2025 10:18

Review from steveny91 is dismissed. Related teams and files:

  • agent-integrations
    • .github/workflows/test-agent-target.yml
@AAraKKe AAraKKe enabled auto-merge September 25, 2025 10:18
@AAraKKe AAraKKe added this pull request to the merge queue Sep 25, 2025
Merged via the queue into master with commit bc2bb16 Sep 25, 2025
23 checks passed
@AAraKKe AAraKKe deleted the aarakke/fix-test-agent-target branch September 25, 2025 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants