Skip to content
This repository was archived by the owner on Oct 7, 2020. It is now read-only.

Commit cc3b955

Browse files
Steven Dakeistio-testing
Steven Dake
authored andcommitted
Update common files (#634)
* Update common files * Use same image as CI so generators work correctly
1 parent c964768 commit cc3b955

File tree

7 files changed

+406
-398
lines changed

7 files changed

+406
-398
lines changed

Makefile

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,8 @@ ifeq ($(LOCAL_ARCH),x86_64)
3333
TARGET_ARCH ?= amd64
3434
else ifeq ($(shell echo $(LOCAL_ARCH) | head -c 5),armv8)
3535
TARGET_ARCH ?= arm64
36+
else ifeq ($(LOCAL_ARCH),aarch64)
37+
TARGET_ARCH ?= arm64
3638
else ifeq ($(shell echo $(LOCAL_ARCH) | head -c 4),armv)
3739
TARGET_ARCH ?= arm
3840
else
@@ -56,7 +58,7 @@ ifeq ($(BUILD_WITH_CONTAINER),1)
5658
export TARGET_OUT = /work/out/$(TARGET_OS)_$(TARGET_ARCH)
5759
CONTAINER_CLI ?= docker
5860
DOCKER_SOCKET_MOUNT ?= -v /var/run/docker.sock:/var/run/docker.sock
59-
IMG ?= gcr.io/istio-testing/build-tools:master-2019-11-08T17-21-18
61+
IMG ?= gcr.io/istio-testing/build-tools:master-2019-11-26T07-29-27
6062
UID = $(shell id -u)
6163
GID = `grep docker /etc/group | cut -f3 -d:`
6264
PWD = $(shell pwd)
@@ -95,6 +97,7 @@ RUN = $(CONTAINER_CLI) run -t -i --sig-proxy=true -u $(UID):$(GID) --rm \
9597
-e TARGET_ARCH="$(TARGET_ARCH)" \
9698
-e TARGET_OS="$(TARGET_OS)" \
9799
-e TARGET_OUT="$(TARGET_OUT)" \
100+
-e USER="${USER}" \
98101
$(ENV_VARS) \
99102
-v /etc/passwd:/etc/passwd:ro \
100103
$(DOCKER_SOCKET_MOUNT) \

common/.commonfiles.sha

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
c90e42b760dbccddf56c75c7f0194988cf57f2be
1+
8d9d2756a8345265a58f1239598ad65c5e019ba4

common/Makefile.common.mk

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -96,15 +96,15 @@ update-common:
9696
@git clone -q --depth 1 --single-branch --branch $(UPDATE_BRANCH) https://github.com/istio/common-files $(TMP)/common-files
9797
@cd $(TMP)/common-files ; git rev-parse HEAD >files/common/.commonfiles.sha
9898
@rm -fr common
99-
@cp -ar $(TMP)/common-files/files/* $(shell pwd)
99+
@cp -a $(TMP)/common-files/files/* $(shell pwd)
100100
@rm -fr $(TMP)/common-files
101101

102102
update-common-protos:
103103
@mkdir -p $(TMP)
104104
@git clone -q --depth 1 --single-branch --branch $(UPDATE_BRANCH) https://github.com/istio/common-files $(TMP)/common-files
105105
@cd $(TMP)/common-files ; git rev-parse HEAD > common-protos/.commonfiles.sha
106106
@rm -fr common-protos
107-
@cp -ar $(TMP)/common-files/common-protos/* $(shell pwd)/common-protos
107+
@cp -a $(TMP)/common-files/common-protos $(shell pwd)
108108
@rm -fr $(TMP)/common-files
109109

110110
check-clean-repo:

common/config/license-lint.yml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,6 @@ whitelisted_modules:
7171
- github.com/daviddengcn/go-colortext
7272
- github.com/dchest/siphash
7373
- github.com/dnaeon/go-vcr
74-
- github.com/docker/docker
7574
- github.com/duosecurity/duo_api_golang
7675
- github.com/dustin/go-humanize
7776
- github.com/facebookgo/stack
@@ -92,9 +91,6 @@ whitelisted_modules:
9291
- github.com/JeffAshton/win_pdh
9392
- github.com/jmespath/go-jmespath
9493
- github.com/jteeuwen/go-bindata
95-
- github.com/juju/errors
96-
- github.com/juju/loggo
97-
- github.com/juju/testing
9894
- github.com/julienschmidt/httprouter
9995
- github.com/koneu/natend
10096
- github.com/kr/logfmt
@@ -118,7 +114,6 @@ whitelisted_modules:
118114
- github.com/russross/blackfriday
119115
- github.com/russross/blackfriday/v2
120116
- github.com/sean-/seed
121-
- github.com/signalfx/com_signalfx_metrics_protobuf
122117
- github.com/smartystreets/assertions
123118
- github.com/smartystreets/goconvey
124119
- github.com/storageos/go-api

common/scripts/check_clean_repo.sh

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,8 @@
1515
# limitations under the License.
1616

1717
if [[ -n $(git status --porcelain) ]]; then
18-
git status
19-
echo "ERROR: Some files need to be updated, please run 'make gen' and include any changed files in your PR"
20-
exit 1
18+
git status
19+
git diff
20+
echo "ERROR: Some files need to be updated, please run 'make gen' and include any changed files in your PR"
21+
exit 1
2122
fi

common/scripts/report_build_info.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ if [[ -n ${ISTIO_VERSION} ]]; then
4343
fi
4444

4545
GIT_DESCRIBE_TAG=$(git describe --tags)
46-
HUB=${HUB:-""}
46+
HUB=${HUB:-"docker.io/istio"}
4747

4848
# used by common/scripts/gobuild.sh
4949
echo "istio.io/pkg/version.buildVersion=${VERSION}"

0 commit comments

Comments
 (0)