project-everest / everest-ci

CI scripts for project everest
3 stars 8 forks source link

Feature request: don't rebuild branches daily, if there are no changes since last build #41

Closed BarryBo closed 7 years ago

BarryBo commented 7 years ago

I get 3 Slack notifications for the "barrybo" branch daily. I am not doing active development on the branch, so it would be more efficient if the build system recognized that and skipped the build.

I don't feel I should delete my branch when I'm not working on a project, only to recreate it later when I do.

vkanne-msft commented 7 years ago

I am surprised who is even building it. I don’t think the CI is building it unless there is active development.

@darrenge, can you kindly check how the builds are getting triggered ?

From: Barry Bond [mailto:notifications@github.com] Sent: Wednesday, January 18, 2017 9:01 AM To: project-everest/everest-ci everest-ci@noreply.github.com Cc: Subscribed subscribed@noreply.github.com Subject: [project-everest/everest-ci] Feature request: don't rebuild branches daily, if there are no changes since last build (#41)

I get 3 Slack notifications for the "barrybo" branch daily. I am not doing active development on the branch, so it would be more efficient if the build system recognized that and skipped the build.

I don't feel I should delete my branch when I'm not working on a project, only to recreate it later when I do.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fproject-everest%2Feverest-ci%2Fissues%2F41&data=02%7C01%7Cvkanne%40microsoft.com%7C0c2cded425e949581bef08d43fc39e6b%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636203556790661383&sdata=Vh%2Fq75kn%2Fq0UW3Ta74lw9lql3404AXqUL222%2BtKIT3g%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FASzQ2Avar091NhWfMOA_ooONNQC3cplbks5rTkVbgaJpZM4LnImj&data=02%7C01%7Cvkanne%40microsoft.com%7C0c2cded425e949581bef08d43fc39e6b%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636203556790661383&sdata=DPRhKTK1MA%2BFp84ZhwmirHCvKIBYN0ujFoOtwKnH8io%3D&reserved=0.

msprotz commented 7 years ago

Is this #39? I also noticed this.

It's about replacing * with master somewhere in VSTS.

vkanne-msft commented 7 years ago

Yup. It is in the Everest-Nightly-Upgrade-Windows. This is the one causing the nightly builds on all branches. Fixing it.

From: Jonathan Protzenko [mailto:notifications@github.com] Sent: Wednesday, January 18, 2017 9:04 AM To: project-everest/everest-ci everest-ci@noreply.github.com Cc: Sreekanth Kannepalli vkanne@microsoft.com; Comment comment@noreply.github.com Subject: Re: [project-everest/everest-ci] Feature request: don't rebuild branches daily, if there are no changes since last build (#41)

Is this #39https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fproject-everest%2Feverest-ci%2Fissues%2F39&data=02%7C01%7Cvkanne%40microsoft.com%7C6711d8ff0566486db1b908d43fc40d08%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636203558643669401&sdata=oALx37fhQz8S8%2Bt7G4hoYvUwaqBS21HYelVXFcmElcA%3D&reserved=0? I also noticed this.

It's about replacing * with master somewhere in VSTS.

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fproject-everest%2Feverest-ci%2Fissues%2F41%23issuecomment-273535628&data=02%7C01%7Cvkanne%40microsoft.com%7C6711d8ff0566486db1b908d43fc40d08%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636203558643669401&sdata=BFIUZEEEz%2BL7dJT5sju6iUOT7h7C7rrDUqPsDGd4QI4%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FASzQ2HV8Jum1qmjq1AxMqChCY4-5pIzfks5rTkYUgaJpZM4LnImj&data=02%7C01%7Cvkanne%40microsoft.com%7C6711d8ff0566486db1b908d43fc40d08%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636203558643669401&sdata=RZOXTalgWH%2BP8Kt9CFVBtBewTjz%2FtDMooCPuVm9UDSM%3D&reserved=0.

vkanne-msft commented 7 years ago

I believe I fixed this. Changed the build definition of Everest-Nightly-Upgrade