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

Fixes #901: mock2 doesn't update wildcard value #902

Merged
merged 1 commit into from
May 6, 2016

Conversation

tiffanyfay
Copy link
Contributor

@tiffanyfay tiffanyfay commented May 5, 2016

Fixes #901

Summary of changes:

  • Make copy of namespace

Testing done:

  • Created task with mock2 and looked at file output.
    Before:
tail /tmp/snap_published_mock_file_orig.log    ✭
2016-05-05 16:32:31.945949686 -0700 PDT|/intel/mock/host9/baz|83
2016-05-05 16:32:31.945950847 -0700 PDT|/intel/mock/host9/baz|89
2016-05-05 16:32:31.945951236 -0700 PDT|/intel/mock/host9/baz|67
2016-05-05 16:32:31.945952907 -0700 PDT|/intel/mock/host9/baz|71
2016-05-05 16:32:31.945953295 -0700 PDT|/intel/mock/host9/baz|68
2016-05-05 16:32:31.945953668 -0700 PDT|/intel/mock/host9/baz|80
2016-05-05 16:32:31.945954072 -0700 PDT|/intel/mock/host9/baz|72
2016-05-05 16:32:31.945955032 -0700 PDT|/intel/mock/host9/baz|88
2016-05-05 16:32:31.94595542 -0700 PDT|/intel/mock/host9/baz|83
2016-05-05 16:32:31.945955808 -0700 PDT|/intel/mock/host9/baz|83

After:

tail /tmp/snap_published_mock_file.log   130 ↵ ✭
2016-05-05 16:44:53.121801654 -0700 PDT|/intel/mock/host2/baz|89
2016-05-05 16:44:53.121802752 -0700 PDT|/intel/mock/host3/baz|78
2016-05-05 16:44:53.121803272 -0700 PDT|/intel/mock/host4/baz|70
2016-05-05 16:44:53.121809668 -0700 PDT|/intel/mock/host5/baz|85
2016-05-05 16:44:53.121810209 -0700 PDT|/intel/mock/host6/baz|65
2016-05-05 16:44:53.121810687 -0700 PDT|/intel/mock/host7/baz|87
2016-05-05 16:44:53.121811158 -0700 PDT|/intel/mock/host8/baz|87
2016-05-05 16:44:53.121812339 -0700 PDT|/intel/mock/host9/baz|78
2016-05-05 16:44:53.121812867 -0700 PDT|/intel/mock/bar|88

@intelsdi-x/snap-maintainers

@candysmurf
Copy link
Contributor

👍 LGTM

@tiffanyfay tiffanyfay merged commit a579d11 into intelsdi-x:master May 6, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants