Skip to content

Conversation

@JamesWTruher
Copy link
Collaborator

It looks like using cron on travis-ci will enable us to create daily builds. However, travis-ci doesn't allow for cron jobs to be run at any particular time. Also, it does not appear that it is possible to label these builds, so they will just appear in the same stream as other builds.

The tests are not marked as throwonfailure for the execution of the pester tests because I want to be sure that we always upload our artifacts.

After this change is merged, we can create the cron job (which is now enabled for us in travis-ci)

A cron job needs to be created to start the build, and the variable
TRAVIS_DAILY_BUILD must be set to true
ulimit -n 4096
# do this for our daily build test run
if [[ "$TRAVIS_EVENT_TYPE" == "cron" ]]; then
powershell -c "Import-Module ./build.psm1; Start-PSBootstrap; Start-PSBuild -CrossGen; Start-PSPester -Tag @('CI','Feature','Scenario') -ExcludeTag RequireAdminOnWindows; Start-PSxUnit"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Start-PSBootstrap

Maybe we still want packages out of it? I.e. Start-PSBootstrap -Package; ...; Start-PSPackage

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was wondering about that, my thought that was if we're not doing it for pull requests, i didn't want to do it here. I'm open to an alternative position, though. @andschwa do you have an opinion?

Copy link
Collaborator

@vors vors left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's better than nothing and hopefully, we can get Azure orchestration sooner than later.

set -x
ulimit -n 4096
# do this for our daily build test run
if [[ "$TRAVIS_EVENT_TYPE" == "cron" ]]; then
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we can leave some other hook to allow full builds on CI with REST api.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yah - that's the plan for the future, but I don't know what that hook will look like. For the moment I wanted to get builds started sooner, rather than later.

@JamesWTruher JamesWTruher added Area-Maintainers-Build specific to affecting the build Review - Needed and removed Review - Needed labels Nov 17, 2016
@vors vors merged commit ca0be9a into PowerShell:master Nov 17, 2016
@JamesWTruher JamesWTruher deleted the jameswtruher/dailytravis branch September 23, 2023 05:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-Maintainers-Build specific to affecting the build

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants