Skip to content

feat(credentials): add GCP credential management (#7495) - #7660

Draft
Gabriel Pezé (gabriel-peze) wants to merge 22 commits into
mainfrom
issue/7495
Draft

feat(credentials): add GCP credential management (#7495)#7660
Gabriel Pezé (gabriel-peze) wants to merge 22 commits into
mainfrom
issue/7495

Conversation

@gabriel-peze

Copy link
Copy Markdown
Contributor

Proposed changes

  • Add the GCP credentials management, with status

Testing Instructions

  1. Create and update a credentials GCP
  2. Check for the status

Related issues

@github-actions

Copy link
Copy Markdown
Contributor

📖 Documentation check — ❌ Failed

40 functional file(s) changed, 0 documentation file(s) changed.

Blocking gaps

Update the docs or add the No need documentation label to bypass.

🟠 2 new endpoint(s) in existing controller

  • File: openaev-api/src/main/java/io/openaev/api/credentials/CredentialApi.java
  • Expected doc update: docs/docs/usage/rest-api.md
  • Reason: New endpoints expand the API surface.

Suggestions (non-blocking)

  • 🟡 New configuration propertyopenaev-api/src/main/java/io/openaev/secrets/provider/impl/validators/GcpCredentialConnectivityCheck.javadocs/docs/deployment/configuration.md

Add the No need documentation label to bypass this check when no documentation update is needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

filigran team Item from the Filigran team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants