Page MenuHomePhabricator

Added for running secp tests in TeamCity

Authored by jasonbcox on Mar 12 2019, 23:59.



Allows us to test secp builds which currently do not run on TeamCity
Depends on D2717
Depends on D2685 only to get passing tests

TeamCity images have been updated to include the jdk 8 dependency.

Test Plan
sudo apt-get install openjdk-8-jdk # dependencies to build and run the java tests

This fails as expected without D2685 applied and passes with D2685 applied.
Tested on a TeamCity agent instance and locally.

Diff Detail

rABC Bitcoin ABC
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

jasonbcox created this revision.Mar 12 2019, 23:59
Herald added a reviewer: Restricted Project. · View Herald TranscriptMar 12 2019, 23:59
Herald added a subscriber: schancel. · View Herald Transcript
deadalnix added inline comments.Mar 13 2019, 02:09
8 ↗(On Diff #7692)


jasonbcox planned changes to this revision.Mar 14 2019, 17:52
jasonbcox added inline comments.
8 ↗(On Diff #7692)

Dunno. This was just copied from

jasonbcox updated this revision to Diff 7766.Mar 20 2019, 22:59

Fix various bugs

jasonbcox edited the summary of this revision. (Show Details)Mar 20 2019, 23:04
jasonbcox edited the test plan for this revision. (Show Details)
jasonbcox added inline comments.Mar 20 2019, 23:10
8 ↗(On Diff #7692)

@schancel What situation is the codebase not part of a git repo? I don't see the branch being hit otherwise.

jasonbcox retitled this revision from [WIP] Added for running secp tests in TeamCity to Added for running secp tests in TeamCity.Mar 21 2019, 00:21
jasonbcox added inline comments.Mar 21 2019, 02:06
8 ↗(On Diff #7692)
@jasonbcox answer to your question is when the git repo is stripped in transfer. (E.g. when rsyncing it to a remote machine to build, or server side checkout (vs agent checkout))
Or if you download a zip of the source

Jason Cox
so for running locally, right? I dont think teamcity would ever have hit this path

If agent checkout is turned off it will.
It checks out the code on the master node and only transfers the source to the build environment.
Without ssh keys or the git metadata.
Fabien accepted this revision.Mar 21 2019, 10:33
Fabien added a subscriber: Fabien.
Fabien added inline comments.
1 ↗(On Diff #7766)

#!/usr/bin/env bash ?

This revision is now accepted and ready to land.Mar 21 2019, 10:33
jasonbcox added inline comments.Mar 21 2019, 16:10
1 ↗(On Diff #7766)

I haven't tested this on the TeamCity agents. a future diff maybe

This revision was automatically updated to reflect the committed changes.