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

az k8s-extension extension-types list failed #5356

Closed
t0rben opened this issue Sep 15, 2022 · 5 comments · Fixed by #5359
Closed

az k8s-extension extension-types list failed #5356

t0rben opened this issue Sep 15, 2022 · 5 comments · Fixed by #5359
Assignees
Labels
AKS bug This issue requires a change to an existing behavior in the product in order to be resolved. customer-reported Issues that are reported by GitHub users external to the Azure organization. CXP Attention This issue is handled by CXP team. extension/aks-preview needs-author-feedback More information is needed from author to address the issue.

Comments

@t0rben
Copy link

t0rben commented Sep 15, 2022

This is autogenerated. Please review and update as needed.

Describe the bug

Command Name
az k8s-extension extension-types list Extension Name: k8s-extension. Version: 1.3.3.

Errors:

The command failed with an unexpected error. Here is the traceback:
cf_k8s_extension() takes 1 positional argument but 2 were given
Traceback (most recent call last):
  File "/opt/homebrew/Cellar/azure-cli/2.40.0/libexec/lib/python3.10/site-packages/knack/cli.py", line 233, in invoke
    cmd_result = self.invocation.execute(args)
  File "/opt/homebrew/Cellar/azure-cli/2.40.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 663, in execute
    raise ex
  File "/opt/homebrew/Cellar/azure-cli/2.40.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 726, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/homebrew/Cellar/azure-cli/2.40.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 697, in _run_job
    result = cmd_copy(params)
  File "/opt/homebrew/Cellar/azure-cli/2.40.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 333, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/homebrew/Cellar/azure-cli/2.40.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/command_operation.py", line 112, in handler
    client = self.client_factory(self.cli_ctx, command_args) if self.client_factory else None
  File "/Users/tbock/.azure/cliextensions/k8s-extension/azext_k8s_extension/_client_factory.py", line 21, in cf_k8s_cluster_extension_types_operation
    return cf_k8s_extension(cli_ctx, consts.EXTENSION_TYPE_API_VERSION).cluster_extension_types
TypeError: cf_k8s_extension() takes 1 positional argument but 2 were given

To Reproduce:

Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

  • Put any pre-requisite steps here...
  • az k8s-extension extension-types list -g {} --cluster-name {} --cluster-type {}

Expected Behavior

Environment Summary

macOS-12.6-arm64-arm-64bit, Darwin 21.6.0
Python 3.10.6
Installer: HOMEBREW

azure-cli 2.40.0

Extensions:
connectedk8s 1.3.3
k8s-extension 1.3.3

Dependencies:
msal 1.18.0b1
azure-mgmt-resource 21.1.0b1

Additional Context

@ghost ghost added question The issue doesn't require a change to the product in order to be resolved. Most issues start as that customer-reported Issues that are reported by GitHub users external to the Azure organization. labels Sep 15, 2022
@t0rben
Copy link
Author

t0rben commented Sep 15, 2022

I've tried it additionally using windows 10 enterprise powershell and latest versions of the azure-cli as well as both extensions and got the same behaviour.

@yonzhan yonzhan added extension/aks-preview AKS CXP Attention This issue is handled by CXP team. and removed question The issue doesn't require a change to the product in order to be resolved. Most issues start as that labels Sep 15, 2022
@ghost
Copy link

ghost commented Sep 15, 2022

Thank you for your feedback. This has been routed to the support team for assistance.

@yonzhan
Copy link
Collaborator

yonzhan commented Sep 15, 2022

route to CXP team

@navba-MSFT navba-MSFT self-assigned this Sep 16, 2022
@navba-MSFT
Copy link
Contributor

@t0rben Thanks for reaching out to us and reporting this issue. We are looking into this issue and we will provide an update.

navba-MSFT added a commit to navba-MSFT/azure-cli-extensions that referenced this issue Sep 16, 2022
CLI Command: az k8s-extension extension-types list -g {} --cluster-name {} --cluster-type {}

Results in below error:
  File "/opt/homebrew/Cellar/azure-cli/2.40.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 726, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/homebrew/Cellar/azure-cli/2.40.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 697, in _run_job
    result = cmd_copy(params)
  File "/opt/homebrew/Cellar/azure-cli/2.40.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 333, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/homebrew/Cellar/azure-cli/2.40.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/command_operation.py", line 112, in handler
    client = self.client_factory(self.cli_ctx, command_args) if self.client_factory else None
  File "/Users/tbock/.azure/cliextensions/k8s-extension/azext_k8s_extension/_client_factory.py", line 21, in cf_k8s_cluster_extension_types_operation
    return cf_k8s_extension(cli_ctx, consts.EXTENSION_TYPE_API_VERSION).cluster_extension_types
TypeError: cf_k8s_extension() takes 1 positional argument but 2 were given

This PR fixes this issue.

fixes Azure#5356
@navba-MSFT navba-MSFT added the bug This issue requires a change to an existing behavior in the product in order to be resolved. label Sep 16, 2022
@navba-MSFT
Copy link
Contributor

@t0rben The above PR has been filed to address this. It is currently under review. Please let me know if you have any questions in the meantime.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AKS bug This issue requires a change to an existing behavior in the product in order to be resolved. customer-reported Issues that are reported by GitHub users external to the Azure organization. CXP Attention This issue is handled by CXP team. extension/aks-preview needs-author-feedback More information is needed from author to address the issue.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants