Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added fallthrough check in matching #1010

Conversation

Mixaster995
Copy link
Contributor

@Mixaster995 Mixaster995 commented Jul 7, 2021

Signed-off-by: Mikhail Avramenko [email protected]

Description

Fix for match selector which enables continuation in case of absent nse candidates

Issue link

part of networkservicemesh/deployments-k8s#2007
blocked by networkservicemesh/api#106

How Has This Been Tested?

  • Added unit testing to cover
  • Tested manually
  • Tested by integration testing
  • Have not tested

Types of changes

  • Bug fix
  • New functionallity
  • Documentation
  • Refactoring
  • CI

Signed-off-by: Mikhail Avramenko <[email protected]>
@denis-tingaikin
Copy link
Member

Please add also unit test to cover the use-case with fallthrough

@Mixaster995 Mixaster995 force-pushed the improvement/match_selection branch from ee89adc to 3922e08 Compare July 8, 2021 02:32
@Mixaster995 Mixaster995 force-pushed the improvement/match_selection branch from 3922e08 to 6892900 Compare July 8, 2021 02:37
@Mixaster995 Mixaster995 marked this pull request as ready for review July 8, 2021 02:42
@Mixaster995 Mixaster995 force-pushed the improvement/match_selection branch from ea5e530 to 0162ebc Compare July 8, 2021 06:32
Signed-off-by: Mikhail Avramenko <[email protected]>
@Mixaster995 Mixaster995 force-pushed the improvement/match_selection branch from 0162ebc to 38746af Compare July 8, 2021 06:36
Signed-off-by: Mikhail Avramenko <[email protected]>
Copy link
Member

@denis-tingaikin denis-tingaikin left a comment

Choose a reason for hiding this comment

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

LGTM

@denis-tingaikin denis-tingaikin merged commit 6d7c5b4 into networkservicemesh:main Jul 8, 2021
@Mixaster995 Mixaster995 deleted the improvement/match_selection branch July 8, 2021 11:36
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