Bug #20801
Updated by Brett Smith 10 months ago
@discountConfiguredRAMPercent@ was added to a-d-c in #12199, commit:80da19707253af74bd78c374bfcab64b04d2dbde. The commit message says "Derate advertised node sizes by 5%." What it actually does is _increase_ a container request's RAM requirement by 5% after @ReserveExtraRAM@ and local keepstore requiprements have been rolled in. 5%. Neither seems like what we want: while it's true Crunch won't be able to use all the RAM on the system, the amount it can't use seems likely to be mostly constant based on what else runs from the compute node image rather than a percentage of node size. We already have a configuration knob, @Containers.ReserveExtraRAM@, that lets the administrator tell Crunch that amount. I think we should rely on that exclusively for the goal we're trying to accomplish here, and eliminate @discountConfiguredRAMPercent@ as a source of difficult-to-diagnose "why didn't my container request get the node size I expected" problems.