Skip to content

Remove old CircleCI jobs#1059

Merged
lucacome merged 1 commit intomasterfrom
chore/circleci
Mar 12, 2026
Merged

Remove old CircleCI jobs#1059
lucacome merged 1 commit intomasterfrom
chore/circleci

Conversation

@lucacome
Copy link
Copy Markdown
Member

@lucacome lucacome commented Mar 12, 2026

Summary by CodeRabbit

Release Notes

  • Chores
    • Removed legacy CI/CD pipeline configuration and associated build infrastructure scripts that are no longer actively maintained.

@github-actions github-actions bot added the chore label Mar 12, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 12, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 75bd11f7-a636-45ec-8ac8-246c1388681e

📥 Commits

Reviewing files that changed from the base of the PR and between a6d7aa6 and 6376689.

📒 Files selected for processing (8)
  • .circleci/config.yml
  • ci/Dockerfile
  • ci/build_nginx_opentracing_module.sh
  • ci/do_ci.sh
  • ci/install_gcc4.8.sh
  • ci/install_opentracing.sh
  • ci/run_nginx_ot_docker.sh
  • ci/setup_build_environment.sh
💤 Files with no reviewable changes (8)
  • .circleci/config.yml
  • ci/do_ci.sh
  • ci/setup_build_environment.sh
  • ci/run_nginx_ot_docker.sh
  • ci/Dockerfile
  • ci/build_nginx_opentracing_module.sh
  • ci/install_gcc4.8.sh
  • ci/install_opentracing.sh

Walkthrough

This pull request removes the complete CI/CD infrastructure for building and testing nginx with OpenTracing support, including the CircleCI workflow configuration and all supporting build automation scripts.

Changes

Cohort / File(s) Summary
CI/CD Pipeline Configuration
.circleci/config.yml
Removed CircleCI configuration containing two build jobs targeting Ubuntu 18.04 with different OpenTracing versions (v1.5.1 and v1.6.0) and the associated workflow definition.
Build and Setup Scripts
ci/setup_build_environment.sh, ci/install_gcc4.8.sh, ci/install_opentracing.sh, ci/build_nginx_opentracing_module.sh, ci/do_ci.sh, ci/run_nginx_ot_docker.sh
Removed all build automation scripts that handled environment setup, GCC 4.8 installation, OpenTracing library compilation, nginx module building, and Docker container orchestration.
Docker Configuration
ci/Dockerfile
Removed Dockerfile defining the ubuntu:24.04 based build environment with setup automation.
🚥 Pre-merge checks | ✅ 3
✅ 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 reflects the main change: removal of CircleCI configuration and related CI scripts, which includes the old CircleCI jobs (build_gcc48_opentracing_15/16) and supporting infrastructure.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/circleci

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 and usage tips.

@lucacome lucacome merged commit d7c6578 into master Mar 12, 2026
29 of 30 checks passed
@lucacome lucacome deleted the chore/circleci branch March 12, 2026 02:29
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.

1 participant