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

OFED builder workflow #1132

Merged
merged 28 commits into from
Sep 27, 2024
Merged

OFED builder workflow #1132

merged 28 commits into from
Sep 27, 2024

Conversation

assumptionsandg
Copy link
Contributor

No description provided.

@assumptionsandg assumptionsandg self-assigned this Jul 4, 2024
@assumptionsandg assumptionsandg marked this pull request as ready for review July 9, 2024 16:10
@assumptionsandg assumptionsandg requested a review from a team as a code owner July 9, 2024 16:10
@assumptionsandg assumptionsandg changed the title WIP: OFED builder workflow OFED builder workflow Jul 9, 2024
markgoddard
markgoddard suggested changes Jul 23, 2024
View reviewed changes
Copy link

@markgoddard markgoddard left a comment

Choose a reason for hiding this comment

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

Thanks @assumptionsandg this is a nice start.

.github/workflows/package-build-ofed.yml Outdated Show resolved Hide resolved
.github/workflows/package-build-ofed.yml Show resolved Hide resolved
etc/kayobe/ansible/build-ofed.yml Outdated Show resolved Hide resolved
etc/kayobe/ansible/build-ofed.yml Outdated Show resolved Hide resolved
etc/kayobe/ansible/build-ofed.yml Outdated Show resolved Hide resolved
etc/kayobe/pulp.yml Outdated Show resolved Hide resolved
etc/kayobe/pulp.yml Outdated Show resolved Hide resolved
etc/kayobe/pulp.yml Outdated Show resolved Hide resolved
etc/kayobe/pulp.yml Outdated Show resolved Hide resolved
.github/workflows/package-build-ofed.yml Outdated Show resolved Hide resolved
@assumptionsandg
Push OFED userspace packages
764a1c2
@assumptionsandg
Fix build
78aed99
@assumptionsandg
Fix kernel upgrade
95c4e9f
@assumptionsandg
Replace MLNX with DOCA
2b30dff
@assumptionsandg
Adjust lv_var_tmp to 2G
0ddfcac
@assumptionsandg
Fix workflow
395e9cc
@assumptionsandg
Disable gpg check for doca host
b902590
@assumptionsandg
Fix inputs in workflow
35d8345
@assumptionsandg
Replace with_fileglob
a135c1b
@assumptionsandg
Remove trailing slash in base_path
9d628e8
@assumptionsandg
Install kernel modules
cbb7a1b
@assumptionsandg
Re-add the vault password
1d14550
@assumptionsandg
Remove trailing dash from distribution
c5e7657
@assumptionsandg
Remove LVM configuration
b62f57d 
LVM config is already applied for the seed in the ci-builder
environment, and this should be enough for building OFED.
@assumptionsandg
Use reset-bls-entries playbook in OFED workflow
29bda85
@assumptionsandg
Use replace instead of lineinfile
5a9126d
@assumptionsandg
Remove sed magic
699f244
@assumptionsandg
Move OFED repositories to ofed.yml
022a7ea
@assumptionsandg
Rename build-ofed to build-ofed-rocky
1840b05
@assumptionsandg
Add precheck for noexec
c61deb2
@assumptionsandg
Update workflow
d989478
@assumptionsandg
WIP: OFED documentation
2847ad9
@assumptionsandg
Fix no eol
1c44353
@assumptionsandg
Add a release note
3490003
@assumptionsandg
Add to docs tree
5eaf53c
Alex-Welsh
Alex-Welsh approved these changes Sep 27, 2024
View reviewed changes
Copy link
Contributor

@Alex-Welsh Alex-Welsh left a comment

Choose a reason for hiding this comment

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

Lets not hold this up any longer. There's plenty of improvements we could make but we shouldn't try to start with perfection.

Time to test in prod!

doc/source/contributor/ofed.rst Show resolved Hide resolved
@Alex-Welsh Alex-Welsh enabled auto-merge (squash) September 27, 2024 14:25
@Alex-Welsh Alex-Welsh merged commit d25970b into stackhpc/2024.1 Sep 27, 2024
12 checks passed
@Alex-Welsh Alex-Welsh deleted the ofed branch September 27, 2024 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@markgoddard markgoddard markgoddard requested changes

@Alex-Welsh Alex-Welsh Alex-Welsh approved these changes

Assignees

@assumptionsandg assumptionsandg

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@assumptionsandg @Alex-Welsh @markgoddard