Skip to content

Google Drive Integration - #2443

Merged
matoszz merged 6 commits into
mainfrom
feat-driveintegration
May 30, 2026
Merged

Google Drive Integration#2443
matoszz merged 6 commits into
mainfrom
feat-driveintegration

Conversation

@matoszz

@matoszz matoszz commented May 29, 2026

Copy link
Copy Markdown
Member
  • Adds initial basic google drive integration, which leverages googles live html rendering capability and returns the html in the graphql request
  • Adds the integration provider, the operations, the connection infos
  • Updates the existing resolver to return the needed fields
  • Assumes returned html is controlled via the UI
  • Enhances pkg/gala to have additional context enhancements / injection

@theopenlane-bender

Copy link
Copy Markdown

🔧 Configuration Changes Detected

This PR contains changes that will affect the Helm chart configuration. A draft infrastructure PR has been automatically created to preview these changes:

📋 Draft PR: https://github.com/theopenlane/openlane-infra/pull/963

Changes Preview:

✅ Updated ConfigMap template

  • 🔄 Merged Helm values.yaml
  • 🔐 External secrets configuration updated
  • ✅ Updated ConfigMap template

The draft infrastructure PR will be closed automatically after this core PR is merged.

Comment thread internal/integrations/definitions/googledrive/client.go
Comment thread internal/integrations/definitions/googledrive/operation_export.go
func persistActionPlanInput(ctx context.Context, db *ent.Client, integration *ent.Integration, createInput ent.CreateActionPlanInput) error {
// Custom persistence logic required for ActionPlan.
// Implement lookup and upsert logic below using the createInput fields.
panic("not implemented: persistActionPlanInput")

@golanglemonade golanglemonade May 29, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

might be good to just return a not implemented error instead of panic just in case (or add the implementation)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

These are generated resolvers, and private / unexported - the only way you'd hit this call path is by specifying it as the ingest schema for an integration operation in which case you should be implementing the resolver (e.g. adding the necessary annotations to the schemas and the other related tasks)

Comment thread internal/integrations/operations/ingest_procedure_persist.go
@matoszz
matoszz enabled auto-merge (squash) May 30, 2026 14:31
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
2.3% Coverage on New Code (required ≥ 20%)

See analysis details on SonarQube Cloud

@matoszz
matoszz merged commit a9db46b into main May 30, 2026
22 of 23 checks passed
@matoszz
matoszz deleted the feat-driveintegration branch May 30, 2026 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants