Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deploy command fails even with extras installed #2203

Open
jonbiemond opened this issue Jan 12, 2025 · 0 comments · May be fixed by #2207
Open

Deploy command fails even with extras installed #2203

jonbiemond opened this issue Jan 12, 2025 · 0 comments · May be fixed by #2207
Assignees

Comments

@jonbiemond
Copy link

dlt version

1.5

Describe the problem

When running dlt deploy the command fails with the message below even though dlt[cli] is installed.

WARNING: Please install additional command line dependencies to use deploy command:
pip install "dlt[cli]"
We ask you to install those dependencies separately to keep our core library small and make it work everywhere.
NOTE: Please refer to our docs at 'https://dlthub.com/docs/walkthroughs/deploy-a-pipeline' for further assistance.

This is because dlt[cli] extras do not include all the dependencies needed by the deploy command. Namely the pip dependency.

Expected behavior

No response

Steps to reproduce

  1. Install dlt[cli] into a virtual environment without pip. For example using a package manager such as uv.
uv venv
source .venv/bin/activate
uv pip install "dlt[cli]"
  1. Run the deploy command.
dlt deploy pipeline.py

Operating system

macOS

Runtime environment

Local

Python version

3.11

dlt data source

No response

dlt destination

No response

Other deployment details

No response

Additional information

  1. Do not hide the exception and replace it with a custom message. This makes it difficult to debug. Instead, include the exception message with the warning.
  2. Include the pip dependency in the cli extras.
jonbiemond added a commit to jonbiemond/dlt that referenced this issue Jan 12, 2025
Increase the `pipdeptree` requirement version
to be ">=2.23.2".
This release of `pipdeptree` removes all
references to `pip`.
tox-dev/pipdeptree#402
The deploy command, which uses `pipdeptree`,
may now be used in environments without `pip`.

Fixes dlt-hub#2203
jonbiemond added a commit to jonbiemond/dlt that referenced this issue Jan 12, 2025
Increase the `pipdeptree` requirement version
to be ">=2.23.2".
This release of `pipdeptree` removes all
references to `pip`.
tox-dev/pipdeptree#402
The deploy command, which uses `pipdeptree`,
may now be used in environments without `pip`.

Fixes dlt-hub#2203
jonbiemond added a commit to jonbiemond/dlt that referenced this issue Jan 12, 2025
Increase the `pipdeptree` requirement version
to be ">=2.23.2".
This release of `pipdeptree` removes all
references to `pip`.
tox-dev/pipdeptree#402
The deploy command, which uses `pipdeptree`,
may now be used in environments without `pip`.

Fixes dlt-hub#2203
@jonbiemond jonbiemond linked a pull request Jan 13, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Todo
2 participants