Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add WaaS resources #56

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

trois-six
Copy link
Contributor

@trois-six trois-six commented Jun 29, 2022

Description

WIP! Add WaaS resources to the Terraform provider

Motivation and Context

These resources are missing.

How Has This Been Tested?

On my own SA tenant.

Types of changes

  • New internal API to manage WaaS + convert WaaS + Terraform WaaS resources.
  • Make all *_test.go compile (at least), but tests are not corrected, a lot of work is needed in that field
  • Upgrade go dependencies

Checklist

  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes if appropriate.
  • All new and existing tests passed.

@trois-six trois-six changed the title Feat/add waas Feat: add WaaS resources Jun 29, 2022
@wfg
Copy link
Collaborator

wfg commented Jun 29, 2022

Wow, this is going to be a pretty awesome addition! Thanks for taking it on @trois-six!

@trois-six trois-six changed the title Feat: add WaaS resources feat: add WaaS resources Jun 30, 2022
@Zack-Watt
Copy link

looking forwarded to this being added. Will help my workload a lot.

@trois-six
Copy link
Contributor Author

looking forwarded to this being added. Will help my workload a lot.

The problem is that I won't be able to continue, I just moved to another job and I won't have more time to spend on that + I don't have a dev environment anymore to work on that.

@trois-six
Copy link
Contributor Author

looking forwarded to this being added. Will help my workload a lot.

The problem is that I won't be able to continue, I just moved to another job and I won't have more time to spend on that + I don't have a dev environment anymore to work on that.

... and why the HELL did we expose something that is different that the API in Terraform, the layer of conversation in a (BIG) waste of time for me.

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.

3 participants