Skip to content

[ci] Add Ubuntu 24.10 Build#16667

Closed
dpiparo wants to merge 1 commit intoroot-project:masterfrom
dpiparo:addUbu24_v2
Closed

[ci] Add Ubuntu 24.10 Build#16667
dpiparo wants to merge 1 commit intoroot-project:masterfrom
dpiparo:addUbu24_v2

Conversation

@dpiparo
Copy link
Copy Markdown
Member

@dpiparo dpiparo commented Oct 12, 2024

Build clad with one core only.

@dpiparo dpiparo self-assigned this Oct 12, 2024
@dpiparo dpiparo requested a review from bellenot as a code owner October 12, 2024 12:32
@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 12, 2024

Test Results

    17 files      17 suites   3d 9h 36m 16s ⏱️
 2 661 tests  2 659 ✅ 0 💤 2 ❌
43 513 runs  43 511 ✅ 0 💤 2 ❌

For more details on these failures, see this check.

Results for commit 5136747.

♻️ This comment has been updated with latest results.

${_clad_extra_cmake_args}
BUILD_COMMAND ${CMAKE_COMMAND} --build . ${EXTRA_BUILD_ARGS}
INSTALL_COMMAND ${CMAKE_COMMAND} --build . ${EXTRA_BUILD_ARGS} --target install
BUILD_COMMAND ${CMAKE_COMMAND} --build . ${EXTRA_BUILD_ARGS} -j 1
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Consider adding a comment why '-j 1' is needed (is it a temporary workaround until something is fixed in clad or is it a limitation of the platform and/or MV configuration?).

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I agree. I wanted to test this at scale before documenting further. @vgvassilev any thought about why this is now necessary?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I am not sure I have seen the actual build failure to be able to comment. Can you give me a link to the failure?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Just remove the "-j1"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I do not have access to ubu24 system.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

but you have the line to reproduce the failure locally with the container, right?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@dpiparo dpiparo changed the title [ci] Add ubu24 Build [ci] Add Ubuntu 24.10 Build Oct 13, 2024
@dpiparo dpiparo closed this Oct 16, 2024
@dpiparo dpiparo reopened this Oct 16, 2024
@guitargeek
Copy link
Copy Markdown
Contributor

I just can't reproduce and debug these failues on a clean ubuntu 24.10 install... I don't think we can make progress on this in the next 10 day :(

Would it make sense maybe to add it to the CI with pyroot=OFF? At least we get some coverage then (and also test a new point in the configuration phase space).

@vepadulano
Copy link
Copy Markdown
Member

This PR is fixing the ubuntu 24.10 failures #16837 . We can introduce the new CI build directly with that.

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.

6 participants