Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 29340860: Issue 3996 - Make sure to install current versions of Markdown and Jinja2 modules on web servers (Closed)

Created:
April 27, 2016, 12:26 p.m. by Wladimir Palant
Modified:
April 27, 2016, 2:15 p.m.
Reviewers:
mathias
Visibility:
Public.

Description

Note that old packages have to be removed manually before provisioning the web servers. Repository: hg.adblockplus.org/infrastructure

Patch Set 1 #

Total comments: 5

Patch Set 2 : Addressed comments #

Patch Set 3 : Make sure that apt packages are removed #

Patch Set 4 : Hold apt packages rather than purge #

Unified diffs Side-by-side diffs Delta from patch set Stats (+19 lines, -1 line) Patch
M modules/web/manifests/server.pp View 1 2 3 1 chunk +19 lines, -1 line 0 comments Download

Messages

Total messages: 8
Wladimir Palant
April 27, 2016, 12:26 p.m. (2016-04-27 12:26:40 UTC) #1
mathias
Obviously we tackled the very same issue at the same time, you just happen to ...
April 27, 2016, 12:35 p.m. (2016-04-27 12:35:29 UTC) #2
Wladimir Palant
On 2016/04/27 12:35:29, mathias wrote: > Obviously we tackled the very same issue at the ...
April 27, 2016, 1 p.m. (2016-04-27 13:00:42 UTC) #3
mathias
Indeed, obviously the native package location has precedence in sys.path... Well, in that case we ...
April 27, 2016, 1:21 p.m. (2016-04-27 13:21:28 UTC) #4
Wladimir Palant
On 2016/04/27 13:21:28, mathias wrote: > Indeed, obviously the native package location has precedence in ...
April 27, 2016, 1:39 p.m. (2016-04-27 13:39:51 UTC) #5
mathias
> From what I can tell, ensure=>absent will already do? This will make sure that ...
April 27, 2016, 1:43 p.m. (2016-04-27 13:43:56 UTC) #6
Wladimir Palant
Well, ensure=>'held' it is then...
April 27, 2016, 1:52 p.m. (2016-04-27 13:52:20 UTC) #7
mathias
April 27, 2016, 1:57 p.m. (2016-04-27 13:57:30 UTC) #8
LGTM.

Powered by Google App Engine
This is Rietveld