Skip to content

Conversation

@xmudrii
Copy link
Member

@xmudrii xmudrii commented Nov 8, 2025

As it says in the title, this is required to find the proper set of rules to copy when creating rules for a new branch.

The testdata contained rules for kubernetes/kubernetes which are quite large in size and are based on the master branch. I changed testdata with our rules which are based on the main branch. There are less rules, but it's good enough for what are tests covering.

/assign @xrstf @embik

@kcp-ci-bot kcp-ci-bot added dco-signoff: yes Indicates the PR's author has signed the DCO. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Nov 8, 2025
On-behalf-of: @SAP marko.mudrinic@sap.com
Signed-off-by: Marko Mudrinić <mudrinic.mare@gmail.com>
@kcp-ci-bot kcp-ci-bot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Nov 8, 2025
@embik
Copy link
Member

embik commented Nov 9, 2025

What's going on with the massive drop of testdata, can we quickly add a note in the PR description on that?

Copy link
Member

@embik embik left a comment

Choose a reason for hiding this comment

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

/approve

@kcp-ci-bot kcp-ci-bot added the lgtm Indicates that a PR is ready to be merged. label Nov 10, 2025
@kcp-ci-bot
Copy link

LGTM label has been added.

Git tree hash: 7891843962832bd3ae297bafdbcd07e466f2d7f1

@kcp-ci-bot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: embik

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kcp-ci-bot kcp-ci-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 10, 2025
@kcp-ci-bot kcp-ci-bot merged commit 5e3abb8 into kcp-dev:master Nov 10, 2025
6 checks passed
@xmudrii xmudrii deleted the fix-update-rules branch November 10, 2025 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Indicates the PR's author has signed the DCO. lgtm Indicates that a PR is ready to be merged. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants