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

Frontier: Post-maintenance updates. #2915

Merged
merged 3 commits into from
Jul 20, 2024
Merged

Frontier: Post-maintenance updates. #2915

merged 3 commits into from
Jul 20, 2024

Conversation

dqwu
Copy link
Contributor

@dqwu dqwu commented Jul 19, 2024

A quick fix to build scream on Frontier after recent maintenance
before we find better solutions.

dqwu added 3 commits July 19, 2024 17:10
@dqwu
Add ums and ums002 to load libunwind 1.6.2
155ec3c 
After recent Frontier maintenance, ums/default and ums002/default
are required to load libunwind/1.6.2.
@dqwu
Add <cstdint> to scream_scorpio_types.hpp to use std::int64_t
555aa19 
This fixes a known build error after recent Frontier maintenance:
no type named 'int64_t' in namespace 'std'; did you mean simply
'int64_t'?
@dqwu
Explicitly add gcc/12.2.0 lib path to CMAKE_EXE_LINKER_FLAGS
805ab1f 
This is a simple workaround to fix scream linking errors after
recent Frontier maintenance before we find better solutions.
@dqwu dqwu added cmake Concerns CMake related files or logic Machine File Frontier labels Jul 19, 2024
@dqwu dqwu requested review from ambrad and jgfouca July 19, 2024 22:31
@E3SM-Bot
Copy link
Collaborator

Status Flag 'Pull Request AutoTester' - Testing Jenkins Projects:

Pull Request Auto Testing STARTING (click to expand)

Build Information

Test Name: SCREAM_PullRequest_Autotester_Mappy

  • Build Num: 5674
  • Status: STARTED

Jenkins Parameters

Parameter Name Value
PR_LABELS cmake;Machine File;Frontier
PULLREQUESTNUM 2915
SCREAM_SOURCE_REPO https://github.com/E3SM-Project/scream
SCREAM_SOURCE_SHA 805ab1f
SCREAM_TARGET_BRANCH master
SCREAM_TARGET_REPO https://github.com/E3SM-Project/scream
SCREAM_TARGET_SHA e587a75
TEST_REPO_ALIAS SCREAM

Build Information

Test Name: SCREAM_PullRequest_Autotester_Weaver

  • Build Num: 5915
  • Status: STARTED

Jenkins Parameters

Parameter Name Value
PR_LABELS cmake;Machine File;Frontier
PULLREQUESTNUM 2915
SCREAM_SOURCE_REPO https://github.com/E3SM-Project/scream
SCREAM_SOURCE_SHA 805ab1f
SCREAM_TARGET_BRANCH master
SCREAM_TARGET_REPO https://github.com/E3SM-Project/scream
SCREAM_TARGET_SHA e587a75
TEST_REPO_ALIAS SCREAM

Using Repos:

Repo: SCREAM (E3SM-Project/scream)
  • Branch: dqwu/frontier-071924
  • SHA: 805ab1f
  • Mode: TEST_REPO

Pull Request Author: dqwu

@E3SM-Bot
Copy link
Collaborator

Status Flag 'Pull Request AutoTester' - Jenkins Testing: all Jobs PASSED

Pull Request Auto Testing has PASSED (click to expand)

Build Information

Test Name: SCREAM_PullRequest_Autotester_Mappy

  • Build Num: 5674
  • Status: PASSED

Jenkins Parameters

Parameter Name Value
PR_LABELS cmake;Machine File;Frontier
PULLREQUESTNUM 2915
SCREAM_SOURCE_REPO https://github.com/E3SM-Project/scream
SCREAM_SOURCE_SHA 805ab1f
SCREAM_TARGET_BRANCH master
SCREAM_TARGET_REPO https://github.com/E3SM-Project/scream
SCREAM_TARGET_SHA e587a75
TEST_REPO_ALIAS SCREAM

Build Information

Test Name: SCREAM_PullRequest_Autotester_Weaver

  • Build Num: 5915
  • Status: PASSED

Jenkins Parameters

Parameter Name Value
PR_LABELS cmake;Machine File;Frontier
PULLREQUESTNUM 2915
SCREAM_SOURCE_REPO https://github.com/E3SM-Project/scream
SCREAM_SOURCE_SHA 805ab1f
SCREAM_TARGET_BRANCH master
SCREAM_TARGET_REPO https://github.com/E3SM-Project/scream
SCREAM_TARGET_SHA e587a75
TEST_REPO_ALIAS SCREAM

@E3SM-Bot
Copy link
Collaborator

Status Flag 'Pull Request AutoTester' - AutoMerge IS ENABLED, but the Label AT: AUTOMERGE is not set. Either set Label AT: AUTOMERGE or manually merge the PR...

@ambrad ambrad merged commit 02dab76 into master Jul 20, 2024
13 checks passed
@ambrad ambrad deleted the dqwu/frontier-071924 branch July 20, 2024 00:29
This was referenced Jul 23, 2024
Closed
Merged
dqwu added a commit that referenced this pull request Jul 30, 2024
@dqwu
Merge pull request #2923 from E3SM-Project/dqwu/frontier-072624
5e7b019 
This PR is a follow-up to PR #2915, which optionally runs hipInit
prior to MPI_Init to prevent occasional segmentation faults during
MPI_Init (see OLCFDEV-1655).
brhillman pushed a commit that referenced this pull request Jul 31, 2024
@ambrad @brhillman
Merge pull request #2915 from E3SM-Project/dqwu/frontier-071924
5434820 
Frontier: Post-maintenance updates.
brhillman pushed a commit that referenced this pull request Jul 31, 2024
@dqwu @brhillman
Merge pull request #2923 from E3SM-Project/dqwu/frontier-072624
6145f82 
This PR is a follow-up to PR #2915, which optionally runs hipInit
prior to MPI_Init to prevent occasional segmentation faults during
MPI_Init (see OLCFDEV-1655).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ambrad ambrad ambrad approved these changes

@jgfouca jgfouca Awaiting requested review from jgfouca

Assignees
No one assigned
Labels
cmake Concerns CMake related files or logic Frontier Machine File
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dqwu @E3SM-Bot @ambrad