search

Azure / azure-rest-api-specs

The source for REST API specifications for Microsoft Azure.
MIT License
2.61k stars 5.01k forks source link

microsoft.logic/workflows missing "identity" property #11986

Closed alex-frankel closed 2 years ago

alex-frankel commented 3 years ago

Originally reported here: https://github.com/Azure/bicep/issues/949

Appears to be the case for all apiVersions, but a PUT with the identity property will work as expected. Here's their template sample from docs: https://docs.microsoft.com/en-us/azure/templates/microsoft.logic/2019-05-01/workflows

ghost commented 3 years ago

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @Azure/azure-logicapps-team.

Issue Details
Originally reported here: https://github.com/Azure/bicep/issues/949 Appears to be the case for all apiVersions, but a PUT with the identity property will work as expected. Here's their template sample from docs: https://docs.microsoft.com/en-us/azure/templates/microsoft.logic/2019-05-01/workflows
Author: alex-frankel
Assignees: raych1
Labels: `Logic App`, `Service Attention`, `needs-triage`
Milestone: -
ghost commented 3 years ago

Hi, alex-frankel. Your PR has no update for 14 days and it is marked as stale PR. If no further update for over 14 days, the bot will close the PR. If you want to refresh the PR, please remove no-recent-activity label.

alex-frankel commented 3 years ago

Hi @raych1 - can you confirm that this has been successfully routed to the logic apps team? It looks like the @ mention did not work. No other participants have been added to the issue.

alex-frankel commented 3 years ago

Hi logic apps team – can you all take a look at this issue?

Thanks, Alex

Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: msftbot[bot]mailto:notifications@github.com Sent: Tuesday, December 22, 2020 9:00 AM To: Azure/azure-rest-api-specsmailto:azure-rest-api-specs@noreply.github.com Cc: Alex Frankelmailto:alfran@microsoft.com; Authormailto:author@noreply.github.com Subject: Re: [Azure/azure-rest-api-specs] microsoft.logic/workflows missing "identity" property (#11986)

Hi, alex-frankel. Your PR has no update for 14 days and it is marked as stale PR. If no further update for over 14 days, the bot will close the PR. If you want to refresh the PR, please remove no-recent-activity label.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-rest-api-specs%2Fissues%2F11986%23issuecomment-749583070&data=04%7C01%7Calfran%40microsoft.com%7C65db16a777964ac5dccb08d8a68a5d24%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637442460493573121%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=WoLaOSHxwtF5xy%2BaVsKqqNyG06dqbwJQs0iQZMq9ghY%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACUQVA5QPMOCSA6DX6474M3SWCYB7ANCNFSM4UQY7JXA&data=04%7C01%7Calfran%40microsoft.com%7C65db16a777964ac5dccb08d8a68a5d24%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637442460493583114%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=UtwBvc7U%2FIXUuiyJK8Hig6DccEYov87RogccMGqFtSg%3D&reserved=0.

raych1 commented 3 years ago

@rarayudu, @pankajsn, @tonytang-microsoft-com, I found you're either code owners of microsoft.logic RP or swagger author. Can you have a look at this issue or re-assign properly. Thanks.

alex-frankel commented 3 years ago

Ping 😊 Can anyone take a look at this?

Thanks, Alex

Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: Alex Frankelmailto:alfran@microsoft.com Sent: Tuesday, December 22, 2020 7:08 AM To: Azure/azure-rest-api-specsmailto:reply@reply.github.com; Azure/azure-rest-api-specsmailto:azure-rest-api-specs@noreply.github.com; Jon Fanceymailto:Jon.Fancey@microsoft.com; Nitasha Vermamailto:nitashav@microsoft.com; Tony Tangmailto:tonytang@microsoft.com; Rama Krishna Rayudumailto:rarayudu@microsoft.com; Rohitha Hewawasammailto:rohithah@microsoft.com Cc: Authormailto:author@noreply.github.com Subject: RE: [Azure/azure-rest-api-specs] microsoft.logic/workflows missing "identity" property (#11986)

Hi logic apps team – can you all take a look at this issue?

Thanks, Alex

Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: msftbot[bot]mailto:notifications@github.com Sent: Tuesday, December 22, 2020 9:00 AM To: Azure/azure-rest-api-specsmailto:azure-rest-api-specs@noreply.github.com Cc: Alex Frankelmailto:alfran@microsoft.com; Authormailto:author@noreply.github.com Subject: Re: [Azure/azure-rest-api-specs] microsoft.logic/workflows missing "identity" property (#11986)

Hi, alex-frankel. Your PR has no update for 14 days and it is marked as stale PR. If no further update for over 14 days, the bot will close the PR. If you want to refresh the PR, please remove no-recent-activity label.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-rest-api-specs%2Fissues%2F11986%23issuecomment-749583070&data=04%7C01%7Calfran%40microsoft.com%7C65db16a777964ac5dccb08d8a68a5d24%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637442460493573121%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=WoLaOSHxwtF5xy%2BaVsKqqNyG06dqbwJQs0iQZMq9ghY%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACUQVA5QPMOCSA6DX6474M3SWCYB7ANCNFSM4UQY7JXA&data=04%7C01%7Calfran%40microsoft.com%7C65db16a777964ac5dccb08d8a68a5d24%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637442460493583114%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=UtwBvc7U%2FIXUuiyJK8Hig6DccEYov87RogccMGqFtSg%3D&reserved=0.

alex-frankel commented 3 years ago

Yes. The swagger for the logic needs to be updated to include the identity details. We have a backlogs item for this.

Thanks, Rama.

From: Alex Frankel alfran@microsoft.com Sent: Sunday, January 10, 2021 9:57 PM To: Azure/azure-rest-api-specs reply@reply.github.com; Azure/azure-rest-api-specs azure-rest-api-specs@noreply.github.com; Jon Fancey Jon.Fancey@microsoft.com; Nitasha Verma nitashav@microsoft.com; Tony Tang tonytang@microsoft.com; Rama Krishna Rayudu rarayudu@microsoft.com; Rohitha Hewawasam rohithah@microsoft.com Cc: Author author@noreply.github.com Subject: RE: [Azure/azure-rest-api-specs] microsoft.logic/workflows missing "identity" property (#11986)

Ping 😊 Can anyone take a look at this?

Thanks, Alex

Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: Alex Frankelmailto:alfran@microsoft.com Sent: Tuesday, December 22, 2020 7:08 AM To: Azure/azure-rest-api-specsmailto:reply@reply.github.com; Azure/azure-rest-api-specsmailto:azure-rest-api-specs@noreply.github.com; Jon Fanceymailto:Jon.Fancey@microsoft.com; Nitasha Vermamailto:nitashav@microsoft.com; Tony Tangmailto:tonytang@microsoft.com; Rama Krishna Rayudumailto:rarayudu@microsoft.com; Rohitha Hewawasammailto:rohithah@microsoft.com Cc: Authormailto:author@noreply.github.com Subject: RE: [Azure/azure-rest-api-specs] microsoft.logic/workflows missing "identity" property (#11986)

Hi logic apps team – can you all take a look at this issue?

Thanks, Alex

Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: msftbot[bot]mailto:notifications@github.com Sent: Tuesday, December 22, 2020 9:00 AM To: Azure/azure-rest-api-specsmailto:azure-rest-api-specs@noreply.github.com Cc: Alex Frankelmailto:alfran@microsoft.com; Authormailto:author@noreply.github.com Subject: Re: [Azure/azure-rest-api-specs] microsoft.logic/workflows missing "identity" property (#11986)

Hi, alex-frankel. Your PR has no update for 14 days and it is marked as stale PR. If no further update for over 14 days, the bot will close the PR. If you want to refresh the PR, please remove no-recent-activity label.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-rest-api-specs%2Fissues%2F11986%23issuecomment-749583070&data=04%7C01%7Crarayudu%40microsoft.com%7C27930366d96d40acd72b08d8b5f5abf4%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637459414038837133%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=JpFe4vB5jlnz50BAYROjQRG2dalIrcHE64OwfVppX0Q%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACUQVA5QPMOCSA6DX6474M3SWCYB7ANCNFSM4UQY7JXA&data=04%7C01%7Crarayudu%40microsoft.com%7C27930366d96d40acd72b08d8b5f5abf4%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637459414038847087%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=ewaS0Q6QhPt9H7nFjbxKNwF3OMZ4eKgiWXsrQuBLLY8%3D&reserved=0.

alex-frankel commented 3 years ago

Great – do you have an ETA for when it will be resolved?

Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: Rama Krishna Rayudumailto:rarayudu@microsoft.com Sent: Sunday, January 10, 2021 10:09 PM To: Alex Frankelmailto:alfran@microsoft.com; Azure/azure-rest-api-specsmailto:reply@reply.github.com; Azure/azure-rest-api-specsmailto:azure-rest-api-specs@noreply.github.com; Jon Fanceymailto:Jon.Fancey@microsoft.com; Nitasha Vermamailto:nitashav@microsoft.com; Tony Tangmailto:tonytang@microsoft.com; Rohitha Hewawasammailto:rohithah@microsoft.com Cc: Authormailto:author@noreply.github.com Subject: RE: [Azure/azure-rest-api-specs] microsoft.logic/workflows missing "identity" property (#11986)

Yes. The swagger for the logic needs to be updated to include the identity details. We have a backlogs item for this.

Thanks, Rama.

From: Alex Frankel alfran@microsoft.com Sent: Sunday, January 10, 2021 9:57 PM To: Azure/azure-rest-api-specs reply@reply.github.com; Azure/azure-rest-api-specs azure-rest-api-specs@noreply.github.com; Jon Fancey Jon.Fancey@microsoft.com; Nitasha Verma nitashav@microsoft.com; Tony Tang tonytang@microsoft.com; Rama Krishna Rayudu rarayudu@microsoft.com; Rohitha Hewawasam rohithah@microsoft.com Cc: Author author@noreply.github.com Subject: RE: [Azure/azure-rest-api-specs] microsoft.logic/workflows missing "identity" property (#11986)

Ping 😊 Can anyone take a look at this?

Thanks, Alex

Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: Alex Frankelmailto:alfran@microsoft.com Sent: Tuesday, December 22, 2020 7:08 AM To: Azure/azure-rest-api-specsmailto:reply@reply.github.com; Azure/azure-rest-api-specsmailto:azure-rest-api-specs@noreply.github.com; Jon Fanceymailto:Jon.Fancey@microsoft.com; Nitasha Vermamailto:nitashav@microsoft.com; Tony Tangmailto:tonytang@microsoft.com; Rama Krishna Rayudumailto:rarayudu@microsoft.com; Rohitha Hewawasammailto:rohithah@microsoft.com Cc: Authormailto:author@noreply.github.com Subject: RE: [Azure/azure-rest-api-specs] microsoft.logic/workflows missing "identity" property (#11986)

Hi logic apps team – can you all take a look at this issue?

Thanks, Alex

Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: msftbot[bot]mailto:notifications@github.com Sent: Tuesday, December 22, 2020 9:00 AM To: Azure/azure-rest-api-specsmailto:azure-rest-api-specs@noreply.github.com Cc: Alex Frankelmailto:alfran@microsoft.com; Authormailto:author@noreply.github.com Subject: Re: [Azure/azure-rest-api-specs] microsoft.logic/workflows missing "identity" property (#11986)

Hi, alex-frankel. Your PR has no update for 14 days and it is marked as stale PR. If no further update for over 14 days, the bot will close the PR. If you want to refresh the PR, please remove no-recent-activity label.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-rest-api-specs%2Fissues%2F11986%23issuecomment-749583070&data=04%7C01%7Calfran%40microsoft.com%7Cf6aa9db98a3b4ed73b1f08d8b5f76610%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637459421454567174%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=xiuXCF5ssvnWax2abqY435lMd2VJBZoz9s57CuG%2FvLM%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACUQVA5QPMOCSA6DX6474M3SWCYB7ANCNFSM4UQY7JXA&data=04%7C01%7Calfran%40microsoft.com%7Cf6aa9db98a3b4ed73b1f08d8b5f76610%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637459421454577128%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=i1qzxcgzOWgRc%2Fuky0h%2FyaBibBDQjM7PT939ENfJTtI%3D&reserved=0.

miqm commented 3 years ago

@alex-frankel do you know if the team has some ETA on fixing this?

darren-johnson commented 3 years ago

This is an issue for me too. Is there an update on this?

ecfan commented 3 years ago

Ping @Azure/azure-logicapps-team

alex-frankel commented 3 years ago

Latest ETA from logic apps team is June

Dave14393 commented 3 years ago

Is there any update on this?

ecfan commented 3 years ago

Ping @Azure/azure-logicapps-team

alex-frankel commented 3 years ago

Since this is taking so long to resolve, we are prioritizing https://github.com/Azure/bicep/issues/3000 as a workaround. Current ETA for the swagger fix is sometime between now and end of CY2021..

The bicep workaround we are hoping to get to in our 0.5 release.

anthony-c-martin commented 2 years ago

Closed by #16181