Add preliminary smart container list, throws a warning not an exception
This is a fix for !25104 (merged), which broke several ART tests.
The problem was that we added a new smart collection for VR track jets, which everyone used to add to AllVariables
. Of course AllVariables
is disallowed for collections with smart collections (at least in some derivations), so adding a smart collection broke several of our bigger derivations.
With this MR, the SlimmerHelper
will check for the offending collection in PreliminarySmartCollections
. If the collection is listed as a "preliminary" smart collection it will just throw a warning.
I think this solution is the least evil of several options:
- I can't validate every derivation to make sure this smart collection works for them. This is something that will have to be up to the derivations clients.
- I don't want to break anyone's derivation, and unfortunately several derivations are broken right now.
- I don't want to keep adding VR track jets to
AllVariables
: most people need very few of the variables we're saving there.
I'm guessing some people would opt for a more draconian approach where we simply leave the derivations broken and leave it up to the derivation contacts to fix them. That sort of decision seems above my pay grade and beyond the timescale where we'd want to have the ART tests fixed (maybe a conversation in the derivation coordination meeting would be appropriate at some point).