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

Remove Go 1.9 / AppEngine support; assume go1.12 build tag #3767

Merged
merged 5 commits into from Jul 30, 2020
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
4 changes: 0 additions & 4 deletions .travis.yml
Expand Up @@ -16,8 +16,6 @@ matrix:
env: GO111MODULE=on
- go: 1.12.x
env: GO111MODULE=on
- go: 1.9.x
env: GAE=1

go_import_path: google.golang.org/grpc

Expand All @@ -30,13 +28,11 @@ before_install:
install:
- try3() { eval "$*" || eval "$*" || eval "$*"; }
- try3 'if [[ "${GO111MODULE}" = "on" ]]; then go mod download; else make testdeps; fi'
- if [[ -n "${GAE}" ]]; then source ./install_gae.sh; make testappenginedeps; fi
- if [[ -n "${VET}" ]]; then ./vet.sh -install; fi

script:
- set -e
- if [[ -n "${TESTEXTRAS}" ]]; then examples/examples_test.sh; interop/interop_test.sh; make testsubmodule; exit 0; fi
- if [[ -n "${VET}" ]]; then ./vet.sh; fi
- if [[ -n "${GAE}" ]]; then make testappengine; exit 0; fi
- if [[ -n "${RACE}" ]]; then make testrace; exit 0; fi
- make test
1 change: 0 additions & 1 deletion CONTRIBUTING.md
Expand Up @@ -57,6 +57,5 @@ How to get your contributions merged smoothly and quickly.
- `make vet` to catch vet errors
- `make test` to run the tests
- `make testrace` to run tests in race mode
- optional `make testappengine` to run tests with appengine

- Exceptions to the rules can be made if there's a compelling reason for doing so.
8 changes: 0 additions & 8 deletions Makefile
Expand Up @@ -22,12 +22,6 @@ test: testdeps
testsubmodule: testdeps
cd security/advancedtls && go test -cpu 1,4 -timeout 7m google.golang.org/grpc/security/advancedtls/...

testappengine: testappenginedeps
goapp test -cpu 1,4 -timeout 7m google.golang.org/grpc/...

testappenginedeps:
goapp get -d -v -t -tags 'appengine appenginevm' google.golang.org/grpc/...

testdeps:
go get -d -v -t google.golang.org/grpc/...

Expand All @@ -53,8 +47,6 @@ vetdeps:
deps \
proto \
test \
testappengine \
testappenginedeps \
testdeps \
testrace \
updatedeps \
Expand Down
2 changes: 0 additions & 2 deletions balancer/rls/internal/balancer.go
@@ -1,5 +1,3 @@
// +build go1.10

