Skip to content
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

Jenkins plugin for cart valgrind #267

Open
wants to merge 27 commits into
base: master
Choose a base branch
from

Conversation

mallove79
Copy link
Contributor

No description provided.

Signed-off-by: Ethan Mallove <[email protected]>
Quick-functional: true
Allow-unstable: true
Skip-func-test-vm-valgrind: false

Signed-off-by: Ethan Mallove <[email protected]>
Add skip_post_script parameter for the case of a non-unit test stage

Signed-off-by: Ethan Mallove <[email protected]>
 * '#' does not a Groovy comment make.
 * Create a File object from target_dir to check existence.

Signed-off-by: Ethan Mallove <[email protected]>
It's either *all* named params or no named params.
    https://www.jenkins.io/doc/pipeline/steps/workflow-durable-task-step/#sh-shell-script

Otherwise Groovy interpreter errors:
    Expected named arguments but got [{returnStatus=true}, ...

Signed-off-by: Ethan Mallove <[email protected]>
Signed-off-by: Ethan Mallove <[email protected]>
…ins_plugin_for_cart_valgrind

Signed-off-by: Ethan Mallove <[email protected]>
…nto jenkins_plugin_for_cart_valgrind

Signed-off-by: Ethan Mallove <[email protected]>
There may exist an issue with the Jenkins Valgrind plugin, which
confuses the reporting of valgrind memcheck xml results from
multiple stages:

    https://issues.jenkins.io/browse/JENKINS-66073?jql=component%3D16538

This is why we currently stash all valgrind reuslts before reporting on them.

Testing whether this issue does in fact exist -- instead of stashing Valgrind
results, run publishValgrind immediately (in the same stage Valgrind was run).

Signed-off-by: Ethan Mallove <[email protected]>
…nto jenkins_plugin_for_cart_valgrind

Signed-off-by: Ethan Mallove <[email protected]>
Prepend trace lines with "TRACE:"

Signed-off-by: Ethan Mallove <[email protected]>
Signed-off-by: Ethan Mallove <[email protected]>
Signed-off-by: Ethan Mallove <[email protected]>
Signed-off-by: Ethan Mallove <[email protected]>
We can simpliy call the stash from the Jenkinsfile, instead of skipping
everything in unitTestPost besides the stash.

Signed-off-by: Ethan Mallove <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants