Opened 5 months ago

Last modified 2 months ago

#26076 new defect

test_keygen.sh and test_key_expiration.sh fail on Windows

Reported by: saper Owned by:
Priority: Very Low Milestone: Tor: unspecified
Component: Core Tor/Tor Version: Tor: 0.2.7.3-rc
Severity: Normal Keywords: tor-ci, tor-testing, 034-roadmap-subtask, 034-triage-20180328, 034-included-20180328, 034-deferred-20180602, 035-removed-20180711
Cc: catalyst, saper@… Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description (last modified by teor)

FAIL: src/test/test_keygen.sh
=============================
May 11 01:50:30.175 [warn] Path for GeoIPFile (<default>) is relative and will resolve to C:\projects\appveyor\i686-w64-mingw32\<default>. Is this what you wanted?
May 11 01:50:30.175 [warn] Path for GeoIPv6File (<default>) is relative and will resolve to C:\projects\appveyor\i686-w64-mingw32\<default>. Is this what you wanted?
Tor didn't declare that there would be no encryption
FAIL src/test/test_keygen.sh (exit status: 5)
SKIP: src/test/fuzz_static_testcases.sh

Edit: when we fix this bug, we should revert #26830 and #26853, which skips these tests on Windows

Child Tickets

TicketStatusOwnerSummaryComponent
#26830closedteorSKIP test_keygen.sh on WindowsCore Tor/Tor
#26853closedteorSKIP test_key_expiration.sh on WindowsCore Tor/Tor

Change History (9)

comment:1 Changed 5 months ago by saper

This one looks flakey to me, sometimes it is not there, and the last time I got:

FAIL: src/test/test_key_expiration.sh
=====================================
Setup step #1
=== Starting key expiration tests.
==== Case 1: Test --key-expiration without argument and ensure usage
             instructions are printed.
May 11 09:08:11.312 [warn] Path for GeoIPFile (<default>) is relative and will resolve to C:\projects\appveyor\i686-w64-mingw32\<default>. Is this what you wanted?
May 11 09:08:11.312 [warn] Path for GeoIPv6File (<default>) is relative and will resolve to C:\projects\appveyor\i686-w64-mingw32\<default>. Is this what you wanted?
==== Case 1: ok
==== Case 2: Start Tor with --key-expiration 'sign' and make sure it prints an expiration.
May 11 09:08:11.859 [warn] Path for GeoIPFile (<default>) is relative and will resolve to C:\projects\appveyor\i686-w64-mingw32\<default>. Is this what you wanted?
May 11 09:08:11.859 [warn] Path for GeoIPv6File (<default>) is relative and will resolve to C:\projects\appveyor\i686-w64-mingw32\<default>. Is this what you wanted?
==== Case 2: ok
==== Case 3: Start Tor with --key-expiration 'sign', when there is no
             signing key, and make sure that Tor generates a new key
             and prints its certificate's expiration.
Tor didn't print an expiration
FAIL src/test/test_key_expiration.sh (exit status: 5)
SKIP: src/test/fuzz_static_testcases.sh

comment:2 Changed 5 months ago by saper

Maybe we have some trouble creating reliable temporary directories? During https://ci.appveyor.com/project/torproject/tor/build/1.0.12 the first run completed, but for the second we get "Tor didn't declare that there would be no encryption" which happens when the data directory and the keys folder do already exist. (Just a guess).

comment:3 Changed 4 months ago by nickm

I've just seen a run where the first one failed but the second one succeeded. So, Hmm.

comment:4 Changed 4 months ago by nickm

Parent ID: #25549

unparenting, to close parent.

comment:5 Changed 4 months ago by nickm

Keywords: 034-deferred-20180602 added
Milestone: Tor: 0.3.4.x-finalTor: 0.3.5.x-final

Deferring non-must tickets to 0.3.5

comment:6 Changed 3 months ago by nickm

Keywords: 035-removed-20180711 added
Milestone: Tor: 0.3.5.x-finalTor: unspecified

These tickets are being triaged out of 0.3.5. The ones marked "035-roadmap-proposed" may return.

comment:7 Changed 2 months ago by teor

Version: Tor: 0.2.7.3-rc

comment:8 Changed 2 months ago by teor

Description: modified (diff)

comment:9 Changed 2 months ago by teor

Description: modified (diff)
Summary: AppVeyor CI: Tor didn't declare that there would be no encryption FAIL src/test/test_keygen.sh (exit status: 5)test_keygen.sh and test_key_expiration.sh fail on Windows

Update title, add another related bug

Note: See TracTickets for help on using tickets.