Scott Snyder authored 5 months ago and Duc Ta committed 5 months ago

AthenaConfiguration: Fix CA caching issue.

We can use AccumulatorCache to memoize results of functions returning a CA.
The memoized results are looked up by a hash of the arguments.

However, the hash of a set of configuration flags is based on the object id
(since only locked flags can be hashed).  But this can fail if flag
objects are deleted and recreated, since then we can have flags objects
with different values but the same hash.

When we hash a set of flags, save a reference to it, so that the address
won't be reused.  Shouldn't significantly affect typical configurations
which don't reuse flags objects, but some unit tests do that.

See ATLASRECTS-8070.

Will Buttinger authored 8 months ago and Duc Ta committed 8 months ago

Update AthConfigFlags.py - don't force outputlevel to info if user didn't specify it

Frank Winklmeier authored 11 months ago and Duc Ta committed 11 months ago

athena.py: implement native support for CA

Merge the functionality of the `ThinCAWrapper.sh` and `CARunner.py` into
the main `athena` executable. Reduces the amount of shell scripting and
unifies the user interface.

Also re-use the AthOptionsParser for the CA-case.

Merge the functionality of the `ThinCAWrapper.sh` and `CARunner.py` into
the main `athena` executable. Reduces the amount of shell scripting and
unifies the user interface.

Also re-use the AthOptionsParser for the CA-case.

Make `fillFromArgs` also work for non-standard parsers, i.e. parsers
that do not support all the "athena-like" command line arguments.

Rename the `--pmon` command line option to `--perfmon` to follow the
athena and transform naming. Also select "fastmonmt" as the default MODE
and make the MODE an optional argument.

Edward Moyse authored 1 year ago and Frank Winklmeier committed 1 year ago

AthConfigFlags: restrict --filesInput to comma-separated list

See merge request !70084

(cherry picked from commit f3955109)

6bfd5c79 AthConfigFlags: restrict --filesInput to comma-separated list

Refactor the code needed to dump python profiles into the `Debugging`
module and re-use it in legacy athena and CA. The dump of the temporary
profile for `--drop-and-reload` is no longer needed either if we dump
the profile before the reload (and the second part of the job has no
interesting profile data anyway).

Currently we support the following syntax for `--filesInputs`:
```
   --filesInput file1 file2
   --filesInput file1,file2
```
Change this to only support the second version as was the case in legacy
athena. This also simplifies the command line parsing and avoids
swallowing flags.

Teng Jian Khoo authored 1 year ago and Marija Vranjes Milosavljevic committed 1 year ago

ATR-28968: Reduce INFO messages from central trigger configuration code

Dan Guest authored 1 year ago and Melissa Yexley committed 1 year ago

This reverts commit fb89bdec.

Frank Winklmeier authored 1 year ago and Melissa Yexley committed 1 year ago

Remove the global `ConfigFlags` instance and the safe-guards to delete
the instance manually from the various skeletons and drivers.

This reverts commit fb89bdec.

Remove the global `ConfigFlags` instance and the safe-guards to delete
the instance manually from the various skeletons and drivers.

Dan Guest authored 1 year ago and Melissa Yexley committed 1 year ago

Revert "remove deprecation on __call__, again"

This reverts commit 1b3d761e.