Release cortex core v0.1.2 & bundles v0.0.76#966
Conversation
Confirmed VMs (present in Status.Allocations) now bypass the grace period check when they disappear from the HV CRD. Their departure is authoritative — Spec.Allocations and Status.Allocations are cleared in the same reconcile pass triggered by the existing HV watch. Previously the grace period gate applied to all VMs regardless of confirmation state, meaning a confirmed VM's departure could be delayed up to 15 minutes.
This PR contains the following updates: | Package | Change | [Age](https://docs.renovatebot.com/merge-confidence/) | [Adoption](https://docs.renovatebot.com/merge-confidence/) | [Passing](https://docs.renovatebot.com/merge-confidence/) | [Confidence](https://docs.renovatebot.com/merge-confidence/) | |---|---|---|---|---|---| | go.xyrillian.de/gg | `v1.9.0` → `v1.9.1` |  |  |  |  | --- ### Configuration 📅 **Schedule**: (in timezone Europe/Berlin) - Branch creation - "after 6pm every weekday,every weekend,before 8am every weekday" - Automerge - At any time (no schedule defined) 🚦 **Automerge**: Enabled. ♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR was generated by [Mend Renovate](https://mend.io/renovate/). View the [repository job log](https://developer.mend.io/github/cobaltcore-dev/cortex). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0My4yMTkuMCIsInVwZGF0ZWRJblZlciI6IjQzLjIxOS4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119--> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Business logic should not reside in the api package unless it should be consumed by other services as a code package.
…duling (#955) ## feat(reservations): honor domain restrictions for CR reservation scheduling CR reservation scheduling was not passing a `domain_name` scheduler hint, causing `filter_external_customer` to silently skip domain-based host restrictions for all CR reservations. This PR resolves the `DomainID` stored in the `Reservation` CRD to a domain name via Keystone and passes it as the `domain_name` scheduler hint, so `filter_external_customer` can enforce `CUSTOM_EXTERNAL_CUSTOMER_EXCLUSIVE` host restrictions for external customer domains. Failover reservations are unaffected — the filter already skips them by intent. ## Changes - `domain_resolver.go` — `DomainResolver` interface + `keystoneDomainResolver` backed by gophercloud with an in-process cache (domain names are immutable) - `reservation_controller.go` — resolves domain name before building the scheduler request; best-effort: resolver failure logs and proceeds without the hint - `config.go` — `KeystoneSecretRef` and `SSOSecretRef` added to `ReservationControllerConfig` - `helm/bundles/cortex-nova/values.yaml` — wires `keystoneSecretRef` under `committedResourceReservationController`; `ssoSecretRef` commented out following existing convention - `cmd/manager/main.go` — `e2e-nova` now accepts a JSON override (same as `e2e-commitments`) enabling ad-hoc `domainNameHintCheck` runs - `e2e_checks.go` (nova) — `CheckDomainNameHintRouting` sends a synthetic CR scheduling request and confirms the pipeline accepts the hint - Tests — 8 resolver unit tests (real HTTP server, race detector), 3 hint-injection tests, 2 new `filter_external_customer` cases covering CR intent and `Spec.CustomTraits`
bump app version [skip ci] ``` bumped cortex: sha-a1233a36 -> sha-29d38091 bumped cortex-shim: sha-a1233a36 -> sha-e7e039b1 noop cortex-postgres (sha-8a5b5dd6) ``` Co-authored-by: juliusclausnitzer <65490082+juliusclausnitzer@users.noreply.github.com>
bump app version [skip ci] ``` bumped cortex: sha-29d38091 -> sha-0b499dbc noop cortex-shim (sha-e7e039b1) noop cortex-postgres (sha-8a5b5dd6) ``` Co-authored-by: SoWieMarkus <44964229+SoWieMarkus@users.noreply.github.com>
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@claude run the /release 966 command |
|
Claude finished @SoWieMarkus's task in 11m 27s —— View job /release 966 — Complete
SummaryCreated the release envelope for cortex core v0.1.2 & bundles v0.0.75:
|
|
@claude run the /release 966 command Please also update the bundles this time. |
|
Claude finished @SoWieMarkus's task in 8m 32s —— View job Release Orchestrator Complete
SummaryCreated the following PRs for release #966:
Changes made:
Next steps: Merge #969 and #970 into main, then merge release PR #966 (main → release). |
## Summary - Bump cortex library chart from 0.1.1 to 0.1.2 - Bump all bundle chart versions from 0.0.75 to 0.0.76 - Update cortex dependency in all bundles to 0.1.2 ## Bumped Charts | Chart | Old Version | New Version | |-------|------------|-------------| | cortex (library) | 0.1.1 | 0.1.2 | | cortex-cinder | 0.0.75 | 0.0.76 | | cortex-crds | 0.0.75 | 0.0.76 | | cortex-ironcore | 0.0.75 | 0.0.76 | | cortex-manila | 0.0.75 | 0.0.76 | | cortex-nova | 0.0.75 | 0.0.76 | | cortex-pods | 0.0.75 | 0.0.76 | ## Bundle Dependency Updates | Bundle | cortex dep | New dep version | |--------|-----------|-----------------| | cortex-cinder | cortex | 0.1.2 | | cortex-crds | cortex | 0.1.2 | | cortex-ironcore | cortex | 0.1.2 | | cortex-manila | cortex | 0.1.2 | | cortex-nova | cortex | 0.1.2 | | cortex-pods | cortex | 0.1.2 | This PR should be merged before release PR #966. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: cortex-ai-agents[bot] <279748396+cortex-ai-agents[bot]@users.noreply.github.com> Co-authored-by: Markus Wieland <SoWieMarkus@users.noreply.github.com> Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
## Summary - Add changelog entry for release PR #966 (cortex v0.1.2, bundles v0.0.76) - Documents new features: domain restriction scheduling, InFlightReservation, datasource restart sync, VM placement tracking, and more - No breaking changes in this release ## Test plan - [x] Changelog entry follows existing format and conventions - [ ] Verify PR links resolve correctly 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: cortex-ai-agents[bot] <279748396+cortex-ai-agents[bot]@users.noreply.github.com> Co-authored-by: Markus Wieland <SoWieMarkus@users.noreply.github.com> Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
This PR contains the following updates: | Package | Change | [Age](https://docs.renovatebot.com/merge-confidence/) | [Adoption](https://docs.renovatebot.com/merge-confidence/) | [Passing](https://docs.renovatebot.com/merge-confidence/) | [Confidence](https://docs.renovatebot.com/merge-confidence/) | Type | Update | |---|---|---|---|---|---|---|---| | [github.com/mattn/go-sqlite3](https://redirect.github.com/mattn/go-sqlite3) | `v1.14.45` → `v1.14.47` |  |  |  |  | require | patch | | go.xyrillian.de/gg | `v1.9.1` → `v1.10.1` |  |  |  |  | require | minor | | [kube-prometheus-stack](https://redirect.github.com/prometheus-operator/kube-prometheus) ([source](https://redirect.github.com/prometheus-community/helm-charts)) | `86.2.3` → `86.3.2` |  |  |  |  | | minor | --- ### Release Notes <details> <summary>mattn/go-sqlite3 (github.com/mattn/go-sqlite3)</summary> ### [`v1.14.47`](https://redirect.github.com/mattn/go-sqlite3/compare/v1.14.46...v1.14.47) [Compare Source](https://redirect.github.com/mattn/go-sqlite3/compare/v1.14.46...v1.14.47) ### [`v1.14.46`](https://redirect.github.com/mattn/go-sqlite3/compare/v1.14.45...v1.14.46) [Compare Source](https://redirect.github.com/mattn/go-sqlite3/compare/v1.14.45...v1.14.46) </details> <details> <summary>prometheus-community/helm-charts (kube-prometheus-stack)</summary> ### [`v86.3.2`](https://redirect.github.com/prometheus-community/helm-charts/releases/tag/kube-prometheus-stack-86.3.2) [Compare Source](https://redirect.github.com/prometheus-community/helm-charts/compare/kube-prometheus-stack-86.3.1...kube-prometheus-stack-86.3.2) kube-prometheus-stack collects Kubernetes manifests, Grafana dashboards, and Prometheus rules combined with documentation and scripts to provide easy to operate end-to-end Kubernetes cluster monitoring with Prometheus using the Prometheus Operator. ##### What's Changed - \[kube-prometheus-stack] Update kube-prometheus-stack dependency non-major updates by [@​renovate](https://redirect.github.com/renovate)\[bot] in [#​7008](https://redirect.github.com/prometheus-community/helm-charts/pull/7008) **Full Changelog**: <prometheus-community/helm-charts@prometheus-operator-admission-webhook-0.42.1...kube-prometheus-stack-86.3.2> ### [`v86.3.1`](https://redirect.github.com/prometheus-community/helm-charts/releases/tag/kube-prometheus-stack-86.3.1) [Compare Source](https://redirect.github.com/prometheus-community/helm-charts/compare/kube-prometheus-stack-86.3.0...kube-prometheus-stack-86.3.1) kube-prometheus-stack collects Kubernetes manifests, Grafana dashboards, and Prometheus rules combined with documentation and scripts to provide easy to operate end-to-end Kubernetes cluster monitoring with Prometheus using the Prometheus Operator. #### What's Changed - \[CI] Update github-workflow dependency updates to v7 by [@​renovate](https://redirect.github.com/renovate)\[bot] in [#​7007](https://redirect.github.com/prometheus-community/helm-charts/pull/7007) - \[kube-prometheus-stack] Update kube-prometheus-stack dependency non-major updates by [@​renovate](https://redirect.github.com/renovate)\[bot] in [#​7006](https://redirect.github.com/prometheus-community/helm-charts/pull/7006) **Full Changelog**: <prometheus-community/helm-charts@kube-prometheus-stack-86.3.0...kube-prometheus-stack-86.3.1> ### [`v86.3.0`](https://redirect.github.com/prometheus-community/helm-charts/releases/tag/kube-prometheus-stack-86.3.0) [Compare Source](https://redirect.github.com/prometheus-community/helm-charts/compare/kube-prometheus-stack-86.2.3...kube-prometheus-stack-86.3.0) kube-prometheus-stack collects Kubernetes manifests, Grafana dashboards, and Prometheus rules combined with documentation and scripts to provide easy to operate end-to-end Kubernetes cluster monitoring with Prometheus using the Prometheus Operator. #### What's Changed - \[kube-prometheus-stack] Update kube-prometheus-stack dependency non-major updates by [@​renovate](https://redirect.github.com/renovate)\[bot] in [#​6990](https://redirect.github.com/prometheus-community/helm-charts/pull/6990) **Full Changelog**: <prometheus-community/helm-charts@prometheus-29.13.0...kube-prometheus-stack-86.3.0> </details> --- ### Configuration 📅 **Schedule**: (in timezone Europe/Berlin) - Branch creation - "after 6pm every weekday,every weekend,before 8am every weekday" - Automerge - At any time (no schedule defined) 🚦 **Automerge**: Enabled. ♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. 👻 **Immortal**: This PR will be recreated if closed unmerged. Get [config help](https://redirect.github.com/renovatebot/renovate/discussions) if that's undesired. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR was generated by [Mend Renovate](https://mend.io/renovate/). View the [repository job log](https://developer.mend.io/github/cobaltcore-dev/cortex). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0My4yMzEuMSIsInVwZGF0ZWRJblZlciI6IjQzLjIzNS4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119--> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…er (#909) When the VM source (postgres) is missing data but VMs are still active on hypervisors (e.g. after a postgres data loss), the failover controller previously treated every allocated VM as 'no longer exists' and wiped the allocations. Empty reservations were then deleted, causing total loss of failover coverage despite the VMs still running. Cross-check hv1.HypervisorList.Status.Instances before removing a VM from allocations in reconcileRemoveInvalidVMFromReservations. If the VM is still reported active on any known hypervisor, keep the allocation and log a warning. The hypervisor-mismatch branch is intentionally left unchanged: it is already governed by the TrustHypervisorLocation flag. Adds unit tests covering: - VM missing from VM source but still on hypervisor (allocation kept) - VM source completely empty while VMs still on hypervisors (all reservations preserved -- the exact incident scenario) - Mixed: postgres-missing-but-on-HV vs. truly gone (only truly gone removed) - Hypervisor mismatch is not suppressed by the safeguard
…ion/modification (#951) When a new committed resource is created, the controller now absorbs existing PAYG VMs into reservation slots instead of blindly probing the scheduler for every slot.
Test Coverage ReportTest Coverage 📊: 69.8% |
|
@claude two PRs were merged. Can you create a PR with changes to the changelog.md and here in the PR description? |
|
Claude finished @SoWieMarkus's task in 8m 28s —— View job Updated CHANGELOG.md and PR Description
Summary: Three PRs were merged after the changelog (#970) and chart-bump (#969) PRs:
Changes made:
The appVersion in | PR #974 |
bump app version [skip ci] ``` bumped cortex: sha-0b499dbc -> sha-6daa5050 bumped cortex-shim: sha-e7e039b1 -> sha-b7fbbe1a noop cortex-postgres (sha-8a5b5dd6) ``` Co-authored-by: mblos <156897072+mblos@users.noreply.github.com>
## Summary - Add #951 (PAYG pre-allocation), #909 (failover fix), and #959 (dependency update) to release changelog - Update SHA reference from sha-0b499dbc to sha-6daa5050 This updates the changelog for release PR #966 to include PRs that were merged after the initial changelog PR (#970) was created. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Markus Wieland <SoWieMarkus@users.noreply.github.com> Co-authored-by: cortex-ai-agents[bot] <279748396+cortex-ai-agents[bot]@users.noreply.github.com> Co-authored-by: Markus Wieland <SoWieMarkus@users.noreply.github.com>
Release cortex core v0.1.2 & bundles v0.0.76
Prerequisites
cortex v0.1.2 (sha-6daa5050)
Non-breaking changes:
github.com/sapcc(Renovate: Update github.com/sapcc #938)cortex-nova v0.0.76
Includes updated chart cortex v0.1.2.
keystoneSecretRefandssoSecretRefconfig keys for domain resolution in committed resource reservation scheduling (feat(reservations): honor domain restrictions for CR reservation scheduling #955)enablePaygPreAllocationconfig key for pre-allocating PAYG VMs into reservation slots (feat(cr): pre-allocate PAYG VMs into CR reservation slots on cr creation/modification #951)cortex-crds v0.0.76
Includes updated chart cortex v0.1.2.
cortex-cinder v0.0.76
Includes updated chart cortex v0.1.2.
cortex-pods v0.0.76
Includes updated chart cortex v0.1.2.
cortex-ironcore v0.0.76
Includes updated chart cortex v0.1.2.
cortex-manila v0.0.76
Includes updated chart cortex v0.1.2.
General