Skip to content

Add more e2e RBAC#1199

Merged
Kidswiss merged 1 commit into
developfrom
fix/e2e-rbac
Jun 1, 2026
Merged

Add more e2e RBAC#1199
Kidswiss merged 1 commit into
developfrom
fix/e2e-rbac

Conversation

@Kidswiss
Copy link
Copy Markdown
Contributor

Because kuttl tries to read service accounts from the default namespace for some reason, we added RBAC to allow this.

Otherwise it spends a lot of time re-trying to query those service accounts.

Checklist

  • The PR has a meaningful title. It will be used to auto generate the
    changelog.
    The PR has a meaningful description that sums up the change. It will be
    linked in the changelog.
  • PR contains a single logical change (to build a better changelog).
  • I run make e2e-test against local kindev and all checks passed
  • If no changes have been made you can self approve but otherwise at least two reviews are required
  • Link this PR to related issues or PRs.

@Kidswiss Kidswiss force-pushed the fix/e2e-rbac branch 2 times, most recently from 7b3758e to 682a866 Compare May 29, 2026 11:47
@Kidswiss Kidswiss added the patch label May 29, 2026
@Kidswiss Kidswiss requested review from a team, TheBigLee, mdnix, mikeshootzz and zugao and removed request for a team May 29, 2026 12:38
Copy link
Copy Markdown
Contributor

@mikeshootzz mikeshootzz left a comment

Choose a reason for hiding this comment

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

Rest LGTM

ownerGroup: appcat.vshn.io
ownerKind: XObjectBucket
ownerVersion: v1
proxyEndpoint: 172.18.0.1:9443
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this looks unrelated 🤔

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 is handled by make push-golden depending if you pass the -e DEBUG=false or not, but I can reset it.

Comment thread tests/dev.yml Outdated
registry: dockerhub.vshn.net
grpcEndpoint: 172.18.0.1:9443
proxyFunction: true
proxyFunction: false
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

same here

@Kidswiss
Copy link
Copy Markdown
Contributor Author

Link to passing e2e tests: https://github.com/vshn/component-appcat/actions/runs/26635484023

Because kuttl tries to read service accounts from the default namespace
for some reason, we added RBAC to allow this.

Otherwise it spends a lot of time re-trying to query those service
accounts.

Also fixing a few postgresqlcnpg issues.
@Kidswiss Kidswiss merged commit 952e0f9 into develop Jun 1, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants