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

Split build from publish and test #812

Closed
freddydk opened this issue Nov 12, 2023 · 5 comments
Closed

Split build from publish and test #812

freddydk opened this issue Nov 12, 2023 · 5 comments
Assignees
Labels
enhancement New feature or request wontfix This will not be worked on

Comments

@freddydk
Copy link
Contributor

Summary

Today, the build step in AL-Go is done using Run-AlPipeline from BcContainerHelper.
Run-AlPipeline will build, sign, publish and run tests.
for Performance reasons and due to new code signing requirements, this process needs to change.

  1. Building the apps should be done first
  2. Then Signing
  3. Then publishing and running tests.

Only step 3 requires an actual BC instance (Container or like).
Building and signing can be much faster using this approach.

Additionally refactor the build part of workflows to use an composite action, to capture the build parts in one area.

Current structure:

Image

New structure (single project):

Image

New Structure (multi-project):

Image

It should also be possible run the p1 tests together with the p2 tests to validate that p2 didn't break p1 tests.

Success Criteria

TBD

Implementation

TBD

Try it out

Not yet available

@freddydk freddydk converted this from a draft issue Nov 12, 2023
@freddydk freddydk linked a pull request Nov 12, 2023 that will close this issue
3 tasks
@freddydk freddydk added the enhancement New feature or request label Nov 12, 2023
@fvet
Copy link

fvet commented Mar 7, 2024

@freddydk I see this is labelled '2024 w1' (although draft PR pending). Can we expect this as part of the April release?
Not having to spin up a container for building apps would certainly decrease our build time.

@freddydk
Copy link
Contributor Author

freddydk commented Mar 7, 2024

Nope - not going to make it - will have to go through the plan and adjust, sorry

@mazhelez
Copy link
Collaborator

Splitting the build and test jobs is possible even today by splitting the apps into different AL-Go projects.

Of course, that way apps and test apps will be in different artifacts for delivery/deployment. So, I believe we'll try another approach: having the ability to bundle apps from different AL-Go projects together.

@fvet
Copy link

fvet commented Jun 20, 2024

I'm not quite sure if I follow the suggested solution, but our goal is to keep to one repository, containing all apps, so we can deploy / release them all together as one solution, similar to how the MS multi-app repo's are structured.

image

@freddydk freddydk assigned freddydk and unassigned mazhelez and freddydk Nov 19, 2024
@freddydk freddydk moved this from Grooming to 🔖 Backlog in AL-Go for GitHub Roadmap Nov 21, 2024
@freddydk freddydk moved this from 🔖 Backlog to Grooming in AL-Go for GitHub Roadmap Nov 21, 2024
@freddydk
Copy link
Contributor Author

We will document ways of doing this as part of #1331

@freddydk freddydk added the wontfix This will not be worked on label Dec 16, 2024
@freddydk freddydk self-assigned this Dec 16, 2024
@freddydk freddydk moved this from 🧹 Grooming to 👎 Removed in AL-Go for GitHub Roadmap Dec 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request wontfix This will not be worked on
Projects
Status: 👎 Removed
Development

Successfully merging a pull request may close this issue.

3 participants