Azure-docs: Option 1 and 2 are identical

[sean-nixon]

Both options listed here use the exact same commands to add a completely new key to the key vault. What's the other option supposed to be?

---

Document Details

⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

• ID: 5ec6afe4-89a8-1b96-7ae4-81651004d858
• Version Independent ID: 1d17a759-76a0-c990-3993-4fe32ec3a5ed
• Content: PowerShell - Rotate TDE protector - Azure SQL Database
• Content Source: articles/sql-database/transparent-data-encryption-byok-azure-sql-key-rotation.md
• Service: sql-database
• Sub-service: security
• GitHub Login: @aliceku
• Microsoft Alias: aliceku

[Mike-Ubezzi-MSFT]

@sean-nixon Thank you for bringing this to our attention. I will have a correction made to the documentation.

[Mike-Ubezzi-MSFT]

@aliceku The Manual key rotation (link) and the Option 2: Manual rotation (link) are exactly the same?! @CarlRabeler for awareness.

[CarlRabeler]

@VanMSFT for awareness

[aliceku]

There is only the option of manual key rotation, we had to disable automatic key rotation, because that was breaking the geo-dr scenario. It's just a matter of changing the article to make clear that there is only one option now: manual rotation.

[aliceku]

I don't know why the manual section is duplicated though, the Option #2 can be removed.

[Mike-Ubezzi-MSFT]

Removing section.

[Mike-Ubezzi-MSFT]

Someone beat me to it. @sean-nixon An update will be published within the next 24 hours.

[Mike-Ubezzi-MSFT]

Looking through the history of this document, it used to be ## Option 1: Auto rotation and ## Option 2: Manual rotation but with the recent migration to az.sql, the auto functionality does not exist and manual was being used as a placeholder until the functionality became available. Subsequently, a PR was issued to change Option 2 via the following commit: https://github.com/MicrosoftDocs/azure-docs/commit/97626512c0b15f062c828a11c0b6c22b06cc3335#diff-225fee344f4dee51635ad67c1d7f807a

But I am not seeing the PR where Option 2 has been removed. That needs to be approved and merged.

[aliceku]

We removed option one, the auto-rotation, because it was breaking the geo-dr scenario. We are blocking auto-key rotation now. I had removed that section, but didn't realize that option 2 was duplicated. I just removed the duplicated section.

[Mike-Ubezzi-MSFT]

Thanks @aliceku !
@sean-nixon We will now proceed to close this thread as an update should be published within 24hrs. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.