Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable vips and security scans #3878

Draft
wants to merge 4 commits into
base: ng
Choose a base branch
from
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/ralph/admin/sitetrees.py
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,7 @@ def section(section_name, app, model):
section(_('Databases'), 'data_center', 'Database'),
section(_('Disk Shares'), 'data_center', 'DiskShare'),
section(_('Server Rooms'), 'data_center', 'ServerRoom'),
section(_('VIPs'), 'data_center', 'VIP'),
# section(_('VIPs'), 'data_center', 'VIP'),
section(_('Preboots'), 'deployment', 'Preboot'),
section(
_('Preboot configuration'), 'deployment',
Expand Down
8 changes: 4 additions & 4 deletions src/ralph/api/tests/test_rendering.py
Original file line number Diff line number Diff line change
Expand Up @@ -85,8 +85,8 @@ class APIBrowsableClient(APIClient):
"rack-accessories": "/api/rack-accessories/",
"racks": "/api/racks/",
"regions": "/api/regions/",
"scm-info": "/api/scm-info/",
"security-scans": "/api/security-scans/",
# "scm-info": "/api/scm-info/",
# "security-scans": "/api/security-scans/",
"server-rooms": "/api/server-rooms/",
"services": "/api/services/",
"services-environments": "/api/services-environments/",
Expand All @@ -104,10 +104,10 @@ class APIBrowsableClient(APIClient):
"transitions-job": "/api/transitions-job/",
"transitions-model": "/api/transitions-model/",
"users": "/api/users/",
"vips": "/api/vips/",
# "vips": "/api/vips/",
"virtual-server-types": "/api/virtual-server-types/",
"virtual-servers": "/api/virtual-servers/",
"vulnerabilities": "/api/vulnerabilities/",
# "vulnerabilities": "/api/vulnerabilities/",
"warehouses": "/api/warehouses/"
}

Expand Down
2 changes: 1 addition & 1 deletion src/ralph/assets/admin.py
Original file line number Diff line number Diff line change
Expand Up @@ -167,7 +167,7 @@ def get_service_base_objects_queryset(self):
service_env__service=self.object
).select_related(
'service_env__environment', 'content_type', 'securityscan'
)
).exclude(content_type__model='vip')

def get_context_data(self, **kwargs):
context = super().get_context_data(**kwargs)
Expand Down
2 changes: 0 additions & 2 deletions src/ralph/assets/api/serializers.py
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,6 @@
Memory,
Processor
)
from ralph.configuration_management.api import SCMInfoSerializer
from ralph.lib.custom_fields.api import WithCustomFieldsSerializerMixin
from ralph.licences.api_simple import SimpleBaseObjectLicenceSerializer
from ralph.networks.api_simple import IPAddressSimpleSerializer
Expand Down Expand Up @@ -258,7 +257,6 @@ class BaseObjectPolymorphicSerializer(
"""
__str__ = StrField(show_type=True)
service_env = ServiceEnvironmentSerializer()
scmstatuscheck = SCMInfoSerializer()

class Meta:
model = BaseObject
Expand Down
4 changes: 3 additions & 1 deletion src/ralph/assets/tests/test_api.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
# -*- coding: utf-8 -*-
from unittest import skip
from urllib.parse import urlencode

from ddt import data, ddt, unpack
Expand Down Expand Up @@ -571,7 +572,7 @@ def test_patch_asset_model(self):
ServiceEnvironment: ServiceEnvironmentFactory,
SSLCertificate: SSLCertificatesFactory,
Support: SupportFactory,
VIP: VIPFactory,
# VIP: VIPFactory,
VirtualServer: VirtualServerFactory,
Cluster: ClusterFactory,
ConfigurationClass: ConfigurationClassFactory,
Expand Down Expand Up @@ -708,6 +709,7 @@ def test_filter_by_ip(self):
response = self.client.get(url, format='json')
self.assertEqual(len(response.data['results']), 1)

@skip("SCM Status is disabled")
def test_filter_by_scm_status_check_result(self):
status_checks = {
SCMCheckResult.scm_error: 2,
Expand Down
4 changes: 2 additions & 2 deletions src/ralph/configuration_management/api.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
from rest_framework import status
from rest_framework.response import Response

from ralph.api import RalphAPISerializer, RalphAPIViewSet, router
from ralph.api import RalphAPISerializer, RalphAPIViewSet
from ralph.api.serializers import RalphAPISaveSerializer
from ralph.assets.models import BaseObject
from ralph.configuration_management.models import SCMStatusCheck
Expand Down Expand Up @@ -100,7 +100,7 @@ def create(self, request, hostname):
return Response(self.serializer_class(scan).data, status=res_status)


router.register('scm-info', SCMInfoViewSet)
# router.register('scm-info', SCMInfoViewSet)
urlpatterns = [
url(
r'^scm-info/(?P<hostname>[\w\.-]+)',
Expand Down
37 changes: 6 additions & 31 deletions src/ralph/data_center/admin.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@
from django.utils.safestring import mark_safe
from django.utils.translation import ugettext_lazy as _

from ralph.admin import filters
from ralph.admin.decorators import register
from ralph.admin.filters import (
BaseObjectHostnameFilter,
Expand All @@ -21,8 +20,7 @@
MacAddressFilter,
RelatedAutocompleteFieldListFilter,
TagsListFilter,
TreeRelatedAutocompleteFilterWithDescendants,
VulnerabilitesByPatchDeadline
TreeRelatedAutocompleteFilterWithDescendants
)
from ralph.admin.helpers import generate_html_link
from ralph.admin.mixins import (
Expand All @@ -39,10 +37,7 @@
from ralph.assets.models.components import Ethernet
from ralph.assets.views import ComponentsAdminView
from ralph.attachments.admin import AttachmentsMixin
from ralph.configuration_management.views import (
SCMCheckInfo,
SCMStatusCheckInChangeListMixin
)
from ralph.configuration_management.views import SCMCheckInfo
from ralph.data_center.forms import DataCenterAssetForm
from ralph.data_center.models.components import DiskShare, DiskShareMount
from ralph.data_center.models.hosts import DCHost
Expand All @@ -59,8 +54,7 @@
BaseObjectCluster,
Cluster,
ClusterType,
Database,
VIP
Database
)
from ralph.data_center.views import RelationsView
from ralph.data_importer import resources
Expand All @@ -72,7 +66,7 @@
from ralph.networks.forms import SimpleNetworkWithManagementIPForm
from ralph.networks.views import NetworkWithTerminatorsView
from ralph.operations.views import OperationViewReadOnlyForExisiting
from ralph.security.views import ScanStatusInChangeListMixin, SecurityInfo
from ralph.security.views import SecurityInfo
from ralph.supports.models import BaseObjectsSupport


Expand All @@ -92,15 +86,6 @@ def generate_list_filter_with_common_fields(
),
MacAddressFilter,
IPFilter,
(
'securityscan__vulnerabilities__patch_deadline',
VulnerabilitesByPatchDeadline
),
(
'securityscan__vulnerabilities',
filters.RelatedAutocompleteFieldListFilter
),
'securityscan__is_patched',
]
)
if type(postfix) == list:
Expand Down Expand Up @@ -361,8 +346,6 @@ class DataCenterAssetRelationsView(RelationsView):

@register(DataCenterAsset)
class DataCenterAssetAdmin(
SCMStatusCheckInChangeListMixin,
ScanStatusInChangeListMixin,
ActiveDeploymentMessageMixin,
MulitiAddAdminMixin,
TransitionAdminMixin,
Expand All @@ -379,8 +362,6 @@ class DataCenterAssetAdmin(
change_views = [
DataCenterAssetComponents,
DataCenterAssetNetworkView,
DataCenterAssetSecurityInfo,
DataCenterAssetSCMInfo,
DataCenterAssetRelationsView,
DataCenterAssetLicence,
DataCenterAssetSupport,
Expand All @@ -402,8 +383,6 @@ class DataCenterAssetAdmin(
'show_location',
'service_env',
'configuration_path',
'scan_status',
'scm_status_check',
'property_of',
'order_no'
]
Expand Down Expand Up @@ -621,7 +600,7 @@ class DatabaseAdmin(RalphAdmin):
pass


@register(VIP)
# @register(VIP)
class VIPAdmin(RalphAdmin):
search_fields = ['name', 'ip__address']
list_display = ['name', 'ip', 'port', 'protocol', 'service_env']
Expand Down Expand Up @@ -664,8 +643,6 @@ class DCHostSCMInfo(SCMCheckInfo):

@register(DCHost)
class DCHostAdmin(
SCMStatusCheckInChangeListMixin,
ScanStatusInChangeListMixin,
RalphAdmin
):
change_list_template = 'admin/data_center/dchost/change_list.html'
Expand All @@ -684,9 +661,7 @@ class DCHostAdmin(
'service_env',
'configuration_path',
'show_location',
'remarks',
'scan_status',
'scm_status_check'
'remarks'
]
# TODO: sn
# TODO: hostname, DC
Expand Down
5 changes: 2 additions & 3 deletions src/ralph/data_center/api/routers.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,7 @@
DataCenterViewSet,
RackAccessoryViewSet,
RackViewSet,
ServerRoomViewSet,
VIPViewSet
ServerRoomViewSet
)

router.register(r'accessories', AccessoryViewSet)
Expand All @@ -21,7 +20,7 @@
router.register(r'racks', RackViewSet)
router.register(r'rack-accessories', RackAccessoryViewSet)
router.register(r'server-rooms', ServerRoomViewSet)
router.register(r'vips', VIPViewSet)
# router.register(r'vips', VIPViewSet)
router.register(r'clusters', ClusterViewSet)
router.register(r'cluster-types', ClusterTypeViewSet)
router.register(r'base-object-clusters', BaseObjectClusterViewSet)
Expand Down
4 changes: 0 additions & 4 deletions src/ralph/data_center/api/serializers.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@
NetworkComponentSerializerMixin,
OwnersFromServiceEnvSerializerMixin
)
from ralph.configuration_management.api import SCMInfoSerializer
from ralph.data_center.models import (
Accessory,
BaseObjectCluster,
Expand All @@ -25,7 +24,6 @@
ServerRoom,
VIP
)
from ralph.security.api import SecurityScanSerializer


class ClusterTypeSerializer(RalphAPISerializer):
Expand Down Expand Up @@ -126,8 +124,6 @@ class Meta:

class DataCenterAssetSerializer(ComponentSerializerMixin, AssetSerializer):
rack = SimpleRackSerializer()
scmstatuscheck = SCMInfoSerializer()
securityscan = SecurityScanSerializer()
related_hosts = serializers.SerializerMethodField()

def get_related_hosts(self, obj):
Expand Down
5 changes: 3 additions & 2 deletions src/ralph/data_center/tests/test_view.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
from datetime import datetime, timedelta
from unittest import skip
from urllib.parse import urlencode

from django.contrib.contenttypes.models import ContentType
Expand Down Expand Up @@ -118,7 +119,7 @@ def test_changelist_cluster_location(self):
)
self.assertContains(result, 'DC1 / SR1 / Rack #1')


@skip("Security Scans are disabled")
class DCHostScanStatusInListingTest(ClientMixin, TestCase):

def setUp(self):
Expand Down Expand Up @@ -212,7 +213,7 @@ def test_listing_show_failed_icon_when_scan_error(self):
)
self.assertContains(result, "Scan failed")


@skip("Security Scans are disabled")
class DCHostFilterByPatchDeadline(ClientMixin, TestCase):
def setUp(self):
self.login_as_user()
Expand Down
2 changes: 2 additions & 0 deletions src/ralph/lib/polymorphic/models.py
Original file line number Diff line number Diff line change
Expand Up @@ -331,6 +331,8 @@ def __new__(cls, name, bases, attrs):
if new_class._meta.proxy:
continue
try:
if new_class._meta.model_name == 'vip':
continue
polymorphic_class._polymorphic_descendants.append(new_class)
except AttributeError:
# The exception is for class Polymorphic
Expand Down
6 changes: 3 additions & 3 deletions src/ralph/security/api.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
from django.db import transaction
from rest_framework import serializers

from ralph.api import RalphAPISerializer, RalphAPIViewSet, router
from ralph.api import RalphAPISerializer, RalphAPIViewSet
from ralph.api.serializers import RalphAPISaveSerializer
from ralph.networks.models.networks import IPAddress
from ralph.security.models import any_exceeded, SecurityScan, Vulnerability
Expand Down Expand Up @@ -117,6 +117,6 @@ def create(self, request, *args, **kwargs):
return super().create(request, *args, **kwargs)


router.register(r'vulnerabilities', VulnerabilityViewSet)
router.register(r'security-scans', SecurityScanViewSet)
# router.register(r'vulnerabilities', VulnerabilityViewSet)
# router.register(r'security-scans', SecurityScanViewSet)
urlpatterns = []
7 changes: 4 additions & 3 deletions src/ralph/security/tests/test_api.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
# -*- coding: utf-8 -*-
from datetime import datetime, timedelta
from unittest import skip

from django.urls import reverse
from rest_framework import status
Expand All @@ -12,7 +13,7 @@
VulnerabilityFactory
)


@skip('Security Scans are disabled')
class SecurityScanAPITests(RalphAPITestCase):

def setUp(self):
Expand Down Expand Up @@ -226,7 +227,7 @@ def test_posting_scan_replace_old_one_when_scan_already_exists(self):
scan.id,
)


@skip("Vulnerabilities are disabled")
class VulnerabilityAPITests(RalphAPITestCase):

def setUp(self):
Expand Down Expand Up @@ -317,7 +318,7 @@ def test_patch_vulnerability(self):
self.vulnerability.external_vulnerability_id,
data['external_vulnerability_id'])


@skip("Vulnerabilities are disabled")
class TestExternalVulnerability(RalphAPITestCase):
def setUp(self):
super().setUp()
Expand Down
2 changes: 1 addition & 1 deletion src/ralph/urls/base.py
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@
'ralph.domains.api',
'ralph.operations.api',
'ralph.supports.api',
'ralph.security.api',
# 'ralph.security.api',
'ralph.sim_cards.api',
'ralph.ssl_certificates.api',
'ralph.networks.api',
Expand Down
Loading
Loading