Skip to content

chore: update ci pipeline config of eloqstore#436

Merged
thweetkomputer merged 1 commit into
eloqdata:mainfrom
thweetkomputer:adapt_ci_zc
Jan 21, 2026
Merged

chore: update ci pipeline config of eloqstore#436
thweetkomputer merged 1 commit into
eloqdata:mainfrom
thweetkomputer:adapt_ci_zc

Conversation

@thweetkomputer

@thweetkomputer thweetkomputer commented Jan 21, 2026

Copy link
Copy Markdown
Collaborator

Summary by CodeRabbit

  • Chores
    • Simplified build pipeline configurations by removing unused resource definitions and task input dependencies across debug, nightly, and release pipelines.
    • Cleaned up build scripts by removing conditional logic and directory setup steps from the build process.

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

@coderabbitai

coderabbitai Bot commented Jan 21, 2026

Copy link
Copy Markdown

Walkthrough

This PR removes all eloqstore-related resources, task inputs, job steps, and initialization code from the Concourse CI/CD pipeline and build scripts across debug, nightly, and release tarball configurations.

Changes

Cohort / File(s) Change Summary
Pipeline Definitions
concourse/pipeline/build_debug_tarball.yml, build_nightly_tarball.yml, build_release_tarball.yml
Removed eloqstore_src_main git resource definitions and associated get: eloqstore_src steps from job plans in ubuntu2004, ubuntu2204, and ubuntu2404 jobs
Task Configurations
concourse/tasks/build_debug_tarball.yml, build_nightly_tarball.yml, build_release_tarball.yml
Removed eloqstore_src input declarations from task input lists
Build and Tagging Scripts
concourse/scripts/build_tarball.bash, tag.sh
Removed conditional eloqstore submodule initialization block (pushd/symlink/git submodule sync/popd) triggered when DATA_STORE_TYPE=ELOQDSS_ELOQSTORE

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • githubzilla
  • liunyl

Poem

🐰 The eloqstore hops away,
From pipelines where it used to stay,
Symlinks fade, resources gone,
The build marches onward and on! 🚀

🚥 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 describes the main change: removing eloqstore from CI pipeline configuration across multiple files. It accurately reflects the primary objective of the PR.
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
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

@thweetkomputer thweetkomputer merged commit a3c67fb into eloqdata:main Jan 21, 2026
2 checks passed
@thweetkomputer thweetkomputer deleted the adapt_ci_zc branch January 21, 2026 05:28
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