Landscape Client

Enable support for compression in curl

Bug #297623 reported by Andreas Hasenack on 2008-11-13

	Status	Importance	Assigned to	Milestone
Landscape Client	Fix Released	Wishlist	Andreas Hasenack	Landscape Client 11.02
Landscape Server	Invalid	Wishlist	Unassigned	Landscape Server mountainview
landscape-client (Ubuntu)	Fix Released	Low	Unassigned
Karmic	Won't Fix	Undecided	Unassigned
Lucid	Fix Released	Undecided	Unassigned
Maverick	Fix Released	Undecided	Unassigned

Bug Description

Landscape's staging server already supports gzip encoding in http(s) transfers, and we will put it in place in production too sometime soon.

In order to take advantage of this, the client needs to enable this support in curl. I did some tests and it seems this change is enough:

--- lib/fetch.py 2008-11-05 17:06:55.000000000 -0200
+++ lib/fetch.py 2008-11-11 19:18:04.000000000 -0200
@@ -39,6 +39,7 @@
     curl.setopt(pycurl.FOLLOWLOCATION, True)
     curl.setopt(pycurl.MAXREDIRS, 5)
     curl.setopt(pycurl.WRITEFUNCTION, input.write)
+ curl.setopt(pycurl.ENCODING, "gzip, deflate")
     curl.perform()

return input.getvalue()

See original description

Tags:

Related branches

lp:~ahasenack/landscape-client/297623-curl-compression

Merged into lp:~landscape/landscape-client/trunk at revision 312

Jamu Kakar (community): Approve on 2011-01-24

Thomas Herve (community): Approve on 2011-01-24

lp:ubuntu/natty/landscape-client

lp:~ahasenack/landscape-client/landscape-client-11.02-0ubuntu0.10.10.0

lp:~ahasenack/landscape-client/landscape-client-11.02-0ubuntu0.10.04.0

lp:~ahasenack/landscape-client/landscape-client-11.02-0ubuntu0.9.10.0

lp:ubuntu/lucid-proposed/landscape-client

lp:ubuntu/maverick-proposed/landscape-client

lp:ubuntu/karmic-proposed/landscape-client

Revision history for this message

Andreas Hasenack (ahasenack) wrote on 2008-11-13:

(should be "low" for landscape-client (Ubuntu))

description:	updated
description:	updated
Changed in landscape:
importance:	Undecided → Wishlist
milestone:	none → mountainview-pre-1
status:	New → Confirmed
Changed in landscape-client:
importance:	Undecided → Wishlist
status:	New → Confirmed
status:	New → Confirmed

Mathias Gug (mathiaz) on 2008-11-13

Changed in landscape-client:
importance:	Undecided → Low

Andreas Hasenack (ahasenack) on 2008-11-19

Changed in landscape:
milestone:	mountainview-pre-1 → mountainview-pre-2

Andreas Hasenack (ahasenack) on 2008-12-03

Changed in landscape:
milestone:	mountainview-pre-2 → mountainview-pre-3

Andreas Hasenack (ahasenack) on 2009-01-28

Changed in landscape:
milestone:	mountainview-pre-3 → mountainview

Revision history for this message

Andreas Hasenack (ahasenack) wrote on 2009-04-08:

Let's try to uncouple server bugs from client ones.

Changed in landscape:
status:	Confirmed → Invalid
Changed in landscape-client:
milestone:	none → 1.0.x

Andreas Hasenack (ahasenack) on 2010-07-19

Changed in landscape-client:
assignee:	nobody → Andreas Hasenack (ahasenack)
milestone:	1.0.x → 1.5.4

Andreas Hasenack (ahasenack) on 2010-08-02

Changed in landscape-client:
milestone:	1.5.4 → 1.5.5

Andreas Hasenack (ahasenack) on 2010-08-13

Changed in landscape-client:
milestone:	1.5.5 → later

Andreas Hasenack (ahasenack) on 2011-01-24

Changed in landscape-client:
status:	Confirmed → Fix Committed
milestone:	later → 11.02

Revision history for this message

Launchpad Janitor (janitor) wrote on 2011-03-02:

This bug was fixed in the package landscape-client - 11.02-0ubuntu0.11.04.0

---------------
landscape-client (11.02-0ubuntu0.11.04.0) natty; urgency=low

* New upstream version (LP: #727324)

- Exit gracefully instead of crashing when the filesystem is
read-only (LP: #649997).

- Drop hal requirement (LP: #708502).

- Enable HTTP compression in Curl (LP: #297623).

- Explicitly name log files that need to be rotated (LP: #634236).

- Assorted test suite fixes
-- Andreas Hasenack <email address hidden> Tue, 01 Mar 2011 15:38:11 -0300

Changed in landscape-client (Ubuntu):
status:	Confirmed → Fix Released

Revision history for this message

Martin Pitt (pitti) wrote on 2011-03-30: Please test proposed package

Accepted landscape-client into lucid-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in landscape-client (Ubuntu Lucid):
status:	New → Fix Committed
tags:	added: verification-needed

Revision history for this message

Martin Pitt (pitti) wrote on 2011-03-30:

Accepted landscape-client into maverick-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in landscape-client (Ubuntu Maverick):
status:	New → Fix Committed

Revision history for this message

Martin Pitt (pitti) wrote on 2011-03-30:

Accepted landscape-client into karmic-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in landscape-client (Ubuntu Karmic):
status:	New → Fix Committed

Revision history for this message

Clint Byrum (clint-fewbar) wrote on 2011-04-26:

Revision history for this message

Clint Byrum (clint-fewbar) wrote on 2011-04-26:

Revision history for this message

Martin Pitt (pitti) wrote on 2011-05-03:

This package has been in karmic-proposed for some time without verification. I removed it as karmic is end-of-life now.

Changed in landscape-client (Ubuntu Karmic):
status:	Fix Committed → Won't Fix

Revision history for this message

Andreas Hasenack (ahasenack) wrote on 2011-05-03:

#10

Fix confirmed for lucid and maverick using the package in their respective proposed repositories:
* 11.02-0ubuntu0.10.04.1
* 11.02-0ubuntu0.10.10.1

Martin Pitt (pitti) on 2011-05-04

tags:

added: verification-done
removed: verification-needed

Revision history for this message

Launchpad Janitor (janitor) wrote on 2011-05-04:

#11

This bug was fixed in the package landscape-client - 11.02-0ubuntu0.10.10.1

---------------
landscape-client (11.02-0ubuntu0.10.10.1) maverick-proposed; urgency=low

  * debian/control, debian/rules: Add quilt
  * debian/patches/fix-landscape-monitor.patch: Fix landscape
    monitoring with gir1.0-gudev-1.0 installed. (LP: #747498)

landscape-client (11.02-0ubuntu0.10.10.0) maverick-proposed; urgency=low

* New upstream version (LP: #727324)

- Exit gracefully instead of crashing when the filesystem is
read-only (LP: #649997).

- Drop hal requirement (LP: #708502).

- Enable HTTP compression in Curl (LP: #297623).

- Explicitly name log files that need to be rotated (LP: #634236).

- Assorted test suite fixes

- Use a better load check for the sysinfo wrapper, taking into account the
number of cores (LP: #643565).

- Add an option to bootstrap cloud instances using cloud-init
(LP: #701972).

- Fix packaging for Natty (LP: #688115).

    - Force deletion of all the persist data for the monitoring plugins at
      resynchronization, instead of relying each one of them to do
      (LP: #688161).

- Don't send the mount-activity message to the server anymore
(LP: #688514).

- Workaround a new behavior in NetworkManager where getfqdn would report
localhost instead of useful hostname (LP: #649142).

landscape-client (1.5.5.1-0ubuntu0.10.10.0) maverick; urgency=low

  * The client network plugin would send erroneous data if a network
    interface was removed (and its kernel module removed as well) and
    then readded (LP: #641264).
-- Chuck Short <email address hidden> Tue, 12 Apr 2011 13:56:38 -0400

Changed in landscape-client (Ubuntu Maverick):
status:	Fix Committed → Fix Released

Revision history for this message

Launchpad Janitor (janitor) wrote on 2011-05-04:

#12

This bug was fixed in the package landscape-client - 11.02-0ubuntu0.10.04.1

---------------
landscape-client (11.02-0ubuntu0.10.04.1) lucid-proposed; urgency=low

  * debian/control, debian/rules: Add quilt
  * debian/patches/fix-landscape-monitor.patch: Fix landscape:
    monitoring with gir1.0-gudev-1.0 installed. (LP: #747498)

landscape-client (11.02-0ubuntu0.10.04.0) lucid-proposed; urgency=low

* New upstream version (LP: #727324)

- Exit gracefully instead of crashing when the filesystem is
read-only (LP: #649997).

- Drop hal requirement (LP: #708502).

- Enable HTTP compression in Curl (LP: #297623).

- Explicitly name log files that need to be rotated (LP: #634236).

- Assorted test suite fixes

- Use a better load check for the sysinfo wrapper, taking into account the
number of cores (LP: #643565).

- Add an option to bootstrap cloud instances using cloud-init
(LP: #701972).

- Fix packaging for Natty (LP: #688115).

    - Force deletion of all the persist data for the monitoring plugins at
      resynchronization, instead of relying each one of them to do
      (LP: #688161).

- Don't send the mount-activity message to the server anymore
(LP: #688514).

- Workaround a new behavior in NetworkManager where getfqdn would report
localhost instead of useful hostname (LP: #649142).
-- Chuck Short <email address hidden> Tue, 12 Apr 2011 14:38:20 -0400

Changed in landscape-client (Ubuntu Lucid):
status:	Fix Committed → Fix Released

Andreas Hasenack (ahasenack) on 2012-11-20

Changed in landscape-client:
status:	Fix Committed → Fix Released

Report a bug

This report contains Public information

Everyone can see this information.

You are

Subscribing...

Edit bug mail

Other bug subscribers

Remote bug watches

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