Skip to content

Fix e2e secret reference schema failures - #2

Closed
Breee with Copilot wants to merge 5 commits into
copilot/build-upjet-provider-for-litellmfrom
copilot/fix-e2e-tests-failure
Closed

Breee with Copilot wants to merge 5 commits into
copilot/build-upjet-provider-for-litellmfrom
copilot/fix-e2e-tests-failure

Conversation

Copilot AI commented Aug 26, 2026 •

Copy link
Copy Markdown

The e2e-tests job failed because VectorStore fixtures included an unsupported key field in SecretReference, while Credential reconciliation was blocked by an incorrectly required initProvider secret reference.

  • VectorStore fixtures
    • Remove key from litellmParamsSecretRef; Crossplane SecretReference supports only name and namespace.
  • Credential schema
    • Make the optional init-provider secret reference omittable.
    • Align the generated CRD schema accordingly.
litellmParamsSecretRef:
  name: e2e-vectorstore-values
  namespace: upbound-system

Co-authored-by: Breee <11966385+Breee@users.noreply.github.com>
Copilot AI changed the title Fix e2e tests secret reference schema failures Fix e2e secret reference schema failures Aug 26, 2026
Copilot AI requested a review from Breee August 26, 2026 06:57
Co-authored-by: Breee <11966385+Breee@users.noreply.github.com>
@Breee

Breee commented Aug 27, 2026

Copy link
Copy Markdown
Member

@copilot already dying e2e tests because of panics

Co-authored-by: Breee <11966385+Breee@users.noreply.github.com>

Copilot AI commented Aug 27, 2026

Copy link
Copy Markdown
Author

@copilot already dying e2e tests because of panics

Fixed the nil *litellm.Client metadata panic and the generated initProvider secret-reference validation in e9c6f5f.

Co-authored-by: Breee <11966385+Breee@users.noreply.github.com>
Co-authored-by: Breee <11966385+Breee@users.noreply.github.com>
@Breee Breee closed this Sep 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants