Skip to content

Conversation

@jaysterp
Copy link


This checklist is used to make sure that common guidelines for a pull request are followed.

  • The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

  • I adhere to the Command Guidelines.

tjprescott and others added 30 commits October 15, 2018 16:30
* Updated adls version to 0.0.33

* Updated version

* Update version

* Downgrade to 0.0.32 due to test failures

* Updated test recordings with new version and updated to version 0.0.34 again

* Updated History.rst
* Adding vnet-resource-group

* fixing wording for more clear content

* Removing vnet-resource-group

* allow passing in vnet name or id

* updating container module version

* adding wording change for the LRP in container warning.

* reverting version back to current version

* Adding private option for --ip-address

* updating tests for new parameters

* updating tests recordings

* Removing the ability to create VNET by passing the resource ID

* adding --vnet-name back for backwards compatability

* Fixing based on feedback

* Bumping the version for CItests

* updating container instance sdk version

* changing casing and removing existance check

* fixing no image error validation
* Update tools setup.

* Remove autopep8
… service bug fix (#7569)

* delete container to custom command

* finished up commands

* added test

* history and pep8

* linter, static

* new recording after scrubbing fix

* comment typo
* Fix #5608. Fix #4885.

* Update Metric command.
* adding deployment based for resources

* changing to sdk calls for role assignment

* Fixing naming problem causing authorization client to fail

* Fixing formatting errors

* fixing differnt return statemnet errors

* adding tests for the --scope parameter

* fixing conflicts in recordings

* fixing conflicts and rerecording tests

* removing whitespace

* making non predictble test live only

* Fixing blank lines

* allow role assignment for --no-wait aswell

* removed unused imports
… The behavior of this command has been altered by the following extension: storage-preview (#7606)
* removing warnings for cloudshell proxy

* updating code owners for the container modules

* Updating how env var is accessed

* fixing formatting error in string message

* removing unused import
* Fixed bug where --no-wait option causes vm resize to crash.

* Bump up VM version number
* Update rdbms help examples text from Gen4 to Gen5

* Fix version
* image create now accepts a storage-sku. Unused when source is a VM due to service side issue. Also updated help text and made some changes to some model imports.

* Updated storage-sku help parameter

* Fixed bug introduced by using get_models. For profiles that don't have StorageAccountTypes defined define the possible storage-sku values.

* History changes.

* Bumped up VM version number.

removed extra newline in History.rst

* Added test coverage for --storage-sku

* Comment explaining else statement in vm/_params.py

* Modified  test_vm_managed_disk

* Addressed pep8 warnings.
* initial work on the PR trigger

* task create and update with PR trigger

* use one flag to enable commit and pr trigger events

* nit fixes

* Two event flags and only support one source trigger per task

* PR fixes

* allow empty event list when disabling all trigger events

* format fix

* do not clear event list in update, instead status disabled

* update task table

* nit fix

* bump cli event version

* nit fixes

* update ACR sdk version

* version fix

* update appservice version

* pylint fixes

* pylint fixes

* nit

* pylint fix

* respond to PR comments

* change from null to empty check
* revert to desktop python

* change before do model file concat

* add trimmer

* revert an unrelated change

* incorporate model file trimmer

* revert a few unnecessary change

* lint fix

* remove an irrelevant command file

* backup python to storage accounts

* remove irrelevant comment

* simplify
tjprescott and others added 5 commits October 24, 2018 10:02
* pin flake8 to 3.5.0

* wrong version
* Fixed bug where if update --remove is called with --ids, the correct params aren't passed in subsequent iterations of the update command.

* Fixed Pep8 issues

* Fixed bug by changing generic update logic instead of copying CLI params.
* moved previous 'extensions' event handling logic

* moved all extension management utility to core

* fix ref

static checking

fix test import

* core package

* fix reference

* renamed to register_global_transforms
@jaysterp jaysterp requested a review from djyou as a code owner October 25, 2018 00:53
@jaysterp jaysterp requested a review from ankurkhemani October 25, 2018 00:54
Prasanna-Padmanabhan and others added 3 commits October 25, 2018 08:28
* Updates for GA release.

* Addressing PR feedback:
- Removed 'recommendation generate' command and added a '--refresh' option to 'recommendation list' instead.
- Renamed 'configuration get' to 'configuration list'.
- Renamed 'configuration set' to 'configuration update'.
- Test recording updates.

* Update history with details of the changes.

* Adding generic update and show command.

* Addressing review feedback.

* Update SDK package version.

* Addressing review feedback.

* Fix indentation errors.

* Add support for ids.

* Updates for GA release.

* Addressing PR feedback:
- Removed 'recommendation generate' command and added a '--refresh' option to 'recommendation list' instead.
- Renamed 'configuration get' to 'configuration list'.
- Renamed 'configuration set' to 'configuration update'.
- Test recording updates.

* Update history with details of the changes.

* Adding generic update and show command.

* Addressing review feedback.

* Update SDK package version.

* Addressing review feedback.

* Fix indentation errors.

* Add support for ids.

* Fix test failures.

* Update to latest SDK version

* Update history

* Reference new package version
* Update _help.py

Fixed non-working example with guidance from 
https://docs.microsoft.com/en-us/azure/mysql/quickstart-create-mysql-server-database-using-azure-cli

* Update HISTORY.rst

* Update setup.py

* Fix placeholder
@jaysterp jaysterp merged commit 3941f6f into AzureCR:dev Oct 25, 2018
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.