sivchari

Results 51 issues of sivchari
trafficstars

Currently, when we met the ValidateRequiredField error, we should compare as a raw string. This proposal aims to compare by using errors.Is/As and csting from interface to the type.

### Proposal Details ## Proposal Details ### Proposed addition: ```go func Expired(err error) bool ``` ### Motivation Currently, many codebases should check whether the error is context.Canceled or context.DeadlineExceeded like...

Proposal
LibraryProposal

**Description:** I added 2 linter, usetesting and containedctx, to golangci-lint configuration. These can diagnose common Go bad practice in codes. **Link to tracking Issue(s):** - Resolves: #issue-number **Testing:** **Documentation:**

This is a WIP which I want to do about this issue. https://github.com/kubernetes/kubernetes/pull/131774 Currently, some test code prepares test-data as a raw byte variable. These code are not supported by...

priority/backlog
kind/cleanup

**What this PR does / why we need it**: Part of #11947 **Which issue(s) this PR fixes** *(optional, in `fixes #(, fixes #, ...)` format, will close the issue(s) when...

cncf-cla: yes
size/XXL
area/provider/infrastructure-docker

**What this PR does / why we need it**: **Which issue(s) this PR fixes** *(optional, in `fixes #(, fixes #, ...)` format, will close the issue(s) when PR gets merged)*:...

cncf-cla: yes
size/L
do-not-merge/needs-area

**What this PR does / why we need it**: **Which issue(s) this PR fixes** *(optional, in `fixes #(, fixes #, ...)` format, will close the issue(s) when PR gets merged)*:...

cncf-cla: yes
size/XXL
approved
do-not-merge/hold
needs-rebase
area/ci

[usetesting linter](https://github.com/ldez/usetesting) is automatically enabled when Go 1.24 is used. This linter diagnose the calls that can be replaced by functions defined in testing package. e.g. os.Setenv -> testing.Setenv. It...

help wanted
priority/backlog
kind/cleanup
triage/accepted

/area ci /area testing /area e2e-testing close #11963 **What this PR does / why we need it**: **Which issue(s) this PR fixes** *(optional, in `fixes #(, fixes #, ...)` format,...

cncf-cla: yes
size/L
area/testing
area/ci
area/e2e-testing

#### Description Use grpc.NewClient instead of grpc.DialContext #### Link to tracking issue Fixes #13632 #### Testing #### Documentation

ready-to-merge