[SRU] Backport GenericIPAddressfield from 1.4

Bug #1081391 reported by Julian Edwards
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-django (Ubuntu)
Fix Released
Undecided
Andres Rodriguez
Precise
Invalid
High
Unassigned

Bug Description

The GenericIPAddress field from 1.4 is required in 1.3 as a pre-requisite to a maas backport. This field stores any IP address, v4 or v6.

[Impact]
This is functionality required as part of the MAAS Next Steps blueprint [1], in order to be able to SRU various new fixes to MAAS in precise. This functionality adds a new field that is required for MAAS, in order to be able to store IPv4 and IPv6.

[Test Case]
This has been tested against MAAS. To test this we simply need to branch the MAAS branch which is being prepared for the SRU, and make sure all tests pass. Tests will fail when this SRU has not being installed. Test will pass when this feature is backported.

1. bzr branch lp:~maas-maintainers/maas/1.2 maas-sru
2. cd maas-sru
3. make test

[Regression Potential]
Minimal. This is functionality that produces minimal impact to django. On the other hand, the maas-maintainers team has committed to address any related issues and maintain it from this point forward.

Changed in python-django (Ubuntu):
assignee: nobody → Andres Rodriguez (andreserl)
Changed in python-django (Ubuntu):
status: New → Fix Released
summary: - Backport GenericIPAddressfield from 1.4
+ [SRU] Backport GenericIPAddressfield from 1.4
description: updated
Changed in python-django (Ubuntu Precise):
importance: Undecided → High
Revision history for this message
Dave Walker (davewalker) wrote : Please test proposed package

Hello Julian, or anyone else affected,

Accepted into quantal-proposed. The package will build now and be available in a few hours in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please change the bug tag from verification-needed to verification-done. If it does not, change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

tags: added: verification-needed
Changed in python-django (Ubuntu Precise):
status: New → Fix Committed
tags: added: verification-done
removed: verification-needed
Revision history for this message
Scott Kitterman (kitterman) wrote : Update Released

The verification of this Stable Release Update has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regresssions.

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

This bug was fixed in the package python-django - 1.3.1-4ubuntu1.5

---------------
python-django (1.3.1-4ubuntu1.5) precise-proposed; urgency=low

  [ Julian Edwards ]
  * debian/patches:
    - genericipaddressfield.diff: Backport GenericIPAddressField
      from 1.4 (LP: #1081391)
    - prefetch_related.diff: Backport prefetch_related from 1.4 (LP: #1081388)
    - bug15496-base64-multipart-fix.diff: Include fix for upstream bug #15496
      which makes 'Content-Transfer-Encoding: base64: work for multipart
      messages. (LP: #1081392)
 -- Andres Rodriguez <email address hidden> Tue, 20 Nov 2012 16:00:41 -0500

Changed in python-django (Ubuntu Precise):
status: Fix Committed → Fix Released
Revision history for this message
Scott Kitterman (kitterman) wrote :

I've just removed this update from precise-updates. Unfortunately it wasn't until after I copied it over that I noticed that this is not an appropriate change for an SRU and it's not been reviewed by the SRU team.

Revision history for this message
Scott Kitterman (kitterman) wrote :

Since this was not reviewed by the SRU team, I've removed the SRU. Additionally, since feature additions are not permitted in SRUs by the normal policy, a change like this should be discussed in advance, probably with the tech board, before it is resubmitted.

Changed in python-django (Ubuntu Precise):
status: Fix Released → Invalid
Changed in python-django (Ubuntu Precise):
status: Invalid → New
Revision history for this message
Scott Kitterman (kitterman) wrote :

"Adds a new field that is required for MAAS, in order to be able to store IPv4 and IPv6." is not an SRU worthy fix. The fact that someone at Canonical wants it done doesn't change that. Please do not move this from invalid again without discussion.

python-django is widely used for more than MAAS. "Works with MAAS" is nowhere near sufficient validation that messing with a feature backport post-release is safe for Ubuntu users.

Changed in python-django (Ubuntu Precise):
status: New → Invalid
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.