deploy --build-egg deploys despite help message

[andrewbaxter]

The help says --build-egg TEXT Only build the given egg, don't deploy it however running with --build-egg caused the project to deploy. It's possible that I'm using the command wrong (wanted to inspect the built files that would be deployed) but even misusing it I wouldn't expect it to deploy.

$ shub deploy --build-egg abcd
Building images.scrapinghub.com/project/.:18f9c28c-master.
Steps: 100%|█████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 10/10
The image images.scrapinghub.com/project/.:18f9c28c-master build is completed.
Login to images.scrapinghub.com succeeded.
Pushing images.scrapinghub.com/project/.:18f9c28c-master to the registry.
Layers: 100%|████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 23/23
3249f0ebd44b: 100%|██████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 15.6M/15.6M [308kB/s]
b6e62d926fd2: 100%|██████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 37.3M/37.3M [797kB/s]
3a0aa11d8d4f: 100%|█████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 2.90M/2.90M [62.5kB/s]
d03ddadd158b: 100%|██████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 43.3M/43.3M [948kB/s]
a27fe7e855bf: 100%|███████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 327k/327k [7.20kB/s]
8ee3917962e2: 100%|███████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 4.10k/4.10k [116B/s]
The image images.scrapinghub.com/project/.:18f9c28c-master pushed successfully.
Deploying images.scrapinghub.com/project/.:18f9c28c-master
You can check deploy results later with 'shub image check --id 10'.
Progress: 100%|████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 100/100
Deploy results:
{u'status': u'ok', u'project': ., u'version': u'18f9c28c-master', u'spiders': .}

Checking the project deploys page it reports the code was deployed a minute ago (I was on an outdated commit, so it's unlikely someone else just happened to deploy at the same time).

Also, I also don't see any build directories or eggs in the directory.

[vshlapakov]

@andrewbaxter thanks for the report, I'd consider it as a bug introduced when implementing logic for custom images. The general idea is that in a case of a custom image, only Dockerfile instructions are important, you might even skip adding setup.py file, that's why it ignores the option. However, I agree that I'd expect building an egg file (if it's possible) when using the command in any case.

[andrewbaxter]

Okay yeah, that does make sense. Or just raising an error, I think. In my case I was trying to confirm that a file wouldn't be added to the deployed project that was missing from Dockerignore.