Skip to content

fix(ci): remove legacy Travis CI config - #354

Merged
karelhala merged 1 commit into
RedHatInsights:masterfrom
platex-rehor-bot:bot/RHCLOUD-49644
Jul 31, 2026
Merged

fix(ci): remove legacy Travis CI config#354
karelhala merged 1 commit into
RedHatInsights:masterfrom
platex-rehor-bot:bot/RHCLOUD-49644

Conversation

@platex-rehor-bot

@platex-rehor-bot platex-rehor-bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Description

Remove the legacy Travis CI configuration files (.travis.yml, .travis/custom_release.sh, .travis/deploy_key.enc) that contain an unsafe remote script execution pattern in the Deploy stage. The deploy job downloaded and executed a shell script from a mutable branch without any commit pin, checksum, or signature verification (CVSS 8.0).

Konflux/Tekton is now the authoritative build and deployment system for this repo, making the Travis CI setup obsolete. Updated README and technical reference docs to reflect the current CI/CD architecture.

RHCLOUD-49644


Anything reviewers should know?

  • The .travis/deploy_key.enc was an encrypted SSH deploy key used by Travis to push builds to the learning-resources-build repo. Since Konflux handles deployment now, this key is no longer needed.
  • Recent commit bc4019f already removed a broken Travis badge link from the README, confirming Travis is no longer in active use.
  • All 110 existing tests pass. Lint clean.

Checklist

  • Accessibility: color contrast, keyboard nav, screen reader tested (or N/A)
  • All PR checks pass locally (build, lint, test)
  • No unrelated changes included

AI disclosure

Assisted by: Claude Code

…cution

RHCLOUD-49644

The Travis Deploy stage piped a remote shell script from a mutable
branch directly into the interpreter without integrity verification
(CVSS 8.0). Konflux/Tekton is now the authoritative build system,
so the legacy Travis files are no longer needed. Update README and
technical docs accordingly.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jul 31, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: c57f5212-8439-41da-a77d-4db4ca053afb

📥 Commits

Reviewing files that changed from the base of the PR and between 5487aff and 6c864fd.

📒 Files selected for processing (5)
  • .travis.yml
  • .travis/custom_release.sh
  • .travis/deploy_key.enc
  • README.md
  • docs/TECHNICAL_REFERENCE.md
💤 Files with no reviewable changes (2)
  • .travis.yml
  • .travis/custom_release.sh

Summary by CodeRabbit

  • Documentation
    • Updated deployment documentation to reflect the current Konflux/Tekton pipeline process.
    • Removed outdated Travis CI setup, deployment instructions, diagrams, and file references.
  • Chores
    • Removed legacy Travis CI configuration and release automation.

Walkthrough

Travis CI configuration files (.travis.yml and .travis/custom_release.sh) are removed. README.md and docs/TECHNICAL_REFERENCE.md are updated to remove Travis CI references and describe deployment through Konflux/Tekton pipelines, including changes to a CI/CD mermaid diagram and a file reference table.

Changes

Travis CI removal and documentation update

Layer / File(s) Summary
Remove Travis CI configuration files
.travis.yml, .travis/custom_release.sh
Deletes the Travis CI pipeline configuration and the branch-specific custom release script.
Update deployment and CI/CD documentation
README.md, docs/TECHNICAL_REFERENCE.md
Replaces Travis CI deployment description with Konflux/Tekton pipeline information. Updates the CI/CD mermaid diagram to remove the Legacy Travis CI branch and relabel Tekton as "Tekton/Konflux". Removes the Travis CI configuration example and the .travis.yml row from the CI/CD files table.

Estimated code review effort: 1 (Trivial) | ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely identifies removal of the legacy Travis CI configuration.
Description check ✅ Passed The description explains the security issue and migration, links the issue, documents validation, and includes the required checklist and AI disclosure.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

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

@karelhala
karelhala merged commit 76e86f5 into RedHatInsights:master Jul 31, 2026
9 checks passed
@platex-rehor-bot
platex-rehor-bot deleted the bot/RHCLOUD-49644 branch July 31, 2026 09:52
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.

2 participants