Conversation
Add automated dependency management via Dependabot for GitHub Actions and Python pre-commit hooks. Replace unstable @master reference with stable v2.0.0 release for ludeeus/action-shellcheck to prevent unexpected breaking changes in CI workflows. Changes: - Add .github/dependabot.yml for weekly dependency updates - Update pr-validation.yml: ludeeus/action-shellcheck@master → @2.0.0 - Update shell-script-testing.yml: ludeeus/action-shellcheck@master → @2.0.0 Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
…ts/ directory Move all testing, validation, and CI-related scripts and manifests from scripts/ and openshift/dr-testing/ into a new top-level tests/ directory for better organization and separation of concerns. New structure: - tests/scripts/ - Test and validation scripts (DR, RTO/RPO, validation) - tests/hooks/ - Pre-commit and CI hooks - tests/openshift/ - OpenShift DR testing CronJob manifests - tests/README.md - Comprehensive testing documentation scripts/ now contains only operational scripts: - AAP scaling and cluster management - EFM integration hooks - Shared operational libraries Changes (30 files): - Moved 6 test scripts to tests/scripts/ - Moved 2 CI hooks to tests/hooks/ - Moved 6 DR testing manifests to tests/openshift/dr-testing/ - Updated 15 documentation files with new paths - Updated .pre-commit-config.yaml with new hook paths - Updated CLAUDE.md and README.md repository structure diagrams - Created tests/README.md with complete testing documentation All file moves preserve complete git history via 'git mv'. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This file contains hidden or 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
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.
Summary
This PR introduces two major improvements to the CI/CD infrastructure:
tests/directoryChanges
1. Dependabot Configuration (265a5db)
Added
.github/dependabot.yml:Fixed critical stability issue:
ludeeus/action-shellcheck@master(unstable) with@2.0.0(stable release)pr-validation.ymlandshell-script-testing.ymlExpected benefits:
2. Testing Infrastructure Reorganization (c7c874a)
New
tests/directory structure:Updated
scripts/to contain only operational scripts:Documentation updates (15 files):
.pre-commit-config.yamlupdated with new hook pathsBenefits:
git mvTesting
Pre-commit Hooks
All pre-commit hooks updated and tested with new paths:
# Hooks now correctly reference tests/hooks/ - tests/hooks/check-script-permissions.sh - tests/hooks/validate-openshift-manifests.shCI Workflows
All GitHub Actions workflows validated:
Documentation
All 15 updated documentation files verified:
Files Changed
30 files total:
.github/dependabot.ymltests/README.mdscripts/andopenshift/dr-testing/→tests/Impact
Low risk changes:
Expected Dependabot PRs after merge:
Checklist
Related
This PR sets the foundation for future CI/CD improvements identified in pipeline analysis:
🤖 Generated with Claude Code