Skip to content

Comments

Btweed/pipeline fix#237

Closed
catastrophe-brandon wants to merge 4 commits intoRedHatInsights:masterfrom
catastrophe-brandon:btweed/pipeline-fix
Closed

Btweed/pipeline fix#237
catastrophe-brandon wants to merge 4 commits intoRedHatInsights:masterfrom
catastrophe-brandon:btweed/pipeline-fix

Conversation

@catastrophe-brandon
Copy link
Contributor

No description provided.

@coderabbitai
Copy link

coderabbitai bot commented Feb 16, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


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

catastrophe-brandon and others added 4 commits February 16, 2026 10:25
- Remove chrome-port parameter (no longer needed without chrome sidecar)
- Update pipeline reference to use btweed/remove-chrome-sidecar branch
  from catastrophe-brandon/konflux-pipelines fork
- Update configmap references to use v2 versions for safe migration

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Add debug output to help diagnose e2e test failures:
- Log HCC_ENV_URL and STAGE_ACTUAL_HOSTNAME environment variables
- Display Caddyfile content before starting Caddy

This will verify that the run-application sidecar receives the required environment variables and that the Caddyfile is correctly configured.

Also update claude.md with context about the chrome sidecar removal and the fix applied to the pipeline.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Remove debug output that was added during chrome sidecar removal
troubleshooting. The changes are now working, so the debug logging
is no longer needed.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.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.

1 participant