catch all exceptions and save de snapshot
This commit is contained in:
parent
e36256ee72
commit
0e89af1215
|
@ -1,5 +1,6 @@
|
||||||
import os
|
import os
|
||||||
import json
|
import json
|
||||||
|
import copy
|
||||||
from time import time
|
from time import time
|
||||||
from distutils.version import StrictVersion
|
from distutils.version import StrictVersion
|
||||||
from typing import List
|
from typing import List
|
||||||
|
@ -28,11 +29,12 @@ def save_snapshot_in_file(snapshot_json):
|
||||||
This function allow save a snapshot in json format un a TMP_SNAPSHOTS directory
|
This function allow save a snapshot in json format un a TMP_SNAPSHOTS directory
|
||||||
The file need to be saved with one name format with the stamptime and uuid joins
|
The file need to be saved with one name format with the stamptime and uuid joins
|
||||||
"""
|
"""
|
||||||
|
name_file = "{uuid}_{time}.json".format(uuid=snapshot_json['uuid'], time=int(time()))
|
||||||
|
path_name = "{tmp}/{file}".format(tmp=TMP_SNAPSHOTS, file=name_file)
|
||||||
|
|
||||||
if not os.path.isdir(TMP_SNAPSHOTS):
|
if not os.path.isdir(TMP_SNAPSHOTS):
|
||||||
os.system('mkdir -p {}'.format(TMP_SNAPSHOTS))
|
os.system('mkdir -p {}'.format(TMP_SNAPSHOTS))
|
||||||
|
|
||||||
name_file = "{uuid}_{time}.json".format(uuid=snapshot_json['uuid'], time=int(time()))
|
|
||||||
path_name = "{tmp}/{file}".format(tmp=TMP_SNAPSHOTS, file=name_file)
|
|
||||||
snapshot_file = open(path_name, 'w')
|
snapshot_file = open(path_name, 'w')
|
||||||
snapshot_file.write(json.dumps(snapshot_json))
|
snapshot_file.write(json.dumps(snapshot_json))
|
||||||
snapshot_file.close()
|
snapshot_file.close()
|
||||||
|
@ -78,62 +80,67 @@ class ActionView(View):
|
||||||
# model object, when we flush them to the db we will flush
|
# model object, when we flush them to the db we will flush
|
||||||
# snapshot, and we want to wait to flush snapshot at the end
|
# snapshot, and we want to wait to flush snapshot at the end
|
||||||
|
|
||||||
device = snapshot_json.pop('device') # type: Computer
|
snapshot_json_bakup = copy.copy(snapshot_json)
|
||||||
components = None
|
try:
|
||||||
if snapshot_json['software'] == (SnapshotSoftware.Workbench or SnapshotSoftware.WorkbenchAndroid):
|
device = snapshot_json.pop('device') # type: Computer
|
||||||
components = snapshot_json.pop('components', None) # type: List[Component]
|
components = None
|
||||||
snapshot = Snapshot(**snapshot_json)
|
if snapshot_json['software'] == (SnapshotSoftware.Workbench or SnapshotSoftware.WorkbenchAndroid):
|
||||||
|
components = snapshot_json.pop('components', None) # type: List[Component]
|
||||||
|
snapshot = Snapshot(**snapshot_json)
|
||||||
|
|
||||||
# Remove new actions from devices so they don't interfere with sync
|
# Remove new actions from devices so they don't interfere with sync
|
||||||
actions_device = set(e for e in device.actions_one)
|
actions_device = set(e for e in device.actions_one)
|
||||||
device.actions_one.clear()
|
device.actions_one.clear()
|
||||||
if components:
|
if components:
|
||||||
actions_components = tuple(set(e for e in c.actions_one) for c in components)
|
actions_components = tuple(set(e for e in c.actions_one) for c in components)
|
||||||
for component in components:
|
for component in components:
|
||||||
component.actions_one.clear()
|
component.actions_one.clear()
|
||||||
|
|
||||||
assert not device.actions_one
|
assert not device.actions_one
|
||||||
assert all(not c.actions_one for c in components) if components else True
|
assert all(not c.actions_one for c in components) if components else True
|
||||||
db_device, remove_actions = resource_def.sync.run(device, components)
|
db_device, remove_actions = resource_def.sync.run(device, components)
|
||||||
|
|
||||||
del device # Do not use device anymore
|
del device # Do not use device anymore
|
||||||
snapshot.device = db_device
|
snapshot.device = db_device
|
||||||
snapshot.actions |= remove_actions | actions_device # Set actions to snapshot
|
snapshot.actions |= remove_actions | actions_device # Set actions to snapshot
|
||||||
# commit will change the order of the components by what
|
# commit will change the order of the components by what
|
||||||
# the DB wants. Let's get a copy of the list so we preserve order
|
# the DB wants. Let's get a copy of the list so we preserve order
|
||||||
ordered_components = OrderedSet(x for x in snapshot.components)
|
ordered_components = OrderedSet(x for x in snapshot.components)
|
||||||
|
|
||||||
# Add the new actions to the db-existing devices and components
|
# Add the new actions to the db-existing devices and components
|
||||||
db_device.actions_one |= actions_device
|
db_device.actions_one |= actions_device
|
||||||
if components:
|
if components:
|
||||||
for component, actions in zip(ordered_components, actions_components):
|
for component, actions in zip(ordered_components, actions_components):
|
||||||
component.actions_one |= actions
|
component.actions_one |= actions
|
||||||
snapshot.actions |= actions
|
snapshot.actions |= actions
|
||||||
|
|
||||||
if snapshot.software == SnapshotSoftware.Workbench:
|
if snapshot.software == SnapshotSoftware.Workbench:
|
||||||
# Check ownership of (non-component) device to from current.user
|
# Check ownership of (non-component) device to from current.user
|
||||||
if db_device.owner_id != g.user.id:
|
if db_device.owner_id != g.user.id:
|
||||||
raise InsufficientPermission()
|
raise InsufficientPermission()
|
||||||
# Compute ratings
|
# Compute ratings
|
||||||
try:
|
try:
|
||||||
rate_computer, price = RateComputer.compute(db_device)
|
rate_computer, price = RateComputer.compute(db_device)
|
||||||
except CannotRate:
|
except CannotRate:
|
||||||
pass
|
pass
|
||||||
else:
|
else:
|
||||||
snapshot.actions.add(rate_computer)
|
snapshot.actions.add(rate_computer)
|
||||||
if price:
|
if price:
|
||||||
snapshot.actions.add(price)
|
snapshot.actions.add(price)
|
||||||
elif snapshot.software == SnapshotSoftware.WorkbenchAndroid:
|
elif snapshot.software == SnapshotSoftware.WorkbenchAndroid:
|
||||||
pass # TODO try except to compute RateMobile
|
pass # TODO try except to compute RateMobile
|
||||||
# Check if HID is null and add Severity:Warning to Snapshot
|
# Check if HID is null and add Severity:Warning to Snapshot
|
||||||
if snapshot.device.hid is None:
|
if snapshot.device.hid is None:
|
||||||
snapshot.severity = Severity.Warning
|
snapshot.severity = Severity.Warning
|
||||||
db.session.add(snapshot)
|
db.session.add(snapshot)
|
||||||
db.session().final_flush()
|
db.session().final_flush()
|
||||||
ret = self.schema.jsonify(snapshot) # transform it back
|
ret = self.schema.jsonify(snapshot) # transform it back
|
||||||
ret.status_code = 201
|
ret.status_code = 201
|
||||||
db.session.commit()
|
db.session.commit()
|
||||||
return ret
|
return ret
|
||||||
|
except Exception as err:
|
||||||
|
save_snapshot_in_file(snapshot_json_bakup)
|
||||||
|
raise err
|
||||||
|
|
||||||
def transfer_ownership(self):
|
def transfer_ownership(self):
|
||||||
"""Perform a InitTransfer action to change author_id of device"""
|
"""Perform a InitTransfer action to change author_id of device"""
|
||||||
|
|
Reference in New Issue