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

azure-functions #48

Closed
1 task done
Tracked by #49
Hebilicious opened this issue Jul 5, 2023 · 9 comments
Closed
1 task done
Tracked by #49

azure-functions #48

Hebilicious opened this issue Jul 5, 2023 · 9 comments

Comments

@Hebilicious
Copy link
Collaborator

Hebilicious commented Jul 5, 2023

Describe the feature

Investigate why this worfklow is disabled.
Rename this workflow to "azure-function"

Additional information

  • Would you be willing to help implement this feature?
@Hebilicious Hebilicious added ci azure enhancement New feature or request labels Jul 5, 2023
@Hebilicious Hebilicious changed the title add azure workflow enable all azure workflow Jul 5, 2023
@Hebilicious Hebilicious changed the title enable all azure workflow enable azure functions Jul 5, 2023
@Hebilicious Hebilicious changed the title enable azure functions azure functions Jul 5, 2023
This was referenced Jul 5, 2023
@Hebilicious Hebilicious removed the enhancement New feature or request label Jul 5, 2023
@Hebilicious Hebilicious changed the title azure functions azure-functions Jul 5, 2023
@tobiasdiez
Copy link

tobiasdiez commented Sep 18, 2023

In the past, I had issues as well with the azure deployment. Usually, you can find them out in the log: choose Log stream on the right in the azure function, then select File stream in the dropdown and then invoke the function.

My recent trouble was that symlinks are not preserved (see eg Azure/webapps-deploy#54), which lead to "module not found" errors (since the symlink in the node_modules folder is missing). Maybe helpful in this context: archiverjs/node-archiver#402

If you want, I can help diagnosing azure errors and maybe fix them. For this I obviously need temporary access to the test azure instance. Let me know at code@tobiasdiez.de in case you are interested in working together on the azure deployment.

@GerryWilko
Copy link

Am I right in thinking that Azure Function deployment does not work at present? At the moment I have errors with lru-cache which does appear to be exactly what you are saying @tobiasdiez, the dependency is a symlink into the .nitro folder. Is there any work around for this?

@tobiasdiez
Copy link

tobiasdiez commented Jan 17, 2024

@GerryWilko
Copy link

Hmm, I am having other issues after replacing the symlinks. Images in my public folder arent being served and some scripts are strangely showing integrity errors.

It looks like azure-functions isn't really in a working state. I think I'll have to use azure container instances instead.

@itpropro
Copy link

itpropro commented Jan 18, 2024

Hmm, I am having other issues after replacing the symlinks. Images in my public folder arent being served and some scripts are strangely showing integrity errors.

It looks like azure-functions isn't really in a working state. I think I'll have to use azure container instances instead.

I recommend to use Container Apss, not Container instances. Just bundle the app into a docker image and use that in you Container App.
Container Instances are not really made for long running application, more for temporary testing.

PS: I am currently updating the Nitro Azure preset (which targets Static Web App) to use the new Azure Functions v4 model and newer Node versions.
After that I will also update the Azure Functions preset.

@pi0
Copy link
Member

pi0 commented Jan 18, 2024

Let's track it in nitro side. If no progress, I would deprecate azure functions all together.

We currently support Static Web App which kinda works, this functions that is half broken and Container which I didn't try but I imagine should be a standard conterized app.

@pi0 pi0 closed this as completed Jan 18, 2024
@itpropro
Copy link

Let's track it in nitro side. If no progress, I would deprecate azure functions all together.

We currently support Static Web App which kinda works, this functions that is half broken and Container which I didn't try but I imagine should be a standard conterized app.

Static Web Apps is also broken when you use pnpm btw, as Oryx (the build engine under the hood of the swa deployment) doesn't support pnpm. We also have to update the node version, as we currently default to max 16.

@pi0
Copy link
Member

pi0 commented Jan 18, 2024

I just made a (local) deployment few days ago with SWA 😅 Default is 18 in edgem yes sadly they don't support Node.js 20 for CLI at leat.

@itpropro
Copy link

I just made a (local) deployment few days ago with SWA 😅 Default is 18 in edgem yes sadly they don't support Node.js 20 for CLI at leat.

Local unfortunately behaves differently to the actual service. There are also differences between the service configurations in Azure as well as between free and standard tier 🙈

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

No branches or pull requests

5 participants