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

Add doc for AKS integration #3206

Merged
merged 3 commits into from
Aug 24, 2024
Merged

Conversation

dejanu
Copy link
Contributor

@dejanu dejanu commented Aug 14, 2024

What does this change

Add documentation concerning AKS integration.

What issue does it fix

Closes #3173

Signed-off-by: dejanualex <dejanualexandru@gmail.com>
@schristoff
Copy link
Member

Please don't forget to add your name to the contributing.md file! Thank you so much :)

@dejanu
Copy link
Contributor Author

dejanu commented Aug 16, 2024

Please don't forget to add your name to the contributing.md file! Thank you so much :)

I added it the last time. Thx

Copy link
Contributor

@kichristensen kichristensen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you so much for the PR, looks really good. I have added two small comments

docs/content/docs/integrations/aks.md Outdated Show resolved Hide resolved
docs/content/docs/integrations/aks.md Outdated Show resolved Hide resolved
Signed-off-by: dejanualex <dejanualexandru@gmail.com>
@dejanu
Copy link
Contributor Author

dejanu commented Aug 22, 2024

@kichristensen, I've added Kubernetes mixin, just to illustrate the integration is working, hope it's ok. I've added a small video for demo purposes: https://youtu.be/lB1sjL55dZ0

@kichristensen
Copy link
Contributor

@dejanu Looks really good, thank you so much for the contribution

@kichristensen kichristensen merged commit d954afe into getporter:main Aug 24, 2024
30 checks passed
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.

Add documentation on connecting to AKS
3 participants