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

ci: "upstream" packages validation workflow #4455

Merged
merged 76 commits into from
Dec 9, 2024
Merged

Conversation

ppawlowski
Copy link
Contributor

@ppawlowski ppawlowski commented Sep 5, 2024

Description

This pull request extends the existing Create pre-staging environment workflow by adding the possibility to test upstream packages on pre-staging environment.
In this iteration, the following packages can be verified:

  • flowfuse/driver-k8s
  • flowfuse/nr-launcher

Major changes:

  • publish_k8s_driver and publish_nr_launcher jobs - responsible for building and publishing driver-k8s and nr-launcher node packages when other than the default branch is specified for a particular package
  • build-node-red, upload-node-red and create-custom-stack jobs - responsible for building and publishing Node-RED container image as well as creating custom Stack on pre-staging environment to be consumed after login to the FlowFuse pre-staging instance. Executed if the nr-launcher package is about to be verified.
  • Introduced Node-RED dockerfile and supporting files to build NR container without pulling the whole flowfuse/helm repository. This may change in the future, though.

Additionally, workflow received a bunch of improvements around conditional statements for particular jobs. All of the changes ensure that core functionality will work as previously and no additional jobs won't be executed.

Related Issue(s)

https://github.com/FlowFuse/CloudProject/issues/389

Checklist

  • I have read the contribution guidelines
  • Suitable unit/system level tests have been added and they pass
  • Documentation has been updated
    • Upgrade instructions
    • Configuration details
    • Concepts
  • Changes flowforge.yml?
    • Issue/PR raised on FlowFuse/helm to update ConfigMap Template
    • Issue/PR raised on FlowFuse/CloudProject to update values for Staging/Production

Labels

  • Includes a DB migration? -> add the area:migration label

@ppawlowski ppawlowski marked this pull request as ready for review December 6, 2024 13:00
@ppawlowski ppawlowski requested a review from hardillb December 6, 2024 13:01
@hardillb hardillb merged commit 862fd54 into main Dec 9, 2024
19 checks passed
@hardillb hardillb deleted the ci-upstream-prestaging branch December 9, 2024 10:46
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