Skip to content

Conversation

@opensource-krzysztof
Copy link
Contributor

SLES15 SUPPORT

@opensource-krzysztof opensource-krzysztof requested a review from a team as a code owner December 17, 2024 12:57
name: "${{matrix.os}}, ${{matrix.build_type}}, shared=${{matrix.shared_library}}"
# run only on upstream; forks will not have the HW
if: github.repository == 'oneapi-src/unified-memory-framework'
#if: github.repository == 'oneapi-src/unified-memory-framework'
Copy link
Contributor

Choose a reason for hiding this comment

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

I suppose it should not be commented out

Copy link
Contributor

Choose a reason for hiding this comment

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

Why have you commented it out?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok, commented for testing purpose

Copy link
Contributor

Choose a reason for hiding this comment

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

Please do not resolve issues on your own. The issue is always resolved by the submitter.

Copy link
Contributor

Choose a reason for hiding this comment

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

ok, commented for testing purpose

If it is just for testing then you should:

  1. convert this PR to draft (since it is not read yet)
  2. use a temporary patch like ldorau@a590c27 on my DEBUG_CI_run_only_multi_numa branch to test only one CI job that you are changing and do not occupy other resources, when you do not need them (the resources in public UMF CI are very limited).

Comment on lines +63 to +69
- name: Run tests (on SLES15)
if: matrix.os == 'sles15'
working-directory: ${{github.workspace}}/build
run: |
ctest --output-on-failure --test-dir test -E "umf-provider_os_memory_multiple_numa_nodes"
./test/umf_test-provider_os_memory_multiple_numa_nodes \
--gtest_filter="-*checkModeLocal/*:*checkModePreferredEmptyNodeset/*:testNuma.checkModeInterleave"
Copy link
Contributor

Choose a reason for hiding this comment

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

I cannot see any difference between 'rhel-9.1' and 'sles15', so why there are two separate cases?

Copy link
Contributor

@ldorau ldorau left a comment

Choose a reason for hiding this comment

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

Please squash 3 commits:
image
into one and add an appropriate description like "Add SLES15 to multi-NUMA CI"


- name: Run tests
if: matrix.os != 'rhel-9.1'
if: (matrix.os != 'rhel-9.1') && (matrix.os != 'sles15')
Copy link
Contributor

Choose a reason for hiding this comment

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

I would run all tests on SLES - did it not work? can you send some logs with failing SLES on all tests?

name: "${{matrix.os}}, ${{matrix.build_type}}, shared=${{matrix.shared_library}}"
# run only on upstream; forks will not have the HW
if: github.repository == 'oneapi-src/unified-memory-framework'
#if: github.repository == 'oneapi-src/unified-memory-framework'
Copy link
Contributor

Choose a reason for hiding this comment

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

when done testing, pls un-comment this

@ldorau ldorau marked this pull request as draft December 17, 2024 14:19
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.

4 participants