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

feat: use kuma version when installing it #949

Merged
merged 6 commits into from
Feb 5, 2024
Merged

Conversation

czeslavo
Copy link
Contributor

@czeslavo czeslavo commented Feb 2, 2024

In Kuma addon builder we allow setting a version, but we don't use it when calling helm install.

@codecov-commenter
Copy link

codecov-commenter commented Feb 2, 2024

Codecov Report

Attention: 9 lines in your changes are missing coverage. Please review.

Comparison is base (bf671d1) 60.19% compared to head (d281ff1) 42.41%.
Report is 1 commits behind head on main.

Files Patch % Lines
pkg/clusters/addons/kuma/addon.go 0.00% 8 Missing ⚠️
pkg/clusters/addons/kuma/builder.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##             main     #949       +/-   ##
===========================================
- Coverage   60.19%   42.41%   -17.79%     
===========================================
  Files          50       50               
  Lines        4032     4039        +7     
===========================================
- Hits         2427     1713      -714     
- Misses       1300     2089      +789     
+ Partials      305      237       -68     
Flag Coverage Δ
integration-test 42.41% <0.00%> (-17.79%) ⬇️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

pkg/clusters/addons/kuma/addon.go Outdated Show resolved Hide resolved
pkg/clusters/addons/kuma/addon.go Show resolved Hide resolved
Copy link
Member

@pmalek pmalek left a comment

Choose a reason for hiding this comment

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

One nit, I'd have is to include this change in the changelog. Other than that looks good.

programmer04
programmer04 previously approved these changes Feb 5, 2024
pkg/clusters/addons/kuma/addon.go Outdated Show resolved Hide resolved
@programmer04
Copy link
Member

When your PR will be merged and this #946 too I think we can release a new version

Co-authored-by: Jakub Warczarek <jakub.warczarek@konghq.com>
programmer04
programmer04 previously approved these changes Feb 5, 2024
pmalek
pmalek previously approved these changes Feb 5, 2024
programmer04
programmer04 previously approved these changes Feb 5, 2024
Copy link
Member

@programmer04 programmer04 left a comment

Choose a reason for hiding this comment

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

👍

@czeslavo czeslavo merged commit fe761fa into main Feb 5, 2024
41 checks passed
@czeslavo czeslavo deleted the feat/use-kuma-version branch February 5, 2024 14:23
randmonkey pushed a commit that referenced this pull request Mar 27, 2024
Co-authored-by: Jakub Warczarek <jakub.warczarek@konghq.com>
randmonkey pushed a commit that referenced this pull request Mar 27, 2024
Co-authored-by: Jakub Warczarek <jakub.warczarek@konghq.com>
rainest pushed a commit that referenced this pull request Mar 27, 2024
Co-authored-by: Jakub Warczarek <jakub.warczarek@konghq.com>
czeslavo added a commit that referenced this pull request Mar 29, 2024
* feat: install default allow-all traffic permission when kuma >= 2.6.0 (#957)

* feat: use kuma version when installing it (#949)

Co-authored-by: Jakub Warczarek <jakub.warczarek@konghq.com>

* chore: test on PRs to release branches

* run lint workflow for all branches

* fix linter issues

---------

Co-authored-by: Grzegorz Burzyński <czeslavo@gmail.com>
Co-authored-by: Jakub Warczarek <jakub.warczarek@konghq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants