Skip to content

add stackit-pod-identity-webhook#1

Merged
ske-prow[bot] merged 2 commits intomainfrom
initial-webhook
Mar 2, 2026
Merged

add stackit-pod-identity-webhook#1
ske-prow[bot] merged 2 commits intomainfrom
initial-webhook

Conversation

@crigertg
Copy link
Copy Markdown
Contributor

@crigertg crigertg commented Feb 23, 2026

How to categorize this PR?

/kind enhancement

What this PR does / why we need it:

Introduce the stackit-pod-identity-webhook with build and testing setup.

Special notes for your reviewer:

Breaking changes:

@crigertg crigertg changed the title add initial repo setup and code for stackit-pod-identity-webhook wip: add initial repo setup and code for stackit-pod-identity-webhook Feb 23, 2026
@crigertg crigertg changed the title wip: add initial repo setup and code for stackit-pod-identity-webhook add initial repo setup and code for stackit-pod-identity-webhook Feb 23, 2026
Comment thread .github/renovate.json5 Outdated
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file is WIP and needs further changes and specifications.

@crigertg crigertg force-pushed the initial-webhook branch 2 times, most recently from 3458f98 to ba6ebb6 Compare February 23, 2026 12:56
Comment thread .github/ISSUE_TEMPLATE/bug_report.md Outdated
Comment thread .github/ISSUE_TEMPLATE/bug_report.md Outdated
Comment thread .github/ISSUE_TEMPLATE/bug_report.md Outdated
Comment thread .github/ISSUE_TEMPLATE/bug_report.md Outdated
Comment thread .github/ISSUE_TEMPLATE/feature_request.md Outdated
Comment thread README.md Outdated
Comment thread cmd/stackit-pod-identity-webhook/main.go Outdated
Comment thread pkg/webhook/constants.go
Comment thread pkg/webhook/pod_mutator.go Outdated
Comment thread test/integration/webhook_test.go
@crigertg crigertg force-pushed the initial-webhook branch 4 times, most recently from c459ebc to 130e014 Compare February 24, 2026 13:19
Comment thread .idea/workspace.xml Outdated
Comment thread charts/stackit-pod-identity-webhook/values.yaml
Comment thread cmd/stackit-pod-identity-webhook/main.go
Comment thread pkg/webhook/pod_mutator.go Outdated
Comment thread README.md Outdated
Comment thread charts/stackit-pod-identity-webhook/values.yaml Outdated
Copy link
Copy Markdown

@relusc relusc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀 🚀 🚀

Copy link
Copy Markdown
Member

@xoxys xoxys left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://github.com/stackitcloud/cloud-provider-stackit/blob/main/SECURITY.md might be missing. The rest looks good to me at least in terms of the repo meta files.

Comment thread .github/renovate.json5 Outdated
Comment thread README.md Outdated
Comment thread hack/tools.mk Outdated
Comment thread go.mod Outdated
Comment thread charts/stackit-pod-identity-webhook/templates/certificate.yaml
Comment thread .github/release.yaml Outdated
Comment thread pkg/webhook/pod_mutator_test.go Outdated
Comment thread pkg/webhook/pod_mutator_test.go Outdated
Comment thread pkg/webhook/pod_mutator_test.go Outdated
Comment thread test/integration/helpers.go
Comment thread test/integration/webhook_test.go Outdated
Comment thread go.mod
Comment thread pkg/webhook/pod_mutator.go
Comment thread pkg/webhook/pod_mutator.go Outdated
@maboehm
Copy link
Copy Markdown

maboehm commented Feb 26, 2026

Alright, from my POV this is a mergable first version.
/lgtm

@crigertg crigertg changed the title add initial repo setup and code for stackit-pod-identity-webhook add stackit-pod-identity-webhook Feb 27, 2026
@ske-prow ske-prow bot assigned relusc Mar 2, 2026
@ske-prow ske-prow bot added the lgtm Indicates that a PR is ready to be merged. label Mar 2, 2026
@ske-prow
Copy link
Copy Markdown

ske-prow bot commented Mar 2, 2026

LGTM label has been added.

DetailsGit tree hash: bb45293afe82d2a278e6cc5d52bb4e7d8bf93cff

@ske-prow
Copy link
Copy Markdown

ske-prow bot commented Mar 2, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: relusc

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ske-prow ske-prow bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 2, 2026
@ske-prow ske-prow bot merged commit da139da into main Mar 2, 2026
3 checks passed
@ske-prow ske-prow bot deleted the initial-webhook branch March 2, 2026 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants