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

Release adjustments and documentation #834

Merged
merged 3 commits into from
Aug 21, 2024
Merged

Conversation

didierofrivia
Copy link
Collaborator

This PR removes unnecessary env var CHANNELS and adds the step to follow in order to release a X.Y.Z version of the Kuadrant Operator

@didierofrivia didierofrivia self-assigned this Aug 21, 2024
Signed-off-by: dd di cesare <didi@posteo.net>
Signed-off-by: dd di cesare <didi@posteo.net>
Signed-off-by: dd di cesare <didi@posteo.net>
Copy link

codecov bot commented Aug 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.97%. Comparing base (ece13e8) to head (da63966).
Report is 172 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #834      +/-   ##
==========================================
+ Coverage   80.20%   81.97%   +1.76%     
==========================================
  Files          64       76      +12     
  Lines        4492     6090    +1598     
==========================================
+ Hits         3603     4992    +1389     
- Misses        600      749     +149     
- Partials      289      349      +60     
Flag Coverage Δ
bare-k8s-integration 4.50% <ø> (?)
controllers-integration 71.92% <ø> (?)
gatewayapi-integration 10.95% <ø> (?)
integration ?
istio-integration 56.23% <ø> (?)
unit 31.58% <ø> (+1.55%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
api/v1beta1 (u) 71.42% <ø> (ø)
api/v1beta2 (u) 85.35% <80.00%> (-6.08%) ⬇️
pkg/common (u) 88.13% <ø> (-0.70%) ⬇️
pkg/istio (u) 72.50% <ø> (-1.41%) ⬇️
pkg/log (u) 94.73% <ø> (ø)
pkg/reconcilers (u) ∅ <ø> (∅)
pkg/rlptools (u) 83.64% <ø> (+4.19%) ⬆️
controllers (i) 82.33% <82.24%> (+5.53%) ⬆️

see 42 files with indirect coverage changes

@didierofrivia didierofrivia added the area/doc Improvements or additions to documentation label Aug 21, 2024

2. Create a new directory `operators/kuadrant-operator/0.W.Z` containing:

* Copy the bundle files from `github.com/kuadrant/kuadrant-operator/tree/v0.W.Z/bundle`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

openshift release requires some bits we do not have in this repo:

in metadata/annotations.yaml, the target catalog version

com.redhat.openshift.versions: "v4.12"

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also, replaces field in the CSV is not needed.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd say let's shoot another PR to fix the OpertaorHub/OpenShift operators publishing, since it's not needed until GA. WDYT?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also, it will need to be changed all along the dependencies

@didierofrivia didierofrivia merged commit acad4f2 into main Aug 21, 2024
29 checks passed
@didierofrivia didierofrivia deleted the release-adjustments branch August 21, 2024 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/doc Improvements or additions to documentation
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants