Skip to content

Maintenance changes and updates #115

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
Open
kimpaller wants to merge 115 commits into
base: master
Choose a base branch
from
Open

Maintenance changes and updates #115

kimpaller wants to merge 115 commits into from

Conversation

@kimpaller
Copy link
Collaborator

@kimpaller kimpaller commented Oct 17, 2025 •
edited
Loading

📦 PR #115 Summary: Jenkins Shared Library Maintenance Merge

This pull request merges over 100 commits from the adgt-test-harness branch into master. These updates have been tested in staging and are actively used in (ADGT) daily test harness operations.

🔧 Highlights

✅ Branch Merge

  • Consolidates 113 commits from adgt-test-harness into master
  • Reflects long-term staging usage and stability

🚀 No-OS Pipeline Enhancements

  • Refined stage flow and platform variable handling
  • Improved ELF file filtering and Xilinx naming conventions
  • Added fallback for missing system_top.xsa (dummy creation)
  • Enhanced logging and archiving behavior

🧪 Test Harness Improvements

  • Customized baudrate handling for pyadi-iio examples
  • Adjusted boot wait times and stage skipping logic
  • Improved verbosity marker handling and exception filtering

📁 SCM and Checkout Logic

  • Refactored libiio checkout to support branches/tags
  • Fixed multibranch pipeline SCM behavior for PRs
  • Added repo clone checks and branch format validation

🛠️ Dependency and Agent Setup

  • Added toggles for optional agent dependency installation
  • Defaulted to updating requirements during agent setup
  • Added support for disabling NetBox via CLI flags

🧹 Miscellaneous Fixes

  • Corrected typos and syntax issues
  • Updated default Docker image to Ubuntu 22.04
  • Improved error handling and command failure checks

This PR represents a comprehensive cleanup of the shared library, focusing on robustness, configurability, and smoother CI/CD integration.

tagoylo and others added 30 commits October 28, 2024 09:25
@tagoylo
no-os: update no-os stage flow
4bf8fa6 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
no-os: fix variable used for loading non-xilinx platforms
935d77c 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
no-os: define platform
525a496 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
no-os:update no_os_project var
0ab8a81 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
no-os:update no_os_project config location
da09dde 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
no-os: update mcufla.sh command
ad5fa0d 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
no-os: remove pwd
bc3677c 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
no-os:handle xilinx naming
c741e27 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
no-os:fix filter for elf file
838aa31 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
no-os: add other dependencies
5162df7 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
no-os:add correct link for mcufla.sh
5370d6f 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
no-os: fix filepath
797e010 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
no-os:stop pipeline if file is not found
9c99869 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
no-os: create dummy system_top.xsa
b225bbf 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
no-os: copy system_top.xsa to project folder
e95b2d4 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
no-os: update copying of xsa file
31f4f3b 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
no-os: update checking context
1c5d550 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
no-os: update archiving of logfiles
7ff5d78 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
pyadi-iio test: update serial baudrate source
99fd947 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
nooos: customize baudrate for iio example
4a6317c 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
noos: remove exception of other examples
6418c6c 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
pyaditest: skip stage for other noos examples
6f76976 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
pyadi stage: update skipping of stage with respect to noos example
d055ac7 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
noos: update waiting time to boot
dd30caa 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
pyaditests: update handling of -v marker
049b03d 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
noos: set boot waiting time longer
eb28ee6 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
noos: fix syntax
ca02879 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
noos: update link of mcufla.sh
a3af0dd 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
noos: improve checks for failing commands
b5fd453 
Signed-off-by: Trecia Agoylo <[email protected]>
@kimpaller
add url_template argument
ad4c8f4 
Signed-off-by: kimpaller <[email protected]>
kimpaller and others added 23 commits January 10, 2025 08:59
@kimpaller
Merge pull request #95 from sdgtt/fix-libiio-checkout
eb505bc 
modify libiio checkout to support branch or tag
@tagoylo
add exception to checksum verification
4184318 
Signed-off-by: Trecia Agoylo <[email protected]>
@kimpaller
Merge pull request #96 from sdgtt/checksum-exception
5178000 
add exception to checksum verification
@kimpaller
verify first if board indeed needs recovery
78b8a47 
Signed-off-by: kimpaller <[email protected]>
@tagoylo
Merge pull request #98 from sdgtt/maintenance_tasks
b3b8915 
verify first if board indeed needs recovery
@jpineda3
Fix typo from recent merge
8c1fa82
@jpineda3
Fix scm checkout for PRs on multibranch pipelines
7b4f4ab 
Signed-off-by: Pineda <[email protected]>
@tagoylo
Merge pull request #99 from sdgtt/fix-pr-clone
e546e1b 
Fix scm checkout for PRs on multibranch pipelines
@jpineda3
Fix repo clone for multibranch but not current repo CI
91489b2 
Signed-off-by: Pineda <[email protected]>
@tagoylo
Merge pull request #100 from sdgtt/fix-pr-clone
908dc90 
Fix repo clone for multibranch but not current repo CI
@tagoylo
set dafault image to 22.04
a6eab82 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
Merge pull request #103 from sdgtt/update-default-image
d86aa9b 
set dafault image to 22.04
@bia1708
Gauntlet.groovy: Fix nebula call to netbox
7b4368a 
Add the netbox token and IP as command line arguments for nebula. This
eliminates the added dependency of having to store netbox credentials
inside the config.

Signed-off-by: bia1708 <[email protected]>
@kimpaller
Merge pull request #105 from bia1708/fix-scripts
477dce3 
Gauntlet.groovy: Fix nebula call to netbox
@macylibed9
Pytest-libiio_fix
610377f 
Signed-off-by: Macy Libed <[email protected]>
@tagoylo
Merge pull request #106 from macylibed9/pyaditest-libiio_fix
af5993a 
pytest libiio fix
@Zvckkk
added a toggle for adi_diagnostics and set the default to false.
43d5e5f
@jpineda3
Merge pull request #109 from Zvckkk/remove-adi-diagnostic
04fb949 
added a toggle for adi_diagnostics and set the default to false.
@tagoylo
add way to optionally install agent deps
8172ac0 
Signed-off-by: Trecia Agoylo <[email protected]>
@tagoylo
default to updating requirements in deps installation in agent
ecac6a2 
Signed-off-by: Trecia Agoylo <[email protected]>
@Zvckkk
added space in command line arguement --board-name for netbox disable
5545af8
@tagoylo
Merge pull request #113 from Zvckkk/adgt-test-harness
bb0fa04 
added space in command line arguement --board-name for netbox disable
@kimpaller
Merge pull request #112 from sdgtt/install-deps-on-agent
01079e5 
Install deps on agent
@kimpaller kimpaller changed the title Adgt test harness Maintenance changes and updates Oct 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tagoylo tagoylo tagoylo approved these changes

@jpineda3 jpineda3 Awaiting requested review from jpineda3

@tfcollins tfcollins Awaiting requested review from tfcollins

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@kimpaller @tagoylo @jpineda3 @bia1708 @macylibed9 @Zvckkk