[v1.22.x] contrib/intel/jenkins: Cleanup jenkinsfile to match upstream #10815
+137
−306
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Zach Dworkin <zachary.dworkin@intel.com>
Remove this unused function. It was used for legacy code and is no longer needed. Signed-off-by: Zach Dworkin <zachary.dworkin@intel.com>
This unused function was for legacy code. It is no longer needed Signed-off-by: Zach Dworkin <zachary.dworkin@intel.com>
Replace old build function with inline shell command. This is a legacy function and is no longer needed. Signed-off-by: Zach Dworkin <zachary.dworkin@intel.com>
Remove this legacy function and instead put it in the few stages that require it. Signed-off-by: Zach Dworkin <zachary.dworkin@intel.com>
Remove old slurm functions and instead use it in the one stage it applies to. Signed-off-by: Zach Dworkin <zachary.dworkin@intel.com>
All spaces in stage names should be "_" Only migrated test stages will be named CI_ to match output locations Signed-off-by: Zach Dworkin <zachary.dworkin@intel.com>
Remove legacy related things that are not in use anymore Reorganize top level functions to make replaying easier Remove unused variables Fix weird if statements to be inline ? statements Signed-off-by: Zach Dworkin <zachary.dworkin@intel.com>
Issue ofiwg#10123 is resolved and this test can be re-enabled. Signed-off-by: Zach Dworkin <zachary.dworkin@intel.com>
aingerson
previously approved these changes
Feb 22, 2025
multinode stage needs to run in CI_LOCATION instead of RUN_LOCATION to use the right script. Signed-off-by: Zach Dworkin <zachary.dworkin@intel.com>
ubertest implementation currently requires FI_RMA_EVENT when using RMA and counters This will cause tcp to return ENODATA for these combinations and cause runfabtests to fail. This should get updated in ubertest to not require it but remove testing for now Signed-off-by: Alexia Ingerson <alexia.ingerson@intel.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.