Bug 1195131

Summary: [RFE] download a docker image with cinder and ceph driver and setup it as a provider for ovirt-engine
Product: [oVirt] ovirt-engine Reporter: Sandro Bonazzola <sbonazzo>
Component: RFEsAssignee: Simone Tiraboschi <stirabos>
Status: CLOSED WONTFIX QA Contact: Jiri Belka <jbelka>
Severity: low Docs Contact:
Priority: medium    
Version: ---CC: amureini, bugs, dfediuck, fdeutsch, jbelka, lsurette, mgoldboi, pstehlik, rbalakri, s.kieske, srevivo, ykaul, ylavi
Target Milestone: ---Keywords: FutureFeature
Target Release: ---Flags: ylavi: ovirt-future?
ylavi: planning_ack?
ylavi: devel_ack?
ylavi: testing_ack?
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: ovirt-3.6.0-alpha1.2 Doc Type: Enhancement
Doc Text:
Installing a local cinder instance cold be convenient for different reasons. Using a docker container for it simplify the process.
Story Points: ---
Clone Of:
: 1195133 (view as bug list) Environment:
Last Closed: 2017-12-03 13:07:26 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Integration RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1195133    

Description Sandro Bonazzola 2015-02-23 08:00:50 UTC
create a docker image with cinder and ceph driver (for now), configure as systemd service as part of setup and add as a provider.

Comment 1 Jiri Belka 2016-02-26 10:27:05 UTC
fail

ovirt-engine-3.6.4-0.0.master.20160224185731.git1fcb5b6.el7.centos.noarch
docker-1.8.2-10.el7.x86_64
ovirt-engine-setup-plugin-dockerc-3.6.4-0.0.master.20160224185731.git1fcb5b6.el7.centos.noarch

there's something wrong with cinder image and no error is logged during setup and thus user could be surprised there is no cinder extension visible in engine's Admin Portal.

imo engine-setup should do checks if appropriate ovirt extension-related service managed by docker works.


# docker images | grep cinder
docker.io/kollaglue/centos-rdo-cinder            kilo                aa8a79666fe9        9 months ago        741.6 MB


# docker logs cinder 2>&1 | head
/start.sh: line 14: check_required_vars: command not found
/start.sh: line 14: check_required_vars: command not found
/start.sh: line 14: check_required_vars: command not found
/start.sh: line 14: check_required_vars: command not found
/start.sh: line 14: check_required_vars: command not found
/start.sh: line 14: check_required_vars: command not found
/start.sh: line 14: check_required_vars: command not found
/start.sh: line 14: check_required_vars: command not found
/start.sh: line 14: check_required_vars: command not found
/start.sh: line 14: check_required_vars: command not found


# engine-setup
...
[ INFO  ] Creating cinder
[ INFO  ] Starting cinder
...

the above is all, the user does not see any check for service functionality.

imo there should be line like:


[ INFO  ] Checking cinder service successful

or

[ ERROR|WARNING ] Checking cinder service failed

cinder should listen on :8776

# printf "GET / HTTP/1.1\n\n" | nc localhost 8776
Ncat: Connection refused.

so maybe checking if this service is available via http could be sufficient.

Comment 2 Red Hat Bugzilla Rules Engine 2016-02-26 10:27:13 UTC
Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release.