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

Feature/calico integration #4143

Merged

Conversation

Mixaster995
Copy link
Contributor

@Mixaster995 Mixaster995 commented Dec 21, 2021

Description

Added calico patches

Issue

part of networkservicemesh/integration-k8s-kind#325

Signed-off-by: Mikhail Avramenko <[email protected]>
@Mixaster995 Mixaster995 force-pushed the feature/calico-integration branch from 05384ea to e2dfaaa Compare December 21, 2021 12:20
Signed-off-by: Mikhail Avramenko <[email protected]>
@@ -31,7 +31,11 @@ kubectl create ns nsm-system
2. Apply NSM resources for basic tests:

```bash
kubectl apply -k https://github.com/networkservicemesh/deployments-k8s/examples/basic?ref=bbe46ecf104280e96ae3a32d4757149e792f0d11
if [[ "${CALICO}" == "on" ]]; then
Copy link
Member

@denis-tingaikin denis-tingaikin Dec 22, 2021

Choose a reason for hiding this comment

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

Please don't use env. As I can see we can simply create a separate suite(setup) for calico and add links at supported use-cases.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@denis-tingaikin ok, refactored example structure, please, take a look

@Mixaster995 Mixaster995 force-pushed the feature/calico-integration branch 5 times, most recently from 753366d to 0a3335b Compare December 23, 2021 10:46
Signed-off-by: Mikhail Avramenko <[email protected]>
@Mixaster995 Mixaster995 force-pushed the feature/calico-integration branch from 0a3335b to 42109f6 Compare December 23, 2021 10:48
@Mixaster995
Copy link
Contributor Author

@denis-tingaikin refactored folder structure - please, take a look

@denis-tingaikin denis-tingaikin merged commit 16f8d83 into networkservicemesh:main Dec 23, 2021
nsmbot pushed a commit to networkservicemesh/integration-tests that referenced this pull request Dec 23, 2021
…ployments-k8s@main

PR link: networkservicemesh/deployments-k8s#4143

Commit: 7dc61bf
Author: Network Service Mesh Bot
Date: 2021-12-23 04:55:22 -0600
Message:
  - Update go.mod and go.sum to latest version from networkservicemesh/de…
Signed-off-by: NSMBot <[email protected]>
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.

2 participants