Closed takaishi closed 5 years ago
takaishi
commented
5 years ago タイトルと概要がPRの趣旨とずれていたので修正しました。
この修正はYao::Tenantのインスタンスメソッドの修正で、adminではない場合Yao::Tenantの取得ができません。そのためall_tenantsを使っても問題はないと判断しました。また、もともとall_tenantsを使っていたのを使わなくなったために発生したバグであることからもall_tenantsを使うことは問題ないかと思います。
Yao::Projectについてもall_tenantsを追加していますが、これは openstack server list でテナントを指定した場合にall_tenantsが使われているためそれに揃える目的があります。
buty4649
commented
5 years ago ありがとうございます。そして、私が勘違いしていたのですが、 project_id(tenant_id) はall_tenantsが指定されている場合のみ動くようです。 なので、提案していただいた内容でも全く問題なかったです :bowing_man:
This filter only works when the all_tenants filter is also specified.おーっ、そういう制限があるのですねえ。ありがとうございます！
https://github.com/yaocloud/yao/commit/91e2eeb332c1173f46917bc102bf9faf35474d25 の変更により、Yao::Tenantの servers インスタンスメソッドが空配列を返すようになったためこれを修正します。
原因ですが、HavanaのAPIでインスタンス一覧を取得する際、
all_tenantsオプションがない場合project_idで正しく絞り込めない…というものです。all_tenantsを有効にすることでテナントに所属するインスタンス一覧が返ってきます。