| 2010-05-03 07:25:02 |
Mekk |
bug |
|
|
added bug |
| 2010-05-03 07:25:02 |
Mekk |
attachment added |
|
Dependencies.txt http://launchpadlibrarian.net/47477307/Dependencies.txt |
|
| 2010-05-10 02:00:27 |
pklaus |
attachment added |
|
screenshot of the etckeeper upgrade bug http://launchpadlibrarian.net/48180994/etckeeper_upgrade-bug-screenshot.png |
|
| 2010-05-10 02:03:34 |
pklaus |
summary |
etckeeper hook breaks 9.04->9.10 upgrade with "command line too long" |
etckeeper hook breaks 9.10→10.04 upgrade with "Argument list too long" |
|
| 2010-05-10 08:00:27 |
Daniel Hahler |
etckeeper (Ubuntu): status |
New |
Triaged |
|
| 2010-05-10 08:00:36 |
Daniel Hahler |
etckeeper (Ubuntu): importance |
Undecided |
High |
|
| 2010-05-10 08:00:42 |
Daniel Hahler |
etckeeper (Ubuntu): importance |
High |
Medium |
|
| 2010-05-15 20:41:07 |
Daniel Hahler |
bug watch added |
|
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=581678 |
|
| 2010-05-15 20:41:07 |
Daniel Hahler |
bug task added |
|
etckeeper (Debian) |
|
| 2010-05-15 20:41:39 |
Daniel Hahler |
summary |
etckeeper hook breaks 9.10→10.04 upgrade with "Argument list too long" |
etckeeper hook breaks upgrade with "Argument list too long" |
|
| 2010-05-15 21:40:56 |
Bug Watch Updater |
etckeeper (Debian): status |
Unknown |
New |
|
| 2010-05-17 22:39:25 |
Bug Watch Updater |
etckeeper (Debian): status |
New |
Fix Released |
|
| 2010-05-31 10:12:21 |
Thierry Carrez |
etckeeper (Ubuntu): importance |
Medium |
High |
|
| 2010-05-31 10:12:29 |
Thierry Carrez |
etckeeper (Ubuntu): assignee |
|
Thierry Carrez (ttx) |
|
| 2010-05-31 10:14:01 |
Thierry Carrez |
nominated for series |
|
Ubuntu Lucid |
|
| 2010-05-31 10:14:01 |
Thierry Carrez |
bug task added |
|
etckeeper (Ubuntu Lucid) |
|
| 2010-05-31 10:14:19 |
Thierry Carrez |
etckeeper (Ubuntu Lucid): importance |
Undecided |
High |
|
| 2010-05-31 10:14:21 |
Thierry Carrez |
etckeeper (Ubuntu Lucid): assignee |
|
Thierry Carrez (ttx) |
|
| 2010-05-31 10:14:27 |
Thierry Carrez |
etckeeper (Ubuntu): status |
Triaged |
Fix Released |
|
| 2010-05-31 10:14:33 |
Thierry Carrez |
etckeeper (Ubuntu Lucid): status |
New |
Triaged |
|
| 2010-06-01 15:11:31 |
Thierry Carrez |
etckeeper (Ubuntu Lucid): importance |
High |
Medium |
|
| 2010-06-10 13:29:44 |
Thierry Carrez |
description |
Binary package hint: etckeeper
I just executed distribution upgrade from 9.04 to 9.10.
The upgrade failed nearby the end because of etckeeper failure. I don't remember exact message but it was about command line being too long (I suspect bzr command was executed with all changed etc files, which are numerous)
Definitely deserves some thought.
ProblemType: Bug
DistroRelease: Ubuntu 10.04
Package: etckeeper 0.41ubuntu3
ProcVersionSignature: Ubuntu 2.6.31-21.59-generic
Uname: Linux 2.6.31-21-generic x86_64
NonfreeKernelModules: nvidia
Architecture: amd64
Date: Mon May 3 09:21:23 2010
InstallationMedia: Ubuntu 9.10 "Karmic Koala" - Release amd64 (20091027)
PackageArchitecture: all
ProcEnviron:
PATH=(custom, no user)
LANG=en_US.UTF-8
SHELL=/bin/bash
SourcePackage: etckeeper |
Binary package hint: etckeeper
I just executed distribution upgrade from 9.04 to 9.10.
The upgrade failed nearby the end because of etckeeper failure. I don't remember exact message but it was about command line being too long (I suspect bzr command was executed with all changed etc files, which are numerous)
Definitely deserves some thought.
ProblemType: Bug
DistroRelease: Ubuntu 10.04
Package: etckeeper 0.41ubuntu3
ProcVersionSignature: Ubuntu 2.6.31-21.59-generic
Uname: Linux 2.6.31-21-generic x86_64
NonfreeKernelModules: nvidia
Architecture: amd64
Date: Mon May 3 09:21:23 2010
InstallationMedia: Ubuntu 9.10 "Karmic Koala" - Release amd64 (20091027)
PackageArchitecture: all
ProcEnviron:
PATH=(custom, no user)
LANG=en_US.UTF-8
SHELL=/bin/bash
SourcePackage: etckeeper
== SRU Report ==
Impact:
If the user has thousands of package installed and upgrades to a new release, etckeeper tries to commit after the apt run with a commit message that is too long, resulting in an upgrade failure.
Development branch fix:
Version 0.46 fixed the issue by adding support for --stdin and making etckeeper use it for post-install.d commit messages, so it is no longer subject to the issue.
Minimal patch:
Backporting the fix from 0.46 would result IMHO in a significant patch that also introduces features (--stdin), so this SRU proposes to truncate the commit message instead after 2000 lines. That's an improvement on the current situation (where the message would be completely lost) and avoids the upgrade failure message. See attached in comment 6.
TEST CASE:
This is relatively tricky to reproduce, since this requires to have a very large number of packages installed.
- Install karmic
- Install etckeeper
- Install > 3000 packages
- Upgrade to lucid
Without fix: upgrade fails at the end with "Argument list too long"
With fix: upgrade does not fail at the end
Regression potential:
This patch conservatively truncates the message after 2000 lines. There might have been cases where more than 2000 packages were installed, where the etckeeper commit message would still not generate an error on upgrade. For the few users in that case, some lines at the end of the commit message would be lost. It does not change behavior for users upgrading less than 2000 packages at once, and avoids losing all the message for those that exceeded the limit.
|
|
| 2010-06-10 13:30:29 |
Thierry Carrez |
attachment added |
|
patch http://launchpadlibrarian.net/50072965/patch |
|
| 2010-06-10 13:33:55 |
Thierry Carrez |
etckeeper (Ubuntu Lucid): status |
Triaged |
Fix Committed |
|
| 2010-06-11 07:33:17 |
Thierry Carrez |
removed subscriber Thierry Carrez |
|
|
|
| 2010-06-11 07:33:32 |
Thierry Carrez |
etckeeper (Ubuntu Lucid): status |
Fix Committed |
In Progress |
|
| 2010-06-11 08:43:48 |
Thierry Carrez |
description |
Binary package hint: etckeeper
I just executed distribution upgrade from 9.04 to 9.10.
The upgrade failed nearby the end because of etckeeper failure. I don't remember exact message but it was about command line being too long (I suspect bzr command was executed with all changed etc files, which are numerous)
Definitely deserves some thought.
ProblemType: Bug
DistroRelease: Ubuntu 10.04
Package: etckeeper 0.41ubuntu3
ProcVersionSignature: Ubuntu 2.6.31-21.59-generic
Uname: Linux 2.6.31-21-generic x86_64
NonfreeKernelModules: nvidia
Architecture: amd64
Date: Mon May 3 09:21:23 2010
InstallationMedia: Ubuntu 9.10 "Karmic Koala" - Release amd64 (20091027)
PackageArchitecture: all
ProcEnviron:
PATH=(custom, no user)
LANG=en_US.UTF-8
SHELL=/bin/bash
SourcePackage: etckeeper
== SRU Report ==
Impact:
If the user has thousands of package installed and upgrades to a new release, etckeeper tries to commit after the apt run with a commit message that is too long, resulting in an upgrade failure.
Development branch fix:
Version 0.46 fixed the issue by adding support for --stdin and making etckeeper use it for post-install.d commit messages, so it is no longer subject to the issue.
Minimal patch:
Backporting the fix from 0.46 would result IMHO in a significant patch that also introduces features (--stdin), so this SRU proposes to truncate the commit message instead after 2000 lines. That's an improvement on the current situation (where the message would be completely lost) and avoids the upgrade failure message. See attached in comment 6.
TEST CASE:
This is relatively tricky to reproduce, since this requires to have a very large number of packages installed.
- Install karmic
- Install etckeeper
- Install > 3000 packages
- Upgrade to lucid
Without fix: upgrade fails at the end with "Argument list too long"
With fix: upgrade does not fail at the end
Regression potential:
This patch conservatively truncates the message after 2000 lines. There might have been cases where more than 2000 packages were installed, where the etckeeper commit message would still not generate an error on upgrade. For the few users in that case, some lines at the end of the commit message would be lost. It does not change behavior for users upgrading less than 2000 packages at once, and avoids losing all the message for those that exceeded the limit.
|
Binary package hint: etckeeper
I just executed distribution upgrade from 9.04 to 9.10.
The upgrade failed nearby the end because of etckeeper failure. I don't remember exact message but it was about command line being too long (I suspect bzr command was executed with all changed etc files, which are numerous)
Definitely deserves some thought.
ProblemType: Bug
DistroRelease: Ubuntu 10.04
Package: etckeeper 0.41ubuntu3
ProcVersionSignature: Ubuntu 2.6.31-21.59-generic
Uname: Linux 2.6.31-21-generic x86_64
NonfreeKernelModules: nvidia
Architecture: amd64
Date: Mon May 3 09:21:23 2010
InstallationMedia: Ubuntu 9.10 "Karmic Koala" - Release amd64 (20091027)
PackageArchitecture: all
ProcEnviron:
PATH=(custom, no user)
LANG=en_US.UTF-8
SHELL=/bin/bash
SourcePackage: etckeeper
== SRU Report ==
Impact:
If the user has thousands of packages installed and upgrades to a new release, etckeeper tries to commit after the apt run with a commit message that is too long, resulting in an upgrade failure.
Development branch fix:
Version 0.46 fixed the issue by adding support for --stdin and making etckeeper use it for post-install.d commit messages, so it is no longer subject to the issue.
Minimal patch:
Backporting the fix from 0.46 would result IMHO in a significant patch that also introduces features (--stdin), so this SRU proposes to truncate the commit message. See attached in comment 10.
TEST CASE:
This is relatively tricky to reproduce, since this requires to have a very large number of packages installed. Here is a way to trigger the bug:
- Install etckeeper
- Prepare a /tmp/testfile with more than 128K of data (copy a log file) (for example cp /var/log/udev /tmp/testfile)
- Start upgrading packages
- In parallel, once the packages are downloaded and the /var/cache/etckeeper/packagelist.pre-install is generated, replace it by your testfile (cp /tmp/testfile /var/cache/etckeeper/packagelist.pre-install)
- Wait for the upgrades to complete
This generates an artificially long set of differences in packages installed and will trigger the bug.
Without fix: upgrade fails at the end with "Argument list too long"
With fix: upgrade does not fail at the end, etckeeper commit message is truncated.
Regression potential:
The patch truncates the commit message only in the cases where it would trigger an "argument too long" error, so it should not create a regression.
|
|
| 2010-06-11 09:22:06 |
Thierry Carrez |
attachment added |
|
patch http://launchpadlibrarian.net/50123700/patch |
|
| 2010-06-11 09:26:12 |
Thierry Carrez |
etckeeper (Ubuntu Lucid): status |
In Progress |
Fix Committed |
|
| 2010-06-18 09:26:54 |
Martin Pitt |
bug |
|
|
added subscriber SRU Verification |
| 2010-06-18 09:26:59 |
Martin Pitt |
tags |
amd64 apport-bug lucid |
amd64 apport-bug lucid verification-needed |
|
| 2010-06-18 10:14:14 |
Launchpad Janitor |
branch linked |
|
lp:ubuntu/lucid-proposed/etckeeper |
|
| 2010-06-24 09:18:13 |
Thierry Carrez |
tags |
amd64 apport-bug lucid verification-needed |
amd64 apport-bug lucid verification-done |
|
| 2010-06-24 13:47:52 |
Launchpad Janitor |
etckeeper (Ubuntu Lucid): status |
Fix Committed |
Fix Released |
|
| 2011-09-19 21:17:46 |
Ubuntu Foundations Team Bug Bot |
tags |
amd64 apport-bug lucid verification-done |
amd64 apport-bug lucid testcase verification-done |
|
