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 sql server ad-admin create gives error #12640

Closed
Raka74 opened this issue Mar 18, 2020 · 6 comments
Closed

az sql server ad-admin create gives error #12640

Raka74 opened this issue Mar 18, 2020 · 6 comments
Labels
Service Attention This issue is responsible by Azure service team. SQL az sql

Comments

@Raka74
Copy link

Raka74 commented Mar 18, 2020

This is autogenerated. Please review and update as needed.

Describe the bug

az sql server ad-admin create -g {} -s {} -u {} -i {}` - doesnt work , below error is displayed.

Command Name
az sql server ad-admin create

Errors:

create_or_update() missing 1 required positional argument: 'parameters'
Traceback (most recent call last):
python3.6/site-packages/knack/cli.py, ln 206, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 608, in execute
    raise ex
cli/core/commands/__init__.py, ln 666, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
cli/core/commands/__init__.py, ln 659, in _run_job
    six.reraise(*sys.exc_info())
...
cli/core/commands/__init__.py, ln 306, in __call__
    return self.handler(*args, **kwargs)
azure/cli/core/__init__.py, ln 493, in default_command_handler
    return op(**command_args)
cli/command_modules/sql/custom.py, ln 2074, in server_ad_admin_set
    properties=kwargs)
TypeError: create_or_update() missing 1 required positional argument: 'parameters'

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 sql server ad-admin create -g {} -s {} -u {} -i {}

Expected Behavior

Environment Summary

Linux-4.15.0-1071-azure-x86_64-with-debian-stretch-sid
Python 3.6.5

azure-cli 2.2.0

Extensions:
storage-preview 0.2.8

Additional Context

@yonzhan yonzhan added the SQL az sql label Mar 18, 2020
@yonzhan yonzhan added Service Attention This issue is responsible by Azure service team. triage labels Mar 18, 2020
@ghost
Copy link

ghost commented Mar 18, 2020

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @azureSQLGitHub.

@yonzhan
Copy link
Collaborator

yonzhan commented Mar 18, 2020

sql

@skwasiborski
Copy link

skwasiborski commented Mar 18, 2020

The issue seems to be caused by wrong argument name in /azure/cli/command_modules/sql/custom.py
line 2074 should be:
parameters=kwargs instead of the current properties=kwargs
And it seems to be fixed by this commit.

@jaredmoo
Copy link
Member

Fixed by #12541 . Thank you!

@venkatrajupalepu
Copy link

I'm facing the same issue. Is this in production?

@jaredmoo
Copy link
Member

No, it's not released yet :( next release is https://github.com/Azure/azure-cli/milestone/84

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Service Attention This issue is responsible by Azure service team. SQL az sql
Projects
None yet
Development

No branches or pull requests

5 participants