Epic 9 - #348
Merged
Merged
Conversation
Comprehensive dev-agent story context created for: - 9.0: Fix RabbitMQ vhost/vhostTopic serialization bug - 9.1: Upgrade vault/api from v1.14 to v1.23 - 9.2: Upgrade test dependencies (ginkgo, gomega) - 9.3: Upgrade peripheral and security dependencies - 9.4: Evaluate and plan pkg/errors migration - 9.5: Upgrade Vault version in integration test infrastructure to 2.0.3 Co-authored-by: Cursor <cursoragent@cursor.com>
Fix map-reassignment bugs in convertVhostsToJson and convertTopicsToJson that discarded all but the last entry when multiple vhosts or vhostTopics were configured. Replace log.Fatal with panic to avoid os.Exit killing the operator. Add 4 multi-entry unit tests with independently-constructed expected values. Merge duplicate vhostTopics entries for the same vhostName. Replace helper-derived test expectations and switch to parsed JSON comparison for ordering resilience. Co-authored-by: Cursor <cursoragent@cursor.com>
Upgrade ginkgo v2.27.4 → v2.32.0 and gomega v1.39.0 → v1.42.1 with transitive dependency resolution via go mod tidy. No test code changes required — fully backward-compatible. All unit and integration tests pass. Co-authored-by: Cursor <cursoragent@cursor.com>
…l stories Add 4 new stories to Epic 10 (Operator SDK + Build Tooling Upgrades): - 10-0a: Kustomize v3 to v5 syntax migration (bases to resources, patchesStrategicMerge to patches, vars to replacements) - 10-5: Remove kube-rbac-proxy sidecar, enable controller-runtime authn/authz for metrics - 10-6: Helm chart kube-rbac-proxy removal and metrics rearchitecture - 10-7: CI pipeline adaptation and end-to-end validation Epic renamed to Operator SDK + Build Tooling Upgrades and Metrics Modernization to reflect expanded scope. Co-authored-by: Cursor <cursoragent@cursor.com>
Low-risk dependency bump with no Go source changes required. All existing vault/api usage (ResponseError assertions, DefaultConfig, NewClient, TLSConfig, Secret, EnableAuditOptions, LifetimeWatcherInput) remains backward-compatible. Unit and integration tests pass. Transitive dependency changes: backoff/v3→v4, go-retryablehttp v0.7.8, parseutil v0.2.0, go-sockaddr v1.0.7, hcl v1.0.1-vault-7, go-wordwrap v1.0.1, go-test/deep v1.1.1, mattn/go-colorable v0.1.14, fatih/color v0.18.0. Co-authored-by: Cursor <cursoragent@cursor.com>
- hcl/v2 v2.21.0 → v2.24.0 (transitive: go-cty v1.13.0 → v1.16.3) - sprig/v3 v3.2.3 → v3.3.0 (transitive: dario.cat/mergo v1.0.1, xstrings v1.5.0, cast v1.7.0, decimal v1.4.0, copystructure v1.2.0) - go-logr/logr v1.4.3 → v1.4.4 - golang.org/x/crypto v0.53.0 → v0.54.0 - golang.org/x/net v0.56.0 → v0.57.0 - Transitive x/ bumps: mod, sync, sys, term, text, tools No Go source code changes. All unit and integration tests pass. Co-authored-by: Cursor <cursoragent@cursor.com>
Replace the single go-multierror usage in RandomSecret.isValid() with errors.Join from the standard library. This removes go-multierror as a direct dependency (remains indirect via vault/api).
…rastructure Update Vault version across integration and local-development Helm values, Makefile version variables (VAULT_VERSION, VAULT_CHART_VERSION), and project-context.md. Improve Makefile vault CLI download target to use versioned binary path with symlink (matching kubectl pattern) so version changes trigger a fresh download. All integration tests pass against Vault 2.0.3 with Helm chart 0.34.0. No Go source code changes required. Co-authored-by: Cursor <cursoragent@cursor.com>
Complete Epic 9 retrospective covering 6 stories delivered in 11 days: bug fix (9.0), dependency upgrades (9.1-9.3), pkg/errors migration (9.4), and Vault 2.0.3 infrastructure upgrade (9.5). Mark epic-9 done and retrospective complete. Clean up resolved items in deferred-work.md. Co-authored-by: Cursor <cursoragent@cursor.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
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.
Epic 9: Vault API + Peripheral Dependency Upgrades
epic-9: done
9-0-fix-rabbitmq-vhost-vhosttopic-serialization-bug: done
9-1-upgrade-vault-api-from-v1-14-to-v1-23: done
9-2-upgrade-test-dependencies-ginkgo-gomega: done
9-3-upgrade-peripheral-and-security-dependencies: done
9-4-evaluate-and-plan-pkg-errors-migration: done
9-5-upgrade-vault-version-in-integration-test-infrastructure: done
epic-9-retrospective: done