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

Revert "Bump github.com/dgraph-io/ristretto from 0.0.3 to 0.1.0 (#8500)" #9050

Conversation

clamoriniere
Copy link
Contributor

This reverts commit dad900a.

What does this PR do?

A brief description of the change being made with this pull request.

Motivation

What inspired you to submit this pull request?

Additional Notes

Anything else we should know when reviewing?

Describe how to test your changes

Write here in detail how you have tested your changes
and instructions on how this should be tested in QA.

Describe or link instructions to set up environment
for testing, if the process requires more than just
running the agent on one of the supported platforms.

Checklist

  • A release note has been added or the changelog/no-changelog label has been applied.
  • The need-change/operator and need-change/helm labels has been applied if applicable.
  • The appropriate team/.. label has been applied, if known.
  • If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
  • The config template has been updated if applicable.

Note: Adding GitHub labels is only possible for contributors with write access.

@L3n41c
Copy link
Member

L3n41c commented Sep 2, 2021

I found those errors in AppVeyor:

=== FAIL: pkg/network/config TestIgnoreConntrackInitFailure (unknown)
panic: Failed to find GetQueuedCompletionStatus procedure in kernel32.dll: The specified procedure could not be found.

goroutine 48 [running]:
golang.org/x/sys/windows.(*LazyProc).mustFind(...)
	C:/gopath/pkg/mod/golang.org/x/[email protected]/windows/dll_windows.go:325
golang.org/x/sys/windows.(*LazyProc).Addr(...)
	C:/gopath/pkg/mod/golang.org/x/[email protected]/windows/dll_windows.go:333
github.com/Microsoft/go-winio.getQueuedCompletionStatus(0x12c, 0xc0004bff9c, 0xc0004bffa0, 0xc0004bffa8, 0xc0ffffffff, 0x206ce75, 0x3431880)
	C:/gopath/pkg/mod/github.com/!microsoft/[email protected]/zsyscall_windows.go:357 +0x15b
github.com/Microsoft/go-winio.ioCompletionProcessor(0x12c)
	C:/gopath/pkg/mod/github.com/!microsoft/[email protected]/file.go:167 +0xb6
created by github.com/Microsoft/go-winio.initIo
	C:/gopath/pkg/mod/github.com/!microsoft/[email protected]/file.go:73 +0x85

=== FAIL: pkg/network/config TestIgnoreConntrackInitFailure/via_YAML (unknown)
panic: Failed to find GetQueuedCompletionStatus procedure in kernel32.dll: The specified procedure could not be found.

goroutine 48 [running]:
golang.org/x/sys/windows.(*LazyProc).mustFind(...)
	C:/gopath/pkg/mod/golang.org/x/[email protected]/windows/dll_windows.go:325
golang.org/x/sys/windows.(*LazyProc).Addr(...)
	C:/gopath/pkg/mod/golang.org/x/[email protected]/windows/dll_windows.go:333
github.com/Microsoft/go-winio.getQueuedCompletionStatus(0x12c, 0xc0004bff9c, 0xc0004bffa0, 0xc0004bffa8, 0xc0ffffffff, 0x206ce75, 0x3431880)
	C:/gopath/pkg/mod/github.com/!microsoft/[email protected]/zsyscall_windows.go:357 +0x15b
github.com/Microsoft/go-winio.ioCompletionProcessor(0x12c)
	C:/gopath/pkg/mod/github.com/!microsoft/[email protected]/file.go:167 +0xb6
created by github.com/Microsoft/go-winio.initIo
	C:/gopath/pkg/mod/github.com/!microsoft/[email protected]/file.go:73 +0x85

@clamoriniere clamoriniere merged commit 5391973 into main Sep 2, 2021
@clamoriniere clamoriniere deleted the revert-8500-dependabot/go_modules/github.com/dgraph-io/ristretto-0.1.0 branch September 2, 2021 15:10
L3n41c pushed a commit that referenced this pull request Sep 2, 2021
clamoriniere added a commit that referenced this pull request Sep 2, 2021
zandrewitte pushed a commit to StackVista/stackstate-agent that referenced this pull request Nov 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants