-
Notifications
You must be signed in to change notification settings - Fork 22
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
Release workflow #56
Release workflow #56
Conversation
Signed-off-by: Michael Valdron <[email protected]>
Signed-off-by: Michael Valdron <[email protected]>
Signed-off-by: Michael Valdron <[email protected]>
Signed-off-by: Michael Valdron <[email protected]>
Signed-off-by: Michael Valdron <[email protected]>
Signed-off-by: Michael Valdron <[email protected]>
Signed-off-by: Michael Valdron <[email protected]>
Signed-off-by: Michael Valdron <[email protected]>
Signed-off-by: Michael Valdron <[email protected]>
Signed-off-by: Michael Valdron <[email protected]>
Signed-off-by: Michael Valdron <[email protected]>
Signed-off-by: Michael Valdron <[email protected]>
Signed-off-by: Michael Valdron <[email protected]>
Signed-off-by: Michael Valdron <[email protected]>
Signed-off-by: Michael Valdron <[email protected]>
Signed-off-by: Michael Valdron <[email protected]>
Signed-off-by: Michael Valdron <[email protected]>
… integration testing Signed-off-by: Michael Valdron <[email protected]>
Skipping CI for Draft Pull Request. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #56 +/- ##
=======================================
Coverage 26.43% 26.43%
=======================================
Files 25 25
Lines 1415 1415
=======================================
Hits 374 374
Misses 1020 1020
Partials 21 21 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Michael Valdron <[email protected]>
baabc2f
to
46a57f1
Compare
With these changes release tags trigger the "Release" workflow which first validates the places the version is defined and then pushes the images alongside running integration tests on k8s 1.25 & 1.26 (does not pass due to timeouts, see devfile/api#1313 for more details): If version check fails, the other jobs do not run: |
OpenShift release testing PR: openshift/release#44995 |
N/A |
Integration testing in OpenShift passes, but on Kubernetes it times out. See devfile/api#1313. |
Signed-off-by: Michael Valdron <[email protected]>
Co-authored-by: Armel Soro <[email protected]> Signed-off-by: Michael Valdron <[email protected]>
791cf2a
to
cb65ba2
Compare
Signed-off-by: Michael Valdron <[email protected]>
Signed-off-by: Michael Valdron <[email protected]>
Signed-off-by: Michael Valdron <[email protected]>
Signed-off-by: Michael Valdron <[email protected]>
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: michael-valdron, rm3l The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
* release workflow Signed-off-by: Michael Valdron <[email protected]> * deploy minikube ci script Signed-off-by: Michael Valdron <[email protected]> * deploy operator job Signed-off-by: Michael Valdron <[email protected]> * set imagePullPolicy to 'Never' to use local built image Signed-off-by: Michael Valdron <[email protected]> * release deployment checks on kubernetes 1.25 & 1.26 Signed-off-by: Michael Valdron <[email protected]> * publishing operator bundle requires release deploy checks to pass Signed-off-by: Michael Valdron <[email protected]> * remove unnecessary ingress minikube addon Signed-off-by: Michael Valdron <[email protected]> * olm deploy check Signed-off-by: Michael Valdron <[email protected]> * version check Signed-off-by: Michael Valdron <[email protected]> * fix yq actions Signed-off-by: Michael Valdron <[email protected]> * download yq locally for version check Signed-off-by: Michael Valdron <[email protected]> * install operator-sdk Signed-off-by: Michael Valdron <[email protected]> * operator-sdk makefile variable Signed-off-by: Michael Valdron <[email protected]> * skip olm deploy check Signed-off-by: Michael Valdron <[email protected]> * setup go & operator-sdk for deploy checks Signed-off-by: Michael Valdron <[email protected]> * convert kubernetes deploy checks into kubernetes integration tests Signed-off-by: Michael Valdron <[email protected]> * release ci labelling fixups Signed-off-by: Michael Valdron <[email protected]> * add additional time for registry operator to setup before running k8s integration testing Signed-off-by: Michael Valdron <[email protected]> * make k8s integration testing not required for release image pushing Signed-off-by: Michael Valdron <[email protected]> * pin actions to commit SHA Signed-off-by: Michael Valdron <[email protected]> * use go.mod version Co-authored-by: Armel Soro <[email protected]> Signed-off-by: Michael Valdron <[email protected]> * set up go after source checkout Signed-off-by: Michael Valdron <[email protected]> * use gh provided CI variable instead parameter Signed-off-by: Michael Valdron <[email protected]> * use strategy.matrix for k8s version for integration testing Signed-off-by: Michael Valdron <[email protected]> * quote CI variable Signed-off-by: Michael Valdron <[email protected]> --------- Signed-off-by: Michael Valdron <[email protected]> Co-authored-by: Armel Soro <[email protected]> Signed-off-by: thepetk <[email protected]>
Please specify the area for this PR
ci
registry operator
What does does this PR do / why we need it:
Includes addition of a release workflow to automate the container image publication process.
If workflow is successful, it will create new release tags for
quay.io/devfile/registry-operator
andquay.io/devfile/registry-operator-bundle
, release tags should be eitherv[0-9].[0-9].[0-9]
orv[0-9].[0-9].[0-9]-rc.[0-9]
pattern.Which issue(s) this PR fixes:
Fixes #?
fixes devfile/api#1267
PR acceptance criteria:
Documentation
How to test changes / Special notes to the reviewer: