-
Notifications
You must be signed in to change notification settings - Fork 15
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
fix acc tests for following resources. #29
Conversation
- Instance - Volume - Volume Attach - Port Reason of this change 1: Depending on specification change of HCL2, l HCL grammar for objects and list of objects are changed strictly. Reason of this change 2: Depending on OpenStack settings in each region default zone setting is needed to pass the tests.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1箇所コメントしました。
`, testCreateNetworkForInstance, OS_IMAGE_ID) | ||
`, testCreateNetworkForInstance, | ||
OS_DEFAULT_ZONE, | ||
OS_DEFAULT_ZONE, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OS_DEFAULT_ZONE
を2つ渡しています。 これは大丈夫でしょうか。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
はい、別なリソースに対する渡し方なので問題ないかと。
あとSptinfのプレースホルダーそれぞれに渡しているという意味になります。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
失礼しました。 理解しました。 Approve 済みです。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM:)
Reason of this change 1:
Depending on specification change of HCL2, l
HCL grammar for objects and list of objects are
changed strictly.
Reason of this change 2:
Depending on OpenStack settings in
each region default zone setting is needed
to pass the tests.