Skip to content

Translate Lua modulefiles to Tcl and define default version with .version file - #2680

Open
xdelaruelle wants to merge 2 commits into
openhpc:4.xfrom
xdelaruelle:tcl_modulefiles
Open

Translate Lua modulefiles to Tcl and define default version with .version file#2680
xdelaruelle wants to merge 2 commits into
openhpc:4.xfrom
xdelaruelle:tcl_modulefiles

Conversation

@xdelaruelle

Copy link
Copy Markdown
Contributor

Translate the 5 existing Lua modulefiles into Tcl to harmonize all OpenHPC modulefiles in this language.

Change the 6 existing default symlink into Tcl .version modulerc files to harmonize the way module default version are defined in all OpenHPC packages.

This change is part of the preparatory work for the addition of Environment Modules (#2470)

@adrianreber

Copy link
Copy Markdown
Member

Thanks. I think this might be one of the first PR I will be looking at and testing.

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown

📊 Build Order Analysis Results

Environment: AlmaLinux 10 Container
Total Spec Files Analyzed: 89
Status:Build order analysis completed successfully

📋 Build Order Output

📦 Build Order (84 packages)

➕ Show remaining 69 packages

Analysis performed by OpenHPC Build Order CI
Commit: ad0ebce3
Workflow: 34456093898

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown

📦 Package Count Analysis Results

Environment: UBI 10 Container
Target Branch: 4.x
Version: OpenHPC 4.x
Overall Status:Analysis completed successfully

🏭 Factory Repositories

Status:Factory repositories analysis completed successfully

Release Base OS aarch64 x86_64 noarch Total
4.2 EL_10 71 112 12 195
4.2 openEuler_24.03 66 73 12 151
Total All 137 185 24 346

Analysis performed by OpenHPC Package Count CI
Commit: ad0ebce3
Workflow: 34456094099

@adrianreber

Copy link
Copy Markdown
Member

This probably needs to be split in smaller PRs. The GCC rebuild usually takes almost as long as GitHub Actions allows it. So it probably will not finish.

@xdelaruelle

Copy link
Copy Markdown
Contributor Author

This probably needs to be split in smaller PRs. The GCC rebuild usually takes almost as long as GitHub Actions allows it. So it probably will not finish.

If I do a specific PR for GCC and another for the other packages, do you think it will be ok?

@adrianreber

Copy link
Copy Markdown
Member

This probably needs to be split in smaller PRs. The GCC rebuild usually takes almost as long as GitHub Actions allows it. So it probably will not finish.

If I do a specific PR for GCC and another for the other packages, do you think it will be ok?

Yes, that should work. Only trilinos would be a problem as it takes forever to build.

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown

🚀 CCache Statistics

Configuration 🐧 RHEL 🦊 openEuler
🖥️ x86_64 / gnu15 🟡 36.46% 🔴 3.59%
🖥️ x86_64 / intel 🟡 31.24% -
💪 aarch64 / gnu15 🔴 3.68% 🔴 3.61%
📊 Detailed Statistics

openEuler (aarch64)

Cacheable calls:    4463 / 6196 (72.03%)
  Hits:              161 / 4463 ( 3.61%)
    Direct:          133 /  161 (82.61%)
    Preprocessed:     28 /  161 (17.39%)
  Misses:           4302 / 4463 (96.39%)
Uncacheable calls:  1733 / 6196 (27.97%)
Local storage:
  Cache size (GiB):  0.4 /  5.0 ( 8.14%)
  Hits:              161 / 4463 ( 3.61%)
  Misses:           4302 / 4463 (96.39%)

openEuler (x86_64)

Cacheable calls:    4649 / 6518 (71.33%)
  Hits:              167 / 4649 ( 3.59%)
    Direct:          139 /  167 (83.23%)
    Preprocessed:     28 /  167 (16.77%)
  Misses:           4482 / 4649 (96.41%)
Uncacheable calls:  1869 / 6518 (28.67%)
Local storage:
  Cache size (GiB):  0.4 /  5.0 ( 8.70%)
  Hits:              167 / 4649 ( 3.59%)
  Misses:           4482 / 4649 (96.41%)

RHEL (aarch64/gnu15)

Cacheable calls:    4242 / 5790 (73.26%)
  Hits:              156 / 4242 ( 3.68%)
    Direct:           86 /  156 (55.13%)
    Preprocessed:     70 /  156 (44.87%)
  Misses:           4086 / 4242 (96.32%)
Uncacheable calls:  1548 / 5790 (26.74%)
Local storage:
  Cache size (GiB):  0.4 /  5.0 ( 8.15%)
  Hits:              156 / 4242 ( 3.68%)
  Misses:           4086 / 4242 (96.32%)

RHEL (x86_64/gnu15)

Cacheable calls:    12137 / 19587 (61.96%)
  Hits:              4425 / 12137 (36.46%)
    Direct:          3362 /  4425 (75.98%)
    Preprocessed:    1063 /  4425 (24.02%)
  Misses:            7712 / 12137 (63.54%)
Uncacheable calls:   7450 / 19587 (38.04%)
Local storage:
  Cache size (GiB):   0.6 /   5.0 (12.81%)
  Hits:              4425 / 12137 (36.46%)
  Misses:            7712 / 12137 (63.54%)

RHEL (x86_64/intel)

Cacheable calls:    15312 / 30634 (49.98%)
  Hits:              4783 / 15312 (31.24%)
    Direct:          3556 /  4783 (74.35%)
    Preprocessed:    1227 /  4783 (25.65%)
  Misses:           10529 / 15312 (68.76%)
Uncacheable calls:  15322 / 30634 (50.02%)
Local storage:
  Cache size (GiB):   0.6 /   5.0 (11.98%)
  Hits:              4783 / 15312 (31.24%)
  Misses:           10529 / 15312 (68.76%)

🤖 Generated from workflow run 34456093732

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown

Test Results

108 files  + 75  108 suites  +75   7m 18s ⏱️ + 6m 30s
309 tests +238  291 ✅ +226  18 💤 +12  0 ❌ ±0 
534 runs  +408  508 ✅ +390  26 💤 +18  0 ❌ ±0 

Results for commit af5b573. ± Comparison against base commit d966218.

♻️ This comment has been updated with latest results.

@xdelaruelle

Copy link
Copy Markdown
Contributor Author

This probably needs to be split in smaller PRs. The GCC rebuild usually takes almost as long as GitHub Actions allows it. So it probably will not finish.

If I do a specific PR for GCC and another for the other packages, do you think it will be ok?

Yes, that should work. Only trilinos would be a problem as it takes forever to build.

It seems it was able to finish. So is it still worth to split in 2 PRs?

openEuler job seems to fail on something else. Should I patch something?

@xdelaruelle

Copy link
Copy Markdown
Contributor Author

openEuler job seems to fail on something else. Should I patch something?

I propose #2692 to fix the openEuler build issue with plasma. Rebased this PR on the other

Translate the 5 existing Lua modulefiles into Tcl to harmonize all
OpenHPC modulefiles in this language.

This change is part of the preparatory work for the addition of
Environment Modules (openhpc#2470)

Signed-off-by: Xavier Delaruelle <xavier.delaruelle@cea.fr>
Change the 6 existing "default" symlink into Tcl ".version" modulerc
files to harmonize the way module default version are defined in all
OpenHPC packages.

This change is part of the preparatory work for the addition of
Environment Modules (openhpc#2470)

Signed-off-by: Xavier Delaruelle <xavier.delaruelle@cea.fr>
@adrianreber

Copy link
Copy Markdown
Member

So, CI is happy, good. I would like to have one more discussion in the TSC before starting to merge your PRs. The next meeting is probably on 2026-09-16. From my side it looks ready.

We already agreed in the TSC that it would be a good addition for OpenHPC but let's have one more round in the TSC next week.

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.

2 participants