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

Verbosity level #119

Merged
merged 2 commits into from
Dec 1, 2023
Merged

Verbosity level #119

merged 2 commits into from
Dec 1, 2023

Conversation

eguzki
Copy link
Contributor

@eguzki eguzki commented Nov 21, 2023

What

Deploy limitador with specific verbosity level
Currently, only verbosity level 1 to 4 (both included) are allowed. Any verbosity level out of that range should be rejected

apiVersion: limitador.kuadrant.io/v1alpha1
kind: Limitador
metadata:
  name: limitador-sample
spec:
  verbosity: 2

Verification Steps

dev setup

make local-setup

Deploy the limitador CR with verbosity level set to 2

k apply -f - <<EOF
---
apiVersion: limitador.kuadrant.io/v1alpha1
kind: Limitador
metadata:
  name: limitador-sample
spec:
  verbosity: 2
EOF

Check deployment is correct

kubectl wait --timeout=300s --for=condition=Ready limitador limitador-sample

Should return

limitador.limitador.kuadrant.io/limitador-sample condition met

Check command line

kubectl get deployment/limitador-limitador-sample  -o yaml | yq '.spec.template.spec.containers[0].command'

should look like this

- limitador-server
- -vv
- --http-port
- "8080"
- --rls-port
- "8081"
- /home/limitador/etc/limitador-config.yaml
- memory

Note: -vv has been added to the command line

Signed-off-by: Eguzki Astiz Lezaun <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Nov 21, 2023

Codecov Report

Merging #119 (16c7b03) into main (f6a01e8) will increase coverage by 0.08%.
The diff coverage is 100.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #119      +/-   ##
==========================================
+ Coverage   68.62%   68.70%   +0.08%     
==========================================
  Files          16       16              
  Lines        1157     1160       +3     
==========================================
+ Hits          794      797       +3     
  Misses        324      324              
  Partials       39       39              
Flag Coverage Δ
integration 73.77% <ø> (ø)
unit 66.37% <100.00%> (+0.12%) ⬆️

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

Components Coverage Δ
api/v1alpha1 (u) 100.00% <ø> (ø)
pkg/helpers (u) ∅ <ø> (∅)
pkg/log (u) 31.81% <ø> (ø)
pkg/reconcilers (u) 36.76% <ø> (ø)
pkg/limitador (u) 88.88% <100.00%> (+0.09%) ⬆️
controllers (i) 73.77% <ø> (ø)
Files Coverage Δ
api/v1alpha1/limitador_types.go 100.00% <ø> (ø)
pkg/limitador/deployment_options.go 93.61% <100.00%> (+0.43%) ⬆️

@eguzki eguzki marked this pull request as ready for review November 24, 2023 10:08
@eguzki eguzki requested a review from a team November 24, 2023 10:08
@eguzki
Copy link
Contributor Author

eguzki commented Nov 30, 2023

@Kuadrant/engineering ready for review!

Copy link
Contributor

@KevFan KevFan left a comment

Choose a reason for hiding this comment

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

Looks good to me ! 👍

image

@eguzki eguzki merged commit 583c7a8 into main Dec 1, 2023
11 checks passed
@eguzki eguzki deleted the verbosity-level branch December 1, 2023 08:32
@alexsnaps alexsnaps added this to the v0.8.0 milestone Dec 8, 2023
@eguzki eguzki added the kind/enhancement New feature or request label May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement New feature or request
Projects
Archived in project
Status: To test
Development

Successfully merging this pull request may close these issues.

4 participants