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

Cover cmd-exclude-prefixes-k8s by integration tests #11

Closed
denis-tingaikin opened this issue Sep 8, 2020 · 2 comments
Closed

Cover cmd-exclude-prefixes-k8s by integration tests #11

denis-tingaikin opened this issue Sep 8, 2020 · 2 comments
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@denis-tingaikin
Copy link
Member

denis-tingaikin commented Sep 8, 2020

Description

We've completed cmd-exclude-prefixes-k8s and now we can cover the application by integration tests.

Test scenarios

We need to cover the next scenarios:

  1. Verify that cmd-exclude-prefixes-k8s can correctly read cluster prefixes from kube-system configmap.
  2. Verify that cmd-exclude-prefixes-k8s can correctly collect prefixes if configmap from scenario1 is missed.
  3. Verify that cmd-exclude-prefixes-k8s can correctly read user's prefixes
    3.1. Verify deleting the user's prefixes.
    3.2. Verify updating the user's prefixes.
    3.3. Verify adding the user's prefixes.
  4. Verify that cmd-exclude-prefixes-k8s can correctly read prefixes from envs.
    4.1. Verify that cmd-exclude-prefixes-k8s is working fine with correct prefixes from envs.
    4.2. Verify that cmd-exclude-prefixes-k8s is failing with incorrect prefixes from envs.
  5. Verify collision prefixes collision cases.

Testing notes

We have not completed tests for nsmgr + nsc yet, and for first step we can collect prefixes into alpine pod via shared volume. And check file source via kubectl cp, kubectl exec -ti "pod name" -- cat "path".

@denis-tingaikin
Copy link
Member Author

@tigrulya-exe cc

@Mixaster995
Copy link
Contributor

Mixaster995 commented Dec 14, 2021

closed in favor of networkservicemesh/deployments-k8s#1353

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants