Skip to content
This repository was archived by the owner on Dec 2, 2021. It is now read-only.

CART-XXXX DO NOT REVIEW. with br#383

Open
sylviachanoiyee wants to merge 1 commit intomasterfrom
schan_verbs_br
Open

CART-XXXX DO NOT REVIEW. with br#383
sylviachanoiyee wants to merge 1 commit intomasterfrom
schan_verbs_br

Conversation

@sylviachanoiyee
Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Sylvia Oi Yee Chan <sylvia.oi.yee.chan@intel.com>
Copy link
Copy Markdown
Collaborator

@daosbuild1 daosbuild1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style warning(s) for job https://build.hpdd.intel.com/job/daos-stack/job/cart/job/PR-383/1/
Please review https://wiki.hpdd.intel.com/display/DC/Coding+Rules

FYI: Errors found in lines not modified in the patch:

multi-node-test.sh:224:
(lint) Double quote to prevent globbing and word splitting. [SC2086]

multi-node-test.sh:221:
(lint) Double quote to prevent globbing and word splitting. [SC2086]

Comment thread multi-node-test.sh
scp -i ci_key -r jenkins@${nodes[0]}:"$LOGDIR" \
install/Linux/TESTING/avocado/job-results

sed -i '/DEBUG|/ s/$/ <br\/>/' install/Linux/TESTING/avocado/job-results/*/*/results.xml
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(style) line over 80 characters

@daosbuild1
Copy link
Copy Markdown
Collaborator

@daosbuild1
Copy link
Copy Markdown
Collaborator

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants