tyk icon indicating copy to clipboard operation
tyk copied to clipboard

Update DRL dependency, update API usage, go mod tidy

Open titpetric opened this issue 2 years ago • 7 comments

Description

Related Issue

Motivation and Context

How This Has Been Tested

Screenshots (if appropriate)

Types of changes

  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to change)
  • [ ] Refactoring or add test (improvements in base code or adds test coverage to functionality)

Checklist

  • [ ] Make sure you are requesting to pull a topic/feature/bugfix branch (right side). If pulling from your own fork, don't request your master!
  • [ ] Make sure you are making a pull request against the master branch (left side). Also, you should start your branch off our latest master.
  • [ ] My change requires a change to the documentation.
    • [ ] If you've changed APIs, describe what needs to be updated in the documentation.
    • [ ] If new config option added, ensure that it can be set via ENV variable
  • [ ] I have updated the documentation accordingly.
  • [ ] Modules and vendor dependencies have been updated; run go mod tidy && go mod vendor
  • [ ] When updating library version must provide reason/explanation for this update.
  • [ ] I have added tests to cover my changes.
  • [ ] All new and existing tests passed.
  • [ ] Check your code additions will not fail linting checks:
    • [ ] go fmt -s
    • [ ] go vet

titpetric avatar Sep 01 '22 13:09 titpetric

API tests result: success :white_check_mark: Branch used: refs/pull/4280/merge Commit:
Triggered by: pull_request (@titpetric) Execution page

Tyk-ITS avatar Sep 01 '22 13:09 Tyk-ITS

:boom: CI tests failed :see_no_evil:

CI test log

Building go plugin
go test -race -count=1 -failfast -v -timeout 15m -coverprofile=tyk.cov github.com/TykTechnologies/tyk
# github.com/TykTechnologies/drl
../../../go/pkg/mod/github.com/!tyk!technologies/[email protected]/drl.go:83:22: d.open.Swap undefined (type atomic.Value has no field or method Swap)
../../../go/pkg/mod/github.com/!tyk!technologies/[email protected]/ttlcache.go:79:22: c.open.Swap undefined (type atomic.Value has no field or method Swap)

gofmt

all ok

goimports

gateway/server.go

gogenerate

all ok

If the above are ok, please look at the run or in the Checks tab.

github-actions[bot] avatar Sep 09 '22 15:09 github-actions[bot]

:boom: CI tests failed :see_no_evil:

CI test log

Building go plugin
go test -race -count=1 -failfast -v -timeout 15m -coverprofile=tyk.cov github.com/TykTechnologies/tyk
# github.com/TykTechnologies/drl
../../../go/pkg/mod/github.com/!tyk!technologies/[email protected]/drl.go:83:22: d.open.Swap undefined (type atomic.Value has no field or method Swap)
../../../go/pkg/mod/github.com/!tyk!technologies/[email protected]/ttlcache.go:79:22: c.open.Swap undefined (type atomic.Value has no field or method Swap)

gofmt

all ok

goimports

gateway/server.go

gogenerate

all ok

If the above are ok, please look at the run or in the Checks tab.

github-actions[bot] avatar Sep 09 '22 15:09 github-actions[bot]

API tests result: failure :no_entry_sign: Branch used: refs/pull/4280/merge Commit: 2b9d8562cf9f42652463ea547d119486bbf1bc10 Triggered by: pull_request (@titpetric) Execution page

buger avatar Sep 09 '22 15:09 buger

API tests result: failure :no_entry_sign: Branch used: refs/pull/4280/merge Commit: 91890c29caad42f7fcb97a5ae01c09d632e04b37 Triggered by: pull_request (@titpetric) Execution page

buger avatar Sep 09 '22 15:09 buger

API tests result: success :white_check_mark: Branch used: refs/pull/4280/merge Commit: 3b6d55cbb18ce2c37575a2393bad45aed9cc0971 Triggered by: pull_request (@titpetric) Execution page

buger avatar Sep 09 '22 16:09 buger

API tests result: success :white_check_mark: Branch used: refs/pull/4280/merge Commit: d4c25e9a22740379fa5ca0ceecfe5b968796463d Triggered by: pull_request (@jeffy-mathew) Execution page

Tyk-ITS avatar Dec 07 '22 06:12 Tyk-ITS

:boom: CI tests failed :see_no_evil:

git-state

all ok

Please look at the run or in the Checks tab.

github-actions[bot] avatar Dec 28 '22 17:12 github-actions[bot]

API tests result: failure :no_entry_sign: Branch used: refs/pull/4280/merge Commit: 5db2479b0a96b73c103f328e4bbeaa2d66a9f9be Triggered by: pull_request (@titpetric) Execution page

Tyk-ITS avatar Dec 28 '22 17:12 Tyk-ITS

API tests result: success :white_check_mark: Branch used: refs/pull/4280/merge Commit: 9a17357ffc09249fb3b75dcf476cc895145e8909 Triggered by: pull_request (@titpetric) Execution page

Tyk-ITS avatar Dec 28 '22 17:12 Tyk-ITS

API tests result: success :white_check_mark: Branch used: refs/heads/master Commit: a3cd481fb4737c94d97d6737cdebc422682b1e64 Update DRL dependency, update API usage, go mod tidy (#4280) Triggered by: push (@titpetric) Execution page

Tyk-ITS avatar Dec 28 '22 18:12 Tyk-ITS