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

ArchiveUtilの追加 #272

Merged
merged 3 commits into from
Jul 23, 2019
Merged

Conversation

yamamoto-febc
Copy link
Member

v1から必要なパッケージ類を移行する。

  • util
  • sacloud/ostype
  • sacloud配下の定数やヘルパー群

builderパッケージは別途書き直し。

@yamamoto-febc yamamoto-febc added this to the v2.0.0 milestone Jul 23, 2019
@yamamoto-febc
Copy link
Member Author

要検討:

親アーカイブ/ディスク検索、OSタイプごとのアーカイブ検索などの高レベルAPIをどこに置くか?
数はさほどないため、書き直すbuilderパッケージも含めて全てutilに詰め込む?

@yamamoto-febc yamamoto-febc mentioned this pull request Jul 23, 2019
40 tasks
@yamamoto-febc
Copy link
Member Author

util配下に移行とする。

  • util.ArchiveUtil: アーカイブ検索関連
  • util.ServerUtil: v1のbuilderパッケージを再構成したもの

v1のutilパッケージも再構成する。

  • util/mutexkv: pkgへ移動
  • util/server: ServerUtilへ移動
  • util/setup: v2でのsacloud.StateWaiterが類似機能を持っているため整理する

@yamamoto-febc
Copy link
Member Author

アーカイブ検索時に同一キーを複数用いる場合がある。(Tags.Name)
現在のフィルタの実装はmapなため同一キーを複数指定できない。
これは別PRで対応する。

@yamamoto-febc
Copy link
Member Author

yamamoto-febc commented Jul 23, 2019

#273 #275 にて対応済み

@yamamoto-febc
Copy link
Member Author

yamamoto-febc commented Jul 23, 2019

PRを分けて対応する。

このPRではostypeパッケージとutil/archiveパッケージの導入を行う。

@yamamoto-febc yamamoto-febc changed the title [WIP] v1既存パッケージの移行 ArchiveUtilの追加 Jul 23, 2019
@yamamoto-febc yamamoto-febc marked this pull request as ready for review July 23, 2019 07:33
@yamamoto-febc yamamoto-febc merged commit 1144f13 into sacloud:v2-dev Jul 23, 2019
@yamamoto-febc yamamoto-febc deleted the v2/from-v1 branch July 23, 2019 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant