-
Notifications
You must be signed in to change notification settings - Fork 25
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.
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
[Backport] v3.0.0 improvements to release/v2.x #203
Draft
alexandreLamarre
wants to merge
6
commits into
release/v2.x
Choose a base branch
from
backport-to-release-2
base: release/v2.x
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
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
Fix bug when finding project root Pick the last tag instead of the first one Allow setting GIT_TAG externally too and set on workflow based on tags Fix install-federator e2e CI to use proper method for current charts Rewrite a few e2e scripts to be timeout based loops Add local-e2e target script matching current GHA workflow Remove unnecessary wait for data steps w/ new script Correct dev branch tags Refactor another e2e script to use timeout loop more shellcheck Move timeout onto correct step Fix success condition of alerts script try e2e without branch tags script Seed image name for testing via versions script Add TAG env back to e2e workflow Refactor another script for wait loop Fix inconsistent case of strings Also make sure that Prom Targets are in the "up" state Adds missing parameters to the commands in */developing.md fix(ci): Correct k3s version used for e2e testing (#172) * Add min/max k3s version used for testing I'm adding both so in the future we can test both extremes, but for now I'm going to PR just using 1 of them. * go generate * Add helper script to grab K3S versions as ENVs and use that in local-e2e * Make debug output actually create envs that should work in GHA instead of just human debug output * Source k3s version from `build.yaml` for GHA too * Fix job step ordering so yq will be installed when k3s version env is set * remove conflicting and unused ids * Fix helm-locker yq step * Export tag style of k3s versions * Use tags directly in GHA update libraries & dependencies to k8s 1.32 Signed-off-by: Alexandre Lamarre <[email protected]> add updated helm-locker Signed-off-by: Alexandre Lamarre <[email protected]> add updated helm-project-operator Signed-off-by: Alexandre Lamarre <[email protected]> Add node identification to internal package Signed-off-by: Alexandre Lamarre <[email protected]> Add internal helmcommon package to manage crds Signed-off-by: Alexandre Lamarre <[email protected]> update helm-locker to use shared crds abstraction Signed-off-by: Alexandre Lamarre <[email protected]> update helm-project-operator to use shared crds abstraction Signed-off-by: Alexandre Lamarre <[email protected]> update older sub-packages Signed-off-by: Alexandre Lamarre <[email protected]> update main functions slightly rework crd management Signed-off-by: Alexandre Lamarre <[email protected]> update go generate targets Signed-off-by: Alexandre Lamarre <[email protected]> update to go 1.23 run go mod tidy Signed-off-by: Alexandre Lamarre <[email protected]> update golangci-lint Signed-off-by: Alexandre Lamarre <[email protected]> remove internal go sub modules Signed-off-by: Alexandre Lamarre <[email protected]> update controller & crd generation Signed-off-by: Alexandre Lamarre <[email protected]> Run go generate Signed-off-by: Alexandre Lamarre <[email protected]> update helm-project-operator with new generated code Signed-off-by: Alexandre Lamarre <[email protected]> fix crd generation path Signed-off-by: Alexandre Lamarre <[email protected]> tweak test labels Signed-off-by: Alexandre Lamarre <[email protected]> run go generate Signed-off-by: Alexandre Lamarre <[email protected]> update images to go 1.23 Signed-off-by: Alexandre Lamarre <[email protected]> simplify dev scripts Signed-off-by: Alexandre Lamarre <[email protected]> remove dapper from Makefile Signed-off-by: Alexandre Lamarre <[email protected]> [WIP] simplify CI Signed-off-by: Alexandre Lamarre <[email protected]> vendor chart data in public package Signed-off-by: Alexandre Lamarre <[email protected]> update integration workflow again Signed-off-by: Alexandre Lamarre <[email protected]> use upstream helm during image builds Signed-off-by: Alexandre Lamarre <[email protected]> [WIP] importing images for integration CI Signed-off-by: Alexandre Lamarre <[email protected]> fix integration testing Signed-off-by: Alexandre Lamarre <[email protected]> [temp] disable validate-ci Signed-off-by: Alexandre Lamarre <[email protected]> lint fixes Signed-off-by: Alexandre Lamarre <[email protected]> Add back deprecated flag logic Signed-off-by: Alexandre Lamarre <[email protected]> remove unecessary CI stuff Signed-off-by: Alexandre Lamarre <[email protected]> update helm-project-operator build-chart destination Signed-off-by: Alexandre Lamarre <[email protected]> make crd management consistent with existing flags and add tests Signed-off-by: Alexandre Lamarre <[email protected]> update unit test command to exclude integration tests Signed-off-by: Alexandre Lamarre <[email protected]> Add helm-project-operator chart to examples dir Signed-off-by: Alexandre Lamarre <[email protected]> update path for HPO chart in integration tests Signed-off-by: Alexandre Lamarre <[email protected]> revert public pkg/chart package Signed-off-by: Alexandre Lamarre <[email protected]> vendor correct helm-controller dependencies Signed-off-by: Alexandre Lamarre <[email protected]> update wrangler/go toolchain Signed-off-by: Alexandre Lamarre <[email protected]> remove helm-locker e2e CI Signed-off-by: Alexandre Lamarre <[email protected]> bump rancher/klipper-helm to v0.9.4-build20250113 Signed-off-by: Alexandre Lamarre <[email protected]> add KUBECONFIG to version script Signed-off-by: Alexandre Lamarre <[email protected]> add back embedded controller name logic Signed-off-by: Alexandre Lamarre <[email protected]> Empty-Commit rename internal/helm-locker/pkg -> internal/helm-locker Signed-off-by: Alexandre Lamarre <[email protected]> rename internal/helm-project-operator/pkg -> internal/helm-project-operator Signed-off-by: Alexandre Lamarre <[email protected]> centralize cmd version Signed-off-by: Alexandre Lamarre <[email protected]> update golangci-lint config Signed-off-by: Alexandre Lamarre <[email protected]> bump integration test k3s matrix [1.30.9, 1.32.1] Signed-off-by: Alexandre Lamarre <[email protected]> feat: Update Rancher Project Monitoring to use upstream 66.7.1 (#173) * Update Rancher Project Monitoring to use 0.5.0 chart version that matches Monitoring 66.7.1 * go generate * bump RPM version for small version fix * go generate * fix local-e2e bug * fix ci artifact file name * (test) expand CI wait time * fix: consistently use KUBECTL_WAIT_TIMEOUT * Make install line more readable * Every supported k3s/rke2 versions should have it disabled * make all helm install steps uniform * Capture more details about Project Monitoring install task * expand timeouts for scripts * Update shell for kuberlr-kubectl * Adjust alerts validation script Add initial Renovate configuration (#149) Co-authored-by: renovate-rancher[bot] <119870437+renovate-rancher[bot]@users.noreply.github.com> Pin dependencies Migrate config .github/renovate.json remove dapper Signed-off-by: Alexandre Lamarre <[email protected]> Update Docker File Deps manually downgrade to go 1.23 Signed-off-by: Alexandre Lamarre <[email protected]> ci: Improve ability to use CI in forks & fix multiple tag bug (#189) * Add empty for debug * fix all runs-on to be dynamic * only read vault secrets when repo matches rancher/prom-fed * Install YQ only in GHA runners where YQ isn't available * Ensure package-helm uses the current tag for chart versioning * fix runs-on format to respect matrix.arch Adjust Alertmanager E2E verification to allow common alerts (#190) * Adjust Alertmanager E2E verification to allow InfoInhibitor & PrometheusOutOfOrderTimestamps * Make helm repo url configurable * Simplify checks to verify based on count vs (count - expected)
5f12190
to
ff8fcb3
Compare
Signed-off-by: Alexandre Lamarre <[email protected]>
Signed-off-by: Alexandre Lamarre <[email protected]>
k3s [1.28:1.31] Signed-off-by: Alexandre Lamarre <[email protected]>
Signed-off-by: Alexandre Lamarre <[email protected]>
Signed-off-by: Alexandre Lamarre <[email protected]>
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.