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

Improved describe method #1103

Conversation

Mixaster995
Copy link
Contributor

Signed-off-by: Mikhail Avramenko avramenkomihail15@gmail.com

@Mixaster995 Mixaster995 changed the title improved describe method Improved describe method Nov 22, 2021
@Mixaster995 Mixaster995 self-assigned this Nov 23, 2021
Signed-off-by: Mikhail Avramenko <avramenkomihail15@gmail.com>
@Mixaster995 Mixaster995 force-pushed the enhancement/improve-describe branch from b4cbdc5 to 20b5fb9 Compare November 30, 2021 08:10
}
}

func getNamespaces(nsList *corev1.NamespaceList) []string {
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
func getNamespaces(nsList *corev1.NamespaceList) []string {
func filterNamespaces(nsList *corev1.NamespaceList) []string {

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok, done

Signed-off-by: Mikhail Avramenko <avramenkomihail15@gmail.com>
@denis-tingaikin denis-tingaikin merged commit d8b8af7 into networkservicemesh:main Dec 2, 2021
nsmbot pushed a commit to networkservicemesh/integration-k8s-kind that referenced this pull request Dec 2, 2021
…tegration-tests@main

PR link: networkservicemesh/integration-tests#1103

Commit: d8b8af7
Author: Denis Tingaikin
Date: 2021-12-03 02:09:01 +0300
Message:
  - Merge pull request #1103 from Mixaster995/enhancement/improve-describe
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
@Mixaster995 Mixaster995 deleted the enhancement/improve-describe branch December 3, 2021 02:38
nsmbot pushed a commit that referenced this pull request Dec 3, 2021
…ployments-k8s@main

PR link: https://github.com/networkservicemesh/deployments-k8s/pull/

Commit: d8b8af7
Author: Denis Tingaikin
Date: 2021-12-03 02:09:01 +0300
Message:
  - Merge pull request #1103 from Mixaster995/enhancement/improve-describe
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit that referenced this pull request Dec 3, 2021
…ployments-k8s@main

PR link: networkservicemesh/deployments-k8s#3974

Commit: d8b8af7
Author: Denis Tingaikin
Date: 2021-12-03 02:09:01 +0300
Message:
  - Merge pull request #1103 from Mixaster995/enhancement/improve-describe
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit that referenced this pull request Dec 3, 2021
…ployments-k8s@main

PR link: networkservicemesh/deployments-k8s#3976

Commit: d8b8af7
Author: Denis Tingaikin
Date: 2021-12-03 02:09:01 +0300
Message:
  - Merge pull request #1103 from Mixaster995/enhancement/improve-describe
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit that referenced this pull request Dec 3, 2021
…ployments-k8s@main

PR link: https://github.com/networkservicemesh/deployments-k8s/pull/

Commit: d8b8af7
Author: Denis Tingaikin
Date: 2021-12-03 02:09:01 +0300
Message:
  - Merge pull request #1103 from Mixaster995/enhancement/improve-describe
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit that referenced this pull request Dec 3, 2021
…ployments-k8s@main

PR link: https://github.com/networkservicemesh/deployments-k8s/pull/

Commit: d8b8af7
Author: Denis Tingaikin
Date: 2021-12-03 02:09:01 +0300
Message:
  - Merge pull request #1103 from Mixaster995/enhancement/improve-describe
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