Skip to content
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

Various problems manual steps in New-WinGet-Rest-Source-Azure.md documentation #178

Open
ghost opened this issue Oct 4, 2022 · 2 comments
Assignees
Labels
Issue-Docs Improvements or additions to documentation
Milestone

Comments

@ghost
Copy link

ghost commented Oct 4, 2022

Brief description of your issue

This is in reference to the manual steps in the "Manage Windows Package Manager REST source manually" section of new-winget-rest-source-azure.md.

Broadly, the Azure wizards are out of step with what's in the documentation. This makes following some instructions somewhat difficult because things don't line up, may be in a different wizard page, or missing entirely. This is especially apparent in "Storage Account" and "Function App" setup.

In the "Azure Key Vault Secrets" section, it notes the following secrets: "AzStorageAccountKey", "CosmosAccountEndpoint", "CosmosReadOnlyKey", and "CosmosReadWriteKey" but has actual instructions for "AzStorageAccountKey", "CosmosAccountEndpoint", and "CosmosAccountKey". CosmosAccountKey is not in the initial list of keys, and there's no instructions for "CosmosReadOnlyKey" or "CosmosReadWriteKey". Later when setting up the Function App, it's revealed that you need to set up CosmosReadOnlyKey/CosmosReadWriteKey and CosmosAccountKey is not used at all. It was straightforward enough to figure out what values to put here, but the docs should have been in sync with this.

In the "Azure Function" section, it references a configuration property "WEBSITE_CONTENTSHARE" as "azfun-pkgman3pr-westus-test". It's unclear if this is a proper value or not (and if it's not a proper value, what it should be set to). Also, there's a typo ("URI's").

@ghost ghost added the Issue-Docs Improvements or additions to documentation label Oct 4, 2022
@RDMacLachlan RDMacLachlan self-assigned this Oct 5, 2022
@RDMacLachlan
Copy link
Member

Thank you for the feedback on this.

@nclinch
Copy link

nclinch commented Jan 5, 2023

to add to the list from above...
you cannot set the function to use .net 3.1

Screenshot 2023-01-05 174126

In the Instance Details section, ensure that the following values have been set the function runtime to .net 3.1

Function App name: contoso-function-demo
Publish: Code
Runtime stack: .NET
Version: 3.1
Region: West US

@denelon denelon added this to the 1.9-REST milestone Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue-Docs Improvements or additions to documentation
Projects
Status: To Do
Development

No branches or pull requests

3 participants