Skip to content

feat(dedicated-server): adopt plannedChange apiv6 route behind feature flag#23011

Merged
qpavy merged 2 commits into
release/bm-dc360-plannedchangesfrom
feat/dedicated-planned-change-route
Jul 16, 2026
Merged

feat(dedicated-server): adopt plannedChange apiv6 route behind feature flag#23011
qpavy merged 2 commits into
release/bm-dc360-plannedchangesfrom
feat/dedicated-planned-change-route

Conversation

@Carlolis

@Carlolis Carlolis commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Description

Gate the new /dedicated/server/:serviceName/plannedChange apiv6 route (DC360, INTERNAL rollout) behind the dedicated-server:plannedChange feature flag, falling back to the legacy /plannedIntervention route when the flag is off.

When the flag is on, upgradeTask reads task.changeUuid and calls /plannedChange/:changeUuid; otherwise it reads task.plannedInterventionId and calls /plannedIntervention/:interventionId. PlannedChange exposes the same wantedStartDate the UpgradeTask.scheduledTimeSlot getter reads, so the model is unchanged.

Applied identically to the server dashboard and the cluster-node dashboard resolvers; flag id added to both features batches.

Ticket Reference: #PUBM-54260

qpavy
qpavy previously approved these changes Jul 9, 2026
Carlolis and others added 2 commits July 15, 2026 06:45
…e flag

Gate the new /dedicated/server/:serviceName/plannedChange apiv6 route
(DC360, INTERNAL rollout) behind the dedicated-server:plannedChange
feature flag, falling back to the legacy /plannedIntervention route
when the flag is off.

When the flag is on, upgradeTask reads task.changeUuid and calls
/plannedChange/:changeUuid; otherwise it reads task.plannedInterventionId
and calls /plannedIntervention/:interventionId. PlannedChange exposes the
same wantedStartDate the UpgradeTask.scheduledTimeSlot getter reads, so
the model is unchanged.

Applied identically to the server dashboard and the cluster-node
dashboard resolvers; flag id added to both features batches.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
ref: #PUBM-54260

Signed-off-by: soufien mhelhali <soufien.mhelhali.ext@ovhcloud.com>
@mhelhali-soufien
mhelhali-soufien dismissed stale reviews from qpavy and themself via 8856b46 July 15, 2026 04:45
@mhelhali-soufien
mhelhali-soufien force-pushed the feat/dedicated-planned-change-route branch from 17d877a to 8856b46 Compare July 15, 2026 04:45
@qpavy
qpavy changed the base branch from master to release/bm-dc360-plannedchanges July 16, 2026 10:44
@qpavy
qpavy merged commit d1ad4a9 into release/bm-dc360-plannedchanges Jul 16, 2026
11 of 13 checks passed
@qpavy
qpavy deleted the feat/dedicated-planned-change-route branch July 16, 2026 10:47
@qpavy qpavy mentioned this pull request Jul 16, 2026
1 task
acavalloni pushed a commit that referenced this pull request Jul 16, 2026
…e flag (#23011)

switch api route from plannedIntervention to plannedChange

ref: #PUBM-54260

Signed-off-by: soufien mhelhali <soufien.mhelhali.ext@ovhcloud.com>
Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-authored-by: charles ilief <charles.ilieff@ovhcloud.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants