search

cornelisnetworks / opa-ff

Other
11 stars 20 forks source link

Update to latest from build 10.12.1.0.6 (IFS 10.12.1.0.7) #35

Closed gregkresge-cornelisnetworks closed 1 year ago

gregkresge-cornelisnetworks commented 1 year ago

Successfully built manually on RHEL9 (bld-node-42)

Repo release and tag will be added after merging to master.

gregkresge-cornelisnetworks commented 1 year ago

@davidgonzalez-cornelisnetworks Slightly concerned why we are seeing all of these branding reversals. Were these never cherry-picked over to opa-dev/opa10_12 (seeing these "Intel" references in ifs-all)? Or am I forgetting that this was something that was separately, manually done on the public repos? Either way, I'll get these corrected. Good catches.

gregkresge-cornelisnetworks commented 1 year ago

Looks like this patch was applied to opa-10_11, but never cherry-picked to opa-dev

commit dd7a846049c439bad536ef3bb5942524dac57cf9 Author: Scott Breyer scott.breyer@cornelisnetworks.com Date: Thu Oct 28 11:02:45 2021 -0400

STL-62361 Rebranding for OpenIB Install

Reviewed-by: Gonzalez, David david.gonzalez@cornelisnetworks.com
Reviewed-by: Kresge, Greg greg.kresge@cornelisnetworks.com
Signed-off-by: Breyer, Scott scott.breyer@cornelisnetworks.com
(cherry picked from commit 98513213c5138ac35b9dee9455985a401b21b8ef)
gregkresge-cornelisnetworks commented 1 year ago

Amended commit to not revert rebranding. While I'll get a separate patch staged on opa-dev to address this delta, I think that should cover the misses in this PR.