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

Fix infoblox acme provider documentation #9277

Merged
merged 1 commit into from Aug 25, 2022

Conversation

ldez
Copy link
Member

@ldez ldez commented Aug 24, 2022

What does this PR do?

Replace INFOBLOX_USER with INFOBLOX_USERNAME

Motivation

Have the right env var name.

More

  • [ ] Added/updated tests
  • Added/updated documentation

@ldez ldez added area/documentation status/2-needs-review area/acme kind/bug/fix a bug fix bot/light-review decreases the number of required LGTM from 3 to 1. labels Aug 24, 2022
@ldez ldez added this to the 2.8 milestone Aug 24, 2022
@ldez ldez added this to To review in v2 via automation Aug 24, 2022
@ldez ldez changed the title docs: fix infobloc documentation doc: fix infobloc documentation Aug 24, 2022
@traefiker traefiker merged commit 5ca7fff into traefik:v2.8 Aug 25, 2022
v2 automation moved this from To review to Done Aug 25, 2022
@traefiker traefiker removed status/3-needs-merge bot/light-review decreases the number of required LGTM from 3 to 1. labels Aug 25, 2022
@ldez ldez deleted the fix/infoblox-doc branch August 25, 2022 13:37
@ldez ldez changed the title doc: fix infobloc documentation doc: fix infoblox documentation Aug 25, 2022
@rtribotte rtribotte changed the title doc: fix infoblox documentation Fix infoblox acme provider documentation Sep 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
v2
Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants