Skip to content

chore(dir/helm): remove oidc auth wiring#1378

Merged
tkircsi merged 1 commit into
mainfrom
chore/remove-auth-wiring
Apr 22, 2026
Merged

chore(dir/helm): remove oidc auth wiring#1378
tkircsi merged 1 commit into
mainfrom
chore/remove-auth-wiring

Conversation

@tkircsi
Copy link
Copy Markdown
Contributor

@tkircsi tkircsi commented Apr 22, 2026

Summary

  • Remove the legacy in-repo envoy-authz implementation and Helm chart from dir now that dir consumes the external oidc-gateway OCI chart instead.
  • Clean up the remaining build, release, versioning, and vulnerability-scan wiring that still referenced envoy-authz.
  • Keep the dir apiserver chart behavior intact by retaining the dedicated long-lived EventService/Listen route through the new oidc-gateway streamingPath configuration.

Validation

  • helm lint install/charts/dir/apiserver
  • Repository-wide search confirms no active envoy-authz wiring remains outside historical changelog text
  • Verified edited files are free of linter issues

@tkircsi
chore(dir/helm): remove oidc auth wiring
8b751f6 
Signed-off-by: Tibor Kircsi <tkircsi@cisco.com>
@tkircsi tkircsi requested a review from a team as a code owner April 22, 2026 07:14
@tkircsi tkircsi self-assigned this Apr 22, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 22, 2026 •
edited
Loading

The latest Buf updates on your PR. Results from workflow Buf CI / verify-proto (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed⏩ skipped⏩ skipped✅ passedApr 22, 2026, 7:15 AM

@github-actions github-actions Bot added the size/XL Denotes a PR that changes 2000+ lines label Apr 22, 2026
@tkircsi tkircsi added kind/refactor Categorizes issue or PR as related to cleaning up code, process, or technical debt. area/helm/dir and removed size/XL Denotes a PR that changes 2000+ lines labels Apr 22, 2026
@github-project-automation github-project-automation Bot moved this to Backlog in Discovery Apr 22, 2026
@tkircsi tkircsi added this to the DIR v1.3 milestone Apr 22, 2026
@tkircsi tkircsi linked an issue Apr 22, 2026 that may be closed by this pull request
Remove obsolete monorepo wiring from dir #1361
Closed
@tkircsi tkircsi merged commit 19dbb7b into main Apr 22, 2026
40 checks passed
@tkircsi tkircsi deleted the chore/remove-auth-wiring branch April 22, 2026 07:34
@github-project-automation github-project-automation Bot moved this from Backlog to Done in Discovery Apr 22, 2026
keraron pushed a commit to keraron/dir that referenced this pull request May 5, 2026
@tkircsi @keraron
chore(dir/helm): remove oidc auth wiring (agntcy#1378)
5f3b33c 
Signed-off-by: Tibor Kircsi <tkircsi@cisco.com>
Signed-off-by: Aron Kerekes <arkereke@cisco.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ramizpolic ramizpolic ramizpolic approved these changes

Assignees

@tkircsi tkircsi

Labels

area/helm/dir kind/refactor Categorizes issue or PR as related to cleaning up code, process, or technical debt.

Projects

Discovery
Archived in project

Milestone

DIR v1.3

Development

Successfully merging this pull request may close these issues.

Remove obsolete monorepo wiring from dir

2 participants

@tkircsi @ramizpolic