Merge ~paelzer/qa-regression-testing:fix-dovecot-bionic into qa-regression-testing:master

Proposed by Christian Ehrhardt 
Status: Merged
Merged at revision: 443db76b551614f734fc5106b2973f3e6e7fddd5
Proposed branch: ~paelzer/qa-regression-testing:fix-dovecot-bionic
Merge into: qa-regression-testing:master
Diff against target: 24 lines (+4/-2)
1 file modified
scripts/test-dovecot.py (+4/-2)
Reviewer Review Type Date Requested Status
Steve Beattie Approve
Review via email: mp+334295@code.launchpad.net

Description of the change

Hi,
as part of the dovecot merge for bionic I found that this will need to be changed once the migration is complete.
I also re-found the issue of the slow starting service, but that the repo (but not the dep8 in the package) had a fix for.

It brings back the discussion of making the lib a package or fetching it in the tests, but for now just the simple small change that the banner will be visible again in 18.04

To post a comment you must log in.
Revision history for this message
Steve Beattie (sbeattie) wrote :

Thanks, looks good. Merging.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/scripts/test-dovecot.py b/scripts/test-dovecot.py
2index 7d149c5..6dffc77 100755
3--- a/scripts/test-dovecot.py
4+++ b/scripts/test-dovecot.py
5@@ -147,7 +147,8 @@ Get cracking!''')
6 def test_pop3(self):
7 '''Test POP3 protocol'''
8
9- if self.lsb_release['Release'] == 14.04:
10+ if (self.lsb_release['Release'] == 14.04 or
11+ self.lsb_release['Release'] >= 18.04):
12 message = '+OK Dovecot (Ubuntu) ready.'
13 else:
14 message = '+OK Dovecot ready.'
15@@ -160,7 +161,8 @@ Get cracking!''')
16 def test_pop3s(self):
17 '''Test POP3S protocol'''
18
19- if self.lsb_release['Release'] == 14.04:
20+ if (self.lsb_release['Release'] == 14.04 or
21+ self.lsb_release['Release'] >= 18.04):
22 message = '+OK Dovecot (Ubuntu) ready.'
23 else:
24 message = '+OK Dovecot ready.'

Subscribers

People subscribed via source and target branches