-
Notifications
You must be signed in to change notification settings - Fork 422
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
Developer Experience improvements #131
Conversation
/priority critical-urgent |
/kind feature |
Makefile
Outdated
|
||
.PHONY: lint | ||
lint: dep-ensure ## Lint codebase | ||
@echo If you have generated new mocks, run make copy-genmocks before linting | ||
@echo If you have genereated new mocks, run make copy-genmocks before linting |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SUPER NIT
Generated is spelled wrong here
build/controller_gen.bzl
Outdated
{controller_gen} all && \\ | ||
cp -fR config $$GENDIR | ||
{controller_gen} all && \\ | ||
cp -fR config $$GENDIR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like the spaces were removed before these lines. Was that on purpose?
scripts/ci-bazel-integration.sh
Outdated
@@ -0,0 +1,26 @@ | |||
#!/bin/bash | |||
|
|||
# Copyright 2018 The Kubernetes Authors. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
# Copyright 2018 The Kubernetes Authors. | |
# Copyright 2019 The Kubernetes Authors. |
docs/development.md
Outdated
### Get familiar with basic concepts | ||
|
||
This provider is modeled after the upstream cluster-api project. To get familiar | ||
with resources, concepts and conventions refer to the [upstream gitbook](https://kubernetes-sigs.github.io/cluster-api/). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SUPER DUPER NIT: can we rename it to cluster-api gitbook? Also let's move the link to the bottom of the readme.md for consistency's sake.
docs/development.md
Outdated
### Get familiar with basic concepts | ||
|
||
This provider is modeled after the upstream cluster-api project. To get familiar | ||
with resources, concepts and conventions refer to the [upstream gitbook](https://kubernetes-sigs.github.io/cluster-api/). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
with resources, concepts and conventions refer to the [upstream gitbook](https://kubernetes-sigs.github.io/cluster-api/). | |
with the concepts and conventions, refer to the [cluster-api gitbook](https://kubernetes-sigs.github.io/cluster-api/). |
158dd7e
to
eafd87c
Compare
70c2c23
to
8c40dec
Compare
@@ -25,7 +25,7 @@ COPY vendor/ vendor/ | |||
RUN CGO_ENABLED=0 GOOS=linux GOARCH=amd64 go build -a -o manager sigs.k8s.io/cluster-api-provider-azure/cmd/manager | |||
|
|||
# Copy the controller-manager into a thin image | |||
FROM alpine:3.9 | |||
FROM gcr.io/distroless/base |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For reference to other reviewers
https://github.com/GoogleContainerTools/distroless/blob/master/base/README.md
6328c51
to
f58e84c
Compare
Ready for review: |
overall looks good, let me know once the conflicts are resolved. |
- Update documentation - Update .gitignore - Remove skip_files in golangci config - Bump golang version (`1.12.1`) / switch thin image to `distroless/base` - Clean up Makefile - Bump bazel rules_go (`0.18.1`) - Bump kind version (`d4ff13e4808ee5a816ed92bf2d3348bda12413db`) - Rename build/asm_shim - Add bazel generator for kustomize - Update example specs - Update generate-yaml.sh - Update cmd packages - Add release tool - Regenerate CRDs Signed-off-by: Stephen Augustus <saugustus@vmware.com>
Thanks @awesomenix. Conflicts fixed up! |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: awesomenix, justaugustus The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Signed-off-by: Stephen Augustus saugustus@vmware.com
What this PR does / why we need it:
Update developer toolbox and documentation:
1.12.1
) / switch thin image todistroless/base
0.18.1
)d4ff13e4808ee5a816ed92bf2d3348bda12413db
)Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #127, fixes #108, fixes #98, fixes #107
Release note: