Update Set-AzSynapseSqlPoolAuditSetting.md

[sravanisaluru]

updating the audit action groups that we support for synapse

Description

Checklist

• [x] I have read the Submitting Changes section of CONTRIBUTING.md
• [x] The title of the PR is clear and informative
• [ ] The appropriate ChangeLog.md file(s) has been updated:
  • For any service, the ChangeLog.md file can be found at src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md
  • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header -- no new version header should be added
• [x] The PR does not introduce breaking changes
• [ ] If applicable, the changes made in the PR have proper test coverage
• [ ] For public API changes to cmdlets:
  • [ ] a cmdlet design review was approved for the changes in this repository (Microsoft internal only)
    • {Please put the link here}
  • [ ] the markdown help files have been regenerated using the commands listed here

[msftbot[bot]]

Thank you for your contribution sravanisaluru! We will review the pull request and get back to you soon.

[sravanisaluru]

#sign-off

[sravanisaluru]

#sign-off

[VeryEarly]

please also update this string: https://github.com/Azure/azure-powershell/blob/main/src/Synapse/Synapse/Common/HelpMessages.cs#L244 otherwise your change will be overwrite next time doc was generated

[sravanisaluru]

Hi Yabo ,

could you please help me with this ,

regards sravani

On Thu, Jun 30, 2022 at 11:40 AM Yabo Hu @.***> wrote:

please also update this string:

https://github.com/Azure/azure-powershell/blob/main/src/Synapse/Synapse/Common/HelpMessages.cs#L244 otherwise your change will be overwrite next time doc was generated

— Reply to this email directly, view it on GitHub https://github.com/Azure/azure-powershell/pull/18839#issuecomment-1170806933, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHMY2KRZN4HYELE7QWU5KDLVRU25BANCNFSM52IBLIHA . You are receiving this because you authored the thread.Message ID: @.***>

[BethanyZhou]

@sravanisaluru , as you're intend to reduce the accepted value of AuditActionGroup, we need to update the Enum value below as well. I'm afraid that these changes will cause breaking changes as well. And breaking change is not allowed in current release. Could you raise the request by email and send it to our PM @dcaro ?

https://github.com/Azure/azure-powershell/blob/175bb0682a7dcdeece0b9c07a3995b2dbb13414c/src/Synapse/Synapse/Models/ManagementModels/Auditing/WorkspaceAuditModel.cs#L17-L39

[dcaro]

@sravanisaluru has the list of accepted values already been changed on the API? What happens today if DATABASE_LOGOUT_GROUP value is passed?

[sravanisaluru]

This enum is it applicable for all like azure database azure synapse ? , for azure synapse today we support only 3 action groups so in the accepted values we cannot specify other action groups. but if this used across other services then it can break things. let me know how to proceed.

customer cannot pass other action groups as its not supported for synapse

[sravanisaluru]

this change is applicable only for synapse and it will fail if database_logout_group is passed as a parameter. i assume this update will not change anything for azuresql or MI. let me know if it can be updated only for synapse or will effect other services like azure sql or MI

On Tue, Jul 5, 2022 at 11:56 AM Damien Caro @.***> wrote:

@sravanisaluru https://github.com/sravanisaluru have the accepted values already been changed on the API? What happens today if DATABASE_LOGOUT_GROUP value is passed?

— Reply to this email directly, view it on GitHub https://github.com/Azure/azure-powershell/pull/18839#issuecomment-1174666443, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHMY2KSHEAXNTEKLSYF3CH3VSPIRHANCNFSM52IBLIHA . You are receiving this because you were mentioned.Message ID: @.***>

[isra-fel]

/azp run azure-powershell - security-tools

[azure-pipelines[bot]]

Azure Pipelines successfully started running 1 pipeline(s).