Skip to content
This repository has been archived by the owner on Nov 9, 2020. It is now read-only.

Commit

Permalink
Add log to dump kv_str (#1388)
Browse files Browse the repository at this point in the history
* Add log to dump the kv_str reading from meta file.
* set "policyList" to empty in SetupTest for VsanTestSuite
  • Loading branch information
lipingxue committed Jun 12, 2017
1 parent 45b3a81 commit f5dae96
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 1 deletion.
2 changes: 2 additions & 0 deletions esx_service/utils/kvESX.py
Original file line number Diff line number Diff line change
Expand Up @@ -279,6 +279,8 @@ def load(volpath):
try:
return json.loads(kv_str)
except ValueError:
# Adding this log for DEBUG
logging.warning("kv_str from meta file is %s ", kv_str)
logging.exception("Failed to decode meta-data for %s", volpath)
return None

Expand Down
2 changes: 1 addition & 1 deletion tests/e2e/volume_access_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
// The goal of this test suite is to verify read/write consistency on volumes
// in accordance with the access updates on the volume

// +build runonce
// +build runalways

package e2e

Expand Down
4 changes: 4 additions & 0 deletions tests/e2e/vsan_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,10 @@ func (s *VsanTestSuite) SetUpSuite(c *C) {
}
}

func (s *VsanTestSuite) SetUpTest(c *C) {
s.policyList = []string{}
}

func (s *VsanTestSuite) TearDownTest(c *C) {
if s.volumeName != "" {
out, err := dockercli.DeleteVolume(s.config.DockerHosts[0], s.volumeName)
Expand Down

0 comments on commit f5dae96

Please sign in to comment.