🌱 Add script to check CAPI version alignment #1706
Open
Conversation
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
Collaborator
|
@s04 can you please use that heading? thank you. Otherwise it looks good. |
s04
force-pushed
the
feature/check-capi-version-sync
branch
from
b47f8e8 to
0d681e5
Compare
Contributor
Author
|
@guettli thanks added. |
guettli
changed the title
Collaborator
|
@s04 I am sorry, linting fails. Files need a boilerplate header: #!/usr/bin/env bash
# Copyright 2025 The Kubernetes Authors.
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.can you please add this? |
This script validates that Cluster API versions are consistent across the codebase by reading the desired version from go.mod and checking for mismatches in Makefile, Tiltfile, and test configurations. Documentation files (*.md) are excluded as they may contain example output that doesn't need to be kept in sync. Fixes syself#1705
s04
force-pushed
the
feature/check-capi-version-sync
branch
from
0d681e5 to
2cd8724
Compare
Contributor
Author
|
@guettli thanks for your patience! Added it and ran the linter. Looks good now, will make a note to run the validator in the future. 🔍 121 Total (in 9s) ✅ 106 OK 🚫 0 Errors 💤 15 Excluded |
Collaborator
|
@s04 Please run |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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
Adds a bash script that detects Cluster API version misalignments across the codebase.
Changes
hack/check-capi-version.shthat:go.modUsage
The script exits with status 1 if mismatches are found, making it suitable for CI integration.
Fixes #1705