Skip to content

Commit 26c8a5a

Browse files
committed
Remove validating webhook
1 parent a6c0e9c commit 26c8a5a

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

60 files changed

+38
-9509
lines changed

.github/ISSUE_TEMPLATE/bug.md

+1-2
Original file line numberDiff line numberDiff line change
@@ -5,12 +5,11 @@ labels: kind/bug
55

66
---
77
<!--
8-
Thank you for your interest in Gateway API! Please note that bug reports
8+
Thank you for your interest in Gateway API! Please note that bug reports
99
here should only be used for bugs with the API itself, such as:
1010
1111
- Incomplete or inaccurate validation
1212
- Problems with the API specification
13-
- Unexpected errors with the validating webhook
1413
- Flaws in conformance tests
1514
1615
Bugs that don't fit into one of the above categories should likely be filed

.gitignore

-1
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,6 @@ Session.vim
5050
/www/test_out
5151
.*.timestamp
5252
/site
53-
admission
5453
go.work
5554
go.work.sum
5655

Makefile

+2-6
Original file line numberDiff line numberDiff line change
@@ -58,16 +58,12 @@ all: generate vet fmt verify test
5858

5959
# Run generators for protos, Deepcopy funcs, CRDs, and docs.
6060
.PHONY: generate
61-
generate: update-codegen update-webhook-yaml
61+
generate: update-codegen
6262

6363
.PHONY: update-codegen
6464
update-codegen:
6565
hack/update-codegen.sh
6666

67-
.PHONY: update-webhook-yaml
68-
update-webhook-yaml:
69-
hack/update-webhook-yaml.sh
70-
7167
.PHONY: build-install-yaml
7268
build-install-yaml:
7369
hack/build-install-yaml.sh
@@ -82,7 +78,7 @@ vet:
8278

8379
# Run go test against code
8480
test:
85-
go test -race -cover ./pkg/admission/... ./apis/... ./conformance/utils/...
81+
go test -race -cover ./apis/... ./conformance/utils/...
8682
# Run tests for each submodule.
8783
cd "conformance/echo-basic" && go test -race -cover ./...
8884
cd "gwctl" && go test -race -cover ./...

RELEASE.md

+10-13
Original file line numberDiff line numberDiff line change
@@ -43,10 +43,9 @@ For a **PATCH** release:
4343
- Use `git` to cherry-pick all relevant PRs into your branch.
4444
- Update `pkg/generator/main.go` with the new semver tag and any updates to the API review URL.
4545
- Run the following command `BASE_REF=vmajor.minor.patch make generate` which
46-
will update generated docs and webhook with the correct version info. (Note
47-
that you can't test with these YAMLs yet as they contain references to
48-
elements which wont exist until the tag is cut and image is promoted to
49-
production registry.)
46+
will update generated docs with the correct version info. (Note that you can't
47+
test with these YAMLs yet as they contain references to elements which wont
48+
exist until the tag is cut and image is promoted to production registry.)
5049
- Create a pull request of the `<githubuser>/release-x.x.x` branch into the `release-x.x` branch upstream
5150
(which should already exist since this is a patch release). Add a hold on this PR waiting for at least
5251
one maintainer/codeowner to provide a `lgtm`.
@@ -62,10 +61,9 @@ For a **MAJOR** or **MINOR** release:
6261
- Check out the `release-major.minor` release branch locally.
6362
- Update `pkg/generator/main.go` with the new semver tag and any updates to the API review URL.
6463
- Run the following command `BASE_REF=vmajor.minor.patch make generate` which
65-
will update generated docs and webhook with the correct version info. (Note
66-
that you can't test with these YAMLs yet as they contain references to
67-
elements which wont exist until the tag is cut and image is promoted to
68-
production registry.)
64+
will update generated docs with the correct version info. (Note that you can't
65+
test with these YAMLs yet as they contain references to elements which wont
66+
exist until the tag is cut and image is promoted to production registry.)
6967
- Verify the CI tests pass before continuing.
7068
- Create a tag using the `HEAD` of the `release-x.x` branch. This can be done using the `git` CLI or
7169
Github's [release][release] page.
@@ -76,10 +74,9 @@ For a **MAJOR** or **MINOR** release:
7674
For an **RC** release:
7775
- Update `pkg/generator/main.go` with the new semver tag and any updates to the API review URL.
7876
- Run the following command `BASE_REF=vmajor.minor.patch make generate` which
79-
will update generated docs and webhook with the correct version info. (Note
80-
that you can't test with these YAMLs yet as they contain references to
81-
elements which wont exist until the tag is cut and image is promoted to
82-
production registry.)
77+
will update generated docs with the correct version info. (Note that you can't
78+
test with these YAMLs yet as they contain references to elements which wont
79+
exist until the tag is cut and image is promoted to production registry.)
8380
- Include the changelog update in this PR.
8481
- Merge the update PR.
8582
- Tag the release using the commit on `main` where the changelog update merged.
@@ -90,7 +87,7 @@ For an **RC** release:
9087
- Attach these files to the Github release.
9188

9289
### Promoting images to production registry
93-
Gateway API follows the standard kubernetes image promotion process described [here][kubernetes-image-promotion].
90+
Gateway API follows the standard kubernetes image promotion process described [here][kubernetes-image-promotion].
9491

9592
1. Once the tag has been cut and the image is available in the staging registry,
9693
identify the SHA-256 image digest of the image that you want to promote.

apis/v1/gateway_types.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -56,8 +56,8 @@ type GatewayList struct {
5656
// GatewaySpec defines the desired state of Gateway.
5757
//
5858
// Not all possible combinations of options specified in the Spec are
59-
// valid. Some invalid configurations can be caught synchronously via a
60-
// webhook, but there are many cases that will require asynchronous
59+
// valid. Some invalid configurations can be caught synchronously via CRD
60+
// validation, but there are many cases that will require asynchronous
6161
// signaling via the GatewayStatus block.
6262
type GatewaySpec struct {
6363
// GatewayClassName used for this Gateway. This is the name of a

apis/v1/httproute_types.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -821,7 +821,7 @@ type HTTPHeader struct {
821821
// HTTPHeaderFilter defines a filter that modifies the headers of an HTTP
822822
// request or response. Only one action for a given header name is permitted.
823823
// Filters specifying multiple actions of the same or different type for any one
824-
// header name are invalid and will be rejected by the webhook if installed.
824+
// header name are invalid and will be rejected by CRD validation.
825825
// Configuration to set or add multiple values for a header must use RFC 7230
826826
// header value formatting, separating each value with a comma.
827827
type HTTPHeaderFilter struct {

apis/v1/validation/common.go

-86
This file was deleted.

0 commit comments

Comments
 (0)