[copr] master: W:317, 16: Unnecessary pass statement (unnecessary-pass) (bd0dc7d)
by Miroslav Suchý
Repository : http://git.fedorahosted.org/cgit/copr.git
On branch : master
>---------------------------------------------------------------
commit bd0dc7de25013aa94ae5d1c4c0c86c2a8f4524c3
Author: Miroslav Suchý <msuchy(a)redhat.com>
Date: Fri Nov 6 14:42:47 2015 +0100
W:317,16: Unnecessary pass statement (unnecessary-pass)
>---------------------------------------------------------------
backend/backend/vm_manage/manager.py | 1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a/backend/backend/vm_manage/manager.py b/backend/backend/vm_manage/manager.py
index 3416837..c5858a4 100644
--- a/backend/backend/vm_manage/manager.py
+++ b/backend/backend/vm_manage/manager.py
@@ -314,7 +314,6 @@ class VmManager(object):
result.append(VmDescriptor.load(self.rc, vm_name))
except VmDescriptorNotFound:
self.log.debug("Failed to load VMD: {}".format(vm_name))
- pass
return result
def get_all_vm_in_group(self, group):
8 years, 6 months
[copr] master: W: 32, 4: Unused import fedmsg (unused-import) (4eb8504)
by Miroslav Suchý
Repository : http://git.fedorahosted.org/cgit/copr.git
On branch : master
>---------------------------------------------------------------
commit 4eb8504c115aae26c2487c04e2432984f8557b52
Author: Miroslav Suchý <msuchy(a)redhat.com>
Date: Fri Nov 6 14:33:12 2015 +0100
W: 32, 4: Unused import fedmsg (unused-import)
>---------------------------------------------------------------
backend/backend/helpers.py | 7 -------
1 files changed, 0 insertions(+), 7 deletions(-)
diff --git a/backend/backend/helpers.py b/backend/backend/helpers.py
index 0c11146..42ae03b 100644
--- a/backend/backend/helpers.py
+++ b/backend/backend/helpers.py
@@ -28,13 +28,6 @@ from backend.constants import DEF_BUILD_USER, DEF_BUILD_TIMEOUT, DEF_CONSECUTIVE
CONSECUTIVE_FAILURE_REDIS_KEY, default_log_format
from backend.exceptions import CoprBackendError
-try:
- import fedmsg
-except ImportError:
- # fedmsg is optional
- fedmsg = None
-
-
class SortedOptParser(optparse.OptionParser):
"""Optparser which sorts the options by opt before outputting --help"""
8 years, 6 months
[copr] master: W: 9, 0: Unused import sys (unused-import) (8786a2c)
by Miroslav Suchý
Repository : http://git.fedorahosted.org/cgit/copr.git
On branch : master
>---------------------------------------------------------------
commit 8786a2cf343e70af6282129130d83066a45825c2
Author: Miroslav Suchý <msuchy(a)redhat.com>
Date: Fri Nov 6 14:40:40 2015 +0100
W: 9, 0: Unused import sys (unused-import)
>---------------------------------------------------------------
backend/backend/vm_manage/check.py | 2 --
1 files changed, 0 insertions(+), 2 deletions(-)
diff --git a/backend/backend/vm_manage/check.py b/backend/backend/vm_manage/check.py
index a5e763b..568e6a7 100644
--- a/backend/backend/vm_manage/check.py
+++ b/backend/backend/vm_manage/check.py
@@ -5,8 +5,6 @@ import time
# from multiprocessing import Process
#from threading import Thread
-import sys
-
from ansible.runner import Runner
from backend.helpers import get_redis_connection, format_tb
8 years, 6 months
[copr] master: W: 81, 17: Unused variable 'arch' (unused-variable) (fe6a756)
by Miroslav Suchý
Repository : http://git.fedorahosted.org/cgit/copr.git
On branch : master
>---------------------------------------------------------------
commit fe6a75661cd78e0a1288b2610a093f826f65962a
Author: Miroslav Suchý <msuchy(a)redhat.com>
Date: Fri Nov 6 14:30:41 2015 +0100
W: 81,17: Unused variable 'arch' (unused-variable)
>---------------------------------------------------------------
backend/backend/helpers.py | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/backend/backend/helpers.py b/backend/backend/helpers.py
index 91dca98..695e885 100644
--- a/backend/backend/helpers.py
+++ b/backend/backend/helpers.py
@@ -78,7 +78,7 @@ def chroot_to_branch(chroot):
"""
Get a git branch name from chroot. Follow the fedora naming standard.
"""
- os_name, version, arch = chroot.split("-")
+ os_name, version, _ = chroot.split("-")
if os_name == "fedora":
os_name = "f"
elif os_name == "epel" and int(version) <= 6:
8 years, 6 months
[copr] master: [backend] fix not starting job_grab (16aab5b)
by frostyx@fedoraproject.org
Repository : http://git.fedorahosted.org/cgit/copr.git
On branch : master
>---------------------------------------------------------------
commit 16aab5b4828f2f8dd0c8bf17416578b592229cdb
Author: Jakub Kadlčík <jkadlcik(a)redhat.com>
Date: Fri Nov 6 14:34:31 2015 +0100
[backend] fix not starting job_grab
>---------------------------------------------------------------
backend/backend/actions.py | 2 +-
backend/run/copr_run_job_grab.py | 3 +++
2 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/backend/backend/actions.py b/backend/backend/actions.py
index 3468fdf..d246dfe 100644
--- a/backend/backend/actions.py
+++ b/backend/backend/actions.py
@@ -5,7 +5,7 @@ import time
from urllib import urlretrieve
from munch import Munch
-from copr.client.exceptions import CoprRequestException
+from copr.exceptions import CoprRequestException
from .createrepo import createrepo
from .exceptions import CreateRepoError
diff --git a/backend/run/copr_run_job_grab.py b/backend/run/copr_run_job_grab.py
index 47a549b..b39fabe 100755
--- a/backend/run/copr_run_job_grab.py
+++ b/backend/run/copr_run_job_grab.py
@@ -6,6 +6,9 @@ from __future__ import unicode_literals
from __future__ import division
from __future__ import absolute_import
+import sys
+sys.path.append("/usr/share/copr/")
+
from backend.helpers import get_backend_opts
from backend.daemons.job_grab import CoprJobGrab
from backend.frontend import FrontendClient
8 years, 6 months
[copr] master: Added marshmallow as dep (cfa1175)
by Miroslav Suchý
Repository : http://git.fedorahosted.org/cgit/copr.git
On branch : master
>---------------------------------------------------------------
commit cfa117545a889fbd60b33619d707600cf4eaba6e
Author: Sergio Oliveira <seocam(a)seocam.com>
Date: Thu Nov 5 18:52:24 2015 -0200
Added marshmallow as dep
>---------------------------------------------------------------
python/setup.py | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/python/setup.py b/python/setup.py
index b831232..f24fa06 100644
--- a/python/setup.py
+++ b/python/setup.py
@@ -29,6 +29,7 @@ def find_version(*file_paths):
requires = [
+ 'marshmallow',
'requests',
'requests-toolbelt',
'six'
8 years, 6 months
[copr] master: do not check cert when downloading srpm (e6bf1dc)
by Miroslav Suchý
Repository : http://git.fedorahosted.org/cgit/copr.git
On branch : master
>---------------------------------------------------------------
commit e6bf1dcda3685ea79ea3b28e0ba4e45bcbfa6adb
Author: Miroslav Suchý <miroslav(a)suchy.cz>
Date: Thu Nov 5 20:23:53 2015 +0100
do not check cert when downloading srpm
srpm may reside on server with https with self-signed cert (it just happened)
and we probably should not fail in this case
>---------------------------------------------------------------
dist-git/dist_git/dist_git_importer.py | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/dist-git/dist_git/dist_git_importer.py b/dist-git/dist_git/dist_git_importer.py
index 368bdbc..a4a18cd 100755
--- a/dist-git/dist_git/dist_git_importer.py
+++ b/dist-git/dist_git/dist_git_importer.py
@@ -220,7 +220,7 @@ class SourceDownloader(object):
"""
log.debug("download the package")
try:
- r = get(task.package_url, stream=True)
+ r = get(task.package_url, stream=True, verify=False)
except Exception:
raise PackageDownloadException("Unexpected error during URL fetch: {}"
.format(task.package_url))
8 years, 6 months
[copr] master: since APIv2 we require python-marshmallow (b6283bb)
by Miroslav Suchý
Repository : http://git.fedorahosted.org/cgit/copr.git
On branch : master
>---------------------------------------------------------------
commit b6283bb21d08ac84f31019ef1971aebaf3acef43
Author: Miroslav Suchý <miroslav(a)suchy.cz>
Date: Tue Oct 27 08:41:05 2015 +0100
since APIv2 we require python-marshmallow
>---------------------------------------------------------------
python/python-copr.spec | 4 ++++
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/python/python-copr.spec b/python/python-copr.spec
index 12740dd..d232cb0 100644
--- a/python/python-copr.spec
+++ b/python/python-copr.spec
@@ -29,6 +29,7 @@ BuildRequires: util-linux
BuildRequires: python-setuptools
BuildRequires: python-requests
BuildRequires: python-requests-toolbelt
+BuildRequires: python-marshmallow
BuildRequires: python-six
BuildRequires: python-mock
BuildRequires: pytest
@@ -41,6 +42,7 @@ Requires: python-setuptools
Requires: python-six
Requires: python-requests
Requires: python-requests-toolbelt
+Requires: python-marshmallow
%description
COPR is lightweight build system. It allows you to create new project in WebUI,
@@ -61,12 +63,14 @@ BuildRequires: python3-pytest
BuildRequires: python3-mock
BuildRequires: python3-requests
BuildRequires: python3-requests-toolbelt
+BuildRequires: python3-marshmallow
BuildRequires: python3-six
Requires: python3-setuptools
Requires: python3-six
Requires: python3-requests
Requires: python3-requests-toolbelt
+Requires: python3-marshmallow
%description -n python3-copr
COPR is lightweight build system. It allows you to create new project in WebUI,
8 years, 6 months
[copr] master: [vagrant] use wget instead of outdated dnf copr plugin (592ea6f)
by frostyx@fedoraproject.org
Repository : http://git.fedorahosted.org/cgit/copr.git
On branch : master
>---------------------------------------------------------------
commit 592ea6fe61881d7e95359acc2191a9c859be09ed
Author: Jakub Kadlčík <jkadlcik(a)redhat.com>
Date: Tue Nov 3 14:20:55 2015 +0100
[vagrant] use wget instead of outdated dnf copr plugin
>---------------------------------------------------------------
Vagrantfile | 4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/Vagrantfile b/Vagrantfile
index e1f0fcb..08e10de 100644
--- a/Vagrantfile
+++ b/Vagrantfile
@@ -31,7 +31,9 @@ Vagrant.configure(2) do |config|
# Enable the Copr repository for dependencies
frontend.vm.provision "shell",
- inline: "sudo dnf -y copr enable msuchy/copr"
+ # inline: "sudo dnf -y copr enable msuchy/copr"
+ # WORKAROUND: old DNF plugin uses outdated .repo URL
+ inline: "sudo wget https://copr.fedoraproject.org/coprs/msuchy/copr/repo/fedora-21/msuchy-co... -P /etc/yum.repos.d/"
# Install build dependencies for Copr Frontend
frontend.vm.provision "shell",
8 years, 6 months