Skip to content

Commit

Permalink
Fixes to get CI tests passing.
Browse files Browse the repository at this point in the history
* Disable package build tests since v1 will no longer be packaged. Users installing packages should update to v2. v1 builds are intended for users installing from source.
* Update SSL error message test on CentOS 7.
  • Loading branch information
dwsteele committed Jul 3, 2018
1 parent a96e7e0 commit a52c658
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 3 deletions.
8 changes: 8 additions & 0 deletions doc/xml/release.xml
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,14 @@
<release date="XXXX-XX-XX" version="1.29dev" title="UNDER DEVELOPMENT">
<release-test-list>
<release-development-list>
<release-item>
<p>Disable package build tests since <id>v1</id> will no longer be packaged. Users installing packages should update to <id>v2</id>. <id>v1</id> builds are intended for users installing from source.</p>
</release-item>

<release-item>
<p>Update SSL error message test on CentOS 7.</p>
</release-item>

<release-item>
<p>Update URL for Debian package repository.</p>
</release-item>
Expand Down
4 changes: 2 additions & 2 deletions test/lib/pgBackRestTest/Module/Storage/StorageS3CertTest.pm
Original file line number Diff line number Diff line change
Expand Up @@ -61,10 +61,10 @@ sub run
if ($self->vm() eq VM_CO7)
{
# Tests fails on co7 because by default certs cannot be located. This logic may need to be changed in the future if
# this bug gets fixed by Red Hat.
# this bug gets fixed by Red Hat. UPDATE: The behavior changed here but it does not seems to be fixed.
$self->testException(
sub {storageRepo({strStanza => 'test1'})->list('/')}, ERROR_HOST_CONNECT,
'IO::Socket::IP configuration failed SSL connect attempt failed.*certificate verify failed',
'IO::Socket::IP configuration failed error.*shutdown while in init',
'cert verify fails on ' . VM_CO7);

# It should work when verification is disabled
Expand Down
3 changes: 2 additions & 1 deletion test/travis.pl
Original file line number Diff line number Diff line change
Expand Up @@ -156,7 +156,8 @@ sub processEnd

processBegin("${strVm} test" . (defined($strParam) ? ": ${strParam}" : ''));
executeTest(
"${strTestExe} --vm-host=" . VM_U14 . " --vm-max=2 --vm=${strVm}" . (defined($strParam) ? " ${strParam}" : ''),
"${strTestExe} --vm-host=" . VM_U14 . " --vm-max=2 --no-package --vm=${strVm}" .
(defined($strParam) ? " ${strParam}" : ''),
{bShowOutputAsync => true});
processEnd();
}
Expand Down

0 comments on commit a52c658

Please sign in to comment.