openstack/manila-tempest-plugin
Code Issues Proposed changes

Bump test microversion and fix backup test

Browse Source 
We added new field "backup_type" in share-backups API
in 2.85.

Change-Id: Iaaf3c3914089ad0d27b61a524f14353cae35c4d5
Signed-off-by: Goutham Pacha Ravi <gouthampravi@gmail.com>
This commit is contained in:
agireesh 2024-03-11 20:43:00 +05:30 committed by Goutham Pacha Ravi
parent 1687dd6ef6
commit 077fd3d198
2 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
manila_tempest_tests/config.py
View File

@ -40,7 +40,7 @@ ShareGroup = [
"This value is only used to validate the versions " "This value is only used to validate the versions "
"response from Manila."), "response from Manila."),
cfg.StrOpt("max_api_microversion", cfg.StrOpt("max_api_microversion",
default="2.82", default="2.85",
help="The maximum api microversion is configured to be the " help="The maximum api microversion is configured to be the "
"value of the latest microversion supported by Manila."), "value of the latest microversion supported by Manila."),
cfg.StrOpt("region", cfg.StrOpt("region",

5
manila_tempest_tests/tests/api/test_backup.py
View File

@ -54,11 +54,14 @@ class ShareBackupTest(base.BaseSharesMixedTest):
def test_create_share_backup(self): def test_create_share_backup(self):
backup = self.create_backup_wait_for_active(self.share_id) backup = self.create_backup_wait_for_active(self.share_id)
# Verify backup create API response # Verify backup create API response, we use the configured max API
# version to make this call
expected_keys = ["id", "share_id", "status", expected_keys = ["id", "share_id", "status",
"availability_zone", "created_at", "updated_at", "availability_zone", "created_at", "updated_at",
"size", "progress", "restore_progress", "size", "progress", "restore_progress",
"name", "description"] "name", "description"]
if utils.is_microversion_ge(CONF.share.max_api_microversion, '2.85'):
expected_keys.append("backup_type")
# Strict key check # Strict key check
actual_backup = self.shares_v2_client.get_share_backup( actual_backup = self.shares_v2_client.get_share_backup(