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

Guide to retrieve ske outgoing cluster ip #476

Merged
merged 1 commit into from
Jul 31, 2024
Merged

Guide to retrieve ske outgoing cluster ip #476

merged 1 commit into from
Jul 31, 2024

Conversation

h3adex
Copy link
Contributor

@h3adex h3adex commented Jul 22, 2024

I wrote some documentation on how to gather the outgoing IP address of the SKE cluster for my own reference. I believe this could be helpful for other developers as well. Please feel free to read through it and share any feedback if something is not clear.

Additionally, I suggest we link this guide to the following issue for better visibility: Issue #102.

Thank you! :-)

Copy link

@saurabh2590 saurabh2590 left a comment

Choose a reason for hiding this comment

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

Looks good to me and we were able to use the instructions to look up router IP.

@DiogoFerrao
Copy link
Contributor

Hey @h3adex, sorry for the delayed response to your PR.

Firstly I would like to thank you for this contribution! We always welcome this sort of help and this guide is definitely useful for our users.

Also, I really like the writing on it! I will leave some comments below with some details but it looks great overall!

Once these comments are addressed, we can merge the PR.

@h3adex
Copy link
Contributor Author

h3adex commented Jul 29, 2024

Hi @DiogoFerrao

I've applied all of you comments. I've amended all changes to the latest commit.

Thank you

@joaopalet
Copy link
Contributor

Hi @h3adex,

Looks good to me, thanks for the contribution! Can we merge the PR?

@h3adex
Copy link
Contributor Author

h3adex commented Jul 31, 2024

Yes. Let's go :-)

@h3adex
Copy link
Contributor Author

h3adex commented Jul 31, 2024

@DiogoFerrao

We should also inform this issue about this PR.

@joaopalet joaopalet merged commit 3c53079 into stackitcloud:main Jul 31, 2024
3 checks passed
joaopalet added a commit that referenced this pull request Aug 1, 2024
joaopalet added a commit that referenced this pull request Aug 1, 2024
vicentepinto98 pushed a commit that referenced this pull request Oct 24, 2024
* docs: add guide to retrieve outgoing ske ip-address

* docs: fix volume attach example

* Revert "docs: add guide to retrieve outgoing ske ip-address (#476)"

This reverts commit 3c53079.

---------

Co-authored-by: Mauritz Uphoff <[email protected]>
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.

4 participants