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

Remove #define of cr_log_info needed for v2.2.2 #1276

Merged
merged 1 commit into from
Mar 16, 2017

Conversation

sungeunchoi
Copy link

After this commit, you will need to use Criterion v2.3.0 or later.

Signed-off-by: Sung-Eun Choi sungeunchoi@users.noreply.github.com

After this commit, you will need to use Criterion v2.3.0 or later.

Signed-off-by: Sung-Eun Choi <sungeunchoi@users.noreply.github.com>
@sungeunchoi sungeunchoi requested review from jswaro and removed request for jswaro March 10, 2017 02:18
@sungeunchoi
Copy link
Author

@chuckfossen or @jswaro

@chuckfossen
Copy link

👍

@hppritcha
Copy link
Member

@sungeunchoi which criterion should the LANL-CI point to now?

@sungeunchoi
Copy link
Author

I've installed v2.3.1 internally.

@hppritcha
Copy link
Member

bot:lanl:retest

@hppritcha
Copy link
Member

should I install v2.2.0 of criterion at LANL? still seems to be failing

@jswaro
Copy link
Member

jswaro commented Mar 14, 2017

After this commit, you will need to use Criterion v2.3.0 or later.

@hppritcha Its needs 2.3.1 because 2.3.0 has a logging issue which causes gnitest to take over an hour to finish.

@hppritcha
Copy link
Member

try again with 2.3.1
bot:lanl:retest

@hppritcha
Copy link
Member

grrr. libcsptr junk...
bot:lanl:retest

@hppritcha
Copy link
Member

now I'm getting agitated
bot:lanl:retest

@hppritcha
Copy link
Member

i'm merging so we will get clean CI runs at NERSC now that its switched over to criterion 2.3.1

@hppritcha hppritcha merged commit 73b7ac5 into ofi-cray:master Mar 16, 2017
@sungeunchoi sungeunchoi deleted the update-criterion-version branch March 16, 2017 21:27
hppritcha pushed a commit to hppritcha/libfabric that referenced this pull request Mar 17, 2017
After this commit, you will need to use Criterion v2.3.0 or later.

upstream merge of ofi-cray#1276

Signed-off-by: Sung-Eun Choi <sungeunchoi@users.noreply.github.com>
(cherry picked from commit ofi-cray/libfabric-cray@4e45815)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants