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

Create a keyvault fails in cloud shell #7839

Closed
suhuruli opened this issue Nov 13, 2018 · 2 comments
Closed

Create a keyvault fails in cloud shell #7839

suhuruli opened this issue Nov 13, 2018 · 2 comments
Assignees
Labels
bug This issue requires a change to an existing behavior in the product in order to be resolved. KeyVault az keyvault

Comments

@suhuruli
Copy link

Describe the bug
When I try to do az sf cluster create in Cloud Shell, it fails at the Create a KeyVault stage with the following error:

Found primary certificate parameters in parameters file
Creating key vault
'ObjectsOperations' object has no attribute 'get_current_user'
Traceback (most recent call last):
File "/opt/az/lib/python3.6/site-packages/knack/cli.py", line 197, in invoke
cmd_result = self.invocation.execute(args)
File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/init.py", line 373, in execute
six.reraise(*sys.exc_info())
File "/opt/az/lib/python3.6/site-packages/six.py", line 693, in reraise
raise value
File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/init.py", line 347, in execute
result = cmd(params)
File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/init.py", line 182, in call
return self.handler(*args, **kwargs)
File "/opt/az/lib/python3.6/site-packages/azure/cli/core/init.py", line 440, in default_command_handler
result = op(**command_args)
File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/servicefabric/custom.py", line 238, in new_cluster
parameter_file)
File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/servicefabric/custom.py", line 1822, in _set_parameters_for_customize_template
secret_identifier)
File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/servicefabric/custom.py", line 1140, in _create_certificate
cmd, cli_ctx, vault_resource_group_name, vault_name, location, enabled_for_deployment=True).result()
File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/servicefabric/custom.py", line 1664, in _create_keyvault
object_id = _get_current_user_object_id(graph_client)
File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/servicefabric/custom.py", line 1695, in _get_current_user_object_id
current_user = graph_client.objects.get_current_user()
AttributeError: 'ObjectsOperations' object has no attribute 'get_current_user'

To Reproduce

  1. Open Bash instance for Cloud Shell and run the following command:

az sf cluster create --resource-group demolinuxsecure --location westus --certificate-output-folder . --certificate-password --certificate-subject-name demolinuxsecure0808.westus.cloudapp.azure.com --vault-name demolinuxsecure0808 --vault-resource-group demolinuxsecure --template-file AzureDeploy.json --parameter-file AzureDeploy.Parameters.json

using the template here: https://github.com/suhuruli/service-fabric-cluster-templates/tree/master/10-VM-Ubuntu-2-NodeType-Secure

Expected behavior
I expect the deployment to work since the exact same steps work on my local Mac machine.

Environment summary
Cloud Shell.

@yugangw-msft
Copy link
Contributor

if you run az account show, what do you get?

@suhuruli
Copy link
Author

suhuruli commented Nov 13, 2018

Edit: I fixed the formatting.

When I run az account show I get the following output:
{
"environmentName": "AzureCloud",
"id": "1111111-22222-3333333",
"isDefault": true,
"name": "Demo",
"state": "Enabled",
"tenantId": "1111111-22222-333333",
"user": {
"name": "[email protected]",
"type": "user"
}
}

I also just confirmed that on my Mac, the version of CLI I was using is 2.0.45 and on Cloud Shell it is 2.0.50. When I upgraded my Mac to the latest (2.0.50), it is also breaking there now.

@tjprescott tjprescott added bug This issue requires a change to an existing behavior in the product in order to be resolved. KeyVault az keyvault labels Nov 14, 2018
a-santamaria pushed a commit to a-santamaria/azure-cli that referenced this issue Nov 16, 2018
williexu pushed a commit to williexu/azure-cli that referenced this issue Dec 4, 2018
williexu pushed a commit that referenced this issue Dec 6, 2018
* fix live tests for sas token urls to use datetime (#7866)

* Convert test back to scenario test. (#7862)

* Remove unused scripts. (#7860)

* fix get curren user id azure-graphrbac Breaking change fixes #7839 (#7864)

* Use knack 0.5.1. (#7780)

* test: make webjob tests record only as create command is unavailable (#7870)

* Deprecate expand param. (#7868)

* Updated help.py - generate-sas example (#7873)

Added MacOS example for generating SAS token with expiration time. The call to the date function `date -d` works on Linux, but not on MacOS. I provided MacOS sample.

* Low priority vmss test and help update. (#7882)

Updated help text and test location for low priority vmss

* Added --force to az vm restart. Added simple test. (#7885)

* Fix requests version > 2.20 (#7858)

* Bump CLI versions

* [vm] Enable password and ssh key authentication during linux VM creation (#7863)

* authentication-type can be set to all to permit both ssh key and password authentication

* Added tests and updated authentication type validation error.

* Added history entry, bumped version. Checked out file recording from dev branch.

* Addressed PEP8 issues.

* Updated test_vm_defaults auth tests.

* Updated other profile's unittests.

* Now infers authentication_type, other updates. Do not need to explicitly pass  all.

* Updated help text.
Fix conflict by checking out ephemeral os disk recording from dev branch.

* Style changes.

* Properly handle parsing of ids from -o tsv on windows. (#7898)

* Properly handle parsing of ids from -o tsv on windows.

* Removed unused import.

* [Storage] bump mgmt sdk version to 3.1.0 and set new api-version to use. (#7869)

* changed api-version in recordings and set to new ver of sdk

* version bump and history

* fix immutability test

* re-record all

* fixed recordings

fix recording

* bumped version after release

* test: live test fixes in appservice, vm, resource (#7899)

* ci: fix build break (#7900)

* Updated adls version (#7859)

* core: support cross tenant resource provisioning for multi-tenant service principal (#7916)

* core: fix the error on matching SP creds (#7923)

* resource: fix live test failure of policyset (#7922)

* [CI] reenable extension checking in CI (#7933)

* reenable extension checking in CI

minor edit to get travis to run

edit

* minor edit

* BotService: Bugfixing, code refactoring, reorganization and UX revamping (#7924)

* Close #7528. (#7920)

* Minor updates to core/generate_ssh_keys and batchai/_generate_ssh_keys (#7895)

* [image] create. Add --os-disk-caching parameter. (#7919)

* graph: support custom key identifier (#7913)

* support key-id description

* linter error fixes

* consolidate

* use official sdk

* update history

* BotService: fix tests, improve test robustness and update recordings (#7941)

* BotService: fix tests, mprove test robustness and update recordings

* BotService; fixing tests

* BotService: Update recordings

* webapp: Adding support for az webapp up (#7930)

* core: fix a breaking in kv when run under service principal (#7946)

* Disable version check from CI. (#7958)

* test: add test coverage for regression from auth (#7949)

* core: fix a breaking in kv when run under service principal

* test: add test coverage for regression from auth

* [AKS] Remove "(PREVIEW)" from AAD arguments  (#7960)

* [AKS] Remove "(PREVIEW)" from AAD arguments

* Bump module version and update HISTORY.rst

* Bug fixes for azure-cli-botservice (#7956)

* Az bot: fix bug when publishing Node.js bots, fix minor bugs

* Az bot: rerecord BotTests

* Az bot: fix static check error

* [core] Updated location help text (#7951)

* Updated location help text.

* Update parameters.py

* EventHubs help fixes (#7961)

* EventHubs help fixes

#Fixes 7937. Fixes #7938.

* Update _help.py

* Fixes for VPN Client Generate (#7962)

* Remove mock dependency.

* Fix linter issues.

* Condense the table format for task list and change all column headers to NAME
@haroldrandom haroldrandom added bug This issue requires a change to an existing behavior in the product in order to be resolved. KeyVault az keyvault labels Oct 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue requires a change to an existing behavior in the product in order to be resolved. KeyVault az keyvault
Projects
None yet
Development

No branches or pull requests

5 participants