From e3af89a3ad19666195d625db89e7eb5dadfce792 Mon Sep 17 00:00:00 2001 From: Rishabh Dave Date: Tue, 8 Feb 2022 00:14:42 +0530 Subject: [PATCH] monitoring: mention PyYAML only once in requirements Following error occurs while running "sudo install-deps.sh" - ERROR: Double requirement given: PyYAML==6.0 (from -r requirements-lint.txt (line 5)) (already in pyyaml (from -r requirements-alerts.txt (line 1)), name='PyYAML') PyYAML is mentioned twice as a requirement. It is mentioned once in both the following files - monitoring/ceph-mixin/requirements-lint.txt monitoring/ceph-mixin/requirements-alerts.txt These requirements were added in commits 44d3e4c264506154373ffaeb13d6c924c580e6b5 and 4750ac0d7766a8a089adf073415af0ac0d3f81d9. Fixes: https://tracker.ceph.com/issues/54185 Signed-off-by: Rishabh Dave (cherry picked from commit a6f5efb620c429f81ea13992c2f77b4ca55458bc) --- monitoring/prometheus/tests/requirements.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/monitoring/prometheus/tests/requirements.txt b/monitoring/prometheus/tests/requirements.txt index bd25cccf86d77..e518e69644e66 100644 --- a/monitoring/prometheus/tests/requirements.txt +++ b/monitoring/prometheus/tests/requirements.txt @@ -1,2 +1,2 @@ -pyyaml -bs4 \ No newline at end of file +pyyaml==6.0 +bs4 -- 2.39.5