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

🚧 QA-689 wip #512

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

lluiscampos
Copy link
Member

No description provided.

@mender-test-bot
Copy link

There was an error running your pipeline, see logs for details.

@lluiscampos lluiscampos force-pushed the QA-689-refactor-cicd-part2 branch 4 times, most recently from 0ebfbf2 to 8de0866 Compare March 6, 2025 13:09
Ticket: QA-689

Signed-off-by: Lluis Campos <lluis.campos@northern.tech>
@lluiscampos lluiscampos force-pushed the QA-689-refactor-cicd-part2 branch from 8de0866 to f4243a9 Compare March 6, 2025 19:12
The default behaviour does not change: a pipeline with the default
values (all versions to `master`) will build, test and potentially
publish all packages.

However on demand, by setting any of the packages versions to empty
string, these won't be built/tested/published. This allows for release
builds were only one or a subset of packages are needed.

Two main changes:
* On the test level, mark all tests with per-package markers and add
  some logic in the collection of tests to skip the once we don't have
  version information about
* On the CI/CD level, refactor the whole pipeline to produce CI jobs per
  package. This seems like an overkill but the explicitness of it is
  preferable to complex alternatives such a dynamic pipelines.

Ticket: QA-689

Signed-off-by: Lluis Campos <lluis.campos@northern.tech>
@lluiscampos lluiscampos force-pushed the QA-689-refactor-cicd-part2 branch from f4243a9 to 8246c8c Compare March 7, 2025 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants