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

artifacts のサイズ制限にかかってappveyor のビルドに失敗する #514

Closed
m-tmatma opened this issue Oct 2, 2018 · 35 comments
Labels
CI appveyor など CI 関連 【ChangeLog除外】
Milestone

Comments

@m-tmatma
Copy link
Member

m-tmatma commented Oct 2, 2018

artifacts のサイズ制限にかかってappveyor のビルドに失敗する
Maximum allowed artifact storage size of 50000 Mb will be exceeded.

#512 (comment)

AppVeyor が失敗するのでログを見てみると

Maximum allowed artifact storage size of 50000 Mb will be exceeded.

と最後に出ていました。

@m-tmatma m-tmatma added the CI appveyor など CI 関連 【ChangeLog除外】 label Oct 2, 2018
@m-tmatma
Copy link
Member Author

m-tmatma commented Oct 2, 2018

@KENCHjp
Copy link
Member

KENCHjp commented Oct 2, 2018

いま、保存期間が6か月らしいですが、うちの使い方みてると3か月とかむしろ1か月でもいいような。
#109
保存期間調整できるのかな。。。

@berryzplus
Copy link
Contributor

berryzplus commented Oct 2, 2018

うーむ。

Ilya Finkelshteyn's Avatar Ilya Finkelshteyn on 27 Aug, 2018 08:58 PM
Retention policy is now in place. Also default absolute limit is increased. If someone's artifacts collection pace is faster than artifact retention policy, please open a new issue describing your specific situation.

このレスを意訳すると、
古い成果物を削除する基準を見直してデフォの容量増やしたよ。もし新しい基準よりも速いペースで成果物を作っていて困った人がいたら、新しいスレ立てて自分たち固有の状況を説明してちょ。
になる気がする。

(引用文は https://help.appveyor.com/discussions/problems/10253-maximum-allowed-artifact-storage-size-of-1000-mb-will-be-exceeded の最後で problem を閉じた人が書いたメッセージです。)

@berryzplus berryzplus reopened this Oct 2, 2018
@berryzplus
Copy link
Contributor

berryzplus commented Oct 2, 2018

↑間違った

@berryzplus
Copy link
Contributor

berryzplus commented Oct 2, 2018

誰かissue立ててこないと対応してもらえない予感。

https://help.appveyor.com/discussions/problems

@m-tmatma
Copy link
Member Author

m-tmatma commented Oct 3, 2018

急ぎであれば、問い合わせお願いします

@m-tmatma
Copy link
Member Author

m-tmatma commented Oct 3, 2018

送りました。
appveyor/ci#2672

@m-tmatma
Copy link
Member Author

m-tmatma commented Oct 3, 2018

対応されたみたい。
appveyor/ci#2672 (comment)

master をリビルド中です。

@m-tmatma m-tmatma changed the title appeyor の 過去のartifacts を削除する appeyor が artifacts のサイズ制限にかかってビルドに失敗する Oct 3, 2018
@m-tmatma
Copy link
Member Author

m-tmatma commented Oct 3, 2018

タイトル修正

@m-tmatma m-tmatma added this to the next release milestone Oct 3, 2018
@m-tmatma m-tmatma changed the title appeyor が artifacts のサイズ制限にかかってビルドに失敗する artifacts のサイズ制限にかかってappeyor のビルドに失敗する Oct 3, 2018
@m-tmatma
Copy link
Member Author

m-tmatma commented Oct 3, 2018

master のビルド通りました。
閉じます。

@m-tmatma m-tmatma closed this as completed Oct 3, 2018
@m-tmatma
Copy link
Member Author

m-tmatma commented Oct 3, 2018

@m-tmatma
Copy link
Member Author

再度サイズを拡張するようにお願いしました。
appveyor/ci#2708

@k-takata
Copy link
Member

20日ちょっとで20GB使い切っていることを考えると、半年なら180GBぐらい必要な計算になってしまいますね。

@KENCHjp
Copy link
Member

KENCHjp commented Oct 25, 2018

まぁ、今の使い方は、極論すると、保有期間1日でもいいわけで、半年なんていらないんすよね。
CIで容量くってしまうなら、CIテストの時はDry Runするような仕組みがあってもいいかもですね。

@m-tmatma
Copy link
Member Author

artifacts を rotate するように依頼しましょうか?

@m-tmatma
Copy link
Member Author

artifacts を rotate するように依頼しましょうか?

artifacts の保持期間を短くするように依頼しておきました。
appveyor/ci#2708 (comment)

@KENCHjp
Copy link
Member

KENCHjp commented Oct 25, 2018

We extended your limit to 100Gb for now to unblock you, looking better solution.

100Gbになっちゃった。。。

@m-tmatma
Copy link
Member Author

再度サイズを拡張するようにお願いしました。
appveyor/ci#2708

#579 (comment)
で通った。

@k-takata
Copy link
Member

looking better solution.

何らかの対策はすべきでしょうね。今のペースだとまた1ヶ月後にはいっぱいになるでしょうから。

@berryzplus
Copy link
Contributor

アーティファクトの整理します?

当面「allのみ or 個別のみ」にする、とか。

x64版が欲しい人は自分でビルドしてる気配があるので、
x64版のバイナリはリリース目途が立つまで回収しない、とか。

@ds14050
Copy link
Contributor

ds14050 commented Oct 27, 2018

PR で作成されるアーティファクトの第一の目的はリリース代わりではなくデバッグだと思うんです。x64-Debug が一番バグを引き当てやすいかなと思って動作確認では最初に選んでいます。

@berryzplus
Copy link
Contributor

ちゃんと見るときはPRブランチごとソースをフェッチしてDebugビルドするのでDebugのアーティファクトは不要と考えていました。x64については現状で試験提供の認識です。絶賛バシラー募集中ですが、自分でなりたいとは思っていません。

成果物を見るだけならwin32-Release派です。
実際に使われるバージョンにこそ対応する意味があると思うので。

@m-tmatma
Copy link
Member Author

当面「allのみ or 個別のみ」にする、とか。

#587 で個別のみにしました。

m-tmatma added a commit that referenced this issue Oct 28, 2018
すべてを含んだ zip を作成しないようにする (#514 の軽減策)
@berryzplus
Copy link
Contributor

#587 マージによって、生成されるバイナリ量が半分になりました。
従来の倍のペースでPRしない限り、当分は大丈夫ってことですね・・・。

artifactsの保持期間については、根本的に別なあり方を認めてもらったほうがいい気もしています。
こちら側の立場でラクなのは制限を越えたら古いものから削除、ですがそれは難しいのかしら・・・。

@m-tmatma
Copy link
Member Author

m-tmatma commented Dec 18, 2018

恒例の時期がやってきました。
https://ci.appveyor.com/project/sakuraeditor/sakura/builds/21088621/job/q5f33l77ws3x8vw4

10/26 ~ 12/19 の 54日間で 30GB 使ったことになる。
#587 を入れたので、まあ予想通り。

@m-tmatma
Copy link
Member Author

再度サイズ制限を拡張してもらえるように依頼しました。
appveyor/ci#2708 (comment)

@m-tmatma
Copy link
Member Author

恒例の時期がやってきました。
https://ci.appveyor.com/project/sakuraeditor/sakura/builds/21088621/job/q5f33l77ws3x8vw4

10/26 ~ 12/19 の 54日間で 30GB 使ったことになる。
#587 を入れたので、まあ予想通り。

対応してもらった。
リビルドかけた。

@m-tmatma
Copy link
Member Author

失敗してた PR はすべてビルド通った。

@m-tmatma
Copy link
Member Author

サイズ制限はアカウント単位らしい
appveyor/ci#2789 (comment)

@k-takata
Copy link
Member

zip形式から7z形式に変えればさらに半分くらいに減るかもしれません。
(7zの圧縮は遅いので、ビルド時間は確実に延びますが。)

@KageShiron
Copy link
Member

手元で試してみましたが、サイズの半分ぐらいがchmファイル(=圧縮済み)なので劇的には変わらないっぽいです。(ASMの方は半分ぐらいになりました。)

image

@k-takata
Copy link
Member

やっぱりdoxygen要らないのでは?w

@berryzplus
Copy link
Contributor

doxygen生成のchmでかすぎ・・・
ソース触る人が地図代わりにできるように吐き出すものなので、
releaseには要らないのかも・・・とか言い出すとまた複雑な実行制御を組み立てる話にw
作り方だけ共有して、appveyorでは作らない、というのも選択なのかも知れませんね。

@ds14050
Copy link
Contributor

ds14050 commented Dec 30, 2018

releaseには要らないのかも・・・とか言い出すとまた複雑な実行制御を組み立てる話にw

#62698b0d81 を思い出してくださいね。

@m-tmatma m-tmatma changed the title artifacts のサイズ制限にかかってappeyor のビルドに失敗する artifacts のサイズ制限にかかってappveyor のビルドに失敗する Feb 23, 2019
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this issue Jun 11, 2019
…l-artifacts

すべてを含んだ zip を作成しないようにする (sakura-editor#514 の軽減策)
@m-tmatma
Copy link
Member Author

#514 (comment)

再度サイズ制限を拡張してもらえるように依頼しました。
appveyor/ci#2708 (comment)

19 Dec 2018
から半年以上たっている。

当時と比べて開発スピードは格段に下がっているが(ビルドされる頻度が下がっているが)、
7 か月発生していないので閉じます。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI appveyor など CI 関連 【ChangeLog除外】
Projects
None yet
Development

No branches or pull requests

6 participants