jenkins-bot has submitted this change and it was merged.
Change subject: Clear manifests before every collect
......................................................................
Clear manifests before every collect
Bug: T95210
Change-Id: I0c55e346e39add99f651350076ff2631165e70fc
---
M destiny/collector.py
1 file changed, 3 insertions(+), 1 deletion(-)
Approvals:
Legoktm: Looks good to me, approved
jenkins-bot: Verified
diff --git a/destiny/collector.py b/destiny/collector.py
index 20d8294..e981818 100644
--- a/destiny/collector.py
+++ b/destiny/collector.py
@@ -49,6 +49,7 @@
"""
manifest_files = glob.glob(ManifestCollector.MANIFEST_GLOB_PATTERN)
self.log.info("Collecting manifests with pattern %s",
ManifestCollector.MANIFEST_GLOB_PATTERN)
+ manifests = []
for manifest_file in manifest_files:
fileparts = manifest_file.split('/')
toolname = fileparts[3] # FIXME: Have extra validation to make
sure this *is* a tool
@@ -63,7 +64,8 @@
with open(manifest_file) as f:
manifest = Manifest(toolname, yaml.safe_load(f))
- self.manifests.append(manifest)
+ manifests.append(manifest)
+ self.manifests = manifests
self.log.info("Collected %s manifests", len(self.manifests))
def run(self):
--
To view, visit https://gerrit.wikimedia.org/r/202291
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I0c55e346e39add99f651350076ff2631165e70fc
Gerrit-PatchSet: 1
Gerrit-Project: operations/software/tools-manifest
Gerrit-Branch: master
Gerrit-Owner: Yuvipanda <[email protected]>
Gerrit-Reviewer: Legoktm <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
