openstack
/
charm-swift-proxy
Code Issues Proposed changes

Update patch set 6 

Patch Set 6:

(1 comment)

Patch-set: 6
This commit is contained in:
Gerrit User 11805 2023-10-11 12:39:02 +00:00 committed by Gerrit Code Review
parent 3501102211
commit 2ca27f4e2f
1 changed files with 17 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
90a552996f3399c2657c42f4df67d370fd1276de
View File

@ -50,6 +50,23 @@
"message": "Thanks for continuing to try and fix this Freyes. I think Alex\u0027s change that was in his personal branch would be useful to get the debug output:\n\ndiff -Naur /tmp/b/zaza-openstack-tests/zaza/openstack/utilities/swift.py ./zaza/openstack/utilities/swift.py\n--- /tmp/b/zaza-openstack-tests/zaza/openstack/utilities/swift.py 2023-10-11 08:19:00.914932903 -0400\n+++ ./zaza/openstack/utilities/swift.py 2023-10-11 08:19:12.427135837 -0400\n@@ -366,4 +366,7 @@\n result \u003d zaza.model.run_on_leader(proxy_app, cmd, model_name\u003dmodel_name)\n expected \u003d (\u0027{num}/{num} hosts matched, 0 error[s] while checking hosts.\u0027\n \u0027\u0027.format(num\u003dexpected_hosts))\n- return bool(result[\u0027Stdout\u0027].strip(\u0027\\n\u0027) \u003d\u003d expected)\n+ actual \u003d result[\u0027Stdout\u0027].strip(\u0027\\n\u0027)\n+ logging.info(\u0027is_ring_synced: expected: \"%s\" vs actual: \"%s\"\u0027,\n+ expected, actual)\n+ return bool(actual \u003d\u003d expected)",
"revId": "90a552996f3399c2657c42f4df67d370fd1276de",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": false,
"key": {
"uuid": "b2b19cd7_33490fdf",
"filename": "/PATCHSET_LEVEL",
"patchSetId": 6
},
"lineNbr": 0,
"author": {
"id": 11805
},
"writtenOn": "2023-10-11T12:39:02Z",
"side": 1,
"message": "Ah I see you already have a change in master z-o-t for logging. Let\u0027s backport that to stable/antelope.",
"revId": "90a552996f3399c2657c42f4df67d370fd1276de",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
}
]
}