ols-jenkaas

Merge lp:~maxiberta/ols-jenkaas/idempotent-ppa-purge into lp:~ols-jenkaas-admins/ols-jenkaas/trunk

  1. idempotent-ppa-purge
  2. Merge into trunk
Proposed by Maximiliano Bertacchini
Status: Merged
Merged at revision: 620
Proposed branch: lp:~maxiberta/ols-jenkaas/idempotent-ppa-purge
Merge into: lp:~ols-jenkaas-admins/ols-jenkaas/trunk
Diff against target: 12 lines (+1/-1)
1 file modified
setup/seed-slave (+1/-1)
To merge this branch: bzr merge lp:~maxiberta/ols-jenkaas/idempotent-ppa-purge
Reviewer Review Type Date Requested Status
Daniel Manrique (community) Approve
Review via email: mp+358651@code.launchpad.net

Commit message

Make call to ppa-purge idempotent.

To post a comment you must log in.
Revision history for this message
Daniel Manrique (roadmr) wrote :

Today I learned about || :

:)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'setup/seed-slave'
--- setup/seed-slave 2018-11-12 16:20:42 +0000
+++ setup/seed-slave 2018-11-12 16:54:10 +0000
@@ -5,7 +5,7 @@
5# Note these PPAs were deprecated, so we need to remove them explicitly5# Note these PPAs were deprecated, so we need to remove them explicitly
6# otherwise they'll remain forever on slaves which have it configured.6# otherwise they'll remain forever on slaves which have it configured.
7sudo apt install -y ppa-purge7sudo apt install -y ppa-purge
8sudo ppa-purge -y ppa:vila/ppa8sudo ppa-purge -y ppa:vila/ppa || :
9sudo rm -vf /etc/apt/sources.list.d/*lxd-stable*.list9sudo rm -vf /etc/apt/sources.list.d/*lxd-stable*.list
1010
11sudo add-apt-repository -y ppa:mojo-maintainers11sudo add-apt-repository -y ppa:mojo-maintainers

Subscribers

People subscribed via source and target branches