Skip to content

Commit

Permalink
Change _all_fields config for normalised records
Browse files Browse the repository at this point in the history
Change _all_fields in the normalised_summary record definition to be
equivalent to _db_fields, rather than _msg_fields. This is necessary as
_all_fields should be a superset of fields used when converting to and
from the Python record object. It doesn't change what gets written out
to a message, but having it set incorrectly meant that unloading
old-style normalised records from a new-style database was causing an
error when the apel message writer came across the ServiceLevelType
field which it didn't expect at all.
  • Loading branch information
tofu-rocketry committed Oct 14, 2024
1 parent ade2e76 commit c041e92
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion apel/db/records/normalised_summary.py
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ def __init__(self):
"NormalisedWallDuration", "NormalisedCpuDuration",
"NumberOfJobs"]
# All allowed fields.
self._all_fields = self._msg_fields
self._all_fields = self._db_fields

def _check_fields(self):
'''
Expand Down

0 comments on commit c041e92

Please sign in to comment.