🧪 Updated workflow to accept test name

[sshveta]

Related PR konveyor/tackle2-ui#2829

Summary by CodeRabbit

• Chores
  • Added a new workflow input parameter for specifying custom UI test file paths, allowing overrides to default test configurations.
  • Updated repository references in workflow documentation.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

A new workflow input ui_test_spec was added to the global CI bundle workflow to allow specifying test file paths for UI testing. This input overrides ui_test_tags, is passed to the e2e-ui-integration-tests step, and the ui_tests_ref reference was updated to point to the tackle2-ui repository.

Changes

Cohort / File(s)	Summary
Workflow UI Test Configuration \\.github/workflows/global-ci-bundle.yml	Added new ui_test_spec input (string, default "") to both workflow_call and workflow_dispatch sections for specifying comma-separated test file paths; updated ui_tests_ref description to reference tackle2-ui repository; passed ui_test_spec as test_spec parameter to the UI tests workflow step

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• Update global CI workflows to run UI tests from tackle2-ui repository #177: Introduces/standardizes the centralized run-ui-tests action that this PR integrates with by adding the ui_test_spec parameter and test_spec mapping to the UI test invocation.

Suggested reviewers

• sjd78
• aufi

Poem

🐰 A workflow grows a testing branch,
With spec paths in a comma'd bunch,
From tackle2 the refs now flow,
Let UI tests run smooth and slow! ✨

Pre-merge checks

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: adding a new workflow input to accept test specifications, which allows specifying test names/paths to run.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In @.github/workflows/global-ci-bundle.yml:
- Line 513: The workflow is passing an unsupported input "test_spec" to the
action konveyor/tackle2-ui/.github/actions/run-ui-tests@main; either remove the
test_spec line from the workflow invocation (so only test_tags, tests_ref, and
base_url are passed) or update the action implementation (action metadata and
entrypoint) to declare a "test_spec" input and use it to override the internal
spec generation (ensure the code that currently builds test_spec from test_tags
checks for a non-empty inputs.test_spec and uses that instead).

📜 Review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7a31e9b and adb8a2a.

📒 Files selected for processing (1)

• .github/workflows/global-ci-bundle.yml

🧰 Additional context used

🧠 Learnings (2)

📓 Common learnings

Learnt from: aufi
Repo: konveyor/ci PR: 97
File: .github/workflows/global-ci-bundle.yml:141-145
Timestamp: 2025-06-26T06:08:36.244Z
Learning: In the global-ci-bundle.yml workflow, the base_tag parameter is primarily designed for automated workflow_call executions, not manual workflow_dispatch usage. For workflow_call, operator_bundle defaults to empty so make-bundle runs and uses base_tag. For workflow_dispatch (web UI), operator_bundle has a default value, so users need to manually clear it to use base_tag.

📚 Learning: 2025-06-26T06:08:36.244Z

Learnt from: aufi
Repo: konveyor/ci PR: 97
File: .github/workflows/global-ci-bundle.yml:141-145
Timestamp: 2025-06-26T06:08:36.244Z
Learning: In the global-ci-bundle.yml workflow, the base_tag parameter is primarily designed for automated workflow_call executions, not manual workflow_dispatch usage. For workflow_call, operator_bundle defaults to empty so make-bundle runs and uses base_tag. For workflow_dispatch (web UI), operator_bundle has a default value, so users need to manually clear it to use base_tag.

Applied to files:

• .github/workflows/global-ci-bundle.yml

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)

• GitHub Check: Verify api and ui tests run via global-ci-bundle.yml with default input / e2e-ui-integration-tests
• GitHub Check: Verify api and ui tests run via global-ci-bundle.yml with default input / e2e-api-integration-tests
• GitHub Check: Verify api and ui tests run via global-ci.yml with default inputs / e2e-ui-integration-tests
• GitHub Check: Verify api and ui tests run via global-ci.yml with default inputs / e2e-api-integration-tests

🔇 Additional comments (3)

.github/workflows/global-ci-bundle.yml (3)

147-153: LGTM! Well-structured input parameter.

The new ui_test_spec input is properly defined with clear documentation and an appropriate default value that maintains backward compatibility.

---

230-236: LGTM! Consistent with workflow_call definition.

The ui_test_spec input for workflow_dispatch correctly mirrors the workflow_call definition, maintaining consistency across trigger types.

---

129-129: The repository name in the descriptions (lines 129 and 212) is correct. The konveyor/tackle2-ui repository is the official Konveyor Tackle UI repository, and these descriptions accurately align with the action reference at line 510 (konveyor/tackle2-ui/.github/actions/run-ui-tests@main). No changes needed.

[coderabbitai]

⚠️ Potential issue | 🔴 Critical

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash

# Try to fetch the action.yml file from the tackle2-ui repository
# Using raw GitHub content URL
echo "=== Checking action.yml for test_spec support ==="
curl -s "https://raw.githubusercontent.com/konveyor/tackle2-ui/main/.github/actions/run-ui-tests/action.yml" | head -100

Repository: konveyor/ci

Length of output: 3023

---

🌐 Web query:

konveyor tackle2-ui PR 2829 test_spec parameter github

💡 Result:

I couldn't find a Konveyor tackle2-ui pull request #2829 or any reference to a test_spec parameter matching that PR. Do you have a link, exact repo (GitHub vs mirror), or the PR number to check? I can search GitHub and return the PR details and file/parameter changes once you confirm.

---

The action does not support the test_spec parameter as an input.

The konveyor/tackle2-ui/.github/actions/run-ui-tests@main action only accepts three inputs: test_tags, tests_ref, and base_url. The test_spec parameter passed on line 513 is not defined in the action's inputs and will be silently ignored. The action internally generates test_spec from test_tags via the "Build test spec from test tags" step.

Either update the action to accept and use test_spec as an input parameter (which would allow it to override the tags-based spec generation), or remove this parameter from the workflow call.

🤖 Prompt for AI Agents

In @.github/workflows/global-ci-bundle.yml at line 513, The workflow is passing
an unsupported input "test_spec" to the action
konveyor/tackle2-ui/.github/actions/run-ui-tests@main; either remove the
test_spec line from the workflow invocation (so only test_tags, tests_ref, and
base_url are passed) or update the action implementation (action metadata and
entrypoint) to declare a "test_spec" input and use it to override the internal
spec generation (ensure the code that currently builds test_spec from test_tags
checks for a non-empty inputs.test_spec and uses that instead).