Bug 1361845
| Summary: | Require ovirt-imageio-proxy | ||
|---|---|---|---|
| Product: | [oVirt] ovirt-engine | Reporter: | Amit Aviram <aaviram> |
| Component: | Packaging.rpm | Assignee: | Amit Aviram <aaviram> |
| Status: | CLOSED CURRENTRELEASE | QA Contact: | Natalie Gavrielov <ngavrilo> |
| Severity: | medium | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | future | CC: | aaviram, acanan, amureini, bugs, didi, sbonazzo, tnisan, ylavi |
| Target Milestone: | ovirt-4.0.2 | Flags: | rule-engine:
ovirt-4.0.z+
rule-engine: blocker+ rule-engine: planning_ack+ amureini: devel_ack+ acanan: testing_ack+ |
| Target Release: | 4.0.2.4 | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | If docs needed, set a value | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2016-08-12 14:22:21 UTC | Type: | Bug |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | Storage | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
|
Description
Amit Aviram
2016-07-31 08:02:14 UTC
Sandro, your opinion on this? (In reply to Amit Aviram from comment #1) > Sandro, your opinion on this? I personally think such functionality should not be mandatory. If it's packaged and designed to be optional, why bother every user out there with more packages, configuration, etc.? See also bug 1084489. Consider e.g. users that do not use ISOs - only install machines from templates, glance, PXE etc. That said, one might claim that not requiring this is a regression, as currently the command line iso uploader is required by the engine, and should be obsoleted by the new uploader. So I do not object. Keeping needinfo on Sandro so he will comment too. To help decide, and for completeness, please provide output of 'yum install ovirt-imageio-proxy' on an existing clean engine machine (which never had the proxy) so we can see which extra packages will be added and how much space will be needed. Thanks. Nothing against adding the dependency if it has been decided to provide the image uploader proxy by default. If this bz gets planning ack I'm fine with merging. If we go this way, for 4.1 you may consider dropping requirement on iso-uploader and image-uploader which are deprecated in 4.0 and supposed to be dropped in 4.1. Dependencies Resolved ============================================================================================================================================================================================================================================ Package Arch Version Repository Size ============================================================================================================================================================================================================================================ Installing: ovirt-imageio-proxy noarch 0.3.0-0.201607260726.git4faf259.el7.centos ovirt-master-snapshot 39 k Installing for dependencies: ovirt-imageio-common noarch 0.3.0-1.el7 centos-ovirt40-candidate 22 k ovirt-imageio-proxy-setup noarch 0.3.0-0.201607260726.git4faf259.el7.centos ovirt-master-snapshot 23 k python-requests noarch 2.6.0-1.el7_1 rhel-7-server-rpms 93 k python-urllib3 noarch 1.10.2-2.el7_1 rhel-7-server-rpms 100 k python-webob noarch 1.2.3-6.el7 rhel-7-server-optional-rpms 202 k systemd-python x86_64 219-19.el7_2.11 rhel-7-server-rpms 99 k Transaction Summary ============================================================================================================================================================================================================================================ Install 1 Package (+6 Dependent packages) Total download size: 577 k Installed size: 2.0 M Fine with me. Thanks for the info. Aharon/Gil - can you help with qa-ack here please? Verified, rhevm-4.0.2.4-0.1.el7ev.noarch ovirt-imageio-proxy-0.3.0-0.el7ev.noarch ovirt-imageio-common-0.3.0-0.el7ev.noarch |