camilamacedo86 / kubebuilder
82%
master: 66%

LAST BUILD BRANCH: metrics-port
DEFAULT BRANCH: master
Repo Added
Token ZL3OZCITlsqdEPu1QR9RvS6f34N5v77Oj regen
Build 2915 Last
Files 55
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH webhooks-path-change
branch: webhooks-path-change
CHANGE BRANCH
x
Sync Branches

coverage: 82.116%. Remained the same
11035844586

push

github

camilamacedo86 
:warning: (go/v4) decouple webhooks from APIs - Move Webhooks from `api/<version>` or `api/<group>/<version>` to `internal/webhook/<version>` or `internal/webhook/<group>/<version>`

This PR decouples the webhooks from the API, aligning with the recent breaking changes introduced in controller-runtime to ensure that kubebuilder still compatbile with its next release.  Webhooks are now scaffolded under `internal/webhook` to comply with the latest standards.

**Context:**

Controller-runtime deprecated and removed the webhook methods in favor of CustomInterfaces (see [controller-runtime#2641](https://github.com/kubernetes-sigs/controller-runtime/issues/2641)). The motivation for this change is outlined in [controller-runtime#2596](https://github.com/kubernetes-sigs/controller-runtime/issues/2596).

See that the current master branch already reflects these changes, using the CustomInterfaces: [kubebuilder#4060](https://github.com/kubernetes-sigs/kubebuilder/pull/4060).

**Changes:**

- Webhooks are now scaffolded in `internal/webhook/<version>` or `internal/webhook/<group>/<version>`.
- However, to ensure backwards compatibility, a new `--legacy` flag is introduced. Running `kubebuilder create webhook [options] --legacy` will scaffold webhooks in the legacy location for projects that need to retain the old structure. However, users will still to address the breaking changes in the source code by replacing the old methods by the new CustomInterfaces.

2282 of 2779 relevant lines covered (82.12%)

15.79 hits per line

Relevant lines Covered
2779 RELEVANT LINES 2282 COVERED LINES
15.79 HITS PER LINE
Source Files on webhooks-path-change

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
11035844586 webhooks-path-change :warning: (go/v4) decouple webhooks from APIs - Move Webhooks from `api/<version>` or `api/<group>/<version>` to `internal/webhook/<version>` or `internal/webhook/<group>/<version>` This PR decouples the webhooks from the API, aligning with the r... push camilamacedo86 github
82.12
11035749575 webhooks-path-change :warning: (go/v4) decouple webhooks from APIs - Move Webhooks from `api/<version>` or `api/<group>/<version>` to `internal/webhook/<version>` or `internal/webhook/<group>/<version>` This PR decouples the webhooks from the API, aligning with the r... push camilamacedo86 github
82.12
11035551607 webhooks-path-change :warning: (go/v4) decouple webhooks from APIs - Move Webhooks from `api/<version>` or `api/<group>/<version>` to `internal/webhook/<version>` or `internal/webhook/<group>/<version>` This PR decouples the webhooks from the API, aligning with the r... push camilamacedo86 github
82.12
10978989618 webhooks-path-change :warning: (go/v4) decouple webhooks from APIs - Move Webhooks from `api/<version>` or `api/<group>/<version>` to `internal/webhook/<version>` or `internal/webhook/<group>/<version>` This PR decouples the webhooks from the API, aligning with the r... push camilamacedo86 github
81.77
10975094654 webhooks-path-change :warning: (go/v4) decouple webhooks from APIs - Move Webhooks from `api/<version>` or `api/<group>/<version>` to `internal/webhook/<version>` or `internal/webhook/<group>/<version>` This PR decouples the webhooks from the API, aligning with the r... push camilamacedo86 github
81.77
10975061421 webhooks-path-change :warning: (go/v4) decouple webhooks from APIs - Move Webhooks from `api/<version>` or `api/<group>/<version>` to `internal/webhook/<version>` or `internal/webhook/<group>/<version>` This PR decouples the webhooks from the API, aligning with the r... push camilamacedo86 github
81.77
10975017715 webhooks-path-change Update pkg/plugins/golang/v4/scaffolds/internal/templates/main.go Co-authored-by: Varsha <varshaprasad96@gmail.com> push web-flow github
81.77
10974991625 webhooks-path-change :warning: (go/v4) decouple webhooks from APIs - Move Webhooks from `api/<version>` or `api/<group>/<version>` to `internal/webhook/<version>` or `internal/webhook/<group>/<version>` This PR decouples the webhooks from the API, aligning with the r... push camilamacedo86 github
81.77
10974028539 webhooks-path-change :warning: (go/v4) decouple webhooks from APIs - Move Webhooks from `api/<version>` or `api/<group>/<version>` to `internal/webhook/<version>` or `internal/webhook/<group>/<version>` This PR decouples the webhooks from the API, aligning with the r... push camilamacedo86 github
81.77
10956192090 webhooks-path-change :warning: (go/v4) decouple webhooks from APIs - Move Webhooks from `api/<version>` or `api/<group>/<version>` to `internal/webhook/<version>` or `internal/webhook/<group>/<version>` This PR decouples the webhooks from the API, aligning with the r... push camilamacedo86 github
81.77
See All Builds (2915)

Badge your Repo: kubebuilder

We detected this repo isn’t badged! Grab the embed code to the right, add it to your repo to show off your code coverage, and when the badge is live hit the refresh button to remove this message.
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

Refresh