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

Revert "Fix/revert (#410)" #413

Conversation

Bolodya1997
Copy link

@Bolodya1997 Bolodya1997 commented Oct 14, 2021

This reverts commit d541552.

Description

Changes are working, error was on CI because of not updated cmd-nse-firewall-vpp version.
https://github.com/networkservicemesh/deployments-k8s/blame/ae9f8f8a82b9303beab3d36d606263cf4c21b90a/images.yaml#L8

Valid version should be 68bed4e, because:

  1. PR introducing new version - Update from update/networkservicemesh/sdk-vpp cmd-nse-firewall-vpp#84.
  2. workflow run triggered by this PR - https://github.com/networkservicemesh/cmd-nse-firewall-vpp/runs/3879037037?check_suite_focus=true#step:7:5

Issue

Closes #411.

This reverts commit d541552.

Signed-off-by: Vladimir Popov <vladimir.popov@xored.com>
@denis-tingaikin
Copy link
Member

OK, thanks @Bolodya1997!

We merged reverts yesterday to make CI green on integration tests. Will merge this PR as soon as we get a green build on kind with the latest connect merges from @edwarnicke.

@denis-tingaikin denis-tingaikin merged commit 2d2aa47 into networkservicemesh:main Oct 22, 2021
nsmbot pushed a commit to networkservicemesh/cmd-nse-firewall-vpp that referenced this pull request Oct 22, 2021
…k-vpp@main

PR link: networkservicemesh/sdk-vpp#413

Commit: 2d2aa47
Author: Vladimir Popov
Date: 2021-10-22 16:03:40 +0700
Message:
  - Revert "Fix/revert (#410)" (#413)
This reverts commit d541552ca8a75892013e1a9a269a433c111053bb.

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nse-icmp-responder-vpp that referenced this pull request Oct 22, 2021
…k-vpp@main

PR link: networkservicemesh/sdk-vpp#413

Commit: 2d2aa47
Author: Vladimir Popov
Date: 2021-10-22 16:03:40 +0700
Message:
  - Revert "Fix/revert (#410)" (#413)
This reverts commit d541552ca8a75892013e1a9a269a433c111053bb.

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nsc-vpp that referenced this pull request Oct 22, 2021
…k-vpp@main

PR link: networkservicemesh/sdk-vpp#413

Commit: 2d2aa47
Author: Vladimir Popov
Date: 2021-10-22 16:03:40 +0700
Message:
  - Revert "Fix/revert (#410)" (#413)
This reverts commit d541552ca8a75892013e1a9a269a433c111053bb.

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-forwarder-vpp that referenced this pull request Oct 22, 2021
…k-vpp@main

PR link: networkservicemesh/sdk-vpp#413

Commit: 2d2aa47
Author: Vladimir Popov
Date: 2021-10-22 16:03:40 +0700
Message:
  - Revert "Fix/revert (#410)" (#413)
This reverts commit d541552ca8a75892013e1a9a269a433c111053bb.

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit that referenced this pull request Jan 25, 2022
…k-kernel@main

PR link: networkservicemesh/sdk-kernel#413

Commit: 63d40e5
Author: Denis Tingaikin
Date: 2022-01-25 08:36:13 +0300
Message:
  - Merge pull request #413 from glazychev-art/iprule_add
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
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.

Fix error in proxylistener
2 participants