Skip to content

Conversation

ajanikow
Copy link
Collaborator

No description provided.

@cla-bot cla-bot bot added the cla-signed label Jul 24, 2025
@ajanikow ajanikow requested a review from Copilot July 24, 2025 13:33
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR introduces a new upgrade mode called "optional-replace" for ArangoDB deployment server groups. This mode provides a conditional replacement strategy that falls back to replacing members when upgrades fail with specific exit codes, particularly code 30 for compaction failures.

  • Adds the "optional-replace" upgrade mode as a new enum value across all API versions
  • Updates validation logic to accept the new upgrade mode
  • Documents the new mode in API specifications and user documentation

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pkg/crd/crds/database-deployment.schema.generated.yaml Adds "optional-replace" enum value to all upgrade mode definitions in CRD schema
pkg/apis/deployment/v2alpha1/server_group_spec_upgrade_mode.go Defines the new constant and updates validation logic for v2alpha1
pkg/apis/deployment/v1/server_group_spec_upgrade_mode.go Defines the new constant and updates validation logic for v1
pkg/apis/deployment/v2alpha1/server_group_spec.go Adds documentation for the new upgrade mode in v2alpha1 spec
pkg/apis/deployment/v1/server_group_spec.go Adds documentation for the new upgrade mode in v1 spec
docs/api/ArangoDeployment.V1.md Updates API documentation to include the new upgrade mode option
CHANGELOG.md Records the feature addition in the changelog
@ajanikow ajanikow merged commit 9cd2858 into master Jul 24, 2025
3 checks passed
@ajanikow ajanikow deleted the feature/optional_upgrade_api branch July 24, 2025 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

2 participants