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 changes in begin and dial #1682

Merged

Conversation

NikitaSkrynnik
Copy link
Contributor

Description

Revert PRs:

  1. Use a context with exteded timeout on Requests in begin #1656
  2. Add a timeout for Closes in begin.Server #1650
  3. Add more mutexes in dial chain element to fix race conditions #1670

because netlink is fixed now and doesn't cause inteface leaks: networkservicemesh/sdk-kernel#686

Issue link

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 functionality
  • Documentation
  • Refactoring
  • CI

…orkservicemesh#1656)"

This reverts commit 6fad31a.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>
…1650)"

This reverts commit 3016313.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>
…networkservicemesh#1670)"

This reverts commit b66e1bf.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>
Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>
@denis-tingaikin denis-tingaikin merged commit 083d4e2 into networkservicemesh:main Oct 15, 2024
15 checks passed
nsmbot pushed a commit to networkservicemesh/cmd-csi-driver that referenced this pull request Oct 15, 2024
…k@main

PR link: networkservicemesh/sdk#1682

Commit: 083d4e2
Author: Nikita Skrynnik
Date: 2024-10-15 16:56:31 +0700
Message:
  - Revert changes in `begin` and `dial` (#1682)
* Revert "Use a context with exteded timeout on Requests in begin (#1656)"

This reverts commit 6fad31a9f0aa1e2b927d93da26423cd20aa354eb.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add a timeout for Closes in begin.Server (#1650)"

This reverts commit 301631365421a9d916a5ab7224fa9fbd847320b2.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add more mutexes in dial chain element to fix race conditions (#1670)"

This reverts commit b66e1bf47abb91cdf494b5eef05fda13c6d36b71.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

---------

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-ipam-vl3 that referenced this pull request Oct 15, 2024
…k@main

PR link: networkservicemesh/sdk#1682

Commit: 083d4e2
Author: Nikita Skrynnik
Date: 2024-10-15 16:56:31 +0700
Message:
  - Revert changes in `begin` and `dial` (#1682)
* Revert "Use a context with exteded timeout on Requests in begin (#1656)"

This reverts commit 6fad31a9f0aa1e2b927d93da26423cd20aa354eb.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add a timeout for Closes in begin.Server (#1650)"

This reverts commit 301631365421a9d916a5ab7224fa9fbd847320b2.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add more mutexes in dial chain element to fix race conditions (#1670)"

This reverts commit b66e1bf47abb91cdf494b5eef05fda13c6d36b71.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

---------

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nse-vfio that referenced this pull request Oct 15, 2024
…k@main

PR link: networkservicemesh/sdk#1682

Commit: 083d4e2
Author: Nikita Skrynnik
Date: 2024-10-15 16:56:31 +0700
Message:
  - Revert changes in `begin` and `dial` (#1682)
* Revert "Use a context with exteded timeout on Requests in begin (#1656)"

This reverts commit 6fad31a9f0aa1e2b927d93da26423cd20aa354eb.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add a timeout for Closes in begin.Server (#1650)"

This reverts commit 301631365421a9d916a5ab7224fa9fbd847320b2.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add more mutexes in dial chain element to fix race conditions (#1670)"

This reverts commit b66e1bf47abb91cdf494b5eef05fda13c6d36b71.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

---------

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nse-remote-vlan that referenced this pull request Oct 15, 2024
…k@main

PR link: networkservicemesh/sdk#1682

Commit: 083d4e2
Author: Nikita Skrynnik
Date: 2024-10-15 16:56:31 +0700
Message:
  - Revert changes in `begin` and `dial` (#1682)
* Revert "Use a context with exteded timeout on Requests in begin (#1656)"

This reverts commit 6fad31a9f0aa1e2b927d93da26423cd20aa354eb.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add a timeout for Closes in begin.Server (#1650)"

This reverts commit 301631365421a9d916a5ab7224fa9fbd847320b2.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add more mutexes in dial chain element to fix race conditions (#1670)"

This reverts commit b66e1bf47abb91cdf494b5eef05fda13c6d36b71.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

---------

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

PR link: networkservicemesh/sdk#1682

Commit: 083d4e2
Author: Nikita Skrynnik
Date: 2024-10-15 16:56:31 +0700
Message:
  - Revert changes in `begin` and `dial` (#1682)
* Revert "Use a context with exteded timeout on Requests in begin (#1656)"

This reverts commit 6fad31a9f0aa1e2b927d93da26423cd20aa354eb.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add a timeout for Closes in begin.Server (#1650)"

This reverts commit 301631365421a9d916a5ab7224fa9fbd847320b2.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add more mutexes in dial chain element to fix race conditions (#1670)"

This reverts commit b66e1bf47abb91cdf494b5eef05fda13c6d36b71.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

---------

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-registry-memory that referenced this pull request Oct 15, 2024
…k@main

PR link: networkservicemesh/sdk#1682

Commit: 083d4e2
Author: Nikita Skrynnik
Date: 2024-10-15 16:56:31 +0700
Message:
  - Revert changes in `begin` and `dial` (#1682)
* Revert "Use a context with exteded timeout on Requests in begin (#1656)"

This reverts commit 6fad31a9f0aa1e2b927d93da26423cd20aa354eb.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add a timeout for Closes in begin.Server (#1650)"

This reverts commit 301631365421a9d916a5ab7224fa9fbd847320b2.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add more mutexes in dial chain element to fix race conditions (#1670)"

This reverts commit b66e1bf47abb91cdf494b5eef05fda13c6d36b71.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

---------

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nsmgr-proxy that referenced this pull request Oct 15, 2024
…k@main

PR link: networkservicemesh/sdk#1682

Commit: 083d4e2
Author: Nikita Skrynnik
Date: 2024-10-15 16:56:31 +0700
Message:
  - Revert changes in `begin` and `dial` (#1682)
* Revert "Use a context with exteded timeout on Requests in begin (#1656)"

This reverts commit 6fad31a9f0aa1e2b927d93da26423cd20aa354eb.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add a timeout for Closes in begin.Server (#1650)"

This reverts commit 301631365421a9d916a5ab7224fa9fbd847320b2.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add more mutexes in dial chain element to fix race conditions (#1670)"

This reverts commit b66e1bf47abb91cdf494b5eef05fda13c6d36b71.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

---------

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-registry-proxy-dns that referenced this pull request Oct 15, 2024
…k@main

PR link: networkservicemesh/sdk#1682

Commit: 083d4e2
Author: Nikita Skrynnik
Date: 2024-10-15 16:56:31 +0700
Message:
  - Revert changes in `begin` and `dial` (#1682)
* Revert "Use a context with exteded timeout on Requests in begin (#1656)"

This reverts commit 6fad31a9f0aa1e2b927d93da26423cd20aa354eb.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add a timeout for Closes in begin.Server (#1650)"

This reverts commit 301631365421a9d916a5ab7224fa9fbd847320b2.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add more mutexes in dial chain element to fix race conditions (#1670)"

This reverts commit b66e1bf47abb91cdf494b5eef05fda13c6d36b71.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

---------

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-cluster-info-k8s that referenced this pull request Oct 15, 2024
…k@main

PR link: networkservicemesh/sdk#1682

Commit: 083d4e2
Author: Nikita Skrynnik
Date: 2024-10-15 16:56:31 +0700
Message:
  - Revert changes in `begin` and `dial` (#1682)
* Revert "Use a context with exteded timeout on Requests in begin (#1656)"

This reverts commit 6fad31a9f0aa1e2b927d93da26423cd20aa354eb.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add a timeout for Closes in begin.Server (#1650)"

This reverts commit 301631365421a9d916a5ab7224fa9fbd847320b2.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add more mutexes in dial chain element to fix race conditions (#1670)"

This reverts commit b66e1bf47abb91cdf494b5eef05fda13c6d36b71.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

---------

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/sdk-k8s that referenced this pull request Oct 15, 2024
…k@main

PR link: networkservicemesh/sdk#1682

Commit: 083d4e2
Author: Nikita Skrynnik
Date: 2024-10-15 16:56:31 +0700
Message:
  - Revert changes in `begin` and `dial` (#1682)
* Revert "Use a context with exteded timeout on Requests in begin (#1656)"

This reverts commit 6fad31a9f0aa1e2b927d93da26423cd20aa354eb.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add a timeout for Closes in begin.Server (#1650)"

This reverts commit 301631365421a9d916a5ab7224fa9fbd847320b2.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add more mutexes in dial chain element to fix race conditions (#1670)"

This reverts commit b66e1bf47abb91cdf494b5eef05fda13c6d36b71.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

---------

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-map-ip-k8s that referenced this pull request Oct 15, 2024
…k@main

PR link: networkservicemesh/sdk#1682

Commit: 083d4e2
Author: Nikita Skrynnik
Date: 2024-10-15 16:56:31 +0700
Message:
  - Revert changes in `begin` and `dial` (#1682)
* Revert "Use a context with exteded timeout on Requests in begin (#1656)"

This reverts commit 6fad31a9f0aa1e2b927d93da26423cd20aa354eb.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add a timeout for Closes in begin.Server (#1650)"

This reverts commit 301631365421a9d916a5ab7224fa9fbd847320b2.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add more mutexes in dial chain element to fix race conditions (#1670)"

This reverts commit b66e1bf47abb91cdf494b5eef05fda13c6d36b71.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

---------

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/sdk-kernel that referenced this pull request Oct 15, 2024
…k@main

PR link: networkservicemesh/sdk#1682

Commit: 083d4e2
Author: Nikita Skrynnik
Date: 2024-10-15 16:56:31 +0700
Message:
  - Revert changes in `begin` and `dial` (#1682)
* Revert "Use a context with exteded timeout on Requests in begin (#1656)"

This reverts commit 6fad31a9f0aa1e2b927d93da26423cd20aa354eb.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add a timeout for Closes in begin.Server (#1650)"

This reverts commit 301631365421a9d916a5ab7224fa9fbd847320b2.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add more mutexes in dial chain element to fix race conditions (#1670)"

This reverts commit b66e1bf47abb91cdf494b5eef05fda13c6d36b71.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

---------

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-dashboard-backend that referenced this pull request Oct 15, 2024
…k@main

PR link: networkservicemesh/sdk#1682

Commit: 083d4e2
Author: Nikita Skrynnik
Date: 2024-10-15 16:56:31 +0700
Message:
  - Revert changes in `begin` and `dial` (#1682)
* Revert "Use a context with exteded timeout on Requests in begin (#1656)"

This reverts commit 6fad31a9f0aa1e2b927d93da26423cd20aa354eb.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add a timeout for Closes in begin.Server (#1650)"

This reverts commit 301631365421a9d916a5ab7224fa9fbd847320b2.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* Revert "Add more mutexes in dial chain element to fix race conditions (#1670)"

This reverts commit b66e1bf47abb91cdf494b5eef05fda13c6d36b71.

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

* fix linter issues

Signed-off-by: NikitaSkrynnik <nikita.skrynnik@xored.com>

---------

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.

2 participants