Skip to content

fix: remove clickhouse connect from langflow image#1801

Open
lucaseduoli wants to merge 1 commit into
release-cpdfrom
fix/remove_clickhouse
Open

fix: remove clickhouse connect from langflow image#1801
lucaseduoli wants to merge 1 commit into
release-cpdfrom
fix/remove_clickhouse

Conversation

@lucaseduoli

Copy link
Copy Markdown
Collaborator

This pull request makes a minor change to the Dockerfile.langflow to address package management. The main update is the removal of the clickhouse-connect package after installation.

  • Dependency management:
    • Uninstalls the clickhouse-connect package after all dependencies are installed to ensure it is not present in the final Docker image.

@lucaseduoli lucaseduoli self-assigned this Jun 8, 2026
@coderabbitai

coderabbitai Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

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.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 3fb5703a-5f77-4e95-a151-3e8f1c6cee1f

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
  • Commit unit tests in branch fix/remove_clickhouse

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.

@github-actions github-actions Bot added docker bug 🔴 Something isn't working. labels Jun 8, 2026

@edwinjosechittilappilly edwinjosechittilappilly left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pre empt approval. Merge only after test in CPD

@github-actions github-actions Bot added the lgtm label Jun 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug 🔴 Something isn't working. docker lgtm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants