-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[CI] Fix archive warnings #18462
[CI] Fix archive warnings #18462
Conversation
❕ Build Aborted
Expand to view the summary
Build stats
Test stats 🧪
Steps errorsExpand to view the steps failures
Log outputExpand to view the last 100 lines of log output
|
Jenkinsfile
Outdated
// For such, the directory is the one when using the target -C <directory> | ||
// Otherwise, it will use the ** glob pattern | ||
def directory = '**' | ||
if (target.contains('-C ')) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Target format
flags -C folder/subfolder barFoo
^---^ ^-----------------^ ^----^
flags Directory target
For instance:
target: "-C metricbeat crosscompile"
Directory=metricbeat
Target=crosscompile
target: "STRESS_TEST_OPTIONS='-timeout=20m -race -v -parallel 1' -C libbeat stress-tests"
Directory=libbeat
Target=stress-tests
target: "-C generator/_templates/beat test"
Directory=generator/_templates/beat
Target=test
…w CI Worker as the workspace will be empty
I'm a bit skeptical with this approach :( Just thinking if we could add a wrapper to the make/mage to collect the tests to the a temporary folder in the root folder .... @kuisathaverat , what do you think? This proposal requires to change the signature and pass the folder where the project runs, aka to parse |
it is too complicated to avoid an issue you can workaround by changing the catchError
|
What does this PR do?
Use folder of the module to filter what files should be archived and junit shown.
Why is it important?
To avoid issues when reaching the search of over 10000 files. See JENKINS-51913