Skip to content

Commit

Permalink
DAOS-14698 crtl: Create container with attributes
Browse files Browse the repository at this point in the history
Fix pylint errors.

Features: control dfuse container daos_cmd
Required-githooks: true

Signed-off-by: Cedric Koch-Hofer <[email protected]>
  • Loading branch information
kanard38 authored and knard-intel committed Nov 15, 2024
1 parent af54e76 commit d230c87
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
1 change: 1 addition & 0 deletions src/tests/ftest/dfuse/container_attrs.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -34,3 +34,4 @@ container_03:
type: POSIX
attrs: dfuse-attr-time:42,dfuse-dentry-time:404
control_method: daos
path: foo
6 changes: 3 additions & 3 deletions src/tests/ftest/util/dfuse_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -512,7 +512,7 @@ def destroy_sub_cont(self, cont):
return False

if cont not in self.sub_conts:
self.log.info(f"DAOS container {cont} is not a DFuse sub-container")
self.log.info("DAOS container %s is not a DFuse sub-container", str(cont))
return False

host = self._running_hosts[0]
Expand All @@ -522,8 +522,8 @@ def destroy_sub_cont(self, cont):
result = run_remote(self.log, host, str(cont.daos))
if not result.passed:
self.log.info(
f"DFuse sub-container {cont} can not be destroyed on {result.failed_hosts}: "
f"{result.output[0].stderr}")
"DFuse sub-container %s can not be destroyed on %s: %s",
cont, result.failed_hosts, result.output[0].stderr)
return False

self.sub_conts.remove(cont)
Expand Down

0 comments on commit d230c87

Please sign in to comment.