trigger ci#26
Open
testcara wants to merge 85 commits into
Open
Conversation
Add e2e workflow
Create trigger-pull.js
Konflux Staging update cara-konflux-ui-test-b1228
- Add secret parameters to e2e-main-pipeline - Update run-e2e-konflux-ui task with Cypress and Slack credentials - Update check-org-membership task with GitHub token - Update IntegrationTestScenario with all secret parameters - Add example PipelineRun for manual testing This migration enables running E2E tests in Kubernetes with proper credentials management instead of GitHub Actions secrets.
- Check EVENT_TYPE before parsing PR-specific fields - Use jq default values to handle missing/malformed JSON fields - Fix 'parse error: Expected value before comma' for push events
- Set pull_request_number to JSON null instead of empty value - Set pull_request_author to JSON null instead of string 'null' - Fix 'parse error: Expected value before comma' in job-spec JSON For push events, the generated JSON will now be: "pull_request_number": null, "pull_request_author": null, instead of: "pull_request_number": , "pull_request_author": "null",
The issue was that although GH_TOKEN was set in bash (verified by debug output),
it wasn't being passed to the Node.js process started by 'npx cypress'.
Changed from:
export GH_TOKEN
npx cypress run ...
To:
GH_TOKEN="${GH_TOKEN}" npx cypress run ...
This ensures environment variables are explicitly passed to the child process.
**Key changes:** - Removed pr_check.sh dependency from run-e2e-konflux-ui task - Implemented Slack reporting directly in Tekton task - Uses native Tekton field references for PipelineRun metadata - Simplified message format (PipelineRun name instead of hardcoded URL) **Benefits:** - No more script dependencies in Tekton pipelines - More declarative and Kubernetes-native approach - pr_check.sh can now be removed from integration-tests or kept only for GitHub Actions - Easier to maintain and debug **Migration path:** - pr_check.sh still used by GitHub Actions workflows - Can be gradually deprecated as GitHub Actions are phased out
testcara
force-pushed
the
main
branch
3 times, most recently
from
ce96dac to
fe3bbbd
Compare
testcara
force-pushed
the
main
branch
5 times, most recently
from
afc8fed to
6e0a746
Compare
testcara
force-pushed
the
main
branch
9 times, most recently
from
02e8b2f to
e30c490
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes
Description
Type of change
Screen shots / Gifs for design review
How to test or reproduce?
Browser conformance: