-
Notifications
You must be signed in to change notification settings - Fork 124
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
[Bug]: build pipelines not working for third party dependencies (PTE nor appsource deps) #1299
Comments
@freddydk you're right. I tested the installApps URLs in postman and indeed they return an HTML preview of the file. not the file itself. Let me see if I can correct this and re-run the CI/CD pipelines |
@freddydk thanks for the suggestion! That was was the issue (google drive turns out is a poor CDN endpoint). All pipelines are passing now that I moved the dependency app files from Google drive to a dedicated GitHub repo and changed the URLs in installApps accordingly Next question... That dependencies repo is currently public and we want to make it private-only via GitHub personal access tokens The raw download of such a file according to my research is as follows: I assume I need to use classic tokens and not fine-grained tokens? I've been trying to use a fine grained token, granting the content:read-only permission to the repository where the dependencies are and setting our org as the owner. Despite this I'm getting a 404 error from postman. GPT suggests that fine-grained tokens require request bearer headers but I'm unsure if BCContainerHelper supports that (my guess is no), hence me exploring the classic token route. |
I would use NuGet instead. And then provide the people who need access to the NuGet feed access, so they themselves can create their secrets, |
duly noted! I'm going to reach out our dependency ISVs if they have private nuget feeds I can consume |
AL-Go version
6.0
Describe the issue
we are currently standing up several AL Go repositories (both AL Go PTE and AL Go Appsource).
according to the AL Go workshop documentation, the build pipelines should work out of the box for the following use cases
as far as i understand, the build pipelines work both when docker is used ("useCompilerFolder": false, "doNotPublishApps": false) and when docker IS NOT used ("useCompilerFolder": true, "doNotPublishApps": true). My observations suggest otherwise.
the external dependencies (both appsource and PTE) are not written by my company. As such, the external PTE dependencies are hosted as full apps on google drive (for now) and we expose them to AL go via the InstallApps property setting. For external AppSource dependencies, default nuget functionality is used where ALGO fetches the dependency from the whitelisted public nuget repository.
Expected behavior
all scenarios should pass the CICD build pipeline. no build failures.
Steps to reproduce
request access to https://github.com/SteveKrisjanovsD365/PTEDemo/ where I have the following branches to reproduce (you can run the CICD in the non-main branches as we have no delivery steps added - this is a throwaway repo)
Alternately you can reproduce it from scratch if you prefer. "installApps" setting is used fore the external PTE dependencies (one full app per URL, not a runtime pkg), and "useCompilerFolder" + "doNotPublishApps" to utilize or skip docker (true, true to skip docker, false, false to use docker)
the branches that FAIL have the following build errors:
20241108-appsourcedep-usedocker-yes (compiles fine in VS Code):
20241108-ptedep-usedocker-no (compiles fine in VS Code):
20241108-ptedep-usedocker-yes (compiles fine in VS Code):
Additional context (logs, screenshots, etc.)
see attached logs for each of the relevant branches (see "steps to reproduce" notes)
the only one that succeeds is branches [main] (no dependencies) and [20241108-appsourcedep-usedocker-no] (one external appsource dep, docker NOT used). All the ohers fail.
logs_30668419473-20241108-appsourcedep-usedocker-yes.zip
logs_30668621671-20241108-ptedep-usedocker-no.zip
logs_30669787918-20241108-ptedep-usedocker-yes.zip
logs_30668189200-20241108-appsourcedep-usedocker-no.zip
The text was updated successfully, but these errors were encountered: