Copy of Lb2Lll dir from AP to test too long rendered yamls

See LbAnalysisProductions#41 (closed)

added 1 commit

• 31b5361d - Disable automatically_configure

Compare with previous version

@mmulder, allow me to chime in and make a comment - it is great that you add a test for an issue you've found, but I don't think you need the whole shabang you committed to test the issue ;-). Just a simple job will do, no?

Hi @erodrigu; normally, indeed not. However, in this case, the error was caused after rendering the yaml (which requires the options files to be all there) and seems to come from having a too long yaml file.

So at least @cburr recommended me to place the files as I had them in the repository.

I did remove any irrelevant files > 10k as far as I could see from this folder, in order to not take too much space. But there might still be some remaining files that could be removed (mostly helper files that are not directly part of the options files per job, but are called by them).

OK, you know best. I just wanted to make sure that the point was taken into account, as tests should ideally be just what's needed and no more :).

I agree, thanks for making sure! :)

mentioned in merge request LbAnalysisProductions!31 (merged)

Hi @mmulder and @cburr. The associated issue is closed, see the top of this page, but this MR did not get merged. Is this an oversight? Can you finalise this "thread" as you see appropriate? Advance thanks.

Hi @erodrigu, I am not sure. I think @cburr wanted this code to use for testing even if it is fixed now?

Makes complete sense, of course. Hence my question as this should then be finalised (pipeline failing) and merged :-).

added Testing WP2 - Analysis Productions labels

@cburr, what's your take on this? At this stage one can no longer re-run the pipeline, but that's not necessarily an issue. What's your take on this MR?

@cburr, @mmulder, can we try and converge on this long standing MR please?

Can you please comment, @cburr, @mmulder?

Hi @erodrigu, I have to say that I did not look at this; I presume @cburr solved this as part of the hackathon?

Not sure. I see no related MR or anything, so ... @cburr?

Still looking at what is dormant ... Can you @cburr please comment? We've been in "not sure, to be seen" mode for 7 months now. Do you need any help?

The alternative is that we shamelessly close this. Fingers crossed, then, that the problem is either fixed or won't hit us back.

The problem was fixed a long time ago (LbAnalysisProductions!31 (merged)). This repository only exists to be able to run tests without making a mess in the main AnalysisProductions repository and having an open MR is occasionally useful when playing with the API but we can close this.

closed