Red Hat Bugzilla – Bug 862260
[RFE] engine-iso-uploader: update command-line usage (URL based location, etc.)
Last modified: 2018-02-14 03:44:52 EST
engine-iso-uploader command-line interface is hard to use, there is no explicit method to select transport (ssh, nfs), there is implicit behaviour, such as if one specifies ssh user then ssh is used.
It is also hard to understand how it can be used with or without engine, and it is very hard to extend in coherent approach.
Alter the command-line interface to be similar to other network tools, by making it URL based.
engine-iso-uploader [options] command source [source...] destination
-c|--config file - read configuration from file.
-o|--option option=value - set configuration option
--engine - engine to use (rest api) URL notation.
list - list iso images
upload - upload iso images
download - download iso images
source - URL of source.
destination - URL of destination.
Supported local destinations:
Supported remote destinations:
nfs://*ISO_DOMAIN - '*' marks host as iso domain, query location from server.
nfs://host/path - bypass engine, put image directly at path.
ssh://host/path - bypass engine, put image directly at path.
If URL is missing password and password is not provided within netrc
1. Using URL notation to provide locations obsoletes the need to specify protocol, host, port, user, password, path. It is standard notation known by target audience.
2. Using well formed command: utility command source... destination is commonly used for transfers.
3. Using standard netrc notation to provide host credentials is better than proprietary format.
4. Using options as name=value provides the ability to extend configuration without altering the command-line usage.
This was discussed some time ago with Keith Robertson and Itamar Heim.
Maybe we want also a list-domains command in order to preserve the functionality provided by current list command.
Having also the download command could be useful. Maybe ovirt-engin-iso-uploader will be reductive for the tool, ovirt-engine-iso-ctl or ovirt-engine-iso-manager could be more appropriate.
We should merge the image and iso uploaders into single tool
If there is only one ISO domain, does engine-iso-uploader have to ask for its name? This is sooooo annoying.
Doesn't seems weekathon material to me.
Yaniv, can we close this one since ovirt-iso-uploader is deprecated in favor of ovirt-imageio?
I would wait for the replacement feature to be in.
(In reply to Yaniv Lavi from comment #8)
> I would wait for the replacement feature to be in.
We are not going to change the command line nevertheless. Closing for the time being.