-
Notifications
You must be signed in to change notification settings - Fork 33
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
[WIP] Docker in the nightly testing #1033
base: main
Are you sure you want to change the base?
Changes from 13 commits
143f4a2
46b60fb
726fd23
81abd42
0d602d9
bfb7e5e
ede9ec5
b565cd3
69bde86
888c185
b610766
8d27aea
3207e4c
6f9ed9b
c7053bf
ae630ec
a15b792
10ab5cf
cd042a6
84f5db8
5d68917
233a67d
1dc7cca
8fd1ab1
6fc641a
e298464
eef0189
6246020
75afc7b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -33,6 +33,13 @@ include(CMakePackageConfigHelpers) | |
include(GNUInstallDirs) | ||
find_package(PkgConfig) | ||
|
||
find_package(Valgrind REQUIRED) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. FYI, we already have finding valgrind https://github.com/oneapi-src/unified-memory-framework/blob/main/src/utils/CMakeLists.txt#L32 |
||
if(Valgrind_FOUND) | ||
message(STATUS "Found Valgrind!") | ||
else() | ||
message(FATAL_ERROR "Valgrind not found!") | ||
endif() | ||
|
||
# Define a list to store the names of all options | ||
set(UMF_OPTIONS_LIST "") | ||
list(APPEND UMF_OPTIONS_LIST CMAKE_BUILD_TYPE) | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,22 @@ | ||
# Copyright (C) 2025 Intel Corporation | ||
# Under the Apache License v2.0 with LLVM Exceptions. See LICENSE.TXT. | ||
# SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception | ||
|
||
if(NOT VALGRIND_ROOT AND DEFINED ENV{VALGRIND_ROOT}) | ||
set(VALGRIND_ROOT "$ENV{VALGRIND_ROOT}" CACHE PATH "Valgrind base directory location (optional, used for nonstandard installation paths)") | ||
mark_as_advanced(VALGRIND_ROOT) | ||
endif() | ||
|
||
# Search path for nonstandard locations | ||
if(VALGRIND_ROOT) | ||
set(Valgrind_INCLUDE_PATH PATHS "${VALGRIND_ROOT}/include" "${VALGRIND_ROOT}/valgrind/current/usr/include" NO_DEFAULT_PATH) | ||
set(Valgrind_BINARY_PATH PATHS "${VALGRIND_ROOT}/bin" NO_DEFAULT_PATH) | ||
endif() | ||
|
||
find_path(Valgrind_INCLUDE_DIR valgrind/memcheck.h HINTS ${Valgrind_INCLUDE_PATH}) | ||
find_program(Valgrind_EXECUTABLE NAMES valgrind PATH ${Valgrind_BINARY_PATH}) | ||
|
||
include(FindPackageHandleStandardArgs) | ||
find_package_handle_standard_args(Valgrind DEFAULT_MSG Valgrind_INCLUDE_DIR Valgrind_EXECUTABLE) | ||
|
||
mark_as_advanced(Valgrind_INCLUDE_DIR Valgrind_EXECUTABLE) |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -6,15 +6,15 @@ black==24.3.0 | |
# Tests | ||
packaging==24.2 | ||
# Generating HTML documentation | ||
pygments==2.19.1 | ||
sphinxcontrib_applehelp==2.0.0 | ||
sphinxcontrib_devhelp==2.0.0 | ||
sphinxcontrib_htmlhelp==2.1.0 | ||
sphinxcontrib_serializinghtml==2.0.0 | ||
sphinxcontrib_qthelp==2.0.0 | ||
pygments==2.18.0 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. we should rather not downgrade packages, rather move them up |
||
sphinxcontrib_applehelp==1.0.0 | ||
sphinxcontrib_devhelp==1.0.0 | ||
sphinxcontrib_htmlhelp==2.0.1 | ||
sphinxcontrib_serializinghtml==1.1.5 | ||
sphinxcontrib_qthelp==1.0.3 | ||
breathe==4.35.0 | ||
sphinx==8.1.3 | ||
sphinx_book_theme==1.1.3 | ||
sphinx==7.1.2 | ||
sphinx_book_theme==1.0.1 | ||
# Spelling check in documentation | ||
pyenchant==3.2.2 | ||
sphinxcontrib-spelling==8.0.0 |
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.
ok, at the beginning you can run more simple tests (like BasicBuilds or FastBuilds). Let's say Valgrind and Fuzz tests are a little more complicated and can be added separately.
Also, you don't have to update existing jobs to use dockers, as discussed - for the starters we can add one extra job running on dockers in nightly build