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.

Sign up for GitHub

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: add egctl ratelimit config support #2674

Merged
merged 3 commits into from
Feb 26, 2024

fix: add todo information and supplementary function comments

af39b78
Select commit
Loading
Failed to load commit list.
Merged

feat: add egctl ratelimit config support #2674

fix: add todo information and supplementary function comments
af39b78
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Feb 26, 2024 in 1s

34.26% of diff hit (target 60.00%)

View this Pull Request on Codecov

34.26% of diff hit (target 60.00%)

Annotations

Check warning on line 83 in internal/cmd/egctl/config.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/egctl/config.go#L83 

Added line #L83 was not covered by tests

Check warning on line 26 in internal/cmd/egctl/config_cmd.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/egctl/config_cmd.go#L26 

Added line #L26 was not covered by tests

Check warning on line 40 in internal/cmd/egctl/config_cmd.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/egctl/config_cmd.go#L39-L40 

Added lines #L39 - L40 were not covered by tests

Check warning on line 51 in internal/cmd/egctl/config_ratelimit.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/egctl/config_ratelimit.go#L33-L51 

Added lines #L33 - L51 were not covered by tests

Check warning on line 55 in internal/cmd/egctl/config_ratelimit.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/egctl/config_ratelimit.go#L54-L55 

Added lines #L54 - L55 were not covered by tests

Check warning on line 63 in internal/cmd/egctl/config_ratelimit.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/egctl/config_ratelimit.go#L58-L63 

Added lines #L58 - L63 were not covered by tests

Check warning on line 68 in internal/cmd/egctl/config_ratelimit.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/egctl/config_ratelimit.go#L65-L68 

Added lines #L65 - L68 were not covered by tests

Check warning on line 71 in internal/cmd/egctl/config_ratelimit.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/egctl/config_ratelimit.go#L70-L71 

Added lines #L70 - L71 were not covered by tests

Check warning on line 82 in internal/cmd/egctl/config_ratelimit.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/egctl/config_ratelimit.go#L74-L82 

Added lines #L74 - L82 were not covered by tests

Check warning on line 88 in internal/cmd/egctl/config_ratelimit.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/egctl/config_ratelimit.go#L85-L88 

Added lines #L85 - L88 were not covered by tests

Check warning on line 97 in internal/cmd/egctl/config_ratelimit.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/egctl/config_ratelimit.go#L93-L97 

Added lines #L93 - L97 were not covered by tests

Check warning on line 99 in internal/cmd/egctl/config_ratelimit.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/egctl/config_ratelimit.go#L99 

Added line #L99 was not covered by tests

Check warning on line 111 in internal/cmd/egctl/config_ratelimit.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/egctl/config_ratelimit.go#L110-L111 

Added lines #L110 - L111 were not covered by tests

Check warning on line 138 in internal/cmd/egctl/config_ratelimit.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/egctl/config_ratelimit.go#L137-L138 

Added lines #L137 - L138 were not covered by tests

Check warning on line 144 in internal/cmd/egctl/config_ratelimit.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/egctl/config_ratelimit.go#L143-L144 

Added lines #L143 - L144 were not covered by tests

Check warning on line 158 in internal/cmd/egctl/config_ratelimit.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/egctl/config_ratelimit.go#L157-L158 

Added lines #L157 - L158 were not covered by tests

Check warning on line 163 in internal/cmd/egctl/config_ratelimit.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/egctl/config_ratelimit.go#L162-L163 

Added lines #L162 - L163 were not covered by tests

Check warning on line 169 in internal/cmd/egctl/config_ratelimit.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/egctl/config_ratelimit.go#L168-L169 

Added lines #L168 - L169 were not covered by tests

Check warning on line 175 in internal/cmd/egctl/config_ratelimit.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/egctl/config_ratelimit.go#L174-L175 

Added lines #L174 - L175 were not covered by tests

Check warning on line 180 in internal/cmd/egctl/config_ratelimit.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/egctl/config_ratelimit.go#L179-L180 

Added lines #L179 - L180 were not covered by tests

Check warning on line 195 in internal/cmd/egctl/config_ratelimit.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/egctl/config_ratelimit.go#L194-L195 

Added lines #L194 - L195 were not covered by tests

Check warning on line 200 in internal/cmd/egctl/config_ratelimit.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/egctl/config_ratelimit.go#L199-L200 

Added lines #L199 - L200 were not covered by tests

Check warning on line 149 in internal/cmd/egctl/envoy_stats.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/egctl/envoy_stats.go#L149 

Added line #L149 was not covered by tests

Check warning on line 172 in internal/cmd/egctl/envoy_stats.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/cmd/egctl/envoy_stats.go#L172 

Added line #L172 was not covered by tests

Check warning on line 106 in internal/infrastructure/kubernetes/ratelimit/resource.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

internal/infrastructure/kubernetes/ratelimit/resource.go#L101-L106 

Added lines #L101 - L106 were not covered by tests
View more details on Codecov