Skip to content

Improve patch type to randomized resource name#199

Merged
xinWeiWei24 merged 4 commits into
mainfrom
improve_patch/vitto
Aug 4, 2025
Merged

Improve patch type to randomized resource name#199
xinWeiWei24 merged 4 commits into
mainfrom
improve_patch/vitto

Conversation

@vittoriasalim
Copy link
Copy Markdown
Contributor

@vittoriasalim vittoriasalim commented Aug 4, 2025

This PR improves the patch operation by supporting dynamic resource names through a randomized suffix. Users can specify a name pattern (e.g., benchmark-{suffix}), and a suffix will be generated based on the defined keySpaceSize.

Example Usage

  - patch:
      version: v1
      resource: configmaps
      namespace: default
      patchType: merge
      name: benchmark # name pattern
      keySpaceSize: 100 # randomize suffix
      body: |
        {
          "metadata": {
            "labels": {
              "test-label": "mutation-test"
            }
          }
        }
    shares: 100

@vittoriasalim vittoriasalim requested a review from fuweid August 4, 2025 05:24
@vittoriasalim vittoriasalim marked this pull request as ready for review August 4, 2025 05:26
@vittoriasalim
Copy link
Copy Markdown
Contributor Author

vittoriasalim commented Aug 4, 2025

TODO: aggregate patch and delete url result

Copy link
Copy Markdown
Contributor

@xinWeiWei24 xinWeiWei24 left a comment

Choose a reason for hiding this comment

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

lgtm

@xinWeiWei24 xinWeiWei24 merged commit 001207b into main Aug 4, 2025
4 checks passed
@vittoriasalim vittoriasalim deleted the improve_patch/vitto branch August 6, 2025 04:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants