Merge lp://qastaging/~danilo/landscape-charm/fix-package-search into lp://qastaging/~landscape/landscape-charm/trunk

Proposed by Данило Шеган
Status: Merged
Approved by: Данило Шеган
Approved revision: 278
Merged at revision: 279
Proposed branch: lp://qastaging/~danilo/landscape-charm/fix-package-search
Merge into: lp://qastaging/~landscape/landscape-charm/trunk
Diff against target: 85 lines (+19/-19)
2 files modified
lib/tests/test_templates.py (+3/-3)
templates/service.conf (+16/-16)
To merge this branch: bzr merge lp://qastaging/~danilo/landscape-charm/fix-package-search
Reviewer Review Type Date Requested Status
Alberto Donato (community) Approve
Free Ekanayaka (community) Approve
🤖 Landscape Builder test results Approve
Review via email: mp+259195@code.qastaging.launchpad.net

Commit message

Fix package-search configuration in service.conf

This fixes [package-search] section in service.conf to match what we usually do for LDS (though our old charm used port 9090: I don't think it matters much which we choose, as long as they are in the 9090-9099 range so as not to overlap with our other services).

This branch also fixes the whitespacing issue (to some extent) by using {%- or -%} which truncates the space to the left and to the right (note the minus sign).

Description of the change

Fix package-search configuration in service.conf

This fixes [package-search] section in service.conf to match what we usually do for LDS (though our old charm used port 9090: I don't think it matters much which we choose, as long as they are in the 9090-9099 range so as not to overlap with our other services).

This branch also fixes the whitespacing issue (to some extent) by using {%- or -%} which truncates the space to the left and to the right (note the minus sign).

To post a comment you must log in.
Revision history for this message
🤖 Landscape Builder (landscape-builder) wrote :

Command: make ci-test
Result: Success
Revno: 278
Branch: lp:~danilo/landscape-charm/fix-package-search
Jenkins: https://ci.lscape.net/job/latch-test/946/

review: Approve (test results)
Revision history for this message
Free Ekanayaka (free.ekanayaka) wrote :

Nice, +1

review: Approve
Revision history for this message
Alberto Donato (ack) wrote :

+1, looks good

review: Approve
Revision history for this message
🤖 Landscape Builder (landscape-builder) wrote :
Download full text (15.5 KiB)

The attempt to merge lp:~danilo/landscape-charm/fix-package-search into lp:landscape-charm failed. Below is the output from the failed tests.

./dev/ubuntu-deps
Reading package lists...
Building dependency tree...
Reading state information...
python-software-properties is already the newest version.
The following packages were automatically installed and are no longer required:
  linux-headers-3.2.0-80 linux-headers-3.2.0-76-virtual linux-headers-3.2.0-76
  linux-headers-3.2.0-79 linux-headers-3.2.0-79-virtual
  linux-headers-3.2.0-80-virtual
Use 'apt-get autoremove' to remove them.
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
OK
Hit http://ppa.launchpad.net precise Release.gpg
Hit http://ubuntu-cloud.archive.canonical.com precise-updates/cloud-tools Release.gpg
Hit http://ppa.launchpad.net precise Release
Hit http://ubuntu-cloud.archive.canonical.com precise-updates/cloud-tools Release
Hit http://ppa.launchpad.net precise/main Sources
Hit http://ubuntu-cloud.archive.canonical.com precise-updates/cloud-tools/main amd64 Packages
Hit http://ppa.launchpad.net precise/main amd64 Packages
Hit http://ubuntu-cloud.archive.canonical.com precise-updates/cloud-tools/main i386 Packages
Hit http://ppa.launchpad.net precise/main i386 Packages
Ign http://ubuntu-cloud.archive.canonical.com precise-updates/cloud-tools/main TranslationIndex
Hit http://ppa.launchpad.net precise/main TranslationIndex
Hit http://ppa.launchpad.net precise/main Translation-en
Hit http://security.ubuntu.com precise-security Release.gpg
Ign http://ubuntu-cloud.archive.canonical.com precise-updates/cloud-tools/main Translation-en_US
Ign http://ubuntu-cloud.archive.canonical.com precise-updates/cloud-tools/main Translation-en
Hit http://security.ubuntu.com precise-security Release
Hit https://private-ppa.launchpad.net precise Release.gpg
Hit https://private-ppa.launchpad.net precise Release.gpg
Hit https://private-ppa.launchpad.net precise Release
Hit https://private-ppa.launchpad.net precise Release
Hit https://private-ppa.launchpad.net precise/main amd64 Packages
Hit https://private-ppa.launchpad.net precise/main i386 Packages
Hit https://private-ppa.launchpad.net precise/main TranslationIndex
Hit http://security.ubuntu.com precise-security/main Sources
Hit https://private-ppa.launchpad.net precise/main amd64 Packages
Hit https://private-ppa.launchpad.net precise/main i386 Packages
Hit https://private-ppa.launchpad.net precise/main TranslationIndex
Hit https://private-ppa.launchpad.net precise/main Translation-en
Hit https://private-ppa.launchpad.net precise/main Translation-en
Hit http://security.ubuntu.com precise-security/universe Sources
Hit http://security.ubuntu.com precise-security/main amd64 Packages
Hit http://security.ubuntu.com precise-security/universe amd64 Packages
Hit http://mirrors.digitalocean.com precise Release.gpg
Hit http://mirrors.digitalocean.com precise-updates Release.gpg
Hit http://mirrors.digitalocean.com precise Release
Hit http://mirrors.digitalocean.com precise-updates Release
Hit http://mirrors.digitalocean.com precise/main Sources
Hit http://mirrors.digitalocean.com precise/universe Sources
Hit http://mirrors.digitalocean.com precise...

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches