Skip to content

Commit b236958

Browse files
committed
Merge branch 'main' into hotfix/azure-ai-evaluation/1.13.1
2 parents 8e5cb87 + c1205a6 commit b236958

File tree

729 files changed

+72406
-36508
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

729 files changed

+72406
-36508
lines changed

.github/CODEOWNERS

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -320,6 +320,10 @@
320320
# PRLabel: %Operator Nexus - Network Cloud
321321
/sdk/networkcloud/ @Azure/azure-sdk-write-networkcloud
322322

323+
# ServiceLabel: %Planetary Computer
324+
# PRLabel: %Planetary Computer
325+
/sdk/planetarycomputer/ @mandarinamdar @chahibi
326+
323327
# AzureSdkOwners: @annatisch
324328
# ServiceLabel: %Azure Projects
325329
# ServiceOwners: @annatisch

doc/dev/mgmt/sdk-breaking-changes-guide.md

Lines changed: 42 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -205,16 +205,31 @@ op IotDpsResourceGetCustomized(
205205
);
206206
```
207207

208-
## 6. Removal of Unreferenced Models
208+
## 6. Common Types Upgrade
209+
210+
**Changelog Pattern**:
211+
212+
Multiple changes related to common infrastructure types such as `SystemData` and `IdentityType`:
213+
214+
```md
215+
- Deleted or renamed model `IdentityType`
216+
```
217+
218+
**Reason**: Common types are upgraded to their latest versions during TypeSpec migration.
219+
220+
**Impact**: Low impact since these are common infrastructure types rarely used directly by users.
221+
222+
**Resolution**: Accept these breaking changes.
223+
224+
## 7. Removal of Unreferenced Models
209225

210226
**Changelog Pattern**:
211227

212228
Multiple removals of unreferenced models that are typically not used in the SDK:
213229

214230
```md
215-
- Deleted or renamed model `ReplicationUsageList`
216-
- Deleted or renamed model `VaultList`
217-
- Deleted or renamed model `VaultUsageList`
231+
- Deleted or renamed model `ProxyResourceWithoutSystemData`
232+
- Deleted or renamed model `Resource`
218233
```
219234

220235
**Reason**: Unreferenced models are removed during TypeSpec migration.
@@ -223,7 +238,27 @@ Multiple removals of unreferenced models that are typically not used in the SDK:
223238

224239
**Resolution**: Accept these breaking changes.
225240

226-
## 7. Parameters Changed to Keyword-only
241+
## 8. Removal of Pageable Models
242+
243+
**Changelog Pattern**:
244+
245+
Multiple removals of models following the pattern `xxxList`:
246+
247+
```md
248+
- Deleted or renamed model `ElasticSanList`
249+
- Deleted or renamed model `SkuInformationList`
250+
- Deleted or renamed model `SnapshotList`
251+
- Deleted or renamed model `VolumeGroupList`
252+
- Deleted or renamed model `VolumeList`
253+
```
254+
255+
**Reason**: Python will not expose pageable models for list APIs.
256+
257+
**Impact**: Low impact since these models are typically not used directly by users.
258+
259+
**Resolution**: Accept these breaking changes.
260+
261+
## 9. Parameters Changed to Keyword-only
227262

228263
**Changelog Pattern**:
229264

@@ -239,7 +274,7 @@ Entries showing the usage of passing parameters positionally is disabled:
239274

240275
**Resolution**: Accept these breaking changes.
241276

242-
## 8. Removal of Parameter `if_match`
277+
## 10. Removal of Parameter `if_match`
243278

244279
**Changelog Pattern**:
245280

@@ -257,7 +292,7 @@ Removal of parameter `if_match` and addition of `etag/match_condition` for the s
257292

258293
**Resolution**: Accept these breaking changes.
259294

260-
## 9. Removal of multi-level flattened properties
295+
## 11. Removal of multi-level flattened properties
261296

262297
**Changelog Pattern**:
263298

doc/dev/test_proxy_troubleshooting.md

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -275,6 +275,9 @@ To run recorded tests successfully when recorded values are inconsistent or rand
275275
proxy provides a `variables` API. This makes it possible for a test to record the values of variables that were used
276276
during recording and use the same values in playback mode without a sanitizer.
277277

278+
Note that the recorded variables **must** have string values. For example, trying to record an integer value for a
279+
variable will cause a test proxy error.
280+
278281
For example, imagine that a test uses a randomized `table_uuid` variable when creating resources. The same random value
279282
for `table_uuid` can be used in playback mode by using this `variables` API.
280283

@@ -293,14 +296,14 @@ class TestExample(AzureRecordedTestCase):
293296
@recorded_by_proxy
294297
def test_example(self, **kwargs):
295298
# In live mode, variables is an empty dictionary
296-
# In playback mode, the value of variables is {"table_uuid": "random-value"}
299+
# In playback mode, the value of variables is {"current_time": "<previously recorded time>"}
297300
variables = kwargs.pop("variables", {})
298301

299-
# To fetch variable values, use the `setdefault` method to look for a key ("table_uuid")
300-
# and set a real value for that key if it's not present ("random-value")
301-
table_uuid = variables.setdefault("table_uuid", "random-value")
302+
# To fetch variable values, use the `setdefault` method to look for a key ("current_time")
303+
# and set a real value for that key if it's not present (str(time.time()))
304+
# Note that time.time() is converted from a float to a string to record it properly
305+
current_time = variables.setdefault("current_time", str(time.time()))
302306

303-
# use variables["table_uuid"] when using the table UUID throughout the test
304307
...
305308

306309
# return the variables at the end of the test to record them

eng/ci_tools.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ pkginfo==1.12.1.2
1111
pip==24.0
1212
typing-extensions==4.12.2
1313
pyproject-api==1.8.0
14-
cibuildwheel==2.16.5
14+
cibuildwheel==2.23.3
1515
importlib-metadata==8.5.0
1616
build==1.2.2.post1
1717

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
---
2+
description: 'Run the package validation checks for the SDK package.'
3+
---
4+
## Goal
5+
Run the validation checks for the SDK package by collecting the required information from the user and executing the `azsdk_package_run_check` tool.
6+
7+
## Instructions
8+
1. **Collect Required Information**:
9+
- Prompt the user for the absolute package path to the package directory for the SDK package they want to check.
10+
- Prompt the user to specify values for each of the parameters required by the azsdk_package_run_check tool. Present the user with options to pick from for the allowed values specified by the parameter schema.
11+
12+
2. **Execute Check**:
13+
- Use the `azsdk_package_run_check` tool with the package path and check type.
14+
15+
3. **Present Results**:
16+
- If the package has passed all validation checks, highlight and finish
17+
- If the package is not ready, display the specific check types that need to be fixed for each language.
18+
19+
## Expected User Interaction Flow
20+
1. Ask: "What is the exact path of the package you want to check for validation?"
21+
2. Ask: "Please specify the check type you want to run (e.g., 'all', 'linting', 'format', etc.)"
22+
3. Execute the check using the provided information
23+
4. Display results and next steps
Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
---
2+
description: 'Verify Setup'
3+
---
4+
5+
## Goal
6+
This tool verifies the developer's environment for SDK development and release tasks. It returns what requirements are missing for the specified languages and repo.
7+
8+
Your goal is to identify the project repo root, and pass in the `packagePath` to the Verify Setup tool. For a language repo, pass in the language of the repo.
9+
10+
## Examples
11+
- in `azure-sdk-for-js`, run `azsdk_verify_setup` with `(langs=javascript, packagePath=<path>/azure-sdk-for-js)`.
12+
- in `azure-sdk-for-python`, run `azsdk_verify_setup` with `(langs=python, packagePath=<path>/azure-sdk-for-python, venvPath=<path-to-venv>)`.
13+
14+
## Parameter Requirements
15+
WHENEVER Python is included in `langs`, BEFORE RUNNING `azsdk_verify_setup`, you MUST ASK THE USER TO SPECIFY WHICH virtual environment they want to check. DO NOT ASSUME THE VENV WITHOUT ASKING THE USER. After obtaining the `venvPath`, you can run the tool.
16+
17+
The user can specify multiple languages to check. If the user wants to check all languages, pass in ALL supported languages and STILL ASK for a `venvPath`. Passing in no languages will only check the core requirements.
18+
19+
## Output
20+
Display results in a user-friendly and concise format, highlighting any missing dependencies that need to be addressed.

0 commit comments

Comments
 (0)