New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Listing Functions in an Azure Function App fails because of Read-Only Resource Lock #1664

Closed
tomkerkhove opened this Issue Aug 9, 2017 · 11 comments

Comments

Projects
None yet
6 participants
@tomkerkhove
Copy link

tomkerkhove commented Aug 9, 2017

OS and Browser

  • OS: Windows 10
  • Browser : Google Chrome

Repro steps

  • Provision Azure Function App with sample function
  • Add read-only resource lock on the resource group
  • List all functions in the provisioned Azure Function App

Any error messages

No

Expected outcome

I'd expect to be able to check all functions, deployed code, montioring, etc without the capability to change anything/save changes

Mention any other details that might be useful.

  • Removing the resource lock fixes the issue
  • Delete-lock has no impact

With read-only resource lock:
not listed

With read-only resource lock:
listed

@lindydonna

This comment has been minimized.

Copy link
Contributor

lindydonna commented Aug 10, 2017

This is by design, as you can only view top-level resources with a read resource lock. Functions are not top level resources, they are part of the site content. Listing functions requires access to the SCM site, which only owners and contributors have.

@lindydonna lindydonna closed this Aug 10, 2017

@tomkerkhove

This comment has been minimized.

Copy link

tomkerkhove commented Aug 11, 2017

@lindydonna

This comment has been minimized.

Copy link
Contributor

lindydonna commented Aug 11, 2017

@tomkerkhove If that is your use case, I recommend you use Application Insights for monitoring. That provides far more features and readers should still be able to access the telemetry. See https://blogs.msdn.microsoft.com/appserviceteam/2017/04/06/azure-functions-application-insights/

@tomkerkhove

This comment has been minimized.

Copy link

tomkerkhove commented Aug 12, 2017

Monitoring is only one if the things that I'm missing. What if I just want to check the config or code that is running, this should be possible with a Read-only lock.

This is a show stopper for me to be honest.

@tomkerkhove

This comment has been minimized.

Copy link

tomkerkhove commented Aug 30, 2017

Isn't there an option to reopen this issue and provide a way around this?

@paulbatum

This comment has been minimized.

Copy link
Member

paulbatum commented Aug 30, 2017

The portal could list functions via the ARM API. This would not be blocked by the readonly lock. In fact this change is likely to happen as part of Azure/azure-functions-host#1824. So I'm reopening this issue.

@paulbatum paulbatum reopened this Aug 30, 2017

@paulbatum paulbatum added this to the Triaged milestone Aug 31, 2017

@paulbatum paulbatum modified the milestones: Triaged, backlog Oct 3, 2017

@tomkerkhove

This comment has been minimized.

Copy link

tomkerkhove commented Dec 15, 2017

Is there an update on this? This is very very annoying and is discouraging Resource Locks. /cc @jeffhollan

@ehamai

This comment has been minimized.

Copy link
Contributor

ehamai commented Mar 15, 2018

@ahmelsayed - ReadOnly locks blocks access to our App Settings API which requires write access so the tree view just assumes most of your experience isn't going to work and blocks everything. But should that be a problem for showing the list of functions, the code editor, and showing monitoring in readonly mode for all types of FunctionApps (v1, v2, linux v2, etc...)?

@tomkerkhove

This comment has been minimized.

Copy link

tomkerkhove commented Mar 15, 2018

That would be good enough for me to be honest :)

@btardif btardif modified the milestones: sprint 29 (s92), s94 Jun 12, 2018

@btardif btardif modified the milestones: s94, s96, s98 Jul 31, 2018

@btardif btardif added the OLD label Sep 8, 2018

@btardif btardif added wontfix and removed Improvement labels Sep 26, 2018

@hartra344

This comment has been minimized.

Copy link
Member

hartra344 commented Sep 28, 2018

Migrated item to MsAzure Dev Ops

@tomkerkhove

This comment has been minimized.

Copy link

tomkerkhove commented Sep 28, 2018

Hm, why is this not public?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment