Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add testing on AppVeyor #1226

Closed
wants to merge 1 commit into from

Conversation

alex85k
Copy link
Contributor

@alex85k alex85k commented Oct 16, 2014

After successful merging of #889 , the testing process had become faster. Together with caching functionality on AppVeor com it gives us a chance to automatically test build on Windows (like Travis does for Linux).

I have an initial setup of testing by appveyor: https://ci.appveyor.com/project/alex85k/project-osrm
https://github.com/alex85k/Project-OSRM/blob/develop/appveyor.yml

Tried to use cached folder with 20Mb .7z (dependencies - Ruby+gems added, boost 1.56 and protobuf 2.6 used).

Todo: test and publish the dependencies with Ruby for debug/release on official server.

@DennisOSRM
Copy link
Collaborator

Awesome, I'll put this on the official server tomorrow.

@alex85k
Copy link
Contributor Author

alex85k commented Oct 16, 2014

It may need some testing. Strange that Travis and Appveyor seem to ignore this PR :)

@alex85k
Copy link
Contributor Author

alex85k commented Oct 16, 2014

Published the lates debug and release depedencies:
https://dl.dropboxusercontent.com/u/63393258/libs_osrm_Debug_test.7z
https://dl.dropboxusercontent.com/u/63393258/libs_osrm_Release_test.7z
(expat and protoc.exe included for future use)

@alex85k
Copy link
Contributor Author

alex85k commented Oct 16, 2014

Debug libs are incorrectly packed, will republish them tomorrow.

@DennisOSRM
Copy link
Collaborator

The files are now available from the http://build.project-... host

@emiltin
Copy link
Contributor

emiltin commented Oct 17, 2014

is it possible to get appveyor to wrap long lines? at the moment the cucumber output is hard to read.

@alex85k
Copy link
Contributor Author

alex85k commented Oct 17, 2014

The files are now available from the http://build.project-... host

Could you please republish the debug version from dropbox? It had release stxxl.lib version instead of debug version.

The files with Ruby are called libs_osrm_Release_test.7z and libs_osrm_Debug_test.7z, maybe it is better to keep both on http://build.project-osrm.org

@alex85k
Copy link
Contributor Author

alex85k commented Oct 17, 2014

is it possible to get appveyor to wrap long lines? at the moment the cucumber output is hard to read.

I can only patch cucumber ruby script in depndencies .7z to add line breaks :)
Republished .7z files to dropbox.

@alex85k
Copy link
Contributor Author

alex85k commented Oct 17, 2014

Strange that debug version did not fit in 30 min (with only features/testbot activated) and I do not see the cache working...
https://ci.appveyor.com/project/alex85k/project-osrm

@DennisOSRM
Copy link
Collaborator

what happens if an old archive has been cached? Is there a way to replace it?

@alex85k
Copy link
Contributor Author

alex85k commented Oct 17, 2014

what happens if an old archive has been cached? Is there a way to replace it?

I guess in current version we'll need to run a delete script manually...

Maybe curl -zcan help (it should download only newer file)?

@alex85k
Copy link
Contributor Author

alex85k commented Oct 17, 2014

is it possible to get appveyor to wrap long lines? at the moment the cucumber output is hard to read.

Cucumber hack seems to work: https://ci.appveyor.com/project/alex85k/project-osrm/build/job/cno8vuq26qdc8h8j

The dependenices should be ready for deployment now.

Debug testing is too slow (can enable only 200-300 steps to fit in 30 min).

Will push curl -z together with the updated paths like http://build.project-osrm.org/libs_osrm_Release_test.7z and reduced debug testing when dependencies are publised.

@DennisOSRM
Copy link
Collaborator

Great, will deploy files soon'ish. Waiting on another thing and will get to it afterwards.

@DennisOSRM
Copy link
Collaborator

@alex85k files have been updated

@DennisOSRM
Copy link
Collaborator

@alex85k can you rebase the branch on latest develop and check that it works with the recent changes from #1231.

@alex85k
Copy link
Contributor Author

alex85k commented Oct 17, 2014

Sorry, can not find the updated files.
Need 20-30 Mb _test.7z from the Dropbox (elsewhere old branches appveyor can be broken)
(like http://build.project-osrm.org/libs_osrm_Release_test.7z)

Will rebase now. By the way, -z made the curl crazy: https://ci.appveyor.com/project/alex85k/project-osrm/build/job/k51hqf0f0wyr00sr

@alex85k
Copy link
Contributor Author

alex85k commented Oct 17, 2014

Repushed... Will need to replace address with official.

@DennisOSRM
Copy link
Collaborator

uploading the files a second time. For some reason this didn't work though the web server although the files were visible locally.

@DennisOSRM
Copy link
Collaborator

Found the reason: uploaded to the wrong directory. This has been fixed now.

@DennisOSRM
Copy link
Collaborator

Appears to work, but runs into the 30 min threshold

@alex85k
Copy link
Contributor Author

alex85k commented Oct 17, 2014

OK, repushed with official adresses. Tried features/testbot for debug this time...
Let release have another chance to build... (then we'll need to skip some tests or pay for AppVeyor :) )

@DennisOSRM
Copy link
Collaborator

I will send the AppVeyor folks an email if the would be interested in supporting us on Monday.

@alex85k
Copy link
Contributor Author

alex85k commented Oct 17, 2014

By the way, does not MapBox have a paid account? :)
I can see big appveyor.yml here: https://github.com/BergWerkGIS/mapnik-dependencies

@DennisOSRM
Copy link
Collaborator

Good catch. Another possible route

@alex85k
Copy link
Contributor Author

alex85k commented Oct 18, 2014

By the way, this time it passed the tests and even have 5 min left:
https://ci.appveyor.com/project/DennisOSRM/osrm-backend/build/1.0.303/job/u2rtpu3oelvhckdd
It seems Appveyor have "slow" and "fast" VMs. Better to have "fast" only :) (but maybe it depends on load)

As a reserve variant, we can always patch cucumber to stop testing if the time is near...

@DennisOSRM
Copy link
Collaborator

Yep they recently rolled out a new set of VMs. Not sure if the old ones are meant to stay but that's perhaps a thing to discuss with the folks.

@DennisOSRM
Copy link
Collaborator

Mapbox will support the windows build by running the backend builds from their account. Works great and takes only five mins to build. Will work next week on transitioning the build to their account.

@TheMarex
Copy link
Member

TheMarex commented May 1, 2015

This is now taken care of.

@TheMarex TheMarex closed this May 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants