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

Use go:embed instead of go-bindata #4370

Merged
merged 12 commits into from
Oct 28, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ jobs:
lint:
name: Lint
runs-on: ubuntu-latest
container: weaveworks/eksctl-build:ff2eff428ff2b334c713fd07916a10db37f60d29
container: weaveworks/eksctl-build:83999a3391d78ed18a2e6093328d1122c927956f
steps:
- name: Checkout
uses: actions/checkout@v2
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/integration-tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ jobs:
name: Run integration tests
environment: "Integration tests"
runs-on: ubuntu-latest
container: weaveworks/eksctl-build:ff2eff428ff2b334c713fd07916a10db37f60d29
container: weaveworks/eksctl-build:83999a3391d78ed18a2e6093328d1122c927956f
env:
SSH: "/root/.ssh"
steps:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/release-candidate.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ jobs:
rc:
name: Trigger release candidate build
runs-on: ubuntu-latest
container: weaveworks/eksctl-build:ff2eff428ff2b334c713fd07916a10db37f60d29
container: weaveworks/eksctl-build:83999a3391d78ed18a2e6093328d1122c927956f
steps:
- uses: actions/github-script@v3
name: Get integration tests status
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/release.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ jobs:
rc:
name: Trigger release build
runs-on: ubuntu-latest
container: weaveworks/eksctl-build:ff2eff428ff2b334c713fd07916a10db37f60d29
container: weaveworks/eksctl-build:83999a3391d78ed18a2e6093328d1122c927956f
steps:
- name: Checkout
uses: actions/checkout@v2
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/tag-release.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ jobs:
name: Tag release from version
environment: release
runs-on: ubuntu-latest
container: weaveworks/eksctl-build:ff2eff428ff2b334c713fd07916a10db37f60d29
container: weaveworks/eksctl-build:83999a3391d78ed18a2e6093328d1122c927956f
steps:
- name: Checkout
uses: actions/checkout@v2
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/update-generated.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ jobs:
update_aws_node:
name: Update aws-node and open PR
runs-on: ubuntu-latest
container: weaveworks/eksctl-build:ff2eff428ff2b334c713fd07916a10db37f60d29
container: weaveworks/eksctl-build:83999a3391d78ed18a2e6093328d1122c927956f
env:
UPDATE_BRANCH: update-aws-node
GOPRIVATE: ""
Expand Down
1 change: 0 additions & 1 deletion .requirements
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ github.com/cloudflare/cfssl/cmd/[email protected]
github.com/cloudflare/cfssl/cmd/[email protected]
github.com/golangci/golangci-lint/cmd/golangci-lint
github.com/goreleaser/goreleaser
github.com/kevinburke/go-bindata/go-bindata
github.com/onsi/ginkgo/[email protected]
github.com/vektra/mockery/cmd/mockery
github.com/github-release/github-release
Expand Down
2 changes: 1 addition & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
ARG BUILD_IMAGE=/weaveworks/eksctl-build:ff2eff428ff2b334c713fd07916a10db37f60d29
ARG BUILD_IMAGE=/weaveworks/eksctl-build:83999a3391d78ed18a2e6093328d1122c927956f
FROM $BUILD_IMAGE as build

WORKDIR /src
Expand Down
14 changes: 1 addition & 13 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -16,16 +16,8 @@ generated_code_deep_copy_helper := pkg/apis/eksctl.io/v1alpha5/zz_generated.deep
generated_code_aws_sdk_mocks := $(wildcard pkg/eks/mocks/*API.go)

conditionally_generated_files := \
userdocs/src/usage/schema.json \
$(generated_code_deep_copy_helper) $(generated_code_aws_sdk_mocks)

all_generated_files := \
pkg/nodebootstrap/bindata/assets.go \
pkg/addons/default/assets.go \
pkg/addons/assets.go \
pkg/apis/eksctl.io/v1alpha5 \
$(conditionally_generated_files)

.DEFAULT_GOAL := help

##@ Utility
Expand Down Expand Up @@ -172,10 +164,6 @@ delete-integration-test-dev-cluster: build ## Delete the test cluster for use wh
## not present, the generation of assets will not fail but will not contain it.
.PHONY: generate-always
generate-always: pkg/addons/default/assets/aws-node.yaml ## Generate code (required for every build)
@# go-bindata targets must run every time, as dependencies are too complex to declare in make:
@# - deleting an asset is breaks the dependencies
@# - different version of go-bindata generate different code
@go-bindata -v
go generate ./pkg/apis/eksctl.io/v1alpha5/generate.go
go generate ./pkg/nodebootstrap
go generate ./pkg/addons/default/generate.go
Expand All @@ -193,7 +181,7 @@ generate-all: generate-always $(conditionally_generated_files) ## Re-generate al

.PHONY: check-all-generated-files-up-to-date
check-all-generated-files-up-to-date: generate-all ## Run the generate all command and verify there is no new diff
git diff --quiet -- $(all_generated_files) || (git --no-pager diff $(all_generated_files); echo "HINT: to fix this, run 'git commit $(all_generated_files) --message \"Update generated files\"'"; exit 1)
git diff --quiet -- $(conditionally_generated_files) || (git --no-pager diff $(conditionally_generated_files); echo "HINT: to fix this, run 'git commit $(conditionally_generated_files) --message \"Update generated files\"'"; exit 1)

### Update maxpods.go from AWS
.PHONY: update-maxpods
Expand Down
3 changes: 1 addition & 2 deletions build/docker/build_image_manifest
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
"github.com/cloudflare/cfssl v1.6.1"
"github.com/golangci/golangci-lint v1.42.0"
"github.com/goreleaser/goreleaser v0.182.1"
"github.com/kevinburke/go-bindata v3.22.0+incompatible"
"github.com/onsi/ginkgo v1.16.4"
"github.com/vektra/mockery v1.1.2"
"github.com/github-release/github-release v0.10.0"
Expand All @@ -15,5 +14,5 @@
"k8s.io/code-generator v0.21.2"
"sigs.k8s.io/mdtoc v1.0.1"
100644 blob a73dcb54fd6215c023ffea7f781e313b41841135 build/docker/Dockerfile
100644 blob e02e5f2ad4516bd586dd870006e14a8a5a168093 .requirements
100644 blob 5683496f7ee0443658a663785d87e404096459d6 .requirements
100755 blob c1129ff1ff85ac2c53f908a577675ea59a9325a7 build/scripts/install-build-deps.sh
2 changes: 1 addition & 1 deletion build/docker/image_tag
Original file line number Diff line number Diff line change
@@ -1 +1 @@
ff2eff428ff2b334c713fd07916a10db37f60d29
83999a3391d78ed18a2e6093328d1122c927956f
1 change: 0 additions & 1 deletion go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -185,7 +185,6 @@ require (
github.com/julz/importas v0.0.0-20210419104244-841f0c0fe66d // indirect
github.com/justinbarrick/go-k8s-portforward v1.0.4-0.20200904152830-b575325c1855
github.com/kballard/go-shellquote v0.0.0-20180428030007-95032a82bc51
github.com/kevinburke/go-bindata v3.22.0+incompatible
github.com/kevinburke/rest v0.0.0-20210106114233-22cd0577e450 // indirect
github.com/kevinburke/ssh_config v1.1.0 // indirect
github.com/kisielk/errcheck v1.6.0 // indirect
Expand Down
2 changes: 0 additions & 2 deletions go.sum
Original file line number Diff line number Diff line change
Expand Up @@ -1132,8 +1132,6 @@ github.com/k0kubun/pp v2.3.0+incompatible/go.mod h1:GWse8YhT0p8pT4ir3ZgBbfZild3t
github.com/karrick/godirwalk v1.7.5/go.mod h1:2c9FRhkDxdIbgkOnCEvnSWs71Bhugbl46shStcFDJ34=
github.com/kballard/go-shellquote v0.0.0-20180428030007-95032a82bc51 h1:Z9n2FFNUXsshfwJMBgNA0RU6/i7WVaAegv3PtuIHPMs=
github.com/kballard/go-shellquote v0.0.0-20180428030007-95032a82bc51/go.mod h1:CzGEWj7cYgsdH8dAjBGEr58BoE7ScuLd+fwFZ44+/x8=
github.com/kevinburke/go-bindata v3.22.0+incompatible h1:/JmqEhIWQ7GRScV0WjX/0tqBrC5D21ALg0H0U/KZ/ts=
github.com/kevinburke/go-bindata v3.22.0+incompatible/go.mod h1:/pEEZ72flUW2p0yi30bslSp9YqD9pysLxunQDdb2CPM=
github.com/kevinburke/rest v0.0.0-20210106114233-22cd0577e450 h1:Lr2sEj5wWzk82b/L8LsLzsCxywQaOpcr0ti/qcfzCOk=
github.com/kevinburke/rest v0.0.0-20210106114233-22cd0577e450/go.mod h1:pD+iEcdAGVXld5foVN4e24zb/6fnb60tgZPZ3P/3T/I=
github.com/kevinburke/ssh_config v0.0.0-20190725054713-01f96b0aa0cd/go.mod h1:CT57kijsi8u/K/BOFA39wgDQJ9CxiF4nAY/ojJ6r6mM=
Expand Down
Loading