Skip to content

Conversation

gcp-cherry-pick-bot[bot]
Copy link

Cherry-picked fix: match logic misbehave (#7218)

  • add rule name in ur for mutate existing

Signed-off-by: ShutingZhao shuting@nirmata.com

  • fix match logic

Signed-off-by: ShutingZhao shuting@nirmata.com

  • linter fixes

Signed-off-by: ShutingZhao shuting@nirmata.com

  • fix the match logic to only apply to the new object, unless it's a delete request

Signed-off-by: ShutingZhao shuting@nirmata.com

  • fix unit tests

Signed-off-by: ShutingZhao shuting@nirmata.com


Signed-off-by: ShutingZhao shuting@nirmata.com

* add rule name in ur for mutate existing

Signed-off-by: ShutingZhao <shuting@nirmata.com>

* fix match logic

Signed-off-by: ShutingZhao <shuting@nirmata.com>

* linter fixes

Signed-off-by: ShutingZhao <shuting@nirmata.com>

* fix the match logic to only apply to the new object, unless it's a delete request

Signed-off-by: ShutingZhao <shuting@nirmata.com>

* fix unit tests

Signed-off-by: ShutingZhao <shuting@nirmata.com>

---------

Signed-off-by: ShutingZhao <shuting@nirmata.com>
@realshuting realshuting enabled auto-merge (squash) May 18, 2023 21:59
@codecov
Copy link

codecov bot commented May 18, 2023

Codecov Report

Merging #7237 (7d4af74) into release-1.10 (d50e503) will increase coverage by 0.00%.
The diff coverage is 4.00%.

@@              Coverage Diff              @@
##           release-1.10    #7237   +/-   ##
=============================================
  Coverage         34.54%   34.55%           
=============================================
  Files               219      219           
  Lines             21457    21461    +4     
=============================================
+ Hits               7412     7415    +3     
- Misses            13322    13324    +2     
+ Partials            723      722    -1     
Impacted Files Coverage Δ
pkg/webhooks/resource/utils.go 14.03% <0.00%> (-1.06%) ⬇️
pkg/engine/engine.go 74.09% <100.00%> (+1.80%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@realshuting realshuting merged commit f161a5e into release-1.10 May 18, 2023
@realshuting realshuting deleted the cherry-pick-1f91fb-release-1.10 branch May 18, 2023 22:31
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.

1 participant