/*
*
* Copyright 2020 gRPC authors.
Expand Down
2 changes: 0 additions & 2 deletions balancer/rls/internal/balancer_test.go
@@ -1,5 +1,3 @@
// +build go1.10

/*
*
* Copyright 2020 gRPC authors.
Expand Down
2 changes: 0 additions & 2 deletions balancer/rls/internal/builder.go
@@ -1,5 +1,3 @@
// +build go1.10

/*
*
* Copyright 2020 gRPC authors.
Expand Down
2 changes: 0 additions & 2 deletions balancer/rls/internal/client_test.go
@@ -1,5 +1,3 @@
// +build go1.10

/*
*
* Copyright 2020 gRPC authors.
Expand Down
2 changes: 0 additions & 2 deletions balancer/rls/internal/config.go
@@ -1,5 +1,3 @@
// +build go1.10

/*
*
* Copyright 2020 gRPC authors.
Expand Down
2 changes: 0 additions & 2 deletions balancer/rls/internal/config_test.go
@@ -1,5 +1,3 @@
// +build go1.10

/*
*
* Copyright 2020 gRPC authors.
Expand Down
2 changes: 0 additions & 2 deletions balancer/rls/internal/keys/builder.go
@@ -1,5 +1,3 @@
// +build go1.10

/*
*
* Copyright 2020 gRPC authors.
Expand Down
2 changes: 0 additions & 2 deletions balancer/rls/internal/keys/builder_test.go
@@ -1,5 +1,3 @@
// +build go1.10

/*
*
* Copyright 2020 gRPC authors.
Expand Down
2 changes: 0 additions & 2 deletions balancer/rls/internal/picker.go
@@ -1,5 +1,3 @@
// +build go1.10

/*
*
* Copyright 2020 gRPC authors.
Expand Down
2 changes: 0 additions & 2 deletions balancer/rls/internal/picker_test.go
@@ -1,5 +1,3 @@
// +build go1.10

/*
*
* Copyright 2020 gRPC authors.
Expand Down
2 changes: 0 additions & 2 deletions channelz/service/func_linux.go
@@ -1,5 +1,3 @@
// +build !appengine

/*
*
* Copyright 2018 gRPC authors.
Expand Down
2 changes: 1 addition & 1 deletion channelz/service/func_nonlinux.go
@@ -1,4 +1,4 @@
// +build !linux appengine
// +build !linux

/*
*
Expand Down
2 changes: 1 addition & 1 deletion channelz/service/service_sktopt_test.go
@@ -1,4 +1,4 @@
// +build linux,!appengine
// +build linux
// +build 386 amd64

/*
Expand Down
2 changes: 1 addition & 1 deletion channelz/service/service_test.go
Expand Up @@ -58,7 +58,7 @@ type protoToSocketOptFunc func([]*channelzpb.SocketOption) *channelz.SocketOptio

// protoToSocketOpt is used in function socketProtoToStruct to extract socket option
// data from unmarshaled proto message.
// It is only defined under linux, non-appengine environment on x86 architecture.
// It is only defined under linux environment on x86 architecture.
var protoToSocketOpt protoToSocketOptFunc

// emptyTime is used for detecting unset value of time.Time type.
Expand Down
2 changes: 1 addition & 1 deletion channelz/service/util_sktopt_386_test.go
@@ -1,4 +1,4 @@
// +build 386,linux,!appengine
// +build 386,linux

/*
*
Expand Down
2 changes: 1 addition & 1 deletion channelz/service/util_sktopt_amd64_test.go
@@ -1,4 +1,4 @@
// +build amd64,linux,!appengine
// +build amd64,linux

/*
*
Expand Down
30 changes: 0 additions & 30 deletions credentials/go12.go

This file was deleted.

2 changes: 0 additions & 2 deletions credentials/internal/syscallconn.go
@@ -1,5 +1,3 @@
// +build !appengine

/*
*
* Copyright 2018 gRPC authors.
Expand Down
30 changes: 0 additions & 30 deletions credentials/internal/syscallconn_appengine.go

This file was deleted.

2 changes: 0 additions & 2 deletions credentials/internal/syscallconn_test.go
@@ -1,5 +1,3 @@
// +build !appengine

/*
*
* Copyright 2018 gRPC authors.
Expand Down
4 changes: 4 additions & 0 deletions credentials/tls.go
Expand Up @@ -241,6 +241,10 @@ var cipherSuiteLookup = map[uint16]string{
tls.TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256: "TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256",
tls.TLS_ECDHE_RSA_WITH_CHACHA20_POLY1305: "TLS_ECDHE_RSA_WITH_CHACHA20_POLY1305",
tls.TLS_ECDHE_ECDSA_WITH_CHACHA20_POLY1305: "TLS_ECDHE_ECDSA_WITH_CHACHA20_POLY1305",
// Go 1.12
tls.TLS_AES_128_GCM_SHA256: "TLS_AES_128_GCM_SHA256",
tls.TLS_AES_256_GCM_SHA384: "TLS_AES_256_GCM_SHA384",
tls.TLS_CHACHA20_POLY1305_SHA256: "TLS_CHACHA20_POLY1305_SHA256",
}

// cloneTLSConfig returns a shallow clone of the exported
Expand Down
2 changes: 0 additions & 2 deletions examples/features/xds/client/main.go
@@ -1,5 +1,3 @@
// +build go1.11

/*
*
* Copyright 2020 gRPC authors.
Expand Down
2 changes: 0 additions & 2 deletions examples/features/xds/server/main.go
@@ -1,5 +1,3 @@
// +build go1.11

/*
*
* Copyright 2020 gRPC authors.
Expand Down
6 changes: 0 additions & 6 deletions install_gae.sh

This file was deleted.

2 changes: 0 additions & 2 deletions internal/channelz/types_linux.go
@@ -1,5 +1,3 @@
// +build !appengine

/*
*
* Copyright 2018 gRPC authors.
Expand Down
4 changes: 2 additions & 2 deletions internal/channelz/types_nonlinux.go
@@ -1,4 +1,4 @@
// +build !linux appengine
// +build !linux

/*
*
Expand Down Expand Up @@ -37,6 +37,6 @@ type SocketOptionData struct {
// Windows OS doesn't support Socket Option
func (s *SocketOptionData) Getsockopt(fd uintptr) {
once.Do(func() {
logger.Warning("Channelz: socket options are not supported on non-linux os and appengine.")
logger.Warning("Channelz: socket options are not supported on non-linux os.")
})
}
2 changes: 1 addition & 1 deletion internal/channelz/util_linux.go
@@ -1,4 +1,4 @@
// +build linux,!appengine
// +build linux

/*
*
Expand Down
2 changes: 1 addition & 1 deletion internal/channelz/util_nonlinux.go
@@ -1,4 +1,4 @@
// +build !linux appengine
// +build !linux

/*
*
Expand Down
2 changes: 1 addition & 1 deletion internal/channelz/util_test.go
@@ -1,4 +1,4 @@
// +build linux,go1.10,!appengine
// +build linux

/*
*
Expand Down
2 changes: 0 additions & 2 deletions internal/credentials/go110.go
@@ -1,5 +1,3 @@
// +build go1.10
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rename this file?
Or delete?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Renamed.


/*
*
* Copyright 2020 gRPC authors.
Expand Down
2 changes: 0 additions & 2 deletions internal/credentials/go110_test.go
@@ -1,5 +1,3 @@
// +build go1.10

/*
*
* Copyright 2020 gRPC authors.
Expand Down
31 changes: 0 additions & 31 deletions internal/credentials/gobefore110.go

This file was deleted.

2 changes: 0 additions & 2 deletions internal/profiling/buffer/buffer.go
@@ -1,5 +1,3 @@
// +build !appengine

/*
*
* Copyright 2019 gRPC authors.
Expand Down
43 changes: 0 additions & 43 deletions internal/profiling/buffer/buffer_appengine.go

This file was deleted.

2 changes: 0 additions & 2 deletions internal/profiling/buffer/buffer_test.go
@@ -1,5 +1,3 @@
// +build !appengine

/*
*
* Copyright 2019 gRPC authors.
Expand Down
2 changes: 0 additions & 2 deletions internal/profiling/profiling_test.go
@@ -1,5 +1,3 @@
// +build !appengine

/*
*
* Copyright 2019 gRPC authors.
Expand Down