6:22 a.m.
From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has uploaded a new change for review.
Change subject: build: no one needs python-rtslib
......................................................................
build: no one needs python-rtslib
commit df5ebfd removed all reference to rtslib. There is no need to pull
the package any more.
Change-Id: Iac4280c1ed36693e61bcfb80ae55c1313b762344
Signed-off-by: Dan Kenigsberg <danken(a)redhat.com>
---
M automation/build-artifacts.packages.el7
M automation/build-artifacts.packages.fc24
M automation/check-merged.packages.el7
M automation/check-merged.packages.fc24
M automation/check-patch.packages.el7
5 files changed, 0 insertions(+), 5 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/47/66247/1
diff --git a/automation/build-artifacts.packages.el7
b/automation/build-artifacts.packages.el7
index 7f2a880..58927c9 100644
--- a/automation/build-artifacts.packages.el7
+++ b/automation/build-artifacts.packages.el7
@@ -19,7 +19,6 @@
python-mock
python-netaddr
python-pthreading
-python-rtslib
python-setuptools
python3-netaddr
python3-nose
diff --git a/automation/build-artifacts.packages.fc24
b/automation/build-artifacts.packages.fc24
index 08c7506..84be9bd 100644
--- a/automation/build-artifacts.packages.fc24
+++ b/automation/build-artifacts.packages.fc24
@@ -19,7 +19,6 @@
python-mock
python-netaddr
python-pthreading
-python-rtslib
python-setuptools
python3-decorator
python3-netaddr
diff --git a/automation/check-merged.packages.el7 b/automation/check-merged.packages.el7
index 1c43150..aaee0b1 100644
--- a/automation/check-merged.packages.el7
+++ b/automation/check-merged.packages.el7
@@ -18,7 +18,6 @@
python-mock
python-netaddr
python-pthreading
-python-rtslib
python-six
python34-nose
python34-six
diff --git a/automation/check-merged.packages.fc24
b/automation/check-merged.packages.fc24
index 83cccaa..cb869f7 100644
--- a/automation/check-merged.packages.fc24
+++ b/automation/check-merged.packages.fc24
@@ -22,7 +22,6 @@
python-mock
python-netaddr
python-pthreading
-python-rtslib
python-six
python3-decorator
python3-netaddr
diff --git a/automation/check-patch.packages.el7 b/automation/check-patch.packages.el7
index 937f0ea..e702fe2 100644
--- a/automation/check-patch.packages.el7
+++ b/automation/check-patch.packages.el7
@@ -17,7 +17,6 @@
python-mock
python-netaddr
python-pthreading
-python-rtslib
python-setuptools
python-six
rpm-build
--
To view, visit https://gerrit.ovirt.org/66247
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iac4280c1ed36693e61bcfb80ae55c1313b762344
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com>
8:18 a.m.
New subject: Change in vdsm[master]: automation: no one needs python-rtslib
From Yaniv Bronhaim <ybronhei(a)redhat.com>:
Yaniv Bronhaim has posted comments on this change.
Change subject: automation: no one needs python-rtslib
......................................................................
Patch Set 2: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/66247
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iac4280c1ed36693e61bcfb80ae55c1313b762344
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
Gerrit-HasComments: No
8:41 a.m.
New subject: Change in vdsm[master]: automation: no one needs python-rtslib
From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has posted comments on this change.
Change subject: automation: no one needs python-rtslib
......................................................................
Patch Set 2: Verified+1
still passes CI
--
To view, visit https://gerrit.ovirt.org/66247
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iac4280c1ed36693e61bcfb80ae55c1313b762344
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Leon Goldberg <lgoldber(a)redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
Gerrit-HasComments: No
10:14 a.m.
New subject: Change in vdsm[master]: automation: no one needs python-rtslib
From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has posted comments on this change.
Change subject: automation: no one needs python-rtslib
......................................................................
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.ovirt.org/66247
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iac4280c1ed36693e61bcfb80ae55c1313b762344
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Leon Goldberg <lgoldber(a)redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
Gerrit-HasComments: No
10:15 a.m.
New subject: Change in vdsm[master]: automation: no one needs python-rtslib
From Dan Kenigsberg <danken(a)redhat.com>:
Dan Kenigsberg has submitted this change and it was merged.
Change subject: automation: no one needs python-rtslib
......................................................................
automation: no one needs python-rtslib
commit df5ebfd removed all reference to rtslib. There is no need to pull
the package any more.
Change-Id: Iac4280c1ed36693e61bcfb80ae55c1313b762344
Signed-off-by: Dan Kenigsberg <danken(a)redhat.com>
Reviewed-on: https://gerrit.ovirt.org/66247
Continuous-Integration: Jenkins CI
Reviewed-by: Yaniv Bronhaim <ybronhei(a)redhat.com>
Reviewed-by: Leon Goldberg <lgoldber(a)redhat.com>
---
M automation/build-artifacts.packages.el7
M automation/build-artifacts.packages.fc24
M automation/check-merged.packages.el7
M automation/check-merged.packages.fc24
M automation/check-patch.packages.el7
5 files changed, 0 insertions(+), 5 deletions(-)
Approvals:
Yaniv Bronhaim: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
Leon Goldberg: Looks good to me, but someone else must approve
Dan Kenigsberg: Verified; Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/66247
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iac4280c1ed36693e61bcfb80ae55c1313b762344
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Leon Goldberg <lgoldber(a)redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
2777
days inactive
2777
days old
vdsm-patches@lists.fedorahosted.org
4 comments
1 participants
participants (1)
-
Code Review