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

spanner: cannot find module providing package cel.dev/expr #9031

Closed
LeVadim opened this issue Nov 18, 2023 · 9 comments · Fixed by cncf/xds#89
Closed

spanner: cannot find module providing package cel.dev/expr #9031

LeVadim opened this issue Nov 18, 2023 · 9 comments · Fixed by cncf/xds#89
Assignees
Labels
api: spanner Issues related to the Spanner API. priority: p3 Desirable enhancement or fix. May not be included in next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Comments

@LeVadim
Copy link

LeVadim commented Nov 18, 2023

Client

Spanner

Environment

Local OSX Sillicon

Go Environment

$ go version
go version - go1.21.4 darwin/arm64

$ go env
GO111MODULE='on'
GOARCH='arm64'
GOBIN=''
GOCACHE='/Users/Macbook/Library/Caches/go-build'
GOENV='/Users/Macbook/Library/Application Support/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='arm64'
GOHOSTOS='darwin'
GOINSECURE=''
GOMODCACHE='/Users/Macbook/go/pkg/mod'
GONOPROXY='github.com/...'
GONOSUMDB='github.com/...'
GOOS='darwin'
GOPATH='/Users/Macbook/go'
GOPRIVATE='github.com/...'
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/usr/local/go'
GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/usr/local/go/pkg/tool/darwin_arm64'
GOVCS=''
GOVERSION='go1.21.4'
GCCGO='gccgo'
AR='ar'
CC='clang'
CXX='clang++'
CGO_ENABLED='1'
GOMOD='/Users/Macbook/...'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -ffile-prefix-map=/var/folders/dm/zy86pnt94t10jbc9q8_1scg00000gp/T/go-build556399722=/tmp/go-build -gno-record-gcc-switches -fno-common'

Code
go get -u cloud.google.com/go/spanner

go: downloading github.com/googleapis/gax-go v1.0.3
go: downloading github.com/cncf/xds v0.0.0-20231117232044-0f5e0d9dbc12
go: downloading github.com/cespare/xxhash v1.1.0
go: downloading github.com/cncf/udpa v0.0.1
go: cloud.google.com/go/spanner imports
google.golang.org/grpc/xds/googledirectpath imports
google.golang.org/grpc/xds imports
google.golang.org/grpc/xds/internal/xdsclient/xdslbregistry/converter imports
github.com/cncf/xds/go/xds/type/v3 imports
cel.dev/expr: cannot find module providing package cel.dev/expr

go mod tidy gives the same error

Expected behavior
The download or go mod tidy would work without errors

Actual behavior
It crashes with errror: cel.dev/expr: cannot find module providing package cel.dev/expr

Screenshots
Screenshot 2023-11-18 at 2 57 36 PM

@LeVadim LeVadim added the triage me I really want to be triaged. label Nov 18, 2023
@product-auto-label product-auto-label bot added the api: spanner Issues related to the Spanner API. label Nov 18, 2023
@LeVadim
Copy link
Author

LeVadim commented Nov 18, 2023

@arhea Helped under linked github issue

Solution to add this replace statement helped

replace github.com/cncf/xds => github.com/cncf/xds v0.0.0-20231116142803-3a472e524827

@arhea
Copy link

arhea commented Nov 18, 2023

*Temporary fix that reverts the previous version. Once cncf/xds#76 is fixed, this should be removed. Looks like it is just a problem with the cel.dev domain which I'm sure will get fixed soon.

@sergiitk
Copy link

Yes, CEL folks are working on it: cncf/xds#76 (comment)

@sergiitk
Copy link

FYI - the change that caused cncf/xds#76 has been rolled back in cncf/xds#79 until the CEL team fixes the domain settings.

@noahdietz noahdietz added type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. priority: p2 Moderately-important priority. Fix may not be included in next release. and removed triage me I really want to be triaged. labels Nov 29, 2023
@noahdietz
Copy link
Contributor

@sergiitk I'm just here triaging bugs, will this be fixed with a new xds release/dependency update?

@noahdietz noahdietz added priority: p3 Desirable enhancement or fix. May not be included in next release. and removed priority: p2 Moderately-important priority. Fix may not be included in next release. labels Nov 29, 2023
@rahul2393
Copy link
Contributor

sergiitk Is it still happening?
If not please help closing the ticket.

@sergiitk
Copy link

sergiitk commented Feb 2, 2024

Yes, this is still happening. I had a chat with CEL folks this week, they're close to rolling out the domain settings with necessary redirects.

@sergiitk
Copy link

The fix: cncf/xds#89.

@sergiitk
Copy link

sergiitk commented Apr 1, 2024

FYI The fix has been merged. @rahul2393 feel free to close this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: spanner Issues related to the Spanner API. priority: p3 Desirable enhancement or fix. May not be included in next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants