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

ovh: allow to use ovh.conf #2215

Closed
1 task done
yomovh opened this issue Jun 21, 2024 · 3 comments · Fixed by #2216
Closed
1 task done

ovh: allow to use ovh.conf #2215

yomovh opened this issue Jun 21, 2024 · 3 comments · Fixed by #2216

Comments

@yomovh
Copy link

yomovh commented Jun 21, 2024

Welcome

  • Yes, I've searched similar issues on GitHub and didn't find any.

How do you use lego?

Through Terraform ACME provider

Detailed Description

Hello there !

We have some OVHcloud customers that are using the ability to provide the API credentials through ovh.conf file (see README) notably in OVHcloud terraform provider.

When they are also using acme provider in a plan, they need to change the way they provide the credentials since in acme, the only supported way seems to be through environment variables

This feature request would be to enable the usage of ovh.conf file in acme for the OVHcloud provider.

@ldez ldez changed the title [OVHcloud provider] Enable to use ovh.conf like go-ovh & terraform provider ovh: enable to use ovh.conf like go-ovh & terraform provider Jun 21, 2024
@ldez ldez self-assigned this Jun 21, 2024
@ldez ldez changed the title ovh: enable to use ovh.conf like go-ovh & terraform provider ovh: enable to use ovh.conf Jun 21, 2024
@ldez
Copy link
Member

ldez commented Jun 21, 2024

Hello @yomovh,

You are a product manager at OVH, and your request is for your customers, it would feel fair if you could sponsor this project.

https://github.com/sponsors/ldez

I will do a PR to support this in any case.

@ldez ldez changed the title ovh: enable to use ovh.conf ovh: allow to use ovh.conf Jun 21, 2024
@yomovh
Copy link
Author

yomovh commented Jun 21, 2024

That's a fair point, I will pull some strings and see what happens.

@ldez
Copy link
Member

ldez commented Jun 21, 2024

I created a PR but I need a change inside go-ovh: ovh/go-ovh#84

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

Successfully merging a pull request may close this issue.

2 participants