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

Sunny/bicep deployer #3

Merged
merged 29 commits into from
May 12, 2023
Merged

Conversation

sunnycarter
Copy link
Collaborator


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

Related command

General Guidelines

  • Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
  • Have you run python scripts/ci/test_index.py -q locally?

For new extensions:

About Extension Publish

There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

src/aosm/setup.py Outdated Show resolved Hide resolved
@Cyclam
Copy link
Collaborator

Cyclam commented May 8, 2023

If the intention is for this command group to also include generating network service designs (as the README says), then I don't think definition is the right word

src/aosm/azext_aosm/_configuration.py Outdated Show resolved Hide resolved
src/aosm/azext_aosm/_configuration.py Outdated Show resolved Hide resolved
src/aosm/azext_aosm/_configuration.py Outdated Show resolved Hide resolved
src/aosm/azext_aosm/_configuration.py Outdated Show resolved Hide resolved
src/aosm/azext_aosm/_configuration.py Outdated Show resolved Hide resolved
src/aosm/azext_aosm/deploy/deploy_with_arm.py Outdated Show resolved Hide resolved
src/aosm/azext_aosm/deploy/pre_deploy.py Show resolved Hide resolved
src/aosm/azext_aosm/deploy/pre_deploy.py Show resolved Hide resolved
src/aosm/azext_aosm/deploy/pre_deploy.py Outdated Show resolved Hide resolved
src/aosm/azext_aosm/util/constants.py Outdated Show resolved Hide resolved
Suggestion from cyclam

Co-authored-by: Cyclam <95434717+Cyclam@users.noreply.github.com>
@sunnycarter sunnycarter merged commit 057ef05 into add-aosm-extension May 12, 2023
@jordlay jordlay deleted the sunny/bicep-deployer branch May 15, 2023 10:44
jddarby pushed a commit that referenced this pull request Oct 19, 2023
…count container copy, api-version 2023-09-15-preview (Azure#6703)

* init changes for 20230915-preview api

* Changes to add enablePriorityBasedExecution and defaultPriorityLevel

* Removed isPreview flag from CMK related properties and added key-uri to cosmosdb update operation.

* Added test and test recording for priority based execution

* corrected resource group name and whitespace corrections

* white space corrections

* Add cosmosdb copy command

* Add tests

* Dts/cross account copy (#3)

* Deprecate old command

* Add test recording for dts

---------

Co-authored-by: Nitesh Vijay <niteshvijay@microsoft.com>

* Fix build error

* Fix Static Analysis

* Use boolean type

* Change help text

* Update help text

* Updated Vendored SDK

* Updated test account name in test_cosmosdb-merge-scenario.py

* Whitespace correction and added linter exclution for enable pbe

* Updated linter_exclusions

* Remove id part

* Corrected account name in test_cosmosdb-merge-scenario.py and generated test recordings

* Generated test recording for test_cosmosdb_materializedview

* Add support for MI Cassandra in 2023-09-15

* Add -n for jobname

* Fix breaking change

* Modified setup.py and history.rst

* Fix static check

* Whitespace correction

* Re-recorded tests

* rerecorded test test_cosmosdb_sql_materializedview_scenario

* removed linter exclusion for enable_priority_based_execution

* Rerecorded all tests

* Updated HISTORY.rst

* Rerecorded tests

* rerecorded test_cosmosdb_sql_materializedview_scenario

* rerecorded test_cosmosdb_sql_materializedview_scenario

* rerecorded tests

---------

Co-authored-by: Achint Agrawal <acagrawal@microsoft.com>
Co-authored-by: AdrianSibajaRetana <54075415+AdrianSibajaRetana@users.noreply.github.com>
Co-authored-by: Nitesh Vijay <niteshvijay@microsoft.com>
Co-authored-by: Nitesh Vijay <niteshvijay1995@gmail.com>
Co-authored-by: Guanzhou <guanzhousong@microsoft.com>
@atindall9041 atindall9041 mentioned this pull request Mar 7, 2024
3 tasks
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.

3 participants