[MIR] vblade-persist

Bug #801244 reported by Chuck Short
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
runit (Ubuntu)
Won't Fix
Undecided
Chuck Short
vblade-persist (Ubuntu)
Won't Fix
Undecided
Robie Basak

Bug Description

Rationale: Dependency for nova apart of the server-o-cobbler specification in order to get nova, glance, and swift into main.
Security: No known security history.
Quality Assurance: Package works out of the box with no prompting. There is no major bugs in Ubuntu and the is no major bugs in Debian.
Standards Compliance: FHS and Debian Policy compliant.
Maintenance: Simple package that the Ubuntu Server Team will take care of.
Dependencies: All are in main except for runit which will have a MIR done shortly.

Michael Terry (mterry)
Changed in vblade-persist (Ubuntu):
assignee: nobody → Michael Terry (mterry)
Revision history for this message
Michael Terry (mterry) wrote :

Approved from a packaging/maintainability perspective. Though it would be nice to see a bug subscriber.

Assigning to ubuntu-security for a quick pass. I'm not familiar with this protocol, and this package is creating /dev symlinks, so I want to be sure there aren't any concerns.

Changed in vblade-persist (Ubuntu):
assignee: Michael Terry (mterry) → Ubuntu Security Team (ubuntu-security)
status: New → Confirmed
Changed in vblade-persist (Ubuntu):
assignee: Ubuntu Security Team (ubuntu-security) → Jamie Strandboge (jdstrand)
status: Confirmed → In Progress
Revision history for this message
Jamie Strandboge (jdstrand) wrote :

Well coded and coded defensively with good input validation. verify_interface() could be better in the face of no ip and no ifconfig, but we always have those, so no biggie. Would be nice if it used 'set -e' in /usr/sbin/vblade-persist, but I don't see anything that is wrong.

Feel free to promote, but note the dependency on runit which is still in universe (I don't see a bug for it either).

Changed in vblade-persist (Ubuntu):
assignee: Jamie Strandboge (jdstrand) → nobody
status: In Progress → Confirmed
Revision history for this message
Michael Terry (mterry) wrote :

Marking Fix Committed then, but you should file a MIR for runit too.

Changed in vblade-persist (Ubuntu):
status: Confirmed → Fix Committed
Matthias Klose (doko)
Changed in runit (Ubuntu):
assignee: nobody → Chuck Short (zulcss)
status: New → Incomplete
Revision history for this message
Steve Langasek (vorlon) wrote :

vblade-persist depends on runit, which is not appropriate to include in main and will not be supported by the Foundations team. If the functionality of this package is required in main, it should be ported to systemd.

Revision history for this message
Michael Terry (mterry) wrote :

Haha, indeed. I hadn't even looked at what runit was yet. Runit-assumptions do unfortunately seem to be embedded into how this package works.

Changed in vblade-persist (Ubuntu):
status: Fix Committed → Incomplete
Revision history for this message
Steve Langasek (vorlon) wrote :

I think this MIR was originally filed for a previous iteration of the package, and the submitter (Chuck) is no longer responsible for the state of this package. Assigning to Robie (the uploader of the version (re?)introducing the recommends, and subscribing the ubuntu server team.

Changed in vblade-persist (Ubuntu):
assignee: nobody → Robie Basak (racb)
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Thanks Steve for making us aware of that old MIR.

In fact vblade is on its way to be demoted and going out of main.
Yes it is still in main, but the upload of 22-1ubuntu1 was not only to merge but also to prepare it to become a sync package later on and with that change it should be demoted too.

That said I think we can set the vblade portion of this bug to "Won't Fix"

Changed in vblade-persist (Ubuntu):
status: Incomplete → Won't Fix
Michael Terry (mterry)
Changed in runit (Ubuntu):
status: Incomplete → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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