Skip to content

Devcenter dataplane SDK v2023-04-01, integration tests, and samples#36245

Merged
ShivangiReja merged 66 commits intomainfrom
dev/olegoid/devcenter-dataplane-2023-04-01-tests
Sep 23, 2023
Merged

Devcenter dataplane SDK v2023-04-01, integration tests, and samples#36245
ShivangiReja merged 66 commits intomainfrom
dev/olegoid/devcenter-dataplane-2023-04-01-tests

Conversation

@olegoid
Copy link
Contributor

@olegoid olegoid commented May 12, 2023

Description

Please add an informative description that covers that changes made by the pull request and link all relevant issues.

If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.

All SDK Contribution checklist:

  • The pull request does not introduce [breaking changes]
  • CHANGELOG is updated for new features, bug fixes or other significant changes.
  • I have read the contribution guidelines.

General Guidelines and Best Practices

  • Title of the pull request is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

  • Pull request includes test coverage for the included changes.

Pipeline runs

Dry runs on the release pipeline:
https://dev.azure.com/azure-sdk/internal/_build?definitionId=5622&_a=summary

List of tests

DevBox

  • get_pool
  • list_pools
  • get_dev_box
  • get_remote_connection
  • get_action
  • list_actions
  • skip_action
  • delay_action
  • delay_all_actions
  • get_schedule
  • list_schedules
  • list_dev_boxes
  • list_all_dev_boxes
  • list_all_dev_boxes_by_user
  • begin_create_dev_box
  • begin_delete_dev_box
  • begin_start_dev_box
  • begin_stop_dev_box
  • begin_restart_dev_box

DevCenter

  • get_project
  • list_projects

Environments

  • begin_delete_environment
  • begin_create_or_update_environment
  • list_environment_types
  • list_catalogs
  • list_environments
  • list_all_environments
  • get_environment_definition
  • list_environment_definitions
  • list_environment_definitions_by_catalog
  • get_catalog
  • get_environment

Also updated samples.

sebrenna and others added 11 commits March 29, 2023 13:36
* Add session records

* Fix build errors in samples

* Update test project file

* Add devbox client tests

* Add devcenter client tests

* Add env client tests (incomplete)

* Add devcenter admin permissions to the bicep config

* Remove old devbox tests

* Update env tests + session recordings

* Environments tests + session records

* Remove stale session records

* Update env client tests and records

* Update env session records

* Update dev-center tests

* Update session records + tests

* Update env session records

* Update dev center session records

* Update env tests + session records

* Fix typo

* Update samples

* Update dev-box client session records

* Add pool schedule to the bicep script

* Fix test naming

* Fix naming
@olegoid olegoid changed the base branch from main to dev/sebrenna/devcenter-dataplane-2023-04-01-preview May 12, 2023 16:28
@pallavit
Copy link
Contributor

other than small nits and a couple of follow up questions, changes look good to me. @ShivangiReja let us know if you have any other comments.

@ShivangiReja
Copy link
Member

/azp run net - devcenter - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@olegoid
Copy link
Contributor Author

olegoid commented Sep 22, 2023

/azp run net - devcenter - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@olegoid
Copy link
Contributor Author

olegoid commented Sep 22, 2023

/azp run net - devcenter - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@olegoid
Copy link
Contributor Author

olegoid commented Sep 23, 2023

/azp run net - devcenter - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ShivangiReja
Copy link
Member

ShivangiReja commented Sep 23, 2023

In the case of PRs, we run tests in playback mode; live tests are not executed on PRs. We run live tests for the nightly and weekly pipeline. If running all tests live takes 3-4 hours total, that's fine.

For example, you can see the nightly pipeline for Search here, which typically takes approximately 6 hours to complete.

Could you please consider running the tests in live mode for this PR, or alternatively, could you create an issue to track this work and enable it in future PRs?

@olegoid
Copy link
Contributor Author

olegoid commented Sep 23, 2023

Filed an issue that requires our team to enable live tests for the SDK:
#38927

@ShivangiReja ShivangiReja merged commit e96129d into main Sep 23, 2023
@ShivangiReja ShivangiReja deleted the dev/olegoid/devcenter-dataplane-2023-04-01-tests branch September 23, 2023 00:36
yaotongms pushed a commit to yaotongms/azure-sdk-for-net that referenced this pull request Oct 12, 2023
matthohn-msft pushed a commit to matthohn-msft/azure-sdk-for-net that referenced this pull request Oct 27, 2023
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.

7 participants