Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Update 09-aks-cluster.md #70

Closed
wants to merge 1 commit into from
Closed

Update 09-aks-cluster.md #70

wants to merge 1 commit into from

Conversation

lfrost1
Copy link

@lfrost1 lfrost1 commented Nov 23, 2022

variable name fix

@lfrost1
Update 09-aks-cluster.md
2e80080 
variable name fix
@ckittel ckittel requested a review from ferantivero December 1, 2022 21:29
@ckittel
Copy link
Member

ckittel commented Dec 7, 2022 •
edited
Loading

Thank you @lfrost1, we are including a variation of this fix in #66, specifically, changing the variable name when setting the cert.

@ckittel
Copy link
Member

ckittel commented Dec 14, 2022

Going to now close as the fix was merged in #66. Again, thank you for helping identify this, @lfrost1! Your PR was much appreciated.

@ckittel ckittel closed this Dec 14, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@ferantivero ferantivero Awaiting requested review from ferantivero

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lfrost1 @ckittel