Remove statements setting bytestream components OutputLevel to DEBUG in
production.

Frank Berghaus authored 4 years ago and Vakhtang Tsulaia committed 4 years ago

The extends template takes care of the onerous tasks of implementing the
functionality required for interactions with the service interface. This
just streamlines the code a bit and ensures the interface accessors are
correctly implemented.

Add docstrings to the bytestream configuration module and the functions
that set up reading from and writing to bytestream. Remove remaining
global data members.

Add docstrings to the bytestream configuration module and the functions
that set up reading from and writing to bytestream. Remove remaining
global data members.

Make the bytestream write configuration compliant with the style found
in the Athena configuration modules. Update the one client. Executing
the configuration now copies 10 events from the default RAW input into a
new bytestream file to test the configuration.

Refactor the read configuration for bytestream in component accumulator
style. Looking at code from the athena configuration team it seems that
we use camelCase for functions and methods. Since that changes the
interface it requires a update to the clients.

Frank Berghaus authored 4 years ago and Walter Lampl committed 4 years ago

The ByteStreamEventStorageInput service provided a legacy job option called FullFileName to specify bytestream input. With the move to EventSelector this option was all but removed from ByteStreamEventInputSvc. The EventSelectorByteStream provieded the input option to specify input. It captured the FullFileName option of the InputSvc. This commit removes the relevant parts of the ByteStreamEventStorageInputSvc and EventSelectorByteStream. It also attempts to move all references in other job options (unless they specify being for an older release).

Updated `ComponentAccumulator` configuration tests which dump the configuration
to a pickle file. In python3 the output file needs to be explicitly opened in binary mode.
I.e. ```f = open('configdump.pkl','wb')```
rather than ```f = open('configdump.pkl','w')```

ProxyProviderSvc is in SGComps, not StoreGate.

Fix warnings/errors flagged by `flake8`. Mostly unused variables/imports
or stylistic code problems.

Former-commit-id: 6b28f423

Former-commit-id: 45675d18

Metadata tools are now private, not public.
Fixes crash in TrigUpgradeTest.



Former-commit-id: dda7ea8d

Former-commit-id: 49b9ded0

Walter Lampl authored 6 years ago and Frank Winklmeier committed 6 years ago

Former-commit-id: 18aec75c

This reverts commit 05908ffa [formerly a8b54d55], reversing
changes made to b2a24b9e [formerly ad7dbb4f].


Former-commit-id: 1da4678a

Fix configuration for IOVDbMetaDataTool and ByteStreamMetaDataTool
changing from public to private tools.



Former-commit-id: 24c83545

Fix configuration for IOVDbMetaDataTool and ByteStreamMetaDataTool
changing from public to private tools.



Former-commit-id: 28b2fe8f

Former-commit-id: 6a791025