Spec URL: https://github.com/ALU-CloudBand/vitrage-dashboard/blob/master/openstack-vitrage-ui.spec
SRPM URL: https://github.com/ALU-CloudBand/vitrage-dashboard/blob/master/openstack-vitrage-ui-1.0.3-dev8.src.rpm
Description: The Vitrage Horizon plugin
sent a pull request to fix minor issues, currently preventing it building.
another pull request.
Based on my specfile: https://raw.githubusercontent.com/mrunge/vitrage-dashboard/master/openstack-vitrage-ui.spec
I have a copr repo with all vitrage components: https://copr.fedorainfracloud.org/coprs/mrunge/vitrage/
vitrage-dashboard bundles third party libraries, which are already used by horizon (d3) and others. Please unbundle them.
Can you push the change to openstack gerrit so the ui guys can approve it
I'd drop the vendor directory completely. That won't be accepted. It requires generating XStatic packages for those deps not already included in OpenStack packages.
another issue I found from playing around with the package is: API pieces are currently missing
yes now I see that from the devstack installation
the api is copied I guess we need to do the same in the spec file
There doesn't seem to be a requirement for copying api over. E.g see ironic-ui:
since https://github.com/openstack/vitrage-dashboard/commit/7b4cbda9305402fb97714a261a198cf362fb6de2 was merged, we don't need to copy API over any more. Please tag a new release and include that into the package.
Was new release tagged as requested in comment 10 ?
Answering myself, there isn't a release yet which includes Matthias' fix from comment 10. But we can proceed with RDO Trunk review which builds from master.
Here is the licensecheck output, project is APLv2 licensed:
vitrage-dashboard/.coveragerc: *No copyright* UNKNOWN
vitrage-dashboard/.gitreview: *No copyright* UNKNOWN
vitrage-dashboard/.jshintrc: *No copyright* UNKNOWN
vitrage-dashboard/.mailmap: *No copyright* UNKNOWN
vitrage-dashboard/.testr.conf: *No copyright* UNKNOWN
vitrage-dashboard/CONTRIBUTING.rst: *No copyright* UNKNOWN
vitrage-dashboard/HACKING.rst: *No copyright* UNKNOWN
vitrage-dashboard/LICENSE: *No copyright* Apache (v2.0)
vitrage-dashboard/MANIFEST.in: *No copyright* UNKNOWN
vitrage-dashboard/README.rst: *No copyright* UNKNOWN
vitrage-dashboard/babel.cfg: *No copyright* UNKNOWN
vitrage-dashboard/openstack-common.conf: *No copyright* UNKNOWN
vitrage-dashboard/requirements.txt: *No copyright* UNKNOWN
vitrage-dashboard/setup.cfg: *No copyright* Apache
vitrage-dashboard/setup.py: Apache (v2.0) GENERATED FILE
vitrage-dashboard/test-requirements.txt: *No copyright* UNKNOWN
vitrage-dashboard/tox.ini: *No copyright* UNKNOWN
The following files have been reviewed in Gerrit:
- Spec: https://raw.githubusercontent.com/rdo-packages/vitrage-dashboard-distgit/rpm-master/openstack-vitrage-ui.spec
- SRPM: http://188.8.131.52:8080/v1/AUTH_b50e80d3969f441a8b7b1fe831003e0a/rdoartifacts/67/4667/11/gate/DLRN-rpmbuild/Z20e036e1b2594625bb9a8d818fda84ae/artifacts/centos/repos/b7/3c/b73c3875060c865628316c85386197986df95441_dev/openstack-vitrage-ui-1.1.1-0.20170201134555.b73c387.el7.centos.src.rpm
Please note that the SRPM has been generated by DLRN, so some differences are expected (Version, Release and Source0).
what's the next step ?
the ocata and ocata-uc tags were uncommented in rdo.yaml
what do I need to do so I can see get the rpms from rdo ?
Matthias, can you have a look at the spec/SRPM, and set rdo-review+ if appropriate?
comments 4 and 6 are not addressed.
could you please include a comment about licenses of bundled libraries?
Some of them are a bit dated already. Maybe it makes sense to remove them and to
use the ones used in horizon? (e.g d3?)
to elaborate more: d3 lib is not licensed under ASL 2.0, as hinted in license field in the spec file.
I updated the spec file with license ASL and BSD
Matthias, please check PS2 in https://review.rdoproject.org/r/4873
left my comment on the review.
We now have builds for vitrage-dashboard in the RDO Trunk repos.