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.

Sign up for GitHub

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

Jump to bottom

Valgrind testing on master merge times out #428

Open
dschwen opened this issue Mar 23, 2020 · 2 comments
Open

Valgrind testing on master merge times out #428

dschwen opened this issue Mar 23, 2020 · 2 comments

Comments

@dschwen
Copy link
Member

dschwen commented Mar 23, 2020

https://civet.inl.gov/branch/422/

We currently do not have a clean master merge for Magpie. The rests in question (DPA stuff) should probably be marked "heavy" or simplified.

Ping @snschune
@snschune
Copy link
Contributor

Heavy is fine as long as it's tested for master merge. Is it?

@dschwen
Copy link
Member Author

dschwen commented Mar 24, 2020

Hm, maybe increasing the timeout then or simplifying the tests.

snschune pushed a commit to snschune/magpie that referenced this issue Mar 24, 2020
Simplify two tests to avoid valgrind timeouts (idaholab#428)
5d29dff
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dschwen @snschune