Skip to content

Feat/terraform provider#2577

Draft
RobbertDM wants to merge 30 commits intoconveyordata:mainfrom
RobbertDM:feat/terraform-provider
Draft

Feat/terraform provider#2577
RobbertDM wants to merge 30 commits intoconveyordata:mainfrom
RobbertDM:feat/terraform-provider

Conversation

@RobbertDM
Copy link
Contributor

@RobbertDM RobbertDM commented Jan 31, 2026

Don't review yet, in draft, more of a scratchpad for myself.
I know it's not a small single-resource PR like you said but high-level, I think this comes closer to what you envision for a terraform provider?

  • Filter out deprecated API routes to generate openapi-no-deprecated.json (OpenAPI 3.1.0)
  • SDK generation using the recently open-sourced oapi-codegen-dd from Doordash https://github.com/doordash-oss/oapi-codegen-dd because that supports OpenAPI 3.1.0
  • terraform vibe coded from generated SDK (this is why I don't want you to review yet, I want to do more functional tests first, see below)
  • tf docs generated using github.com/hashicorp/terraform-plugin-docs

TODOs from your comments at #2235

  • Test with local portal deployment
  • end-to-end test, spinning up fresh portal, unleashing lots of resources onto that portal, somehow verifying that they have been made, terraform destroy them, somehow verify that they're gone
  • end-to-end test in CI/CD
  • See if that no-deprecated dance is necessary, could be that the doordash oapi-codegen has a filter for this itself that you can set in the config file.
  • Apparently there is a Go CLI, do you still want to revive that thing using the newly generated SDK? The last code generation is from june 2024 and still called bifrost.gen.go :D
  • Update the release notes document if needed in the release notes
  • When updating the UI please provide screenshots or videos to the reviewers

@nclaeys nclaeys marked this pull request as draft February 1, 2026 14:07
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