Skip to content

Add CLI examples for DMS Schema Conversion operations#10423

Open
vluhans wants to merge 1 commit into
aws:developfrom
vluhans:dms-schema-conversion-examples
Open

Add CLI examples for DMS Schema Conversion operations#10423
vluhans wants to merge 1 commit into
aws:developfrom
vluhans:dms-schema-conversion-examples

Conversation

@vluhans

@vluhans vluhans commented Jun 18, 2026

Copy link
Copy Markdown
Member

Add command examples for 34 DMS operations covering Schema Conversion, data providers, instance profiles, and migration projects.

Operations covered:

  • cancel-metadata-model-conversion, cancel-metadata-model-creation
  • create-data-provider, create-instance-profile, create-migration-project
  • delete-data-provider, delete-instance-profile, delete-migration-project
  • describe-conversion-configuration, describe-data-providers, describe-extension-pack-associations, describe-instance-profiles, describe-metadata-model, describe-metadata-model-assessments, describe-metadata-model-children, describe-metadata-model-conversions, describe-metadata-model-creations, describe-metadata-model-exports-as-script, describe-metadata-model-exports-to-target, describe-metadata-model-imports, describe-migration-projects
  • export-metadata-model-assessment, get-target-selection-rules
  • modify-conversion-configuration, modify-data-provider, modify-instance-profile, modify-migration-project
  • start-extension-pack-association, start-metadata-model-assessment, start-metadata-model-conversion, start-metadata-model-creation, start-metadata-model-export-as-script, start-metadata-model-export-to-target, start-metadata-model-import

All examples have been tested against the real DMS service and validated with pytest tests/functional/docs/test_examples.py (11756 tests passed).

@vluhans vluhans requested a review from a team as a code owner June 18, 2026 14:54
@codecov-commenter

codecov-commenter commented Jun 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.60%. Comparing base (70d3b38) to head (54c25cd).
⚠️ Report is 77 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #10423      +/-   ##
===========================================
+ Coverage    93.42%   93.60%   +0.18%     
===========================================
  Files          211      211              
  Lines        17312    17355      +43     
===========================================
+ Hits         16174    16246      +72     
+ Misses        1138     1109      -29     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@aemous aemous added documentation This is a problem with documentation. pr:needs-review This PR needs a review from a Member. dms labels Jun 26, 2026
@aemous aemous self-assigned this Jul 8, 2026
@aemous aemous self-requested a review July 8, 2026 12:06

@aemous aemous left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nit: Region uses us-east-1 instead of us-west-2

File: All files referencing us-east-1 (e.g., cancel-metadata-model-conversion.rst, line 6).

The style guide specifies using us-west-2 as the default sanitized region unless the resource can only exist in a different region. Please update to us-west-2 or provide justification for why us-east-1 is required here.

Comment thread awscli/examples/dms/cancel-metadata-model-conversion.rst Outdated
@aemous

aemous commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Nit: Region uses us-east-1 instead of us-west-2

File: All files referencing us-east-1 (e.g., cancel-metadata-model-conversion.rst, line 6).

The style guide specifies using us-west-2 as the default sanitized region unless the resource can only exist in a different region. Please update to us-west-2 or provide justification for why us-east-1 is required here.

Discussed internally. We can keep us-east-1 in these examples for consistency with existing DMS examples.

@vluhans vluhans force-pushed the dms-schema-conversion-examples branch from a83ead6 to 54c25cd Compare July 8, 2026 15:47
@aemous aemous self-requested a review July 8, 2026 16:12

@aemous aemous left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM.

@aemous aemous added the v1 label Jul 8, 2026
@aemous aemous added pr:ready-to-merge This PR is ready to be merged. and removed pr:needs-review This PR needs a review from a Member. labels Jul 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dms documentation This is a problem with documentation. pr:ready-to-merge This PR is ready to be merged. v1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants