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

appveyor で master がビルドされないことがある #300

Closed
m-tmatma opened this issue Jul 24, 2018 · 17 comments
Closed

appveyor で master がビルドされないことがある #300

m-tmatma opened this issue Jul 24, 2018 · 17 comments
Labels
🐛bug🦋 ■バグ修正(Something isn't working) CI appveyor など CI 関連 【ChangeLog除外】
Milestone

Comments

@m-tmatma
Copy link
Member

m-tmatma commented Jul 24, 2018

appveyor で master がビルドされないことがある

https://ci.appveyor.com/project/sakuraeditor/sakura/branch/master
を見ると

b9ffcf3
に対するビルドになっているが、

それ以降のコミットで
5a60bab
(#265) など複数の PR をマージしているが

それらに対するビルドが走っていない。

@m-tmatma m-tmatma added 🐛bug🦋 ■バグ修正(Something isn't working) CI appveyor など CI 関連 【ChangeLog除外】 labels Jul 24, 2018
@m-tmatma
Copy link
Member Author

説明欄で PR 番号と commit Hash を間違えたので訂正

@m-tmatma
Copy link
Member Author

appveyor の画面で "New Build" をクリックすると、最新の master に対してビルドが走った。
https://ci.appveyor.com/project/sakuraeditor/sakura/build/1.0.485

@m-tmatma
Copy link
Member Author

Scheduled builds の機能がworkaround にならないかと思ったが、

デフォルトではフリーアカウントが無効にされているみたいだ。
https://www.appveyor.com/blog/2017/11/08/sheduled-builds-for-free-accounts/

ただ必要性を説得できれば有効にはできるみたい。

@k-takata
Copy link
Member

ただ必要性を説得できれば有効にはできるみたい。

xx のデイリービルドに使いたいと言って有効にしてもらったことがあります。
(問題が頻繁に発生するのでなければ手動でいいのではと思いますが。)

@m-tmatma
Copy link
Member Author

https://help.appveyor.com/discussions/questions/16419-always-build-closed-pull-requests

Pull request (PR) build is build of "virtual merge" of changes made in head branch as if they merged into base branch (usually master). PR build answer the question "what if I merge those changes into master branch". If PR is closed, this usually means that answer to this question is not important anymore. So when AppVeyor notified that PR is closed (via webhook), AppVeyor cancels all running or queued builds for that PR. If you need PR builds to finish even when PR is closed, you use that setting.

とあるので有効にしてみた。

appveyor-build-closed-pr

@m-tmatma
Copy link
Member Author

上記を設定して、#264 に対する merge commit (c11b203) が入ったが
ビルドされていないみたい。

以下で緑のチェックが付いているものが appveyor でビルドに成功したものだが、
https://github.com/sakura-editor/sakura/commits/master

手動でビルドさせたものを除いて、merge commit の場合にはビルドが走っていないぽい。

@m-tmatma
Copy link
Member Author

とあるので有効にしてみた。

おっと、ページ末尾の Save を押してなかった。

@m-tmatma m-tmatma added this to the next release milestone Jul 25, 2018
@m-tmatma
Copy link
Member Author

質問しました。
appveyor/ci#2540

@m-tmatma
Copy link
Member Author

↑ 設定を変更しても #302 のマージコミットがビルドされなかったからです。

@kobake
Copy link
Member

kobake commented Jul 26, 2018

すみません、これ以前に自分が WebHook のオプションをいじっていて、「Pushes」の項目を OFF にしていたことが原因な気がします。ON にし直してみます。

@kobake
Copy link
Member

kobake commented Jul 26, 2018

ON にしました。

@kobake
Copy link
Member

kobake commented Jul 26, 2018

何故これを OFF にしていたかというと、全てのプルリクエストで何故か二重に CI ビルドが走ってしまっているのが気になっていて、Pull requests オプション ON だけで十分に CI ビルドが走っているようだったので Pushes のオプションは OFF にしてしまっていました。

たぶん今回 Pushes を ON にすることにより、master ビルドは直る可能性がありますが、PR のビルドは二重に走ると思います。これはこれで少し困るので対策考えたいです。

m-tmatma added a commit to m-tmatma/sakura that referenced this issue Jul 26, 2018
@m-tmatma
Copy link
Member Author

たぶん今回 Pushes を ON にすることにより、master ビルドは直る可能性がありますが、PR のビルドは二重に走ると思います。これはこれで少し困るので対策考えたいです。

#307 でテスト用の PR 作ってみましたが、二重には走っていないようです。
https://ci.appveyor.com/project/sakuraeditor/sakura/build/1.0.510

@kobake
Copy link
Member

kobake commented Jul 26, 2018

二重に走るときと走らないときがあって、条件の切り分けがまだできてないです。今後、ビルドが二重に走るの見た方いれば教えていただければと。

@m-tmatma
Copy link
Member Author

ただ必要性を説得できれば有効にはできるみたい。

xx のデイリービルドに使いたいと言って有効にしてもらったことがあります。
(問題が頻繁に発生するのでなければ手動でいいのではと思いますが。)

簡単に有効にしてもらえるみたいです。
https://srz-zumix.blogspot.com/2018/01/appveyor.html
https://help.appveyor.com/discussions/problems/9891-enable-scheduled-builds-support

@m-tmatma
Copy link
Member Author

m-tmatma commented Aug 7, 2018

いまのところ master はちゃんとビルドされている。

@m-tmatma
Copy link
Member Author

問題なさそうなので閉じます

@ds14050 ds14050 added 🐛bug🦋 ■バグ修正(Something isn't working) CI appveyor など CI 関連 【ChangeLog除外】 labels Sep 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛bug🦋 ■バグ修正(Something isn't working) CI appveyor など CI 関連 【ChangeLog除外】
Projects
None yet
Development

No branches or pull requests

4 participants