Ubuntu

[needs-packaging] [FFE] Please include uvp-monitor in Ubuntu

Bug #1714968 reported by Łukasz Zemczak
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ubuntu
Fix Released
Wishlist
Unassigned

Bug Description

Please allow the addition of the uvp-monitor source and binary package into the Ubuntu universe.

[Rationale]

T-Systems requested including of the uvp-monitor tool in Ubuntu for their T-Systems cloud. It's a tool providing enhancements to VMs running on UVP/Xen by collecting and exporting recource usage data as well as VM hostnames. It will only be used on T-System cloud images, having no real effect on regular images if installed.

The source code of the project is a real mess, but only a small portion of the code is being used for the uvp-monitor tool. The packages, after being built in a PPA, have been tested both by me and T-Systems and confirmed to be working as expected. The Foundation team will be responsible for providing package updates whenever a new version will be requested or needed for any other reason.

Since artful is in Feature Freeze, this bug is also a request for a Feature Freeze Exception. The package will not be seeded on any released images and is a direct request from T-Systems to get our cloud images on their platform up-to-date feature-wise.

[Testing]

The initial version of the uvp-source package has been made available in the ppa:sil2100/testing PPA:

https://launchpad.net/~sil2100/+archive/ubuntu/testing

If required, the package can be tested on a T-Systems cloud image:
 * Deploy a Ubuntu VM on T-Systems
 * Install uvp-monitor, confirm that it's running
 * On your host, go into the instance management web UI, go into the details of the running image and check monitoring data
 * Make sure data such as disk/CPU usage is being registered

See original description
Łukasz Zemczak (sil2100)
description: updated
Revision history for this message
Brian Murray (brian-murray) wrote :

*** This is an automated message ***

This bug is tagged needs-packaging which identifies it as a request for a new package in Ubuntu. As a part of the managing needs-packaging bug reports specification, https://wiki.ubuntu.com/QATeam/Specs/NeedsPackagingBugs, all needs-packaging bug reports have Wishlist importance. Subsequently, I'm setting this bug's status to Wishlist.

summary: - [FFE] Please include uvp-monitor in Ubuntu
+ [needs-packaging] [FFE] Please include uvp-monitor in Ubuntu
Changed in ubuntu:
importance: Undecided → Wishlist
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

Since the copyright situation (and quality) of the code is a bit tricky, I'm actually thinking if we shouldn't get this package to partner instead.

Revision history for this message
Łukasz Zemczak (sil2100) wrote :

I guess let's try getting this included in the primary archive - from what I see we don't want to include new packages into partner (and generally just get partner removed).

Revision history for this message
Łukasz Zemczak (sil2100) wrote :

Uploaded to artful NEW queue. A few notes about the package:
 * I recently noticed that the generated .deb package is not completely lintian clean. There are some errors coming from the init.d file that's being shipped with the package. Those can be surely fixed if needed.
 * I patch up the Makefile to make the package building process a bit more intuitive. I couldn't decide if I should do all the hacking-around in debian/rules or simply patch up the Makefile. I can remove some of the Makefile changes though. But even in that case, we still *have* to modify the Makefile since otherwise the target we want to build will try to build the custom libxenstore and generate a statically linked uvp-monitor binary that we don't need.

Revision history for this message
Andy Whitcroft (apw) wrote :

Ugg there is such a malange of bits in there. Of the bits we are using there is only one bit to my eye which is suspect. In uvp-monitor/securec is source for a library which does not appear to claim to be GPL-2 indeed it appears to claim to be proprietry. It does appear that this code was submitted to the upstream github repo by Huawei but that probabally ought to be queried with the vendor.

Revision history for this message
Steve Langasek (vorlon) wrote :

Note that I have not done a full review of this source to satisfy myself about the legalities, and am not asserting anything either way about the package as a whole (and as a result am also not accepting it directly); I have only reviewed enough to be address Andy's previous comment.

The code under uvp-monitor/securec contains copyright notices of the form:

  * Copyright @ Huawei Technologies Co., Ltd. 1998-2014. All rights reserved.

"All rights reserved" is not a statement of any license or lack thereof. This is particular legal boilerplate language, historically required under certain (now obsolete) copyright treaties, which is part of the declaration of *copyright*.

If there is a top-level distribution license, there is no conflicting statement in this directory about distribution terms (which "all rights reserved" is not), and there is evidence that the code was submitted upstream to a GPL-licensed project by the copyright owner, that is sufficient grounds for assuming this is on the up-and-up and this does not need to be a blocker for acceptance into the archive.

Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in ubuntu:
status: New → Confirmed
Revision history for this message
Daniela Ebert (d-ebert) wrote :

Any update on this?

Revision history for this message
Iain Lane (laney) wrote :

(cleaning up ~ubuntu-release bugs)

This package is in xenial and yakkety via -updates, and artful and bionic proper. So I believe this bug is done.

Changed in ubuntu:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.