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 hack create' Issue encountered running lab @ https://github.com/microsoft/Reactors/tree/master/AI_1 #1653

Merged
merged 1 commit into from
Jan 5, 2017

Conversation

troydai
Copy link
Contributor

@troydai troydai commented Jan 5, 2017

This is autogenerated. Please review and update as needed.

Describe the bug

Command Name
az hack create

Errors:

invalid syntax (_database_utils.py, line 30)
Traceback (most recent call last):
python2.7/site-packages/knack/cli.py, ln 206, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 528, in execute
    self.commands_loader.load_arguments(command)
azure/cli/core/__init__.py, ln 299, in load_arguments
    self.command_table[command].load_arguments()  # this loads the arguments via reflection
cli/core/commands/__init__.py, ln 291, in load_arguments
    super(AzCliCommand, self).load_arguments()
...
cliextensions/hack/azext_hack/custom.py, ln 12, in <module>
    from ._database_utils import Database
cliextensions/hack/azext_hack/_database_utils.py, ln 30
    def __init__(self, name: str, delegate, params):
                           ^
SyntaxError: invalid syntax

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 hack create --name {} --runtime {} --location {} -a{} --output {}

Expected Behavior

Environment Summary

Linux-4.4.0-18362-Microsoft-x86_64-with-Ubuntu-18.04-bionic
Python 2.7.15+
Shell: bash

azure-cli 2.0.81

Extensions:
hack 0.4.1

Additional Context

Copy link
Member

@derekbekoe derekbekoe left a comment

Choose a reason for hiding this comment

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

LGTM

@troydai
Copy link
Contributor Author

troydai commented Jan 5, 2017

I will merge once the CI run on my own repo's queue passes.

@troydai troydai merged commit 5c836cf into Azure:master Jan 5, 2017
@troydai troydai deleted the troy-ci branch January 5, 2017 19:52
thegalah pushed a commit to thegalah/azure-cli that referenced this pull request Jan 6, 2017
* Azure/master:
  Fix PEP8 for azure-cli (Azure#1661)
  [VM/VMSS] Enable Multi-Cloud VM/VMSS Create (Azure#1654)
  Fix blob type validator (Azure#1656)
  Improve @file handling logic (Azure#1648)
  Name private_config.json differently in VM test (Azure#1653)
  Update regex of parsing sdk function argument comments (Azure#1650)
  Reorganize test run in automation (Azure#1624)
  Fix issue 1644. (Azure#1647)
  [Network] VNet Gateway Fixes and Enhancements (Azure#1643)
  [Network] Application Gateway Commands and Fixes (Azure#1606)
  [Docs] Correcting unittest location in docs (Azure#1642)
  [ACS] Add a table transform for acs show to match acs list. (Azure#1641)
  [ACS] Add a unit test and fix a bug with az browse ... (Azure#1640)
  [Component] Fix pip seg fault on 'az component update' (Azure#1639)
  fixed typo (Azure#1634)
@azure-sdk azure-sdk changed the title Name private_config.json differently in VM test 'az hack create' Issue encountered running lab @ https://github.com/microsoft/Reactors/tree/master/AI_1 Sep 24, 2020
@azure-sdk azure-sdk added customer-reported Issues that are reported by GitHub users external to the Azure organization. Service Attention This issue is responsible by Azure service team. extension/hack and removed cla-not-required labels Sep 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
customer-reported Issues that are reported by GitHub users external to the Azure organization. extension/hack Service Attention This issue is responsible by Azure service team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants