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

Update wordpress use case for summit demo #87

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

yomovh
Copy link
Member

@yomovh yomovh commented Nov 13, 2024

Kind of the Pull Request

  • ✨ New demos ✨
  • 🐛 Bug 🐛
  • [x ] 🌟 Enhancement 🌟
  • 📝 Documentation 📝

Products targeted

  • 🧠 AI 🧠
  • 💿 DBaaS 💿
  • ⎈ Managed Kubernetes Service / Managed Private Registry / Managed Rancher Service ⎈
  • [x ] 🏗️ IaC 🏗️

Purpose of this Pull Request

  • replace terraform by opentofu in CLI examples
  • update instance default flavor to gen 3
  • bump ovh provider to 1.0.0

* replace terraform by  opentofu in CLI examples
* update instance default flavor to gen 3
@philippart-s
Copy link
Collaborator

hi @yomovh,

why do you replace Terraform by Opentofu CLI, other examples are using Terraform, it may cause confusion for users.
Maybe it's better to have both of the examples and if we're going to pick one, I think it should be the same in all the examples.

@scraly any ideas/opinion on the subject?

@scraly
Copy link
Collaborator

scraly commented Nov 15, 2024

I will do the complete review next week but before to do it, yes instead of replacing tf CLI to OpenTofu, you can keep the Terraform cli usage and you can add a note explaining than user can execute Terraform or OpenTofu cli.

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